Editing points on gradient not working

Thanks for building this library! I'm having this bug where when trying to select the gradient color to edit (on the bottom bar),  I get the error Cannot read property 'y' of undefined. I upgraded to version 1.2.0 but still seeing the issue

@melissafzhang Thanks! I fixed this bug in version 1.2.1. Please update! If something is wrong, write a new issue.


Pick up more name pairs, e.g. long,lat

There's no reason this tool couldn't be smarter at spotting potential latitude/longitude column pairs - since they come in pairs, there's not much risk of seeing a lat column and triggering when there's no sensible lon to accompany it.

Came up in office hours. Let's go for:

lon, long, lng for longitude
lat for latitude

Suggestion: look for these as _latitude, _longitude style pairs too.
Basically it should kick in if it looks like there are only two columns that could be representing locations in latitude/longitude format.

what happens if the fields appear in a sqlite database converted from geojson, and the the datasette-geojson-map plugin is installed?

This won't solve the conflicting problem where two plugins want to display a map at the same time - that's going to have to come later, maybe after the Datasette 1.0 release.

I have a power plants table with these columns:
['rowid', 'country', 'country_long', 'name', 'gppd_idnr', 'capacity_mw', 'latitude', 'longitude', 'primary_fuel', 'other_fuel1', 'other_fuel2', 'other_fuel3', 'commissioning_year', 'owner', 'source', 'url', 'geolocation_source', 'wepp_id', 'year_of_capacity_data', 'generation_gwh_2013', 'generation_gwh_2014', 'generation_gwh_2015', 'generation_gwh_2016', 'generation_gwh_2017', 'generation_data_source', 'estimated_generation_gwh']
This did NOT display a map... because of my logic that says that if there are multiple ambiguous lat/lon columns it should ignore them.
It thought that country_long and longitude were rival columns!
I might need some kind of priority mechanism here, such that country_long is ignored if there were exact matches for latitude and longitude. Not sure how best to do that.

My current tests look like this:
@pytest.mark.parametrize(
    "input,expected",
    (
        ([], []),
        (["a", "b"], []),
        (["a", "b", "latitude", "longitude"], ["latitude", "longitude"]),
        (["a", "b", "lat", "lon", "c"], ["lat", "lon"]),
        (["a", "b", "lat", "lng", "c"], ["lat", "lng"]),
        (["a", "b", "lat", "long", "c"], ["lat", "long"]),
        # Returns [] because there are multiple potential options:
        (["a", "lat", "lon", "c", "latitude", "longitude"], []),
        # Wildcard matches
        (["a", "foo_latitude", "foo_longitude"], ["foo_latitude", "foo_longitude"]),
        (["a", "foo_latitude", "bar_longitude"], ["foo_latitude", "bar_longitude"]),
        (["a", "foo_lat", "foo_long"], ["foo_lat", "foo_long"]),
        (["a", "foo_lat", "foo_lon"], ["foo_lat", "foo_lon"]),
        (["a", "foo_lat", "foo_lng"], ["foo_lat", "foo_lng"]),
    ),
)
def test_location_columns_from_columns(input, expected):
    actual = location_columns_from_columns(input)
    actual_cap = location_columns_from_columns([col.upper() for col in input])
    assert actual == expected
    assert actual_cap == [col.upper() for col in expected]

I'm going to say that the prefixes have to match. That should fix the country_long problem.

Made myself a test database like this:
echo '[
    {
        "name": "San Francisco",
        "lat": 37.7749,
        "lon": -122.4194
    },
    {
        "name": "New York",
        "lat": 40.7128,
        "lon": -74.006
    },
    {
        "name": "Cleveland",
        "lat": 41.4993,
        "lon": -81.6944
    }
]' | sqlite-utils insert strange.db latlon -
echo '[
    {
        "name": "San Francisco",
        "lat": 37.7749,
        "lng": -122.4194
    },
    {
        "name": "New York",
        "lat": 40.7128,
        "lng": -74.006
    },
    {
        "name": "Cleveland",
        "lat": 41.4993,
        "lng": -81.6944
    }
]' | sqlite-utils insert strange.db latlng -
echo '[
    {
        "name": "San Francisco",
        "lat": 37.7749,
        "long": -122.4194
    },
    {
        "name": "New York",
        "lat": 40.7128,
        "long": -74.006
    },
    {
        "name": "Cleveland",
        "lat": 41.4993,
        "long": -81.6944
    }
]' | sqlite-utils insert strange.db latlong -
echo '[
    {
        "name": "San Francisco",
        "location_lat": 37.7749,
        "location_lng": -122.4194
    },
    {
        "name": "New York",
        "location_lat": 40.7128,
        "location_lng": -74.006
    },
    {
        "name": "Cleveland",
        "location_lat": 41.4993,
        "location_lng": -81.6944
    }
]' | sqlite-utils insert strange.db location -

All four tables look like this:


Add support for larger DartPad API Doc examples

An API example that's longer than 10 or 20 lines can be a little unwieldy to maintain as part of a DartDoc comment. On the other hand, interactive DartPad examples that do a more than demonstrate instantiating a class or calling a method, like this one https://master-api.flutter.dev/flutter/animation/Curve2D-class.html, are pretty compelling.
It would be helpful if larger examples could be included by reference. We'd need a way to refer to a file/directory in a stable repo. Maybe somewhere in the samples repo?
/// {@tool dartpad <new source option here>}
/// {@end-tool}

This approach might also simplify analyzing and testing example code; at least the kind of example code that's a functional small app rather than a code fragment.

CC @gspencergoog @JoseAlba @johnsonmh @willlarche

cc @devoncarew, for the impact on the IDE support for examples.
I think this is a good idea, but I want to make sure that this is used only for large examples. One of the goals of the dartdoc examples is to make it so that if you are perusing the code, you get the same information as if you were perusing the docs website. If we could, I'd put a webview with the docs inline in the IDE, Dartpad and all (but we can't).
For large examples, this breaks down, since then you're paging past pages and pages of comments when you're looking for actual code, so this seems like a good idea. I think the examples should be part of the Flutter tree itself, since it's important to make sure that the examples and the code are in sync with each other, or they might not compile when the user goes to run them.

Also, it's nice to have the example code right there next to the code you're changing: makes it easier to remember to update it.

Now that samples live in their own independent file, this problem should no longer exist.


Dynamic router deferred routes should allow mutations that immediately resolve them

It seems the route-deferred return value cannot be used with a mutation that immediately resolves the route.  This should be possible, as it is not technically safe to side-effect within the will-enter method.

This is fixed by the new target-ready! and is in the docs.


Select All Occurences of a symbol / references Command

Hey ! It might be interesting to add the command: "Select All Occurences of Selected Symbol"
Because I often need to replace an occurrence of a symbol, for example in Typescript.

What do you think about it?

Did you try the F2 rename feature?

Did you try the F2 rename feature?

