Crash caused by trying to add custom trades.

crash-2017-06-20_12.22.31-server.txt
This is the file I had modified:
{
	Profession: "minecraft:smith",
	Career:"armor",

	Offers:{
		Recipes:[

			{action:"remove", buy:{id:"minecraft:emerald"},sell:{id:"minecraft:diamond_chestplate"}, CareerLevel:3},
			{action:"add", buy:{id:"minecraft:emerald",Count:{min:10,max:12}},sell:{id:"minecraft:diamond_helmet",Count:1,tag:{ench:"random"}}, CareerLevel:3, chance:0.25},
			{action:"add", buy:{id:"minecraft:emerald",Count:{min:16,max:19}},sell:{id:"minecraft:diamond_chestplate",Count:1,tag:{ench:"random"}}, CareerLevel:3, chance:0.25},
			{action:"add", buy:{id:"minecraft:emerald",Count:{min:14,max:17}},sell:{id:"minecraft:diamond_leggings",Count:1,tag:{ench:"random"}}, CareerLevel:3, chance:0.25},
			{action:"add", buy:{id:"minecraft:emerald",Count:{min:8,max:10}},sell:{id:"minecraft:diamond_boots",Count:1,tag:{ench:"random"}}, CareerLevel:3, chance:0.25}
			{action:"add", buy:{id:"minecraft:iron_horse_armor",Count:1},sell:{id:"minecraft:iron_ingot",Count:{min:8,max:11}, CareerLevel:1}
			{action:"add", buy:{id:"minecraft:golden_horse_armor",Count:1},sell:{id:"minecraft:gold_ingot",Count:{min:8,max:11}, CareerLevel:2}
			{action:"add", buy:{id:"minecraft:diamond_horse_armor",Count:1},sell:{id:"minecraft:diamond",Count:{min:5,max:7}, CareerLevel:3}

		]
	}
}

Thanks!

Hi - you were close!
The lines you added were missing a curly bracket after the sell count (notice that the lines above have two and you have one), and they're also missing a comma at the end - all the lines except the last one need a comma.
Here's a fixed file:
{
	Profession: "minecraft:smith",
	Career:"armor",

	Offers:{
		Recipes:[
			{action:"remove", buy:{id:"minecraft:emerald"},sell:{id:"minecraft:diamond_chestplate"}, CareerLevel:3},
			{action:"add", buy:{id:"minecraft:emerald",Count:{min:10,max:12}},sell:{id:"minecraft:diamond_helmet",Count:1,tag:{ench:"random"}}, CareerLevel:3, chance:0.25},
			{action:"add", buy:{id:"minecraft:emerald",Count:{min:16,max:19}},sell:{id:"minecraft:diamond_chestplate",Count:1,tag:{ench:"random"}}, CareerLevel:3, chance:0.25},
			{action:"add", buy:{id:"minecraft:emerald",Count:{min:14,max:17}},sell:{id:"minecraft:diamond_leggings",Count:1,tag:{ench:"random"}}, CareerLevel:3, chance:0.25},
			{action:"add", buy:{id:"minecraft:emerald",Count:{min:8,max:10}},sell:{id:"minecraft:diamond_boots",Count:1,tag:{ench:"random"}}, CareerLevel:3, chance:0.25},
			{action:"add", buy:{id:"minecraft:iron_horse_armor",Count:1},sell:{id:"minecraft:iron_ingot",Count:{min:8,max:11}}, CareerLevel:1},
			{action:"add", buy:{id:"minecraft:golden_horse_armor",Count:1},sell:{id:"minecraft:gold_ingot",Count:{min:8,max:11}}, CareerLevel:2},
			{action:"add", buy:{id:"minecraft:diamond_horse_armor",Count:1},sell:{id:"minecraft:diamond",Count:{min:5,max:7}}, CareerLevel:3}
		]
	}
}

I put it in PasteBin as well for you: https://pastebin.com/pJDJXssV

Thank you, seems to be working fine now! I knew it would be something I'd missed! Still learning :)


Remove unnecessary optional keywords and add documentation to UpdateGridpoolOrderRequest

What's needed?
There are redundant optional keywords being used in the UpdateGridpoolOrderRequest message.
Proposed solution
// Request to update an existing order for a given Gridpool.
message UpdateGridpoolOrderRequest {
  // Represents the order properties that can be updated after an order has been
  // placed. At least one of the optional fields must be set for an update to
  // take place.
  message UpdateOrder {
-   // The updated limit price at which the contract is to be traded.
+   // Optional; The updated limit price at which the contract is to be traded.
    // This is the maximum price for a BUY order or the minimum price for a SELL
    // order.
-    optional frequenz.api.common.v1.market.Price price = 2;
+   frequenz.api.common.v1.market.Price price = 2;

-   // The updated quantity of the contract being traded, specified in MWh.
+  // Optional; The updated quantity of the contract being traded, specified in MWh.
-   optional frequenz.api.common.v1.market.Energy quantity = 3;
+  frequenz.api.common.v1.market.Energy quantity = 3;

-   // Applicable for STOP_LIMIT orders. This is the updated stop price that
+   // Optional; Applicable for STOP_LIMIT orders. This is the updated stop price that
    // triggers the limit order.
-   optional frequenz.api.common.v1.market.Price stop_price = 4;
+  frequenz.api.common.v1.market.Price stop_price = 4;

-   // Applicable for ICEBERG orders. This is the updated price difference
+   // Optional; Applicable for ICEBERG orders. This is the updated price difference
    // between the peak price and the limit price.
-   optional frequenz.api.common.v1.market.Price peak_price_delta = 5;
+  frequenz.api.common.v1.market.Price peak_price_delta = 5;

-   // Applicable for ICEBERG orders. This is the updated quantity of the order
+   // Optional; Applicable for ICEBERG orders. This is the updated quantity of the order
    // to be displayed in the order book.
-   optional frequenz.api.common.v1.market.Energy display_quantity = 6;
+  frequenz.api.common.v1.market.Energy display_quantity = 6;

-   // Updated execution options such as All or None, Fill or Kill, etc.
+   // Optional; Updated execution options such as All or None, Fill or Kill, etc.
    optional OrderExecutionOption execution_option = 7;

-   // This is an updated timestamp defining the time after which the order
+   // Optional; This is an updated timestamp defining the time after which the order
    // should be cancelled if not filled. The timestamp is in UTC.
    google.protobuf.Timestamp valid_until = 8;

-    // Updated user-defined payload individual to a specific order.
+   // Optional; Updated user-defined payload individual to a specific order.
    // This can be any data that the user wants to associate with the order.
    google.protobuf.Struct payload = 9;

-   // Updated user-defined tag to group related orders.
+   // Optional; Updated user-defined tag to group related orders.
    optional string tag = 10;
  }

  // ID of the Gridpool the order belongs to.
  uint64 gridpool_id = 1;

  // The order identifier
  uint64 order_id = 2;

  // Field mask specifying which fields should be updated
  google.protobuf.FieldMask update_mask = 3;

  // The fields that can be updated
  UpdateOrder update_order_fields = 4;
}

// ...

// Represents an order in the electricity market.
message Order {
  // The delivery area where the contract is to be delivered. The representation
  // of the delivery area may vary by jurisdiction.
  frequenz.api.common.v1.grid.DeliveryArea delivery_area = 2;

  // The delivery period for the contract, specified as a start and end
  // timestamp in UTC. It represents the period during which the contract is
  // expected to be fulfilled.
  frequenz.api.common.v1.grid.DeliveryPeriod delivery_period = 3;

  // The type of order, such as LIMIT, STOP_LIMIT, ICEBERG etc.
  // This determines how the order is to be executed in the market.
  OrderType type = 4;

  // Indicates if the order is on the Buy or Sell side of the market.
  MarketSide side = 5;

  // The limit price at which the contract is to be traded. This is the maximum
  // price for a BUY order or the minimum price for a SELL order.
  frequenz.api.common.v1.market.Price price = 6;

  // The quantity of the contract being traded, specified in MWh.
  frequenz.api.common.v1.market.Energy quantity = 7;

- // Applicable for STOP_LIMIT orders. This is the stop price that triggers the
+ // Optional; Applicable for STOP_LIMIT orders. This is the stop price that triggers the
  // limit order.
  optional frequenz.api.common.v1.market.Price stop_price = 8;

- // Applicable for ICEBERG orders. This is the price difference between the
+ // Optional; Applicable for ICEBERG orders. This is the price difference between the
  // peak price and the limit price.
- optional frequenz.api.common.v1.market.Price peak_price_delta = 9;
+ frequenz.api.common.v1.market.Price peak_price_delta = 9;

-  // Applicable for ICEBERG orders. This is the quantity of the order to be
+  // Optional; Applicable for ICEBERG orders. This is the quantity of the order to be
  // displayed in the order book.
- optional frequenz.api.common.v1.market.Energy display_quantity = 10;
+ frequenz.api.common.v1.market.Energy display_quantity = 10;

  // Optional; Execution options such as All or None, Fill or Kill, etc.
  optional OrderExecutionOption execution_option = 11;

- // Do not use if ExecutionOption is set to FOK or IOC. This is an optional UTC
+ // Optional; Do not use if ExecutionOption is set to FOK or IOC. This is an optional UTC
  // timestamp defining the time after which the order should be cancelled if
  // not filled.
- optional google.protobuf.Timestamp valid_until = 12;
+ google.protobuf.Timestamp valid_until = 12;

  // Optional user-defined payload individual to a specific order. This can be
  // any data that needs to be associated with the order.
  //
  // The field can store e.g. JSON objects containing details involved in the
  // order. This feature can simplify application development by eliminating the
  // need for complicated state management to remember the specifics of each
  // order.
  //
  // By embedding this "state" within the order itself, you can include
  // specifics like which microgrids consume or provide how much power. This
  // makes it easier to manage complex orders and can simplify the logic
  // required in applications.
  //
  // Example JSON payload:
  // {
  //   "microgrids": [
  //     {
  //       "microgrid_id": "1",
  //       "mwh": 1.0
  //     },
  //     {
  //       "microgrid_id": "2",
  //       "mwh": 0.5
  //     }
  //   ]
  // }
  //
  // In this example, if the order is exectuted, these microgrids might consume
  // the electricity mentioned in the example JSON payload.
- optional google.protobuf.Struct payload = 13;
+ google.protobuf.Struct payload = 13;

  // Optional user-defined tag to group related orders.
  optional string tag = 14;
}

Use cases
No response
Alternatives and workarounds
No response
Additional context
No response

The recommendation makes sense.
I'd also recommend adding that these fields are optional to the field documentations, so that users get informed of it when they use these fields in their IDEs/code-editors.

fields are optional to the field documentations

Haven't I done this?

Haven't I done this? Whats on your mind beyond of what I added to the documentation?

Oh right, you did that already. I'd maybe extend it to add a suggestion what happens when it is left empty, wherever it is possible.

Why is the optional keyword  not removed for the stop price?
optional frequenz.api.common.v1.market.Price stop_price = 8;

Why is the optional keyword not removed for the stop price?

That was a mistake. Its fixed now.

And what is the 'sane default' of delivery_period and delivery_area?

There is none. If they are not specified they will be rejected. They have to be explicitly given. They also are not optional hence I'm not sure where you copied the lines above from. So far these are not optional.

I'm not sure where you copied the lines above from

From the GridpoolOrderFilter here, and same applies to PublicTradeFilter.
But yes I recon they should not have the optional keyword, I'll change it with the rest.

But yes I recon they should not have the optional keyword, I'll change it with the rest.

No this is part of the filter.  To me that looks correct. These are however used in combination with the field masks.

if they are not specified there is no default

I thought their default would precisely be None. And thus the reason why the optional keyword was not needed here.

They can never be a "default" as with an enum

My understanding was that the 'sane default'  for enum is 0 or UNSPECIFIED as we have it. So that for these we needed to keep the optional keyword when needed.

since they are composite types

Forget about the composite types stuff. Please have a look at this table here.

The changes have been merged in #21.
@camille-bouvy-frequenz can we close this issue?


Custom dataset location impossible?

Steps to reproduce
import qcodes as qc
from qcodes.dataset.sqlite_base import init_db, connect

dataDB = 'C:/Users/Junkie/Documents/Programming/Python/QCoDeS/datapool/experiments.db';
qc.config["core"]["db_location"] = dataDB;

conn = connect(dataDB, debug=True)
init_db(conn)

from qcodes.dataset.experiment_container import Experiment
from qcodes.dataset.measurements import Measurement
from qcodes import new_data_set, new_experiment

Expected behaviour
Use the database under the path dataDB for the storage of measurements..
Actual behaviour
It uses the database created under the current working directory instead.
System
operating system
qcodes branch
feature/dataset
qcodes commit

The file qcodes\dataset\experiment_container.py, host of new_data_set and new_experiment, contains the following line
DB = qcodes.config["core"]["db_location"].
As this is executed as soon as any part of qcodes is imported, the variable DB is set to the default config that is saved to the disk as parts of the qcodes repository.


entrypoints.json

  "css": [
    "/build/app.b82aa2d9.css"
  ]

?

@lxregistry could you explain where do you see this code and what is your question here, please?

I think this is related to the problems we're having with the compiled assets. Let's close this because that problem was already solved. Thanks.


[semantic-arc-opts] Instead of tracking owned value introducers, track incoming values that are seen from owned value introducers.

TLDR: This will allow me to in a forthcoming commit to eliminate the extra run
of the peephole optimizer to seed the phi arg analysis since this eliminates
potential data invalidation issues. That being said, I am formalizing the
language/model in the pass a little in this commit, so I wanted to give a long
description below.

In this commit, I am formalizing some of the language around this optimization
away phi nodes specifically and more towards a notion that Andy and I have
discussed called "Joined Live Ranges". The idea is a "Live Range" in OSSA is an
owned value together with its set of forwarding uses. The owned value is in a
certain sense the equivalence class representative of all of the values that are
derived from the owned value via forwarding instructions.
This works well as long as all of the "forwarding" instructions only derive
their ownership from a single operand. Some instructions without that property
are branch, tuple, struct. Since such a "forwarding" instruction derives its
ownership by merging the ownership of multiple operands, if multiple of its
operands are non-trivial owned operands, we are in a sense joining together the
live ranges of those non-trivial owned operands. The main implication of this is
that if we want to convert the joined live range from owned to guaranteed, we
can only do it if we can also do it for all of its incoming values at the same
time.
This creates a conundrum though since this pass has been written like a peephole
pass which is ill-suited to be converted into this form. So instead, this pass
seeds an analysis from peephole runs that /could/ have been converted from owned
to guaranteed except for a specific joined live range. Then after we reach a
fixed point, we use that information from the failed peepholes to try to
eliminate the joined live ranges.
This approach is a good approach, but if implemented in certain ways exposes the
risk of problems around touching invalidated data. Specifically, before this
patch, this was implemented with the concern that each branch
instruction (currently the only supported type of joined live range instruction)
could be paired with multiple owned value introducers. This is because
initially, we were allowing for tuple, struct with multiple non-trivial operands
to be used as owned value introducers. This implied that we could not rely on
just storing a bool per joined live range operand since we would need to know
that all introducers associated with the operand were optimizable.
After some thought/discussion, I realized that this sort of optimization can
also apply to tuple/struct/similar insts. This let to the shift in
thinking/realization that we could talk instead about general joined live ranges
and handle tuple/struct/similar instructions just like they were branch/phi! As
such, if we assume that all such instructions are treated as separate
LiveRanges, then we know that our branch can only have a single owned value
introducer since we do not look through /any/ joined live ranges when computing
LiveRanges.
This realization then allows us to store a single bool value marking the operand
on the joined live range as being one that we /may/ be able to optimize. This
then allows us to simplify the state we handle and not have to worry about
storing memory to instructions that may be eliminated. Since we have a joined
live range use, we know that any value that is used by the joined live range can
not be eliminated by the peephole optimizer.

@atrick here is the other part!

@swift-ci test

@swift-ci asan

Should be NFC.


Unity no .meta file , prefabs is no work

Hey dude，unity sample scene git clone no work , cannot find .meta file

Same for me. The prefabs in the scene are broken and missing.

I updated windows foundation to 2.7.3 and textmesh pro to 2.1.6 and fixed prefabs


 This doesn't cause a malfunctioning program, but it causes a severe slow-down due to exceptions being slow.
To Reproduce
In Visual Studio, enable stopping the program execution when an InvalidOperationException gets thrown.
Start the following program:
using System;

using Microsoft.Data.Sqlite;

namespace TestSqliteInvOpEx
{
    class Program
    {
        static void Main(string[] args)
        {
            using (var conn = new SqliteConnection("Data Source=:memory:"))
            {
                conn.Open();

                using (var createTableCmd = conn.CreateCommand())
                {
                    createTableCmd.CommandText = "create table test (a int)";
                    createTableCmd.ExecuteNonQuery();
                }

                using (var insertCmd = conn.CreateCommand())
                {
                    insertCmd.CommandText = "insert into test (a) values (1)";
                    insertCmd.ExecuteNonQuery();
                }

                using (var queryCmd = conn.CreateCommand())
                {
                    queryCmd.CommandText = "select * from test";
                    using (var reader = queryCmd.ExecuteReader())
                    {
                        while (reader.Read())
                        {
                            Console.WriteLine(reader.GetValue(0));
                        }

                        reader.Close();
                    }
                }

                conn.Close();
            }
        }
    }
}

Stack trace:
 	Microsoft.Data.Sqlite.dll!Microsoft.Data.Sqlite.SqliteDataReader.NextResult()	Unbekannt
 	Microsoft.Data.Sqlite.dll!Microsoft.Data.Sqlite.SqliteDataReader.Dispose(bool disposing)	Unbekannt
 	System.Data.Common.dll!System.Data.Common.DbDataReader.Dispose()	Unbekannt
>	TestSqliteInvOpEx.dll!TestSqliteInvOpEx.Program.Main(string[] args) Zeile 38	C#

Additional context
Microsoft.Data.Sqlite version: 3.0.0
Target framework: .NET Core 3.0
Operating system: Windows 10 Pro 1903

@fubar-coder Is it intentional that you're not disposing the DbCommand? Do you see different results if you do dispose it?

@ajcvickers I don't understand. I am disposing the command (via using) and I'm not sure that I am allowed to dispose a command before I dispose the data reader.

@fubar-coder Oops. Yes, mis-read the code.

Note for triage: essentially the same as #18307, which we never really got to the bottom of.

@roji If you are looking for 3.1 work, then you could try digging into the SQLite code for this and the other linked issue. Another set of eyes on the code might help.
/cc @bricelam

Duplicate of #18307

Closing this as issue is tracked in the reopened issue #18307


Difference icon_type between folium and st_folium

Hello,
I have a rendering issue between Folium and st_folium.
Indeed, st_folium doesn’t display the numbers in my bubbles. With Folium, I use the icons fa-1, fa-2,…
And when I use st_folium, the numbers in the markers disappear.

code:
import streamlit as st
import folium
from folium.map import Icon
from streamlit_folium import st_folium

st.set_page_config(layout='wide')

m = folium.Map(location=[39.949610, -75.13282], zoom_start=14, height=500)

icons_list = ["1", "2", "3", "star", "heart"]

for i, icon in enumerate(icons_list):
    folium.Marker(
        [39.949610, -75.150282 + i/100],
        icon=Icon(icon=icon, prefix='fa', color='red', icon_color='white')
    ).add_to(m)

cols = st.columns(2)

with cols[0]:
    "## Using `components.html`"
    st.components.v1.html(
        m.get_root()._repr_html_(), height=500,
    )

with cols[1]:
    "## Using `st_folium`"
    st_folium(m, returned_objects=[], use_container_width=True, height=500)

Thank you to edsaac from discuss.streamlit.io for the code and the picture

I suspect this might be resolved with our newest release, as we've improved how the various required libraries for folium are loaded. Please let us know if this is still an issue

Closing this, as I would guess the newest version of the package solves this issue. If not, happy to re-open


Problem with range.start and range.end

Hi,
I discover a problem with the last version of the library, I want to put the month name on the header on each month.
I make a vertical scroll and itemSize = 50.
I use this to take the month on the header.
func calendar(_ calendar: JTAppleCalendarView, willDisplaySectionHeader header: JTAppleHeaderView, range: (start: Date, end: Date), identifier: String) {

        switch identifier {
        case "MonthHeaderFirstView":

            let headerCell = (header as? MonthHeaderFirstController)

            let startDate = range.start
            print("RANGE START: \(range.start)")

            let month = currentCalendar.dateComponents([.month], from: startDate).month!

            let year = currentCalendar.component(.year, from: startDate)

            headerCell?.firstMonthLabel.text = String(month) + " - " + String(year)
            break;

        case "MonthHeaderSecondView":
            let headerCell = (header as? MonthHeaderSecondController)

            let endDate = range.end
            print("RANGE END: \(range.end)")

            print("--------------------------")
            let month = currentCalendar.dateComponents([.month], from: endDate).month!

            let year = currentCalendar.component(.year, from: endDate)

            headerCell?.secondMonthLabel.text =  String(month) + " - " + String(year)

            break;

        default:
            let headerCell = (header as? MonthHeaderFirstController)

            let startDate = range.start

            let month = currentCalendar.dateComponents([.month], from: startDate).month!

            let year = currentCalendar.component(.year, from: startDate)

            headerCell?.firstMonthLabel.text =  String(month) + " - " + String(year)
            break;
        }
    }

func calendar(_ calendar: JTAppleCalendarView, sectionHeaderIdentifierFor range: (start: Date, end: Date), belongingTo month: Int) -> String {
        if month % 2 > 0 {
            return "MonthHeaderFirstView"
        }
        return "MonthHeaderSecondView"
    }

But It has a problem on February and some months, the range start will return this "2017-02-28 23:00:00 +0000"
Here's a screenshot.

I hope you can help me :)
Thanks.

I do not understand the last part of your question.
You said

the range start will return this "2017-02-28 23:00:00 +0000"

Which function is returning this?
Is it returning this for all the months?
Based on your question, I am not sure what the problem is. For me to know  the problem, I neeed to know what you are getting and what was supposed to be the expected result, and how they are different.

I want to put the month number on the header in the calendar, how can I do it?
My calendar shows max. 2 headers at the same time on the screen.

I did this:
    func calendar(_ calendar: JTAppleCalendarView, willDisplaySectionHeader header: JTAppleHeaderView, range: (start: Date, end: Date), identifier: String) {
        let date = range.start
        let month = testCalendar.component(.month, from: date)
        // etc etc

And it works well.
Can you tell me what version number of the calendar you are using?

I use the last versión 6.1.2.
I'll try your code later, I'll tell you if it works ;)

I can't try this now, I'll try it on Monday at Work.
I'll tell you if it works.
Thanks ;D

It works, thank you so much :D

Awesome. I'll close this issue now.
Btw, you calendar looks very cool. Why dont you post its picture here --> https://github.com/patchthecode/JTAppleCalendar/issues/216 ?
cheers man.


fix: eth_estimateGas opts to fallback estimation for contract revert

Description:
This PR modifies the EthImpl class to improve error handling for gas estimation.

Add logic to return predefined.CONTRACT_REVERT JSON-RPC error with the revert reason in case of contract revert
Extend predefined.CONTRACT_REVERT to cover more edge cases when decoding error messages
Extend tests for decodeErrorMessage and predefined.CONTRACT_REVERT

Related issue(s):
Fixes #2830
Fixes #2821
Notes for reviewer:

Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

Shouldn't we include some tests for this new scenario?

Good point, I will extend the tests for eth_estimateGas

Nit: I think we should the remove the "Fixes #2821" from this PR's description, and replace it by something like "Related to #2821". The reason is that we wouldn't want to close #2821 when we merge this PR, but instead close it when the Axelar team can effectively confirm the issue is solved, which is when we make a new release containing this fix.


Phenotype -  CYP2C9, diplotype *2/*3

Hi Steven,
According to CPIC guidelines, diplotype *2/*3 (CYP2C9)  should be assigned poor_metabolizer instead of intermediate metabolizer.

Hi @ankita86, I'm sorry it took a while to get back to you. I finally updated the phenotyper() method for CYP2C9 as you can see below:
from pypgx.phenotyper import phenotyper
print(phenotyper("cyp2c9", "*2", "*3"))

Which gives:
poor_metabolizer

This change has been made in the 0.3.0-dev branch if you want to use the updated version of pypgx.

Please feel free to let me know if you have any questions.


是否支持删除指定的log类型

是否支支持删除指定的log类型，例如
include: ['log', 'warn']
或者通过注释不删除某行log
// exclude log console.log('这行不会被删除')
因为transform.js是被压缩过的，看不到源码

Yes, but I think warn and error if written should be what the user wants to be printed

Currently supported, please upgrade to the latest version


Referredresourceupdate

Improving the referred resource update approach, skipping the spec not changed cases.

/lgtm

/approve


Node.js should use multi cores.

Normally Node.js runs with a single process usually it's enough but with heavy load works it is required to use hardware resources such as multi-cores. We can use all CPU resources using cluster module.

I will apply cluster module in server.js.

@hyungheo, I'm closing this issue.


Post Verification Setting

http://www.elkarte.net/community/index.php?topic=3304.msg22076;topicseen#new
(0 for no limit, moderators are exempt)
Problem is, being exempt does not seem to apply to Global Moderators.

I was looking into this and it's rather odd: apparently the moment is_moderator is set allowedTo('moderate_board) returns false, even if it is allowed, but later the same call returns true.
So... looking at where and what changes in between...


Ignores compilerOptions.generate

Describe the bug
@jakobrosenberg and I are trying to generate two builds of a vite app and we can't get v-p-s to always force ssr to false
Jake came up with the following workaround:
https://github.com/roxiness/routify.dev/commit/e8638a8ca48fd9cf503fdd60a608217853b27720#diff-58e6f63d87181b1c6a8cb6e5f1691df04aa32854456efcd52ca71c8541375d26R14
Reproduction
We have these scripts in the package.json:
{
    "build:client": "vite build --outDir dist/client",
    "build:server": "vite build --outDir dist/server --ssr src/main.js",
}

and trying to force ssr to always be false for svelte here:
    svelte({
      compilerOptions: { generate: "dom", format: "esm" },

Logs
https://gist.github.com/ghostdevv/0850dada3de50545b3b2e556f688afe4

System Info
System:
    OS: Linux 5.17 undefined
    CPU: (12) x64 AMD Ryzen 5 3600 6-Core Processor
    Memory: 430.21 MB / 15.62 GB
    Container: Yes
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.9.1 - ~/.nvm/versions/node/v16.9.1/bin/node
    Yarn: 1.22.18 - ~/.nvm/versions/node/v16.9.1/bin/yarn
    npm: 7.24.2 - ~/.nvm/versions/node/v16.9.1/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chromium: 101.0.4951.54
    Firefox: 100.0
  npmPackages:
    @sveltejs/vite-plugin-svelte: ^1.0.0-next.44 => 1.0.0-next.44
    svelte: ^3.48.0 => 3.48.0
    vite: ^2.9.9 => 2.9.9

Severity
annoyance

generate is indeed hardcoded: https://github.com/sveltejs/vite-plugin-svelte/blob/main/packages/vite-plugin-svelte/src/utils/compile.ts#L24
If you don't want an ssr build and instead render the result yourself with jsdom, not passing --ssr to vite build could work:
"build:server": "vite build --outDir dist/server src/main.js"

I don't think we should allow users to override generate. If the build is compiling for SSR, we should generate for SSR, otherwise it opens up a footgun for the users to break their builds. Similarly that format is always esm because the build needs ESM.
I think the workaround is the way to go for now for this special case, otherwise @dominikg had also suggested using experimental.dynamicCompilerOptions to force it to generate: 'dom' (the options aren't validated thereafter).

I don't see the footgun. It's a very specific option that nobody accidentally types. If you do add it, it does exactly what it's documented to do. It's self-explanatory. Vite has 100s of options, which, if used incorrectly, could break a build.
Pros of fixing this

predictability - options do what they say.
broader scope of use cases - eg.: faster app tests, simpler SSR and isomorphic client / server code.
enforces correct configs.
if you try to fix an unrelated issue with generate: 'dom', you will instantly know that it doesn't work.

Cons

if you break your config, you break your build (this is a pro)

It's your choice, but I would favor explicitness, transparency and predictability any day of the week. Arbitrarily changing the behavior of documented options only leads to confusion and unnecessary hacks.
At any rate, thanks for maintaining this. ❤

It is documented that generate is non-configurable. To be honest, I don't quite agree with the pros listed but it may be a matter of perspective, if you call vite build --ssr and expect a non-ssr output because of generate: 'dom', it conflicts with the intention of the command.
I'm still leaning to not fixing this. Dominik has discussed with me too of how this usecase was needed, and I understand how this is unexpected from your end, but I think it's an edge case that the workaround is the solution for now.

Maybe this plugin can throw if you've passed in one of the non-configurable options, rather than silently ignoring them?

<EMAIL_ADDRESS>has been released and includes a warning when users add essential compilerOptions to their config.
dynamicCompilerOptions can be used as an escape hatch at your own risk. Closing this as it is documented - and now warned behavior. not a bug

@bluwy is there a specific entry in the pros list you don't agree with?
As for SSR. SSR simply means prerendering HTML on the server. There's nothing edge or "non-ssr" about using a DOM bundle for that. In fact DOM bundles have been used for SSR for about a decade now.
Thanks for pointing out experimental.dynamicCompilerOptions. I'll give that a try. 🙂


README a bit confusing (parseUserAgent default)

This line:

Example: iPhone. If the option parseUserAgent is set to false, then it will return an empty string

Makes it seem like the default is true. At least to me. Totally your call. I'd suggest:

Example: iPhone. parseUserAgent is set to false by default and will return an empty string.

And how about this:

Example: iPhone. Unless you explicitly set parseUserAgent to true (which defaults to false, as mentioned below) it will always return an empty string.

Yeah, that's good too. Also, I never could get it to work : (

Did you tried like it's in here: https://github.com/rguerreiro/express-device/blob/master/example/app_with_parse.js ?

Yes. I'm looking into the code, and it looks like npm update didn't pull the latest version... Let me look closer.

Yeup, wrong version : ) My fault.


Update Changelog.md

Update the changelog to the same version the release-tag-system has.
Further explanation: I am the nuget-package maintainer. I currently see new releases only via a mail alert on changelog.md. Therefore the changelog needs to be up to date if I'm supposed to keep track of it. The alternative would be to delete changelog.md altogether, although I have not looked into any notification systems for the release tag system.

@fkorak Version 1.9.0 lives in a different branch than master. However I don't see any problem of updating the master changelog, although I won't track to changes in master branch.


python3Packages.imgtool: init at 1.10.0

Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Kindly requesting a review from @pjones, @Taneb and @NickCao as imgtool uses some of their maintained packages as dependencies.

This pull request has been mentioned on NixOS Discourse.


Alternative for older APIs

Task/Issue URL: https://app.asana.com/0/414730916066338/1202360890245130/f
Description
Fix some of the lint wranings.
Steps to test this PR
API 23

[ ] install from this branch on API 23 device
[ ] install another VPN app, eg NetGuard and enable it
[ ] launch DDG app and try to enable AppTP
[ ] verify a dialog explaining a VPN conflict appears
[ ] click on "open settings"
[ ] verify android VPN settings are opened

API < 26

[ ] install from this branch on API < 26
[ ] enable AppTP
[ ] install a game, eg. "Art of War"
[ ] open App tracking protection -> scroll down -> having problems with an app -> show all apps
[ ] verify protection for the game is disabled and it shows the byline "Known issues"

Current dependencies on/for this PR:

develop

PR #1988   👈

This comment was auto-generated by Graphite.


Unable to broadcast engines in spark

Hello,
I am attempting to follow the example seen here for using the library with spark data frames: https://microsoft.github.io/presidio/samples/deployments/spark/
However, I receive the following error which seems to stem from trying to reference the broadcasted engine in a UDF. I have tried researching this quite a bit with no luck. Any suggestions would be very helpful.
Error: pyspark.errors.exceptions.base.PySparkRuntimeError: [BROADCAST_VARIABLE_NOT_LOADED] Broadcast variable 37 not loaded.
Environment: Databricks runtime 14.3 LTS
Code: Pretty much copied from example line for line, just changed the lines in which I read the data into a data frame.

I was able to fix this by downgrading DBR  to 12.2 - Issue is most likely related to introduction of spark connect to DBR but did not confirm.


unable to download files

Describe the bug
hi, im using this package to download updated free databases
I follow all steps but i receive this message : Edition ID: "GeoLite2-Country" not found in maxmind.com.
This message is in Client.php # 160.
To Reproduce
This is what i done:

Create file geoupdate.php by example in documentation
Call this script from browser :  http://mydomain.com/geoupdate.php
See Message on browser screen

Desktop (please complete the following information):

OS: window 10
Browser firefox
Version 88

Additional context
I tried to call generated url from Client.php in headers method link #226.
https://download.maxmind.com/app/geoip_download?edition_id=GeoLite2-ASN&suffix=tar.gz&license_key=mylicense
This works.
Thanks in advance

Please update tronovav / geoip2-update to the latest version. Now the latest version is v2.1.5
In this release, we have added in the answer a detailed description of the causes of errors in downloading database updates.
It seems to us that you simply made a mistake when specifying the license key in the client configuration.
Even if this is not the case, you will now see more detailed information about your cause.


Enhancement: Add more complex git output scopes to diff syntax

This commit addresses the following flaws of the diff.sublime-syntax

missing rules for git output:

deleted file mode
rename from
rename to
similarity index

the whole git output is scoped as as punctuation

the linefeed \n is not included in the git output rules which
causes bad results if a color scheme wants to highlight the
background of the git output.

This commit therefore

adds rules for the missing output messages
introduces normal keyword, constant, string scopes for the output
and encapsulates everything in a meta scope based on the existing
meta.diff.header for easier color-scheme creation.
optionally includes the \n? in any rule.

Hotfixes should be submitted to master branch. As part of the PR process,
you will be asked to provide the information necessary to make that happen.
Pull requests on new features should be submitted to dev branch and will be
regularly merged into master after evaluations over an extended period of
time.
The goal is to be able to create color schemes with nice looking git diff headers.

Cool @deathaxe


Use garden.privileged_image_plugin if GrootFS is enabled

Garden runC does not set the privileged image plugin flags if the garden.privileged_image_plugin BOSH property is not set.

Thanks, @glestaris, looks like we missed that one. Prioritizing for the Diego team to pull in.
Best,
Eric


Is there a way to shorten the number of digits generated for an id?

The 19 digits are fine, but I don't need that level of precision so I'd like to lower the amount of digits to somewhere around 12. Is this possible?

The number is always a 64 bit number. Binary 00000...00001 (64 bits) can be represented in decimal as 1. However, 01111.....111111 (The first bit is never used) simply requires 20 digits to represent. If you need less digits then you can do some bitwise arithmetic to get rid of some bits of, for example, the timestamp part. You can configure IdGen to use more or less bits for any given part (timestamp/generator/sequence) and then decide which bits you want to get rid of.
Alternatively, you could consider using a hexadecimal (which would require 16 'digits' to represent the max. value) or even Base36 (which would require 13 'digits') or base 64 (which would require 11 digits).
You could even consider using something like HumanoID to encode your ID in a few easy to remember words - given a large enough dictionary. I'm not aware of a .Net implementation, but there's bound to be one. If not, the PHP code is easily readable and portable to .Net I think.
But, quite honestly, if you worry about the number of digits it takes to represent an ID then most likely IdGen isn't for you. It's not designed to be compact or 'human friendly'. It's designed to be low-latency, distributed, uncoordinated, (roughly) time ordered, compact and highly available.

That's very informative. Thanks for the explanation.

Care to share why the requirement of 12 digits and/or what solution you've decided on?

I am developing an api with a UI that is utilizing these ids to do various operations. For example, create a duplicate of the record with a given id and change a few fields within. Unfortunately, JavaScript is incorrectly displaying the id since the id is larger than the max integer size in Javascript which then causes an invalid reference of a given id. At least, that's my understanding.
Here's an example of the IDs as shown in DB vs UI:
UI:<PHONE_NUMBER>425173000
DB:<PHONE_NUMBER>425172992
I don't necessarily need it to be 12 digits, I just need it to be a little shorter so that JavaScript doesn't mess up the data.

@JohnnySanabria Please see the FAQ. Just treat the ID's as strings instead of numbers. Other than that, what I said here still stands:

Alternatively, you could consider using a hexadecimal representation of an ID (which would require 16 'digits' to represent the max. value) or even Base36 (which would require 13 'digits') or base 64 (which would require 11 'digits').


HBASE-25718 Backport HBASE-25705 to branch-2

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 36s
Docker mode activated.

-0 :warning:
yetus
0m  6s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 47s
branch-2 passed

+1 :green_heart:
compile
0m 27s
branch-2 passed

+1 :green_heart:
shadedjars
5m 59s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 27s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 22s
the patch passed

+1 :green_heart:
compile
0m 27s
the patch passed

+1 :green_heart:
javac
0m 27s
the patch passed

+1 :green_heart:
shadedjars
6m  1s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 26s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
2m 41s
hbase-client in the patch passed.

25m 38s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3109/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3109

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux bdd161f86129 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 1d3ea38f1e

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3109/2/testReport/

Max. process+thread count
344 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3109/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 23s
Docker mode activated.

-0 :warning:
yetus
0m  6s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 53s
branch-2 passed

+1 :green_heart:
compile
0m 32s
branch-2 passed

+1 :green_heart:
shadedjars
7m 31s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 30s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 33s
the patch passed

+1 :green_heart:
compile
0m 33s
the patch passed

+1 :green_heart:
javac
0m 33s
the patch passed

+1 :green_heart:
shadedjars
7m 33s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 28s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
3m  3s
hbase-client in the patch passed.

32m 23s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3109/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3109

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 1cb6f9f829b7 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 1d3ea38f1e

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3109/2/testReport/

Max. process+thread count
211 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3109/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 12s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 39s
branch-2 passed

+1 :green_heart:
compile
1m  4s
branch-2 passed

+1 :green_heart:
checkstyle
0m 36s
branch-2 passed

+1 :green_heart:
spotbugs
1m 12s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 16s
the patch passed

+1 :green_heart:
compile
1m  0s
the patch passed

+1 :green_heart:
javac
1m  0s
the patch passed

+1 :green_heart:
checkstyle
0m 36s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
11m 55s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
1m 19s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 15s
The patch does not generate ASF License warnings.

33m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3109/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3109

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux 1a722eaa264e 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 1d3ea38f1e

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
95 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3109/2/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Added Anubhav Adhikari's introduction

@The-Lively-Developers-Community/maintainers - can someone help me resolve merge conflicts?

@anubhav1206 - thanks for adding your introduction. Make sure to join our community, by creating an issue here: https://github.com/The-Lively-Developers-Community/become-a-member :)!


Question

Does anyone knows if this directive is being used in production?

Yes, we use it in production at our company. And next time, don't use mysterious issue titles like "Question"... ;)

OK, Thanks.


Add release.yaml workflow

https://github.com/networkservicemesh/integration-tests/issues/1667

@denis-tingaikin #195


Allow to pass the profile name to use

For example, you may want to setup awscli::profile for several user accounts. For every user, the title must be unique:
awscli::profile { "awscli default profile for $user": ... }
Using the new $profile_name var, you can now manage the "default" profile (for example) for all users.

Looks a bit like my PR #23. I need to get around to writing some tests so @justindowning will merge mine in.

Oh, I missed that! @justindowning feel free to proceed as you see fit.

Closing in favor of #23.


hubot not responding to redis

my hubot rocket.chat adapter works well except for every time i started hubot, I had to reset the following items:
export ROCKETCHAT_USER=bot
export ROCKETCHAT_PASSWORD=bot
export ROCKETCHAT_AUTH=password
export HUBOT_YOUTUBE_API_KEY=<your token>

I get the feel hubot is not really responding to redis that i installed. I do have hubot-redis-brain installed as well. Is there any other setting I need to do in order to make it work? Or should i just re-install the whole thing? what might be the possible reason?

Redis isn't going to magically remember things.  You have to explicitely save them there.
Also.. Those are environment variables needed by hubot.  These are needed by the hubot adapter before it ever even gets to the point of connecting to the redis brain.

thanks, all i did is installed redis and hubot-redis-brain, nothing else. i didn't setup or add anything to environment. Unfortunately i am out of clue here, how to do and what to do? is there tutorial you know that can help me? or i need to ask hubot github community?

@jy2008 let me break it up in two parts.
export ROCKETCHAT_USER=bot
export ROCKETCHAT_PASSWORD=bot
export ROCKETCHAT_AUTH=password
export HUBOT_YOUTUBE_API_KEY=<your token>

These are environment variables... you have to set them every time.
My suggestion is to create a file called start with these contents:
#!/bin/bash

export ROCKETCHAT_USER=bot
export ROCKETCHAT_PASSWORD=bot
export ROCKETCHAT_AUTH=password
export HUBOT_YOUTUBE_API_KEY=<your token>

bin/hubot -a rocketchat

then do: chmod +x start and then you run your bot by just doing: ./start
As far as using the brain.  Please look at the hubot brain documentation.  To store something in the brain you will have to actually save it to the brain.

thank you very much, that is crystal clear! :)


[docs] Adds Badges for Docker and License

See https://microbadger.com/labels for more details

Note to self: After next release is completely done, file a PR to add commit hash to the README here.


Neoscopes not loading

I tried to install the package through packer. Everything seems to be installed correctly but when I try to use one of the plugin function I get a error loading module. Using a protected call for the plugin, printing "No neoscope" if the loading fails, I can actually confirm the plugin is not loaded.

Sorry for delay, forgot to set up a watch on the repository :)
Could you please share how you are loading the plugin? Are you requiring neoscope or neoscopes (the second one is correct).

I'm loading the plugin this way:
local ns_status_ok, ns = pcall(require, "neoscopes")
if not ns_status_ok then
    print('No neoscopes')
  return
end

After opening a file, or just launching nvim, is I inspect the :messages I can see the No neoscopes printed (and therefore neoscopes setup/keybinds do not work)

Ah, ok. Could you please try updating the plugin? I have pushed another fix yesterday which may actually fix this issue as well.

I'm not at home at the moment, I'll try to update as soon as I get back home in like 2 hours or so

Sorry for being late, finishing my work took longer than expected. I've updated and now the plugins seems to be loaded. I'll do some tests and, if that's the case, I'll close this issue. Thank you 😄

Absolutely no worries at all, thank you for reporting it and sorry for the troubles.


Is it CRLF or just LF?

The text states:
is the SHA-256 hash computed over the resource URI (scope as described in the previous section) and the string representation of the token expiry instant, separated by CRLF
Then the pseudo-code uses simply '\n':
SHA-256('https://<yournamespace>.servicebus.windows.net/'+'\n'+<PHONE_NUMBER>)
Can you make these consistent and correct?

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

ID: 13478d1d-11eb-930a-02c4-559a1ef89fa6
Version Independent ID: be36cdf9-1e5e-c47f-e699-652dc42ee2b9
Content: Azure Service Bus access control with Shared Access Signatures
Content Source: articles/service-bus-messaging/service-bus-sas.md
Service: service-bus-messaging
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@dlstucki Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

It is just \n according to https://github.com/Azure/azure-sdk-for-net/blob/4bab05144ce647cc9e704d46d3763de5f9681ee0/sdk/servicebus/Microsoft.Azure.ServiceBus/src/Primitives/SharedAccessSignatureTokenProvider.cs#L143.
The SHA-256 pseudo-code is misleading. The signing is done with HMACSHA256 using the SAS policy's primary or seconday keys as input key. (https://github.com/Azure/azure-sdk-for-net/blob/4bab05144ce647cc9e704d46d3763de5f9681ee0/sdk/servicebus/Microsoft.Azure.ServiceBus/src/Primitives/SharedAccessSignatureTokenProvider.cs#L167)

It wasn't hard for me to figure out which was correct. The point of this issue is to have the documentation fixed to be consistent and point users at the right format. Read the initial description. (I'll update title to clarify).

I just wanted to point out the source and also that the pseudo-code is at best misleading (it doesn't mention they key at all), and also I wanted to get some attention on this issue, which has been open for 11 months now.

\n is a bit unclear. However, "SHA-256('https://.servicebus.windows.net/'+'\n'+<PHONE_NUMBER>)" is not misleading, it is dead wrong.
Also a real example should be added, \n and HMACSHA256 and url encoding stuff may make much sense with an example and save a lot of time for us.

Updated this section:
https://docs.microsoft.com/en-us/azure/service-bus-messaging/service-bus-sas#generate-a-shared-access-signature-token
#please-close

Updated this section:
https://docs.microsoft.com/en-us/azure/service-bus-messaging/service-bus-sas#generate-a-shared-access-signature-token
#please-close


add workflow for publishing on push to main

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

Project coverage is 91.37%. Comparing base (15005da) to head (859e539).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage   91.37%   91.37%
=======================================
  Files           4        4
  Lines          58       58
=======================================
  Hits           53       53
  Misses          5        5

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


update PPP

Pull Request Test Coverage Report for Build 297

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

Totals

Change from base Build 296:
0.0%

Covered Lines:
1742

Relevant Lines:
3051

💛  - Coveralls


Unable to Mirror git project in Azure Devops with option "Use new URL: https://dev.azure.com..."

Environment Details

Pipeline

[x] Build

Agent Type

[x] Hosted Agent

Agent OS

[x] Linux (Image: ubuntu-latest:  Ubuntu 22.04.1)

Agent Tool Versions

git version: 2.39.1

Mirror Task Configuration

Mirror Git Repo Task version: 1.1.12
Mirror Source Repo System

[X] Azure DevOps (formerly VSTS)

[x] Using PAT for Mirror Source Repo
[ ] Mirror Source Repo is Public
[x] Using SSL Verification on Mirror Source Repo
Mirror Destination Repo System

[x] GitHub

[x] Using PAT for Mirror Destination Repo
[ ] Mirror Destination Repo is Public
[x] Using SSL Verification on Mirror Destination Repo

Description

Selected option to "Use new URL: https;//dev.azure.com/" it fails to Mirror our repositories.

yml looks like this:
trigger:
- '*'

# Also trigger on other repositories that are significant
resources:
  repositories:
    - repository: Repo
      type: git
      name: Project/Repo
      trigger:
      - '*'
  # Additional repository triggers

pool:
  vmImage: ubuntu-latest

steps:
# - script: echo $(Build.SourceBranchName)
# - script: echo $(Build.SourceBranch)
# - script: echo $(Build.Repository.Uri)
# - script: echo $(Build.Repository.Name)

- task: mirror-git-repository-vsts-task@1
  displayName: 'Mirroring'
  inputs:
    sourceGitRepositoryUri: $(Build.Repository.Uri)
    sourceGitRepositoryPersonalAccessToken: $(AzureDevopPAT)
    sourceVerifySSLCertificate: true
    destinationGitRepositoryUri: 'https://github.com/MyCompany/$(Build.Repository.Name)'
    destinationGitRepositoryPersonalAccessToken: '$(GitHubPAT)'
    destinationVerifySSLCertificate: true

Log Output
Starting: Mirroring
==============================================================================
Task         : Mirror Git Repository
Description  : A straightforward utility to mirror one Git repository to another location
Version      : 1.1.12
Author       : Swellaby
Help         : [More Information](https://github.com/swellaby/vsts-mirror-git-repository)
==============================================================================
/usr/bin/git -c http.sslVerify=true clone --mirror [https://***@myCompany<EMAIL_ADDRESS>Repo.git
Cloning into bare repository 'Repo.git'...
fatal: unable to access<EMAIL_ADDRESS>URL using bad/illegal format or missing URL
##[error]Error: The process '/usr/bin/git' failed with exit code 128
Finishing: Mirroring

@tofindoutsysadmin had the same issue, using the default clone url provided by azure devops is not working but could resolve this using the url in the adresbar, so changing it to https://dev.azure.com/myCompany/Project/_git/Repo/ (removing myCompany@) will resolve this issue


iOS - System chat - Chat with your setup specialist message not aligned and gets out of box

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

issue reproducible when validating #41290
Version Number: 1.4.79-6
Reproducible in staging?: y
Reproducible in production?: no, (the chat is not displayed to check)
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Open the app and log in with a new account
Select Manage my teams expenses on onboarding modal
Complete the onboarding flow
Tap the search icon
Open Expensify chat

Expected Result:
The message "Chat with your specialist in #admins for help" is displayed in the box
Actual Result:
The message flows out of the box. Changing device font size might help reproduce the issue.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos

https://github.com/Expensify/App/assets/38435837/efe243ea-6d4a-4738-844c-9f6fc09c7320

View all open jobs on GitHub

FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

I don't think it needs to block the deploy, but let's fix it.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The report system chat footer gets out of the box.
What is the root cause of that problem?
The container itself already sets a padding,
https://github.com/Expensify/App/blob/66b4072db8ec86a8eed2d6e765a52eae3c9c68d4/src/components/Banner.tsx#L74-L78
but the text content still overflows the banner/box. If we look at the banner text, we apply a flex1 style that should fix this issue,
https://github.com/Expensify/App/blob/66b4072db8ec86a8eed2d6e765a52eae3c9c68d4/src/components/Banner.tsx#L99-L105
but in our case, we use custom text content which doesn't have the flex1 style.
https://github.com/Expensify/App/blob/66b4072db8ec86a8eed2d6e765a52eae3c9c68d4/src/components/Banner.tsx#L94
https://github.com/Expensify/App/blob/66b4072db8ec86a8eed2d6e765a52eae3c9c68d4/src/pages/home/report/SystemChatReportFooterMessage.tsx#L73
What changes do you think we should make in order to solve the problem?

Add styles.flex1 style to the content.
https://github.com/Expensify/App/blob/66b4072db8ec86a8eed2d6e765a52eae3c9c68d4/src/pages/home/report/SystemChatReportFooterMessage.tsx#L73
Then, make sure to not render the Text component if the text is empty.
https://github.com/Expensify/App/blob/66b4072db8ec86a8eed2d6e765a52eae3c9c68d4/src/components/Banner.tsx#L96-L106
{shouldRenderHTML && text ? (
    <RenderHTML html={text} />
) : text && (
    ...

Hello, @MitchExpensify
Please invite me to slack channel.
My email<EMAIL_ADDRESS>

@shubham1206agra what do you think of these proposals?

Waiting on an a review on proposals, Melvin

Reminder set to pay - Do we need BZ steps here @shubham1206agra ?

Paid and contracts ended!


Kill non-plussed stats in otherinfo, this is useless in 2021

Card (297):
HP 4313 (4960)
ATK 1958 (2252)
RCV 408 (469)

Inherit (297):
HP 4313 (4960)
ATK 1958 (2252)
RCV 408 (469)

also figure out voice awakening!!!!!!!!!!!!!!!!!

also figure out voice awakening!!!!!!!!!!!!!!!!!


Fixes #106: set an exact version for vkbansal's eslint presets so tha…

…t we don't need a newer version of eslint's react plugin.

So I decided to use an exact version, because the newer react eslint plugin prohibits old-style proptypes. I got this error when I tried:
/home/julien/travail/git/react-contextmenu/examples/DynamicMenu.js
  1:1  error  React.PropTypes is deprecated since React 15.5.0, use the npm module prop-types instead  react/no-deprecated

I've fixed this issue myself. Thanks for the PR

I've fixed this issue myself. Thanks for the PR

I see, thanks !


Handling PyQt4.QtCore.QDate.toPyDate vs PySide2.QtCore.QDate.toPython

Hello, we are using Qt.py to help with our transition from PyQt4 to PySide2 and I've run into some problems with QDate objects. I have a dialog that needs to be able to function in PyQt4 in one environment, and in PySide2 inside of Maya 2018. It is using a QDateEdit, QDateEdit.date() returns a QDate, but in PyQt4 we need to call QDateEdit.date().toPyDate() to get a datetime object back and in PySide2 we need to call QDateEdit.date().toPython().
I tried handling this in QtSiteConfig.py with the following update_compatibility_members method:
def update_compatibility_members(members):
    """This function is called by Qt.py to modify the modules it exposes.

    Arguments:
    members (dict): The members considered by Qt.py
    """
    members['PyQt4']["QtCore.QDate"] = {
        "toPython": "QtCore.QDate.toPyDate",
    }
    members['PyQt4']["QtCore.QDateTime"] = {
        "toPython": "QtCore.QDate.toPyDate",
    }

The code is executing, print statements inside of the method are visible, but when my bindings are PyQt4 QDate objects still do not have a toPython method on them.
Any help to get this working site wide would be greatly appreciated. For now we are just using hasattr to check for toPython on the QDate, and doing QDate.toPython = QDate.toPyDate if it isn't find, but it would be nice to fix it centrally. Thanks!
Kenny

Thanks @KennethLowe, seems like a simple misaligned member in PySide2.
What I'd suggest is done is to unify them like this.
from Qt import QtCompat
QtCompat.QDate.toPyDate()

Such that this new QDate would represent a compatibility wrapper around the Qt 4 and Qt 5 versions. We can't (shouldn't) add it to the original QDate as it would mean (1) monkey patching and (2) leaving the old one behind and nearby, running the risk of either intermixing them in code or using the wrong one.
Have a look at adding this to the _compatibility_members dictionary, and see whether it produces the results you're looking for. Then a PR would trigger the automatic tests and hopefully solve the issue, then I'd be able to merge.


[tests-only][full-ci] bump web commit id upto 2022-01-07

Description

updates owncloud/web commit id for the tests

Part of

https://github.com/owncloud/QA/issues/711

Needs adjustments in the oCIS settings service acceptance tests, not all the steps about creating files/foldrs there seem to have been ported to the in the server part of (middleware) step definitions


MappingContext Becomes Incorrect During Filtering

Object A     Subobject B         Sub-subobject C     attribute
While filtering Object A, the mapping context becomes incorrect.  When it process attribute, it thinks the destination class is Sub-subobject C instead of Object A.

Can you report a breaking test case?

[https://drive.google.com/file/d/0B09xvORaY2UOeFl4UVhIV2hWLTA/view?usp=sharing]
zip file containing test case

Ok, i replicated the test case in a single class (easier to work on):
https://github.com/andreabertagnolli/orika/blob/issue-164/tests/src/main/java/ma/glasnost/orika/test/community/Issue164Test.java
the mapping context gets changed after the mapping of objA, when starts the mapping of SourceSubObjectA to SubObjectA, and it can't manage the objB mapping correctly.

At this point I wonder if the MappingContext is the right place to keep resolvedSourceType and resolvedDestinationType, because that values are changed during the mapping at different levels...
I found that those fields where added to MappingContext in this commit.
https://github.com/orika-mapper/orika/commit/70cccd551a801ee88a8daf1291a29173f21d6d97
dunno... someone's got any idea?


[antimentionspam] log to modlog

When muting the user currently no mod log case is created, it would be nice to see a modlog case created for when they are muted since i would assume most use the [p]unmute command to unmute them which does generate a modlog case so it looks odd as it appears they get unmuted for no reason lol

Changes required for this already exist (90975d0) on the branch where I'm incorporating things to be merged when Red 3.2 is available.

sounds good, i had thought of this before and forgot so figured i would write it down so i didn't forget again

Also, closing, duplicate #121


VS 2017 version

When we could expect version for VS2017?

I'm also wondering when  we could expect Umbraco.ModelsBuilder.CustomTool for VS2017

@zpqrtbnk Could you please give us some rough timeframe for this? Thanx!

Installed the RTM version of 2017 last week. Now I need to port the extension - which from what I am reading is not "immediate". We're working hard on putting together a new release of Umbraco at the moment, so I won't be able to do this extension in the coming 2-3 weeks.
The most reasonable timeframe would be approx 1 month, unless someone wants to try to do it.

Curses... 😄
Maybe I should give it a stab this weekend.
Since I've some how messed up my VS2015 installation so it won't open up any of my projects.

@zpqrtbnk
With some Iron Fist(ing) on Netflix as background noise I messed around a bit and managed to get it working.
Installs and tested it on a site running Umbraco 7.5.10

@ruant Mr.Robot is a more appropriate choice in this particular case :+1:

Excellent, need to test!

Quick Q re. the PR: I understand that it's now mandatory to build with VS 2017 (if we want to be able to target VS 2012 up to VS 2017) - is that correct?
And just because I don't have 2012 installed... you've tested the extension on 2012, right?
Planning a next minor release pretty soon and would love to merge!

@zpqrtbnk I have not tried building it with other then VS2017, but I guess VS2017 is need.
I can try it out in VS2015 later tonight, to see if it builds.
I have not tested it on 2012 no. The question is, do it need to support 2012?
I also haven't tested it in 2015 other then that the installer listed it as an option to install to during installation.
I'll check that it works with 2015 as well tonight.

I don't think we need to support 2012, no, and I'm happy killing the support entirely.
If you can test that it works on 2015 then that'd be fantastic. And then I can merge and it'll be part of the next Core minor version.
Many, many, (many) thanks for helping here!

So glad you guys are working on this, I just forked the repo to have a go and then found this issue and the work you've done so far. This is the only reason I have to keep going back to VS2015 at the moment so looking forward to the next update. I have also read that you have to build VS2017 VSIX files only in VS2017.

@zpqrtbnk
Glad to help 👍
Had a quick look at it yesterday.
I didn't mannage to make it build in 2015.
But I'm no expert, so it might be possible to get it to work.
Worth mentioning is that my VS2015 installation seams to be a bit screwed up after I installed VS2017.
So that might be the reason also.
Regarding testing the tool in VS2015 it's the same, VS2015 won't open up any of our projects, but my collegues manages. Same code 😕
So if anyone can test out the VSIX built in VS2017, on VS2015 and confirm that it works, that would be awesome.
Installer

I'll give it a go.

"All VSIX projects will require a major version one-way upgrade to Visual Studio 2017."

...which might explain why you can't now build it in VS2015. See https://docs.microsoft.com/en-us/visualstudio/extensibility/how-to-migrate-extensibility-projects-to-visual-studio-2017

@zpqrtbnk @ruant I have installed it and used it with no visible errors on the following:
Microsoft Visual Studio Enterprise 2015 (Version 14.0.25421.03 Update 3)
Microsoft Visual Studio Enterprise 2017 Version 15.0.26228.10 D15RTWSVC
I hope that helps!

Thanks all for your helps, considering it all OK, will merge asap.

Merged - note that the solution will open in VS2015, except for the VSIX project which will be marked as incompatible - VS2017 now required to build the whole thing.
A HUGE thank you to you all.

Well I have an issue with the VSIX that is produced, the Visual Studio Gallery refuses it, and it cannot install on my machine, says that "the file is not a valid VSIX package" with the following log:
29/03/2017 19:00:50 - -------------------------------------------
29/03/2017 19:00:50 - vsixinstaller.exe version:
29/03/2017 19:00:50 - 15.0.26228.0 built by: D15REL
29/03/2017 19:00:50 - -------------------------------------------
29/03/2017 19:00:50 - Command line parameters:
29/03/2017 19:00:50 - C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\Common7\IDE\VSIXInstaller.exe,D:\d\Zbu ModelsBuilder\build\Release\v3.0.6\Umbraco.ModelsBuilder.CustomTool-3.0.6.vsix
29/03/2017 19:00:50 - -------------------------------------------
29/03/2017 19:00:50 - Microsoft VSIX Installer
29/03/2017 19:00:50 - -------------------------------------------
29/03/2017 19:00:50 - Failed to initialize settings for Isolated Shell C:\Program Files (x86)\Microsoft SQL Server\130\Tools\Binn\ManagementStudio\ssms.exe: Impossible de trouver le point d'entrée '?' dans la DLL 'PkgdefMgmt.dll'.
29/03/2017 19:00:50 - Initializing Install...
29/03/2017 19:00:50 - Microsoft.VisualStudio.ExtensionManager.InvalidExtensionPackageException: The file is not a valid VSIX package.
   à Microsoft.VisualStudio.ExtensionManager.InstallableExtensionImpl.ReadVSIXManifestFromPackage(Stream stream, CultureInfo preferredCulture)
   à Microsoft.VisualStudio.ExtensionManager.InstallableExtensionImpl..ctor(String path, CultureInfo preferredCulture)
   à Microsoft.VisualStudio.ExtensionManager.ExtensionEngineImpl.CreateInstallableExtension(String extensionPath, CultureInfo preferredCulture)
   à Microsoft.VisualStudio.ExtensionManager.ExtensionEngineImpl.CreateInstallableExtension(String extensionPath)
   à VSIXInstaller.App.GetInstallableData(String vsixPath, Boolean isRepairSupported, IEnumerable`1& skuData)
   à VSIXInstaller.App.Initialize(Boolean isRepairSupported)
   à VSIXInstaller.App.Initialize()
   à System.Threading.Tasks.Task`1.InnerInvoke()
   à System.Threading.Tasks.Task.Execute()

No idea what it means exactly, reopening the issue...

This is exactly this issue and no, the VSIX does not contain manifest.json nor catalog.json. I've read the migration notes but it's not helping so far...

Oops, it may be that our build script still references VS2015... looking into it...

Turns out it's the build script we're using to release MB that has issues. Fixed somehow, version 3.0.6 has been uploaded to NuGet and to Visual Studio Gallery. Should be part of next minor release of Cms (see U4-9683).


android cannot compile ,gradle error  KOTLIN_STAT_ENABLED_PROPERTY, but two days ago is compiled normal

Description

What went wrong:
A problem occurred evaluating project ':app'.

KOTLIN_STAT_ENABLED_PROPERTY

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.

What went wrong:
A problem occurred configuring project ':app'.

compileSdkVersion is not specified. Please add it to build.gradle

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
==============================================================================

Get more help at https://help.gradle.org/

React Native Version
0.71.3
Output of npx react-native info
System:
OS: macOS 12.6.1
CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
Memory: 72.66 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 16.16.0 - /usr/local/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 8.11.0 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.1, iOS 16.1, macOS 13.0, tvOS 16.1, watchOS 9.1
Android SDK: Not Found
IDEs:
Android Studio: 2022.1 AI-221.6008.13.2211.9619390
Xcode: 14.1/14B47b - /usr/bin/xcodebuild
Languages:
Java: 11.0.2 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.3 => 0.71.3
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
npm install && yarn android
Snack, code example, screenshot, or link to a repository
d

I couldn't reproduce this locally with 0.71.4 which suggests something is up with your development environment.
Does npx react-native doctor indicate everything is fine with your environment?
My Env for Reference:
❯ yarn android
yarn run v1.22.19
$ react-native run-android
warn Multiple Podfiles were found: ios/Podfile,vendor/bundle/ruby/2.7.0/gems/cocoapods-core-1.12.0/lib/cocoapods-core/Podfile. Choosing ios/Podfile automatically. If you would like to select a different one, you can configure it via "project.ios.sourceDir". You can learn more about it here: https://github.com/react-native-community/cli/blob/master/docs/configuration.md
info Starting JS server...
info Launching emulator...
info Successfully launched emulator.
info Installing the app...
Starting a Gradle Daemon (subsequent builds will be faster)

> Task :app:installDebug
Installing APK 'app-debug.apk' on 'Pixel_4_API_31(AVD) - 12' for :app:debug
Installed on 1 device.

BUILD SUCCESSFUL in 3m 21s
38 actionable tasks: 38 executed
info Connecting to the development server...
8081
info Starting the app on "emulator-5554"...
Starting: Intent { cmp=com.current/.MainActivity }
✨  Done in 207.98s.
❯ npx react-native info
info Fetching system and libraries information...
System:
    OS: macOS 13.2.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 259.09 MB / 32.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 17.9.0 - ~/.nvm/versions/node/v17.9.0/bin/node
    Yarn: 1.22.19 - ~/.yarn/bin/yarn
    npm: 8.18.0 - ~/.nvm/versions/node/v17.9.0/bin/npm
    Watchman: 20230306 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.12.0 - /Users/blakef/.rvm/gems/ruby-2.7.6/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
    Android SDK:
      API Levels: 26, 27, 28, 29, 30, 31, 32, 33
      Build Tools: 23.0.1, 25.0.2, 27.0.3, 28.0.2, 28.0.3, 29.0.2, 30.0.2, 30.0.3, 31.0.0, 33.0.0
      System Images: android-31 | Google APIs ARM 64 v8a, android-31 | Google Play ARM 64 v8a
      Android NDK: Not Found
  IDEs:
    Android Studio: 2022.1 AI-221.6008.13.2211.9514443
    Xcode: 14.2/14C18 - /usr/bin/xcodebuild
  Languages:
    Java: javac 12 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.2.0 => 18.2.0
    react-native: 0.71.4 => 0.71.4
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found
❯ yarn list react-native
yarn list v1.22.19
warning Filtering by arguments is deprecated. Please use the pattern option instead.
└─<EMAIL_ADDRESS>✨  Done in 0.18s.


Fix pip command

Fix the target of the pip install command, since requirements.txt is in the root.
Tested manually:
  ~/workspace/chatbot-rag-app                                                                                ✘ INT  chatbot-rag-app  14:25:16
❯ pip install -r requirements.txt

Collecting aiohttp==3.8.5 (from -r requirements.txt (line 7))
  Downloading aiohttp-3.8.5-cp311-cp311-macosx_11_0_arm64.whl.metadata (7.7 kB)
Collecting aiosignal==1.3.1 (from -r requirements.txt (line 13))
  Downloading aiosignal-1.3.1-py3-none-any.whl.metadata (4.0 kB)
...

Thanks!


GDScript: no error when there is a trailing comma in function arguments

Godot version: v4.0.dev.custom_build 35ff38b3d
OS/device including version: windows 10
Issue description:
func f(p, ): ## <-- no errors
	print(p)

func _ready():
	f(0, ) ## <-- no error

When running with the GDScript test runner for this snippet, generates this output file:
Parser bug: invalid inheritance.

This may be done by design. Trailing commas in function declarations and calls make it possible to write more diff-friendly code:
# If we add one more argument, we have to modify only 1 line.
hello(
    "foo",
    "bar",
    "baz",
)

# If we add one more argument, we have to modify 2 lines.
hello(
    "foo",
    "bar",
    "baz"
)

Therefore, I would prefer this behavior to stay in :slightly_smiling_face:

Yes, this is intended. Many users asked for this so I added it.

I guess the problem comes from having this inline over multiple lines, but if that's difficult to solve then perhaps not worth it.


feat(lang-server): Add autocomplete for shadowDatabaseUrl

@albertoperdomo Can you check to see if the documentation needs more?
Closes #708

Sorry for dropping the ball. LGTM!

Oh no issues at all! I only assigned you last night :D
I figured if there were changed, I could do it in another PR!


fix #18

All tests are green, so I guess this fixes it then?

@modulovalue, thank you for your PR!
While your solution does fix the issue, it drops some really useful type information. I've created PR #22, which seems to fix the issues with all types retained.


Cluster image links are broken

The cluster images are still pointing to the old source

I also noticed that the demo code is referencing the wrong resources, so looks like this would help!

I signed it!

I had this problem today too. You can configure the images path like this (or set an URL):
var mcOptions = {
				imagePath: '../images/m'
			};
var markerCluster = new MarkerClusterer(map, markers, mcOptions);

fixed by 2f659a3

fixed by 2f659a3


[IZPACK-1238] UserInputPanel: input fields not focused automatically on panel activation - post-fix

File/dir input fields still didn't receive the initial focus correctly.
This small change fixes this problem for them.

IzPack project » izpack #751 SUCCESS
This pull request looks good
(what's this?)


Logo size to be increased

Hi,
Ritika here.
Below attached is the picture of website and I noticed that the logo on our website could be larger to enhance visibility as the main focus. Additionally, making the home, about, and features icons clickable with a hover effect for color change would improve interactivity. @Tejashri-Taral and @sanjay-kv
Please assign me this task.

@Tejashri-Taral Why won't the issue get fixed. There is a possiblity so as to make the visiblity to new users about the logo. Please take the consideration and let me work on this task


Add environment to container name

It's helpful for logging purposes to distinguish between different deployment environments, so add that to the container name, like "project-staging".

Did you mean to PR this against newrelic/centurion?

Uh, yes.


Take Over Brightness Option

A friend and I have the same issue..
sometimes we want to make it darker or brighter in the room that we are in and have it stay at the set brightness.. however we would like AL to keep adapting the color of the lights, but not the brightness..
This can be achieved by looking for manual events where the brightness is set.. and switch off the AL Adapt Brightness switch.. but this is a big pain to do..
Would it be possible to have an option like Take Over Control - but only prevents AL from changing the brightness (or turns off the adapt brightness switch) if the brightness is manually controlled?
kinda in the same topic but can be a different request.. it would be nice if there was a manual control switch , or if when taking over control (existing) it turns off the adaptive lighting switch..  it’s kinda hard to know when you are still taking over control.. but like I said a different request.. but it would be nice for take over brightness to use that kind of logic ..

There is a switch you will have to do what you want, see the 3rd and 4th points here https://github.com/basnijholt/adaptive-lighting?tab=readme-ov-file#bulb-features

It would be two automations per zone, looking for turn on events with brightness, across 7 zones.. it ends up being a lot to add in.. but it AL is already listing to events to figure out if manual control is overriding the AL, it seems like I could be a check added into AL without much overhead.. I think there would be a large benefit to many to have this feature implemented.. but that is just my opinion.

I Agree with zpriddy. Most people don't change the color temperatures of there lamps manually. Having a checkbox with 'Keep colortemperature synced' would be an awesome feature.

I was trying to setup some automation for this.. However the only solution would be to trigger off of every state_changed event or any. state change event from every adaptive lighting switch. Then add a condition to the automation to see if there are any lights in the manual control config.. and if there are.. then set the color temp to the value in the trigger event/switch value using the transition period in the event..
However that is a LOT of events that get sent depending on the number and configuration of your AL setup.. Im not quite sure its very feasible to take that approach.. At the very least its a lot of automation triggers to be running..

Hi, I join the group who would appreciate the independence of brightness and color temperature adaptivity. For now I use this set up as a workaround, hopefully it will help others as an idea.
Goal

Adapt brightness for each room/light group separately.
When the brightness is changed (for example a scene is set), disable adapting brightness, however continue with adapting the color temperature.
Adapt color temperature regardless the brightness settings.
Have the same color temperature in the whole house at the time.

Set up

I have a new Entry for every room/light group (like hallway, kitchen spotlights, kitchen dinner table...). In this instance I define the main settings for the brightness adaptation behaviour.
I have checked take_over_control.
In the instance entities I disable Adapting Color switch.
I have created a virtual light in configuration.yaml, for this I need 3 Helpers (input_number.virtual_light_brightness, input_number.virtual_light_color_temp, input_boolean.virtual_light_state).

light:
- platform: template
  lights:
    virtual_light:
      unique_id: "virtual_light"
      friendly_name: "Virtual light"
      value_template: "{{ states('input_boolean.virtual_light_state') }}"
      level_template: "{{ states('input_number.virtual_light_brightness') | int }}"
      temperature_template: "{{ states('input_number.virtual_light_color_temp') | int }}"
      turn_on:
        service: input_boolean.turn_on
        entity_id: input_boolean.virtual_light_state
      turn_off:
        service: input_boolean.turn_off
        entity_id: input_boolean.virtual_light_state
      set_level:
        service: input_number.set_value
        target:
          entity_id: input_number.virtual_light_brightness
        data_template:
          value: "{{ brightness }}"
      set_temperature:
        service: input_number.set_value
        target:
          entity_id: input_number.virtual_light_color_temp
        data_template:
          value: "{{ color_temp }}"

And I have created a new instance of Adaptive Lighting for that Virtual light where I set up the default behaviour for the color temperature adaptation. The great benefit is, that it can be different to the brightness settings of other instances.
Also, the brightness entity switch might be disabled to have no updates with any effect.
I keep turned on the Virtual light. In its parameters, the adapted values are hold. I am interested especially in color_temp.
I have a new automation to update the color_temp of any light when turned on, or when Virtual light color_temp has changed.

alias: "Update color temp by Virtual light"
description: ""
trigger:
  - platform: state
    entity_id:
      - light.virtual_light
    attribute: color_temp_kelvin
    id: Color update
  - platform: state
    entity_id:
      - light.kitchen_dinner_table_light_1
      - light.kitchen_dinner_table_light_2
      - light.kitchen_dinner_table_light_3
      - light.kitchen_dinner_table_light_4
      - light.kitchen_spot_light_1
      - .... // define your list of all lights. It's needed to keep it descriptive here, otherwise you would need to listen to all events which might be too crazy.
    from: "off"
    to: "on"
    id: Some light is turned on
condition: []
action:
  - choose:
      - conditions:
          - condition: trigger
            id:
              - Color update
        sequence:
          - service: script.set_color_if_lights_on
            data:
              kelvin: "{{ state_attr('light.virtual_light', 'color_temp_kelvin') }}"
              entity_ids:
                - light.kitchen_dinner_table_light_1
                - light.kitchen_dinner_table_light_2
                - light.kitchen_dinner_table_light_3
                - light.kitchen_dinner_table_light_4
                - light.kitchen_spot_light_1
                - ... // define your list of all lights
      - conditions:
          - condition: trigger
            id:
              - Some light is turned on
        sequence:
          - service: script.set_color_if_lights_on
            data:
              kelvin: "{{ state_attr('light.virtual_light', 'color_temp_kelvin') }}"
              entity_ids: "{{ trigger.entity_id }}"
            enabled: true
mode: queued
max: 10

For the updating only turned on lights I use this script:

alias: Set Color if Lights On
sequence:
  - variables:
      kelvin: "{{ kelvin }}"
      light_entities: "{{ entity_ids }}"
      entities_to_regulate: >
        {{ expand(light_entities) | selectattr('state', 'eq', 'on') |
        map(attribute='entity_id') | list }}
  - condition: template
    value_template: "{{ entities_to_regulate | count > 0 }}"
  - service: light.turn_on
    data_template:
      entity_id: "{{ entities_to_regulate | join(', ') }}"
    data:
      kelvin: "{{ kelvin }}"
mode: restart

To have a correct behaviour when setting a scene, I need manually update the scene definition in scenes.yaml to omit all the color change definitions. I keep there only brightness definitions. Otherwise the scene would change the color to something pre-defined and in x seconds (default is 90 in Adaptive lighting) the color will be changed back to the Virtual light color temperature which is annoying.

Know limitations

The only limitation I observe is that during the night, when the color temperature of Virtual light is set to the minimum, there is no update of the color_temp, which causes no update of the manually changed color of any light back to the correct Virtual light value.

I have one LED strip which I normally leave at 50% brightness because it is overall much brighter than my other bulbs. adaptive_lighting.apply turns everything to the same brightness whenever it's activated; I'd prefer if it didn't touch the brightness control at all. Having brightness controlled manually, while color control is still automatically managed, would be super useful for me.


ValueError: Attempted to map inputs that were not found in graph_def: [input:0]

Hello!
I have asked this question in stackoverflow but at this point it merits a long post.
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
NA
TensorFlow installed from (source or binary):
pip
TensorFlow version (use command below):
1.8
Python version:
Python 2.7.12
Bazel version (if compiling from source):
NA
GCC/Compiler version (if compiling from source):
NA
CUDA/cuDNN version:
9.2
GPU model and memory:
7611MiB (gpu nvidia p4)
Exact command to reproduce:
ValueError: Attempted to map inputs that were not found in graph_def: [input:0]

Describe the problem
Trying to create a frozen graph, technically succeeding at doing that but then unable to run it. I am unsure about what the tags do. I want to use the model for inference,  but wasn't able to load it in a frozen graph in training and serving mode.  I suppose I have to restore the session? the documentation does not show that. And I double checked my output node's name is softmax. I saw the frozen graph in tensorboard and looks fine, but does not load into the graph. Help? I found people with issues like this but none resolved or their issues I already fixed in my code.
Source code / logs
This is the code to freeze from and not frozen  .pb file and its variables.
import tensorflow as tf
export_dir='../catmod'
with tf.Session(graph=tf.Graph()) as sess:
    # I have not been able to find how the tags affect this. Does training work? I tried as well with serve
    # serve gave the same error of input:0  * I checked with saved_model_cli
    tf.saved_model.loader.load(sess, [tf.saved_model.tag_constants.TRAINING], export_dir)
    output_graph = "frozen_graph.pb"
    # changing nodes from refswitch to switch then moving it accordingly
    for node in gd.node:
        if node.op == 'RefSwitch':
            node.op = 'Switch'
            for index in xrange(len(node.input)):
                if 'moving_' in node.input[index]:
                    node.input[index] = node.input[index] + '/read'
        elif node.op == 'AssignSub':
            node.op = 'Sub'
            if 'use_locking' in node.attr: del node.attr['use_locking']
    gd = sess.graph.as_graph_def()
    output_nodes=['softmax']
    output_graph_def = tf.graph_util.convert_variables_to_constants(
       sess, # The session is used to retrieve the weights
       gd,
       output_nodes# The output node names are used to select the usefull nodes
    )

    # Finally we serialize and dump the output graph to the filesystem
    with tf.gfile.GFile(output_graph, "wb") as f:
        f.write(output_graph_def.SerializeToString())
sess.close()

This code is functional as I have used it with pre-frozen graphs.  (parts of) Code to load the frozen graph which I used with other frozen graphs that I got from the internet:
import tensorflow as tf
import numpy as np
from tensorflow.python.platform import gfile
from tensorflow.python.client import timeline
from tensorflow.python.util import compat

### Loading the model

def getModel(pb_frozen):
    with gfile.FastGFile(pb_frozen,'rb') as f:
        graph_def = tf.GraphDef()
        graph_def.ParseFromString(f.read())
    return graph_def

# Running the model
tf.logging.info("Starting execution")

gpu_options = tf.GPUOptions(per_process_gpu_memory_fraction=0.50)
tf.reset_default_graph()

g = tf.Graph()

dummy_input = np.random.random_sample((batch_size,224,224,3))
outlist=[]
# creating the graph
with g.as_default():
    inc=tf.constant(dummy_input, dtype=tf.float32)
    dataset=tf.data.Dataset.from_tensors(inc)
    dataset=dataset.repeat()
    iterator=dataset.make_one_shot_iterator()
    next_element=iterator.get_next()
    out = tf.import_graph_def(
      graph_def=  getModel(pb_frozen), # loading graph
      input_map={"input":next_element},
      return_elements=['dense/bias']
    )
    out = out[0].outputs[0]
    outlist.append(out)

And then is run in a session.

I'm not sure I fully understand what you're trying to do here - the code you have to manipulate the graph before writing it out seems curious (for example, why are  all the RefSwitch nodes being changed to Switch?).
Perhaps you can describe the higher level objective here and we can figure out a more appropriate path to it - modifying individual nodes in the graph seems fishy.
As for the error message itself, it is suggesting that there is no node named input in the the graph defined by pb_frozen.
Also, perhaps some clarification around the SavedModel / frozen graph may be in order.
Here's a really quick summary:

A GraphDef defines a computation, which may depend on parameters (tf.Variables in Python). The GraphDef does not contain the value of those parameters.
A checkpoint contains a map from variables referenced in a GraphDef to their values.
A SavedModel packages the GraphDef and the checkpoint into a single bundle.
A "frozen" graph is a term used to represent a graph that fully defines a model, i.e., there is no checkpoint needed since all the variables in the model have been "frozen" into constant tensors that can be included in the GraphDef.

Actually, a SavedModel can package multiple graphs. Use the saved_model_cli command-line tool to inspect the contents of a SavedModel . See documentation. Using that tool, you'll be able to determine which tag is appropriate, what the names of the input and output tensors are etc.
Hope that helps.

Hello!
I modified the nodes because I had a bug when they weren't switch and I found on stackoverflow that the way of overcoming that bug is to just switch them individually.
Anyway the big picture:
I have various trained models in checkpoints that I then export into a pb/variables file which needs to then be converted into a frozen graph for me to then use a graph optimizer. So far I had been working with frozen graphs online and didn't expect this process of freezing a model from checkpoints to be this complicated.
I checked the tags of my pb files with that comment and I have one with serve and the other with train tags. I put them through the code above with only the last layer name and managed to make visualizable graph through tensorboard that I can't load using the second part of the code. At this point I am pretty sure that when I am exporting the graph to a saved pb or when I am freezing it there is an issue creating the input layer such that i can load it an then do inference with it.
Thank you!
Nyla

Nagging Assignee @asimshankar: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @asimshankar: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @asimshankar: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @asimshankar: It has been 59 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @asimshankar: It has been 74 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @asimshankar: It has been 89 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @asimshankar: It has been 104 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.