[Feature] Support Fallback to Full Parse When Using --partial-parse-file-path

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
When using the default target path or specifying --target-path without a partial_parse.msgpack file present, dbt falls back to a full parse, "Unable to do partial parsing because saved manifest not found. Starting full parse". This is very useful for cases where the existence of partial_parse.msgpack can't be guaranteed. However, when --partial-parse-file-path is used we lose this behaviour due to the error "Error: Invalid value for '--partial-parse-file-path': ... does not exist."
Describe alternatives you've considered
Getting around this today requires the use of --target-path or copying artifacts to the default target/ prior to running dbt. Both of these are viable options but have two minor downsides:
1. --target-path requires you modify the output path of artifacts to the directory where the partial p
arse artifacts may be stored.
2. Optionally copying artifacts to the desired target path requires an extra step prior to a dbt run.
Who will this benefit?
Users of --partial-parse-file-path who can't gaurantee artifact existence (e.g. CICD setup).
Are you interested in contributing this feature?
Yes
Anything else?
No response

Thank you for proposing this idea @d-cole !
I can definitely see how it would be convenient to provide an empty (or non-existent) folder when using --partial-parse-file-path. It could be similarly handy to do something similar with the --state or --defer-state flags.
But since --partial-parse-file-path is an intentionally undocumented private/hidden flag not meant for dbt users, I'm going to close this as "not planned" (along with https://github.com/dbt-labs/dbt-core/issues/10561).
But even if --partial-parse-file-path was a "public" flag meant to be used by end users, the "does not exist" error is what we've chosen for other flags like --state, --defer-state, etc. whenever the expected file doesn't exist. So this would have been not planned either way.


Feature/issue 1122 generalize is_constant_struct with parameter packs

Summary
This PR generalizes the is_constant_struct trait with parameter packs.
It replaces is_constant_struct<A> && is_constrant_struct<B> with is_constant_struct<A, B>
and !is_constant_struct<A> || !is_constrant_struct<B> with !is_constant_struct<A, B>.
Tests
Extended the existing is_constant_struct tests.
Side Effects
/
Checklist

[x] Math issue #1122

[x] Copyright holder: Rok Češnovar, Univ. of Ljubljana

[x] the basic tests are passing

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

@bob-carpenter, would you mind looking at this PR?

I can review it.

@bob-carpenter based on your comments I started thinking that we could maybe clean all this is_constant related traits we currently have.
Currently on develop we have:

is_constant
is_constant_struct (the same as is_constant as you note in the comments)
contains_nonconstant_struct (essentially a parameter packed version of !is_constant_struct<>)
is_nonconstant_struct (used only in contains_nonconstant_struct)

I propose a few options that would clear this up. Please comment if you like any of them or should leave this as is:
A)

make the is_constant_all<> trait that would replace is_constant and is_constant_struct
replace contains_nonconstant_struct with !is_constant_all<>
is_nonconstant_struct is removed
B)
make the is_constant_all<> trait that would replace is_constant and is_constant_struct
make the is_nonconstant_any trait that would replace  contains_nonconstant_struct and all the !is_constant_struct<> (80%+ is_constant_struct are used with !).
is_nonconstant_struct is removed
C)
same a B) but the is_constant_any uses the current name contains_nonconstant_struct. Also replace !is_constant_struct<> with it.

My preference would be A, with B the backup if you prefer is_nonconstant_any<> over !is_constant_all<>.

I like the plan to consolidate all calls into is_constant_all with ! for negation.

I updated the top-level comment in the PR to reflect that we are now consolidating all these traits. This has now also become a significantly larger PR. However, most of the changes are from is_constant_struct to is_constant_all and contains_nonconstant_struct to is_constant_all.
Non-find-and-replacing changes are in the prim/scal/meta/is_constant, prim/mat/meta/is_constant and prim/arr/meta/is_constant where I fixed the comments issue you raised in the review.

Please ping me when this is ready to review again. Thanks.

@bob-carpenter It is ready. I had a few issues with leftover tests, its fixed now.


The sequence of vin for transactions is incorrect

For example, the sequence of vin is [4 NEO, 3 NEO, 40 NEO]

UI: https://neoscan.io/transaction/8E23D99DA5D2BC836D9D31846AC26A33C9CDD07ACDF2568CCE8BCA71912AFD54
API: https://neoscan.io/api/main_net/v1/get_transaction/8e23d99da5d2bc836d9d31846ac26a33c9cdd07acdf2568cce8bca71912afd54

but the sequence of vin from blockchain is [40 NEO, 3 NEO, 4 NEO]

Blockchain: http://seed1.redpulse.com:10332?id=0&method=getrawtransaction&params=["8e23d99da5d2bc836d9d31846ac26a33c9cdd07acdf2568cce8bca71912afd54", 1]

@yongjiema while vout order is important as it is used to specify a vin, I think the sequence of vin has little importance, I guess it will be better to just use the same order returned by the nodes, but except from display point of view is there another reason that will make this bug a higher priority?

I just noticed it will be potential issue here.

I'm changing the label to enhancement as I don't see any urgent reason to deal with it. I guess a way to solve this issue is to change the vin mode to add an "index" field, this index can be derived from the json response of the node for a block. When displaying the vins we can then just do an "order_by" index for the vin.

I agree with the plan, it's not urgent for me either. Do you welcome the PR from me for the enhancement?

@yongjiema yes you can do a PR, but it will require a resync so it is harder to handle in release cycle

I just encounter a issue related to it, some unit testings are failed due to the order is different.

@yongjiema can you precise which test is failing?


Saving search engine not working as expected

The selected value on the settings manager for the app is not populated correctly with the selected item.

solved display issue, not correct method used on winforms form for combobox

solved in last release


v11: Update to Intel MPI 2021.13

This updates ESMA_env to v4.29.1 which moves to Intel MPI 2021.13 at NCCS on SLES15.
This should be taken in concert with https://github.com/GEOS-ESM/GEOSgcm_App/pull/652
This is tested zero-diff to Open MPI.

I've removed the blocker since https://github.com/GEOS-ESM/GEOSgcm_App/pull/652 has been approved


Labelling the field names for better language-support in validation-messages

Currently, the validation messages just return the untranslated field names. This works in english but not any other languages.
I have added two methods that return the translated field names.
The getValidatorMessages method is there if anyone wants to fully customize the messages.

@mtellenbach Nice one, looks good to me. Just needs to be tested and confirmed working.

Merged in f34690715537d891a586637ac7058bcda3612b5d
Thanks for this!


add test datasets

Coverage remained the same at 72.637% when pulling 25bd8120a651124c9d3636ab90f8fc841fa7fa83 on phobson:build-update into 63a93efde84c4c186e0c9821b357cf43b69ddab1 on Geosyntec:master.


There is no value for network in /home/admin/raspiblitz.info

Using WIfi and no Ethernet
Building sdcard by using RaspbianStretchDesktop.sh
At reboot while still connected though hdmi, and login as admin/raspiblitz
i get the following error message when running There is no value for network in /home/admin/raspiblitz.info

RaspbianStretchDesktop.sh is in between versions - please use the sd card image or even better wait some more days for the version 1.0-rc1 release. thx.


Feature: creat new app through new client interface

Feature Request
we are thinking bout adding a new client interface (mainly in java client  ) which can create a new app through only rpc request to the server side~
Is your feature request related to a problem? Please describe:

our customer code will run in docker or one of some container type in k8s in the future,  in the container circumstance, customer code can not conveniently use the shell tool------"run.sh" to create a new pegasus app;
Describe the feature you'd like:

we want to realize a client interface which can create a new app through only rpc requests to the server side
Describe alternatives you've considered:

Teachability, Documentation, Adoption, Migration Strategy:

Get/put etc. are business flow, but create/delete apps are management flow. I think mixed two flow together into client is not a good idea. We can't implement authentication in client, which is a basicly requirement for management flow.

Get/put etc. are business flow, but create/delete apps are management flow. I think mixed two flow together into client is not a good idea. We can't implement authentication in client, which is a basicly requirement for management flow.

I think client side can contain not only normal put/get, read/write functionality, but also can include some "tools like" interface for "management usage"  you have mentioned;  authentication is another topic,  since normal read or write also need it;

Get/put etc. are business flow, but create/delete apps are management flow. I think mixed two flow together into client is not a good idea. We can't implement authentication in client, which is a basicly requirement for management flow.

I think client side can contain not only normal put/get, read/write functionality, but also can include some "tools like" interface for "management usage" you have mentioned; authentication is another topic, since normal read or write also need it;

How to prevent common users from calling this interface? In most scenarios, only administrators have permission to create creat a new app.

Get/put etc. are business flow, but create/delete apps are management flow. I think mixed two flow together into client is not a good idea. We can't implement authentication in client, which is a basicly requirement for management flow.

I think client side can contain not only normal put/get, read/write functionality, but also can include some "tools like" interface for "management usage" you have mentioned; authentication is another topic, since normal read or write also need it;

How to prevent common users from calling this interface? In most scenarios, only administrators have permission to create creat a new app.

previously, we also cannot limit  permissions of normal users to create a app, since they can also use the shell tools;  In this case, I will add kerberos to the related rpc interface to let only the administrators to create new app;

Thanks to @GiantKing 's reminding, we should keep in mind not expose too many privilege to common Pegasus users.
But I also think we should achieve the goal by using authority control mechanism in Pegasus, not to provide Java API is not reasonable cause any body can use the shell tool to create table now, without any permission.

I think it's OK to add createTable interface . Many other storage systems support this interface, for example: Dynamo. And Here is the official document
And In my opinion, it's better to enable authentation to ensure safety.


Scala Project for algo

Hi, wangzheng
I've setup  a proper scala project structure to organise the code, and I've done chapter 05 Array with Scala Test to demo the usage, I will add more scala code based on the progress I made about the course.
The original files from scala folder has been merged to the project I may refactor it when I arrive that chapter
thanks

ch06 linked list also added


fix: improve bulk add routes performance

Without this change, the test will timeout. We are adding 500 routes or so in our application and each time a new route is added sortChildren is invoked and is slow and gets increasingly slower as new routes are added. This change only invokes sortChildren on the final add.
The cb function was removed as it was unused. Let me know if you'd like it back.

It's an interesting problem you found of having a very large number of routes at the same level, increasing the sort time on each addition. But sorting only sort siblings, not all routes. And all routes added are not necessarily at the same level, meaning that a strategy of sorting on the last added route won't work for every case.

