Permutations without repitation

How to get permutations without repitations?
array(1,2,3,4,5,6)
all perms for 3: example: 1,2,3 1,2,4 etc

Hey hi!
Good question! I think this is not implemented.
You're looking for all the combinations of a given size right?

Have you tried this?
<?php

declare(strict_types=1);

use drupol\phpermutations\Iterators\Combinations;

include 'vendor/autoload.php';

$input = [1,2,3,4,5,6];

var_dump(iterator_to_array(new Combinations($input, 3)));

Yes bro but I need without repitations - and I need it with good performence because my input have 49 numbers and param is = 6 ;)

So for example I need results with combinations without repitations so 1,2,3 but 3,2,1 (this one is worng) - its called in math permutations without repitations

Hi "bro",
I gave you the solution up there, feel free to re-open the issue if it doesn't fit your needs.

Hi
thank you so much for help, I was using this package long time ago not right way :)
Regards

You're welcome!


DOBS: Too many requests

This continues #1262 .   We're starting to hit the digital ocean rate limits during our provisioning.  Digital Ocean limits requests to 5,000 per hour and provides feedback on how many you have left.
Would it be possible to "scale back" any non-essential requests when the threshold is near?
Example:
if RateLimit-Remaining <  total rexray volumes * 25
  skip status checks, etc.

Having the same issue. Especially after running into #1321 as well.


How do I add a new human language to be supported by TagUI?

user email

eg adding Bulgarian language

For adding and using new languages, check out this section - https://github.com/kelaberetiv/TagUI#native-languages
The bg code can be added to tagui/src/languages/build.csv to let TagUI self-generate the Bulgarian language definitions before making modifications manually. Creating the bulgarian.csv manually is fine too, to look something like this structure - https://github.com/kelaberetiv/TagUI/blob/master/src/languages/serbian.csv
// more details on ISO-639-1 Code and Google Translate here - https://cloud.google.com/translate/docs/languages


Already existing device DAC provisioning (CON-826)

Describe the question/query that you have
I heard from 1 of my customers that it is possible to commission DACs from some kind of espressif server. Where can I find more information on how it works?
Additional context
...

It is not possible to update/install DACs on existing/in-field devices through this service yet.

We are working on it. I will post an update here, once it is available.


GlobalFilter does not respect disableFilters

Describe the bug
I think this is a bug, but it could be a feature request. When using useGlobalFilter, if you set disableFilters: true on a column passed into the useTable properties, it doesn't actually disable filtering on that column.
To Reproduce
Steps to reproduce the behavior:

Create a table with the useGlobalFilter.
On one+ of the columns in the collection, add disableFilters: true as a property of that column.
Attempt to search for a field known to be in the column with disableFilters: true and you'll see that it is filtered by that text.

Expected behavior
My expectation would be that disableFilters would apply to both regular filters and the global filter.
Codesandbox!
https://codesandbox.io/s/cranky-swanson-p9h0c
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOS 10.15.3 (Catalina)
Browser: Chrome (and Firefox)
Version: 80 (and 71)

Created https://github.com/tannerlinsley/react-table/pull/1963 to add this prop...

#1963 was merged, this is now fixed.


[BugFix] Fix list iceberg partition with glue

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

[FE PR Coverage Check]
:disappointed: fail : 0 / 2 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/connector/iceberg/IcebergMetadata.java
0
2
00.00%
[134, 135]

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

@Mergifyio backport branch-2.5


[REEF-247]: Port ranges should to be configurable for listening

Defined a port range provider and a default implementation.
JIRA: REEF-247
This closes #

I did a pass. Mostly cosmetic stuff.

Still some more beauty issues.


Could not install package 'React.AspNet 1.5.2

In final vs2015 (not the RC version) MVC webapplication project using NETFramework,v4.6, when I install the React.AspNet package get the following errors::
Package 'React.AspNet 1.5.2' does not exist in project 'ReactJSWeb' Package 'React.AspNet 1.5.2' does not exist in folder 'E:\Projects\vs2015\ReactJSWeb\packages' Install-Package : Could not install package 'React.AspNet 1.5.2'. You are trying to install this package into a project that targets '.NETFramework,Version=v4.6', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author. At line:1 char:1 + Install-Package React.AspNet + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + CategoryInfo : NotSpecified: (:) [Install-Package], Exception + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PackageManagement.PowerShellCmdlets.InstallPackageCommand

I think this maybe due to the fact that ReactJS.NET had a dependency on ASP.NET Beta 4, whereas you're using Beta 5. I just pushed ReactJS.NET 1.5.3 which switches the dependencies to Beta 5. Try upgrading to 1.5.3 and let me know if that works for you. Thanks!

Closing this out since I haven't heard back, but let me know if you're still encountering issues.

Sorry for delayed reply, I was busy in my project. Thanks a lot for your great work and your help!


Problem!

I have certainly did something wrong - maybe deleting the system administrator Abraham Root or trying to change language to french. Now, I can't access the timetable managers screen nor courses, students… administration menus. What can I do to correct this situation or go back to previous status, please?

Looks like you have deleted the record for the admin user (Abraham Root) in the Administration > Academic Sessions > Managers page. Also, by default, French localization is not included in the official build of UniTime, though big chunk of UniTime has been translated to French -- see https://help.unitime.org/localization for more details.
For the default admin/admin user to work, there needs to be:

a record in the users table (user name, hashed password using MD5 and external id) - typically to be defined Administration > Other > Users page

mysql> select * from users where external_uid='1';
+----------+--------------------------+--------------+
| username | password                 | external_uid |
+----------+--------------------------+--------------+
| admin    | ISMvKXpXpadDiUoOSoAfww== | 1            |
+----------+--------------------------+--------------+
1 row in set (0.00 sec)

(this authentication can be replaced by other means, e.g., using LDAP or CAS authentication)

a record in the timetable_manager table (Administration > Academic Session > Timetable Managers) with the matching external id

