getEnumType doesn't work when SSR

The following types were generated through the protobuf-es.
  static readonly fields: FieldList = proto3.util.newFieldList(() => [
    ~
    { no: 2, name: "status", kind: "enum", T: proto3.getEnumType(Event_Status) },
    ~
  ]);

-----------------------

export enum Event_Status {
  /**
   * @generated from enum value: STATUS_UNSPECIFIED = 0;
   */
  UNSPECIFIED = 0,

  /**
   * @generated from enum value: STATUS_ACTIVE = 1;
   */
  ACTIVE = 1,

  /**
   * @generated from enum value: STATUS_SUSPENDED = 2;
   */
  SUSPENDED = 2,
}

// Retrieve enum metadata with: proto3.getEnumType(Event_Status)
proto3.util.setEnumType(Event_Status, ~, [
  { no: 0, name: "STATUS_UNSPECIFIED" },
  { no: 1, name: "STATUS_ACTIVE" },
  { no: 2, name: "STATUS_SUSPENDED" },
]);

I found that the "status" value is a string for SSR and a number for CSR.
Is this the intended behavior?
Also, how do you resolve the issue of enum type values changing with different renderings?
Thanks.

@Syunki0403, you say that "getEnumType" doesn't work. Can you clarify what exactly you mean by that? Please provide a short code example, describe what you expected to happen, and what actually happened.
Enum values are always integers in generated code. When a Protobuf message is serialized to JSON, enum values become strings. It looks like you are (inadvertently?) serializing your messages to JSON. Take a look at the documentation here to learn about the two approaches for crossing the serialization boundary.

There is a Hooks that calls the API to retrieve the event that matches the eventId as follows.
import { useSuspenseQuery } from '@connectrpc/connect-query';

import { getCurrentEvent } from '@/services/v1/event_connectquery';

import { useApiHeaders } from './use-api-headers';
import { useEventId } from './use-event-id';

export const useCurrentEvent = () => {
  const { eventId } = useEventId();
  const { headers } = useApiHeaders();
  const { data } = useSuspenseQuery(getCurrentEvent, { eventId }, { callOptions: { headers } });

  console.log(data.status);

  return { currentEvent: data };
};

The log in "data.status" at the time of page rendering shows "STATUS_ACTIVE", indicating that it is a string type.
I would like to run invalidateQueries in TanStack Query to reacquire getCurrentEvent due to data changes.
import { createConnectQueryKey } from '@connectrpc/connect-query';
import { useQueryClient } from '@tanstack/react-query';

import { getCurrentEvent } from '@/services/v1/event_connectquery';
-------
  ~
  const queryClient = useQueryClient();

  queryClient.invalidateQueries({ queryKey: createConnectQueryKey(getCurrentEvent, { eventId }) });
  ~

When the log "data.status" is reacquired, it shows 1, indicating that a numeric type is being acquired.
I call this "getEnumType doesn't work".

Thanks for the details. I'm assuming you have a schema similar to this:
syntax = "proto3";

message Event {
  Status status = 1;
  enum Status {
    STATUS_UNSPECIFIED = 0;
    STATUS_ACTIVE = 1;
    STATUS_SUSPENDED = 2;
  }
}

The generated property for the field will be status: Event_Status. This will always be a JavaScript Number at runtime.
When the message is serialized to JSON, the property will be "status": "STATUS_ACTIVE" by default, following the canonical JSON format. The zero value STATUS_UNSPECIFIED is omitted by default. There are several options for the JSON format that change what's serialized, also implemented by protobuf-es.
Regardless of the options used, parsing the message from JSON should lead to status: Event_Status being a JavaScript number again.
I don't know under which circumstances it would happen that the property is a string after parsing JSON. Here's the function that parses and enum field: https://github.com/bufbuild/protobuf-es/blob/8252042088cfbd46207690f3186b8b4c34dacbe9/packages/protobuf/src/private/json-format.ts#L578 It only returns type number.
This might be an issue in connect-query, or a gotcha in combination with server-side rendering. I cannot transfer issues between organizations. Can you re-open this on https://github.com/connectrpc/connect-query-es?


Build via GitHub Actions

The build process works (using zotero-plugin), but this build doesn't yet function correctly.

Moving the plugin code into a subdir doesn't work / requires other changes to the esbuild. Should probably use a "build" dir and package only that (src doesn't need to be zipped since it's bundled).
Edit: it seems it's actually functional, but should still be cleaned up.
The install.rdf and update.rdf should be generated via generator-zotero-plugin, but neither require('zotero-plugin/rdf') nor import 'zotero-plugin/rdf' works.
I had to restart Zotero after installing for the plugin to become functional.

Might be better to switch already from the Zotero-6-only update.rdf  to the new Zotero 6 + 7 updates.json manifest, but this isn't yet supported by zotero-plugin.

I had to restart Zotero after installing for the plugin to become functional.

Maybe because the generated install.rdf lacks <em:bootstrap>true</em:bootstrap>.

I had to restart Zotero after installing for the plugin to become functional.

Maybe because the generated install.rdf lacks <em:bootstrap>true</em:bootstrap>.

Hmm, I don't understand this:

Building tag 0.2.1, but package version is 0.2.1


Trotec PAC 2000 E a bit glitchy (Remote YX1F)

Home Assistant version: 0.111.4
SmartIR version: 1.12.0
SmartIR configuration:

smartir:
    check_updates: false

climate:
  - platform: smartir
    name: Stefan Zimmer Klima
    unique_id: stefan_zimmer_klima
    device_code: 1780
    controller_data: <IP_ADDRESS>
    temperature_sensor: sensor.temperature
    power_sensor: binary_sensor.ac_power

Describe the bug
I own a Trotec PAC 2000 E and noticed that, despite the remote being a YX1F and SmartIR offering a YX1F6 code, it's a bit glitchy. The ON command does work, however when I send an OFF command it send a "Lights Off" command instead and switches the AC unit in an unknown state since I can't see the display anymore.
I'm not sure if this is simply an incompatible remote (since the 6 at the end is missing on my remote) or if the codes are wrong (which I doubt)? If so I'm more than happy to supply a new remote code for this specific one :)
Debug log

Not applicable.

Additional context
Add any other context about the problem here.

I've now created a JSON file with my Broadlink for my YX1F remote. My model is a Trotec PAC 2000 E. Not sure if the remote code I chose is okay but it'd be nice if it could be added :)
I didn't include the "Auto" mode since it doesn't seem to be supported by SmartIR? Not sure. If it is I can easily provide a code for it.
1781.zip

Auto mode is "heat_cool" in Home Assistant.


Data module for NLP Perspectives

I've created the data module for nlpperspectives, please check it.

thank you, I've fixed it based on your recommendation.


blackjack: move FirstTurn function to tests

Fixes #1558

Some CI tests are failing but I believe this is some sort of problem with the CI - it's saying the function is declared twice, but the function was just moved to another file. Caching issue maybe?


[Backport v1.1.x] Do not return error for efi.ReadLoadOption

When encountering a boot-entry for a device-path that go-efilib is
unable to parse we should just skip that entry instead of erroring out.
Backporting this manually and minimally since cherry-picking 2e56a099b
had conflicts.
Signed-off-by: Fredrik Lönnegren<EMAIL_ADDRESS>

Codecov Report
Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Please upload report for BASE (v1.1.x@c20fa28). Learn more about missing BASE report.

Files
Patch %
Lines

pkg/efi/manager.go
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff            @@
##             v1.1.x    #2093   +/-   ##
=========================================
  Coverage          ?   75.34%
=========================================
  Files             ?       67
  Lines             ?     6814
  Branches          ?        0
=========================================
  Hits              ?     5134
  Misses            ?     1310
  Partials          ?      370

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