Thanks @troch , any suggestions? At this point it takes over 4s on a powerful machine to load roughly 500 routes which is obviously an unacceptable startup time hit for a webapp. Is there a way to rework the sort algorithm to sort children routes as well (only when adding multiple routes at once)?

A sort of all nodes (starting with the root node and going through the different children) after adding multiple routes would be the right option, however it depends on the usage. It could be expensive to re-sort if you add 3 more routes to a tree already containing 100. With router5 you can add all routes at once, or one by one. Maybe we could have the following:

Current behaviour doesn't change from this package perspective: it sorts when it adds a route.
This package adds a new method for sorting all nodes, and an option to add / its constructor to disable sorting
When a router is created with Router5, routes are added, then the sort method is called

Please consider not using the word 'unacceptable', it is a very strong word in English and your comment comes across as blaming.

Those options would be fine. I don't mean unacceptable in any blaming sense so I apologize if it came across like that. I just mean there is currently no way around this issue for us and we can't progress with the startup hit. Thanks for looking into this!

Superseded by #23


Result builder autocomplete does not work with buildPartialBlock

Describe the bug
When in a result builder context, autocomplete does not suggest the new buildPartialBlock functions.
Steps To Reproduce
Steps to reproduce the behavior:

Define a @resultBuilder
Type build and invoke autocomplete

Expected behavior
Both buildPartialBlock endpoints should be options.
Environment (please fill out the following information)

OS: macOS 12
Xcode 14 beta 2

cc @rintaro, does code completion need special handling for this?

Yes, result builder functions are custom. We need to add buildPartialBlock entry to https://github.com/apple/swift/blob/main/lib/IDE/CompletionOverrideLookup.cpp#L496-L517
But I realized buildPartialBlock is not included in ResultBuilderBuildFunction.
@xedin I think ResultBuilderBuildFunction is used for diagnostics too, there's no need for buildPartialBlock at this point?

@rintaro I think this is just an oversight, we need to add buildPartialBlock - two versions of it, but I don't think there are any diagnostics that are specific to it at the moment.


Nodejs pdf2png.js example - text is not rendered

I'm following the below example from this repo for nodejs to render a pdf to an image:
https://github.com/mozilla/pdf.js/blob/master/examples/node/pdf2png/pdf2png.js
But it doesn't render any text at all.
After surfing though other issues here on github, I've added disableFontFace:false which partially fixed the problem - it now renders some text while other parts of it are still missing.
I'm seeing a bunch or warnings in server logs like below:
Warning: Error during font loading: Font HelveticaLTStd-Bold is not available
Warning: fontRes not available
Warning: TypeError: Cannot read property 'length' of undefined
Warning: Error during font loading: this._map[charCode].charCodeAt is not a function

Is it possible to configure pdf.js to render all text despite of font availability (fallback to default or something)?
Attach (recommended) or Link to PDF file here:
pdf-example.pdf
Configuration:

Web browser and its version: -
Operating system and its version: nodejs 8
PDF.js version: 2.2.228
Is a browser extension: no

Steps to reproduce the problem:

Use the example code from this repo:
https://github.com/mozilla/pdf.js/blob/master/examples/node/pdf2png/pdf2png.js
add disableFontFace:false
use this sample pdf to create an image:
pdf-example.pdf
check the resulting image.

What is the expected behavior? (add screenshot)
it renders all the text:

What went wrong? (add screenshot)
It doesn't render all the text, producing warnings Error during font loading :

I'm seeing a bunch or warnings in server logs like below:
Warning: Error during font loading: Font HelveticaLTStd-Bold is not available
Warning: fontRes not available
Warning: TypeError: Cannot read property 'length' of undefined
Warning: Error during font loading: this._map[charCode].charCodeAt is not a function

This part of the issue should have been fixed by PR #11218; any remaining issues would be a duplicate of issue #4244.

oh that's a bummer.. It doesn't seem to fallback to the default font as per PR #11218 (which would be just great) and the #4244 is from 2014... any chances that there might be a fix/workaround any soon?


docs/readme: crucial from curcial

Changed typo from word curcial to crucial

Everything looks good in this one! :+1: Thanks for the patience!

@homu r+

:pushpin: Commit e2b0537 has been approved by hobofan

:zap: Test exempted - status


Test package

Converted project to PyPi python package
Updated internal processing to match Dataflow pipeline architecture

broke translators into raw -> standard and standard -> enhanced

Is this good to merge now?

Is this good to merge now?

Looks like still just a couple comments to shore up first.

I believe everything is resolved. I resolved all of them on my end, hopefully that is in sync with what you see


Create an example with SSR

Discussed here #22

I can take this up?

Yeah @Guru107. It's yours. :100:

Hi @Guru107
Can I help you with this one?

Hey @matheusml was caught up in other work wasn't able to look at it. I m planning to getting it done over the weekend. Will definitely bug you if I need any help. Cheers 👍

Hey @matheusml check out the Repo . Let me know your feedback.

@Guru107 this is looking awesome!

This is awesome @Guru107
Please add this to the examples/react folder!

@matheusml this issue can be closed


Use NPM package Plausible Analytics

Summary
See Motivation.
Basic example

NPM package
plausible/plausible-tracker
plausible-tracker

Motivation
Add missing data adblockers by using NPM package as an alternative to proxy. See also: Adblockers and using a proxy for analytics

58408dbc69536ebe896980ef690bd460b267c5cc


molokai seems not work

my .vimrc:
set rtp+=~/.vim/bundle/Vundle.vim
call vundle#begin()
" alternatively, pass a path where Vundle should install plugins
"call vundle#begin('~/some/path/here')

" let Vundle manage Vundle, required
Plugin 'VundleVim/Vundle.vim'
Plugin 'L9'
Plugin 'fatih/vim-go'
Bundle 'scrooloose/nerdtree'
Plugin 'Valloric/YouCompleteMe'
Plugin 'vim-scripts/taglist.vim'
Plugin 'tomasr/molokai'
call vundle#end()

let g:molokai_original = 1
let g:rehash256 = 1
set sw=4

" colorscheme molokai

I build vim from source and run it in docker...   On my mac it work correctly

Wait, you commented out the last line, where the colorscheme should be set to molokai?
Maybe overstating the obvious here, but shouldn't you remove the double quote in the last line?

Oh... my mistake


BUG: Position of the Move to Top Button Is Not Perfect

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The "Move to Top" button is not positioned correctly on the screen. Its placement can interfere with other elements and does not follow the expected design standards.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned!
Make sure to attach before & after screenshots in your PR and include the issue number.
i am the project mentor you can always reach out to me for any help!
Also, please star ⭐ the repo before working on it.
Happy open-source contributing! 😊


Why namespaces instead of classes?

is there a reason behind that?
regards

This is because most functions are really free functions, they don't have much state, except for a few things. If I did it as class, I'd have to make it non-constructable, non-copyable, etc.-etc.
Don't see much reasons for making it a class if no one will ever have instances of it.


Is both DB2 and Oracle PLSQL syntax supported by activejdbc?

Many thanks to Igor for answering my previous query.
Is there any sample code for DB2 or Oracle PLSQL support?
Many legacy codes are modeled within such environment.

@gencube unfortunately there is no such support. General purpose of ActiveJDBC is an ORM. If you need  to call stored procs, you will have to use standard JDBC features. You might find the DB-Migrator useful to create such procs: http://javalite.io/database_migrations. Also, if you have just a question, just post on StackOverflow and tag activejdbc and javalite


Convert analyzer template test to node:test

node:test is pretty nice, and Uvu has some rough edges. This is just a first test: new tests would be written to node:test and existing tests would be ported over, starting with the analyzer.

Arg, this is why the new tests aren't running on CI: https://github.com/nodejs/node/issues/50658#issuecomment-1806581766
Should we update Node to 21?


Traceback after 1.7 Update

I'm getting this traceback after having the plugin updated to the new version
Traceback (most recent call last):
  File "/opt/sublime_text/sublime_plugin.py", line 141, in create_window_commands
    cmds.append(class_(window))
  File "advanced_new_file.commands.copy_file_command in /home/vessel/.config/sublime-text-3/Installed Packages/AdvancedNewFile.sublime-package", line 11, in __init__
TypeError: super(type, obj): obj must be an instance or subtype of type

Sublime 3 Build 3083
Ubuntu 15.04
After I added ANF to Ignored Packages, everything works fine.

Can you try restarting Sublime Text. That happens from time to time due to how ST loads plugins, along with how I've structured the inheritance of the plugin. I haven't had a chance to dig through fixing it completely.

Oh, yeah. It works. Thanks

No problem. I'm trying to catch up on the handful of bugs/requests I have, so I'll see if I can solve this. Thanks for using AdvancedNewFile.


Enter pain prompt

Added EnterPainPrompt.kt, which is a Card component that gets displayed in the Home Screen depending on if the user has entered their muscle Pain or Not

blocked by #21


USDCUSD Oracle Config

Description
This PR adds 4 price sources for the USDCUSD oracle. Oracle parameters have been calibrated:
aggregation:
    mid:
      maxPercentageDeviation: 0.01
      maxPercentageBidAskSpread: 0.005

reporter:
    blockBased:
        minReportPriceChangeThreshold: 0.0005

Rates and config parameters look good to me :)


[5.7.0] Scim2/users api doesn't return response when User name is added having '$' character.

Add<EMAIL_ADDRESS>from Tenant admin in add new user section.
Home>Identity	>Users and Role>Add>Add New User
Enter username as<EMAIL_ADDRESS>and Defined password.It will be added successfully .
After adding user run the scim2/users api from JMeter for the tenant and we see the following error comes in WSo2 server logs.
ERROR -  Error occurred while accessing Java Security Manager Privilege Block
Error occurred while retrieving SCIM user information
org.wso2.charon3.core.exceptions.CharonException: Error occurred while retrieving SCIM user information
Caused by: org.wso2.carbon.user.core.UserStoreException: Error occurred while accessing Java Security Manager Privilege Block
Caused by: java.lang.IllegalArgumentException: No group with name {c1}
at java.util.regex.Matcher.appendReplacement(Matcher.java:849)
at java.util.regex.Matcher.replaceFirst(Matcher.java:1004)
at java.lang.String.replaceFirst(String.java:2178)
Looks like the User added<EMAIL_ADDRESS>is not valid therefore JAVA API fails in replaceFirst() method.
So this user name is actually invalid and There should be validation in Tenant side for adding the New User as Valid email id.

This bug cannot be reproduced in 5.8.0-m24-SNAPSHOT
Steps :
UsernameJavaRegEx property under UserStoreManager in user-mgt.xml set as following to allow $ for usernames.
<Property name="UsernameJavaRegEx">^[\S]{3,30}$</Property>

