Search by version

Is it possible to search which feed has a particular version of the package? Something along the lines of:
Microsoft.OData.Edm And version:7.6.0-beta

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

ID: 4488541f-e00f-aa81-4731-9649800e2640
Version Independent ID: 59b5b0cf-0d9f-c710-d9f5-e56a578f74f4
Content: advanced package search syntax
Content Source: docs/project/search/advanced-package-syntax.md
Product: devops
Technology: devops-collab
GitHub Login: @KathrynEE
Microsoft Alias: sumallel

Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a few options where you might consider asking your question:

Azure DevOps on Stack Overflow

Azure DevOps Support Bot

Currently you cannot search for packages using version. we are not indexing the version information. The intent is for users to find the package name and then traverse the version by getting into the package details. If you believe this is a feature that you would like to have in future releases then please consider creating a new feature request here:
https://developercommunity.visualstudio.com/search.html?f=&type=question+OR+problem+OR+idea&type=question+OR+problem+OR+idea&c=&redirect=search%2Fsearch&sort=relevance&q=package+search


Getting error in Train.py

Hello,Sir i'm getting error when i run Train.py.
FileNotFoundError:No such File or Directory: Optimized_thetas/model_2017-05-29_21-08-44_10.0_h50.pkl

Hi @BilalAh, This error is caused by the absence of the optimized_thetas directory. The commit 048d983 fixes the issue.


Fix io.anserini.ltr.FeatureExtractorCli for argument -collection clueweb

This pull request will close #688.
I would be happy if I could improve the robustness of io.anserini.ltr.FeatureExtractorCli further by adding some integration-tests.
If there is some interest in that just let me know, I would be glad to help.

Thanks for your contribution!
Other than minor formatting changes, we can merge.

@mam10eks do we have an update on how this is going?

Hi @lintool,
I am still working on this.
I have found some related problems in the RankLibReranker and this motivated me to clean up both parts.
Since I am working on this in my spare time I think I need two more weeks until I can finish this and have everything properly tested.
My goal is to reproduce some results reported in the "Learning to Rank for Information Retrieval" book (Tie-Yan Liu) with the Anserini-LTR-Pipeline to be sure that everything is fine.

@mam10eks How is the progress going? Are you still working on it?

Hi @MathBunny,
sorry for the long delay.
I had planned too much and there have been some changes in the Master Branch since then.
I will fix those stuff tomorrow and then provide a small merge request instead of a large one.

@MathBunny,
From my side this can be merged now.
There are still other issues in the LTR-Pipeline.
Is anyone else working on that?
I still work on that in my spare time, but it needs some weeks (I will split this into "smaller merge requests")
.

hi @MathBunny can you take a look? you're more familiar with the code than I am at this point... do a CR and if it's okay we'll merge.


[tsp-client] Support generating a lock file from emitter-package.json

Facilitate generating an emitter-package-lock.json file from an emitter-package.json for the language repositories.
Fixes: https://github.com/Azure/azure-sdk-tools/issues/7691

/check-enforcer evaluate


Explicitly mark Pod that use GPU devices

Is your feature request related to a problem? Please describe.
If enable  nvidia-device-plugin or tke-gpu-manager on kubegems plugins management page ,we should mark a plugin icons which pod used gpu devices. this feature just like nodes which enabled gpu.

Any further updates? @chenshunliang

This feature is merged in v1.22.0-beta.0 .


WSL --class not registered

I did a system reset due to some reason after that i was trying to install docker desktop but it showed some error regarding WSL when I tried to install WSL it showing class not registered I tried so many fixes from another issues and required windows features are also enabled but error still same it was working fine before system reset.

C:\Users\ak680>wsl --install
Class not registered
Error code: Wsl/CallMsi/Install/REGDB_E_CLASSNOTREG

Please help me solve this issue.
WslLogs-2024-10-31_08-19-34.zip

Logs are required for review from WSL team
If this a feature request, please reply with '/feature'. If this is a question, reply with '/question'. Otherwise please attach logs by following the instructions below, your issue will not be reviewed unless they are added. These logs will help us understand what is going on in your machine.
How to collect WSL logs
Download and execute collect-wsl-logs.ps1 in an administrative powershell prompt:
Invoke-WebRequest -UseBasicParsing "https://raw.githubusercontent.com/microsoft/WSL/master/diagnostics/collect-wsl-logs.ps1" -OutFile collect-wsl-logs.ps1
Set-ExecutionPolicy Bypass -Scope Process -Force
.\collect-wsl-logs.ps1

The script will output the path of the log file once done.
If this is a networking issue, please use collect-networking-logs.ps1, following the instructions here
Once completed please upload the output files to this Github issue.
Click here for more info on logging If you choose to email these logs instead of attaching to the bug, please send them to<EMAIL_ADDRESS>with the number of the github issue in the subject, and in the message a link to your comment in the github issue and reply with '/emailed-logs'.
View similar issues
Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it!
Closed similar issues:

wsl not responding just saying class not registered (#11892),  similarity score: 0.75
Error code: Wsl/CallMsi/REGDB_E_CLASSNOTREG (#10882),  similarity score: 0.73
Class not registered Error code: Wsl/CallMsi/REGDB_E_CLASSNOTREG (#11999),  similarity score: 0.73
Class not registered Error code: Wsl/CallMsi/REGDB_E_CLASSNOTREG (#11705),  similarity score: 0.72
Unregistered class Error code: Wsl/CallMsi/REGDB_E_CLASSNOTREG (#11641),  similarity score: 0.71

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

/question

Logs are required for review from WSL team
If this a feature request, please reply with '/feature'. If this is a question, reply with '/question'. Otherwise please attach logs by following the instructions below, your issue will not be reviewed unless they are added. These logs will help us understand what is going on in your machine.
How to collect WSL logs
Download and execute collect-wsl-logs.ps1 in an administrative powershell prompt:
Invoke-WebRequest -UseBasicParsing "https://raw.githubusercontent.com/microsoft/WSL/master/diagnostics/collect-wsl-logs.ps1" -OutFile collect-wsl-logs.ps1
Set-ExecutionPolicy Bypass -Scope Process -Force
.\collect-wsl-logs.ps1

The script will output the path of the log file once done.
If this is a networking issue, please use collect-networking-logs.ps1, following the instructions here
Once completed please upload the output files to this Github issue.
Click here for more info on logging If you choose to email these logs instead of attaching to the bug, please send them to<EMAIL_ADDRESS>with the number of the github issue in the subject, and in the message a link to your comment in the github issue and reply with '/emailed-logs'.
View similar issues
Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it!
Closed similar issues:

wsl not responding just saying class not registered (#11892),  similarity score: 0.75
Error code: Wsl/CallMsi/REGDB_E_CLASSNOTREG (#10882),  similarity score: 0.73
Class not registered Error code: Wsl/CallMsi/REGDB_E_CLASSNOTREG (#11999),  similarity score: 0.73
Class not registered Error code: Wsl/CallMsi/REGDB_E_CLASSNOTREG (#11705),  similarity score: 0.72
Unregistered class Error code: Wsl/CallMsi/REGDB_E_CLASSNOTREG (#11641),  similarity score: 0.71

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

/question

Diagnostic information
Issue was edited and new log file was found: https://github.com/user-attachments/files/17574455/WslLogs-2024-10-31_08-19-34.zip
Detected appx version: <IP_ADDRESS>
Detected user visible error: Wsl/CallMsi/Install/REGDB_E_CLASSNOTREG
Detected user visible error: Wsl/CallMsi/Install/REGDB_E_CLASSNOTREG

how can i resolve this issue

Same error code.
You can try to install the latest WSL from https://github.com/microsoft/WSL/releases
Works for me.

We've recently made some changes to detect broken COM registrations, could you try a new pre-release version of WSL and see if you are able to reproduce this?


Fix touch screen malfunction

Our display connected to the RPi, does not work well.

It turned out that I needed to choose a touchscreen in screen setting. The touchscreen was not checked before.
Preference > Screen Configuration > Layout > Screens > DSI-1 > Touchscreen > 10-0014 Goodix Capacitive TouchScreen


SDXL - Bring up

Model card: https://github.com/tenstorrent/tt-metal/issues/15963
Unit test sheet link:

SDXLT-UT-1024x1024
SDXLT-UT-512x512

Issues :
#16000 - ttnn.SiLU fails with OOM
#16001 - ttnn.group_norm fails with OOM issue
#16002 - ttnn.group_norm fails with low pcc
#16096 - Conv op fails in SDXL-turbo model (resnetblock)
#16125 - Low PCC in Conv with real weights, High PCC with random weights
Progress:

[x] DownBlock2D - PCC>0.99
[ ] CrossAttnDownBlock2D
[ ] ResnetBlock2D
[ ] Transformer2DModel
[ ] BasicTransformerBlock
[ ] Attention
[ ] FeedForward
[ ] GEGLU
[ ] UpBlock2D
[ ] UNetMidBlock2DCrossAttn

@saichandax @Sudharsan-V @vigneshkeerthivasanx @sabira-mcw FYI


update image source URL for proper display

Changed the image source URL
from:
https://github.com/cncf/artwork/blob/master/other/illustrations/ashley-mcnamara/transparent/cncf-cloud-gophers-transparent.png
to:
https://github.com/cncf/artwork/blob/main/other/illustrations/ashley-mcnamara/transparent/cncf-cloud-gophers-transparent.png?raw=true
This modification ensures the image is correctly displayed in the rendered content.

/lgtm
thanks.


CanCan/matchers doesn't work when multiple permissions are checked at the same time

I'm using the be_able_to matcher available at cancan/matchers to check if my Ability is able to do many actions on a model and it fails. Curious thing: if I try them all manually, every single one of them work.
This is the Ability:
class Ability
  include CanCan::Ability

  def initialize(user)
    can [:write, :rate, :read], Job, user_id: user.id
  end
end

And this is my test:
require "rails_helper"

RSpec.describe Ability do
  subject { Ability.new(user) }
  let(:user) { create(:user) }

  it "should be able to write, rate and read his own job" do
    permissions = [:write, :rate, :read]
    permissions.each do |p|
      expect(subject).to be_able_to(
        p,
        Job.new(driver_id: user.id),
        driver_id: user.id
      )
    end

    expect(subject).to be_able_to(
      permissions,
      Job.new(driver_id: user.id),
      driver_id: user.id
    )
  end
end

So, what happens when I run this spec is:

first assertion group (in the permissions.each loop) is ok, all assertions pass
second assertion (using the permissions array) fails

They are testing the same thing and should all pass.
Versions
Rails: 4.2.7
CanCanCan: 1.13.1

@douglascamata if you take a look on matcher implementation you will see that it just call can?
  match do |ability|
    ability.can?(*args)
  end

that means you pass array and it interpret as action name thats why your code fails. Workaround that you use i do the same way. I think about to make pull request to add another matcher which will deal with arrays of actions.

@gingray I don't think it should be a different matcher. Because if with can (to define abilities) I can use an array of abilities, I should be able to ask with can? and have a response that follow the same though process as the definition.
If with can [:write, :rate, :read], Job, user_id: user.id I define that an User can write, rate and read all jobs he owns, when I use can? [:write, :rate, :read], Job, user_id: user.id it should check if the user can write, rate and read all jobs he owns. Which is is case would be true.

@douglascamata from one side you right but from other in terms of REST you shouldn't make action with two or more responsibilities. There also pull request from 2015 to add multiple can? but it still in pull request https://github.com/CanCanCommunity/cancancan/pull/262

I've add pull request to solve this issue https://github.com/CanCanCommunity/cancancan/pull/364

@gingray I don't think about it as something that I would actually use in the code of my app, but something that I would use in the tests for sure.

@douglascamata i've got your point hope pull request will be approved and we can use same method to work with array of abilities from tests.

this should have been patched by #364. please reopen if the problem persists


Improve logging by adding debug logs, and perhaps a log decorator

Create a logger class that wraps multiple loggers, one for only logging errors, one for info, one for debug, and a perhaps a silly one, if needed.

Can try coloring logs using this: https://stackoverflow.com/a/2666796


Streamlining Deployment

Not copy pasting from frontend to backend and changing stuff but having one "deploy it now" command
VUEX

Fixed on the development branch.
To install and build both the front and the backend, copy the files, and start the server:
cd backend && npm run serve


There was a Mistype on the others page and I have OCD.

There was a Mistype on the others page and I have OCD. It said CPW when its actually CWP so I fixed it.

Thanks for the fix :)


Vulkan: Crash due to poor EditorToaster interaction with a GPU error

Godot version
Godot Engine v4.0.dev.calinou.795cd2eb3 (795cd2eb3)
System information
Linux, Elementary OS 5.1, Intel(R) HD Graphics 4400 (HSW GT2), Vulkan API 1.2.182
Issue description
Godot crashed with a huge backtrace--seemingly when a GPU hang occurred, with this partial output (full log: 2021-11-03--godot-editor-toaster-crash-01.txt):
INTEL-MESA: error: ../src/intel/vulkan/anv_device.c:3179: GPU hung on one of our command buffers (VK_ERROR_DEVICE_LOST)
ERROR: Condition "err" is true.
   at: flush (drivers/vulkan/vulkan_context.cpp:1722)
ERROR: Condition "err" is true.
   at: flush (drivers/vulkan/vulkan_context.cpp:1740)
ERROR: Condition "err" is true.
   at: flush (drivers/vulkan/vulkan_context.cpp:1740)
ERROR: Condition "err" is true.
   at: flush (drivers/vulkan/vulkan_context.cpp:1740)
ERROR: Condition "err" is true.

[...snip many duplicate lines...]

ERROR: Condition "err" is true.
   at: flush (drivers/vulkan/vulkan_context.cpp:1740)
================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.0.dev.calinou (795cd2eb38626970e7487fcc0686b33c19fd6ab5)
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x3f040) [0x7f373338a040] (??:0)
[2] /usr/lib/x86_64-linux-gnu/libvulkan_intel.so(+0xfe302) [0x7f37277ad302] (??:0)
[3] /usr/lib/x86_64-linux-gnu/libvulkan_intel.so(+0x2e734d) [0x7f372799634d] (??:0)
[4] /usr/lib/x86_64-linux-gnu/libvulkan_intel.so(+0x2e840b) [0x7f372799740b] (??:0)
[5] /usr/lib/x86_64-linux-gnu/libvulkan_intel.so(+0xb0903) [0x7f372775f903] (??:0)
[6] /usr/lib/x86_64-linux-gnu/libvulkan_intel.so(+0xb149a) [0x7f372776049a] (??:0)
[7] RenderingDeviceVulkan::texture_get_data(RID, unsigned int) (??:0)
[8] RendererStorageRD::texture_2d_get(RID) const (??:0)
[9] CommandQueueMT::CommandRet1<RendererStorage, Ref<Image> (RendererStorage::*)(RID) const, RID, Ref<Image> >::call() (??:0)
[10] CommandQueueMT::_flush() (??:0)
[11] CommandQueueMT::flush_if_pending() (??:0)
[12] RenderingServerDefault::canvas_item_set_modulate(RID, Color const&) (??:0)
[13] CanvasItem::set_modulate(Color const&) (??:0)
[14] EditorToaster::popup(Control*, EditorToaster::Severity, double, String) (??:0)
[15] EditorToaster::popup_str(String, EditorToaster::Severity, String) (??:0)
[16] EditorToaster::_error_handler(void*, char const*, char const*, int, char const*, char const*, bool, ErrorHandlerType) (??:0)
[17] _err_print_error(char const*, char const*, int, char const*, char const*, bool, ErrorHandlerType) (??:0)
[18] _err_print_error(char const*, char const*, int, char const*, bool, ErrorHandlerType) (??:0)
[19] VulkanContext::flush(bool, bool) (??:0)
[20] RenderingDeviceVulkan::_flush(bool) (??:0)
[21] RenderingDeviceVulkan::texture_get_data(RID, unsigned int) (??:0)
[22] RendererStorageRD::texture_2d_get(RID) const (??:0)
[23] CommandQueueMT::CommandRet1<RendererStorage, Ref<Image> (RendererStorage::*)(RID) const, RID, Ref<Image> >::call() (??:0)
[24] CommandQueueMT::_flush() (??:0)
[25] CommandQueueMT::flush_if_pending() (??:0)
[26] RenderingServerDefault::canvas_item_set_modulate(RID, Color const&) (??:0)
[27] CanvasItem::set_modulate(Color const&) (??:0)
[28] EditorToaster::popup(Control*, EditorToaster::Severity, double, String) (??:0)
[29] EditorToaster::popup_str(String, EditorToaster::Severity, String) (??:0)
[30] EditorToaster::_error_handler(void*, char const*, char const*, int, char const*, char const*, bool, ErrorHandlerType) (??:0)
[31] _err_print_error(char const*, char const*, int, char const*, char const*, bool, ErrorHandlerType) (??:0)
[32] _err_print_error(char const*, char const*, int, char const*, bool, ErrorHandlerType) (??:0)

[...snip seemingly recursive backtrace...]

[241] _err_print_error(char const*, char const*, int, char const*, char const*, bool, ErrorHandlerType) (??:0)
[242] _err_print_error(char const*, char const*, int, char const*, bool, ErrorHandlerType) (??:0)
[243] VulkanContext::flush(bool, bool) (??:0)
[244] RenderingDeviceVulkan::_flush(bool) (??:0)
[245] RenderingDeviceVulkan::texture_get_data(RID, unsigned int) (??:0)
[246] RendererStorageRD::texture_2d_get(RID) const (??:0)
[247] CommandQueueMT::CommandRet1<RendererStorage, Ref<Image> (RendererStorage::*)(RID) const, RID, Ref<Image> >::call() (??:0)
[248] CommandQueueMT::_flush() (??:0)
[249] CommandQueueMT::flush_if_pending() (??:0)
[250] RenderingServerDefault::canvas_item_set_modulate(RID, Color const&) (??:0)
[251] CanvasItem::set_modulate(Color const&) (??:0)
[252] EditorToaster::popup(Control*, EditorToaster::Severity, double, String) (??:0)
[253] EditorToaster::popup_str(String, EditorToaster::Severity, String) (??:0)
[254] EditorToaster::_error_handler(void*, char const*, char const*, int, char const*, char const*, bool, ErrorHandlerType) (??:0)
[255] _err_print_error(char const*, char const*, int, char const*, char const*, bool, ErrorHandlerType) (??:0)
-- END OF BACKTRACE --
================================================================
Aborted

(The build I'm using appears to already include the patch for the only other recent EditorToaster issue: https://github.com/godotengine/godot/issues/54442)
Steps to reproduce
I think the GPU hang is probably this one: https://github.com/godotengine/godot/issues/53940
(I encountered this error while attempting to document a different issue.)
But I'm guessing this situation will probably occur with any similar error that occurs related to rendering if the EditorToaster doesn't bail on render failures.
Minimal reproduction project
N/A

Oh, actually, now that I take another look[0] at the backtrace it may be relevant that the issue I was writing up/reproducing was related to:

https://github.com/godotengine/godot/issues/36721
https://github.com/godotengine/godot/pull/54427

[0] Specifically noticing this portion:
[245] RenderingDeviceVulkan::texture_get_data(RID, unsigned int) (??:0)
[246] RendererStorageRD::texture_2d_get(RID) const (??:0)

FWIW from my perspective this seems somewhat less of a rendering issue and more of an issue with the error handler not recognizing when it is causing further errors resulting in nested calls to itself (i.e. the error handler).
While this specific instance was render-related presumably there may also be other ways for the same scenario to occur--which I'm guessing is partly as a result of overloading _err_print_error() to do something more than just "printing". (Possibly related to multi-threading.)

When an error occurs a toast appears. This adds some nodes to the editor canvas if it's on the main thread. As a safeguard, it should probably check to make sure it's not recursively reporting errors.

Looks like the origin of the rendering error is that your device couldn't create a window so no rendering operations could be successfully flushed.

The origin of the specific rendering error in my case was/is that the mesh preview requests an MSAA level that the GPU explicitly doesn't support: https://github.com/godotengine/godot/issues/53940


Add support for namespaced service names

Service ids with \ are not considered viable services. Since symfony 3.4+ recommends and auto configuring services as their FQCN, you need to introduce an alias to configure batch_load_fn.
To fix this, the Overblog\DataLoaderBundle\DependencyInjection\Configuration::SERVICE_CALLABLE_NOTATION_REGEX should be changed. We could use the same capture group definition as for function in PHP_CALLABLE_NOTATION_REGEX, i guess.

Hi, nice catch! Thanks for your feedback. I think, regex will no more be sufficient, we should use compiler pass to test if service exists otherwise fallback to static class notation.

fixed in v0.4.1


Consumer group for custom cloud stream binder not working

I've created custom binder for salesforce platform event bus , use comet-D client and long polling connection to connect to bus,
I followed the reference document guide
https://docs.spring.io/spring-cloud-stream/docs/current/reference/html/spring-cloud-stream.html#spring-cloud-stream-overview-custom-binder-impl
and google pub/sub binder
it is working fine as expected but the consumer group is not working,I passed the consumer group in createConsumerEndpoint method in custom binder class but not configured anywhere in connection.
@Override
protected MessageProducer createConsumerEndpoint(
ConsumerDestination destination,
String group,
ExtendedConsumerProperties properties) {
SalesforceEventInboundChannelAdaptor adapter = new SalesforceEventInboundChannelAdaptor(properties.getExtension(),destination.getName(),empConnector);
ErrorInfrastructure errorInfrastructure =
registerErrorInfrastructure(destination, group, properties);
adapter.setErrorChannel(errorInfrastructure.getErrorChannel());
return adapter;
}

I read few document online about consumer group in spring cloud stream but didn't get any in details. whether it's spring feature or broker should support that to implement like kafka message broker support consumer group on broker side .
it is just mentioned spring create  load balance in front of consumer so that only one consumer process the message.
I have to configure consumer group to autoscale the consumer in Azure cloud, could you please suggest the solution

@Rita0880 I am unfamiliar with the libraries you are using, but the concept of consumer-group should be addressed at the binder level. See how the group is propagated eventually to the underlying Kafka libraries here.

@sobychacko I've checked link ,it pass the consumer group property to org.apache.kafka.clients.consumer.KafkaConsumer (github) which is internally call https://github.com/apache/kafka/blob/7ed3748a462cd1ce7c30bb9a331b94a3cd79a401/clients/src/main/java/org/apache/kafka/clients/consumer/internals/ConsumerCoordinator.java#L94
to coordinate the consumer
they all are apache kafka client specific, spring binder only pass the property value in createConsumerEndpoint , not any spring specific implementation for consumer group.I read, kafka itself supprt consumer group at broker level
comet-d client which I am using ,doesn't have such consumer group support at broker level
if you see the binder class of google pub/sub it is not pass consumer group in inbound adaptor:
https://github.com/spring-attic/spring-cloud-gcp/blob/main/spring-cloud-gcp-pubsub-stream-binder/src/main/java/org/springframework/cloud/gcp/stream/binder/pubsub/PubSubMessageChannelBinder.java#L81
I am not sure consumer group is working for google pub sub...

Thanks @sobychacko
This is what I am looking for, how can I implement at binder level , as per spring doc,it's a spring cloud stream feature, but I am not able find any sample code or document to follow to implement it at binder level,
Could you please share any sample code or document if you have?

Sure, the expectation of the group is from Spring Cloud Stream. But, how it is implemented or used at the binder level is another matter. RabbitMQ does not have the concept of a consumer group, but the binder for RabbitMQ in Spring Cloud Stream maps certain elements from it to the group concept. See the docs for some details on that. Trace through this code also to see how it is done in the Rabbit binder.

@Rita0880 I am not familiar with the internals of the technologies you are building the binder for. But if your goal is to achieve autoscaling, I believe you can probably accomplish that using core Spring Cloud Stream properties such as instanceCount and instanceIndex. If you have some concrete use cases or examples or code to share, then we may be able to provide further guidance.

Consumer groups is either target platform specific functionality (i.e., Kafka) or binder implemented on top of target platform if possible (i.e., Rabbit). There is nothing in core to support it based on how specific it is to the target platform.
I am closing it as there is nothing for us to do here and as a discussion it is better to be moved to Stack Overflow


[BACKLOG-11905] Update icons

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f ui/build.xml clean-all resolve jacoco && ant -f ui/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  ui/src/org/pentaho/di/ui/core/gui/GUIResource.java
  ui/src/org/pentaho/di/ui/partition/dialog/PartitionSchemaDialog.java

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on ui/src/org/pentaho/di/ui/partition/dialog/PartitionSchemaDialog.java. Found 2013. Was expecting 2017

replaced by https://github.com/pentaho/pentaho-kettle/pull/3835


Add option to get the info on running containers

Hello,
Could an option be added to obtain information on containers? A bit like docker inspect?
In fact, it's something that could be “simple” to add, but it's necessary to have the real name of running containers.
This would simplify the development of third-party tools or make it easier to integrate devcontainers with other IDEs.

Dev containers internally uses  Docker, hence, you can still use the docker inspect command to find information on the containers.

In fact, it's something that could be “simple” to add, but it's necessary to have the real name of running containers.

Curious, won't the docker ps command help you with it?

Curious, won't the docker ps command help you with it?

It is, but If I have multiple containers running on my system (which is the case for most pleople), I have to "manually" search for the one I want. As the cli can execute command in the devcontainer, I suppose it van return the name of the container.


Clean up RandomId

Assume a unique name and then use that to internally generate the seed.

Add a macro to define random ids

Hey @ekr-cfa, while reviewing your PR, I'd suggest the following code changes:
👉 Suggestion for #10

Make macro more visible and remove import requirement

You can also review and apply these suggestions locally on your machine.
Learn more about GitKraken Code Suggest

Code Suggest liberates your code reviews from GitHub's restrictive, comment-only feedback style. As simple as suggesting changes in a Google-doc, provide real code suggestions from where you code, e.g. your IDE, and on anything in your project — not just on the lines of code changed in the PR.

Join your team on GitKraken to speed up PR review.


Rename repo

Presumably to lbrysdk.

[ ] rename the repo
[ ] update https://github.com/lbryio/boardbot
[ ] update https://github.com/lbryio/lbry-desktop/blob/master/package.json#L141
[ ] what else?

For me the primary motivations would be:

Calling it what it it is: LBRY SDK. Having SDK in the name like that signals that we really want people to build on it, this is front and center.
lbry as repo name implies that the entirety of LBRY is in there and while it is definitely a mono-repo for most Python code in LBRY it's still missing the blockchain part (lbrycrd) which is a core part of LBRY. Without that core part I don't think it's fair to call it lbry.
I would like to eventually rename the main Python module from lbrynet to just lbry and while it's mostly okay to have ../lbry/lbry/.. directory structure (unless you give it a different name when git cloning) this can sometimes cause difficult to debug issues with configuration / import paths. Lot's of projects do this though (repo name == module name), so it's not a big issue, but if we can address the edge case, why not?

Downsides are:

potentially creates a bunch of work
may have zero impact on actually getting more users

@lyoshenka can we go ahead with the rename?

@eukreign i'm ambivalent. if this is valuable and worth doing in the next month, just do it. otherwise close it?

I'm very in favor of doing this unless there are costs I'm not appreciating.

repo renamed to lbry-sdk


Contracts show no_contracts during configuration flow

System Health details

Waiting for logs

By looking deeper - issue is caused by a contract with status 2, where as the current condition is status=1
https://github.com/GuyKh/iec-custom-component/blob/main/custom_components/iec/config_flow.py#L123


Allow to force open in same window from the recently opened picker

Currently, "Open Recent..." defaults to opening folders in the same window. If you hold Ctrl, they will be opened in a new window instead as indicated in the UI:

With "window.openFoldersInNewWindow": "on", the behavior changes to always opening the folder in a new window, regardless of whether Ctrl is held or not, effectively taking away some flexibility.
I think it would be convenient if "window.openFoldersInNewWindow": "on" flipped the default behavior of "Open Recent..." instead, so that holding Ctrl could still be used to open the folder in the same window. This would be useful for people like me who want to open folders in a new window most of the time, so the more common action would have a simpler shortcut (Enter vs Ctrl+Enter).

via https://github.com/microsoft/vscode/commit/72a381699f549b3f1c7c38b2b9a6d9a1de06f603 pushed support to force open in same window when Alt key is pressed, I think that is a good compromise to allow full control independent of the setting.
Verification

make sure you can force open a recent entry in the recently opened folders quick pick (File > Open Recent > More...) by holding Alt key when clicking even if "window.openFoldersInNewWindow": "on" is configured

With this on macOS, Cmd+Enter opens the folder in a new window when "window.openFoldersInNewWindow": "default" | "off" is set, but Opt+Enter opens the folder in the same window when  "window.openFoldersInNewWindow": "on" is set. These should be the same in my opinion.

@JacksonKearl that was the intent of my change to allow to override your setting. It seems it is working exactly as advertised. Please comment back if you disagree.

I think the key commands should be the same between the two. i.e. it should be Cmd+Enter to make the folder open in the same window when "window.openFoldersInNewWindow": "on", as it is Cmd+Enter to make the folder open in a different window then "window.openFoldersInNewWindow": "off".

@JacksonKearl understood, but that is not what is implemented :). It is non trivial to get all the things right based on settings because you can have different settings for folders vs files, so you would almost have to update the placeholder when the folder is active vs. a file. That is why I did not want to go down that path.


Client components are not loaded until all suspenses resolve in SSR

Currently the framework implements streaming SSR which is then re-hydrated by fetching RSC from the __marz endpoint. I think ideally that I'd want to skip streaming SSR and instead render a single "frame" on the server, so the client still gets a server rendered HTML frame that is then hydrated by RSC. Not sure on this though, so if anyone else has any better suggestions please share in this thread!

Example: if we have a  component that simply awaits a 1 second timeout, and wrap it in , both the initial SSR stream and RSC stream will take 1s to finish, as shown in this waterfall image:

For now, I came up with a workaround where I immediately cancel rendering after starting. This seems to work okay but it's definitely not ideal. Need to look into how other frameworks accomplish this 🤔 Perhaps start by rendering the RSC, then render it to HTML?
https://github.com/hex2f/marz/blob/3ea0f88089b585e3d24af2ff6a96f06ae1d06361/framework/server/worker.tsx#L81-L84

hey, i made this RSC framework thing, i might be able to help with something :)
one question - you mention "frames" in a bunch of places, wdym by that?


Docs bug TransactionsManager and related misleading field

Describe the feature
Docs bug
https://github.com/paradigmxyz/reth/blob/ca98261d43d40cb2332567fac2bb944fe38e9131/crates/net/network/src/transactions/mod.rs#L226-L227
Also improve naming to better match actual flow, pending_transactions (above) is incorrect in context of TransactionsManager. This is a stream of transactions that have been successfully inserted into the pool. They are pending in the context of the TransactionsPool, but in the context of TransactionsManager they are not. TransactionsManager is done with those transactions when notice is received that they have been successfully inserted into the pool and thereby are propagated by TransactionsManager (insertion into pool means they have passed validation). A better name is inserted_transactions or txns_inserted_into_pool or pool_insertions.
Same goes for pending_pool_imports, which is better off being named pending_pool_insertions since the code that the PoolImportFuture contains is
https://github.com/paradigmxyz/reth/blob/ca98261d43d40cb2332567fac2bb944fe38e9131/crates/net/network/src/transactions/mod.rs#L981
which ends up here
https://github.com/paradigmxyz/reth/blob/ca98261d43d40cb2332567fac2bb944fe38e9131/crates/transaction-pool/src/pool/mod.rs#L385-L393
a function that, for the successful case, ends with the code
https://github.com/paradigmxyz/reth/blob/ca98261d43d40cb2332567fac2bb944fe38e9131/crates/transaction-pool/src/pool/mod.rs#L440-L455
which notifies about a successful pool insertion, not a successful pool import. Hence, pending_pool_insertions is a far more correct and therefore helpful name.
Same goes for the PoolImportFuture. Other code in TransactionsManager referring to import should be revised to check if it better be replaced with insert.
Additional context
No response

Can I give it a try?

@emhane What's the difference between pool insertion and pool import? I found the document for these two functions use imports transactions as well.

the tx pool has several stages, so when a tx leaves the tx manager, it becomes pending in the pool.

https://github.com/paradigmxyz/reth/blob/123b95aeccb6f41b29a9b3249f108cf714d31bd4/crates/transaction-pool/src/traits.rs#L52-L71

those docs should be updated too, to reflect what the code actually does, so something like
Inserts all _external_ transactions into pool and queues for import.

And, see, your question perfectly captures the problematic here, developers have a tendency to trust docs (and log messages). So it's very important they directly reflect the code.

It appears that import transactions refers to the synchronization of finalized transactions from other peers. Is that correct?

Also improve naming to better match actual flow, pending_transactions (above) is incorrect in context of TransactionsManager. This is a stream of transactions that have been successfully inserted into the pool.

This is not accurate, the TransactionsManager is only interested in pending txs, which is a small subset of transactions inserted into the pool:
https://github.com/paradigmxyz/reth/blob/9f91c6ad94f94b2ff1a988c932c9ef0772c5bcc1/crates/net/network/src/transactions/mod.rs#L252-L254
https://github.com/paradigmxyz/reth/blob/9f91c6ad94f94b2ff1a988c932c9ef0772c5bcc1/crates/transaction-pool/src/pool/mod.rs#L146-L146
https://github.com/paradigmxyz/reth/blob/9f91c6ad94f94b2ff1a988c932c9ef0772c5bcc1/crates/transaction-pool/src/pool/mod.rs#L440-L443
https://github.com/paradigmxyz/reth/blob/9f91c6ad94f94b2ff1a988c932c9ef0772c5bcc1/crates/transaction-pool/src/pool/mod.rs#L519-L520
https://github.com/paradigmxyz/reth/blob/9f91c6ad94f94b2ff1a988c932c9ef0772c5bcc1/crates/transaction-pool/src/pool/mod.rs#L37-L40
pending is the proper name here, because we only want to send transactions that are executable at the current state


Add links support

Codecov Report

Merging #29 (b5a2c29) into master (b5be42e) will increase coverage by 0.30%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
+ Coverage   92.03%   92.33%   +0.30%
==========================================
  Files           2        3       +1
  Lines         251      261      +10
==========================================
+ Hits          231      241      +10
  Misses         14       14
  Partials        6        6

Flag
Coverage Δ

unittests
92.33% <100.00%> (+0.30%)
:arrow_up:

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

Impacted Files
Coverage Δ

main.go
89.07% <ø> (ø)

function/github.go
100.00% <100.00%> (ø)

function/link.go
100.00% <100.00%> (ø)

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 b5be42e...b5a2c29. Read the comment docs.


Add CI for Windows

(probably using ci.appveyor.com)
@phadej Are you interested to look into this?

I can check this next week.

Setting up the appveyor is trivial, the tests aren't:
  test\Hpack\ConfigSpec.hs:282:
  1) Hpack.Config.readPackageConfig accepts data-files
       Actual list is not a permutation of expected list!
         expected list contains:   ["data/foo/index.html", "data/bar/index.html"]
         actual list contains:     ["data\\bar\\index.html", "data\\foo\\index.html"]
         the missing elements are: ["data/foo/index.html", "data/bar/index.html"]
         the extra elements are:   ["data\\bar\\index.html", "data\\foo\\index.html"]

  test\Hpack\UtilSpec.hs:173:
  2) Hpack.Util.expandGlobs, when expanding *, expands by extension
       expected: ([],["files/bar.js","files/baz.js","files/foo.js"])
        but got: ([],["files\\bar.js","files\\baz.js","files\\foo.js"])

  test\Hpack\UtilSpec.hs:178:
  3) Hpack.Util.expandGlobs, when expanding *, rejects dot-files
       expected: ([],["foo/bar"])
        but got: ([],["foo\\bar"])

  test\Hpack\UtilSpec.hs:183:
  4) Hpack.Util.expandGlobs, when expanding *, accepts dot-files when explicitly asked to
       expected: ([],["foo/.baz"])
        but got: ([],["foo\\.baz"])

  test\Hpack\UtilSpec.hs:188:
  5) Hpack.Util.expandGlobs, when expanding *, matches at most one directory component
       expected: ([],["foo/bar.js"])
        but got: ([],["foo\\bar.js"])

  test\Hpack\UtilSpec.hs:194:
  6) Hpack.Util.expandGlobs, when expanding **, matches arbitrary many directory components
       expected: ([],["foo/bar/baz.js"])
        but got: ([],["foo\\bar\\baz.js"])

Randomized with seed 195582947

All of these should be easy to fix.  But as we have hpack support in stack now, this is kind of important.
@phadej Can you open a PR with your AppVeyor config?

Build for windows is passing now.


Problem with custom stable diffusion

I tried to run a trained .ckpt file using the Fast-Dreambooth Colab notebook, i've done it before without any problem, but now I always get this error message when I want to try it:
ImportError: cannot import name 'image_from_url_text' from 'modules.generation_parameters_copypaste' (/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/generation_parameters_copypaste.py)
PD: I've also tried running it with AUTOMATIC1111 and got the same error, i'll be grateful if someone knows how to fix this.

use the latest colab notebook and check the box "update_repo"


Add hungarian language

Coverage remained the same at 99.34% when pulling 69fa40c25a5614e7e2316f8279942f5f57b6d06a on hansi90:master into 68bf704835742313274e41e3f02fe9572ce0a4ab on charliekassel:master.


Refactor LWRPs into custom resources

Description
This originated from a need to fix the deprecation warnings for
use_inline_resources (FC113)[1] and this commit implements just the bare
minimum required to convert to custom resources while keeping all the
tests passing.
Support for chef-client 12 is dropped as that will be EOL this month[2].
I also removed the exec bit from some template and attribute files.
[1] http://www.foodcritic.io/#FC113
[2] https://www.chef.io/eol-chef12-and-chefdk1/
Issues Resolved
Fixes Travis builds like https://travis-ci.org/sous-chefs/dhcp/jobs/368779524
Contribution Check List

[x] All tests pass.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable

@damacus
https://help.github.com/articles/requesting-a-pull-request-review/

Note: Pull request authors can't request reviews unless they are either a repository owner or collaborator with write access to the repository.

I'm wondering whether we need to enable travis on this repo before this change. Or  at very least, before we release ;)

Just realised how long this has been hanging around.
Next up some tests. I'll sort out Travis shortly


fix query key rule

This PR skips the non-encoded query key test because there is no rule for it yet, and modifies the encoded query key rule as it wasn't matching with any keys_only rule

Rebased


Added resources for TF Model server

Added content and resources for demos of TF Model Server, Note: I will also be using these in my talk at GDG Ahmedabad on 30 May.
Resources added-

Serving a dummy model
Serving a CNN model
Serving model with Kubernetes
End to end example to serving MNIST on Cloud

@Rishit-dagli Thank you very much for your contribution. Those are very helpful resources.


Fix/make list in empty document

this PR fixes inserting a list from a text node that's a direct child from the rootNode. For example when immediately creating a list.
I'm unsure whether the parent.parent logic in getBlocksFromRange is still required, but I left it as is since it does seem to work as expected.
One annoyance I haven't addressed is that when pressing the list button at the end of a textNode it's not always consistent (sometimes the text is included in the li, but not always).

I'm also pretty sure the current behavior is not what we want so with these cases im always struggling a bit: should we let the test pass to document the current behavior, or should we  let it fail to signal that it needs work?

I'm also pretty sure the current behavior is not what we want so with these cases im always struggling a bit: should we let the test pass to document the current behavior, or should we let it fail to signal that it needs work?

I struggled with this as well, it's how I ended up with the somewhat vaguer test with find. Ideally I'd like something like warn, so the test can fail visually (and indicate the behaviour is not what we want) but we also know it's not blocking.

general remark: Something our xml comparisons do (and previous integrations tests as well) is focus quite a bit on the specific implementation of the command. e.g. if at some point we decide an empty list item starts out with a br child we'll need to revisit all make-list tests.

Yes but I think that's a good thing
I think no matter what we do editing operations will always be brittle and depend heavily on us knowing exactly what they do and how they interact.
Breaking tests will then signal us that a previous assumption was broken and force us to think if that will be a problem or not.

I'm also pretty sure the current behavior is not what we want so with these cases im always struggling a bit: should we let the test pass to document the current behavior, or should we let it fail to signal that it needs work?

I struggled with this as well, it's how I ended up with the somewhat vaguer test with find. Ideally I'd like something like warn, so the test can fail visually (and indicate the behaviour is not what we want) but we also know it's not blocking.

I guess the closest that we have available is letting it fail but skipping the test
This will get marked visually as well (see the next-text-node tests for an example)
Come to think of it thats not a bad workflow:

failing test signifies an urgent bug we should fix asap
failing but skipped test signifies potentially problematic behavior that we can live with for the moment

There is also test.todo()
which passes when the test fails and vice versa (signalling that you should remove the todo when the behavior is implemented)
it also visually marks the failing test
it could serve as a third level:

behavior we eventually want but which is not an immediate priority

thing is in this case im not exactly sure what we want
I guess for the newline to be replaced by a list with a single li (and possibly an empty textnode inside the li?)

thing is in this case im not exactly sure what we want
I guess for the newline to be replaced by a list with a single li (and possibly an empty textnode inside the li?)

What I expected to happen (as in consistent with behaviour when there is text and you click the button):

that the contents of the textnode is moved into the li, possibly normalized.

This will get marked visually as well (see the next-text-node tests for an example)

I did notice that triggers a linting error, though I did not look at it yet.

This will get marked visually as well (see the next-text-node tests for an example)

I did notice that triggers a linting error, though I did not look at it yet.

yes, true. There are also other areas where eslint is confused, e.g. when you use a function type as a type for a parameter (complains about unused params) so we need to have a look at the config there

thing is in this case im not exactly sure what we want
I guess for the newline to be replaced by a list with a single li (and possibly an empty textnode inside the li?)

What I expected to happen (as in consistent with behaviour when there is text and you click the button):

that the contents of the textnode is moved into the li, possibly normalized.

Situations like this really prove the value of feature passports, love the direction we're going with those :+1:

feature passport -> encode those into unit tests marked "todo" -> start implementing
see now we're cooking with TDD :smiley:

This will get marked visually as well (see the next-text-node tests for an example)

I did notice that triggers a linting error, though I did not look at it yet.

yes, true. There are also other areas where eslint is confused, e.g. when you use a function type as a type for a parameter (complains about unused params) so we need to have a look at the config there

made a jira story: https://binnenland.atlassian.net/browse/GN-2570

I'm also pretty sure the current behavior is not what we want so with these cases im always struggling a bit: should we let the test pass to document the current behavior, or should we let it fail to signal that it needs work?

I struggled with this as well, it's how I ended up with the somewhat vaguer test with find. Ideally I'd like something like warn, so the test can fail visually (and indicate the behaviour is not what we want) but we also know it's not blocking.

I guess the closest that we have available is letting it fail but skipping the test
This will get marked visually as well (see the next-text-node tests for an example)
Come to think of it thats not a bad workflow:
* failing test signifies an urgent bug we should fix asap

* failing but skipped test

I think I'd prefer todo for

signifies potentially problematic behaviour that we can live with for the moment
and use skip for
behaviour we eventually want but which is not an immediate priority

As todo still runs your test, but skip doesn't this feels a closer match to me

@abeforgit what do you think, do we merge as is?


Initial points

We need

[x] auction (with number in user defined format)
[x] bidders
[x] bids
[x] items (to sale)
[x] Auction packs (many auctions in time frame, one checkout),  (with number in user defined format)

Reqiurements

[x] if bidder adds bid in :prolonging_limit minutes before auction ends, end time is extended for bid.time + prolonging_limit (so auction end when there are no bids in last :prolonging_limit minutes)

[x] for each auction, there can be set of rules for minimal bid according to current auctioned_price (default can be (1..Infinity) => 1), I call it bid_steps_ladder.
real example
minimal_bids = {
  (0...5_000) => 100,
  (5_000...20_000) => 500,
  (20_000...100_000) => 1_000,
  (100_000...500_000) => 5_000,
  (500_000...1_000_000) => 10_000,
  (1_000_000...2_000_000) => 50_000,
  (2_000_000..) => 100_000
  }

[x] first bid do not increase auctioned_price it stays on starting_price

[x] bids cannot be deleted (only canceled by admin)

[x] admin can cancel bids (when? during sale or after it ?)

[x] auction stores all bids history even those cancelled by admin

[x] bidder cannot "overbid" themselves (no following direct bids from same bidder)

[x] but if bid is wit maximal price, You can overbid Yourself (increase max price)

[ ] auctioneer can define format of auction numbers (eg. "YY#####") and auction_pack numbers

[ ] there should be ability to follow auction (notification before end) or item.author (new item in auction)

[ ] item can have category and user can select auction by categories.

[x] AuctionPack can be (un)published /public

[ ] AuctionPack can be open (adding items , bidding)/ closed(bidding ended)

[ ] auctioneer_commission is in % and adds to sold_price in checkout

[x] auction have start_time and end_time

[ ] auction can be highlighted for cover pages

[x] if there is two bids with same max_price, winner is bid which is older

[ ] auction fee, paid by buyer, is set by percentage and for each auction (sale)

[x] there should be two types of bid

[x] one with maximal price (amount =>  maximal bid price; placed in small bids as needed), system will increase bid automagicaly unless it reaches maximum
[x] second direct bid (amount => bid price; immediattely placed)

[ ] immediatelly after auction/AuctionPack is closed, email to winner should be sent.

points

there should be ability to follow auction (notification before end) or item.author (new item in auction)
item can have category and user can select auction by categories.

was delegated to Item (or main app)


I can't seem to load a VFS

.. and it is making me sad.
I am not sure if the URI way of loading vfs, eg: database: "file://./bla.db?vfs=httpvfs" is enabled so it could work.
I tried just hacking in the vfs I wanted in the C code where sqlite3_open_v2 is called but that hasn't worked. It might be this VFS doing some crime, or not loading, or something.
But something is up.
Trying to do this one: https://github.com/psanford/sqlite3vfshttp
For now you can try it with this URL:
export SQLITE3VFSHTTP_URL="https://fly.storage.tigris.dev/underjord-streaming-public/podcast-index.db"

My attempt
config :exqlite,
  force_build: true,
  load_extensions: [ "../sqlite3vfshttp/sqlite3http-ext/httpvfs"]

defmodule Podstream.DB do
    def go do
        {:ok, conn} = Exqlite.Sqlite3.open("./foo.db", mode: :readonly)
        {:ok, statement} = Exqlite.Sqlite3.prepare(conn, "select * from podcasts limit 10")
        :ok = Exqlite.Sqlite3.bind(conn, statement, [])
        result = Exqlite.Sqlite3.step(conn, statement)
        IO.inspect(result, label: "result")
    end
end

// ..

    rc = sqlite3_open_v2(filename, &db, flags, "httpvfs");
    if (rc != SQLITE_OK) {
        return make_error_tuple(env, "database_open_failed");
    }
// ..

Forced mix deps.compile exqlite --force.
From SQLite docs:
The default VFS can be changed by registering or re-registering the VFS using the sqlite3_vfs_register() interface with a second parameter of 1. Hence, if a (unix) process wants to always use the "unix-nolock" VFS in place of "unix", the following code would work:
sqlite3_vfs_register(sqlite3_vfs_find("unix-nolock"), 1);
An alternate VFS can also be specified as the 4th parameter to the sqlite3_open_v2() function. For example:
int rc = sqlite3_open_v2("demo.db", &db, SQLITE_OPEN_READWRITE, "unix-nolock");
Finally, if URI filenames have been enabled, then the alternative VFS can be specified using the "vfs=" parameter on the URI. This technique works with sqlite3_open(), sqlite3_open16(), sqlite3_open_v2(), and when a new database is ATTACH-ed to an existing database connection. For example:
ATTACH 'file:demo2.db?vfs=unix-none' AS demo2;
The VFS specified by a URI has the highest priority. After that comes a VFS specified as the fourth argument to sqlite3_open_v2(). The default VFS is used if no VFS is specified otherwise.

I haven't needed to load a VFS before. I'll take a look at this. We'll probably need to pass a new option or configuration to exqlite when opening to signal that it is VFS and special handling needs to take place.

So I imagine exqlite just passes the query params through and if that is the case, as long as the URI flag thing is switched on it SHOULD be able to do VFS without more modification. But I did not get this working.
Let me know if anything needs clarification.

We had a nice conversation during @lawik s live stream this afternoon.
After I decided to try some more and found this solution to query the remote db.
I followed Lars's directions to compile httpvfs.so,
set the export SQLITE3VFSHTTP_URL=https://fly.storage.tigris.dev/underjord-streaming-public/podcast-index.db,
I copied it to my test phoenix project to make it easier, still needed to supply the full path though.
defmodule Sqlvfs.Db do
  def go do
    {:ok, conn} = Exqlite.Sqlite3.open("memory:")
    :ok = Exqlite.Sqlite3.enable_load_extension(conn, true)

    :ok =
      Exqlite.Sqlite3.execute(
        conn,
        "select load_extension('/home/herman/Projects/sqlite-objects/sqlvfs/httpvfs.so')"
      )

    {:ok, conn} = Exqlite.Sqlite3.open("file:///foo.db?vfs=httpvfs")
    {:ok, stmt} = Exqlite.Sqlite3.prepare(conn, "select * from podcasts limit 1;")
    {:ok, row} = Exqlite.Sqlite3.fetch_all(conn, stmt)
    :ok = Exqlite.Sqlite3.release(conn, stmt)
    :ok = Exqlite.Sqlite3.close(conn)
    IO.inspect(row)
  end
end

I followed this sequence as it mimics the flow we did with the sqlite3 cli.

I can confirm that I am also able to get this to work.

Something is up with the DBConnection implementation and how Ecto is initiating the connection. I can reach the VFS directly with the code above and work with raw data. What I am struggling with is Ecto establishing a pool of connections (1 in this case) and is not able to open the sqlite database at all which doesn't let it load the necessary extensions.
I will need to dig more into the initialization code to understand more.


SendHandler does not populate the HTTPResponse

This library is not adding the HTTPResponse once we send off the request via the SendHandler. Look at the original aws-sdk-go for an example:
https://github.com/aws/aws-sdk-go/blob/main/aws/corehandlers/handlers.go#L112

@ktseytlin @kevinchristen
Can you guys provide some input on this?
I have other handlers that rely on HTTPResponse to be there and unfortunately this is not including it.
related:
https://github.com/newrelic/go-agent/issues/287

DAX does not use HTTP. Can you give us more information about how your application relies on the response?

Thank you for responding back. There is a library (NewRelic) I'm using for tracing that relies on that field to be populated. Here is the code:
https://github.com/newrelic/go-agent/blob/master/_integrations/nrawssdk/v1/nrawssdk.go#L26-L29


Optimize git/oci/helm container resources

Reduce Memory and increase CPU to match usage under load, as observed durring stress tests.
Adjust standard requests to half of new autopilot limits. Boost is allowed on non-autopilot clusters, because no limit is set.

Profiling:

git-sync: go/config-sync-reconciler-profiling
helm-sync & oci-sync: go/config-sync-reconciler-profiling-2

/lgtm
/assign @janetkuo


Add option to color any help command

This allows you to use colored git log --help for example, to get
colored output.

Could I get a review on this one?

:+1:, but we should probably change the plugin name in the future to something like colored. Can you add a basic README? See https://github.com/robbyrussell/oh-my-zsh/issues/7175#issue-365263634 for instructions.

@mcornella, Done. Let me know if the wording is ok.


Intermittent failure in test_files/cte.slt:751

There is a non deterministic test failure on cte.slt it seems, unrelated to this PR and looks like a non-deterministic test...
Failure: https://github.com/apache/arrow-datafusion/actions/runs/8543135777/job/23406267036?pr=9876
External error: query result mismatch:
[SQL] WITH RECURSIVE my_cte AS(
    SELECT 1::int AS a
    UNION ALL
    SELECT a::bigint+2 FROM my_cte WHERE a<3
) SELECT *, arrow_typeof(a) FROM my_cte;
[Diff] (-expected|+actual)
-   1 Int32
-   3 Int32
+   3 Int32
+   1 Int32
at test_files/cte.slt:751

Originally posted by @alamb in https://github.com/apache/arrow-datafusion/issues/9876#issuecomment-2035412270
I believe this was added in https://github.com/apache/arrow-datafusion/pull/9836 from @jonahgao

The same PR tests passed when the same job was rerun:  https://github.com/apache/arrow-datafusion/actions/runs/8543135777/job/23410293407?pr=9876

take


findAll() Function order.

Dear Greg0,
I was wondering if there was any option for ordering the ids by asc or desc, because I was not able to find it. Thanks!

Yes, here is an example for orderBy functionality
https://github.com/Greg0/Lazer-Database/blob/master/examples.md#order-by


Failed to create a Network Interface with no much information except a CloudException

I used to have this piece of code and it used to work for a while ..suddenly I am having an issue with the Interface .create api call, it's throwing a Cloud exception with no further details .
At least what is the proper way to debug this.. ?!
        NetworkInterface networkInterface = azure.networkInterfaces().define(netInterfaceName)
                .withRegion(region)
                .withExistingResourceGroup(resourceGroupName)
                .withExistingPrimaryNetwork(azure.networks().getByGroup(resourceGroupName, network))
                .withSubnet(networkSubnet)
                .withPrimaryPrivateIpAddressDynamic()
                .withNewPrimaryPublicIpAddress()
               .withExistingNetworkSecurityGroup(azure.networkSecurityGroups().getByGroup(resourceGroupName, securityGroup))
                .create();

            VirtualMachine automationVM = azure.virtualMachines().define(vmName)
                    .withRegion(region)
                    .withExistingResourceGroup(resourceGroupName)
                    .withExistingPrimaryNetworkInterface(networkInterface)
                    .withLatestWindowsImage(vmImage.publisher(), vmImage.offer(), vmImage.sku())
                    .withAdminUsername("ABC")
                    .withAdminPassword("XYZ!")
                    .withSize(azureCores.size())
                    .create();

#1531 was fixed in 1.0.0. Can you verify if this is still an issue?
Thanks!

I will check it and tell you

presumably fixed based on the lack of further response


1.9.3-p0 compatibility

Hello
I found ill behavior with mri-1.9.3-p0
~/src/ruby/wrong [master]$ ruby test/config_test.rb
ruby 1.9.3p0 (2011-10-30 revision 33570) [x86_64-darwin10.8.0]
/Users/thenrio/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in require': cannot load such file -- ParseTree (LoadError)     	from /Users/thenrio/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in require'
from /Users/thenrio/src/ruby/wrong/lib/wrong/chunk.rb:7:in require_optionally'     	from /Users/thenrio/src/ruby/wrong/lib/wrong/chunk.rb:13:in <top (required)>'
from /Users/thenrio/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in require'     	from /Users/thenrio/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in require'
from /Users/thenrio/src/ruby/wrong/lib/wrong/assert.rb:5:in <top (required)>'     	from /Users/thenrio/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in require'
WDYT?, Thierry

So what about now ?
Curious and glad you reviewed this PR :)

