The model is automatically killing itself after 3 epochs

I am getting the Total Loss of about 10.032443234 starting , and after 3 epochs it is automatically killed.
Systerm Infomation:-
Processor:- Intel XEON
RAM:- 8 GB
Graphics: NVIDIA GTX-1060 6GB
Python 3.6

The training will end when it can no longer improve.  Please include the message (4-5 lines) towards the end of your output.  It will give a clue as to the problem.  But bottom line is that training has failed.

I also get something similar -
I am trying to train with squeezenet with the raccon dataset (I split it so that image 1-40 is my validation set). I had to change the congif file abit for it to run :
{
    "model" : {
        "backend":              "SqueezeNet",
        "input_size":           416,
        "anchors":              [0.57273, 0.677385, 1.87446, 2.06253, 3.33843, 5.47434, 7.88282, 3.52778, 9.77052, 9.16828],
        "max_box_per_image":    10,
        "labels":               ["raccoon"]
    },

    "train": {
        "train_image_folder":   "path/to/raccoon_dataset-master/raccoon_dataset-master/images/train/",
        "train_annot_folder":   "path/to/raccoon_dataset-master/raccoon_dataset-master/annotations/train/",

        "train_times":          10,
        "pretrained_weights":   "",
        "batch_size":           16,
        "learning_rate":        1e-4,
        "nb_epochs":             50,
        "warmup_epochs":        3,
        "object_scale":         5.0 ,
        "no_object_scale":      1.0,
        "coord_scale":          1.0,
        "class_scale":          1.0,
        "saved_weights_name":   "racoon_detector.h5",

        "debug":                true
    },

    "valid": {
        "valid_image_folder":   "path/to/raccoon_dataset-master/raccoon_dataset-master/images/val/",
        "valid_annot_folder":   "path/to/raccoon_dataset-master/raccoon_dataset-master/annotations/val/",

        "valid_times":          1
    }
}

The recall only decreases during the first 3 (warmup) iterations.
typical print out for loss at this stage:

Loss XY 	[2.52920581e-05]
Loss WH 	[0.0361119919]
Loss Conf 	[0.12189471]
Loss Class 	[0]
Total Loss 	[10.1580324]
I am not sure how it gets to total loss of 10 ?

on the first real epoch I there are warning fro almost all batches and finally a failure :

if all(x == 1 for x in tup) and isinstance(A, _nx.ndarray):
SystemError: error return without exception set
Exception ignored in: <generator object tile.. at 0x000001B577C54E60>
Traceback (most recent call last):
File "C:\Users\dan\AppData\Local\Programs\Python\Python36\lib\site-packages\numpy\lib\shape_base.py", line 1140, in
if all(x == 1 for x in tup) and isinstance(A, _nx.ndarray):
SystemError: error return without exception set
raccoon 0.0000
mAP: 0.0000
Exception ignored in: <async_generator object _ag at 0x000001B39081CAD8>
Traceback (most recent call last):
File "C:\Users\dan\AppData\Local\Programs\Python\Python36\lib\types.py", line 27, in _ag
SystemError: error return without exception set

Seems like the early stopping was the cause, you can either increase the value of patience(number of epochs) or remove that argument altogether.


cmd + space shows up twice

Whe I use cmd + space, the shortcut shows up twice on my screen if I press the spacebar quickly.

Thanks for opening an issue, can you confirm you are on an Intel/x86_64 machine?
This is likely the same issue as #163

Yup, this is a duplicate.  Sorry!

No problem, thank you


feat: New Design

Version

v0.8.0
Development Environment

Any
Current Behavior

Expected Behavior

Apply new design.

Finished


Allow table name with dots by a PinotConfiguration switch

Description
Following changes in https://github.com/apache/pinot/pull/5734,
Allows table name with dots by a shared pinot property. This will allow table name format like namespace.table_name to avoid table_name conflict while providing some mitigation/middle ground before Pinot table goes true multi-tenant.
The use case this one PR enabled will be for some users sharing a same Pinot Cluster, with the below Example rules (not in Pinot, but only with the wrapper/UI or some toolset visible to user):

The Admin enforces the rule that the users cannot use dots in their own table_name like test;
The Admin also enforces the rule that the users will be given a unique "namespace" as prefix to table;
The users can create a table named test and write query freely/naturally with table name like user_namespace.test
Pinot can delegate the authorization of table access via its plugin properly in the query

Extended feature
Extract column name only after the last dot in cases like SELECT db.namespace.table.column_name from db.namespace.table. In the example the table name will be db.namespace.table
Tests Done
Many Unit tests
Release Notes

Added pinot.table.name.dots.enabled config to Controllers and Brokers to allow table name with dots.
Extract column name from queries like "SELECT db.namespace.table.column_name" properly

IMO this doesn't even need a feature flag. We're not breaking any behavior. Keeps things simple.

@apucher Good point. Let me list down the things we are trying to handle with . and discuss a solution:

SELECT ... FROM <db>.<table>
SELECT <table>.<col> FROM ...
In the future, we also want to support:
SELECT <col>.<nested_fields> FROM ...

If we allow . within the table name, we cannot tell if . is the delimiter or part of the table name.
Add @xiangfu0 to the discussion

it should be easy to parse the table name since it needs to be present in the FROM clause.
are nested field selects a thing yet?

it should be easy to parse the table name since it needs to be present in the FROM clause.

I think the <db>.<table> convention is from the connector, and we don't have the database concept in Pinot. When getting it from the FROM clause, we want to match it with the tables available in pinot. If we support dot in the table name, it will make it slightly hard to parse the name. E.g. when getting a.b.c.d.e, we won't know which part is the table name.

are nested field selects a thing yet?

We don't support it yet, but it is in our roadmap, so we should take them into consideration
I think using dot in table name (or any field name) is in general bad practice because dot is already reserved for other purpose. @dongxiaoman I think we should just use _ instead of . for multi-tenant purpose. To avoid name conflict, we can reject namespace with underlying _ or perform a prefix check when adding a new namespace.
Some reference about not using .:

https://dba.stackexchange.com/questions/1166/is-it-okay-to-put-dots-in-sql-server-database-names#:~:text=Complex SQL actions are processed,if the base is fine.&text=These errors may more often,of anything on sql server.
https://stackoverflow.com/questions/41948399/is-it-good-practice-to-use-dots-within-table-names-in-mysql

There's precedence for using dots as separators across databases/schemas in MySQL, MSSQL, and others. It's tricky in the sense that underscores are much more commonly used in name tables. Both, underscores and dots, could ultimately work. My hunch is that we'll ultimately have to support dots (to some degree) as most cloud analytics systems already do:

mssql: https://stackoverflow.com/questions/11426929/can-we-use-join-for-two-different-database-tables
(or the links already shared above)
big query: https://michaelhoweely.com/2020/09/24/how-to-combine-data-from-two-bigquery-public-datasets-using-sql/
athena: https://stackoverflow.com/questions/58985607/unable-to-join-two-tables-from-two-different-databases-in-athena

Agree on using dot to connect db/namespace and table name and column name (this is the standard SQL syntax).
The problem here is that we don't support db/namespace concept natively in Pinot right now, and we are trying to overload the table name to include the namespace as part of it. This can potentially become a blocker for us when adding support of namespace in the future.

@Jackie-Jiang Agree that using dot in table name is not good practice. I already saw JDBC H2 error if I copy/paste the Full integration test because JDBC engine treat dots as separator by default. That is why I hide the support with a feature flag. cc @apucher
Meanwhile, as I mentioned, table name with dots can provide a transition/middle ground before we fully support full multi-tenant/DB concept.
By this PR, we can allow the user write something naturally:
SELECT col1 from mydb.test_table where xxx while the mydb.test_table is managed by Pinot admins, and the user is assigned mydb from the DataWarehouse infrastructure on top of Pinot.
If this is not breaking other features, providing extra flexibility (extra knob) will help support more use cases, it actually expands Pinot's scope

@Jackie-Jiang Sorry for the late response. Can you please have another quick look? All comments addressed


Only delay for status updates when many requests are delayed

Attempt at fixing the delay issues for status updates. After the other current open PRs are merged, we still have one case where a particularly active request with many tasks being launched will have status updates delayed. Offer processing to launch 200+ tasks ma hold the request lock for a fair amount of time, delaying status updates for that particular request. Other request updates are fine, but the overall delay still looks high and can short circuit offer polling.
This changes the logic to only delay other pollers when a higher percentage of requests have delayed status updates. One or two requests with large piles of status updates will then not be able to slow down processing of everything else.

conflicts heavily with https://github.com/HubSpot/Singularity/pull/2028 , merging these changes into there


invoke with returnType: Data throwing error of unsupported type

I'm trying to get a list of Channels from signalR on an invoke method with a returnType of Data and then use the Decodable protocol to then parse the result into an array of channels. However the client is getting an error or Unsupported type. I can use Any as the result, but then I would be unable to use the Codable protocol.
    self.chatHubConnection?.invoke(method: "ListChannels", arguments: ["My Channels", 0, 0], returnType: Data.self ) { result, error in
        if error != nil {
            print("We got an error", error)
        } else {
            let channelList = try! JSONDecoder().decode([Channel].self, from: result!)
            print(channelList)
        }
    }

I worked around this by using JSONSerialization.data.

I think that in you case you can call JSONEncoder.decode() like this because the type implements the Codable protocol. The client should support such types without having to implement type converters. I created #35 to track this.

Serialization is now based on Encodable/Decodable - #55


fetchgit: document --fetch-lfs flag

Motivation for this change
Fixes: 74128561 ("fetchgit: add lfs support")
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Plain git doesn't know the 'lfs' subcommand, so this doesn't work.
$ ./result/bin/nix-prefetch-git --fetch-lfs https://github.com/nixos/nix
git: 'lfs' is not a git command. See 'git --help'.

The most similar command is
        log

(Yes, I know nix doesn't have LFS files, but the point is that git doesn't understand the 'lfs' subcommand.)
Should we add 'git-lfs' by default for nix-prefetch-git (and pkgs.fetchgit), or extend the documentation with examples on how to set it up (ugh?).
CC @jonringer

Result of nixpkgs-review pr 121905 at b6d72172 run on aarch64-linux 1

11 packages built successfully:

bundix
cabal2nix
crate2nix
dep2nix
go2nix
haskellPackages.update-nix-fetchgit
nix-prefetch-git
nix-prefetch-scripts
nix-update-source
update-nix-fetchgit
vgo2nix


Beginner support: Camera position