Implemented: 'performFind' api to fetch picklists with the help of facilityIds ( #1vy48ym )

Closing. Will be done later.


[DUOS-1452][risk=no] Dataset Catalog Refactor

Addresses
https://broadworkbench.atlassian.net/browse/DUOS-1452

Converted from class component to function
Lots of cleanup around item selection
Fixes select-all datasets -> DAR creation bug
Updates styling so inactive datasets are non-selectable
Prevent selection of any inactive datasets
Removed many unnecessary variables and state checks

Have you read Terra's Contributing Guide lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

@JVThomas - great feedback all around, will implement!


Support OpenTTD 14 admin interface changes (protocol 3)

This PR better handles cases where data is unexpectedly missing from the incoming packet buffer, most notably in OpenTTD 14 where the company shares have been removed from company update packets.

(please let me know if this breaks your implementation!)


Checkdocs CI looks like has a internal failure that is not marked as an error.

It looks like the CI loop is reporting that checkdocs is successful currently..
If you look at the output from docnits - it does not look so good.
Complex regular subexpression recursion limit (65534) exceeded at ./util/find-doc-nits line 1004.
This seems to be related to the rather large migration_guide.pod.

So, it seems that this particular error doesn't cause an exception (no die at least), that's why it's undetected.  From debugging printing I just did, it appears we still get a list of links, only surprisingly small.
I'm currently running doc-nits with added regexp debugging enabled around that code section to find out what's really going on.
It may be as simple as having to run this regexp in a while loop and manually push $1 to @lists...

Fix in #15450

This was already reported in #15407: I marked that as duplicate even if it predates this issue, just because most of the relevant discussion happened here.


Feature request: Add Energy Tax to total price

Great ro have this, especially as a component in home-assistant.
Would it be possible to also add the energy tax to the tariff displayed?
Now there is the incl. / excl. BTW option. In NL we also need to pay tax (+vat over the tax) on top of these rates:
for Gas 2023 - € 0,48980 ex BTW = €0,592658 incl BTW
for Electricity - € 0,12599 ex BTW = €0,1524479 incl BTW
so yes, about half the net price is in additional taxes..
Source for taxes: https://www.belastingdienst.nl/wps/wcm/connect/bldcontentnl/belastingdienst/zakelijk/overige_belastingen/belastingen_op_milieugrondslag/tarieven_milieubelastingen/tabellen_tarieven_milieubelastingen?projectid=6750bae7-383b-4c97-bc7a-802790bd1110

Great ro have this, especially as a component in home-assistant.
Would it be possible to also add the energy tax to the tariff displayed? Now there is the incl. / excl. BTW option. In NL we also need to pay tax (+vat over the tax) on top of these rates:
for Gas 2023 - € 0,48980 ex BTW = €0,592658 incl BTW
for Electricity - € 0,12599 ex BTW = €0,1524479 incl BTW
so yes, about half the net price is in additional taxes..
Source for taxes: https://www.belastingdienst.nl/wps/wcm/connect/bldcontentnl/belastingdienst/zakelijk/overige_belastingen/belastingen_op_milieugrondslag/tarieven_milieubelastingen/tabellen_tarieven_milieubelastingen?projectid=6750bae7-383b-4c97-bc7a-802790bd1110

https://student-techlife.com/2023/01/11/energyzero-market-prices-with-new-integration-in-home-assistant/?fbclid=IwAR1DCcvuUQaHfdz3axBNrkgk2fOyAsinzBXRb_UjwthEGoj4claYt7Qq9us

This is not the right place for feature requests for the home assistant integration, please check the community forum for this. This repository is the underlying package that the integration uses to retrieve the data.
But to answer your question, see my response in a forum topic about the integration.

Very helpful, thanks! Let me close this one.

Not the right place, links to guidance were given


Export package without its dependencies

It would be nice to export a package without any of its dependencies. For example, I would like to export the headers and built libraries for libpng, but I would like to have all the dependencies (zlib headers and binaries excluded).

I just discovered vcpkg owns. This solves my problems of identifying the files that belong to each package. Sorry for the confusion.

What solved the problem?
I have the same issue where we store bin, lib and include per library separately in our own repository and we have our own CMake system that fetches these libraries. But doing vcpkg export <lib> --raw exports also the dependency libraries/includes/pdbs so I can't just copy the directories directly.

Turns out there's list of files installed by each package in the vcpkg-export\installed\vcpkg\info\package-triplet.list file. So I made a batch file that copies files referenced here to a separate directory.
Here is the batch file (strip_export.bat) located in vcpkg root directory (where vcpkg.exe is):
@echo off
set export_directory=%~f1
set export_library=%2
set full_file=vcpkg/info/%export_library%.list

pushd %export_directory%\installed
mkdir %export_directory%\stripped
setlocal enabledelayedexpansion
for /F "tokens=*" %%A in (vcpkg/info/%export_library%.list) do (
    set "file_name=%%~dfA"
    set "file_path=%%~dpA"
    set "relative_file_path=!file_path:%CD%\=!"
    xcopy /Y !file_name! %export_directory%\stripped\!relative_file_path!
)
endlocal
popd

Usage: strip_export.bat <export_directory> <export_list_name>
For example I have exported GDAL with vcpkg export gdal --triplet=x64-windows-gdal --raw command and renamed the export directory to vcpkg-export-gdal-2.4.1-13. I would run the strip batch file like this: strip_export.bat vcpkg-export-gdal-2.4.1-13 gdal_2.4.1_x64-windows-gdal
This results in vcpkg-export-gdal-2.4.1-13\stripped directory that contains only the files that come from GDAL package but not the dependencies.

Just curious what's the use case for this feature? Without the dependencies the exported libs wouldn't work.

It would prevent multiple versions of the same dependency in the dependency graph, for a project that would install different exports.
Consider libpng depending on zlib. If I install another export from another lib that depends on zlib, or an export of zlib by itself, they could conflict with one another.
Without this possibility, the only safe way to handle exports a a consumer is to consume only one a time in a project.


Add detailed information to the OSG mkdocs repo 404 pages (SOFTWARE 3290)

This PR is for ticket 3290. I added the same 404 pages from technology.

@brianhlin opps my bad I did not realize that was there and cp'ed everything. It should be good now.

If the CI passes, go ahead and merge. Once it's been pushed, please verify that it works via https://opensciencegrid.org/docs/foo


chore: Update contributing to suggest live branch

May be worth getting rid of the master branch (and the other stale ones) if you're not using them anymore

Docs Build status updates of commit 5105aa2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

CONTRIBUTING.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[Question] Time for pretrain and finetune

Question
How long does it take for you to pre-train and fine-tune the 7B LLaVA model on 8×910B?

@BaleMa Hi, I use about 1 hour for stage 1 and 7 hours for stage 2.


I don't see as much information in Python that in the GUI

Hello,
Thanks for the package 👍
I don't see as much information in Python that in the GUI, even in administrator mode:
PyHardwareMonitor-main> python.exe .\test\TestHardwareMonitor.py
Hardware: AMD Ryzen Threadripper 1950X 1
        Sensor: CPU Core #1, value: 73.18607330322266
[...]
        Sensor: CPU Core #32, value: 11.351579666137695
        Sensor: CPU Total, value: 60.929725646972656
        Sensor: CPU Core Max, value: 99.99630737304688
        Sensor: Package, value:<PHONE_NUMBER>9003906
        Sensor: Core #1, value: 3693.0537109375
        Sensor: Core #1, value: 37.0
        Sensor: Core #1 (SMU), value: 8.992218971252441
        Sensor: Core #1 VID, value: 1.2312500476837158
[...]
        Sensor: Core #16, value: 3693.0537109375
        Sensor: Core #16, value: 37.0
        Sensor: Core #16 (SMU), value: 4.2909746170043945
        Sensor: Core #16 VID, value: 1.2374999523162842
        Sensor: Core (Tctl), value: 89.12500762939453
        Sensor: Core (Tdie), value: 62.12500762939453
        Sensor: Core (SVI2 TFN), value: 1.0499999523162842
        Sensor: Bus Speed, value: 99.81226348876953

The power is not there.
What can I do to get it ?

Did you use the wheel from the release page or the version from the main branch?
I never ended up updating the release as I wanted to correct the package version to have it correlate directly to the LibreHardwareMonitorLib version.
Since the package was never deployed for its original intent I may have slacked off in correcting that and making a new release.

I retract my version question, you obviously had the most recent version with the test script.
With minor modifications to the test script to include the sensor units in the print
+from HardwareMonitor.Util import SensorValueToString

@@ -42,8 +43,8 @@ for hardware in computer.Hardware:
     for subhardware  in hardware.SubHardware:
         print(f"\tSubhardware: {subhardware.Name}")
         for sensor in subhardware.Sensors:
-            print(f"\t\tSensor: {sensor.Name}, value: {sensor.Value}")
+            print(f"\t\tSensor: {sensor.Name}, value: {SensorValueToString(sensor.Value, sensor.SensorType)}")
     for sensor in hardware.Sensors:
-        print(f"\tSensor: {sensor.Name}, value: {sensor.Value}")
+        print(f"\tSensor: {sensor.Name}, value: {SensorValueToString(sensor.Value, sensor.SensorType)}")

When run in an admin-terminal I see the following
> py .\TestHardwareMonitor.py
Hardware: Intel Core i5-8265U
        Sensor: CPU Core #1 Thread #1, value: 19.6 %
        Sensor: CPU Core #1 Thread #2, value: 6.7 %
        Sensor: CPU Core #2 Thread #1, value: 8.9 %
        Sensor: CPU Core #2 Thread #2, value: 11.1 %
        Sensor: CPU Core #3 Thread #1, value: 26.7 %
        Sensor: CPU Core #3 Thread #2, value: 31.1 %
        Sensor: CPU Core #4 Thread #1, value: 10.9 %
        Sensor: CPU Core #4 Thread #2, value: 37.0 %
        Sensor: CPU Total, value: 19.0 %
        Sensor: CPU Core #1, value: 66.0 °C
        Sensor: CPU Core #2, value: 66.0 °C
        Sensor: CPU Core #3, value: 70.0 °C
        Sensor: CPU Core #4, value: 72.0 °C
        Sensor: CPU Package, value: 72.0 °C
        Sensor: CPU Core #1 Distance to TjMax, value: 34.0 °C
        Sensor: CPU Core #2 Distance to TjMax, value: 34.0 °C
        Sensor: CPU Core #3 Distance to TjMax, value: 30.0 °C
        Sensor: CPU Core #4 Distance to TjMax, value: 28.0 °C
        Sensor: Core Max, value: 72.0 °C
        Sensor: Core Average, value: 68.5 °C
        Sensor: CPU Core #1, value: 3600.0 MHz
        Sensor: CPU Core #2, value: 3600.0 MHz
        Sensor: CPU Core #3, value: 3500.0 MHz
        Sensor: CPU Core #4, value: 3400.0 MHz
        Sensor: CPU Package, value: 21.9 W
        Sensor: CPU Cores, value: 19.5 W
        Sensor: CPU Memory, value: 1.4 W
        Sensor: CPU Core, value: 1.053 V
        Sensor: CPU Core #1, value: 1.033 V
        Sensor: CPU Core #2, value: 1.007 V
        Sensor: CPU Core #3, value: 1.027 V
        Sensor: CPU Core #4, value: 0.999 V
        Sensor: Bus Speed, value: 100.0 MHz
...

For my platform the Wattages are displayed. Since you were able to read the power values using the GUI it should be available here as well.
Would it be possible to confirm that the script was indeed run as administrator and if the sensors are definitely not there.
With the sensor type or unit not being printed in the test script I would have totally missed them in the output myself.
Thanks.

I am closing this issue since there has been no further indication presented of the issue persisting at this time.


Feature Request: GitLab compatibility.

Is it possible to make this style compatible with GitLab as well?
That would be awesome!

Sorry for the delay. I haven't tried GitLab yet. But will definitely make a GitLab theme if i have time.


Translate the app to Mandarin Chinese

Translate the app to Mandarin Chinese by creating a zh.json language file based on the existing translations, and registering it in the i18n.js.

Happy to take this on.

Hi @alexismo, awesome, thanks!


xgcm for computing gradients of vectors

Dears all,
I have been computed gradients of velocity using outputs from  ECCO V4. Previously, I carefully verified the formalism for computing curl and divergence using xgcm and face connections. However, I found an error in the computation of velocity gradients at the boundaries between rotated and non-rotated tiles. Despite repeated checks, particularly of the face connections, I was unable to resolve this specific issue. I would greatly appreciate any guidance or suggestions you might have on how to address this problem. I share and snapshot computed for the curl of tau to clarify regions where the computation produce large values.
Best regards

Hi @fecampos thanks for sharing.
Could you please share the full code and data you used to produce this figure? Without that it's impossible for us to help you.

Hi @rabernat,
Thank for your early response. I share my code:
`import xarray as xr
import ecco_v4_py as ecco
import matplotlib.pyplot as plt
import numpy as np
from xmitgcm.llcreader.llcmodel import faces_dataset_to_latlon
import warnings, xgcm
warnings.filterwarnings("ignore")
from dask.distributed import Client, LocalCluster
cluster = LocalCluster(n_workers=20, threads_per_worker=4)
client = Client(cluster)
ds = xr.open_mfdataset('/home/jovyan/efs_ecco/bdarby/data/ECCO_V4r4_PODAAC/stress/OCEAN_AND_ICE_SURFACE_STRESS_mon_mean_2017-01_ECCO_V4r4_native_llc0090.nc')
dg = xr.open_mfdataset('/home/jovyan/efs_ecco/bdarby/data/ECCO_V4r4_PODAAC/grid/*.nc')
face_connections = {'tile':
{0: {'X':  ((12, 'Y', False), (3, 'X', False)),
'Y':  (None,             (1, 'Y', False))},
1: {'X':  ((11, 'Y', False), (4, 'X', False)),
'Y':  ((0, 'Y', False),  (2, 'Y', False))},
2: {'X':  ((10, 'Y', False), (5, 'X', False)),
'Y':  ((1, 'Y', False),  (6, 'X', False))},
3: {'X':  ((0, 'X', False),  (9, 'Y', False)),
'Y':  (None,             (4, 'Y', False))},
4: {'X':  ((1, 'X', False),  (8, 'Y', False)),
'Y':  ((3, 'Y', False),  (5, 'Y', False))},
5: {'X':  ((2, 'X', False),  (7, 'Y', False)),
'Y':  ((4, 'Y', False),  (6, 'Y', False))},
6: {'X':  ((2, 'Y', False),  (7, 'X', False)),
'Y':  ((5, 'Y', False),  (10, 'X', False))},
7: {'X':  ((6, 'X', False),  (8, 'X', False)),
'Y':  ((5, 'X', False),  (10, 'Y', False))},
8: {'X':  ((7, 'X', False),  (9, 'X', False)),
'Y':  ((4, 'X', False),  (11, 'Y', False))},
9: {'X':  ((8, 'X', False),  None),
'Y':  ((3, 'X', False),  (12, 'Y', False))},
10: {'X': ((6, 'Y', False),  (11, 'X', False)),
'Y': ((7, 'Y', False),  (2, 'X', False))},
11: {'X': ((10, 'X', False), (12, 'X', False)),
'Y': ((8, 'Y', False),  (1, 'X', False))},
12: {'X': ((11, 'X', False), None),
'Y': ((9, 'Y', False),  (0, 'X', False))}}}
xgcm_grid = xgcm.Grid(ds, periodic=False, face_connections=face_connections)
taux, tauy = ds.oceTAUX, ds.oceTAUY
zeta = (-xgcm_grid.diff(taux * dg.dxC, 'Y') + xgcm_grid.diff(tauy * dg.dyC, 'X'))/dg.rAz
plt.figure(figsize=(20,8), dpi= 90)
curr_obj = ecco.plot_proj_to_latlon_grid(zeta.XG,zeta.YG,
zeta.isel(time=0).squeeze(),
show_colorbar=True,
show_land=True,projection_type='robin',
user_lon_0=0,plot_type='pcolormesh',
cmin=-1e-6,cmax=1e-6,
cmap='seismic')
plt.title(r"$\nabla \times \tau$",size=30)`
I share the data via dropbox:

grid: https://www.dropbox.com/scl/fi/mwhubukyfi85qaeur7ks8/GRID_GEOMETRY_ECCO_V4r4_native_llc0090.nc?rlkey=ran6n21b668hi2xf1wxs7m80y&st=ucjad4fc&dl=0
data (tau): https://www.dropbox.com/scl/fi/xil8fv1gmdvbpaf4cno8k/OCEAN_AND_ICE_SURFACE_STRESS_mon_mean_2017-01_ECCO_V4r4_native_llc0090.nc?rlkey=fxw5k0f8ekx1l0nr77v4dn798&st=b7f15a1q&dl=0
Thank you so much for your help

Best regards

@fecampos The issue you found may be related to the fact that the wind stress and ocean stress fields may be provided at 'tracer' or 'c' points on the c-grid. In constrast, velocity and lateral flux fields are provided on the 'u' and 'v' points. Consequently, to calculate curl you may need to take some additional steps. A tutorial that works for ECCO V4r4 fields is here:
https://ecco-v4-python-tutorial.readthedocs.io/ECCO_v4_Gradient_calc_on_native_grid.html#
The recipe is basically

rotate the u_x and v_y vectors from the orthogonal ‘x-y’ basis to the orthogonal lambda-phi basis, the basis of parallels and meridians to make the u_lambda and v_phi vectors. these fields are originally at the 'c' points.
calculate the gradients of ‘u_lambda’ and ‘v_phi’ with respect to the model’s ‘x’ direction to make du_lambda_dx and dv_phi_dx vectors.  these fields are now at the 'u' points.
calculate the gradients of ‘u_lambda’ and ‘v_phi’ with respect to the model’s ‘y’ direction to make du_lambda_dy and dv_phi_dy vectors.  these fields are now at the 'v' points.
interpolate the four gradients to the tracer points. after interpolation these fields are all on the 'c' points
rotate the four gradients of the zonal flow with respect to ‘x’ and ‘y’ to the zonal and meridional directions to make the du_lambda_dphi and du_lambda_dlambda vectors. after rotation the fields are still on the 'c' points
rotate the gradients of the meridional flow with respect to ‘x’ and ‘y’ to the zonal and meridional directions to make the dv_phi_dphi and dv_phi_dlambda vectors. after rotation the fields are still on the 'c' points

the wind or ocean stress curl is then calculated via
curl = dv_phi_dlambda - du_lambda_dphi
the wind or ocean stress curl fields are at the 'c' points. keep in mind that curl fields from horizontal velocity and flux fields are at 'zeta' points, so you would need to do one final interpolation to get curl fields from the 'c' points to the 'zeta' points.
Maybe @rabernat knows a simpler way!
-Ian


typeof(GeofenceService): GeofenceService not found, should I use GeofenceLocationService instead?

AppContext.StopService(new Intent(AppContext, typeof(GeofenceService)));
this raises error on GeofenceService not found.
Should I use GeofenceLocationService instead? if so it would be great to update the sample project

Is GeofenceService look for it on the PCL project on Helpers folder.


Delegation history and list for DReps

Why
So Dreps can see and understand who's delegating to them and how much Voting Power is involved
What
Delegators need to know several things about their delegees:

Who's delegated to them
how much voting power has been delegated and by whom
What is the date of the delegation
If an 'un-delegation' event happens

How
This would work as a table in the delegator's detail card. It should be available to the public.

This is in issue: https://github.com/IntersectMBO/govtool/issues/749

This is not completed. Re-opening


Fix logo for terminals that support only 256 colors.

coverage: 100.0%. remained the same
when pulling fb2f7a56dbc818ed978c53e92f9d293f539a223d on fix-logo-256-colors
into 415d42b0970dc7089c492d15000eed4a384557cd on master.


Fix throughput calculation in image-to-text example

What does this PR do?

The throughput calculation in the image-to-text example is currently misleading as it assumes that args.max_new_tokens will always be generated. However, this is not necessarily the case when setting ignore_eos to True.
This PR fixes that and adds a new ignore_eos argument.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


During build: "there is no package called ‘abind’"

There are also a number of dependencies that are not documented in the readme (e.g., R.oo, R.methodsS3...).
So, probably a review of requirements would be helpful to casual users like me - Thanks folks!

@ebridge2 maybe you can easily fix that?


DNN-7020

DNN-7020 - The site settings control now provides an option to include attachments in outgoing messages. The default is off.

I have manually merged this


Have you tought about supporting Piccolo?

Another interesting Python ORM is https://github.com/piccolo-orm/piccolo/
Have you thought about adding support for it?
This is how they currently suggest using pgvector, through raw SQL:
https://github.com/piccolo-orm/piccolo/discussions/988 this points to the other URL ->
https://github.com/piccolo-orm/piccolo/discussions/876

Hi @jackbravo, thanks for sharing. I don't think it's common enough to add/support right now, but someone could create a separate package for it (looks like it supports custom types).


Bulk operation exception changes

Description
Spark connector customer have encountered a challenge related to bulk operations, where identifying the specific failed operations proves difficult. This pull request addresses this issue by enhancing the Exception body to include pertinent details such as the itemId and partitionKeyValue for the failed operation.
By incorporating this improvement, users gain valuable insights into the nature and scope of the failures, enabling more effective troubleshooting and resolution efforts. With access to detailed information on the failed operations, users can now focus their attention on rectifying specific issues, potentially leveraging tools like a Dead Letter Queue to streamline the resolution process.

BulkOperationFailedException

exception = {
            "ClassName": "BulkOperationFailedException",
            "userAgent": "azsdk-java-cosmos/4.55.0-beta.1 MacOSX/10.16 JRE/11.0.10",
            "statusCode": 409,
            "resourceAddress": null,
            "innerErrorMessage": "All retries exhausted for 'UPSERT' bulk operation - statusCode=[409:0] itemId=[d4f258eb-2f20-49a3-a18d-f2592ce43b78], partitionKeyValue=[[\"8489b200-9ac4-4a75-8c5c-4c05dad62b25\"]]",
            "causeInfo": null,
            "responseHeaders": "{x-ms-substatus=0}",
            "itemId": "d4f258eb-2f20-49a3-a18d-f2592ce43b78",
            "partitionKey": "[\"8489b200-9ac4-4a75-8c5c-4c05dad62b25\"]"
        }

All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-cosmos


i found an error on FinRL_multiple_stock_trading.ipynb

this problem is data processing, not alignment，but the state must have an alignment data for a day.
so original file can bring an error if you use ticker_list = config.CSI_300_TICKER
Stock Dimension: 298, State Space: 1789
Traceback (most recent call last):
  File "/home/wac/PycharmProjects/drl_crypto_coin/train/multi_train_drl.py", line 105, in <module>
    env_train, _ = e_train_gym.get_sb_env()
  File "/home/wac/.local/lib/python3.6/site-packages/finrl/env/env_stocktrading.py", line 286, in get_sb_env
    obs = e.reset()
  File "/home/wac/.local/lib/python3.6/site-packages/stable_baselines3/common/vec_env/dummy_vec_env.py", line 62, in reset
    self._save_obs(env_idx, obs)
  File "/home/wac/.local/lib/python3.6/site-packages/stable_baselines3/common/vec_env/dummy_vec_env.py", line 92, in _save_obs
    self.buf_obs[key][env_idx] = obs
ValueError: cannot copy sequence with size 1734 to array axis with dimension 1789

this request fix this error, please merged
pull request:
https://github.com/AI4Finance-LLC/FinRL-Library/pull/46

this problem is data processing, not alignment，but the state must have an alignment data for a day.
so original file can bring an error if you use ticker_list = config.CSI_300_TICKER
Stock Dimension: 298, State Space: 1789
Traceback (most recent call last):
  File "/home/wac/PycharmProjects/drl_crypto_coin/train/multi_train_drl.py", line 105, in <module>
    env_train, _ = e_train_gym.get_sb_env()
  File "/home/wac/.local/lib/python3.6/site-packages/finrl/env/env_stocktrading.py", line 286, in get_sb_env
    obs = e.reset()
  File "/home/wac/.local/lib/python3.6/site-packages/stable_baselines3/common/vec_env/dummy_vec_env.py", line 62, in reset
    self._save_obs(env_idx, obs)
  File "/home/wac/.local/lib/python3.6/site-packages/stable_baselines3/common/vec_env/dummy_vec_env.py", line 92, in _save_obs
    self.buf_obs[key][env_idx] = obs
ValueError: cannot copy sequence with size 1734 to array axis with dimension 1789

this request fix this error, please merged
pull request:
#46

Great, you did the Time Series Reshaping. Does it have to be after the feature engineer stage? I switched it before the feature engineer stage, found that feature engineer got some errors.

this problem is data processing, not alignment，but the state must have an alignment data for a day.
so original file can bring an error if you use ticker_list = config.CSI_300_TICKER
Stock Dimension: 298, State Space: 1789
Traceback (most recent call last):
  File "/home/wac/PycharmProjects/drl_crypto_coin/train/multi_train_drl.py", line 105, in <module>
    env_train, _ = e_train_gym.get_sb_env()
  File "/home/wac/.local/lib/python3.6/site-packages/finrl/env/env_stocktrading.py", line 286, in get_sb_env
    obs = e.reset()
  File "/home/wac/.local/lib/python3.6/site-packages/stable_baselines3/common/vec_env/dummy_vec_env.py", line 62, in reset
    self._save_obs(env_idx, obs)
  File "/home/wac/.local/lib/python3.6/site-packages/stable_baselines3/common/vec_env/dummy_vec_env.py", line 92, in _save_obs
    self.buf_obs[key][env_idx] = obs
ValueError: cannot copy sequence with size 1734 to array axis with dimension 1789

this request fix this error, please merged
pull request:
#46

Great, you did the Time Series Reshaping. Does it have to be after the feature engineer stage? I switched it before the feature engineer stage, found that feature engineer got some errors.

add the intersection method to preprocess data, The purpose is to increase training stability

add the intersection method to preprocess data, The purpose is to increase training stability


Add Pie chart for signal numbers clustered by criticality

Summary of changes
Add Pie chart for signal numbers clustered by criticality
Context and reason for change

Use Accordion for better utilization of space, compactness, and prevent overcrowding of visual data.
Accordion will permit addition of Pie charts to the popup in future without having to change the page styles or layout. Thus, it should be future-proof.
Add expansion icon to Accordion to indicate when accordion is expanded. Better UX.
Add custom colors for different Pie Charts.
Add tests for rendering the new Pie Chart component.
Add function that provides the data needed by the new Pie chart.
Add tests for the new data-providing function.
Extract out a function to help count number of critical signals.
Fix existing test by adding ResizeObserver package. See https://github.com/opossum-tool/OpossumUI/pull/1066#issuecomment-1275748639 and https://github.com/opossum-tool/OpossumUI/pull/1066#issuecomment-1275815744

Fix: #945
Signed-off-by: someshkhandelia<EMAIL_ADDRESS>

Previously, the popup looked like this:

Now, the popup looks like this:

Interesting to note, that the following test is failing:

with the error:

However, commenting out the Accordion components in statistics popup, makes the test pass:

Checking https://github.com/maslianok/react-resize-detector/issues/217 for https://github.com/opossum-tool/OpossumUI/pull/1066#issuecomment-1275748639

Some design points:

"Other" color is not really good on this background
Critical signals pie chart has no highly critical signals, but the legend has.
Both pie charts should have equal margin on left
The both pie charts (especially critical signals) have too big margins on top and bottom.
Pie charts title bars have different height when accordeon is opened and closed. Probably it should be equal to table headers.
Accordeons are too wide on wide screens
It will unlikely be the case, but when there are no signals Critical signals pie chart (empty) still exists (see picture 2)

Picture 2

Regarding https://github.com/opossum-tool/OpossumUI/pull/1066#issuecomment-1276262826, @vasily-pozdnyakov can you please tell which input files have you used for generating the two pictures?

I would keep legends on right side, because it is more readable if we have long license names.
Red and orange colors on Most Frequent Licenses can be confusing as we use the colors for criticality as well.
Critical signals should be colored consistently (I guess high = orange, medium=lighOrange, none=darkBlue) - see Critical Licenses table.

Failing test case - solved
#1066 (comment) was solved by adding the ResizeObserver package(https://yarnpkg.com/package/resize-observer-polyfill): yarn add resize-observer-polyfill
and then including the following line in setupTests.ts:

From https://developer.mozilla.org/en-US/docs/Glossary/Polyfill?retiredLocale=de : A polyfill is a piece of code (usually JavaScript on the Web) used to provide modern functionality on older browsers that do not natively support it.
We are not in this case (we are using a quite recent version of chromium).
Please, check mockResizeObserver in src/Frontend/test-helpers/popup-test-helpers.ts for a different (already existing solution).


pod issue

Invalid Podfile file: uninitialized constant Pod::Podfile::RVCalendarWeekView.

it is working fine on me.. any idea on how to fix this?

I am just following step for installation, pod RVCalendarWeekView then above error comes, Please, let me know.

I think this is your podfile that fails

I am using same pod file which is there in same folder.

You need to create your own podfile in your own project, you don't have to use the podfile used in this project

I have used same file replacing the new pod install text, but still throws
error. one question if it is grid still it can have one column ?
On Tue, Nov 8, 2016 at 9:47 PM, Jordi Puigdellívol<EMAIL_ADDRESS>
wrote:

You need to create your own podfile in your own project, you don't have to
use the podfile used in this project
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BadChoice/RVCalendarWeekView/issues/19#issuecomment-259181790,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADBzyhcAwapvoQMEvFyppfNKh-oNsqAqks5q8KCFgaJpZM4Ksjba
.

--
Thanks & Regards
Sanoj Kashyap
iOS Developer
Skype - sanoj.kashyap
Mob - +91-9590399725

after creating new file it's giving "Invalid Podfile file: uninitialized constant Pod::Podfile::RVCalendarWeekView.
" error.

Could you fi it?

I also faced the same issue.
Its because I had given
pod RVCalendarWeekView instead of  pod 'RVCalendarWeekView'
now I changed it into pod 'RVCalendarWeekView' and its working fine.
@BadChoice  This happened you have given pod RVCalendarWeekView in your documentation. It will better if you change it so that others can simply copy paste and use it

Thanks, readme updated ;)