Been too busy to dig into it. I tried installing 1.9.3rc1 via RVM and
got the dreaded readline library problem and haven't had time to try
again since.
On Wed, Nov 16, 2011 at 12:12 PM, Thierry Henrio
<EMAIL_ADDRESS>wrote:

So what about now ?
Curious and glad you reviewed this PR :)

Reply to this email directly or view it on GitHub:
https://github.com/sconover/wrong/pull/15#issuecomment-2767180

--
Alex Chaffee -<EMAIL_ADDRESS>- http://alexch.github.com
Stalk me: http://friendfeed.com/alexch | http://twitter.com/alexch |
http://alexch.tumblr.com


Install in lunarvim not works

Hello, i was trying to install using the commands in my macos for lunarvim.
But look like it's not working in lunarvim. Any thoughts?
Thanks

I have many thoughts ;) but no information at all to help you with your issue. What do you mean by "not working"? What have you tried?


Does early memorization predict late memorization?

We currently have the following correlation heat-map which indicates that the answer is "yes." We should probably also make confusion matrices for the classifier that takes a model and predicts memorization by the fully trained model by assuming it is the same as the 23M sequence checkpoint.

The correlation looks strong. This is for 13B model.

The trend is consistent in small models (19M)

@lintangsutawika very interesting! It looks like the change between checkpoints is mostly confined to FP cell.. the TN and FN numbers are rather consistent across the course of training (until the end, where incorrect judgements are impossible).
One potential next step is to plot the TP, FP, and FN rates as a function of the number of steps the checkpoint has been trained for. My intuition is that we should plot this with a logarithm on the x-axis (I know this will make it look funky), maybe do it both with and without a log and see what they look like?
Another analysis worth doing is looking at the consistency across checkpoints. Obviously the final checkpoint gets all predictions correct, but once points get categorized correctly do they remain correctly categorized throughout the remaining checkpoints? I guess what I have in mind here is a markov model for how individual predictions move around the four states over the course of training. Does that make sense?
This is a minor thing, but I’m used to these matrices being laid out slightly differently, as shown in the below image. Would you mind reorienting them? It’s messing with my head

