Batch only produces one FHIR resource type

According to the docs, the batch processor ETL should by default produce Patient, Observation, and Encounter. Without options it is only producing Encounter. With resourceList=Patient,Encounter,Observation,Condition,DiagnosticReport,MedicationStatement it only produces Condition.
E.g.
java -Xmx8192M -cp ./pipelines/batch/target/batch-bundled-0.1.0-SNAPSHOT.jar com.google.fhir.analytics.FhirEtl --fhirServerUrl=http://localhost:8080/fhir --outputParquetPath=/tmp/parquet/ --resourceList=Patient,Encounter,Observation,Condition,DiagnosticReport,MedicationStatement --batchSize=10 --jdbcModeHapi=false --jdbcModeEnabled=false

It should produce multiple resource types if the FHIR-store have those (it is part of the continuous build tests). From your arguments it seems you are using the FHIR-search API mode. In that case you should see log lines like this:
Number of resources for [RESOURCE_TYPE] search is [NUMBER]
Can you please  check your logs and see what the number is for each resource type?

Sure. Here's the command and grep of that output
ryzen $ java -Xmx8192M -cp ./pipelines/batch/target/batch-bundled-0.1.0-SNAPSHOT.jar com.google.fhir.analytics.FhirEtl --fhirServerUrl=http://localhost:8080/fhir --outputParquetPath=/tmp/parquet/ --resourceList=Patient,Encounter,Observation,Condition,DiagnosticReport,MedicationStatement --batchSize=10 --jdbcModeHapi=false --jdbcModeEnabled=false | grep "Number of resources"
18:21:09.641 [main] INFO  c.g.fhir.analytics.FhirSearchUtil com.google.fhir.analytics.FhirSearchUtil.createSegments:239 - Number of resources for Patient search is 1639
18:21:55.841 [main] INFO  c.g.fhir.analytics.FhirSearchUtil com.google.fhir.analytics.FhirSearchUtil.createSegments:239 - Number of resources for Encounter search is 128015
18:22:42.283 [main] INFO  c.g.fhir.analytics.FhirSearchUtil com.google.fhir.analytics.FhirSearchUtil.createSegments:239 - Number of resources for Observation search is 571842
18:23:28.373 [main] INFO  c.g.fhir.analytics.FhirSearchUtil com.google.fhir.analytics.FhirSearchUtil.createSegments:239 - Number of resources for Condition search is 74596
18:24:14.496 [main] INFO  c.g.fhir.analytics.FhirSearchUtil com.google.fhir.analytics.FhirSearchUtil.createSegments:239 - Number of resources for DiagnosticReport search is 60200
18:24:14.503 [main] INFO  c.g.fhir.analytics.FhirSearchUtil com.google.fhir.analytics.FhirSearchUtil.createSegments:239 - Number of resources for MedicationStatement search is 0

Do you get any error messages? Is it possible to share the whole pipeline logs?

The log is huge. One repeated error is this:
11:56:57.305 [direct-runner-worker] ERROR c.g.fhir.analytics.FhirSearchUtil com.google.fhir.analytics.FhirSearchUtil.searchByUrl:70 - Failed to search for url: http://localhost:8080/fhir?_getpages=765a4163-6ec6-4987-906e-28e964ea17b5&_getpagesoffset=98170 ;  Exception: ca.uhn.fhir.rest.server.exceptions.InternalErrorException: HTTP 500 : Could not open JPA EntityManager for transaction; nested exception is org.hibernate.exception.JDBCConnectionException: Unable to acquire JDBC Connection

The command used is:
java -Xmx8192M -cp ./pipelines/batch/target/batch-bundled-0.1.0-SNAPSHOT.jar com.google.fhir.analytics.FhirEtl --fhirServerUrl=http://localhost:8080/fhir --outputParquetPath=/tmp/parquet/ --resourceList=Patient,Encounter,Observation,Condition,DiagnosticReport --batchSize=10 --jdbcModeHapi=false --jdbcModeEnabled=false

So, I am not sure why there would be a JDBC error when JDBC is explicitly not being used. It should be only using FHIR search.

The latest run gave only Condition and Patient. Here's the first 1000 lines of stdout.
https://pastebin.com/kyYfKq7n

The log is huge. One repeated error is this:
11:56:57.305 [direct-runner-worker] ERROR c.g.fhir.analytics.FhirSearchUtil com.google.fhir.analytics.FhirSearchUtil.searchByUrl:70 - Failed to search for url: http://localhost:8080/fhir?_getpages=765a4163-6ec6-4987-906e-28e964ea17b5&_getpagesoffset=98170 ;  Exception: ca.uhn.fhir.rest.server.exceptions.InternalErrorException: HTTP 500 : Could not open JPA EntityManager for transaction; nested exception is org.hibernate.exception.JDBCConnectionException: Unable to acquire JDBC Connection

This error is coming from the FHIR server, i.e., when we are fetching resources through the FHIR API (not JDBC), the server returns the above 500 code. It seems to be caused by DB connection starvation on the FHIR server side (HAPI?). My guess is that the size of the connection pool is bigger than what the DB supports by default. Can you share the configuration of your HAPI server? Also if you look at the error logs that you get from HAPI, it may tell us more what the problem is.

The command used is:
java -Xmx8192M -cp ./pipelines/batch/target/batch-bundled-0.1.0-SNAPSHOT.jar com.google.fhir.analytics.FhirEtl --fhirServerUrl=http://localhost:8080/fhir --outputParquetPath=/tmp/parquet/ --resourceList=Patient,Encounter,Observation,Condition,DiagnosticReport --batchSize=10 --jdbcModeHapi=false --jdbcModeEnabled=false

The other issue that I see with this command is that you are using the default "DirectRunner" which is not recommended for production use (we mostly use it for test purposes). If you have a large amount of data, please use --runner=FlinkRunner.
Also is there a particular reason that you don't use the pipeline/controller app and instead run the pipeline directly from the command line? It is fine if you prefer that, I am just curious because the controller app takes care of some of these details.