Add user<EMAIL_ADDRESS>under tenant<EMAIL_ADDRESS>via Management Console
Request
curl -v -k --user test@test.com:password https://localhost:9443/t/test.com/scim2/Users/660944d1-7a08-4903-acb1-1aba32bbdb16

Response
{
   "meta":{
      "created":"2019-03-11T11:46:46Z",
      "lastModified":"2019-03-11T11:46:46Z",
      "resourceType":"User"
   },
   "schemas":[
      "urn:ietf:params:scim:schemas:core:2.0:User",
      "urn:ietf:params:scim:schemas:extension:enterprise:2.0:User"
   ],
   "roles":[
      {
         "type":"default",
         "value":"Internal/everyone"
      }
   ],
   "name":{
<EMAIL_ADDRESS>   },
   "id":"660944d1-7a08-4903-acb1-1aba32bbdb16",
<EMAIL_ADDRESS>}

Thanks @hpmtissera for the contribution. I will close the issue based on the above comment.


Add additional pages.

Add other additional pages listed here:

Developers page.
Submit Feedback page.

And Improve links in navigation bar and footer.

Closing with details in PR above.


Does support to AWS china Region？

Does support to AWS china Region？ cn-north-1

Not currently, but until an endpoint and AMI are added to regions.json in the codebase -- you can add support for yourself by expanding regions.json in your local copy of BWMG.  This will likely require an AMI to be created in China, if nobody else has created and published one yet.


nwg-launchers: 0.6.3 -> 0.7.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nwg-piotr/nwg-launchers/releases.
meta.description for nwg-launchers is: "GTK-based launchers: application grid, button bar, dmenu for sway and other window managers"
meta.homepage for nwg-launchers is: "https://github.com/nwg-piotr/nwg-launchers"
meta.changelog for nwg-launchers is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7.0 with grep in /nix/store/gb2lj3dcmms76r1rbdz6dy3m9jm35n81-nwg-launchers-0.7.0
found 0.7.0 in filename of file in /nix/store/gb2lj3dcmms76r1rbdz6dy3m9jm35n81-nwg-launchers-0.7.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
nwg-launchers

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gb2lj3dcmms76r1rbdz6dy3m9jm35n81-nwg-launchers-0.7.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nwg-launchers https://github.com/r-ryantm/nixpkgs/archive/518e0d65ca47caad3f3b5c2cd3f37b63e17c1efc.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gb2lj3dcmms76r1rbdz6dy3m9jm35n81-nwg-launchers-0.7.0
ls -la /nix/store/gb2lj3dcmms76r1rbdz6dy3m9jm35n81-nwg-launchers-0.7.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

nwg-launchers

Maintainer pings
cc @bbigras for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nwg-piotr/nwg-launchers/releases.
meta.description for nwg-launchers is: "GTK-based launchers: application grid, button bar, dmenu for sway and other window managers"
meta.homepage for nwg-launchers is: "https://github.com/nwg-piotr/nwg-launchers"
meta.changelog for nwg-launchers is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7.1 with grep in /nix/store/faa0xa4j2mibyb0v1p6s9cy26bq7g9vw-nwg-launchers-0.7.1
found 0.7.1 in filename of file in /nix/store/faa0xa4j2mibyb0v1p6s9cy26bq7g9vw-nwg-launchers-0.7.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
nwg-launchers

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/faa0xa4j2mibyb0v1p6s9cy26bq7g9vw-nwg-launchers-0.7.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nwg-launchers https://github.com/r-ryantm/nixpkgs/archive/115db75807436cb56273c65e885c62b0c16ae3ad.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/faa0xa4j2mibyb0v1p6s9cy26bq7g9vw-nwg-launchers-0.7.1
ls -la /nix/store/faa0xa4j2mibyb0v1p6s9cy26bq7g9vw-nwg-launchers-0.7.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

nwg-launchers

Maintainer pings
cc @bbigras for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nwg-piotr/nwg-launchers/releases.
meta.description for nwg-launchers is: "GTK-based launchers: application grid, button bar, dmenu for sway and other window managers"
meta.homepage for nwg-launchers is: "https://github.com/nwg-piotr/nwg-launchers"
meta.changelog for nwg-launchers is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/3qyd41agz1c2qrzpr9rdlsgbf0xl73pg-nwg-launchers-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/3qyd41agz1c2qrzpr9rdlsgbf0xl73pg-nwg-launchers-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
nwg-launchers

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3qyd41agz1c2qrzpr9rdlsgbf0xl73pg-nwg-launchers-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nwg-launchers https://github.com/r-ryantm/nixpkgs/archive/b1c253feba8c73dd18be20653fffb7e542699df3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3qyd41agz1c2qrzpr9rdlsgbf0xl73pg-nwg-launchers-<IP_ADDRESS>
ls -la /nix/store/3qyd41agz1c2qrzpr9rdlsgbf0xl73pg-nwg-launchers-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

nwg-launchers

Maintainer pings
cc @bbigras for testing.

@ofborg build nwg-launchers nwg-launchers.passthru.tests


add prop-types

add prop-types for react >= 15.5.0

@jaysoo any chance you could merge this and cut a new release?

This is not maintained anymore. You can install rn-prompt instead, which is
this very repo but compatible with React ^16
On 7 February 2018 at 02:27:18, Sam Dunster<EMAIL_ADDRESS>wrote:
@jaysoo https://github.com/jaysoo any chance you could merge this and cut
a new release?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jaysoo/react-native-prompt/pull/32#issuecomment-363613337,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADteLMYgn4NPZQrR_yT1fxSbz-gl7kp-ks5tSO3mgaJpZM4RUFRN
.


Tests for CollectBaseDistributionByCycle

thanks to @dekling
fixes #405

@akiezun Review complete.  I like the filtering for comments.  Just two trivial nitpicks.  Up to you if you want to keep the plotting or not.   It's not really being tested properly and we're not really supporting it, but if we have the command line options it doesn't hurt to exercise them.  Squash and merge whenever you're ready.

@akiezun I'd like to amend my review completed note.  Won't the commentPrefix == null case incorrectly filter out every line?
Please add tests for compareTextFile with and without a comment prefix.

Note that this test passes now which I believe should probably not.
    @Test
    public void compareTextFiles() throws IOException {
        IntegrationTestSpec.compareTextFiles(new File("gradlew.bat"),new File("README.md"));
    }

It might also be a good idea to throw an exception if the files are all empty.

Fantastic catch @lbergelson about the filtering! Very nasty bug indeed. I fixed it now and added a test for it. back to @lbergelson

@akiezun Back to you.

back to @lbergelson

@akiezun :+1:  Squash and merge.  Thanks for adding those tests.


Add pyproject.toml for Custom Node Registry

We are building a global registry for custom nodes (similar to PyPI). The registry makes using custom nodes more reliable by reserving globally unique names for each custom node, and supporting semantic versioning.
Here’s some more information on the registry.
Action Required:

[x] Go to the registry. Login and create a publisher id. Comment it here and we will update the PR (or just merge it and change it yourself).
[ ] Write a short the description.
[ ] Merge the separate Github Actions PR and run the workflow.

If you want to publish the node manually, install the cli and run comfy node publish
Please message me on Discord if you have any questions!

@nuanarchy

@nuanarchy

should be good to continue with rest of PR steps now!


MC: templates - category selection issue

On some templates the categories are pre-selected, disabling the input for any that can't be changed.
Two updates here:

It would be better to make these read only OR display a tool tip "Primary category can't be changed on this template" if they're not editable
I'm seeing an 'X' beside the tertiary category that doesn't do anything, we can remove that:

Also, this x appears below the sub-category input on mobile, if we get rid of it it will solve that issue

On mobile the placement of the "x" icon is broken:

It should appear like this:

Still broken for me:

@phoebemirman can we do something simpler here, have the default value be selectable again, so you don't need an 'X' for example see:

https://getbootstrap.com/docs/4.3/examples/checkout/


Redis support?

I see in the dockerfile you're including PECL drivers for redis and there are some references to redis:// urls in SimpleStorage api. Is redis supported as a storage backend? If so, how can I configure cockpit to use it?

truly wondering if an external REDIS service would fix this bug: https://github.com/agentejo/cockpit/issues/799
My PHP knowledge is to dusty and I'm to new to Cockpit to truely get what is happening at some moments.


mo_gaal Tensorflow -> Pytorch

I have tried to convert the mo_gaal model from Tensorflow to Pytorch. I didn't create a new test_mo_gaal.py because the original one works well.
In mo_gaal(Pytorch version), instead of keeping consistent with default hyper-parameter value as of the tensorflow version, I did tuning it and add a dynamic tuner in the model.
All Submissions Basics:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[x] Have you successfully ran tests with your changes locally?
[ ] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?
[ ] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

[x] Have you created a .py in ~/pyod/models/?
[x] Have you created a _example.py in ~/examples/?
[ ] Have you created a test_.py in ~/pyod/test/?
[x] Have you lint your code locally prior to submission?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

87 of 87 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 93.966%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
9032

Relevant Lines:
9612

💛  - Coveralls

The github client issue doesn't happen yesterday for me but does again today:(


Improve  documentation

Major questions need to be answered and documented:

How is <repository> used my official tooling? not at all today...
What are valid type values? Is it freeform?
What should the url link to? A link that can be opened in the browser? A URL you can pass to git clone?

Related:

show the data in UI: https://github.com/NuGet/NuGetGallery/issues/4941
add commit hash to metadata: https://github.com/NuGet/Home/issues/6547

/cc @emgarten @diverdan92

We've updated MSBuild target docs to mention the new RepositoryBranch and RepositoryCommit. We should also include these in some more high level docs.

See also #1046.

Documented here - https://docs.microsoft.com/en-us/nuget/reference/nuspec#repository


Dash.js version

Can I ask which version of dash.js code are you using when building pensive project?
I ask this question because the code of dash.js has changed so much and I want to compare what kind of changes you have made to the dash.js project.
Thanks and regards.

I believe it's dash.js v2.0. We have a mirror copy of the dash.js code for the project: https://github.com/hongzimao/pensieve/tree/master/dash.js

Thanks for your quick response.
I saw the mirror copy of the dash.js code. But the code is after your modifications.
So, I am wondering if I can get the exact version of dash.js so that I can compare the original code with your modifications.
Here list all the versions of dash.js https://github.com/Dash-Industry-Forum/dash.js/tags?after=v2.9.0

