Transform f1_elc_op_mnt_expn xbrl + dbf

Steps:

[x] Map #2114
[ ] Transform

Table Notes

No instant, only duration.
Almost 200 XBRL factoids / DBF row numbers, omg.
Big changes in 2006. Lots of row mapping to do.
Another "last year" and "this year" table.

Transformation Plan

Can probably keep only the current year balance to avoid redundancy.
Simple and tedious to row map.
Output will only have a single column of dollar values.
There are a few sign conventions to apply in transform_main() for the moment.
Bring in FERC account numbers as metadata where we can.
Several groups / subgroups to aggregate via association table or calculation metadata.

Mapping instructions:
The following values should be put in the xbrl_factoid  column of dbf_to_xbrl.csv, in association with the analogous DBF row_number / row_literal value. They can be used more than once, if the same DBF value is associated with different rows in different years that appear in that file.
In the row_type column, categorize each DBF row as one of:

header
reported_value
calculated_value

In the notes column, indicate anything unusual about the mapping (for human eyes only).
administrative_and_general_expenses
administrative_and_general_operation_expense
administrative_and_general_salaries
administrative_expenses_transferred_credit
advertising_expenses
allowances
ancillary_services_market_administration
capacity_market_administration
coolants_and_water
customer_account_expenses
customer_assistance_expenses
customer_installations_expenses
customer_records_and_collection_expenses
customer_service_and_information_expenses
day_ahead_and_real_time_market_administration
demonstrating_and_selling_expenses
distribution_expenses
distribution_maintenance_expense_electric
distribution_operation_expenses_electric
duplicate_charges_credit
electric_expenses_hydraulic_power_generation
electric_expenses_nuclear_power_generation
electric_expenses_steam_power_generation
employee_pensions_and_benefits
franchise_requirements
fuel
fuel_steam_power_generation
general_advertising_expenses
generation_expenses
generation_interconnection_studies
hydraulic_expenses
hydraulic_power_generation_maintenance_expense
hydraulic_power_generation_operations_expense
informational_and_instructional_advertising_expenses
injuries_and_damages
load_dispatch_monitor_and_operate_transmission_system
load_dispatch_reliability
load_dispatch_transmission_service_and_scheduling
load_dispatching
maintenance_of_boiler_plant_steam_power_generation
maintenance_of_communication_equipment_electric_transmission
maintenance_of_communication_equipment_regional_market_expenses
maintenance_of_computer_hardware
maintenance_of_computer_hardware_transmission
maintenance_of_computer_software
maintenance_of_computer_software_transmission
maintenance_of_electric_plant_hydraulic_power_generation
maintenance_of_electric_plant_nuclear_power_generation
maintenance_of_electric_plant_steam_power_generation
maintenance_of_energy_storage_equipment
maintenance_of_energy_storage_equipment_other_power_generation
maintenance_of_energy_storage_equipment_transmission
maintenance_of_general_plant
maintenance_of_generating_and_electric_plant
maintenance_of_line_transformers
maintenance_of_meters
maintenance_of_miscellaneous_distribution_plant
maintenance_of_miscellaneous_hydraulic_plant
maintenance_of_miscellaneous_market_operation_plant
maintenance_of_miscellaneous_nuclear_plant
maintenance_of_miscellaneous_other_power_generation_plant
maintenance_of_miscellaneous_regional_transmission_plant
maintenance_of_miscellaneous_steam_plant
maintenance_of_miscellaneous_transmission_plant
maintenance_of_overhead_lines
maintenance_of_overhead_lines_transmission
maintenance_of_reactor_plant_equipment_nuclear_power_generation
maintenance_of_reservoirs_dams_and_waterways
maintenance_of_station_equipment
maintenance_of_station_equipment_transmission
maintenance_of_street_lighting_and_signal_systems
maintenance_of_structures
maintenance_of_structures_and_improvements_regional_market_expenses
maintenance_of_structures_distribution_expense
maintenance_of_structures_hydraulic_power_generation
maintenance_of_structures_nuclear_power_generation
maintenance_of_structures_steam_power_generation
maintenance_of_structures_transmission_expense
maintenance_of_underground_lines
maintenance_of_underground_lines_transmission
maintenance_supervision_and_engineering
maintenance_supervision_and_engineering_electric_transmission_expenses
maintenance_supervision_and_engineering_hydraulic_power_generation
maintenance_supervision_and_engineering_nuclear_power_generation
maintenance_supervision_and_engineering_other_power_generation
maintenance_supervision_and_engineering_steam_power_generation
market_facilitation_monitoring_and_compliance_services
market_monitoring_and_compliance
meter_expenses
meter_reading_expenses
miscellaneous_customer_accounts_expenses
miscellaneous_customer_service_and_informational_expenses
miscellaneous_distribution_expenses
miscellaneous_general_expenses
miscellaneous_hydraulic_power_generation_expenses
miscellaneous_nuclear_power_expenses
miscellaneous_other_power_generation_expenses
miscellaneous_sales_expenses
miscellaneous_steam_power_expenses
miscellaneous_transmission_expenses
nuclear_fuel_expense
nuclear_power_generation_maintenance_expense
nuclear_power_generation_operations_expense
office_supplies_and_expenses
operation_of_energy_storage_equipment
operation_of_energy_storage_equipment_distribution
operation_of_energy_storage_equipment_transmission_expense
operation_supervision
operation_supervision_and_engineering_distribution_expense
operation_supervision_and_engineering_electric_transmission_expenses
operation_supervision_and_engineering_hydraulic_power_generation
operation_supervision_and_engineering_nuclear_power_generation
operation_supervision_and_engineering_other_power_generation
operation_supervision_and_engineering_steam_power_generation
operations_and_maintenance_expenses_electric
other_expenses_other_power_supply_expenses
other_power_generation_maintenance_expense
other_power_generation_operations_expense
other_power_supply_expense
outside_services_employed
overhead_line_expense
overhead_line_expenses
power_production_expenses
power_production_expenses_hydraulic_power
power_production_expenses_nuclear_power
power_production_expenses_other_power
power_production_expenses_steam_power
power_purchased_for_storage_operations
property_insurance
purchased_power
regional_market_expenses
regional_market_maintenance_expense
regional_market_operation_expense
regulatory_commission_expenses
reliability_planning_and_standards_development
reliability_planning_and_standards_development_services
rents_administrative_and_general_expense
rents_distribution_expense
rents_hydraulic_power_generation
rents_nuclear_power_generation
rents_other_power_generation
rents_regional_market_expenses
rents_steam_power_generation
rents_transmission_electric_expense
sales_expenses
scheduling_system_control_and_dispatch_services
station_expenses_distribution
station_expenses_transmission_expense
steam_expenses_nuclear_power_generation
steam_expenses_steam_power_generation
steam_from_other_sources
steam_from_other_sources_nuclear_power_generation
steam_power_generation_maintenance_expense
steam_power_generation_operations_expense
steam_transferred_credit
steam_transferred_credit_nuclear_power_generation
street_lighting_and_signal_system_expenses
supervision_customer_account_expenses
supervision_customer_service_and_information_expenses
supervision_sales_expense
system_control_and_load_dispatching_electric
transmission_expenses
transmission_maintenance_expense_electric
transmission_of_electricity_by_others
transmission_operation_expense
transmission_rights_market_administration
transmission_service_studies
uncollectible_accounts
underground_line_expenses
underground_line_expenses_transmission_expense
water_for_power

If we fully normalize the data we'll end up with one column called something like annual_o_and_m_expense and another called expense_type. This is cleaner than having a zillion columns, but it causes all of the once column headers to become column values. This means that they don't get defined in the fields.py module and might be harder to search for in our documentation. How do we feel about this? @zaneselvans @cmgosnell

We've talked about this a little bit, and I think we're going to want to do something analogous to the coding tables, but with a process that converts the XBRL metadata into a table explaining what all the different value types mean, which hopefully applies generally across many different tables with the same kinds of transforms happening to them, so we don't have to manage all the metadata by hand like we do with the column annotations now.

I made an issue for this #2138


[docs] Improve the color description in the API pages

Closes https://github.com/mui/material-ui/issues/29688
There was confusion reported for the color prop, regarding why the string value is allowed.
Once we agree on the exact text we want to include, I will add it on the other components too.

It's a bit "wordy" as it's currently written. I think we can tighten it up while also clarifying. Am I understanding correctly that the string value is optional, and is used for defining custom theme colors?

@mbrookes @samuelsycamore can you check the last update? I haven't run the scripts, so the text was not updated in all files. If it is good, will use the same on all components that supports the color prop with string value.


Add home page

In this pull request, I have :

[x] Added home page.
[x] Styled the application.
[x] Used React Router to make the routes work.

Thank you @TedLivist, I truly appreciate your time and effort in reviewing my work 😃


[Menubar Weather] ...

Extension
https://www.raycast.com/koinzhang/menubar-weather
Description

It seems normal in the setting. I can see the last refresh time, but it just shows "no weather info, try again later" on the menubar.
I've tried restarting raycast, the extension, network and even my mac, yet nothing changed.
macOS 12.6 (21G115)
Steps To Reproduce
macOS 12.6 (21G115)
Raycast 1.56.3
menubar weather is the latest version
Current Behaviour
No response
Expected Behaviour
No response

Hi @Antinino. Have you tried uninstalling and reinstalling this extension? Maybe it's due to some internal cached key changes that need to be cleared from the extension cache.

Hi @Antinino. Have you tried uninstalling and reinstalling this extension? Maybe it's due to some internal cached key changes that need to be cleared from the extension cache.

It works after reinstalling! THX for the reply.


Speed up Healpix query

Need to look into what exactly is meant with this comment from the workshop

The "multi" mode for the empirical selection function uses the uniq scheme and np.searchsorted.


Renamed categories

Change tailoring to "Cloth" instead of Tailoring and "Vendor Reagents" instead of Reagents.

fixed in 1.8.1


map2, map3 and friends don't preserve all error messages

For instance:
decoder =
    Decode.map3 (\a b c -> ( a, b, c ))
        (Decode.field "bar" Decode.int)
        (Decode.field "foo" Decode.int)
        (Decode.column 3 Decode.int)

contents =
    """bar
1
2
d
d
5
"""

Decode.decodeCsv Decode.FieldNamesFromFirstRow decoder contents

You would expect that for rows 1,2, and 5 you would get 2 errors (foo not provided, column 3 not found) and 3 errors for rows 3 and 4, but instead you just get 1 error for each row.
This is because they are implemented as Result.map2, Result.map3 etc, which only take the first encountered error, rather than aggregating them as a list.

