Dropdown of the intellisense is not in the correct position

VSCode Version: Code - Insiders 1.19.0-insider (3c470706ded48fc75c05f86bc4f9c7b0f4d6c708, 2017-11-27T05:14:17.223Z)
OS Version: Windows_NT x64 6.1.7601
Extensions:

Extension
Author (truncated)
Version

vscode-eslint
dba
1.4.0

githistory
don
0.2.3

xml
Dot
1.9.2

Go
luk
0.6.66

code-beautifier
mic
1.7.2

vscode-icons
rob
7.16.0

vscode-table-formatter
shu
1.2.1

addDocComments
ste
0.0.8

vscode-import-cost
wix
2.5.1

(1 theme extensions excluded)

Reproduces without extensions: Yes, I checked it
Steps to Reproduce:

Create a file (in my case test.go)
Set the language to Golang
Set the cursor to the End of the file (after the .)
Press Ctrl + Shift + P and reload the window
Press Ctrl + Space to get the intellisense dropdown

The first time after the reload when I try to open the intellisense dropdown it is not on the correct position. When I try it a second time or more, everything is alright. It looks like that:

The testfile to reproduce is:
package main

import (
    "github.com/sqweek/dialog"
)

func main() {
    dialog.Message("%s", "Do you want to continue?").Title("Are you sure?").
}

I'm not so sure, I know that this is a specific issue to Golang intellisense, but the position placement is not done there, it is done in the VS Code core. Although it is the same without any extensions enabled.
Additional I checked if the same behaviour is in an older release persistent. In this VS Code release everything works without any problems with the same test case:

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T18:07:25.676Z)
OS Version: Windows_NT x64 6.1.7601
Extensions:

Extension
Author (truncated)
Version

vscode-eslint
dba
1.2.11

githistory
don
0.2.3

xml
Dot
1.9.2

vsc-material-theme
Equ
1.0.3

markdown-table-formatter
jos
0.2.3

vscode-JS-CSS-HTML-formatter
lon
0.2.3

Go
luk
0.6.66

vscode-icons
rob
7.16.0

addDocComments
ste
0.0.8

(4 theme extensions excluded)

@mxschmitt I am not able to repro this. Do you still see this?

Ah, never mind. I can repro now. I need to press Ctrl+Space right after reload. If I wait for a while, then it doesnt repro. Also if the search/extensions view let is open, then after reload, the cursor is in the side bar. By the time I get the cursor to the editor and try, it doesnt repro. I'll see whats happening

@ramya-rao-a thanks!

@mxschmitt The fix will be out in tomorrow's Insiders build


AllowedMsgAllowance does not subtract fees from spend_limit

Summary of Bug
AllowedMsgAllowance does not subtract the fees from spend_limit. The granter pays the fee correctly, and the module checks spend_limit so if the requested amount of fee is greater than the limit, it rejects the transaction. However, after the transaction, spend_limit, which should have been subtracted by the amount of fee, is intact.
I found the same issue on cosmos-sdk too.
Version
bc28e496b710a91e6ff2c2a8e3699c5d56f3b06d
Steps to Reproduce
$ simd tx feegrant grant $granter $grantee --spend-limit $limit --allowed-messages "/lbm.bank.v1.MsgSend"
$ simd query feegrant grants $grantee // remember spend_limit
$ simd tx bank send $grantee $other $sendamount --fee-account $granter --fees $fee
$ simd query feegrant grants $grantee // spend_limit has not decreased

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

I opened the issue on cosmos-sdk.
https://github.com/cosmos/cosmos-sdk/issues/10563


chore: update to pnpm7

vite-plugin-svelte already switched, kit is following soon and working with pnpm6 and pnpm7 projects at the same time is a hassle locally.

thanks :heart:


Unable to import SQL snippet

This documentation states SQL snippets are supported.
I created a TSQL snippet and imported it into SSMS.  But when attempting to import it into Visual Studio 2017, I receive the error "the snippet files chosen were not valid".
Also, I noticed when authoring the snippet in Visual Studio 2017, Intellisense only offers VB,C#,J#,and XML for the Code element's Lanquage attribute.
VB snippets import fine.

Associated with wrong document


NotamDb initalization failed due to missed message identified during initalization process

We keep getting this error while running FnsClient in our servers. Any way to resolve it?
Here is complete stack trace -
java.lang.Exception NotamDb initalization failed due to missed message identified during initalization process.      LogbackBugsnagAppender.java:71 bugsnag.soschat.LogbackBugsnagAppender.extractThrowable     LogbackBugsnagAppender.java:42 bugsnag.soschat.LogbackBugsnagAppender.append     LogbackBugsnagAppender.java:18 bugsnag.soschat.LogbackBugsnagAppender.append     AppenderBase.java:82 ch.qos.logback.core.AppenderBase.doAppend     AppenderAttachableImpl.java:51 ch.qos.logback.core.spi.AppenderAttachableImpl.appendLoopOnAppenders     Logger.java:270 ch.qos.logback.classic.Logger.appendLoopOnAppenders     Logger.java:257 ch.qos.logback.classic.Logger.callAppenders     Logger.java:421 ch.qos.logback.classic.Logger.buildLoggingEventAndAppend     Logger.java:383 ch.qos.logback.classic.Logger.filterAndLog_0_Or3Plus     Logger.java:522 ch.qos.logback.classic.Logger.error     FnsClient.java:198 com.soschat.notam.source.fns.FnsClient.initalizeNotamDbFromFil     FnsClient.java:366 com.soschat.notam.source.fns.FnsClient.start     NotamAppListener.java:27 com.soschat.notam.source.configuration.NotamAppListener.onApplicationEvent     NotamAppListener.java:14 com.soschat.notam.source.configuration.NotamAppListener.onApplicationEvent     SimpleApplicationEventMulticaster.java:172 org.springframework.context.event.SimpleApplicationEventMulticaster.doInvokeListener     SimpleApplicationEventMulticaster.java:165 org.springframework.context.event.SimpleApplicationEventMulticaster.invokeListener     SimpleApplicationEventMulticaster.java:139 org.springframework.context.event.SimpleApplicationEventMulticaster.multicastEvent     AbstractApplicationContext.java:402 org.springframework.context.support.AbstractApplicationContext.publishEvent     AbstractApplicationContext.java:408 org.springframework.context.support.AbstractApplicationContext.publishEvent     AbstractApplicationContext.java:359 org.springframework.context.support.AbstractApplicationContext.publishEvent     EventPublishingRunListener.java:105 org.springframework.boot.context.event.EventPublishingRunListener.running     SpringApplicationRunListeners.java:78 org.springframework.boot.SpringApplicationRunListeners.running     SpringApplication.java:332 org.springframework.boot.SpringApplication.run     SpringApplicationBuilder.java:139 org.springframework.boot.builder.SpringApplicationBuilder.run     DefaultBinderFactory.java:250 org.springframework.cloud.stream.binder.DefaultBinderFactory.getBinderInstance     DefaultBinderFactory.java:174 org.springframework.cloud.stream.binder.DefaultBinderFactory.doGetBinder     DefaultBinderFactory.java:120 org.springframework.cloud.stream.binder.DefaultBinderFactory.getBinder     BindingService.java:313 org.springframework.cloud.stream.binding.BindingService.getBinder     BindingService.java:214 org.springframework.cloud.stream.binding.BindingService.bindProducer     BindableProxyFactory.java:262 org.springframework.cloud.stream.binding.BindableProxyFactory.createAndBindOutputs     OutputBindingLifecycle.java:55 org.springframework.cloud.stream.binding.OutputBindingLifecycle.doStartWithBindable     LinkedHashMap.java:608 java.util.LinkedHashMap$LinkedValues.forEach     AbstractBindingLifecycle.java:47 org.springframework.cloud.stream.binding.AbstractBindingLifecycle.start     OutputBindingLifecycle.java:34 org.springframework.cloud.stream.binding.OutputBindingLifecycle.start     DefaultLifecycleProcessor.java:182 org.springframework.context.support.DefaultLifecycleProcessor.doStart     DefaultLifecycleProcessor.java:53 org.springframework.context.support.DefaultLifecycleProcessor.access$200     DefaultLifecycleProcessor.java:360 org.springframework.context.support.DefaultLifecycleProcessor$LifecycleGroup.start     DefaultLifecycleProcessor.java:158 org.springframework.context.support.DefaultLifecycleProcessor.startBeans     DefaultLifecycleProcessor.java:122 org.springframework.context.support.DefaultLifecycleProcessor.onRefresh     AbstractApplicationContext.java:893 org.springframework.context.support.AbstractApplicationContext.finishRefresh     ServletWebServerApplicationContext.java:163 org.springframework.boot.web.servlet.context.ServletWebServerApplicationContext.finishRefresh     AbstractApplicationContext.java:552 org.springframework.context.support.AbstractApplicationContext.refresh     ServletWebServerApplicationContext.java:142 org.springframework.boot.web.servlet.context.ServletWebServerApplicationContext.refresh     SpringApplication.java:775 org.springframework.boot.SpringApplication.refresh     SpringApplication.java:397 org.springframework.boot.SpringApplication.refreshContext     SpringApplication.java:316 org.springframework.boot.SpringApplication.run     SpringApplication.java:1260 org.springframework.boot.SpringApplication.run     SpringApplication.java:1248 org.springframework.boot.SpringApplication.run     SpringNotamSourceApp.java:14 com.soschat.notam.source.SpringNotamSourceApp.main     NativeMethodAccessorImpl.java:-2 jdk.internal.reflect.NativeMethodAccessorImpl.invoke0     NativeMethodAccessorImpl.java:62 jdk.internal.reflect.NativeMethodAccessorImpl.invoke     DelegatingMethodAccessorImpl.java:43 jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke     Method.java:566 java.lang.reflect.Method.invoke     MainMethodRunner.java:48 org.springframework.boot.loader.MainMethodRunner.run     Launcher.java:87 org.springframework.boot.loader.Launcher.launch     Launcher.java:50 org.springframework.boot.loader.Launcher.launch     JarLauncher.java:51 org.springframework.boot.loader.JarLauncher.main

When was the last pull performed for the code? A bug was fixed that would prevent reinitialization processes to start if a message was missed during initialization. Please ensure you have the most recent code version and follow up if the issue still persists. Also, when posting about missed messages, please include the time the message was missed.

Let me take the latest and rebuild our service.

