Virtio-gpu-gl through already existing Windows Mesa Driver

Hi
From what I get a virtio-gpu windows guest driver already exist but there is no support for 3D acceleration with VirGL (on host) as renderer
On Linux a 3D acceleration on a virtio-gpu-gl is supported through Mesa Drivers
For windows there is support for Mesa Driver, I would like to investigate the possibility of using this drivers to interact with the virtio-gpu of this repo so to enable 3D acceleration on windows
The basic idea is:

Virtio-gpu-gl driver exposing a vGPU
Mesa driver using such vGPU
WindowsOS using Mesa Driver

(I know about VFIO, GPUPassthrough, Nvidia Grid ecc, but this would be a more Flexible, Hardware Agnostic and Open Source option)
Is this achievable? Or am I missing something?
Thanks,
Luca

Hi Yan
I would like to try to work on it on spare time
I'm experienced c++20 programmer (mostly baremetal) and IT admin but I'm getting a bit lost on the implementation details (with missing or hidden documentation).
Here is as I understood a possible complete architecture:

From what I get full Mesa already support a virtio-gpu and I think the gallium conversion is here
yet I didn't found where it hook up with the actual vGPU
On your side code I did't deep dive into it yet but seem like you already exposing virtio-gpu-gl as a pcie device and in the header the are definitions for 3D commands
What I'm missing?
Thanks,
Luca

@imwhocodes
What is the "virtual pcie" device on that diagram?
In my understanding one of the critical things there is adding the WDDM RENDER stuff to the existing DOD driver.
(Some of the work toward building a FULL WDDM virtio-gpu driver has been done already but it is still not completed yet.
When this job will be done, we can add an OpenGL Installable Client Driver, or any other ICD to the project)
You can take a look at https://studiopixl.com/page3/ . But you need to understand that due to luck of Rendering Driver
functionality this project was done as a kind of hack for the POC purpose only.
Cheers,
Vadim.

@YanVugenfirer  Do you have any clarification or hints?
Thanks

I wonder if it might even wind up being easier to use vitio-gpu venus? I realize that it still isn't merged into qemu so there might not be a lot you can actually do on that end right now but the patches are being worked on. I wonder if it might be easier to get vulkan acceleration in the guest then it would be opengl via venus.
also it might not be too bad to try and port D3D10UMD to NIR from TGSI to try and get that working with virgl, or even zink if the vulkan route could wind up proving more beneficial, (I assume that the performance would be better at the very least). and if that does work I believe from there it can be extended to d3d11. IIRC that should be able to provide desktop acceleration.
well regardless, another resource is a talk that I think was from 2020? it was something called virgl and windows if I am remembering it correctly. that could be a good chunk information there.

@imwhocodes Hi Luca,
I think while ICD can be most probably done with not much effort by reusing Mesa and other existing projects, I think this is a least interesting part.
We need to implement (or reuse from other projects) the layer that will translate DirectX 10+ to something that can be used on the host side.
Best regards,
Yan.

Wanted to pop in and give some bits of info.