No problem.

Preliminary graphs through steps for all model sizes until before 146M.
TPR

FPR

FNR

Also, I tried graphing the number of lines memorized through checkpoints for each model.
It looks like there is a linear growth for all models and that the rate of growth increases as the model size gets larger.

@lintangsutawika This is great work, and I think with a little addition design iteration will be an important component of the memorization paper. Unfortunately the raw underlying memorization data was erroneous, but @uSaiPrashanth is working on fixing that and should have new data by Jan 1. Have you had a chance to read the WIP paper draft?
I’m hoping we can package these results and deliver a paper to ICML (deadline Jan 26). I think this is a realistic but non-trivial deadline to make, and that we have really compelling results so far.
Where is the code you’ve been working with? Is it on a branch / fork somewhere, in a Jupyter Notebook, or what?

I have the code in Jupyter Notebook that I copied from Sai's. I can rerun and adjust the graphs for the paper.
I haven't read the paper draft. I've reached out to Sai for help.

Great. Can you upload that notebook to analysis/memorization

What happens to points that are forgotten? Tracking all the points that were memorized in the first checkpoints, the all model sizes tend to forget what they memorized earlier. Connecting this to above graph, where the number of memorization increases, this probably means that the ones memorized later are new points and that the ones memorized earlier have a slight tendency to be forgotten.