Of course (and that's a pretty useful feature),  F2 rename replace the content without placing cursors...

So what? Isn't the outcome the same? All occurrences of a symbol get a new name? I am not sure what you are asking for. Do you want rename to use multiple cursors instead of the input widget?

So what? Isn't the outcome the same? All occurrences of a symbol get a new name? I am not sure what you are asking for. Do you want rename to use multiple cursors instead of the input widget?

Sorry, I must have misspoken. My need is to be able to target multiple occurrences of a symbol with multiple cursors, not a string. Sometimes this can be useful to rename an element, to add content to it or to identify and visualize it in the minimap. Screenshot n°2 contains four cursors, the effect of which would have been achieved after calling this command.
As it's currently possible to list the references of a symbol, I'd like to be able to have several cursors on these references.
btw thanks for you time

Hey @jrieken, this issue might need further attention.
@MatteoGauthier, you can help us out by closing this issue if the problem no longer exists, or adding more information.

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


Enhancement - manual control button for on, disable button for off

Thanks for this node.  I am happy to have this interface. I thought it was ridiculous that someone would have to go to the admin page to modify schedules. (with other schedulers) Your node solves this problem.
In my Node Red system I have a light turned on at 5:20 am along with an internet radio, Mondays to Friday. No action on the weekend. The problem I have is that when there is a long weekend I would like to easily disable a normally scheduled event. I would rather sleep in than get woken up by the light and music.
I can see how I could go in and modify the schedule, but that is a number of steps and I will again have to restore the normal settings after the day had passed, or until the timer is reactivated. (I just thought of vacation days where I would be off for a number of days)
And the manual turn on button could be helpful for some although it is not as significant as being able to disable the timer.
Ideally I would like to see the disable button on the description bar.

Hi @RudyFiero ,
thank you for your feedback, it is always nice to hear about how people use the scheduler!
The scheduler already provides the functionality to disable a device or a single timer (until it gets turned on again).
The button that disabled the whole device is located next to the add button:

If it was clicked all the timers for this specific device are out of order.
The same button can be found by clicking an existing timer (where you would edit the timer) if this button is activated then only this specific item is disabled.

Please let me know if this helps!

LOL
So the clock is more than just a pretty face.
Once again. Thank you for this useful node.


rawQuery ERROR:  insufficient data left in message; when casting to interval

Bug description
The following query:
await db.$queryRaw`
INSERT INTO "QuickbooksTokens"
("accessToken", "refreshToken", "accessExpiration", "refreshExpiration")
VALUES (
${access},
${refresh},
NOW() + ${accessExpiration}::INTERVAL,
    NOW() + INTERVAL '24 hours'
  )
  `

Produces the postgresql error:
ERROR:  insufficient data left in message
CONTEXT:  unnamed portal parameter $3

Where "accessToken" and "refreshToken" are "String" and "accessExpiration" and "refreshExpiration" are "DateTime".
If you change it to the following:
  await db.$queryRaw`
  INSERT INTO "Tokens"
    ("accessToken", "refreshToken", "accessExpiration", "refreshExpiration")
  VALUES (
    ${access},
${refresh},
NOW() + ${accessExpiration}::TEXT::INTERVAL,
    NOW() + INTERVAL '24 hours'
)
  `

Then it works fine.
How to reproduce
Create table with datetime field, try to run a $queryRaw using an integer interval.
Expected behavior
No response
Prisma information

model Tokens {
  id Int @id @default(autoincrement())
  accessToken String
  refreshToken String
  accessExpiration DateTime
  refreshExpiration DateTime
}

// Add your code using Prisma Client

Environment & setup

OS: Debian
Database: Postgresql14
Node.js version: v18.7.0

Prisma Version
❯ node_modules/.bin/prisma -v
Environment variables loaded from .env
prisma                  : 3.15.2
@prisma/client          : 3.15.2
Current platform        : debian-openssl-3.0.x
Query Engine (Node-API) : libquery-engine 461d6a05159055555eb7dfb337c9fb271cbd4d7e (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli 461d6a05159055555eb7dfb337c9fb271cbd4d7e (at<EMAIL_ADDRESS>Introspection Engine    : introspection-core 461d6a05159055555eb7dfb337c9fb271cbd4d7e (at<EMAIL_ADDRESS>Format Binary           : prisma-fmt 461d6a05159055555eb7dfb337c9fb271cbd4d7e (at<EMAIL_ADDRESS>Default Engines Hash    : 461d6a05159055555eb7dfb337c9fb271cbd4d7e
Studio                  : 0.462.0

Hi @rmzg, we're currently not able to reproduce this locally. May you please provider the values of access, refresh, accessExpiration you're passing to the raw query? Is the QuickbooksTokens model different than Tokens? If so, can you please add that part of the schema to this issue as well?
Most likely, you have some null character strings in your input, which triggers this problem in Postgres. This is also mentioned on StackOverflow.

Ping @rmzg - can you help us out here please? We want to reproduce this, but are currently not able to without additional information from you. Thanks!

I don't have easy access to test this in prisma 4.x at the moment so I'm
going to assume it's fixed later on.
On Fri, Sep 23, 2022, 11:58 AM Jan Piotrowski @.***>
wrote:

Ping @rmzg https://github.com/rmzg - can you help us out here please?
We want to reproduce this, but are currently not able to without additional
information from you. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/prisma/prisma/issues/15199#issuecomment-1256445916,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJZFZZZVEBPKSQ25C4D6T3V7XOTZANCNFSM6AAAAAAQFPP5SQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok, closing this for now. If anyone hits the same problem, jsut comment here again and we can reopen and take another look.


Q4: Send Check Point firewall logs to syslog endpoint

Describe the issue
Configure Check Point firewalls to send logs to SysLog server.
We want to save checkpoint firewall logs and make them available to the OpenSearch SIEM. Check Point does not have a CloudWatch logging agent in the AMI but does support sending logs to a sys log endpoint. The SEA was designed to have an Rsyslog proxy endpoint in the Operations account for this purpose (originally for Fortigate).
Additional context

Notes from conversation with Ryan J

we can forward CP FW logs to a syslog endpoint there is a rsyslog NLB in Operations.
we would have to create an endpoint service in operations and then create an endpoint in the inspection vpc in the perimeter account

CP syslog config may be done through the AMI userdata startup script

Definition of done

Check Point firewalls are configured to send logs to syslog
Logs are visible in SIEM

Waiting for capability coming in Q4 that would work instead of us hacking together a wobbly solution

@NickCorcoran can you please review this issue and determine if it is still a valuable thing to pursue?


add jupyter notebook

Add Jupyter Notebook deployment SDL to Machine Learning category

This pull request has been mentioned on Akash Community. There might be relevant details there:
https://forum.akash.network/t/torrenting-metasearch-engine-data-science-on-akash/2865/1

Thank you!


A lot of "Failed to execute controller command after..."

Describe the bug
Every morning when I wake up, a motion detector right outside the bedroom detects when I'm "up" and turns on around 6-7  lights and more. This worked good in 5.7.0 but after I upgraded to 6.0.0, it takes very long time to execute these commands and the log is filled with "Failed to execute controller command after [...]".
I have attached a log file that starts with when motion is detected in the morning. As we can see in the log, it takes almost 4 minutes to turn a few of the devices.
Device information
Which device(s) is/are affected (make/model)? Mostly Everspring and Fibaro
What are the node IDs? 51, 66, 53 are a few
Last Known Working Configuration

[x] Previously working device (node-zwave-js)

Which library version/docker image/adapter version? 5.7.0

Installation information
How did you install node-zwave-js?

[x] Pkg

Logfile:
failed_to_execute.txt

Power-cycle your Z-Wave stick. In your log, it simply doesn't react at all for a while.

Power-cycle your Z-Wave stick. In your log, it simply doesn't react at all for a while.


Description
How to replicate:

Change the map to Packed Station.
Go to Cargo and order a Laser Crate.
Open the crate and get a Retro Laser Gun.
Go to Science and get the Large Power Cell they have there.
Your Retro Laser now has 225 shots in it.
This also applies (to a lesser degree) to the lasers in the armory on Packed.

Logically, each gun should only take one cell type (small or medium, as large is supposed to be for APCs as I understand it) and be unable to take anything else.

Adding additional checks to what items can be inserted beyond just checking an entity whitelist requires making some changes to ItemSlots. I was going to make those changes either way to support magazine-weapons with their different calibres, so I can just also add in similar checks for power cells when I do that.
Currently I'm just waiting on #5638, because that also modifies item slots and I don't want to create unnecessary conflicts that I then need to resolve.


Switching among different PHY Modes

Is your enhancement proposal related to a problem? Please describe.
I am working with nRF52840 DK using Zephyr 2.0, and now i would like to see the difference among different PHY Modes. In BLE stack, there are totally 4 PHY Modes, which are LE 1M PHY, LE 2M PHY, Coded PHY (S2) and Coded PHY (S8).
I saw some configuration choices like CONFIG_BT_CTLR_PHY_2M and CONFIG_BT_CTLR_PHY_CODED. I suppose if I would like to enable LE 2M PHY, I could easily use CONFIG_BT_CTLR_PHY_2M=y, and set CONFIG_BT_CTLR_PHY_2M=n to use LE 1M PHY. But how could I know which Coded PHY am I using, if I only enable Coded PHY by CONFIG_BT_CTLR_PHY_CODED=y? S2 or S8? Or maybe I still need to do some other configurations?
Additional context
Add any other context or graphics (drag-and-drop an image) about the feature request here.

@BOZHENG001 I think it makes sense to be able to configure this from the application's perspective. Let us hear from others to see how we can add this functionality.

Makes sense. I have had similar requests already and have a branch ready. I can submit a PR soon.


Release 1.0 test

@hklai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.1

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.1

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.1

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

prow/daily-e2e-pilot-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-pilot-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.1

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

prow/daily-e2e-pilot-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-pilot-no_auth

prow/daily-upgrade-tests-1.0.2.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.2

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.1

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

prow/daily-e2e-pilot-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-pilot-no_auth

prow/daily-upgrade-tests-1.0.2.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.2

prow/daily-e2e-bookinfoTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-bookinfoTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.1

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

prow/daily-e2e-pilot-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-pilot-no_auth

prow/daily-upgrade-tests-1.0.2.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.2

prow/daily-e2e-bookinfoTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-bookinfoTests

prow/daily-e2e-simpleTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.1

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

prow/daily-e2e-pilot-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-pilot-no_auth

prow/daily-upgrade-tests-1.0.2.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.2

prow/daily-e2e-bookinfoTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-bookinfoTests

prow/daily-e2e-simpleTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-simpleTests

prow/daily-e2e-mixer-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.1

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

prow/daily-e2e-pilot-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-pilot-no_auth

prow/daily-upgrade-tests-1.0.2.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.2

prow/daily-e2e-bookinfoTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-bookinfoTests

prow/daily-e2e-simpleTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-simpleTests

prow/daily-e2e-mixer-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-mixer-no_auth

prow/daily-e2e-dashboard.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.1

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

prow/daily-e2e-pilot-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-pilot-no_auth

prow/daily-upgrade-tests-1.0.2.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.2

prow/daily-e2e-bookinfoTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-bookinfoTests

prow/daily-e2e-simpleTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-simpleTests

prow/daily-e2e-mixer-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-mixer-no_auth

prow/daily-e2e-dashboard.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-dashboard

prow/daily-perf-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-perf-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.1

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

prow/daily-e2e-pilot-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-pilot-no_auth

prow/daily-upgrade-tests-1.0.2.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.2

prow/daily-e2e-bookinfoTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-bookinfoTests

prow/daily-e2e-simpleTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-simpleTests

prow/daily-e2e-mixer-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-mixer-no_auth

prow/daily-e2e-dashboard.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-dashboard

prow/daily-perf-tests.sh
d25887b8a136daf430b3dbf6ac6e1ced076ff7a4
link
/test daily-perf-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

prow/daily-e2e-pilot-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-pilot-no_auth

prow/daily-upgrade-tests-1.0.2.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-upgrade-tests-1.0.2

prow/daily-e2e-bookinfoTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-bookinfoTests

prow/daily-e2e-simpleTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-simpleTests

prow/daily-e2e-mixer-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-mixer-no_auth

prow/daily-e2e-dashboard.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-dashboard

prow/daily-perf-tests.sh
d25887b8a136daf430b3dbf6ac6e1ced076ff7a4
link
/test daily-perf-tests

prow/daily-upgrade-tests-1.0.1.sh
d25887b8a136daf430b3dbf6ac6e1ced076ff7a4
link
/test daily-upgrade-tests-1.0.1

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-unit-tests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-unit-tests

prow/daily-e2e-pilot-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-pilot-no_auth

prow/daily-e2e-bookinfoTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-bookinfoTests

prow/daily-e2e-simpleTests.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-simpleTests

prow/daily-e2e-mixer-no_auth.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-mixer-no_auth

prow/daily-e2e-dashboard.sh
216996124908ed777c8df87df40481997f4aec90
link
/test daily-e2e-dashboard

prow/daily-perf-tests.sh
d25887b8a136daf430b3dbf6ac6e1ced076ff7a4
link
/test daily-perf-tests

prow/daily-upgrade-tests-1.0.1.sh
d25887b8a136daf430b3dbf6ac6e1ced076ff7a4
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
d25887b8a136daf430b3dbf6ac6e1ced076ff7a4
link
/test daily-upgrade-tests-1.0.2

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Basic authentication support

It could be great to have basic authentication support and set the parameters in the athena.json file.

Hey @yael-lorenzo-olx
Could you please elaborate more? Could you provide some examples or use cases?

After using it, it is a bit hard to think of multi purpose model to do it. Perhaps this is for documentation and just as an informative case.
We are using the FluentRequestInterface to create the WEB requests and it has a function withBasicAuth($username, $password);
We are loading from athena.json file the user and pwd to set this up.


[IOTDB-573]claim the error output can be igonred in UT/ITs

PR #1015 introduced WrappedRunnable but print too many error messages in UT.
I make the error messages easy to be distinguished and reduce the number of test threads.

LGTM


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.1
Time: Sun Mar 20 2016 11:47:18 GMT-0400 (Eastern Daylight Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\Ed\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\inject\GPMInject\interface\mouseButtonNavigation.js:8:8.
    at BrowserWindow.callIntoRenderer (C:\Users\Ed\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\lib\rpc-server.js:201:19)
    at emitTwo (events.js:92:20)
    at BrowserWindow.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #470


Add support for cucumber

Hi,
I've been planning on adding a cucumber runner.
The ideia is to add some flags that control which files get sent as task. That way we could execute only rspec or only cucumber or both.
In the following images I've identified the spots where changes will be made.

What do you think about this plan to implement cucumber support?

Since we're not using Cucumber, we can't have strong support on Cucumber, and we don't have a plan to do so.
But if you can send us a pull request, we can review your implementation. (but remember I can't promise it can be merged or not)


Delete double/float formatting code moved to shared

Depends on https://github.com/dotnet/coreclr/pull/15246
Contributes to https://github.com/dotnet/coreclr/pull/15246

Cherry-picked into https://github.com/dotnet/corert/pull/5043


Does this plugin work with current simhub release?

Hi, I am looking for a plugin in order to use my slipro with simhub. This project looked like a good one but impossible for me to set up working . Thanks for your help or your comments. Cheers. Gege.

Happy on RD.


Move core visibility trees from itwinjs-core

Moved core visibility trees used by the tree widget to this package. Updated itwin packages to 3.5.0 because some of tests are using new API release in only in 3.5.0

Let's omit the JoesHouse.bim dataset. There're only a few ModelsTree integration tests that use it and they're pretty much worthless anyway. We can add proper integration tests targeted towards specific issues when they come up.
I'm not sure I like the current directory structure. I suggest this:./components
  - trees
    - ...common stuff
    - category-tree
      - ...newly added category tree related stuff
      - CategoriesTree.tsx
    - models-tree
      - ...newly added models tree related stuff
      - ModelsTree.tsx
    - spatial-tree
      - ...newly added spatial tree related stuff
      - SpatialTree.tsx

Updated structure and removed JoeHouse imodel and tests based on it.


Multiple numberformats in the same column?

I'm sorry if this was answered before but I wasn't able to find it.
I'm getting data from a stored procedure that I have no control over. The columns have whole numbers and decimal numbers (percentages) intermixed.
Is there a way to check the number for a decimal point and multiply it by 100 then add a %, if there is no decimal point just add the commas like normal?
I have another question about formatting each column with a different numberformat, but I can create a new issue if needed.
Thank you for the great script.

It should be possible for you to write a custom number formatter for this, yes.

I've implemented my own aggregator.
It uses different format per row. And I use "format" subfield to define data format. It passed to getFormat function.
	function getFormat(short) {
		if (short == '%') {
			return $.pivotUtilities.numberFormat({ digitsAfterDecimal: 2, scaler: 100, suffix: "%" });
		}
		if (short == '$') {
			return $.pivotUtilities.numberFormat({prefix:'$'});
		}
		if (short == '#') {
			return $.pivotUtilities.numberFormat({digitsAfterDecimal: 0});
		}
		if (short == '$M') {
			return $.pivotUtilities.numberFormat({digitsAfterDecimal: 2, scaler: 1000, prefix: '$'});
		}
		return $.pivotUtilities.numberFormat();
	}

I had plans to improve this function to use row/column name: use last character as format description like "Users, #", "Profit, $", "KPI, %"

Thank you guy, I was able to get it working. sorry it took me so long to get back to you.


Make Release smaller

See #57.
While testing this branch, I noticed that you have to delete any existing<EMAIL_ADDRESS>in order to get the smaller version, since the zip command only adds/updates new and changed to the (possibly already existing) zip archive.
This could cause problems when refactoring code, which might be needed in the future...
Should we always delete any existing<EMAIL_ADDRESS>then, before creating the new release? Or is there some option in the zip command that always creates a completely new archive (I don't think so)?

I think it's ok to delete the zip file if it already exists as this is very temporary thing. You do not want to have this lying around the source tree anyways.
Btw, I think you can move the changelog.md to the docs directory as well. I am pretty confident that there are no external links to this document yet.

That was quick :)


[terra-date-picker] Separate date input field into 3 distinct inputs

Issue Description
It would be helpful if the date picker automatically inserted the separator character as the user is typing the date.
Issue Type

[x] New Feature
[ ] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
The separator character should automatically be inserted by the date picker as the user enters the date.
Here is an example in en-US locale:  After the month is entered, the forward slash should automatically be inserted and the cursor should be moved past it ready for the user to input the day.  Once the day has been entered, then a forward slash should automatically be inserted and the cursor moved past it ready for the user to input the year.
Current Behavior
The separator character must be manually typed by the user.

Steps to Reproduce

Navigate to the date picker examples.
Try entering a date manually into the date picker.
Observe no separators are automatically inserted.

The changes made on #595 now limits the date input field to accept only numeric values and the "/" and "." separators (depending on the locale).

From leadership sync today, we are going to split the day, month and year into 3 separate inputs instead of a single one and require the user to type the "/".
cc: @amichaelparker, @neilpfeiffer

@ryanthemanuel / @avinashg1994, I was just talking with a solution designer about the timeline for this change. She mentioned that Ryan had indicated it would be available in Q3. I'm curious if this change is expected to be passive. We have some components we'd like to get it pulled into for Q3, but without knowing what it might take to uplift, it's hard to know whether this would be possible. Any info you can provide here would be appreciated. Thanks.

@rockhold, the intention is to keep things passive, but that being said this is still in progress and splitting the date input requires quite a lot of changes.
cc: @ryanthemanuel

Will internationalization be supported? Not every county uses '/' in their date format, few countries use '-' and few use '.'  WIKI. How will the new date picker work in that case?

@saigorantla,
Yes, Internationalisation will be supported.
DatePicker already supports the following formats and the same will also be supported as part of this upgrade.

This issue is still active. Please don't close.


Configuring format and localization of calendar period button in datepicker

Hi, I have this format at the header of datepicker:

How can I change it?
Please, notice that I am using 'fi-FI' locale, but it does not affect calendar period button area :(
Hope you guys can help me!
P.S. I'm using version '2.0.0-beta.11'. I also tried to reproduce problem on plunker (https://stackblitz.com/edit/angular-material2-issue-8naqf1), but with no success.

Please refer troubleshooting by connecting with the community at StackOverflow, Gitter, or Google Groups.


Sparkline not working in 3.52.0 when data defined as {x,y}

Description
The sparkline chart is not working in version 3.52.0, but working correctly in 3.50.0

Codepen https://codepen.io/olton/pen/mdZLLoY
-->

fixed in 3.53.0


pushs to master are not detected correctly

@rultor release, tag is 0.0.31

@rultor release, tag is 0.0.31

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.31

@yegor256 Done! FYI, the full log is here (took me 9min)


Is it possible to search by Id and the name

Hi, I am using angular chosen for one of my project. There is a selection box where I got few products I am displaying with chosen Select plugin, they can be searched by name at the moment. But I need it to work with productId as well. So what I need is to be able to search the product by name as well as Id. Is it possible to achieve that using this directive.
Let say my product object looks like this
[
{
"productId": 11,
"name": "Bananas"
},
{
"productId": 22,
"name": "oranges"
}
]
Please have a look thanks.

That would be very nice add on. But It seems to be out of scope for angular-chosen. If it can be added to chosen then we could expose an attribute in the directive to specify what key to search by.

Closing the issue.


Library not handling 401 error response?

Hi,
I was reading the part related to interceptors in the documentation, which states:

Since 3.1 the library uses a default HttpInterceptor that takes care about transmitting the access_token to the resource server and about error handling for security related errors (HTTP status codes 401 and 403) received from the resource server.

Reading at the library code version 4+ (4.0.2 to be precise), I can see that the Authorization header is sent, providing that the flag sendAccessToken is set, however I don't see any handling of the "401 Unauthorized" http response.
Can anyone shed some light? Maybe I'm missing something?

Thanks for looking into it @jeroenheijmans
I noticed #414, it is related, but different in the sense that it is more about hooking custom logic, like retries, on error responses.
I went to open this question-issue as I was confused by the statement that the library does handle 401 and 403.
I think it makes sense that the library comes with default logic for redirecting to login on 401 error.

Hah, good points. I clarified the other issue as well.
The retries mentioned there are secondary, the actual important thing is logging the user in (silently, if possible). Same as your intention, I still think? It's just that you seem to prefer what I called "Option 4", making the login upon 401/403 (and retry if the login was done silently), whereas I think it's an application concern and thus lean to "Option 1".
In any case, I would agree that the current documentation you quoted is not appropriate. Possibly the original author intended to suggest that 401s/403s are "prevented" by the default interceptor provided by the library, which is the case.
So in any case, I'm all for changing that bit of documentation to match current library behavior.
I'm not so sure if I'm fore "Option 4" / having the library redirec on 401s (certainly not by default), since those flows can vary greatly between applications.

I see. So the 401 / 403 that would occur because of the lack of the Authorization header are handled, not the 401 / 403 in general 🙂

I think so, yeah. AFAIK there's no feature in the library to handle 401/403 when the bearer token is already being sent.

Think the questions in this thread got answered, and since related issues and PRs have been resolved as well I'm going to close this issue.
Let us know if the issue should be re-opened (and what open issue still remains)!


After updating to 3.0.0 the base file names are not based on the source file name anymore

Now, after the update to version 3.0.0 (and replacing name by filename), it uses the name of the "main"-chunk, instead of the name of by worker files:
Search.worker.ts results into main.CuC84dZH.js, main.CuC84dZH.js.map and main.CuC84dZH.js.LICENSE.txt.
TypeDetection.worker.ts results into main.CsGA6GIM.js, main.CsGA6GIM.js.map and main.CsGA6GIM.js.LICENSE.txt
Beside that, I have the main.HeMLeGN2.min.js and main.HeMLeGN2.min.js.map containing the main chunk.
I've also tried to switch to chunkFilename or use [id].worker.js, but this doesn't help.
Any idea how to get the old behaviour?

Operating System: Windows 10 Pro
Node Version: 14.4.0
NPM Version: 6.14.5
webpack Version: 4.44.1
worker-loader Version: 3.0.0

Expected Behavior
The output files should have the same base file name as the source files.
Actual Behavior
The output files get the name of the "main" chunk.
Code
{
	test: /\.worker\.ts$/u,
	use: [
		{
			loader: 'worker-loader',
			options: {
				name: 'js/[name].[hash].js',
				publicPath: '/' // CDN path here
			}
		},
		{
			loader: 'babel-loader'
		}
	]
},

I'm importing the Worker on this way:
import Worker, { EventData, TypeInformation } from './TypeDetection.worker';

Hm, yes, you are right, it is bug, non critical (becase you have worker suffix), but will be great to fix it


Ignore principals order when comparing policies

Description of your changes
AWS returns principals items in a random order so provider-aws should ignore it to not consider policies changed.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Existing and new tests

@max-melentyev can you rebase on the latest master and resolve the conflicts?

rebased


Unable to update schedules and periods via CDK

I've followed the readme and deployed the CDK application for v1.4.2 solution. Now I want to update the schedules and periods through CDK and not via console or using scheduler-cli directly. However, I can't see any option to do so. I tried to update the  setup_demo_data.py file and hoped the DynamoDB would reflect the changes I made. Unfortunately, that did not work. Is there anything I can do so that I can update the schedules and periods via changing code in Git instead of doing manual operations of CRUD on the DB.

Any help with this @gockle @CrypticCabub ?

Hi @ramarahul
We do not have a construct for creating schedules through CDK directly, but we do have a custom resource that will allow you to manage schedules using Infrastructure as Code via CloudFormation templates.
You can find the documentation for this in our implementation guide here
And here is an example of how you can implement the IG's example schedule using the custom resource (note that this version takes in the ServiceToken ARN as an input parameter to make managing it easier)
AWSTemplateFormatVersion: 2010-09-09
Parameters:
  ServiceInstanceScheduleServiceTokenARN:
    Type: String
    Description: (Required) service token arn taken from InstanceScheduler outputs
Metadata:
  'AWS::CloudFormation::Designer': {}
Resources:
  StopOnWeekends:
    Type: 'Custom::ServiceInstanceSchedule'
    Properties:
      NoStackPrefix: 'True'
      Name: mon-9am-fri-5pm
      Description: start instances at 9am on monday and stop them at 5pm on friday
      ServiceToken: !Ref ServiceInstanceScheduleServiceTokenARN
      Timezone: America/New_York
      Periods:
      - Description: 9am monday start
        BeginTime: '09:00'
        EndTime: '23:59'
        WeekDays: mon
      - Description: all day tuesday-thursday
        WeekDays: tue-thu
      - Description: 5pm friday stop
        BeginTime: '00:00'
        EndTime: '16:59'
        WeekDays: fri

Posting in this thread as it is the only issue found on github related to instance scheduler so far.
I am having hard times finding ServiceInstanceScheduleServiceTokenARN .. Where can I find it?

hi @oussemabhouri
The service token ARN can be found in the CloudFormation outputs of your main hub stack.
To access these, go to CloudFormation, select your deployed Instance Scheduler hub stack, and then select "Outputs" from the navbar above the stack information

Got it running.
Thank you
Caleb Pearson @.***> schrieb am Di. 28. Nov. 2023 um
14:45:

hi @oussemabhouri https://github.com/oussemabhouri
The service token ARN can be found in the CloudFormation outputs of your
main hub stack.
To access these, go to CloudFormation, select your deployed Instance
Scheduler hub stack, and then select "Outputs" from the navbar above the
stack information
—
Reply to this email directly, view it on GitHub
https://github.com/aws-solutions/instance-scheduler-on-aws/issues/453#issuecomment-1829877147,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM35XFNAMSWAJ5DRRPU4ZLDYGXTHFAVCNFSM6AAAAAAXQ226QGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRZHA3TOMJUG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Often Graph API to get Outlook calendars and events fail with "Unknown Error"

Describe the bug
I'm using following Graph API calls in a Java Backend:

https://graph.microsoft.com/v1.0/me/events
https://graph.microsoft.com/v1.0/me/calendars

Often API returns following error:
{
    "error": {
        "code": "UnknownError",
        "message": "",
        "innerError": {
            "date": "2023-05-12T07:50:26",
            "request-id": "3a497a63-9a12-4492-8d7a-8548cc1c8b6c",
            "client-request-id": "cc09d87e-1b67-4631-f6dc-b7c0ca29d7d8"
        }
    }
}

Sometime same APIs ran without any issue.
I ran APIs also with MS Graph Explorer and I got same issues.
I tried with my AD credentials<EMAIL_ADDRESS>where is configured a Exchange onprem mailbox with Exchange hybrid connector.
To Reproduce
Steps to reproduce the behavior:

Go to https://developer.microsoft.com/en-us/graph/graph-explorer
Run query with above APIs
See error

Expected behavior
No unknow errors.
Screenshots
NA
Desktop (please complete the following information):

OS: Windows 11
Browser Edge
Version 113.0.1774.35

Additional context
I noticed that a collegue with an Exchange Online mailbox on same tenant got always successfull results.

Hey @itague,
This may be a problem with the Outlook API.
Please post this on the Graph Q&A forum where the appropriate team will be able to help.


Fix meta property declaration to be optional

According to the comment immediately above the property, meta is supposed to be optional. This PR brings the type description in line with that sentiment.

Coverage remained the same at 100.0% when pulling 6ea25b72a4a0d1eb8792fb8238bfc3dfd853a4fe on smithad15:patch-1 into e55a7364b1caa694b4a6bc9a0b66d7815e5761aa on acdlite:master.

Thank you for the PR, but please see the discussion in #53.

Understood, however, this does cause some confusion between the documentation and the type enforcement. If you are hesitant to change the type declarations while waiting for TS2.3, might I suggest changing the documentation to reflect the current limitations instead? Having a property described as optional, and then enforcing its inclusion only for TypeScript users is a little counterintuitive. Personally, I've had to implement my own type description in my projects in order to work around this. In the end, it's up to you. Thanks for the library!

@smithad15 thanks for the response, sorry for my slow response. Please feel free to submit that documentation change that is relevant until TS 2.3, but for now I'm going to close this PR in favour of #53.


ChunkingSink does not write Requests/Responses without body

Detailed Description
ChunkingSink should write requests/responses without a body. It should use the delegate sink directly as it only chunks the bodies.
Your Environment
* Version used: 2.1.0

Reproducing
// create a sink which prints to console
var sink = new ChunkingSink(new DefaultSink(new DefaultHttpLogFormatter(), new HttpLogWriter() {

	@Override
	public void write(Precorrelation precorrelation, String request) throws IOException {
		System.out.println(request);

	}

	@Override
	public void write(Correlation correlation, String response) throws IOException {
		System.out.println(response);
	}

}), 100);
// just use whatever does not matter for this test
var corr = new Correlation() {

	@Override
	public String getId() {
		return UUID.randomUUID().toString();
	}

	@Override
	public Instant getStart() {
		return Instant.now();
	}

	@Override
	public Instant getEnd() {
		return Instant.now();
	}

	@Override
	public Duration getDuration() {
		return Duration.ofSeconds(5);
	}

};
// will be printed
sink.write(corr, MockHttpRequest.create().withBodyAsString("bodyAsString"));
sink.write(corr, MockHttpRequest.create().withBodyAsString("bodyAsString"), MockHttpResponse.create().withBodyAsString("bodyAsString"));
// will not be printed
sink.write(corr, MockHttpRequest.create());
sink.write(corr, MockHttpRequest.create(), MockHttpResponse.create());

Thanks for the test case! I'll try to take a look soon.

Fixed in 2.2.1


PhantomJS-prebuilt dependency

Hi,
isnt phantomjs supposed to go in the dev-dependencies?
adding 50mb to every distrubution of your project for an as far as i have seen unused tool looks a bit weird.
Could you maybe explain why this is a dependency and not a dev-dependency for testing?

@SHoogland Please follow issue #100 for further details. (Working) Suggestions very welcome!

Ow sorry, didnt look hard enough after all 🙈

@SHoogland No worries! ;) As I said, I'd totally appreciate any working suggestion to avoid including PhantomJS as regular dependency, but unfortunately I'm not aware of any (without dropping functionality)


Create service principal in AAD

Add secret information to the secrets for the dotnet-podcast repo forked to my account.


Add fulfillment settings card

I want to merge this change because... it adds fulfillment settings card in orders settings.

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://qa.staging.saleor.cloud/graphql/

Task linked: SALEOR-3858 Add Fulfillment Settings card

Everything looks ok, but by default both fulfillment settings should be enabled.

They will be. These settings need to be enabled by default on backend, but backend doesn't have them implemented yet. So, as soon as backend has it implemented, it will be also enabled here, because dashboard simply reflects what settings state is saved on backend.

Task linked: SALEOR-4021 Make fulfillment approvement naming consistent


Sequence bytes in class B transactions

Sequence bytes were intended to be used to order data packages of a Mastercoin transaction.
Very sadly there were multiple transactions which did not follow the spec on this and were (and are?) parsed as valid nevertheless with the assumption that the data is in order.
I assume this means the requirement of ordered sequence bytes can be dropped now? If this is the case, what are we going to do about this in the future? If it's not a requirement, it's wasted space and if it becomes a requirement in the future, then we need to discuss how the transition will happen.

Sequence bytes are ignored during parsing. /close


Small issue with not display filesize by the batch download

Bug description
When you start a batch download you always have a column with the filesize which is by sankaku complex empty (same also with gelbooru and probably also other sources)
Steps to reproduce
Just start a batch download and you will see that the column is empty
Expected behavior
Seeing the filesize inside this window
Screenshots
https://i.imgur.com/55maAGv.png
System information
Windows 10 with the latest nightly version

Indeed, Sankaku (in regex mode) does not provide a filesize.
But Grabber could indeed show the size once it starts downloading and for already downloaded files. :+1:

@Bionus this also happens with gelbooru like you can see here: https://i.imgur.com/CsLCtXi.png


Improve the Phyx summary page

Two minor fixes may be needed:

The phyloref table adds a column for each phylogeny, but currently only uses the label of the phylogeny (e.g. "Fig 1 from Brochu 2003"). Is this clear enough, or should we add "Phylogeny: " to the front of the phylogeny label (e.g. "Phylogeny: Fig 1 from Brochu 2003")?

[ ] If we change this, remember that our default name for unlabeled phylogenies is Phylogeny 1, so we want to make sure we don't end up writing this as Phylogeny: Phylogeny 1!

https://github.com/phyloref/klados/blob/ce4e31b1de251ff10a22cbc5b2e17ceb41f73436/src/components/phyx/PhyxView.vue#L105-L107

The phylogeny table refers to the "Description" of each phylogeny, but this field no longer exists. Should we replace this field with the curator notes? Or some other field, such as the source, number of leaf nodes, list of leaf nodes, etc.?

https://github.com/phyloref/klados/blob/ce4e31b1de251ff10a22cbc5b2e17ceb41f73436/src/components/phyx/PhyxView.vue#L214

Perhaps prefix with "Phylogeny for ..."? But generally yes for using some prefix.
As for the table, it seems the most natural change would be to remove this column from the table as well. I don't think there's a good reason to maintain the number of columns, and as you implicitly suggest, there is no clear reason by which to select from the list of fields that aren't currently reported the particular field to start reporting in the table.


UKHPI - 3.2.3: Consistent Navigation

See https://docs.google.com/spreadsheets/d/1YS3N5ps9pJGFOxBDi9C2DwA37WQPIVax/edit?gid=1215429860#gid=1215429860&range=24:24

@tomguilbert The requirement for this ticket is to have consistent navigation elements across all HMLR applications. But, as mentioned in the FE planning meeting, this is not a HMLR service and naturally has different navigation elements compared to the other services. Another challenge with this would be the fact that the logo cannot link to the LR landing page because this is not a HRML app, instead it must link to the UKHPI landing page.

Agreed- let's can this one.


sanctuary fixes

reset git state before running tests
fix a bunch of nits with string/error reporting in CI
trim null terminators from input files

It'd be great to have more context and reason behind these changes 🙏

@Xanewok apologies. I added it to the commit message, but for some reason, it wasn't copied to the PR. Added!


Drop Django 1.9 and Python 3.3 from CI

As agreed at https://groups.google.com/d/msg/wagtail-developers/nTn-rthPMsA/ORP3GA-hCQAJ

@kaedroho Good point! Merged in 63d52af0a32b7f1d9e20c29a943c7ff5106b9427, with documentation about supported versions updated in 187908e7738e64030ca72fdde8a0f7642fae1098.


Update go version

https://hub.docker.com/layers/library/golang/1.19.1/images/sha256-122f3484f844467ebe0674cf57272e61981770eb0bc7d316d1f0be281a88229f?context=explore
Optionally change the golang version to "latest" so that you don't have to update it as often.

Merged into https://github.com/onemorebsmith/kaspa-stratum-bridge/pull/21


[Typo/Mistranslation] 07-design-report-in-power-bi-desktop-enhanced

On Practice 1, task 3 step 1 through 3, the column and table names need to stay in English so it will align with the lab environment. It is very confusing for the learners to see the column/table names in Korean on the instruction and then have to match it with the corresponding English names on the lab environment.
Current:
연도
연도
연도
To be:
Year
Year
Year

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Locfile at PHO - Project HO disabled - must follow up with the PM

Discussed with the PM - update made to the older HB locfiles and this will seed the TM when handedback


DXE-2799 export-appsec missing metadata parameter for custom rules

As per appsec API documentation, when creating new custom rules we have the possibility to add parameter metadata which includes the metadata used for unstructured custom rules.
Here is an example of an export of a Custom Rule via API :
HTTP/1.1 200 OK
Connection: keep-alive
Content-Encoding: gzip
Content-Length: 642
Content-Type: application/json
Date: Mon, 19 Jun 2023 10:49:37 GMT
Server: nginx
Vary: accept-encoding
X-RateLimit-Remaining: 99
x-ids-session-id: db6f5df1-4e8f-442e-829f-75626e3e7803
x-ratelimit-remaining-system: 9988
x-trace-id: 92bb649032c14d6f

{
    "id": 60164190,
    "inspectRequest": true,
    "inspectResponse": false,
    "loggingOptions": [
        {
            "id": "CLIENT_TLS_FINGERPRINT_MATCH"
        }
    ],
    "metadata": "<match:metadata-stage value=\"client-request\">\n\t<match:uri.wildcard compare-by-skipping-path-segment-parameter=\"on\" normalize=\"on\" result=\"true\" value=\"/*/careers/* /*/careers\">\n\t\t<assign:variable>\n\t\t\t<name>VAR_UNAUTHORIZED_ACCESS</name>\n\t\t\t<value>true</value>\n\t\t\t<hidden>on</hidden>\n\t\t</assign:variable>\n\t\t<match:uri.wildcard compare-by-skipping-path-segment-parameter=\"on\" normalize=\"on\" result=\"true\" value=\"/*/careers/home /*/careers/home/\">\n\t\t\t<assign:variable>\n\t\t\t\t<name>VAR_UNAUTHORIZED_ACCESS</name>\n\t\t\t\t<value>false</value>\n\t\t\t\t<hidden>on</hidden>\n\t\t\t</assign:variable>\n\t\t</match:uri.wildcard>\n\t</match:uri.wildcard>\n\t<match:variable name=\"VAR_UNAUTHORIZED_ACCESS\" value=\"true\">\n\t\t<security:firewall.action>\n\t\t\t<msg>Unauthorized Access to legacy jobs career paths</msg>\n\t\t\t<tag>CUSTOM_AKA/BLOCK_LEGACY_JOBS_CAREER_PATHS</tag>\n\t\t\t<id>60164190</id>\n\t\t\t<action>%(WAF_CUSTOM_R60164190_ACTION)</action>\n\t\t\t<data>%(AK_PATH)</data>\n\t\t</security:firewall.action>\n\t</match:variable>\n</match:metadata-stage>",
    "name": "BLOCK - Specific paths",
    "operation": "AND",
    "ruleActivated": true,
    "structured": false,
    "tag": [
        "CUSTOM_AKA/VISIBILITY_EMAIL_PARAMETER"
    ],
    "version": 6
}

and here is the TF code generated with the export-appsec command where we can observe that the metadata parameter is missing:
resource "akamai_appsec_custom_rule" "block__specific_paths_60164190" {
  config_id = akamai_appsec_configuration.config.config_id
  custom_rule = jsonencode(
    {
      "loggingOptions" : [
        {
          "id" : "CLIENT_TLS_FINGERPRINT_MATCH"
        }
      ],
      "name" : "BLOCK - Specific paths",
      "operation" : "AND",
      "tag" : [
        "CUSTOM_AKA/VISIBILITY_EMAIL_PARAMETER"
      ]
    }
  )
}

Hi @brrbrr,
thank you for reporting this issue. We will look into this and get back to you soon
Best regards,
Tatiana Slonimskaia

Hi @brrbrr, unfortuantely only the structured custom rules can be managed through the API and Terraform. Unstructured customer rules can be only managed via UI.


Cannot access the docs

Hello there!
I would like to inform that seems to have an issue regarding the docs for this lib (as of 2023-10-23 18:08 UTC)
https://downforeveryoneorjustme.com/papooch.github.io?proto=https

Recovered link from:  https://www.npmjs.com/package/nestjs-cls?activeTab=readme
Original Url: https://papooch.github.io/nestjs-cls/

Hi, thanks for the report, but there is nothing in the root of https://papooch.github.io, so it shows 404. The sub-path /nestjs-cls works for me on https://papooch.github.io/nestjs-cls/. This is also confirmed by another down detector tool. Are you by chance in a region where your ISP could be blocking the github.io domain?
If you can't access the website, they're always available in markdown format in the docs folder. (e.g. Quick Start)

Hi, thanks for the report, but there is nothing in the root of https://papooch.github.io, so it shows 404. The sub-path /nestjs-cls works for me on https://papooch.github.io/nestjs-cls/. This is also confirmed by another down detector tool. Are you by chance in a region where your ISP could be blocking the github.io domain?
If you can't access the website, they're always available in markdown format in the docs folder. (e.g. Quick Start)

Thank you for the response!
Well, I don't know (yet) why it isn't working for me
I even tried usinc CURL

I tried using a VPN, Its working now, I am not able to describe why It only works for me through a VPN (Asia), I am currently located in Brazil.

Working like a charm now, without VPN.


Money

Hello, I noticed something, I don't know if it's a bug or a configuration error.
When I start a match I noticed that the money we receive is not the same on the main interface and on the scoreboard he adds the number 9 at the bottom left in the indication of the money.
CSS : counterstrikesharp-with-runtime-build-246-windows-a87bd25
MatchZy : MatchZy-0.7.12-with-cssharp-windows
Metamod : 2.0/1293

thank you in advance.

Hey!
This is weird, I don't think plugin is causing this issue as money is not changed in a live match.
Are you facing this issue withou MatchZy plugin? Or is there any other plugin which can cause this issue?
A wild guess, but this can be an issue with CS2 also. It can be an issue of the language that you're using in CS2, can you check if this looks fine in English?

I removed the MatchZy plugin and I don't have the bug.

I reinstalled the MatchZy plugin.

Good morning.
I have just tested your version of the plugin, there are no bugs under the version of (counterstrikesharp-with-runtime-build-244-windows-7f5103d).
But when I use the latest version of (counterstrikesharp-with-runtime-build-247-windows-f8451c2) the bug appears.
Your plugin is experiencing bugs with the latest counterstrikesharp version.
Can you check.

Hey @Geant200
Can you tell me (step by step) when the money is bugged? I will try debug it for you :-) /Miksen

I just retested with the latest version of (counterstrikesharp-with-runtime-build-247-windows-f8451c2)
The bug has disappeared, I will do some tests and if it happens again I will report back to you ^^.
Thanks ^^.

Hey!
Closing this issue since this is not reproducing now, you can open this if you feel this issue still persists.


fix invalid regex pattern causing test failures

replaced invalid regex pattern " * " with " .* "
ensured compatibility with ecmascript syntax

my contribution here was very small so I added this as a comment, I'm closing this place


Prevent drawing an extra line in draw_line_segment_mut

In cases where is_steep was true, (y, x) was out of bounds, but
(x, y) was in bounds, the else clause would draw an extra line segment by
mirroring the out-of-bounds part of the line into the image.
Example
extern crate image;
extern crate imageproc;

use image::{ImageBuffer, Rgb};

fn main() {
    let mut img = ImageBuffer::<Rgb<u8>, Vec<u8>>::new(400, 200);
    let red = Rgb { data: [255, 0, 0]};
    imageproc::drawing::draw_line_segment_mut(&mut img, (-100., 500.), (400., -300.), red);
    img.save("line.png").unwrap();
}

Before:

After:

Thanks!


CMake updated to use /Zc:inline and /Zc:lambda

The /Zc:inline switch enforces C++11 rules on inline visibility which reduces redundant comdats in each OBJ file (i.e. all the DirectXMath inline functions you didn't call in that translation unit). This switch is on by default with MSBuild, but is not on by default for command-line builds. This update adds this switch to CMake builds as well.
This reduces the size of the Release obj/lib files in half.
There's also a /Zc:lambda conformance switch for VS 2019 16.8 or later which I'm enabling for coverage.

Related updates for test suite: https://github.com/walbourn/uvatlastest/commit/77cc67726be688346778c3bb01c6dff43cc8ee5a https://github.com/walbourn/uvatlastest/commit/77cc67726be688346778c3bb01c6dff43cc8ee5a


When graphqlgen large schema will get stuck

Description
In one of my projects, the datamodel has five hundred lines. My schema.graphql file imports the generated prisma.graphql schema file and has a hundred lines of Query and Mutation.
I get stuck when I run graphqlgen. Cpu and memory are slightly higher. I noticed a prompt for the find prettier, but I still didn't succeed after removing the prettier configuration.
My three computers have linux, mac, and windows systems, and they are not working properly. But the surprise is that my mac has been successful once, just once.
Steps to reproduce
When schema.graphql too large.
My other small project run graphqlgen perfectly.
Expected results
Graphqlgen work success.
Actual results
Stucked. And console has no message.
Versions

graphqlgen: 0.4.0
OS name and version: mac: 16G memory, mojava(darwin-x64). windows: win10 16G memory i7.
Node: node-v10.8.0

Supplement：
When wait 1 hour later, console log err as follows：
graphqlgen
<--- Last few GCs --->

[14832:000002002A721AF0]  8948164 ms: Mark-sweep 1159.4 (1202.0) -> 1159.4 (1165.5) MB, 3679.4 / 0.0 ms  last resort GC in old space requested
[14832:000002002A721AF0]  8948951 ms: Mark-sweep 1159.4 (1165.5) -> 1159.4 (1165.5) MB, 786.8 / 0.0 ms  last resort GC in old space requested

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0000038B35425EE1 <JSObject>
    0: builtin exit frame: concat(this=000002C19A6AE511 <JSArray[40009211]>,000002C19A682201 <JSArray[18728217]>)

    1: flatten(aka flatten) [C:\Users\T47\AppData\Roaming\npm\node_modules\graphqlgen\dist\utils.js:~49] [pc=000002BC79E8D883](this=0000006577382311 <undefined>,a=000002C19A6AE511 <JSArray[40009211]>,b=000002C19A682201 <JSArray[18728217]>)
    2: arguments adaptor frame: 4->2
    ...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory

Same issue.

@mentrie If That for the suggestion.  But if I downgrade, then I'm hit with the #280 bug 😢

@LawJolla Right. For the past 1hr, 0.4.0 is still running 😢

Same issue here. Can't use at all until this is resolved. My system has 32 GB of RAM, and it still doesn't work. Haven't let it run for an hour, but after several minutes I still have plenty of RAM left.

I've spend few hours debugging the package and it seems that here :
https://github.com/prisma/graphqlgen/blob/03336f01d1257643c4b24577a397df44bd00076a/packages/graphqlgen/src/generators/common.ts#L228 it turns into the endless loop.
When i add Mutation to my schema with the same name as one in the prisma.graphql it runs until it fails.

@mentrie Thanks for the suggestion. But if I downgrade, then I'm hit with the #280 bug 😢

that has been fixed. I've downgraded to 0.3.0 and it worked even with the schema that fails on 0.4.0

@mentrie Thanks for the suggestion. But if I downgrade, then I'm hit with the #280 bug 😢

that has been fixed. I've downgraded to 0.3.0 and it worked even with the schema that fails on 0.4.0

@woss It's still an issue for me in 0.3.0.  This project looked promising, but it now appears abandoned.  I've given up and gone back to Javascript to actually get things built... pesky deadlines. :)

Anyone have a schema that leads to this issue that they can share?

@jasonkuhrt  I have figured out its problem, it has nothing to do with the size of the schema, but an error when using the imported Input Type.
like this example : https://github.com/prisma/graphql-prisma-typescript/blob/master/src/schema.graphql
when you add some Mutation using imported Input Type it will stuck:
# import * from "./generated/prisma.graphql"
type Mutation {
  addPaymentMethod(
    PaymentCreateInput
  ): MutationResult!
  book(
    BookingCreateInput
  ): MutationResult!
}

This bug renders graphqlgen unusable for me.
I was more specific with my imports from Prisma generated code.
// I used
# import Post, User, UserCreateInput, UserUpdateInput from './generated/prisma.graphql'

// VS
# import * from "./generated/prisma.graphql"

node -v
v11.4.0

npm -v
6.4.1

OSx 10.12.6

prisma -v
prisma/1.22.2

https://ibb.co/0DjYqkG
Any hacky workaround ideas for now? A nightly I can test?

Quick update: in VS code, using cmd + . then selecting add all missing imports adds all missing imports - who would've thought?

@Victorkangsh iiuc:

This issue occurs in conjunction with graphql import

I should be able to repro with
https://github.com/prisma/graphql-prisma-typescript/blob/master/src/schema.graphql

But a more minimal repro can be had by importing one schema (A) into another (B) and using A types in B Mutation fields args.

Did I get that right?

@jasonkuhrt  I think the problem is nested InputType.
I use this config:

when I try this:

it's not work, and i copy in schema:

still stuck.
But when I remove nest Type , it worked:

graphql import is no problem, I test this:

all success

@jasonkuhrt

Anyone have a schema that leads to this issue that they can share?

https://github.com/vdiaz1130/graphqlgen-issue310

Anyone have a schema that leads to this issue that they can share?

To provide some more context.
I tried debugging it with console.logs, and I see that it gets stuck around getDistinctInputTypes method in common.js file.
I could see that the type it's trying to parse is Mutation

I worked around this by not importing from Prisma generated. I simply copied over my schema and removed all the Prisma specific things (@relation, @default, etc). Also using version 0.3.0 of graphqlgen.
If in your Prisma generated code you have an error on the Model interface, delete the import and reimported via VS code "cmd + ." It should import from a ''prisma-client-lib/dist/types' - Fix this before you generate with graphqlgen.
After generating with graphqlgen, If you get missing imports from your generated code and you are using VS Code, go next to a type, class, interface etc, then hit "cmd + ." Select "add all missing imports".

Thanks @jasonkuhrt !
Is there a particular release we should try?  I still see 0.4.0 as the current.

Before considering this issue resolved I'd like to hear from multiple users if its their experience that the issue is resolved too.

I added the line you added in the pull request to my local code.
It's been running for ~90 mins without stopping.
Looks like it's still reproducing.

@LawJolla I'm not doing releases yet myself I'm afraid (might start to this or next month though) but maybe you could pull down from git repo (but build steps and all...). Given the diff is trivial what @atardadi did might be easiest.
@atardadi thanks for the feedback! Good to know...

@vdiaz1130 thanks for the repo repro. I'd like to extract the minimal:

the model file(s)
the schema file(s)

I need and repro with the graphqlgen test suite.
But, can you advise how I should get prisma-client-lib installed and setup exactly? Even better might be a repro that doesn't need that dep to begin with (would simplify a bit I suppose).

I was so excited with graphqlgen :(
Even if there is a way around, there are a lot of issues unfixed!

@Victorkangsh ok thanks for confirming, interesting... I think then, since this issue was originally created by you we should close it. Others have raised other issues here but going by your feedback, they are different issues. As such I would like to see them tracked as separate issues and have focused discussion.
Does anyone strongly object?

@Victorkangsh ok thanks for confirming, interesting... I think then, since this issue was originally created by you we should close it. Others have raised other issues here but going by your feedback, they are different issues. As such I would like to see them tracked as separate issues and have focused discussion.
Does anyone strongly object?

I don't mind, although I have the same issue - once my schema grew large it doesn't generate it for me.
We could try publishing a new version and see if that solves it for me.
Otherwise, I will create a separate issue


Make Example / Unit Tests which show the Problem with FK constraints and Entity existence

Should show two entities with an FK relation over a technical name, write a new entity which references another by its technical name without making a lookup for the UUID first.

Done: https://github.com/shopware/platform/pull/144


Ready for packagist.org

This pull request accomplishes the following:
PHP 7.1 support.
Union types.
Analyze and use types found in function signature.
Align and group the generated phpDoc parameters.
Use proper type names.
Add support for float type.
Modernize the code to use PRS-4 auto-loading.
Port tests to phpunit (note 3 tests are disabled, partial parsing isn't supported any more).
Fixed some edge cases.
Add phpDoc and type-hinting to the code base.

Why use camelCase for bin/PHPTracerWeaver? Doesn't seem like a common convention for shell scripts.
Move trace.sh in bin/ as well.

@troelskn any other changes you would like done to this before it is ready for merge?

Don't think so. Good to see this project coming back up to speed.

Ok, if you merge it I think I am about ready with the next batch of changes :)


Does VSTS login username and password can be used to connect with git repo

The document only list three authentication types (PAT, SSH and alternate credentials) to connect with VSTS git repo. What about using VSTS login usename (email address) and password?

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

ID: 3f8b4989-21f0-53c5-dfce-6d5f1a66b8b2
Version Independent ID: 2f4419a7-408b-40cd-9190-985a9976d556
Content
Content Source
Service: unspecified

I'll find out and update you shortly.

@lvsezhufu If you are using Git Credential Manager for Windows and connecting to your VSTS Git repo, you can use your VSTS username and password in the Git Credential login window.

@steved0x But actually not. For some VCS GUI the email address and password can not be recognized (like GitKranken).  While for some VCS GUI the email address and password can be recognized (like sourcetree). That's confusing.


Need to fetch data from the server for Followers/Following user rather then fetch at the initial loading

Completed on
https://github.com/Roman-Ojha/Social/commit/8b887ff92667ee4b9d4acc037d98f46dd26d1937
https://github.com/Roman-Ojha/Social/commit/058c666545f105504267311f215a25c463bb1c14


Cast int cell to string in excel file

Let's say I want to parse the attached file.
It has in the A column string or int, but I want every cells of that column to be string.
I tried with this package.yaml
resources:
  - name: example
    type: table
    path: example.xlsx
    scheme: file
    format: xlsx
    encoding: utf-8
    mediatype: application/vnd.ms-excel
    schema:
      fields:
        - name: A
          type: string
        - name: B
          type: string
    pipeline:
      steps:
        - type: table-normalize
        - type: field-merge
          fromNames:
            - A
            - B
          name: AB

and runned this python script
from frictionless import Package

p = Package.from_descriptor('package-issue.yaml')

p.extract()

for r in p.resources:
    r.transform()
    print(r.to_view())

I have this error when I try to tranform it with field-merge
frictionless.exception.FrictionlessException: [step-error] Step is not valid: "field_merge" raises "sequence item 0: expected str instance, NoneType found"

Is there a way to cast all ints in string ?
example.xlsx

@l-vincent-l
Thanks for reporting! We will investigate both issues

Hi @l-vincent-l,
Thanks!. Can you please try the following methods:

First method: Set the format of the column in excel file to string and then read the resource. And use preserve_formatting=True as follows:

descriptor = {
  "resources": [
    {
      "name": "example",
      "type": "table",
      "path": "example.xlsx",
      "scheme": "file",
      "format": "xlsx",
      "encoding": "utf-8",
      "mediatype": "application/vnd.ms-excel",
      "schema": {
        "fields": [
          {
            "name": "A",
            "type": "string"
          },
          {
            "name": "B",
            "type": "string"
          },
          {
            "name": "C",
            "type": "string"
          },
          {
            "name": "D",
            "type": "any"
          },
          {
            "name": "E",
            "type": "integer"
          }
        ]
      }
    }
  ]
}
from frictionless import Package, Resource, formats, Pipeline, steps
control = formats.ExcelControl(preserve_formatting=True)
p = Package.from_descriptor(descriptor)
pipeline = Pipeline(
  steps=[
            steps.field_merge(name="F", from_names=["C", "B"]),
        ],
)
for resource in p.resources:
    resource = Resource(resource.to_descriptor(), control = control)
    print(resource.to_view())
    transform = resource.transform(pipeline)
    print(transform.to_view())

+-------+-----+-----+-----+---+
| A     | B   | C   | D   | E |
+=======+=====+=====+=====+===+
| '001' | 'b' | '1' | 'a' | 1 |
+-------+-----+-----+-----+---+
| '002' | 'c' | '1' |   1 | 1 |
+-------+-----+-----+-----+---+

+-------+-----+---+-------+
| A     | D   | E | F     |
+=======+=====+===+=======+
| '001' | 'a' | 1 | 'b-1' |
+-------+-----+---+-------+
| '002' |   1 | 1 | 'c-1' |
+-------+-----+---+-------+

Second method: Use CSV format.

descriptor = {
  "resources": [
    {
      "name": "example",
      "type": "table",
      "path": "example.csv",
      "scheme": "file",
      "schema": {
        "fields": [
          {
            "name": "A",
            "type": "string"
          },
          {
            "name": "B",
            "type": "string"
          },
          {
            "name": "C",
            "type": "string"
          },
          {
            "name": "D",
            "type": "any"
          },
          {
            "name": "E",
            "type": "integer"
          }
        ]
      }
    }
  ]
}
from frictionless import Package, Resource, formats, Pipeline, steps
package = Package.from_descriptor(descriptor)
pipeline = Pipeline(
  steps=[
            steps.field_merge(name="F", from_names=["C", "B"]),
        ],
)
for resource in package.resources:
    print(resource.to_view())
    transform = resource.transform(pipeline)
    print(transform.to_view())

+-------+-----+-----+-----+---+
| A     | B   | C   | D   | E |
+=======+=====+=====+=====+===+
| '001' | 'b' | '1' | 'a' | 1 |
+-------+-----+-----+-----+---+
| '002' | 'c' | '1' | '1' | 1 |
+-------+-----+-----+-----+---+

+-------+-----+---+-------+
| A     | D   | E | F     |
+=======+=====+===+=======+
| '001' | 'a' | 1 | 'b-1' |
+-------+-----+---+-------+
| '002' | '1' | 1 | 'c-1' |
+-------+-----+---+-------+

Files used
example.csv
example.xlsx

Jumping in (I work with @l-vincent-l ) to add a bit of context to the initial question
We use Frictionless/Table Schema in a workflow where we do not control the data/dataset production. Instead, we actually use Frictionless/Table Schema to add a postprocess schema so to "normalize" the data before it get imported.
Does that usage makes sense to you ?
Thanks!

Thanks @yohanboniface. I did not understand completely. Could you please explain in detail your usecase with data ? Question that I have is:
-> Is data generated/produced of same type or different in terms of files, format?

@shashigharti we have an ETL where we ingest data that we do not produce. The dataset file and format is known, but we cannot change it.
In other words, we want to be able to import any dataset from anywhere on the Internet, "just" by defining a "schema file" on our side that would give to our ETL the rules needed to "normalize" the data before importing it in our database. By normalizing, I mean making sure for example that an int column only contains int, or mapping "pseudo booleans" to a real boolean column, or making sure that some administrative codes are valid, etc.
The ETL output is a normalized table in a postgresql database (each resource ends in a separate table).
To give a bit more details: we currently have our own code base, very similar to Frictionless, and we are evaluating the opportunity to use Frictionless, instead of reinventing the wheel.
To have an example of how we describe our ETL steps currently: https://datahub.incubateur.tech/infrastructure/insitu/-/blob/main/insitu/datasets/numerique.yml
Hope this helps! :)

Thanks @yohanboniface, now it is clear.
I will discuss this with @roll  and reply here.


Support spring-webmvc for jakarta

There is a change from spring boot3 (spring 6) to jakarta.servlet.
Currently, brave-instrumention-spring-webmvc only supports javax.servlet.
Are there any plans to support jakarta.servlet?
In the long run, I think spring users would be better off with micrometer tracing, but I think existing users would be happier with brave-instrumention-spring-webmvc as well.

Spring mvc natively supports micrometer observation. They work with micrometer tracing which has a brave bridge.

On the side note, the

Are there any plans to support jakarta.servlet?

It is supported arleady https://github.com/openzipkin/brave/tree/master/instrumentation/servlet-jakarta

It is supported already https://github.com/openzipkin/brave/tree/master/instrumentation/servlet-jakarta

servlet-jakarta is coresspond to servlet, not the jakarta version of spring-webmvc


Feedback v1

[x] Fix Head Badges

[x] Rename What is Laravel Auth?:
Laravel Auth is an internal Laravel Nova Authentication replacement to gain more control over authorizing into Laravel Nova.

[x] Rename it to laravel-auth-assets

      "@php artisan vendor:publish --tag=auth-assets --ansi --force",

[x] Rename it to laravel-auth:install

php artisan auth:install

[x] Maybe explain a little more whats before and after (What exactly needs to be removed?)

        Nova::routes()
            ->withAuthenticationRoutes()
            ->withPasswordResetRoutes();

[x] Invalid Example .env file

[x] Switch to named route

    Nova::userMenu(function (Request $request, Menu $menu) {
            return $menu
                ->append(MenuItem::externalLink('Logout', '/auth/logout'));
        });

[x] rename to --tag=laravel-auth-config

php artisan vendor:publish --tag=auth-config

[ ] How to manage translations?

[x] Logo Optional

[x] Adjust initial Logo size about -50%

[x] Social Button Gray instead of blue

[ ] How to make services optional? there needs to be a way to enable disable it.

[x] Define Redirect Path after Login + Logout

[x] Two Logout Items

 Nova::routes();

And if I remove the extra Menu Item its only one but this does not work.

Moved unchecked issues to seperate issues


cytoband renderings from real data

Also in chromosome.ts you're importing props but that's not being used. ESLint is causing the build to fail on that account and some other reasons.


[BUG] pip install -U leptonai error

Describe the bug
pip install -U leptonai error that require install the Rust package manager. not friendly
Detail:
Collecting hf-transfer (from leptonai)
Using cached https://mirrors.aliyun.com/pypi/packages/a1/a9/9ba1c6d974555246fbdfc7268005152bce6d9283333560e7916bd3642a7a/hf_transfer-0.1.5.tar.gz (21 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... error
error: subprocess-exited-with-error
× Preparing metadata (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [6 lines of output]
  Cargo, the Rust package manager, is not installed or is not on PATH.
  This package requires Rust and Cargo to compile extensions. Install it through
  the system's package manager or via https://rustup.rs/

  Checking for Rust toolchain....
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
Environment (please complete the following information):

OS: MAC
Python version: Python 3.9.6
Pip pip 24.0

Could you try pip install hf-transfer and see if it succeeds? I suspect this is due to the dependency library issue.

Traced to https://github.com/leptonai/leptonai/pull/211 - it enabled faster execution but is probably not good across the board. We'll revert it and fix this.


Enhancement: Extend strip_trailing_period to Handle Strings and Floats

Current Functionality
The strip_trailing_period function currently removes trailing periods from floats in a specified DataFrame column. However, it does not handle string values or mixed data types effectively.

Proposed Enhancement
Extend the functionality of the strip_trailing_period function to support:

Strings with Trailing Periods:

Remove trailing periods from string values like "123.".

Mixed Data Types:

Handle columns containing both numeric and string values seamlessly.

Graceful Handling of NaN:

Skip processing for NaN values, leaving them unchanged.

Robust Type Conversion:

Convert numeric strings (e.g., "123.") back to float where applicable.
Retain strings if conversion to float is not possible.

Benefits

Improved Usability:

Handles real-world scenarios where columns may contain mixed data types or string representations of numeric values.

Enhanced Robustness:

Ensures no unintended changes to NaN or non-applicable values.

Backward Compatibility:

Retains existing functionality for numeric columns while extending it for string processing.

Example
Input DataFrame
data = {
    "income": ["123.", 456.0, "789.0", "test.", "test", 100, None],
    "other": [1, 2, 3, 4, 5, 6, 7],
}
df = pd.DataFrame(data)

Resolved via https://github.com/lshpaner/eda_toolkit/pull/68


Add the option to open more than one specific pages on startup

Attempted to solve issue: #1316

@ChihweiLHBird Could you share a screenshot of the page?

@ChihweiLHBird Could you share a screenshot of the page?

Change the content from 'Open a specific page' to 'Open a specific page or pages'. Also, update the resource file.

The content has been updated. How to update the resource file? Does Visual Studio do it automatically?

Change the content from 'Open a specific page' to 'Open a specific page or pages'. Also, update the resource file.

The content has been updated. How to update the resource file? Does Visual Studio do it automatically?
Under the strings folder, edit the resources.resw file in the en-US folder and then build the project to apply to resource files for other languages.

Change the content from 'Open a specific page' to 'Open a specific page or pages'. Also, update the resource file.

The content has been updated. How to update the resource file? Does Visual Studio do it automatically?

Under the strings folder, edit the resources.resw file in the en-US folder and then build the project to apply to resource files for other languages.

My Visual Studio does not apply it to other languages when building the project...

My Visual Studio does not apply it to other languages when building the project...

You need to install the Multilingual App Toolkit Extension.

Instead of 'delete page' and 'edit page', it would be better if it says 'remove page' and 'change page location'.

Instead of 'delete page' and 'edit page', it would be better if it says 'remove page' and 'change page location'.

How about "Change Page"?

Instead of 'delete page' and 'edit page', it would be better if it says 'remove page' and 'change page location'.

How about "Change Page"?

First of all, these are not pages but locations. So change page location should be more appropriate.

Okay, and it is updated.

Make the edit button before the change buttons. It looks weird to me,

How is it now?

How is it now?

Great!

I am going to the bed. Let me know if you find some other issues. I will fix them tomorrow. (●'◡'●)

Great work! This should also help with restoring opened locations in the future.

Thank you!

Can you decrease size of Pages text and align(left) path bars with Pages text.

All done, thank you for reviewing.

And it is better to change Add page text to "Plus" button and make it little bit smaller.

Still not aligned to the left

Sorry about that. It should be okay now.

And it is better to change Add page text to "Plus" button and make it little bit smaller.

What icon is it?

And it is better to change Add page text to "Plus" button and make it little bit smaller.

What icon is it?

Glyph=""

@ChihweiLHBird

@ChihweiLHBird

Would you like left margin (of the add button) to be 10 or a higher value?

@ChihweiLHBird Look's great! Here's some final feedback:

Add text back to button
Fix alignment with pages text with controls below

It looks weird being so close to "pages:"

And it is better to change Add page text to "Plus" button and make it little bit smaller.

@XPoppyX It is just an example, we could leave it at the right side.
@duke7553 I think it is better when button text(Add pages) is not repeating text before button(Pages), but you have the final say.

I would prefer the one come with Edge.

Everyone, how is this?


Add link to https://web.dev/window-controls-overlay/

See https://github.com/GoogleChrome/web.dev/pull/5282 for the reciprocal link.

Docs Build status updates of commit 94ca2a1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps-chromium/experimental-features/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 5b6cf5c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps-chromium/experimental-features/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#sign-off and merged.


Reproduce SNAP gates paper

Reproduce Efficient cavity control with SNAP gates with  with qgrad

We can try to make this target state with S = 0, 1, 2

Qutip code to get this state
from qutip import fock
import numpy as np

from scipy.special import binom

def binomial(hilbert_size, S, N=None, mu=None):
    """
    Binomial code
    """
    if N == None:
        Nmax = int((hilbert_size)/(S+1)) - 1
        try:
            N = np.random.randint(0, Nmax)
        except:
            N = Nmax

    if mu is None:
        mu = np.random.randint(2)

    c = 1/np.sqrt(2**(N+1))

    psi = 0*fock(hilbert_size, 0)

    for m in range(N):
        psi += c*((-1)**(mu*m))*np.sqrt(binom(N+1, m))*fock(hilbert_size, (S+1)*m)

    rho = psi*psi.dag()
    return rho.unit()


WIP: Enable Multi-Arch Building and Pushing for registry viewer

What does this PR do / why we need it
Updates CI to allow for the registry viewer to be built for multiple architectures. A list of changes are explained below:

Added scripts/build_multi_arch.sh which allows for the registry viewer to be built and pushed for multiple architectures. This script also supports both podman and docker so we are free to use whatever engine is available. Currently we are using docker as part of the github actions where this script is being called.
Added github action setups for both qemu and docker buildx to ci.yaml and pushimage-next.yaml to support the building of multi-arch images.
Added a new timeout value to our main Dockerfile so that Yarn install will not timeout prematurely. When building for a different architecture the emulation can be slow and Yarn was timing out before completion.
Previously the landing-page and registry-viewer apps were calling a docker-build command as part of our CI to check the builds. I altered these commands to now build a multi-arch image.

Which issue(s) does this PR fix
fixes https://github.com/devfile/api/issues/1548
PR acceptance criteria

[ ] Unit Tests
[ ] E2E Tests
[ ] Documentation
Update the sidebar if there is a new file added or an existing filename is changed

How to test changes / Special notes to the reviewer

Closing this PR, I need to test it locally against my own fork and reopen a PR.


[BEAM-10827] Fix forward the google-cloud-core version

google-cloud-spanner==1.18.0 was released 22 hrs ago, it
falls in-between the range in setup >=1.13.0,<2, but requires
google-cloud-core>=1.4.1.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @pabloem
PTAL, thx

LGTM!

Run Python PreCommit


opt-in reactive providers

Description
We forgot to add the providers for reactive components to the #ifdef
This fixes this, and contains a driveby fix on a global namespace.
Tests
Tested in Playground and test project

changelog?


Wrap Logic functions and ccode

cc @certik, @richardotis

It fails with:
/home/travis/build/symengine/symengine.py/symengine/lib/symengine_wrapper.cpp:274:32: fatal error: symengine/infinity.h: No such file or directory

compilation terminated.

Which is probably a symengine bug.
Otherwise +1.

From the build log from travis-ci, the wrong version of symengine was checked out, see here. That was exactly why the header file "symengine/infinity.h" couldn't be found. The same error also occurred on appveyor, see here.
I think something might go wrong when git performed the merge of this patch. Could anyone take a look at this problem and consider to merge this pull request into master branch? So that more features of symengine library would be available in this python binding. :-)

Superseded by #168


Console errors in production

In production, sometimes I have errors that occur in this piece of code.
componentWillUnmount() {
    try {
        google.visualization.events.removeAllListeners(this.wrapper);
    }
    catch(err) {
        console.error("Error removing events, error : ", err);
    }
}

They appear because the component on my page rerender several times. Does anyone faced with this problem and how to fixed it. Chart works well, just annoying errors in the console.

I've the same problem 😕

Fixed in 1.4.2 !


Flaky test: "FuzzyFinder file-finder behavior toggling when the project has multiple paths shows all files for the current project and selects the first"

https://github.visualstudio.com/Atom/_build/results?buildId=43735&view=logs&jobId=6de74252-975b-522d-7a3d-227ca79ca6c1&taskId=b7ded238-9370-50c9-8500-e02f948495a4&lineStart=378&lineEnd=386&colStart=1&colEnd=1
FuzzyFinder
  file-finder behavior (ripgrep=false, scoringSystem=fast)
    toggling
      when the project has multiple paths
        it FuzzyFinder file-finder behavior (ripgrep=false, scoringSystem=fast) toggling when the project has multiple paths shows all files for the current project and selects the first
          TypeError: Cannot read property 'textContent' of null
            at it (/Users/vsts/agent/2.153.2/work/1/s/node_modules/fuzzy-finder/spec/fuzzy-finder-spec.js:172:65)

Another one:
https://github.visualstudio.com/Atom/_build/results?buildId=43222&view=logs&jobId=6de74252-975b-522d-7a3d-227ca79ca6c1&taskId=b7ded238-9370-50c9-8500-e02f948495a4&lineStart=374&lineEnd=382&colStart=1&colEnd=1