Welcome 👍


Updating padding for active and none-active structure tabs

Helps them breathe

 UX Issue: Is the text in tabs in structure a bit cramped?


[Bug]: Uncaught TypeError: a is undefined

Expected Behavior
After clicking Export, give me the view to generate the pdf.
Current Behavior
After clicking export with the remastered mapping and the remastered pdf i get the issue: Uncaught TypeError: a is undefined
Steps to Reproduce

Set mapping to current rotgrind remastered
choose pdf template of rotgrind remastered
click on export

Context
No response
Version
0.24.6
Foundry VTT Version
11.315
Operating System
Linux
Browser / App
Firefox
Game System
pf2e
Modules Disabled

[ ] I can confirm that disabling all of my other modules doesn't fix this issue.

Fixed the bug with changing the following line 468 in the mapping file:
FROM
{ "pdf": "languages", "foundry": actordata.system.traits.languages.value.concat([actordata.system.traits.languages.custom]).filter(function(a) {return a.trim() != ''}).join(", ") },
TO
{ "pdf": "languages", "foundry": actordata.system.traits.languages.value.concat([actordata.system.traits.languages.custom]).filter(function(a) {return a !== undefined ? a.trim() : ''}).join(", ") },

After applying this fix you will get to the issue https://github.com/arcanistzed/pdf-sheet/pull/121 and can be fixed like described there.