First off, thanks for this amazing card!
I have my OBJ loaded and working well. The starting camera view is looking away from my object. Is there documentation for how best to use the camera position and rotation appearance fields - or a recommended way to export an OBJ from SweetHome3D to center the default camera? When I type in {x:0, y:0, z:0} into the appearance fields, I keep getting NnN positions.
I am sure this is user error - my technical background is limited. Home Assistant is a fun project for me to learn.
Again, thanks for your work here! Really excited to take advantage of it!

Hello @aaronganek , thanks.
coordinate are usually messed up by the THREE.js loader. To avoid this: all objects needs to be at bottom right compared to coordinate 0,0 in the Sweethome3d model (based on the 2d perspective). Also avoid to put objects too far from the focus of your house (ex a tree or a mailbox).
If you have followed the previous guideline, the best way to change the default point of view is:

When you are in view edit mode, set the view according to your preference (position and rotation of your camera)
Double click in an area where there are no objects.
A pop-up will appear
Copy the test inside the pop-up
Click the card edit button and enter in code editor for the configuration
Remove any previous paramater related to the position and rotation
go to the bottom of the config and paste the copied data
save

If you follow these 2 steps, you should have the right default view when you open the card

Hello aarong, I close the issue you can follow my procedure above to assist you


⚠ Bofrost.es has degraded performance

In a8eca06, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1570 ms

Resolved: Bofrost.es performance has improved in de7cfbf after 6 minutes.


Bug: Crowdstrike analytics rule not working

Describe the bug
The current CrowdStrike analytics rule only uses the table which is used by the Crowdstrike agent source.
It does not use the Falcon Data Replicator, which is the other S3 source method also in the Crowdstrike package.
I've managed to build a detection rule which triggers on data replicator detections:
CrowdstrikeReplicatorLogs_CL
     | where EventType_s == "Event_ExternalApiEvent"
     | extend customFields=parse_json(custom_fields_message_s)
     | where customFields["ExternalApiType"] == "Event_DetectionSummaryEvent"
     | summarize by Timestamp=unixtime_milliseconds_todatetime(tolong(customFields["UTCTimestamp"])),
        Name=tostring(customFields["DetectName"]),
        Hostname=tostring(customFields["ComputerName"]),
        Severity=tostring(customFields["SeverityName"]),
        Tactic=tostring(customFields["Tactic"]),
        Technique=tostring(customFields["Technique"]),
        Description=tostring(customFields["DetectDescription"]),
        User=tostring(UserName_s),
        LocalIP=tostring(customFields["LocalIP"])
;

To Reproduce

Install Crowdstrike source
Setup the S3 replicator
Install the built-in analytics rule(s)
Never see anything happening since the table used keeps empty

Expected behavior
All analytics rules working.

Hi @hazcod are you still facing this issue, please confirm, thanks!

We wanted to check on the status of Issue https://github.com/Azure/Azure-Sentinel/issues/6631 . The issue is pending for more than x days. Please let us know if you need any assistance to review this Issue. As per our standard operating procedures if no response is received in the next 7 business days, we will close this Issue. Thank you for your cooperation.

Hi @hazcod we are working with the concerned team, will update you once we hear back from them, thanks!

Hi @hazcod we are still waiting for update, will update you by end of this week, thanks!

Hi @hazcod can you please provide us details on whether your concern is with an existing rule or you are looking for a new rule to be added to the solution, thanks!

@v-vdixit It's the existing rule for CrowdStrike.

Hi @hazcod can you please provide a link for this rule, we are unable to locate it in the solution, thanks!

Gentle Reminder: We are waiting for your response on this issue. If you still need to keep this issue active, please respond on it in the next 2 days. If we don't receive response, we will be closing this issue as per our standard procedures, thanks!

The analytics rules in Content Hub refer to table CrowdStrikeFalconEventStream, which does not exist/fill up when using CrowdStrike Falcon Data Replicator. It uses CrowdstrikeReplicatorLogs_CL

Thanks for the update @hazcod we will discuss with the team and update you for same.

Hi @hazcod we are discussing this with the team, will raise PR with changes by 25th July 2023, thanks!

Hi @hazcod we need some time to check on this rule so we will not be raising the PR currently, will provide you update by 27th July 2023, thanks!

Hi @hazcod we need some time to check on this rule so we will not be raising the PR currently, will provide you update by 04th Aug2023, thanks!

Hi @hazcod we need some time to check on this rule so we will not be raising the PR currently, will provide you update once we are working on it.

We are adding this item in our backlog as a feature request ,so closing this issue.


Missing cmdbench module

I tried to run cg-producer Docker image version a6e779a3 and got the following error:
ModuleNotFoundError: No module named 'cmdbench'

It seems that cg-producer/setup.py should have installed it, but looking at the Dockerfile it seems that only setup.py of pycg is actually executed.

On the monster deployment this issue crashes the pycg deployment:
/scp:magiel@monster.ewi.tudelft.nl:~/fasten-k8s-deployments/fasten/pypi-cg-producer $ kubectl logs pycg-6b576b586d-464l2 -n fasten
Traceback (most recent call last):
  File "/cggen/entrypoint.py", line 28, in <module>
    from pycg_producer.producer import CallGraphGenerator
  File "/cggen/pycg_producer/producer.py", line 29, in <module>
    import cmdbench
ModuleNotFoundError: No module named 'cmdbench'


⚠ Bofrost.at has degraded performance

In 3b32d2b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1976 ms

Resolved: Bofrost.at performance has improved in 2cf340c.


Fix video not loading #808

Route collision with thumbnail path and video path. Results in videos not loading properly.
Changing the route definition order fixes the issue.

Thank you!


Fix python-ligo-lw requirement

1.9.3 included a bump in the python-ligo-lw requirement, this was missed in #11.
This also adds @aepace as a co-maintainer of the recipe.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Proguard error on Android SDK 23

Your library cannot be used together with Android sdk 23, as proguard throws an error:
Warning: de.duenndns.ssl.MemorizingTrustManager: can't find referenced method 'void setLatestEventInfo(android.content.Context,java.lang.CharSequence,java.lang.CharSequence,android.app.PendingIntent)' in library class android.app.Notification
This is a real show stopper - with that dependency your library cannot be used any more.

There seems to be a pull request to solve this issue: https://github.com/ge0rg/MemorizingTrustManager/pull/46
When will be your next release fixing that issue? :-)

See https://github.com/klassm/MemorizingTrustManager/commit/42254987aaea7a207712f3f0b799b3cf2d845dcf
I've merged those two available pull requests allowing to build an aar that can be used against sdk 23. I won't create a pull request - feel free to take what you want.

Fixed in master! :)


What is correct way to setup HighStock

im trying like this
 import Highcharts from 'highcharts'
 import StockInit from 'highcharts/modules/stock'

and init it in mounted section
mounted () {
      StockInit(Highcharts)
}

there is my options
        testOptions:  {
          rangeSelector: {
            selected: 1
          },

          title: {
              text: 'AAPL Stock Price'
          },

          series: [{
              name: 'AAPL',
              data: [
              [1416182400000, 7.204, 7.232, 7.037, 7.072, 889594.5],
              [1416268800000, 7.058, 7.107, 6.933, 6.968, 930296.44],
              [1416355200000, 6.961, 7.003, 6.92, 6.982, 591656.75],
              [1416441600000, 6.947, 7.024, 6.891, 6.982, 683544.62],
              [1416528000000, 6.975, 7.141, 6.891, 7.1, 1146732],
              [1416787200000, 6.968, 7.204, 6.905, 7.086, 2227368.5],
              [1416873600000, 7.065, 7.253, 7.024, 7.225, 1567802.62],
              [1416960000000, 7.232, 7.495, 7.183, 7.385, 2184918.75],
              [1417046400000, 7.482, 7.593, 7.322, 7.454, 2103856.75],
              [1417132800000, 7.426, 8.239, 7.364, 8.239, 4844839.5],
              [1417392000000, 8.35, 8.655, 8.037, 8.065, 4465213.5],
              [1417478400000, 8.03, 8.905, 7.989, 8.711, 3490411.5],
              [1417564800000, 8.711, 9.245, 8.426, 8.697, 3997530],
              [1417651200000, 8.725, 9.301, 8.558, 9.295, 3450180.75],
              [1417737600000, 9.357, 10.176, 8.989, 9.69, 4413697],
              [1417996800000, 9.468, 10.364, 9.322, 10.176, 3928342.5],
              [1418083200000, 9.989, 10.385, 9.12, 9.12, 4045420],
              [1418169600000, 9.045, 9.572, 8.641, 9.454, 3527844.25],
              [1418256000000, 9.19, 9.725, 9.107, 9.266, 2237995.75],
              [1418342400000, 9.253, 9.461, 9.1, 9.28, 1643909.12],
              [1418601600000, 9.149, 9.149, 8.815, 9.03, 1800936.62],
              [1418688000000, 8.975, 9.593, 8.933, 9.593, 2469073],
              [1418774400000, 9.6, 10.482, 9.461, 10.266, 4400166],
              [1418860800000, 10.301, 10.399, 9.739, 9.94, 2470251],
              [1418947200000, 9.891, 10.149, 9.628, 10.016, 1935405.88],
              [1419206400000, 9.947, 10.822, 9.885, 10.253, 3467391.25],
              [1419292800000, 10.01, 10.211, 9.67, 9.843, 1797850.75],
              [1419379200000, 9.836, 9.912, 9.1, 9.37, 2010271],
              [1419465600000, 9.565, 9.843, 9.35, 9.801, 1997275.88],
              ],
              tooltip: {
                  valueDecimals: 2
              }
          }]
        },

but chart looks like ordinary lineChart

Same problem for Exporting module

https://codesandbox.io/s/jjyqvv0k13 @IsmailMouh @pashtetbezd you can see this demo

@Rodimusbot  you should pass instance of Highchars.
<vue-highcharts :options="options" :highcharts="Highcharts" ></vue-highcharts>

I update the demo, you can see demo on codesanbox  for more details.


findDOMNode was called on an unmounted component