Added notebook in a PR for now.
https://github.com/EleutherAI/pythia/pull/50


added clause to rps tournament test

Exposes a condition that the grader checks for, but which isn't at all described.

Thanks!


[Bug] Data is lost on nested associations when preload is enabled for new records

Demonstration

As you can see, the first time I render the blueprint, it's like there is no data at all. Then I load the data from the model itself and rerender the exact same blueprint with the exact same record. Now the data is present.
Code
Blueprinter.configure do |config|
  config.extensions << BlueprinterActiveRecord::Preloader.new(auto: true)
end

class PostBlueprint < ApplicationBlueprint
  identifier :id

  fields :name

  association :locale, blueprint: Rcf::Admin::LocaleBlueprint
  association :post_group, blueprint: Rcf::Admin::PostGroupBlueprint
  association :field_values, blueprint: Rcf::Admin::FieldValueBlueprint

  view :list do
    excludes :field_values
    association :post_group, blueprint: Rcf::Admin::PostGroupBlueprint, view: :list
  end
end

class FieldValueBlueprint < ApplicationBlueprint
  identifier :id

  fields :position, :value, :file_url, :meta

  association :field, blueprint: Rcf::Admin::FieldBlueprint
  association :subfield_values, blueprint: Rcf::Admin::FieldValueBlueprint