From: Hongzi Mao<EMAIL_ADDRESS>Sent: Tuesday, July 21, 2020 10:29 PM
To: hongzimao/pensieve<EMAIL_ADDRESS>Cc: Yaxiong Xie<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [hongzimao/pensieve] Dash.js version (#109)
I believe it's dash.js v2.0. We have a mirror copy of the dash.js code for the project: https://github.com/hongzimao/pensieve/tree/master/dash.js
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/hongzimao/pensieve/issues/109#issuecomment-662205161, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJ64W57TQ452PE7VWVAWIQLR4ZFIVANCNFSM4PEHFCRQ.

I see what you need to do. I just checked our development branch and it seems we downloaded a version around Oct 2016. Unfortunately I can't find the exact dash version. But can you diff on the dash versions before this date and after v2.0 (i.e., v2.0.0 - v2.3.0 in https://github.com/Dash-Industry-Forum/dash.js/tags?after=v2.6.1). It's likely that you can find the right version by finding the least amount of diff.
I don't think we add more than a thousand lines of code in dash. Remember to only diff on the source code but not the compiled files like dash.all.min.js.
Hope this helps!

Feel free to confirm here the files and the code we modified when you find the version with least amount of diff. We can then double check. Thanks!

Got it. I will update when I get the results.
Thanks for your help, it really helps.

From: Hongzi Mao<EMAIL_ADDRESS>Sent: Tuesday, July 21, 2020 10:49 PM
To: hongzimao/pensieve<EMAIL_ADDRESS>Cc: Yaxiong Xie<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [hongzimao/pensieve] Dash.js version (#109)
I see what you need to do. I just checked our development branch and it seems we downloaded a version around Oct 2016. Unfortunately I can't find the exact dash version. But can you diff on the dash versions before this date and after v2.0 (i.e., v2.0.0 - v2.3.0 in https://github.com/Dash-Industry-Forum/dash.js/tags?after=v2.6.1). It's likely that you can find the right version by finding the least amount of diff.
I don't think we add more than a thousand lines of code in dash. Remember to only diff on the source code but not the compiled files like dash.all.min.js.
Hope this helps!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/hongzimao/pensieve/issues/109#issuecomment-662210686, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJ64W53C5GFDVCBWQ2NAZADR4ZHTNANCNFSM4PEHFCRQ.

I guess it is dash.js-2.3.0.

This looks right. We mainly changed AbrController.js.

Yep. I believe we found the correct version.


Open up a little the DelegatedAuthenticationClientLogoutAction

For customizations...

Backported to the 6.6 branch: https://github.com/apereo/cas/pull/5788


adding deployment steps to .NET Aspire + Redis to Azure

Adding a new deployment article to deploy .NET Aspire that uses Redis to Azure
https://github.com/dotnet/docs-aspire/issues/696

Fixes #696 (if available)
Fixes #430

Internal previews

📄 File
🔗 Preview link

docs/caching/caching-components-deployment.md
Tutorial: Deploy a .NET Aspire app with a Redis Cache to Azure

@IEvangelist  applied all suggestions in batch.


Please add license to this repo

Something like https://github.com/boto/boto3/blob/develop/LICENSE
In order for the team I'm on to get repos approved for internal use it has to have a license file.
Thanks!

See the "license" link on http://testfixtures.readthedocs.io/en/latest/?
That links to: https://github.com/Simplistix/testfixtures/blob/master/docs/license.txt
...which appears to be what you're asking for.

Good deal thanks!

Also see 814149e438da106d49fa92d3585e9c1a54a266a3


zfs test_maximum_size_enforced: test the right scenario and verify that

See https://clusterhq.atlassian.net/browse/FLOC-2509
Note that this pull request depends on #1716

Creating a proper pull request against master.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1196 ms

Resolved: Bofrost.at performance has improved in c7f4df4.


Fix for Python 3

:star: PR built and available in a preview environment itamaro-python-http-demo-pr-1 here


My cost estimate looks wrong.

Environment - GKE
Backend - remote, Terraform Cloud
Regions - us-east1 & us-east4
I've tried get cost estiomation via CLI and Terraform Run Tasks, and from distant seems works fine about cost reports, or comments updates in case of GitHub Actions. According to billing, my average monthly total cost is about $10,000 for both environments - staging & production, wich cannot correspond with Infracost report.
For instance:
      "some-node-pool" = {
-       machine_type = "n2-standard-4"
+       machine_type = "n2-standard-8"
        min_count    = 1
        max_count    = 6
        disk_size_gb = "50"
        auto_upgrade = false
      }

Terraform Cloud Run Task:
Monthly cost will increase by +$1,022 (+1%)
Previous monthly cost: $70,647
New monthly cost: $71,669
Time generated: Feb 7, 2022, 4:28:50 PM
~ module.k8s.module.gke.google_container_node_pool.pools["some-node-pool"]
+$1,022 ($1,535 → $2,557)
~ Instance usage (Linux/UNIX, on-demand, n2-standard-4 → n2-standard-8)
+$1,022 ($1,533 → $2,555)
Monthly cost change
Amount:  +$1,022 ($70,647 → $71,669)
Percent: +1%
My terraform directory structure.
#  tree terraform
terraform
├── environments
│   ├── production
│   │   ├── main.tf
│   │   ├── production.auto.tfvars
│   │   ├── variables.tf
│   │   └── versions.tf
│   └── staging
│       ├── main.tf
│       ├── staging.auto.tfvars
│       ├── variables.tf
│       └── versions.tf
├── main.tf
└── modules
    ├── cloudflare
    │   ├── main.tf
    │   ├── readme.md
    │   ├── variables.tf
    │   └── versions.tf
    ├── google-cloud-provider
    │   ├── main.tf
    │   ├── output.tf
    │   └── variables.tf
    ├── kubernetes
    │   ├── main.tf
    │   ├── output.tf
    │   ├── readme.md
    │   ├── variables.tf
    │   └── versions.tf
...

# cat terraform/main.tf
terraform {
  required_version = ">= 0.12"

  backend "remote" {
    hostname = "app.terraform.io"
    organization = "<orgName>"

    workspaces {
      prefix = "prefix"
    }
  }
}

Any comments or suggestions are welcome.

Thanks @thuban-arrakis, yeah this seems quite a bit off.
Can you run a infracost breakdown against this project and let us know:

Any relevant parts of the full Infracost breakdown
Currently we use the autoscaling.min_count attribute, or if that doesn't exist the node_count or the initial_node_count for calculating the cost. What attribute would you expect to be used for this?

If you're able to generate the Terraform plan JSON (terraform plan -out plan.save, terraform show -json plan.save | jq) and send us any relevant parts that would help us look into this as well.

Thanks @thuban-arrakis, yeah this seems quite a bit off.
Can you run a infracost breakdown against this project and let us know:

Any relevant parts of the full Infracost breakdown
Currently we use the autoscaling.min_count attribute, or if that doesn't exist the node_count or the initial_node_count for calculating the cost. What attribute would you expect to be used for this?

If you're able to generate the Terraform plan JSON (terraform plan -out plan.save, terraform show -json plan.save | jq) and send us any relevant parts that would help us look into this as well.

Closing this issue, it's all about prevous .infracost cache =)
Cost estimation work perfectly fine.

Thanks @thuban-arrakis. I'm wondering why the cache wasn't invalidating when the files were changed? If you have any insight on that it might be good to check if our cache is working as expected.


Anchor file for YOLOV4

I have one question should we change the anchors value for YOLOV4 when trained on my dataset  or it will be the same ?

You do not need to change it .


Disconnect condition in Examples

The examples are very simple and basic, no doubt by design, however the example code can lead to a situation where the connection drops and is not handled.
I had a project where the remote sensor that was publishing a value to an MQTT server worked fine on the bench, but at it's remote location, it was failing at seemingly random intervals, but usually within 24 hours of a reset.  This morning it failed within 15 minutes of a reset, so I knew something was wrong with the code.  The remote location is across a, somewhat unreliable at times, WiFi bridged connection.  (yep, need to fix that too...  :-)  )
As the example code stands, if the Arduino misses a PING packet exchange, it will disconnect.  In the example code, this disconnect is permanent.  This can happen on unreliable links (WiFi) or if the publisher is unable to communicate with the server for whatever reason (ISP outage).  The symptom is that all other code on the Arduino continues to operate normally, with the exception of the MQTT client functionality.
The good news is that the functionality is already in the library, it's just not shown in the examples.  Simply check the result of the client.loop() statement in the main loop and act upon it.
FALSE == we're disconnected so it's time to try reconnecting
TRUE == all is good.
In the examples, replace:
client.loop()

with
if(!client.loop()) client.connect("arduinoPublisher");

or, if want to check the client.connect() as well, use something like this:
  if (!client.loop()) {
    Serial.print("Client disconnected...");
    if (client.connect("arduinoPublisher")) {
      Serial.println("reconnected.");
    } else {
      Serial.println("failed.");
    }
  }

I don't expect an update the library for this, but I did want to document this here on GitHub so that it might be useful to future users of your excellent library.  Thanks Nick!!

The basic example has been updated to include reconnect logic, and a new example of non-blocking reconnect logic added.

Hi,
I´m still trying to run the simple ESP8266 DEMO, but after awhile the MQTT connection seems to disconnect.
I use this Code:
/*
 Basic MQTT example

  - connects to an MQTT server
  - publishes "hello world" to the topic "outTopic"
  - subscribes to the topic "inTopic"
*/

#include <ESP8266WiFi.h>
#include <PubSubClient.h>
#include "MySecret.h"

// Update these with values suitable for your network.
IPAddress server(192, 168, 1, 203);
long lastMsg = 0;

void callback(const MQTT::Publish& pub) {
  // handle message arrived
}

WiFiClient wclient;
PubSubClient client(wclient, server);

void setup() {
  // Setup console
  Serial.begin(115200);
  delay(10);
  Serial.println();
  Serial.println();

  client.set_callback(callback);
}

void loop() {
  if (WiFi.status() != WL_CONNECTED) {
    Serial.print("Connecting to ");
    Serial.print(ssid);
    Serial.println("...");
    WiFi.begin(ssid, password);

    if (WiFi.waitForConnectResult() != WL_CONNECTED)
      return;
    Serial.println("WiFi connected");
  }

  if (WiFi.status() == WL_CONNECTED) {

    if (!client.loop()) {
      Serial.println("### Client has disconnected...");
      client.disconnect();

      // add you code to reconnect here.
      if (client.connect("arduinoClient")) {
        Serial.println("MQTT-Client connected");
        //client.publish("outTopic","hello world");
        //client.subscribe("inTopic");
    }
  }

/*
    if (!client.connected()) {
      Serial.println("MQTT-Client disconnected");

      if (client.connect("arduinoClient")) {
        Serial.println("MQTT-Client connected");
        //client.publish("outTopic","hello world");
        client.subscribe("inTopic");
      }
    }

    if (client.connected())
      client.loop();
*/

  }

    long now = millis();
  if (now - lastMsg > 300000) {
    lastMsg = now;
    client.publish("/30/Status/MQTT","on");
  }

}

I also post this Problem here:
https://github.com/Imroy/pubsubclient/issues/37
I hope, that someone got the same problem and solved it finally.......
Greetings from Hamburg
Pf@nne

@Pfannex use knolleary's mqtt_reconnect_nonblocking.ino https://github.com/knolleary/pubsubclient/blob/master/examples/mqtt_reconnect_nonblocking/mqtt_reconnect_nonblocking.ino
I have no recoonet at least one month now.
FYI:
https://github.com/chaeplin/esp8266_and_arduino/blob/master/_02-mqtt-sw-temperature/_02-mqtt-sw-temperature.ino
https://github.com/chaeplin/esp8266_and_arduino/blob/master/_03-hx711-scale/_03-hx711-scale-esp-12/_03-hx711-scale-esp-12.ino
(disable serial out also help)

@chaeplin,
thanks for your hints!
I modified knollearys mqtt_esp8266.ino example to the reconnect example.
https://github.com/Pfannex/pubsubclient/blob/master/examples/mqtt_esp8266/mqtt_esp8266.ino
But I always got a lot of disconnects!
MQTT disconnected....
MQTT disconnected....
MQTT disconnected....
MQTT disconnected....
MQTT disconnected....
MQTT disconnected....

Are you able to run my sketch on your hardware?
Which ESP-library do you use?
Thanks for your Help!

https://gist.github.com/chaeplin/2b69fb46479584a354b7
a modified one, no reconnect.
https://github.com/esp8266/Arduino, Stable version.

I got some reconnects again......
I think I used not the right ESP8266 library (Board)

Right now I saw that there is a specific board for my ESP8266-12e!
NodeMCU 1.0 (ESP-12e Module)
I changed the settings, now I´m waiting...... :-)

pubsubclient : git version / no modification
mqtt : mosquitto ver 1.3.4 / Debian
Arduino IDE 1.6.5 / with Board manager / OS X
esp8266 : esp12
Connecting to Myhome
...........
WiFi connected
IP address:
<IP_ADDRESS>
failed, rc=-1
MQTT disconnected....
MQTT connected....
0
0
0
0
0
0
0
0
0
0
0
0

Still reconnects!
mosquitto.conf
# Place your local configuration in /etc/mosquitto/conf.d/
#
# A full description of the configuration file is at
# /usr/share/doc/mosquitto/examples/mosquitto.conf.example

pid_file /var/run/mosquitto.pid

persistence true
persistence_locat

ion /var/lib/mosquitto/

log_dest file /var/log/mosquitto/mosquitto.log

include_dir /etc/mosquitto/conf.d

Looks like yours.....
mosquitto version 1.4.4 on Debian Wheezy (Raspberry PI 2)
Arduino IDE 1.6.5
I´m not sure to use the right ESP-board,
does my board  looks like yours?

My board is generic ESP8266.
Do you have more codes than https://github.com/Pfannex/pubsubclient/blob/961700bc79c604bb187f8f62bd236a124339e369/examples/mqtt_esp8266/mqtt_esp8266.ino ?
Try to change MQTT_KEEPALIVE in PubSubClient.h from 15 to 60.
(Mine is 15)

You can find some more code here
https://github.com/Pfannex/AktSen
I changed MQTT_KEEPALIVE from 15 to 60 without success.
I also changed the ESP-12e into a ESP-12, also without success.
I´m very close to change the ESP against a W5100 wired controller.
What else can I do?
Where are I´m wrong?

I've got a couple esp8266 -12 on order - should arrive later this week.
Will see how I get on over the weekend.
On Mon, 16 Nov 2015 19:56 Pfannex<EMAIL_ADDRESS>wrote:

You can find some more code here
https://github.com/Pfannex/AktSen
I changed MQTT_KEEPALIVE from 15 to 60 without success.
I also changed the ESP-12e into a ESP-12, also without success.
I´m very close to change the ESP against a W5100 wired controller.
What else can I do?
Where are I´m wrong?
—
Reply to this email directly or view it on GitHub
https://github.com/knolleary/pubsubclient/issues/41#issuecomment-157152038
.

Great!
thank you very much!  I'm a little bit desperately.....

@Pfannex
I think board type is not a problem(flash size, serial speed...).
I have checked my code to mosquitto server(1.4.5/Wheezy) on pi B+ and 2. MQTT_KEEPALIVE is 15.
No reconnect at leat 10 mins.
tcp        0      0 <IP_ADDRESS>:1883     <IP_ADDRESS>:4097     ESTABLISHED 17310/mosquitto  off (0.00/0/0)

@Pfannex
The following is log when I kill mosquitto in server. Infinite rc=-2(no delay).
0
0
0
0
0
failed, rc=-3
MQTT disconnected....
failed, rc=-2
MQTT disconnected....
failed, rc=-2
MQTT disconnected....
failed, rc=-2
MQTT disconnected....
failed, rc=-2
Ping your esp8266 when disconnected to check network is ok.
Check mosquitto on server.

@Pfannex
Your log start with rc=-3,
-3 : MQTT_CONNECTION_LOST - the network connection was broken
-2 : MQTT_CONNECT_FAILED - the network connection failed
-1 : MQTT_DISCONNECTED - the client is disconnected cleanly
0 : MQTT_CONNECTED - the cient is connected
https://cloud.githubusercontent.com/assets/7262813/11170690/f5f254ba-8bdb-11e5-9d6b-b8dd44ec5a8e.PNG

Ping your esp8266 when disconnected to check network(Wifi) is ok.

I have to check this out.
But I saw the in the disconnect state the ESP did not send any TCP-frames for a reconnect!
So I think, that there is no TCP-binding active.
Meanwhile I don't think, that the PubSubClient are the problem.
I think that the Ethernet-connection is the problem.
Is it possible to compile a Sketch and Flash it without Arduino?
So we can find a problem into my Arduino installation.

do ping test using this https://gist.github.com/chaeplin/72f7817bcf7f10bdbdff
(no pubsub, just wifi)
and how about using Staging version ?

I have disabled
boolean reconnect() { too.
I also run a simple WiFi sketch, I got about 3 disconnects a day.
But all disconnects were detected, a reconnect will start immediately.
What do you mean with "Staging version"?
Right now your test is running.....
Without any reconnects since 30 minutes.
Stay tuned.... :-)
btw. special thanks for your help so far

@Pfannex esp8266/Arduino has two version, stable and staging.

Right now your test is running.....
Without any reconnects since 60 minutes.
I canceled your test and installed "staging 2.0.0-rc1".
I run the previous Sketch.......
stay tuned.....

1h.......that's a long time!
I will report the overnight result tomorrow morning.

No reconnect or hanging since 8h....  pray....

@Pfannex
My esps had random hang(reset) a month ago. So I dosabled "Serial.begin", no more hang.
#define DEBUG_PRINT 0
void setup() {
if (DEBUG_PRINT) {
Serial.begin(115200);
}
}

20 hours!
I think, you solved the problem!
Great hint, thank you for your help!
I also disabled the RS232....... outside debugging
We will see if its stable, so stay tuned.....

@Pfannex tuned

The MQTT-Client works fine!
No Problems so far!
Thanks...!
But now I got a Problem with the ESP-Board.
Did you heard something like that?
https://github.com/esp8266/Arduino/issues/1078

i resolved this problem by adding a delay(10); in the library's loop function as here below.
boolean PubSubClient::loop() {     if (connected()) {     	delay(10);

The delay(10); worked flawlessly for me too, 4 years later. Could it be added to the code?

Hello everyone. I have this code, but I don´t know why the pubsubclient state is returning -1. I'm using a ESP32 devkit v1. I'm running Mosquitto on a Raspberry Pi for MQTT server with user and password. I'm sure the error is not in the server, because with a JavaScript client works perfectly. This is my code:
const char* ssid = "my_ssid";
const char* password = "my_network_password";
IPAddress mqttServer(192, 168, 1, 36); // The IP server
const int mqttPort = 9001;
const char* mqttUser = "user";
const char* mqttPassword = "pass";
void callback(char* topic, byte* payload, unsigned int length) {
// My callback function
}
WiFiClient wifiClient;
PubSubClient client(mqttServer, mqttPort, callback, wifiClient);
void setup() {
Serial.begin(9600);
WiFi.begin(ssid, password);
while (WiFi.status() != WL_CONNECTED) {
Serial.println("Connecting to network...");
delay(500);
}
Serial.println("Connected to network");
Serial.println("IP address: ");
Serial.println(WiFi.localIP());
Serial.println("==================================================");
String clientId = "ESP32";
clientId += String(random(0xffff), HEX);
if (client.connect(clientId.c_str(), mqttUser, mqttPassword)) {
Serial.println("MQTTClient connected");
} else {
Serial.println("MQTTClient can´t connect");
}
}
void loop() {
while (!client.connected()) {
reconnect();
}
client.loop();
// Rest of code
}
void reconnect() {
Serial.println("Connecting to MQTT server...");
String clientId = "ESP32";
clientId += String(random(0xffff), HEX);
client.connect(clientId.c_str(), mqttUser, mqttPassword);
delay(10000);
if (client.connected()) {
Serial.println("MQTTClient connected");
// Rest of code
} else {
Serial.println("Error code: ");
Serial.println(client.state()); // It returns -1
delay(5000);
}
}

@pierotoscano1496 double post. I replied to your issue in issue #203


Notifications/export: publish message on workflow finishing

When the workflow completes message should be sent to the pub/sub service so that other systems, such as the informatics gateway, can use the results

[x] #171

@jackschofield23  can we close this issue as complete?

Closed


garbled codes in some code files

There are garbled codes in some code files, which may affect semantic analysis, such as: s969976977.java

Your assessment of this being garbled code is probably based on the funny comments with all the question marks.
But really, this is a correct Java problem that is part of the accepted answers to problem p02407.
(By the way, instead of using a screen capture and inserting an image here, it is better to use a literal code block.)

Your assessment of this being garbled code is probably based on the funny comments with all the question marks. But really, this is a correct Java program that is part of the accepted answers to problem p02407. (By the way, instead of using a screen capture and inserting an image here, it is better to use a literal code block.)

Thank you！


Add tflite-micro and some other AI support

Summary
Recently, we ported tflite-micro on nuttx-apps with an API demo, it's a C++11 library for AI inference.
Impact
Totally new feature, no one use it yet.
Testing
CONFIG_TFLITEMICRO
CONFIG_TFLITEMICRO_TOOL

macOS build error seems CI problem also occurring on other builds unrelated to this change.

Thank you @jihandong amazing work!! :-)
Would it be possible to add Documentation and example / test applications in a separate PR in a free moment? :-)

I'd like to. But compared to the background knowledge of AI, tflite-micro is very esay to use and there are already many good documents. Share some steps here:

First u need to prepare your own tflite model file.
Register needed operators by tflite::MicroMutableOpResolver<T>::Add*.
Prepare a piece of memory as memory pool(tflite-micro calls it "arena") for intermediate calculations.
Initialize a MicroIntepreter, every instance represents a parsed model.
Finally, ou can get the input and output addresses(from the memory pool) through interpreter::GetInput and interpreter::GetOutput, interpreter::Invoke would do once inference.

Hope these steps help u. :)