I'm getting this error after I click "Pay $10", when my page then rerenders without <StripeCheckout>. I'm not calling findDOMNode, but I'm guessing Checkout is?
inner-284ced8932f4a8…085eeb162a4124.js:6 Uncaught Invariant Violation: Minified React error #44; visit http://facebook.github.io/react/docs/error-decoder.html?invariant=44 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
    at n (https://checkout.stripe.com/m/lib/inner-284ced8932f4a845e0085eeb162a4124.js:6:27558)
    at Object.r [as findDOMNode] (https://checkout.stripe.com/m/lib/inner-284ced8932f4a845e0085eeb162a4124.js:62:194)
    at t (https://checkout.stripe.com/m/lib/inner-284ced8932f4a845e0085eeb162a4124.js:37:22320)
    at r (https://checkout.stripe.com/m/lib/inner-284ced8932f4a845e0085eeb162a4124.js:58:14240)

I'm getting the same error. I don't suppose you found a solution, did you?

I got the same error ? Btw, how can we enable apple pay, google pay like alipay ?

Did you find a solution?

I'm getting the same error - React 16.8


[feat] add support for new insert/update syntax, M.SQL import syntax, and trying to rewrite new mysql driver.

[dev] Update gitignore
[feat] add M.SQL import style
[feat] add support for new insert/update syntax
[feat] trying to rewrite/build the mysql driver.

suggestions:

revert the changes that make password -> passwd. This kind of renaming may not be so convenient for updating the code.
temperarely delete the commits related to the new mysql driver. It should be in an independent branch.


Make current_tenant thread safe

Thank you for your contribution!  The code addition is super straightforward, as expected.  The real trick is on the testing side.  My recommendation is to simplify the testcase such that the new thread attempts to overwrite the current tenant variable.  There are some good examples available here: https://stackoverflow.com/questions/6241384/how-do-i-manage-ruby-threads-so-they-finish-all-their-work
describe '#current_tenant=` do
  it 'is threadsafe' do
    Multitenant.current_tenant = :old_tenant
    overwriter = Thread.new { Multitenant.current_tenant = :new_tenant }
    overwriter.join
    expect(Multitenant.current_tenant).to eq :old_tenant
  end
end

Thank you for your contribution! The code addition is super straightforward, as expected. The real trick is on the testing side. My recommendation is to simplify the testcase such that the new thread attempts to overwrite the current tenant variable. There are some good examples available here: https://stackoverflow.com/questions/6241384/how-do-i-manage-ruby-threads-so-they-finish-all-their-work
describe '#current_tenant=` do
  it 'is threadsafe' do
    Multitenant.current_tenant = :old_tenant
    overwriter = Thread.new { Multitenant.current_tenant = :new_tenant }
    overwriter.join
    expect(Multitenant.current_tenant).to eq :old_tenant
  end
end

Thankyou for your review and feedback. Changes are incorporated, kindly give another go.

Thanks @wireframe!


Have a look here

Still a work in progress. https://github.com/ClosetMonkey/ReactPHPZMQ

Hey @ClosetMonkey, looks like a cool project, thanks for sharing! 👍
This ticket encourages me to start a discussion about adding "GitHub Discussions" to all ReactPHP repositories. There's currently only the possibility to open up a new issue, as you did, but I think this kind of ticket is more suited for a "GitHub Discussion" (under a category show your work).
I think we can close this ticket for now but we can transfer it into a Discussion (if everyone agrees to GitHub Discussions) for more visibility at a later point.

@ClosetMonkey Interesting one! Make sure to add to our wiki (https://github.com/reactphp/reactphp/wiki/Users) and/or leave a link in https://github.com/friends-of-reactphp/zmq.
I'll assume this is resolved and will close this for now, please feel free to report back otherwise :+1:

Working out some bugs first but then will do :thumbsup:


Enable SignalChildrenProcesses by default

Bug Description
Slurm process tracking is currently not configured to kill child processes of a job. Set SignalChildrenProcesses=yes in cgroup.conf to enable this functionality.
To Reproduce
NA
Environment
NA
Relevant log output
NA

Additional context
No response

Surprising that Slurm doesn't have this enabled by default. This will be helpful for when we eventually enable Open OnDemand in Charmed HPC since VSCode Server is notorious for leaving behind child processes that bog down cluster performance.


Fix snippet

Fix snippet.
BEFORE:

AFTER:

Hi @charliethinker, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnetcore/tutorials/first-mvc-app/search.md
:white_check_mark:Succeeded

aspnetcore/tutorials/first-mvc-app/start-mvc/sample2/src/MvcMovie/Controllers/MoviesController.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@charliethinker  looks good, can you sign the CLA?

Hi @charliethinker, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnetcore/tutorials/first-mvc-app/search.md
:white_check_mark:Succeeded

aspnetcore/tutorials/first-mvc-app/start-mvc/sample2/src/MvcMovie/Controllers/MoviesController.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@charliethinker, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

@charliethinker, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

@charliethinker  Thanks! Much better now!


[REQUEST] Implement semantic verification of fields route_sort_order in routes.txt

Is your feature request related to a problem? Please describe.
This numeric fields should be user configurable so that suspicious values can be reported.
Describe the solution you'd like

[x] Add new Warning notice:

SuspiciousRouteSortOrderNotice

[x] Add support to provide thresholds for this field via configuration file and/or command line

[ ] Determine default values for said thresholds

Not sure how route_sort_order could be suspicious? It's just a integer for ordering records.

We could imagine that for an agency managing 300 routes, having a route with field route_sort_order equal to 6000 would be suspicious.

Yes, I guess so. I guess I'm just used to seeing people use somewhat arbitrary integers for ordering. I also haven't seen this field much myself in practice - I'd be interested in seeing actual values used in practice.


[Feature][Seatunnel-web]Add support to configure placeholder with default value in the job config.

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs

The primary modification in this Pull Request focuses on org.apache.seatunnel.app.utils.JobUtils#replaceJobConfigPlaceholders and its application.
I am currently encountering an issue when ${configParam:default} is set in the job configuration.
The ConfigFactory.parseString(jobConfigString) operation fails with the error:
String: 1: Token not allowed in path expression: ':' (you can double-quote this token if you really want it here).
I plan to investigate the variable substitution implementation on the Seatunnel engine side.

Executed integration tests, all are passed

Manually tested conditional query

Thanks @Hisoka-X for reviewing and merging the PR.


Copy frameworks only when installing

Is there a way to set this flag for a Copy Files phase?

Hi @Elland. What do you mean exactly by Copy frameworks?
At the time there isn't a way to control that on any of the build phases. This happens through a buildActionMask property on the build phase.
What is your use case?

Hey @yonaskolb, I wanted the generated project configuration to only copy frameworks when installing, for faster build+launch times.

I'm running into this now, where I need this checkmark checked in my Embed Frameworks build phase:

Here's what my project file looks like without the checkmark:
/* Begin PBXCopyFilesBuildPhase section */
		FC21B861D79C5E0F1CFC6599 /* Embed Frameworks */ = {
			isa = PBXCopyFilesBuildPhase;
			buildActionMask = 12;
			dstPath = "";
			dstSubfolderSpec = 10;
			files = (
				637EF3B657CC45F576440008 /* GooglePlaces.framework in Embed Frameworks */,
				9CB073B21DCB818D51414013 /* GoogleMaps.framework in Embed Frameworks */,
				84A3FAC8FC437CBCF002801A /* GoogleMapsCore.framework in Embed Frameworks */,
				238BC22DAEFDEA871E9C4988 /* GoogleMapsBase.framework in Embed Frameworks */,
			);
			name = "Embed Frameworks";
			runOnlyForDeploymentPostprocessing = 0;
		};
/* End PBXCopyFilesBuildPhase section */

And here's with the checkmark:
/* Begin PBXCopyFilesBuildPhase section */
		FC21B861D79C5E0F1CFC6599 /* Embed Frameworks */ = {
			isa = PBXCopyFilesBuildPhase;
			buildActionMask = 8;
			dstPath = "";
			dstSubfolderSpec = 10;
			files = (
				637EF3B657CC45F576440008 /* GooglePlaces.framework in Embed Frameworks */,
				9CB073B21DCB818D51414013 /* GoogleMaps.framework in Embed Frameworks */,
				84A3FAC8FC437CBCF002801A /* GoogleMapsCore.framework in Embed Frameworks */,
				238BC22DAEFDEA871E9C4988 /* GoogleMapsBase.framework in Embed Frameworks */,
			);
			name = "Embed Frameworks";
			runOnlyForDeploymentPostprocessing = 1;
		};
/* End PBXCopyFilesBuildPhase section */

This setting should go on the target I think since it applies to all the frameworks being embedded. @yonaskolb if you have guidance on how this might be done (and the level of difficulty to do) I might be able to pull it off. Thanks!

Just talked to @jsorge offline about a way forward and landed on a property on Target called onlyCopyFilesOnInstall which will affect all PBXCopyFilesBuildPhase in that target, and also the Carthage script that copies the frameworks

Thanks @yonaskolb that's great to hear! 💪

Sorry this PR took so long to open!

Just talked to @jsorge offline about a way forward and landed on a property on Target called onlyCopyFilesOnInstall which will affect all PBXCopyFilesBuildPhase in that target, and also the Carthage script that copies the frameworks

Sorry, that asking here. But I strongly need to set “onlyCopyFilesOnInstall=true“ flag to one build phase, an “onlyCopyFilesOnInstall=false“ on another. Is it possible?

@Noegon So you've got multiple embed framework build phases and one needs this turned on and another doesn't? The way I built the PR that's not possible, and I'm not sure offhand what it would take to make that possible.
I'm very curious about your use-case though. Can you share any more specifics about what you're doing?

/* Begin PBXCopyFilesBuildPhase section /
FC21B861D79C5E0F1CFC6599 / Embed Frameworks / = {
isa = PBXCopyFilesBuildPhase;
buildActionMask = 8;
dstPath = "";
dstSubfolderSpec = 10;
files = (
637EF3B657CC45F576440008 / GooglePlaces.framework in Embed Frameworks /,
9CB073B21DCB818D51414013 / GoogleMaps.framework in Embed Frameworks /,
84A3FAC8FC437CBCF002801A / GoogleMapsCore.framework in Embed Frameworks /,
238BC22DAEFDEA871E9C4988 / GoogleMapsBase.framework in Embed Frameworks /,
);
name = "Embed Frameworks";
runOnlyForDeploymentPostprocessing = 1;
};
/ End PBXCopyFilesBuildPhase section */

/* Begin PBXCopyFilesBuildPhase section /
FC21B861D79C5E0F1CFC6599 / Embed Frameworks / = {
isa = PBXCopyFilesBuildPhase;
buildActionMask = 8;
dstPath = "";
dstSubfolderSpec = 10;
files = (
637EF3B657CC45F576440008 / GooglePlaces.framework in Embed Frameworks /,
9CB073B21DCB818D51414013 / GoogleMaps.framework in Embed Frameworks /,
84A3FAC8FC437CBCF002801A / GoogleMapsCore.framework in Embed Frameworks /,
238BC22DAEFDEA871E9C4988 / GoogleMapsBase.framework in Embed Frameworks /,
);
name = "Embed Frameworks";
runOnlyForDeploymentPostprocessing = 1;
};
/ End PBXCopyFilesBuildPhase section */


Bump Scala to 2.12.20

:mag: Description
Issue References 🔗

Scala 2.12.20 release https://github.com/scala/scala/releases/tag/v2.12.20

Describe Your Solution 🔧
Types of changes :bookmark:

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

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklist 📝

[x] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

cc @bowenliang123

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (d7219fc) to head (8736f2c).
Report is 2 commits behind head on master.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #6676   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         683     684    +1
  Lines       42213   42233   +20
  Branches     5755    5755
======================================
- Misses      42213   42233   +20

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks, merged to master (1.10.0).


[ToggleButton, ToggleButtonGroup] Use context to pass data between ToggleButtonGroup and ToggleButton

One possible implementation that allows people to use ToggleButton in ToggleButtonGroup without caring about how nested ToggleButton is.
This was discussed in #12921
Let me know if this is something you might merge and I'll take care of testing as well

I'm really interested in a comparison between controller components with the cloneElement pattern and the new context api.  That way we could think about rewriting every controller-like component if the context api is better suited for these types of components. If it turns out that cloneElement is fine then I would stick to that pattern for consistency sake. Didn't spend much time thinking about this so I'm interested in opinions.
@RobertPurcea  Not a review but I'm glad you showed how this could look like 👍

Some v0 and a few early versions of v1 components used the old context feature for this sort of thing, but they were removed prior to release of v1. @oliviertassinari might know the details.

Answered in https://github.com/mui-org/material-ui/issues/12921#issuecomment-423835514

@oliviertassinari might know the details.

I don't, or I don't remember.

The old context API was documented as experimental which was probably one argument against using it.

The old context API was documented as experimental which was probably one argument against using it.

No, we use it for other things, even still (e.g the theme), so it wasn't that.

What was the decision in the end?

What's the status of this?  I see the PR has a solution but looks like the tests weren't updated.
Has there been a final verdict on whether using the Context API is an acceptable solution?  Seems like it's more flexible and it has my vote.
I'm seeing the Tooltip problems and will probably create a wrapping component to push the props through in the meanwhile but would rather have it work out of the box.  Context API really seems like the way to go.

This is in my mind the accepted solution but if this gets pushed forward it should include a more generic component that can also be used in RadioGroup and possibly Select.

The new hook API reduces the overhead from using the context, at least from a boilerplate point of view. I don't know about the runtime implications. I see pros and cons going both directions. I won't take side here, if not the side that requier doing less.
@eps1lon You have mentioned the Select, as far as I know, the issue is at a different level. We need to rewrite the focus handling logic.

@oliviertassinari The component I'm talking would only take care about handling what items are selected in a collection. Focus is a different concern and should therefore be handled in a different component.

I would vote for using context, it's already powerful and hooks make using it even better 🥇

I made a basic implementation a few months ago which used render props but never finished it: https://github.com/eps1lon/material-ui/commit/7df852f845cb983c68665cc8e6a1f928f9dcc010
Someone can pick this up and port this to react context instead of render props API. It's a pretty common concern so it would be nice if we could use it in ToggleButtonGroup´, RadioGroup, Menu` etc. Needs further investigation though.

I've been playing with this https://github.com/joshwooding/material-ui/tree/selectable-group
https://joshuawooding-mui.netlify.com/lab/selectable-group/
I still need to investigate integrating it with ToggleButtonGroup and other components.

Closing in favour of #12961
Thanks for the inspiration @RobertPurcea
Thanks for the starting point @eps1lon


Add amityschools.txt

School Site: amityregion5.org

@mihirmjog Could you please provide a proof of the fact that amityschools.org is an email domain of the school, which owns a website amityregion5.org? I did not find any references to amityschools.org at the website.

@philipto My apologies, I should have provided proof.
Please go to AmityRegion5.org > Search "amityschools.org" within the widget.
I can try getting additional proof if needed.

I can also provide a link to amity's library resource center that instructs students on how to use their<EMAIL_ADDRESS>account.
In addition, the "Education Technology Plan Template" - which is a report submitted to the Connecticut State Department of Education to list the uses of technology in schools- has several mentions of amityregion5.org (see page 14, 32 and 34).
Please let me know if you need anything else.

@mihirmjog Thank you for the clarification! Pull request merged.


[Website] [WIP] New Docs

We are working on a revamp of the docs. This PR will be used to keep track of the overall effort, which will happen on a new-docs branch.
Reviews for individual changes will be performed on separate PRs.
We have set up a staging site at https://jest-bot.github.io/jest/ which will host all changes that have landed in the new-docs branch.
Refer to the Jest Docs-a-thon project for status updates.
F.A.Q.
Q. How do I build the Jest website?
A. This README (https://github.com/facebook/jest/tree/master/website) should cover it. Basically, run npm install in website/ then run npm start. You will then be able to load the website locally at http://localhost:8080/jest/index.html
Q. What should I work on?
A. We're tracking progress using a GitHub Project (https://github.com/facebook/jest/projects/1). Anything under TODO is fair game. Once you find something you want to work on, move it to the Claimed column. You can also refer to the docs feedback below where other issues have been highlighted.
Q. I have some changes ready for review. What do I do?
A. Open a pull request on GitHub against the new-docs branch.
Q. How do I open a PR against the new-docs branch?
A. Navigate to https://github.com/facebook/jest/compare then choose base:new-docs as your target branch. If you've pushed your local branch to GitHub, you should be able to find it under compare:.
Q. My branch does not appear under compare when creating a new PR, why?
A. This can happen when you're working off a fork of facebook/jest and you pushed your branch to your own fork. The easiest way to open a PR in this case would be to click on the prompt to open a PR when visiting https://github.com/facebook/jest. You'll need to make sure to change the base branch back to new-docs when doing this.
Q. Who can review my PRs?
A. Post the link to your PR in the Jest Docs WG Messenger Group and/or add the PR to the Needs Review column in https://github.com/facebook/jest/projects/1. Ping @hramos if you can't find someone to review your PR.

@hramos Do we plan on versioning our API in documentation?
From the user perspective I think it would be great to know from which Jest version certain methods or config options are available.
We could also mark deprecated APIs and keep them in docs for, lets say, 2 versions behind current
Something simple like this:

What do you think?

@thymikee I think this makes a ton of sense. For features that we add, we should add this in the documentation going forward.

@thymikee we don't have that information available anywhere, do we? It seems like we need to go back to every method in the API reference and manually tag it with the earliest version number it should be available on. I'm not strongly in favor or against this, but if you'd really like to add versioning I suggest opening a new issue to discuss this as this PR will soon get a lot of commit comments on it related to the docs sprint which will make it hard to have a discussion.

Nope, this would require us to crawl through changelog and apply it manually. I'm not sure how the docs structure is going to change after this "makeover" so I just wanted to let you know about the idea :)

I think it is fine if we just keep adding this kind of documentation going forward, for future versions. There is no value in going back to older versions and figuring this out for the past.

@hramos it's 78 commits strong now. Can we resolve the conflicts and merge this PR and iterate on it from master?

@cpojer sounds good to me, as we're not yet deploying the site from master.


When deleting an item the cursor jumps to the previous current item instead of the closest remaining one

Nope, not fixed.

Commit 1deee47 erroneously references #122 instead of #123.


chore: implement cancel button

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


Snapshot Feature doesnt work

Hi,
i just tried to enable Snapshot Feature on my cams. But it wont work. I can enable it on the settings page (saved and rebooted cam and its still activated), but the snapshot page of the web interface just keeps "loading" without showing any snapshot.
My other two cameras (with yihack MStar) show a snapshot instant. I reflashed all 4 cameras with stock firmware yesterday, paired them with app, deleted them afterwards and reflashed them with the yiHacks.
So there shouldnt be a problem :)
Any hints how to enable snapshot :)?

What happens if you run the web service?
http://<IP_ADDRESS>:8080/cgi-bin/snapshot.sh

It seems it has Problems loading the Page. There is no error message. Its just a blank Page and IT loads and loads and loads....

Yeah
I tested it. The cam Was online for around 1 hour. I opened the snapshot Page and it crashed and rebootet.

Is swap file enabled?

No. I let All settings 'Stock', beside disabling cloud, setting a User and password, renaming the camera and disabling snapshot for recorded Video

Check the faq page.
The snapshot requires a lot of memory.
Enabling swap file the cam will be stable.

I will try it. Is there any reason to NOT use swap file? 'you cant' buy sd Cards smaller than 16 GB nowadays. So there should be enough space at any cam :)

Swap file is not safe for a sd card, after a couple of years the card probably dies.
Not all people want this.

Hey. With swap file it works.
Thanks a lot for the help.
You could add a comment near the snapshot Option in Web interface that swap file is 'needed' for it to work flawless :)


Remove deprecated SyntaxTree methods for beta5

Purpose
$subject.
Fixes #33593
Approach
N/A
Samples
N/A
Remarks
N/A
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Are there any other usages outside the lang repo?

Are there any other usages outside the lang repo?

Only the debugger was using it as I know of. If by any chance someone uses it outside, they have to move to the new API.

Are there any other usages outside the lang repo?

Only the debugger was using it as I know of. If by any chance someone uses it outside, they have to move to the new API.

Then it's better to mention this in slack right?


Added interface functionality

Now it's possible to do the speed test over a specific interface for example eth0 or tun1 etc

This is unfortunately not portable, and only works on linux. The current implementation of requiring a bind IP works across all OSes.


added "--generate-ssh-keys" to "az aks create"

on file articles/application-gateway/tutorial-ingress-controller-add-on-new.md I added "--generate-ssh-keys" to "az aks create"
this is requirment from new  version of azure command line

@saostad : Thanks for your contribution! The author(s) have been notified to review your proposed change. @caya

LGTM. #sign-off

@mscatyao: I'm sorry - only the author of this article, @caya, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Fix Google Maps URL to work with newest version of the URL

The URL scheme doesn't work with the newest version of the Google Maps app (it can't find the route). Removing the map point name fixes the problem.

👍 for the google maps fix but you should make other PR's for your other fixes..


fix: include ggml sources

whisper-rs-sys published on crates.io is lacking of essential ggml src files and won't compile.
Should fix #187 #190

whisper-rs-sys 0.11.1 published, cargo should automatically resolve it as a new dep if you do a cargo update.


1.17 AND1.18 NOT WORKING BRUH

Describe the bug
A clear and concise description of what the bug is.
Whenever I try too open 1.17 or 1.18 it doesnt work its shows exit code 0 and crashes all the other versions work properly
Desktop (please complete the following information):
OS & version [e.g. Windows 10, 1822]:
Windows 11 latest
MSFT Build of OpenJDK Version:
The output from java --version
To Reproduce:
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots, Logs etc
If applicable, add screenshots, logs to help explain your problem.
Additional context
Add any other context about the problem here.

@JarJarVader Thank you for the report!
Thank you for the report!
Please submit this issue over at the Mojang bug tracker. Please search for existing reports first; in case you do not find any, create a new report and let us know about the issue number here (e.g. MC-123456). The Mojang team will take the first look. If an OpenJDK bug is identified, the Mojang team will contact the Microsoft Build of OpenJDK team to address the issue.


Document AWS Lambda compatibility

Is it possible to use AWS Distro for OpenTelemetry to instrument AWS Lambda functions?
From aws-otel.github.io :

Use AWS Distro for OpenTelemetry to instrument your applications running on Amazon Elastic Compute Cloud (EC2), Amazon Elastic Container Service (ECS), and Amazon Elastic Kubernetes Service (EKS) on EC2, and AWS Fargate, as well as on- premises.

If so, I recommend updating the documentation.

Hi joebowbeer,
It's on our roadmap to support AWS Lambda, but we do not support it today. Stay tuned for updates soon!

Hello,
I spent some time working through integrating Lambda <> OpenTelemetry in a NodeJS environment. Below are some of the areas that I think could see improvement:
Timing of OTEL auto-instrumentation.
We followed a similar approach to what is described in the guide Tracing with the AWS Distro for OpenTelemetry JavaScript SDK, although my setup file is named telemetry.js instead of tracing.js.
When working with simple examples, it's easy to add a line similar to the following at the top of your code. This allows opentelemetry to intercept calls to require in order to automatically instrument supported modules.
require('./telemetry.js');

Unfortunately this approach an fall apart pretty quickly if you add babel/webpack into the mix to minimize the size of your lambda functions. A recommended alternative is to launch your application with a node argument that allows telemetry.js to load before your main code executes. The command typically looks something like this:
node --require ./telemetry.js  src/index.js

This turned out to be mildly challenging in a Lambda environment. I was eventually able to launch our NodeJS lambda functions using this approach, but it involved:

Creating a new lambda layer that included our telemetry.js setup, a custom telemetry-wrapper execution wrapper, and all the node_module dependencies to run said telemetry.js file. Note: if you are building a new layer from scratch, make sure your dependencies fall under nodejs/node_modules.

telemetry-wrapper
#!/bin/bash

# the path to the interpreter and all of the originally intended arguments
args=("$@")

# the extra options to pass to the interpreter
extra_args=("--require" "/opt/telemetry.js")

# insert the extra options
args=("${args[@]:0:$#-1}" "${extra_args[@]}" "${args[@]: -1}")

# start the runtime with the extra options
exec "${args[@]}"

Add a custom environment variable to our Lambda to execute the custom execution wrapper AWS_LAMBDA_EXEC_WRAPPER=/opt/telemetry-wrapper

OTEL Exporters need to flush before shutting down
It's not clear to me if this is a responsibility for AWS or OTEL, but the OTEL exporters don't seem to have the oportuntiy to flush and pending spans before a lambda shuts down. I was able to sole this with a call await api.trace.getTracerProvider().getDelegate().shutdown();
While doing research to find a working solution to the timing issue, I found Sentry's approach to Lambda's pretty clean, hopefully this use case is something that OpenTelemetry gives some attention to in the future.
Connecting OTEL to the external X-Ray trace.
If you turn on X-Ray support for a Lambda, you automatically get high level instrumentation out-of-the-box. If you setup the aws-otel-collector and use otel libraries to export traces to X-Ray, you can get detailed instrumentation. If you have both X-Ray support enabled at the Lambda level, and your own otel instrumentation, this will result in two separate traces being generated for every invocation of the Lambda function. I managed to work around this by abusing the AWSXRayPropagator class. :-)
Recording exceptions does not seem to work
For some reason, recording exceptions does not seem to work as expected. I haven't had time to dig deeper into what's going on, only I don't see any indication of the error or stack trace in x-ray.
Other thoughts:
I found this discussion interesting, but I'm not sure if a custom tracer would have helped with the issues I ran into:

Supply auto-configured base X-Ray tracer #3

It would be nice if this opentelemetry plugin supported a Lambda environment:

@opentelemetry/resource-detector-aws

Below is an example that demonstrates some of the topics discussed above:
import "core-js/stable";
import "regenerator-runtime/runtime";

const path = require('path');

const { NoRecordingSpan } = require('@opentelemetry/core');
const api = require('@opentelemetry/api');

const deployment = require('../package.json');
const axios = require('axios');

function instrumentHandler(handler) {
    return async (event, context, callback) => {
        const tracer = api.trace.getTracer(deployment.name, deployment.version);

        // Currently AWSXRayPropagator expected to be passed HTTP headers,
        // not a Lambda environment map.
        const mockHttpRequestHeaders = {
            'X-Amzn-Trace-Id': process.env._X_AMZN_TRACE_ID
        };

        // propagate remote AWS X-Ray span to current execution context
        await api.context.with(api.propagation.extract(mockHttpRequestHeaders), async () => {

            const remoteSpan = new NoRecordingSpan(api.context.active());

            const handlerName = path.basename(process.env._HANDLER)

            const handlerSpan = tracer.startSpan(handlerName, {
                parent: remoteSpan,
                kind: api.SpanKind.CONSUMER
            });

            let handlerReturn = null;
            let handlerError = null;
            try {
                handlerReturn = await tracer.withSpan(handlerSpan, async () => {
                    return handler(event, context, callback);
                });
            } catch (error) {
                handlerError = error;
                // FIXME: Recorded exceptions to not make it to x-ray, attached to the respective
                // span. By allowing the exception to bubble up, Lambda's x-ray integration
                // will ultimately record the exception at a higher level span.
                handlerSpan.recordException(error);
            }

            handlerSpan.end();

            // ensure exporter(s) have a chance to flush  spans before
            // lambda fn shuts down or freezes
            await api.trace.getTracerProvider().getDelegate().shutdown();

            if (handlerError) {
                throw handlerError
            }

            return handlerReturn;

        });
    };
}