mysql> select * from timetable_manager where external_uid='1';
+----------+--------------+------------+-------------+-----------+------------------+--------------------+------------+
| uniqueid | external_uid | first_name | middle_name | last_name | email_address    | last_modified_time | acad_title |
+----------+--------------+------------+-------------+-----------+------------------+--------------------+------------+
|      470 | 1            | Abraham    | NULL        | Root      | [demo<EMAIL_ADDRESS>| NULL               | NULL       |
+----------+--------------+------------+-------------+-----------+------------------+--------------------+------------+
1 row in set (0.00 sec)

the timetable managers are associated with one or more roles (tmtbl_mgr_to_roles table) and they may be associated with one or more departments (dept_to_tt_mgr table, this is important for roles that are session or department dependent -- e.g., a timetable manager can only make changes to courses of his/her departments; a session admin can only make changes to academic sessions from which he/she has at least one department).
mysql> select * from tmtbl_mgr_to_roles where manager_id = 470;
+------------+----------+----------+------------+----------------+
| manager_id | role_id  | uniqueid | is_primary | receive_emails |
+------------+----------+----------+------------+----------------+
|        470 |        1 |      510 |          1 |              1 |
+------------+----------+----------+------------+----------------+
1 row in set (0.00 sec)

Roles are defined in the roles table with the permissions in rights table. If you need to reset the roles and their permissions back to their defaults, you can copy the content of these two tables from the blank-data.sql. We also have an XML file that can be used to export/import roles and their permissions (using Administration > Academic Sessions > Data Exchange page) -- this is useful for making a backup before making permission changes. The various XML formats are described at https://www.unitime.org/uct_interfaces.php.
Please notes that some roles are automatically associated based on having a record in some other table: there is the Student role if we have a student record, the Instructor role if there is an instructor record, the Advisor role if there is an instructor record, the No Role when the user is authenticated but no other role exists for the term, and the Anonymous for access that does not require authentication. If you want to disable some of these, the better approach is to mark the role as disabled (not enabled) on the Administration > Other > Roles page instead of deleting the role.
Please take a look at the administration manual (Help > Manuals > Administration) as it provides some useful information on how to set up UniTime.


Remove getTable call from updatePartitionStatistics

@rohangarg LGTM
This seems to fix ANALYZE failure on Avro tables with avro.schema.url.
Can you confirm & add a test (in TestAvroSchemaUrl)?

@findepi

This seems to fix ANALYZE failure on Avro tables with avro.schema.url.
Can you confirm & add a test (in TestAvroSchemaUrl)?

I tried the analyze query on a partitioned and unpartitioned avro table. For unpartitioned table, the analzye still fails. For partitioned tables, it has started working (if it wasn't working before). should I just add an analyze command in the partitioned table test?

Is it ready for re-review

merged, thanks!


video/avc matching compontent failed

[Delete any non-applicable sections, but we may ask for more information or close the issue if the template is not followed. Please reference the SmartDeviceLink GitHub Best Practices for further instructions on how to enter an issue.]
Bug Report
[Summary]
VideoStreamManager start encoder failed.
※When I use SDL Android SDK 4.6.0, there is no problem.The error occurs when I use SDL Android SDK 4.9.0.
※This issue does not happen on XiaoMi Phone, but it does happen on HUAWEI phones(android 9 and 10).
Reproduction Steps

[Step 1]
Refer to [Android Guides 10/03/2019 11:49 AM], and write a Navigation app(With Mapbox).
[Step 2]
Connect the APP to a SDL Emulator with USB.
[Step 3]
Click the APP icon on the SDL Emlator

Expected Behavior
[Some expected behavior]
The video streaming should be successful
Observed Behavior
[Some observed behavior]
The following errors were output:
E/libc : Access denied finding property "ro.kirin.product.platform"
E/HwExtendedCodec: mime: video/avc matching compontent failed!
I/OMXClient: IOmx service obtained
I/Acodec: In onAllocateComponent create component, codec name: OMX.hisi.video.encoder.avc
W/HwExtendedUtils:hw configLocalPlayBack err = -22
W/OMXUtils: do not know color format 0x7f000001 =<PHONE_NUMBER>
E/ACodec: [OMX.hisi.video.encoder.avc] failed to set output port definition parameters.
E/ACodec:[OMX.hisi.video.encoder.ac] configureCodec returning error -22
E/ACodec: signalError(omxError 0x80001001, internalWrror -22)
OS & Version Information

Android Version: [API Level 26/28]
SDL Android Version: [4.9.0]
Testing Against: [A SDL Emulator developed by our company]

Test Case, Sample Code, and / or Example App
[Paste a link to a PR, gist, or other code that exemplifies this behavior]

HI @bill-lzb, can you test with the latest release of SDL 4.10.1?

I'm going to close this issue for inactivity. Please test with the 4.11.0 release and report if you still see the issue.


fix(universal): rewrite universal quickstart to make it work and align it with Kubernetes quickstart demo

Our previous quickstart was in a terrible shape recently, basically every step was impossible to finish. I've rewritten this quickstart demo for universal so it can be run on a single machine. And aligned it with the Kubernetes demo for consistency
Fixes: #1960
When reviewing this guide, please try running it on our machine!!

It would be nice to run this on different os than osx, as I only run it locally on my mac by now


MQTT Version 5.0 Implementation?

Hi, is there any plan to implement MQTT 5.0 ?

Fair enough, thanks

There are several Mqtt brokers available for testing now:
https://github.com/eclipse/paho.mqtt.testing
https://flespi.com/mqtt-broker#mqtt-checklist
https://vernemq.com/mqttv5.html

@chkr1011 Maybe re-open this issue, to keep track of the progress of V5 support?

v3.0.0 will have MQTTv5 support. For individual issues with that I prefer separate issues.


[Feature] Add materialized view definition

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #4787
Problem Summary(Required) ：

add MaterializedView meta class
add MaterializedViewManager to manage the relation between base tables and MaterializedView

[FE PR Coverage check]
:disappointed: fail : 193 / 252 (76.59%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/Table.java
2
6
33.33%

:large_blue_circle:
com/starrocks/catalog/Database.java
1
2
50.00%

:large_blue_circle:
com/starrocks/catalog/MaterializedView.java
151
205
73.66%

:large_blue_circle:
com/starrocks/catalog/PartitionInfo.java
1
1
100.00%

:large_blue_circle:
com/starrocks/alter/MaterializedViewManager.java
38
38
100.00%

run starrocks_fe_unittest

[FE PR Coverage check]
:disappointed: fail : 183 / 235 (77.87%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/Table.java
2
6
33.33%

:large_blue_circle:
com/starrocks/catalog/Database.java
1
2
50.00%

:large_blue_circle:
com/starrocks/server/GlobalStateMgr.java
1
2
50.00%

:large_blue_circle:
com/starrocks/catalog/MaterializedView.java
50
75
66.67%

:large_blue_circle:
com/starrocks/catalog/RangePartitionInfo.java
70
91
76.92%

:large_blue_circle:
com/starrocks/catalog/OlapTable.java
15
15
100.00%

:large_blue_circle:
com/starrocks/persist/gson/GsonUtils.java
5
5
100.00%

:large_blue_circle:
com/starrocks/catalog/PartitionInfo.java
3
3
100.00%

:large_blue_circle:
com/starrocks/alter/MaterializedViewManager.java
36
36
100.00%

[FE PR Coverage check]
:heart_eyes: pass : 146 / 177 (82.49%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/Table.java
2
6
33.33%

:large_blue_circle:
com/starrocks/catalog/Database.java
1
2
50.00%

:large_blue_circle:
com/starrocks/server/GlobalStateMgr.java
1
2
50.00%

:large_blue_circle:
com/starrocks/catalog/MaterializedView.java
50
75
66.67%

:large_blue_circle:
com/starrocks/catalog/OlapTable.java
15
15
100.00%

:large_blue_circle:
com/starrocks/catalog/RangePartitionInfo.java
33
33
100.00%

:large_blue_circle:
com/starrocks/persist/gson/GsonUtils.java
5
5
100.00%

:large_blue_circle:
com/starrocks/catalog/PartitionInfo.java
3
3
100.00%

:large_blue_circle:
com/starrocks/alter/MaterializedViewManager.java
36
36
100.00%

I'm reviewing this PR, please don't merge it.

[FE PR Coverage check]
:disappointed: fail : 104 / 142 (73.24%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/Table.java
2
6
33.33%

:large_blue_circle:
com/starrocks/catalog/MaterializedView.java
46
75
61.33%

:large_blue_circle:
com/starrocks/catalog/OlapTable.java
17
22
77.27%

:large_blue_circle:
com/starrocks/catalog/RangePartitionInfo.java
31
31
100.00%

:large_blue_circle:
com/starrocks/persist/gson/GsonUtils.java
5
5
100.00%

:large_blue_circle:
com/starrocks/catalog/PartitionInfo.java
3
3
100.00%

run starrocks_be_unittest

[FE PR Coverage check]
:heart_eyes: pass : 122 / 145 (84.14%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/MaterializedView.java
55
78
70.51%

:large_blue_circle:
com/starrocks/catalog/Table.java
6
6
100.00%

:large_blue_circle:
com/starrocks/catalog/OlapTable.java
22
22
100.00%

:large_blue_circle:
com/starrocks/catalog/RangePartitionInfo.java
31
31
100.00%

:large_blue_circle:
com/starrocks/persist/gson/GsonUtils.java
5
5
100.00%

:large_blue_circle:
com/starrocks/catalog/PartitionInfo.java
3
3
100.00%


ServerlessJS recipe

We should add a recipe for ServerlessJS https://github.com/Microsoft/vscode-chrome-debug/issues/495#event-1244061243

If someone can come up with this, I would be forever grateful. I've spent almost a week now trying to get VSCode + Serverless Framework + Typescript working, and I can only ever get like 80% of the way there. Debugger will break on the ./webpack/**/*.js files, but on nothing else, including non-TS files.


Arm64  builds

I wanted to give this a try on my raspberry pi server running arm64 but the build and dockerfile fails. It's not listed as a target in the makefile

There is a ARMv5 build but i'm not sure if its the right one. What kind of RPI do you have?

There is a ARMv5 build but i'm not sure if its the right one. What kind of RPI do you have?

I finally got it to work by converting it to a go modules project with go mod init and running a go mod tidy. Finally just a go build in go 1.15. Might have been a few other commands in there but the go compiler listed them. Works fine like this. Not sure why you aren't using go modules yet.
https://github.com/camccar/pgweb

RE modules: This is an old project so i just haven't gotten a chance to upgrade to modules.
RE build: I don't see you having binary build instructions added here https://github.com/camccar/pgweb/blob/master/Makefile#L57-L59, if you have them feel free to submit a PR and i will incorporate those builds into target releases.

I’ll try to get to making a PR this weekend. If all the test still pass

I just didn't understand how to use dep. I can build it now without modules. but it is missing the arm64 target. I added that in the PR. Would be awesome if you got an arm64 version in docker hub. In the mean time I just wrote another dockerfile to reference it locally.


LoctionIndexTree.query doesn't visit all nodes

See failing ignored test LocationIndexTreeTest.testBoundingBoxQuery2.
I thought maybe it isn't even meant to visit all nodes, but only all nodes with edges. But that isn't what's happening, either.

Yeah, something is wrong here.
The LocationIndex.Visitor is indeed too low level and only returns stored nodes and not all nodes (on purpose, but we should document this). But the EdgeVisitor should work as intended and yet it returns a very different node count (102 vs. 68). The best workaround at the moment seems to be:
EdgeExplorer edgeExplorer = graph.createEdgeExplorer();
index.query(graph.getBounds(), new LocationIndex.Visitor() {
    @Override
    public void onNode(int nodeId) {
        EdgeIterator iter = edgeExplorer.setBaseNode(nodeId);
        while (iter.next()) {
            set.add(iter.getBaseNode());
            set.add(iter.getAdjNode());
        }
    }
});

But even here there is a mismatch (102 vs. 98).

Thanks for clarifying! I'll test again and possibly "tighten" it a bit. Main thing is that I know what is supposed to come out there. I'll just ask again if I miss something. :-)

I'll just ask again if I miss something. :-)

Sure :)
In theory the class is not that complicated :) ... but to keep memory usage low there are some tweaks:

every level of the "quad" tree could have 16 or 64 leafs instead of just 4 (this per-level leaf count is calculated up front). See #1485 for a rough visualization.
From the in-memory tree InMemConstructionIndex (Java objects) we call prepareIndex and store only integers (node IDs). For every cell that an edge crosses we store e.g. the base node of this edge into these crossed cells.
But I had problems with "perfect" line rasterization and so we sometimes store a node ID in a cell X which is not crossed by the edge and only near it. Although we can handle this when querying the index it would be nice to get rid of this inefficiency.
We do not have to store the boundaries of every cell somewhere. Instead we can calculate them implicit when we do the query as we know the size of the smallest cell (deltaLat, deltaLon)

And additionally it has to be very fast as our CH routing is so fast :)

But I had problems with "perfect" line rasterization and so we sometimes store a node ID in a cell X which is not crossed by the edge and only near it.

I think we do only store a node ID in those cells X that are crossed by the edge. As we should, because that's the correct thing to do.
That's just not what Bresenham does. It's what Supercover does.
And it is what we do, except that the class is called Bresenham, not Supercover.
Correct?
http://eugen.dedu.free.fr/projects/bresenham/

Or is it the other way round, and we do use Bresenham, so we sometimes don't store a node ID in a cell X even though it is crossed by the edge, and we try to handle that during query?

Ah, ups, indeed. It was the other way around and we store the node IDs in too few cells and use bresenham. This is also the reason that for every location lookup we need to look at least into the 8 surrounding cells.
This supercover algorithm looks like the thing I was looking for - nice! After so many years you answered my question :)

This supercover algorithm looks like the thing I was looking for - nice! After so many years you answered my question :)

Actually, that's from your comment in the class. :-D

but it didn't work in our case as it works with integer values and not decimals like required for lat+lon.

Can't we scale the coordinates and apply the algorithm to their integer representation?

Can't we scale the coordinates and apply the algorithm to their integer representation?

I tried this at that time without success for certain real world cases, also someone in the one answer tried to find a solution for this, but it also didn't work for me.

Actually, that's from your comment in the class. :-D You even give references etc.

Ah, ok :) ... hmmh, then I likely also tried it without success. I tried so many things at that time, because I was (and am) pretty sure there is a way and I dislike(d) this imprecision. But at the end the workaround via fetching also the "next cells" worked too good to invest more time.