Thank you @jihandong amazing work!! :-)
Would it be possible to add Documentation and example / test applications in a separate PR in a free moment? :-)

I'd like to. But compared to the background knowledge of AI, tflite-micro is very esay to use and there are already many good documents. Share some steps here:
1. First u need to prepare your own tflite model file.

2. Register needed operators by `tflite::MicroMutableOpResolver<T>::Add*`.

3. Prepare a piece of memory as memory pool(tflite-micro calls it "arena") for intermediate calculations.

4. Initialize a `MicroIntepreter`, every instance represents a parsed model.

5. Finally, ou can get the input and output addresses(from the memory pool) through `interpreter::GetInput` and `interpreter::GetOutput`, `interpreter::Invoke` would do once inference.

Hope these steps help u. :)

Its not really for me, but for any user or newcomer to NuttX that would be interested in this subject :-)
We have project documentation available at:
https://nuttx.apache.org/docs/latest/
Source code for this Documentation is here:
https://github.com/apache/nuttx/tree/master/Documentation
We usually provide starting point "examples" that presents some basic functionalities and acts as project templates:
https://github.com/apache/nuttx-apps/tree/master/examples
Its also good way to announce your work results to the world :-)

let us remove VELAPLATFORM and change-id from commit message:


crash when switching out of transforms module