// eslint-disable-next-line no-unused-vars
export const handler = instrumentHandler(async (event, context, callback) => {
    console.log(process.env);

    const response = await axios.get('https://ifconfig.co/json')
    console.log(response.data);
});

After further investigation, it seems that the approach highlighted above likely does not handle the scenario in which Lambda's are frozen, and potentially never unfrozen. See the following open issue for details:
https://github.com/open-telemetry/opentelemetry-js/issues/1739

A external-type lambda extension is needed?

@joebowbeer  - Possibly, it looks like some effort has been put towards using Lambda extensions for OTEL support. TBH I'm not familiar enough with Lambda's life-cycle or how the OTEL Collector works to know what additional challenges might be involved in the extension approach.
Maybe someone from one of the following projects can provide recommendations.
https://github.com/open-telemetry/opentelemetry-lambda-extension

@codeboten
@carlosalberto

https://github.com/aws-observability/aws-otel-lambda

@wangzlei
@alolita
@lonewolf3739

Done.


Problem using webgl renderer on mobile

Hi for all. Using pixi version 4.5.6 i got a problem with WebglRenderer. It isn't working on Samsung J1 Mini (CanvasRenderer works great). On Xiaomi Redmi 3 Pro WebGlRenderer it works slowly (20 fps when stage has 1 child - Sprite). On both devices works pixi v. 4.5.2 (newer versions doesn't works). Thanks.

Hello! Thanks for reporting this.
We need some more information about what you're actually using to preview. Does it happen on these 2 devices with this example: http://pixijs.github.io/examples/#/basics/basic.js
I assume you're using Chrome? Do you happen to know what version of browser/Android? Anything else you could provide would be useful. Thanks.

Basic example doesn't work (CanvasRenderer works only) no Samsung J1. Application was built with cordova: 6.2.3. Android 5.1.1. No one plugins. Default cordova project with basic.js.


Customer feedback | az iot dps enrollment | explain --device-information and fix spelling in example

Describe the bug
Feedback category
Need more information /and/
Typo in example
Reference command
az iot dps enrollment create --device-information
Summary
From Learn Customer Feedback, a customer is requesting more information on the --device-information parameter. There is also a typo in the example (See screen print below).
Request

Fix the typo in the current example
And more information to the parameter short-summary

Where is this device information shown?
What is a good business purpose for this parameter use?

Doc link
https://learn.microsoft.com/en-us/cli/azure/iot/dps/enrollment
Customer feedback
"
The document is ok but I have two attention points. --device-information seems to be the official name but the examples show --device-info. Please be consistent. And, what does this field do? Where is this device info shown? in the device twin??? I have no clue where this is kept. Please explain better.
"
Screen print of typo in example

Related command
az iot dps enrollment create
Errors
n/a
Issue script & Debug output
n/a
Expected behavior
n/a
Environment Summary
Azure CLI 2.62.0
Additional context
No response

Thank you for opening this issue, we will look into it.

Hi @dbradish-microsoft, thanks for the feedback, will investigate and update!


Incorrect links for "PARAMETER OF"  in documentation search results

This links to https://typst.app/docs/reference/types/array/#map-parameters-mapper. But id map-parameters-mapper doesn't exist. Instead, it should link this: https://typst.app/docs/reference/types/array/#methods-map-parameters-mapper.
Same with parameter of insert: https://typst.app/docs/reference/types/dictionary/#insert-parameters-key should be replaced with https://typst.app/docs/reference/types/dictionary/#methods-insert-parameters-key.
So the issue is with all "PARAMETER OF " links.

It's probably only broken for method parameters, not for normal functions. Since methods are currently being reworked to functions in type scopes, I would fix it together with that.

This is fixed now.


Adding u, VR DECam filters

Hi, I would like to open a PR to be able to use the u, VR DECam filters along with the existing ones (grizY). The builtins.py code seems straightforward to modify. How can I upload the *.dat files?
http://svo2.cab.inta-csic.es/svo/theory/fps3/index.php?id=CTIO/DECam.u
http://svo2.cab.inta-csic.es/svo/theory/fps3/index.php?id=CTIO/DECam.VR_filter&&mode=browse&gname=CTIO&gname2=DECam#filter