I redid my setup and have the following error:
19:03:30.163 [CHAIN DataSource (at Create.Values2/Read(CreateSource) (org.apache.beam.runners.flink.translation.wrappers.SourceInputFormat)) -> FlatMap (FlatMap at FetchResources2/ParDo(Search)/ParMultiDo(Search)) -> FlatMap (FlatMap at FetchResources2/ParDo(Search)/ParMultiDo(Search).output) (1/2)#0] ERROR c.g.fhir.analytics.FhirSearchUtil com.google.fhir.analytics.FhirSearchUtil.searchByUrl:70 - Failed to search for url: http://localhost:8080/fhir?_getpages=19680b29-323f-4b39-921f-79a109ce9231&_getpagesoffset=625754 ;  Exception: ca.uhn.fhir.rest.server.exceptions.ResourceGoneException: HTTP 410 : HAPI-0417: Search ID "19680b29-323f-4b39-921f-79a109ce9231" does not exist and may have expired

Here are the steps that I took:

Clone and generate 1000 patients with Synthea.
Start HAPI, e.g. docker run -p 8080:8080 hapiproject/hapi:latest
Load the patient bundles into HAPI. This takes a while.
Ensure Java 17 is being used by setting env classpath
Build the JAR file for pipelines. No errors. All tests pass.
Run pipelines, e.g. java -Xmx8192M -cp ./pipelines/batch/target/batch-bundled.jar com.google.fhir.analytics.FhirEtl --fhirServerUrl=http://localhost:8080/fhir --outputParquetPath=/tmp/parquet/ --resourceList=Patient,Encounter,Observation,Condition,DiagnosticReport --batchSize=2 --jdbcModeHapi=false --jdbcModeEnabled=false --runner=FlinkRunner --parallelism=2

I need to access the raw parquet files to import them into duckdb. When I use the docker compose method in the docs it hangs after 13% and doesn't recover in several hours.

The problem you have faced is known, please see this issue and in particular the "expiry" problem. The root cause is that HAPI keeps a list of resources associated to a search and that expires after a certain time (I think by default it is one hour). That's why we want to implement better API based solutions (i.e., the above issue).
A few points:

If your FHIR server is HAPI, can you try the JDBC mode instead? For that you need to provide the HAPI DB --fhirDatabaseConfigPath (like this) and enable --jdbcModeHapi.
You are setting --parallelism=2; this means only two threads will be used. If you have more cores on your machine, please consider increasing parallelism.
Re. the controller not making any progress, if you can share the configuration and some logs of the controller, that would be helpful.


Can't change mocked module with conditional exports

Describe the bug
Hi, I can't mock the constants imported from the local module, that are used in the conditional exports. See the code examples below for more context. If I run these tests, it will throw the error:
ReferenceError: Cannot access 'mockIsDev' before initialization
 ❯ Object.get IS_US [as IS_US] paths.test.ts:5:27
      4| const mockIsDev = vi.fn().mockReturnValue(false);
      5|
      6| vi.mock('./constants', () => ({
       |                          ^
      7|   get IS_US() {
      8|     console.log('inside', mockIsDev());
 ❯ eval paths.ts:3:27

Here is the simplified version of the failing code. We use conditional exports to reduce the bundle size of our application.
./constants.ts
export const IS_US = process.env.REGION === "US"

./paths.ts
import { IS_US } from './constants';

export const signInPath = IS_US ? () => '/sign-in' : () => '/login';

./paths.test.ts
import { vi, test, expect } from 'vitest';
import { signInPath } from './paths';

const mockIsDev = vi.fn().mockReturnValue(false);

vi.mock('./constants', () => ({
  get IS_US() {
    console.log('inside', mockIsDev());
    return mockIsDev();
  },
}));

test('path is correct for US', () => {
  mockIsDev.mockReturnValue('hello');
  expect(signInPath()).toBe('/sign-in');
});

I have no idea how to test it, I tried using functions as they're also hoisted, nothing really helps. Would appreciate if you could help me to solve this issue 🙏
Reproduction
https://stackblitz.com/edit/vitest-dev-vitest-wknbcs?file=paths.ts
System Info
System:
    OS: macOS 13.0
    CPU: (8) arm64 Apple M1 Pro
    Memory: 57.09 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 19.1.0 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - ~/.yarn/bin/yarn
    npm: 8.19.3 - /opt/homebrew/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 107.0.5304.110
    Firefox: 106.0.5
    Safari: 16.1
  npmPackages:
    @vitejs/plugin-react: ^2.2.0 => 2.2.0
    vitest: ^0.25.6 => 0.25.6

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

So far, I've found several solutions:

Declare mockIsDev before importing the signInPath. That way module will be mocked BEFORE it's imported
Declare mockIsDev in separate file and import before the signInPath. Works in the same way as 1
In a separate file, reassign vi.mock to const mockModule = vi.mock.bind(vi) and use carefully call in the right place, it won't be hoisted. I end up with this solution and declared the generic function to mock constants module, as I'll be reusing it a lot, so now I just import mockConstants and mocked functions (such as mockIsDev) and call mockConstants before importing the module with conditional export (e.g. ./paths)

If you find a better ways to resolve this problem, please let me know! 🙏

vi.mock is hoisted, so you don't have access to variables defined in global scope.
You can use vi.doMock which does the same, but is not hoisted. But you will need to load module asynchronously after.


Popup with IsOpen set to True in xaml causes exception at runtime

When setting IsOpen="True" for a Popup, it throws because the visual tree is still initializing I think. Not sure if it's supposed to work.
Repro
<Popup IsOpen="True">
    <TextBlock Text="Hello World!" />
</Popup>

If placed as a direct child of Window, it doesn't show anything. If placed deeper in the tree, e.g.:
<Window ...>
    <StackPanel>
        <Popup IsOpen="True">
            <TextBlock Text="Hello World!" />
        </Popup>
    </StackPanel>
</Window>

it throws:
System.InvalidOperationException: Attempted to open a popup not attached to a TopLevel

If set manually in the Window constructor, after InitializeComponent (AvaloniaXamlLoader.Load(this)) it works as expected.

It's not supposed to work when window isn't yet shown.

This works perfectly fine under WPF

probably it needs to not throw an exception and defer opening until window is opened (attached to visual tree?)

I think attaching to visual tree doesn't work. I've stumbled upon this some time ago by trying to open combobox in datagrid when row is selected in editorControl.OnAttachedToVisual(Logical)Tree, and it still throws. But maybe that was just DataGrid issue.

@kekekeks what you think about not throwing exceptions and displaying popup when attached to visual tree?

That should work, yes.

Note that we also need to wait until window is completely presented, otherwise it might still fail to show the popup properly.

Just tested it and looks like waiting for "AttachedToVisualTree" is enough.
Also the logic I implemented is following:

IsOpen/Open call may open popup, or schedule opening if window is not there.
If scheduled, popup will open when attached to visual tree
Scheduling is cleared when popup is opened or when closed was requested.

Just tested it and looks like waiting for "AttachedToVisualTree" is enough.

wont be enough once we add wayland backend

I don't really have access to machine with wayland backend, could you guide me on how I should check whether window is ready to display?


Binding Ctrl-h to some action prevent backspace to be working fine

In my config file I have Ctrl-h bound to the previous-window command. Unfortunately having this setting makes also the backspace key triggering this command. This behavior prevent me to actually using backspace for what it is meant for.
You mention that ctrl-h and backspace are somehow the same key sequence. However in tmux I do have the similar setting and it works. So it might be possible to have pymux override this default behavior and adopting something similar to what tmux does.
Here is an excerpt from my config file
bind-key -n C-l next-window
bind-key -n C-h previous-window

Hi @jczulian,
I just fixed the issue in this commit of prompt-toolkit: https://github.com/jonathanslenders/python-prompt-toolkit/commit/b586e477c374a78dd1808ac9818ea6be60b87ccc
If you have time, can you try to checkout the latest master branch of prompt-toolkit, install that and try again. If it doesn't work, we'll see tomorrow.
Cheers,
Jonathan

Hi @jonathanslenders,
I have just tried it, works perfectly. Thank you very much, I can now get back to using pymux instead of using tmux.
Keep the good work and thanks a million for your help.


Create Django API endpoint for database reset and integrate Cypress before hook

will look into the possibility once the new restructuring of forms are merged


BLE_HID_DEVICE example error  (IDFGH-10957)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hello I am trying to build something using bluetooth/esp_hid_device example. But when I am try to build this I got undefined reference to `esp_bt_gap_set_cod' error. I define to enable hid_device
#define CONFIG_BT_HID_DEVICE_ENABLED 1
Thanks for your helps
I am using ESP32s3 devkitc-1.

@monurbstnc
thanks for report the issue,
I am using the master branch (commit a1c0d9c3a1d1bea876f57c9229065ecd7f048b33), and everything is working fine.
What version of ESP-IDF are you using?

@monurbstnc
Can you provide me with the complete log, please?

I solve the problem, thanks for the helps.


Revert "Resolve tenant admin user id from realm config only if user id exist in the realm"

Reverts wso2/carbon-kernel#3529

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/4413385598

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/4413385598
Status: success


Add Olawale to Rookie list

Description
Welcome! We are excited to have you on board. This is your first formal step in making it official that you are a part of the Polkadot Heroes program as either a hero or a rookie. Please fill out your profile details in this pull request. Remember to place your profile in alphabetical order in the list, add your profile image to the ./images subfolder of either the Heroes/ or Rookies/ folder, and fill out all the informational fields in your card. You can copy and paste someone else's card and replace their information with your own.
Make sure to preview your additions in the Heroes/README.md or the Rookies/README.md file to confirm it looks as expected.
Checklist:

[x] My name is alphabetically ordered by first name in the README
[x] My profile picture is added to the /images folder of either the Heroes/ or the Rookies/ directory
[x] All the fields are filled out for my profile (see the other profiles for examples)

Add Olawale Rookie list

check all 3 tasks

Hi @lexycole!
Thanks for opening the PR to add yourself.
Can you please follow the instructions here and also update the Rookies/README.md file to include your information:

Add your Profile Card
Create your profile card in the appropriate README.md file (either Rookies/README.md or Heroes/README.md), following the provided template or copy and paste an earlier profile card of another member to use for reference. Make sure your profile is in alphabetical order in the list.


feat:(index) add case for boolean field

I noticed that when I have a boolean field and I include it in the pane, I get "Untitled" if the field is true, otherwise I get blank.
This commit makes it render "true", "false" or leaves it blank if it is not defined.

Can we merge this?

Can we merge this?

Let me know if I need to make it not-a-draft?


docs: Update ReadMe now that musl is not needed; support Linux ARM64

What ❔

Update ReadMe now that musl is not needed.
Note support for Linux ARM64.
Harmonize case for musl and ARM64.

@FlashSheridan , I believe we need to port this to zkvyper as well?


How to run SV3D_u on MacBook with M1 chip

To run SV3D_u on a single image I followed the instruction to download the sv3d_u.safetensors and run python scripts/sampling/simple_video_sample.py but got the errors below and realized xFormers is not supported on Mac.
Was anyone able to find a workaround to run it on Mac?
OMP: Info #276: omp_set_nested routine deprecated, please use omp_set_max_active_levels instead.
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/torchvision/io/image.py:13: UserWarning: Failed to load image Python extension: dlopen(/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/torchvision/image.so, 0x0006): Symbol not found: __ZN3c1017RegisterOperatorsD1Ev
  Referenced from: <61623A3D-DA3C-3AAD-B2F0-D363151DDB3F> /Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/torchvision/image.so
  Expected in:     <03BFC270-EF54-36FC-B102-371F5164D3C9> /Users/alisonqiu/Library/Python/3.10/lib/python/site-packages/torch/lib/libtorch_cpu.dylib
  warn(f"Failed to load image Python extension: {e}")
WARNING[XFORMERS]: xFormers can't load C++/CUDA extensions. xFormers was built for:
    PyTorch 1.11.0 with CUDA None (you have 2.3.0)
    Python  3.10.1 (you have 3.10.1)
  Please reinstall xformers (see https://github.com/facebookresearch/xformers#installing-xformers)
  Memory-efficient attention, SwiGLU, sparse and more won't be available.
  Set XFORMERS_MORE_DETAILS=1 for more details
Traceback (most recent call last):
  File "/Users/alisonqiu/Downloads/React/generative-models/scripts/sampling/simple_video_sample.py", line 349, in <module>
    Fire(sample)
  File "/Users/alisonqiu/Library/Python/3.10/lib/python/site-packages/fire/core.py", line 143, in Fire
    component_trace = _Fire(component, args, parsed_flag_args, context, name)
  File "/Users/alisonqiu/Library/Python/3.10/lib/python/site-packages/fire/core.py", line 477, in _Fire
    component, remaining_args = _CallAndUpdateTrace(
  File "/Users/alisonqiu/Library/Python/3.10/lib/python/site-packages/fire/core.py", line 693, in _CallAndUpdateTrace
    component = fn(*varargs, **kwargs)
  File "/Users/alisonqiu/Downloads/React/generative-models/scripts/sampling/simple_video_sample.py", line 98, in sample
    model, filter = load_model(
  File "/Users/alisonqiu/Downloads/React/generative-models/scripts/sampling/simple_video_sample.py", line 340, in load_model
    model = instantiate_from_config(config.model).to(device).eval()
  File "/Users/alisonqiu/Downloads/React/generative-models/sgm/util.py", line 175, in instantiate_from_config
    return get_obj_from_str(config["target"])(**config.get("params", dict()))
  File "/Users/alisonqiu/Downloads/React/generative-models/sgm/models/diffusion.py", line 48, in __init__
    model = instantiate_from_config(network_config)
  File "/Users/alisonqiu/Downloads/React/generative-models/sgm/util.py", line 175, in instantiate_from_config
    return get_obj_from_str(config["target"])(**config.get("params", dict()))
  File "/Users/alisonqiu/Downloads/React/generative-models/sgm/modules/diffusionmodules/video_model.py", line 153, in __init__
    linear(model_channels, time_embed_dim),
  File "/Users/alisonqiu/Downloads/React/generative-models/sgm/modules/diffusionmodules/util.py", line 296, in linear
    return nn.Linear(*args, **kwargs)
  File "/Users/alisonqiu/Library/Python/3.10/lib/python/site-packages/torch/nn/modules/linear.py", line 98, in __init__
    self.weight = Parameter(torch.empty((out_features, in_features), **factory_kwargs))
  File "/Users/alisonqiu/Library/Python/3.10/lib/python/site-packages/torch/utils/_device.py", line 78, in __torch_function__
    return func(*args, **kwargs)
  File "/Users/alisonqiu/Library/Python/3.10/lib/python/site-packages/torch/cuda/__init__.py", line 284, in _lazy_init
    raise AssertionError("Torch not compiled with CUDA enabled")
AssertionError: Torch not compiled with CUDA enabled

+1


loadTexture() not firing txLoaded event inline unless at least one animation loop has occurred

We have encountered this problem in our code, where we intend to access the width of the rendered text, and to do that we force it to loadTexture(). For consistency reasons, we rely on txLoaded event to set the renderWidth of the element, in this case we access it immediately after executing loadTexture(). However, it does not work as expected because txLoaded fails to fire inline the first time. This code from this jsfiddle best explains the problem.
class LoadTextureExample extends lng.Application {
	static _template() {
		return {
    	Text1:{ x: 0, y: 0 },
    }
	}

  _init() {
  		let renderWidth;
  		this.tag('Text1').on('txLoaded', () => {
      	renderWidth = this.tag('Text1').renderWidth;
      	console.log("Text1 loaded");
        });

      requestAnimationFrame(() => { // Frame 1
      	console.log('txLoaded does not execute in between 1 and 2')
        this.tag('Text1').patch({ text: { text: 'Text one', textColor: 0xffffffff }});
        console.log(1);
        this.tag('Text1').loadTexture(); // console logs Text1 loaded after 2 and not inline, bug?
        console.log('renderWidth', renderWidth); // as a result this is undefined
        console.log(2);

        requestAnimationFrame(() => { // Frame 2
          console.log('at least one render loop has occurred, now txLoaded executes inline');
          this.tag('Text1').patch({ text: { text: 'Text one one', textColor: 0xffffffff }});
          console.log(1);
          this.tag('Text1').loadTexture(); // now console logs Text1 loaded inline i.e. before 2
          console.log('renderWidth', renderWidth); // this is 230, as expected
          console.log(2);
        });
      });
    }
}

const options = {
	stage: {
		w: 250,
		h: 250,
		clearColor: 0xFF000000,
		useImageWorker: false
	},
	debug: true
}
const app = new LoadTextureExample(options);
document.body.appendChild(app.stage.getCanvas());

Hi @hiratariq! Sorry for the delay. Much of the team has been on vacation.
Unfortunately, this is by design. We do not call txLoaded for a texture until it and its Element are "Activated" (see Lifecycle Events). This activation process does not occur until the first render loop.
If you're using loadTexture() in this way, you can actually access the w/h of the texture directly:
class LoadTextureExample extends lng.Application {
	static _template() {
		return {
    	Text1:{ x: 0, y: 0 },
    }
	}

  _init() {
    console.log('txLoaded does not execute in between 1 and 2')
    this.tag('Text1').patch({ text: { text: 'Text one', textColor: 0xffffffff }});
    this.tag('Text1').loadTexture();
    console.log('renderWidth', this.tag('Text1').texture.source.w);
    console.log('renderHeight', this.tag('Text1').texture.source.h);
  }
}

https://jsfiddle.net/uxdnr1zt/


Schedule.On doesn't trigger during warmup (initiated by SetWarmup)

There's currently no way to have scheduled events trigger during warmup. This is a problem because often scheduled events drive algorithm logic. Consequently, if you want to use warmup functionality, you must replace scheduled events with some kind of replacement on your own. While not an enormous  project it introduces an additional risk for bugs.
The most straightforward fix would be to simply make scheduled events happen during warmup, the problem is that's a subtle interface breaking change on existing code. Therefore, maybe a boolean property or some kind of parameter in scheduled events methods is better to enable this behavior.

Thanks @4real -- will look into making the behavior the same for warm up.

Great news. I was trying to reproduce scheduled events manually to get the warm up phase. But like @4real said, it's quite messy to manually add handlers in OnData for this.


Add support to create / update / delete custom roles for organisations

API can be found here - https://docs.github.com/en/rest/orgs/custom-roles#create-a-custom-role
I plan on working on this feature

Hi @gmlewis , could we have this released? We would like to leverage the new API in the terraform github provider.

Hi @gmlewis , could we have this released? We would like to leverage the new API in the terraform github provider.

Yes, I'll work on a new release.

This has been released here: https://github.com/google/go-github/releases/tag/v48.0.0


Self-documenting ChefSpec test

This default ChefSpec test is more idiomatic and does a better job of revealing the intent of the test.

I concede that phrases like "more idiomatic" and "better job of revealing the intent" are clear indicators that this PR is akin to shaving a yak.
I could also see an argument against this change as writing ChefSpec tests won't usually involve passing a block to expect.

I'm cool with it. :+1:

:shipit:

Merged as e117637aab7caf08def07584ca502f279c9dc8bf
Thanks!


Sort individual feature importance by residual for regression

This PR orders the samples in individual feature importance by abs(true_y - predicted_y) for regression. It will show the predictions which are better first than the prediction which are way off in the model.
Description
Before: sort by index

After: sort by abs(true_y - predicted_y)

Checklist

[x] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #1487 (aa427db) into main (93b9385) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1487   +/-   ##
=======================================
  Coverage   89.30%   89.30%
=======================================
  Files          38       38
  Lines        1617     1617
=======================================
  Hits         1444     1444
  Misses        173      173

Flag
Coverage Δ

unittests
89.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

@tongyu-microsoft I'm curious what prompted this change. Was this a feature request from somebody?
As a user, I'd be a bit surprised about the ordering since we had it by index so far and that's immediately obvious. The residual ordering is not immediately obvious unless we point it out in writing. Perhaps that should be added? If you haven't yet you may want to consult with a designer 🙂

@tongyu-microsoft I'm curious what prompted this change. Was this a feature request from somebody? As a user, I'd be a bit surprised about the ordering since we had it by index so far and that's immediately obvious. The residual ordering is not immediately obvious unless we point it out in writing. Perhaps that should be added? If you haven't yet you may want to consult with a designer 🙂

@romanlutz, I think sorting based on some regression metric might be useful for understanding as to why model might be predicting with higher accuracy for some samples and doesn't predict the same way for other samples. For classification scenario as well, we differentiate between which samples were correctly predicted by model vs which were not correctly predicted. Agree, that we should document this in the dashboard and should use some standard regression metric like r2_score to order these.

I think it would also be nice to see the values by which these are sorted as a separate column.  If they are sorted by abs(true_y - predicted_y), it might be nice to have a columns with name something like "Abs difference" right after the index, so it's clear that this is what the data is sorted by.  It doesn't necessarily have to be part of this PR though.


[WIP] avoid metaprogramming for tx creation.

The tx object has everything needed to avoid having to do metaprogramming to create a new tx object. Because the buffers are not copied the amount of memory used is basially the same.
also, I removed the try catch inside the toBuffer, because it was enmascarating errors. Basically by transforming everything into an Error is hard to handle speific situations.

Looks much easier to read! Thank you

I did the metaprogramming approach initially because we only had one call: Object.setPrototypeOf(transaction, Transaction.prototype). But then we needed to add Input and Output, and the Script class, and the code grew. This is a ncie change. If someone needs more performance, decodeTx is there.

One question - many of the other classes have Cannot create X: message errors for their constructors. Do you find that error too verbose? I think that if we remove that here, we should just remove it everywhere. What do you think?

Oh, I like the error message. I just don't like absorbing all the possible errors. Because if there is an error that can be handled outside in some way it gets a lot harder if every error gets converted into new Error(cannot create tx: ${e.message}. For example, if the original error had some extra property it gets lost. The same with the class.

LGTM


Communes are not always LAU2

The dataset you use to inform the 'LAU2'/'commune' spatial resolution is not strictly 2013 LAU2. For instance, the data for the UK is more correctly 'wards'. The IDs for these wards don't always match LAU2 and, in the case of Scotland and Northern Ireland, their boundaries don't match LAU2 either. I suspect that all the communes with None values in the TRUE_COMM_ column are not what they seem, i.e.:

This isn't a problem for your paper's analysis, since you don't aggregate based on IDs, but rather by using spatial joins. I only noticed the issue when trying to match known electricity consumption in the UK, at the LAU1 resolution to data coming out of this. I thought would be easiest (ha!) to do it by aggregating using LAU2 -> LAU1 correspondence tables).

Thanks for pointing this out. The data is unfortunately very confusing. Codes and classification change often, and are somehow inconsistent. For example, I also use the DEGURBA dataset for LAU 2013 data in the analysis, which matches mostly, but not always the codes. Back when I did the analysis, this was the only existing dataset available. I was always hoping eurostat would publish a newer dataset (based on the new classification in which there is only one LAU level left), checked regularly their website -- but nothing.
I looked it up once more just now, and indeed they published a newer dataset last June. So if there is reason to, one could update the analysis with LAU 2018 and NUTS 2016.

Yeah, I agree that it is a total mess of codes (just the UK has about 5 different ways of classifying areas of a similar size and their codes/boundaries change at different points in time to each other). The unified LAU from 2017 onwards would be possible, but it's a closer match to LAU1 than LAU2 (e.g. ~400 UK areas instead of ~10,000). Would this pose a problem, do you imagine?

No, that should be straightforward (as in: you update the URL, rerun the workflow, it breaks, and you start debugging 😜).
But seriously, the only thing that comes to my mind is the DEGURBA dataset, which wouldn't match anymore (which is necessary only for the analysis of the potentials, that you may not be interested in). And most probably the pre-processing of the LAU data would need to be updated, in particular the mapping of column names.

Yeah, I wouldn't expect it to be too much of a problem programmatically, but do we care about the super-high resolution of 2013 'LAU2' beyond the study you undertook in this paper?
RE DEGURBA, the latest correspondence tables include this classification, so updating to 2018 would work for LAU & DEGBURA, provided we're happy with the somewhat coarser resolution

Whether or not we care about the resolution depends on the use case. If we are talking about euro-calliope, I'd say coarser is better (and especially newer is better).

Agreed. I'll update the finest resolution to LAU2017/2018 after/if we make the repository split discussed in #3


rhel: Installation of CC 3.0 on RHEL systems

This commit includes a script based in centos installs scripts to
install clear containers 3.0 in RHEL systems.
This guide will install Docker CE from CentOS repos and extra
requeriment from CentOS repositories.
This document is not intended to be run in production systems.
Fixes: #463
Depends on: #372
Signed-off-by: Geronimo Orozco<EMAIL_ADDRESS>

Coverage remained the same at 89.402% when pulling fad8d3e93c51accf3707c326ce28c602b882b057 on gorozco1:rhel-install-doc into 2b2602d93c7aad1fef1166abff085d2685745c95 on clearcontainers:master.

@gorozco1 - one further thing to update is the final paragraph inversions.txt which (on #372) is documented to be for CentOS only. Once #372 lands, this PR should probably also mention RHEL to avoid any confusion.

Based on discussion from dev meeting today this is not a valid scenario to install cc in rhel we need to rework and fine tuning with docker-ce


Add Core class

closes #4

This looks hype, the comments are so well written. Left one nitpicky comment that's not super important. How did you choose which axis corresponds to which number?


Missing method "find_with_linked"

Motivation
We have find_with_related and find_also_linked, but there's no find_with_linked.
For instance, I want to get all users who are in a group whose name is "admin" (not primary key). I have a "membership" table for the N-N relation. I would want to write:
User::find().find_also_linked(Group).filter(Group::Column::Name.eq("admin")).all(db).await?;

Then there's also the question of inner join vs left join. I know the answer is probably going to be pointing me to https://www.sea-ql.org/SeaORM/docs/advanced-query/custom-select/, but it's just a seemingly missing part of the API.

Somewhat related, but I'm looking to get a Vec<(User, Vec<Group>)>.

Actually, I was looking at the custom-select page, and even with custom joins and so on, I can't figure out how to get a Vec<(User, Vec<Group>)>: You can't derive FromQueryResult for a struct with Vec inside, and I'm not sure how to aggregate across rows.

So, from what I understand, the only way to do it given the current code (if we want to leverage the link) is to use find_also_linked and then post process the results: we'll get a (User, Some(Group)) for each pair of (user, group), plus a (User, None) for users without groups. If we sort by user, they'll be correctly grouped and we can do some stream processing. Not the prettiest but it works. I still feel like this should be done by the framework.

I think this omission is not intentional and this can be added.
@billy1624 I think it should be a matter of refactoring into prepare_select_two_for_linked and construct a SelectTwoMany?

Closed by https://github.com/SeaQL/sea-orm/pull/1743


[CI] Codecov.io failing with 404

Search before asking

[X] I searched in the issues and found nothing similar.

Example failure
https://github.com/devinbost/pulsar/actions/runs/4494617987/jobs/7907322924
Exception stacktrace
[2023-03-22T22:25:28.804Z] ['error'] There was an error running the uploader: Error uploading to https://codecov.io/: Error: There was an error fetching the storage URL during POST: 404 - {'detail': ErrorDetail(string='Could not find a repository, try using repo upload token', code='not_found')}
[2023-03-22T22:25:28.804Z] ['verbose'] The error stack is: Error: Error uploading to https://codecov.io/: Error: There was an error fetching the storage URL during POST: 404 - {'detail': ErrorDetail(string='Could not find a repository, try using repo upload token', code='not_found')}
at main (/snapshot/repo/dist/src/index.js)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
[2023-03-22T22:25:28.804Z] ['verbose'] End of uploader: 1651 milliseconds
. . .
[2023-03-22T22:26:31.441Z] ['error'] There was an error running the uploader: Error uploading to https://codecov.io/: Error: There was an error fetching the storage URL during POST: 404 - {'detail': ErrorDetail(string='Could not find a repository, try using repo upload token', code='not_found')}
[2023-03-22T22:26:31.442Z] ['verbose'] The error stack is: Error: Error uploading to https://codecov.io/: Error: There was an error fetching the storage URL during POST: 404 - {'detail': ErrorDetail(string='Could not find a repository, try using repo upload token', code='not_found')}
at main (/snapshot/repo/dist/src/index.js)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
[2023-03-22T22:26:31.442Z] ['verbose'] End of uploader: 1564 milliseconds
Error: Codecov: Failed to properly upload: The process '/home/runner/work/_actions/codecov/codecov-action/v3/dist/codecov' failed with exit code 255
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I'm getting this repeatedly upon multiple re-runs of the CI.

This can be referred to https://github.com/apache/pulsar/pull/19947. I'm not sure if it's before or after the PR merged.
cc @lhotari

For the upload issue, Codecov now recommends to [specify an upload token for the build]((https://community.codecov.com/t/upload-issues-unable-to-locate-build-via-github-actions-api/3954):

Public repositories that rely on PRs via forks will find that they cannot effectively use Codecov if the token is stored as a GitHub secret. The scope of the Codecov token is only to confirm that the coverage uploaded comes from a specific repository, not to pull down source code or make any code changes.
For this reason, we recommend that teams with public repositories that rely on PRs via forks consider the security ramifications of making the Codecov token available as opposed to being in a secret.

I'll first create a ticket to ASF infra for creating the token for apache/pulsar. After the token is available, I can work on an improvement where this token is used for builds and PRs that target apache/pulsar. For builds in own forks, I'll add instructions for setting that up.

I created https://issues.apache.org/jira/browse/INFRA-24399 for the creation of the Codecov upload token for apache/pulsar.

INFRA-24399 closed with The ASF does not have a business relationship with CodeCov, we do not pay for their services. If you want to use CodeCov, you are limited to their free offering.
Perhaps nothing we can do from our side :/

Closing as Won't Fix on INFRA ticket.
If anyone has ideas to ease the case, please reopen and submit a patch.


Resizing event of the chart

In the examples page, when you change from an example to another and try to resize the graph, multiples exceptions are thrown in the console: Error: Invalid value for  attribute transform="translate(NaN,5)". Seems that the resize event is not properly deleted.

I know this is an old thread, but I just ran into this issue, so I'm guessing it hasn't been addressed. Upon checking the original nvd3 code (version 1.7.1), I found that the nv.utils.windowResize function actually returns an object with a clear() method that unregisters the  events . However, this object is never used by the angular directive.
I forked my own version and messed with the code for a possible solution (PR #79)

This was merged


Problems running app not launching into NativeScript after qrcode and app setup there is a build failure.

Describe the bug
Created app across platforms as requested. Originally requested for next.js but not displaying in preview window other than qrcode and request to install NativeScript app.
Have done this and scanned qrcode, appears to start to work then fails on build.
Have tried several times but same issues.
Some errors shown leading to this...
I notice there are some dependency issues that need to be fixed. Let me update the package.json with the correct dependencies and ensure everything is properly configured.
Let me help you fix this issue. The main problem is that we need to properly set up the Next.js application structure and ensure all required files are in place.
I notice that we're trying to run a Next.js application in a NativeScript environment. This is causing the conflict since these are two different frameworks with different purposes. Let me help you create a proper cross-platform solution using NativeScript with React.
Let me help you fix the preview bundle loading issue. The error occurs because we need to ensure the NativeScript project structure is correct and properly configured. Let's update the necessary files:
Let me help fix the preview bundle loading issues. We need to simplify the project structure and ensure it's properly configured for NativeScript. I'll update the necessary files:
I've upgraded to PRO because I ran out of credit for today because this has failed so many times.
This is the current screen...you can see there's nothing again

Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-gfhizj
Steps to reproduce
I don't think you need me to explain - the above explanation should be enough
Expected behavior
I'm not getting anything yet - just want to see something
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 128
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 51517109,
  "usedJSHeapSize": 49312481,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = shawnurquhart
Chat ID = 5b42860c1a76

Additional context
No response

This is a screen shot from the NativeScript android app - after scanning the qrcode, it appeared to iteract with the browser installation and display the android device connected but an error on the phone suggesting a build error and that was it.


CircularReplication with 3 shard / 2 replicas on a 3 node cluster

Hi,
Can we set CircularReplication on a 3 node k8-cluster, with 3 shards and 2 replicas ?
similar to https://altinity.com/blog/2018/5/10/circular-replication-cluster-topology-in-clickhouse

Am trying with,
apiVersion: "clickhouse.altinity.com/v1"
kind: "ClickHouseInstallation"
metadata:
  name: "circular"
spec:
  defaults:
    templates:
      podTemplate: dist-template
  configuration:
    clusters:
      - name: "cluster"
        layout:
          shardsCount: 3
          replicasCount: 2
  templates:
    podTemplates:
      - name: dist-template
        podDistribution:
          - type: CircularReplication
        spec:
          containers:
            - name: clickhouse
              image: yandex/clickhouse-server:<IP_ADDRESS>

All pods except chi-circular-cluster-1-1-0 get attached,  1-1-0 gets stuck with,
Warning  FailedScheduling  <unknown>  default-scheduler
0/4 nodes are available:
1 node(s) didn't match pod affinity rules,
1 node(s) had taint {node-role.kubernetes.io/master: }, that the pod didn't tolerate,
2 node(s) didn't satisfy existing pods anti-affinity rules, 3 node(s) didn't match pod affinity/anti-affinity.

It does work if i use 3 shards and 3 replicas, but not for 2 replicas
Details

operator v0.12
AWS EKS / Kind: v0.18

Thanks !

@kushagra391 do you have k8s cluster with 1 master + 3 worker nodes? or 1 master + 2 workers?

i tested both on

kind: 1 master and 3 nodes
AWS EKS: 1 master with 12 nodes

on EKS am getting,
Warning  FailedScheduling  11s   default-scheduler
0/12 nodes are available:
10 node(s) didn't match pod affinity rules,
12 node(s) didn't match pod affinity/anti-affinity,
2 node(s) didn't satisfy existing pods anti-affinity rules.

and similar to kind, 3 * 3 setup works in EKS as well.

could you share
kubectl get sts -n <your_namespace> chi-circular-cluster-0-1 -o yaml
kubectl get sts -n <your_namespace> chi-circular-cluster-0-1 -o yaml
kubectl get sts -n <your_namespace> chi-circular-cluster-1-1 -o yaml
?
clickhouse-operator doesn't schedule stateful set itself, it just translates Kind: ClickHouseInstallation affinity rules to standart kubernetes pod affinity/anti-affinity rules via kube-scheduler and controller-manager

@kushagra391
also let's try another combination of pod distributions in ClickHouseInstallation
    podDistribution:
          - type: ShardAntiAffinity
          - type: ReplicaAntiAffinity
          - type: ClickHouseInstallationAffinity
          - type: PreviousTailAffinity

@Slach sure, i pasted all 3 outputs into https://gist.github.com/kushagra391/a1f16c7d7fc1ca852cc379f454bba719
am using kind, hopefully you can recreate this setup,
kind create cluster --config cluster-config.yaml

kubectl apply -f https://raw.githubusercontent.com/Altinity/clickhouse-operator/0.12.0/deploy/operator/clickhouse-operator-install.yaml
kubectl create ns clickhouse
kubectl -n clickhouse apply -f clickhouse_config.yaml

with the other set of combination ShardAntiAffinity / ReplicaAntiAffinity / ...
chi-circular-cluster-2-1-0 is getting stuck.
kubectl -n clickhouse2 get pods -o wide
NAME                         READY   STATUS    RESTARTS   AGE   IP             NODE           NOMINATED NODE   READINESS GATES
chi-circular-cluster-0-0-0   1/1     Running   0          77s   <IP_ADDRESS>   kind-worker3   <none>           <none>
chi-circular-cluster-0-1-0   1/1     Running   0          71s   <IP_ADDRESS>   kind-worker    <none>           <none>
chi-circular-cluster-1-0-0   1/1     Running   0          65s   <IP_ADDRESS>   kind-worker    <none>           <none>
chi-circular-cluster-1-1-0   1/1     Running   0          55s   <IP_ADDRESS>   kind-worker3   <none>           <none>
chi-circular-cluster-2-0-0   1/1     Running   0          44s   <IP_ADDRESS>    kind-worker2   <none>           <none>
chi-circular-cluster-2-1-0   0/1     Pending   0          31s   <none>         <none>         <none>           <none>

@kushagra391 did you try to change podDistribution: ?

i tried,
- type: ShardAntiAffinity
          - type: ReplicaAntiAffinity
          - type: PreviousTailAffinit

as a workaround, this is working.
I guess, this is close enough :)
        podDistribution:
          - type: ShardAntiAffinity
          - type: ReplicaAntiAffinity
          - type: MaxNumberPerNode
            number: 2

kubectl -n cht get pods -o wide
NAME                       READY   STATUS    RESTARTS   AGE   IP            NODE           NOMINATED NODE   READINESS GATES
chi-rollworks-test-0-0-0   1/1     Running   0          51s   <IP_ADDRESS>   kind-worker3   <none>           <none>
chi-rollworks-test-0-1-0   1/1     Running   0          40s   <IP_ADDRESS>   kind-worker    <none>           <none>
chi-rollworks-test-1-0-0   1/1     Running   0          29s   <IP_ADDRESS>   kind-worker2   <none>           <none>
chi-rollworks-test-1-1-0   1/1     Running   0          23s   <IP_ADDRESS>   kind-worker3   <none>           <none>
chi-rollworks-test-2-0-0   1/1     Running   0          18s   <IP_ADDRESS>   kind-worker    <none>           <none>
chi-rollworks-test-2-1-0   1/1     Running   0          7s    <IP_ADDRESS>   kind-worker2   <none>           <none>

glad to hear about a workaround


Failed to install turtlebot3_gazebo

Hi, I am a newbie to ROS. I tried to install the turtlebot3_simulations with the following steps:
On Ubuntu 18.04.1 LTS

installing full ros-melodic
setting up a catkin workspace
git clone turtlebot3, turtlebot3_msgs, turtlebot3_simulations
editing the turtlebot3_simulations/turtlebot3_gazebo/CMakeLists.txt from "DEPENDS gazebo" to "DEPENDS GAZEBO" (issue here?)
running catkin_make and sourcing the setup.bash

but I can only roslaunch
turtlebot3_bringup
turtlebot3_example
turtlebot3_navigation
turtlebot3_teleop
turtlebot3_description
turtlebot3_msgs
turtlebot3_slam
catkin_make throws following warnings:
CMake Warning (dev) at /usr/share/cmake-3.10/Modules/FindBoost.cmake:911 (if):
Policy CMP0054 is not set: Only interpret if() arguments as variables or
keywords when unquoted.  Run "cmake --help-policy CMP0054" for policy
details.  Use the cmake_policy command to set the policy and suppress this
warning.
Quoted variables like "chrono" will no longer be dereferenced when the
policy is set to NEW.  Since the policy is not set the OLD behavior will be
used.
Call Stack (most recent call first):
/usr/share/cmake-3.10/Modules/FindBoost.cmake:1558 (_Boost_MISSING_DEPENDENCIES)
/usr/share/OGRE/cmake/modules/FindOGRE.cmake:318 (find_package)
/usr/lib/x86_64-linux-gnu/cmake/gazebo/gazebo-config.cmake:175 (find_package)
turtlebot3_simulations/turtlebot3_gazebo/CMakeLists.txt:23 (find_package)
This warning is for project developers.  Use -Wno-dev to suppress it.
Is there a quick workaround for this?
Thanks for the help!

Encountered same problem

Hi @ryfr, @daniel9612 :)
I'm sorry for late reply.
I'm really sorry, but it does not support the melodic version yet. I will let you know when I release ros-melodic.
Thanks,
Gilbert

Ok, thanks.
Which is the last supported version?
Ros-lunar?

This issue will be closed since there were no actions for a while. You can reopen this issue to show this issue to the users whenever. Thanks.


Speed up write step

👋 @kkga
In todo.md, I saw:

use streams to open and write markdown files

This PR makes the write step run concurrently rather than writing each output file one-by-one.
In my testing, it saves a few seconds for very large projects.
There's no pressure to merge this, it's not blocking me, etc.
I'm just sharing it in case it's interesting!

@healeycodes This is great, thank you!


⚠ Samsung has degraded performance

In 1becde2, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2814 ms

Resolved: Samsung performance has improved in c829301.


Fix TSC diagnostic file parsing (handle parsing diagnostics with multiple blocks)

This patch fixes parts of the .dia file reading. As originally
implemented, the reader would fail and emit an error if a diagnostic had
sub-diagnostics, i.e, a warning with notes.
Part of the issue was that the parser could only handle one block at a
time and couldn't handle the nesting of blocks. After fixing this, it
became apparent that there were some other issues. The parser was
consuming the records in a post-order traversal of the diagnostic graph,
when it really needs to be handling them in a pre-order ordering. This
wasn't an issue as a post-order and pre-ordering are the same if you can
only handle trees with a root node. Emitted in the wrong order, the
filemap, flagmap, and categorymap wouldn't be updated correctly,
resulting on dropping diagnostic information on the floor, when really
the information would come later.
In the end, I recorded the diagnostic records in the pre-order traversal
order into the diagnosticRecords property. When exiting from the last
active blockID in a diagnostic tree, I would run over the lists of
records, emitting a diagnostic for each. This way we emit diagnostics in
a pre-order ordering once we have collected all of the information for
the diagnostics in the tree.
Now, there's even more fun to be had. When constructing the diagnostic,
the code made an assumption that filename records, category records, and
flag records would come strictly before the diagnosticInfo, source
range, and fixIt records. This isn't the case. To work around that, I
run over the records, handling all the filenames, categories, and flags
first. Once those have been populated into their respective maps, we
iterate over the records again, handling the diagnosticInfo, source
ranges, and fixIts with the fully populated maps. This should prevent us
from dropping pieces of diagnostics on the ground, simply because
they're in a weird order.
Noticed this issue after posting https://github.com/apple/swift/pull/59136.

@swift-ci please test

@swift-ci please test

This is great — thank you @etcwilde!  One future improvement here would be to fold the nested annotations in under each SerializedDiagnostics.Diagnostic (as a new SerializedDiagnostics.Diagnostic.Annotation type) so that client code can reason about top-level diagnostics as discrete issues, with the "here is the location of the previous declaration" etc annotations attached to the top-level issues.  That can be done in a follow-on PR, though, and I can see if I can help sketch that out if you want.


I need some help

I need some help, because i used but when i run de backend stop here and never continue

I loggin and get redirect but doesn't work

It's not a lot of information to work with. One thing that might be an issue when running it in docker is that you need to expose the 8080 port so that after authorization flow the response is sent to it.
Please use GitHub Discussions or a Discord for questions in the future.

Closing the issue. If you want to discuss it more, please do so in Discussions or Discord.


Comment extra YAML listing until they're available

While || true also works, it's easy to miss that and swallow errors in the future when more YAMLs are added.

/lgtm
/approve


Change the word 'layout' to 'delay'

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

https://developer.mozilla.org/en-US/docs/Web/Performance/Critical_rendering_path
In the documentation for CRP, there is an improper usage of a word in the 'Layout' section, 4th paragraph, 3rd line. It should be  While a 20ms delay on load, instead it is  While a 20ms layout on load

Fixed; thanks for reporting!


Preview placeholder for non-tabular resources

At the moment, if we go to showcase page of a dataset with a non-tabular resource, there would be no preview. It would have a title of the resource but no content underneath, which does not look great.

states resource - http://datahub.io/examples/vega-views-tutorial-topojson
excel resource - https://datahub.io/anuveyatsu/excel-lucky-monkey-72

This is happening because we cannot display non-tabular resources as tables so our frontend-showcase-js library skipping such files when creating preview containers.
We could implement something like preview placeholders, e.g., see Github example https://github.com/datapackage-examples/excel/blob/master/excel.xlsx :

Notice, topojson is implemented in github - https://github.com/datapackage-examples/vega-airports/blob/master/data/us-10m.json
Acceptance criteria

[ ] 2 examples above do not have empty space in previews - they have nice placeholders

Tasks

[ ] analysis
[ ] mockup of the placeholder
[ ] refactor the template

// EXTRA

[ ] have file size/rowcount next to each preview table's title

Analysis

Where to implement, frontend OR frontend-showcase-js?

We cannot implement it in dpr-js because it looks up for views to render preview components (and we're not going to generate preview views for such resources). So we need to implement this in the frontend.

As a placeholder, we can have a box with a light grey background. In the middle of the box, we can inform users that this resource does not have a preview but they can view raw content (link).
Implementation
We just need to have placeholder element in resource-preview div elements. It would be replaced by dpr-js if there is a preview component. Rendering process would be:

Initial template rendering - have placeholders for all previews
Once dpr-js is loaded it replaces previews with generated components (handsontable, leafletmap, more when we implement)
If there is no preview component, a placeholder would remain.

Think this is FIXED. We are saying:

This is a preview version. There might be more data in the original version.


Fixes minor typos

Corrects minor comment misspellings in videostab.hpp

This change is

Thanks! :+1:


onError() does not get called in case of invalid date typed in keyboard

Use case
I have a "Save" button in my page which I want to disable as long as the date typed in the DatePicker is not valid. So I want to set one error flag in state which will be used to disable/enable the "Save" button. The DatePicker field is not an mandatory field though. I thought of using onError() to set the error flag in case user typed wrong date (could be the format is not correct or its out side of min and max date), but the function does not get called. I see only error message below the field.

Tech
Version

material-ui-pickers
1.0.0-rc.3

material-ui
1.0.0-beta.36

React
16.2.0

Browser
Chrome Version 65.0.3325.146

Platform
Windows

Steps to reproduce

Expected behavior
onError() should be called whenever the date entered through keyboard is not correct
Actual behavior
onError() does not get called
Also if I type any value outside of max date it does not give any error. For e.g. if I have set max date as 01-01-2200 and I type 2200123 for the year field then I dont see any error message below the field
I am using following code snippet
        <MuiPickersUtilsProvider utils={DateFnsUtils}>
            <div className={this.props.classes.dateSelector}>
                <DatePicker
                    keyboard
                    value={selectedDate}
                    invalidDateMessage={"Date is not in 'DD MMM YYYY' format"}
                    minDateMessage={"Date can not be earlier than 01 Jan 1900"}
                    maxDate={'2200-01-01'}
                    maxDateMessage={"Date can not be further than 01 Jan 2200"}
                    onError={this.handleError}
                    label={this.props.label}
                    format="DD MMM YYYY"
                    onChange={this.handleDateChange} />
            </div>
        </MuiPickersUtilsProvider>

It is actually called with rc.7


Hide error message when execution is fixed

The error message introduced in https://github.com/enso-org/enso/pull/6918 needs to be hidden when the execution is no longer failing.
This is blocked by https://github.com/enso-org/enso/issues/7002

@farmaazon @vitvakatu Should be unblocked. Also, the execution failure messages should be less frequent now because (normal) program execution interruptions are not reported as failures (they never should have been).


Fixed #31509 - enable faulthandler automatically during tests

I mention some issues with this here: https://code.djangoproject.com/ticket/31509#comment:5

Sorry about that. I will write documentation and tests for this.

Sorry about that. I will write documentation and tests for this.

No worries.
Great work so far, thank you.

Quick question - where is the appropriate place to write documentation?

Please read the appropiate documentation on writing documentation linked from https://docs.djangoproject.com/en/dev/internals/contributing/ : https://docs.djangoproject.com/en/dev/internals/contributing/writing-documentation/ .
The new argument to the test runner should be documented alongside the others like --parallel - just grep for where that is.
The release note should be in 3.1.txt, but since the cutoff for features is monday, you might need to rebase and target 3.2.

I am not sure offhand why the tests are failing.

I believe the tests are failing because Jenkins is overwriting sys.stderr and faulthandler.enable() tries to call fileno on it (which isn't supported).

Indeed:
In [1]: import faulthandler

In [2]: import sys

In [3]: import io

In [4]: sys.stderr = io.StringIO()

In [5]: faulthandler.enable()
---------------------------------------------------------------------------
UnsupportedOperation                      Traceback (most recent call last)
<ipython-input-9-7989c6ca1bf2> in <module>
----> 1 faulthandler.enable()

UnsupportedOperation: fileno

I can't reproduce it locally. I'm not sure how it's replaced, it would be part of the Python process.

We could always use faulthandler.enable(file=sys.__stderr__), which according to https://docs.python.org/3/library/sys.html#sys.stderr will point to the original sys.stderr when the python program was started.

Should I target this at 3.2 now?

We could always use faulthandler.enable(file=sys.__stderr__)

That seems reasonable to me. But I'd like to understand how Jenkins is overriding stderr.
Another workaround is from sys import stderr, which will grab what stderr points to at import time, even if something else overrides sys.stderr. I'm not sure that's better than sys.__stderr__ though, since it will only work if stderr == __stderr__ anyway.

Should I target this at 3.2 now?

Yes please.

@ovkulkarni Thanks :+1: I pushed edits, added missing docs changes, added test for already enabled faulthandler and for sys.stderr that is not a file.
I also doc'd arguments added in Django 3.0 and 3.1.

@adamchainz Do you want to take a look at my changes?


Add YugaByte DB to the Database and Data Analytics section

Please only report on one project or product per issue
Guidelines for getting added to the CNCF Cloud Native Landscape:

We generally will only list a company in one box, to represent its major or best known offering. We occasionally make exceptions for large companies.
Projects with at least 250 GitHub stars that clearly fit in an existing category are generally included.
We are unlikely to create a new category for products/projects as we'd rather find the best home with the current options.
We are generally not including commercial versions of open source software, when that version just includes minor tweaks.
Closed source products need to link to a clear description of your product; no stealth mode companies.
Your project or company needs a logo.

Project or Product name:
YugaByte DB
Is it open source? (if you are tempted to say partially, please answer no):
Yes, distributed under Apache 2.0
What are the best URL(s) for it? Repo and homepage (if different) for open source, product homepage for closed source?:
https://github.com/YugaByte/yugabyte-db
https://www.yugabyte.com/product/
What one category does it best belong in?
Database and Data Analytics
What's the URL for the best logo (and it needs to include the name)?
https://s3-us-west-2.amazonaws.com/assets.yugabyte.com/yugabyte_logo_for_cncf_landscape.ai
https://s3-us-west-2.amazonaws.com/assets.yugabyte.com/yugabyte_logo_for_cncf_landscape.png
Anything else we need to know?:
See YugaByte DB in action on Kubernetes: https://docs.yugabyte.com/quick-start/install/#kubernetes

@alexcontini Please convert page 2 of https://s3-us-west-2.amazonaws.com/assets.yugabyte.com/yugabyte_logo_for_cncf_landscape.ai into an SVG.

@dankohn the zipped svg is attached
yugabyte.svg.zip

Closed with https://github.com/cncf/landscape/commit/86b7ebcc39c149561581fe91db091fffb3e4ee0a


[bug] can't connect to tauri.localhost

Describe the bug

When I exec npm run tauri dev it's fine , however, my angular project is mutil language version, so as the screenshot shows, it will build the mutile language folder, and the entry command is node proxy-server.mjs
how can I use Tauri build a release version work?
Reproduction
No response
Expected behavior
No response
Full tauri info output
can't connect to tauri.localhost

Stack trace
No response
Additional context
No response

thanks for your solution, I will try


Fix Flake8 . issues #91

Fixes #91
Account Number:<PHONE_NUMBER>de48e4bf2ed2a9b4b839e6b2603018ecdfbf4cffdb2065e01a3ed1

@vosi Thanks for helping me get this right. I've made all the changes you requested. Let me know if I've done anything wrong, or if there are other changes to be made. Thanks again for your help :)

@AbhayAysola one more thing, do not use plain noqa, please define a rule that should be ignored, like you did for import *

@AbhayAysola one more thing, do not use plain noqa, please define a rule that should be ignored, like you did for import *

Sure, I will do that.

@AbhayAysola i still can't accept it,

check my change request
check the ticket description again and uncomment 2 lines is the file

@AbhayAysola i still can't accept it,

check my change request
check the ticket description again and uncomment 2 lines is the file

My bad, I messed up with commits and stuff and I thought I had made those changes but I didn't. 😓
Should be good now.

@vosi Do you have any idea why flake8 thinks there's an import in line 4 of update_stack.py ? I don't understand why the checks are failing.

@vosi Do you have any idea why flake8 thinks there's an import in line 4 of update_stack.py ? I don't understand why the checks are failing.

right now it's configured to merge branches to master internally to do flake test on a merged branch ... I might change that later ... or not
but as a quick fix, I'd suggest you rebasing or updating your branch with upstream

Ok, should be working by now.

paid


Option to accumulate over multiple frames

Hi, would it be possible to add an option to accumulate AO over multiple frames? Some models look quite bad with lower quality settings (especially light surfaces) & cranking up the quality isn't always an option for lower end devices. I would love to help on this but my knowledge on GLSL is lacking 😄

I get why you're asking but I am strongly against adding temporal reprojection to the library as it would make things much more complicated, add overwhelming amounts of smearing, and requires exepensive-to-generate motion vectors. It would compromise performance just as much.
However, I would by open to making it so the camera could accumulate AO over multiple frames when frozen - as a toggleable setting for like... static viewing. Would you be interested in that?

accumulate AO over multiple frames when frozen

Actually this is what I meant :D The option to accumulate when the camera is static would be great!

That should theoretically be doable - the issue is that it would require setting up a seperate render target and piping the AO through that... should be possible though. I'll get to it for the 1.8 release. Should be added within the month.

This should enable taking just ~4 samples a frame and getting results that look better then current ultra settings in less than a second - assuming stationary camera & scene.
<img width="1507" alt="Screenshot 2024-01-16 at 8 26 45 PM" src="https://github.com/N8python/n8ao/assets/43304488/f5125bba-537a-4112-86d8-5

6dee322b462">
(Top is w/ accumulation, bottom w/out. Same perf cost)

W/out it for reference.

This is really nice!

Commited.

Just updated the package on my side. Looks great! The accumulated AO is very good, for some scenes it completely changed the look for the better (as per your recommendation to set denoiseRadius: 0).
I tried experimenting with a mixed quality approach. Where I'd set the quality to the "Low" config while the camera was moving, then switching to your recommended config for accumulation when static. I was hoping to have a smooth transition but it looks like the frame gets cleared as soon as accumulation kicks in. Although stepping though the N8AOPass.ts I couldn't see this happening.
Do you know what this could be?

Ah... I set it up so the accumumlation WOULD be cleared if you change the settings of N8AO... as the old samples have been invalidated. Wdym cleared? Like all the AO dissapears?

The behaviour you've described aligns with what I'm seeing to be honest. The static AO comes back quickly but there is a noticeable transition between dynamic and static AO. I'm going to see if I can configure this to achieve a smooth transition.


make preference more like apple style

Description

add pictures for appearance changing view2.
Change tab settings to xcode style.

Checklist

[X] I read and understood the contributing guide as well as the code of conduct
[X] My changes generate no new warnings
[X] My code builds and runs on my machine
[X] Review requested

Screenshots

If you still decide not to, then I can use svg. To be honest, the amount of code that SwiftUI requires is huge. May exceed image size

If you still decide not to, then I can use svg. To be honest, the amount of code that SwiftUI requires is huge. May exceed image size

I just built this. This is exactly 50 lines of code (2KB). So I will not argue further. 2KB compared to 1MB of images (combined) is worth it to spend 10 minutes of your time to implement it in a efficient way. I will not argue further.

Also if we need to change it some day in the future we can easily change the code instead of re-creating it in Figma, etc.

If you still decide not to, then I can use svg. To be honest, the amount of code that SwiftUI requires is huge. May exceed image size

I just built this. This is exactly 50 lines of code (2KB). So I will not argue further. 2KB compared to 1MB of images (combined) is worth it to spend 10 minutes of your time to implement it in a efficient way. I will not argue further.

If you still decide not to, then I can use svg. To be honest, the amount of code that SwiftUI requires is huge. May exceed image size

I just built this. This is exactly 50 lines of code (2KB). So I will not argue further. 2KB compared to 1MB of images (combined) is worth it to spend 10 minutes of your time to implement it in a efficient way. I will not argue further.

It doesn't have any blur effect. And the wallpaper is not very similar to the style of MacOS

Also if we need to change it some day in the future we can easily change the code instead of re-creating it in Figma, etc.

But it isn't very beautiful

It doesn't have any blur effect. And the wallpaper is not very similar to the style of MacOS

That's because it is a proof of concept put together in less than 10 minutes. But instead of doing it yourself, you argue with me for over 1 hour whether 2KB of code is more efficient than 1MB of images. 🙄

So how do you simulate the style of the wallpaper, which involves complex mathematical formulas. Also it only took me a minute to make this image. Obviously more efficient than 10 minutes.

In summary: We will not add about 1MB (even 270KB is huge) to our package size just for 3 images that take up about 2cm on screen each. I showed a proof of concept of how easy it is to recreate an image like that with pure SwiftUI. Code is much more maintainable than heavily compressed PNGs. Also we have to take in consideration that once we host the app on our own servers we, as an open source community, cannot afford any unnecessary increase of app size since traffic costs a lot of money once it scales.

@underthestars-zhy Even if it is 193kb it can't beat 2kb 🤷🏻♂

Not to mention the jpg compression artefacts.

Closing this.
Tab Width Setting has been implemented in:

#385

The Appearance picker in the proposed style does not make sense in this context.
Apple uses this in macOS System Preferences to indicate to users what Appearance means. In Xcode they use a system menu picker instead since the user already knows what Appearance is.
There's no need to spend three times the screen real-estate for redundant information.
See System Preferences:

See Xcode:


Allow to specify what is current time

Currently it is hardcoded as browser current time, but it would be usefull to allow user specify that is current time (something like this http://gist.github.com/398904).
In my case it necessary because I have to display intervals like '2 seconds' ago and very often browser time is different from server time in couple of seconds. With this patch I can specify current time as server time and display correct intervals.

Why is it still open?


feat(Data/UInt, Data/Fin/Basic) instances

Add new typeclass instances for Fin and UInt8. This incorporates gebner's suggestions to add boolean methods for checking under/overflow, and adding separate instances of e.g. Fin UInt8.size. The plan was to make a macro that would extend the UInt stuff to all the UInt sizes, but I couldn't figure out how to deal with namespaces and identifiers properly in a macro, so I'll have to get some outside guidance and improve the macro docs.

Also if anyone looking at this can figure out this thing, I can get rid of one of the weird h0 lemmas: https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/simp.20type.20annotations/near/260877092

I figured out the macro thing, I'll PR the improved version.


Document DestinationRule mTLS conflict

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: frankbu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [frankbu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: frankbu, rcaballeromx
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [frankbu,rcaballeromx]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Web Browser Compatibility List

Hello everyone.  Is there a web browser compatibility list somewhere?

So far I tested the following and it looks good:

FF 62
Edge 38
Chrome 67
IE 11
iPhone 5S, Safari 7

I just went back from Safari iOS 7 up to 11 and tested the phone validator.
Same works fine for Android from 4.3 onwards (Chrome, Firefox)

Hey, since there are React component and basic JS version, so they might cover different browser compatibility. https://caniuse.com/#search=oninput
Basically any browser supports onInput Event should just work.
We are experiencing some cursor moving issue on Android devices, particularly Samsung devices.


npm run-script build build doesn't work

See https://github.com/GoogleChromeLabs/llaminator/pull/22
The error message says that index.ts's "MIME type ('video/mp2t') is not executable."
I'm guessing that's because we're trying to include a ".ts" file as a script, and .ts is being misinterpreted.

Seems to work fine here:
$ npm run-script build

><EMAIL_ADDRESS>build
> rollup -c rollup.config.mjs

src/index.ts → dist...
created dist in 101ms

src/sw.ts → dist...
created dist in 346ms

sorry, to be more specific - the build step succeeds, but then serving up the output directory with a static server will give you a runtime error when index.html tries to include index.ts as a script


Update Fukukitaru skill name

Because yea... the current one is... yknow. Not like this is much better, but at least it's not blatantly 18+ anymore.

I'll leave this for someone to do at some other time, considering the above.


corrected spelling and grammar (I)

Coverage remained the same at 85.564% when pulling b2b74fb99b20c63d699e99d5aa6e8655770b1afa on minorchanges into cfdd8937219c7e467b370bcdf4b689686fe3c0b4 on master.

Coverage remained the same at 85.564% when pulling b2b74fb99b20c63d699e99d5aa6e8655770b1afa on minorchanges into cfdd8937219c7e467b370bcdf4b689686fe3c0b4 on master.


Is .NET4.5 still supported in SharpDX for WinRT?

I installed "SharpDX Toolkit for VisualStudio" into VS2012 and create project "Toolkit Game" for "Windows Store" platform.
Project creation succeeds, but build failed.

warning MSB3274: The primary reference "SharpDX" could not be resolved because it was built against the ".NETCore,Version=v4.5.1" framework. This is a higher version than the currently targeted framework ".NETCore,Version=v4.5".
warning MSB3275: The primary reference "SharpDX.Toolkit.Game" could not be resolved because it has an indirect dependency on the assembly "SharpDX, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b4dcf0f35e5521f1" which was built against the ".NETCore,Version=v4.5.1" framework. This is a higher version than the currently targeted framework ".NETCore,Version=v4.5".
...

.NET4.5 (and VS2012) is no longer supported?

SharpDX2.6.3 WinStore is relying on Win8.1 which is requiring using .NET4.5.1


Make Symbol#valueDeclaration optional in public API

compiler/types.ts uses some non-nullable typings to avoid frequent
assertions in the TS compiler. This causes a loss of type information
and makes using typescript's public API dangerous since an external
consumer may assume that a certain type is defined when it is not.
(See #24706 for more details).
One such case is Symbol#valueDeclaration, which this PR provides a
public API typing fix for. This is implemented by adding an additional
processing step to the LGK compiler that overwrites definitions with the
@optional JSDoc tag to include the ? token in their signature.
Specifically, /** @optional */ valueDeclaration: Declaration ->
valueDeclaration?: Declaration. The relevant change in the public API
definitions is at lib/typescript.d.ts:2285.
Part of #24706

Ideally the update to all LGK should not be necessary, just attaching it in this PR to demonstrate that this change works. There may be better ways to fix this, too -- one way is to add a flag like stripInternal that removes definitions with the @internal annotation. I didn't think this use case needed a new compiler flag, but I would love to hear any feedback.

I agree. valueDeclaration is the worst offender in my opinion, but replacing just the public API isn't the right way to go. We would welcome a PR that changes the actual type and inserts ! in the right places, but I'm going to close this PR.


Add a License

Looks interesting. I'm new in my Godot journey so not using that many addons yet but can easily anticipate wanting this.
I'd have a hard time justifying using it without the ability to diagnose and fix it though! I'm not asking for a full blown set of contribution guidelines, but can you pick and add a license?

I 100% forgot to add a License, it was supposed to be MIT! Thanks for noticing it.
Closing this issue now that I added the license.


Cannot install @azure/service-bus

Package Name: @azure/service-bus
Package Version: 1.0.0-preview.4
Operating system: Arch Linux
[x] nodejs

version: v10.15.3

Describe the bug
When installing @azure/service-bus, it fails with error due to failing preinstall script. See attached error message.
To Reproduce
Steps to reproduce the behavior:

Create empty directory
Run npm install @azure/service-bus

Expected behavior
Package is downloaded without error.
Screenshots
➜  tmp npm i @azure/service-bus

><EMAIL_ADDRESS>preinstall /home/lenkan/code/node_modules/@azure/amqp-common
> node ../../../common/scripts/rush-welcome.js

internal/modules/cjs/loader.js:584
    throw err;
    ^

Error: Cannot find module '/home/lenkan/code/common/scripts/rush-welcome.js'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:582:15)
    at Function.Module._load (internal/modules/cjs/loader.js:508:25)
    at Function.Module.runMain (internal/modules/cjs/loader.js:754:12)
    at startup (internal/bootstrap/node.js:283:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3)
npm WARN enoent ENOENT: no such file or directory, open '/home/lenkan/code/package.json'
npm WARN code No description
npm WARN code No repository field.
npm WARN code No README data
npm WARN code No license field.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>preinstall: `node ../../../common/scripts/rush-welcome.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>preinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!

Same here, cannot install the module.

><EMAIL_ADDRESS>preinstall c:\Git_work\alien-vault\allegro-methodus\modules\@allegro\service-bus\node_modules\@azure\amqp-common
> node ../../../common/scripts/rush-welcome.js

internal/modules/cjs/loader.js:583
    throw err;
    ^

Error: Cannot find module<EMAIL_ADDRESS>    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:581:15)
    at Function.Module._load (internal/modules/cjs/loader.js:507:25)
    at Function.Module.runMain (internal/modules/cjs/loader.js:742:12)
    at startup (internal/bootstrap/node.js:279:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:752:3)

Same problem for me:
><EMAIL_ADDRESS>preinstall /Users/matteodipaolantonio/Desktop/infinitas/service-liber-api/node_modules/@azure/amqp-common
> node ../../../common/scripts/rush-welcome.js

module.js:549
    throw err;
    ^

Error: Cannot find module '/Users/matteodipaolantonio/Desktop/infinitas/service-liber-api/common/scripts/rush-welcome.js'
    at Function.Module._resolveFilename (module.js:547:15)
    at Function.Module._load (module.js:474:25)
    at Function.Module.runMain (module.js:693:10)
    at startup (bootstrap_node.js:191:16)
    at bootstrap_node.js:612:3

We do have the same problem with the latest version  although it fails with a different error message
error C:\Development\GraphQL\node_modules\@azure\service-bus\node_modules\@azure\amqp-common: Command failed.
We think its related to the recent @azure/amqp-common_1.0.0-preview.4 version bump which is a dependency of  l @azure/service-bus

I worked around the issue for now by doing
npm i --save<EMAIL_ADDRESS>
before running
npm i --save<EMAIL_ADDRESS>

Apologies, this slipped through our review for the preview. We've got the fix merged and will be cutting a preview.5 release shortly.


implement T5 LLM prototype

User story

As a project team
I want / need to implement the first draft of the T5 LLM
So that we can evaluate the functionality of this LLM and discover problems with this LLM.

Acceptance criteria

one or more source code files for the T5 LLM are created
a separate folder for the T5 LLM source code is created
the code framework is created according to the defined software architecture
the LLM framework is compatible with the LLM-Server framework
source code compiles without errors

Definition of done (DoD)

source code is uploaded
the bill of materials section of the planning documents is updated

Notes

T5 LLM

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

Source code and readme are uploaded here: https://github.com/amosproj/amos2023ss03-qachat/tree/main/QAChat/LLM_Prototypes/T5
We were able to run like this:

But because of a problem from hugginface side we are not able to run again:

Regarding the creation of sentence embeddings from T5, unlike BERT, T5 does not have a CLS token and returns multiple outputs.
To obtain the embeddings, there are three possible methods: (i) using the representation of the first token in the encoder, (ii) taking the average of all token representations from the encoder, and (iii) using the representation of the first token in the decoder.
Recently, Google has introduced ST5 (Sentence T5), which is a tool for generating sentence embeddings and can be found at:  https://tfhub.dev/google/collections/sentence-t5.
Additional details can be found at: https://arxiv.org/abs/2108.08877


Add kubevirtci deployment support and functional tests

/cc @0xFelix
What this PR does / why we need it:
This PR adds kubevirtci and a simple set of functional tests to the project.
At present these tests are crudely written in bash and only exercise the admission the generated resources and the admission of a VirtualMachine referencing these resources. The latter is useful as it exercises the validation codepaths ensuring the common-instancetypes are actually useful.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Draft until https://github.com/kubevirt/common-instancetypes/pull/26 and https://github.com/kubevirt/kubevirt/pull/9103 land allowing the functional tests to pass.
Release note:

NONE

/approve
/lgtm
Looks good to me, thanks!


⚠ File access (beta) has degraded performance

In e3831e5, File access (beta) (https://beta.fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 3232 ms

Resolved: File access (beta) performance has improved in d42745e.


Make index creation deterministic

By ensuring randstrobes are sorted by all their fields.
I thought that the comparison function I added in #386 (commit 2e4ff9500e68d6e465735dd276d362cf71851dcd) was good enough, but now that I ran on some other datasets that I hadn’t used for testing previously, it became apparent that that’s not the case because I was getting non-reproducible results.
This slows down index creation again a little bit (now takes ~61s instead of ~60s for CHM13), but it’s still a lot faster than before parallel sorting.

Merging as I need this for further experiments


added support for protocol 48258

Notably, it seems like all the TrainXXX, BuildXXX and MorphXXX have been renamed to just XXX in the latest version of the balance data.

Thanks @frugs !!

No problem! I found the process a little painful so I might send you a follow up pull request with added documentation around this, if that's ok?

yepper
On Sun, Dec 11, 2016 at 4:35 PM, Hugo Wainwright<EMAIL_ADDRESS>wrote:

No problem! I found the process a little painful so I might send you a
follow up pull request with added documentation around this, if that's ok?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ggtracker/sc2reader/pull/17#issuecomment-266309787,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAvzqQv3_s1LZd6a7HMh-d5SCK3i3-hOks5rHGyegaJpZM4LJ_Jo
.


RuntimeWarning in preparing data for display in variable explorer leaks to console

Problem Description
Originally reported as numpy/numpy#11029
In numpy 1.14, umath.minimum.reduce and umath.maximum.reduce will raise RuntimeWarning if they encounter a NAN value. It seems the variable explorer is not filtering these warnings, so they turn up on the console when

max,min display is enabled in the variable explorer
there is a NAN value in the ndarray

What steps will reproduce the problem?

Use numpy 1.14.0 or later
Turn on max/min display by clicking to the variable explorer tab, clicking on the setup gear in the corner, and making sure "Show arrays min/max" checkbox is checked.
in the console, `import numpy as np; A=np.full(16, np.nan)'

Note that numpy 1.14.0 - 1.14.3 have a bug, doing A=np.full(7, np.nan) would not show the warning.
What is the expected output? What do you see instead?
In the console, I see two RuntimeWarnings
Package Versions

Spyder: 3.2.8
Python: 3.5.2
Qt: 5.9.3
PyQt: 5.9.2
Operating System: Ubuntu 16.04

Dependencies

IPython >=4.0    :  6.3.1 (OK)
cython >=0.21    :  0.28.2 (OK)
jedi >=0.9.0     :  0.12.0 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
numpy >=1.7      :  1.15.0.dev0+52316e3 (OK)
pandas >=0.13.1  :  None (NOK)
psutil >=0.3     :  5.4.5 (OK)
pycodestyle >=2.3:  2.4.0 (OK)
pyflakes >=0.6.0 :  1.6.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pylint >=0.25    :  1.8.4 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
rope >=0.9.4     :  0.10.7 (OK)
sphinx >=0.6.6   :  1.7.3 (OK)
sympy >=0.7.3    :  None (NOK)

Thanks for reporting. We'll try to fix this in our next release.

Yup, reproduced.


Warning when Synchronous api used?

Is it possible to issue an exception whenever a sync API is used (e.g. ToArray() instead of ToArrayAsync), similar to QueryClientEvaluationWarning?
Ideally this could be turned off on a per query basis in case there is a good reason to use a sync API (a bit like the c# #pragma warning disable)

@clement911 It's possible that we could block synchronous execution of the query, which seems to be the gist of what you are asking for. However, this is the first feedback we have seen for this, and there are many scenarios where using synchronous APIs is desirable--for example, when using SQLite which doesn't have a truly asynchronous implementation at the database level. (In such cases using the async APIs just adds overhead but with no value.)
We're not planning to implement this, but as always we could reconsider given good feedback.

This is not what I'm asking.
I just would like to have some sort of events I can hook up to when a sync API is used.
So that we can log it and then remove sync calls that were accidentally introduced in the code base.

@clement911 Something like this? https://docs.microsoft.com/en-us/dotnet/api/microsoft.entityframeworkcore.diagnostics.relationaleventid.commandexecuting?view=efcore-2.1

Yes, that looks promising, I see there is an IsAsync property.
But, how do I hook up to that event? Any sample?

@clement911 https://github.com/aspnet/EntityFramework.Docs/issues/583 links to a couple of issues with examples.

Interesting API..
I think that will do thanks.


Fix SelectBox async methods

This PR is required for https://github.com/x-team/auto/issues/1358 and for https://github.com/x-team/auto/issues/1407
It updates the async methods of SelectBox and fix some hidden bugs here and there.
@nicksp as I mentioned before, Styleguidist have some limitations so the showcases for SelectBox now miss the "full UX" for QA since it requires a container to hold transitional states. This is not a problem and can be worked around. I'm just not investing more time on this now to get things (ListsEditor) going, so please keep that limitation in mind when checking this PR. :)
I tested this on the application and this is how it looks:

Still need to deal with transitional states on AddToList container, so the https://github.com/x-team/auto/issues/1358 can remain open.
Auto-ui preview

I'm not seeing this error on my log 🤔
It doesn't look related to any change from this PR. Will double check...

Thanks @bernardodiasc also, given that we aren't that interested in having transitional states in auto-ui, would is still remaining on https://github.com/x-team/auto/issues/1358? Could you leave a note there?
And I'm curious if these transitional state will work when integrated into Auto ?

This is how it looks:

The UI will respond only after server response with the updates in this PR. It's still required to make some updates on Auto to make transitions work on AddToList, the effort would be XS. I mentioned this in the https://github.com/x-team/auto/issues/1358, I suggested to repurpose that issue to address this detail, since the SelectBox are now stable. And it's ok to do that task later because the feature works fine without it (it's more an enhancement than a bug).

@bernardodiasc Could you see the error I showed above? If not, then it might be smth related to my machine and we're good to merge changes.


Add Kubernetes 1.29 staging packages bundle files, deprecate 1.24 packages

Adding 1.29 staging packages bundle input files.
Update Helm chart tags to latest for other Kubernetes versions
Removing 1.24 packages input files.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm
/woof


⚠ Bofrost.at has degraded performance

In a8a59f0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1822 ms

Resolved: Bofrost.at performance has improved in 8390f84.


Automatic update Pypi

Automatically update the Pypi submission whenever a new commit is done in the master branch. Also start using tags.

Integrated.


Hover is offset when using CSS zoom.

I've zoomed out my whole webpage using CSS zoom. However I notice now that the tooltip appears now at offset location of the true data point. Is there anyway to fix this?

Ok, I've found a solution. (It is better that I not use zoom at all in my body, but currently I have to.)
I set the zoom again for the wrapping div of the chart to the inverse of the zoom that was applied to the page.
So my body is styled like this:
body {
  zoom: 0.8;
}

And with inline styling on the wrapper div I did this:
<div style={{ zoom: (1 / 0.8)}}>
  <Bar data={data} options={options} />
</div>

So I've used the inverse of the zoom here, in my case 1 / 0.8. You could also apply this in CSS, performing a calculation ( I believe this is possible in CSS3).


What are the constraints to make ResizeNearestNeighbor work on Edge TPU compiler

Eventhough I have successed int8 quantization, I recently experience failing on ResizeNearestNeighbor in Edge TPU. In the coral official website, they state that ResizeNearestNeighbor is allowed. So why when I am checking the condition on graph, it fails?

Furthermore, when I check the frozen graph before tflite int8 conversion, it does not have Fake quantize node, is this the reason?

My edgecompiler cannot perform this and does not give any clue
Edge TPU Compiler version 14.1.317412892

Internal compiler error. Aborting!

My quantized model is stored here: [https://drive.google.com/drive/folders/14L4Yrsk_9rB6UOgJn8La7n1epanAFI0B?usp=sharing]

Hello @dtlam26, is this still failing with the latest compiler release?

Yes, I am using the version from git source 14.1.317412892.
`$ ./compiler/x86_64/edgetpu_compiler /home/dtlam26/Documents/Coral_Project/RetinaNet/export/retina.tflite
Edge TPU Compiler version 14.1.317412892
Internal compiler error. Aborting!
`
You can check my models I attached above to verify,

@dtlam26 oh, sorry, I suggest trying it with the compiler from our apt repo, it is up to 15.x now, the edgetpu repo was deprecated, more info here:
https://coral.ai/news/updates-11-2020

what about the pycoral, without edgeTPU compiler, how can I compile my model to fit it edgeTPU configuration? As I attached above, I have try to compile with 15.0 version and it still failed. Is that the lastest version?. I get it through python3-edgetpu

For the pycoral, I install it from wheel and got the missing in runtime as bellow:
ERROR: Could not find a version that satisfies the requirement tflite-runtime==2.5.0 (from pycoral==1.0.0) (from versions: none)
ERROR: No matching distribution found for tflite-runtime==2.5.0 (from pycoral==1.0.0)

@dtlam26 You can get the tflite-runtime whl at : https://www.tensorflow.org/lite/guide/python
and then try to install the pycoral.

@dtlam26 You can get the tflite-runtime whl at : https://www.tensorflow.org/lite/guide/python
and then try to install the pycoral.

appreciate, I got pycoral now. But the newest edge_compiler from apt still fails:
edgetpu_compiler retina.tflite
Edge TPU Compiler version 15.0.340273435

Internal compiler error. Aborting!

@dtlam26 There is a compiler binary available at https://github.com/google-coral/edgetpu/tree/master/compiler/x86_64/edgetpu_compiler_bin , I am not sure about the version of this one since I am working on a Mac right now. Can you try this one and share the result ?

@dtlam26 There is a compiler binary available at https://github.com/google-coral/edgetpu/tree/master/compiler/x86_64/edgetpu_compiler_bin , I am not sure about the version of this one since I am working on a Mac right now. Can you try this one and share the result ?

I have go to the location file, but can not execute with these errors:
figlet: relocation error: libc.so.6: symbol _dl_exception_create, version GLIBC_PRIVATE not defined in file ld-linux-x86-64.so.2 with link time reference
dirname: relocation error: libc.so.6: symbol _dl_exception_create, version GLIBC_PRIVATE not defined in file ld-linux-x86-64.so.2 with link time reference
dirname: relocation error: libc.so.6: symbol _dl_exception_create, version GLIBC_PRIVATE not defined in file ld-linux-x86-64.so.2 with link time reference

However, I think that is the eel2 branch version, so I have checkout and compile, but still fails. You can verify with my model that I share in the drive above:
git checkout c48c888
./edgetpu_compiler retina.tflite
Edge TPU Compiler version 14.1.317412892

Internal compiler error. Aborting!

Hello, i mean the compiler, not the runtime:
sudo apt install edgetpu-compiler
@Naveen-Dodda could you check this one out?

@Naveen-Dodda could you check this one out?

Yes, for clarify, I have tried to install with the sudo apt install edge-compiler at first. Then I go with the git compling method as shown above

From this issue with Upsampling module. Is it also because my model input for upsampling layer is too big to keep precision so that the compiler stop working?