Support ESL Events

Hello,
I like your crate and I'm interested in the event listener feature. Do you plan to work on it soon or can I submit a merge request with some API design suggestions? I have some ideas that might improve the crate.
Thanks for your time and work.

Thank you for using this crate!
I plan to work on it, but not in the near future. Your suggestions and PRs are welcome.

@KaranGauswami I see that you are moving things around a little bit in the crate. I will submit a PR once you finalize and merge the draft.


Non-transactional Java-based migrations support

Until now JdbcMigration and SpringJdbcMigration implementations could only execute within a transaction. Flyway 5.2.0 introduces a new JavaMigration type which supersedes both existing ones and additionally can be configured to optionally execute without a transaction.

@axelfontaine I have been using Flyway for over 2 years now and the big issue I have is that I have lots of existing migrations from 5.x. If I switch over to 6.x will it fix up those existing migrations in my database or will I really screw up my production database?
I now find myself urgently needing to upgrade from 5.x to 6.x as a result of this.. https://github.com/flyway/flyway/issues/2326
I need to update a query that is a production issue for me and I am in a bit of a bind, this breaking change from 5.x to 6.x is not what I need right now.
I really do not understand why you got rid of the existing interface SpringJdbcMigration??
You could have kept it for those existing users and ensured that whatever migrations they had would be preserved and handled correctly the new way. An interface by definition should never be changed. The other issue is that, given that SpringJdbcMigration was an interface, I (and probably many more people), would have the opportunity to boiler plate their migration code, for example
class VX__Migration extends BoilerPlate implements SpringJdbcMigration {
    void migrate(JdbcTemplate template) throws Exception {
         super.someBoilerPlateCode(template);
         ..
         Specific code
    }
}

My existing code follows this pattern a lot..
My question is... If I refactor, like this:
abstract class BoilerPlate extends BaseJavaMigration {

    abstract void migrate(JdbcTempate template);

   @Override
   public void migrate(Context context) throws Exception {
     JdbcTemplate template = new JdbcTemplate(new SingleConnectionDataSource(context.getConnection(), true));
     this.migrate(template);
   }
}

Will this work for all the existing migrations that existing in my database from 5.x. Does 6.x recognize existing 5.x migrations and upgrade their records in the database accordingly...

@geoffreywiseman do you have any insights on how to go about upgrading from SpringJdbcMigration ?


complete callback called before the animation is over

Hello,
I have an issue when a try to execute a method in the  complete callback (on Chrome). It seems the callback is called before the animation is over. Here is an example:  https://jsfiddle.net/tvh7qc37/7/. If you open the console, you will see that the code is stopped by the debugger before the animation is over.
thank you for your help !

I'm not entirely sure how debugger works as I never rely on it, but I'm pretty sure the complete callback is always called after the animation. You can try replacing your debugger statement with something like console.log(document.querySelector(".foreground").style.transform) and you'll always get scale(0).


Added PLL7 (USB2 PLL) Support

PLL7 is required to use the secondary USB controller. PLL3 is only used for the primary USB controller.
I briefly read through the i.MX RT1060 specification and noticed no meaningful differences between the USB1 and USB2 PLLs. Assuming I haven't overseen anything, PLL7 should have an identical implementation as PLL3, with the only variation being the USB controller number in the register names.

PLL7 doesn't exist on the smaller i.MX RT MCUs, like the 1010 and 1020. That's why CI is failing.
Consider exporting PLL7 only when building for the 1060 MCUs. It might look similar to how we include and export PLL1 when building for the 1060.

My mistake. Thanks for the pointers. Hopefully it's a bit better now.


CA-202 - Replace S3 deprecated arguments

Replaces deprecated S3 arguments with resource blocks.
Replaces deprecated IAM policy document argument names.

:tada: This PR is included in version 1.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Use <CODE_REVIEW></CODE_REVIEW> tags to enclose source code

The system prompt should mention the <CODE_REVIEW></CODE_REVIEW> tags which enclose the inserted source files.
The prompt should mention the following:
Before proposing or writing any code, conduct a thorough review of the existing code, enclosed within <CODE_REVIEW></CODE_REVIEW> tags.

Done


One of the images is 9MB large

The "header-scientist.jpg" image is over 9MB large, and takes several seconds to download.  It should be scaled to something reasonable
https://github.com/OA-PASS/pass-ui-static/blob/master/img/header-scientist.jpg

Addressed in PR #16


Discussion: how to design addition

There has been multiple ideas / discussions about handling addition properly and I wanted to document them here as decisions are made.
Handling missing dimensions
The current plan is to require the user to call add_dim() when increasing the dimensions of an expression via addition. Without add_dim() an error will be thrown.
Example
>>> import polars as pl
>>> food_cost = pl.DataFrame({
        "month": ["Jan", "Feb", "Mar", "Jan", "Feb", "Mar"],
        "year":     [2023, 2023, 2023, 2024, 2024, 2024],
        "monthly_cost": [1000, 1010, 1020, 1030, 1040, 1050]
    }).to_expr()
>>> rent = pl.DataFrame({
        "year": [2023, 2024],
        "monthly_cost": [1000, 1100]
    }).to_expr()

# This should throw an error
>>> food_cost + rent

# This shouldn't
>>> food_cost + rent.add_dim("month")