The .dat files are hosted on the github.io website, https://github.com/sncosmo/sncosmo.github.io/tree/master/data/bandpasses. You can open a PR there first, then we can test the code with a PR here.

Thanks, I opened a pull request for the u band in sncosmo.github.io. I don't see the file with atmosphere + filter + instrument component for the VR filter, but that's rarely used so lower in priority.


[Node.js] Enable record/playback using pipeline

Using pipeline to facilitate record and playback which
is more easy to use.

@halton PTAL thanks.


ARM_ERR_ARM_GO_AT on RPI 1

Hello,
in order to test the device i do the following
1- I built arminfo
2- i disabled the gpio 4 with echo command as you explained in another issue
3- i run via SSH sudo ./arminfo  i get the following error.
ARM Init failed!
ARM_ERR_ARM_GO_AT: 'ARM command Error, can't switch to AT command.'
I changed cmdline.txt for this
dwc_otg.lpm_enable=0 root=/dev/mmcblk0p2 rootfstype=ext4 elevator=deadline rootwait
then i type
echo 4 > /sys/class/gpio/export
echo out > /sys/class/gpio/gpio4/direction
echo 1 > /sys/class/gpio/gpio4/value
Now sometimes it succeed but most of the time it failed with various error
ARM Init failed!
ARM_ERR_ARM_GET_REG: 'ARM command Error, from get register.'
or
ARM Init failed!
ARM_ERR_PORT_WRITE_READ: 'Port Error, at the port reading/writing.'
or
ARM Init failed!
ARM_ERR_ARM_GO_AT: 'ARM command Error, can't switch to AT command.'
Can you help ?
regards
Michel

Hi,
It is possible if the radio module is busy by sending a message to Sigfox for example.
You can drive the reset pin (during 10ms) before to call ./arminfo.

The problem was fix by issuing several time the echo command.
We also changed the arminfo code to receive a 64bit representation of the serial num.
Your 32bit cast is not correct for registration. But now we have a more critical issue with initialization sequence. Il will open another issue for it.
Regards
Michel


Update fading_edge_scrollview to 4.1.1

Hello bro, you need to upgrade fading_edge_scrollview to version 4.1.1 because the version 3.0.0 has an issue with PageController and ScrollController. Upgrading to 4.1.1 solve this

This is truly disastrous, as the package does not work AT ALL now.

This is truly disastrous, as the package does not work AT ALL now.

@BigYajuu I agree, as a temporal fix you can copy all the marquee code to a local widget and delete marquee lib from pubspec, then install the fading_edge_scrollview 4.1.1 and the problem will be solved.

This is truly disastrous, as the package does not work AT ALL now.

@BigYajuu I agree, as a temporal fix you can copy all the marquee code to a local widget and delete marquee lib from pubspec, then install the fading_edge_scrollview 4.1.1 and the problem will be solved.

An even easier solution is to simply add this to your pubspec.yaml
dependency_overrides:
  fading_edge_scrollview: ^4.1.1

This is truly disastrous, as the package does not work AT ALL now.

@BigYajuu I agree, as a temporal fix you can copy all the marquee code to a local widget and delete marquee lib from pubspec, then install the fading_edge_scrollview 4.1.1 and the problem will be solved.

An even easier solution is to simply add this to your pubspec.yaml
dependency_overrides:
  fading_edge_scrollview: ^4.1.1

In my particular case, doing that cause a conflict beteween marquee lib pubsec and my project's one, so i didn't have another option, if that works for you great job!!
Cheers mate!

In the upcoming two days you will get an update to 2.3. 😎  be prepared.


Tabs auto-convert to spaces when saving makefile in Opensuse 42

Hi Sir,
I work in makefile recently using your config under OpenSuse 42, I get the following error when I executing my makefile.
makefile:2: *** missing separator (did you mean TAB instead of 8 spaces?).  Stop.
I can insert Tabs when I editing files, but tabs auto convert to spaces when saving. The same config works in Windows but get error under Linux.

I worked on a Makefile today myself and verified that this is not a problem for me. You'd need to provide instructions detailing how to reproduce this from scratch with an empty checkout of this config if I'm to guess at what might be going wrong.

I download the latest .emacs.d config, then start up emacs. C-X C-F open a makefile,
test : test.o
        cc -o test test.o

test.o : test.c
        cc -c test.c

This is the simple makefile. I can insert tab before cc command, but after C-X C-S save the file, tabs before cc command change to spaces. Open a shell then execute make command, get error
makefile:2: *** missing separator (did you mean TAB instead of 8 spaces?).  Stop

I tested it under Arch Linux.The Makefile is normal.
I have never used OpenSUSE, but I suspect that the system text encoding or keyboard layout settings.
That is the system by default tab into a space.
I can not provide good advice. But if you don't customized configuration, you can try delete ~/.emacs.d/ and  reinstall this configuration.

Thanks!
I have already deleted .emacs.d/ and re-download the from Github, restart emacs, it doesn't work.
After I started emacs -nw -Q in console and tried vim, it worked.
I know, it must be some configuration in my OpenSuse conflict with emacs.d config, but it is werd.

@viscroad That is, it does not load any configuration but can be used……
Have you used any other configuration? Try the other configuration files on GitHub might be better......
Put your system's language and character encoding. This helps solve the problem.

I know, it must be some configuration in my OpenSuse conflict with emacs.d config, but it is weird.

What happens if you start Emacs using the --no-site-file option? Does it work better then?

@YuanPre
Thanks! Limited accessing Internet these days, ha....
system language: en_US.UTF-8
keyboard layout: layout:     us

@purcell
Sir, sorry for late replying, --no-site-file works the same result, after save makefile, we get spaces instead original tabs... ...

Do you use a GUI or a CLI version?
And Your DE(Desktop Environment)？
Yeah ... now the problem seems to be your character in the preservation of the document was escaped ... ...
Have you tried vim or any other editor?
And……Tested on my openSUSE Tumbleweed with KDE Plasma.....
It's normal......

@yuanyuanpre
the following sys info. Tried both in GUI and CLI. I also tried in vim, it was fine. emacs -nw -Q was fine. It's weird... ...
~> uname -r
4.1.12-1-default
~> lsb_release -a
LSB Version:    core-5.0-amd64:core-5.0-noarch:desktop-5.0-amd64:desktop-5.0-noarch:imaging-5.0-amd64:imaging-5.0-noarch:languages-5.0-amd64:languages-5.0-noarch
Distributor ID: SUSE LINUX
Description:    openSUSE Leap 42.1 (x86_64)
Release:        42.1
Codename:       n/a
~> kde4-config -v
Qt: 4.8.6
KDE Development Platform: 4.14.10
kde4-config: 1.0
~> plasmashell --version
plasmashell 5.4.2

Yeah...Your system is like mine, except the kernel. My kernel is manually compiled linux-4.10.11-x86_64. The emacs version is 25.2 .
But the Makefile is passed on my system...
Really strange ah...
It seems beyond my ability.
Sorry for I couldn't solve your problem.
I have limited ability of emacs because I use emacs for only two months...

@YuanPres
My Emacs version is 24.3. What differences from us are kernel and Emacs version. I keep eyes on this and try to find the root cause. Whatever, thank you very much!!!  Maybe I will send you email when I fix this issue.
GNU Emacs 24.3.1
Copyright (C) 2013 Free Software Foundation, Inc.
GNU Emacs comes with ABSOLUTELY NO WARRANTY.
You may redistribute copies of Emacs
under the terms of the GNU General Public License.
For more information about these matters, see the file named COPYING.

@viscroad , same with issue-444

Closing this because there's no sign it's an issue with this configuration specifically.


Expand bindless bounds checking to runtime-sized descriptor arrays

Runtime-sized descriptor arrays are allowed by VK_EXT_descriptor_indexing as the final binding in a descriptor set layout. Since these arrays are not encapsulated in a struct type, OpArrayLength cannot be used to determine their length. Rather, the lengths for these arrays will be passed in a "debug input" buffer by the validation layer. The binding and format of the input buffer is documented in instrument.hpp.
To facilitate coordination with the validation layers, bounds checking of runtime arrays is off by default and must be enabled explicitly when creating the bindless bounds instrumentation pass.

My latest changes have now passed CI.

Thanks Jeff. Is (2) core or part of VK_EXT_descriptor_indexing?

2 is also new to descriptor_indexing, it falls under the runtimeDescriptorArray feature. 3 falls under runtimeDescriptorArray and descriptorBindingVariableDescriptorCount.

OK. Previously I had designed this with the assumption that there would be only one RuntimeArray of descriptors per descriptor set. Now we know there can be multiple.
So, given the debug input buffer:
struct {
uint32_t data[];
};
and a RuntimeArray of descriptors at (set=s, binding=b),
previously its length was at
data[ s + 1 ]
but now its length will be at
data[ data[ s + 1 ] + b]

@s-perron It looks like the CI build is failing because it is picking up a version of googletest where INSTANTIATE_TEST_CASE_P is deprecated, and spirv-tools is making heavy use of it. Do you have any quick thoughts on what to do here or should I track this down?

It should already be fixed.  Can you rebase?

Doh! Thanks.

@s-perron Any ideas on why the shaderc smoketest is failing at hlsl_structbuffer_fn2_comp?

A change in the validator is causing a unit test in glslang to fail.  Not a problem for you.  If other test pass, we can still merge.
@jeffbolznv Have all of your concerns been addressed?

I'm not really familiar with this code and haven't reviewed in detail, but it looks like the design has been updated to handle multiple runtime arrays per set, so I don't object.


Feature Request: import and sign raw transactions

Would it be possible for sparrow to sign an unsigned raw transaction hex that's created with bitcoin-core or bitcoinjs ?
Load transaction works great:
After loading, it could check that all inputs are controlled by a specific wallet and offer to sign it.
Thanks for considering,

Trying to understand this use case better. Both Bitcoin Core and bitcoinjs provide functionality to create PSBTs, which is the more appropriate data structure for an unsigned bitcoin transaction. Why would you not use them?

I didn't have a chance to try this yet but I'll do that next time, I assume loading an unsigned psbt that has an input controlled by a loaded wallet in sparrow will allow me to sign it with sparrow.

Yes, so long as the PSBT has been reasonably well constructed.


forge not finding existing branch

