Add support for currently unsupported JSON/JSONB operators

Diesel currently supports the postgres json and jsonb types. We do not provide built-in support for various operators available for these types. This is a tracking issue for adding support for these operators.
The general strategy for adding support for new operators is as following:

Define the operator via infix_operator!(). The contains operator  would be defined as following: infix_operator!(KeyExists, " ? ", backend: Pg); These operators can be defined here. If there is already an existing definition, this step could be skipped.
Define a new helper type here. Helper types are a tool for diesel users to simplify type definitions. Again if there is already a matching type definition this could be skipped.
Add a new ExpressionMethods extension trait for the corresponding types. This only needs to happen once for jsonb   and once for jsonb + json.  See the PgRangeExpressionMethods trait here as example.
Add a new method to the corresponding ExpressionMethods trait. See here for an example for a different operator/type. This definition should:

Use the previously defined helper type
Contain a documentation with an example

Check if compile test output is up to date by running TRYBUILD=overwrite cargo test inside of diesel_compile_tests. (This is required if you've added a new operator based on infix_operator!
Submit a PR with the change

Operator list:

[ ] json/jsonb -> integer (indexing by index, returns json/jsonb) **
[ ] json/jsonb -> text (indexing by key name, returns json/jsonb) **
[ ] json/jsonb ->> integer (indexing by index, returns field as text) **
[ ] json/jsonb ->> text (indexing by key name, returns field as text) **
[ ] json/jsonb #> text[] (indexing by path, returns field as json/jsonb) *
[ ] json/jsonb #>> text[] (indexing by path, returns field as text) *
[ ] jsonb @> jsonb (checks if the second value is contained by the first value, returns a boolean) *
[ ] jsonb <@ jsonb (checks if the first value is contained by the second value, returns a boolean) *
[x] jsonb ? text (checks if a string exists as top level key in the given json, returns a boolean) *
[ ] jsonb ?| text[] (checks if any of the strings exists as top level key in the given json, returns a boolean) *
[ ] jsonb ?& text[] (checks if all of the strings exist as top level key in the given json, returns a boolean) *
[x] jsonb || jsonb (concats two json values, returns jsonb) *
[ ] jsonb - text (remove the key given as string from the json, returns jsonb) **
[ ] jsonb - text[] (removes all keys given as string from the json, returns jsonb) **
[ ] jsonb - integer (removes the field with the given index from the json, returns jsonb) **
[ ] jsonb #- text[] (removes the given path from the json, returns jsonb) *
[ ] jsonb @? jsonpath (Does JSON path return any item for the specified JSON value?, returns a boolean) ***
[ ] jsonb @@ jsonpath (Returns the result of a JSON path predicate check for the specified JSON value. Only the first item of the result is taken into account. If the result is not Boolean, then NULL is returned., returns a boolean`) ***

Items marked with one * can follow the instructions directly, items marked with ** require additional work to unify the different input types behind a common operator/function call (See here for a general example on how to abstract over expressions of different types), items marked with *** likely require additional design work.
Please add a comment to this issue if you plan to work on a specific operator.
If there is anything unclear about how to add a support for a specific operator just ask and we will try to answer your questions.

Believe it or not, I was considering suggesting this some time last week!
I will gladly take this task up for the one asterisk operators and maybe this work will evolve into discussion on other operators.

@czotomo Any new feature should target the master branch.

Jsonb concat PR: https://github.com/diesel-rs/diesel/pull/3035

Alright, let's continue with the trivial ones.
I'm going to implement
jsonb ? text

Alright, I'll implement the array and object indexing operators by ints and strings, I don't see a reason to split this work.
json/jsonb -> integer => json/jsonb
json/jsonb -> text => json/jsonb
json/jsonb ->> integer  => text
json/jsonb ->> text => text

Hey, I'd like to get my hands dirty with these two operators if that's ok..
jsonb ?| text[]
jsonb ?& text[]

@czotomo You can likely combine these two operations

json/jsonb -> integer => json/jsonb
json/jsonb -> text => json/jsonb

in one function by using a similar trick as for implementing the PgJsonbExxpressionMethods trait for jsonb + nullable jsonb. To do that you can use a generic type for the sqltype passed into AsExpression and restrict that type to Integer or Text by using am auxiliary trait.
(Also these operators need there own ExpressionMethods trait as these need to be implemented for the Json type as well. Again thats possible by using a corresponding auxiliary trait implemented for jsonb and json + their nullable variants.)
@mrbuzz :+1: go for it

Hi, I'd like to try adding support for these two operators.
jsonb @> jsonb
jsonb <@ jsonb

@czotomo Would it be possible to team up on this? Quite new to Rust, but have plenty of development experience. I'd be happy with just being a fly on the wall, and just see how the problem is solved.

@gamesbrainiac sure, please reach out to me on Diesel's gitter so we can sync

I'd like to take these three operators as my next task:
jsonb - text
jsonb - text[]
jsonb - integer
I might need some guidance as I'm relatively new to Rust, I'll start by looking at the example code

Me and @czotomo have made some progress, but we're a little stick. If @mrbuzz wants to join in, we plan of having another crack at it tomorrow.

@gamesbrainiac @czotomo Is there any information that I could provide to help you? As additional side note: Opening a WIP PR could also be a good idea, as this makes it easier for others to comment on specific code issues, as they have access to the whole change.

@mrbuzz I'm happy to answer any question that comes up. Just ask .

I'll try these operators next.
json/jsonb #> text[]
json/jsonb #>> text[]

Me and @czotomo have made some progress, but we're a little stuck. If @mrbuzz wants to join in, we plan of having another crack at it tomorrow.

That'd be cool! Thanks for the invite. At what time are you meeting? I am a little short on time today but I'll see what  I can do

@zaneduffield Yes those would need marker traits for json/jsonb. So basically this thing here: https://github.com/diesel-rs/diesel/blob/b479e8352f6712bbc2f6bdcd734c8e32a934e6b9/diesel/src/pg/expression/expression_methods.rs#L1554-L1564
but so that the helper trait (JsonbOrNullableJsonb) is implemented for Json and Nullable<Json> as well.

This is the progress we've made so far. https://github.com/gamesbrainiac/diesel/tree/master
We tried the tests, but it keeps failing. It should work, but it's hard to figure out what's wrong.
@mrbuzz Do you have gitter? We usually communicate using that.

@weiznich is there a diesel development / maintenance Gitter room where people who know ins and outs hang out, or are the conversations strictly performed in issues' and PRs' comments?

@gamesbrainiac
The following change fixes the test:
@@ -1363,7 +1363,7 @@ pub trait PgAnyJsonExpressionMethods: Expression + Sized {
     ///     .values((name.eq("Claus"), address.eq(&santas_address)))
     ///     .execute(conn)?;
     ///
-    /// let santas_postcode = contacts.select(address.retrieve_as_object("postcode")).get_result::<serde_json::Value>(conn)?;
+    /// let santas_postcode = contacts.select(address.retrieve_as_object::<diesel::sql_types::Text, _>("postcode")).get_result::<serde_json::Value>(conn)?;
     /// assert_eq!(santas_postcode, serde_json::json!("99705"));
     ///
     ///
@@ -1388,7 +1388,8 @@ pub trait PgAnyJsonExpressionMethods: Expression + Sized {
     ///
     /// let roberts_second_address_in_db = contacts
     ///                             .filter(name.eq("Robert Downey Jr."))
-    ///                             .select(address.retrieve_as_object(1))
+    ///                             .select(address.retrieve_as_object::<
+    ///                             diesel::sql_types::Integer, _>(1))
     ///                             .get_result::<serde_json::Value>(conn)?;
     ///
     /// let roberts_second_address = serde_json::json!({
@@ -1405,9 +1406,9 @@ pub trait PgAnyJsonExpressionMethods: Expression + Sized {
     /// #     Ok(())
     /// # }
     /// ```
-    fn retrieve_as_object<T, ST>(self, other: T) -> dsl::RetrieveAsObjectJsonb<Self, T, ST>
+    fn retrieve_as_object<ST, T>(self, other: T) -> dsl::RetrieveAsObjectJsonb<Self, T, ST>
     where
-        ST: SqlType + TypedExpressionType,
+        ST: SqlType + TypedExpressionType + TextOrInteger,
         T: AsExpression<ST>,
     {
         Grouped(RetrieveAsObjectJsonb::new(self, other.as_expression()))
@@ -1432,6 +1433,5 @@ impl<T> PgAnyJsonExpressionMethods for T
 where
     Self::SqlType: JsonOrNullableJsonOrJsonbOrNullableJsonb,
     T: Expression,
-    T::SqlType: TextOrInteger,
 {
 }

To explain that a bit:
The change in line 1432 (removing T::SqlType: TextOrInteger) fixes the problem that the trait is not implemented for any jsonb column. That happens because that condition reads like: If the column is (a json or jsonb or nullable json or nullable jsonb) and (text or integer). That something that cannot be true at any point of time, as a column has just one sql type. What we cant here is to assert that the left side of the expression is JsonOrNullableJsonOrJsonbOrNullableJsonb while the right side is TextOrInteger. So we remove the trait bound here and add it in line 1405, where other represents the right side of the expression.
The other changes work around the fact that rustc cannot infer the correct sql type for &str and integer, so that they must be specified directly. That's a bit unfortunate, as this makes the current interface hard to use. I would like to hear opinions here so that we hopefully can find a better solution.
@czotomo It's fine to ask these question in the normal gitter room, but if there is need for that we can just create a new room for that.

@mrbuzz I'm happy to answer any question that comes up. Just ask .

Just opened a draft PR #3044 for the remove key operators. I used an extra generic parameter to carry the input type as @gamesbrainiac and @czotomo did in their implementation.

This is the progress we've made so far. gamesbrainiac@3e857bb
We tried the tests, but it keeps failing. It should work, but it's hard to figure out what's wrong.
@mrbuzz Do you have gitter? We usually communicate using that.
Yep! Same name as on Github. Feel free to ping me any time

I just want to leave a comment here that I'm away for this week. So don't expect prompt answers from me. I will try yo answer the outstanding questions next week.

@mrbuzz @gamesbrainiac @czotomo Sorry for being away so long. I've pushed #3092 which contains all of your work + some fixes to the operators. Additionally it adds the missing operators. Feel free to leave comments about potential improvements on that PR.
Thanks again for your work :heart:

@weiznich should these operators be mentioned in the changelog for version 2?

@czotomo Yes that would be great


SVG images - do not generate thumbnail if getImagineImage() returns false

A user on the forum reported that using an SVG image caused errors when upgrading from 8.0.3 to 8.1.0.
"after upgrading my site from 8.0.3, where everything works fine, to 8.1.0 i got the following error on every page, including dashboard:
Call to a member function getSize() on boolean"
https://www.concrete5.org/community/forums/installation/error-after-upgrade-from-8.0.3-to-8.1.0/

I was able to reproduce the error when uploading SVG images through the file manager.
Steps to reproduce:

upload an SVG image in the file manager
this will trigger an "Upload Failed" error modal with the filename and Call to a member function getSize() on boolean error
close the error modal
refresh the page
the SVG image is uploaded and listed, but has a size of 0.00 KB

When an SVG image is uploaded/rescanned and ImageMagick is not available, then getImagineImage() returns false:
https://github.com/concrete5/concrete5/blob/develop/concrete/src/Entity/File/Version.php#L997

I've upgraded from 5.7 to 8 and I've on every page the "Call to a member function getSize() on boolean" error.
Any idea how I can find the file is causing the error?

Has this been pulled into the main project yet? I've done a bunch of research and there seems to be no way at all to fix Concrete5 when this happens aside from reinstalling the framework, which is VERY excessive!
Is there any other way to resolve this issue in the newer versions or merely wait for the core fix?

@MarcDBoss
This pull request was merged into the develop branch and will be in the next release.
After making a full backup (database and files), have you tried applying this fix manually?

I couldn't find any resources on handling the error manually I had to
completely uninstall the latest version. We will be trying to proceed with
the legacy version to avoid anything like this in our productions code.
On Mar 2, 2017 12:45 AM, "Karl Dilkington"<EMAIL_ADDRESS>wrote:

@MarcDBoss https://github.com/MarcDBoss
This pull request was merged into the develop branch and will be in the
next release.
After making a full backup (database and files), have you tried applying
this fix manually?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/concrete5/concrete5/pull/5069#issuecomment-283562536,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH2B39xndPD8BAL-D6PzQKnTreLh9ChYks5rhldtgaJpZM4L3ZfA
.

Make sure you have GD and/or imagick installed, without these PHP can't process images at all.


design a proper 3d model of iPhone 16 Pro

you can use any platform to design but as per requirement we will use .gltf or .glb ,so work accordingly.

Hi @NiranjanKumar001
Can you please assign me this

Kindly assign to me.

done

Thanks!


KYLIN-4315 use metadata numRows in beeline client for quick row counting

see https://issues.apache.org/jira/browse/KYLIN-4315

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@c58c9bf). Click here to learn what that means.
The diff coverage is 0%.

@@           Coverage Diff            @@
##             master   #1024   +/-   ##
========================================
  Coverage          ?   25.1%
  Complexity        ?    6179
========================================
  Files             ?    1433
  Lines             ?   87269
  Branches          ?   12218
========================================
  Hits              ?   21910
  Misses            ?   63210
  Partials          ?    2149

Impacted Files
Coverage Δ
Complexity Δ

...apache/kylin/source/hive/HiveTableMetaBuilder.java
0% <0%> (ø)
0 <0> (?)

...rg/apache/kylin/source/hive/BeelineHiveClient.java
4% <0%> (ø)
5 <0> (?)

...va/org/apache/kylin/source/hive/HiveTableMeta.java
0% <0%> (ø)
0 <0> (?)

...va/org/apache/kylin/source/hive/CLIHiveClient.java
0% <0%> (ø)
0 <0> (?)

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 c58c9bf...9e7c478. Read the comment docs.

Check list

[x] Step 1. [OPTIONAL]Understand background/root cause/design basically(one hour EST) . Its issue type is

[ ] bug fix
[ ] new feature
[x] enhancement

[x] Step 2. Test cases are designed and documented (30 minutes EST).
[x] Step 3. Prepare specific env, for example:

mock data (maybe some specific data type)
test env (maybe install a RDBMS instance).

[x] Step 4. Verify and make sure test cases passed.
[x] Step 5. Paste manual important steps and screenshots here (20 minutes EST).

If you find difficulty in pick most important evidence, please attach diagnosis package.

[x] Step 6. Do more check in user perspective (20 minutes EST)

[x] Doc need be updated? And if it is updated? Ask help for release manager if so.
[x] If it is a breaking change so we should notify Kylin community? Ask help for release manager if so.

[x] Step 7.  Summarize this test (20 minutes EST).

Total estimate

4 hours for small issue from optimistic estimate.

Note
If you find some unexpected and unrelated error/mistake, please DO report it if it is truly a mistake, I think we may research and fix it in the future.
If you find background information/root cause analysis is not complete or ambiguous, please try to contact to author or do a quick research and record what you find. It is a good chance to learn something interesting.
If you find it is not easy to design testcase, please notify release manager.

Design test cases
Prepare test env

create cube and build segment (2012-01-01 ~ 2013-01-01)

dimension : LSTG_FORMAT_NAME, ops_region, part_dt
measure : sum(price)

Case-1
Make sure query-1 and query-2 has the same result.

query-1 (Hive)

select LSTG_FORMAT_NAME, stddev_pop(SUM_M)
from (
      select LSTG_FORMAT_NAME, ops_region, sum(price) as SUM_M
      from KYLIN_SALES
      where part_dt >= '2012-01-01' and part_dt < '2012-06-01'
      group by LSTG_FORMAT_NAME, ops_region
) a
group by LSTG_FORMAT_NAME
order by LSTG_FORMAT_NAME

query-2 (Kylin)

select LSTG_FORMAT_NAME, stddev_sum(ops_region, price)
from KYLIN_SALES
where part_dt >= '2012-01-01' and part_dt < '2012-06-01'
group by LSTG_FORMAT_NAME
order by LSTG_FORMAT_NAME

Case-2
Make sure query-1 and query-2 has the same result.

query-1 (Hive)

select stddev_pop(SUM_M)
from (
      select LSTG_FORMAT_NAME, ops_region, sum(price) as SUM_M
      from KYLIN_SALES
      where part_dt >= '2012-01-01' and part_dt < '2012-06-01'
      group by LSTG_FORMAT_NAME, ops_region
) a

query-2(Kylin)

select stddev_sum(ops_region, price)
from KYLIN_SALES
where part_dt >= '2012-01-01' and part_dt < '2012-06-01'

Prepare data and env

[x] Built on commit_SHA : d14eef5fee3ef5a305b35b9121bde28fc2400d9d

[x] Hadoop env : CDH 5.7

[x] Just sample cube is OK.

Verify test case
Case 1

Result of Kylin

Result of Hive

DROP TABLE IF EXISTS kylin_intermediate_kylin_sales_cube_f91987ee_443f_cadb_f3cb_04accea358b6;
CREATE EXTERNAL TABLE IF NOT EXISTS kylin_intermediate_kylin_sales_cube_f91987ee_443f_cadb_f3cb_04accea358b6
(
`KYLIN_SALES_TRANS_ID` bigint
,`KYLIN_SALES_PART_DT` date
,`KYLIN_SALES_LEAF_CATEG_ID` bigint
,`KYLIN_SALES_LSTG_SITE_ID` int
,`KYLIN_CATEGORY_GROUPINGS_META_CATEG_NAME` string
,`KYLIN_CATEGORY_GROUPINGS_CATEG_LVL2_NAME` string
,`KYLIN_CATEGORY_GROUPINGS_CATEG_LVL3_NAME` string
,`KYLIN_SALES_LSTG_FORMAT_NAME` string
,`KYLIN_SALES_SELLER_ID` bigint
,`KYLIN_SALES_BUYER_ID` bigint
,`BUYER_ACCOUNT_ACCOUNT_BUYER_LEVEL` int
,`SELLER_ACCOUNT_ACCOUNT_SELLER_LEVEL` int
,`BUYER_ACCOUNT_ACCOUNT_COUNTRY` string
,`SELLER_ACCOUNT_ACCOUNT_COUNTRY` string
,`BUYER_COUNTRY_NAME` string
,`SELLER_COUNTRY_NAME` string
,`KYLIN_SALES_OPS_USER_ID` string
,`KYLIN_SALES_OPS_REGION` string
,`KYLIN_SALES_PRICE` decimal(19,4)
)
STORED AS SEQUENCEFILE
LOCATION 'hdfs://sandbox.hortonworks.com:8020/kylin/kylin_metadata_wrp_310/kylin-c9804baf-044f-4537-132a-81ce89d76e10/kylin_intermediate_kylin_sales_cube_f91987ee_443f_cadb_f3cb_04accea358b6';
ALTER TABLE kylin_intermediate_kylin_sales_cube_f91987ee_443f_cadb_f3cb_04accea358b6 SET TBLPROPERTIES('auto.purge'='true');
INSERT OVERWRITE TABLE `kylin_intermediate_kylin_sales_cube_f91987ee_443f_cadb_f3cb_04accea358b6` SELECT
`KYLIN_SALES`.`TRANS_ID` as `KYLIN_SALES_TRANS_ID`
,`KYLIN_SALES`.`PART_DT` as `KYLIN_SALES_PART_DT`
,`KYLIN_SALES`.`LEAF_CATEG_ID` as `KYLIN_SALES_LEAF_CATEG_ID`
,`KYLIN_SALES`.`LSTG_SITE_ID` as `KYLIN_SALES_LSTG_SITE_ID`
,`KYLIN_CATEGORY_GROUPINGS`.`META_CATEG_NAME` as `KYLIN_CATEGORY_GROUPINGS_META_CATEG_NAME`
,`KYLIN_CATEGORY_GROUPINGS`.`CATEG_LVL2_NAME` as `KYLIN_CATEGORY_GROUPINGS_CATEG_LVL2_NAME`
,`KYLIN_CATEGORY_GROUPINGS`.`CATEG_LVL3_NAME` as `KYLIN_CATEGORY_GROUPINGS_CATEG_LVL3_NAME`
,`KYLIN_SALES`.`LSTG_FORMAT_NAME` as `KYLIN_SALES_LSTG_FORMAT_NAME`
,`KYLIN_SALES`.`SELLER_ID` as `KYLIN_SALES_SELLER_ID`
,`KYLIN_SALES`.`BUYER_ID` as `KYLIN_SALES_BUYER_ID`
,`BUYER_ACCOUNT`.`ACCOUNT_BUYER_LEVEL` as `BUYER_ACCOUNT_ACCOUNT_BUYER_LEVEL`
,`SELLER_ACCOUNT`.`ACCOUNT_SELLER_LEVEL` as `SELLER_ACCOUNT_ACCOUNT_SELLER_LEVEL`
,`BUYER_ACCOUNT`.`ACCOUNT_COUNTRY` as `BUYER_ACCOUNT_ACCOUNT_COUNTRY`
,`SELLER_ACCOUNT`.`ACCOUNT_COUNTRY` as `SELLER_ACCOUNT_ACCOUNT_COUNTRY`
,`BUYER_COUNTRY`.`NAME` as `BUYER_COUNTRY_NAME`
,`SELLER_COUNTRY`.`NAME` as `SELLER_COUNTRY_NAME`
,`KYLIN_SALES`.`OPS_USER_ID` as `KYLIN_SALES_OPS_USER_ID`
,`KYLIN_SALES`.`OPS_REGION` as `KYLIN_SALES_OPS_REGION`
,`KYLIN_SALES`.`PRICE` as `KYLIN_SALES_PRICE`
 FROM `DEFAULT`.`KYLIN_SALES` as `KYLIN_SALES`
INNER JOIN `DEFAULT`.`KYLIN_CAL_DT` as `KYLIN_CAL_DT`
ON `KYLIN_SALES`.`PART_DT` = `KYLIN_CAL_DT`.`CAL_DT`
INNER JOIN `DEFAULT`.`KYLIN_CATEGORY_GROUPINGS` as `KYLIN_CATEGORY_GROUPINGS`
ON `KYLIN_SALES`.`LEAF_CATEG_ID` = `KYLIN_CATEGORY_GROUPINGS`.`LEAF_CATEG_ID` AND `KYLIN_SALES`.`LSTG_SITE_ID` = `KYLIN_CATEGORY_GROUPINGS`.`SITE_ID`
INNER JOIN `DEFAULT`.`KYLIN_ACCOUNT` as `BUYER_ACCOUNT`
ON `KYLIN_SALES`.`BUYER_ID` = `BUYER_ACCOUNT`.`ACCOUNT_ID`
INNER JOIN `DEFAULT`.`KYLIN_ACCOUNT` as `SELLER_ACCOUNT`
ON `KYLIN_SALES`.`SELLER_ID` = `SELLER_ACCOUNT`.`ACCOUNT_ID`
INNER JOIN `DEFAULT`.`KYLIN_COUNTRY` as `BUYER_COUNTRY`
ON `BUYER_ACCOUNT`.`ACCOUNT_COUNTRY` = `BUYER_COUNTRY`.`COUNTRY`
INNER JOIN `DEFAULT`.`KYLIN_COUNTRY` as `SELLER_COUNTRY`
ON `SELLER_ACCOUNT`.`ACCOUNT_COUNTRY` = `SELLER_COUNTRY`.`COUNTRY`
WHERE 1=1 AND (`KYLIN_SALES`.`PART_DT` >= '2012-01-01' AND `KYLIN_SALES`.`PART_DT` < '2012-02-01')
;
select count(*) from kylin_intermediate_kylin_sales_cube_f91987ee_443f_cadb_f3cb_04accea358b6;


Tag push event

Started work on tag push event

Hi guys, do you known where this version will be merged to master ?

Hi guys, do you known when this version will be merged to master ?


Ignoring tsconfig.json in Angular 2 CLI generated apps

From @ncrawlins on May 8, 2016 15:53

VSCode Version: 1.0.0
OS Version: Windows 10

Maybe it's a problem with the project setup, but VSCode doesn't honor the experimentalDecorators flag and other tsconfig settings for any file you open AFTER the first file in an Angular CLI generated project.  You can close all files and wait a second and then open the files in a different order and the ones reporting errors will switch.
Steps to Reproduce:

Install angular 2 cli from NPM
use "ng new test" to create a new project.
Open the created directory in code.  The first TS file you open won't have any problems but all the ones you open after that won't use the tsconfig.json.

Copied from original issue: Microsoft/vscode#6200

The problem is the generated tsconfig.json file. It contains a files section and list only two files main and typings.d.ts. Therefore the files under source aren't considered part of the project defined by the tsconfig.json file.
Removing the files section makes it work as expected. Opened https://github.com/angular/angular-cli/issues/684.
I will also move this to the TS team since they should already issue an error on the first opened file in the src folder.

I'm not sure it's quite that simple.  Aren't files referenced by main.ts or it's dependencies also supposed to be included under that configuration? Because in that image the VtcnetClientAppComponent is included in main.ts and yet it will still occasionally shows errors.
For example, in a larger project, if every single file I open is a dependency of main.ts, everything works fine. No matter how many times I open and close a file it never shows errors.  However, if I open any new files that aren't in the dependency chain yet, then I will sporadically see those errors.  And from that point on I will sporadically see those errors on all files, even ones included in the dependency chain of main.ts that shouldn't be showing the error.  I hope that makes sense.

I will sporadically see those errors on all files, even ones included in the dependency chain of main.ts that shouldn't be showing the error. I hope that makes sense.

can you provide steps to reproduce the issue?

//cc: @alexeagle,
@alexeagle why does the template include "files" and not just include all files in the directory?

@mhegazy
the github at this repo is a very simple example.  https://git.io/vrfeN

Open the repo in VS Code.  Because main.ts imports A, B, C and D the es6 target is applied to all files as you navigate between them and no errors are shown.
Comment out all references to objD in main.ts and reload VS Code.
I would expect errors only in objD since it is no longer implicitly in the files list.  However, switching between files will show errors "randomly" on ALL the dependencies.  Sometimes revisiting a file or closing a file and hitting the refresh button can change the error status for a file despite making no changes.

@zhengbli can you take a look

I can't repro either of the issues. @ncrawlins @dbaeumer can you test using the TS nightly?

@zhengbli
With the nightly build it's better but it still happens in a special case.   View main.ts but don't have it in your working files.  Then click on any of the objX module files.  It will show an incorrect error.  If you click away from that objX file (but not into main.ts) and then back to it, the error will be gone.
If main.ts is in your list of working files, everything seems to work fine.

@hansl for the decision about including only the entry points in "files". I think this is just a tradeoff vs. exclude of node_modules, typings/main, typings/main.d.ts

@ncrawlins Thanks for the procedure, now I can repro this issue. Investigating.

@alexeagle is correct, the reason they are there is in order to support an entry point for a broccoli-typescript plugin to build the appropriate dependency model.

I think the typical treeshaker setup would be to give entry points to some other tool via its configuration (eg rollup https://github.com/rollup/rollup/wiki/JavaScript-API#entry )
But it's an interesting setup to use TypeScript's files config to serve both purposes...


Correct typo in JSON schema type keyword

A typo caused the markdownlint configuration JSON schema to contain an invalid keyword named type: rather than the intended type keyword.

Thanks!


Running az webapp create gives an unexpected error

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp create
Errors:
The command failed with an unexpected error. Here is the traceback:
No module named 'decorator'
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/fabric/connection.py", line 5, in <module>
    from invoke.vendor.six import StringIO
ModuleNotFoundError: No module named 'invoke.vendor'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/hackzor/.local/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/home/hackzor/.local/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 561, in execute
    self.commands_loader.load_arguments(command)
  File "/home/hackzor/.local/lib/python3.10/site-packages/azure/cli/core/__init__.py", line 507, in load_arguments
    self.command_table[command].load_arguments()  # this loads the arguments via reflection
  File "/home/hackzor/.local/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 318, in load_arguments
    super(AzCliCommand, self).load_arguments()
  File "/home/hackzor/.local/lib/python3.10/site-packages/knack/commands.py", line 104, in load_arguments
    cmd_args = self.arguments_loader()
  File "/home/hackzor/.local/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 125, in arguments_loader
    op = self.get_op_handler(self.op_path)
  File "/home/hackzor/.local/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 59, in get_op_handler
    handler = import_module(mod_to_import)
  File "/usr/lib64/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/hackzor/.local/lib/python3.10/site-packages/azure/cli/command_modules/appservice/custom.py", line 27, in <module>
    from fabric import Connection
  File "/usr/lib/python3.10/site-packages/fabric/__init__.py", line 3, in <module>
    from .connection import Config, Connection
  File "/usr/lib/python3.10/site-packages/fabric/connection.py", line 10, in <module>
    from decorator import decorator
ModuleNotFoundError: No module named 'decorator'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp create --name {} --resource-group {} --plan {} --deployment-container-image-name {} --docker-registry-server-password {} --docker-registry-server-user {} --https-only

Expected Behavior
Environment Summary
Linux-6.0.10-100.fc35.x86_64-x86_64-with-glibc2.34, Fedora Linux 35 (Workstation Edition)
Python 3.10.8
Installer: PIP

azure-cli 2.43.0

Extensions:
ssh 1.1.3

Dependencies:
msal 1.20.0
azure-mgmt-resource 21.1.0b1

Additional Context

route to CXP team

@Strobodov We are looking into it.

@Strobodov Please the response here https://github.com/Azure/azure-cli/issues/24865

Can confirm that there seems to be something wrong with my installation. Issue can be closed for now.


Fixed issue #6

Fixes #6
The grid disappears because the key release event is not called when another window gets active. The problem is fixed by manually calling the release function when the MainWindow gets inactive.

Tested; it works, thank you!


What are the reported losses for the improved WGAN?

In the improved wgan code, there is no printing/plotting of the loss. I'm trying to do this, but I'm confused by the output of train_on_batch for the Critic.
When I do print discriminator_model.metrics_names, I see this:
['loss', 'sequential_2_loss', 'sequential_2_loss', 'sequential_2_loss']

What are each of these? These don't line up with the model.compile line:
discriminator_model.compile(optimizer=Adam(0.0001, beta_1=0.5, beta_2=0.9),
                            loss=[wasserstein_loss,
                                  wasserstein_loss,
                                  partial_gp_loss])

First is the sum of the next three losses, where the next three are in the order as defined, wasserstein_loss, wasserstein_loss, partial_gp_loss.
For example, I get returned -5.6893063,-109.37327,102.22689,1.4570715, then
-109.37327 + (wasserstein)
102.22689 + (wasserstein)
1.4570715 (gradient penalty)
=-5,6893 (total loss) which is indeed what we expect from the first parameter = -5.6893063

Thanks! Make sense. The first one being the sum of all losses was the key piece I was missing.

@AmirAlavi What kind of results did you get on MNIST dataset. For me it shows black images after 9th epoch and never recovers. Please note I have not changed anything in the source code.


No access URL generated for mysql host

Version: main
UI: dev-e632ec6
Commit ID: dbf312b122cd051d8f7f34ca32df1cfea7e0475e
Env: Ubuntu 22.04 amd64
Test Steps:

Deploy a mysql db using mysql module.
View access URLs of the instance.

Test Result:

No access URL generated for mysql host.

Closed in favor of https://github.com/seal-io/walrus/pull/1861, a walrus resource's endpoints or walrus_endpoints outputs determine the endpoints.


Add support for cursor positioning

Richeditor should support cursor positioning like EditText.setSelection does.

I would need this feature too


Copy-paste bug in line 43 of background.ts?

Reading the source code of background.ts, I see these two lines:
        if (header.value?.toLowerCase().includes('mobile')) header.value = uaMaker(MOBILE_UA_SUFFIX, true)
        else header.value = uaMaker(MOBILE_UA_SUFFIX, false)

Shouldn't the second line use DESKTOP_UA_SUFFIX instead of MOBILE_USA_SUFFIX?

You are right. We haven't caught this error because the uaMaker change has not shipped yet.


Improve search field labelling

Clear distinction needs to be made between location based searches and ISP based searches.
We also might consider adding location search to the top nav for easy access to location changing functionality.

I've added labels.


Update examples-of-bulk-access-to-data-in-azure-blob-storage.md

Fixing data source name and code syntax

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

Hi @suedeyloh , thank you so much for your contribution! Yes, you are right, the name of the storage account should new invoices. Thank  you for making our docs better! #sign-off

OOPS I bet I messed up the merge, sorry.  :-(  I was trying to figure out how to merge, and saw this was still "open."  I'm going to keep my fingers off stuff now.
  - A fellow Microsoftie :-)

no worries :) please feel free to contact me internally for help with writing! #sign-off


[terraform-ibm-icd-postgresql] Expose provider visibility in the DA

The IBM provider supports passing a value for visibility. The default is "public" which means public endpoint is always used for api calls.
We should probably expose this value as a variable, and possibly default it to "private", especially since there may be CBR rules in place that only allow services to be talked to via the private network.

PR: https://github.com/terraform-ibm-modules/terraform-ibm-icd-postgresql/pull/521


Type error

lib/core/pup.ts
Error lines, 55, 59 - Property 'name' does not exist on type 'ProcessConfiguration'.deno-ts(2339)

😄  It's just a couple of days old, so the details are still forming. But it should start to settle soon, keep the feedback flowing!


No MQTT commands work

I've managed to install the script and discovery without issue. The problem lies when I issue out MQTT commands.
I can verify that the script connects to MQTT Broker and subcribe to the necessary topics. I've been able to listen and receive messages. However, when I issue activity commands there is no response. Only when I issue an activity command as "off" would EVERYTHING turn off. (as said in the documentation)
So clearly there seems to be an issue with responding to MQTT commands.

I've managed to install the script and discovery without issue. The problem lies when I issue out MQTT commands.
I can verify that the script connects to MQTT Broker and subcribe to the necessary topics. I've been able to listen and receive messages. However, when I issue activity commands there is no response. Only when I issue an activity command as "off" would EVERYTHING turn off. (as said in the documentation)
So clearly there seems to be an issue with responding to MQTT commands.

I'm new here, could You plase just give me an information with what can I install scripts? I've donwloaded the scripts but don't know in what to run them :\


Unit tests

Please @amycatgirl for the love of god
Add unit tests
Please 🥺

I forgot to mention this. I am working on a rewrite from the ground up to support unit testing and typescript.
The repo is private for the time being, but once the rewrite gets into a functional state (the core functionality), I'll push this into its own, separate public branch under this repo.
I am trying to make sure that I can unit test as much as I possibly can.

Once I get to command handling, I'll make sure to tackle #1 as well, since using classes for commands is getting a bit tiring.

Note to self: tomorrow, bump package to version 2.0.0, rename package name to autokickbot


Make stdio configurable

This PR allows stdout and stdin of Select to be configured in similar fashion to Prompt, useful when you want to output to, for example, osStderr instead.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


How to display "server status (up or down)" in Deploy with curl

Hi All
i am working on push deployment status into deploy widget with curl, but "Number of servers up or down in each environment" aways is zero. so How to display it with curl.
thanks a lot.

I haven't looked into this. My suggestion would be to see how that value is generated on the UI and track back to the DB. Once you see what values are required in the DB then you can try to figure out what needs to be passed.

thanks a lot

Were you able to figure it out?


Multi-line comments support

Would be nice to support proper syntax highlighting for multi-line comments:

We've done that in our Monaco-based project with:
monaco.languages.setMonarchTokensProvider("jsonata", {
    tokenizer: {
      root: [
        [/\/\*.*\*\//, "jsonata-comment"],
        [/'([^']*)'/, "jsonata-string"],
        [/"([^"]*)"/, "jsonata-string"],
        [/\$[a-zA-Z0-9_]*/, "jsonata-variable"],
        [/[a-zA-Z0-9_]+/, "jsonata-names"],
        { include: "@whitespace" },
      ],
      comment: [
        [/[^/*]+/, "jsonata-comment"],
        [/\/\*/, "jsonata-comment", "@push"], // nested comment
        ["\\*/", "jsonata-comment", "@pop"],
        [/[/*]/, "jsonata-comment"],
      ],
      whitespace: [
        [/[ \t\r\n]+/, "white"],
        [/\/\*/, "jsonata-comment", "@comment"],
        [/\/\/.*$/, "jsonata-comment"],
      ],
    },
  });

Fixed with 1.0.0


playing with processing modifications

It crashed with https://github.com/dabreegster/abstreet/issues/182 after playing for some time but it seems unrelated.
This PR is mostly request for somehow passing rng where it is necessary

Thanks! I now have access to tags and will play with a bit.

Model is still quite bad and results in many bizarre things, but at least in Kraków it is mostly limited by simulation - with people walking across entire city center because pedestrian-only shortcuts are not simulated, missing public transport (trams).
I would be happy to revisit it once simulation improves and is able to more closely model my city.
Is there some sane way to add traffic from outside map or leaving map and going outside? Especially people coming from outside using buses (it seems that buses are never leaving map)

In summary: it is a hilarious simplification filled with bizarre results, but at least it is clear improvement over "rush of trips at midnight, later nothing"

Sorry for delayed responses yesterday, and thanks for getting this working! I'm going to change the base to be the master branch, not bldg_types, to see the diff properly. Github warns it might make some of the old conversation threads vanish. :\

Is there some sane way to add traffic from outside map or leaving map and going outside?

You could use TripEndpoint::Border(i, None), where i comes from map.all_incoming_borders() or map.all_outgoing_borders(). After we merge this one, I can stub out some code for it.

Especially people coming from outside using buses (it seems that buses are never leaving map)

I don't know yet how to model buses and map boundaries more reasonably. Right now, buses insist on making loops around the stops that're inside the map bounds. If the bounds cut off part of a route, then the bus does really strange things to glue together the two directions of the route. This is one of the main reasons why light rail in Seattle is hard; there's no way to turn around using the separate tracks except near the start/end of the entire route.
The more realistic thing is probably to have buses disappear/reappear at appropriate border intersections. But then some modelling questions:

How long should the bus spend off-map?
If someone is riding to a destination off-map, do we just measure the trip time to leave the map?
If someone boards near the end of the route and needs to ride off-map, then reappear and alight at one of the first stops in the other direction, how should that work?

I can file another issue to track this; I've also got a google doc somewhere with other "transit simulation is too simple" problems

I can file another issue to track this

I would be happy to discuss that, I have some reasonably seeming ideas for problems that you mentioned.

I'll merge and do the formatting cleanup. Thanks for the PR!


v19.0.3 remove Entity::setId and move complete abstraction

[Current coverage][cc-pull] is 98.80%

Merging [#474][cc-pull] into [master][cc-base-branch] will increase coverage by 0.70%

@@             master       #474   diff @@
==========================================
  Files            27         27
  Lines           420        417     -3
  Methods         103        102     -1
  Messages          0          0
  Branches          0          0
==========================================
  Hits            412        412
+ Misses            8          5     -3
  Partials          0          0

Powered by Codecov. Last updated by [b869b76...d8a65b9][cc-compare]
[cc-base-branch]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/compare/b869b76cf97a23cbb18c7eb4437db8cc01135e8c...d8a65b9562ab7d1da2f79f46db7c16d8d27e002f
[cc-pull]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/pull/474?src=pr


suggestion: replace "created, check downloads" button action with close instead of key re-gen

I have created a controller account, the creds have been downloaded, but when I press a big red button i get another account generated rather than closing the window (which is a logical further step)

links https://github.com/airalab/robonomics.app/issues/163


ARROW-12512: [C++][Dataset] Support writing CSV datasets

This refactors the CSV writer to expose a writer class, then hooks it up to Datasets.

Closing for now - will submit the CSV writer separately first.


1.19.1 Animation is wrong for crawling.

Animation is just the swimming one, for some reason.

mod list?

modlist.txt

Just looked at the FastAnim code. It injects it's own variant of setAngles method into all mob's *Model classes. It could indeed break compatibility.


Textures missing for Object3D Babylon.js visualisation [App]

Describe the bug
When logging an .obj file, the texture .mtl and .png files are not uploaded and not visible in the Babylon.js visualisation in the dashboard.
To Reproduce
Steps to reproduce the behavior:

call wandb.Object3D("./path/to/model.obj")
open dashboard
see visualisation without texture

Expected behavior
When uploading the files manually to the Babylon.js sandbox, the object is rendered wih its texture. The wandb client should allow for .mtl and .png texture files to be uploaded with the .obj file.
Screenshots
Babylon.js

wandb

Operating System

OS: macOS
Browser firefox
Version 86

@frederikschubert thanks for the feature request, this is a good idea and touches a few different places in the app. We won't be able to fix this quickly, but we're tracking internally as a request on Object 3D and will increase priority as more requests come in.

Is this possible by now?


AIShield AI Security Monitoring Solution updates

Required items, please complete

Added new Analytics rules, parser and sample data
Change(s):
See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

yes

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Hi @udanashivm,
Please add new Analytical Rules and parser in input file.
After adding rules and parser in input file please repackage this solution using V3 tool. please refer below link FYR.
https://github.com/Azure/Azure-Sentinel/blob/master/Tools/Create-Azure-Sentinel-Solution/V3/README.md
Thanks..!!

Hi @udanashivm we are waiting for your feedback. Thanks

Hi @udanashivm, please give the response we are waiting for your feedback. Thanks..!!

Hi @udanashivm, please give the response we are waiting for your feedback. Thanks...!!

Hi @udanashivm, please give the response we are waiting for your feedback. Thanks..!!

Hi @udanashivm, please give the response we are waiting for your feedback. Thanks..!!

Hi @udanashivm ,
We wanted to check on the status of PR 10715. PR is pending for more than 60 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR.
Thank you for your cooperation.

Hi @udanashivm,
Since we have not received a response in the last 7 days, we are closing your PR #10715 per our standard operating procedures. If you still need support for this issue, you can re-open the PR at any time.
If you do re-open, we simply request that you ensure the PR has response to the last request.
Thank you for your cooperation.


I Can't Find The Wrapper Offline Files.

Can Someone Help Me? I'm Trying to find it.

same bro.

same bro.

Hello, if you are referring into the 2.0.0 version, everything has been remade. Please tell more information about what do you want to do with the files and I can help you.

If want the assets, resources/app/server/store/3a981f5cb2739137/cc_store (this is optional though), IDK where the code is.


Fix digdag-docs version and release

Fix digdag-docs version and release because there are not incremented.
Update version from "0.8" to "0.9"
Update release from "0.8"  to "0.9.5"

👍  Thank you!


Update qunit.css

Removed extra spaces. Made code more Structured.

This code is vendored from https://qunitjs.com/. Any changes should be made upstream.


Issue with BASELINE RESOLVER - 3.0.11-beta version

Basic information

My operating system: Windows 10
My MZmine version: 3.0.11-beta

What happened
Dear developers,
I am trying to reproduce a job I already performed on the old version MZmine 2.53, I started from Thermo raw data and I adopted the same parameters for mass detection and ADAP chromatogram builder, when it came to the deconvolution I tried to use the "baseline resolver" algorithm which I think is the equivalent of "baseline cut-off" in the 2.53 version, in fact, I put the same parameters, but this time I got this error for each feature list.

Could you give an explanation for this?
Parameters
-mass detention: MS1 noise level: 10000, MS2 noise level 100
-ADAP chrom builder: Scans MSlevel 1, Min group size n of scans: 3, group intensity: 10000, min intensity:10000, scan accuracy 20 ppm
-Baseline resolver: MS/MS scan pairing (RT toll:0.3, MS1MS2 toll: 5ppm), Min peak height: 10000, peak duration 0-2 min, Baseline level:10000 ---  ERROR ---
Thank you in advance,
Giovanni

Could you share the raw data file? And maybe your parameters as a batch processing file?
Did you check if the chromatograms list contains features?
Best,
Robin

Yes, I checked and they contain some features, please find attached the wetransfer link containing all the requested files.
https://we.tl/t-nRoBJ7jZ9P
Thank you in advance Robin,
Best,
Giovanni

You can check out the latest development build here:
https://github.com/mzmine/mzmine3/releases
If the issue persits or you find any other problems, you can open a new issue.
Thanks for the report!

I apologize I am not a developer, now if I download the
MZmine_Linux_portable.zip
https://github.com/mzmine/mzmine3/releases/download/Development-release/MZmine_Linux_portable.zip
version,
this error should be fixed?
Thank you again.
Giovanni
Il giorno ven 25 mar 2022 alle ore 13:48 Steffen Heuckeroth <
@.***> ha scritto:

You can check out the latest development build here:
https://github.com/mzmine/mzmine3/releases
If the issue persits or you find any other problems, you can open a new
issue.
Thanks for the report!
—
Reply to this email directly, view it on GitHub
https://github.com/mzmine/mzmine3/issues/612#issuecomment-1078996177,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARJ7P6L76JBNC5BJB6TUNYLVBWYYFANCNFSM5REETHFQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I shared them through the WeTransfer link, were you able to download them?
Il lun 21 mar 2022, 12:58 Robin Schmid @.***> ha
scritto:

Could you share the raw data file? And maybe your parameters as a batch
processing file?
Did you check if the chromatograms list contains features?
Best,
Robin
—
Reply to this email directly, view it on GitHub
https://github.com/mzmine/mzmine3/issues/612#issuecomment-1073807089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARJ7P6PRM5PIQ6ZSOU7QNTLVBBP43ANCNFSM5REETHFQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add files via upload

Local updates from the Bloomington Pestilli lab as of 8/22/2018

@DanNBullock So there are some code changes that you didn't make but you are making it part of your PR? Are you committing this from some git repo that's communally edited?
I did noticed that we are still using /N/dc2/projects/lifebid/code/mba on some of our App - probably used by other scripts as well? I can't change how people do things, but for Brainlife App, I think we should be switching to use this repo - which is cloned under /N/u/brlife/git/mba.. Do you know if I need to merge some changes made on /N/dc2/projects/lifebid/code/mba to this repo before I switch over?

Closing this as we don't understand what these changes are. Please send a new PR if necessary.


Antidote configuration

Unified and simplified Antidote configuration.

Changed shell command to use the release shell instead
Added part on how to use the relase in the Readme
Rebar now understands symlinks for _checkouts, fixed readme
Removed deprecated env entrypoint script
Removed launch-nodes script, uses legacy RELX_REPLACE_OS_VARS functionality
Removed all schema files except the riak_core schema for compatibility purposes. Especially the erlang-vm.schema file caused problems when trying to use the NODE_NAME env variable
Configuration use the new .src functionality to enable native OS variable replacement in the root configuration file
Split up smaller configuration files, currently only for the network configuration
Enabled packing files (data antidote, data riak, logging files) in a root directory if specified via ROOT_DIR_PREFIX
Same for logging and data dirs (LOGGER_DIR_PREFIX, DATA_DIR_PREFIX)
Removed vars.config, it was only used for the Mustache templating system of Rebar, which was only used for the schema files of cuttlefish, which we do not use
Fixed ranch to version 1.7.1 for now, otherwise it would upgrade to a release candidate with breaking API changes
Bumped antidote to version 0.4.0 (both rebar.config and antidote.app.src)
Moved documentation from antidote.app.src to config/sys.config where all configuration for the release resides
Added a validation function to ensure the antidote data directory exists
Commented out stats calls which are not implemented yet in the antidote_stats module (metadata_updated, metadata_update_stable)
Removed partially obsolete max ETS tables argument. The amount is not limited anymore, and 8000 is a reasonable default size of the internally used table for named tables.
Removed the ERL_MAX_PORTS env parameter. This is no longer used and is replaced by vm argument +Q. Furthermore, the default is much higher (65536) than this value (4096).
Added kernel network config for documentation purposes
Removed eunit_opts

Verbose output is worse than default output, as it doesn't show failures at the end
Eunit_logs is not useful

Added eunit.coverdata to coverage stats, coverage has increased by ~7%

Consequences:

[x] Docker files have to be updated
[x] Update documentation for Antidote on how to use a release, go into more detail than the readme
VM arguments are copy pasted from a rebar default from many years ago. Check if vm args improve performance, if not disable them
Check if there is a way to specify coverage output path for ct_run

The new dockerfile can be seen here.

Why are we bumping from v0.1.1. to v0.4.0?

Why are we bumping from v0.1.1. to v0.4.0?

There are multiple versions everywhere and I didn't know which version to take. The docker-antidote version is later than v0.2.0 and the release configuration (rebar) is 0.0.2. So I guess the version should be something like v0.2.1 everywhere? Initially I thought the docker-compose files will stop working, but they are still working.

Why is coveralls showing a decrease in coverage? There shouldn't be, does it have problems with it git rebase? It seems to compare the coverage to the previous build.

New statistics:


Cardboard boxes delete Magneticraft Shelving Units

Just encountered this nasty bug. I tried to pack a shelving unit using a cardboard box. The shelving unit's GUI opened for a second instead of packing, then both it and the box disappeared. Everyone nearby checked their inventories but could not find it. We tested it on another shelving unit and got the exact same results. We checked the server console and didn't see any errors.
Mekanism Version: 1.7.10-<IP_ADDRESS>8
Magneticraft Version: 0.6.1-final

This is an example of an interaction that would call for a blacklist entry. There are simply too many special cases for me to account for.

@bloxgate Do you have your forge config to remove erroring tile entities ?

@Alexismachina No I don't. Both settings for removing entities are set to false.

Hum...yeah i guest just add it to the blacklist like aidan said would be the best thing :/


Reflection, Query Builders and Scope Creep

What started out as a journey to implement reflection in Nebula... turned into a journey of reflecting inward... and also, I decided to change a lot of stuff.
Fixes #19, fixes #36, fixes #37, fixes #52, fixes #68, fixes #69
Draft of release notes are below - I'll update them here as I wrap up the pull request.

NebulaFactory.cls
The framework now supports reflection for some interfaces. To utilise your own interface implementation, open the custom metadata type 'Nebula Interface Implementations', find the record of the desired interface and enter the name of your Apex class. Nebula will then inject your code into the framework at runtime.
1. IAggregateResultQueryBuilder
2. IQueryArgumentFormatter
3. IQueryFilter
4. ISearchQueryBuilder
5. ISObjectQueryBuilder
To leverage Apex's implementation of reflection, all relevant classes now use a parameterless constructor - any setup work needed has been moved from constructors to methods.
SObject Repository
DML.cls

All DML methods now return the corresponding list of database results - see IDML.cls. Previously, these were void methods. This change is inherited by SObjectRepository.cls as well (it extends DML.cls)

SObjectRepository.cls

The member variable sobjectRepository.Query is now an instance of SObjectQueryBuilder.cls - see 'Query Builders' section for more info

New member variable sobjectRepository.AggregateQuery, an instance of the new class AggregateResultQueryBuilder.cls - see 'Query Builders' section for more info

Query Builders
QueryBuilder.cls has been split into SObjectQueryBuilder.cls, AggregateQueryBuilder.cls & SearchQueryBuilder.cls
QueryBuilder.cls

QueryBuilder.cls is now an abstract class. It only contains protected methods for shared logic that is used by SObjectQueryBuilder.cls & AggregateQueryBuilder.cls. QueryBuilder.cls itself should no longer be used directly.

SObjectQueryBuilder.cls
A new builder class that generates dynamic SOQL queries & returns a list of SObject

7 ways to add fields to your query:
1. addFields(Schema.FieldSet fieldSet)
2. addFields(List<Schema.SObjectField> fields)
3. addAllFields()
4. addAllStandardFields()
5. addAllCustomFields()
6. addAllReadableFields()
7. addAllEditableFields()

Name fields for lookups are automatically added to your query. For example, adding the contact object's field AccountId (a lookup to account):
sobjectQueryBuilder.addField(Schema.Contact.AccountId)
will result in your query having 2 fields added
SELECT AccountId, Account.Name FROM Contact

New method for including children records

New method, getQueryLocator()

AggregateQueryBuilder.cls
A new builder class that generates dynamic SOQL queries & returns a list of AggregateResult

Four methods to add fields to group by
1. groupBy(QueryDate queryDate);
2. groupBy(Schema.FieldSet fieldSet);
3. groupBy(Schema.SObjectField groupByField);
4. groupBy(List<Schema.SObjectField> groupByFields);

Six aggregate methods
1. avg(Schema.SObjectField sbojectField)
2. count(Schema.SObjectField sbojectField)
3. count_distinct(Schema.SObjectField sbojectField)
4. max(Schema.SObjectField sbojectField)
5. min(Schema.SObjectField sbojectField)
6. sum(Schema.SObjectField sbojectField)

Since this is the first round of implementing the class, not all aggregate result features (like 'GROUP BY ROLLUP') have been implemented, but more will be added in the future.
SearchQueryBuilder.cls
A new builder class that generates dynamic SOSL queries & returns a list of a list of SObject (yes, a list of list, that's not a typo)

Add SObject Query Builders to include them in the returned search results
1.  addSObjectQueryBuilders(ISObjectQueryBuilder sobjectQueryBuilder) - add a single SObject query builder
2.  addSObjectQueryBuilders(List sobjectQueryBuilders) - add a list of SObject query builders
The query search group defaults to 'ALL FIELDS', but can be changed to any of the QuerySearchGroup enums
1.  setQuerySearchGroup(QuerySearchGroup searchGroup)

QueryFilter.cls

Completely changed the methods available to handle 4 filtering scenarios
1. filterByField
2. filterByParentField
3. filterByQueryDate
4. filterBySubquery
Added support for specifying 'AND' and 'OR' statements in a query. To use, provide a list of query filters that you want to be wrapped as a set of 'AND' or 'OR' statements to the corresponding method
1. andFilterBy
2. orFilterBy

QueryDate.cls

A new class that can be used in two ways
1. Filter on a date/datetime field, using date functions

QueryDate qd = QueryDate.CALENDAR_MONTH(Schema.Lead.CreatedDate);
QueryFilter filter = new QueryFilter().filterByQueryDate(qd, QueryOperator.EQUALS, 3);
sobjectQueryBuilder.filterBy(filter);
System.assertEquals(filter.getValue(), 'CALENDAR_MONTH(CreatedDate) = 3');

// In SOQL, it would be written as
List<Lead> leads = [SELECT Id
FROM Lead
WHERE CALENDAR_MONTH(CreatedDate) = 3')];

Group by a date function in an aggregate result

QueryDate qd = QueryDate.CALENDAR_MONTH(Schema.Lead.CreatedDate);
aggregateResultQueryBuilder.groupBy(qd);

// In SOQL, it would be written as
List<AggregateResult> results = [SELECT CALENDAR_MONTH(CreatedDate)
FROM Lead
GROUP BY CALENDAR_MONTH(CreatedDate)];

QueryFilterScope.cls

Added 3 new filter scope enums
1. DELEGATED
2. MY_TERRITORY - only works in orgs with territory management enabled
3. MY_TEAM_TERRITORY - only works in orgs with territory management enabled

Record Types
SObjectRecordTypes.cls

Now requires the abstract method Schema.SObjectType getSObjectType() to be implemented

Settings

The custom setting NebulaRepositorySettings__c.object has been renamed to NebulaSObjectQueryBuilderSettings__c.object

Utilities
CollectionUtils.cls

Added 3 new methods for working with collections
1. getLastItem
2. pop
3. splice

Environment.cls

New property, NamespacePrefix

Documentation

Started commenting classes & methods with ApexDoc. This will be an ongoing improvement.

@jamessimone I'm still working on wrapping up some changes for this pull request (and catching up on tests, release notes, etc), but if you're bored and want to take a look, this has most of the changes that I'm making for v2.0

I've added SearchQueryBuilder.cls, giving a total of 3 types of query builders now

SearchQueryBuilder
SObjectQueryBuilder
AggregateResultQueryBuilder

@jongpie I began giving this an in-depth look yesterday, as I wanted to port the AggregateResultQueryBuilder over to our code.  I ended up taking a slightly different approach with a few of the design choices that you made - for example, I had the AggregateResultQueryBuilder extending QueryBuilder without a separate interface.  This made some of the method calls slightly clearer.  I also am not sure that I see the value in adding reflection for the QueryBuilder implementations - it's hard to imagine somebody wanting to override the default implementations (I would argue that the defaults are actually the reason somebody would want to use Nebula to begin with - nobody wants to have to re-write the query-level code!), and the work necessary to support reflection (specifically, moving the setting of the SObjectType for each QueryBuillder out of the constructors) ends up leading to additional hoops to jump through.  Does that make sense?
I also wrapped the implementation for grabbing the chained string for relationship names to another object outside of QueryFilter - the reason for this is that we have a concrete implementation right now that uses the orderBy call in QueryBuilder on a parent field (CustomObject__c.Lead__r.CreatedDate).  So I changed the implementation for IQueryBuilder's interface to accept this object, SObjectFieldWrapper, and that object handlers either a single SObjectField, or a chained list of related SObjectFields.
This SObjectFieldWrapper I also used in the QueryFilter implementation so that there is only one "filterByField" method, so to speak; the first argument is replaced by the wrapper.  More than happy to drop that in a branch if you are interested in taking a look.
I need to look into the rest of the changes you made today, and I am sure you plan to make others as well.  Overall I have to say that the quality of the code is quite high, and aside from the stuff that I've mentioned (which is very specific in terms of feedback), it has been a pleasure to look through all of this!

@jamessimone your comments about reflection for the query builders totally makes sense - I started working on just implementing reflection, but query builder (the one class at the time) was giving me some problems when trying to convert to work with reflection, plus I knew I wanted to eventually make some enhancements to QueryBuilder.cls anyway, and well... I went down a rabbit hole of refactoring and changing. In the end, I think you're right - I don't think there's a benefit to making the query builder classes use reflection. I'm going to undo some of the reflection-related changes that I made and move some of the code back into the constructors.
Your SObjectFieldWrapper class sounds like a great idea, that might actually help with a few other things that I want to add to the query builder classes in the future - if you're cool with throwing it into a branch, I'd love to take a look.
Last thing to note - I added SearchQueryBuilder.cls for SOSL queries. I'm not sure if you saw that change already or not, but it involved a few more changes to SObjectQueryBuilder.cls

@jongpie I will throw that in a branch, then!  Also, two other notes:

the getFirstQueryResult / getQueryResult calls in AggregateResultQueryBuilder can be eliminated entirely (!)
I don't know if you will want to follow this approach, but I changed the aggregate methods in AggregateResultQueryBuilder to take in a string as well as the SObjectField, with the string being the alias.  I get what you were going for with the default names, and it would be nice to have that as a fallback (though with method overloading that's gonna grow the class by quite a bit) - we have production code already referencing distinct aliases in the aggregate results, and it was more convenient for me to just require an alias than it was to do any other thing.  Your thoughts?

@jamessimone couple of notes for you!

My brain is fried, so I'm sure I'm overlooking something - why can the getFirstQueryResult / getQueryResult calls in AggregateResultQueryBuilder be eliminated?
I definitely plan to add the ability to alias the aggregate functions, but I personally will end up using the auto-alias names, so I want to support both options. I originally started implementing 2 versions for each aggregate function (see below), but then I wanted to see if there was a cleaner way. So far, overloading still seems like the best option for supporting both

Auto-aliased: avg(Schema.SObjectField numericSObjectField)
Manually aliased: avg(Schema.SObjectField numericSObjectField, String fieldAlias))

I've renamed SObjectFieldWrapper to QueryField (and for now, I'm sticking with your implementation of passing SObject fields in the constructor - seems to be the shortest code when using QueryField). It's now being used in these public methods (and probably a few other classes that I'm forgetting + test classes)

AggregateResultQueryBuilder - groupBy() and orderBy() now use QueryField
SObjectQueryBuilder - orderBy() now uses QueryField
QueryFilter - you changed this already for SObjectFieldWrapper, I just changed it to the new class name, QueryField

So, now we can group by, filter on and order by parent fields, which is pretty freakin' sweet. Thanks again for the help!
I'm still cleaning up the code, catching up on tests and stripping out some of the reflection changes that I implemented, so still a bit more work for me to do on this PR, but keep the feedback coming if you have more ideas.

@jamessimone welp, I made 2 more changes...

I had previously forgotten to update the aggregate functions in AggregateQueryBuilder.cls to use QueryField instead of Schema.SObjectField. That's done, so now you could do something like this (my examples aren't fully working code, just snippets of the relevant bits, let me know if they're unclear though)

// Get the min created date of the parent object (account)
[SELECT MIN(Account.CreatedDate) FROM Contact];

by doing something like this
QueryField parentAcctCreatedDate = new QueryField(new List<SObjectField>(Contact.AccountId, Account.CreatedDate));
// Get the min created date of the parent object (account)
aggregateQueryBuilder.min(parentAcctCreatedDate);

All the aggregate functions for AggregateQueryBuilder have been overloaded so you can supply your own field alias - just for you, dawg. Example:

// Automatic field alias
aggregateResultQueryBuilder avg(myQueryField);
// Bring Your Own Field Alias (BYOFA)
aggregateResultQueryBuilder avg(myQueryField, 'wubbaLubbaDubDub');

@jongpie due to the magical mysteries of Apex, you don't even need to cast a List<SObject> to a List<AggregateResult> - so your base getQueryResults() and getFirstQueryResult() methods can be called without needing specific methods to get the aggregate results.
ALSO - I ended up making the 'getQuery()' method on QueryBuilder virtual and added the following to the Aggregate's over-ridden implementation:
'        if(this.aggregateFunctionList.isEmpty()) {
return super.getQuery();
}'
That way if you have repositories overriding the default query builder, you can still do regular queries so long as you aren't doing anything with aggregates.
Grouping by parent fields is pretty awesome (as are those BYOA's) - thank YOU for adding those in!

@jamessimone I think I'm at a stopping point now. Do you mind taking another glance at the changes? Test classes are still lacking, but the main classes should be good now. I've essentially removed all of the reflection bits, even though that was the original aim. Since I ended up refactoring so much other stuff and new adding query builders, etc, I don't think there's now much value in supporting reflection right now. It may get added back in the future, but I prefer using constructors for some parameters.
I also want your opinion on QueryFilter in particular - it's still using a parameterless constructor, with separate methods for different filtering scenarios. I want to try to move back to passing parameters in the constructor - essentially, each 'filterby' method would get converted to a constructor. Thoughts? Should I move back to parameters in the constructors? Or keep the methods as-is?

@jongpie I will take a look!

@jamessimone please don't hate me, but I just pushed one last set of changes for SearchQueryBuilder.cls. I realized this weekend that I want all 3 query builders to be usable as soon as they are constructed - meaning, as soon as you construct one, you should be able to call getQueryResults() and not receive a runtime error. In the real world, the query builders would also need filters added, query fields to be selected, etc but I think it's nice to have the base object be usable.
SearchQueryBuilder.cls didn't work this way - you had to call a few of the builder methods before the search query would be usable. I've changed the constructors in SearchQueryBuilder. As a result, SObjectRepository.cls no longer has the query builder variables (this.Query, this.AggregateResult and this.SearchResult) - instead, each method in SObjectRepo now just instantiates it's own query builder as needed.
AggregateResultQueryBuilder.cls had a small change added - if you run something like this (no 'group by' or aggregate functions specified), then a default 'COUNT(Id)' is added to the aggregate query.
new AggregateResultQueryBuilder(Schema.Account.SObjectType).getQueryResults();

@jongpie no hate, and no worries!  Reviewing the rest of your changes (I am mostly caught up) was one of my plans this weekend - sorry I spent it going over Lightning stuff instead.  I plan to take a look and get this fully reviewed by the end of the week.

@jamessimone thanks for the review! I'm going to keep those methods for aggregate results for now. I plan to merge this to dev and master later today.


Increase s390x capacity for supporting more production load

This PR is for managing loads in the production system where we have multiple buildpipeline running and requesting nodes.
Also increasing the node capacity from 1CPU, 4GiB RAM ro 2CPU 8GiB ram

/lgtm


Tooltips could be shown on hover when app descriptions are cut by an ellipsis

On the pages were multiple applications are shown, it would be nice to have an on hover tooltip to read the full description without having to enter to the app page

I agree that the cut off descriptions don't look very good, but I don't think that tooltips are a good solution: you would still see the cut off descriptions, so it still wouldn't look very good.
The obvious solution would be to leave enough space for two lines of description text. Alternatively, we might want to consider dropping the descriptions altogether.

At the begining we tried tooltips in the descriptions but we disabled them because they were very annoying.
We are planning to revisit the tiles design. Maybe we'll do them wider and/or having two lines. The alternative solutions seems fine too

+1 yes, but I suggest using just "title" attribute, will solving the problem.
(sory for my english)


add topmed reference dataset

LMK if this needs unit tests, I thought maybe not

this looks good to me.  I ported the clinvar unit tests because we already had them and they were useful, but we don't need them for the others.


Fix for A1111 v1.3.0 Txt2Img bug, Added lines suggested by Ruakuu

See https://github.com/Interpause/auto-sd-paint-ext/issues/152
Tested and works on my setup

You added these changes to the img2img code, not the txt2img code


Wrap usage of global flags in default template

What type of PR is this?
(REQUIRED)
Minor enhancement of an existent function

(not sure !)
What this PR does / why we need it:
(REQUIRED)
It adds the wrap function to global flags in default template

Which issue(s) this PR fixes:
(REQUIRED)
Fixes a part of  https://github.com/urfave/cli/issues/1465

Special notes for your reviewer:
(fill-in or delete this section)

Testing
(fill-in or delete this section)

Release Notes
(REQUIRED)

As per @dearchap comment https://github.com/urfave/cli/pull/1483#pullrequestreview-1102236643 I think it's better to work on the original problem first: allowing user to set wrapAt in App


examples/posix_spawn: Fix parallel build errors at link time

During generating symtab.c, compile seqeuence would start with
incomplete symtab.c. This change will fix such a situation.
Signed-off-by: liuhaitao<EMAIL_ADDRESS>

@masayuki2009 I apply your elf change to posix_spawn since same problem occurred.

@masayuki2009 I apply your elf change to posix_spawn since same problem occurred.

@liuguo09 Thanks, I thought I should apply the same change to posix_spawn after the new nightly build completes to check if my approach was correct.

According to comments in #118 from @liuguo09

You are right. It seems the mv operation to symtab.c reduces the probability
of build break(from my local build, it is easy to hit the link error without mv operation),
but it doesn't root cause the link errors in the latest nightly build.

Thanks for the comment, though I think we have another root cause of the link errors in nightly build. This PR might reduce the probability as well. So let's merge this PR.

According to comments in #118 from @liuguo09

You are right. It seems the mv operation to symtab.c reduces the probability
of build break(from my local build, it is easy to hit the link error without mv operation),
but it doesn't root cause the link errors in the latest nightly build.

Thanks for the comment, though I think we have another root cause of the link errors in nightly build. This PR might reduce the probability as well. So let's merge this PR.

Another try:
https://github.com/apache/incubator-nuttx-apps/pull/123

I confirmed that this PR would be effective.
See https://github.com/apache/incubator-nuttx-apps/pull/118#issuecomment-600904711.


UI response very slow

Type: Performance Issue
I disabled all extetions and restart the vscode, the problem still exists
VS Code version: Code 1.85.2 (8b3775030ed1a69b13e4f4c628c612102e30a681, 2024-01-18T06:40:32.531Z)
OS version: Darwin x64 22.3.0
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
3, 4, 4

Memory (System)
16.00GB (0.78GB free)

Process Argv
--crash-reporter-id c1859d29-42ad-43f2-87b0-731a0edd0ab3

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
   49	   213	  2531	code main
   13	    49	  2539	   gpu-process
    0	    16	  2540	   utility-network-service
    8	   131	  2587	shared-process
    0	     0	 27022	     /bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=
    6	   279	 26398	window [3] (Visual Studio Code)
    0	    49	 26695	fileWatcher [3]
    2	    82	 26696	extensionHost [3]
   51	    66	 27013	window [4] (Issue Reporter)

Workspace Info
;

Extensions: none
A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscorecescf:30445987
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfcontrol:30979541
d34g3935:30971562
bg6jg535:30979843
8f97e687:30977000
dsvsc020:30976470

I was simply dragging the mouse to adjust the width of the search page, and the code main CPU usage would spike to 100%.

I restarted my MBP. It fixed. I don't know why.

Can we close the issue?
If you want us to investigate, please add the exact steps to reproduce the issue. A animated gif could be helpful.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Option to collect package json name entry

Summary
To help analyze the collected data, and allow to understand what project's are linked to the results, it would be great to be able to get the name entry of the project's package.json file, or the name of the project's github repository (or both).
Job stories
Allow innersource library maintainers to know which project's are using the library and get in touch with them.
If possible we could be interested to collaborate on this feature, as part of Airbus opensource contributions.
Acceptance criteria
Option to collect package.json name entry, github project's name, or both.

Hi @eric-helier - thanks for the suggestion! Here's how we're handling this on our end. Each Elastic document has the field resource.attributes.analyzed.path which is the analyzed GitHub repository's SHA-256 encoded path of format host/owner/repository like github.com/my-org/my-repo. Since there's no protocol in that path, it allows us to uniquely identify the analyzed repositories. But as you mentioned in this issue, since these paths are hashed, without doing anything, we can get unique repo counts, but can't see who they are. That is by design.
I'm not sure if @jdharvey-ibm mentioned this in that other issue, but we are hosting a Kibana instance for all telemetry reporting and data visualizations. This is internal to IBM. Without doing anything, those data tables and charts use the SHA-256 values everywhere.
If we know what we're looking for, like github.com/org-a/repo-a, and github.com/org-b/repo-b, we can also SHA-256 hash those paths, and then look for matches in Kibana documents.
We have a Chrome extension that we built for Kibana. Every second, it looks for known hashes, and swaps them out with known values.
For repo paths that we know about, npm dependencies that we know about, etc... we do decryptions in Kibana to make sense of this data. That allows our anonymized data to stay anonymous, and allows us to make sense of the data that we separately should know about.
I understand this part of IBM Telemetry has not been open-sourced. @jdharvey-ibm please elaborate if I'm missing something or it if could be explained better. Hope this helps if you want to take a similar approach to identify data.

Hi @mattrosno and thank you for getting back to me with some details on how you work around project's identification on your side. We'll think about it and see if possible to have a similar approach on our side.
This seems tricky, since we generally don't have access to project's repositories on github enterprise. One goal of using telemetry for us is to discover all the projects using our libraries.
Maybe we can negotiate an extract of all the project's names using our libraries, that would allow us to do the SHA-256 encoding and comparison, similar to your process.
As a general need, I was thinking there could be an optional feature to allow for de-anonymising the package.json name entry or github repository name, so we don't need to do this kind of additional work around using telemetry-js collected data.
Maybe this feature could be documented with a strong disclaimer to do it only if not working in IBM, as I understand no one should do that in your context.


My home is a duplex building. I can't see the first floor directly. Is there any solution？

Thanks！

Only way to support levels coming from sweethome3d is to use my plugin to export the model

@wenxu502, please give feedback otherwise I close the issue


Prevent FLAML logging warning on import

Why are these changes needed?
Previous FLAML PR (#61) did not fully remove the FLAML warning showing when running AutoGen workflows due to FLAML in completion.py.
So, this PR will add a temporary logging NullHandler during import of FLAML functions in completion.py, then will remove the NullHandler afterwards.
Related issue number
#61
Checks

[ ] I've included any doc changes needed for https://autogen-ai.github.io/autogen/. See https://autogen-ai.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

@thinkall FYI. This might be an issue to solve in FLAML.

@thinkall FYI. This might be an issue to solve in FLAML.

The warning message in FLAML is expected. I would suggest adding a logger instead of importing it from FLAML. This PR also works well though.

@thinkall FYI. This might be an issue to solve in FLAML.

The warning message in FLAML is expected. I would suggest adding a logger instead of importing it from FLAML. This PR also works well though.

Hi @thinkall, I tried that in a previous PR. I don't believe we're importing one now from FLAML. I think it's logging a warning regarding automl automatically.

@thinkall FYI. This might be an issue to solve in FLAML.

The warning message in FLAML is expected. I would suggest adding a logger instead of importing it from FLAML. This PR also works well though.

Hi @thinkall, I tried that in a previous PR. I don't believe we're importing one now from FLAML. I think it's logging a warning regarding automl automatically.

I see. You're right.


added isErrorMessage function

While parsing messages from a ExchangeFeed, I had to filter for errors and I didn't find a dedicated function. So I added it.

@Ike-Clinton thanks! Do you want me to include a test for this change in order to merge it or are you referring only to future changes?

Not up to me! :)
On Thu, Sep 21, 2017 at 03:57 Alessandro Costa<EMAIL_ADDRESS>wrote:

@Ike-Clinton https://github.com/ike-clinton thanks! Do you want me to
include a test for this change in order to merge it or are you referring
only to future changes?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/coinbase/gdax-tt/pull/24#issuecomment-331081691, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFij8dKiSJe0XV5B41ggedHt2YZM2K1Nks5skhbzgaJpZM4PeeAN
.

@Ike-Clinton sorry. I can't assign the PR but I saw that @CjS77 or @mihar merged previous PRs. I hope you can find the time to take a look at this one. Thanks ;)

@kostola, where are you filtering for errors? Once messages are in the stream, or from the underlying WS API?
I don't think we use ErrorMessage anywhere yet, so there's an opportunity to redefine it at this point to capture any use cases that fit.
For one thing, I'm thinking of changing the interface definition to
export interface ErrorMessage extends StreamMessage {
    message: string;
    details?: any
}

Then it fits into the streaming message system more elegantly and also allows users to pass metadata on (something that will possible expand to other messages).
If you're filtering from the API level, then the message should be coerced into the defined interface; or an 'error' message should be emitted (not to be confused with an error message that is sent along the message stream) and handled appropriately by its instantiator.

@CjS77 I modified the method as you suggested.
I am filtering messages coming from an ExchangeFeed, in particular authentication error messages.
I left the type check because also UnknownMessage interface contains a message field of type any.

Perfect sense. So what we really need to do is have those messages handled and pushed as ErrorMessage instances rather than embedded in UnknownMessage.
I'll merge your PR in now, but I'll open an issue to implement the above-mentioned change, but will let you carry on in the meantime.


Orca cmdb 22

orchestra cmdb updates with 2 web services added:

select ticket table
select like ticket table

Tested, nothing special


Added fix for dead ReLU detector with Activation layer

fixes https://github.com/keras-team/keras-contrib/issues/261

It's done.
Tests are in the same state as they were before this PR

LGTM, if tests pass then will merge.


Enhance duckdbExtension

What type of PR is this?

What does this PR do / why is it needed ?

Which issue(s) this PR fixes:

Fixes #
Other notes for reviewers:
Does this PR introduce a user-facing change?

@PrateekGarg-gs @gs-kotang can we merge change in https://github.com/finos/legend-engine/pull/2836 into this PR?
@aziemchawdhary-gs I want to merge these soon as they would help with my work on the DataCube PoC https://github.com/finos/legend-engine/pull/2864 to improve the grid for REPL which is backed entirely by DuckDB for now.
Could we merge this?


Use span-based string.Concat instead of string.Substring

PR Summary
Avoid some string allocations by using string.Concat(ReadOnlySpan, …) overloads.
Before:
str1 + str2.Substring(...) + str3

After:
string.Concat(str1, str2.AsSpan(...), str3)

PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[NA] Experimental feature(s) needed

[NA] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[NA] Documentation needed

[NA] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[NA] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[NA] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

@adityapatwardhan, @daxian-dbw Can you please review?

@iSazonov Could you please review?


Can't drag resources into property inspector.

Operating system or device - Godot version:
Linux - Master
Issue description:
Dragging a *.png file into a Sprites Texture slot doesn't change the texture.
It also doesn't draw that orange "hey, you can drop me here" helper box anymore.

Yeah previews are disabled for the moment, will come back later
On May 7, 2017 7:25 AM, "Andreas Haas"<EMAIL_ADDRESS>wrote:

Operating system or device - Godot version:
Linux - Master
Issue description:
Dragging a *.png file into a Sprites Texture slot doesn't change the
texture.
It also doesn't draw that orange "hey, you can drop me here" helper box
anymore.
[image: dnd_bug]
https://cloud.githubusercontent.com/assets/8281916/25778468/9c55cb32-32fe-11e7-8d00-53bfc3e06c44.gif
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/8676, or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z28yNN_OYXO3PzDahOZMdNt1xMhMRks5r3WPggaJpZM4NTCBh
.

@reduz this isn't about previews.
It's about not being able to drag resources into the property inspector.
In the gif you can see that it's not possible to set the texture of a sprite this way.

Can't reproduce anymore on latest master. However, someone on irc mentioned that it's still not working for them.

I still have this problem in master (f9fc2ef).

Can reproduce this too with current master on Linux.

Can't reproduce on master 9a502d3. I think it's fixed?

I have problems only with textures, and not all of the time.

Just got this issue, I've added a new image to the project directory and didn't let me to use it as a texture via drag and drop.

Can confirm that it's reproducible by adding a new image. icon.png can be dropped just fine while the new image can't.

this is a duplicate, i remember seeing it before..

ah yeah, #8116


Replace connected-react-router with an actively maintained package

We currently use this package for Redux bindings to React Router, but it does not seem to be actively maintained anymore, and does not have support for React Router v6. This blocks us from updating React Router.
An alternative package providing similar functionality and support for React Router 6 is redux-first-history.

closing


Volumes created with 1.3.1 crash the Finder

I upgraded our build server from 10.11 to 10.12 and ran into a problem with dmgbuild 1.2.1 that we were using, so I upgraded it to 1.3.1 and python 3.
The DMGs are created correctly, and the volumes mount correctly, but the volumes that it produces causes Finder to crash when I try to open them (tested on two 10.12.6 machines):

*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[__NSCFData getCString:maxLength:encoding:]: unrecognized selector sent to instance 0x610000249db0'
terminating with uncaught exception of type NSException
abort() called
Application Specific Backtrace 1:
0   CoreFoundation                      0x00007fffd13422cb __exceptionPreprocess + 171
1   libobjc.A.dylib                     0x00007fffe615a48d objc_exception_throw + 48
2   CoreFoundation                      0x00007fffd13c3f04 -[NSObject(NSObject) doesNotRecognizeSelector:] + 132
3   CoreFoundation                      0x00007fffd12b4755 forwarding + 1061
4   CoreFoundation                      0x00007fffd12b42a8 _CF_forwarding_prep_0 + 120
5   Foundation                          0x00007fffd2cfb9b8 parseFloats + 89
6   Foundation                          0x00007fffd2cfb939 NSRectFromString + 59
7   Finder                              0x000000010589e57f Finder + 2745727
8   Finder                              0x000000010566150c Finder + 398604
...

This occurs both with my previously working setup, as well as the volumes that 'python3 setup.py test' produces, in both the released 1.3.1 tarball as well as master. How can I start tracking this problem down?

On 22 Sep 2017, at 13:22, Matt Barringer<EMAIL_ADDRESS>wrote:

*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[__NSCFData getCString:maxLength:encoding:]: unrecognized selector sent to instance 0x610000249db0'
terminating with uncaught exception of type NSException
abort() called

If I had to guess, from the backtrace, this is a Python 3 problem; quite likely we’re sending a “bytes” object rather than an “str” object to the biplist code, which is causing it to generate an NSData instead of an NSString in the binary plist.
Have you tried using Python 2 instead?
Kind regards,
Alastair.
--
http://alastairs-place.net

With python 2.7.13 installed globally via Homebrew, I have a different error, which was why I switched to python 3:
E   internal_error: PyObjCRT_SizeOfType: Unhandled type '0x75', u'f', u'd']

That is despite uninstalling 2.7.13 after the OS upgrade and then reinstalling.
But it turns out that running dmgbuild within a Python 2.7.13 virtualenv works perfectly, so I'll close this. Sorry for the trouble!

I am encountering this issue as well on Python3, and using Python 2.7.13 is not really a practical option for me at the moment, since the invocation of dmgbuild is part of a larger build process that goes on in a Buildbot virtualenv for a Python3 app.
If dmgbuild is going to support Python3 compatibility then this issue should probably be re-opened since it seems like a legitimate bug.

Agreed, this shouldn't have been closed. I already have what I hope is a fix, which I've just checked in; I'll push out a release if it works (I'm just fixing the CI set-up because pyenv's maintainer appears to have pulled it from PyPI…)

OK. This is fixed in dmgbuild 1.3.2.


What is the license of irqstat?

Hi,
Thanks for sharing this powerful tool.
What is the license of irqstat?
This tool does help us a lot on debugging NUMA interrupts related problems.
We want to include this tool into our commercial products for debugging support.
However, there is no license information. I understand the tool is simple and small, but having a license information would be very helpful for making the decision.
Could you add LICENSE file into this project?
We can determine next step per legal evaluations.
Thanks.

Hi Oliver,
I am glad irqstat is helpful to you, and would like for it to be available
for you to use.  Let me think about it for a day and I'll decide on a
license and update the project.
Lance
On Sun, Mar 29, 2015 at 11:03 PM, Oliver Yang<EMAIL_ADDRESS>wrote:

Hi,
Thanks for sharing this powerful tool.
What is the license of irqstat?
This tool does help us a lot on debugging NUMA interrupts related problems.
We want to include this tool into our commercial products for debugging
support.
However, there is no license information. I understand the tool is simple
and small, but having a license information would be very helpful for
making the decision.
Could you add LICENSE file into this project?
We can determine next step per legal evaluations.
Thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/lanceshelton/irqstat/issues/1.

Lance,
I consulted with our lawyer and got confirmed that GPLv3 is not encouraged to be used in our product.
Our Linux distribution is using many other popular Licesnse like: GP:Lv2, BSD and so on.
Anyway, you are the owner, our legal team will review our request after you determine it.

Hi Oliver,
irqstat is now licensed under the MIT license.  It is similar to a BSD license.
Lance


init.sh script not working

Currently,  the script init.sh is not working because we removed the rust-toolchain file. We should either remove it completely (as it has been replaced by https://github.com/integritee-network/integritee-node/blob/master/ci/install_rust.sh I believe?) or fix it.

Yes, not needed anymore.


add support for input-group-sm and input-group-lg in prepend/append

As the title says. Add input_group_class: option so that prepend/append can decorate the input group with additional classes, like input-group-sm or input-group-lg

Looks great. Nice job on the README as well. Could you make the following additional changes so that I can merge this PR?

Mention this new feature in the CHANGELOG
Add a test

Thanks!

@mattbrictson Thanks for considering this. I updated the patch.

Excellent, thanks! ✨


Addressing documentation issues with some map control related functions

Some map control functions have some docs related errors (outlined in this GitHub Issue). This commit should help fixing those errors and others I noticed.

Did a build of your PR and everything looks good to me!
https://awpy-mapcontrol-docfix.readthedocs.io/en/latest/

awpy/analytics/map_control.py:81:0: R0914: Too many local variables (16/15) (too-many-locals)
Is this a new lint btw?
We have it configured to max 15 in pyproject
https://github.com/pnxenopoulos/awpy/blob/87167047ac88c9a425769403048e2786018296e2/pyproject.toml#L213C16-L213C16

Hm I don't think this is a new lint - I recall hitting this issue previously with my Map Control PR.
I think it will get fixed very soon however. I think in Jan-Eric's new type propagation PR, he removes the neighbor_info argument from _bfs and _calc_frame_map_control_tile_values. I think this should reduce the local variables a bit. My thought is that Jan-Eric's PR could be merged in first and I can take those changes and see if that fixes the lint issue. If it does, then we can go forward with this PR. If not, I can fix the local variable issue myself. Let me know what you guys think @JanEricNitschke @hojlund123

Yeah, it's not a new lint. It's just that pylint is currently configured to allow basically one linter warning and still pass.
(The same warning is currently in main: https://github.com/pnxenopoulos/awpy/actions/runs/5707191025/job/15463655598#step:10:13)
But the type propagation PR does remove it anyway. I also have a CI update PR as draft that i will add some more commits to soon which changes the pylint threshold to allow 0 linter warnings in the future. This also currently has a fix for this warning but it is less elegant than he one in the type PR.
So i do not think we do not necessarily need to wait for the type PR as this warning is already in main anyway. (But the type PR should also be ready anway too)


Inconsistent starting title when using "test dialogue" button.

Describe the bug
The dialogue starts at inconsistent titles when editing and testing using the built-in text editor and test dialogue button.
Affected version

Dialogue Manager v2.25.1
Found bug on Godot 4.1.1, Also confirmed on new clean project in Godot 4.1 stable.

To Reproduce
Steps to reproduce the behavior:
create a new dialog file, and create a few titles. This is what I used.
~ one
one
=> END
~ two
two
=> END
~ three
three
=> END
Always test the dialogue using the "test dialogue" button.
First time through you should get "one" of course.
Now make a change. I moved the entire "three" block to the top.
Don't save anything. (I used ctrl-S). Run again.
I got "two"...
I just keep making small singular changes to the file and re-running it (messing with it) - adding new lines, deleting text, moving titles up and down in order etc. - and the output changes. But again, seemingly randomly.
Saving (ctrl-s shortcut) also seems to have an effect - sometimes.
I've managed to get all 3 numbers just by messing with the text and changing order.
Expected behavior
Consistent behavior.
When using the test dialogue button, it should always give the same "expected" output. Right now I'm just crossing my fingers that it starts where I expect.
I'm not sure what the intended behaviour is when running a dialogue file this way. Does it jump to the first title? Do I tell it where to start with a jump at the very top outside of any titles - Just tested this with Nathan default, it crashed. Submitting a seperate bug report.

It should be starting from the closest title above the current cursor.

Thanks for the quick reply. Totally works as expected!
It crashes when there is nothing above the cursor though.

Found this in the documentation:
Your custom test scene must extend BaseDialogueTestScene which will provide you with a resource property which is the DialogueResource currently open and a title property which is the nearest title to the cursor position.
This information should probably be put under (or duplicated in?) the "Running a test scene" heading.
Also should probably read "nearest title above the cursor position."