end

Edit: Reproduction exemple
# frozen_string_literal: true

require "bundler/inline"

gemfile(true) do
  source "https://rubygems.org"

  gem "rails"
  # If you want to test against edge Rails replace the previous line with this:
  # gem "rails", github: "rails/rails", branch: "main"

  gem "sqlite3", "~> 1.4"

  gem "blueprinter"
  gem "blueprinter-activerecord"
end

require "active_record"
require "blueprinter"
require "blueprinter-activerecord"
require "minitest/autorun"
require "logger"

# This connection will do for database-independent bug reports.
ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: ":memory:")
ActiveRecord::Base.logger = Logger.new(STDOUT)

ActiveRecord::Schema.define do
  create_table "posts", force: :cascade do |t|
    t.string :name
  end

  create_table "post_values", force: :cascade do |t|
    t.references :post, null: true
    t.bigint "parent_id"
    t.string :name
    t.string :value
  end
end

class Post < ActiveRecord::Base
  has_many :post_values, dependent: :destroy

  accepts_nested_attributes_for :post_values, allow_destroy: true, reject_if: :all_blank
end

class PostValue < ActiveRecord::Base
  belongs_to :post, inverse_of: :post_values, optional: true

  belongs_to :parent, class_name: "PostValue", inverse_of: :subvalues, optional: true

  has_many :subvalues,
    -> { order(position: :asc) },
    foreign_key: :parent_id,
    class_name: "PostValue",
    dependent: :destroy,
    inverse_of: :parent

  accepts_nested_attributes_for :subvalues, allow_destroy: true, reject_if: :all_blank
end

class PostValueBlueprint < Blueprinter::Base
  identifier :id

  fields :name, :value

  association :subvalues, blueprint: PostValueBlueprint
end

class PostBlueprint < Blueprinter::Base
  identifier :id

  fields :name

  association :post_values, blueprint: PostValueBlueprint
end

class BugTest < Minitest::Test
  def expected_output
    {
      id: nil,
      name: "Test",
      post_values: [
        {
          id: nil,
          name: "Root value",
          subvalues: [
            { id: nil, name: "Child 1 level 1", subvalues: [], value: "asdasdads" },
            { id: nil, name: "Child 2 level 1", subvalues: [], value: "hjgghj" },
            { id: nil, name: "Child 3 level 1", subvalues: [], value: "tyrhtfghdf" }
          ],
          value: "root"
        }
      ]
    }
  end

  def test_preloading_enabled
    Blueprinter.configure do |config|
      config.extensions << BlueprinterActiveRecord::Preloader.new(auto: true)
    end

    post = Post.new

    post.assign_attributes(
      id: nil,
      name: "Test",
      post_values_attributes:[
        {
          id: nil,
          name: "Root value",
          value: "root",
          subvalues_attributes: [
            { id: nil, name: "Child 1 level 1", value: "asdasdads" },
            { id: nil, name: "Child 2 level 1", value: "hjgghj" },
            { id: nil, name: "Child 3 level 1", value: "tyrhtfghdf" },
          ]
         }
      ]
    )

    assert_equal expected_output, PostBlueprint.render_as_hash(post)
  end

  def test_preloading_disabled
    post = Post.new

    post.assign_attributes(
      id: nil,
      name: "Test",
      post_values_attributes:[
        {
          id: nil,
          name: "Root value",
          value: "root",
          subvalues_attributes: [
            { id: nil, name: "Child 1 level 1", value: "asdasdads" },
            { id: nil, name: "Child 2 level 1", value: "hjgghj" },
            { id: nil, name: "Child 3 level 1", value: "tyrhtfghdf" },
          ]
         }
      ]
    )

    assert_equal expected_output, PostBlueprint.render_as_hash(post)
  end
end

I don't think the issue is related to this extension. Remember, the extension only runs when the object is an ActiveRecord::Relation or descendant. In your example, it looks like you're passing in a single ActiveRecord object, so the extension isn't run.
There could be an issue in Blueprinter itself.

I don't think the issue is related to this extension. Remember, the extension only runs when the object is an ActiveRecord::Relation or descendant. In your example, it looks like you're passing in a single ActiveRecord object, so the extension isn't run.
There could be an issue in Blueprinter itself.

The problem disapear if I remove
config.extensions << BlueprinterActiveRecord::Preloader.new(auto: true)

I don't think the issue is related to this extension. Remember, the extension only runs when the object is an ActiveRecord::Relation or descendant. In your example, it looks like you're passing in a single ActiveRecord object, so the extension isn't run.
There could be an issue in Blueprinter itself.

I added a reproduction example. Hope this helps investigate faster :)
If I remove the line
config.extensions << BlueprinterActiveRecord::Preloader.new(auto: true)

the issue is gone. So I do believe it is related

Thanks for the example, will look. And you're right, associations would trigger the extension.

I've duplicated this in our test suite in the fix-new-records branch. It may be difficult to fix in Blueprinter 1.x, given how extension hooks interact with rendering. Ideally this extension wouldn't be called at all in this case - some of the ideas we've tossed around for BP 2.0 may easily allow for that.
$ bundle exec rake test N=/test_auto_preload_with_new_records/
Run options: -n /test_auto_preload_with_new_records/ --seed 25987

# Running:

F.

Finished in 0.038318s, 52.1948 runs/s, 52.1948 assertions/s.

  1) Failure:
PreloaderExtensionTest#test_auto_preload_with_new_records_via_attributes [test/preloader_extension_test.rb:181]:
--- expected
+++ actual
@@ -1 +1 @@
-{:id=>nil, :name=>"asdf", :widgets=>[{:id=>nil, :name=>"A", :price=>nil}, {:id=>nil, :name=>"B", :price=>nil}, {:id=>nil, :name=>"C", :price=>nil}]}
+{:id=>nil, :name=>"asdf", :widgets=>[]}