Portal Create New Site Unable to unzip portal content

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request

Module Version

4.0.2

Affected Resource(s)

ArcGIS_Portal

Configuration Files
3 machines setup in AZURE:

Machine 1: Portal, Portal Web Adaptor, Server Web Adaptor
Machine 2: ArcGIS Server
Machine 3: Datastore

Shared configuration on AZURE file share

# Copy-paste your DSC JSON configurations here - for large configs,
# please use a service like Dropbox and share a link to the ZIP file.

Expected Behavior
Portal site should have created
Actual Behavior
Run 1: Creating Portal Site issue

28/02/2023 9:53:26 AM: [portalmachine]:                            [[ArcGIS_Portal]Portalportalmachine] Making request to https://macineURL.com:7443/arcgis/portaladmin/createNewSite to create the site

28/02/2023 10:11:51 AM: [portalmachine]:                            [[ArcGIS_Portal]Portalportalmachine] Response received from create site {
    "error":  {
                  "code":  500,
                  "message":  "Unable to unzip portal content.",
                  "details":  null
              }
}
28/02/2023 10:11:51 AM: [portalmachine]: LCM:  [ End    Set      ]  [[ArcGIS_Portal]Portalportalmachine]  in 1188.5050 seconds.
28/02/2023 10:11:51 AM: [portalmachine]: LCM:  [ End    Set      ]
28/02/2023 10:11:51 AM: Operation 'Invoke CimMethod' complete.

Run 2: Created Portal Site all else went well but failed to register web adaptors
Run 3: Registered Web Adaptors, all other parts of config completed successfully
Run 4: Ran through without issues
Steps to Reproduce
I can't see any issues with the build yet, so the question is, has Portal installed correctly?
See checks in factoids, all appears aok.
If not, should I:
1. start again: restore the snapshots for all servers, delete all content from shared config
2. restore only the portal machine, clear out
Important Factoids
Reindexed Portal, all aok there
checked Portal logs and this message appears for:
SEVERE The process of creating a new site failed. Reverting site creation. Unable to unzip portal content.
I am assuming it cleans itself up. Subsequent messages in logs appear things went swell with Portal Site creation.
Everything in Portal, AGS and DS appear to be working as expected.

References

Hi @lachlanwainwright,
Here are some additional validation checks to try:

Ensure hosting server federation validates in portal home -> settings -> servers (or can also check in portaladmin endpoint)
Ensure relational data store in server manager validates. As well as any other data stores types you may have like tile cache.
Check system properties in portaladmin to ensure webcontexturl and private portal url are set as desired.
Create some content, like publish a hosted feature layer or test some of the workflows out that your organization will be doing.

If these checks pass then I think it is safe to say the deployment is golden.
Thanks,
Cameron K.

Thanks Cameron
Checkes done and passed without issue.
I have also run a webgisdr as another check and that passed perfectly too.
None of the logs show any severe errors so it's golden.
Thanks again Cameron on the ESRI DSC crew


I created a new project but there was a problem - Unhandled 'error' event