Handling unmatched indices
What happens if indices exist in one side of the some but not the other? For example, in the previous example, if one of the two tables contained extra years.
I propose the default is an error is thrown (#7). However there should be two alternatives.
.keep_unmatched(): If there are unmatched rows (rows where the index exists in only one of the two tables), these are kept.
.drop_unmatched(): Only indices that are in common between both dataframes are kept.
Example
>>> food_cost = pl.DataFrame({
        "month": ["Jan", "Feb", "Mar", "Jan", "Feb", "Mar", "Jan"],
        "year":     [2023, 2023, 2023, 2024, 2024, 2024, 2025],
        "monthly_cost": [1000, 1010, 1020, 1030, 1040, 1050, 1060]
    }).to_expr()
>>> rent = pl.DataFrame({
        "year": [2023, 2024],
        "monthly_cost": [1000, 1100]
    }).to_expr()

# This returns an error because food_cost has year=2025 but not cost_of_taxes
>>> food_cost + rent.add_dim("month")

# This is fine and drops (Jan, 2025)
>>> food_cost.drop_unmatched() + rent.add_dim("month")

# This is fine and keeps (Jan, 2025)
>>> food_cost.keep_unmatched() + rent.add_dim("month")

# What about the reverse situation
>>> food_cost = pl.DataFrame({
        "month": ["Jan", "Feb", "Mar", "Jan", "Feb", "Mar"],
        "year":     [2023, 2023, 2023, 2024, 2024, 2024],
        "monthly_cost": [1000, 1010, 1020, 1030, 1040, 1050]
    }).to_expr()
>>> rent = pl.DataFrame({
        "year": [2023, 2024, 2025],
        "monthly_cost": [1000, 1100, 1200]
    }).to_expr()

# The behavior is the same... except for this case!
>>> cost_of_housing + cost_of_taxes.add_dim("month").keep_unmatched()

# The above line throws an error since it is impossible to know which month should be applied in the added "month" dimensions, when there is no year match.
# Basically, `add_dim()` con only be used when there are no unmatched indices or when using `.drop_unmatched()`

Internals
Review of expression representation
Each expression is a dataframe with columns (dim1, dim2, dim3, ...) and also columns (__variable_id) and (__coef).
Approaches to adding
Adding requires performing a concatenation of both dataframes followed by a group_by(...).sum() operation.
(An alternative approach would be doing an outer join on all but the coefficient column, and then summing the coefficient columns using 0 instead of null. However this is almost surely slower and has no advantage.)
The concatenation approach is great however it doesn't check for unmatched indices. To check for unmatched indices, an outer join has to be done on the dimension columns. Rows containing null reveal unmatched indices. Additionally, if using drop_unmatched(), an inner join has to be performed on each operand individually to keep only the values in common. Finally, if adding a dimension a join needs to also be performed.
Unmatched strategy types
Each term would have an unmatched_strategy property which is an enum with one of three options.
error: This is the default and means an error should be thrown if the term has an unmatched row.
drop: Enabled via .drop_unmatched() this means unmatched rows are ignored (dropped).
keep: Enabled via .keep_unmatched() this means unmatched rows are kept (equivalent to giving a default 0 value to the other summation terms).
Proposed implementation decision tree

Scenario
add_dim()
Left and right terms' unmatched strategy
Operation required before concatenation

1 (Default)
None
error with error
1 outer join to check for unmatched

2
None
drop with drop
Two inner joins to remove unmatched on each term

3
None
keep with keep
None

4
None
drop with error
Filter drop term via right join with check

5
None
keep with error
Check error term via right join on keep term

6
None
drop with keep
Same as 4 but without check

7
One of the terms
error with error
Outer join on .add_dim() term with check

8
Both of the terms
error with error
Outer join on each .add_dim() term with check

TODO continue table for other cases
Performance and multi-element additions
The .concat() operation can be applied to a sum of multiple elements however the .join() cannot. Therefore, a sped-up addition for multi-element sums can only be done in scenario 3 (no added dimensions and .keep_unmatched())
One possibility is adding a configuration flag to the library called disable_unmatched_checks. If it is set to True, then .keep_unmatched() would become the default and a faster addition would be performed. Should this be implemented? If so, it could then be worthwhile to implement a LazySumExpression that allows for summation terms to accumulate and only computes the summation when it's needed.
Note that the performance speeds-up reported in #15 are not the full picture since they omitted the .group_by().sum() operation.
Implementation plan
For now, I suggest that I implement the add_dim() separately which (although less optimal) is a lot simpler.

Great ideas!
About handling missing dimensions
I prefer the explicit broadcasting involved in add_dim() to silently adding "missing" dimensions as is done in GAMS. In GAMS the broadcasting remains obvious, since the expressions include the dimensions of all variables (and not just their names).
About unmatched indices
In GAMS, values for missing indexes are zero or non-existing. Having as default a similar behavior in pyoframe is reasonable,. This is an argument for using scenario 3 as default, and is the most efficient of the alternatives, since it does not require any joins before concatenation.  However, I am also here in favor of making it explicit when unmatched indices should be ignored. So I support having the more cautious strategy of Scenario 1 as default.
Possibly, the case of drop_unmatched is sufficient infrequent that we can leave it to the user can do the match manually.
food_cost.filter(pl.col("year").is_in(rent.data.select("year"))) + rent.add_dim("month")
Speed improvement of multi-element additions
I redid the example of #15
In [22]: m = pf.Model()
    ...: m.x = pf.Variable(pl.DataFrame({"p": np.arange(2000)}), pl.DataFrame({"t": np.arange(200)}))
    ...: m.y = pf.Variable(pl.DataFrame({"p": np.arange(2000)}), pl.DataFrame({"t": np.arange(200)}))
    ...: m.z = pf.Variable(pl.DataFrame({"p": np.arange(2000)}), pl.DataFrame({"t": np.arange(200)}))
    ...: %timeit m.x + m.y + m.z
    ...: %timeit pl.concat([m.x.to_expr().data + m.y.to_expr().data + m.z.to_expr().data], how="vertical_relaxed").group_by(["p", "t", "__variable_id"]).sum()
495 ms ± 36.1 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
102 ms ± 5.22 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

Aha thank you for re-running the performance test! The 5x speedup is significant and good to know!
Seems like we agree to use add_dim(). Regarding, drop_unmatched I think it's actually a relatively common use case. E.g. from the nutrition we could do
m.max_nutrients = sum("food", m.Buy * food_nutrients).drop_unmatched() <= max_nutrient

instead of needing
m.max_nutrients = sum("food", m.Buy * food_nutrients.within(max_nutrient)) <= max_nutrient

to ensure a constraint is only created when max_nutrient exists.
I'm also hesitant to allow for a default of zero (without an explicit keep_unmatched) since it would mean that the following expression:
m.max_nutrients = sum("food", m.Buy * food_nutrients) <= max_nutrient

sets an upperbound of 0 whenever max_nutrient is undefined, which is an unwanted behaviour. I rather this expression return an error.
I think we can default to requiring the explicit keep_unmatched but with a system-wide setting
Config.disable_unmatched_checks = False
This flag could be set to True once a model has been properly tested.


Make fzf-cd-widget not conflict with user defined functions

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
The fzf-cd-widget uses builtin cd but still conflicts with custom functions called cd. I don't understand why, as when I ran builtin cd -- $dir manually, like in the fzf_cd_widget, it doesn't conflict...
Thanks

Hey @junegunn , please reopen as that pull request did not actually fix the issue... My issue was created after that pull request was closed, and I'm still having the same problem


The Deprotector freezes every time during the scanning for file names

I tried to use the Deprotector several times on the map in this link: https://xgm.guru/p/wc3/ACG-Arena-Original--l8.
The Deprotector hangs every time during the scanning.

Are you using default options? (no checkboxes selected)
I tested on my PC with this map & it worked fine, no freezing. Maybe try leaving it overnight?
That's an extremely large map (file size 630MB), so that's probably why it's so slow.
It may also help to try on a faster PC, the program can become extremely slow if your PC runs out of RAM while scanning.

Yes, I was using the default options. Is there any way you can share the deprotected map or a listfile generated from the deprotected map?

https://drive.google.com/file/d/16Kb6lDoKf760C8mnI4PdMjvXL_aU3D0S/view?usp=drive_link
Honestly, I don't think it's going to be very helpful for you. Although it opens in world editor, it has errors on save. It's an extremely big/complicated map, & the deprotector isn't perfect. Fixing it will probably take you a lot of work. Unless you're an expert map creator, I suggest starting with a simpler map.
If you just want to edit war3map.j, you don't need any deprotection at all. Just use MPQEditor with the protected map.
http://www.zezula.net/en/mpq/download.html
I have a tutorial video here explaining how to edit war3map.j directly:
https://youtu.be/Og1bGnSEoy8


Prices shifted by one hour

It seems that the prices are shifted by one hour. It may be a timezone problem, but I have HA currently set Central European Time/Prague+1:00 which should be OK.
Look at the example - OTE and raw data side by side. It shows that the price 1.2CZK/KWh starts at 09:00

Here is the time from HA where it starts at 08:00.

Basically HA shows the price one hour earlier than it should.

I think HA shows correct price but you might be interpreting OTE prices incorrectly. "9. hodina" on OTE means 10:00 - 11:00. The day starts with "1. hodina" and it means 0:00 - 1:00. Does it make sense?

I think HA shows correct price but you might be interpreting OTE prices incorrectly. "9. hodina" on OTE means 10:00 - 11:00. The day starts with "1. hodina" and it means 0:00 - 1:00. Does it make sense?

just a little correction: 9th hour is from 8:00-9:00

Of course :facepalm: Thank you

Closing now. @shaman79 please reopen if you still face some problems.


README pls

How would we use this for our own services?

This is being built as a near-term solution for failover and connection severing for the MySQL service. It is not yet in a final release of that service, and should be considered experimental. Once we've got it released we could evaluate other use cases.
For more info: https://github.com/cloudfoundry/cf-mysql-release/blob/release-candidate/docs/proxy.md


Unify DeleteAction API

Hey there,
in order to fix https://github.com/tinacms/tinacms/issues/884 I've prepared a little PR for you. I hope this covers everything and solves it like intended.
I saw you are using conventional commits, I recommend to setup husky to check for that :)
Cheers
Marc

I guess those failing pipelines are not related to my actual code changes?

Amazing! Thank you 😄
Correct. Failing pipelines are because your fork doesn’t have the same secrets set. We will merge into another branch to verify the pipeline.
We’re already using husky to run prettier, so using it to check commit formats is a great idea. Thanks for that too 👍

Looks great! Just a package-lock.json conflict in the way :)

@all-contributors please add @mmintel for code


DO NOT MERGE Test https://github.com/cilium/cilium-cli/pull/2513

/ci-eks

/ci-ipsec-e2e

/ci-e2e-upgrade


unTimeout on-start relies on cache; often fails

When the bot starts & attempts to unTimeout DQs that have expired during downtime, it relies on Discord.js's cache:
https://github.com/jtkiesel/bigbro/blob/14d5afc7151d644ee3ba6113ba311d9c061efa29/src/index.ts#L98-L110
https://github.com/jtkiesel/bigbro/blob/14d5afc7151d644ee3ba6113ba311d9c061efa29/src/commands/dq.ts#L47-L62
However, this is unlikely to have been cached on-start, and as such, this often fails.
We should probably look into the following:

Use something along the lines of const guild = await client.guilds.fetch(guildID); const member = guild.member(memberID).
Only remove the DB record if the member is unTime'd out successfully

The switch to using Discord's timeout API rather than our own has caused this issue to be OBE.


Drest/feature data pipelines/fixes 1

@Pralish Great job on getting started on the bug fixes! In the meantime, I fixed the test suite so we know the amount of test coverage we need to add to get data pipelines merged.
Right now we are at 66%, I have attached the coverage report below
coverage.zip

We can start by bumping the coverage on these controllers:

@donrestarone , Thank you so much  for the test cases. I have finished working on the feedbacks and will be working on the unit tests.


feat: rate limit middleware

this middleware allows to rate limit clients using x/time/rate
closes #43
PS: not sure about the api yet, that's why no tests either.
Looking for feedback.

Codecov Report