Summary of Bug
Expected command forge node start --provenance-branch release/1.7.x to fetch and build that version, however, getting result of The entered branch, release/1.7.x, does not exist.. The branch exists in github.
Version
v1.0.1 from pip
Steps to Reproduce
Run forge node start --provenance-branch release/1.7.x

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Fixed on commit 221f79a4940aa63adcf315be36432819f7b738ab


Revert "untap: add --force switch"

Reverts Homebrew/brew#9422

Review period skipped due to critical label.

Review period skipped due to critical label.


Not working on latest windows build

can you kindly take a look, the app not working since i update to build 19045.2673 (22h2)
got The instruction at 0x00007ffe2d4c0522 referenced memory at 0x00000000. The memory could not be read. Click on OK to terminate the program everytime i open. am i missing something

Weird. I got the same update and am having no issues so far.
Could you try reinstalling the app and restarting your PC and see if that changes anything?


Problems with Ginga Eiyuu Densetsu Gaiden (1999)

Reported by werndlymob, Jul 3, 2013
Here is a bit of a strange one. JMM claims that I am missing one episode for Ginga Eiyuu Densetsu Gaiden (1999) but when I check the episode list they are all marked as available (btw there is a small spelling error in the UI, Episodes->Availability->No Files Aavailable).
I have the Central Anime release, used to have the LD/DVD version but now switched to the BD version but in both cases it claims that I am missing 1 episode while still showing all episodes as available.
There are no specials or anything either for it so I can't for my life understand what I am supposed to be missing.
http://forum.team-mediaportal.com/threads/my-anime-3.107475/page-52#post-1000107

Pretty sure this was fixed in Version <IP_ADDRESS>.


Carrousel more like mockup design

Carrousel is more like desktop design, i use a div below the carrousel section with the class container and i put a margin: auto; this for center the div and i put a max-width: 960px; this is for not grow more in the sides.

please add a picture of the result of your work

Here is the picture.

Looking good

Very good

you have conflicts

 Carrousel Desktop design update (2)

Se crea un div para la parte del boton y los puntos; se le agrega una clase para centrarlos. y se resolvieron los conflictos en el html.

Can I make an observation?
I think there isn't space between points and button.
VD, do you think that is important?

Can I make an observation?
I think there isn't space between points and button.
VD, do you think that is important?

That part is now in develop branch, this pr is only for center carrousel and center botton galery and that little points.


python3Packages.tempest: remove unittest2 requirement

Motivation for this change
related: https://github.com/NixOS/nixpkgs/pull/154650
related: https://github.com/NixOS/nixpkgs/pull/155005
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 155057 run on x86_64-linux 1

2 packages built:

python310Packages.tempest
python39Packages.tempest

@SuperSandro2000 Why did you just merge even though I suggested a change?

Because I was lazy and I stopped careing about is only cosmetically and people get way to often way to angry over such changes.

Really? I didn't know that.

I lied. It's a 406 :D

https://github.com/NixOS/nixpkgs/pull/155279


Prolog files are identified as Perl6 and Perl code

I've noticed that heavily commented Prolog code is often detected as Perl6 or just Perl. See this repository for example. If there is quite a bit of normal Prolog code, recognition works OK. As far as I know some Perl code is sometimes recognized as Prolog too.
Misrecognized files are easy to find because comments in them are not properly highlighted.