Bug Report or Feature Request (mark with an x)
- [ ×] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [x ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions

node -- v8.12.0;
npm --6.4.1;
ng --7.0.2;
Windows (10)
Repro steps

The log given by the failure

E:>ng new kl
? Would you like to add Angular routing? Yes
? Which stylesheet format would you like to use?
If you do pwd in the terminal where you are trying to do ng new what is the output?

@alan-agius4
As your tips, when I got your reply.
First, I restarted the PC and created a new project with powershell, there isn't any issue.
Then I tried to create another project with cmd, it can be also created.
That's so weird. The issue is gone. I don't know why.
`? Would you like to add Angular routing? Yes
? Which stylesheet format would you like to use?
If you do execute pwd and where cmd in the terminal where you are trying to do ng new what is the output?

Thanks @alan-agius4  i could resolve for me.
I tried to create an project in "D:" and got the same error as @MangoWHL.
Then i created a project in "D:\Test" and all was ok.
So i think the Problem was the Path.

I can reproduce the exact same behavior on Windows 10 if you try ng new in the root
of any of my drives. Make and enter a folder, and the error goes away.

I get the same issue when creating a project on root of a drive for example:
R:\
Create a project one level deep works:
R:\test
ng --version

Angular CLI: 7.0.3
Node: 8.9.4
OS: win32 x64

I get the same issue when creating a project on root of a drive for example:
F:\
Create a project one level deep works:
 F:\TestDirectory
But i want to create project on root of a driver
how can i ?

@amirzenoozi create the project one level deep and then move it?

Hi, all we have identified the bug that is causing this error, as at the moment we are not handling root paths correctly in windows.
We landed a PR https://github.com/angular/angular-cli/pull/12951 to fix the issue

i am getting same issue but not in root drive
e:\code\FrontEnd\Angular-by-bradtraversy
still getting same error


Use POST endpoint instead of expand the cluster list

Description
org_overview endpoint currently used the GET endpoint to retrieve all the cluster reports from Aggregator. It can lead in some situations to very long URLs that can be handled differently depending on configuration.
This change makes the endpoint to use the POST endpoint version to retrieve the reports for a list of clusters, which will avoid the possible URL problems
Type of change

Refactor (refactoring code, removing useless files)

Testing steps
Regular CI
Checklist

[x] make before_commit passes
[x] updated documentation wherever necessary
[x] added or modified tests if necessary
[x] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report

Merging #584 (cc3a07e) into master (30e674e) will decrease coverage by 0.24%.
The diff coverage is 40.62%.

@@            Coverage Diff             @@
##           master     #584      +/-   ##
==========================================
- Coverage   55.20%   54.95%   -0.25%
==========================================
  Files          12       13       +1
  Lines        1355     1412      +57
==========================================
+ Hits          748      776      +28
- Misses        509      537      +28
- Partials       98       99       +1

Impacted Files
Coverage Δ

server/server.go
55.80% <40.62%> (-3.19%)
:arrow_down:

conf/configuration.go
63.63% <0.00%> (-0.95%)
:arrow_down:

server/handlers.go

server/endpoints.go

server/endpoints_v1.go
100.00% <0.00%> (ø)

server/endpoints_v2.go
100.00% <0.00%> (ø)

server/handlers_v1.go
51.49% <0.00%> (ø)

Continue to review full report at Codecov.

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


Did I install it wrong?

Hey there! my first time trying a c++ library, I could "#include "clip.h" and write some code but I get those two errors

this is the code im trying to run

anyone know what im doing wrong?
thanks very much

Hi @Aze1337, you need to link your .exe with the clip.lib.
You can see an example in the Wiki with cmake: https://github.com/dacap/clip/wiki#how-to-use-it
I hope it helps!

Hello @dacap Thank you a lot for your response!
I'm still unsure how to do it - I've followed this https://en.wikibooks.org/wiki/C%2B%2B_Programming/Compiler/Linker/Libraries/Configuring_Visual_Studio

I have put this folder path here,

and this folder

in here

but whenever I compile, it gets me those errors

do you possibly know what I'm doing wrong ? thanks again

You've configured the directories but didn't specify the clip.lib as an input to the linker (check the Linker > Input), anyway I don't remember the Visual Studio settings, but I think there should be the field to add the clip.lib as an input to the linker.

just specified it there

less errors now but still not sure what else to do

Oh sorry about that, you have to link to shlwapi.lib too (add that file in the additional dependencies field too).
Generally clip is made to be used with cmake (so dependencies are included automatically), so this manual configuration is not expected in the regular use case, but in your case, you will need to specify the dependencies manually.

Oh sorry about that, you have to link to shlwapi.lib too (add that file in the additional dependencies field too).
Generally clip is made to be used with cmake (so dependencies are included automatically), so this manual configuration is not expected in the regular use case, but in your case, you will need to specify the dependencies manually.

Thank you very much for your help! I can compile & run just fine :D


Documentation issue

On this Using Bundlers page a wrong statement is made.
"Keep in mind that every browser that supports service workers also supports most ES2015 features (the primary exception being import statements), so there's usually no need to transpile your code.
This means you can freely use features like classes and syntax like async/await when writing service worker code, which is fantastic as most service worker APIs are promise-based."
async/await was introduces in ES2017, not ES2015

We removed that bit from the latest version of that page.


Workbox-Google-Analytics not working correctly on Node

Library Affected:
workbox-google-analytics v6.5.4
Browser & Platform:
All browsers and NodeJs (NextJS)
Issue or Feature Request Description:
When trying to use the node library for workbox-google-analytics in my NextJS app, I get a SyntaxError: Cannot use import statement outside a module as soon as I try to initialize the instance at the moment of compiling the code for the dev env. As shown in the screenshot below.

It is worth noting that when I try to run the code from the NPM RunKit the same error is shown when trying to initialize the instance as shown below.

I tried running the libraries locally using the workbox CLI's CopyLibraries directive as a workaround but I couldn't initialize the workbox instance either.
I appreciate any help regarding this topic.

Have you solved this problem? I stumbled on it.

Have you solved this problem? I stumbled on it.

Hi @Simple-Jian in the end I ended up using the fetch implementation inside the node file. Like manually calling the API with all the params in it.


ERROR  in installing frame field learning package

I'm running the code on Windows 10 system. I successfully executed bash setup.sh
Post that I ran python setup.py install and got this output:
Installed d:\anaconda3\envs\topo\lib\site-packages\frame_field_learning-0.0.1-py3.6.egg
Processing dependencies for frame-field-learning==0.0.1
Finished processing dependencies for frame-field-learning==0.0.1

But when I run python main.py --run_name "frame_field_learning/runs/mapping_dataset.unet_resnet101_pretrained.field_off.train_val | 2020-09-07 11:54:48" --in_filepath grid_100-100_3698.0.tif
This throws an error
ERROR: The frame_field_learning package is not installed! Execute script setup.sh to install local dependencies such as frame_field_learning in develop mode.
Please reply @Lydorn
Thanks in Advance

May a different python env you used when you ran main.py, especially conda env. I setup up it and ran it. I have never seen this error.

@Ankit-Vohra I have encountered the same issue. The setup.sh file didn't run directly so I ran the contents one by one. The frame_field_learning is installed but when I run main.py it shows not installed again. Did you encounter the same issue and how did you solve it if you did? I would be glad if you could provide some insights. Thanks.

If the error remains, adding all the imports to 'requirements.txt' file and then running the setup and main file should work. I managed to do it this way.

If the error remains, adding all the imports to 'requirements.txt' file and then running the setup and main file should work. I managed to do it this way.

Hello, I tried your method, but I didn't understand what "imports" is. Do you mean that put all the ".py" file names in the "lydorn_utils" folder into "requirements.txt"?

If the error remains, adding all the imports to 'requirements.txt' file and then running the setup and main file should work. I managed to do it this way.

Hello, I tried your method, but I didn't understand what "imports" is. Do you mean that put all the ".py" file names in the "lydorn_utils" folder into "requirements.txt"?

Hello,
I have attached the editted requirements.txt file that I used. You can try and see if this works for you. Wish you luck.
requirements_editted.txt

If the error remains, adding all the imports to 'requirements.txt' file and then running the setup and main file should work. I managed to do it this way.

Hello, I tried your method, but I didn't understand what "imports" is. Do you mean that put all the ".py" file names in the "lydorn_utils" folder into "requirements.txt"?

Hello,
I have attached the editted requirements.txt file that I used. You can try and see if this works for you. Wish you luck.
requirements_editted.txt

I have sloved this issue by edit "main.py" ---- "import("frame_field_learning.local_utils")'"  to  "import frame_field_learning.local_utils", but thank you for your kind reply.


flakes: split sources

This PR splits flakes/manual.toml into multiple files under flakes/manual/, and modifies .github/workflows/check-flake-files.yml so that it only runs on the modified files.

Hey, sorry this took me so long to take a look at.
I really like the direction this is going. This does however only address the PR checks and not the actual import. That would be where the actual issues surface when a downstream flake breaks. So I think we should loop over the groups in the import as well, write them to a yaml (maybe? it should at least be concat'able) and bulk-import that into ES.

I've also lost track of the PR, but I think the one-line change in .github/workflows/import-to-elasticsearch.yml does exactly that?


Add deadline for pre-install job

according to docs if pod fails Job will start a new pod.
So if config contains error (e.g. typo in rethinkdb URL) job will create infinite count of pods.
In my case, it was 12k pods and broken cluster :)
In this PR I add deadline. The job doesn't create any new pods after deadline.
300 seconds seems reasonable for me. I beleive pre-install job finishes in 5 minutes in all cases.

Sorry for the delay.  Yeah, the hook-failed solves my problem. Should I add it to pre-install-job.yaml? It seems useful.

@nesterov-n sure

Stale pr. Closing it


rootston: Debug output damage flag stopped working in 0.6

@Timidger Can this be closed as Rootston isnt here anymore?


Can’t use PC without adding package cloud to our maven repo proxy

This is because one dependency - truth generator api - is published only to package cloud. Publishing TG to repo1 (maven central) would avoid this.
Building and running PC was tested in a clean environment on Confluent’s build system, so the package cloud repo works, but in some environments, it’s not able to be reached automatically without extra config steps to the proxy setup.
https://github.com/confluentinc/parallel-consumer/issues/264#issuecomment-1128532160
Workaround: configure maven proxy to allow access to package cloud.
edit:
Ah - I just realised it was incorrectly set as a compile time dependency, which is incorrect. It's only needed to run the tests. Will fix in:

#306
and release in <IP_ADDRESS>

Ah - I just realised it was incorrectly set as a compile time dependency, which is incorrect. It's only needed to run the tests.

Released: https://github.com/confluentinc/parallel-consumer/releases/tag/<IP_ADDRESS>


How can I customize the toolbar XML?

One of the great things about the Android toolbar is that it is more customizable than the ActionBar. I haven't found a direct way to change the XML with this library. For example, I want to add a spinner dropdown to the toolbar and change the hamburger icon to an up arrow for some fragments.
The best solution I've come up with is to set the toolbar's visibility to GONE and make my own. I'm not sure if this will cause other integration problems.
What's the best way to customize the toolbar XML?
p.s. Thanks for the great library :)

i'm also curious of the best practice recommended for this

I'm interested on this, too. Same case, I want to add a Spinner that mimics material specs.

Done it, but I need to do some other tricks to fix colors. Added the spinner following (this post)[https://blog.danielbetts.net/2015/01/02/material-design-spinner-toolbar-style-fix/]
Just note that on the "Adding spinner to your toolbar" you should use getToolbar() in place of 'toolbar' or ((MaterialNavigationDrawer) getActivity()).getToolbar() if you're inside a fragment.

Fixed. It was almost illegible text on dark background, just added android:background="#FFFFFF" and changed android:textColor to #87000000 on the TextView inside toolbar_text_spinner_dropdown.xml


Inference Issue: Missing MHT Weights in NeuralPLexer Checkpoint

Environment:

Python Version: 3.9.18
Operating System: Linux

Issue Description:
I was performing batched structure sampling using the NeuralPLexer model checkpoint (version v2, published December 13, 2023) from Zenodo. The inference command (no template) was as follows:
python NeuralPLexer-main/neuralplexer/inference.py --task=batched_structure_sampling --input-receptor "opt.pdb" --input-ligand "test.sdf" --out-path "test" --model-checkpoint "neuralplexermodels_downstream_datasets_predictions/models/complex_structure_prediction.ckpt" --n-samples 16 --chunk-size 4 --num-steps=40 --cuda --sampler=langevin_simulated_annealing

Encountered Issue:
The process emitted an error indicating the inability to load pretrained MHT weights, suggesting their absence from the complex_structure_prediction.ckpt file, which seems essential for the _resolve_ligand_encoder during inference.
...
Could not load pretrained MHT weights, skipping
...

Investigative Steps:
To assess the importance of MHT weights for the inference process, I added a debug line in wrappers.py (line 470):
        # Assuming static ligand graph
        if "lig_atom_attr" not in batch["features"].keys():
            print("!!!!!!")
            self._initialize_ligand_embeddings(batch)
        return batch

This modification confirmed the critical role of MHT weights in the inference.
Request for Assistance:
Has anyone else faced a similar issue with missing MHT weights during NeuralPLexer inference? I am looking for any advice or workaround to address this problem. Any shared experiences or suggestions would be highly appreciated.
output_log.txt

Do you know where the file "neuralplexermodels_downstream_datasets_predictions" is? I can't find it in the repository.

I ran into the same issue as reported by JohnBrown2022. Any help on this would be greatly appreciated. Thanks in advance.

This issue seems related to https://github.com/zrqiao/NeuralPLexer/issues/3, no?

I believe my new PR resolves this warning. However, I believe NeuralPLexer still loads the pretrained MHT weights (in spite of this warning being logged) since the inference.py calls Lightning's load_checkpoint() function without enforcing strict matching of key names. I suspect this issue originally came from the names of the ligand and protein encoder modules being changed in the released source code after the complex_structure_prediction.ckpt weights were initially changed.

I believe my new PR resolves this warning. Nonetheless, I believe NeuralPLexer still correctly loads the pretrained MHT weights during inference (in spite of this warning being logged) since the neuralplexer/inference.py script calls Lightning's load_from_checkpoint() function without enforcing strict matching of key names.
https://github.com/zrqiao/NeuralPLexer/blob/301f10de0d214c73956b374fc384c0d37ddbf544/neuralplexer/inference.py#L631
I suspect this issue originally came from the names of the ligand and protein encoder modules being changed in the released source code after the complex_structure_prediction.ckpt weights were initially changed. In which case, this is an easy fix.

I also ran into this issue. I downloaded the package last week, thus hopefully this PR is already included. Would this warning affect any of its predictions? Thank you in advance!


Add nested data types suport fot athena get_columns_in_relation macro

Description
I’m facing the problem with incremental materialization for iceberg  tables. Macro athena__get_columns_in_relation is currently getting fields types from information_schema . However there are some problems:
nested fields are stored as row(field type) instead of struct<field: type>
fields with type binary are stored as varbinary
Checklist

[x] You followed contributing section
[x] You added unit testing when necessary
[x] You added functional testing when necessary

@svdimchenko have a look at https://www.conventionalcommits.org/en/v1.0.0/ and adapt the PR title accordingly.

@svdimchenko Will be nice if you can provide the models that you use to test this out, e.g. with some mock data. This will help the reviewers.

@svdimchenko the proposed test model doesn't help me. Could you add a dbt model with mock data to help us to fully test this?

@mattiamatrix shall we merge this?

What I don't understand is that we already have a utils macro that gets correct type based on information schema or maybe I am missing something ? 🤔

I think that indeed in here we could aim to use the data_type macro, as it should render the right data type.

@svdimchenko this PR needs to be re-adapted. In https://github.com/dbt-athena/dbt-athena/pull/174 we replace the get_columns method with a glue api col, therefore we don't use the information schema from athena/trino anymore.
Could you check if this feature is still relevant for your use case? or if the issue is solved using the latest version (main)??
cc @Jrmyy

not more actual


use of const in strict mode

I'm sorry, I'm new in node. I've got a problem said 'use of const in strict mode' with connect-mongo when trying to support connect-flash.I check my code according to your readme guidance but it doesn't work.How can I solve the problem please?

What is your node version?
If it's under 4.0 you must use require('connect-mongo/es5').
Le ven. 18 déc. 2015 à 04:26, mosquito1994<EMAIL_ADDRESS>a
écrit :

I'm sorry, I'm new in node I've got a problem said 'use of const in strict
mode' with connect-mongo when trying to support connect-flashI check my
code according to your readme guidance but it doesn't workHow can I solve
the problem please?
—
Reply to this email directly or view it on GitHub
https://github.com/kcbanner/connect-mongo/issues/209.

require('connect-mongo/es5') not works because of
Error: Cannot find module 'connect-mongo/es5'

OK. Working on it!

Ok, after npm update
require('connect-mongo/es5') working fine
Good to be documented on npm page, thanks

I added a test to check ES5 version loading.
Also updated the way we deal with those versions + published 1.0.2.

Make sure your package.json reads "connect-mongo": "1.0.0" in "dependencies". Also your require statement must look like: require('connect-mongo/es5')
The newest version of connect-mongo uses a different version of javascript called ES6.

Make sure your package.json reads "connect-mongo": "1.0.0" in "dependencies". Also your require statement must look like: require('connect-mongo/es5')
The newest version of connect-mongo uses a different version of javascript called ES6.

I've the same problem
I tried the following code
var express = require('express');
var expressSession = require('express-session');
var mongoStore = require('connect-mongo')(expressSession);
var sessionStore = new MongoStore({
host: '<IP_ADDRESS>',
port: '27017',
db: 'session',
});
but i get the following error:
/Users/alf/node_modules/connect-mongo/src/index.js:3
const Promise = require('bluebird');
^^^^^
SyntaxError: Use of const in strict mode.
at exports.runInThisContext (vm.js:73:16)
at Module._compile (module.js:443:25)
at Object.Module._extensions..js (module.js:478:10)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at Object. (/Users/roger/node_modules/connect-mongo/index.js:1:80)
at Module._compile (module.js:460:26)
at Object.Module._extensions..js (module.js:478:10)

@rogercastaneda You must use the ES5 version, and use a connection string.

require('connect-mongo/es5')
Thanks. It's work for me.

Thanks, it was my fault, i was using MongoStore instead of mongoStore. It works now :)


beckhoff constants dont work

https://github.com/tomookay/transline2003/assets/144595622/42c54c15-1139-4e41-8693-277c7dc82ea6
its happening again!

i think it was unsafe memory allocation by using a loop in an array that was larger than a const bounds, but there is no check to say its outside the array as the instance call is defined by an in_out


generate idanode url with custom name

Please generate new file called peersv2 to maintain compatibility with previous versions of @scrypta/core


catpuccin does too much escaping

Is there an existing issue outlining your problem?

[X] I have searched the existing issues and they do not solve my problem.

Describe your problem.
Before, this used to work:
set -g @catppuccin_session_text "#{=10:#{s/^(TFS|KB|SUMMIT-)([0-9]*).*/KB-\\2/:session_name}}"

Basically, I want to apply some formatting to the session name before showing it.
Unfortunately, it does not work any more. For example, session name KB1234_xyz should be displayed as KB-1234, but now the result is always KB-\2.
Checking tmux show -A, it seems that catpuccin sets
status-left* "...#{=10:#{s/^(TFS|KB|SUMMIT-)([0-9]*).*/KB-\\\\2/:session_name}}..."

in other words, the backslashes are escaped. I don't think the plugin should be doing it. If I want to add a \ as text, I can escape it myself.
Paste your configuration.
set -g @catppuccin_session_text "#{=10:#{s/^(TFS|KB|SUMMIT-)([0-9]*).*/KB-\\2/:session_name}}"

Attach screenshots.
No response
What tmux version are you seeing the issue on?
tmux built from recent commit [651891c3]
Any additional comments?
No response

The full config:
set -g @plugin 'catppuccin/tmux'
#set -g @catppuccin_flavour 'mocha' # latte,frappe, macchiato or mocha (default)

set -g @catppuccin_window_left_separator "" # "" "null"
set -g @catppuccin_window_middle_separator "▌" # ":" - was not working in one version of catpuccin
set -g @catppuccin_window_right_separator "" # "null" # remove space between windows
set -g @catppuccin_window_status_enable "yes"
# set -g @catppuccin_window_status_icon_enable "no"
set -g @catppuccin_icon_window_current "null" # Other preference could be "★" , but current window is already color-coded
set -g @catppuccin_icon_window_last "" # "" "-" ; the original dash is too big for my taste
set -g @catppuccin_window_default_text "#{s/^(TFS|KB|SUMMIT-)([0-9]*)/KB-\\2/:window_name}"
set -g @catppuccin_window_current_text "#{s/^(TFS|KB|SUMMIT-)([0-9]*)/KB-\\2/:window_name}"

set -g @catppuccin_status_left_separator "" # "null"
set -g @catppuccin_status_right_separator "null"
set -g @catppuccin_status_fill "all"

set -g @catppuccin_status_modules_left "session"
set -g @catppuccin_session_icon "null"
set -g @catppuccin_session_text "#{=10:#{s/^(TFS|KB|SUMMIT-)([0-9]*).*/KB-\\2/:session_name}}█"

set -g @catppuccin_status_modules_right "user host"
set -g @catppuccin_user_icon "null"
set -g @catppuccin_host_icon "null"

I restart tmux, rename the session to KB123_aaa, then get this:

The session name in the bottom left corner should be shown as KB-123

It seems that the problem (for me, at least), is in get_tmux_batch_option function. If I bypass it:

Everything works:

@j-xella
Can you check if #282 fixes it? Just checking out that PR and running ./catppuccin.tmux in an tmux session should be enough to see if it fixes it.

Going to revert #240 until I can find a fix for this.

#282 fixes the other issue I reported, #280. But not this issue.

@j-xella Can you test https://github.com/vdbe/catppuccin-tmux?
Merged the fix into main of my fork. Works for me but can never be sure.
Problem was that the -v prints the escape string/value and without it you get the unescaped/value string.

Seems to be working. See the screenshot, you can compare the names of sessions/windows with what is displayed in the status bar.


Register family check output as individual fonts?

While trying to make a reporter group per-font reports into sections and checks, I noticed that family checks are an annoying-to-deal-with edge case, because they embed which font a message object belongs to in-band (and only when the check does it).
I'd like to achieve something like this:
Spacing:
  Some spacing check:
    UFO1:
    - WARN: blah blah
    UFO2:
    - WARN: blah blah

  Some other spacing check (family check):
    UFO1:
    - WARN: blah blah
    UFO2:
    - WARN: blah blah

instead of
Spacing:
  Some spacing check:
    UFO1:
    - WARN: blah blah
    UFO2:
    - WARN: blah blah

  Some other spacing check (family check):
    Family Check:
    - WARN: UFO1: blah blah
    - WARN: UFO2: blah blah

For this, the reporter currently needs intimate knowledge of where the checks store that information. Is there a better way?

Not sure exactly what you mean. Is it, that in your second example the check is defined something like this: def some_other_spacing_check(ufos) and it is yielding the results of each ufo in one go?
You could try to: def some_other_spacing_check(ufo, ufos) which would call the check once for each ufo, though maybe you'd need to do the same calculations again for each call, thus:
Define a @condition def family_spacing_analysis(ufos) then  def some_other_spacing_check(ufo, family_spacing_analysis) and just yield the results for the current ufo stored in the return value of family_spacing_analysis the @condition would run just once and it's result would be cached.

Is it, that in your second example the check is defined something like this: def some_other_spacing_check(ufos) and it is yielding the results of each ufo in one go?

Yep. I'd like to have the check run on all fonts but return a separate check result dict for each, so that my reporter can be dumb and just treat everything as a check result for an individual font.

Define a @condition def family_spacing_analysis(ufos) then def some_other_spacing_check(ufo, family_spacing_analysis) and just yield the results for the current ufo stored in the return value of family_spacing_analysis

Interesting idea. Will have a look, thanks.

I don't think so, we settled on something or other internally.


[Question]: How to change the text to sql model in NLSQLTableQueryEngine

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Can you confirm if I provide GPT4 in Settings.llm, does NLSQLTableQueryEngine method uses the same for text to sql conversion? If not, how can I configure it.

Do we have any SQL agents in llama index something similar to the one we have in langchain?

@sreenivasanm6 it does use the settings llm. you can also pass it in directly to the query engine llm=llm
There is no sql agent, but you can give a sql query engine as a tool to an agent.


NPM 2.1.0 release doesn't include Message components

See: https://runkit.com/frobinsonj/60c60716f281aa0019cb5eca
Seems to have undone changes applied in v2.0.2 also.

Thanks for the heads up, should be fixed in latest release


Make logging optional and update docs

Hello! :wave: This PR contains a few changes, notably:

Updating the README.md to use the new initializer instead of the deprecated one.
Add a renamed to the deprecated initializer.
Updating the docs to use the simpler initializer.
Make the logging parameter on the initializers optional, the motivation for this is in cases the user don't or can't do logging, they don't have to construct a logger and bootstrap the logging system with SwiftLogNoOpLogHandler, so this code:

LoggingSystem.bootstrap(SwiftLogNoOpLogHandler.init)
let serviceGroup = ServiceGroup(
    services: [service],
    cancellationSignals: [.sigint],
    logger: .init(label: "")
)

becomes
let serviceGroup = ServiceGroup(
    services: [service],
    cancellationSignals: [.sigint]
)

and it also allows the user to do logging outside the ServiceGroup without necessarily doing logging within it.
If there is anything to improve, let me know! :smiley:
thanks dimi and gwynne for helping along the way and answering my questions :)

Thanks for the change. Let's get this landed!


[Enhancement]: Add oc client binary to microshift-aio

What would you like to be added:
In the microshift-aio image, replace the upstream kubectl with OKD's oc and kubectl.
Why is this needed:
Users may want to use OC commands to do things; version should match OKD's.

OKD version of last rebase: 4.8.0-0.okd-2021-10-10-030117
Note: This will need to be updated with every rebase in images/microshift-aio/Dockerfile or will need to create a version command to output this for scripting.

/assign


Write Tests for Type- Narrowing

No of people: 2
Write test cases for all the type-narrowing cases

[ ] If stmt
[ ] Ternary Expr
[ ] While Stmt
[ ] From Clause
[ ] From Do Stmt
[ ] Match Stmt

Report bugs if there are any which are not reported

In-line with the spec's definition of narrowing this needs to focus on the changes we did to narrowing to allow for semantic subtyping also.


Don't dispose of Stream in StreamReader

In .NET 4.0, disposing the StreamReader also destroys the underlying
Stream. By removing the using here we ensure that the Stream won't be
destroyed. The StreamReader will be later collected by GC, which will
call Finalize(), which doesn't dispose the Stream.
@thirkcircus

Can you please put this back how it was? We have copied they say seq and elastic search sinks are working, and if there's not an issue then why change it?
https://github.com/serilog/serilog-sinks-seq/blob/master/src%2FSerilog.Sinks.Seq%2FSinks%2FSeq%2FHttpLogShipper.cs#L282

Oh wait, I just checked the net40 version:
https://github.com/serilog/serilog-sinks-seq/blob/master/src/Serilog.Sinks.Seq/Sinks/Seq/HttpLogShipper-net40.cs#L295
Sorry!

since we don't know how the kinesis net40 impl diverged from the Seq net40 impl, i've just diffed the original Seq HttpLogShipper-net40 to the kinesis one - and looks like there's another streamreader that needs fixing...
original file has two streamreaders;
https://github.com/serilog/serilog-sinks-seq/blob/master/src/Serilog.Sinks.Seq/Sinks/Seq/HttpLogShipper-net40.cs#L273
https://github.com/serilog/serilog-sinks-seq/blob/master/src/Serilog.Sinks.Seq/Sinks/Seq/HttpLogShipper-net40.cs#L294
@endofcake , you happy to also fix the one in TryReadLine - that's the original bug I noticed.

Awesome thanks Justin.

it'd be nicer if we didn't have to backport our application to net40, then we'd never even have noticed this bug...  :wink:

@thirkcircus , sure, here we go https://github.com/serilog/serilog-sinks-amazonkinesis/pull/11.


GitHub & WordPress.org out of sync

Latest version on wordpress.org is 1.0.6
Last stable release tag on github is 1.0.3

Updated to 1.0.6


Research retrieving test results from ctest and recording in Azure DevOps

See: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/test/publish-test-results?view=azure-devops&tabs=yaml

https://github.com/Azure/azure-sdk-for-c/pull/343

Using CMocka to generate xml files and CI updated to publish results

Publishing one test result for each pipeline (win86, win64, lin64, mac64)


[Bug] [Agent] No agent option displayed in WebUI

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating system information
Linux
Python version information

=3.11

DB-GPT version
main
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[ ] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[X] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
Device: 1 GPU
Models information
Any model
What happened
I followed the instructions from this doc:https://www.yuque.com/eosphoros/dbgpt-docs/pom41m7oqtdd57hm ,but no plugin option displayed in WebUI. I have done cookie configuration, and inserted the metada to MySQL.
What you expected to happen
Plugin option displayed in WebUI correctly.
How to reproduce
Follow the steps in https://www.yuque.com/eosphoros/dbgpt-docs/pom41m7oqtdd57hm
Additional context
What's the meaning of the metadata manually inserted? I see model infos in the SQL, should they match the actual model?
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

i run into the same issue. but i modify the dbgpt/app/component_configs.py. add the rm.register_resource(resource_instance=my_tool). Then this ui has the tools.


Add default openBrowser function

All platforms other than Darwin, Linux, and Windows fail to build gup due to a missing openBrowser function.
This commit adds a default openBrowser function for those other systems to allow gup to build on other platforms like the BSDs, directing users to open the bug report URL in their preferred browser as intended.

@memreflect
Thank you for your PR. I will merge this PR.
(Note that the gup command is officially only supported on Linux, Mac, and Windows.)


Windows 10S support: exclude *.node native NodeJS libraries from the generated Electron ASAR

TODO: add in package.json's Electron-Builder configuration: "asarUnpack": "**/*.node" (otherwise, the LevelDown etc. libs get extracted into a temporary directory by the Electron runtime, which is forbidden on Windows 10S)

Note that the LevelDown and PouchDB/LevelDB adapter NodeJS native libs include 6MB of unnecessary data inside the prebuilds folders ... maybe we can remove these at package-time, to make the application a little bit lighter?

Note that this issue is a blocker only for Windows 10S, not for Windows 10 (or Mac, or Linux ... as far as we know).
The only way to solve this issue is by making sure *.node native NodeJS libs are located outside of the Electron ASAR archive, using the "asarUnpack": "**/*.node" electron-builder directive inside package.json (which results in the filesystem structure shown in the above screenshot)
But, when we exclude lcp.node from the ASAR, we then have security/integrity issues that break LCP support. We actually have a solution for this already, but my recommendation is to only apply it specifically for the Windows 10S build in order to preserve the most robust technique for all other target platforms.
Therefore, I suggest we keep lcp.node inside the ASAR for now, but we can of course externalize  the leveldown.node NodeJS native libs. This means that Thorium will be compatible with Windows 10S, but LCP support will be broken (for now anyway). In the future, we can craft a Thorium build especially aimed at Windows 10S deployment, which will use a special version of lcp.node that works outside of the Electron ASAR.

Note that the app.asar.unpacked folder sibling to the app.asar file archive contains filesystem entries that are referenced (via relative paths) inside the app.asar "central directory" (header that lists all available resources inside the ASAR virtual filesystem used by Electron). In other words, attempting to ASAR-extract the app.asar without the presence of its sibling app.asar.unpacked folder fails.

Note that instead of asarUnpack for external-assets/lcp.node (see https://www.electron.build/configuration/configuration#configuration-asarUnpack ), optionally we could use extraResources or extraFiles (preferably the former, see https://www.electron.build/configuration/contents#extraresources and https://www.electron.build/configuration/contents#extrafiles ), which would require removing external-assets from the files configuration (see https://www.electron.build/configuration/contents#files ). This technique would apparently remove the risk of *.node duplicates inside the ASAR when mixing asarUnpack with files (I found this information in the Electron issue tracker, but wasn't able to reproduce the bug myself).
This is currently a moot point as we are going to keep lcp.node inside the ASAR.

Fixed via https://github.com/readium/readium-desktop/pull/407


Allowing Thread.DelayRepeat to connect immediately rather than yield

Currently with Thread.Delay, the function will always yield for the intervalTime before connecting to the heartbeat.  Having the option to connect to the heartbeat immediately would allow the function to be useful in more use-cases.
My proposed solution
function Thread.DelayRepeat(intervalTime, func, yieldMethod, ...)
    local args = table.pack(...)
    local nextExecuteTime
    yieldMethod = yieldMethod or "before"
    assert(yieldMethod == "before" or yieldMethod == "after", "yieldMethod must be \"before\" or \"after\"")

    if yieldMethod == "before" then
        nextExecuteTime = (time() + intervalTime)
    elseif yieldMethod == "after" then
        nextExecuteTime = 0
    end

    local hb
    hb = heartbeat:Connect(function()
        if (time() >= nextExecuteTime) then
            nextExecuteTime = (time() + intervalTime)
            func(table.unpack(args, 1, args.n))
        end
    end)
    return hb
end```

I've actually run into needing this a few times recently. The reason I haven't immediately implemented a change like this is due to backwards-compatibility: By adding a third argument before the 'varargs' ..., it could mess up existing code that relies on passing those arguments.
However, considering Knit is still in alpha, this is the exact time in its lifespan to make breaking changes. I think there's a good reason to add something like this.
I would prefer to stay away from using string values to indicate behavior though. Instead, we could do something where yieldMethod is renamed to something like immediateExecution (or something less verbose), and is just a boolean true|false.

@CaptinLetus Made a PR for this. Let me know what you think: https://github.com/Sleitnick/Knit/pull/28/files#diff-da7f058710b3a2b5b96e5ee4d6367fa53f143f6b3c5fcc3c48c7da783fa32045R151

A much cleaner implementation that my own to accomplish the same result.  The only 2 possible yieldMethod in my solution are before and after so a boolean makes perfect sense here.  Looks great!


Add support for color temperature on Fanelee

Add support for color temperature on Fanelee. This device doesn't support arbitrary color temperatures, just 6 specific ones. After selection, identify the closest. Additionally, the color temperature is suffixed with K.

Cool. This will make a lot of people happy. I'll merge for now and do a release in a day or so.


Drop down prediction menu

Hello! First of all, thank you very much for this autocomplete menu widget, I really needed it.
The thing is, when I search for something in the entry box, the autocomplete displaces everything. Is there a way to avoid it? Also, when I successfully search for something, the autocomplete dropdown doesn't disappear, is there a way to make it go away?
Thanks in advance.

Terribly sorry not to get back to you on time. In case you or future readers still need help with this, shoot me an email.


How to inherit values from the parent to the child helmfile?

We are trying to use helmfile in our pipeline. For this we hoped to use a parent helmfile (with repository configuration and helm-defaults) and subhelmfiles that INHERIT from this master helmfile.
Unfortunately this was not possible to achieve.
Are we missing out something?

@RafalMaleska What do you want to inherit?
One of the rules of helmfile is that each sub-helmfile should be usable independently. So let's assume that your sub-helmfile should be complete on its own i.e. should work without inheritance at all.
It does support inherting values expicitly from the parent to the sub-helmfiles by using this syntax:
helmfiles:
- path: path/to/subhelmfile.yaml
   values:
   - foo: {{whatever available in the parent}}

Does it help?

@mumoshu thank You very much.
We have established something similar with templating and including it in each subhelmfile. But Your solution is much much cleaner.
Thank You

@RafalMaleska Glad to help!
I'm now closing this as resolved. Please feel free to join your slack channel if you have any further questions!


Lipschitz bound regularization

Dear friends,
since we need to integrate everything into sQUlearn from the AutoQML project, I was wondering if the Lipschitz bounds regularization stuff I worked on can be integrated and how I can help to realize this.
Here is the reference https://arxiv.org/abs/2311.11871 and you can find an example implementation here aswell https://github.com/daniel-fink-de/training-robust-and-generalizable-quantum-models. to my knowledge Frederic was able to use it in his research.
Best,
Danya

Probably it's best to ask Christian first if it is really necessary before you start all the work.
In principle, it should not be too difficult since you can implement it as a loss, for which a class exists. It's currently a work in progress, but the Loss class is going to contain the QNN and therefore all the data you need for the Lipschitz bounds. In the Loss class, you also have access to the parameter values.
It's best to take a look at ./src/squlearn/qnn/loss.py.

Actually that is why I raised the feature request, because Christian asked me to.
But if there is this loss class already, I see no issue as we just calculate a norm over the weigths (those that multiply the data).


Allows to save the Twitter IDP without the mandatory input field API key

Environment
Standalone 5.4.0 GA pack
DB - H2
Browser - Chrome Version 61.0.3163.100 (Official Build) (64-bit)
JDK - jdk1.8.0_121
Steps to recreate the issue

Run Standalone IS 5.4.0 GA pack
Register Twitter as a IDP using Twitter configuration without giving a value to API key
Allows to save the IDP

Observation
Even though API key is a mandatory field allows to save the IDP. No error message is displayed.

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


fix #1675

Pull request checklist

[x] All tests pass. Demo project builds and runs.
[x] I have resolved any merge conflicts.
[x] I have followed the coding style, and reviewed the contributing guidelines. Confirmation: @Lucashuang0802

This fixes issue #1675
What's in this pull request?

re-assign font and textColor to textView since font and textColor will be resetted when selectable gets changed to NO. Ref: http://stackoverflow.com/questions/19049917/uitextview-font-is-being-reset-after-settext
postpone the execution of didReceiveMenuWillShowNotification in order to include Custom Action the first time the menu pops up. (Minor sample app bug)

[Current coverage][cc-pull] is 62.11% (diff: 0.00%)

Merging [#1759][cc-pull] into [develop][cc-base-branch] will decrease coverage by 0.06%

@@            develop      #1759   diff @@
==========================================
  Files            62         62
  Lines          2249       2251     +2
  Methods         621        621
  Messages          0          0
  Branches        136        136
==========================================
  Hits           1398       1398
- Misses          782        784     +2
  Partials         69         69

[][cc-pull]

Powered by Codecov. Last update [d73fcbf...01c723f][cc-compare]
[cc-base-branch]: https://codecov.io/gh/jessesquires/JSQMessagesViewController/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/jessesquires/JSQMessagesViewController/compare/d73fcbf8208786e49f3e676e759652bd6fc22a4a...01c723f3b2bf7938c9d934098ed7c7a627546eba?src=pr
[cc-pull]: https://codecov.io/gh/jessesquires/JSQMessagesViewController/pull/1759?src=pr

Thanks @Lucashuang0802 ! a few comments

The description you gave is more clear. Thx. @jessesquires

Thanks @Lucashuang0802 ! 🎉


CH4 feedback after QWR playthrough

CH4 Wards: Increase size of ward placement location
Change "Use the Dust" to "Use the Dust of Appearance".
Camera pan from player to enemy outpost: needs to start from current hero position.
Riki: "Yes I dont know I'm done" -> fix dont to don't
Camera pans after Well Played from cliff to KOTL.

I was about to change the size of ward placement location but this doesn't look so bad. Max distance from the marker right now is 150.

Camera pan from player to enemy outpost: needs to start from current hero position.

It starts from current camera position, it would be pretty weird if it snapped to the current hero's position first no?

Camera pans after Well Played from cliff to KOTL.
Also starts from current camera position. Could make it not pan at all to KOTL and Kunkka maybe?

I was about to change the size of ward placement location but this doesn't look so bad. Max distance from the marker right now is 150.

Weird, she tried to click very close to it and basically couldn't. Maybe she presses the wrong button or something? Idk. Seems good.

Camera pan from player to enemy outpost: needs to start from current hero position.

It starts from current camera position, it would be pretty weird if it snapped to the current hero's position first no?

Camera pans after Well Played from cliff to KOTL.

Also starts from current camera position. Could make it not pan at all to KOTL and Kunkka maybe?

Both cases I remember it had a snap in both cases. Is it a function in both cases? If it's not, it'll take the camera position that the player had at the section's start.

Both cases I remember it had a snap in both cases. Is it a function in both cases? If it's not, it'll take the camera position that the player had at the section's start.

Don't remember exactly the code for the build that she played, but right now it's changed to _ => getPlayerCameraLocation() for the starting position in #412  - think there were some setCameraTarget calls that might've caused the snapping.

Great, if that works as expected now then it's good.

Resolved all issues listed here


Training process bug

I have followed all the steps mentioned in the repo and the training went smooth enough until this point. Most probably it has a bug in rasat/seq2seq/utils/spider.py. Here is the error description:
ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (1034,) + inhomogeneous part.

Can you please help me to fix the issue?

After examining the spider.py, I have noticed the comment "# Replace -100 in the labels as we can't decode them.". Which value the label_ids should be replaced?


feat(clerk-js): Autoswitch to phone input if the only sign in initial value is the phone number

Description

Start with the phone input if the only sign in initial value is the phone number.

Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[x] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Small grammar fixes in GradleMigration.md

Some little tinythings that were a tiny bit confusing as I was reading the text. I thought this would improve the reading experience a little bit. Ignore if you don't think this adds any value 😊

Thanks so much, and sorry for a little sloppiness :)

It was just super minor things anyway, no worries. Thanks for accepting this 🤗


Create a utility class.

To calculate the distance between 2 points.

@jsor worked on a Geokit lib. It should be better to use Geocoder in his lib. Geokit is suitable to provide utility classes, Geocoder is not really designed for that.

Okay, i agree.
Closed.


Flutter macOS desktop: Freezes in release build mode

I'm creating a macOS desktop application. It works well in debug and profile mode, but when running in release mode it will work for 2-20 seconds and then freeze. When it freezes, there is no output or error messages of any kind - perhaps there would be if I ran it from Xcode, but I can't get release mode to work from Xcode at the moment (issue for another day). Resizing the app when it is frozen will simply stretch its last displayed frame as if it were a bitmap.
I've tried to reproduce this with other Flutter applications on macOS, but they seem to work fine, so it must be something specific to my application. I'd rather not share my code publicly at the moment, so if there's a way I can send it to a Flutter team member privately please let me know.

Logs
(no logs since I am running in release mode)
[✓] Flutter (Channel master, 1.21.0-2.0.pre.75, on Mac OS X 10.15.3 19D76, locale en-US)
    • Flutter version 1.21.0-2.0.pre.75 at /Users/ethanelshyeb/fluttersdk/flutter
    • Framework revision 3577da53ff (4 hours ago), 2020-07-20 09:54:50 -0700
    • Engine revision dcc9a4048d
    • Dart version 2.9.0 (build 2.9.0-21.0.dev 9dca49e71e)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/ethanelshyeb/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio 4.0 Preview.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.3.1, Build version 11C504
    • CocoaPods version 1.8.4

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio 4.0 Preview.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] IntelliJ IDEA Ultimate Edition (version 2019.3)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 41.1.5
    • Dart plugin version 193.5233.116