I was looking at doing a fix for this, but I'm somewhat confused. The decoding error is { row : Int, column: Column, problems: List Problem }, but I can't quite work out when is it possible to have any other number than exactly one problem?
I think we'd need to change the type to look more like List { row : Int, column: Column, problem: Problem } to support this.


Throws error if task not saved

HI,
this extension works really well, but only if the task was saved before. If i create a new task in recent kanboard version, and paste an image, i get an error:
Internal Error: SQL Error: SQLSTATE[22P02]: Invalid text representation: 7 ERROR:  invalid input syntax for type integer: ""
this does not occure if the task was saved to board before. should be fixed by readme, or code or whatever

hmmm  can you please post your kanboard configuration?
I've put a little bit of effort into the possibility of pasting images into new tasks, so it generally should work

heyo, I just fixed it ✌


Sass Challenge:Use @each to Map Over Items in a List

Challenge Name
Use @each to Map Over Items in a List

https://beta.freecodecamp.org/en/challenges/sass/use-each-to-map-over-items-in-a-list
Issue Description

My code clearly works and provide the right result in the preview section but for some reason I don't
understand I don't pass the challenge.
Browser Information

Browser Name, Version:  Chrome 64.0.3282.186
Operating System: Windows 7
Mobile, Desktop, or Tablet:

Your Code

Screenshot

Looking in the source it appears the test is missing the . required for class selectors
See below the assert is using blue-bg and should be .blue-bg
"assert($('blue-bg').css('background-color') == 'rgb(0, 0, 255)', 'message: Your .blue-bg class should have a background-color of blue.');",

Do you know if anybody is going to fix this issue?

@sjames1958gm Nice catch there! Are you interested in fixing the tests?
Here is the Contributing Guide that you should refer to, and the Contributors chat room if you need help.


Maven Deploy process

Define a process, or CI job, whereby we can deploy artifacts to Sonatype from a tag to prevent the need to burn through version numbers because Sonatype is unavailable

This is already handled by https://github.com/smallrye/smallrye/wiki/Release-Process


perf: improve reader and parser by using iterators instead of string view operations

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (c33df45) 99.39% compared to head (6cdac31) 99.39%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #57   +/-   ##
=======================================
  Coverage   99.39%   99.39%
=======================================
  Files          55       55
  Lines        5654     5655    +1
=======================================
+ Hits         5620     5621    +1
  Misses         34       34

Files Changed
Coverage Δ

include/emio/detail/conversion.hpp
100.00% <ø> (ø)

include/emio/ranges.hpp
95.65% <ø> (-0.45%)
:arrow_down:

include/emio/detail/format/formatter.hpp
98.71% <100.00%> (ø)

include/emio/detail/format/parser.hpp
100.00% <100.00%> (ø)

include/emio/detail/parser.hpp
97.81% <100.00%> (+0.46%)
:arrow_up:

include/emio/detail/scan/parser.hpp
100.00% <100.00%> (ø)

include/emio/detail/scan/scanner.hpp
99.41% <100.00%> (ø)

include/emio/reader.hpp
100.00% <100.00%> (ø)

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

Overall code size even reduced by 320 bytes! :) [Except format_int test - +20 bytes].
Performance is quit good as well!

format nothing but long text now takes ~0.400 μs instead of 12.5 μs.
different between compile-time and runtime string is reduced (e.g. ~20 μs instead of ~40 μs in format big hex).
emio seems to outperform snprintf nearly everywhere (except double & format nothing but long text).
And now even outperforms fmt when formatting complex hex values


Make Titan/JanusGraph poll for Interrupted on parallelMutate

https://github.com/awslabs/dynamodb-titan-storage-backend/blob/1.0.0/src/main/java/com/amazon/titan/diskstorage/dynamodb/DynamoDBDelegate.java#L319

@kgenge Here is the backlog item for improving Titan/JanusGraph handling of interrupts.
related to #50


[rtl/rv_core_ibex] Fix RTL assertion for CDC

Increase upper bound for cycles to wait in IbexIcacheScrambleKeyRequestAfterFenceI_A.
Partially addresses #16689

This change goes in an RTL file, but it is really more of a DV issue, since the affected code is not synthesized.


Help decoding multiple messages using rust by resolving #987

Closes #987
Say multiple messages were encoded into a continuous byte buffer.  When trying to decode the first message, we might first use a MessageHeaderDecoder to get its templateId at offset=0, then decode its body using correspoded message decoder at offset+=headerLength.  This is ok for current message decoder's header method with an implicit header offset=0.  But when trying to decoder the second and following messages, the MessageHeaderDecoder step is ok however the body decoding step using header method would fetch bytes always from the first message body, thus is not expected.
This unexpected result occurs only when using message decoder's header method to gain ownership.  Use message decoder's wrap method behaves as expected.
This pr is just a simple workaround, simple but works (see tests/issue_987_test.rs).

Thanks @wbprime, I think this looks good.  Can someone apply this PR?


$CurrentIndex() issue on first load

Hi,
I need to get the current slide index, but on first load it return -1.
Below my code:
$(document).ready(function () {
   var galleryOptions = {
          $Loop: 1,
          $DragOrientation: 0,
          $SlideDuration: 500,
          $ArrowNavigatorOptions: {
               $Class: $JssorArrowNavigator$,
               $ChanceToShow: 2,
               $Steps: 1
           },
          $ThumbnailNavigatorOptions: {
               $Class: $JssorThumbnailNavigator$,
               $ChanceToShow: 2,
               $ActionMode: 1,
               $SpacingX: 5,
               $Cols: 6,
               $Align: 0
         }
   };
});

var gallery = new $JssorSlider$('gallery', galleryOptions);

var currentSlideIndex = gallery.$CurrentIndex(); // return -1

I can't figure out why. Do you have any idea?
Thanks!

I think you can get the $CurrentIndex after a few mini seconds.

The issue has been fixed.


IGAPP-1333: Upgrade mapbox

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.

Request failed due to a permanent error: stream was reset

See here. I think this message was already logged before I switched to maplibre. And as maplibre is a mapbox (now rnmapbox) fork the tickets also apply to this project.


Impossible to enable the GNSS sub-module

Hello.
I have an issue using GNSS sub-module.
As I understand, to use the GNSS I need to enable 1.8V FET switching GPIO5 to ON state (according to your schematic). But when I try to do it, I get an error:

So, when I try to get information about supported GPIO pins and I got the following:

Information on the device PCB: XY-A7608 2022-7-18 V1.2
Modem information:
Modem: A7608E-H
Modem Name: A7608E-H
ATI
SMS DONE
Manufacturer: INCORPORATED
Model: A7608E-H
Revision: A7600M7_V5.0
IMEI:<PHONE_NUMBER>82459
+GCAP: +CGSM,+FCLASS,+DS
OK
I have bought it on 2023/03/23 on aliexpress in the official store. I have chosen the version with GPS (T-A7608E-H With GPS)

https://github.com/Xinyuan-LilyGO/T-A7608X/blob/6948e4b3aaa5b3c99b2eb59e57a9626872016d85/examples/Arduino/GPS_BuiltIn/GPS_BuiltIn.ino#L277


fixes #39 (inconsistent fonts on page)

Turns out the issue is a result of taking zeplin at its word, css-wise.  Unfortunately, its auto-generated, overly-specified boilerplate css doesn't include backup fonts, thus defaulting to Times New Roman if one doesn't have Muli ... it also references css variables that we don't have defined :(. I've tried to excise the bits that are either breaking things or unused in the page.  Now looks like this:

Thanks!!


Feature: db.insert function

Is your feature request related to a problem?
There is no easy way to insert multiple records. This can made easier by creating a new method specifically for performing insert operations.
Describe the solution
A new method could be added to the SDK which could have the following signature:
db.insert(table: str, data: Union[Dict, List[Dict]], options: Optional[?])

This would allow you to insert a singular record or a list of records.
Alternative methods
Currently the method that we are using is to either generate a prepared string to do an insert statement:
db.query(```
INSERT INTO company {
	name: 'SurrealDB',
	founded: "2021-09-10",
	founders: [person:tobie, person:jaime],
	tags: ['big data', 'database']
};```)

or
db.query(```
INSERT INTO person [
   {id: "person:jaime", name: "Jaime", surname: "Morgan Hitchcock"},
   {id: "person:tobie", name: "Tobie", surname: "Morgan Hitchcock"},
];```)

SurrealDB version
1.0.0+20230913.54aedcd for macos on x86_64
surrealdb.py version
surrealdb.py  0.3.1 for macOS on x86_64 using Python 3.11.2
Contact Details
No response
Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hey @MatthewAry thank you for the suggestion. We are working on the rust/python client and we also want to ensure that the features across all clients are going to be the same. This does not mean that a batch insert is a bad idea, we will put it to the team and work the feasibility of supporting the batch inserts in general. However, I appreciate that this is not helpful to you right as this time, so below is a function that will generate the SQL statement for a batch insert based off a list of dicts:
def construct_batch_insert(table_name, records):
    """
    Construct a batch insert SQL query for a custom SQL syntax as provided in the example.

    Args:
    table_name (str): The name of the table into which records will be inserted.
    records (list of dict): A list of dictionaries, each representing a row to insert.

    Returns:
    str: A SQL query string for batch insertion.
    """
    # Start the query with the table name
    query = f"INSERT INTO {table_name} [\n"

    # Add each record as a dictionary string
    record_strings = []
    for record in records:
        # Convert each record dictionary to a string in the format: {key: "value", key: "value"}
        record_str = ', '.join([f'{key}: "{value}"' for key, value in record.items()])
        record_strings.append(f"   {{{record_str}}}")

    # Join all record strings with a comma and a newline
    query += ',\n'.join(record_strings)

    # Close the bracket for the record list and end the query with a semicolon
    query += "\n];"

    return query

# Example usage
records = [
    {"id": "person:jaime", "name": "Jaime", "surname": "Morgan Hitchcock"},
    {"id": "person:tobie", "name": "Tobie", "surname": "Morgan Hitchcock"}
]

# Generate the query
query = construct_batch_insert("person", records)
print(query)

The code will give the following printout:
INSERT INTO person [
   {id: "person:jaime", name: "Jaime", surname: "Morgan Hitchcock"},
   {id: "person:tobie", name: "Tobie", surname: "Morgan Hitchcock"}
];

This means that your batch insert can be the following:
db.insert(construct_batch_insert("person", [
    {"id": "person:jaime", "name": "Jaime", "surname": "Morgan Hitchcock"},
    {"id": "person:tobie", "name": "Tobie", "surname": "Morgan Hitchcock"}
]))

Does this approach suit you for now while we work on incorporating batch inserts?