Merging #44 (269b780) into main (7aaf336) will increase coverage by 1.22%.
The diff coverage is 94.28%.

@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
+ Coverage   56.66%   57.88%   +1.22%
==========================================
  Files          19       20       +1
  Lines        1036     1071      +35
==========================================
+ Hits          587      620      +33
- Misses        394      396       +2
  Partials       55       55

Impacted Files
Coverage Δ

ratelimiter/ratelimiter.go
94.28% <94.28%> (ø)

Continue to review full report at Codecov.

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


Is UseNNUE working?

Hi,
I played a game from a certain position, beetween "false" and "true" versions. Then I reversed the colors and the games was identical.
Alex

I tested it in console and the switching between UseNNUE true and false works for go command.
While testing it, I also found that when you write uci  command, the line that reports option name UseNNUE type check default will always report false, because it is hardcoded. I forgot to change the line to report the actual value. I will fix this in new version.
Here are what I did to test if it is working:

Run the chess engine in console window
Run command debug on. This will show if the UseNNUE is true or false.
Run command setoption name UseNNUE true. Engine should respond with UseNNue = true. If the flag true or false is misspelled or entirely left out, UseNNUE parameter will default to true. If the communication between the engine and GUI is broken, this might be the reason, why it always defaults to true.
 I also get slightly different moves.

Regrdas,
Janez

Yes, you are exactly right. I will fix that in next release.
Note to me: fix 2 bugs.


Some common interfaces are missing

For example, HashMap doesn't have .byKey, .byValue, HashSet doesn't have opApply, etc.
Are those deliberately left out, or are there plans to implement them in the future? If it's the latter case, are you accepting pull requests? I really want to contribute to this project.

If it's the latter case, are you accepting pull requests?
https://github.com/economicmodeling/containers/pulls?q=is%3Apr+is%3Aclosed

The changes included in #104 add a lot of methods to the containers including byKey and byValue. I've removed opApply from the containers and byKeyValue should be used instead.

Those referenced in the first message are here now. Open specific issues with the expected function names if any remain.


Unable to create charcter

I get a "Sorry..." message when trying to download a character I made... https://imgur.com/C8xFXnx

Hmm, what error are you getting in the console? My guess would be that it isn't able to download the WandererStringsLoader.dll because something is blocking it, or because you are including invalid characters somewhere.

I will check the console now

I can't get an error to show in the console but the response header when I download says Status Code: 304.

I don't know what you did but I was just able to download my character and it worked... thanks!


Add BytesIO buffer for binary content

This should solve #1222

Coverage remained the same at 53.433% when pulling 815514c936e387401ea38221eb51dc6a773f2c98 on Tixifni:master into bfdef523eb7e934469d5c4a34652626b723d7481 on django-extensions:master.

Could you please add some test(s) for this ?
So we know this works on both Py 2.7 and Py 3.x and that we don't regress later :-)

Sure, will do later today

bump


added a python based rock paper scissors game

I have created a python based rock, paper, scissor game. This is how it looks like -

Good Job 💯


work with React 0.14?

I am hitting some of the same issues that others have reported, especially #8.
Based on the lack of responses by the author to other issues, I'm guessing this library has been abandoned. It would be nice to document that fact in the README so others don't waste time trying to get it to work.

:+1:

+1

+1

:+1:

:+1:

+1

+1

+1

Based on the lack of responses by the author to other issues, I'm guessing this library has been abandoned. It would be nice to document that fact in the README so others don't waste time trying to get it to work.

Please use that time making it work with 0.14 :)
I've been focused on the Router.

I have added a pull request making this library work with react 0.14: https://github.com/ryanflorence/react-magic-move/pull/18
The library is great!!

@ryanflorence It takes 10 seconds to mark it as deprecated, much longer to become acquainted with the library enough to update it!
 As someone who wasted 20 minutes trying to get this to work, though, a quick deprecation notice would have been highly appreciated =)

https://github.com/ryanflorence/react-magic-move/commit/2fa595c4050f49fb139c0e0052d8eadd05b94f73

You rock. Thank you =)


exit app on xapp error

Just followed the example from the npm page for artsy-xapp - hopefully this'll catch 502s.

Yeah, I agree with this - merge on green

Thanks Orta!
On Tue, Aug 28, 2018 at 4:55 PM Orta<EMAIL_ADDRESS>wrote:

@orta commented on this pull request.
In src/index.js
https://github.com/artsy/force/pull/2830#discussion_r213468875:

@@ -30,6 +30,10 @@ setup(app)

// Connect to Redis
cache.setup(() => {

// if we can't get an xapp token, just exit and let the whole system try
// again - this prevents a sustained broken state when gravity returns a
// 502 during force startup.
artsyXapp.on('error', process.exit)

I'll add this to #2829 https://github.com/artsy/force/pull/2829
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/artsy/force/pull/2830#discussion_r213468875, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAk1hVEEIXleIDNPiYfYreEfV141YW3Fks5uVa41gaJpZM4WQFLd
.


Update index.html

Fix symbols style that was broken by the recent commit updating palette 10 to 12

(I'm not 100% sure this'll fix it lol)

I'm not sure this is necessary anymore.


Windows Multiplechoice doesn't work

Using master. When using Windows the multiplechoice throws an exception.
func(c *ishell.Context) {
    choice := c.MultiChoice([]string{
        "Golangers",
        "Go programmers",
        "Gophers",
        "Goers",
    }, "What are Go programmers called ?")
    if choice == 2 {
        c.Println("You got it!")
    } else {
        c.Println("Sorry, you're wrong.")
    }
},

This code will reproduce the problem

I was having same issue on windows and confirm that solution chzyer/readline#132  partially solves the issue ( when replacing all 0 arguments in readline.func(0) with int(os.Stdout.Fd())
So good thing is we have a working choice list on windows.
Remaining issue is that with these changes the select list gets appended on each selection. Example below is a select list with cancel && 2 options, making a couple of selections..
Project list:
 >   Cancel
   [ 1224 ]  test
Project list:
     Cancel
 > [ 1224 ]  test
Project list:
     Cancel
   [ 1224 ]  test
Project list:
     Cancel
 > [ 1224 ]  test
Project list:
     Cancel
 > [ 1224 ]  test
   [ 1217 ] more test```


CleanupExecuteReaderAsync OperationId is empty GUID

WriteCommandAfter: CleanupExecuteReaderAsync has empty GUID as OperationId
Hello!
I'm writing the library for tracing Sql.Client. And I ran into a problem that the CleanupExecuteReaderAsync operation (or correctly call it command), which is produced as "Microsoft.Data.SqlClient.WriteCommandAfter" event, contains Guid.Empty OperationId. I've been confused with this behaviour because it is "after" command.
Event example:
{
    OperationId: 00000000 - 0000 - 0000 - 0000 -<PHONE_NUMBER>00,
    Operation: "CleanupExecuteReaderAsync",
    ConnectionId: 910 f6790 - b10b - 4143 - b723 - 915229 b16bb5,
    Command: SqlCommand {
        Connection: SqlConnection {
            RetryLogicProvider: SqlRetryLogicProvider {
                Retrying: null,
                RetryLogic: SqlRetryLogic {
                    PreCondition: null,
                    NumberOfTries: 1,
                    Current: 0,
                    RetryIntervalEnumerator: SqlNoneIntervalEnumerator {
                        GapTimeInterval: 00: 00: 00,
                        MaxTimeInterval: 00: 00: 00,
                        MinTimeInterval: 00: 00: 00,
                        Current: 00: 00: 00
                    },
                    TransientPredicate: "System.Predicate`1[System.Exception]"
                }
            },
            StatisticsEnabled: False,
            ConnectionString: "",
            ConnectionTimeout: 15,
            CommandTimeout: 30,
            AccessToken: null,
            Database: "",
            DataSource: "",
            PacketSize: 8000,
            ClientConnectionId: 910 f6790 - b10b - 4143 - b723 - 915229 b16bb5,
            ServerVersion: "The property accessor threw an exception: InvalidOperationException",	// this cause because SqlConnection.get_ServerVersion() uses SqlConnection.GetOpenTdsConnection() that throws ADP.ClosedConnectionError() if SqlInternalConnectionTds innerConnection is null when I dump this event
            ServerProcessId: 0,
            State: Closed,
            WorkstationId: "pay-994d47d-qwls4",
            Credential: null,
            FireInfoMessageEventOnUserErrors: False,
            CanCreateBatch: False,
            Site: null,
            Container: null
        },
        RetryLogicProvider: SqlRetryLogicProvider {
            Retrying: null,
            RetryLogic: SqlRetryLogic {
                PreCondition: null,
                NumberOfTries: 1,
                Current: 0,
                RetryIntervalEnumerator: SqlNoneIntervalEnumerator {
                    GapTimeInterval: 00: 00: 00,
                    MaxTimeInterval: 00: 00: 00,
                    MinTimeInterval: 00: 00: 00,
                    Current: 00: 00: 00
                },
                TransientPredicate: "System.Predicate`1[System.Exception]"
            }
        },
        Notification: null,
        Transaction: null,
        CommandText: "select 1;",
        ColumnEncryptionSetting: UseConnectionSetting,
        CommandTimeout: 30,
        CommandType: Text,
        DesignTimeVisible: True,
        Parameters: [],
        UpdatedRowSource: Both,
        Site: null,
        Container: null
    },
    TransactionId: null,
    Statistics: [DictionaryEntry {
        Key: "BuffersReceived",
        Value: 1
    }, DictionaryEntry {
        Key: "BuffersSent",
        Value: 1
    }, DictionaryEntry {
        Key: "BytesReceived",
        Value: 45
    }, DictionaryEntry {
        Key: "BytesSent",
        Value: 48
    }, DictionaryEntry {
        Key: "CursorOpens",
        Value: 0
    }, DictionaryEntry {
        Key: "IduCount",
        Value: 0
    }, DictionaryEntry {
        Key: "IduRows",
        Value: 0
    }, DictionaryEntry {
        Key: "PreparedExecs",
        Value: 0
    }, DictionaryEntry {
        Key: "Prepares",
        Value: 0
    }, DictionaryEntry {
        Key: "SelectCount",
        Value: 1
    }, DictionaryEntry {
        Key: "SelectRows",
        Value: 1
    }, DictionaryEntry {
        Key: "ServerRoundtrips",
        Value: 1
    }, DictionaryEntry {
        Key: "SumResultSets",
        Value: 1
    }, DictionaryEntry {
        Key: "Transactions",
        Value: 0
    }, DictionaryEntry {
        Key: "UnpreparedExecs",
        Value: 1
    }, DictionaryEntry {
        Key: "ConnectionTime",
        Value: 0
    }, DictionaryEntry {
        Key: "ExecutionTime",
        Value: 0
    }, DictionaryEntry {
        Key: "NetworkServerTime",
        Value: 1
    }],
    Timestamp:<PHONE_NUMBER>95323
}

To reproduce
You need to wait when a connection will be closed
Expected behavior
I expect this command has some not empty id. :)
Further technical details
Microsoft.Data.SqlClient version: 3.0.1
.NET target: .net6.0
SQL Server version: Microsoft SQL Azure (RTM) - 12.0.2000.8
Operating system: Docker container(image aspnet:6.0)

I think the reason is that operationId here is empty because it this command doesn't have a parent operation

Interesting. Can you give me a small reproduction app? It doesn't need the listener that you're using to see the value because it'll be visible in the debugger, it's just a lot easier to debug these things when you have a clean reproduction of the exact issue rather than having to work your way towards a problem.