[✓] IntelliJ IDEA Community Edition (version 2017.2.6)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 20.0.2
    • Dart plugin version 172.4343.25

[!] VS Code (version 1.47.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    ✗ Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (3 available)
    • macOS (desktop)  • macos      • darwin-x64     • Mac OS X 10.15.3 19D76
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 84.0.4147.89

! Doctor found issues in 2 categories.

We need stack traces/errors/logs to make this issue actionable.

so if there's a way I can send it to a Flutter team member privately please let me know.

We cannot look at code that is not released under an open source license. I recommend trying to come up with a minimal reproduction that does not depend on your application and sharing that publicly

@jonahwilliams Ok, I traced it to the following code:
class AstWidget extends StatefulWidget {
  const AstWidget();

  @override
  _AstWidgetState createState() => _AstWidgetState();
}

class _AstWidgetState extends State<AstWidget> {
  @override
  Widget build(BuildContext context) {
    Future.delayed(
      Duration(milliseconds: 4),
      () {
        final el = toDiagnosticsNode(name: "astWidget").getProperties()[2] as ObjectFlagProperty<StatefulElement>;
        print((el.value.renderObject as RenderBox).localToGlobal(Offset.zero));
      },
    );
    /// other stuff here that doesn't matter
    return Text("No widget");
  }
}

This is clearly a strange code pattern and it's easy to see why it might be invalid. That said, it's strange that it works in both debug and profile mode, as well as for at least a few frames of release mode, but then fails.

Why don't you wrap that logic in a try/catch with a print to see what is failing?

I'm almost certain this freeze is just a regular UI bug. If your widget has properties that are excluded from release mode you'll get some amount of NoSuchMethodError or  otherrwise

The problem is that since I'm running in release mode (and the problem doesn't occur in debug or profile mode), I can't see the output of print() or any exceptions that are logged. Unless there's a way to do that that I'm not aware of.