2 runs, 2 assertions, 1 failures, 0 errors, 0 skips

I've duplicated this in our test suite in the fix-new-records branch. I think what's happening is that when a preload for the child association is run, ActiveRecord wipes out any unsaved records on that association if they were added via attributes.
Ideally this extension wouldn't be called at all in cases like this (i.e. non-ActiveRecord::Relation objects being rendered). It's been a recurring challenge for this extension, given how extension hooks interact with rendering. Some ideas have been tossed around for BP 2.0 that could fix it. At the moment, it's not clear how we could fix this in BP 1.0. I'll keep thinking about it, though.
$ bundle exec rake test N=/test_auto_preload_with_new_records/
Run options: -n /test_auto_preload_with_new_records/ --seed 25987

# Running:

F.

Finished in 0.038318s, 52.1948 runs/s, 52.1948 assertions/s.

  1) Failure:
PreloaderExtensionTest#test_auto_preload_with_new_records_via_attributes [test/preloader_extension_test.rb:181]:
--- expected
+++ actual
@@ -1 +1 @@
-{:id=>nil, :name=>"asdf", :widgets=>[{:id=>nil, :name=>"A", :price=>nil}, {:id=>nil, :name=>"B", :price=>nil}, {:id=>nil, :name=>"C", :price=>nil}]}
+{:id=>nil, :name=>"asdf", :widgets=>[]}

2 runs, 2 assertions, 1 failures, 0 errors, 0 skips

Thanks for the quick investigation. I am not sure if it's actually wiped out by active record because if I were to do these in those step, the blueprint would extract all attributes fine:
MyBlueprint.render_as_hash(record)
--> empty association

# Simply navigating the attributes
record.my_associations.each(&:any_value)
--> any value[]

MyBlueprint.render_as_hash(record)
--> Now associations are filled properly

To demonstrate that, here is another test that passes successfully with preload
def test_preloading_enabled_with_navigation
    Blueprinter.configure do |config|
      config.extensions = [BlueprinterActiveRecord::Preloader.new(auto: true)]
    end

    post = Post.new

    post.assign_attributes(
      id: nil,
      name: "Test",
      post_values_attributes:[
        {
          id: nil,
          name: "Root value",
          value: "root",
          subvalues_attributes: [
            { id: nil, name: "Child 1 level 1", value: "asdasdads" },
            { id: nil, name: "Child 2 level 1", value: "hjgghj" },
            { id: nil, name: "Child 3 level 1", value: "tyrhtfghdf" },
          ]
         }
      ]
    )

    # This here makes the test pass
    post.post_values.each { |post_value| post_value.subvalues.each { |subvalue| subvalue.name } }

    assert_equal expected_output, PostBlueprint.render_as_hash(post)
  end

Maybe it gives you some ideas?

@jamesst20 Thanks, I'll play around with that. But yes, I think it's quite possible it's an AR bug - or at least quirky behavior that "makes sense" to those with a perfect understanding of AR's internals.

 :)

Hi @jamesst20, yes I've been looking at this off and on and sadly don't see a good solution. I wouldn't call it a Rails "bug" - rather unexpected behavior when attribute assignment and preloading are both used. I've not been able to find a way to detect this case (an association with unsaved members) without potentially triggering an unwanted query. It wouldn't surprise me if an undocumented way exists, but that would have its own risks.
We're currently working on some significant updates to Blueprinter's internals which should give us some relief here (i.e. preloading wouldn't happen at all in the above example), but there's no ETA on that.

IMHO for the 2.0 verison of the Blueprint renderer and extensions, we need a way to ensure this extension is only called once per "outer" MyBlueprint.render call. "Inner" rendering calls (like rendering an association's Blueprint) shouldn't call the hook this extension uses. This would ensure the extension only runs if the "outer" render call site passes an ActiveRecord::Relation.


Implement markdown in the prompt

@lankybutmacho
Somewhat the same reason as why we do it for the secure storage and safe area, it allows users to choose which libraries they want to use rather than us dictating. Some of our customers have strict requirements (K)
Also, most users are unlikely to use markdown so kinda nice that it's optional and doesn't bloat by default


Fix misspelling of mergable

https://github.com/go-gitea/gitea/pull/25812#issuecomment-2099833692
Follow #30573

It's already in the list but seems the spellchecker is not considering compound words like this.

Yes, I saw that.


Customize how method parameters are serialized

Add annotations for defining how method parameter in an API trait are serialized:
When calling an API method in the client, we need to serialize the parameters of this method, send it to the backend, deserialize it in the backend and then finally call the API method implementation. Each parameter itself can be serialized as defined by your implicit Serializers and Deserializers, but multiple parameters need to be combined into a single, serializable type to be send in a request. You can configure, how method parameters are serialized with the following options:

@SerializeParametersAs.CaseClass: Automatically generate a case class representing the parameter lists of each API method: case class _sloth_Api_fun(a: Int, b: String, c: Double). When sending a request, this case class will be used to hold the method parameters, is then serialized and sent to the backend. The backend deserializes the request to this case class and calls the API method implementation. This is the default behaviour.
@SerializeParametersAs.Tuple: Instead of generating a dedicated case class, we just use the Tuple type in scala. So instead of the case class, we will serialize and deserialize a tuple: (Int, String, Double).
@SerializeParametersAs.Single: This serialization method expects every API method to only have one parameter. It will then use this single parameter type as the data transfer object. This is the same behaviour as Tuple with one argument, but does not compile for more than one argument.

Example:
@SerializeParametersAs.CaseClass // or: @SerializeParametersAs.Tuple, @SerializeParametersAs.Single
trait Api {
    def fun(a: Int, b: String)(c: Double): F[Int]
}

see: https://github.com/cornerman/sloth/issues/10

Thanks! I can finally avoid full-auto.
The only thing is, I would rather avoid annotations. Is it possible to "configure" it some other way, perhaps have a different macro invocation? I'd rather have the api itself be fully "ignorant" of the ways it's used (such as Sloth).

The only thing is, I would rather avoid annotations. Is it possible to "configure" it some other way, perhaps have a different macro invocation?

I see, that would be possible as well and would even give more flexibility. One reason why I chose annotations was that I did not want to add another option that can diverge between client and server (the annotation applies this setting to all usages of this API). If you can do Client.withTuples(...) and Router.withTuples, you might configure Client and Router differently by mistake. This is the same as configuring a different PickleType for Client and Router.
An example would be:
val client  = Client.serializeParametersAsTuple[Json, F]
val router  = Client.serializeParametersAsCaseClass[ByteBuffer, F]
// These two would not work together.

We could accept, that you can do these mistakes, they will lead to a runtime errors, when you launch your app. As a helper, we can introduce some kind of ServerClientContract to glue together some configuration for client and server. This could hold the PickleType and the SerializeParametersAs configuration. So you could define your server/client like this:
val jsonConfig = ServerClientContract.serializeParametersAsTuple[Json]

val client = jsonConfig.Client.wire[F]
val router = jsonConfig.Router[F].route[Api](ApiImpl)

What do you think?

As a sidenote: I will probably make Tuple serialization the default. It compiles faster and has less requirements on the used serialization api regarding auto-derivation.

If tuples will be the default I don't care if it's changed via an annotation because I will probably go with the default.
As for ensuring the client and server are generated the same, for that reason and others I've been doing the following. I define this in a cross-compiled module:
trait ApiModule {
  type ApiResult[T] = Kleisli[Future, Option[ActingUser], T]

  type Api[F[_]]

  type ServerApi = Api[ApiResult]
  type ClientApi = Api[Future]

  protected implicit def circeSerializer[T: Encoder]: Serializer[T, String] = _.asJson.noSpaces
  protected implicit def circeDeserializer[T: Decoder]: Deserializer[T, String] =
    parser.decodeAccumulating[T](_).toEither.left.map(Errors)

  protected implicit def monadError(implicit executionContext: ExecutionContext): MonadError[Future, Throwable] =
    cats.instances.future.catsStdInstancesForFuture

  protected def route(implicit c: ExecutionContext): (Router[String, ApiResult], ServerApi) => Router[String, ApiResult]
  protected def wire: Client[String, Future, ClientException] => ClientApi

  val prefix = getClass.getName

  def routeAlone(apiImpl: ServerApi)(implicit executionContext: ExecutionContext): Router[String, ApiResult] =
    route.apply(Router[String, ApiResult], apiImpl)

  def wirePrefixed(f: String => Client[String, Future, ClientException]) = wire(f(prefix))
}

Then specific apis extend ApiModule and define an Api trait inside, also implementing the two methods that actually call the macros:
object DoctorContactsApiModule extends ApiModule {
  trait Api[F[_]]
    extends CRUDApi[F, LLookup[DoctorContact], DoctorContact, LKEnt[DoctorContact]]
      with NotesApi[F, DoctorContact] {
    def forDoctor(doctor: LLookup[Doctor]): F[Seq[LKEnt[DoctorContact]]]
  }
  override protected def route(implicit executionContext: ExecutionContext) = _.route(_)
  override protected def wire = _.wire[ClientApi]
}

(CRUDApi and NotesApi define some methods that several APIs share.)
This file only has to import io.circe.generic.auto._ (and orphan encoder and decoder instances for things it can't auto-derive).
Essentially, I've decoupled "run the macro on the set of methods in this trait" from any server-specific or client-specific code.
The downside to this is that the macro logic runs four times -- it has to generate the router and the client from the trait, in a module that's compiled for both JVM and JS.
Previously, incremental compilation wasn't smart enough to run the macro whenever the API trait changed. This way they are always in the same file. Also, since I'm calling the macro for each API in its own file, I don't trigger rerunning the macro for an API that wasn't changed just because it's in a file that was invalidated.

I'm glad to see progress on #10 here! But I do have a reservation about the current proposal. To me, needing an annotation defining the transport type accompanying the trait definition is a serious case of leaking implementation details into the interface. The API interface shouldn't be tied to it's transport - in theory, you could have the same interface over different transports, e.g. a binary interface over websockets with a fallback to JSON over HTTP.
Would it not be possible, instead of annotations on the trait, to have different client (and server) macros like CaseClassClient, TupleClient, SingleArgumentClient ? It's a choice the user has to make at the time of implementing the transport along with the PickleType, etc.

@ddw-tsc Thank you for your Input :)
Actually, I totally agree with you, the annotations leak information and force a certain style all users of this API. This is mainly the reason I have not merged this PR yet. I would like to define this configuration outside of annotations but this was the first version that I could implement quite easily.
There is just one problem with moving it out of the Api-trait: We need to do the same on the server and the client (either both use case-class or both use tuple or both use single argument). They would not work together if configured differently. For this, I think we should have some kind of Client-Server-Contract that you can use to configure both - so you get consistent behaviour and define it once. Ideas are very welcome :)

In fact the issue of guaranteeing a shared serialization format exists already, since you can have different imports in the file that wires the server and that wires the client.
Above I posted my solution to this. It consists of decoupling "using the wiring macro that generates the code" from the actual server or client implementation. This way I run both macros alongside the trait. This has the downside that when the trait needs to be cross-compiled (e.g. JVM and JS) both macros need to run twice. If there was one macro that did both code generatings it might be a bit faster but ultimately it will be slower, especially if you use full-auto codec derivation e.g. from circe-generic. However it guarantees that the macro for both client and server are run with the same imports in scope. (Having one macro would make the guarantee even stronger since you could technically have an import in a local scope, but that would be very unlikely to happen unintentionally.)
(In my implementation I structure this by putting my API traits inside their own module which extends a base trait that facilitates this setup, however that is just to make it more convenient.)
Of course this doesn't guarantee that your Encoder and Decoder for a given type are symmetric; that is a general issue...

IMO, as nafg has demonstrated above, once you have the general building blocks, you can easily define extra constraints to ensure types match up. As a high-level concept, couldn't one do something like the following?
trait TupleRPC[A] ... {
  val client: TupleClient[A] = ...
  val server: TupleServer[A] = ...
  ...
}

Then , if I have object myRPC extends TupleRPC[MyApiTrait]..., calls to myRPC.client and myRPC.server will use the same strategy. And you can add more parameters for the serialized type, etc. to the TupleRPC trait to make sure more things line up. But I'd still recommend this being an additional building block on top of the raw client & server macros, because sometimes not all server types are simple to express on the client side, so one should have the option of falling back to raw wiring as it was in autowire.

@nafg @ddw-tsc What you are proposing makes sense to me! I also think, it would be good to keep the raw building blocks and invent a structure and API around it to get convenient helpers for the server-client contract.
Currently, I am just really short on time and will not be able to pick this up right now. If anyone is interested, a PR or some help would be awesome :)