@AndrewP-GH as Wraith mentioned, Please provide us with small application that reproduces the issue and we can look into the issue.

Fix is available here https://github.com/dotnet/SqlClient/pull/1511

I tried to reproduce this bug on 5.1.0 and seems it's fixed now. This can be closed.

It seems strange that we accidentally fixed it, but good. Thanks for your work.


rust_version & kubecontext together cause right prompt to not appear

Running next branch with antigen. The entire right prompt doesn't show. Each one works without the other.
POWERLEVEL9K_INSTALLATION_PATH=$ANTIGEN_BUNDLES/bhilburn/powerlevel9k
POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=(context dir vcs)
POWERLEVEL9K_RIGHT_PROMPT_ELEMENTS=(status root_indicator background_jobs pyenv
rust_version kubecontext time)
antigen theme bhilburn/powerlevel9k powerlevel9k

Ooof. That's not great.
Okay, we need to figure out when it broke. Are you familiar with git bisect at all, @neverfox?

Hey @neverfox - I just wanted to raise this one again. Are you still seeing the issue? Can you pull the latest master and let me know if the problem still exists?

Closing this one due to inactivity. Feel free to re-open if you are still having issues!

@neverfox ZSH automatically hides the right prompt if the prompt line gets too long. Maybe this is the case here. Could you post a screenshot of your prompt?

@dritter Unfortunately, I no longer user powerlevel9k, so I couldn't recreate the issue now without modifying my setup. I apologize for not following up sooner at the time.


ACR Beta 2 API Experiment

Addresses:
https://github.com/azure/azure-sdk-for-net/issues/19393
https://github.com/azure/azure-sdk-for-net/issues/19397
https://github.com/azure/azure-sdk-for-net/issues/20334

Closing as changes have been implemented in https://github.com/Azure/azure-sdk-for-net/pull/20579


ValueError exception when using da.stack() is too verbose.

I was attempting to stack 1-d arrays that were read in from a list of hdf5 files and I blindly followed some boiler-plate here (array-hdf5). I had ~10,000 arrays that I attempted to stack and the traceback was too verbose as it lists the size of each individual array that I was attempting to stack.
It think the Traceback should  skip displaying the sizes of the individual arrays or handle them in a smarter way by return. Note that after actually reading the stack, concatenate, and block documentation it was clear that I should be using concatenate and so it was really a user error on my part lol.
Anyways, the problem still remains that the traceback is essentially unbounded. Plus, it's a simple fix as you only need to remove the  "..Got %s", [x.shape for x in seq]" portion of the ValueError() or maybe return only a list of the unique shapes tuples by doing set([x.shape for x in seq]) or something along those lines.
MWE
import numpy as np
import dask.array as da

N=100
rand_size = np.random.randint(1e3,1e4,size=N) # N random array sizes
rand_arr = [np.random.rand(size) for size in rand_size] # N randomly sized arrays
da_rand_arr = [da.from_array(arr, chunks=1e2) for arr in rand_arr] # Convert them to dask arrays
x = da.stack(da_rand_arr, axis=0)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-1-1bc07fcfdcc1> in <module>()
      6 rand_arr = [np.random.rand(size) for size in rand_size] # N randomly sized arrays
      7 da_rand_arr = [da.from_array(arr, chunks=1e2) for arr in rand_arr] # Convert them to dask arrays
----> 8 x = da.stack(da_rand_arr, axis=0)

~/miniconda3/envs/astroconda3/lib/python3.6/site-packages/dask/array/core.py in stack(seq, axis)
   3637     if not all(x.shape == seq[0].shape for x in seq):
   3638         raise ValueError("Stacked arrays must have the same shape. Got %s",
-> 3639                          [x.shape for x in seq])
   3640
   3641     ind = list(range(ndim))

ValueError: ('Stacked arrays must have the same shape.

Yeah, I like the idea of returning the first size that fails. Returning a list of only the unique sizes still has the same problem in the event that all arrays have different sizes.

Something like: "all shapes so far were {}, but at chunk {} found shape {}"

Ok, so here's what I came up with:
if not all(x.shape == seq[0].shape for x in seq):
    idx = np.where(np.asanyarray([x.shape for x in seq]) != seq[0].shape)[0]
    raise ValueError("Stacked arrays must have the same shape.\n"
                     "The first {0} had shape {1}, while array "
                     "{2} has shape {3}".format(idx[0],
                                                seq[0].shape,
                                                idx[0] + 1,
                                                seq[idx[0]].shape))

Example:
N=20
rand_size = np.random.randint(1e2,1e3,size=N)
rand_size = np.insert(rand_size,[1]*20,rand_size[0])
rand_arr_2d = [np.random.rand(size, size) for size in rand_size]
da_rand_arr_2d = [da.from_array(arr, chunks=(50,50)) for arr in rand_arr_2d]
da.stack(da_rand_arr_2d)

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-19-b0cddef797aa> in <module>()
----> 1 da.stack(da_rand_arr_2d)

~/dask/dask/array/core.py in stack(seq, axis)
   3790                          "The first {0} had shape {1}, while array "
   3791                          "{2} has shape {3}".format(idx[0], seq[0].shape,
-> 3792                                                     idx[0]+1, seq[idx[0]].shape)
   3793                          )
   3794

ValueError: Stacked arrays must have the same shape.
The first 21 had shape (354, 354), while array 22 has shape (995, 995)

I ran flake8 dask, which caught a lack of whitespace around the idx[0] + 1 line, so I fixed that. Then I ran py.test dask/array --verbose on my local copy and the test_stack_err() failed on the second assertion. I went and printed out str(e) and for some reason it only displays the first line in the error message supplied to ValueError(). Anyways, I removed the '\n' at the end of the first sentence and it passed. A second solution is to update the test function to explicitly access the value attribute of the raised exception (e.g. str(e.value) vs. str(e)). Any preference? I don't think there is much gained by including the '\n', so I'm fine with just removing it and only changing one file.


RTPS Domain hostid part of the GUID match first interface IP

Given the code here to determine and assign a participant GUID, the "hostid" portion of the GUID (first 4 bytes) is initialized with eProsima prefix (0x010f) and the last two digit of the IPv4 address of the first network interface found.
On system with several interfaces, most of which internals, this scheme is not sufficient for generating a mostly unique hostid to be used for the RTPS Domain.
When the Fast-RTPS process is started as part of the boot sequence, the process ID can often be the same on multiple machines causing multiple participant to have the exact same GUID and confuse each-others.
A more robust approach (on linux and osx) would be to make a call to the gethostid(2) method and use the information received to initialized the hostid portion of the RTPS GUID.

Pull request #53 suggested a different way to adress this, by using the MD5 hash of all the returned interfaces. It may be a nice mechanism ... ¿thoughts?

I supposed we could do something with the hash of the MAC address of the interface, yes. This would be a lot more unique compared to an ip especially when you have many interfaces. Considering using all of the MAC addresses for all interfaces might actually work better and cover most if not all cases.

Just looked at the original pull request. I'll play around with it and see how I can make it work more broadly.

I think the PR to address this issue has been merged. Can we close this issue then?

Should automatically close tomorrow when it gets into master


[BUG] Timber::get_pages_menu() throws warnings when using Polylang

Expected Behavior
Expected to not get any warning messages.
Actual behavior
Warning:
Undefined array key "menu" in
wp-content\mu-plugins\polylang\frontend\frontend-nav-menu.php
on line 280
Warning:
Undefined array key "theme_location" in
wp-content\mu-plugins\polylang\frontend\frontend-nav-menu.php
on line 294
Steps to reproduce behavior
No response
Notes
No response
What version of Timber are you using?
2.0
What version of WordPress are you using?
6.4.3
What version of PHP are you using?
8.1
How did you install Timber?
Installed or updated Timber through Composer

Hi @lnorby , unfortunately we cannot fully support every plugin. That being said, can you check how you are setting up your menu's with Timber?
if you are using the Timber::get_menu function you could try this:
$menu = Timber::get_menu( 'my-menu', [
'menu' => YOUR_MENU_ID,
'theme_location' => 'YOUR_REGISTERED_MENU_LOCATION'
]
 );

I am using Timber::get_pages_menu(). I tried the following, and it worked. No warnings now.
Timber::get_pages_menu( [
'menu' => '',
'theme_location' => '',
] )
Maybe you could set these as default arguments for the method.

Hmm, looking at the default arguments for wp_page_menu()'s which Timber is using, there are no menu and theme_location arguments. https://developer.wordpress.org/reference/functions/wp_page_menu/
So adding these two parameters to get_pages_menu is not something that is standard in WordPress as well. Polylang might be checking for the wrong parameters when handling a pages menu?

The wp_page_menu() function itself is working fine with Polylang, no warnings.

Hi @lnorby ,
I took another look at this and I think what leads to your error is the following:

[the pages menu by wordpress takes a wp_page_menu and transforms it to an actual menu
to be compatible with both the wordpress pages function and the wordpress menu we apply both the  wp_page_menu_args  and wp_nav_menu_args filters in Timber::get_page_menu()
Since Polylang is adding a filter here to wp_nav_menu_args that assumes menu and 'theme_location' are always set, this fails. I think it is right for them to assume those properties are always set since they are by default set by WordPress. and thus the error occurs.

@timber/rangers , is this something the encourage to fix by the user themself or do we want to improve the compatibility since we are actively calling the wp_nav_menu_args without providing all the arguments?

@cafesk8 , I think we are almost ready to release 2.3.0. Hopefully somewhere still in this week.


pep8 issues

for python, there is a style-guide called pep8 which most python programmers use.
while your code isn't formatted badly, it doesn't quite conform to pep8.
would you like this improved in a pull request?

Hi Thomas,
thanks for your comments. Feel free to send a pull request, any help is appreciated.


WIP (don't merge yet!!): Added a separate gradle file to maintain all dependency versions.

Work in progress

I really think we should go for .kts

Ok, well let me finish the rework in groovy first. After that I can take a look how the migration work.


Could you also add a dataset card using the template guide here : https://github.com/huggingface/datasets/blob/master/templates/README_guide.md
If you can't fill some fields then just leave [N/A]

Quick amendment: [N/A] is for fields that are not relevant: if you can't find the information just leave [More Information Needed]

@lhoestq since datset cards are optional for this sprint I'll add those later. Good for merge.

Indeed we only require the tags to be added now (the yaml part at the top of the dataset card).
Could you add them please ?
You can find more infos here : https://github.com/huggingface/datasets/blob/master/ADD_NEW_DATASET.md#tag-the-dataset-and-write-the-dataset-card

@lhoestq added tags, I'll fill rest of the info after current sprint :)

The tests are failing tests for other datasets, not this one.

@lhoestq  could you tell me why these tests are failing, they don't seem related to this PR.


sample_prior_predictive is the only method with default extend_idata=False

I don't see why this should behave differently than the rest
https://github.com/pymc-labs/pymc-marketing/blob/369a5054398dbb6bbb372ef1eb61bb953be26fd2/pymc_marketing/model_builder.py#L543

Closed with #741


Write bad attr value

Im  used xml decopile type. Original attr value is 192

After rebuild is 65472

Files

After rebuilding arsc, your editor writing wrong values in attr

Method of decompile XML or JSON ?
What is the type of of attr ? What are the values ?

Fixed with this
Thanks , you found serious issue.
Next time please avoid such large screen shots or crop to min  size possible, or you can say @attr/cornerSize entry value supposed to be 192 but found 65472


chore: use executeCallbacks for cleaning up multiple functions returned from an effect

In many builder's effects, there are multiple cleanup functions that should be executed when the effect reruns and when the effect is destroyed. Instead, we can just return all the cleanup functions from an effect inside executeCallbacks and avoid the need to call the cleanup functions manually at the start of each effect and in the action's destroy method.
However, some builders call tick() before assigning the cleanup functions, which creates a promise. In order to allow returning cleanup functions that are created after a tick, PR https://github.com/melt-ui/melt-ui/pull/1140 should be merged first. That PR introduced the runAfterTick prop to the effect function. Thanks to that PR, we will be able to run the entire effect callback after a tick, which allowed this PR to return all the cleanup functions in one go after a tick with executeCallbacks.

Will close for now, as effect may be refactored later on


How to check the version with python code?

pyasn1.__version__  raise error. So pyasn1 does not follow https://www.python.org/dev/peps/pep-0396/

What kind of error? It's a bit of chicken-and-the-egg problem, but which pyasn1 version are you using?
The __version__ attribute works for me:
>>> pyasn1.__version__
'0.4.5'
>>>

I use pip to install pyasn1 and the latest version was 0.4.4. ?

Yes, 0.4.4 is the latest and it seems to have __version__ in place:
$ python3 -m venv venv
$ . venv/bin/activate
(venv) $ pip install pyasn1
Collecting pyasn1
  Using cached https://files.pythonhosted.org/packages/d1/a1/7790cc85db38daa874f6a2e6308131b9953feb1367f2ae2d1123bb93a9f5/pyasn1-0.4.4-py2.py3-none-any.whl
Installing collected packages: pyasn1
Successfully installed pyasn1-0.4.4
(venv) $ python
Python 3.5.4 (default, Oct  9 2017, 12:07:29)
[GCC 6.4.1 20170727 (Red Hat 6.4.1-1)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import pyasn1
>>> pyasn1.__version__
'0.4.4'
>>>

It is so strange, I used pythonista to run it. Several minutes age, I couldn't find version. Now it shows. Maybe it is a bug on pythonista.


Initial implementation of managed labels

See #47. This is the initial implementation of managed labels and
annotations - that is, the ability to set a label (or annotation) in a
HierarchyConfiguration object, and have that label (...) propagated to
all descendants, similar to the way objects are propagated. As with
objects, only allowlisted labels are propagated, as defined by the
command line option '--managed-namespace-[label|annotation]'.
Still to come: validator support, better conditions for conflicts,
better testing for external namespaces, better testing for more regexes,
documentation.
Tested: see new integ tests.

Hey @rjbez17 let's maybe give this a day or two for others to chime in before we submit this?

Sure thing. Is this a draft and you'll add the other pieces into this PR or follow up with a separate PR?

Sure thing. Is this a draft and you'll add the other pieces into this PR or follow up with a separate PR?

No, this is the commit I want to submit. The other pieces can come in other PRs, hopefully including PRs submitted by other people. I just wanted to do the core work myself, but there are people volunteering to do things like add validators so I'd like to take them up on it.

@adrianludwin I did a first scan - mainly focusing on the "api" to this new feature. I think I need understand more of the HNC concepts before looking into the reconcile logic. 😅 It seems a bit complicated.

Yeah I need to (re?) write a guide to how HNC works internally to make it easier for people to onboard. It's not that complicated once you get it, but it's not exactly intuitive either.

/assign @rjbez17
I think we're ready for a review whenever you are, thanks!

/lgtm


Document "fastclock=0"

Add information somewhere that you need to build with fastclock=0 if you're doing anything involving the LCD with the external clock module connected.

This should be now done, can you verify and update issue accordingly?

Yes, this is fixed!


Eloquent set created_at does not work with special dates

Laravel Version: 5.4
PHP Version: 7.0
Database Driver & Version: mysql

Description:
Inserting a date works sometimes but sometimes not. Here is an example. Can somebody help me ?

Steps To Reproduce:

@wbdavidhoeck steps to reproduce?

Closing for lack of activity. Please try the forums if you're still facing an issue.


Integration test for istioctl metrics

Resolves https://github.com/istio/istio/issues/16975 by creating a test of istioctl metrics.
Eventually there should be an integration test for all istioctl commands.  When we tried to graduate this command in 1.3 there was feedback that we need an integration test before graduating.
I am looking for feedback on how the test is structured.  Currently it reproduces TestIngessToPrometheus_IngressMetric to set up the data, then does the test.  Should I combine those tests (for speed) or leave them separate (for understandability)?

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

Test name
Commit
Details
Rerun command

lint_istio
d9acc8c5b4f535158af7f6b382d70ee9e80d5044
link
/test lint_istio

integ-mixer-k8s-tests_istio
d9acc8c5b4f535158af7f6b382d70ee9e80d5044
link
/test integ-mixer-k8s-tests_istio

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.

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

Test name
Commit
Details
Rerun command

integ-mixer-k8s-tests_istio
d9acc8c5b4f535158af7f6b382d70ee9e80d5044
link
/test integ-mixer-k8s-tests_istio

lint_istio
b883988c6702b1bae9157f8dd76ecd9ca258967a
link
/test lint_istio

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.


Why model.Summary() shows all layers in sub model.

In old version(I forget which one). Sub model just shows model's name but not all layers.
But now,it shows all layers.
My versoin: TF2.0(GPU)
demo code:
class Swish(keras.layers.Layer):
    def __init__(self):
        super(Swish, self).__init__()
        self.weight = self.add_weight(initializer='uniform',trainable=True)

    def __call__(self, inputs):
        return inputs+tf.sigmoid(self.weight*inputs)

the full code :https://github.com/tensorflow/tensorflow/issues/34637

@HLSS-Hen, Tried replicating the issue but got error at from Models import MCN. Looks like file or class method is missing. Please provide us more information to reproduce the issue. Thanks!

@ HLSS-Hen，尝试复制该问题，但在遇到错误from Models import MCN。看起来缺少文件或类方法。请提供更多信息以重现该问题。谢谢！
No, there is.
MCN.py is the file you miss.

@HLSS-Hen, MCN.py file doesn't have Models class method.

@gadagashwini Models is just a file.

@HLSS-Hen, Just to verify, MCN.py and Models are they same file or different.

@gadagashwini
you can use following code(use any database with shape [?,256,256,3]):
import os
os.environ['TF_AUTO_MIXED_PRECISION_GRAPH_REWRITE_IGNORE_PERFORMANCE'] = '1'
import math
import tensorflow as tf
from tensorflow import keras

class Swish(keras.layers.Layer):
    def __init__(self):
        super(Swish, self).__init__()
        self.weight = self.add_weight(initializer='uniform',trainable=True)

    def __call__(self, inputs):
        return inputs+tf.sigmoid(self.weight*inputs)

class Conv(keras.Model):
    def __init__(self,filters,kernel_size=1,strides=1,padding='valid'):
        super(Conv, self).__init__()
        self.conv = keras.layers.Conv2D(filters,kernel_size,strides,padding)
        self.bn = keras.layers.BatchNormalization()
        self.ac = Swish()

    def __call__(self,inputs):
        return self.ac(self.bn(self.conv(inputs)))

class SEBlock(keras.Model):
    def __init__(self, filters):
        super(SEBlock, self).__init__()
        self.conv0 = keras.layers.Conv2D(filters//4,1,1)
        self.drop = keras.layers.Dropout(0.25)
        self.conv1 = keras.layers.Conv2D(filters,1,1)
        self.bn = keras.layers.BatchNormalization()
        self.ac = Swish()

    def __call__(self,inputs):
        x = self.conv1(self.drop(self.conv0(tf.reduce_mean(inputs,[1,2],keepdims=True))))
        return self.ac(self.bn(tf.sigmoid(x)*inputs))

class ResBlock(keras.Model):
    def __init__(self, filters):
        super(ResBlock, self).__init__()
        self.conv0 = keras.layers.Conv2D(filters//4,1,1)
        self.drop = keras.layers.Dropout(0.25)
        self.conv1 = keras.layers.Conv2D(filters,3,1,'same')
        self.bn = keras.layers.BatchNormalization()
        self.ac = Swish()

    def __call__(self,inputs):
        x = self.conv1(self.drop(self.conv0(inputs)))
        return self.ac(self.bn(inputs+x))

def mcn_520(width, growth,input_shape=[256,256,3]):
    fs = int(width*growth)
    inputs=keras.layers.Input(input_shape)
    x=keras.layers.Conv2D(fs,8,2)(inputs)
    x=keras.layers.MaxPool2D(2)(x)
    x1=Conv(fs//width)(SEBlock(fs)(x))
    x2=Conv(fs//width)(ResBlock(fs)(x))
    for i, depth in enumerate([2, 3, 5, 4]):
        for _ in range(int(6*depth)):
            fs+=int(math.sqrt(fs*width))
            t=keras.layers.Concatenate()([x,x1,x2])
            t=keras.layers.Dropout(0.25)(t)
            t=Conv(fs//width, 1, 1)(t)
            t=keras.layers.Dropout(0.25)(t)
            x1=SEBlock(fs//width)(t)
            x2=ResBlock(fs//width)(t)
            t=keras.layers.Concatenate()([t,x1,x2])
            t=keras.layers.Dropout(0.25)(t)
            t=Conv(growth,1,1)(t)
            x=keras.layers.Concatenate()([x,t])
        if i != 3:
            fs //= 2
            x=keras.layers.MaxPool2D(2)(Conv(fs)(x))
            x1=keras.layers.MaxPool2D(2)(Conv(fs//width)(x1))
            x2=keras.layers.MaxPool2D(2)(Conv(fs//width)(x2))
    x=keras.layers.GlobalMaxPool2D()(x)
    x=keras.layers.Dropout(0.25)(x)
    outputs=keras.layers.Dense(1000,activation='softmax')(x)
    return keras.Model(inputs=inputs,outputs=outputs,name='MCN520')

for gpu in tf.config.experimental.list_physical_devices('GPU'):
    tf.config.experimental.set_memory_growth(gpu, True)
logical_gpus = tf.config.experimental.list_logical_devices('GPU')

starategy=tf.distribute.MirroredStrategy()
with starategy.scope():
    model=MCN.mcn_520(2,24)
    model.summary()
    model.compile(
        loss=tf.keras.losses.SparseCategoricalCrossentropy(),
        optimizer=tf.keras.optimizers.SGD(),
        metrics=[tf.keras.metrics.TopKCategoricalAccuracy(1,'top1'),tf.keras.metrics.TopKCategoricalAccuracy(5,'top5')]
        )
    fit_ds,val_ds=ds(BATCH_SIZE)
    model.fit(
        fit_ds, # please use your database
        epochs=1000000,
        steps_per_epoch=BATCHS_PER_APLY_GRADIENTS*200,
    )

@HLSS-Hen, Please find the attached the gist and help us in reproducing the issue. Thanks!

@gadagashwini  , sorry I took some time to  visit the page. And my google account have some wrong.
Please change     model=MCN.mcn_520(2,24) to     model=mcn_520(2,24) .
If you don't set in your dataset. please remove model.fit to the end.

I could replicate the issue with Tf 2.0.
Please see the gist here. Thanks!

@HLSS-Hen This model is related to your issue but check the approach ('base_model') used to show pretrained model as one layer. Thanks.

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!