Pass through Vulkan and OpenGL calls to the host via venus and the like.
Bundle DXVK (and VKD3D) like Intel does for their Windows driver (which doesn't have D3D11 and older support) for Direct3D support.

DXVK and VKD3D don't officially support Windows (only Linux/Wine) so you'd have to provide support for any bugs that occur there, though that could be useful for improving the projects if you were to fix the bugs and upstream them.

Wanted to pop in and give some bits of info.

Pass through Vulkan and OpenGL calls to the host via venus and the like.
Bundle DXVK (and VKD3D) like Intel does for their Windows driver (which doesn't have D3D11 and older support) for Direct3D support.

DXVK and VKD3D don't officially support Windows (only Linux/Wine) so you'd have to provide support for any bugs that occur there, though that could be useful for improving the projects if you were to fix the bugs and upstream them.

Sure. That's the usual mode of operation for us - if needed we fix issues in other projects and upstream them.

Could something like microsoft's gpu-pv easier?

@voidpointertonull @YanVugenfirer @orowith2o
Any update in this features?
One other option could be wait for RADV Native context to land
From what I understood it is something similar to Nvidia Grid but open-source and for AMD commercial cards, but it is not clear to when and if it will be mainstream (and if the AMD driver will support it also on WindowsOS)
I think a more general virtio-gpu approach would be better anyway

Any updates regarding this feature?
It would be a nice addition to qemu/kvm (which virtio is a part of), if it supported DirectX 3d acceleration like vmware workstation or virtualbox. There is project on github, which runs 3d acceleration for Direct3D to DirectX 9.0c games to run on older versions of Windows (Windows 9x, Windows 2000 and Windows XP) in qemu: https://github.com/kjliew/qemu-3dfx. You could provide a similar feature, but for modern Windows (Windows 7 and later). There should be an option in virtio, because not every system supports multiple gpu for pass through and many gpu models do not support virtualization.

This feature is implemented by #943

Hi
@max8rr8 Thank you so much, I'm going to try it ASAP
I have 3 questions:

Is this based on virgl or virgl-venus (from the code it seem the first)
Which DirectX it support on guest? 10? 11? 12? (I suppose 10 from code)
Do it support Vulkan on guest? (I suppose no)

Thank for your effort!

Is this based on virgl or virgl-venus (from the code it seem the first)

virgl is used by gallium frontends (opengl and directx10/11) while venus is used for vulkan. Driver currently support virgl with possibility to support venus too in future.

Which DirectX it support on guest? 10? 11? 12? (I suppose 10 from code)

10.0 is supported by default, it might be possible to enable 10.1 and 11 though not tested. 12 is quite different from 10/11 and i don't think it can be supported anytime soon.

Do it support Vulkan on guest? (I suppose no)

Vulkan support (through venus) requires support for blob resources from kernel-side driver(that is a bit tricky) and some additional work to support windows on venus from mesa. So no it is not supported now, but can be in future.

Does this mean we can play 3d games on Windows guest without any GPU passthrough?

You can try, but the performance isn't going to be great, you probably will get much better result by using proton or gpu passthrough.
My main motivation for working on this project is 3d acceleration of desktop apps like word, excel, browser, visual studio, etc... to lower cpu usage and improve UX compared to software rendering.

has it been completed?

when such a driver is comming out, this will be a gamechanger!!


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

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

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #470


About problems that arise when running the test.py script

Thanks for your great contribution, I have encountered some problems while using your script:
I changed the entry part of test.py to fix the use of config and checkpoint：

if name == 'main':
parser = argparse.ArgumentParser(description='Hyperparams')
parser.add_argument('--config', help='config file path', default='config/pan_pp/pan_pp_r18_ic15_736_det_only.py')
parser.add_argument('--checkpoint', nargs='?', type=str, default='checkpoints/pan_r18_ic15/panpp_r18_ic15_det.pth.tar')
parser.add_argument('--report_speed', action='store_true', default=True)
parser.add_argument('--vis', action='store_true')
parser.add_argument('--debug', action='store_true')
args = parser.parse_args()
main(args)

In addition, I have prepared the data set in a format, i got an error when i python test.py, the error message is as follows:

Traceback (most recent call last):
File "test.py", line 168, in
main(args)
File "test.py", line 156, in main
test(test_loader, model, cfg)
File "test.py", line 81, in test
for idx, data in enumerate(test_loader):
File "/opt/conda/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 530, in next
data = self._next_data()
File "/opt/conda/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1224, in _next_data
return self._process_data(data)
File "/opt/conda/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1250, in _process_data
data.reraise()
File "/opt/conda/lib/python3.8/site-packages/torch/_utils.py", line 457, in reraise
raise exception
TypeError: Caught TypeError in DataLoader worker process 0.
Original Traceback (most recent call last):
File "/opt/conda/lib/python3.8/site-packages/torch/utils/data/_utils/worker.py", line 287, in _worker_loop
data = fetcher.fetch(index)
File "/opt/conda/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 49, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/opt/conda/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 49, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/data/project/pan_pp/dataset/pan_pp/pan_pp_ic15.py", line 482, in getitem
return self.prepare_test_data(index)
File "/data/project/pan_pp/dataset/pan_pp/pan_pp_ic15.py", line 465, in prepare_test_data
img = transform_pic(img)
File "/opt/conda/lib/python3.8/site-packages/torchvision/transforms/transforms.py", line 95, in call
img = t(img)
File "/opt/conda/lib/python3.8/site-packages/torchvision/transforms/transforms.py", line 135, in call
return F.to_tensor(pic)
File "/opt/conda/lib/python3.8/site-packages/torchvision/transforms/functional.py", line 147, in to_tensor
img = torch.from_numpy(np.array(pic, mode_to_nptype.get(pic.mode, np.uint8), copy=True))
TypeError: array() takes 1 positional argument but 2 were given

I checked  img = transforms.ToTensor()(img) in dataset/pan_pp/pan_pp_ic15.py and there was no error in its type. I wondered what to do in this case.

Is anyone there?

Is the author still alive @whai362

hi,Have you solved the problem


Log refactoring

Small refactoring to centralise the check of the boolean debugMessages and the prefix "Remove Adblock Thing:" into one log function.

Nice work cleaning up some of the code 👍


Is it possible to handle speech front-end signal processing tasks?

🚀 The feature, motivation and pitch
like Adaptive noise suppression，Acoustic echo cancellation，Speech Seperation task， thanks！
Alternatives
No response
Additional context
No response

We may support these tasks, but perhaps there are already off-the-shelf repos supporting them, like asteroid.

Is there any theoretical support for your hypothesis? Could you share it with us? Perhaps you can take a look at our Spatial Audio Understanding. We believe LLM can implicitly achieve source separation, but that requires the construction of a finely-tuned dataset.


Create the voters table

**Title:**
Create the voters table

**Body:**
**User Story**

* As a user, I can request a list of voters alphabetized by last name.

* As a user, I can request a single voter's information.

* As a user, I can create a voter.

* As a user, I can update a voter's email.

* As a user, I can delete a voter.

resolved


improvemen Provide the user with a suggestion to avoid th… (#39631)

cherry-pick #39631 to branch-2.1

run buildall


Syllabus: Ensure access codes uniqueness

We need to ensure that the syllabus "access codes" being generated are unique.

This repository is no longer maintained. It has been superseded by IBM Quantum Learning.


seamlessly export variables of current debugging python program into ipython notebook

It would be great to get the variables in the current stack into ipython notebook and test it.  It's like what one can do in Debug Console (restricted space), but in the ipython environment (more interactive).

/extPython


Updating Book Code to DeepChem 2.4.0

We need to work to update the book code to DeepChem 2.4.0. I've gotten a report on gitter than the chapter 6 code isn't working. What else do we need to update?
CC @peastman

Hi! I am reading "Deep Learning for the Life Sciences" right now. So far I realized that two parts of codes could not be run. The first one is under page 78 (in Chapter 5):
tasks, datasets,trasformers = dc.molnet.load_pdbbind(featurizer='grid', split='random', subset='core)
The second one is under page 91 (in Chpater 6). The whole blocks of codes can't be run. This is what I found so far, and there might will have more because of the updates.
Thank you very much for your help!!!! :-D
-Jennifer

Hi! I am reading "Deep Learning for the Life Sciences" right now. So far I realized that two parts of codes could not be run. The first one is under page 78 (in Chapter 5):
tasks, datasets,trasformers = dc.molnet.load_pdbbind(featurizer='grid', split='random', subset='core)
The second one is under page 91 (in Chpater 6). The whole blocks of codes can't be run. This is what I found so far, and there might will have more because of the updates.
Thank you very much for your help!!!! :-D
-Jennifer

Also, for the first error I mentioned above (the one under page 78). I tried to redefine featurizer = 'grid as featurizer = grid_featurizer, and grid_featurizer had been already defined in page 77.  After this small change, it works for that line.
However, when I tried to run the code after that,
sklearn_model = RandomForestRegressor(n_estimators=100) model= dc.models.SklearnModel(sklearn_model) model.fit(train_dataset),
it shows me another error that ValueError: Found array with 0 feature(s) (shape=(3881, 0)) while a minimum of 1 is required. .

Also, for the first error I mentioned above (the one under page 78). I tried to redefine featurizer = 'grid as featurizer = grid_featurizer, and grid_featurizer had been already defined in page 77.  After this small change, it works for that line.
However, when I tried to run the code after that,
sklearn_model = RandomForestRegressor(n_estimators=100) model= dc.models.SklearnModel(sklearn_model) model.fit(train_dataset),
it shows me another error that ValueError: Found array with 0 feature(s) (shape=(3881, 0)) while a minimum of 1 is required. .

Also, I found same problem in Chapter 7, page 120.
model = dc.models.TensorGraph(learning_rate=learning_rate, model_dir='model')
It will still report me like:
AttributeError: module 'deepchem.models' has no attribute 'TensorGraph'''
I believe similar type of error went through the whole book since we are all using DeepChem 2.4.0 now. Please let me know if there is anything I can help with :-D. So far this is the problem that I still stick with. Thank you!

Also, I found same problem in Chapter 7, page 120.
model = dc.models.TensorGraph(learning_rate=learning_rate, model_dir='model')
It will still report me like:
AttributeError: module 'deepchem.models' has no attribute 'TensorGraph'''
I believe similar type of error went through the whole book since we are all using DeepChem 2.4.0 now. Please let me know if there is anything I can help with :-D. So far this is the problem that I still stick with. Thank you!

Another error found in chapter 7 page 124:
`image_dir = 'BBBC005_v1_images'
label_dir = 'BBBC005_v1_ground_truth'
rows = ('A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P')
blurs = (1, 4, 7, 10, 14, 17, 20, 23, 26, 29, 32, 35, 39, 42, 45, 48)
files = []
labels = []
for f in os.listdir(label_dir):
if f.endswith('.TIF'):
for row,blur in zip(rows, blurs):
fname = f.replace('_F1', '_F%d'%blur).replace('A','%s'%row)
files.append(os.path.join(image_dir, fname))
labels.append(os.path.join(label_dir,f))
loader = dc.data.ImageLoader()
dataset = loader.featurize(files, labels)
splitter = dc.splits.RandomSplitter()
train_dataset, valid_dataset, test_dataset = splitter.train_valid_test_split(dataset, seed=123)`
When I run this code following the instruction from: https://github.com/deepchem/DeepLearningLifeSciences/blob/master/Chapter07/cell_segmentation.py, it sends me an error that shows IndexError: list index out of range,

Another error found in chapter 7 page 124:
`image_dir = 'BBBC005_v1_images'
label_dir = 'BBBC005_v1_ground_truth'
rows = ('A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P')
blurs = (1, 4, 7, 10, 14, 17, 20, 23, 26, 29, 32, 35, 39, 42, 45, 48)
files = []
labels = []
for f in os.listdir(label_dir):
if f.endswith('.TIF'):
for row,blur in zip(rows, blurs):
fname = f.replace('_F1', '_F%d'%blur).replace('A','%s'%row)
files.append(os.path.join(image_dir, fname))
labels.append(os.path.join(label_dir,f))
loader = dc.data.ImageLoader()
dataset = loader.featurize(files, labels)
splitter = dc.splits.RandomSplitter()
train_dataset, valid_dataset, test_dataset = splitter.train_valid_test_split(dataset, seed=123)`
When I run this code following the instruction from: https://github.com/deepchem/DeepLearningLifeSciences/blob/master/Chapter07/cell_segmentation.py, it sends me an error that shows IndexError: list index out of range,

Thank you for the reports! I'm currently working through another bug right now, but will turn around and try to fix some of these once I'm done with that bug.

Thank you for the reports! I'm currently working through another bug right now, but will turn around and try to fix some of these once I'm done with that bug.

I can go through all the book examples and bring them up to date.

I can go through all the book examples and bring them up to date.

Thank you for the reports! I'm currently working through another bug right now, but will turn around and try to fix some of these once I'm done with that bug.

Thank you very much!!! Also, Deep Learning for the Life Sciences is such an amazing book!! Thank you for all the efforts you put into this book! I super enjoy when read it!!!!

Thank you for the reports! I'm currently working through another bug right now, but will turn around and try to fix some of these once I'm done with that bug.

Thank you very much!!! Also, Deep Learning for the Life Sciences is such an amazing book!! Thank you for all the efforts you put into this book! I super enjoy when read it!!!!

I can go through all the book examples and bring them up to date.

Thank you very much!!! Please let me know anytime if there is anything I could help with. I am reading through the book right now and will not stop until I finish the whole book!! :-D Nice book, I love it!!

I can go through all the book examples and bring them up to date.

Thank you very much!!! Please let me know anytime if there is anything I could help with. I am reading through the book right now and will not stop until I finish the whole book!! :-D Nice book, I love it!!

Thank you!  I'm really glad you're enjoying it.  We really need to do an updated edition one of these days, since so much has changed in DeepChem.  For example, the whole TensorGraph framework doesn't exist anymore in 2.4.  We've moved to Keras and PyTorch as our modelling APIs.  In the mean time, we'll try to keep the examples in the repository up to date.

Thank you!  I'm really glad you're enjoying it.  We really need to do an updated edition one of these days, since so much has changed in DeepChem.  For example, the whole TensorGraph framework doesn't exist anymore in 2.4.  We've moved to Keras and PyTorch as our modelling APIs.  In the mean time, we'll try to keep the examples in the repository up to date.

Note to self: we need to see if there's a way to speed up RdkitGridFeaturizer.  We use it for the PDBBind examples in chapter 5.  Prevously we just specified featurizer="grid", and that told it to download a pre-featurized version of the dataset.  But now we actually run the featurization locally, and it's really slow.

Note to self: we need to see if there's a way to speed up RdkitGridFeaturizer.  We use it for the PDBBind examples in chapter 5.  Prevously we just specified featurizer="grid", and that told it to download a pre-featurized version of the dataset.  But now we actually run the featurization locally, and it's really slow.

Thank you! I'm really glad you're enjoying it. We really need to do an updated edition one of these days, since so much has changed in DeepChem. For example, the whole TensorGraph framework doesn't exist anymore in 2.4. We've moved to Keras and PyTorch as our modelling APIs. In the mean time, we'll try to keep the examples in the repository up to date.

Thank you very much for your reply! I will keep reporting the errors/issues from the book here. Please let me know anytime if there is any other things I could help with code update. I am extremely excited that I can talk directly with the author of this fascinating book! This world is magical!!!! :-D

Thank you! I'm really glad you're enjoying it. We really need to do an updated edition one of these days, since so much has changed in DeepChem. For example, the whole TensorGraph framework doesn't exist anymore in 2.4. We've moved to Keras and PyTorch as our modelling APIs. In the mean time, we'll try to keep the examples in the repository up to date.

Thank you very much for your reply! I will keep reporting the errors/issues from the book here. Please let me know anytime if there is any other things I could help with code update. I am extremely excited that I can talk directly with the author of this fascinating book! This world is magical!!!! :-D

Error in Chapter 8 page 146:
Because we want to use DRModel to define and build model, we first import valid module following the codebook:
from model import DRModel, DRAccuracy, ConfusionMatric, QuadWeightedKappa
And there is an error says ModuleNotFoundError: No module named 'model'.

Error in Chapter 8 page 146:
Because we want to use DRModel to define and build model, we first import valid module following the codebook:
from model import DRModel, DRAccuracy, ConfusionMatric, QuadWeightedKappa
And there is an error says ModuleNotFoundError: No module named 'model'.

The featurization was still running when I went to bed, which had been about eight hours.  It had finished when I got up this morning.  But the result seems to be rather messed up.  The X array is a 1D array containing arrays, rather than a 2D array.  About 10% of those arrays have 0 length, apparently because featurization failed on those molecules.  And a few elements are None.
@rbharath do you have any ideas about what's going on?

The featurization was still running when I went to bed, which had been about eight hours.  It had finished when I got up this morning.  But the result seems to be rather messed up.  The X array is a 1D array containing arrays, rather than a 2D array.  About 10% of those arrays have 0 length, apparently because featurization failed on those molecules.  And a few elements are None.
@rbharath do you have any ideas about what's going on?

Hmm, not sure. My first guess is that perhaps the options are set incorrectly? RdkitGridFeaturizer can generate both voxels and flat feature vectors and the options need to be set to generate the flat feature vectors.
Unfortunately, the RdkitGridFeaturizer has been due for a major overhaul for a while. I'm working on fixing AtomicConvModel right now but I can make my next project a pass over RdkitGridFeaturizer to stabilize and document the code

Hmm, not sure. My first guess is that perhaps the options are set incorrectly? RdkitGridFeaturizer can generate both voxels and flat feature vectors and the options need to be set to generate the flat feature vectors.
Unfortunately, the RdkitGridFeaturizer has been due for a major overhaul for a while. I'm working on fixing AtomicConvModel right now but I can make my next project a pass over RdkitGridFeaturizer to stabilize and document the code

Ah that's definitely a bug then! It shouldn't be behaving like that.

Ah that's definitely a bug then! It shouldn't be behaving like that.

Issues find in Chapter 9 page 158 (I am not so sure everyone has the sample problem with me, but just in case I will still post it here):
model.fit_sequences(generate_sequences(50))
This command line takes way longer time than expected. I tried five times, it just keeps running and never stop. The shortest time it takes is 3 hour. It runs 3 hour, but sill shows In [*].

Issues find in Chapter 9 page 158 (I am not so sure everyone has the sample problem with me, but just in case I will still post it here):
model.fit_sequences(generate_sequences(50))
This command line takes way longer time than expected. I tried five times, it just keeps running and never stop. The shortest time it takes is 3 hour. It runs 3 hour, but sill shows In [*].

You want a good GPU for running that example.  Doing it on a CPU would take a very long time.

You want a good GPU for running that example.  Doing it on a CPU would take a very long time.

You want a good GPU for running that example. Doing it on a CPU would take a very long time.

Thank you very much! I am always thinking it's might caused by my computer :-D Since it is not a bug caused by version update, I will delete that comment then. I will also post all errors in the whole book caused by deepchem version change here before next Monday. Thank you very much for all your help! As a newbie in this area, I am so glad that I have the opportunity to directly talk to the author of this fascinating book and get involved!  LOVE IT!!!!! Enjoy your weekend!!!

You want a good GPU for running that example. Doing it on a CPU would take a very long time.

Thank you very much! I am always thinking it's might caused by my computer :-D Since it is not a bug caused by version update, I will delete that comment then. I will also post all errors in the whole book caused by deepchem version change here before next Monday. Thank you very much for all your help! As a newbie in this area, I am so glad that I have the opportunity to directly talk to the author of this fascinating book and get involved!  LOVE IT!!!!! Enjoy your weekend!!!

Error update: Chapter 11 page 187, from first block of codes:
loader = dc.data.CSVLoader(tasks=tasks, smiles_field="SMILES", featurizer=featurizer)
should be revised as (because new version does not support smiles_field anymore):
loader = dc.data.CSVLoader(tasks=tasks, feature_field="SMILES", featurizer=featurizer)

Error update: Chapter 11 page 187, from first block of codes:
loader = dc.data.CSVLoader(tasks=tasks, smiles_field="SMILES", featurizer=featurizer)
should be revised as (because new version does not support smiles_field anymore):
loader = dc.data.CSVLoader(tasks=tasks, feature_field="SMILES", featurizer=featurizer)

Error update: Chapter 11 page 185, from last lock of codes:
training_score_list = [] validation_score_list = [] transformers = [] cv_folds = 10 for i in range(0,cv_folds):     model = generate_graph_conv_model()     train_dataset, valid_dataset, test_dataset = splitter.train_valid_test_split(dataset)     model.fit(train_dataset)     train_scores = model.evaluate(train_dataset, metrics, transformers)     training_score_list.append(train_scores["mean-matthews_corrcoef"])     validation_scores = model.evaluate(valid_dataset, metrics, transformers)     validation_score_list.append(validation_scores["mean-matthews_corrcoef"])     print(training_score_list)     print(validation_score_list)
The error reports Classification metrics can't handle a mix of multilabel-indicator and continuous-multioutput targets

Error update: Chapter 11 page 185, from last lock of codes:
training_score_list = [] validation_score_list = [] transformers = [] cv_folds = 10 for i in range(0,cv_folds):     model = generate_graph_conv_model()     train_dataset, valid_dataset, test_dataset = splitter.train_valid_test_split(dataset)     model.fit(train_dataset)     train_scores = model.evaluate(train_dataset, metrics, transformers)     training_score_list.append(train_scores["mean-matthews_corrcoef"])     validation_scores = model.evaluate(valid_dataset, metrics, transformers)     validation_score_list.append(validation_scores["mean-matthews_corrcoef"])     print(training_score_list)     print(validation_score_list)
The error reports Classification metrics can't handle a mix of multilabel-indicator and continuous-multioutput targets

Hi Dr. Ramsundar and Dr. Eastman,
So far I've updated all errors I found in the book. Greatly appreciated all your help!! I will keep working on reviewing them and finding the best solution. Please let me know anytime if there is anything I could help with :-D Hope you two have a lovely day!
Sincerely,
-Jennifer Ma

Error update: Chapter 11 page 185, from last lock of codes:
training_score_list = [] validation_score_list = [] transformers = [] cv_folds = 10 for i in range(0,cv_folds): model = generate_graph_conv_model() train_dataset, valid_dataset, test_dataset = splitter.train_valid_test_split(dataset) model.fit(train_dataset) train_scores = model.evaluate(train_dataset, metrics, transformers) training_score_list.append(train_scores["mean-matthews_corrcoef"]) validation_scores = model.evaluate(valid_dataset, metrics, transformers) validation_score_list.append(validation_scores["mean-matthews_corrcoef"]) print(training_score_list) print(validation_score_list)
The error reports Classification metrics can't handle a mix of multilabel-indicator and continuous-multioutput targets

I also encountered the same error, have you found a solution?

@ZkrKarry Fixes are in #27 and should be merged in soon!


Need to have an ability to log bad-precaching-response errors

Library Affected:
workbox-precaching
Browser & Platform:
any
Issue or Feature Request Description:
In some situations some resources could be lost in k8 infrastructure
We need to have an ability to log such errors and react on them

You can do this by adding in a plugin implementing the handlerDidError callback to the underlying PrecacheStrategy that workbox-precaching uses. There's a helper method, addPlugins(), that you can call to add it (since most developers won't instantiate the PrecacheStrategy directly).
import {addPlugins, precacheAndRoute} from 'workbox-routing';

addPlugins([{
  handlerDidError: async ({request, event, error, state}) => {
    // Swap this for you custom logging function:
    console.log(`Precaching request for ${request.url} failed: ${error.message}`);

    // Explicitly return null, as we're not providing a fallback response.
    return null;
  }
}]);

precacheAndRoute(self.__WB_MANIFEST);

thanks!


Message "Wrong Package (Size less tha expected)"

hi, i set up a new GC97 , the esp32 is configured , the webpage is up and i could configure all the settings, it connect to my Wifi and seem to see the mttq server but there is a message "Wrong Package (Size less tha expected)" in the webpage and there is no data from the GC97 on the webpage . it could be a bad soldering or a problem with the esp32 board?

@syrusfem Yes, very probably, this is a wrong connection with ESP and GC97 board. Check for TR/RX pairs. Also, sometimes the 2K resistance on RX line can be shortened (no guarantee on that action)

thanks i'll try to resolder che connections , idem the sort of the resistance.
A question: this is the second GC97 i am settin up , but the name is see as the same from homeassistant (i think it broadcast a fixed name) and this confuse the data on homeassistant there i a way to change che name fromn CG97 to another?

Change the MQTT Topic name in the MQTT settings: it must be unique for each device.

@syrusfem  Use unique MQTT topic on each device. Try to reboot ESP after the making a changes and wipeout your "GC97" objects in the HA.

yes i tried that too, changed name of devices  rebooted the devices, deleted original and new GC97 in HA but no results. i'll retry but HA see the 2 differently named GC97 every time but the first that connect with HA have data exposed the second none even if it is see in Mqtt so the values are passed but seem as HA accept only one device .

ok, redone all , changed MQTT topic , rebooted and after that set the MQTT communication with homeassistant and rebooted, same problem , MQTT see the monitoring device but the info are not exposed (even the IP of the device is not exposed) only difference is that a single info is present: battery energy (i think it is the total kwh of the battery) it is a info that i have not exposed on the other GC97 so maybe the sensors have the same name even with different topic and that is the problem? all other data is not  passed or homeassistant did not expose it. Only difference from the two ESP is that the old have an old firmware but it work so i don t think that is the problem.
only thing i have not used is the option: "Retain Flag Enabled" could be useful? i don't know the function.

sorry for all the messages, i know that your is a work of passion so if there is no solution no problem for me. Just another bit of info, restarting HA has redone the problem where there is a swap of sensor readiongs from the 2 sensors, i post here the error in HA , it seems as even with different MQTT  topic HA recognise the data as the same from the two sensors.

I'll check it, but a little bit later. Just I don't have a test instance for now.

I'll check it, but a little bit later. Just I don't have a test instance for now.

thanks!

Question: could i extract data from the webpage of the GC-97? It could bypass the need of mqtt data.

@syrusfem  You can use the json data. Why need to parse the webpage itself? It's just a static page that loads json and render it. Check the documentation for details.

thnks using dev tools of chrome i found json values here: http://<IP_ADDRESS>/get/all.json  .

@syrusfem
http://<ip_address>/get/all.json -- whole data package
http://<ip_address>/get/gc97.json -- hc97 data only
http://<ip_address>/get/esp.json -- ESP state data
http://<ip_address>/get/state.json -- ESP uptime, common state.

thanks, capturing json data to HA sensor is as good as mqtt for meand it work good!


Java: Update README for local maven installation instructions

Since the artifacts aren't available in Maven Central yet, this is a possible temporary workaround, but I don't think this should go into a README as this is not a typical or recommended way to consume the artifacts.

Since the artifacts aren't available in Maven Central yet, this is a possible temporary workaround, but I don't think this should go into a README as this is not a typical or recommended way to consume the artifacts.

Yes.  Agreed.  We will want to remove this (or move to the Developer.md for how to use the ExampleApp) as soon as the Maven Central artefacts are up and available.

I'm having an issue getting classifier resolution to work with this approach. Trying to see if I can make it work with install:install-file and will report back.

I'm having an issue getting classifier resolution to work with this approach. Trying to see if I can make it work with install:install-file and will report back.

I didn't find an option to push the JAR locally through the mvn command line with a classifier.  I'd prefer to have that option, although when pushing locally, it's not relevant.
The classifier is added as a suffix to the JAR name.  I believe if you add the name to the JAR file pushed by mvn it may work.

I'm trying that (but with linux-x86_64) and it isn't quite working in a Gradle project:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileKotlin'.
> Could not resolve all files for configuration ':compileClasspath'.
   > Could not resolve io.valkey:valkey-glide:1.0.0.
     Required by:
         project :
      > A dependency was declared on configuration 'linux-x86_64' which is not declared in the descriptor for io.valkey:valkey-glide:1.0.0.

But actually it looks like the artifacts are now on Maven Central so hopefully this workaround isn't needed anymore. I'll try to use those artifacts now.

That error was because I was specifying the classifier in the wrong way in the dependency definition with build.gradle.kts.  I did:
implementation("io.valkey", "valkey-glide", "1.0.0", osdetector.classifier)

But that sets the configuration property not the classifier property. doh :(
This works:
implementation("io.valkey", "valkey-glide", "1.0.0", classifier = osdetector.classifier)

And it looks like I can just do:
implementation("io.valkey", "valkey-glide", "1.0.0")

But this might only be working for linux-x86_64 because that is the only artifact with configuration in the Gradle metadata file:
https://repo1.maven.org/maven2/io/valkey/valkey-glide/1.0.0/valkey-glide-1.0.0.module

Here is the project that works for me (on linux-x86_64):
https://github.com/jamesward/hello-valkey-glide-kotlin
It'll be interesting to see if the classifier doesn't work for others on different OS / Architectures.

I guess that gradle resolves linux-x86_64 version of the module if classifier is omitted.

I am successfully using valkey-glide 1.0.0 in a Maven project:
https://github.com/sullis/valkey-playground/blob/main/pom.xml
I am using os-maven-plugin to get the right classifier


Include option for autoplay direction

Please, use spaces instead of tabs and I will merge your pull request.

@equinox7 actually I fixed indentation please check https://github.com/baladkb/lbSlider/commit/36624a693cd62e5e5ed9ea4e0dc483e61c4a10ed

You have used tabs. Please, replace them with spaces.
And default autoPlayDirection should be 'right-to-left'.

@equinox7 please check now.

You did not change default option to 'right-to-left', it is still 'left-to-right'. But I can do it by myself.

@equinox7 thanks.

@baladkb Thanks


Log logs with log

Fix #7

Closing this due to below comment :)
https://github.com/ypresto/TimberTreeUtils/issues/7#issuecomment-225328014


not show default option after fetch datas

when i fetching the datas for options default value do not match as selected in Input type = select component.
if it static so work fine.
but the simple html select works correctly.
https://svelte.dev/repl/e20365bfaed443a9b2dc66564b4dac4b?version=3.24.0

Hi @romanown , it seems to be due to the option value actually being a string.  This works with a small update:
let a = "2";

https://svelte.dev/repl/0c2998b324e14f53b8afe8e1493369f6?version=3.24.0
Not sure why that would be different from select, but seems reasonable workaround.


[222+] Migration Aura Flow wrapper to LWC

What is missing from the application? Please describe.
Currently the application uses an Aura component for wrapping LWCs that are used in Flow (see here).
Describe the solution you'd like
Once the new metadata target is available for flow screens the wrapper should be removed.

Implemented with Winter'20 release.


flask to deploy on line

when use flask deploy a fine tuned- classification task online, each post will reload the model. this takes about 4 seconds to respond, is there any solutions to avoid reloading?

#94
You can refer to this.
import predict_online function in   run_classifier_predict_online.py.
It will create a session and graph initializing at first, and you won't initial model every time you send a request

#94
You can refer to this.
import predict_online function in run_classifier_predict_online.py.
It will create a session and graph initializing at first, and you won't initial model every time you send a request

thanks!

@mokundong You may refer to bert-as-service for serving and deploying BERT models.

@mokundong You may refer to bert-as-service for serving and deploying BERT models.

it works for classifier task?

@mokundong You may refer to bert-as-service for serving and deploying BERT models.

it works for classifier task?

It didn't work for me for Question answering


Experimental implementation of Halyard Summary tool

Codecov Report

Merging #53 into master will decrease coverage by 0.09%.
The diff coverage is 79.09%.

@@             Coverage Diff             @@
##             master      #53     +/-   ##
===========================================
- Coverage     80.57%   80.47%   -0.1%
- Complexity      652      657      +5
===========================================
  Files            36       37      +1
  Lines          4011     4298    +287
  Branches        779      838     +59
===========================================
+ Hits           3232     3459    +227
- Misses          520      546     +26
- Partials        259      293     +34

Impacted Files
Coverage Δ
Complexity Δ

...in/java/com/msd/gin/halyard/tools/HalyardMain.java
87.09% <ø> (ø)
11 <0> (ø)
:arrow_down:

...ava/com/msd/gin/halyard/tools/HalyardBulkLoad.java
80.56% <0%> (-1.33%)
10 <0> (ø)

...java/com/msd/gin/halyard/tools/HalyardSummary.java
80.21% <80.21%> (ø)
5 <5> (?)

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 0ab6440...4be8861. Read the comment docs.


Changing gotoh directory name broke directory path

From nohup/cronlog:

Traceback (most recent call last):
File "/home/covid/gotoh2.py", line 262, in
aligner = Aligner()
File "/home/covid/gotoh2.py", line 25, in init
files = pkgres.resource_listdir('gotoh2', 'models')
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 1190,$
resource_name
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 1486,$
return self._listdir(self._fn(self.module_path, resource_name))
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 1574,$
return os.listdir(path)
OSError: [Errno 2] No such file or directory: '/home/covid/models'

Looks like it's still trying to run the gotoh2 package from the local directory instead of the default Python package installation directory..

Hacky solution was to edit lines 25, 29 from "gotoh" -> "gotoh-src" in local files.

This isn't acceptable - we need to figure out a more permanent fix.

Can you please post a minimal reproducible example?

ewong347@Langley:/home/covid$ python gotoh2.py --append test.fa GISAID-2020-05-15.fasta NC_045512.fa

Traceback (most recent call last):
File "gotoh2.py", line 262, in
aligner = Aligner()
File "gotoh2.py", line 25, in init
files = pkgres.resource_listdir('gotoh2', 'models')
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 1190, in resource_listdir
resource_name
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 1486, in resource_listdir
return self._listdir(self._fn(self.module_path, resource_name))
File "/usr/lib/python2.7/dist-packages/pkg_resources/init.py", line 1574, in _listdir
return os.listdir(path)
OSError: [Errno 2] No such file or directory: '/home/covid/models'

Ok, the problem was having a script in the directory named gotoh2.py.  Seems to be a self-referencing problem.  Use this:
art@Langley:/home/covid$ python update.py -ref NC_045512.fa GISAID-2020-05-15.fasta test.fa
Aligning new sequence hCoV-19/USA/IL-NM06/2020|EPI_ISL_444520|2020-03-17

Updated gotoh2 installation on Langley to latest commit (with changes from @RouxCil merged in - e7605993f2f6fd0ae05f317a3903ec7f83cdd9ad)


Is there a way to namespace data with key-prefixing for logical isolation of data in multi tenant environments?

We are running many instances of E2E tests on the same redis instance - and we would like a way to prefix each instance with their own namespace so as to only be concerned with their own data.
Is there any way to accomplish this?


Improve exception handler for BugdownRenderingException

Currently, if we have a BugdownRenderingException, we send an email to admins with an exception report that just contains a sanitized version of the input in question, e.g. this:
Failed message: '[xxxx](xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx)'
This block  in zerver/lib/bugdown/__init__.py contains the logic in question:
        if len(rendered_content) > MAX_MESSAGE_LENGTH * 2:
            raise BugdownRenderingException()
        return rendered_content
    except Exception:
        cleaned = privacy_clean_markdown(content)

        # Output error to log as well as sending a zulip and email
        log_bugdown_error('Exception in Markdown parser: %sInput (sanitized) was: %s'
                          % (traceback.format_exc(), cleaned))
        subject = "Markdown parser failure on %s" % (platform.node(),)
        mail.mail_admins(
            subject, "Failed message: %s\n\n%s\n\n" % (cleaned, traceback.format_exc()),
            fail_silently=False)

        raise BugdownRenderingException()
    finally:
        current_message = None
        db_data = None

I believe we don't use our standard exception reporting logic in zerver/logging_handlers.py and zerver/lib/error_notify.py, in large part because those can end up sending a Zulip message by ERROR_BOT, and we really don't want that to happen with a Bugdown exception, since that could throw the same exception in the exception reporting code.
But we want to have the other nice features of error_notify.py, especially the data about which user got the exception (so one can contact the user).
So I think there's two reasonable options for how to improve this:
(1) Go through the normal logging_handlers.py handler code path, but check the exception class in there, and use that to suppress the ERROR_BOT code path.  With a unit test, this should be pretty safe.
(2) Call the error_notify functions directly, passing in the arguments needed.  The thing I don't like about this is that it seems liable to break over time if/when we extend the error_notify API, but that could be fine with a good test.
Tagging this as a priority, since it makes debugging markdown exceptions feel unattractive as a project.
@gnprice thoughts?

@zulipbot claim


Support dynamic lazy imports

Cloudflare workers support imports natively, including runtime import() expressions.
So potentially we could lazy load files by uploading such modules as "external" alongside the entry-point file.
Currently, our build process will bundle all static imports (and statically analysable dynamic imports) into the main entry-point bundle: import()-ed modules are wrapped in a function, actually executing the import() executes the inlined code.
One thing this approach doesn't currently support is imports where the module specifier is not known at build time. For example:
a.js
export const foo = 'A';

b.js
export const foo = 'B';

index.js
async function main() {
  const p = Math.random() > 0.5 ? 'a' : 'b';
  const foo = (await import(`./${p}.js`)).foo;
}
main();

This issues tracks the support for being able to do this via Wrangler 2.

Closing as a duplicate of #637, which is a superset of this really.

Actually, it's different. That's about lazy imports (await import(<string literal>)), while this is about dynamic imports (await import(variable))

Closing for now, since --no-bundle supports this use case. If there isn't an issue already open for tracking the new module work that's in-progress, we should make one.

Should the docs for --no-bundle be updated to reflect that using --no-bundle supports dynamic imports?
https://developers.cloudflare.com/workers/wrangler/commands/

@irvinebroque Yes, they probably should

Maybe we should reopen this, and make it work even with the bundling usecase
ref: https://github.com/cloudflare/workerd/issues/2372

This seems to work, even without --no-bundle combined with a TS worker. I thought I mention it because this issue is still open. If anyone is interested on how to get this to work - please comment and I'll provide some code.


fix: Collection Gallery Image low quality

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #7141
[ ] Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[x] My fix has changed UI

Copilot Summary

🤖 Generated by Copilot at e6290d3
Added custom banner images for collections. Updated collection-banner component to use toOriginalContentUrl function for IPFS images.

🤖 Generated by Copilot at e6290d3

Sing, O Muse, of the skillful coder who devised
A wondrous feature for the collection-banner component,
That it might display the glorious images of collections
Fetched from the IPFS hash by the toOriginalContentUrl function.

much better now (video compression doesn't do justice ofc 😄 )
https://github.com/kodadot/nft-gallery/assets/36627808/8099140f-7cc6-492a-a39d-85d5d46c7118

pay 30 usd

😍 Perfect, I’ve sent the payout
💵 $30 @ 4.25 USD/DOT ~ 7.059 $DOT
🧗 16SjUbGKSdjCdWTy3NNT3JxbRVGGqD4mwkHpc6BD9U2Rp29Z
🔗 0x56c408b4cf6a4f5c050496b93dd3b6cb9fed0a3f13ac0521cc4fd791e4ac2ff7
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


fluidsynth: 2.2.9 -> 2.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/FluidSynth/fluidsynth/releases.
meta.description for fluidsynth is: "Real-time software synthesizer based on the SoundFont 2 specifications"
meta.homepage for fluidsynth is: "https://www.fluidsynth.org"
meta.changelog for fluidsynth is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.3.0 with grep in /nix/store/nvdvwfbkjq01y8yb9v6aagi5si1s105f-fluidsynth-2.3.0
found 2.3.0 in filename of file in /nix/store/nvdvwfbkjq01y8yb9v6aagi5si1s105f-fluidsynth-2.3.0

Rebuild report (if merged into master) (click to expand)

955 total rebuild path(s)

955 package rebuild(s)

First fifty rebuilds by attrpath
ChowKick
SDL2_mixer
SDL_mixer
Sylk
_1oom
_20kly
abbaye-des-morts
abuse
adapta-gtk-theme
alfaview
alfis
almanah
altair
amberol
anki-bin
anytype
apache-directory-studio
apostrophe
appimage-run
appimage-run-tests
apple-music-electron
aravis
arcanPackages.all-wrapped
arcanPackages.arcan
arcanPackages.arcan-wrapped
arcanPackages.durden-wrapped
arcanPackages.pipeworld-wrapped
arcanPackages.prio-wrapped
arcanPackages.xarcan
asc
ashpd-demo
astroid
audacious
audacious-plugins
audaciousQt5
audio-recorder
augustus
authenticator
aws-workspaces
bada-bib
ballAndPaddle
balsa
banking
baobab
baresip
barrage
beekeeper-studio
beneath-a-steel-sky
bespokesynth
bespokesynth-with-vst2

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nvdvwfbkjq01y8yb9v6aagi5si1s105f-fluidsynth-2.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fluidsynth https://github.com/r-ryantm/nixpkgs/archive/631e127173893f75da35865f8e1c0b68834f7042.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nvdvwfbkjq01y8yb9v6aagi5si1s105f-fluidsynth-2.3.0
ls -la /nix/store/nvdvwfbkjq01y8yb9v6aagi5si1s105f-fluidsynth-2.3.0/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @cillianderoiste @lovek323 for testing.

Cross-compiled for aarch64 and the command fluidsynth --version runs fine in an emulator.


Local and Remote Memory Bandwidth (LMB and RMB) metrics are 0

          Hello,

I'm experiencing an issue where Local and Remote Memory Bandwidth (LMB and RMB) metrics are not displayed in Prometheus, despite proper configuration and troubleshooting steps taken.
Steps and observations:

Based on a suggestion, I disabled the Linux RDT driver via RESCTRL with export PCM_USE_RESCTRL=0 and unmounted resctrl to allow PCM direct access to RDT. However, this did not resolve the issue.
I attempted to create a custom monitoring solution that collects only the LMB and RMB using the output from pqos-msr and integrate this data with PCM data in Prometheus. Unfortunately, I encountered issues running PCM server and pqos monitoring simultaneously. The error message indicates that monitoring on core 0 is already started.

I would appreciate any insights or potential solutions
Thank you for your assistance.
Originally posted by @bellalzohir in https://github.com/intel/pcm/issues/761#issuecomment-2254615723

please see my responses in the original issue


request a tiny tweak to improve debugging clang include dir locating

Brief Description
Can we print the full path for the clang header, instead of relative. which is not so helpful.
the throw line, SetupIncludes() in  src/Parser/ParserOptions.cs
   private void SetupIncludes()
    {
        // Check that the builtin includes folder exists.
        if (!Directory.Exists(BuiltinsDir))
            throw new Exception($"Clang resource folder '{BuiltinsDir}' was not found.");

//  tested it later on Linux,  a property can be put into {}.
OS:  Linux (include version and/or distro)
Used headers
Used settings
Target: MSVC/GCC/Clang
Other settings
Stack trace or incompilable generated code

I had problem to run build.sh on windows git-bash.exe.
unix2does build.sh does not help.   Later I found, I must delete the old cppsharp repo, and clean git clone, otherwise some old lua downlaod, cause error.
$ ./generate.sh
Error: invalid option 'configuration'

I'm happy you've resolved the problem. Do you have further related trouble or may we close this?

We should change the code to print an absolute path of the dir before closing this, as suggested.

Thank you, please close it after add more print.


fix(bch-js): Updating to v6.2.14

:tada: This PR is included in version 2.4.9 :tada:
Your semantic-release bot :package::rocket:


Support *zap.SugaredLogger on zap instrumentation

It appears the current instrumentation does not support Suggared Logger interface. It also does not return the actual Zap Interface and creates conflict with app usage.

@vmihailenco the second part makes sense was just wondering if that could become a quick drop in replacement but having SuggaredLogger sounds great.

Sugared API should be supported in v1.0.5 - see otelzap


FIX "data.typ" test

Today I tried a simple example with csv crate, which counts correctly. Maybe it's a problem with the version.


Feature Request - Grid Filter mod filter support for 1.10 (SkyFactory 3)

Issue description:
Request Grid Filter mod filter support for 1.10 (SkyFactory 3)
What happens:
Option doesn't exist
Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: 12.18.2215
Refined Storage: 1.2.17

Maybe!


openxr: Sync with upstream 1.0.31

May help with #83674, but I don't count on it.

This doesn't help #83674, I'll revert to 1.0.28 instead.


Use @StateObject backport to avoid crash on iOS 13

I've used another package, shaps80/SwiftUIBackports, to enable this; how do you feel about introducing other dependencies?

I don't think it's a good idea to import a whole package just for this. Maybe the package can be used as a guide/template for a custom @StateObject instead? https://github.com/shaps80/SwiftUIBackports/blob/main/Sources/SwiftUIBackports/Shared/StateObject/StateObject.swift

@maguerrieri Thanks very much for opening this PR - I hadn't realised the library was crashing on iOS 13!
I agree with @theedov that adding a dependency on SwiftUIBackports feels a bit heavy, and I also wonder if a backported StateObject could be added manually without the dependency. Perhaps it could default to SwiftUI.StateObject in iOS 14 and above? I also wonder if it would be enough to add this in an ios13 branch that supports iOS 13, and make the main library support >= iOS 14. My assumption is that very few people are building SwiftUI apps that support iOS 13, but maybe I'm wrong?

Thanks for the feedback!
For the package dependency, what would y'all think of reducing it to just the StateObject backport? Like, fork the package and remove everything else?
I do also like the idea of using real StateObject on 14+, but trying briefly I didn't come up with a neat way to switch implementations for a property wrapper. I can fiddle with it some more this week; do y'all have any suggestions there?

Thanks @maguerrieri! I'll try to experiment with lightweight ways of achieving this too.

I think the best approach for this would be to maintain an ios13 compatibility branch with the proposed change and bump the main library's minimum deployment to iOS/tvOS 14.0.

That seems like a reasonable solution, though how will it interact with release versioning? Does SPM handle the semver metadata field okay? e.g. 0.3.1+ios13, or something?

I've set up an iOS 13 branch with your changes - thanks a lot @maguerrieri ! I've also bumped the minimum deployment targets for iOS and tvOS to 14.

how will it interact with release versioning? Does SPM handle the semver metadata field okay? e.g. 0.3.1+ios13, or something?

As far as I can tell, SPM allows you to specify metadata in a dependency version, but I'm assuming it wouldn't resolve newer versions automatically (e.g.  I don't think specifying from: "0.3.1+ios13" would resolve to a version 0.3.2+ios13 if available).

Great, thanks! 🙂
We exactly specify all of our versions, so not getting automatic updates suits me fine. Any other ideas on how to handle versioning?

We exactly specify all of our versions, so not getting automatic updates suits me fine. Any other ideas on how to handle versioning?

Thanks for clarifying @maguerrieri. In that case, I think specifying either the ios13 branch or a specific commit is probably the best approach. Reading the semver spec, it seems like build metadata is intended for other purposes.


Weather Widget: Forecast shows today instead of tomorrow

With the weather widget, the forecast option starts with today instead of tomorrow, resulting in a duplicate when you also have current weather turned on. I suppose that could be intentional, but if so, it would be nice to have the option either way.

This is intentional, as the "current" weather truly is current, where as "today" in the forecast is what it will be on average for the day. For example, you might wake up in the morning and it will show it is cold on the current, but on the forecast it shows it will be warm for the day so you can properly prepare for the day.
That being said, maybe I can add an option that will skip the current day within the forecast.

My pleasure!


node directory not found

Node directory not found...beginning NodeJS download for Windows.
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (35) schannel: next InitializeSecurityContext failed: Unknown error (0x80092012) - The revocation function was unable to check revocation for the certificate.
tar: Error opening archive: Failed to open 'node.zip'
The system cannot find the file specified.
Could Not Find G:\Unreal\PixelStreamingtest\Windows\Samples\PixelStreaming\WebServers\SignallingWebServer\platform_scripts\cmd\node.zip
Node version:
The system cannot find the path specified.
CoTURN directory found...skipping install.
The system cannot find the path specified.
The system cannot find the path specified.
Press any key to continue . . .
Please guys, any help? i don't even know what to do

It’s downloading a zip file, so it seems like you’re using nvm-windows, which is a different project.

If I’m incorrect, then please fill out the entire issue template - it’s there for a reason.

First off, i really thank you for your response.

the image above is what I downloaded as nodejs

and sincerely I don't know how to proceed because I followed the tutorial in the getting started with UE

If I’m incorrect, then please fill out the entire issue template - it’s there for a reason.

maybe cause I'm new to the pixel streaming thing but could you walk me through a step-by-step or just direct me to a tutorial that does in UE5.03 because it seems the versions are different
kind regards

That’s node itself - this repo is for nvm, a node version manager. I have no idea what a pixel is or how it works.


feat: Add bit size to const opcode

Please read contributing guidelines and remove this line.

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 5a71bb95 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,412
179,524
716,068

l1_rollup_calldata_gas
221,692
866,908
3,448,408

l1_rollup_execution_gas
314,179
984,648
3,668,065

l2_block_processing_time_in_ms
1,159
4,349
17,252

note_successful_decrypting_time_in_ms
305 (-1%)
965 (-3%)
3,641 (-2%)

note_trial_decrypting_time_in_ms
12.2 (-20%)
98.7 (-20%)
142 (-3%)

l2_block_building_time_in_ms
17,727
70,460
282,201

l2_block_rollup_simulation_time_in_ms
13,062
51,918
208,296

l2_block_public_tx_process_time_in_ms
4,637
18,465
73,663

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,656 (-8%)
25,931 (-7%)

note_history_successful_decrypting_time_in_ms
2,462 (+3%)
4,667 (+1%)

note_history_trial_decrypting_time_in_ms
168 (+73%)
246 (+12%)

node_database_size_in_bytes
17,322,064
33,099,856

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
311
44,481
26,193

private-kernel-ordering
193 (+1%)
44,513
14,929

base-rollup
1,407
128,970
881

root-rollup
82.3
4,088
677

private-kernel-inner
414
71,640
26,193

public-kernel-private-input
240
32,615
26,193

public-kernel-non-first-iteration
239
32,657
26,193

merge-rollup
7.62
2,608
881

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,575
38,271 (+1%)

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
573 (+1%)
1,214

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.504 (-21%)
572


FeignClient Health Indicator Feature

The idea of this feature is to automatically create HealthIndicator for FeignClient
@FeignClient(name = "healthClient", health = "healthMethod")
public interface HealthClient {

    @RequestMapping(method = RequestMethod.GET, value = "/health")
    ResponseEntity<String> healthMethod();

}

Can you explain more about why you want this feature and what exactly this feature does?

Sometimes our app depend on other app, so to make our app run well, we need to make sure that other app is live. To do that, we need to create HealthIndicator manually.
With the ability to automatically create HealthIndicator in FeignClient, it's really helpful because we  don't need to create HealthIndicator manually

I've marked it for team discussion, I don't know if this is something everyone would want/need.

Codecov Report

Merging #1972 into master will increase coverage by 0.1%.
The diff coverage is 89.79%.

@@            Coverage Diff            @@
##           master    #1972     +/-   ##
=========================================
+ Coverage   76.22%   76.33%   +0.1%
=========================================
  Files         209      210      +1
  Lines        6377     6426     +49
  Branches      794      797      +3
=========================================
+ Hits         4861     4905     +44
- Misses       1178     1182      +4
- Partials      338      339      +1

Impacted Files
Coverage Δ

.../netflix/feign/FeignClientHealthConfiguration.java
89.47% <89.47%> (ø)

...ork/cloud/netflix/feign/FeignClientsRegistrar.java
77.66% <90.9%> (+0.78%)
:arrow_up:

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 34350cb...ab95b5f. Read the comment docs.

cmiiw, at the moment, there are no feign-based built-in health-indicators,
https://docs.spring.io/spring-boot/docs/current/reference/html/production-ready-endpoints.html#_auto_configured_healthindicators
is it possible to make this as one of those?
@khannedy , @spencergibb

I dont understand why you wouldn't just use the health endpoint from actuator.

@ryanjbaxter actually this code will automatically register HealthIndicator for actuator, so it will automatically displayed in actuator endpoint

@khannedy it looked to me like you were creating another endpoint, /health, I was confused why you needed a new endpoint.

@ryanjbaxter not it's just feign endpoint for target server, maybe target server using different endpoint for healthcheck, so we can customize the endpoint, but in the app, the health indicator is registered in actuator feign
for example we have auth service
http://auth-service/
with health endpoint
http://auth-service/healthcheck
in our app, example retail service, so we can create feign for auth service
@FeignClient(name="authService", url="http://auth-service", health="healthMethod")
interface AuthFeign {

   @RequestMapping("/healthcheck")
   ResponseEntity<String> healthMethod();

}

our retail-service endpoint still using actuator health endpoint
http://retail-service/health
and the result for example
{
   "status" : "UP",
   "authFeign" : {
      "status" : "UP",
      "responseCode" : 200,
      "responseBody" : "..."
   },
   "redis" : {
      "status" : "UP"
   },
   "..." : {
   }
}

I think a generic declarative HTTP based health indicator might be something we could think about, but we don't want to maintain a Feign based health indicator.


Fix issue with lazy val on Scala Native + Scala 3.2

Scala Native depends on the scala3-library 3.1.3.
To work on Scala 3.2 you need to explicitly depend on scala3-library
for 3.2.0.
In Scala.js it is a single artifact generated in the dotty repository,
so it's platformed. In Scala Native, however, we need to depend on the
jvm artifact, which is used by the compiler, so we patch
scalaLibraryIvyDeps to be platformed = false

Looks good to me, so far.


Upload of album not complete

Testserver: qa-gluons, spot.mpdl.mpg.de/rest
Test date: 23.02.2017
Device: Tablet Nexus 9 Android 5.1.1.
User: standard user
Action: upload album
select album -> "select all" -> upload -> select collection ->
Observation:
only one item (out of four) was uploaded to collection
or
upload task wasn't created for the items

I tried different collections and albums, cannot reproduce the bug.

this happens only on qa-gluons, it works fine on gluons

cannot reproduce this on spot.mpdl.mpg.de/rest

Testserver: qa-imeji.mpdl.mpg.de
Test date: 28.09.2017
Device: Tablet Nexus 9 Android 5.1.1.
result: ok, cannot reproduce on qa-imeji anymore


Possibility to create drop-zone dynamically on drag-start

It would be nice to have an optional callback in dragstartHandler before ANGULAR_DRAG_START event is triggered.
We would need it to dynamically create the element which holds the 'drop-channel' directive on drag-start.
Creating this element on ANGULAR_DRAG_START is to late.

+1

Is anyone interested in sending a PR for this feature ?

I accomplished something similar by creating the element in the html, but ng-hide it until a drag is in progress.  However, in the ANGULAR_DRAG_START event callback, I had to do $scope.$applyAsync() to get the element to show up.  regular old $apply() doesn't work.


[windows] XFAIL cross-module-optimization test in Windows

@eeckstein: Maybe you can help to find out what's happening here. The new test failed on Windows. It seems that the metadata for the class Test.Container.Base is not being loaded, or it is not pointing to the right place. The executable segfaults in Windows trying to print something, and we believe that it is the first line of the test program [Test.Container.Base]. Any help to understand what's happening would be appreciated.
Just in case it is not an easy fix. This should mark the test as failing on Windows.
Introduced in #28407 and first seen in https://ci-external.swift.org/job/oss-swift-windows-x86_64/2104/

There's another Windows error at the moment, so this will not report a success anyway.
@swift-ci please test Windows platform

@swift-ci please smoke test

@swift-ci please clean test Windows platform

Maybe metatype lookup by mangled name for private types is not working. The reason why it should work is because metaypes are now referenced by symbolic addresses in the mangled name. This was done recently by @jckarter. Maybe for some reason there is a problem with that on Windows.
If this is really the case, there are 2 possibilities to fix that:

Support the PrivateDeclName mangling node in MetadataLookup.cpp in the runtime.

Change the mangling scheme for private entities. This is more heavy weight and I would not recommend it. But if you want to try, I have a prototype patch here: https://github.com/eeckstein/swift/commit/b4b9f1341a0d721dfdb429279da6c4ea4599d2e2
It might be useful to check if it solves the problem.

The first thing what you could do is to debug the runtime and see if swift_getTypeByMangledName works or fails.
Unfortunately I don't have a possibility to debug this myself on windows.

Under no circumstances should we try to demangle private type names from strings. The issue with Windows is likely that always-use-symbolic-references is not enabled on Windows, because swift-reflection-dump does not yet support resolving indirect references across DLLs for PE object files.

Thanks for the answers. I will try to see if that change makes a difference. Breaking more tests is not the perfect solution, though. It seems that we are crashing into the same problem with swift-reflection-dump, but I don't know if I have the knowledge or the time to deal with it.

From an engineering tradeoffs point of view, my judgment would be that getting the performance, memory, and code size benefits of compacted mangled names and cross-module optimization is probably more important than swift-reflection-dump. It is hopefully straightforward to update swift-reflection-dump, though, since for PE you should hopefully be able to recognize indirect references into the import table and map those to import entries to extract the symbol name they're referring to.

Thanks for the clue. Effectively removing that block makes the test pass in Windows. In exchange it breaks at least two tests in the Reflection folder. We will try to work on swift-reflection-dump, but I don't think we can promise any timeline. Would anyone have a problem XFAIL-ing this test for the time being?

XFAILing the affected reflection tests on Windows seems fine to me.

@swift-ci please smoke test

Changed the implementation and disabled the two test that seems to fail because of this changes. Let's see if CI finds some other ones.
@swift-ci please clean test Windows platform

@swift-ci please clean test Windows platform


trouble saving options (parameter must implement Countable, cannot modify header info)

When I try to save my options at wp-admin/options-general.php, I get the following errors:

Warning: count(): Parameter must be an array or an object that implements Countable in /app/public.built/wp-admin/includes/template.php on line 1447

Warning: Cannot modify header information - headers already sent by (output started at /app/public.built/wp-admin/includes/template.php:1447) in /app/public.built/wp-includes/pluggable.php on line 1210

Any idea how to fix this? Thanks

Also, sometimes all my pages and menus go missing, and I get the following error:

Warning: mysqli_set_charset(): Error executing query in /app/public.built/wp-includes/wp-db.php on line 823

I saw that someone had a similar problem and its solution was to edit wp-config.php with something like the following:
define( 'DB_CHARSET',           'utf8mb4'                         );
define( 'DB_COLLATE',           'utf8mb4_general_ci'              );

So I did that, reinstalled wordpress and everything, but still I get this error. I think it occurs when I use the site too much.

ah, the fix was very easy, this is what I did:
             "type": "package",
             "package": {
                 "name": "WordPress/WordPress",
-                "version": "4.8.3",
+                "version": "4.9.1",
                 "dist": {
                     "type": "zip",
-                    "url": "https://github.com/WordPress/WordPress/archive/4.8.3.zip"
+                    "url": "https://github.com/WordPress/WordPress/archive/4.9.1.zip"
                 }
             }
         },

problem solved. thanks for the amazing software.


Unable to download large mailbox PST files

Describe the bug
When I download the large mailboxes (15GB) as PST file it's redirecting to "hostanme/download.php" blank page and nothing happen after that.  But the restore session still running on the VBO server and PST file downloading to the VBO server temp directory.

This is a known issue of the portal when exporting large accounts (mailbox/onedrive/site). This is on my todo to find a workaround but so far the only solution I could think of would be to run the export process in the background, store the file somewhere and create a sort of 'download basket'.


Update New-SPEnterpriseSearchCrawlContentSource.md

Fixed example formatting.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/New-SPEnterpriseSearchCrawlContentSource.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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


Aggressive Retries from "error instantiating route53 challenge solver"

Describe the bug:

When using the Route53 Solver for ACME, I am attempting to use an IRSA (IAM Role for Service Accounts).  However, I have a permissions error, causing sts:AssumeRoleWithWebIdentity to fail.
When this happens, cert-manager goes into an aggressive loop, retrying to delete the challenge:
E1006 00:10:50.308276       1 sync.go:282] cert-manager/challenges/finalizer "msg"="error cleaning up challenge" "error"="error instantiating route53 challenge solver: unable to assume role: WebIdentityErr: failed to retrieve credentials\ncaused by: AccessDenied: Not authorized to perform sts:AssumeRoleWithWebIdentity\n\tstatus code: 403, request id: eb150160-1a0a-4f2a-8b7f-32afa59f863f" "dnsName"="sandbox.example.com" "resource_kind"="Challenge" "resource_name"="app-external-tls-8tqdh-164930377-4091118618" "resource_namespace"="default" "resource_version"="v1" "type"="DNS-01"
I1006 00:10:50.358522       1 util.go:84] cert-manager/controller/orders/handleOwnedResource "msg"="owning resource not found in cache" "related_resource_kind"="Order" "related_resource_name"="app-external-tls-8tqdh-164930377" "related_resource_namespace"="default" "resource_kind"="Challenge" "resource_name"="app-external-tls-8tqdh-164930377-4091118618" "resource_namespace"="default" "resource_version"="v1"
E1006 00:10:50.414506       1 sync.go:282] cert-manager/challenges/finalizer "msg"="error cleaning up challenge" "error"="error instantiating route53 challenge solver: unable to assume role: WebIdentityErr: failed to retrieve credentials\ncaused by: AccessDenied: Not authorized to perform sts:AssumeRoleWithWebIdentity\n\tstatus code: 403, request id: 79f7c88a-c0d4-4654-8b81-ae3353e3cd2e" "dnsName"="sandbox.example.com" "resource_kind"="Challenge" "resource_name"="app-external-tls-8tqdh-164930377-4091118618" "resource_namespace"="default" "resource_version"="v1" "type"="DNS-01"
I1006 00:10:50.473561       1 util.go:84] cert-manager/controller/orders/handleOwnedResource "msg"="owning resource not found in cache" "related_resource_kind"="Order" "related_resource_name"="app-external-tls-8tqdh-164930377" "related_resource_namespace"="default" "resource_kind"="Challenge" "resource_name"="app-external-tls-8tqdh-164930377-4091118618" "resource_namespace"="default" "resource_version"="v1"
E1006 00:10:50.510828       1 sync.go:282] cert-manager/challenges/finalizer "msg"="error cleaning up challenge" "error"="error instantiating route53 challenge solver: unable to assume role: WebIdentityErr: failed to retrieve credentials\ncaused by: AccessDenied: Not authorized to perform sts:AssumeRoleWithWebIdentity\n\tstatus code: 403, request id: f4a66d72-d3c9-4bf7-b2a6-578e214df9d1" "dnsName"="sandbox.example.com" "resource_kind"="Challenge" "resource_name"="app-external-tls-8tqdh-164930377-4091118618" "resource_namespace"="default" "resource_version"="v1" "type"="DNS-01"

It is trying to do this HUNDREDS of times per second.  I had to
kubectl scale --replicas=0 -n cert-manager deployments/cert-manager to stop it / try to cleanup the cluster issuer.
Expected behaviour:

Some kind of backoff.
Stop after N or the backoff is high enough it doesn't matter.

Steps to reproduce the bug:

Misconfigure IRSA for Route53 ACME sovler.
Anything else we need to know?:
Environment details::

cert-manager version: 1.9.1
Install method: helm

/kind bug

I ran into this too

/remove-lifecycle stale

Steps to reproduce the bug:
Misconfigure IRSA for Route53 ACME sovler.

@pquerna could you please explain how you fixed your configuration?

/reopen

Any misconfiguration that prevents it from assuming a role would work I assume. In my case I accidentally told it to assume a policy instead of a role with the policy attached. Even after I fixed the config, the existing solver wouldn't clean up and it very aggressively gave errors like above, like several per second aggressive, until I manually patched out the finalizers. It was so disruptive that new request weren't even picked up until I killed the solver.

/remove-lifecycle stale

Inserting five seconds of sleep before retrying anything would be appropriate.

/remove-lifecycle stale
Still an issue, a retry backoff should be standard design.

similar request to https://github.com/cert-manager/cert-manager/issues/6308

/remove-lifecycle stale
Still an issue, a retry backoff should be standard design.

/remove-lifecycle stale

Related

https://github.com/cert-manager/cert-manager/issues/7234
https://github.com/cert-manager/cert-manager/issues/6308
https://github.com/cert-manager/cert-manager/issues/6230

A beta-release is now available which contains the fix for this issue. Please test and feedback if you have time.

https://github.com/cert-manager/cert-manager/releases/tag/v1.16.0-beta.0


pip and conda package names incompatible

The conda-forge package is installed as pysal and the PyPI one as PySAL. Therefore this situation can now happen after e.g. (pip install pysal && conda install geopandas):
$ conda list pysal
# packages in environment at /Users/rgommers/anaconda3:
#
# Name                    Version                   Build  Channel
pysal                     1.14.3                   py36_0    conda-forge
PySAL                     1.14.3                    <pip>

Due to the different names, conda does not pick up that pysal was already installed with pip, and just reinstalls it when it's needed as a dependency for something else.
The best solution would be to name the package on PyPI all-lowercase. This is normal practice for Python packages (e.g. NumPy and SciPy are packages as numpy and scipy) and avoids such weirdness. And users would anyway expect that, they'd type pip install pysal not pip install PySAL.

Due to the different names, conda does not pick up that pysal was already installed with pip, and just reinstalls it when it's needed as a dependency for something else.

That is true even with identical names and there is not much we can do about it for now.
BTW both pip and conda uses lower case only internally but pip abstracts this out so packages seems case sensitive while conda chooses to make everything lower case by default.

Thanks @ocefpaf. I mixed up two things; installing with pip after having installed already with conda does work as expected. Sorry for the noise.


Alpha-ess update broke charge and discharge sensors

Hi Charles,
I am sure you are aware that Aplha-ess was down last night for maintenance and it seems that they have changed something that is related the the charge and discharge sensors. While all other sensors continue to work as expected these two seem to be reporting 0 values today
Ralph's

I wasn't aware - thanks for letting me know.  I will try to see what changes need to be made.

It does seem as if quite a lot has changed/been broken  - the new product information menu item for example, and my predictive power has been nulled out. I had reported an issue where phantom PV generation was appearing overnight and Alpha responded that they found they had been calculating some values incorrectly and that they were working on a fix. AFAIK that has not been resolved yet, or at least the ticket hasn't been closed. In addition, I think I see new keys in the SticsByDay - can't be certain because my notes on this aren't great but I don't remember all of the ones currently being returned.
It would be really useful if we were able to compare our systems to see if the data returned is in the same format for all types. I have a Smile B3.

Right now I'm not able to logon to the alphaess.com site at all - I keep getting a 404 error.  It looks like the official mobile app is also not reporting real time statistics either.  I'm going to leave it a few more hours to see if Alpha will finish up making their changes.  I deleted the downtime message from them - does anyone still have it?  Can you see what time they thought they would be done at?

Yes, the message was:
To provide you with better service, the system of Alpha ESS Monitoring with be maintained from 13:00 to 18:00 (UTC+8, Beijing) on October 30, 2021.

They were supposed to be done at 6pm(+8gmt)on Saturday, I suspect they have fudged the update on there side, I am still having intermittent issues with the mobile app as well, sometime 500 sometime "no system" errors

It does look as if the whole system has gone down. My Smile has lost connection to the cloud.

I also got this message when attempting to login this morning:
To provide you with better service, the system of Alpha ESS Monitoring will be maintained from 17:00 to 19:00 (UTC+8, Beijing) on October 31, 2021. We are notifying you that you will not be able to login to the Alpha ESS Monitoring website (www.alphaess.com) during the system maintenance. We appreciate your understanding.

My battery failed to charge last night during the Octopus Go period. Hoping they fix it before power hour at 12:00-13:00 today.

Site appears to be back up - the statistics by period API (https://www.alphaess.com/api/Power/SticsByPeriod) which I use for most data is working as expected - with at least one new improvement where the charge from PV and grid is now correctly broken out.
Unfortunately the charge/discharge figures from the system statistics API (https://www.alphaess.com/api/Statistic/SystemStatistic) appear to still be broken for the current day - all previous days are showing correctly but today is returning zero values.  That's the only API call I've been able to find that correctly returns the right figures for discharge.  It may be that there's just a delay in that coming back up after all of their changes.    More monitoring needed on this to see if it gets fixed or if I need a workaround.
Oh and yes the grid charging setup is broken on my Alpha also - I keep trying to enable it and it keeps getting reset back to disabled.

Seems like charge/discharge are working again this morning, i wonder if they gave up and rolled back or fixed it but i am seeing stats again. Not tried the charge/discharge function but i did see that it was not working yesterday is i tried to charge when prices were very low but it would just keep reverting to off.
I am guessing the release team did not have a very good weekend...


Updated us/oh/miami to more up-to-date source

This source has addresses from newer developments that were not present in the previous static zip file dump

This layer has address points if that's preferable:
http://services2.bhamaps.com/arcgis/rest/services/AGS_Miami_CO_OH_GISMap/MapServer/0

@trescube Thanks a bunch for pointing that out 🙌 Updated the source, and doing another sanity check when the source job has completed.

I can confirm that the data looks good AND that it has addresses from new developments in the county that were not in the previous dataset.
This is good to merge in my end.

Thanks, @MiniCodeMonkey!


Floating label overlapping input with value

http://localhost:8000/dist/e2e/input/form-inputs/

Fixed https://github.com/driftyco/ionic/commit/181a07024e8ee602fd989e36b41e9b86ca537b90


Units of marker, camera, and cursor positions

This PR addresses #10.
Camera and cursor coordinates from spinning-ball can now be reported in degrees. This PR incorporates the latest spinning-ball, and sets degrees as the default unit for cameraPos, cursorPos, and marker.position.
Along the way, I fixed a bug in tile-setter that was messing up feature selection across the antimeridian.

This PR introduces several breaking changes to the API:

cameraPos and cursorPos are both now methods returning arrays, rather than links to arrays
The signature of the method to add a marker is now .addMarker({ position }) rather than .addMarker({ lonLat })
Longitude and latitude are reported in degrees (affects cameraPos and cursorPos methods, and marker.position)

For stable code, this should trigger an increment of the major version number. However, since we are still at an early phase with major version 0, I think we should instead do our first increment of the minor version number, and tag this as 0.1.0. According to the semver documentation, this would be consistent with the expectations for versions 0.y.z

Sorry for the force push. This is just updating the version number as discussed. I put the wrong version number in the first commit message, so the force push fixes it


ProgramData files (faforever patch) not downloaded upon ladder game start

only custom game creates the files in programdata, ladder search does not.
That means installing the client on a clean maching and starting with a ladder match actually creates a crash, since init.lua and other files are missing.

Did you test that that it actually does?

This happened two times on two different PCs
The first time, there was no prior installation of the client nor of the game, so programdata files just weren't there. It crashed at launch because of init.lua missing, etc, and I had to host a custom game so that it would download required files.
The second time, there was an outdated FAF patch version since I hadn't played in a long time. Updated the client, tried to match Ladder against askaholic, the game crashed on launch because the patch version was different than the one I was trying to connect to. Had to create a custom games so that the client would update my FAF patch, then ladder worked.

Tried on production client downloads .nx2 files and faforever.faf and starts game normally

Can not reproduce this

Did you do this on the test server?
Because might be something there is configured wrongly...

I believe this happened when we were testing on the test server, yes.

Tell me if you find this happening on prod...

https://i.imgur.com/bj4pknj.png

As this issue is obviously not solved, please re-open.


[TEST] Fix unit test

:white_check_mark: Build finished in 3m 24s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/streaming/impls/mqtt

:exclamation: No tests found!
:information_source:  This is an automatic message

:white_check_mark: Build finished in 2m 16s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/streaming/impls/mqtt

:exclamation: No tests found!
:information_source:  This is an automatic message


Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!

input_variable and model have been mapped to cuda，while the RuntimeError still occured.
File "/lib/python3.6/site-packages/nni/compression/torch/quantization/quantizers.py", line 59, in update_ema
biased_ema = biased_ema * decay + (1 - decay) * value
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!
this happens in QAT Quantizer

Thank you for your asking. Can you post your code here?

The correct order of calling QAT should be like the example below.
optimizer = torch.optim.SGD(model.parameters(), lr=0.01, momentum=0.5)
quantizer = QAT_Quantizer(model, configure_list, optimizer)
quantizer.compress()
model.to(device)

If you map your model to cuda previous to using using quantizer.compress(), this error may occured. And when you train your model or do the inference, all of your data should be mapped to cuda first.

Thank you very much for both your contribution and  answer. My implementation has problems as you mentioned. This problem has been solved.

closing the issue as the problem has been solved.

Hi~ @linbinskn , I meet the same problem during using DataParallel, and I found QAT_Quantizer.compress() is inherited from Compressor.compress() which does nothing but just return self.bound_model. So what is meaning by calling quantizer.compress() before model.to('cuda')? And how can I solve the problem of RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!, really thanks if you have any advice.
https://github.com/microsoft/nni/blob/02eab99b9bb280e385a7af71b4c6c4a73bae1f04/nni/compression/pytorch/compressor.py#L118-L130

Maybe, when you are building model, you can set device_map like:
def build_model(pretrained_model_name_or_path: str, task_name: str):
is_regression = task_name == 'stsb'
num_labels = 1 if is_regression else (3 if task_name == 'mnli' else 2)
model = BertForSequenceClassification.from_pretrained(pretrained_model_name_or_path, num_labels=num_labels,device_map="cuda:1")
return model


callvirt instance support

Calling method from object.
When i call method from struct with interface, like a.GetValue(), i get
IL_002c: callvirt instance !0 class NlpGpuTools.Data.Types.IBaseMathPrimitive`1<float32>::GetValue()
Even if interface and target method has InlineMethod attribute.
Can you add InlineMethod.Fody support for this case?

I think callvirt's target method can be determined at runtime only
Do you have sample of code for this case?

Not needed. Now making weaver for custom INumeric without any reference to interface for ilgpu.


Error with click v8.0.3

Describe the bug

I was getting this error when importing ehrapy
ModuleNotFoundError: No module named 'click._bashcomplete'

A bit of googling seemed to suggest this might be an issue with newer versions of click. Downgrading to v7.1.2 seemed to fix it.
To Reproduce
Steps to reproduce the behavior:
Import ehrapy with click v8.0.3 installed
Expected behavior

Successful import with no error
System [please complete the following information]:

OS: MacOS 10.15
Language Version: Python 3.8
Virtual environment: conda

Additional context

Possibly (probably) I'm just doing something wrong but thought I would raise it just in case

Thanks for reporting. We can restrict click to 7 < 8 versions, but I think that we may have resolved this already on development. Will check.

Closed because I cannot reproduce it. We should test this with a wheel later...


What is the PieSlice control?

:) Thanks for the screen shot. I know what you mean. I plan to deprecate it and replace it with the RingSegment. I'll do that while I am thinking about it. Thanks for the reminder.

But what is it? Here's a demo: http://blog.jerrynixon.com/2012/06/windows-8-animated-pie-slice.html