I've factored out my ApiModule and related code to GitHub (https://github.com/nafg/sloth-apimodule) and published to bintray jcenter (io.github.nafg.sloth-api-module:core:0.1.0)

Will switch to tuple generation as only method of serialization. There for closing this one. See #100
In the end, I think the configuration is not really beneficial. Tuple serialization works out of the box with most libraries. And if you really want a nice json serialization with pretty names, then you can just use any case class as your single method parameter.


Calling APIs from GB Studio

This was possible in GB Studio version 2.0.0 beta 4, and there is a great example provided by Chris Maltby here: https://chrismaltby.itch.io/gb-api-poc
In my opinion there is a huge potential for e-learning applications, and I have created a proof of concept example for SCORM integration based on the included demo project. If you select "Complete it" in the menu, the game communicates to the LMS that the course is completed, which you may see after closing the game window: https://app.cloud.scorm.com/sc/InvitationConfirmEmail?publicInvitationId=9f931d26-231a-4cf2-a6b1-7e2a63e4e8a9
It would be cool if a similar mod and description could be provided for the latest version of GB Studio. If not, would it at least be possible to post the release files for GB Studio version 2.0.0 beta 4 back on GitHub?

UPDATE 2. I also figured out that it works with 2.0.0 beta 5, but instead of "_script_variables" you should look for "_script_v" in the game.map file. Perhaps there is no need for 2.0.0 beta 4 version then.


PR #96100 - cli: evaluate readiness prior to node decommission

Related PR: https://github.com/cockroachdb/cockroach/pull/96100
Commit: https://github.com/cockroachdb/cockroach/commit/cefa614009a1a50431b3e6aa628d91de923d60f4
Fixes: https://github.com/cockroachdb/cockroach/issues/91893 https://github.com/cockroachdb/cockroach/issues/91893

Release note (cli change): cockroach node decommission operations now
preliminarily check the ability of the node to complete decommissioning,
given the cluster configuration and the ranges with replicas present
on the node. This step can be skipped by using the flag --checks=skip.
When errors are detected that would result in the inability to complete
node decommission, they will be printed to stderr and the command will
exit, instead of marking the node as decommissioning and beginning the
node decommission process. When the strict readiness evaluation mode
is used by setting the flag --checks=strict, any ranges that need any
preliminary actions prior to replacement for the decommission process
(e.g. ranges that are not yet fully upreplicated) will block the
decommission process.

As mentioned in the original PR, the flags for controlling this feature on cockroach node decommission are:
--dry-run               Only evaluate decommission readiness and check decommission status, without
                        actually decommissioning the node.

--checks string         Specifies how to evaluate readiness checks prior to node decommission. Takes
                        any of the following values:
                            - enabled  evaluate readiness prior to starting node decommission.
                            - strict   use strict readiness evaluation mode prior to node decommission.
                            - skip     skip readiness checks and immediately request node decommission.

It's worth noting once again that the default is for the checks to be enabled, (aka the default is --checks=enabled), which means that in some cases, cockroach node decommission will exit in error rather than starting a decommission, if the checks determine that it is not possible to move all replicas at the time the decommission is started.  It should also be noted that these checks specifically determine the ability to move the replicas while following all constraints specified in zone configs - the checks do not (currently, at least) evaluate the effects of moving those replicas, I.e. on CPU or disk usage on the remaining nodes.
Please feel free to reach out if there are any questions about this new functionality.


train doesn't run: ValueError: not enough values to unpack (expected 3, got 2)

Hello. train doesn't run:
danusya@werewolf:~/fast-neural-style$ python3 neural_style/neural_style.py train --dataset ~/dataset/ --style-image ~/Caleido/Color_Kaleidoscope_2017-6-24_15141.png  --vgg-model-dir caleido_vgg --save-model-dir caleido_model --epochs 2 --cuda 0
/usr/local/lib/python3.5/dist-packages/torchvision/transforms/transforms.py:156: UserWarning: The use of the transforms.Scale transform is deprecated, please use transforms.Resize instead.
  "please use transforms.Resize instead.")
Traceback (most recent call last):
  File "neural_style/neural_style.py", line 210, in <module>
    main()
  File "neural_style/neural_style.py", line 204, in main
    train(args)
  File "neural_style/neural_style.py", line 50, in train
    style = utils.preprocess_batch(style)
  File "/home/danusya/fast-neural-style/neural_style/utils.py", line 59, in preprocess_batch
    (r, g, b) = torch.chunk(batch, 3)
ValueError: not enough values to unpack (expected 3, got 2)

Are you using png images?  I ran into this too and found out that this project can only handle jpg.

Yep. Ok, thanks, I'll try without png.

As @xaionaro  pointed out this might be an issue with the format of image, you can convert the image to relevant format using the following code:
from PIL import Image
content_image = Image.open('/path/to/image')
content_rgb_image = content_image.convert('RGB')
content_rgb_image.save('/path/to/output/image')

@abhiskk  - Actually, it would be helpful if this code was incorporated into the project so people can use any type of image. The current error: ValueError: not enough values to unpack (expected 3, got 2) is completely misleading.
@xaionaro - if you want a quick way to convert all png into jpg just install imagemagick and do: mogrify -format jpg *.png

@achaiah - Point noted, I will try to incorporate this soon. Any PRs are also welcome. This code is also part of the pytorch/examples repo, so if you want you can open a PR in that repo also.


Support for Volume resize

FEATURE REQUEST

Below are the manual steps to resize cStor volume:
Prerequisite:
Please make sure all the cStorVolumeReplica CR of the volume needs to be in Healthy state,          before proceeding with steps.
Steps to update cStor volume:
Step 1: Update the volume size in all the pool pods
zfs set volsize=<new size> <dataset name>
Note: This has to be performed inside all the pool pods. You can get the dataset name through
zfs list command.
Step 2: Update the LUN size in istgt.conf by following procedure
- Enter inside the cstor target pod.
- Go to cstor-istgt container and update the LUN size in '/usr/local/etc/istgt/istgt.conf' file.
- Kill istgt process ( ps -aux | grep istgt).
Step 3: Restart all the pool pods.
Step 4: Go to the node where application is running and rescan iscsi using following command
sudo iscsiadm -m node -R
Step 5: Verify if the new size is reflected using lsblk command.
Step 6: Resize the filesystem in the node using
sudo resize2fs </dev/sd*> utility.
Step 7: Verify if the size inside the application pod.
Step 8: If everything is successfull edit/patch the respective PersistentVolume PV object to updated size.

The steps for expanding the storage size of a cStor volume provisioned through CSI provisioner are documented here.
https://docs.openebs.io/docs/next/alphafeatures.html#expand-cstor-volume-created-using-csi-provisioner


Update WebTerminal CodeSandbox

What will this achieve?
Change in web terminal's API.

Closing this since I’m guessing this has been addressed in the recent updates to examples. 🤞


chore: ci: factor vale check out into separate workflow

This is a step towards getting deployments to work for external PRs

Superseded by #202


Include video upload in 'insert/edit media' panel

Describe the feature you'd like
When editing a page, there is an Insert/edit media option:

It would make the UI clearer if there could be a third tab there that allowed for uploading videos. I see the functionality is already there using the 'Attachments' feature. This same upload logic would be good to see integrated into a third tab to keep all the video adding functions together. Once uploaded, the default response would then be for BookStack to insert the newly uploaded video into the opened page as it does when you insert a link from the attachments tab (note, this is not inserting an embedded video to the BookStack link, as that opens a whole can of worms on permissions and rendering, it would just include a link to the file the same way the attachment option does).
Describe the benefits this feature would bring to BookStack users
Explain the measurable benefits this feature would achieve.
From a user interaction perspective, and when writing docs on how to upload videos, this ergonomic improvement would be a great help.

Thanks for the request @maggie0002,
This is already open in #882 so I'm going to close this off as a duplicate. Video on the web is still a tricky thing in terms of support, as shown within that issue, which makes me hesitant as if we provided a flow I'd want it to be easy and reliable for users.

Thanks for the request @maggie0002,
This is already open in #882 so I'm going to close this off as a duplicate. Video on the web is still a tricky thing in terms of support, as shown within that issue, which makes me hesitant as if we provided a flow I'd want it to be easy and reliable for users.


Sweep: Add new API in views.py to support USA NID

Sweep: Add new API in views.py to support USA NID


Proposal: change "networks" to "configs"

Currently, the config command and config.json are structured around the idea that a user will have different configurations for different networks he deploys his zkapp to.
However, there are many more reasons to have different configurations:

deploying different smart contracts, with the same key, to the same network
deploying multiple copies of the same smart contracts, which requires using different keys, to the same network

I think those are very typical, not very esoteric use cases that users will run into pretty quickly.
Both of these and more is actually supported by the current config.json: To deploy two different smart contracts (to the same network), just create two different "networks" which reference different smart contracts. This works perfectly -- however, the naming is weird, so much so that many users might not even have the idea of using "networks" like that.
Thus, I propose to change the naming and overall presentation from "we are creating configurations for different networks", to "we are creating different configurations, which might differ by network / smart contract / zkapp keys"

We briefly talked about this, but need to think more deciding a solution, to ensure it would work well everywhere it appears and that it wouldn't get confusing given we have conig.json which we already refer to as a 'config' too, and a command called 'config'. It might work

Solved using deploy <alias> (PR). There was consensus around this new naming.