since I'm running in release mode [...] I can't see the output of print()

Why not?

Hi @ethanblake4
I cannot reproduce the freeze issue in release mode even after 20 sec

Code Sample
import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(
        child: AstWidget(),
      ),
      floatingActionButton: FloatingActionButton.extended(
          label: Text('next'),
          icon: Icon(Icons.arrow_right),
          onPressed: () {
            Navigator.push(
                context, MaterialPageRoute(builder: (_) => Second()));
          }),
    );
  }
}

class AstWidget extends StatefulWidget {
  const AstWidget();

  @override
  _AstWidgetState createState() => _AstWidgetState();
}

class _AstWidgetState extends State<AstWidget> {
  @override
  Widget build(BuildContext context) {
    Future.delayed(
      Duration(milliseconds: 4),
      () {
        final el = toDiagnosticsNode(name: "astWidget").getProperties()[2]
            as ObjectFlagProperty<StatefulElement>;
        print((el.value.renderObject as RenderBox).localToGlobal(Offset.zero));
      },
    );

    /// other stuff here that doesn't matter
    return Text("No widget");
  }
}

class Second extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(
        child: Container(
          child: Text('Hello World'),
        ),
      ),
      floatingActionButton:
          FloatingActionButton(child: Icon(Icons.add), onPressed: () {}),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel master, 1.21.0-2.0.pre.83, on Mac OS X 10.15.6 19G73, locale en-GB)
    • Flutter version 1.21.0-2.0.pre.83 at /Users/taha/Code/flutter_master
    • Framework revision 9e658bc3d5 (6 hours ago), 2020-07-21 03:56:07 +0200
    • Engine revision dcc9a4048d
    • Dart version 2.9.0 (build 2.9.0-21.0.dev 9dca49e71e)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.47.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.2