The application crashes when switching out of the transforms module:

start application
unhide slicer interface
switch to transforms module
switch to any other module

It appears to be a segfault as the parent widget of what I assume is a qSlicerTransformsModuleWidget is set to null.

The crash does not occur when we turn off custom QSS

To try to identify what may be leading the moduleWidget to be invalid, you could also look into having a breakpoint (or qDebug() call) in the destructor of the widget ...
Then when styling is enabled, you could see what lead to destruction of the object in the first place

Actually if you look at the picture on Slicer forum then you migh notice that the app crashes during QWidget::setParent(). More precisely at QWidgetPrivate::inheritStyle and QWidgetPrivate::setStyle_helper within QWidget::setParent() process.
I believe that QWidget tries to inherit the style from parent to all children or something like that. That is why these two functions are called recursively. Finally at some point QWidgetPrivate::setStyle_helper crashes.
So it is not about qSlicerTransformsModuleWidget is NULL but it is something with style inheritage when we set new parent.
The main questions that I don't have answers:

why it crashes only on Linux
why it crashes only with qSlicerTransformsModuleWidget

If open qSlicerTransformsModuleWidget.ui in SlicerDesigner then few warnings will appear:
The enumeration-type property contentsFrameShape could not be read.
The set-type property decimalsOption could not be read.
The enumeration-type property TypeOfTransform could not be read.

I googled that and there already been a similar problem with SlicerSALT. And it also was connected with Linux only. @jcfr don't you remember how you solved this error?
I don't know exactly but I guess some CTK widgets have styles incorrectly managed.

The source of the problem is GlyphPointsNodeComboBox.
To check that go to qMRMLTransformNodeDisplayWidget and after the UI is setup set the its parent to nullptr:
void qMRMLTransformDisplayNodeWidgetPrivate
::init()
{
  Q_Q(qMRMLTransformDisplayNodeWidget);
  this->setupUi(q);

  this->GlyphPointsNodeComboBox->setParent(nullptr);
  /* some other code */
}

Then even you are on Linux and you have StyleSheet aplied then switching to another module from Transforms won't lead to crash.
Though I don't yet understand what is wrong with that widget there...

Finally I've found what exactly causes the crash.
void qMRMLNodeComboBox::changeEvent(QEvent *event) calls qMRMLNodeComboBoxPrivate::updateDelegate and here new item delegate is created.
The strangest thing is that if you put these three qDebug() lines before that line then everything works as expected:
void qMRMLNodeComboBoxPrivate::updateDelegate(bool force)
{
/* some code */
      if (force ||
          qobject_cast<qMRMLNodeComboBoxDelegate *>(this->ComboBox->itemDelegate()))
        {
        qDebug() << "ONE";
        qDebug() << "TWO";
        qDebug() << "THREE";
        this->ComboBox->setItemDelegate(
            new qMRMLNodeComboBoxMenuDelegate(q->parent(), q->comboBox()));
        }
/* some code */
}

I hope somebody will find a clue why this is happening and how to solve this.

@kerim371 Thanks so much for you investigation and for reporting your observations! Sorry for the late reply I have been traveling. Thanks for all your effort to narrow down the problem.

The strangest thing is that if you put these three qDebug() lines before that line then everything works as expected:

Very weird. I wonder if it is related to compiler optimization; perhaps turning off optimization and removing the debug lines would also prevent the crash.

Share your results please when you're done testing.
It would be nice if we could solve this.


Skip looking up new build for x amount of time

Besides having faster server startup, for those of us with slow internet just checking for a new build can take an extra second or two (or timeout.) It'd be great if it only checked for a new Paper build every 24 hours, with possibly an option for force checking.

Implemented in ccc8f55/1.0.2-SNAPSHOT on my repo.
As documented here: https://docs.gradle.org/current/userguide/build_environment.html, properties are resolved such that a value set in GRADLE_USER_HOME/gradle.properties will take precedence over a value set in the project gradle.properties, which allows for those with slow internet to set an override for their system.
See here for the format for this property until I document it in the wiki: https://github.com/jpenilla/run-paper/blob/master/src/main/kotlin/xyz/jpenilla/runpaper/util/DurationParser.kt#L33-L42

Looks great, thanks!

1.0.2 has been released.


CORS headers not set when deploying to Fly.io

Describe the Bug
The expected header Access-Control-Allow-Origin is not set when deploying to Fly.io using the guides. As a result, we can‘t use it with Partytown, which requires the header to be set.
Access to fetch at 'https://analytics.<DOMAIN>/script.js' from origin 'http://localhost:4321' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

accept-ranges: bytes
cache-control: public, max-age=0
content-encoding: gzip
content-security-policy: default-src 'self';img-src * data:;script-src 'self' 'unsafe-eval' 'unsafe-inline';style-src 'self' 'unsafe-inline';connect-src 'self' api.umami.is cloud.umami.is;frame-ancestors 'self' undefined
content-type: application/javascript; charset=UTF-8
date: Fri, 13 Sep 2024 00:52:28 GMT
etag: W/"a11-19196e5b838"
fly-request-id: <REDACTED>
last-modified: Wed, 28 Aug 2024 02:52:03 GMT
server: Fly/0049c9a9 (2024-09-10)
vary: Accept-Encoding
via: 2 fly.io
x-dns-prefetch-control: on

Database
PostgreSQL
Relevant log output
No response
Which Umami version are you using? (if relevant)
v2.13.2
Which browser are you using? (if relevant)
Brave
How are you deploying your application? (if relevant)
Fly.io

disable browser plugin: uBlock Origin

@yeskunall, this is not related to Fly.io. The problem is, that the static asset/tracker script script.js is served by the Next.js server without an Access-Control-Allow-Origin header.
This itself is fine, as the header is not needed when a browser is instructed to load the script via a script tag. But, the header is required when the script is programmatically fetched via XmlHttpRequest or fetch.
As Partytown uses fetch inside the web worker to get the tracking script, the header Access-Control-Allow-Origin header is necessary in order to make Umami work with Partytown.
The header can easily be added for the tracker script. I tried it locally. Umami then correctly works with Partytown.
I will create a PR to suggest the change. 🙂

@peter-kuhmann you’re absolutely correct. It has nothing to do with Fly.io. I updated the issue title and comment to reflect that.
Closing as this is now fixed.


Map file management

Here's my proposal for how we should manage the map files. We would bundle the CARLA maps with the torchdrivesim package itself, and provide a mechanism to include additional paths for maps files using the TDS_RESOURCE_PATH environment variable. All the files are stored in a safe and human-readable format, mostly JSON and OSM. There's also a function to download and save all the map information through the IAI API. Traffic light control will be added in #42.
I included the right-handed Town03 map as an example, but ultimately I'm planning to replace it with all the left-handed CARLA maps before merging. You can run python -m examples.simulate to see that everything runs out of the box, and it will produce simulation.gif in the main folder.
Let me know what you think and if everyone's happy I can finish this, include the map files we want, and add a couple of tests.

I think this is ready to merge, but let me know if you spotted anything bad in my recent commits.

