[Enhancement] Display first frame(thumbnail) without player.play

Is it possible to display a first frame with in Video Widget, without calling player.play()

It's not.

I did exactly that:-

Showing the first frame during entry slide-in up until await player.play() completes
Showing the last frame of the video after video completion (streams.isCompleted) up until slide-out completes

However, as shown in the video, I am having 2 problems

The player takes about 1 to 2 seconds after await player.play() to actually start playing the video. You can see the black screen in between the first frame and play, in the screencast below.
After completing the video, the player shows the first frame, instead of pausing on the last frame. You can see the brief flicker to the first frame on completion of the video below.

Here's the screencast. The slide animation is a 3 second one, during which the outgoing one displays the last frame of the video (separately held screenshot in jpg format, not displayed via mediaKit) and the incoming one displays the first frame until the video starts.
Screencast from 26-03-23 09:37:14 PM IST.webm

I optimized it for audio applications initially. Now it needs to be updated to meet video rendering requirements i.e. no missed frames, initial frame draw etc. etc. Specifically here (how I've abstracted internal libmpv playlist management to be more friendly within Player):
Project is still at early stage (only first version out). For normal applications, which just want to play video in a performant & stable way, this is still good. Further tweaks & optimizations here & there are left.
https://github.com/alexmercerind/media_kit/blob/03d8a702d5c0600e94548fd68a6b71c1485a0e49/media_kit/lib/src/libmpv/player.dart#L138-L165
https://github.com/alexmercerind/media_kit/blob/03d8a702d5c0600e94548fd68a6b71c1485a0e49/media_kit/lib/src/libmpv/player.dart#L88-L135

I was tweaking around with the player code. As you have mentioned, it gets slightly better when the width & height are specified during initialization - the black flicker that occurs before playing (as seen in my screencast) got reduced to a moment's flicker.
videoController = await VideoController.create(player.handle,  width: videoResolutionX, height: videoResolutionY);

Another hack - initialize the player with play: true. Then, pause & seek to zero. This gets the video fully ready to play. On next call to play, the video gets unpaused which takes a lot less time.
await player.open(mk.Playlist(
          [
            mk.Media('asset://assets/${widget.itemVideo.filename}')
          ]
      ), play: true, evictCache: false);
await player.pause();
await player.seek(Duration.zero);

Now there's zero flicker on start. Yet to test it thoroughly though. That's about the first problem. Still facing the second problem - on completion of the playlist, the player rests on the first frame of the video instead of blank / last frame. Let me tweak around and post an update on that problem.
It's pretty impressive to see what media_kit can do while barely at 1.0  :+1:

@srix55
media_kit is not 1.0.0, it's 0.0.1! Expect breaking changes & massive API refinements (like next update) before 1.0.0.

Another hack - initialize the player with play: true. Then, pause & seek to zero. This gets the video fully ready to play. On next call to play, the video gets un-paused which takes a lot less time.

Yes!
I was referring to these minor workarounds & refinements (or "hacks" as you said). The internal Dart sided logic needs to be changed to fit every possible case. Currently AFAIK video is redundantly reloaded due to jump. I'll optimize the things to their best.

Yes... 0.0.1 :smile: & can play 5 hd playlists in parallel
Let me fool around with the code a bit. Currently trying to plug the memory leak in linux.

Should be fixed in #90.
https://user-images.githubusercontent.com/28951144/228332680-432ebe01-8f53-4f17-a214-d1d78379505d.mp4


search box in FormBuilderSearchableDropdown not exist in version 8.0.0?

aftar I upgrade to 8.0.0 I have not seen search box in FormBuilderSearchableDropdown.how do I show it again?

Kindly note that the API for dropdown_search has changed in the recent versions that are compatible with Flutter 3.
Please refer to the package on what has changed.

I got it

@huongtran84 Can you tell what needs to be done to show the search box? Thank you!

FormBuilderSearchableDropdown(
items: _items,
popupProps: PopupProps.menu(showSearchBox: true)
)


Generate shares for an ethereum private key

How can this library be used to generate shares for an ethereum private key?
I tried chaning line 34 of tecdsa.py to:
pub_key, priv_key = paillier.generate_paillier_keypair(n_length=512)
then the ecnil object's he_private_key.q and he_private_key.p values are the correct size to create an ethereum wallet. I changed this value because with the default key size of 3072, the private key ints are too big to create an ethereum wallet. (As a side question, which attribute of he_private_key is actually storing the private key? p or q? ) When I do n_length=512, however, I get an OverflowError: Overflow detected in decrypted number when calling ts_online_protocol
Ultimately I'd like to use tinysig to a) generate a valid ethereum private key and b) split that key into shares which can be used in combination to sign transactions.
So my questions are:

Which attribute of the ecnil object stores the private key? p or q?
How can tinysig be used to generate shares of an ethereum private key?
How can the shares from question (2) be combined to sign a transaction?

I posted this question on the ethereum stack exchange as well

Hi @rmurray2,
tl:dr: this is not the right library for that.

The shares that this library provides are tied up to the underlying MPC protocol. So, they would only work/be secure in an environment that has the required preprocessing material. Which type of shares do you want? Usually, those are additive shares or shamir shares of the key.

The pub_key and priv_key you mention from the paillier function are NOT the keys for ecdsa (aka Ethereum signature scheme). They are part of the protocol that is described here. So, they are supposed to be big and the size of the keys matter: with small keys the scheme will not work due to overflow inside the homomorphic ecnryption scheme.

Note: tinysig implementation is an emulation of the protocol proposed here. As such, it is supposed to be used as a whole. The shares are given by the protocol distributed_key_generation_protocol and saved within the emulated network. You can print the network at any point in debug mode and I believe you can read the shares from there. Anyway, these shares are very specific to this protocol (you can check their shape in Fig 2 (pag 8)). You can also explore all this through the notebook we put together.
FYI: tinysig implementation is a prototype implementation of the proposed protocol and is only meant for educational purposes. The library is not audited and should be used with cautious.


[Feature Request]

Add link sharing support
So other video players can be used for anime
Like md player

done

go into options -> player -> always use external player


GridMap only supports 16bit IDs while MeshLibrary supports 32bit IDs

Godot version: v3.2.3.stable.mono.official

OS/device including version: Microsoft Windows 19041.630

Issue description: I am working on a very simple game and decided I wanted to use gridmap to paint the terrain. I wrote a tiny bit of C# script to take a list of textures and build simple 3d mesh tiles using them to create my mesh library. For the MeshLibrary IDs I decided to just hash the texture names so the IDs would stay consistent every time I regenerated the MeshLibrary. This is the important bit because I generated those ids like this:
Math.Abs(itemName.GetHashCode())
This generated entirely valid MeshLibrary entries:

However when I went to actually draw my grid tiles nothing happened (looking like this):

The problem is this code from gridmap.cpp:
void GridMap::set_cell_item(const Vector3i &p_position, int p_item, int p_rot) {
   // ... Lots of unrelated stuff ...
	Cell c;
	c.item = p_item;
	c.rot = p_rot;

	cell_map[key] = c;
}

Cell is defined like this:
	union Cell {
		struct {
			unsigned int item : 16;
			unsigned int rot : 5;
			unsigned int layer : 8;
		};
		uint32_t cell;
                // Constructor omitted...
	};

And there's the problem. Item is bitmapped to only the first 16 bits. This means that the latter 16 bits are just chopped right off without any error or warning! The shorter and completely wrong ID is then stored in the gridmap.
In summary:

MeshLibrary supports 32 bit positive IDs
GridMap only supports 16 bit IDs
GridMap takes 32 bit IDs and then cuts off the top 16 bits without any warning or error

This issue was very frustrating since GridMap doesn't even bother to put out a warning on invalid IDs. I suggest adding a warning (at least to the editor). As for actually fixing the issue I'm not sure what would be best, maybe switch MeshLibrary to use a short so this specific mistake becomes impossible? Also set_cell_item should absolutely take a short.
Minimal reproduction project: GridMapKeyLimitation.zip

Any plans on supporting 32bit IDs for GridMap ?


Create/update secrets from file

Hiya,
I wanted to add a couple functions to dcos_secret.py that allow for the creation of secrets from files (following this documentation). I made the changes locally and wanted to push them to a branch so I could make a PR, but I don't have permissions to do so. Can I get permissions to make a branch or something, so you can review my changes and possibly (hopefully) incorporate them into master?
Thanks!
Tobi

Update: forked it and created a PR here https://github.com/dcos-labs/ansible-dcos-module/pull/7

PR #7 was merged. Thanks again!


Crashes in KernelBase.dll

Hi,
I was not able to clone the submodules, so I got the HSR source code by Download ZIP, and also those listed directly in .gitmodules (not including every recursive gitmodules) by Download ZIP.
I'm able to build HSRSample_DX12.exe, however, it crashes in Hybrid-Stochastic-Reflections-master\sample\libs\cauldron\src\DX12\base\ResourceViewHeaps.h
D3D12_GPU_DESCRIPTOR_HANDLE GPUView = m_pHeap->GetGPUDescriptorHandleForHeapStart();
The callstack:
KernelBase.dll!00007ff852c1441c()	Unknown
DXGIDebug.dll!00007ff82dea3866()	Unknown
d3d12SDKLayers.dll!00007fffe20e2ee8()	Unknown
d3d12SDKLayers.dll!00007fffe2014712()	Unknown
d3d12SDKLayers.dll!00007fffe20145cb()	Unknown
d3d12SDKLayers.dll!00007fffe2058324()	Unknown
HSRSample_DX12.exe!CAULDRON_DX12::StaticResourceViewHeap::AllocDescriptor(unsigned int size, CAULDRON_DX12::ResourceView * pRV) Line 110	C++
HSRSample_DX12.exe!CAULDRON_DX12::ResourceViewHeaps::AllocDSVDescriptor(unsigned int size, CAULDRON_DX12::DSV * pRV) Line 157	C++
HSRSample_DX12.exe!CAULDRON_DX12::GBuffer::OnCreate(CAULDRON_DX12::Device * pDevice, CAULDRON_DX12::ResourceViewHeaps * pHeaps, const std::map<unsigned int,enum DXGI_FORMAT,std::less,std::allocator<std::pair<unsigned int const ,enum DXGI_FORMAT>>> & formats, unsigned int sampleCount) Line 174	C++
HSRSample_DX12.exe!SampleRenderer::OnCreate(CAULDRON_DX12::Device * pDevice, CAULDRON_DX12::SwapChain * pSwapChain) Line 118	C++
HSRSample_DX12.exe!HSRSample::OnCreate() Line 150	C++
HSRSample_DX12.exe!RunFramework(HINSTANCE__ * hInstance, char * lpCmdLine, int nCmdShow, CAULDRON_DX12::FrameworkWindows * pFramework) Line 117	C++
HSRSample_DX12.exe!WinMain(HINSTANCE__ * hInstance, HINSTANCE__ * hPrevInstance, char * lpCmdLine, int nCmdShow) Line 1144	C++
[External Code]
If I launch the Release build from VS, the memory can go as high as 1.9 GiB without showing the window.
Thanks.

Hi. It's recommended to use git clone --recursive to make sure the right versions of the dependencies are used. Could you please set both bValidationEnabled and  bGpuValidationEnabled here and see if the validation layer reports anything?

Hi Anton,
Updated the original post: those listed directly in .gitmodules (not including every recursive gitmodules) by Download ZIP
The 2 bool's are already true.
D3D12 ERROR: ID3D12DescriptorHeap::GetGPUDescriptorHandleForHeapStart: GetGPUDescriptorHandleForHeapStart is invalid to call on a descriptor heap that does not have DESCRIPTOR_HEAP_FLAG_SHADER_VISIBLE set. If the heap is not supposed to be shader visible, then GetCPUDescriptorHandleForHeapStart would be the appropriate method to call. That call is valid both for shader visible and non shader visible descriptor heaps. [ STATE_GETTING ERROR #1315: DESCRIPTOR_HEAP_NOT_SHADER_VISIBLE]
D3D12: BREAK enabled for the previous message, which was: [ ERROR STATE_GETTING #1315: DESCRIPTOR_HEAP_NOT_SHADER_VISIBLE ]

Thanks, what if you use this new implementation where it checks for the SHADER_VISIBLE bit?

3435973836  = 0xCCCCCCCC seems to be an uninitialized variable, where does that come from in callstack?

decal_mask.png was corrupted in the ZIP, now fixed, thank you very much, Anton.
Now the Release build works.
The Debug build once triggered an exception on a warning for clearValue, it disappears running again:
pCmdLst1->ClearRenderTargetView(m_GBuffer.m_HDRRTV.GetCPU(), clearValuesFloat, 0, nullptr);

decal_mask.png was corrupted in the ZIP, now fixed, thank you very much, Anton.

That's interesting. Thanks for reporting.

This is back:
D3D12 WARNING: ID3D12CommandList::ClearRenderTargetView: The clear values do not match those passed to resource creation. The clear operation is typically slower as a result; but will still clear to the desired value. [ EXECUTION WARNING #820: CLEARRENDERTARGETVIEW_MISMATCHINGCLEARVALUE]

This is back:
D3D12 WARNING: ID3D12CommandList::ClearRenderTargetView: The clear values do not match those passed to resource creation. The clear operation is typically slower as a result; but will still clear to the desired value. [ EXECUTION WARNING #820: CLEARRENDERTARGETVIEW_MISMATCHINGCLEARVALUE]


Took a bit of a wrong

Hi guys,
First let me thank you for such a useful tool! I use ScoutApp a lot when I work with legacy projects and need SASS support.
I used to have v 2.0.2 and it worked wonderfully. Supported many concurrent projects and felt very fast. But there was a few bugs, that of course got resolved in later versions. So I upgraded to 2.68.
Immediately I noticed the app is slower, or rather UI feels much slower because of all the added transitions that don't quite match each other. Sometimes screens fade in, but when closed they would slide out from the view.  If it was up to me, I would get rid of the transitions and just leave it a plain UI that works and does it quickly :)
Sincerely,
Alex Usanin

@alexus2005
Hey! Thanks for the feedback. We just released a new version of Scout-App today, it's got a lot of new features focused on making the app easier to use. I'd love your input on it.
Here are the release notes for it, downloads are at the bottom:

https://github.com/scout-app/scout-app/releases/tag/v2.12.12


Debian packaging

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=762869

OWSLib is in Debian for many years now


add pre-commit config file

Also gitignore vscode so my git status is clean.
The config file was copied from typeshed with fixes to some paths.

I don't think we want to blacken the test files for now, it moves the comments around too much.

Should we also remove the separate black and isort checks from the CI, like we did over at typeshed? (https://github.com/python/typeshed/pull/6376)

We should also update the section on running tests in CONTRIBUTING, but that can be done in a separate PR.

Everything seems to be working now so I'm merging :) Thanks @JelleZijlstra!


Add more support with Spring for Apache Kafka

Add missed JsonDeserializer into type hints
Add @InitializationHint for some spring-kafka JSON components
which need to initialize their static props for ClassUtils.isPresent()
Improve kafka sample for JSON data interaction
Add a @TypeHint for end-user data classes used in the sample
Improve docker-compose.yml for extra image of the current kafka sample

Depends on: https://github.com/spring-projects/spring-kafka/pull/1881

/CC @garyrussell

Didn't you test it?
You need to update the verify.sh https://github.com/spring-projects-experimental/spring-native/blob/main/samples/kafka/verify.sh

Huh? What is that? Never used those .sh. Tried build.sh once, but it was silent and in the end it produces an .exe which is not always working on Windows.
Yeah.. Thanks for the pointer! Let me see what that verify.sh does for us and fix it respectively!
Docs don't mention this file though...

See https://github.com/spring-projects-experimental/spring-native/blob/main/scripts/test.sh
called from https://github.com/spring-projects-experimental/spring-native/blob/main/samples/kafka/build.sh

Repushed after rebasing to main.
Checked locally with those build.sh and verify.sh: works well even on Windows for me ! 😄

Hmmm - on MacOS:
***************************
APPLICATION FAILED TO START
***************************

Description:

Native reflection configuration for org.springframework.kafka.support.serializer.JsonDeserializer is missing.

Correct. You must build and install locally the main project first.
That JsonDeserializer was missed in the original KafkaHints, so I added it as a part of my PR.

Ah; ok; sorry.

Well, there is no strong dependency on spring-kafka in the main project. Right now it is only a matter of a sample. spring-native can be released as is.
We may find something else though for better compatibility before that August 16th...


How to display tsc output?

With v2.6.0 I could see tsc output/errors inside my console, things like:
ExpenseReport.ts(66,28): 7006 Parameter 'expense' implicitly has an 'any' type.
ExpenseReportsHttpService.ts(7,25): 2304 Cannot find name 'ExpenseReportJsonOutput'.
app.ts(6,15): 7006 Parameter '$componentMapperProvider' implicitly has an 'any' type.

Now, even with fullReporter, I do not get tsc output anymore.

I tested it and I can still see errors. Can you create a small example (gulpfile and typescript files) that can reproduce this?
Which reporter you use shouldn't matter (as long as you don't use nullReporter), since that will only change how errors are shown, not which errors.

See https://github.com/tkrotoff/gulp-typescript-issues and https://github.com/tkrotoff/gulp-typescript-issues/commit/8266b2814db83acd506b7ce3c89aa4a351771fcf
$ cat node_modules/gulp-typescript/package.json | grep 'version'
  "version": "2.5.0",
$ gulp scripts
[00:29:13] Using gulpfile ~/gulp-typescript-issues/gulpfile.js
[00:29:13] Starting 'scripts'...
gulp-typescript-issues/app/app.ts(2,18): 7006 Parameter 'name' implicitly has an 'any' type.
[00:29:14] scripts: .tmp/app/app.js.map
[00:29:14] scripts: .tmp/app/app.js
[00:29:14] scripts: 2 items
[00:29:14] Finished 'scripts' after 1.14 s
$ npm update
[...]
$ cat node_modules/gulp-typescript/package.json | grep 'version'
  "version": "2.7.5",
$ gulp scripts
[00:30:16] Using gulpfile ~/gulp-typescript-issues/gulpfile.js
[00:30:16] Starting 'scripts'...
[00:30:18] scripts: .tmp/app/app.d.ts
[00:30:18] scripts: .tmp/app/app.js.map
[00:30:18] scripts: .tmp/app/app.js
[00:30:18] scripts: 3 items
[00:30:18] Finished 'scripts' after 1.32 s

On my real project I have the same problem with ts.createProject and fullReporter.

I found it, an error about the project root directory is hidden, as it's automatically fixed by gulp-typescript. But when an error is generated, further errors are blocked. I've moved the error filter. The hidden error won't block the generation of other errors any more.

Thx, it works now


Show notification when switching to another page (Vuerouter)

I using vue.js & vue router.
Is it possible during the transition to another page  with : this.$router.push(/page-1), show notification on the page / page-1 .
Is it possible to make the notification appear on the next page?

@ivansteff any answer to this ? :)


Incorrect type for load function

The generated type for the load function is marked as value?: undefined but it should actually be value? string.
The generated type looks like this:
/**
 * Load config
 *
 * @param   {undefined}         value     Pass no value for to determine config file automatically
 * @param   {string}            value     Pass an absolute path to load from alternate location
 * @returns {object}                      The Alias HQ instance
 */
export function load(value?: undefined): object;

I believe the JSDoc here is incorrect, the value param should be combined into one and written as [value] instead to mark as optional, according to the docs: https://jsdoc.app/tags-type.html
The code line in question is here:
https://github.com/davestewart/alias-hq/blob/19309203d65893dab77c2d71e1fa26a16e8034ad/src/index.js#L190

Ah, good spot!
Thanks, I'll take a look.
Planning to fit in lots of lovely OSS work over Christmas so hopefully not too long

Fixed and published on NPM as 5.3.2 👍

Crap. Just realised I left the other line in. LMK if this is an issue right now, otherwise I'll fix in a future update.

Seems to work fine for now.

Thanks for fixing this.

Oh, nice, I like how the popup shows the actual docs text.
Is that VS Code?

Oh, nice, I like how the popup shows the actual docs text.
Is that VS Code?

Yes that's the VSCode intellisense, just by hovering over it. Should work in .js files too.


Recommended testing setup?

So I've spun up AdoBot-IO locally using a local mysql (had to use prior to 8.0 as it had auth issue).
I'm trying to test using Adobot against Android VM (8.1).  I have the VM up, it appears to be able to reach the local instance of Adobot-IO, at least from the browser.  I configure the settings but nothing seems to happen in the actual Adobot-IO app.  I wasn't sure if there was something I needed to do besides pointing the URL and clicking save settings.  The Adobot-IO doesn't have anything in the logs about anything trying to connect other than when I go through the browser.  I'm willing to help deveop some of the added features listed in the README as I would like them as well, but would also like to get a local development environment up so I can actually test.
Any suggestions would be appreciated.
Here are my adobot-IO logs:
https://gist.github.com/xamox/611389c236c44dad3d889e3e36a422cd

Here I can get to adobot-io from the VM:

Or in the adobot setup I tried using the host IP:

Use the actual IP of your server, not localhost. I hope you realize that localhost translates to <IP_ADDRESS> (android's loopback address) and not the IP of your server.


Cant open files in the search results when using "Find All References"

This issue has been moved from a ticket on Developer Community.

[severity:I'm unable to use this version] [regression] [worked-in:2019]
I have a python project and when I use the "Find All References", I see a window with all the references but double clicking an entry doesn't open the file.... yet it shows me the file name, line number and column offset of the reference

Original Comments
Feedback Bot on 10/15/2021, 03:25 PM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.
Feedback Bot on 10/18/2021, 10:34 AM:
Thank you for sharing your feedback! Our teams prioritize action on product issues with broad customer impact. See details at: https://docs.microsoft.com/en-us/visualstudio/ide/report-a-problem?view=vs-2019#faq. In case you need answers to common questions or need assisted support, be sure to use https://visualstudio.microsoft.com/vs/support/. We’ll keep you posted on any updates to this feedback.

Original Solutions
(no solutions)

I just tried this and I can't repro with the latest VS and pylance.

by the latest I mean, VS Version 17.7.0 Preview 3.0 and Pylance 2023.6.40

it is because VS doesn't know how to handle %3A in 'file:///c%3A/xxx/PythonApplication2.py' in Location.uri

I reactivated original feedback ticket and routed it to the VS Editor, feel free to close this one.


feat: timeout on step

with a fix on https://github.com/runabove/venom/compare/timeoutStep?expand=1#diff-9db914ba50bc6d534b92b9539cc7b3a1R59

#4

Steven: finally, merged, next PR is in conflict with this.


See Tokens held by a address

On Accounts page, wee the fungible tokens (gas object) this address own
Improve UI so that users can easily identity what are NFTs and what are FTs

Done!


BREAKING: remove path module

Use https://github.com/denoland/deno_path_util/tree/main instead.

Failing CI seems unrelated.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.45%. Comparing base (0c7f83e) to head (8132383).
Report is 131 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #915      +/-   ##
==========================================
+ Coverage   81.43%   81.45%   +0.01%
==========================================
  Files          97       97
  Lines       23877    24820     +943
==========================================
+ Hits        19445    20217     +772
- Misses       4432     4603     +171

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


x86: PC-relative addressing with a 32-bit address size should mask upper bits.

When the 67 (address size override) prefix is used in long mode on an instruction that uses RIP-relative addressing, the effective address needs to be truncated and zero extended (see: "<IP_ADDRESS> Effect of Address-Size Prefix on RIP-Relative Addressing" of AMD64-APM Vol.1).
This PR fixes this by introducing a new subtable pcRelSimm32_32 that is used when decoding RIP-relative addresses in the Addr32_64 subtable. (Note: The PR also removes a 64-bit only constructor for addr32 which is impossible to reach, since addrsize must be equal to 1 for the addr32 to be used which conflicts with the bit64=1 constraint.)
e.g.:

678b0500000000 at RIP=0x1_0000_0000: "MOV EAX,[0x7]" with mem[0x7]=[0x11 0x11 0x11 0x11], mem[0x100000007]=[0x22 0x22 0x22 0x22]

Hardware Reference (AMD CPU & Intel CPU): { RAX=0x11111111 }
x86:LE:64:default (Existing): "MOV EAX,dword ptr [0x100000007]" { RAX=22222222 }
x86:LE:64:default (This patch): "MOV EAX,dword ptr [0x7]" { RAX=0x11111111 }

See also section <IP_ADDRESS> in the Intel 64 and IA-32 Architecture Software Development Manual Vol 2, which uses pretty much identical language.


Trouble with mapproject

Sometimes maproject cannot project onto the datum, but it can over a DEM. Here's a sample command:
mapproject WGS84<PHONE_NUMBER>.27391028_sc00110_c1_PAN.tiff tmp.tif
In this case the RPC model is embedded inside of image. More details are available offline.

Fixed in ca90dafd15a57b9bb3db84d059607be0d6429582


Synchronize definitions and timeline saving

Definitions and current timeline are kept in memory inside a singleton, and saved to disk when needed.
This data is only saved to the config file on timeline end/start.
While in the editor, default values are loaded as I can't seem to make the saves singleton load.
User can request a definitions reset either by a dedicated function, or by passing an argument to the start method.
Current timeline is saved so user can read it while running or load it back on next run. This feature combined with definitions saving makes a complete save/restore system, in which users can exit mid-timeline without negative effects.
I also added a lot of utility functions in DialogicClass.gd, to allow the user to edit and query variables and glossary definitions. It only repeats functions in the singleton, but this way the user only has one class to worry about and it prevents him from accessing unsafe methods.
Closes #102

@coppolaemilio definitions are now saved in json. Hopefully I did not break anything. I didn't notice any bug but it needs more testing as this change is quite important

While I was at it, I also saved the state (current timeline) in json as well.

Sorry I wasn't able to test until now. I'll do the usually test and QA stuff and try to merge soon 👍🏻

Starting from an empty project:

As usual I do not test on empty project :) It is fixed

Testing an empty project with no variables and just a timeline that has one text event:

Fixed both bugs!
The first one was because I initialized saves with an empty dictionary. Instead I initialized it with {'variables': [], 'glossary': []}, just like I did for default definitions.
As for the dialog not working with the editor node, this was because it returned the file name without .json. I changed the start method to allow for timeline name or filename, so I changed how the timeline variable was set with the start method but forgot about the inspector.
All

Yes! they look fixed on my end 🎉 I'll merge now and keep looking for stuff. I'll need to really go back over all the changes because it is starting to feel too complex. I would love if we could simplify more stuff after v1.0 is stable


'Viocon\Container' not found

Hi, I setup the project manually from github, and it is missing the above class.
Any assistance?

try
composer install
inside pixie directory

This can be closed @usmanhalalit .


Update skus.md

The following fact is currently scattered across the doc, some parts haven't been mentioned anywhere. Relevant pages are a bit disjointed. It can be confusing for users who don't have pre-existing knowledge or solid architectural skills. The following paragraph aims to help the readers:
"To have 99.99% SLA for your virtual machines, you need to create and configure a standard SKU load balancer which includes creating a zone-redundant standard SKU IP address. You also need to deploy two or more VMs to different availability zones in the same region."

#Please-close


RT-541: Template tag to render GSC notes data.

Just a template, tag, and tests.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@0665820). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #237   +/-   ##
==========================================
  Coverage           ?   97.01%
==========================================
  Files              ?       73
  Lines              ?     4488
  Branches           ?        0
==========================================
  Hits               ?     4354
  Misses             ?      134
  Partials           ?        0

Continue to review full report at Codecov.

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


Feature/hooks

Remove Recompose dependency in context-oidc
Add useReactOidc Hooks
Modify acces to context (remove provider, add context)
Packages cleaning in solution
Babel 7 in solution
Move package dep to Peer Deps (like React, React router, etc)

close #203
close #92
Edit : It will be the first major update 🥇

Good job @youf-olivier :)

awesome olivier !

Well done Olivier !

Thanks. But Anyone can approuve this PR please ? :D

Why don't you update your version number in package.json file ?

Because it's done during the build.

I think a "premajor" will be nice for testing
https://github.com/lerna/lerna/tree/master/commands/version#positionals
:D

Yes, I need as many tests as possible. So I will release a premajor


导出csv文件名不断增长/Export csv file name continuously getting longer

原本使用2.3.11版本，导出功能没有问题，每个文件可以写得很大，只生成一个csv文件。升级到2.5.22后，并没有修改配置文件，使用同样命令导出，发现文件会被以13M左右大小切割，并且文件名会不断追加变长，导致最后超过hdfs文件名长度限制，写入失败
Steps to reproduce the behavior (Required)
EXPORT TABLE db.table PARTITION (p202412)
(a,b,c,d,e,f,g)
TO 'hdfs://ip:port/{file_path}/ods/etl_time=2024-12-23/'
PROPERTIES
(
    'column_separator' = '\t',
    'load_mem_limit' = '2147483648',
    'timeout' = '3600',
    'export_max_bytes_per_be_per_task'='2147483648'
)
WITH BROKER 'hdfs_broker'
(
    'username' = '',
    'password' = ''
);

Expected behavior (Required)
导出命令执行后，应当正常导出对应文件

Real behavior (Required)
2.5.22版本，在不改变任何配置的情况下，使用相同export语句导出。size被自动切分，文件名长度不断增加

StarRocks version (Required)
2.5.22

@tracymacding @srlch hi guys, i've found the root of this bug, contributed in this topic, FYI.应该就是这个原因了，希望能尽快出一个patch或hotfix版本，解生产之急
https://forum.mirrorship.cn/t/topic/16587/2

@leonhart i will check it ASAP.

@leonhart Maybe you can try SELECT INTO OUTFILE instead of export to avoid this problem

@leonhart Maybe you can try SELECT INTO OUTFILE instead of export to avoid this problem

thx, i'll try it

select_query_stmt
INTO OUTFILE "<file_path>"
FORMAT AS CSV
PROPERTIES
(
    "broker.name" = "xxxx",
    "broker.username" = "xxxx",
    "column_separator" = ",",
    "line_delimiter" = "\n",
    "max_file_size" = "100MB"
);

You can try this, but OUTFILE is a deprecated feature in newer version of SR. Maybe you can use FILE  to export data in higher versions. @leonhart

@srlch thanks a lot. it works for me. hope the community could figure this bug out and fix it soon.;)


Fix segfault in jody hash

One application of mine has suddenly crashed after running for weeks without a glitch. The system is Intel Xeon E5-2620 v4.
 As you can see the panic issued by this piece of code here https://github.com/segmentio/fasthash/blob/f3727d1efd1ebf7ce313d35dea4ad7096d92e72d/jody/hash.go#L102 which has comment about potential segfault. :)
I've fixed hash computation on tail data. Unfortunately this leads to some performance degradation but I haven't found a way to avoid it.
name                     old time/op    new time/op    delta
Hash64/jody/algorithm-4    12.7ns ± 1%    13.3ns ± 1%  +4.99%  (p=0.000 n=19+20)

name                     old speed      new speed      delta
Hash64/jody/algorithm-4  2.83GB/s ± 2%  2.70GB/s ± 1%  -4.65%  (p=0.000 n=19+20)

Hi - I'm running into a similar issue with the new code. In my case I can reproduce very reliably (both in the earlier version, and this version).
 fatal error: checkptr: pointer arithmetic result points to invalid allocation

 goroutine 161 [running]:
 runtime.throw(0x11204c7, 0x40)
 	/usr/local/go/src/runtime/panic.go:1116 +0x72 fp=0xc000071340 sp=0xc000071310 pc=0x46a4d2
 runtime.checkptrArithmetic(0xc00020f510, 0xc0000713f0, 0x1, 0x1)
 	/usr/local/go/src/runtime/checkptr.go:43 +0xbe fp=0xc000071370 sp=0xc000071340 pc=0x4397be
 github.com/segmentio/fasthash/jody.AddString64(0x1c26a908921dd5e0, 0xc00020f50c, 0x4, 0x1c26a908921dd5e0)

Hi - I'm running into a similar issue with the new code. In my case I can reproduce very reliably (both in the earlier version, and this version).
 fatal error: checkptr: pointer arithmetic result points to invalid allocation

 goroutine 161 [running]:
 runtime.throw(0x11204c7, 0x40)
 	/usr/local/go/src/runtime/panic.go:1116 +0x72 fp=0xc000071340 sp=0xc000071310 pc=0x46a4d2
 runtime.checkptrArithmetic(0xc00020f510, 0xc0000713f0, 0x1, 0x1)
 	/usr/local/go/src/runtime/checkptr.go:43 +0xbe fp=0xc000071370 sp=0xc000071340 pc=0x4397be
 github.com/segmentio/fasthash/jody.AddString64(0x1c26a908921dd5e0, 0xc00020f50c, 0x4, 0x1c26a908921dd5e0)

@jnjackins could you show the input which caused the panic and backtrace up to jody hash code?

@jnjackins could you show the input which caused the panic and backtrace up to jody hash code?


A geocode unit test is failing intermittently

Overview
A geocode unit test is failing intermittently
Expected Behavior
All test pass
Actual Behavior
One geocoding test sometimes fails
Steps to Reproduce

Run cibuild

Demo
======================================================================

FAIL: test_successfully_geocoded_item_has_correct_results (api.tests.FacilityListItemGeocodingTest)

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

Traceback (most recent call last):

  File "/usr/local/src/api/tests.py", line 421, in test_successfully_geocoded_item_has_correct_results

    "1400 John F Kennedy Blvd, Philadelphia, PA 19107, USA",

AssertionError: 'Philadelphia, PA, USA' != '1400 John F Kennedy Blvd, Philadelphia, PA 19107, USA'

- Philadelphia, PA, USA

+ 1400 John F Kennedy Blvd, Philadelphia, PA 19107, USA

Will be fixed by #321


[TabBarIOS] The tintColor attribute of TabBarIOS is not working

[Bug]The tintColor attribute of TabBarIOS isn't effective！Should it be white？

I just cloned fresh from master and it works fine for me - are you using the latest version?

Yes it is when clone the master。

Thanks for chiming in @cmcewen


Update Endpoints samples:

Change default service names to the new cloud.goog format.
Change app.yaml beta_settings to endpoints_api_service.

Travis says:
[ERROR]   The project com.google.cloud.speech:grpc-sample:1.0 (/home/travis/build/GoogleCloudPlatform/java-docs-samples/speech/grpc/pom.xml) has 1 error
[ERROR]     Unresolveable build extension: Plugin kr.motd.maven:os-maven-plugin:1.4.1.Final or one of its dependencies could not be resolved: Failed to read artifact descriptor for kr.motd.maven:os-maven-plugin:jar:1.4.1.Final: Could not transfer artifact kr.motd.maven:os-maven-plugin:pom:1.4.1.Final from/to central (https://repo.maven.apache.org/maven2): Access denied to: https://repo.maven.apache.org/maven2/kr/motd/maven/os-maven-plugin/1.4.1.Final/os-maven-plugin-1.4.1.Final.pom , ReasonPhrase:Forbidden. -> [Help 2]

I'll let you fix and then I'll review.

I actually have no idea how to debug this Travis failure, does it not seem unrelated to this change?

Prior review said DO NOT MERGE until docs are coordinated. Once Sepehre asks, it's ok to merge. @tswast @jerjou

Please merge.


chore: add options to dev tools

Description
chore: add options to dev tools
Type of change
Please delete options that are not relevant.

[x] Chore

How has this been tested

[x] Unittest
[ ] Others:

Checklist:

[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Let's make the world better✨😋🐍🌎

Codecov Report

Merging #34 (b32e825) into main (65449f5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #34   +/-   ##
=======================================
  Coverage   95.50%   95.50%
=======================================
  Files          21       21
  Lines         601      601
  Branches       35       35
=======================================
  Hits          574      574
  Misses         18       18
  Partials        9        9

Impacted Files
Coverage Δ

pypj/setting.py
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Upgrade pitest-maven-plugin 1.14.3 -> 1.14.4

This PR contains the following updates:

Package
Type
Update
Change

pitest-maven-plugin (source)
build
patch
1.14.3 -> 1.14.4

Release Notes

hcoles/pitest (pitest-maven-plugin)
v1.14.4
Compare Source

#1246 Priotitise previous killing test when using history files

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade pitest-maven-plugin 1.14.3 -> 1.14.4

See:
- https://github.com/hcoles/pitest/releases/tag/1.14.4
- https://github.com/hcoles/pitest/compare/1.14.3...1.14.4


remove: DDPJobDefinition from SDK

Issue link
https://issues.redhat.com/browse/RHOAIENG-5096
What changes have been made
This pr contains a removal of the  DDPJobDefinition from the codeflare-sdk, as well as updated tests to reflect the change.
A follow up pr will be made to update the docs and demo notebooks upon being merged.
Verification steps
Run the sdk tests and ensure they are functioning correctly.
Run a demo notebook to ensure no breaks.
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

@VanillaSpoon  Tried running tests mnist_raycluster_sdk_oauth_test.py  and demo notebooks with cluster configuration defined , App wrapper creation is fine but Ray Cluster creation is Failing..
Tried running on installing Main branch aswell , same error
@ChristianZaccaria Does we missed any cluster configuration here?
tests/e2e/mnist_raycluster_sdk_oauth_test.py::TestRayClusterSDKOauth::test_mnist_ray_cluster_sdk_auth Insecure request warnings have been disabled Written to: /opt/app-root/src/.codeflare/appwrapper/mnist.yaml AppWrapper 'mnist' has been created in the namespace: 'test-ns-8iw5r' ╭──────────────────────╮ │   🚀 Cluster Queue   │ │      Status 🚀       │ │ +-------+----------+ │ │ | Name  | Status   | │ │ +=======+==========+ │ │ | mnist | queueing | │ │ |       |          | │ │ +-------+----------+ │ ╰──────────────────────╯ Waiting for requested resources to be set up...
cluster = Cluster(ClusterConfiguration(     name='jobtest',     namespace='test-dw',     num_workers=2,     head_cpus="500m",     head_memory=2,     min_cpus=1,     max_cpus=1,     min_memory=4,     max_memory=4,     num_gpus=0,     image="quay.io/project-codeflare/ray:latest-py39-cu118",     instascale=False,     write_to_file=True, ))

@Srihari1192 Those clusterconfiguration look good to me, the openshift_oauth is removed as we are defaulting to it to be true. And all oauth resources are created by the RC Controller in Main

You can remove torchx_config function in cluster.py and torchx_scheduler = "ray" as well as imports involving torchx at the top of unit_test.py and remove torchx from requirements.


baremetal: fix rebuild root no password

这个 PR 实现什么功能/修复什么问题:
修复 baremetal 重装系统没有密码的 bug
是否需要 backport 到之前的 release 分支:

release/2.10.0

/cc @wanyaoqi
/area baremetal

Codecov Report

Merging #2249 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2249      +/-   ##
=========================================
- Coverage    7.09%   7.05%   -0.05%
=========================================
  Files         452     451       -1
  Lines       71256   71092     -164
=========================================
- Hits         5057    5015      -42
+ Misses      65782   65663     -119
+ Partials      417     414       -3

Flag
Coverage Δ

#aFlag
7.05% <0%> (-0.05%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/baremetal/tasks/rebuild.go
0% <0%> (ø)
:arrow_up:

pkg/util/rbacutils/rbac.go
69.96% <0%> (-3.41%)
:arrow_down:

pkg/util/httputils/httputils.go
47.45% <0%> (-0.59%)
:arrow_down:

pkg/compute/guestdrivers/qcloud.go
3.75% <0%> (-0.06%)
:arrow_down:

pkg/multicloud/ucloud/ufile.go
0% <0%> (ø)
:arrow_up:

pkg/multicloud/aws/storagecache.go
0% <0%> (ø)
:arrow_up:

pkg/multicloud/ucloud/storagecache.go
0% <0%> (ø)
:arrow_up:

pkg/multicloud/aws/bucket.go
0% <0%> (ø)
:arrow_up:

pkg/util/fileutils2/seeker.go

pkg/compute/models/networks.go
0.82% <0%> (ø)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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

/lgtm
/approve


CM-18 - Set referrer policy

It is a quick-fix. We need more research.
It looks like we hit the issue because of the behaviour described here: https://developers.google.com/web/updates/2020/07/referrer-policy-new-chrome-default.
Open questions:

[x] The behaviour change was planned for Chrome version 85, we are now running tests with 89. Why did we see the issue only now?
[x] Do we need to inform our customers/adjust any integrations?
[x] It would be good to have some research on how the privacy behaviour will develop over time. Do we need a long term plan?

Closing this one, for https://github.com/LiveIntent/live-connect/pull/39


FZF doesn't work. It doesn't open any file

Hi! This is my first experience with FZF and I known it because I'm migrating my dotfiles to neovim. I'm working to use this tool with neovim but it doesn't work.
I installed it with Plug.
Plug 'junegunn/fzf', { 'dir': '~/.fzf', 'do': 'yes \| ./install' }
Plug 'junegunn/fzf.vim'

https://github.com/gianarb/dotfiles/blob/feature/neovim-migration/nvimrc#L12-L13

In vim after :FZF I search the file and I press ENTER but won't open nothing.
Into the terminal the problem is in this screenshot.

After ENTER it prints the path of searched file.
Thanks for your feedback! :)

Cannot reproduce with your nvimrc. :FZF works just fine. The terminal emulator of neovim is known to have issues; see https://github.com/junegunn/fzf/labels/neovim. Maybe you're running into the problem and upgrading neovim to the latest might help, but I'm not sure.

Into the terminal the problem is in this screenshot.
After ENTER it prints the path of searched file.

It's the expected behavior. fzf is just a filter, like grep, it doesn't take any action instead it just prints the selected lines.

Perfetct! :) THanks for your feedback! I close this issue :)

So upgrading neovim solved the issue?

I installed neovim from github repo yesterday.. There are 5 new commits I'm compiling them :)

Hmm, I have much older version of neovim than you, so I don't think those 5 commits will solve the problem. You might want to check if the :FZF command (and the other commands from fzf.vim repo) work on plain Vim.

mmm this is magic.. but into the last 5 commits of neovim there is the resolution of my problem o.O..
It works! o.O

Ha!
/cc @tarruda @justinmk

This bug was introduced and fixed yesterday, you got unlucky and cloned between the bug/fix :smile:

I'm very lucky!


Added setOutputBuffering() to RouteInterface.php

Closes #2272

Coverage remained the same at 97.312% when pulling 18fce2db64a22fd40ec0138e37142f7840fab685 on RyanNerd:output-buffering-route-interface into f68e58cee9f6213647f88cfc62fb05240b01e3a7 on slimphp:3.x.

Thanks.


MultiplayerSpawner not working when instantiated on client after the host

Godot version
v4.0.alpha10.official [4bbe7f0b9]
System information
Windows 10
Issue description
I've been getting some very buggy behavior with the MultiplayerSpawner when instantiating it
If you're connected to a server as a client, and instantiate a scene containing a MultiplayerSpawner AFTER the host has already instantiated that same scene & the host has already instantiated scenes for the MultiplayerSpawner to spawn, the MultiplayerSpawner just won't work on the client
The following error will be spit out by the client:

Steps to reproduce
Simply load a scene containing a MultiplayerSpawner as a client after the host has already loaded that same scene & has instantiated scenes for the MultiplayerSpawner to spawn
Minimal reproduction project
MultiplayerSpawner bug.zip

I've also come across this in #62027. What is odd is that the multiplayer_bomber example works fine, and I believe it creates the server scene first. So it seems this problem only occurs when you join a bit later than the server


Bug: Importing posts

Every time a new post gets published we send a ping to certain services, along with sending segment analytic data (post created, post published etc).
These pings triggered a crawler to scan the posts that were created.
This also occurs during an import, which causes a huge traffic spike if there a lot of posts in the import.
Ghost version: 1.14.1 and probably previous v1 versions

For a short term solution to reduce the load generated by imports & 1.0 migrations, I will disable the rpc ping on import.
Longer term:
I'm not sure whether the problem is that

Either the post.published event shouldn't fire on import (which poses a problem for other event driven systems like sitemaps) and that we should have a slightly different event post.imported.
Or whether we should just pay attention to this properly in event listeners :thinking_face:

Seeing as the events do get access to the context that the event was fired during an import, paying attention properly when implementing event systems may be easier. Having a whole different set of events would introduce some complexity.
Something to think about 🤔

Then again, the only case where we need the post.published event on import, is for sitemaps. And actually sitemaps are implemented incorrectly.
A future incarnation is coming, whereby all the model events that affect resources are processed into url.added,url.changed or url.removed events. The sitemap would be driven from that. This processor should legitimately listen to import events...
Meanwhile slack pings, rpc pings, any other kind of integration we build would only ever need to listen to post.published & could be safe in the knowledge this event is correct.
I will implement a short term bug fix today, and then add this refactoring of post.published as a todo on the existing work on sitemaps, which can be considered tracked by the ancient #4588.


feat(icon): add e2e icon tests

PR includes basic tests for icon and removes unnecessary code from e2e-app.ts and e2e-app.html, similarly found in PR #650.
Closes #560.
R: @jelbourn

LGTM other than the names of the tests

@jelbourn Ok, I changed the test names.

@jesselpalmer minor lint error

@jelbourn ok, line ending added, though strangely they don't appear on GitHub.

LGTM


Extend the BorTag GC to AllocIds

As suggested in https://github.com/rust-lang/miri/issues/3080#issuecomment-1732505573
We previously solved memory growth issues associated with the Stacked Borrows and Tree Borrows runtimes with a GC. But of course we also have state accumulation associated with whole allocations elsewhere in the interpreter, and this PR starts tackling those.
To do this, we expand the visitor for the GC so that it can visit a BorTag or an AllocId. Then when the GC runs, we use the reachable set of AllocIds to remove elements from the intptrcast state.

@rustbot author

:umbrella: The latest upstream changes (presumably #3122) made this pull request unmergeable. Please resolve the merge conflicts.

This isn't ready, I'm still thinking about your two comments I didn't mark as resolved.

@rustbot author

@rustbot author

CI has been running for an hour. That's not good. I'll look into it.

The slow part of the CI is inserting all the live AllocIds into the reachable set. Aside from asking the interpreter to track a FxHashSet<AllocId> for us then hand it to the GC via clone I'm not sure what else to do other than run the GC less often, but the CI is slow because we're running the GC super often. Maybe trim down the size of that test?
The cde that's causing the GC to lose track of a pointer is here:
https://github.com/rust-random/getrandom/blob/bd14b7a243c7cda78f698742a84d1a0bbfcba972/src/macos.rs#L14-L21
type GetEntropyFn = unsafe extern "C" fn(*mut u8, libc::size_t) -> libc::c_int;

pub fn getrandom_inner(dest: &mut [u8]) -> Result<(), Error> {
    static GETENTROPY: Weak = unsafe { Weak::new("getentropy\0") };
    if let Some(fptr) = GETENTROPY.ptr() {
        let func: GetEntropyFn = unsafe { mem::transmute(fptr) };
        for chunk in dest.chunks_mut(256) {
            let ret = unsafe { func(chunk.as_mut_ptr(), chunk.len()) };

The slow part of the CI is inserting all the live AllocIds into the reachable set. Aside from asking the interpreter to track a FxHashSet for us then hand it to the GC via clone I'm not sure what else to do other than run the GC less often, but the CI is slow because we're running the GC super often. Maybe trim down the size of that test?

That test? We run all tests on Linux with "do GC after every step".
https://github.com/rust-lang/miri/blob/d36c3e7cac5d97aefd86f0a2ecb93e9b29f70324/.github/workflows/ci.yml#L38-L40
And that seems to be good, we caught a bug?
For performance, here's one idea: we don't actually need the set. We need a function fn(AllocId) -> bool that tells us whether the AllocId is used anywhere. So instead of adding all live allocations to the set, we can make that function a 3-way disjunction, where it checks the global alloc_map, the local alloc_map, and the collected set. That might help?

The cde that's causing the GC to lose track of a pointer is here, and it has a wild diagnostic:

Ah, that's the old macOS getrandom doing strange integer pointer shenanigans. If this triggers a GC bug then the first thing to do is extract it into a dedicated test so that we don't have to rely on a pass-dep test to catch this. It should then also reproduce on all targets; the reason it's macOS-only is that getrandom uses different code for different targets.

Here's a minimization:
#![feature(rustc_private)]
extern crate libc;

extern "Rust" {
    pub fn miri_run_provenance_gc();
}

type GetEntropyFn = unsafe extern "C" fn(*mut u8, libc::size_t) -> libc::c_int;

fn main() {
    let name = "getentropy\0";
    let addr = unsafe { libc::dlsym(libc::RTLD_DEFAULT, name.as_ptr() as *const _) as usize };
    unsafe {
        miri_run_provenance_gc();
    }

    let ptr = addr as *mut libc::c_void;
    let func: GetEntropyFn = unsafe { std::mem::transmute(ptr) };
    let dest = &mut [0u8];
    unsafe { func(dest.as_mut_ptr(), dest.len()) };
}

Run with ./miri run -Zmiri-tag-gc=0 to get the error we get in CI.

Note that this code runs fine without explicitly invoking the provenance GC, because in the minimization there is no basic block end in the important region of the program. I always add code to run the GC on every machine step while debugging to avoid getting tripped up by this.

Ah! I think the GC isn't picking up the contents of extra_fn_ptr_map: https://github.com/rust-lang/rust/blob/4d7f952a02d0bca67c98a6b74895b7e3fbe38341/compiler/rustc_const_eval/src/interpret/memory.rs#L106

I think this is going to be easier to do as a rustc PR, so that I can iterate on the interface between Miri and the interpreter core code. I'll set up that later today.

I think this is going to be easier to do as a rustc PR, so that I can iterate on the interface between Miri and the interpreter core code. I'll set up that later today.

However, rustc doesn't run the gc-interval=1 tests.

Ah! I think the GC isn't picking up the contents of extra_fn_ptr_map: https://github.com/rust-lang/rust/blob/4d7f952a02d0bca67c98a6b74895b7e3fbe38341/compiler/rustc_const_eval/src/interpret/memory.rs#L106

Aha!
If we follow my proposal, the API we need is an efficient "is this AllocId live". So that's a single fn(&self, AllocId) -> bool, I think. Basically a more efficient version of get_alloc_info that doesn't compute the size. Maybe we just want that function to be split into two, one for size+align and one for AllocKind?

Subsumed by https://github.com/rust-lang/rust/pull/118029


Draft: update ci to include tests for v0.27.x

Update kind version to v0.19.0 and add 1.27.x to the test matrix.

Codecov Report

Merging #463 (2a69322) into main (0d5ff08) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #463   +/-   ##
=======================================
  Coverage   70.10%   70.10%
=======================================
  Files          17       17
  Lines         950      950
=======================================
  Hits          666      666
  Misses        263      263
  Partials       21       21


[v4.12.0] Fixed EnumArray rule and enhanced validatedArray types

#123  👈
dev

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

Join @GoodM4ven and the rest of your teammates on  Graphite


'MAKE_EXE-NOTFOUND' is not recognized as an internal or external command, operable program or batch file.[QUESTION]

Hi there,
When I using cmake to build the C++ producer sdk on windows, I have this error message:

This is my log file:
CMakeOutput.log
What is 'MAKE_EXE-NOTFOUND' command? I installed the make.exe but it didn't work. Does anyone has this problem before?

I have not seen any issues like this. Could you make sure you have a clean build with the correct dependencies set. We have a windows Travis.ci build and the script that you can check out. Also, try to shorten the path and retry

Any updates on this issue?

Sorry I forgot to update. I try to use X64 Native Tools Command Prompt instead of Cmake Gui and the problem fixed. Thank you very much for your help!

@daling11 , the  X64 Native Tools Command Prompt you mentioned is the Windows build-in command line prompt?, i faced the same issue, and i used the Windows default command line prompt with administrator privilige


fix: Address Rule AVP1000 Incorrect AvaloniaProperty storage

What does the pull request do?

Address Rule AVP1000 Incorrect AvaloniaProperty storage
What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

@emmauss Can you review?

You can test this PR using the following package version. 11.0.999-cibuild0030586-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


fix: proper check for absolute specifier

Looking at your code, I believe you have a bug with the check around if the specifier is absolute or not 😳 Or am I missing something?

@cspotcode  I was just looking around and found this pr, and this needs to be merged. I hope you can see my comment :)

Merged and published as v0.0.4


Build error

Hi, its cool project!
It greats work in editor. But i cant build demo scene to PC.
Assets\Scripts\UIToolkitXRAdapter\XRAdapter\InputSystenEventSystemPatcher.cs(10,6): error CS0246: The type or namespace name 'InitializeOnLoadAttribute' could not be found (are you missing a using directive or an assembly reference?)
Assets\Scripts\UIToolkitXRAdapter\XRAdapter\InputSystenEventSystemPatcher.cs(10,6): error CS0246: The type or namespace name 'InitializeOnLoad' could not be found (are you missing a using directive or an assembly reference?)
If put InputSystenEventSystemPatcher to Editor folder and change field CurrentlyFocusedPanel to public, it builds properly. But nothing happens in runtime, UI button is not pressed.
Unity 2021.1.6f1
PS Sorry for bad english.

Hi :)
Sorry for the delayed response😅 and thanks for the feedback.
I pushed a fix to the master, which works on my machine. I would be happy, if you test it on your machine.
This is an academic project, so I wrote a report about it. I will add the report as a documentation soon, this may also help to improve the understanding of the project.
Cheers,
Bernward


wss

Does it not support the wss protocol?

Hi @Sunny0218,
Sure, you can refer to my other project: https://github.com/tommy44458/light-vue3-starter
In production env, it connect to a public broker by was protocol.

@tommy44458 Thank you.I can connect my mqtt borker use wss.  There is one more question I would like to ask you. In vue project, if I use app.use(mqttVueHook, CONFIG) to connect to mqtt broker, then is the whole vue instance project is a mqtt client, no matter I subscribe or publish any topic, it is based on the current vue instance which is a mqtt client? If I want different components to act as different mqtt clients, what can I do?

@Sunny0218,
I'm sorry for not getting back to you sooner. I have released a new version, v1.2.19, which supports multiple MQTT clients within a single project. You can refer to the "Multi-Client" section in the README for more information.

@Sunny0218,
I'm sorry for not getting back to you sooner. I have released a new version, v1.2.19, which supports multiple MQTT clients within a single project. You can refer to the "Multi-Client" section in the README for more information.

thank you so much!


New RNAseq library_preparation_kits

Select the target metadata schema name (can be multiple), or choose 'Other' if not listed.
RNAseq
If 'Other' is selected, please enter the schema name below.
No response
Describe the change request (use a numbered list if many).
Add

Chromium Next GEM Single Cell 3’ GEM, Library & Gel Bead Kit v3.1, 16 rxns PN-1000121
Chromium Next GEM Single Cell 3' HT Kit v3.1, 48 rxns PN-1000348 | 8 rxns PN-1000370

We have a standard string pattern to label library kits: manufacturer; model/name; product code.
Below are the new kit labels following the pattern:

10X Genomics; Chromium Next GEM Single Cell 3’ GEM, Library & Gel Bead Kit v3.1, 16 rxns; PN 1000121
10X Genomics; Chromium Next GEM Single Cell 3' HT Kit v3.1, 48 rxns; PN 1000348
10X Genomics; Chromium Next GEM Single Cell 3' HT Kit v3.1, 8 rxns; PN 1000370

@HerrLufty Could you please confirm these to the requester?
PS. I need to break down the last library kit item because it contains two product numbers (PN).

Confirmed...go ahead with the add


⚠ ku-s has degraded performance

In a52a288, ku-s ($KU_API_STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 18779 ms

Resolved: ku-s performance has improved in 6ab647a after 49 minutes.


istio-demo-auth.yaml contains invalid yaml (empty values)

in https://github.com/istio/istio/releases/download/1.0.6/istio-1.0.6-linux.tar.gz, install/kubernetes/istio-demo-auth.yaml contains this:
apiVersion: "config.istio.io/v1alpha2"
kind: kubernetesenv
metadata:
  name: handler
  namespace: istio-system
spec:
  # when running from mixer root, use the following config after adding a
  # symbolic link to a kubernetes config file via:
  #
  # $ ln -s ~/.kube/config mixer/adapter/kubernetes/kubeconfig
  #
  # kubeconfig_path: "mixer/adapter/kubernetes/kubeconfig"

which causes this error when I attempt to kubectl apply it in a k8s 1.15.0 system:
error: error validating "istio-demo-auth.yaml": error validating data: unknown object type "nil" in kubernetesenv.spec; if you choose to ignore these errors, turn validation off with --validate=false

Same appears to be true in https://github.com/istio/istio/releases/download/1.0.7/istio-1.0.7-linux.tar.gz.
in istio 1.1.7, its now:
apiVersion: "config.istio.io/v1alpha2"
kind: handler
metadata:
  name: kubernetesenv
  namespace: istio-system
  labels:
    app: mixer
    chart: mixer
    heritage: Tiller
    release: istio
spec:
  compiledAdapter: kubernetesenv
  params:
    # when running from mixer root, use the following config after adding a
    # symbolic link to a kubernetes config file via:
    #
    # $ ln -s ~/.kube/config mixer/adapter/kubernetes/kubeconfig
    #
    # kubeconfig_path: "mixer/adapter/kubernetes/kubeconfig"

And it looks the same in istio 1.2.0.
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[ ] Developer Infrastructure
Expected behavior
Expect release tarballs to contain not contain yaml with empty values.
Steps to reproduce the bug
Download release tarball, look at (or attempt to apply) ./install/kubernetes/istio-demo-auth.yaml
Weirdly, I notice that kubectl doesn't always notice the problem the first time I apply the yaml.  But once it has started to notice, it sees it every time I re-apply after that.
Version (include the output of istioctl version --remote and kubectl version)
istio release tarballs for 1.0.6, 1.0.7, 1.1.7, 1.2.0
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:40:16Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:32:14Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}

How was Istio installed?
Installed according to the calico docs at https://docs.projectcalico.org/v3.3/getting-started/kubernetes/installation/app-layer-policy
Environment where bug was observed (cloud vendor, OS, etc)
k8s 1.15.0, ubuntu 16.04 VM on GCE n1-standard-2
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

So to make sure I fully understand, this doesn't work on 1.0.x, but works on 1.1 and 1.2 right? Is this just a new validation in k8s 1.15? Because I would imagine others would have ran into this otherwise.
So some additional context, Istio 1.0 came out long before 1.15 so obviously hasn't been tested on kubernetes 1.15 which is why I ask.


VER-685 [AL] helpline-frontend: (FE) Modify content on underlying hel…

…pline pages

Changes already merged.


Platform settings styleguide update

I'm documenting our recent design updates for the settings pages inside www.opencollective.com.
Background
We have never done a proper design to cover all the options we put inside the platform's settings. The developer team initially designed the settings pages in the early versions of Open Collective. We have been building on top of that with isolated updates adding new features or facelifting existing ones. We seized the opportunity provided by #5289 and took the time to set a set of rules of text and spacing that make our settings accessible. Now that we have a more complex platform and require users to spend time setting the suitable options to use the platform and deal with sensitive aspects like security.
Font use and inner spacing

Layout

🎨 Figma file reference
cc @Betree

@Memo-Es With the latest dashboard version, this doesn't seem relevant anymore. Should we close it?


Don't commit associations on assign; defer them until save.

This breaks existing functionality, but is probably correct. Since it's not backwards compatible, I wanted @jnunemaker to look it over.
The general problem here is described in #218, and is that assigning associations immediately saves the parent document and the association. This is both surprising and inconsistent with how it's done in other ORMs, and I'd argue that it needs to change.
This fix defers association saves/nullifies until the owner record is saved (but after it is validated!) - this means the deprecation of :dependent => :delete. The current implementation makes no sense at all, IMO, since :delete infers "I don't want this model to do its normal teardown". Additionally, this means that assigning an association to nil will not immediately delete the associated record, which is extremely dangerous behavior, IMO - a simple assignment should never destroy data in the database!
The basic fix here is to just defer association commits to before_save and eliminate delete operating on associated records. Travis is happy with it, but I did remove the specs pertaining to the eliminated behavior to get it there :)
Anyhow, please review and let me know what you think. If the current behavior is there for a reason that I'm not comprehending, I'd love to find a way to improve it.

Just closing since it is super old and shows up for me.


Add drawer

Somebody said Drawers are good.
Implement simple and informative app drawer to the project.

I would like to contribute to this issue by creating the app drawer. Can we have further discussion on this?


[744] Caching of API requests takes into account query params

Trello card URL:
https://trello.com/c/TjcF4e0g
Changes in this PR:

These were not being forwarded so we had a problem where requests for jobs.json?page=1 would be cached and then we'd request jobs.json?page=1 only to get a cache hit and the results from page 1, rather than page 2.
We changed the cloudfront setting for this manually on staging, verified it fixed it before applying to production until this change can be made. https://www.terraform.io/docs/providers/aws/r/cloudfront_distribution.html#query_string

Next steps:

[x] Terraform deployment required?

 API caching respects query parameters so each page can be fetched


Reset URIs on playback pipeline also

Related to #34280. Missed doing it for playback in that PR.

API change check
API changes are not detected in this pull request.


[question] is syntastic_c_remove_include_errors works for gcc.vim file too?

Hello,
TL;DR:
I'm looking forward to ignore Include header errors such as
src/daemon.c|44 col 10 error| fatal error: config.h: No such file or directory

But placing let g:syntastic_c_remove_include_errors = 1 does not help, where Syntastic shows the following on the status bar:
[Location List] :SyntasticCheck gcc (c)

I see syntastic_c_remove_include_errors is included in syntax_checkers/c/make.vim but  not in syntax_checkers/c/gcc.vim
Should this work or I'm effectively hitting an issue of this flag not being honored?

Looking :help syntastic-commands I found:
                                       'g:syntastic_c_remove_include_errors'
Type: boolean
Default: 0
By default, errors in files included from the file being checked are shown.
Set this variable to 1 to remove messages about errors in included files.
Please note that this means syntastic will silently abort checks if there are
fatal errors in one of the included files.

However, this is not the case on my use case (Using vim on a linux-terminal to open a C project ). My guess is that (as mentioned previously) this flag is not being used on gcc.vim, but still, I'm unsure if this file is the current used on my use-case described.

As you note, syntastic_c_remove_include_errors is about errors that happen inside included files, while your error is located in src/daemon.c.
In principle you could use syntastic_quiet_messages to silence it (cf. :h 'syntastic_quiet_messages'), but in practice that would be less than useful since the error is fatal, the compiler simply bails out at that point.  So you can remove the error, but you won't get any further errors anyway.  This is a limitation of gcc, as far as I know there is no way to trick it to keep going after stumbling upon missing include files.  Your best option is to comment out the #include. shrug

I understand now, thanks for the explanation @lcd047
closing the issue now.


Remove pre 1.36 backwards compatibility code

Back when I submitted #185 I did not get the impression that dropping support for older Rust was an option. But it seemed to have gone through without much fanfare in #188, so I guess we can remove all this backwards compatibility code now then.

bors r+

Yea I merged that because I feel that we're going to be publishing a new major version soon anyways.


Error while running mix churn

Hey! Thanks for awesome project!
** (SyntaxError) nofile:1: syntax error before: '='
    lib/churn/processor/cyclomatic_complexity.ex:13: Churn.Processor.CyclomaticComplexity.calculate/1
    lib/churn/processor.ex:11: Churn.Processor.process/2
    (elixir 1.10.4) lib/stream.ex:572: anonymous fn/4 in Stream.map/2
    (elixir 1.10.4) lib/enum.ex:3686: Enumerable.List.reduce/3
    (elixir 1.10.4) lib/stream.ex:1609: Enumerable.Stream.do_each/4
    (elixir 1.10.4) lib/enum.ex:3383: Enum.reverse/1
    (elixir 1.10.4) lib/enum.ex:2982: Enum.to_list/1

I am in an umbrella app and using default config except I've changed lib to apps. If I change the value to apps/*/lib or apps/*/test it works OK.

Thanks! I have added this to apps/*/lib to the default configuration and every documentation page in the project. The interesting thing for me is that the error occurred but only for some of my umbrellas. I need to investigate it deeper.


DDA Streaming: Listdir for non-storage directories returns only first 500 files

Need to implement proper paginated fetching there

Hi @kbolashev, would like to work on this task. Can you give a heads-up so that I can start.

Hi @hjain5164 ! Thank you very much for reaching out
Really sorry, but it looks like I accidentally filed this bug.
Right now our backend's pagination seems to be working a bit incorrectly, so for now the client receives all of the items at once, so this will only have to be fixed once the backend bug is fixed haha

Hi @hjain5164 !
Thanks for reaching out
I did some more tests and it seems like our backend for now returns all the files for non-storage dirs when you listdir, so seems like it's something that we should fix on the backend first before fixing in the client. I'll close this for now, but thanks so much for reaching out again


Add --fix option to automatically resolve lint errors/warnings

Hey folk, not necessarily an issue, more of a feature request / question.
I'm sure you're aware of the fix feature of eslint.
Would this be something that we could expect to see for sass-lint too? It'd be absolutely amazing. Tried looking through issues but haven't found anything similar.
Maybe if there's an already on-going effort or if there were discussions about this I could provide some assistance?
Thank you for the awesome tool and looking forward to hearing back!

HI @elodszopos this was actually being worked on here but unfortunately the author wasn't keen on following our suggestions on how to get this work released / following our contribution guidelines which made it really hard for us to manage or ultimately merge.
We'd obviously be very happy to see this effort come back to life. We'd request that PR's were broken up on a rule by rule basis and included the relevant tests and documentation but I'd be happy to help with these areas if someone wants to just write the functionality.
We even have this branch https://github.com/sasstools/sass-lint/tree/feature/auto-fix ready for all PR's to be applied to so we could release a bulk of these fix issues and test them independently of our main develop branch.
TLDR: We'd love to see this functionality! 🎉

I can see that there was a little back-and-forth with this one. Guidelines are important. Everyone's time matters. I'll most definitely take a look.
So, CLI integration in one PR, followed by others that are on a per rule basis with tests? Make sense @DanPurdy ? Thanks for the quick info and the detailed answer, really helpful.

Bump to get this reviewed and merged? This would be perfect.

+1
I hate having to manually fix extra whitespaces or nextline issues..

+1

+1

+1

+1

Please stop adding +1s it doesn't help at all

+1

+1

Hi,
is there any advance for this issue ? PR is made already. So ETA for merging into upstream ?
cheers

+1

+1

New repo just for this https://github.com/srowhani/sass-lint-auto-fix


build shared lib only if BUILD_SHARED_LIBS is set

Signed-off-by: Hannes Harms<EMAIL_ADDRESS>

Hiya! Could I check for the reason for why you'd like to no longer be forcing SHARED library builds?

Tha change is rquired to build ROS2 as a static set of librarys using the standart cmake feature BUILD_SHARED_LIBS=off
With this change I can link rmw_dds_common a static lib and run ros2 applications without a ROS2 installation.

Got it! There's some internal discussion about this, and we can change it, though it'll necessitate some changes in upstream ROS libraries, so it might take a little while to resolve.

Bumping @hannes09

I have added ament_add_default_options, but now the build fails.

We need to do a release of ament_cmake.  I'll get to that a little later today.

@ros-pull-request-builder retest this please

I forgot ci.. my bad

Linux
Linux-aarch64
Windows


Advanced Output

Hi Ingo,
Thank you very much for this awesome script. It's just what I need.
Not being being a PowerShell Pro, I was wondering if you help me to adjust the out of your script as I do not need the message but fields from the message.
I have a PowerShell script that does it but I do not know adapt yours to have the same  output.
`$events = Get-WinEvent -FilterHashTable @{Logname="Security";ID="5140"} -ErrorAction Stop
Write-Output "TimeCreated, EventID, User, Domain, LogonID, LogonType, Source, ObjectType, ObjectName, HandleID, ProcessID, AccessMask" > C:$(Get-Date -f yyyy-MM-dd)-file_access_audit.csv
foreach ($event in $events) {
$event2=new-object psobject -Property @{
TimeCreated = $event.TimeCreated
EventID = $event.Id
User = $event.Properties[1].Value
Domain = $event.Properties[2].Value
LogonID = $event.Properties[3].Value
LogonType = $event.Properties[8].Value
Source = $event.Properties[18].Value
ObjectType = $event.Properties[5].Value
ObjectName = $event.Properties[6].Value
HandleID = $event.Properties[7].Value
AccessMask = $event.Properties[9].Value
ProcessID = $event.Properties[10].Value
ProcessName = $event.Properties[11].Value
}
if ($event2.User -NotLIKE  "*$")  {
Write-Output "$($event2.TimeCreated), $($event2.EventId), $($event2.User), $($event2.Domain), $($event2.LogonID), $($event2.LogonType), $($event2.Source), $($event2.ObjectType), $($event2.ObjectName), $($event2.HandleID), $($event2.ProcessID), $($event2.AccessMask)" >> C:$(Get-Date -f yyyy-MM-dd)-file_access_audit.csv
}
}`
Any chance you could help me?
TY in advance.
BR
Greg

Hi @ggriffinorg,
have you tried the switch -FormatOutput? All the information(properties) will be stored in the custom object in Data and are separated by "|". This could be easily used for importing into PowerBi or anything similar.
Ciao,
Ingo

Hi Ingo,
Thank you for the tip.
I will try it out.
BR
Greg


Enable and document customization

Enable to customize this installer and document how to build installers for custom Theia-based products.

We (EclipseSource) intend to work on this on behalf of ST Microelectronics.

We are currently working on a GoogleDoc to create the documentation on how to customize the blueprint product to build installers for custom Theia-based products.

We are currently working on a GoogleDoc to create the documentation on how to customize the blueprint product to build installers for custom Theia-based products.

We would like to contribute the documentation to the Theia webpage and opened an issue in the corresponding repo to discuss how and where to contribute it:
https://github.com/theia-ide/theia-website/issues/131

We would like to contribute the documentation to the Theia webpage and opened an issue in the corresponding repo to discuss how and where to contribute it:
https://github.com/theia-ide/theia-website/issues/131

Available here: https://theia-ide.org/docs/blueprint_documentation/


CI: ra_vfs and proptest are not cached

Because they get matched by:
rm -fr ./target/debug/{deps,.fingerprint}/{*ra_*,*test*,*gen_lsp*,*thread_worker*}

Consider https://github.com/rust-lang/cargo/issues/573#issuecomment-468793358 as a workaround.

wow, that's genions! It never occured to me to use -no-deps for this
use case.
On Sat, 20 Jul 2019 at 16:37, Laurențiu Nicola<EMAIL_ADDRESS>wrote:

Because they get matched by:
rm -fr ./target/debug/{deps,.fingerprint}/{ra_,test,gen_lsp,thread_worker}
Consider rust-lang/cargo#573 (comment)
https://github.com/rust-lang/cargo/issues/573#issuecomment-468793358 as
a workaround.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rust-analyzer/rust-analyzer/issues/1565?email_source=notifications&email_token=AANB3M25NB4PQZXNMJ65XALQAMIIBA5CNFSM4IFORRO2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HAN6BFA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANB3M27WVNI74OPRV7AQC3QAMIIBANCNFSM4IFORROQ
.

Hmm, cargo clean -p $(cd crates; ls)?

let me try this out...

I think it should be:
for crate in $(cd crates; ls); do
cargo clean -p $crate
done

Let's see how https://github.com/rust-analyzer/rust-analyzer/pull/1567 looks like

fixed


Is it possible to replace cookie sessions with localStorage?

I am working on an universal app and we cannot depend on cookies. Is it possible to have the user session saved in local storage instead of a cookie?
I know localStorage cannot be used on the server side, so perhaps, the user session could be loaded after server side rendering and before client side rendering. Would this be ok?

You may want to refer to issue #608 discussing authentication using cookies vs localStore. It has some links about why you should use cookies for storing authentication tokens. Also it shows an implementation using the react-cookie library which allows you to use cookies on the server side.
You can definitely use localStorage but in order to have the server rendering the same thing as the client, you need to get anything in your localStorage needed for rendering up to the server somehow - and cookies are the logical way. If you want to store a lot of stuff in localStorage for rendering you'll probably have to give up on accurate server side rendering of those parts (but maybe that's ok for you - you only need server side rendering for SEO purposes and speedy initial page loads).
Out of curiosity, why can you not depend on cookies?

Very nice answer @adailey14 , thank you.
Because I am looking to have authentication routes that work for a web and mobile app. For what I have read so far, I understand that we cannot use cookies in mobile apps, am I wrong?

@Dindaleon Yeah I think cookies in a mobile app are a tricky proposition - not recommended. Personally I am planning to use a react-native implementation for a mobile app, rather than trying to use this project directly within a mobile app. I suspect there will be a number of tricky pieces to get this code to work right as a mobile app, considering the code would have to run in a server side environment, a browser, and a mobile app. I wonder if anyone else has tried that yet?
I'm thinking you would end up wrapping this code in an app container of some kind, like phonegap, or your own custom container. The client side code would be downloaded with the app, so the app would only over make API calls. I think this would mean you would never trigger a server side render from your mobile app, right?
So in that case... I think you would be clear to just use localStorage and include the auth information in your api call headers. So probably you would want your code to detect if it's in the mobile app vs a browser, and use a purely localStore strategy for the mobile app, but include a setting a cookie strategy in the browser to allow server side rendering to work. Maybe that 'detection' should happen at build time.
If you're not doing the 'container' route, and instead your 'app' is really just a wrapped web browser that will be pulling all the code from the server, then you would need to do something where you copy auth info from localStorage into a cookie before each request so you get server side rendering to work serving your app.
These are all just guesses, I don't know if there is a best practice here. Would love to hear if anyone else has experience doing any of the above.

@adailey14 I am planning on using react-native as well. I migh just create a new route for the mobile app that only uses localstorage. I am working with a hapi server.


what the TdApi.CheckAuthenticationPassword mean?

i have downloaded and run java example project. when I typed the phone number and the verification code, the below code is executed.
case TdApi.AuthorizationStateWaitPassword.CONSTRUCTOR: {                 String password = promptString("Please enter password: ");                 client.send(new TdApi.CheckAuthenticationPassword(password), new AuthorizationRequestHandler());                 break;             }
but I enter any password, the error returned.
Error {   code = 400   message = "PASSWORD_HASH_INVALID" }

The error means that a wrong password was entered.

ok. it's my fault


Configure CI to publish UI toolkit package on tag

Publish UI toolkit to @vegaprotocol npm organisation

@ciaran- Added you as an assignee to this as it appears that you may do this as part of lp dashboard work. If that is not the case please reassgin to me.

@dexturr Yep makes sense - happy to pick this up today!
Did you have any preconceptions about how you want this to happen that you think I should know?
Otherwise I'll just crack on and see what I can get working (:

Noting this compatibility issue with the built artifact from ui-toolkit when trying to add to the website project:
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=16.15.1". Got "16.14.0"

Another one: vega.xyz is using react 17 - monorepo is using 18. Bumping the website build to 18 doesn't seem to have caused any problems locally, so gonna suggest we try going forward with that but will check with the website team before we do.

Also - how do we feel about the volume of peerDependencies? There's quite a large number of them just from using ui-toolkit to import a single type. Do we think that's OK?

Also - how do we feel about the volume of peerDependencies? There's quite a large number of them just from using ui-toolkit to import a single type. Do we think that's OK?

I think this was me misunderstanding how the nx output works. We can elimate these with a flag: "buildableProjectDepsInPackageJsonType": "dependencies" in each of the project.json files.


Config valitation for  "Mail" rejects long TLDs

There are some TLDs that are longer than the maximum of four letters. This leads to a failed config check.

Issue #55 fix this bug


timestamp conversion issue

Hi,
When we define a field with type = timestamp, it is created into DB as "timestamp without time zone".
While creating a new entry from the UI side using the OData model, it passes value in TZ format in the batch request which is saved in DB without a time zone.
In a case where there is a timezone different then UTC, it is saving wrong values.
createdat and modifiedat are just showing wrong values when Application is running on other than UTC time zone.
Thanks,
Rajdeep Bhuva

Hi @rbhuva,
it would help if you could provide the sample OData request or maybe directly send a pull request enhancing the unit tests.
Best Regards
Gregor

Hi @gregorwolf,
I have added the fix with the test cases on the pull request: https://github.com/sapmentors/cds-pg/pull/243
Kindly review the pull request.
Thanks,
Rajdeep Bhuva

Hi @rbhuva,
thanks for working on this topic. During our tests we also found that the "milli-seconds" were swallowed up by
const convertToISO = (element) => {
return new Date(element + 'Z').toISOString()
}
Example:
data written: 2012-12-03T07:16:23.574Z
date read:  2012-12-03T07:16:23.000Z   (output of convertToISO)
Kindly check if a "loopback" test might be added to the test implementation (just to ensure that both values match). Something like:
  test(' -> Timestamp', async () => {
    const value = '2012-12-03T07:16:23.574Z';
    const response = await request.post('/beershop/TypeChecks').send({
      type_Timestamp: value,
    })
    expect(response.status).toStrictEqual(201)

    const verify = await request.get(`/beershop/TypeChecks(${response.body.ID})`).send()
    expect(verify.body.type_Timestamp).toStrictEqual(value);
  })

Hi @narmbruster,
As per your suggestion, I have added the loopback test. Both values are matching along with milliseconds.
Regards,
Rajdeep Bhuva

Thank you @narmbruster for the improvement of the PR from @rbhuva. I've created a new version and will close this issue.


ARROW-11440: [Rust] [DataFusion] Add method to CsvExec to get CSV schema

I needed this in Ballista so that I could implement serde for the CsvExec operator.

Codecov Report

Merging #9377 (c1b0832) into master (f05b49b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #9377   +/-   ##
=======================================
  Coverage   81.94%   81.94%
=======================================
  Files         231      231
  Lines       53374    53378    +4
=======================================
+ Hits        43739    43743    +4
  Misses       9635     9635

Impacted Files
Coverage Δ

rust/datafusion/src/physical_plan/csv.rs
73.79% <100.00%> (+0.74%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #9377 (c1b0832) into master (f05b49b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #9377   +/-   ##
=======================================
  Coverage   81.94%   81.94%
=======================================
  Files         231      231
  Lines       53374    53378    +4
=======================================
+ Hits        43739    43743    +4
  Misses       9635     9635

Impacted Files
Coverage Δ

rust/datafusion/src/physical_plan/csv.rs
73.79% <100.00%> (+0.74%)
:arrow_up:

Continue to review full report at Codecov.

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


Deps upgrade

Coverage increased (+0.006%) to 99.576% when pulling c970a35d72b0608e76dab0be5d7692fcee3e35cb on ddolcimascolo:ncu into 61ce361f374b7c9053af234441ce0c98356bd6b0 on linagora:master.

Coverage increased (+0.006%) to 99.576% when pulling<PHONE_NUMBER>b2fb00ed886904286d5911496ecd25 on ddolcimascolo:ncu into 61ce361f374b7c9053af234441ce0c98356bd6b0 on linagora:master.


Revert "fix(core): improve NonReadonly type (#1237)"

This reverts commit 5b76e3b61261bc8563f656e1f847b67c4ac833a8.
Fixes #1348
Status

READY
Description
Readonly properties should be excluded from request parameters

@ezequiel can you also review this PR: https://github.com/anymaniax/orval/pull/1354


support kubernetes v1.8

Gets rid of the deprecated --api-servers flag in favour of using kubeconfig. This is the minimum change required to run a v1.8 cluster. Any future improvements can be made in another PR :)

LGTM

I'm super busy at the moment and don't have time to really test a cluster. I'll defer to you guys.


Built & test on debian:buster

Due to some bug in the gcc version that ships with debian:buster, this fails to build.
It is currently not a very important goal to support older releases and so I will close this. We might revisit it in the future, trying to work around the issue, but given all the areas in which we have a fair bit of work cut out for ourselves, it's not very likely.


Update variable name substitution logic

Account for names that would conflict with ARC (prefix is "alloc",
"new", ... )
Replace characters with text equivalents ( "+" -> "plus", "%" ->
"percent", ...)

TODO: This is a shorter term patch and we should strive to have a more
robust substitution policy across languages.

@maicki - Updated:

Added a test fixture for these cases in Examples
Updating rendering code to be stable (order of alloc , copy, ... properties weren't stable in header before)
Removed code around +, %, etc since I think we should not treat those as valid schema property names

Fixes https://github.com/pinterest/plank/issues/138


A concept: Include scons-local in the repo

From SCons' website:

The scons-local package is one that you can execute standalone, out of a local directory. It's intended to be dropped in to and shipped with packages of other software that want to build with SCons, but which don't want to have to require that their users install SCons.

Granted, this would only really benefit Windows users (though I imagine those make up at least a chunk of Godot users), since installing SCons is trivial everywhere where an actual Unix shell & package manager is available, but it would help, and not bloat the repository itself by that much either (the zipped scons-local package is about ~640kB, and SCons itself is MIT-licensed so there wouldn't be an issue with including it that way either).

I think it's a bad idea, see https://github.com/godotengine/godot/pull/12843#issuecomment-343761752 for details.


Deprecate one page category and add some others

As part of some content changes we want to modify "Enforcement Action" pages that currently have category "Administrative Filing" and instead set them to have a new category "Stipulation and consent order".
We also want to create a new "Enforcement Action" page category "Administrative adjudication", as well as a new category page type for "Administration adjudication docket" that also has distinct "Administrative adjudication" and "Stipulation and consent order" categories.
This PR defines those new categories and migrates existing pages.
@sarahsimpson09
Additions

New "Enforcement Action" page categories "Administrative adjudication" and "Stipulation and consent order".
New "Administration adjudication docket" category page type with distinct "Administrative adjudication" and "Stipulation and consent order" categories.
New unit test that verifies that all page categories have unique keys.

Removals

Deprecated "Enforcement Action" page "Administrative Filing" category.

Testing

Using a production data dump, run the migration in this PR and note that categories have changed on appropriate pages.

Screenshots
New categories as they appear in the Wagtail admin:

New filterable list category options:

Checklist

[X] PR has an informative and human-readable title
[X] Changes are limited to a single goal (no scope creep)
[X] Code can be automatically merged (no conflicts)
[X] Passes all existing automated tests
[X] New functions include new tests
[X] Reviewers requested with the Assignee tool :arrow_right:

@chosak This looks good! One thing-- can we make the existing Enforcement category sentence case while we're at it?
Federal District Court Case --> Federal district court case

@sarahsimpson09 sure, I added that change in b66534f:


Fixing auth

Description of Changes

Fixing an issue where the sha1 threw a 401 because it was missing par…t of the string
Related Issues

Fixes #13

Codecov Report

Merging #14 into master will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   94.11%   94.11%
=======================================
  Files           4        4
  Lines          34       34
  Branches        2        2
=======================================
  Hits           32       32
  Partials        2        2

Impacted Files
Coverage Δ

src/api/webhook/repository.ts
88.23% <0.00%> (ø)

Continue to review full report at Codecov.

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


Extract Generic Database Helpers to its own Library

The core of the code that we use to do database migrations and delta detection doesn't change very often, and the tests for that code are slow. Let's pull all that code into a new helper library (I'm nominating a new library or two in the Baseline repository).
When we do this, let's try to make as much as the coupling as possible to DbConnection instead of NpgsqlConnection as a preparation to taking Marten cross database in a possible v5 release
What should be pulled out

CommandBuilder -- Jasper has its own copy of all this code
ADO.Net extension methods that could easily be useful outside of Marten
ISchemaObject and most of its implementations
Code like FunctionDelta and TableDelta that do database structure difference detection
DdlRules -- I'm betting that almost nobody but the original shop that wrote Marten cares about this, but still

Tasks to finish

[ ] GH action to build and publish Weasel.Postgresql
[ ] Move ISchemaObject and some of the extension methods to the main Weasel library
[ ] Try out all the patch command options in the CLI. Make sure there's at least smoke test coverage
[ ] Hunt down [Obsolete]
[ ] IDbObjects needs to be async all the way down
[ ] IDocumentSchema needs to be async all the way down
[ ]

GH action to build and publish Weasel.Postgresql

@jeremydmiller I will take a crack at this once we are ready to publish a package to NuGet.


 This is because the child process is a terminal provider, which invokes ts-node or tsx, which in turn invoke other processes. All these processes dont get killed when the main process gets killed.
Solution:
Precompile the scripts on change using esbuild, so that we just launch node, which we can then kill

Fixed on Linux by spawning with shell just on windows

fixed by using esbuild for compilation on file-change and using fork(...) when running a script.
TODO: Display error when compilation failed

finished todo with commit 4fbc2bd


revise_01

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions

Description of the Change

Alternate Designs

Benefits

Possible Drawbacks

Applicable Issues

几个原有的题型格式可能因为对齐出错，无法在atom上插入题型格式了
麻烦帮我确认下有没有改对
MCP/MCQx/explanation/pretextteaching/c2e/matching

@dongliang1993 请看一下这个commits的语法