[✓] Connected device (4 available)
    • SM M305F (mobile) • 32003c30dc19668f • android-arm64  • Android 10 (API 29)
    • macOS (desktop)   • macos            • darwin-x64     • Mac OS X 10.15.6 19G73
    • Web Server (web)  • web-server       • web-javascript • Flutter Tools
    • Chrome (web)      • chrome           • web-javascript • Google Chrome 84.0.4147.89

• No issues found!

@TahaTesser did you try resizing the window? you need to make sure that the widget is forced to rebuild

Hi @ethanblake4
Yes, i cannot reproduce the issue even after window resizes and rebuilding the widget

flutter doctor -v
[✓] Flutter (Channel master, 1.21.0-2.0.pre.94, on Mac OS X 10.15.6 19G73,
    locale en-GB)
    • Flutter version 1.21.0-2.0.pre.94 at /Users/taha/Code/flutter_master
    • Framework revision adc9dde3ba (10 hours ago), 2020-07-21 16:17:23 -0700
    • Engine revision 5a73d4dc25
    • Dart version 2.9.0 (build 2.9.0-21.0.dev 84e5eeaea4)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.47.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.12.2

[✓] Connected device (5 available)
    • SM M305F (mobile)      • 32003c30dc19668f          • android-arm64  •
      Android 10 (API 29)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            • iOS
      13.6
    • macOS (desktop)        • macos                     • darwin-x64     • Mac
      OS X 10.15.6 19G73
    • Web Server (web)       • web-server                • web-javascript •
      Flutter Tools
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 84.0.4147.89