@adscib
Hi Adam, I just noticed the carla maps here only contain Town01~Town04 and Town10HD, could we include Town07 as well? It's used in the torchdriveenv related data, including training and validation scenario cases.
The rl-env labeling tool supports "Town01", "Town02",  "Town03", "Town04",  "Town06", "Town07", "Town10HD". For torchdriveenv, Town04 and `Town06 are excluded, because they are too large and will cause issues in the API server side.


Notification badges

Closes #594
This PR adds support for notification badges on the app's icon on the home screen.
I originally was going to mimic the behavior of the notification bell in the UI itself (no count, just an empty badge). However, as I worked on these changes, I changed approaches.
Since badges correspond to push notifications, I thought it made sense to show the badge based on receiving push notifications, instead of just doing some kind of polling on the notes API. This also gives the implicit benefit of only showing badges to users who have enabled push notifications, which is ideal.
The changes work as follows:

Track the number of push notifications received. Increment the count by 1 for every notification received that we show. e.g. notifications that get merged with current notifications don't increment the count. Skipped notifications don't increment the count, etc.
Every time the count of notifications goes up, update the app's badge w/this new number
Every time a notification is clicked, decrement the count and update the app's badge accordingly
If the count is updated to 0, clear the badge altogether instead of displaying 0
When a user loads the notifications page in the UI itself, clear all open notifications and reset the internal count to 0
When the notification bell header polls and indicates no new notes, clear all open notifications and reset the internal count to 0

So there's some slight tweaks to the existing behavior here, plus the new badge behavior as well.
As usual, I am open to feedback!

Mhh, looks like I can't test this since setAppBadge is only supported on Safari

Ah, sorry, Chrome on Windows and macOS or Safari! I didn't realize it wasn't on Linux.


JSON format for Chakra ET

Summary

Added JSON format for Chakra ET
To maintain backward compatibility, a wrapper is included that presents a common API for Protobuf and JSON

Test Plan

Added wrapperTests.cpp and test data

recheck

recheck

recheck

Hi @rvinaybharadwaj,
Thank you for the PR! I have a few requests:
Could you ensure that the code passes all checks (file formatting, tests, etc.)?
Could you provide step-by-step commands to help us reproduce the results?
What is the expected outcome, and how can we verify it?
Thanks again!

Hi @rvinaybharadwaj,
Thank you for the PR! I have a few requests:

Could you ensure that the code passes all checks (file formatting, tests, etc.)?
Could you provide step-by-step commands to help us reproduce the results?
What is the expected outcome, and how can we verify it?  (Could you share us example input and output?)

Thanks again!

Hi @JoongunPark,

I have run clang-format on the code and the tests are breaking on not finding a module during the setup phase. So, I don't think it's an issue related to my code. And I am not sure why the CLA check is failing even though I am a member of MLCommons.

I have added wrapper tests, which are functionally equivalent to the feeder tests and have included test data. You can run it through the existing google test framework.

The data in chakra.0.json is the same as the data in chakra.0.et You should be able to achieve the same result as the protobuf ET and the wrapper tests should all pass.

Let me know if you have any more questions.

Hi @rvinaybharadwaj,
Thank you for the update!

It looks like this branch has new conflicts after the recent merge. Could you please resolve them?
For the cpp_linter, you might want to update the GitHub workflow to fix it. Here’s the link: cpp_lint.yml.
Regarding the CLA issue, I believe someone from MLCommons needs to add you. @tushar-krishna or @srinivas212 should be able to help with this.

I'll run some tests and share the results.

Hi @rvinaybharadwaj,
Thank you for the update!

It looks like this branch has new conflicts after the recent merge. Could you please resolve them?
For the cpp_linter, you might want to update the GitHub workflow to fix it. Here’s the link: cpp_lint.yml.
Regarding the CLA issue, I believe someone from MLCommons needs to add you. @tushar-krishna or @srinivas212 should be able to help with this.

I'll run some tests and share the results.

Hi @JoongunPark ,
Thanks for the feedback.

Merge conflicts have been fixed.
I checked cpp_lint.yml in .github/workflows and it is exactly the same as in the main branch.

Hi @rvinaybharadwaj,
Thank you for updating your PR!
I have two last requests:

Could you please ensure that the "CLA" test passes? It's mandatory, and unfortunately, I can't assist with this part.
It seems you've successfully addressed the original issue with the C++ Link (the module installation problem). However, there are still a few minor errors. You can check them here: link. Could you please take a look and fix those as well?

Thank you for your efforts!

Hi @rvinaybharadwaj,
Thank you for updating your PR! I have two last requests:

Could you please ensure that the "CLA" test passes? It's mandatory, and unfortunately, I can't assist with this part.
It seems you've successfully addressed the original issue with the C++ Link (the module installation problem). However, there are still a few minor errors. You can check them here: link. Could you please take a look and fix those as well?

Thank you for your efforts!

Hi @JoongunPark,
All errors have been fixed.

@rvinaybharadwaj
Thank you for resolving all the issues! I've started reviewing your code in detail, and here’s my understanding so far:
1. New Classes

JSONNode: Handles execution trace (ET) parsing from JSON files.
WrapperNode: Abstracts ET handling for multiple formats (e.g., .et and .json), working within the ETFeeder component.

2. Dependency Changes

pyproject.toml: Updated to support protobuf==5.*. This resolves some installation conflicts with the original version, ensuring compatibility.

3. Test Coverage

Unit Tests (tests/feeder/wrapperTests.cpp): Verifies the JSON parsing functionality.
Test Data (tests/data): Includes new JSON-format test files.

4. Linting Updates

clang-format-lint: Clang format (11->16). Linter version (v0.11 -> v0.18.1)

5. New Third-Party Library

json.hpp (src/third_party/utils/json.hpp): Utilized by JSONNode for JSON parsing.

Also, I have a question regarding JSON format trace. This PR does not include trace conversion from ET to JSON.
Is this PR assuming to get JSON trace with exsting JSONizer?
Thank you again!

@rvinaybharadwaj Thank you for resolving all the issues! I've started reviewing your code in detail, and here’s my understanding so far:
1. New Classes

JSONNode: Handles execution trace (ET) parsing from JSON files.
WrapperNode: Abstracts ET handling for multiple formats (e.g., .et and .json), working within the ETFeeder component.

2. Dependency Changes

pyproject.toml: Updated to support protobuf==5.*. This might resolve some installation conflicts with the original version.

3. Test Coverage

Unit Tests (tests/feeder/wrapperTests.cpp): Verifies the JSON parsing functionality.
Test Data (tests/data): Includes new JSON-format test files.

4. Linting Updates

clang-format-lint: Clang format (11->16). Linter version (v0.11 -> v0.18.1)

5. New Third-Party Library

json.hpp (src/third_party/utils/json.hpp): Utilized by JSONNode for JSON parsing.

Also, I have a question regarding JSON format trace. This PR does not include trace conversion from ET to JSON. Is this PR assuming to get JSON trace with exsting Jsonizer?
Thank you again!

@JoongunPark
Thank you for reviewing the code and summarizing. Your understanding of different parts of the code is correct. The existing JSONizer outputs a slightly different format, but I was assuming it easy to do a JSON dump directly from Pytorch or any other tool.

@rvinaybharadwaj Thank you for resolving all the issues! I've started reviewing your code in detail, and here’s my understanding so far:
1. New Classes

JSONNode: Handles execution trace (ET) parsing from JSON files.
WrapperNode: Abstracts ET handling for multiple formats (e.g., .et and .json), working within the ETFeeder component.

2. Dependency Changes

pyproject.toml: Updated to support protobuf==5.*. This might resolve some installation conflicts with the original version.

3. Test Coverage

Unit Tests (tests/feeder/wrapperTests.cpp): Verifies the JSON parsing functionality.
Test Data (tests/data): Includes new JSON-format test files.

4. Linting Updates

clang-format-lint: Clang format (11->16). Linter version (v0.11 -> v0.18.1)

5. New Third-Party Library

json.hpp (src/third_party/utils/json.hpp): Utilized by JSONNode for JSON parsing.

Also, I have a question regarding JSON format trace. This PR does not include trace conversion from ET to JSON. Is this PR assuming to get JSON trace with exsting Jsonizer?
Thank you again!

@JoongunPark Thank you for reviewing the code and summarizing. Your understanding of different parts of the code is correct. The existing JSONizer outputs a slightly different format, but I was assuming it is easy to do a JSON dump directly from Pytorch or any other tool.

Regarding the JSON format trace, I believe we should discuss whether to align it with the existing JSONizer or consider using a different format. This decision might be crucial, as I recall your initial plan was to create a merged, unified JSON trace. We can also involve @TaekyungHeo for further discussion on this matter.

Based on the discussion in the Chakra meeting on Dec 12, 2024 -

I check for the file format in wrapper_node.cpp #L26 . Since, I use filename.find_last_of("."), renaming the JSON file to filename.et.json should work without any changes.
In the future, if the protobuf ET files are renamed to filename.et.pb, we need to make the corresponding change in wrapper_node.cpp #L27


flaky-results: fix crash_replica_processes

https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1362/testReport/junit/tests/test_backing_image/test_ha_salvage_with_backing_image/
https://github.com/longhorn/longhorn-manager/pull/1338#issuecomment-1138380264

Test result - https://ci.longhorn.io/job/private/job/longhorn-tests-regression/1372/


Chose whether we want focus or scrollIntoView when changing expandedIds

When using an external search bar, the focus gets eaten after changing the expandedIds to show the found items, see gif

@yhy-1 @mellis481 I leave it to you to determine the priority of this. I’d like to get an issue template added to this repo since it’s burdensome to get bug reports that don’t contain the code or the steps to reproduce. @VinceBT-BG please provide the the code (like the props you sent in) AND the steps to reproduce.

Here you have a simple example that works, just press the letter a inside the input
import React from 'react'
import TreeView, { INode, flattenTree } from 'react-accessible-treeview'

function flatten<T>(arr: T[][]): T[] {
	return ([] as T[]).concat(...arr)
}

type NestedArray<T> = Array<NestedArray<T> | T>

function flattenDeep<T>(input: NestedArray<T>): T[] {
	return flatten(input.map(x => (Array.isArray(x) ? flattenDeep(x) : [x])))
}

const findParents = (data: INode[], node: INode): number[] => {
	if (!node?.parent) {
		return []
	}
	const parentNode = data?.filter(dataNode => dataNode.id === node?.parent)
	return [node.parent, ...findParents(data, parentNode[0])]
}

const folder = {
	name: '',
	children: [
		{
			name: 'Fruits',
			children: [{ name: 'Avocados' }, { name: 'Bananas' }, { name: 'Berries' }, { name: 'Oranges' }, { name: 'Pears' }],
		},
		{
			name: 'Drinks',
			children: [
				{ name: 'Apple Juice' },
				{ name: 'Chocolate' },
				{ name: 'Coffee' },
				{
					name: 'Tea',
					children: [{ name: 'Black Tea' }, { name: 'Green Tea' }, { name: 'Red Tea' }, { name: 'Matcha' }],
				},
			],
		},
		{
			name: 'Vegetables',
			children: [{ name: 'Beets' }, { name: 'Carrots' }, { name: 'Celery' }, { name: 'Lettuce' }, { name: 'Onions' }],
		},
	],
}

const data = flattenTree(folder)

export const App = () => {
	const [expandedIds, setExpandedIds] = React.useState<number[]>([])
	const [searchValue, setSearchValue] = React.useState<string>('')

	const [hiddenIds, setHiddenIds] = React.useState<number[]>([])

	const allIds = React.useMemo(() => data.map(element => element.id), [])

	React.useEffect(() => {
		if (!searchValue) {
			setHiddenIds([])
			return
		}
		// Filter elements containing part of search value
		const searchResult = data.filter(element => element.name.toLowerCase().includes(searchValue.toLowerCase()))
		// For each result, find all parents and flatten array
		console.log(searchResult.map(result => findParents(data, result)))
		const parents = flattenDeep(searchResult.map(result => findParents(data, result)))
		// Expand parents without changing the already expanded elements
		const nextExpandedIds = [...new Set([...expandedIds, ...parents]).values()]
		if (JSON.stringify(expandedIds) !== JSON.stringify(nextExpandedIds)) {
			setExpandedIds(nextExpandedIds)
		}
		const allChildren = flatten(searchResult.map(node => [...node.children]))
		const idsToShow = [...parents, ...searchResult.map(el => el.id), ...allChildren]
		setHiddenIds(allIds.filter(id => !idsToShow.includes(id)))
	}, [allIds, expandedIds, searchValue])

	return (
		<>
			<input placeholder="Enter a for example" onChange={event => setSearchValue(event?.target.value)} style={{ border: '1px solid red' }}></input>
			<TreeView
				data={data}
				expandedIds={expandedIds}
				nodeRenderer={({ element, isBranch, getNodeProps, level, handleExpand }) => {
					return (

							{isBranch && '>'}
							{element.name}

					)
				}}
			/>
		</>
	)
}

We need better reproduction steps.
typically people send us “reproduction steps,” “expected results,” and “actual results.”

I have updated my message, is this ok for you ?

Thank you for taking the time to document this. Now that I can see what you were looking to do, I can tell you that your proposed enhancement is (a) outside of the scope of work that my team can provide and (b) is problematic enhancement since it would change the way that the screen reader shortcuts are meant to work.
Instead, please direct your users to use the default search functionality that already exists inside the component.

Type a character* | Focus moves to the next node with a name that starts with the typed character. If multiple characters are typed in rapid succession, focus moves to the next node with a name that starts with the string of characters typed.

This is taken from https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/tree_role#keyboard_interactions which is what the widget was programmed to follow.

Thank you for clarifying. I’ll reopen this so that you or someone else can PR the library. To be clear, we won’t support a search solution outside of component. So if you want to PR the library to search within the children/folders then we would appreciate the contribution. Our team however does not have capacity at this time.


dgram multicast not working in nodejs v11

Consider this very simple test.js
require('dgram').createSocket('udp4').bind(60547);
setTimeout(function() { echo "exiting after 1 minute" }, 60000);

With node v10 we can spawn as many instance of test.js as we want.
With node v11.13 and later spawning the second instance results in the following error:
Error: bind EADDRINUSE
    at exports._errnoException (util.js:742:11)
    at dgram.js:222:28
    at dns.js:82:18
    at process._tickCallback (node.js:378:11)

To fix it, we need to specify an option as following:
require('dgram').createSocket({'type': 'udp4', 'reuseAddr': true}).bind(60547);

Fixed in 0.8.1


catch quota errors

Closes https://github.com/CartoDB/cartodb/issues/14921

@gonzaloriestra would you mind pasting some screenshots to see how these errors look like 🙏?

:green_heart: change privacy from within BUILDER
:green_heart: change privacy from modal dialog within BUILDER
:green_heart: change privacy from dashboard
:green_heart: duplicate a dataset from dashboard
:red_circle: duplicate a public map (generic modal error with broken button + 403 in console)
:green_heart: create empty dataset
:green_heart: create empty map when there are no table quota
:green_heart: add layer from new layer in BUILDER
:green_heart: drag and drop .carto (without enough dataset or public map quota)
:green_heart: drag and drop table
:green_heart: from dataset view -> duplicate
:green_heart: from dataset view -> create from query
:green_heart: sync from data library
:green_heart: create regular api key
Public maps quota error:

Table quota error:

Table quota error (only when creating from SQL):

API keys error:

And the wrong one (duplicating a public map) shows:

Thanks @gonzaloriestra for the screenshots. Are these copies coming from the Backend? I'm not sure users will understand them (specially the "(duplicating a public map)" one). How hard is it to change them? Thanks!

@alonsogarciapablo Alberto is fixing the one about "duplicating a public map" to be more specific.
Regarding the others, it's easy to change the texts. What do you prefer to show?

I will suggest something here. Is it easy to include the actual limits in the error messages (eg: the actual number of datasets)?

Let's not open the 🍈 of including the limits in the message. I will make a proposal to change the copies and we should be good to go!

👍
I fixed the duplicate a public map issue, let's update those copies and 🚀

:heavy_check_mark: Acceptance OK
:green_heart: duplicate a public map

Working on those copies now! Could you guys please share a screenshot of the new "duplicate a public map"? Thank you!

@alonsogarciapablo it's the same as the other public map quota errors: https://user-images.githubusercontent.com/14979109/58874101-2d1d8000-86c8-11e9-9a0b-6e16d39fb416.png

Public maps quota error / Duplicating a public map

New copy proposal:

Your map's privacy couldn't be changed
You have reached the limit of public maps for your plan
Make some of your maps private or upgrade your account to get unlimited public maps

Table quota error:

New copy proposal:

Your dataset couldn't be created
You have reached the limit of datasets for your plan
Remove some of your datasets or upgrade your account to get unlimited datasets

Table quota error (only when creating from SQL):

New copy proposal:

Your dataset couldn't be created
You have reached the limit of datasets for your plan. Upgrade your account to get unlimited datasets

API keys error:

New copy proposal:

You have reached the limit of API keys for your plan


[FEAT] Intents to list entities

I think it would be useful to have an intent like "list my lights" or "list my areas" or even "list lights in " to know how entities are identified. I'm probably not the only one who can't remember if its "kitchen main lights" or "kitchen room lights" or just "kitchen lights" in Home Assistant.

I think for those of us fortunate enough with a screen of the Mark2 it would be great to have it listed there.
But currently I have it on the home assistant android app.

I'll add something for this, with "voice pagination" for long entity lists (I have hundreds, so wouldn't use it myself, but it's a good feature for short lists).
@amunizp You're in luck! Say open Home Assistant dashboard and there's a beautiful GUI that lists everything. :)


Possible to reduce GitHub permissions?

I think Adoptoposs is a super idea and the website looks great!
Would it be possible to reduce the necessary GitHub permissions, though? Here's what I'm seeing:

Read access to public repositories is obviously fine. Write access is a little scary, and I wasn't sure why it was needed. The same goes to write access to personal user data. I'm also slightly concerned about read-access to private project boards, but I haven't dug into see if that exposes any proprietary information from corporate GitHub orgs that I'm in.

Hey, @fhunleth. Thanks for reporting, I think you are absolutely right.
I just had another look into the GitHub API scopes and as far as I could see in a quick test, following scopes should be sufficient:
read:user user:email read:org
(currently we have user public_repo read:org – https://github.com/adoptoposs/adoptoposs/blob/staging/config/config.exs#L42)
I’ll double check later today if all features still work as expected, and change it to the more restricted ones.

@fhunleth :rocket: more restricted permission are deployed. You should be able to log in under read-only permission after revoking access for the Adoptoposs OAuth app in your GitHub Applications settings.

I tried again. I actually didn't have a login so I didn't have to revoke access. The new permissions look much better. However, "Adoptoposs says sorry!" I hope that you received an error report, since I don't think that I have anything additional to add. I just told GitHub to grant access and I got redirected to the 500 error screen.

I see some database connection errors for about that time of your comment. From what I can see it looks like these are not related to the last deployment and it was an unfortunate coincidence to appear shortly after. Anyway, thanks for letting me know. :+1:


Weird syntax coloring on vue files

I noticed that there is a weird syntax coloring behavior as I add new lines or move code, see in the bellow gif:

The issue gets fixed once I open another tab and return back, or after I save the file and it gets reformated.

Are you using 0.30.1? If yes please provide the vscode info by Help: About command.

Yes, I am using the latest version of both Volar and VS Code
Version: 1.63.2 (user setup)
Commit: 899d46d82c4c95423fb7e10e68eba52050e30ba3
Date: 2021-12-15T09:40:02.816Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19043

Can you reproduce it in https://github.com/johnsoncodehk/volar-starter?

I couldn't replicate this anymore in any of my projects, I assume it was a temporary issue with my system.


fix(IAM Identity): update service after recent API changes

PR summary
This PR contains an update to the IAM Identity service - including integration tests and examples - after some recent changes to the API definition.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

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


Add YatharthNAgmote's 1 file

I have created a branch, created  file and made a commit and opened a pull request.

Good


Add yoga layout support

I'd like to add yoga layout, because Grid is pretty buggy and it would make it more similar to React Native. There are a few things that need to be done to make this work.

[ ] Add fixed position support to libui (https://github.com/andlabs/libui/issues/306)
[x] Add yoga-layout on the JS side
[x] Create a wrapper called View over the libui Fixed widget

I'm almost done with a version that works on Linux, I just need a few more days to work out the bugs. Once the proton-native side is ready, then there shouldn't be any more changes needed since everything else should be implemented in libui.
For libui, the notable changes that I've made (on my fork) is add a fixed component using GtkFixed and add a way to get and set the size of any component.
I need to find a way to implement that in Windows and Mac, but I assume they have easy support for fixed positioning and setting/getting the size.
As for libui-node, I've added all of those bindings. Yoga isn't being built together with libui-node, but added with proton-native. I've had to change the compile options for libui-node because it and yoga were conflicting.
The component is implemented as View and supports all of the options that View has on React Native. I need to see if I can implement setting the background color so that it's easier to see nested views.

. I've had to change the compile options for libui-node because it and yoga were conflicting.

Can you details where they conflict? And which options you need to. Change?

I couldn't find exactly what conflicted, but I found this in nbind's issues and it worked. https://github.com/kusti8/libui-node/commit/57a957c4f23d89da53cfb141d96964eefc14bbf5

I've added a PR to track my changes. All I need to do is add fixed positioning for Windows and Mac for libui and merge that.

@kusti8 Is it still in development? I also have a question about running Yoga on JS side, it's interesting to see how it can impact perf 🤔

@roman01la This PR needs to be merged first: https://github.com/andlabs/libui/pull/332.


Roll engine 107fe823431a..3f4f606105a8 (1 commits)

https://github.com/flutter/engine/compare/107fe823431a...3f4f606105a8
git log 107fe823431a64302b63179ad5147e592d33a17f..3f4f606105a8f6217cfbe08e193aa9a54b19dbef --no-merges --oneline
3f4f60610 Simplify loading of app bundles on Android (flutter/engine#9360)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (chinmaygarde@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.