@armFAActr I updated to latest code and here is what I see in logs, it keeps invalidating notam db and reinitialize it because of missed messages -
2021-04-12 15:27:04,167 INFO - dev-76 notam-source -  - : - FnsClient.initalizeNotamDbFromFil(187) | NotamDb initalized
2021-04-12 15:27:04,168 WARN - dev-76 notam-source -  - : - FnsClient$2.onMissed(93) | Missed Message Identified, setting NotamDb to Invalid and ReInitalizing from FNS Initial Load | Missed Messages {3883032:2021-04-12T19:21:49.780864Z, 3883023:2021-04-12T19:21:49.780864Z, 3883036:2021-04-12T19:21:49.780864Z, 3883024:2021-04-12T19:21:49.780864Z, 3883015:2021-04-12T19:21:46.931346Z, 3883014:2021-04-12T19:21:46.931346Z, 3883029:2021-04-12T19:21:49.780864Z, 3883013:2021-04-12T19:21:46.931346Z, 3883028:2021-04-12T19:21:49.780864Z

When you look at your SCDS metrics page, are there any discarded messages shown during the time after connecting?

@armFAActr where I can see that, this is what I see in metrics page -

also, it takes forever to get notam db initialized. Why there is so much delay? it keeps printing this - Waiting for New FNS Initial Load File, current file modified time: Wed Apr 14 16:51:02 EDT 2021

To help isolate the potential issue with the missed message you keep seeing. I have created a basic client that just connects, consumes, and tracks messages using the missed message tracker.
Try to download this (https://s3.amazonaws.com/am.transfer/SWIM/fns-mmt-test.zip), update the app.conf file with your connection details, and then run it using the following command:
java "-Dconfig.file=app.conf" -jar .\fns-mmt-test-1.0.jar.
Let me know if you still see missed messages;

@armFAActr here are the logs, I don't see any missed messages though.
missed-messages.txt

@armFAActr got missed messages now - see in attached file
missed-messages.txt

Please check and ensure you do not have any filters on your SCDS subscription. fns-client requires all notmas messages to function.

also, it takes forever to get notam db initialized. Why there is so much delay? it keeps printing this - Waiting for New FNS Initial Load File, current file modified time: Wed Apr 14 16:51:02 EDT 2021

the FIL file is updated every 3 minutes at the most and can be longer if the processes of generating the FIL file takes longer. During initialization the fns-client checks for a FIL file that is updated after the initialization processes starts to ensure no notams are missed between the last time FIL was updated and when the initialization processes started.

Please check and ensure you do not have any filters on your SCDS subscription. fns-client requires all notmas messages to function.

@armFAActr
Doesn't look like that. This is what we have -

Ok, will look into the logs again and will follow up

I did notice in the graph you sent showed it going from one to two connection. Did you run the client twice? Make sure no other clients are connected and try to rerun the test tool sent previously and let me know if you still see missed messages. SCDS uses queues which don't equally distribute messages to multiple clients. If you need to run two clients then you will need to creat another subscription.

I did notice in the graph you sent showed it going from one to two connection. Did you run the client twice? Make sure no other clients are connected and try to rerun the test tool sent previously and let me know if you still see missed messages. SCDS uses queues which don't equally distribute messages to multiple clients. If you need to run two clients then you will need to creat another subscription.


IO position not updated while reading a CSV file

While reading CSV files the pos method always return first line position.
It seems the issue appeared since this commit (found using git bisect)
To illustrate it i made a test file.
I also made a fork on my own repo with the test file and a csv file (in test/csv/proof) and tried to fix it.
My solution is to pass down the @input instance from Parser to Scanner and update position while iterating within each_line method.
I wanted to make sure it was an issue (or maybe a bad use from me) before opening a Pull Request
I am using Ruby 2.7.2
Best regards,
Emmanuel KONZI.

Could you show your use case of CSV#pos?

Yes. Basically It looks like this:
#file = File.expand_path(File.join(File.dirname(__FILE__), file_name))
CSV.open(file, 'rb', {col_sep: ';', encoding: 'ISO-8859-1'}) do |io|
  pos = io.tell
  io.each { |_| positions << pos; pos = io.tell }
end

I got a full example of the test in a gist

Thanks.
Could you also provide why do you want to use CSV#pos and how did you use CSV#pos in real use case?

Could you also show the positions value of the example script on your environment?

Thanks.
Could you also provide why do you want to use CSV#pos and how did you use CSV#pos in real use case?

In my application i have an autocomplete dropdown for some cities zipcode from a csv file.
So to be able to search effectively in that file we use the CSV#pos to build an index for the cities name chuncks.
For example: We know in that file Moscow is at position 45 and Monrovia at the position 367.  Our index is build with the chunks of Moscow and Monrovia like this:
M =>45, Mo => 45, Mos => 45, Mosc => 45, ... Moscow =>45
M => 367, Mo => 367, Mon => 367, ..., Monrovia => 367

So when someone enters M or Mo we can propose line Moscow and Monrovia based on the fact that we know M or Mo are on the lines 45 and 367.
Does it make sense ?

Could you also show the positions value of the example script on your environment?

@kou here is an output of the script.
➜  csv git:(master) ✗ ruby test/csv/proof/proof_test.rb

Loaded suite test/csv/proof/proof_test
Started
0
450
450
450
450
450
450
450
450
450
450
F
==============================================================================================================================================================================================================================================================================
     12:     positions = @csv_reader.read_positions
     13:     puts positions
     14:     # Before commit 8505ff0 this test was passing. But now position stays at the first line.
  => 15:     assert_equal positions, positions.uniq, "Not unique"
     16:   end
     17:
     18:   class CsvReader
test/csv/proof/proof_test.rb:15:in `test_read_positions'
Not unique
<[0, 450, 450, 450, 450, 450, 450, 450, 450, 450, 450]> expected but was
<[0, 450]>

diff:
? [0, 450, 450, 450, 450, 450, 450, 450, 450, 450, 450]
Failure: test_read_positions(ProofTest)
==============================================================================================================================================================================================================================================================================

Finished in 0.007358 seconds.
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
1 tests, 1 assertions, 1 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
0% passed
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
135.91 tests/s, 135.91 assertions/s

How about using CSV#lineno instead of CSV#pos for your use case?
We may read a large chunk at once for performance instead of reading each line. So I don't want to recommend users that they expect CSV#pos returns the position of the current parsing line.

Yes we could have used lineno but we would have to change the way we look into the file. For now we are using CSV#seek (which we found pretty convenient).
Please could you show me some valid examples of CSV#pos usage ?
If  CSV#pos is not expected to return the position then we need to update the docs or check if other io-delegated methods keep working.

How about caching a parsed CSV object in memory instead of using CSV#pos and CSV#seek? It'll be faster.
I don't think CSV#pos is useful for parsing generally.  And CSV#seek isn't safe for parsing generally. For example, it doesn't work for a CSV file that has BOM. Because we need to read the first some bytes for the file. It also doesn't work for a case that needs to read headers the CSV file. The current implementation also doesn't work if CSV#seek is called after starting parsing. If we support it, we need to care it like CSV#rewind.

Yes it will be faster for sure but we got a really big csvfile so am wondering about memory growth issues. Anyway thanks for the advice, we will try to experiment it.
Then i guess the changes about CSV#pos behaviour are known from you and is not an issue from what i can see. Thanks for all the replies. Can i close the issue ?
But am still waiting for some valid example of those io-delegated methods (such as CSV#pos)

I think that it's time you switch to other approach such as using DB from CSV in your situation.
I close this.
I don't think most of those IO-delegated methods are useful. They still exist for historical reasons. They may be useful with CSV.open but I'm not sure...


Feature: Warning message after deleting profile picture

Is your feature request related to a problem? Please describe

Describe the solution you'd like
Once you click on the 'remove picture' on your profile dashboard, it immediately removes the picture without flashing any error message to the user.
Describe alternatives you've considered
No response
Additional context
No response
Are you working on this?
Yes

@tanmoysrt Can you please review this ?

Hi @Simer13 can you update the attached image in issue description

@tanmoysrt I updated the image

@Simer13, please assign this issue to me.

@tanmoysrt please assign me this issue

@Simer13 you can work on ths issue

@tanmoysrt Ok I will. Thanks!

i am working on this


Generate script doesn't work as expected in Schema Compare after include/exclude

Steps to Reproduce:

Start a schema compare with two databases.
Include/exclude some of the compare results.
Try to generate script.

Expected: Script is generated.
Actual: Nothing happens until include/exclude dependency calculation finishes.
Anticipated: Add a visual trigger to show that include/exclude dependency calculation is underway (a toast notification or a task created) and Generate Script/Apply buttons should be disabled during the time when the dependency calculation is happening.

I was very frustrated until I discovered that it was actually recalculating in the background. It absolutely needs a visual indicator.

IMO it shouldn't let you generate a script or apply the changes to a DB before it has recalculated the Include/Exclude.
I have accidently deployed a change where I have explicitly removed the include tick because of this.


Fix issue #309: server.wait_closed()

Fixes issue #309.

Still working on #309...
Apart from that, the decorator that allows writing async tests is a great idea! Two suggestions to make it even better:

It could apply @asyncio.coroutine automatically to remove the need to stack two decorators on async tests.
I'm not completely sold on the @make_sync name. @async_test could work — decorator names usually describe a feature of the object being decorated. Or perhaps @run_in_event_loop.

Currently each test already run it its own event loop:
    def setUp(self):
        self.loop = asyncio.new_event_loop()
        asyncio.set_event_loop(self.loop)

    def tearDown(self):
        self.loop.close()

Oh, right. It's been several months since I looked. What I may have been thinking of then is that there can be different decorators that each do varying levels of setup (e.g. (1) doing nothing other than running the loop, (2) also starting and stopping the server, (3) also starting and stopping the client and server, etc). And depending on things, perhaps it could also make sense to put starting and stopping the loop in the decorators instead of splitting things by putting some logic in the setUp and tearDown and some in the decorators.

Yes, that makes sense. Good ideas.

and I neglected websockets for several months as well, nothing wrong with that ;-)

I filed #403 to follow up on the async test refactor.


Dont work in Windows 10 on "npm start"

Case 1: You installed through npm
If you installed this package as a node module through npm, you can't run the start script. The npm version of this package is meant to be required by an application.
If you want to test this take a look at https://github.com/fbessou/angular2-schema-form-demo
Case 2: You have cloned from github
Your screenshot does not include the informative part of the message. A guess would be  that you have to set the environment variable ENV_NODE to production manually before calling npm start(or try to use a unix shell):
set NODE_ENV=production

Note that currently this module (and angular 2) is currently unstable and should not be used in production code

Sorrry. I tried to install "https://github.com/fbessou/angular2-schema-form-demo" on windows 10 and it failed. On ubuntu 16.04 it works.

Seems like the port 3002 is already taken on your system. You can change it in webpack.config.js in the devServer section.

3002 is free. I tried on 3000.....it doesnt help. .Seems like the webpark-dev-server not compatible with windows.

Sorry, it was a problem with the port.

Ok, I'm closing this issue then.

Ok, thx)


Error in installing hough voting layer (cudatoolkit = 9.1)

Great work on the paper!
I am trying to use some of it for a related application. I have created a virtual environment using Anaconda and Pip as recommended. Details of the environment follow. (PyTorch didn't seem to have any binaries for CUDA 9.1, so I used the ones compiled with 9.0). python setup.py install gives multiple errors (see below). Any recommendations regarding what may have caused this error and how to fix it would be deeply appreciated.
Environment details
Collecting environment information...
PyTorch version: 1.0.0
Is debug build: No
CUDA used to build PyTorch: 9.0.176
OS: Ubuntu 18.04.4 LTS
GCC version: (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
CMake version: version 3.10.2
Python version: 3.7
Is CUDA available: Yes
CUDA runtime version: 9.1.85
GPU models and configuration:
GPU 0: GeForce GTX 1080 Ti
GPU 1: GeForce GTX 1080 Ti
Nvidia driver version: 440.100
cuDNN version: Could not collect
Versions of relevant libraries:
[pip3] numpy==1.18.5
[pip3] torch==1.0.0
[pip3] torchvision==0.2.1
[conda] blas                      1.0                         mkl
[conda] cuda90                    1.0                  h6433d27_0    pytorch
[conda] mkl                       2020.1                      217
[conda] mkl-service               2.3.0            py37he904b0f_0
[conda] mkl_fft                   1.1.0            py37h23d657b_0
[conda] mkl_random                1.1.1            py37h0573a6f_0
[conda] numpy                     1.18.5           py37ha1c710e_0
[conda] numpy-base                1.18.5           py37hde5b4d6_0
[conda] pytorch                   1.0.0           py3.7_cuda9.0.176_cudnn7.4.1_1    pytorch
Error
I get multiple errors (as included in the following file) when trying to install the Hough voting layer using python setup.py install
error.txt

Hi @ektas0330, my guess is that your CUDA runtime version and PyTorch CUDA version are slightly off. If that's the case, the compilation success is not guaranteed. Please try using a version of PyTorch that aligns with your CUDA runtime version.


Deadlock in ShelfPlugin

When a shelf plugin is not properly closed, reusing the shelf file results in an error:
DBPageNotFoundError: (-30986, 'BDB0075 DB_PAGE_NOTFOUND: Requested page not found')

I am having some problems when i am developing a plugin that extends from ShelfPlugin. The problem is the same that is described above. The first iteration loads everything successfully and creates the ".db" file. But when i am trying to access to the ".db" file, in the next iteration, it gives me the same error.
The part of the code where i am loading the values into the ".db" file is described below:
    if 'classifier' not in self.sh:
        classifier = utilities.train(data,self.sh['classes'])
        self.sh['classifier'] = classifier
    self._classifier = self.sh['classifier']

You need to deactivate the plugin, delete it, or close the shelf via the plugin's close method.


starting container with non-empty request body was deprecated since v1.10 and removed in v1.12

Expected behavior
start container
Actual behavior
very briefly a 400 error is shown with the following message:
starting container with non-empty request body was deprecated since v1.10 and removed in v1.12
immediately followed by showing an empty black log page
Information about the Issue
this problem started after an upgrade of Docker for Mac 17.06.0-rc1
Steps to reproduce the behavior

fresh install of Docker for Mac and Kitematic and it will happen

I got the same problem！！Kitematic works well before I have my lunch
I forgot if if it's occur after I update my docker.
masOS

Same to me. Able to start via CLI, but unable to via Kitematic interface.

@DmitriyBaklikov @basvanbeek
Hey guys~
I reinstall the stable version of docker-for-mac, and the problem sloved.
Seems like something goes wrong with the rc version of docker.

Thanks for the report everyone, I'll check out the RC and will see what's going on.

Opened a bug on the moby/moby project:
https://github.com/moby/moby/issues/33501

Same here. Updated to  17.06.0-rc1-ce-mac13 (18169) and right away Kitematic cannot start any container. Cli works. Let me know if you need info.

Looks like it's a bug on the proxy that is pushed with Docker for Desktop - we'll await a fix on their part.
Thanks everyone for the QA and bug report!


Add Snack Parts Updated (fixed KerbalStuffBot submission)

Closes #3022

Damn, another one of these weird ones:
FATAL CKAN.NetKAN.Program (null) - Unexpected character encountered while parsing value: <. Path '', line 0, position 0
Is this something about the zip file?

KerbalStuff is presently having issues, likely resulting in this failure. Force a rebuild at a later time and it should be fine. JSON is valid and manually looking at how this should install/its relationships looks fine. Good to go once bots give a go-ahead.


[BUG] Cwrapper preloader going wrong on iFrame mode

Describe the bug
The cwrapper mode preloader wile in iFrame mode shows a second menu while loading.
Steps To Reproduce
Steps to reproduce the behavior:

Enable iFrame mode
Set preloader mode to 'crwapper'
Open default home page
See error

Expected behavior
A preloader in fullscreen only in the iFrame
Screenshots
https://github.com/jeroennoten/Laravel-AdminLTE/assets/42849081/9b934778-90a7-4c28-b4ed-92faabcc6466
Environment

Item
Version

Laravel
11

Project
v3.11.0

OS
WSL ubuntu

Additional context
Maybe, check if iFrame mode is enabled before the preloader rendering is a solution.

Hi @ruanpepe , for what I have experimented, the iFrame mode has lots of bad visual effects and they are inherited from the underlying AdminLTE  package, when they show a page embedded in the iFrame mode, the sidebar for example is removed using Javascript, but there is a transition animation that don't make it look smooth. Also, the AdminLTE package appears to be currently unmaintained , so I think we can't expect any improvements on how the i Frame mode works and we can't do much on this package to solve those issues.
Now, in relation to your problem, are you sure it's exclusively related to preloader on cwrapper mode, i.e, have you tested the same with the preloader disabled?
Also, as a workaround, you may try to increase the value in the loading_screen option of the iframe section (for example, to 2000). That way, maybe the loading screen may hide those bad visual effects that happen while loading the tab.

Hi @dfsmania. How are you?
Yes. The iFrame mode is very useful, but it comes with some visual issues.
I tried now without preloader, and the behavior is still going bad. Then, i've been using the preloader with fullscreen mode and it was "hiding" the menu bug. I'll keep the preloader in fullscreen mode.
Thanks for your time.


Transition group is not working in

Version
2.6.11
Reproduction link
https://jsfiddle.net/78ucsog3/
Steps to reproduce
I've created a reproduction of the issue. Basically, transition animations are not working in the iframe. I have a case where I need the render vue components in the iframe, it works great, but animations are not working.

I copied the example List transitions from https://vuejs.org/v2/guide/transitions.html#List-Entering-Leaving-Transitions
Wrapped it in custom  component.
Transitions are not triggering. I also injected css transition classes to the iframe but still no luck.

What is expected?
Transition animation should trigger in the iframe
What is actually happening?
Transition animation is not triggering in the iframe

You need to add the CSS to the iframe's document and it will work.

Please, next time consider using the forum, the Discord server or StackOverflow for questions first. But feel free to come back and open an issue if it turns out to be a bug 🙂

It worked well as yo said. I corrected it in the fiddle in case someone needs : https://jsfiddle.net/L0srtdxe/61/


Fix skeleton import from glTF

For some glTF files, the order of bones in the skeleton array wasn't matching the joints array in the meshes.
Fixes #17808.
NOTE: I can't be completely sure this won't break any model that was being correctly imported so far.

Cherry picked into 3.0.3


Two authentication elements with same api-name and api-location being shown on rapidoc screen

So im trying to provide an api-key called "Authorization" which is in the swagger spec. but instead i recieve two instances of the Authorization key with the location header on my rapidoc screen.
Is this expected behavior or is this a bug.
Rapidoc screen:

code:

from api:

This is an expected behavior,
OpenAPI spec only allows you to define api-key but you cannot provide its value
This is a very common feature that our users have requested where they like to supply their own api-key and value in a declarative manner and also like to control it programmatically. Therefore RapiDoc provides a way to supply API Key and its value from outside the spec using attributes
The api-key-name, key-value and key-location is dynamically injected into the spec and has no connection with keys that already exist in the spec.
RapiDoc UI supports sending multiple or selective api keys during TRY call,
It is common for APIs to ask for multiple API keys to resolve authorization scope
For you case I dont think it will be an issue have multiple api-keys with same name (Authorization ) as long as you make sure that only one key is applied.
If you are concerned that it may confuse your user, then based on your needs remove the api-key either from spec or attribute


Deprecate Accessors for Legacy Error Handlers

cherry-pick to 2.8.x

... and cherry-picked to 2.8.x

I haven't decided yet whether to completely remove these in 3.0 (although I will definitely remove the legacy error handler implementations, hence my other PR https://github.com/spring-projects/spring-kafka/pull/2100).


Suggestion - UI Revamp (based on Paint.NET)

I was thinking about a rework of the UI. There are many paint/photo editor software and for example I really like the simpleness of the Paint.NET (http://www.getpaint.net/index.html)
The UI is pretty simple and it's really user-friendly (atleast for me). Every window can be moved and the extra features can be accessed through the menu bar. Also, you can edit multiple files within one instance.

I think the same concept / layout could be used at Piskel too and it would be easier to make it compatible with lower resolutions. (mobile)
Though I really like the current layout, it's not so extensible, really dependent on the resolution and static / not customizable.
Frames, Layers, Palettes, Tools, Primary / Secondary color, Preview -> separate window (movable)
Settings -> menu bar (opens a dialog box rather than a drawer)
All of this in the dark theme with later additional support for a light theme.
I could tackle the revamp after creating some prototypes and posting it here for feedback.

Adding myself to this so I can follow along.
We've added a css file to our fork of Piskel to give it a different style. We haven't changed any dom elements to do this, and our hope was that we'd still be able to pull updates from here without too many conflicts. Depending on how you make these changes, I'll probably still try to keep the code-dot-org fork in sync.

I achieved the dynamic UI with very minimal changes to the DOM and CSS.
I think the framelist should stay on the left with a button to hide/show and/or dock/undock.
I was just playing around. The header will be replaced with a menu bar for accessing some other features. The right side settings drawers will be replaced with dialogs and the actions will be moved to the menu bar.
What do you guys think? Should I continue working on it? @juliandescottes

Wow! I like the improvements, although I would like to be able to dock all the dialogs to the edges of the workspace because I don't like floaty windows getting in the way of the my drawing. Without docking, their placement just feels too arbitrary and cluttered. However, being able to arrange them however I like would be useful, especially for varying screen sizes. Also, I like the idea of being able to hide / show windows. I would prefer collapsing dialogs to just the dialog title instead of hiding the entire dialog.
Should window layout be stored in the browser, in the server account, or just destroyed with the session?
Also a minor nitpick: Could you make the dialog shadows be transparent black? Grey shadows on top of a darker background always stick out to me.
Wait - those aren't shadows, those are oddly thick dialog borders. My bad. The border color isn't very distinct from the background of the drawing area, and should shadows be added?
Keep up the great work!

👍
Thanks for the proposition. I think overall having more customization for the UI is the way to go.
tldr: this is going in the right direction, but there's a lot to discuss ! :)
One concern with this approach is the default behavior. We need to make sure that by default, the UI looks nice, works well when resizing the window, when adding layers & colors.
With floating panels, I think some panels should also be reworked:

move the color pickers with the color palettes (to actually make a panel similar to the colors panel of paint . net)
palette and layer panels should be resizable vertically (and no longer resize automatically depending on their content)

Floating panels would also enable some other nice features in the future

preview panel could be resizable (and we could remove the current preview popup, which is often inconvenient)
tools (& transform tools) panels could also be resized to have more horizontal icon layouts

I also agree with @epicabsol , it would be nice to pair this with a good docking mechanism. This means docking on the edges of the viewport, as well as between panels.

Should window layout be stored in the browser, in the server account, or just destroyed with the session?

For now we can only store it in the browser, later on we could think about saving user preferences on the piskel account.
Some minor feedback on the current approach:

keyboard shortcut icon (bottom left) will have to move(to a menu maybe, otherwise it will overlap with the framelist)
cursor coordinates (bottom right) are no longer visible. Needs add a background to make sure they're always readable?
zoomed out margin color (lightgrey) should maybe be the same color as the background color now ?

While I like being able to move around the tools, it'd be great to have a flag or option to be able to lock tools in place (from the code). Code.org is using piskel in Gamelab, which a tool for middle/high school students to program their own games. I'd be worried that if we used this UI for our use case, students could get in a bad state and have trouble getting back. Also schools tend to have much smaller screens with lower resolution and we wouldn't want anything to be more complicated for them.

@david-szabo97 Hi! Are you still interested in working on this?
I think this would really be a great feature, provided we can make it optional and ensure the original layout still works the same as today.

@juliandescottes Hi! It would be great to work on this but I'm kinda busy.
If I will have time for this and this is still undone then I can take it.


fix(asinput): add dangerIconDescription as prop

Also, allow passing validationMessage as an element.
These are some additions to the PR @mikix merged last week. I only noticed these things after I tried to use isValid and validationMessage in studio-frontend.

Coverage decreased (-0.4%) to 99.24% when pulling c6fd26f0bc36005b2af6ae7c2fd9c648e0c71d0b on thallada/async-validation-tweaks into cf54fd626c94d580bf6f3b74e79b3a82ad301b7f on master.

Question: is specifying dangerIconDescription necessary for proper a11y? If so, can there be a reasonable default in the common "danger" theme case?

@mikix I'm hesitant to give a default because users of Paragon might get English when they don't expect it. Paragon doesn't have an infrastructure to do translations and I don't think it currently has any display text. It expects users to pass in translated strings themselves.


Running flux.1-schnell

Is it possible to create loras of of flux.1 schnell using this code?

Not yet. That is something that I wanted to add later.

Have you made any progress in this matter?


Make all package managers use the new dependency graph format

Currently, only the Gradle analyzer uses the new dependency graph format introduced in https://github.com/oss-review-toolkit/ort/pull/3502. We should take advantage of the new format for more (ultimately all) package manager implementations.
Package managers that still need to be migrated are unchecked in this list:

[ ] Bazel
[ ] Bower
[ ] Bundler
[ ] Cargo
[ ] Carthage
[ ] CocoaPods
[ ] Composer
[ ] Conan
[ ] GoMod
[x] Gradle
[ ] GradleInspector
[x] Maven
[x] Npm
[ ] NuGet
[ ] Pub
[ ] Pip
[ ] Pipenv
[ ] Poetry
[ ] Sbt
[ ] SpdxDocumentFile
[ ] Stack
[ ] SwiftPm
[ ] Yarn
[x] Yarn2

To better track the progress, I propose to use this as an umbrella ticket and create separate tickets for specific package managers.

The rework of the Maven package manager (#3895) is now done (see #3894). Unfortunately, the results from first tests are rather disappointing wrt reduction of file sizes. I don't have concrete numbers yet from a direct comparison (a test build is still running), but the result file in the new format is still quite big.
It seems that the situation for Maven projects is a bit different than for Gradle Android projects. In the latter case, a project typically has high redundancy due to the various scopes with almost identical dependency trees. A Maven module in a multi-module build in contrast has rather disjunct scopes, and packages do not appear multiple times in the dependency tree. As the dependency graph is constructed on a project-base, there is not much benefit from the new format.
There is, however, a lot of redundancy between the dependencies of the single Maven sub projects. So a promising approach could be to construct a global dependency graph for an analyzer run. This would be rather easy with the current DependencyGraphBuilder class; a new builder instance would not be created for each project, but only a single one, and there would need to be a convention to construct unique scope names qualified with project IDs.
A major problem would be that this is quite a big change in the data format of the analyzer and affects other model classes as well. Especially for projects it would be tricky to re-associate them with their part of the dependency graph.

There is, however, a lot of redundancy between the dependencies of the single Maven sub projects.

Oh, then I must have misunderstood something about the current new dependency graph format, as I though exactly that is already possible: The re-use of re-occurring subtrees within a single project / scope.
So for example, if we had
project
+- scope
   +- depA
   |  +- depC
   |     +- depD
   |     +- depE
   +- depB
      +- depC
         +- depD
         +- depE

then I though depC (incl. all its transitive dependencies, i.e. the whole subtree) would only be stored once, and referred to from depA and depB. Is this not the case?

Yes, within a single project, this reuse is possible, and dependencies are shared. What I meant was that there are obviously not so many duplicated elements in the dependency trees of single projects, so the graph format does not have a positive effect.

But would it really be such a big change to support this subtree reuse across projects? Wouldn't we basically just need to take care that identifiers for subtrees are unique across projects? And that could easily be archived by not using consecutive indices, but hashes on the subtree instances, or?

I can do some experiments in this direction. Seems to be an interesting problem.

Here's a brief migration guide that I came up with while working on https://github.com/oss-review-toolkit/ort/pull/9073:

Refactor existing code so that all metadata for a dependency is available in a single data class.
Implement a DependencyHandler that returns various ORT model classes for that data class.
In the specific package manager class, create a val graphBuilder = DependencyGraphBuilder(...) member variable that takes an instance of that DependencyHandler.
In resolveDependencies(), call graphBuilder.addDependency(DependencyGraph.qualifyScope(...), ...) on each dependency.
Change creating a Project from setting scopeDependencies to setting scopeNames = graphBuilder.scopesFor(projectId).
Change the returned ProjectAnalyzerResult to take an emptySet() as packages.
Override createPackageManagerResult() with PackageManagerResult(projectResults, graphBuilder.build(), graphBuilder.packages()).


Fix S3 creation and access from MLflow

The ACL issue with S3 creation is fixed
The k8s service acccount in mlflow namespace should be able to assume the node role to access the S3 bucket now.

@strickvl just tested this with the latest changes from develop. Looks good!


Update CHANGELOG

fixed issue #8294: wrong equals behaviour on arrays with ArangoSearch

fixed internal issue #528: ArangoSearch range query sometimes doesn't work
correctly with numeric values

FYI: "internal" refers to backlog repo in this case: arangodb/backlog#528

FYI: "internal" refers to backlog repo in this case: arangodb/backlog#528

Yes. IResearch has it's own repo for this. I can't change it atm.


Update NSRDB downloads to V4

NSRDB V4.0.0 data (1998-2023) is now available from new NREL Developer API endpoints:
https://developer.nrel.gov/docs/solar/nsrdb/nsrdb-GOES-tmy-v4-0-0-download/
https://developer.nrel.gov/docs/solar/nsrdb/nsrdb-GOES-aggregated-v4-0-0-download/
SAM's Advanced Downloads option can access these endpoints, but the direct download from the Location and Resource page still accesses the older V3.2.2 (1998-2022) API endpoint.
Note there is an "aggregated" and "conus" endpoint. The latter seems to be a subset of the former, but need to determine the correct endpoint to use.
To Do:

Update NSRDB download code to access V4.0.0 endpoint
Spot-check V4.0.0 files for issues
Test for locations around the world

Data in weather files from GOES V4 endpoing is offset so that DNI peaks around 8 pm instead of Noon. Awaiting fix from NSRDB team before updating in SAM.
Test URL to download example file:
https://developer.nrel.gov/api/nsrdb/v2/solar/nsrdb-GOES-tmy-v4-0-0-download.csv?api_key=DEMO_KEY&full_name=sam_user&email=test.email@email.org&reason=SAM-test&interval=60&utc=false&leap_day=false&attributes=dni,dhi,ghi,dew_point,air_temperature,surface_pressure,wind_direction,wind_speed,surface_albedo,clearsky_dhi,clearsky_dni,clearsky_ghi&names=tmy&wkt=POINT(-122.676 45.5118)

@cpaulgilman, is this issue with GOES V4 being tracked/reported out somewhere publicly? Or is the SAM team simply communicating directly with the NSRDB team?
It would be nice to get public updates on this from the NSRDB team...

@cpaulgilman, is this issue with GOES V4 being tracked/reported out somewhere publicly? Or is the SAM team simply communicating directly with the NSRDB team?
It would be nice to get public updates on this from the NSRDB team...

To be honest, we accidentally stumbled on the update to GOES V4. We discovered the offset issue and reported it to the NSRDB team. I just heard back from them that it's been fixed, and my testing confirms the fix, so we'll proceed with the update in SAM For the December 12, 2024 release.


nixos/zeronet: add fileserverPort option

Motivation for this change
Without it, zeronet tried to write one to the read-only config file and
crashed
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

/cc @Chiiruno

LGTM


feat(consensus): add event handling to the state machine

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2161
#2160  👈
#2159
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @matan-starkware and the rest of your teammates on  Graphite

crates/sequencing/papyrus_consensus/src/state_machine.rs line 193 at r2 (raw file):

    }

    fn advance_step(&mut self, step: Step) -> Vec<StateMachineEvent> {

Consider renaming
Suggestion:
advance_to_step

crates/sequencing/papyrus_consensus/src/state_machine.rs line 163 at r2 (raw file):

    }

    // A proposal from a peer (or self) node.

Why self?
Code quote:
(or self)


Modal: Is it possible to inject its own controller?

As the title suggests - with ui.bootstrap, you can inject a controller by specifying it upon initializing. I don't really see the same option with QuantumUI. Does it exist, and if so, how?
I know you can specify a controller through the view, however it doesn't allow me to interact with the modal itself (ex: self-close upon successful login).

+1


Update install.md

Typo in composer command

Coverage decreased (-0.004%) to 98.507% when pulling d6d8ff73125dd20e2675f64c73f1d08f208c04fe on ktzouno:patch-1 into 9fe9658071c2b9064cf972dca014a48e5ecdccf6 on creof:master.

Coverage decreased (-0.004%) to 98.507% when pulling d6d8ff73125dd20e2675f64c73f1d08f208c04fe on ktzouno:patch-1 into 9fe9658071c2b9064cf972dca014a48e5ecdccf6 on creof:master.

Coverage decreased (-0.004%) to 98.507% when pulling d6d8ff73125dd20e2675f64c73f1d08f208c04fe on ktzouno:patch-1 into 9fe9658071c2b9064cf972dca014a48e5ecdccf6 on creof:master.

Coverage decreased (-0.004%) to 98.507% when pulling d6d8ff73125dd20e2675f64c73f1d08f208c04fe on ktzouno:patch-1 into 9fe9658071c2b9064cf972dca014a48e5ecdccf6 on creof:master.

Thanks !


Type definition claims missing property?

RxJS version:
master
Code to reproduce:
///<reference path="./typings/main/ambient/es6-shim/index.d.ts" />
///<reference path="./spec/es5.d.ts" />

import * as Rx from '.dist/cjs/Rx';

Rx.Observable.of(1).subscribe(_ => {
  console.log(_);
});

Actual behavior:
../RxJS/dist/cjs/Observable.d.ts(11,13): error TS2339: Property 'observable' does not exist on type 'SymbolConstructor'.
Additional information:
tsconfig.json
{
  "compilerOptions": {
    "removeComments": false,
    "preserveConstEnums": true,
    "sourceMap": true,
    "declaration": true,
    "target": "es5",
    "module": "commonjs"
  },

  "files":["index.ts"]
}

Maybe I'm missing something essential after having es5 declarations, @david-driscoll - would you mind to point out what I did missing?

That is really weird, I'm seeing it too.... it's not picking up https://github.com/ReactiveX/rxjs/blob/master/src/symbol/symbol.d.ts for some reason.... must be the compiled output... I'll dig in tonight.

interesting in dist/cjs/symbole there is `///
which is wrong...  I guess probably the only way to fix it up is part of the post build steps (where package.json is dropped in, etc).
Does that sound like a decent solution?

isn't it because of ambient referencing in original code (observable.ts)?

It gets stranger... it's gotta be a bug in the TypeScript code, but I have a one line fix that appears to the solve the problem.

Closed via https://github.com/ReactiveX/rxjs/pull/1569.

@david-driscoll , reopened this issue by update with latest beta https://github.com/ReactiveX/rxjs/pull/1569#issuecomment-205857392 . Seems ambient type import is not way to go.

re-closing this issue by https://github.com/ReactiveX/rxjs/pull/1582. To try out this with npm package, may need next version publish though.


Add --watch to test app

So we can easily try out watching a directory on macOs or Windows.

Now that https://github.com/adammurdoch/native-platform/pull/58 is merged we can remove the System.exit().

I removed the System.exit() call.


Build error, when downloading: https://naciscdn.org/naturalearth/110m/physical/ne_110m_coastline.zip

The ne_110_coastline.zip file is required for cartopy plotting. But the servers has been unstable and resulted build errors in CI/CD in a few occasions.
 /usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/site-packages/cartopy/io/__init__.py:241: DownloadWarning: Downloading: https://naciscdn.org/naturalearth/110m/physical/ne_110m_coastline.zip
  warnings.warn('Downloading: {}'.format(url), DownloadWarning)
Traceback (most recent call last):
  File "/usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/urllib/request.py", line 1346, in do_open
    h.request(req.get_method(), req.selector, req.data, headers,
  File "/usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/http/client.py", line 1253, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/http/client.py", line 1299, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/http/client.py", line 1248, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/http/client.py", line 1008, in _send_output
    self.send(msg)
  File "/usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/http/client.py", line 948, in send
    self.connect()
  File "/usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/http/client.py", line 1415, in connect
    super().connect()
  File "/usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/http/client.py", line 919, in connect
    self.sock = self._create_connection(
  File "/usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/socket.py", line 843, in create_connection
    raise err
  File "/usr/share/miniconda/envs/e3sm_diags_env_dev/lib/python3.9/socket.py", line 831, in create_connection
    sock.connect(sa)
TimeoutError: [Errno 110] Connection timed out

There is a fix merged in cartopy 6 days ago to address this issue by pointing to a new reliable server:
https://github.com/SciTools/cartopy/pull/1833

That has made it to conda-forge: https://github.com/conda-forge/cartopy-feedstock/pull/116

Thank you Xylar. CI is now green.

You can also include the (200+ MB) cartopy_offlinedata package so the data gets downloaded as part of the conda environment from the start: https://github.com/conda-forge/cartopy_offlinedata-feedstock.  This prevents issues when trying to download cartopy data on machines with a firewall or on compute nodes without access to the internet.

yes, Tom and I talked about this option. 200+ MB is no small. I guess let's see if the new server is stable enough and if we have the need to support machines without access to the online package.

At typical conda environment is about 1 GB so it's not in the noise but typically also not more than an additional 20%.

We have had a lot of trouble with MPAS-Analysis before we included this, but mostly because of the firewall at LANL.

I see. yeah, in this case, it makes sense to follow what MPAS-Analysis does for including these offline data then...


Disnake migration

This migrates the bot to disnake, as per recent discussions.
What this includes:

Changes all references from discord.py to disnake, including in tests
Bump version of bot-core so discord.py wasn't required as a sub dep
Use monkey patches from bot-core
Fix the single breaking change I could find

The internal bot.http.edit_message function now has a required files kwarg, which is only used in the help channels cog.
Set this to None as we never upload files using that message

This not included in this PR:

Updating the tags to reference disnake. These were left with the discord module, and can be updated later if needed
Migrating the bot to use all utils that have been moved to bot-core

This would inflate this PR even further, so didn't want to pull it into scope.

P.S this is quite a large PR, so reviewing by the commits may be helpful. I tried to keep the huge rename within the single commit https://github.com/python-discord/bot/pull/2103/commits/27b18baaf785091e4e2d5dcdca3ff18981dfcd44, and didn't include anything else in there.

Just as point of order / reminder, please do not force push in this PR as that will mess with the "Viewed" markers to the right.

Good work on this by the way, I ran both Pyright and Mypy in the command line with most type checking disabled (or ignored when reading the output) and I could not find any complaints about discord or missing imports (although Mypy failed to find the typings for disnake.ext.commands which may make this test incomplete).

Thanks for addressing my comments Chris. I have looked at the diff of each file and not yet found any odd changes - I could also not find any differences between discord.py and disnake when migrating other than the new text-in-voice feature resulting in some methods returning VoiceChannel which may not include all attributes/methods (send() and edit() would be present I believe). Any issues that may arise from this that come to mind?
Additionally, this pull request is pretty much incompatible with every other pull request currently open. Has there been any thought put into figuring out which ones should be merged before this is merged, and which ones should be merged afterwards (so that we can let those know)?

[ ... ] VoiceChannel** which may not include all attributes/methods (send() and edit() would be present I believe). Any issues that may arise from this that come to mind?
Not that I can think of, we also don't use text in voice channels in the server.

Additionally, this pull request is pretty much incompatible with every other pull request currently open. Has there been any thought put into figuring out which ones should be merged before this is merged, and which ones should be merged afterwards (so that we can let the author of those know to update to disnake)?

I think any PRs that are open when this is merged will just need to be rebased, I don't think there are any PRs open that should block this.
We of course can extend the offer to help anyone with an open PR to resolve conflicts.

Marked as do not merge to allow me to squash fixup commits on this PR before it hits main


EF Core samples: Add a way to update Enrollment Data when editing a Student Record

Help us make content visible

Tell us what search terms you used and how you searched docs.
Tell us what docs you found that didn't address your concern.

Describe the new topic
There is a walkthrough that details many-many relationship of Instructor-Course-Enrollment (and Instructor-Department) but for Student-Enrollment-Course this would be a simple add and represents a headscratcher to me when I've completed the Razor and MVC EFCore walkthroughs.
Kudos for adding SQLite3 support to the walkthroughs BTW, really adds simplicity when building on non-Windows platforms.

Explain why this topic is needed.
"Black Hole" that I solved by scaffolding around the Enrollment table

Suggest a location in the Table of Contents.
Somewhere around Part 6/7 in the Razor EFCore walkthrough

Write an abstract. In one short paragraph, describe what this topic will cover.
Unification of the sample app by closing the circle of important missing functionality

Create an outline for the new topic. We'll help review the outline and approve it before anyone writes a topic.

This tutorial is to get you started on EF with ASP.NET Core. We won't be adding any new scenarios.


Add an option to provide the Wasm URL

Hello, could you, please, add an option to allow a full path URL of the Wasm file to be provided at runtime? It is not always available before that. Something like:
PhysX({  wasmPath: 'https://.../physx.wasm.wasm' }).then('...');

If the options object is not provided, it would default to whatever it is doing at the moment.

Hmm might be not that easy. The javascript is entirely generated by the emscripten compiler, so that one would need to have an option to do that. I did a quick google search but found nothing to achieve that.

Back in the days, when prestomation first made his initial project public, I used it to test an integration with PlayCanvas:
https://playcanvas.com/project/737359/overview/playcanvas-with-nvidia-physx
However, I had to modify the original sources to point to the Wasm file path. Similar with your library, I had to modify the original js file:

This is because the PlayCanvas servers add a hash to the filenames of the project assets, so the default name will fail to fetch.

Ok, does the intercept approach work? I guess that would be the easiest solution.
I don't want to add manual changes to the 'official' javascript file I distribute, because that makes it very difficult to reproduce the build and to publish updates. It might be an option to write some kind of script, which changes the javascript automatically as part of the build process but that would also require some work.

I haven't tried the service worker yet. Will do a test tonight and let you know.

Hmm, so after fighting with it, I wasn't able to use a service worker. After some research, I think I found the default way  of doing it in Emscripten - Module.instantiateWasm:
https://emscripten.org/docs/api_reference/module.html#Module.instantiateWasm
You can consider adding support for it. I will close the issue, as I my toolchain replaces instantiateAsync() with my custom function instead in the final bundle.


[Feature Request] Add support for FabricTailor skins

Display FabricTailor heads on discord.
Hope it is possible, thanks!

Oh so you like piracy.... skinrestorer is made for offline mode servers. I see no reason to support that

FabricTailor's primary purpose is to allow changing of skins on regular server/in singleplayer, so I see no reason for it to be unsupported; however, SkinRestorer will probably never be supported officially, as it is specifically designed for offline mode servers.
Also, @emielderckx your phrasing sounds a little like you are attacking the issue author, there are better ways of explaining that piracy is unsupported :P

@emielderckx you are right. I should have thought about it before asking for support. My bad.
@Tom-The-Geek thank you for taking my request in consideration.

Hi there, author of FabricTailor here :).
You could request skin value from FT's interface TailoredPlayer
(see code). Then decode that string (it's encoded using Base64 format), what you're looking for is SKIN -> url value.

posting screenshot here as well

Leaving https://github.com/crafatar/crafatar/issues/54 for reference.

There seems to be a way to do this using https://mc-heads.net/
(comment in crafatar issue)


Database

Cleans up the interface to DB - now you just do an "adduser" as a dict with each key representing column name and value representing the corresponding data for that column.

This change is

Thanks!


fixed inifite loop in remove

Fixed a bug where remove will get stuck in an infinite loop, when removing a point with multiple non equal data entries.

Released to crates.io as https://crates.io/crates/kiddo/0.2.1.
Danke schön, Max! 👍🏼


The message property of SlashContext object is never populated[BUG]

Trying to implement slash commands with my existing bot. Basically just looking to pass the message object through to an existing function. I set up the slash commands just fine, but when the callback executes, the message property of the context object is None. Looking at the code it looks like it's only populated after the message is sent.
My bot has a custom interface for registering commands, I'm not using the discord commands ext. I realise this code won't mean much out of context, but I figured providing it might help if I'm doing something wrong. Basically, an interface represents a single command. interface.func is the callback function that takes a formatted message object and performs some task, like sending a response. The interface has subcommands which are their own nested interfaces, and the flags and arguments are processed separately.
All I need is the message object of the invoking message, so I can pass it to my framework and process it as normal. Does it have something to do with the deferred status of the message? Is there a way to bypass it and access the message object before it's finished being deferred? Or just disable it alltogether?
Thanks, sorry for the kind of chaotic bug report. Please ask any questions you need
# map message callback function to slash callback
def make_async(f):
    async def a(ctx, *args, **kwargs):
        # this runs when the slash command is executed
        ### ctx.message == None ###
        return f(self.format_message(ctx.message), *args, **kwargs)
    return a
# register the command
self.slash.add_slash_command(
    make_async(interface.func), # map the message callback to a slashcommand callback
    interface.name, interface.desc[:100], options=[create_option(
        "arguments", "The arguments and flags to supply to the command", option_type=1, required=False)]
)
# register the subcommands
for subcommand in interface.subcommands:
    self.slash.add_subcommand(
        make_async(subcommand.func),
        interface.name, name=subcommand.name, description=subcommand.desc[:100], options=[
            create_option(
                "arguments", "The arguments and flags to supply to the command",
                option_type=3, required=False
            )
        ]
    )```

Hiya!
So Looking at the code it looks like it's only populated after the message is sent. is intentional, but that's not because of this library, this is more of a Discord bit; when you do a slash command, there's no "message" to reference, compared to d.py's message saying, for example, -say 1234 (If the slash equivalent, let's say, /say 1234, command and args is sent, there's no message symbolising the command itself, but as you said earlier, only the message that the code sends after with ctx.send).
In a lower level, I suppose, the message of the invocation command exists in d.py because it relies on message-based events (like Message intents), and slash commands are its own event.
Basically, the message object of the invoking message doesn't really exist (can't get /say 1234 but you can get the message from await ctx.send("1234")

Yea, that's what I figured. I guess I'm just going to have to reconstruct a message-looking object to pass back into my framework and just cross my fingers. Thanks for the fast response :)


add clarification on use of pickZones function for ZRS resources

@bmoore-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @bmoore-msft
Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you make your changes in the private repo, review the articles on staging, and fix any validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
We'll close this PR. Thank you.

@ShannonLeavitt - I don't have access to the pr repo so I've made changes here for the last few years... if that's required I'll let @mumian take it from here...

@bmoore-msft @mumian We can review this update here. The content team should review and sign off on the update. Thanks.

I have created a separate PR on this - https://github.com/MicrosoftDocs/azure-docs-pr/pull/168481.  You can close this one.

I have created a separate PR on this - MicrosoftDocs/azure-docs-pr#168481. You can close this one.

Awesome. Thanks @mumian.

Thank you both...


Looks like the sentence is incomplete: "Ensure that the storage account"

Hello,
please note that "Ensure that the storage account" in the Tip box of this page is incomplete.
Kind regards,
Anne

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

ID: a48a8e54-a148-77fa-a315-3f0a1448dbb9
Version Independent ID: 6aad06fb-153e-4e32-9767-02c280e5e049
Content: Use Azure Machine Learning labeling in Language Studio - Azure Cognitive Services
Content Source: articles/cognitive-services/language-service/custom/azure-machine-learning-labeling.md
Service: cognitive-services
Sub-service: language-service
GitHub Login: @aahill
Microsoft Alias: aahi

@aahill
Could you please review add comments on this, update as appropriate.

@QueryCoordinator
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


Update limitations to note Hybrid Azure AD Join Scenarios

Updating documentation to match limitations noted on: https://docs.microsoft.com/en-us/azure/active-directory/device-management-hybrid-azuread-joined-devices-setup

@egbill3eagle : Thanks for your contribution! The author, @swkrish, has been notified to review your proposed change.

These limitations are not valid and have been corrected in the original article. #close

#please-close


Autocomplete non-functional.

Issue
Autocomplete does not work.
Steps to reproduce
OS: Windows 11 + WSL.
Motoko plugin version: v0.16.4
DFX version: 0.20.1

Connect VSCode to WSL.
Open terminal.
Create a new project with dfx new.
Choose motoko for backend, anything for frontend, and no extra features.
Open folder in VSCode.
Paste the following code in main.mo:

import List "mo:base/List";

actor Bug {
  var testList = List.nil<Text>();

  testList.
};

Try to get autocomplete to work with testList.

Hi, thanks for opening the issue! The extension currently only supports local variable and import autocompletion (e.g. completing List or testList). We will increase the priority of this feature based on your feedback.

Ah okay, thank you. Yes this kind of support would be super useful; maybe my memory is just bad, but i wouldn't have to flip to the docs as often.


Fix operators links in Introduction docs

Description:
All the operator links in the overview are broken.
For example:

❌ filter (broken)
✔ filter (fixed)

Thank you, @shaunlebron


chore: terminate on wrong genesis

Purpose

How does the solution address the problem

Changes made

prnet cleaned-up successfully


Implement Resource Consumer

This is @socaa's intern project.
Background
We are working on autoscaling for Kubernetes in 3 dimensions:

cluster size autoscaling
horizontal autoscaling of pod - changing the size of replication controller #2863
vertical autoscaling of pod - changing its resource limits #10782

We need a tool which will help us to test those kinds of autoscaling.
Resource Consumer
Resource Consumer will consist of two parts: container consuming resources and library managing this load.
Container
The container which consumes specified amount of resources (cpu/mem). Docker container, uploaded to Google Container Registry, open sourced, possibly written in go (it’s not a requirement, but it’s strongly encouraged). The resources to consume should be set and mutated by sending appropriate http request to the container.
The interface should allow to consume the given amount of cpu/mem for the given period of time. It should consist of the following methods:
ConsumeCPU(milicores, duration_sec)
ConsumeMem(megabytes, duration_sec)
GetCurrentStatus()

Each http request will be handled by spawning a new process which will consume given amount of resource and exit after given timeout.The request cannot be cancelled.
Consuming CPU
Consuming the whole core can be done easily. The problem is how to consume some part of it. One of possible ways to achieve it is to split time into small quants, then perform heavy compute operation during some percentage of quants and sleep during the other ones. It is a part of the task to figure out the best way to consume CPU.
Consuming memory
There are few possible ways how to handle memory consumption. We want to make sure the memory can be correctly freed. The only problem is that the container use some memory itself, so that this amount should be considered in calculations as well. It is a part of the task to figure out the best way to consume memory.
Library
Client side library written in go will be linked with e2e tests and should consist of two kinds of methods: static consumption of resources and dynamic one.
Static consumption
This part of library should allow to consume constant amount of resource over time:
ConsumeCPU(node, percentage)
ConsumeMem(node, percentage)
CreateConsumingPod(node) : PodID
ConsumeCPU(pod, percentageOfRequest)
ConsumeMem(pod, percentageOfRequest)

Dynamic consumption
This part of library should allow to create service on top of replication controller, and then request to consume the given amount of resources by all replicas together based on service level load balancing. It can be implemented with sending a lot of consumption request for short period of time and small amount of resources. The interface should consist of:
CreateConsumingService() : RcID
ConsumeCPU(rc, milicores)
ConsumeMem(rc, megabytes)

In both cases there might be an option to add possibility to modify/stop consumption if needed.
Use cases
Cluster size autoscaling

consume more resources on each node that is specified for autoscaler
observe that cluster size increased

Horizontal autoscaling of pod

create consuming RC and start consuming appropriate amount of resources
observe that RC has been resized
observe that usage on each replica decreased

Vertical autoscaling of pod

create consuming pod and start consuming appropriate amount of resources
observed that limits has been increased

Milestones

[ ] Resource Consumer application

[ ] http API skeleton server

[ ] define API
[ ] implement mock methods which just return theirs names

[ ] implement cpu consumption
[ ] implement memory consumption
[ ] figure out how to verify that the resource consumer works correctly and do it
[ ] add possibility to specify initial consumption request
[ ] [optional] add a status page
[ ] create Dockerfile, build and deploy Docker image

[ ] Client library for e2e test

[ ] static consumption
[ ] dynamic consumption using Kubernetes service based load balancing

[ ] Implement few e2e test for autoscaling using Resource Consumer to prove it is useful

[ ] TBC depending what part of autoscaling will be implemented
[ ] TBC

[ ] Ensure that tests are not flaky.
[ ] Document how to use Resource Consumer
[ ] Final presentation

cc @jszczepkowski @fgrzadkowski

/cc me

cc @AnanyaKumar

/sub

/sub

/sub

Network ingress and egress being added.
On Aug 5, 2015 3:44 PM, "ramr"<EMAIL_ADDRESS>wrote:

/sub
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11570#issuecomment-128172777
.

@erictune could you explain what do you mean?

@piosz I think he's referring to PR 12187

The alpha version is released as gcr.io/google_containers/resource_consumer:alpha. It supports consuming cpu only.

The alpha version is released as gcr.io/google_containers/resource_consumer:beta. It supports consuming both cpu and memory.
cc @derekwaynecarr

Good job @socaa!


Android example fails to run: UnsatisfiedLinkError

Bevy version
b6a647cc013861b182be5f4c85b60b2709388210
Operating system & version
Building on Linux x86_64 for Android.
What you did
Followed the https://github.com/bevyengine/bevy/tree/latest/examples#android,
but with cargo apk build --example android, with manual installation of tha apk.
The 215-megabyte "android" application appeared on the device. But upon starting it the window immediately closes.
What you expected to happen
Some Android view with some UI appears
What actually happened
03-04 20:28:08.412  9304  9304 D AndroidRuntime: Shutting down VM
03-04 20:28:08.413  9304  9304 E AndroidRuntime: FATAL EXCEPTION: main
03-04 20:28:08.413  9304  9304 E AndroidRuntime: Process: rust.example.android, PID: 9304
03-04 20:28:08.413  9304  9304 E AndroidRuntime: java.lang.UnsatisfiedLinkError: Unable to load native library "/data/app/~~6E5dAovNIGKNfvRukbcJ_g==/rust.example.android-DAiOvJ6LjHGyMIvo9MQiIw==/lib/arm64/libandroid.so": dlopen failed: cannot locate symbol "__cxa_pure_virtual" referenced by "/data/app/~~6E5dAovNIGKNfvRukbcJ_g==/rust.example.android-DAiOvJ6LjHGyMIvo9MQiIw==/lib/arm64/libandroid.so"...
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.NativeActivity.onCreate(NativeActivity.java:178)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.Activity.performCreate(Activity.java:8000)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.Activity.performCreate(Activity.java:7984)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1309)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:3404)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3595)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.servertransaction.LaunchActivityItem.execute(LaunchActivityItem.java:85)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:95)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2066)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.os.Handler.dispatchMessage(Handler.java:106)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.os.Looper.loop(Looper.java:223)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at android.app.ActivityThread.main(ActivityThread.java:7660)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at java.lang.reflect.Method.invoke(Native Method)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
03-04 20:28:08.413  9304  9304 E AndroidRuntime:        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)

Additional information
Running on Pixel 3 XL with Android 11.
rustc 1.60.0-nightly (0c292c966 2022-02-08)

Hi!
I also ran into this error. Unfortunately, I wasn't able to fix it.
Here is what I found out so far:
The resources (including the app icon) are not correctly included in the apk any more. I think the configuration options for cargo-apk have changed.
By modifying the cargo.toml (ll.520ff.), I was able to fix some issues:

apk_label -> apk_name
res -> resources
maybe unnecessary: crate_type  (l.520): i changed from ["cdylib"] to ["lib", "cdylib"] (don't know what this means, but this is how it looks in the example from rust-windowing/android-ndk-rs

I also added a new section [package.metadata.android.application]:
[package.metadata.android.application]
icon = "@mipmap/ic_launcher"
label = "Bevy Example"

I moved the icon option from the previous section and added the label option.
Now, the app appears correctly in the application list. Assets are included again. I checked via adb, the libandroid.so file is also installed with the apk.
Btw. I think #4295 might be related to this issue. At least the second part of the error message is the same.
Maybe there are some other breaking changes from android-ndk-rs / cargo-apk? Or is libandroid.so compiled from Bevy code?
I unfortunately have no more ideas how to fix this issue, thus no PR. Maybe someone else has an idea?

some people have been able to run on android with https://github.com/bevyengine/bevy/pull/4139

Thanks! Will have a look into it.
But I think the issues from #4139 are different from this one? I don't have any RustStdoutStderr messages in the adb log, so no directly rust / bevy code related error messages.
Maybe because the .so is not loaded correctly?

I was actually just reinstalling my linux machine to test if this(#4295) was happening elsewhere too. I ran into the exact link error Vi ran into.
@konradmoesch the Rust code doesn't seem to be loading at all and it's purely a Java error at the moment. It's not saying the file doesn't exist, but it's saying the symbol doesn't exist.
Between this and #4295 though, it's complaining about different symbols depending on which OS is compiling.

Hm okay, but the libandroid.so is the compiled rust code, right?
So the missing symbol is caused by some error in the bevy code or the impl of the example? Or is this a compiler related error or an issue in the NDK?
I am new to Bevy and also quite new to rust (started maybe one year ago), sorry. But I'd love to get a bit more into it.

I'm in the same boat really, much greener though(a month or so).
From what I can gather, konradmoesch's assumption that it's an upstream issue from cargo-apk makes sense. It's building the .so file, and the device reports only missing symbols. It may also be the toolchain issue as well.
I'm currently using nightly(mainly because the rust-analyzer) so maybe the toolchain messing the builds up?
I'll see if I can get the stable Rust to work. I'm going to also try quadmacro and see if that works and maybe see what is done differently if it does.

I manually linked libc++_shared.so as described in this answer.
Now, the missing symbol error disappeared and I see the rust logging (I get
rust.example.android I/RustStdoutStderr: thread '<unnamed>' panicked at 'called 'Option::unwrap()' on a 'None' value', [...]\ndk-glue-0.6.1\src\lib.rs:60:39)
Will see if I can find a fix for this issue as well, at least some rust code is running now.

Jinx. I just did the link this morning too. I found that screenshot posted on several Rust projects.
I'm looking into 4139 now. Now that we are getting something Rust launching, the rest should be easy.

This issue should be closed with https://github.com/katyo/oboe-rs/issues/28#issuecomment-1001103335 as the solution. Now I'm running into the unwrap error now, though on a different line. Time to chase this down.

But I don't understand why its not required to link libc++ in the branch from MannevilleF.
I can't find this. All the Bevy forks I have found all fail in the same way without the build.rs fix.

I mean this branch by ManevilleF (pull req #4139)
This branch does not link libc++ manually iirc


TD3 Policy and Target Policy Naming Conflict

Description
When I try to instantiate a TD3 model, I get an error in the init function on line 136:
ValueError: Variable input/model/pi_fc0/w already exists, disallowed. Did you mean to set reuse=True or reuse=tf.AUTO_REUSE in VarScope?

It's possible I misunderstand the variable naming going on, but won't the following lines from td3.py init() always cause a naming conflict because of the reuse=False in the scoping?
with tf.variable_scope("input", reuse=False):
    self.policy_tf = self.policy(self.sess, self.observation_space, self.action_space, **self.policy_kwargs)`
    self.target_policy_tf = self.policy(self.sess, self.observation_space, self.action_space, **self.policy_kwargs)

Code Example
You can replicate the issue with the following code:
import gym

from stable_baselines.common.policies import MlpPolicy
from stable_baselines.common.policies import FeedForwardPolicy
from stable_baselines import TD3

class MyMlpPolicy(FeedForwardPolicy):
    def __init__(self, sess, ob_space, ac_space, n_env=1, n_steps=1, n_batch=None, reuse=False, **_kwargs):
        super(MyMlpPolicy, self).__init__(sess, ob_space, ac_space, n_env, n_steps, n_batch, reuse, feature_extraction="mlp", **_kwargs)

env = gym.make('CartPole-v0')
model = TD3(MyMlpPolicy, env)

System Info

Stable-baselines version: 2.7.0, installed via pip on 09/10/19
GPU: N/A
Python Version: 3.5
Tensorflow version: 1.9.0-rc0

Hello,
as mentioned in the documentation, you should be using td3.policies, and a continuous action environment like Pendulum-v0, not cartpole, because TD3 only support continuous actions.
the following code works:
import gym

from stable_baselines.td3.policies import FeedForwardPolicy
from stable_baselines import TD3

class MyMlpPolicy(FeedForwardPolicy):
    def __init__(self, sess, ob_space, ac_space, n_env=1, n_steps=1, n_batch=None, reuse=False, **_kwargs):
        super(MyMlpPolicy, self).__init__(sess, ob_space, ac_space, n_env, n_steps, n_batch, reuse, feature_extraction="mlp", **_kwargs)

env = gym.make('Pendulum-v0')
model = TD3(MyMlpPolicy, env)

Works now, thanks for the help.

Agents could check if action/observation spaces are one of the supported type and throw a bit more informative exception. A quick PR for later time :)

there is already a check for that ;)

Ah my bad, I did not notice the issue was with using wrong policies. My bad! Perhaps a check for that, but it is already well-documented with proper highlights ^^


Could not find any CSX files

Environment data
dotnet --info output:
.NET Command Line Tools (2.0.0-preview1-005977)
Product Information:
Version:            2.0.0-preview1-005977
Commit SHA-1 hash:  414cab8a0b
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.14393
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.0.0-preview1-005977\
Microsoft .NET Core Shared Framework Host
Version  : 2.0.0-preview1-002111-00
Build    : 1ff021936263d492539399688f46fd3827169983
C# Extension version:
1.10
Steps to reproduce
open VScode
Errors exist but not highlighted
no suggestion when typing ctrl + space
Expected  behavior
working with errors highliting
Actual behavior
when seeing omnishap output , there is  Could not find any CSX files

I hope that somebody from the omnisharp team answers this, as I'm stuck with for more than 1 week

are you trying to use VS Code to edit CSX (C# script) files?
If so, this message indicates no files have been find (recursively) in the folder currently being open - make sure that CSX files are really located there (also, permissions are important).
If you are not explicitly trying to work with C# scripts, then this is not an error of any sort - this message is purely informational and simply means that Omnisharp will not provide any language services for C# scripts, but only for MSBuild / project.json projects.

no. I m only working with c# classes. Asp.net Core and angular. If it just informational, so why Errors are not being highlighted ?

Note: this is not a bug. This is just an informative message in the OmniSharp Log that it didn't find CSX files. Since you aren't using CSX, that's 100% expected.
Please provide the requested information in the other bug you filed. We can't answer your question without digging in further.


update MosaicML inputs and outputs

As of today (July 7, 2023), the MosaicML API uses "inputs" for the prompt

but we have
https://github.com/hwchase17/langchain/blob/0ed2da70200ada39f5319f604b4c5248bc98e81b/langchain/llms/mosaicml.py#L125
Furthermore, the output should be inside the "outputs" key

but we have
https://github.com/hwchase17/langchain/blob/0ed2da70200ada39f5319f604b4c5248bc98e81b/langchain/llms/mosaicml.py#L163-L170
This PR adds support for this new format.
<rant>
It is beyond me, why they apparently have versioned their API (endpoints look like https://models.hosted-on.mosaicml.hosting/mpt-7b-instruct/v1/predict; note the v1 in there), but just keep changing the API without upping the version number. There is a good reason for
https://github.com/hwchase17/langchain/blob/0ed2da70200ada39f5319f604b4c5248bc98e81b/langchain/llms/mosaicml.py#L160-L161
</rant>
CC @hwchase17, @baskaryan

makes sense to me, cc would be great if you could double check @dakinggg

thanks @pmeier!


Does not run on Windows

uvloop does not run on Windows. I paid for the course but can't really use it. :-(

Hi. uvloop is a performance optimization I believe. I don't think it's required for the app to run. Just remove it from the requirements.txt and you should be fine.

BTW, I do agree it's annoying that it isn't supported on Windows, but that said, it's also not a big deal. If you really want to run it, you could use LSW (Linux Subsystem for Windows).

I was able to run it by removing uvloop. Thank you. I was quite disappointed (and stressed) because I told my supervisor that I could get fastapi talking to all the important things this week.
Michael
From: Michael Kennedy @.>
Sent: Monday, October 16, 2023 11:54 PM
To: talkpython/fastapi-and-the-microsoft-identity-platform-course @.>
Cc: Michael Derobio @.>; Author @.>
Subject: Re: [talkpython/fastapi-and-the-microsoft-identity-platform-course] Does not run on Windows (Issue #1)
This email was sent from an external source. Don't click links or open attachments from an unknown or unexpected sender. Report suspicious e-mails using the Phish-Alert button in Outlook, if available, or forward to<EMAIL_ADDRESS>BTW, I do agree it's annoying that it isn't supported on Windows, but that said, it's also not a big deal. If you really want to run it, you could use LSW (Linux Subsystem for Windows).
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/talkpython/fastapi-and-the-microsoft-identity-platform-course/issues/1*issuecomment-1765664603__;Iw!!G-vmT1M064aM!THwgy1kQbZhraxDNPoeL8ZRD1UH1bzulKF0vOTKyMY_7cu3iwEc1oGACK8aNDxP7AlkwPJtmmGpZ_idvRGoRFfjLOCHodbw$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/A5G5STQAAXH52RYGKPB7D7TX7YFPNAVCNFSM6AAAAAA6DAVDFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRVGY3DINRQGM__;!!G-vmT1M064aM!THwgy1kQbZhraxDNPoeL8ZRD1UH1bzulKF0vOTKyMY_7cu3iwEc1oGACK8aNDxP7AlkwPJtmmGpZ_idvRGoRFfjLA-rPoPc$.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Gradle dependency check

Is your feature request related to a problem? Please describe.
Recently it was noticed that dependencies like Guava have been outdated for a expanded amount of time. The used version was 21.0, while the latest was 30.1.1.
Describe the solution you'd like
Add a GitHub action which automatically checks Gradle dependencies for potential vulnerabilities, to prevent this issue from happening again in the future.

e.g. https://github.com/dependency-check/dependency-check-gradle


updated geogebra to 6.0.785

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110959&drop=dogfoodAlpha

@microsoft-github-policy-service agree


Memory use in Actionform

I suddenly noticed my Django processes memory use stretching out like a chewing gum on viagra,  using 500 MB each instead of 50 MB before, bringing the whole server down, and some research led to this autocomplete light.
I was using Autocomplete Light in an actionform:
class ArticleActionForm(ActionForm):
import autocomplete_light
article_id = autocomplete_light.ChoiceField('ArticleAutocomplete')
I'm not 100% sure that this use is officially supported, but it caused to preload all articles (several million) into memory on process startup (without even viewing the form).
I'd appreciate somemone could comment this is a bug or whether I'm using the thing wrong.
Thanks
Joris

Yeah, you're using ChoiceField instead of ModelChoiceField, that's what's wrong.
ChoiceField renders all options :dancer:

Perhaps you'd consider this a bug, but that's fixed in v3 already, you might want to backport it: https://github.com/yourlabs/django-autocomplete-light/blob/v3/src/dal/widgets.py#L43-L51

Great! Thanks for your feedback!


feat: set enable_road_slope_simulation to true

Description
Since enable_slope_compensation in the Autoware's controller is true by default, the simulator's enable_road_slope_simulation should be true by defaul.t
https://github.com/autowarefoundation/autoware_launch/blob/7df85a85c7eba27cc1b96ceb58304d8515c7b9ee/autoware_launch/config/control/trajectory_follower/longitudinal/pid.param.yaml#L8

Tests performed

psim
Effects on system behavior

The simple planning simulator will simulate a force by a road slope.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@maxime-clem @danielsanchezaran Could you review the PR.


Couverture de code

if condition_hold:
	print("the if-block is covered")
else:
	print("the else-block is covered")

https://fr.wikipedia.org/wiki/Couverture_de_code

JaCoCo, couverture de code pour Java
https://github.com/jacoco/jacoco

Istanbul, couverture de code pour Javascript
https://istanbul.js.org/

Coverage.py
https://coverage.readthedocs.io/en/7.6.0/

Couverture pour Rust
https://blog.rust-lang.org/2022/04/07/Rust-1.60.0.html#source-based-code-coverage

grcov collects and aggregates code coverage information for multiple source files (in java and js)
https://github.com/mozilla/grcov

Reaching 100% Code Coverage in Rust

What is code coverage?
https://www.atlassian.com/continuous-delivery/software-testing/code-coverage

Code coverage at Google

Code coverage and test suite effectiveness: Empirical study with real bugs in large systems

How effective are code coverage criteria?

Four common types of code coverage
https://web.dev/articles/ta-code-coverage

A Practical Tutorial on Modified Condition/Decision Coverage
http://shemesh.larc.nasa.gov/fm/papers/Hayhurst-2001-tm210876-MCDC.pdf

A large-scale study of test coverage evolution

Production monitoring to improve test suites

Code Coverage Criteria for Asynchronous Programs
https://www.franktip.org/pubs/fse2023.pdf

Cross platform code coverage for .NET
https://github.com/coverlet-coverage/coverlet

LCOV Code Coverage


Rework VOSI parsing using astropy xml handling

Coverage increased (+8.3%) to 74.983% when pulling a89c29fc0516457fa2bcaca28cde7d91f148d791 on vosi-rework into b4e651f9acb349303015932615991ba8d682ecfc on master.


Handle invalid custom terminal profile

Issue Type: Bug

Add a custom profile that uses an older schema to settings:

"terminal.integrated.profiles.osx": {
  "bash (login)": {
    "pathOrPaths": "bash",
    "args": ["-l"]
  }
}

Run the Select Default Profile command

Nothing happens, but there's an error in the console:
ERR The "path" argument must be of type string. Received type undefined: Error: The "path" argument must be of type string. Received type undefined

Would be nice to show an error notification
VS Code version: Code - Insiders 1.55.0-insider (8437d38d364ba93b53c1e659c20b1c86f710ec28, 2021-03-24T08:10:36.007Z)
OS version: Darwin x64 19.6.0

Since the settings schema tells the user what's wrong with that profile, I'm going to just ignore the profiles from config that don't contain path or source values.

fine with you @Tyriar ?

Looks good


apiMasters routes complete

 API Master File Front End

Edits to be made to apiMasters.jsx:

Combine fan dashboard and profile GETs into one GET
https://github.com/Blue-Tang-Clan/buskin/blob/d973447b46edc94af261f204090162c06e62443f/src/apiMasters.js#L7-L15
Combine artist dashboard and profile into one get
https://github.com/Blue-Tang-Clan/buskin/blob/d973447b46edc94af261f204090162c06e62443f/src/apiMasters.js#L17-L25
Remove timestamp from createEvent body and add date, start_time and end_time fields
https://github.com/Blue-Tang-Clan/buskin/blob/d973447b46edc94af261f204090162c06e62443f/src/apiMasters.js#L52-L61
Ensure all POST requests place ids in the body and not in the URL (createEvent, saveEvent, followArtist)
Change both PATCH requests to PUT requests and send with body that accounts for every profile column
https://github.com/Blue-Tang-Clan/buskin/blob/d973447b46edc94af261f204090162c06e62443f/src/apiMasters.js#L86-L92


I want to contribute some features, why I got xaml intelligence error but build success?

These problems are due to Visual Studio ignore it

So, how to solve this?

So, how to solve this?

I do not know
Maybe older versions of Visual Studio
also These errors may be due to the SharedProject
I recently moved the demo to the Prism, which is out of the Shared mode. You can test it.
https://github.com/ghost1372/HandyControl/tree/PrismDemo
https://github.com/ghost1372/HandyControl/tree/PrismDemo/src/Net_GE45/HandyControlDemo.Prism

I'm using Microsoft Visual Studio Community 2019 16.8.2.

I'm using Microsoft Visual Studio Community 2019 16.8.2.

I mean, previous versions of Visual Studio may not have these issues

:(

:(

Have you tested the new demo?


EmailPasswordAuth class Reset password function : wrong parameters order

Hello everybody!
I was using the resetPassword function from EmailPasswordAuth class and the documentation suggest the follwing function signature resetPassword(token: String, tokenId: String, newPassword: String).
When using it that way I've always got the following response : [Service][BadRequest(48)] invalid token data
I tried changing the parameters order when calling the function (as this issue was already raised for other SDKs) like this : app.emailPasswordAuth.resetPassword(newPassword, token, tokenId) instead of app.emailPasswordAuth.resetPassword(token, tokenId, newPassword) and that time it worked.
Hence, there might be a problem in the function signature or elsewhere.

@eloigrndl Thanks for the report


Can you guys try using rwkv instead of Transformers?

Can you guys try using rwkv instead of Transformers?

Hi, maybe it is a future work. Could you provide a PR for this?

Hi, maybe it is a future work. Could you provide a PR for this?

This is not my project but uses rwkv-v6 instead of Transformers.
https://github.com/yynil/Di-R

i think it's better to try mamba instead of rwkv, which is now considered as next gen architecture.

i think it's better to try mamba instead of rwkv, which is now considered as next gen architecture.

But the performance is weaker! rwkv v6 is obviously more powerful than mamba!