This is great. Thank you for making this. I know that is not a native solution and that developing one is not feasible right now but this does meet my needs for now!

Not sure if this helps when someone needs but the function will not work with version 2 because Surrealdb will not eagerly parse the string to the record type, and it will not work with types like array, object .. Here is my updated code (This code is tested in my project, and is generated by Chatgpt, so use it at your own risks)
`def construct_batch_insert(table_name, records):
"""
Construct a batch insert SQL query for a custom SQL syntax as provided in the example.
Args:
table_name (str): The name of the table into which records will be inserted.
records (list of dict): A list of dictionaries, each representing a row to insert.

Returns:
str: A SQL query string for batch insertion.
"""

def format_value(value):
    """Recursively format values, handling strings, numbers, lists, and dicts."""
    if isinstance(value, str):
        if value.startswith("<record<language>>"):
            # Keep this pattern as unquoted
            return value
        else:
            # Wrap other strings in quotes
            return f'"{value}"'
    elif isinstance(value, (int, float)):
        # Return numbers as-is
        return str(value)
    elif isinstance(value, list):
        # Recursively format lists
        return "[" + ", ".join([format_value(item) for item in value]) + "]"
    elif isinstance(value, dict):
        # Recursively format dicts
        return "{" + ", ".join([f"{k}: {format_value(v)}" for k, v in value.items()]) + "}"
    else:
        # Default case for other types
        return repr(value)

# Start the query with the table name
query = f"INSERT INTO {table_name} [\n"

# Add each record as a dictionary string
record_strings = []
for record in records:
    record_str = []
    for key, value in record.items():
        formatted_value = format_value(value)
        record_str.append(f"{key}: {formatted_value}")

    # Join the key-value pairs and format as a dictionary string
    record_strings.append(f"   {{{', '.join(record_str)}}}")

# Join all record strings with a comma and a newline
query += ",\n".join(record_strings)

# Close the bracket for the record list and end the query with a semicolon
query += "\n];"

return query`


CORS origin is undefined.

cors was working for me yesterday, today I have woken up and now CORS has stopped working and it looks like whenever I do a request the origin is undefined.
Heres my code:
app.use(cors({
  origin: function(origin, callback) {
    console.log('origin', origin); //This prints "origin undefined"
    var originIsWhitelisted = insertWhitelistedLogic();
    console.log(originIsWhitelisted );
    callback(null, originIsWhitelisted);
  },
  credentials: true
}));

I'm having the same issue. Just started today. Not sure what's up.