There are currently some very basic heuristics for Perl and Prolog, but looks like most of those files are all/mostly comments.
vim modeline support would also be helpful here (/cc #1210)
/cc @arfon

@bkeepers, well, if Linguist could understand Emacs (and Vi) mode lines, it would solve the problem (at least, for me and those who use the mode lines).

PR in progress here: https://github.com/github/linguist/pull/2041

@mrkkrp - Linguist does now indeed support Emacs and Vim-style modelines.

@arfon, I'm sorry, it doesn't seem to work. In this repository every file has the following (Emacs) mode line:
%% -*- Mode: Prolog; -*-

Emacs does recognize it, and as far as I know it's perfectly valid mode line. However, as you can see some files are still recognized by Linguist as Perl files. May be I'm missing something?

Hi @mrkkrp - looks like we're requiring mode and not Mode: https://github.com/github/linguist/blob/6d770ab68f7aa9b4ea8a31c260b20cf962971daf/lib/linguist/strategy/modeline.rb#L4
@bkeepers @larsbrinkhoff perhaps we should change the regex to:
/-\*-\s*[mM]ode:\s*(\w+);?\s*-\*-/

It's case insensitive, so
%% -*- mOde: proLog; -*-

is also a valid mode line. So I guess /-\*-\s*mode:\s*(\w+);?\s*-\*-/i is good enough.

OK this is fixed in https://github.com/github/linguist/pull/2083 @mrkkrp. I probably won't cut a new gem until next week with these changes as a new gem was deployed just a couple of days ago.


NDRS-691: Add instrumentation around ContractRuntime

REF: https://casperlabs.atlassian.net/browse/NDRS-691
Sample Output:

I would say we will also need timers/gauges for this. cc @EdHastingsCasperLabs

I would say we will also need timers/gauges for this. cc @EdHastingsCasperLabs

I thought we simply want to track the amount of times these two specific requests are made yes? there is already some instrumentation around the contract runtime

I would say we will also need timers/gauges for this. cc @EdHastingsCasperLabs

I thought we simply want to track the amount of times these two specific requests are made yes?

I think we also want to measure how long it takes. cc @EdHastingsCasperLabs @henrytill

there is already some instrumentation around the contract runtime

Do we measure timings of these endpoints though?

I don't remember the exact context of this work and the ticket isn't much help, so it's hard for me to comment on the specifics here.  But in general I tend to agree with @goral09, timers are useful.

I don't remember the exact context of this work and the ticket isn't much help, so it's hard for me to comment on the specifics here.  But in general I tend to agree with @goral09, timers are useful.

@goral09 checking the code, we do track the timing of both requests, however they seem to be under the get_balance metric. I think it should be under a separate histogram metric for each request. I shall make that change

@goral09 checking the code, we do track the timing of both requests, however they seem to be under the get_balance metric. I think it should be under a separate histogram metric for each request. I shall make that change

@goral09 I think the PR is sufficient now

@goral09 I think the PR is sufficient now

bors r+

bors r+


🐞 Bug: DockerFile vulnerabilities in production

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Summary
The suggested improvements aim to enhance the Dockerfile by reducing the number of layers and making the file more concise. This can lead to a smaller Docker image size, improving efficiency and resource utilization.
Steps To Reproduce

Go to https://github.com/CircuitVerse/CircuitVerse
Fork the repo & cd into circuitverse
Open DockerFile & run using ./bin/docker_run.ps1 or  if you are on Mac - ./bin/docker_run

Expected Behavior
Details
The Dockerfile improvements focus on enhancing efficiency and readability.

Combining RUN Commands:
Purpose: Reduce Docker image layers to minimize size.
Approach: Consolidate multiple RUN commands into a single command for optimized builds.

Combining chown Commands:
Purpose: Improve Dockerfile readability and conciseness.
Approach: Merge multiple chown commands into a single instruction for better maintainability._

Impact
1. Reduced number of layers, leading to a smaller Docker image size.
2. Improved readability with consolidated chown commands.
3. Streamlined and more efficient build process, especially beneficial for production environments.
Device Information [optional]
- OS: Windows 11
- Editor: Vs Code
- PaaS: Docker Desktop

Are you working on this issue?
Yes

@tachyons Can you verify this issue and assign it to me!!

Hi @Madhu-mac If you take a look at production dockerfile https://github.com/CircuitVerse/CircuitVerse/blob/master/Dockerfile.production , there is very few layers in final image. It's a multi-stage build dockerfile.

https://github.com/CircuitVerse/CircuitVerse/blob/master/Dockerfile this dockerfile is for development not intended for production. If we reduce the layers, it can cause issue in caching

@tanmoysrt The dockerfile is running without any errors in production while running in local but while pushing the code we met with installation errors like lint, husky & yarn files not installed properly

@tanmoysrt The dockerfile is running without any errors in production while running in local but while pushing the code we met with installation errors like lint, husky & yarn files not installed properly

Share some screenshot of errors


Warning generated every time I use console

Hello.
In multiple applications, every time I use a console route I get the following warning:
HP Warning:  cli_set_process_title(): cli_set_process_title had an error: Not initialized correctly in /Users/geeh/www/modules.zendframework.com/vendor/zfcampus/zf-console/src/Application.php on line 534

This happens both inside and outside of ZF2 app.
Thank you.

Hello Gary, I have the same problem on my Mac: It seems this functionality is blocked for security reasons (by Apple).
While waiting to find a better solution, I work around the problem with @cli_set_proc_title().

@GeeH can you confirm that your working on a Mac? Which PHP versions are you using?

Same problem here with no real solution except using root
https://github.com/liip/php-osx/issues/139

I can indeed - I’m using Mac OS X 10.10.3 and PHP version 5.6.5
On 25 April 2015 at 12:59:46, Ralf Eggert<EMAIL_ADDRESS>wrote:
Same problem here with no real solution except using root
liip/php-osx#139
—
Reply to this email directly or view it on GitHub.

If you look at the method there is already a version check implemented.
https://github.com/zfcampus/zf-console/blob/master/src/Application.php#L528
Maybe we could add an OS check as well. Would PHP_OS be sufficient to identify Mac OS X? What do you get from var_dump(PHP_OS);?
Other suggestions to identify OS X?

Or maybe php_uname() could do the job?

string(6) “Darwin"
HTH :P
On 25 April 2015 at 12:59:46, Ralf Eggert<EMAIL_ADDRESS>wrote:
Same problem here with no real solution except using root
liip/php-osx#139
—
Reply to this email directly or view it on GitHub.

I will try to provide a fix later on...

http://en.wikipedia.org/wiki/Darwin_(operating_system)


chore: Modified the SSO Create Dialog

Modified the UI of Create Dialog in settings/SSO as it was making a issues with overall user experience (mentioned here)
Preview

Old (Mobile)
New (Mobile)

Old (PC)
New (PC)

@boojack I am noticing two commits in this PR, since I'm kind of new to all this should I remove the merge commit or not?

@MehadND Let's hold the changes to SSO section in this moment, because it's working just fine right now. And I'm planning on maybe turning the dialog into a drawer, so there's more room for more information.
BTW, thanks for your contribution!


MatthewsCorrCoef has massive slow down on both CPU and especially GPU

🐛 Bug
It's extremely slow to compute the Matthew Correlation Coefficient since torchmetrics == 0.10.0.
To Reproduce
During my testing, I am seeing massive slow down when computing the Matthew Correlation Coefficient, especially on the GPU (using pytorch-lightning to build, train and test a deep learning model's performance on this metric).
I have compiled a code sample (next section) and below shows the result over different versions of torchmetrics.

Code sample
from tqdm.auto import tqdm
import time
import torch
from torchmetrics import MatthewsCorrCoef

torch.manual_seed(1)

b, h, w = 10, 1080, 1920
device = "cuda" # or cpu

def generate(b, h, w):
    prob = torch.rand(b, h, w).to(device)
    truth = torch.randint(0, 2, (b, h, w)).to(device)
    return prob, truth

batches = []
for _ in range(10):
    batches.append(generate(b, h, w))

mcc = MatthewsCorrCoef(num_classes=2).to(device)

t1 = time.time()
for detections, targets in tqdm(batches):
    mcc.update(detections, targets)
print(f"{time.time() - t1:.5f}")

Expected behavior
It's supposed to be faster in the range of ~0.1 second for GPU and ~0.9 second for CPU in this naive benchmark.
Environment

TorchMetrics version (pip): 0.10.0
Python & PyTorch Version (e.g., 1.0): python=3.9, pytorch=1.12.1
Any other relevant information such as OS (e.g., Linux): ubuntu18.04, (same behavior with 20.04)

wow, for GPU it is huge :octopus: :face_with_spiral_eyes:


IconDialog Callback not working in dialog fragment

I am showing the icon dialog from dialog fragment. I implemented the callback like below
I have already one fragment, I call the diaglog fragment inside of the this fragment. And in opened diaglog fragment I am showing IconDialog. Maybe something wrong with my fragment show settings.
First diaglog fragment call from fragment.
    @Override public void onResume() {
        super.onResume();

        if (!getUserVisibleHint()) {
            return;
        }

        // Set listener for float action button which has been defined in main activity.
        // Here we will override the listener which can be work for our current fragment.
        floatingActionButton = getActivity().findViewById( R.id.fab );
        floatingActionButton.setOnClickListener( new View.OnClickListener() {
            @Override public void onClick(View view) {
                CategoryInputDialog categoryInputDialog = new CategoryInputDialog();
                categoryInputDialog.show( getFragmentManager(), CategoryInputDialog.class.getName() );
            }
        } );
    }

Second dialog fragment(Icon dialog) call from first dialog fragment on createview.
public class CategoryInputDialog extends DialogFragment implements IconDialog.Callback {....}

imageViewIconSelection.setOnClickListener( new View.OnClickListener() {
            @Override public void onClick(View v) {
                IconDialog iconDialog = new IconDialog();

                iconDialog.show( getChildFragmentManager(), IconDialog.class.getName() );
            }
        } );

    @Override public void onIconDialogIconsSelected(Icon[] icons) {
        selectedIcons = icons;
    }

Callback goes always activity's callback.
private void callSelectCallback() {
        try {
            Icon[] icons = new Icon[selectedItems.size()];
            for (int i = 0; i < icons.length; i++) {
                icons[i] = selectedItems.get(i).icon;
            }

            if (getTargetFragment() != null) {
                // Caller was a fragment
                ((Callback) getTargetFragment()).onIconDialogIconsSelected(icons);
            } else {
                // Caller was an activity
                //noinspection ConstantConditions
                **((Callback) getActivity()).onIconDialogIconsSelected(icons);**
            }

        } catch (ClassCastException e) {
            // Callback interface is not implemented by caller
        }
    }

getTargetFragment returns null even if I set the setTargetFragment.
Currently I solve that issue with shared view model. I got the icon on main activity and set values to dialog fragment.

Are you using the latest version? The code you referenced last is from v2.2.0 and was changed in this commit to include parent fragment. Try using v2.5.0 it should work fine.

Thanks a lot for your quick response,
I was using old version 2.1.0. Because I am a bit confused while readin Read me. May be it would be nice to mention always latest version on the read me.
With below implementation it is working.

public class CategoryEditDialog extends DialogFragment implements IconDialog.Callback {
}

                IconDialog iconDialog = new IconDialog();
                iconDialog.show( getChildFragmentManager(), IconDialog.class.getName() );

    @Override public void onIconDialogIconsSelected(IconDialog dialog, @NonNull Icon[] icons) {
        selectedIcon = icons[0];
        iconViewIconSelection.setIcon( icons[0] );
    }


Can't leave graph view if there is a catalog stack in environment

Rancher Version: Master 6/15
Steps to Reproduce:

Go to catalog and add something
Once it started, go to graph view in stack details page
From drop-down try to change environments

Results: service.js:467 Uncaught TypeError: Cannot read property 'filterBy' of null and you are left in a state where none of the buttons work anymore so you can get off page unless you redirect it manually using url
Expected:  Should not error and mess up my environment

Version - v1.1.0-rc3
Verified fixed


Document requestVideoFrameCallback API

The requestVideoFrameCallback (https://wicg.github.io/video-rvfc/) needs to be documented. It has landed in Chrome and is pretty useful
It seems that Firefox deems it worth-prototyping (https://github.com/mozilla/standards-positions/issues/250#issuecomment-613289284).
It would be important to add a note next to all references to HTMLVideoElement.mozPaintedFrames saying that this information is accessible via requestVideoFrameCallback.
Created the issue to validate that we want to document that on MDN or whether it is too early. If this sounds like an API that should be on DM, please assign this issue to me and I'll be happy working on it

To me, this feels a bit early to document. I've not seen anything over here about an intent to prototype or intent to ship going out yet. Another question for me would be — what evidence have we got that the spec is stable?
Saying that, it does seem to have some momentum and interest among web devs.

@chrisdavidmills I am a bit unsure how to gather whether a spec is stable or not
The "worth prototyping" standard position from Mozilla might be a good signal that this will be enabled in multiple rendering engine
What other steps can I take to ensure of the stability of this?
Will probably give that documentation a go to test Yari and the new content repo

@gsouquet fair answers here; it is not always the easiest question to answer. I think creating docs for this as part of the Yari testing would be great (keep the source code somewhere safe so it dfoesn't get lost), and you can always add browser compat data now.
As for finding more about implementation status, we can always ask. @padenot, has any progress been made on implementing requestVideoFrameCallback() in Gecko yet?

No.


Saeed/learner 504

Description
LEARNER-504
I'm not sure, this PR will fix the desired crases or not because underlying crashes are not replicate able at our end. But I'm hopeful it will, and definitely, this will increase the stability of the code. Moreover, this PR remove strong reference to self in blocks which would avoid retains cycles. This PR also addresses the issue that sometimes app got stuck in loading on relaunch.
Notes
A full regression should be executed on this PR so that we should have surety that these changes are not breaking the app. Because manly the changes are made in the code that is interacting with the database.
How to test this PR

Codecov Report

Merging #1031 into master will decrease coverage by <.01%.
The diff coverage is 5.88%.

@@            Coverage Diff             @@
##           master    #1031      +/-   ##
==========================================
- Coverage    48.4%   48.39%   -0.01%
==========================================
  Files         406      406
  Lines       25692    25695       +3
==========================================
+ Hits        12435    12436       +1
- Misses      13257    13259       +2

Impacted Files
Coverage Δ

Source/OEXInterface.m
27.34% <0%> (-0.04%)
:arrow_down:

Source/OEXDBManager.m
1.34% <0%> (+0.01%)
:arrow_up:

Source/OEXNetworkManager.m
27.48% <0%> (-0.17%)
:arrow_down:

Source/OEXDownloadViewController.m
0% <0%> (ø)
:arrow_up:

Source/OEXNetworkInterface.m
35.82% <0%> (-1.11%)
:arrow_down:

Source/OEXDownloadManager.m
17.24% <75%> (+0.28%)
:arrow_up:

Continue to review full report at Codecov.

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


Everything should be installed in /Library instead of /System/Library

Related to https://github.com/w0lfschild/mySIMBL/pull/95

@w0lfschild made this clear, SIMBL doesn't work with SIMBL.osax being installed in /Library, thus this PR is useless.


doc_id for nested objects should be computed from their JSON

The algorithm for computing the ID is as follows:

If doc_id is supplied in the config as the value of a JSON path, set X to the value of JSON path. If the JSON path returns an array, use the first element.
Trim X of leading and trailing whitespace
If X contains only alphanumeric and underscore, use it as is, otherwise hash it and log a warning. It would be nice if we don't have to do the hashing as it is likely to break someone's logic
Otherwise, the system should parse the JSON, write it as a string with keys sorted, and then hash it.

please refer to #113


Remove VNet Integration using ARM Template while moving to SKU where it's unsupported

[Enter feedback here]
I'm using VNet integration for function app(hosted on Standard tier) and deployed using ARM Template.
How can I disconnect active integration using ARM template. If I deploy app again without above nested resource, it does not remove this existing connection. This may be useful in most of the scenarios but I'm trying to switch my function app to Consumption Plan where vnet integration is not supported. So instead of disconnecting the vnet integration, deployment fails with error that the vnet int not possible for this SKU.
I'm aware of az cli way of disconnecting the vnet but would like the arm to handle it if it is possible.
{
"type": "Microsoft.Web/sites/networkConfig",
"apiVersion": "2019-08-01",
"name": "[concat(parameters('siteName') ,'/virtualNetwork')]",
"dependsOn": [
"[resourceId('Microsoft.Web/sites', parameters('siteName'))]"
],
"properties": {
"subnetResourceId": "parameters('integrationSubNetId')]",
"isSwift": true
}
}

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

ID: a7a98803-1438-b1b5-f543-7dd88bc4294e
Version Independent ID: 37ff1d0f-ed8e-5e4d-1f4c-1b9f6cffb938
Content: Integrate app with Azure Virtual Network - Azure App Service
Content Source: articles/app-service/web-sites-integrate-with-vnet.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

Hi @vaibhavramdasi,  thanks for your patience. We apologize for any frustration this issue may have caused.
At this time, VNET integration does not formally support ARM templates. The engineering team is aware of this limitation (#20348) and is working to resolve it. However, you can try using this workaround ARM template. Please note that this template is not supported/managed by Microsoft. At this point, using Azure CLI is probably the best way to remove a regional virtual network integration from your function application.
az functionapp vnet-integration remove --name                                        --resource-group                                        [--slot]                                        [--subscription]
We will close this out for now, but if you feel you need more information please just let us know and we will be more than happy to reopen this issue.