• No issues found!

@TahaTesser Sorry for the delay. can you try wrapping it in a LayoutBuilder like this:
child: LayoutBuilder(builder: (context, c) {
    return AstWidget();
})

This way it is forced to rebuild every frame.
Then resize the window for a few seconds and it should freeze.

Hi @ethanblake4
Added LayoutBuilder the code sample, still not able to reproduce this issue with lots of resizes and setState

Code Sample
import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(child: LayoutBuilder(
        builder: (context, c) {
          return AstWidget();
        },
      )),
      floatingActionButton: FloatingActionButton.extended(
          label: Text('next'),
          icon: Icon(Icons.arrow_right),
          onPressed: () {
            Navigator.push(
                context, MaterialPageRoute(builder: (_) => Second()));
          }),
    );
  }
}

class AstWidget extends StatefulWidget {
  const AstWidget();

  @override
  _AstWidgetState createState() => _AstWidgetState();
}

class _AstWidgetState extends State<AstWidget> {
  @override
  Widget build(BuildContext context) {
    Future.delayed(
      Duration(milliseconds: 4),
      () {
        final el = toDiagnosticsNode(name: "astWidget").getProperties()[2]
            as ObjectFlagProperty<StatefulElement>;
        print((el.value.renderObject as RenderBox).localToGlobal(Offset.zero));
      },
    );

    /// other stuff here that doesn't matter
    return Text("No widget");
  }
}

class Second extends StatefulWidget {
  @override
  _SecondState createState() => _SecondState();
}

class _SecondState extends State<Second> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(
        child: Container(
          child: Text('Hello World'),
        ),
      ),
      floatingActionButton: FloatingActionButton(
          child: Icon(Icons.add),
          onPressed: () {
            setState(() {});
          }),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel master, 1.21.0-6.0.pre.160, on Mac OS X 10.15.6 19G73,
    locale en-GB)
    • Flutter version 1.21.0-6.0.pre.160 at /Users/taha/Code/flutter_master
    • Framework revision 31ee51a302 (8 hours ago), 2020-08-03 17:27:21 -0700
    • Engine revision fa16c3d0ba
    • Dart version 2.10.0 (build 2.10.0-2.0.dev 0f0e04ec3a)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-30, build-tools 30.0.1
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 48.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.47.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.13.1

[✓] Connected device (5 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E            • ios
      • iOS 13.6
    • iPhone 11 (mobile)     • 1EB615AF-5AB9-4AAA-A587-A6642BB79484 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-13-6 (simulator)
    • macOS (desktop)        • macos                                • darwin-x64
      • Mac OS X 10.15.6 19G73
    • Web Server (web)       • web-server                           •
      web-javascript • Flutter Tools
    • Chrome (web)           • chrome                               •
      web-javascript • Google Chrome 84.0.4147.105

• No issues found!

I cannot reproduce this issue on latest master channel.
Can you upgrade to master and try again
flutter channel master
flutter upgrade --force
flutter doctor -v

Without additional information we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Feature Request: Key-based readonly flag.

Is your feature request related to a problem? Please describe.
My motivation underneath is to lock interpolations so that they cannot be overwritten.
A general form would be to implement key-based read-only checks.
Describe the solution you'd like
I could think of three solutions 2) is likely the easiest to implement, 1) while more limited is more straight forward, and 3) needs to build up on something first.

A readonly_interpolations flag on nodes.
Maybe a bit more involved: If the flag is set check on write if the current value is an interpolation if so raise.

Or the more general and probably most simplest solution that adds a set to check to the flags.

def set_readonly_keys(conf, keys:
    conf._set_flag("readonly_keys", [keys])

# And checked at the same places:
# replace:
if self._get_flag("readonly"): raise ...`
# with
if self._get_flag("readonly") or key in self._get_flag("readonly_keys"): raise ...

Future: On top of 2) metadata could be used to indicate readonly keys:

protected_value : float = field(default=SI("..."), metadata={"omegaconf_readonly_key"})

Do you think 2) is a possible solution?

You can try using something like:
def set_readonly_keys(conf, keys):
    for key in keys:
        OmegaConf.set_readonly(conf._get_node(key), True)

Or to make all interpolations read-only automatically:
def set_readonly_interpolations(conf):
    if conf._is_interpolation():
        OmegaConf.set_readonly(conf, True)
    elif isinstance(conf, DictConfig):
        for key in conf:
            set_readonly_interpolations(conf._get_node(key))
    elif isinstance(conf, ListConfig):
        for key in range(len(conf)):
            set_readonly_interpolations(conf._get_node(key))

Unfortunately this requires accessing some internal API, maybe OmegaConf should expose some public API to make it more convenient.

Thank you works great!
Close this issue if you like or if you prefer to keep it open.

Small addition to the set_readonly_keys would be to support dotlist style.

Close this issue if you like or if you prefer to keep it open.

I'll keep it open as I think it'd be good to make this possible without accessing the internal API.

Small addition to the set_readonly_keys would be to support dotlist style

This can probably be done by using select_node() (from omegaconf._impl import select_node), instead of _get_node().


Link to new page

Add the ability to specify that the icon should open it's link in a new page

there's already capability to open link in new tab, simply pass target="_blank" to the SocialIcon component. See https://github.com/jaketrent/react-social-icons/issues/111

Could this please be added to the documentation on the NPM site? I had to hunt around for this and was thinking about opening a PR myself


helper.transform_asg_style_tags() not protected when resource['values'] is None

Description

To Reproduce
The issue is a bit flaky. It doesn't look directly related with a specific resource (we are using terraform-compliance on a high number of cicd pipelines and this issue is only affecting a few of them)
Example: when generating a plan that applies the whole infrastructure from scratch it doesn't fail, however when the plan includes updates, terraform-compliance fails with the Exception:
        | Name | .+    |
          AttributeError: 'NoneType' object has no attribute 'get'

Feature File:
https://github.com/joaoubaldo/cli/blob/issue-567-handle-none-resource-values/tests/functional/test_issue-567/test.feature
Plan File:
https://github.com/joaoubaldo/cli/blob/issue-567-handle-none-resource-values/tests/functional/test_issue-567/plan.out.json
Used terraform-compliance Parameters:
I was able to isolate the issue by running radish directly with the parameters terraform-compliance uses.
bin/radish --write-steps-once /feature/directory/ --basedir /terraform_compliance/steps --user-data=plan_file=my-plan.tfplan.json --user-data=exit_on_failure=False --user-data=terraform_executable=None --user-data=no_failure=False --user-data=silence_mode_enabled=False --user-data=debugging_mode_enabled=False
Running via Docker:
No
Error Output:
        | Name | .+    |
          AttributeError: 'NoneType' object has no attribute 'get'

Expected Behavior:
Failed / passed scenarios without Exceptions.
Tested Versions:

terraform-compliance version: 1.3.29
terraform version: 1.0.2
python version: 3.9.7

Additional Context:
From what I was able to debug I have found a potential issue here
https://github.com/terraform-compliance/cli/blob/811d1785da5281901726cf0b627f53dbdb244deb/terraform_compliance/common/helper.py#L507)
When resource['values'] is None, the original Exception will be thrown.
I confirm that having a None protection here fixes my problem but I am not certain that having resource['values'] as None at this point means the issue should be fixed somewhere else first.
I'll create a PR and reference the issue.

Thanks a lot for the PR 🎉
Looks great.