Can you try console.logging request.header.origin? I am getting the same result as you guys, but I think this module isn't responsible for the definedness of the request object.
app.use((req,res,next) => {    console.log(req.headers.origin) // undefined    next() })
This may help:
http://stackoverflow.com/questions/29531521/req-headers-origin-is-undefined

Same problem here.  I believe the value we would want is in request.headers.referer, no?
origin used to work, so I'm not sure what changed.

Am also having the same problem, i need to restrict access to some end points, but can not identify the origin. Already use JWT to authenticate, but would like to have another level of access control by origin. request.headers.referer return absolute URI,  anyone got a solution for undefined origin?
Tks

Hi guys!
What do you think about this approach???
Just change the line 223 from:
originCallback(req.headers.origin, function (err2, origin) {
to:
originCallback(req.headers.origin || req.headers.host, function (err2, origin){
Tested on browsers, it woks! I didn't test on requests from servers...

Origin is where it comes from, host is the current domain. They are much different things, and if it falls back to host it should always be true.

Same problem here. anybody have any clue how to fix this?

SAME HERE

is your browser sending the origin header?

For POST type request yes and for GET type no.

Any solution on this issue? I am experiencing the same issue with @bharat-dpd

Hi,
Facing same issue "For POST type request yes and for GET type no." any fix?

What i did to resolve this is below
app.use(function(req,res,next){ 			req.headers.origin = req.headers.origin || req.headers.host; 			next(); 		})
I know this is not a best approach to resolve this. just overriding the origin with host if not exist.

There is not enough information in this thread to determine what is going on. All the proposed changes above are against the CORS specification, which we did not make up. If anyone is still having an issue, I suggest opening a new issue and provide enough detail to reproduce the issue completely, including things like version of this module, version of Node.js, complete code to reproduce and complete instructions for how to reproduce.


Feature: automatically configure memory_limiter and GOMEMLIMIT

The memory_limiter processor has the following guidance under Best Practices:

It is highly recommended to configure the GOMEMLIMIT environment variable as well as the memory_limiter processor on every collector. GOMEMLIMIT should be set to 80% of the hard memory limit of your collector. For the memory_limiter processor, the best practice is to add it as the first processor in a pipeline. This is to ensure that backpressure can be sent to applicable receivers and minimize the likelihood of dropped data when the memory_limiter gets triggered.

This guidance is implemented in the official Helm chart for the OTel Collector:

https://github.com/open-telemetry/opentelemetry-helm-charts/blob/8e489da14e034a66b23b0fc8b80cd1a82af21b74/charts/opentelemetry-collector/templates/_config.tpl#L21
https://github.com/open-telemetry/opentelemetry-helm-charts/blob/8e489da14e034a66b23b0fc8b80cd1a82af21b74/charts/opentelemetry-collector/templates/_config.tpl#L4-L13
https://github.com/open-telemetry/opentelemetry-helm-charts/blob/8e489da14e034a66b23b0fc8b80cd1a82af21b74/charts/opentelemetry-collector/values.yaml#L289-L292

We could explore doing something similar, potentially substituting their use of K8s' resources.limits.memory with BPM's limits.memory: https://github.com/cloudfoundry/bpm-release/blob/master/docs/config.md#limits-schema.
The UX for this feature would be a new Job property of something like memory that allows an operator to specify the maximum amount of memory the OTel Collector process should use. Based on that, we could:

Set BPM's limits.memory.
Configure GOMEMLIMIT to 80% of limits.memory.
Configure the memory_limiter processor if the operator hasn't already done it via the Job's config property (similar to how the Helm chart allows overriding the auto-configuration).

We would want to see if BPM implements limits.memory via cgroups and in a way that is compatible for the memory_limiter processor's limit_percentage and spike_limit_percentage config options. Otherwise, we would need to use the non-percentage config options and manually calculate the values.

Implemented (1) and (2) in #43, thanks @oppegard! We're somewhat reticent to implement (3) right now, as it seems a little heavy-handed.
Closing this as completed for now.


Added some checks to ensure there is a requirements tree before walking it

Pre-Flight checklist

[X] Testing

[X] Code is tested
[X] Changes have been manually tested

What are the relevant tickets?
Fixes #1222
What's this PR do?
Adds a check to see if there is a requirements tree for the program before generating the course ID list. (If the program record predated the requirements tree, there wouldn't be a root node and you'd get a 500 error.)
How should this be manually tested?

Create a program.
Manually delete the requirements tree nodes associated with that program. (For local testing, I did delete from courses_programrequirement where program_id = ? where ? is the program ID in a database shell.)
Try to load the program via the API: /api/programs/<program id>/ - it should work properly, with an empty set for the requirements list.

Codecov Report

Merging #1223 (f7311af) into main (9c98c7b) will increase coverage by 4.95%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1223      +/-   ##
==========================================
+ Coverage   84.08%   89.04%   +4.95%
==========================================
  Files         296      145     -151
  Lines       11608     3907    -7701
  Branches      801        0     -801
==========================================
- Hits         9761     3479    -6282
+ Misses       1624      428    -1196
+ Partials      223        0     -223

Impacted Files
Coverage Δ

courses/serializers.py

courses/widgets.py

authentication/utils.py

main/serializers.py

micromasters_import/admin.py

users/urls.py

main/message.py

courses/admin.py

main/models.py

users/admin.py

... and 141 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Adding method to stop a test

This change is

@stephen-soltesz can we add this?

Sure thing I'll look into adding a few tests

@stephen-soltesz I don't have permission to merge, can you please do it? thanks

@meggamind I've invited you as a collaborator for this repo. This should make some things easier.
General guidelines we all strive for:

be kind
make things a little better
don't break everything at once

Welcome.


Add UPDATE_EXPECT envvar to update tests

Perhaps ensure that the environment variable has the name of the test to be copied (or maybe a list separated by commas/spaces for multiple tests)? So that wrong tests aren't accidentally copied.

Too complicated IMO, that's what git is for. The workflow for these kinds of things is "use version control to see what changed"

Too complicated IMO, that's what git is for. The workflow for these kinds of things is "use version control to see what changed"

Not sure if I follow; the idea is simply to have greater control upon which tests are copied, as sometimes you are implementing your own tests while others are failing (and you'll tackle that later), for example. Without this flexibility, you will just have to resort to the old way of doing it.
Also, to see what changed, just run the tests - the failing ones have (clearly) changed (and they can't be distinguished, in principle!).

sometimes you are implementing your own tests while others are failing

Not sure why this would be a problem. If you make a change and it updates tests it wasn't supposed to, then you can either revert in git or simply fix the code, re-run the tests, and the wrongly changed images will be reverted back. So the idea is that the tests always match what your code is doing and if some of them shouldn't have changed, they'll get updated back to the originals once you fix your code.


Update info about IT Way

Hi, I'm the founder of IT Way.
Just updated info about it.
NOTE: I think it should be squashed

Thanks!


How to set  css , js or images files responses ?

How to set css , js or images files responses ?

Static content should be served by a dedicated HTTP server (nginx, apache) in front of the reactphp application.
However, if the content is generated in your application, then you'd only need to return a response with the appropriate content headers (text/css, application/javascript, image/png, etc) and just give the string as an argument to the request body.
Moreover, if you want to read the file from disk and send it as a response, preferably you'd have to setup a loop that allows async disk I/O and pipe the stream to the response.

Thanks @ssipos90! I believe this has been answered and I'm closing this for now, please feel free to come back to this ticket with more details if this problem persists and we can reopen this :+1:


feat(frontend): enable suggested sepolia tokens

Motivation
With https://github.com/dfinity/oisy-wallet/pull/3337 we enable USDC for users that have not yet touched the "enabling" of this token. This MR implements the same behaviour for its testnet token SepoliaUSDC and ckSepoliaUSDC.
Changes

add sepolia tokens of USDC to suggested constants

Tests
https://github.com/user-attachments/assets/1e77e90e-b87b-408f-a75f-f17406b25c69
local test with ...(nonNullish(LOCAL_CKUSDC_LEDGER_CANISTER_ID) ? [LOCAL_CKUSDC_LEDGER_CANISTER_ID] : []):

Chain Fusion should NOT contain test tokens.
(Blocking just because this is something really important and because Antonio already started reviewing as well)

Ok, I was not aware of this. Would you mind explaining it for my benefit?
So I guess I'll close this MR, or do we want to have SEPOLIA_USDC_TOKEN suggested and i just remove the ckSepoliaUSDC suggestion?

Would you mind explaining it for my benefit?

It was a product decision.
It was also a technical decision:
a. This facilitated the introduction of the concept of "Chain Fusion" on a codebase that required a network as a mandatory option.
b. It helps prevent issues on networks where addresses are identical for production and test environments. Specifically, it helps prevent bugs that could, for example, result in transferring tokens on the incorrect network as lont as a comprehensive test suite is not in place.

Closing this PR. see https://github.com/dfinity/oisy-wallet/pull/3337 for MR without ckSepolia


Count-in selection frequency when offering closest matches

Hi,
Thanks for this great tool, it's one of the few I use frequently for some time now.
What would be very cool for me is to give more weight/importance to the most-selected history entries.
It should not be too difficult to do it, but it probably would be much faster for you to do it, than for me, since you already know the code.
The good thing is that you don't have to make this weight (the count of recent selections) persistent across sessions. For the first implementation, you might lose the weight when the shell is closed, and it would still be a very good improvement over the current code.
I'm not sure if it's clear what I'd like to see, so let me know if you would like some clarifications.

Please note that the new go version sorts the result regardless of the number of matches. --sort=N number is silently ignored.


More diverse pollen views

I am not sure what weather service the card was made for originally, but pollen allergies are normally for certain species of plants and not the entire type.
That means it would be nice to have the option to show tree species, like larch, birch, hazel and so on.
Since there will be different trees and flowers, then a single icon would not do and finding icons for the different species and make people recognize them would be hard, so a text option would be nice too.

I have to bring up this old post again because I am also curious if there is a way to show more than 3 pollen views. Since I got 6 sensors displaying various trees and other. There is no documentation for the pollen part here.
Has somebody figured out a way to display more pollen views?


I wrote some OpenRC scripts to run MCSManager as a service with OpenRC. Should I open a pull request?

I have adapted the systemd service unit files into OpenRC scripts to run on my server. I figured I may as well share them while I am at it. This enables MCSManager to be run as a service on OpenRC based Linux distributions like Alpine Linux. Alpine Linux is popular for containers like LXC and Docker. This could open up the possibility for making MCSManager deployable via Docker and maybe a TurnKey LXC LinuX Container. I need to look a bit deeper at TurnKey before being completely certain of that.
I am testing this on Alpine Linux 3.18.2 (with nodejs 18.16.1), but it should run on any version that has a compatible version of nodejs. The oldest compatible version of Alpine Linux seems to be 3.13, as it ships with nodejs version 14.20.1 (the README.md specifies 14.17.6 or newer). Alpine Linux 3.12 is a bit too old, since it ships with nodejs version 12.22.12 (the nodejs-current package is only 14.5.0).
The daemon script gets saved at /etc/init.d/mcsm-daemon:
#!/sbin/openrc-run

name="MCSManager Daemon"
command="/usr/bin/node /opt/mcsmanager/daemon/app.js"
command_background="yes"
directory="/opt/mcsmanager/daemon"
pidfile="/var/run/mcsm-daemon.pid"
output_log="/var/log/mcsm/daemon.log"
error_log="/var/log/mcsm/daemon.err"
supervisor="supervise-daemon"

depend() {
	need net localmount
	use logger
	after firewall
}

The web script gets saved at /etc/init.d/mcsm-web:
#!/sbin/openrc-run

name="MCSManager Web"
command="/usr/bin/node /opt/mcsmanager/web/app.js"
command_background="yes"
directory="/opt/mcsmanager/web"
pidfile="/var/run/mcsm-web.pid"
output_log="/var/log/mcsm/web.log"
error_log="/var/log/mcsm/web.err"
supervisor="supervise-daemon"

depend() {
	need net localmount
	use logger
	after firewall
}

I can also open a pull request for updating the English documentation on how to use the OpenRC scripts, if it is desired. There are a few more small steps beyond just saving the above files into their respective directories.

Of course, you can submit PR in this repository.
https://github.com/MCSManager/Documentation
You can create a new section based on this file:
https://github.com/MCSManager/Documentation/blob/main/_sidebar.md


streamdiceJS : Encryption is not using numbers and most special characters in ciphertext messages

This bug refers to the JavaScript implementation of streamdice found in ./streamdiceJS/
Quick test of bug
Go to streamdiceJS web and type a long Lorem Ipsum. Encrypted message does not include numbers nor special characters, even though these characters are also being shuffled in the keyboard hashmap. This may be a result of some escape character, but it could also be a flaw or omittance in the program.

Fixed!


ite/it8xxx2: add support for jtag debug interface

If enabled, jtag pins will be configured as debug interface at chip startup.

@keith-zephyr Please revisit @GTLin08 Please review


feat:updated kafka to 20.0.2

Description
updated kafka to 20.0.2

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


Add CLI doc updates for GitLab

These 2 docs need updates:

https://docs.mattermost.com/manage/command-line-tools.html#use-the-cli-on-gitlab-omnibus
https://docs.gitlab.com/ee/integration/mattermost/#mattermost-command-line-tools-cli (I'm not sure who normally owns updating this ... so let me know any concerns!)

More context here: https://gitlab.com/gitlab-org/omnibus-gitlab/-/issues/6490#note_745347313.
Let me know any questions!

@justinegeffen @cwarnermm how can I help move this forward. The deadline for GitLab 14.6 (where MM 6.1 is going to be included) is drawing close.

Thanks, @nab-77! A deadline for this one wasn't clear, so I appreciate knowing the urgency on this one. I'll review today and follow up with you about any gaps or information I need assistance with to complete these doc updates by end of day.

Please let me know how I can help.

Using the GitLab OAuth2 integration with Mattermost: No formal documentation specifies how to use mmctl auth login --strict URL --name HOST --access-token STRING properly. Which token? From where? I found a help written for developers using the API with some guidance on how to acquire a Mattermost token. Sadly, Mattermost session tokens are not able to be created from within Mattermost because the GitLab integration appears to disable the "Manage Tokens" features completely--- even for Super Attempting to use an Auth Token from within GitLab fails as does using a web-browser session token. Basically, without the mattermost command, based on the scant information available and scanning the source code, it appears to me right now that mmctl cannot even be used against GitLab + Mattermost installations? This issue is likely MUCH more urgent than just for the upcoming GitLab release.


Could documentation show in-notebook animation with embed=False?

The documentation shows two modes for using JSAnimation.
Could the documentation show how to arrange for a third case: that where the control panel and display stay inline in the IPython Notebook, but where the individual frames are written to separate files? That would let me embed an animation in a notebook but without ballooning my .ipynb file to many megabytes in size, which would let me version-control it more effectively than if it contains megabytes of difference every time I slightly adjust a 100-frame animation.
Thanks for writing JSAnimation!

Just a big thanks for the JSAnimation you have coded. I have re-used it as a template modified by jinja2 template engine to animate frames produced outside matplotlib.
I have proposed it on the ipython project  as an issue https://github.com/ipython/ipython/issues/8184


[WIP] v4.0.0 Release Summary

NOTE: v4.0.0 is under development on develop_v4 branch.
Current: 4.0.0-alpha1
New architecture: more, more easier to use

Simpler workflow and out of the box features
Support adding the effect component at runtime
The effect for TextMeshPro does not require pre-generated materials
Use texcoord2 and texcoord3 as default

Shader stripping

Remove unnecessary shader variants on preprocess (for Unity 2018.2 or later)

Support Unity Package Manager (UPM)

You can install UIEffect using UPM, or install under Packages directory

UIEffectCapturedImage will be separated from UIEffect

For faster release
UIEffectCapturedImage will be refactored as another project and work independently
You can install it using UPM, or install under Packages directory

UIEffect will no longer support Unity 2017 or earlier

For faster release
It may work but is not recommended

Documentation

Usage of all features
API reference
Demos

Loadmap for v4.0.0
v4.0.0-alpha1 (19/07/31)

Experiment of new architecture
Support Unity Package Manager (UPM)
UIEffectCapturedImage will be separated from UIEffect
Simple demo

v4.0.0-alpha2 (19/08/x)

Shader stripping

Built asset size test

Fix bugs

v4.0.0-alpha3 (19/x/x)

Demo and test
Documentation

Use docFx

Fix bugs
Refactor

v4.0.0-beta (19/x/x)

Fix bugs

v4.0.0 (19/x/x)

Release

Hello there,
Is there any update for the new version?

Hi @buihuuloc
Sorry for development so late.
I am busy and I cannot spend much time on this project... X(
I will release new alpha version in the near future.

Sorry for my late reply.
UIEffect v5 is now released.
Please try it. :+1:
🔄 Upgrading from v4 to v5


📃: ONLINE BOOKSTORE

🔴 Title :creating online bookstore
🔴 Tech stack :HTML.CSS
🔴 Objective :Create a responsive homepage layout for an online bookstore using HTML (frames and other elements) and CSS.
🔴 Summary :
📸 Screenshots

✅ Details to Include When Taking the Issue:
Name :
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.):

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

@iamrahulmahato I am unable to push the code  its showing permission denied.what should i do about it


Failed test run return BUILD SUCCESS

if tests failed, maven return BUILD SUCCESS
and jenkins does not mark build as Failed

invalid


Not showing up in the Formbuilder (October 2.0.23)

Expected behaviour
I would like to see the awesomeiconslist in the Widget List of the Form Builder.
Actual behaviour
It does not appear in the Widget List. If i add it to the Yaml File, the Form Builder says "unknown control type: awesomeiconslist".
Steps to reproduce the behaviour
October 2 with builder plugin. Try to add the awesomeiconslist to the form.

It is just the form builder which shows the array. The icon list appears in the form, if i execute it. Sorry :)

No worries! :)
Glad it worked (I assume)


Frontend code cleanup

In addition to https://github.com/tonythomas01/gdrive_to_commons/issues/9, do some more frontend code cleanup. Run HTML, JS, CSS files through relevant linters and fix corresponding issues.

Closing due to wide scope. Also we integrated prettier to take care of linting.


Support for generic components

First of all, great work on this library.
I was wondering, since components utilizing

This would be particularly helpful for getting type hints in scoped slots, and should be very easy to implement. I can work on a PR.

Here's the PR. https://github.com/MaxLeiter/sortablejs-vue3/pull/113


(TextureButton) Texture miniature does not update when redrawn in external program

Godot version
4.2 dev 5
System information
Wiindows 10
Issue description
So I've changed texture in Photoshop, and it updated everywhere except here:

Also if I delete that texture and drop it again problem is still there.
Reloading project fixes problem :)
Steps to reproduce
Try to add texture, then change it in some graphic program and save. Come back to Godot.
Minimal reproduction project

I suspect it has something to do with Editor resource preview's preview cache mechanism.


Generic Components

What cheatsheet is this about? (if applicable)
Advanced cheatsheet
What's your issue or idea?
We should probably have a short section on authoring generic components -- components that take generic prop interfaces (or even state interfaces).

This should include how these work with higher-order components. As far as I know you loose generic type information when you wrap generic components. Which is somewhat obvious if you think about it. However it can be easily forgotten if you publish these components in a library.

absolutely. i just havent had time to get to this yet. feel free to take the lead if you want to, if not i'll definitely get to it in coming months

Hi,
I think this has been added in the Advanced Readme & HOC Readme
Covers Generic Prop & State Interfaces:
https://github.com/typescript-cheatsheets/react-typescript-cheatsheet/blob/master/ADVANCED.md#generic-components
Covers Higher Order Components:
https://github.com/typescript-cheatsheets/react-typescript-cheatsheet/blob/master/HOC.md#docs-example-use-hocs-for-cross-cutting-concerns
Please do go through and let us know if anything remains. Closing this issue for now, please don't hesitate to re-open if anything needs changes.


Removed RNs from the description of KLAY

RNs are not part of Klaytn.
Proposed changes
Updated the description of KLAY by removing the term RN, which is not part of Klaytn anymore.
Types of changes
Please put an x in the boxes related to your change.

[x] Minor Issues and Typos
[ ] Major Content Contribution
[ ] Others

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to reach out. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING GUIDELINES
[x] I have signed the CLA
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Related issues
N/A
Further comments
N/A

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


python311Packages.django-tastypie: 0.14.5 -> 0.14.6

Diff: https://github.com/django-tastypie/django-tastypie/compare/refs/tags/v0.14.5...v0.14.6
Changelog: https://github.com/django-tastypie/django-tastypie/releases/tag/v0.14.6
Description of changes

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 253059 run on x86_64-linux 1

4 packages built:

python310Packages.django-tastypie
python310Packages.django-tastypie.dist
python311Packages.django-tastypie
python311Packages.django-tastypie.dist


Add simple icu visualizer.

One drawback is that icu has options about its namespace.
Sometimes it could be icu, sometimes icu_Version (e.g. icu_61).

Sometimes it could be icu, sometimes icu_Version (e.g. icu_61).

I think we can support it only by using AlternativeType. So please support also icu namespace

Done.


A bug in blender amass

hi, when I apply the 07_amass.py, error occurs.
type object 'XRFeitoriaBlenderFactory' has no attribute 'apply_motion_data_to_actor'
File "E:\Project\xrfeitoria\xrfeitoria\utils\functions\blender_functions.py", line 69

Sorry, for that. Please try to use dev_plugins first:
git clone https://github.com/openxrlab/xrfeitoria.git
cd xrfeitoria
pip install .
python -c "import xrfeitoria as xf; xf.init_blender(replace_plugin=True, dev_plugin=True)"

This would be a formal feature when a new version is released.

Sorry, for that. Please try to use dev_plugins first:
git clone https://github.com/openxrlab/xrfeitoria.git
cd xrfeitoria
pip install .
python -c "import xrfeitoria as xf; xf.init_blender(replace_plugin=True, dev_plugin=True)"

This would be a formal feature when a new version is released.

Sorry to interrupt, I run your command and add replace_plugin=True, dev_plugin=True to blender.07_amass.py. But there's a new error ImportError: cannot import name 'Dict' from 'xrfeitoria.utils.functions.blender_functions'. Could you tell me how to solve it? Thanks!

Sorry, for that. Please try to use dev_plugins first:
git clone https://github.com/openxrlab/xrfeitoria.git
cd xrfeitoria
pip install .
python -c "import xrfeitoria as xf; xf.init_blender(replace_plugin=True, dev_plugin=True)"

This would be a formal feature when a new version is released.

Sorry to interrupt, I run your command and add replace_plugin=True, dev_plugin=True to blender.07_amass.py. But there's a new error ImportError: cannot import name 'Dict' from 'xrfeitoria.utils.functions.blender_functions'. Could you tell me how to solve it? Thanks!

I don't know the exact reason why this happened. But I noticed in the second line, the log says Reuse existing engine process. This message suggests that an instance of Blender is already running. To address this, you can try terminating the existing Blender process and then try to run the code again. This may resolve the issue.
XRFeitoria would first search for an existing engine process for efficiency. Once found, it would reuse the engine, and execute the following codes on this existing engine. Normally, this would be fine, but when using the existing engine, the dev_plugin and replace_plugin are ignored actually, which would cause failure in replace_plugin.

@HaiyiMei You're right. Problem solved, thanks!

Hi @Time-Lord12th @ChenyangWang95, we released v0.6.0 of XRFeitoria.
Please update using:
pip install xrfeitoria -U

And you may need to use xf.init_blender(..., replace_plugin=True)


feat: [DOC-3250]: add documentation for delegate selectors as an expression

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[x] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

@shalini-agr reviewed and approved. Let me know when to merge.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Missing host makes ansible playbook fail

Following the documentation to deploy PlasmaBio with Ansible, I encounter an issue with the following command:
$ ansible-playbook all.yml -u ubuntu
[WARNING]: No inventory was parsed, only implicit localhost is available
[WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'

PLAY [all] ******************************************************************************************************************************************************************
skipping: no hosts matched

[...]

The hosts file has been modified accordingly to reflect the new server IP address. Ansible documentation says hosts file is usually in /etc/ansible/hosts but that does not seem relevant here.

Fixed by #30.


add external healthcheck capability

abstracted healthcheck, with default as currently defined
additional '--check' cmdline arg to provide command to determine availability
additional '--arg=NAME' to provide way of configuring additional args
can be used in conjunction with healthcheck to provide more configurability
changed result messaging to support showing healthcheck and relevant env vars

Has merge conflicts now unfortunately. Nice idea!


Disallow having  the same name of a component and page directive in Blazor

Describe the bug
Server Side Blazor App
Having @ page' directive name, same as component name,  bricks app and VS.
To Reproduce
Steps to reproduce the behavior:

Using this version

Microsoft.NETCore.Platforms  3.0.0.preview4.19212.13
runtime.win-x64.Microsoft.NETCore.DotNetAppHost 3.0.0-preview4-27615-11
Visual Studio 16.1.0 Preview 1.0

Run this code

@page "/apptest"

<AppTest></AppTest>

or
<apptest></apptest> since it's not case sensitive

Navigate to page

IIS maxes all RAM very fast. No exception is thrown at all. App continues to work , barely.

@page "/test"

<test></test>

@functions
{
   protected override async Task OnInitAsync()
        {

          await Task.Delay(30);

        }
}

Set breakpoint on method and it will go on indefinitely when navigated to page.
So maybe issue some warning? It's not something obvious at first sight.
Also here's the end of dump when components start pilling up in memory.
00007ffb1da86040     4780       401142 System.String
00007ffb1da86588      259       544655 System.Byte[]
00007ffb1e458be0        2    125829408 Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiff[]
00007ffb1e478f88        2    143987760 System.Collections.Generic.Dictionary`2+Entry[[System.Int32, System.Private.CoreLib],[Microsoft.AspNetCore.Components.Rendering.ComponentState, Microsoft.AspNetCore.Components]][]
00007ffb1daa73f0  4418250    200740656 System.Int32[]
00007ffb1e1ce920  4417966    212062368 WebApplication2.Pages.test
00000239b59ab0f0   209655    247207184      Free
00007ffb1e456e40       22    251659152 Microsoft.AspNetCore.Components.RenderTree.RenderTreeEdit[]
00007ffb1e458028  4417980    282750720 Microsoft.AspNetCore.Components.RenderFragment
00007ffb1e478cc0  8835967    282750944 System.Collections.Generic.Stack`1[[System.Int32, System.Private.CoreLib]]
00007ffb1e456958  8835969    282751008 Microsoft.AspNetCore.Components.RenderTree.ArrayBuilder`1[[Microsoft.AspNetCore.Components.RenderTree.RenderTreeFrame, Microsoft.AspNetCore.Components]]
00007ffb1e433980  4417984    353438720 Microsoft.AspNetCore.Components.Rendering.ComponentState
00007ffb1e475a28  8835968    424126464 Microsoft.AspNetCore.Components.RenderTree.RenderTreeBuilder
00007ffb1e433f40  8835973<PHONE_NUMBER> Microsoft.AspNetCore.Components.RenderTree.RenderTreeFrame[]
Total 53251096 objects

Please also provide the file name for these components. It sounds like you have a component that recursively renders itself.

I think Dan is correct that this is simply an infinite recursion. You have a component that renders itself without any limits. This is like having a C# class whose constructor instantiates another copy of itself.
Closing as not a bug, but if you feel we have misdiagnosed this, please post further details. Thanks!


Converted identifiers to snake case

At Jerome Kelleher's suggestion, I have worked my way out from the _protocol_definitions.py file and converted from camelCase to snake_case all of the identifiers that are (1) used in the bodies of requests or responses, and (2) touched by the unit tests.  This is a significant chunk of the camelCase identifiers in the code, though not 100% of them.
All of the unit tests that succeeded in the unmodified snake_case branch succeed after I made these changes.
(BTW, this is my first contribution to the code, so in my ignorance I may have broken something - for instance, the docs. Let me know if so.)

Great work, thanks @hjellinek! This is very useful, and I think we should merge this much onto the snake_case branch as soon as the minor kinks pointed out above are worked out. There are two directions we can go from here:

Leave this as it is for now, and try to keep it up to date by merging back in develop periodically.
Go for the full conversion now, and try to merge back in changes to develop as they happen.

I'm leaning towards (1), as what we have now should be enough to do a good protobuf evaluation.

Excellent stuff @hjellinek. I think modulo my question about the CLI arguments still working, we should squash these commits and merge.
+1.

+1 after the sqash
what is the plan for the snake_case branch?

The plan for the snake_case branch currently is to do a decent evaluation of protobuf based on this. If we do changeover to protobuf, this will be the basis of a full snake_case conversion, which will ultimately get mapped back to develop.

Merging; thanks for the surgical precision @hjellinek!


Upgrade Twine.app to v2.0.10

Binaries are now hosted on bitbucket.org rather than twinery.org.

No problem, I will update and resubmit.


VT Capabilities Callback?

Is your feature request related to a problem? Please describe.
There's no great way to dynamically chose specific set of object pools based on the content of various server messages, like number of soft keys, or data mask size.
Describe the solution you'd like
It would be nice to have a callback or something to allow the user to select the correct object pool at connection time.

Duplicate of #376 ?

Yeah I suppose it is a duplicate.


Use SPI refactor PredicateJudge

// Describe your PR here; eg. Fixes #issueNo
Fixes #1496

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #1501 (9927860) into master (6992443) will decrease coverage by 0.74%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1501      +/-   ##
============================================
- Coverage     64.99%   64.24%   -0.75%
+ Complexity     2073     2065       -8
============================================
  Files           407      407
  Lines          8788     8783       -5
  Branches        882      882
============================================
- Hits           5712     5643      -69
- Misses         2648     2714      +66
+ Partials        428      426       -2

Impacted Files
Coverage Δ
Complexity Δ

...n/base/condition/judge/ContainsPredicateJudge.java
100.00% <ø> (ø)
2.00 <0.00> (ø)

...gin/base/condition/judge/EqualsPredicateJudge.java
100.00% <ø> (ø)
2.00 <0.00> (ø)

...gin/base/condition/judge/GroovyPredicateJudge.java
100.00% <ø> (ø)
2.00 <0.00> (ø)

...ugin/base/condition/judge/MatchPredicateJudge.java
100.00% <ø> (ø)
3.00 <0.00> (ø)

...ugin/base/condition/judge/RegexPredicateJudge.java
100.00% <ø> (ø)
2.00 <0.00> (ø)

...lugin/base/condition/judge/SpELPredicateJudge.java
100.00% <ø> (ø)
3.00 <0.00> (ø)

...base/condition/judge/TimerAfterPredicateJudge.java
60.00% <ø> (ø)
2.00 <0.00> (ø)

...ase/condition/judge/TimerBeforePredicateJudge.java
100.00% <ø> (ø)
3.00 <0.00> (ø)

...in/base/condition/judge/PredicateJudgeFactory.java
88.88% <100.00%> (-3.97%)
5.00 <2.00> (+1.00)
:arrow_down:

...enyu/client/tars/TarsServiceBeanPostProcessor.java
25.33% <0.00%> (-65.34%)
5.00% <0.00%> (-9.00%)

... and 4 more

Continue to review full report at Codecov.

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


consider making task_types input optional with a default

Conventional commits have a well-established set of task types, and it would save a lot of configuration duplication if the action allowed a default value for the task_types input.
e.g. ["feat","fix","docs","test","ci","refactor","perf","chore","revert"] like the docs example.
This would be a non-breaking change too. Thoughts?

I would even go with conventional-config used by commitlint: https://commitlint.js.org/reference/rules.html#type-enum.


Whichbrowser

@NielsLeenheer i fixed here the open erros from #27

fixing coding standards
fixing unittests

Important are some things for the future, to prevent the double work:

i added some notes how to check phpunit and the coding standards here
whichbrowser/parser needs to use semantiv versioning which is really important, otherwise we need to fix the version to a specific one. (do not break the API in minor and patch updates)

I'll make sure the unittests and coding standards are run before submitting PRs.
Regarding versioning: right now the versioning is still based on the server component which hasn't changed API. But once the PHP API stabilises I will release a 1.1 package and the PHP API will be officially supported regarding backwards compatibility.

That sound good. If versioning is not following the semver "style" all users will have problems when using the whichbrowser/parser when BC breaks are introduced.
To detect such breaks in this package i created issue #29 to test the lowest and highest installable dependency and test if there was a change.

There will be no backwards compatibility problems for 1.0 as long as 1.0.1046 or higher used. This is because the Provider uses functionality that was introduced in that version. That functionality will not change in upcoming releases of 1.0.
It would also be safe to allow 1.1 already at this point, because it will also be backwards compatible with 1.0.1046 or higher.


How to Handle payload=org.springframework.kafka.support.KafkaNull?

Hi,
While we are using @StreamListener, is there any way to handle Null (KafkaNull) Payload?
I tried to use @Payload(required = false) in arguments, that is not working. I guess, this work well with @KafkaListener.
Another option I tried is enableDlq for the consumer, and this message goes in there with Null Payload. But, this message is of no use to me.
Please suggest, is there any better approach?
Regards,
Adersh

No, there can never be a Message with null payload. That won'y be allowed by the underlying framework (i.e., Spring Integration). Now, KafkaNull is not null. It's an object that ". . .represents NULL Kafka payload. . ." . This means in theory you should be able to receive Message<KafkaNull>.
That said. . . .
Could you elaborate on your use case. I am not clear as to the value of null payload in general.
Also, please use Stack Overflow channel to ask questions. We use GitHub for reporting issues only.

Sorry for the delay; we were at SpringOne Platform last week.
Kafka records can have a null payload - this is termed a tombstone record for compacted logs (indicating the key has been deleted). Since a Spring-Messaging Message<?> can't have a null payload, we represent this condition with KafkaNull.INSTANCE.
In the @KafkaListener we have a customized payload resolver, which allows (required=false) on the payload...
			argumentResolvers.add(new PayloadArgumentResolver(messageConverter) {

				@Override
				protected boolean isEmptyPayload(Object payload) {
					return payload == null || payload instanceof KafkaNull;
				}

			});

We should add similar logic to @StreamListener.
The outbound channel adapter already converts KafkaNull to null when passing to Kafka.

Actually; since SCSt uses Spring Integration's HandlerMethodArgumentResolversHolder, the fix could be done there so SI apps can also benefit from it.
We'll just need a doc change here.


Time Miscalculations

Varying Transport.bpm values cause Tone.Time miscalculations.
Example here:
https://jsfiddle.net/timboie/1vztuhz1/
Affects both dev and release builds.

Time.js Line 113 : has the potential of returning large floating point numbers.
The following line of code should resolve the issue to the thousandths decimal:
quarters = parseFloat(quarters.toFixed(4));

thank you! pls submit a PR and if possible also a validation test if you can.

Ok will do


Error when testing samples with labels other than {0, 1} in the MNIST dataset.

Environment

Qiskit Machine Learning version:
Python version:
Operating system:

What is happening?
I followed the instructions in the document "Torch Connector and Hybrid QNNs." When running with samples labeled 0 and 1, it works fine. However, when I change the data to distinguish samples with other labels (e.g., 6 and 8 or any other pair), the training throws an error: "IndexError: Target 8 is out of bounds." Has anyone encountered this problem? Please suggest a solution. Thank you!

How can we reproduce the issue?
Train Dataset
-------------
Set train shuffle seed (for reproducibility)
manual_seed(42)
batch_size = 1
n_samples = 100 # we will concentrate on the first 100 samples
Use pre-defined torchvision function to load MNIST train data
X_train = datasets.MNIST(
root="./data",
train=True,
download=True,
transform=transforms.Compose([
transforms.ToTensor()
])
)
Filter out labels (originally 0-9), leaving only label 0 and 1
idx = np.append(
np.where(X_train.targets == 6)[0][:n_samples],
np.where(X_train.targets == 8)[0][:n_samples]
)
X_train.data = X_train.data[idx]
X_train.targets = X_train.targets[idx]
Define torch dataloader with filtered data
train_loader = DataLoader(X_train, batch_size=batch_size, shuffle=True)
What should happen?

Any suggestions?
No response

Hi, @phanducthien82, thanks for raising this. Just to check, what version of Qiskit machine learning are you using?
The code snippets you shared are useful, however, it would be even more helpful to have complete code from start to end that shows the buggy behaviour. This would help us look for potential issues and advise on library usage. Would you be able to share the code in this thread?

06_torch_connector
This is my cocde

@phanducthien82 you may need to save it explicitly in Kaggle, or at least I couldn't find your code at the link. Apart from that, I'd recommend posting the code in a comment here directly, instead of Kaggle, for consistency. Hope this helps!

This is my code
`import numpy as np
import matplotlib.pyplot as plt
from torch import Tensor
from torch.nn import Linear, CrossEntropyLoss, MSELoss
from torch.optim import LBFGS
from qiskit import QuantumCircuit
from qiskit.circuit import Parameter
from qiskit.circuit.library import RealAmplitudes, ZZFeatureMap
from qiskit_algorithms.utils import algorithm_globals
from qiskit_machine_learning.neural_networks import EstimatorQNN, SamplerQNN
from qiskit_machine_learning.connectors import TorchConnector
set seed for random generators
algorithm_globals.random_seed = 42
Additional torch-related imports
import torch
from torch import cat, no_grad, manual_seed
from torch.utils.data import DataLoader
from torchvision import datasets, transforms
import torch.optim as optim
from torch.nn import (
Module,
Conv2d,
Linear,
Dropout2d,
NLLLoss,
MaxPool2d,
Flatten,
Sequential,
ReLU,
)
import torch.nn.functional as F
Train Dataset
-------------
Set train shuffle seed (for reproducibility)
manual_seed(42)
batch_size = 1
n_samples = 100 # we will concentrate on the first 100 samples
Use pre-defined torchvision function to load MNIST train data
X_train = datasets.MNIST(
root="./data",
train=True,
download=True,
transform=transforms.Compose([
transforms.ToTensor()
])
)
Filter out labels (originally 0-9), leaving only label 0 and 1
idx = np.append(
np.where(X_train.targets == 6)[0][:n_samples],
np.where(X_train.targets == 8)[0][:n_samples]
)
X_train.data = X_train.data[idx]
X_train.targets = X_train.targets[idx]
Define torch dataloader with filtered data
train_loader = DataLoader(X_train, batch_size=batch_size, shuffle=True)
n_samples_show = 6
data_iter = iter(train_loader)
fig, axes = plt.subplots(nrows=1, ncols=n_samples_show, figsize=(10, 3))
while n_samples_show > 0:
images, targets = data_iter.next()
axes[n_samples_show - 1].imshow(images[0, 0].numpy().squeeze(), cmap="gray")
axes[n_samples_show - 1].set_xticks([])
axes[n_samples_show - 1].set_yticks([])
axes[n_samples_show - 1].set_title("Labeled: {}".format(targets[0].item()))

n_samples_show -= 1

Test Dataset
------------
Set test shuffle seed (for reproducibility)
manual_seed(42)
n_samples = 50
Use pre-defined torchvision function to load MNIST test data
X_test = datasets.MNIST(
root="./data",
train=False,
download=True,
transform=transforms.Compose([
transforms.ToTensor()
])
)
Filter out labels (origined 0-9), leaving only labels 0 and 1
idx = np.append(
np.where(X_test.targets==6)[0][:n_samples],
np.where(X_test.targets==8)[0][:n_samples]
)
X_test.data = X_test.data[idx]
X_test.targets = X_test.targets[idx]
Define torch dataloader with filtered data
test_loader = DataLoader(X_test, batch_size=batch_size, shuffle=True)
Define and create QNN
def create_qnn():
feature_map = ZZFeatureMap(2)
ansatz = RealAmplitudes(2, reps=1)
qc = QuantumCircuit(2)
qc.compose(feature_map, inplace=True)
qc.compose(ansatz, inplace=True)
# REMEMBER TO SET input_gradients=True FOR ENABLING HYBRID GRADIENT BACKPROP
qnn = EstimatorQNN(
    circuit = qc,
    input_params = feature_map.parameters,
    weight_params = ansatz.parameters,
    input_gradients = True, # allow hybrid gradient backprob
)
return qnn

qnn4 = create_qnn()
Define torch NN module
class Net(Module):
def init(self, qnn):
super(Net, self).init()
self.conv1 = Conv2d(1, 2, kernel_size=5)
self.conv2 = Conv2d(2, 16, kernel_size=5)
self.dropout = Dropout2d()
self.fc1 = Linear(256, 64)
self.fc2 = Linear(64, 2) # 2-dimensional input to QNN
self.qnn = TorchConnector(qnn) # Apply torch connector, weight chosen uniformly at random from interval [-1, 1]
self.fc3 = Linear(1, 1) # 1-dimensional output from QNN
def forward(self, x):
    x = F.relu(self.conv1(x))
    x = F.max_pool2d(x, 2)
    x = F.relu(self.conv2(x))
    x = F.max_pool2d(x, 2)
    x = self.dropout(x)
    x = x.view(x.shape[0], -1)
    x = F.relu(self.fc1(x))
    x = self.fc2(x)
    x = self.qnn(x) # Apply QNN
    x = self.fc3(x)
    return cat((x, 1 - x), -1)

model4 = Net(qnn4)
Define model, optimizer, and loss function
optimizer = optim.Adam(model4.parameters(), lr=0.001)
loss_func = NLLLoss()
Start training
epochs = 10  # Set number of epochs
loss_list = []  # Store loss history
model4.train()  # Set model to training mode
for epoch in range(epochs):
total_loss = []
for batch_idx, (data, target) in enumerate(train_loader):
optimizer.zero_grad(set_to_none=True)  # Initialize gradient
output = model4(data)  # Forward pass
loss = loss_func(output, target)  # Calculate loss
loss.backward()  # Backward pass
optimizer.step()  # Optimize weights
total_loss.append(loss.item())  # Store loss
loss_list.append(sum(total_loss) / len(total_loss))
print("Training [{:.0f}%]\tLoss: {:.4f}".format(100.0 * (epoch + 1) / epochs, loss_list[-1]))
`

Explanation
Hi @phanducthien82, so I used the code you posted above - there isn't a bug just a slight mistake in your code. All that's happening is that you're inputting a target of 6 and 8 directly into the loss function, which is expecting either a 0 or 1, as there are 2 classes. There is a little more to it once you start getting into output shapes and numbers of classes and internal one hot encoding but an easy fix is just to make sure your target array has values 0 where the image is the number 6 and values 1 where the image is the number 8. If you want to extend this to say 3 classes of images, say images of the numbers 6, 7, and 8, then you would need to have targets of 0, 1, and 2.
Fix
This can easily be done in torch by setting the X_train.targets and X_test.targets manually. When the idx are found, the np.where statements are done consecutively, this results in X_train.targets[idx] being a tensor of 6s followed by 8s. This means X_train.targets can be replaced with a tensor of 0s followed by 1s. This can be quickly done using torch.zeros and torch.ones and then concatenated together in-place, no sweat. Replace line 61 with:
X_train.targets = torch.cat((torch.zeros((n_samples)), torch.ones((n_samples))))

and replace line 105 with:
X_test.targets = torch.cat((torch.zeros((n_samples)), torch.ones((n_samples))))

The code reuses n_samples so makes life even easier. Later the code uses the DataLoaderclass to shuffle everything up for you. With these changes, the code ran fine for me and if you need to add another class add another tensor using torch.full((n_samples), class_number) which will create a tensor of shape n_samples filled with the class_number specified. This will need to be concatenated with the other targets. For more info on creating torch tensors look here or check out the documentation.

Thanks a lot. It worked


Add link generator to DashboardClient

What this PR does / why we need it:
Usage: request.DashboardClient.CreateLink(request.Context(), key)

Returns a link response that has a link component with the object's name as the link text as default.

Which issue(s) this PR fixes

Fixes #1444

Can you create an issue to add this to an example plugin as well? I think showing the usage of this will be useful to plugin authors.

I'm inclined to say storybook should be for component rather than API documentation. We can add an example plugin and create a page for documenting what DashboardClient can do outside of code comments.

Where do you see us moving these to? https://reference.octant.dev/?path=/docs/docs-plugins-2-go-plugins--page

My thought was that having the "how to" plugin guides and component reference in the same place is useful

That's fine where it is. I'll add sample code in https://github.com/vmware-tanzu/octant-example-plugins


main dist missing on master

dist was mistakenly removed from master.
closes #183

Thanks for your work mate! It works correctly now!


Language Translation: French

Summary

French Translation
Change the order of languages in the menu to be alphabetical (I left english on top since it's the default)

Change Type

[x] Translation: French

Checklist

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] My changes do not introduce new warnings

It looks like you fixed the issue with the German select option, lgtm!


[release-1.4]Avoid passing token when using http

Cherry pick of https://github.com/kubernetes/heapster/pull/2048
/cc @kawych

/ok-to-test
/lgtm


InputPin not implemented for PA0 (and probably others)

I was told on the embedded rust matrix chat that this crate is no longer used with the advent of stm32f3xx_hal, so I've used that crate instead to get around the issue. This crate isn't listed as deprecated though, and there's no link on the README that I can see to go and use stm32f3xx_hal.
I'm happy to either submit a PR to implement InputPin for PA0 and the others (you can see that it's not available here) or submit a PR amending documentation, or anything else.
Let me know how I can help, cheers.

Ah, I see there is a PR for it already https://github.com/japaric/stm32f30x-hal/pull/16/files

Thanks! I have started using stm32f3xx-hal, it would be nice if this crate either accepted the PRs that fix issues with it, say that it's no longer maintained.
I found it because the discovery book uses it, that should also be updated, since it's probably many people's entry point into embedded rust.


bug(schematics): themingApi missing from schematics

Ever since this commit, we can no longer run the themingApi schematic outside of ng update. This means that if you have a codebase with just scss, you will no longer be able to convert to ng12. One use-case is that you have a theme in its own repository or module, and import it in the projects where you need it.
So my proposal is: bring it back. I don't see any reason to remove it as a standalone schematic, given its usefulness.
If bringing it back is a problem, then it should at least be removed from the documentation. I spent several hours trying to figure out where this darn themingApi is at.
I think it makes more sense to just read the above description, but here's the template stuff:
Reproduction
Steps to reproduce:

make a project with just @angular/material^12
run ng g @angular/material:themingApi

Expected Behavior
Angular CLI converts the scss to new style, as advertised here.
Actual Behavior
Error:
An unhandled exception occurred: Schematic "themingApi" not found in collection "@angular/material".

Environment

Angular: n/a
CDK/Material: 12.0.0-rc3 and above
Browser(s): n/a
Operating System (e.g. Windows, macOS, Ubuntu): Windows 10

Workaround for now, for anyone visiting: use version 12.0.0-rc2 and run ng g @angular/material:themingApi - it should work, although it might be buggy compared to the current script.
Alternatively, copy the scss files into a valid angular project and run ng update --from=11 @angular/material@12 --migrate-only.

Leaving a comment here to let you know that this is still a thing and that I'm the author of this issue - merged work and personal github accounts together, and lost this issue in the process.


[BUG] Property Type Change

Hi,
In version 1.7.2, it is no longer possible to change the property type of columns.

thanks!  I will give it a check
The template feature needed a refactor and something could be affected.

@kazdonkai I could not reproduce it. Think was fixed with https://github.com/RafaelGB/obsidian-db-folder/issues/114

I reinstalled it and it is now available.
thanks.


Replace [ExpectedException] with Assert.Throws

This commit replaces all occurrences of the [ExpectedException] attribute on unit test methods with the more recent and flexible Assert.Throws syntax. As outlined in issue #34, this has some benefits:

improves separation between the Arrange and Act stages of Arrange-Act-Assert;
improves test code consistency;
removes an obstacle for migrating to more recent versions of NUnit.

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


Trigger Notification if there is a Merge Conflict

This is the pull.yml file I am using for my forked repo.
version: "1"
rules:
  - base: feature
    upstream: master
    mergeMethod: merge
  - base: master
    upstream: parent_repo:master
    mergeMethod: hardreset

This keeps the master branch of the forked repo up-to-date with the parent repo.  It keeps the feature branch of the forked repo updated via the master the branch of the forked repo by merging the same.
So the pull app opens a PR, performs the merge operations and closes the PR on its own. However, if a merge between feature branch and master branch leads to a conflict, the PR remains open waiting for conflicts to be resolved.
To inform the user of such conflict via mail, one could edit the pull.yml file and add the assignee or reviewer parameter. But this would lead to triggering of a notification via mail whenever any PR is opened and closed. Is it possible to set a rule such that if there is a merge conflict, the PR is assigned a assignee or reviewer ?

good idea. I'll look into it. 😬

Is this feasible ?

Set a new parameter for merge conflict reviewer (for pull.yml) say m_reviewer
Check for list of open PRs and get  value of merged attribute for each PR
If it's set to false, then add reviewer of PR to m_reviewer and change subject to include the keyword "merge-conflict"
Adding a reviewer should automatically trigger a notification from Github

I'd call it conflict_reviewer and it can be added when
https://github.com/wei/pull/blob/0abe0c337e3dee4f5b77962e77c827d71674b49b/lib/pull.js#L64
=== false.

Ok, upon checking the above condition for the current PR created, this function can be called to add the reviewer https://github.com/wei/pull/blob/0abe0c337e3dee4f5b77962e77c827d71674b49b/lib/pull.js#L220 and assign a label using this https://github.com/wei/pull/blob/0abe0c337e3dee4f5b77962e77c827d71674b49b/lib/pull.js#L177

I am new to building GitHub apps, is there any doc which I can follow ?

You can look at probot docs, GitHub Apps | GitHub REST API v3 and the wrapper @otokit/rest JS

@wei  Please review PR #172

good idea. I'll look into it. 😬 PRs welcome

@wei , a gentle reminder

@wei Please review PR #172


”Bistrița Bârgăului” is listed in the drop-down menu

Description:
"Bistrița Bârgăului" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Bistrița Bârgăului" in the search bar.

Expected results
The location is listed in the drop-down menu as "Bistrița Bârgăului, Bistrița-Năsăud".


some applications from kde or gnome don't work when installed

Describe the bug
I wanted to install some applications of the Kontact suite: Korganizer and Kmail.
However, after installation, the applications did not work as expected, for example, Kmail did not start and Korganizer had no way of installing calendars.
I know that this is maybe to avoid installing many dependencies, but it should not, in my opinion, be this restrictive as it makes these applications unusable.
Other applications have this problem in the Gnome ecosystem like Evolution.
On ArchLinux, when installing Korganizer and then launching it, it works perfectly out of the box even when KDE isn't installed on the system.
To give options to the users, maybe providing them with a *Full (korganizerFull, kmailFull...) variation would be a good idea.
To Reproduce
Steps to reproduce the behavior for Kmail and Evolution:

install Kmail or Evolution on a clean system (with or without Plasma)
try launching it
it crashes

Steps to reproduce the behavior for Korganizer:

Install Korganizer on a clean system (with or without Plasma)
launch it
It stays stuck
exit and install Akonadi
launch it, it should not get stuck anymore
try adding a calendar (right click on the down left part of the application -> Add Calendar)
Only available option is Knot

Expected behavior
The applications or a *Full version of them should work out of the box after their installation.
This means that they should not crash at startup and that their main functionalities should be present.
Screenshots
Cannot add calendars to Korganizer (after installing Akonadi):

Kmail crashing after installation:

Evolution crashing after install:

Additional context
I have Plasma installed and active on my system.
Notify maintainers

KDE maintainers: @ttuegel @vandenoever @nyanloutre
Gnome maintainers: @hedning @jtojnar @worldofpeace @dasj19
Metadata

system: "x86_64-linux"
host os: Linux 5.4.75, NixOS, 21.03.20201110.dd1b7e3 (Okapi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.4pre20201102_550e11f
channels(narice): "nixpkgs-21.03pre253014.6625284c397"
channels(root): "nixos-21.03pre251971.a371c107116"
nixpkgs: /home/narice/.nix-defexpr/channels/nixpkgs

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: kdeApplications.korganizer, kdeApplications.kmail, gnome3.evolution...
# a list of nixos modules affected by the problem
module:

How did you install these programs? With nix-env, home-manager, configuration.nix ?
Can you retry lauching evolution with services.gnome3.evolution-data-server.enable option set to true?

I was installing it with nix-env.
Following your advice, I enabled the evolution-data-server option and now it works, I didn't know that it was mandatory to have this option so I did not enable it.
Just to know why this has to be enabled, is it not possible to install Evolution without its data server as a standalone application?
This would be much more practical for the user as he would only have to install the app with nix-env or home-manager instead of enabling an option in the NixOS config. Also, users using nix on another distro would also be able to install Evolution.
Also, is enabling the evolution-data-server option mentioned in some documentation?
For Korganizer, there is no equivalent option on nixos so I don't know how to install it, but at least I now have a mail and calendar client :wink:  thanks @symphorien :blush:

evolution is not a standalone executable. It requires some dbus services (evolution-data-server) to be functional, and I think the corresponding dbus configuration files are not modifiable by an unprivileged user on a "normal" distro. So unfortunately, it looks like the design of evolution prevents installing it correctly for an unprivileged user.
I don't think there is any documentation besides the existence of the option: https://search.nixos.org/options?channel=20.09&from=0&size=30&sort=relevance&query=evolution
I have no idea about the kde apps you mentioned.

So I guess on other distros, the fact that installing apps is only done by the root user makes installing apps like Evolution, Korganizer, and the Kontact suite a breeze but on NixOS these sorts of installs are not as easy.
That makes sense and it's a pretty interresting limitation of nix :thinking:.
I'll wait for the kde team on this subject to know if they face the same limitations.
One way of avoiding this limitation is to provide a NixOS config option.
Having a program.evolution.enable option in configuration.nix would be the best solution in my opinion.
The same goes for kdeApplications, they could have programs.*.enable options.
What are your thoughts on this possible solution?

To be more precise, the issue is not that root would be required – in this case e-d-s is D-Bus service that runs on a session bus so it can be completely installed to user profile. The issue is that we need to make the D-Bus service running on the system aware of the services (e.g. by linking relevant files to ~/.local/share/dbus-1/services/). Nix packages are self-contained so they cannot really affect environment that way.
See also https://github.com/NixOS/nixpkgs/issues/59833#issuecomment-650786182 for a possible solution but that would no longer be simple.

Having a program.evolution.enable option in configuration.nix would be the best solution in my opinion.

see https://github.com/NixOS/nixpkgs/pull/103135

Hey! I did a fresh (and fully reproducible to the user config files) install of NixOS on my system and hit the korganizer problem once more.
Apparently you have to install akonadi as well as akonadi-mime so in the system's packages so that the application doesn't crash on certain actions which really plagued my experience until now because I didn't know it was a documented issue #38887.
Would it be possible to have the same sort of wrapping for korganizer at a system's level as it needs certain dependencies?
And would it be possible to add the full dependencies to korganizer (or at least what can be made as a dependency) like akonadi?
I think it is a really confusing and frustrating problem that a lot of NixOS beginners face.

For me, to make kaddressbook work, what did it was to have
kaddressbook
plasma5Packages.akonadi
plasma5Packages.kdepim-runtime

in my config.
You may want to try with adding kdepim-runtime and then restart akonadi akonadictl restart and also restart the client that you want to use.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/kaddressbook-kontact-akonadi-cannot-add-address-book/25227/5

Closing this, as this is a none actionable collection of multiple issues. Please open separate issues for kde and gnome things.


Implement a wiki

Starting a wiki now will help to prevent issues later.

First, find or create a template or layout
Add a link to the comments below for approval
If approved will be assigned to you

Will implement a more robust README for now. Not needed at the current stage of development.


Filter Synthetic Methods in PipelineOptionsFactory

Fixes the testReturnTypeConflcitThrows when compiled with java 8, which inserts a bridge method which is reported as a a conflicting type (even though it is an effect of a conflicting type rather than a cause, and the user cannot change anything about it)

R: @lukecwik

Add a comment to PipelineOptions / PipelineOptionsFactory stating that we don't support java8's default methods.


Tgttosawaf option

If it it possible make it so the chickens in tggttos are players to make it feel more like the event but of course make the hit boxes the same as the chickens to make it fair to others

Cannot implement since the player hitbox is significantly larger than then chicken hitbox and there really isn't a way around that,
I could make the hitbox smaller, however that'll just make hitting them more difficult because the real hitbox is smaller than the apparent hitbox
Thank you for the suggestion :D


listener exit with ctrl-c, escape, or q

Looks fine but could you add a note to the doc about how to use the systemcmd?

In the module docs or printed at runtime?

Looks fine but could you add a note to the doc about how to use the systemcmd?

@LorenzMeier I see that I need to use the GitHub "request changes" otherwise what I write is ignored :smile:.

In the module docs or printed at runtime?

In the module docs: #12167


About us and time for action updated.

try putting in the background image image. and connect the forms page in the index file to the Let's Go button.

Could you add the CSS files in their respective folders. Also make sure the HTML files are not included in any folder

that way we'll have to chage the href links for every page..as files will be relocated


Generate Shadow Jar on Java 19

Shadow plugin 8.1.1 works well with Java 19, CI failure due to integration tests, see
> Task :shadowJar
Warning in /home/runner/work/StringResExporter/StringResExporter/build/libs/string-res-exporter-0.2.0-SNAPSHOT-all.jar:META-INF/services/org.apache.poi.extractor.ExtractorProvider:

https://scans.gradle.com/s/5dceyafzhjl5w/tests/overview?outcome=FAILED

Relocation works well with intenal.jar, which includes classes with the byte code version 63.


Fixed the Background UI issue

I have fixed the issue. Hope so it's fine by you

@SauravMob Hi, the fix is almost correct, ie no white screen on any resolution, but now all desktop screen resolutions have that scrollbar. Expected behavior should be: If the container can fit into the screen, then the scroll bar should not be there, otherwise it should be scrollable.

Done. I have fixed the scrollbar issue you can have a look and let me know if any other issue pops up.


...and fixes

Current coverage is 96.92%

Merging #1096 into master will increase coverage by +0.03% as of d3041e8

@@            master   #1096   diff @@
======================================
  Files           75      75
  Stmts         9505    9505
  Branches      1742    1742
  Methods          0       0
======================================
+ Hit           9210    9213     +3
+ Partial        101     100     -1
+ Missed         194     192     -2

Review entire Coverage Diff as of d3041e8

Powered by Codecov. Updated on successful CI builds.

Thx!


Label wrapped vertical size

This vertical size calculation is wrong if word_wrap=True.
https://github.com/xc-jp/pyedifice/blob/a9b003969f592667e76ee0f5bdbeaf0059304482/edifice/base_components.py#L880

https://github.com/pyedifice/pyedifice/issues/41


Warn instead of error for unused imports

Rationale for this change
It's very frustrating when working on datafusion that unused imports cause scripts and tests to fail at build time, and not run.
Also, while looking in to this, it seems the pre-commit.sh script wasn't failing when clippy failed.
This doesn't affect CI where unused imports will still cause failure I believe.
What changes are included in this PR?

remove unused_imports = "deny" from the root Cargo.toml, so the default behaviour of warning occurs
make pre-commit.sh fail if clippy fails

Are these changes tested?
Manually, yes.
Are there any user-facing changes?
No.

See https://github.com/samuelcolvin/datafusion/pull/2 - confirming that CI still fails on unused imports.


os.environ throws TypeError

Steps to reproduce the problem
The os.environ["QISKIT_BRAKET_PROVIDER_MAX_DELAY"] must be set as a string, meaning that when it is set and the _get_result_from_aws_tasks is called, then the retry decorator throws a TypeError since it cannot compare an int and a str.
What is the current behavior?
TypeError is raised when os.environ["QISKIT_BRAKET_PROVIDER_MAX_DELAY"] is set.
What is the expected behavior?
Place an int() in the retry decorator to eliminate the problem.

This would probably also also fix #57


Migrate off deprecated travis-ci.org

Both kpt and kpt-functions-sdk use travis-ci.org, which is deprecated. We should migrate off, taking into account guidance from the Github Admin team.

Will keep this open to address travis + kpt-functions-sdk

@prachirp can you tell me where there are traces of travis in that repo?  I don't see it...
I also checked the function catalog and nothing there either.
Thank you for your help 🙇

@mikebz You're right, I also verified that we don't use travis in that repo. 🙂