Using it with C++

Hi,
Any good way to use this library wrapped by a C++ class? I'm having troubles finding ways to register callbacks that are class members and couldn't come up with any good workarounds (I'm not experienced with C++) . Any suggestion is much appreciated.
Thanks.

I created a simple wrapper over your library. So far it suits my needs, maybe it helps someone else as well: https://github.com/candale/esp_mqt_arduino_wrapper

i will add to sample sections in README file,
thanks


The title “Voluntari” is misaligned

URL: https://oportunitatisicariere.ro/
Browser:  Chrome
Device: IPhone pro 13
OS: iOS
Steps to Reproduce:
Open the URL.
Go to the "Echipa" section
Verify the graphics and their titles.
Observe the alignment of the titles.
Actual Results
The title “Voluntari” is misaligned compared to the graphic.
Expected Results
All the titles are well alligned

Hi @DeedeeDL , the issue was fixed, can you please check if everything works well now?
Thank you
CC: @porobertdev

Hi @DeedeeDL , the issue was fixed, can you please check if everything works well now? Thank you CC: @porobertdev

@DeedeeDL So I suppose everything is fine?

@DeedeeDL from frontend perspective the bug was fixed.
Can you please re-test it?
Thank you

retested. passed. good job !


chore: update axoasset to get zstd fix

yeah we entirely promised zstd, and thought we were doing zstd, and then, just, didn't do that in the impl.


Is there any way to force navigation when clicking on a router-link even if the route doesn't change?

What problem does this feature solve?
Reload current page's state or refresh the page.
For example, I have a <HeaderBar> which contains a Home button, just like the Facebook one.
If I'm in another page (rather than Home), when I hit this button, it will navigate to Home. But when I'm in Home page, hit this button it does nothing.
It's expected to reload the page, and scroll to the top, right?
Please don't tell me to write out an action to refresh the state and scroll to top. This header bar is a shared component, and the Home button can be hit at any where.
What does the proposed API look like?
Some thing like
<router-link :to="{ name: 'Home', force: true }">Home</router-link>

or
this.$router.replace({ name: 'Home', force: true });

@posva
If so, Navigation Guards (beforeRouteLeave, beforeRouteUpdate) will not be triggered.
How do you deal with that?

yeah, they are not supposed to be triggered as you are in the same route with no param change or whatsoever... You need to refactor the code in methods and call them in both places. Cheers

[Solved]
This is my solution, if any body needs.
I create a base component, and use it instead of <router-link>.
If you hit the link when you're in that page already, it will create a query name _, so the fullPath will be change and the page will be "reload".
You can change the query name to whatever you want, and use the timestamp or some unique string to make the fullPath changes.
<template lang="html">
  <router-link v-bind="$attrs" :to="to" @click.native="clickHandler">
    <slot></slot>
  </router-link>
</template>

<script>
export default {
  name: 'BaseLink',

  props: {
    to: {
      type: [Object, String],
      default: '/',
    },
  },

  data() {
    return {
      currentRoute: null,
    };
  },

  created() {
    this.currentRoute = this.$route.name;
  },

  methods: {
    clickHandler($event) {
      if (!($event.ctrlKey || $event.metaKey) && this.currentRoute === this.to.name) {
        const query = { ...this.$route.query, _: new Date().getTime() };
        const newRoute = { ...this.$route, query };
        this.$router.replace(newRoute);
      }
    },
  },
};
</script>

I doubt this method, it looks dirty and contaminate the history (and generate no real-meaning query in the url).  To be clear, you don't want a "real, F5 like" refresh since your script have to be reload and the browser have to reparse it, right? (The OP's solution is in that direction.)
More simple way is to assign an unique key in <router-view /> using one of the properties stored in the params. Manipulate this instead of query would be better.

@leoyli
Yeah, I've tried to bind a key to router-view before, but it didn't work.
<router-view :key="$route.fullPath"/>

Since the fullPath have no changes, how does it suppose to reload the page?
Can you give me an example, plz. Talk is cheap, you know.

You can have something like
<router-view :key="$route.params.reload"/>

And you manipulate like:
<router-link :to="{ params: { reload: Data.now } }" replace/>

Remember to is passing to router.push(), but doing that, it is more declarative and controllable. I'm using this to decide if the page of component should be rerendered or not since they will based on id params obtained from URL entry, and my child component can still using nesting <router-view>.

@leoyli
Did you try it in actually?
It doesn't work. The theNameYouGive doesn't make the $route change, it never refreshes the page.

@trandaison
Sorry my bad.  Looks like <router-link> can only change the params when it assigned together with name and this name can not be the same as the current route; or in your route definition you have :param bound with the url.
That is to say in any cases URL have to be change unless you assign key to an internal state (data) and you bind the property name in the key attribute, e.g. <router-view :key="counter">, and use @click="counter++" to update the value in the component.

Try: npm i vue-state-router instead of vue-router. Totally same API.
Gives you these feature:

when navigate with push mode, every component will be reload, and scroll back to top
when with replace mode, every component will reuse, and scroll to saved position

I've just ran into this problem also.
Use case:

Router setup as /profile/:userId to a component called "Profile"
On the profile it loads the data from a api call, however let's say I have a "Friends" tab on the page and I click their profile :to profile with the userId params
It won't load because the component is already loaded

Same problem.
Resolved : click link profile event () => window.location.href= '/profile/${userId}'

I don't agree with closing this issue essentially based on the assumption that everyone uses router the same way and without many of the more complex side-features.
I have run into this and I am not trying to reload the page; I am trying to move focus to a particular part of the page based on a route parameter but I want it to happen when the parameter is 'called' (ie a button clicked) even if focus has already been moved there.
I can code around this, but it makes the code significantly more mangly and odd.

@posva is there a technical limit? I do not see how this would not be the desired default effect.

@cafe4it but doing so your app is not SPA anymore, the whole app will be reload again.

This is my trick to get through this situation:

Create a component named <base-link> just likes the <router-link> and use this one instead of <router-link>.
If the :to route the same as the current route, render <router-link> in template.
Else, render a <a> with :href=this.$router.resolve(this.to).route.fullPath and add an click event handler to redirect to another page named 'Refresh';
In the Refresh page, replace the route back to the referer.

Everything works just fine so far.

I have a nice trick. It works with nuxt-link from Nuxt.js, but it can also work with router-kink I believe.
First, you register a click event on your link component
<nuxt-link to="/home" @click.native="onClick" />
After that, you check in the click handler if you are on the same route
if(this.$route.path === '/home') {
    // do some staff here
   // example
   window.scrollTo({ top: 0, behavior: 'smooth' })
}

The condition will be true only when you click the navigation link for the same route

I'm interested in having some sort of functionality here that does this natively.
I want the vue-router to think of the URL like a form-input. So I can render text into it without triggering changes that depend on the URL. At the sametime the URL is a control input.

I'm interested in having some sort of functionality here that does this natively.
I want the vue-router to think of the URL like a form-input. So I can render text into it without triggering changes that depend on the URL. At the sametime the URL is a control input.

@posva Can this be re-opened? There seems to be a lot of community support for a option to force a re-render. Please see the use case I shared above, I would love the option to force without the additional complexity of adding a watcher to accomplish something so simple. Thanks in advance.

@posva Can this be re-opened? There seems to be a lot of community support for a option to force a re-render. Please see the use case I shared above, I would love the option to force without the additional complexity of adding a watcher to accomplish something so simple. Thanks in advance.

I added a query param that includes current time as milliseconds and it's working for me.
this.$router.push({ name: 'post', params: { type: 'detail' }, query: { id: 123, time: Date.now() } });

I added a query param that includes current time as milliseconds and it's working for me.
this.$router.push({ name: 'post', params: { type: 'detail' }, query: { id: 123, time: Date.now() } });


How do I get the logs after a disconnect?

Did you check the docs?

[X] I have read all the vscode-neovim docs
[X] I understand the fundamental limitations of the plugin and believe this request to be reasonable

Is your feature request related to a problem? Please describe.
I am experiencing random disconnects when working on dev containers and jupyter notebooks. My issue is that when that happens, the output logs disappear. Is there a way to recover them?
For reference, I look at the logs like this:

View the logs via Output: Focus on Output View and select vscode-neovim
But (obviously) the vscode-neovim section does not exist anymore post disconnect

Describe the solution you'd like
A link to a log file in the disconnect pop up
Describe alternatives you've considered
None
Additional context
No response

You can see where the files are stored by clicking "..." in the output window:

That should give you an idea of where logs are stored.

A link to a log file in the disconnect pop up

vscode extensions have no control or visibility into the "disconnect" event, AFAIK.


Bucket Notification Trigger functionality broken - missing some functions and validations

Bug Report
Issue
The bug is related to the bucket notification trigger functionality for Python.
Code referenced: https://nitric.io/docs/reference/python/storage/bucket-on
Bug 1:
In the code snippet shown below, the second argument to the decorator doesn't seem to be having any validation.
Any random value can be supplied to the second argument and it works.
This does not seem to be an expected behaviour. Only a valid path should be allowed.
@assets.on("write", "/images/cat")
async def create_cat_image(ctx):
  print(f"A cat image was written")

On checking the codebase, there doesn't seem to be any checks for notification_prefix_filter. The filter is most likely meant for filtering events containing the provided key (in form of the second argument).

Expected
The intended behaviour has already been described above.
Nitric CLI Version:
1.37.6

Thanks @g-greatdevaks
I've written up a PR to add on to BucketRef here: https://github.com/nitrictech/python-sdk/pull/136
For Bug 1, what type of validation are you expecting? Is there are particular character set  that you would expect this to be limited to?
If we impose validation on the prefix, we'll likely implement it in a decorator for the nitric providers, in both the nitric CLI and core repositories so we can apply the same validation to all language implementations.
I'm assuming the kind of validation you're expecting would be some subset of the following:
https://docs.aws.amazon.com/AmazonS3/latest/userguide/object-keys.html
https://cloud.google.com/storage/docs/objects
https://learn.microsoft.com/en-us/rest/api/storageservices/naming-and-referencing-containers--blobs--and-metadata#unicode-characters-not-recommended-for-use-in-container-or-blob-names
Let me know if thats in line with what you were thinking.

Thanks for the prompt response, @tjholm 💯
For Bug 1, I am a bit unclear about the functionality of the prefix filter key (the second argument).
Two types of validations I was thinking about (based on my understanding that the second argument is meant for such checks), as stated below:

Object name validation to match with the standards, as you rightly highlighted.
A check on the defined path (as per the example on the official document, the path looks similar to the hierarchy where the object is going to be stored in the Storage Bucket) - whether the path is valid or invalid.

@g-greatdevaks in the file/object stores we support objects are typically stored in a flat structure within their buckets, essentially they're key/value pairs. While using paths e.g. image/latest/pic.png is useful for organization and using the bucket as if they were a traditional file system, that's just a convention. There is a good write up of this in the Google Cloud Storage docs.
The way the prefix filter works is essentially a "starts with" check on file events. Any files that match the desired event type (e.g. "write" for first write) and has a key that starts with the provided prefix will match and trigger the function.
@tjholm given the object stores can be quite permissive with valid key values, I suspect the only safe limitation we could put on the prefix filter would be preventing strings containing newline/carriage characters. Most other characters appear to be valid in one store or another.

We found an issue with the prefix filtering on local environments which I expect caused the confusion, they were effectively not working. That's now fixed in the CLI. We've also added the missing .on function to the BucketRef like in other SDKs. If you try again now with CLI v1.38.1+  and Python SDK v1.0.4+ these issues should be resolved.

Thanks for the prompt fix, Jye.

Thanks for fixing it.


[FEATURE] Is it possible to get dashboard where we can see particular bot token usage

:-)

you can use analytic providers like langsmith, langfuse for that


Align checkboxes & radios

Cross browser aligning checkboxes and radios is hard because every browser seems to have different widths/height and align possibilities. This misaligns our checkboxes and radios in different ways.
I found a solution for this by setting the height of the input to the line height. This won't stretch the input but instead align it to the middle of the element which will perfectly align checkboxes and radios in every browser.
The only disadvantage is that the line height will depend on the line height of the body, but I think that's fine in almost any situation. The current situation is way worse imo.
Before:

https://getbootstrap.com/docs/4.3/components/forms/#checkboxes-and-radios
https://getbootstrap.com/docs/4.3/content/reboot/#forms
https://getbootstrap.com/docs/4.3/components/input-group/#checkboxes-and-radios

After

https://deploy-preview-28711--twbs-bootstrap.netlify.com/docs/4.3/components/forms/#checkboxes-and-radios
https://deploy-preview-28711--twbs-bootstrap.netlify.com/docs/4.3/content/reboot/#forms
https://deploy-preview-28711--twbs-bootstrap.netlify.com/docs/4.3/components/input-group/#checkboxes-and-radios

Closes #27755

Can we look at putting this in https://github.com/twbs/bootstrap/pull/28450? I haven’t had a ton of time to revisit that draft PR, so it might be gnarly, but it’s my intent to ship that in v5 and combine these two sets of components. I didn’t think to make them explicitly linked to line-height, but that PR got closer than the current ones in v4.

Can we look at putting this in https://github.com/twbs/bootstrap/pull/28450? I haven’t had a ton of time to revisit that draft PR, so it might be gnarly, but it’s my intent to ship that in v5 and combine these two sets of components. I didn’t think to make them explicitly linked to line-height, but that PR got closer than the current ones in v4.

I didn’t think to make them explicitly linked to line-height

It's not what I really prefer, but this seems to be the only solution to properly align the native checkboxes.
This PR will have merge conflicts with #28450, maybe I shall just move the changes in reboot to that PR?

Gonna close this since this PR introduces some issues when focus states are applied.


ImageGatherExtended does not appear to work

MoltenVK reports shaderImageGatherExtended as true, but using it seems to cause an error:
00:00:11.985 |E| .NET ThreadPool Worker [mvk-error] VK_ERROR_INITIALIZATION_FAILED: Shader library compile failed (Error code 3):
00:00:11.985 |E| .NET ThreadPool Worker program_source:334:48: error: no matching member function for call to 'gather'
00:00:11.985 |E| .NET ThreadPool Worker     float4 _68 = spvDescriptorSet2.fp_t_tcb_16.gather(spvDescriptorSet2.fp_t_tcb_16Smplr, float2(_63, _65), _1164, component::x).xyzw;
00:00:11.985 |E| .NET ThreadPool Worker                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~
00:00:11.985 |E| .NET ThreadPool Worker /System/Library/PrivateFrameworks/GPUCompiler.framework/Versions/32023/Libraries/lib/clang/32023.21/include/metal/metal_texture:5164:24: note: candidate function not viable: no known conversion from 'const constant spvUnsafeArray<int2, 4>' to 'metal::int2' (aka 'int2') for 3rd argument
00:00:11.985 |E| .NET ThreadPool Worker   METAL_FUNC vec<T, 4> gather(sampler s, float2 coord, int2 offset = int2(0), component c = component::x) const constant METAL_CONST_ARG(c)
...

Looking at the latest MSL spec, I don't see any overloads that handle providing a fixed array of offsets. I also don't see anything special in spirv-cross to emulate this.
This used to "work" in a previous version, but I don't know if it was just broken before or actually worked. The commit where things changed might be https://github.com/KhronosGroup/SPIRV-Cross/commit/1047c13d40a2fccd5198ad8992618af2dab0303f, since that's the only thing that changed around texture gather at that time.

It was broken, because it wasn't passing down the offsets at all.
There is no support for this in Metal. While it would be easy to emulate by doing four read()s, the problem is handling sampler wrap modes, which from my reading of the Vulkan standard affect the operation of the OpImageGather SPIR-V instruction and, indeed, Metal's gather() method, but which don't affect read(). We do not have this information when the shader is compiled, and there's no way to get it from the sampler.

You can center the coordinate on a texel on a regular texture sample instead of using a texel fetch to still respect the sampler wrap mode. Regardless, this feature should be reported as false if it’s not supported.

Regardless, this feature should be reported as false if it’s not supported.

The reason that shaderImageGatherExtended is enabled, is that it covers two features, one of which is supported by Metal, and the flag is enabled to allow support for that.
Eventually, the other will be covered by the final version of the Metal Portability extension. See this discussion for more info.


zine: 0.12.1 -> 0.13.0

Changelog: https://github.com/zineland/zine/releases/tag/v0.13.0
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 220857 run on x86_64-linux 1

1 package built:

zine


Are tiles cached?

When viewing an image with OSD and zooming in then panning around, are the tiles cached so if you were to go back to a recent zoom level and position it would already have it?  Or is it a fresh get every time?
Thanks guys.

Tiles are cached: https://openseadragon.github.io/docs/OpenSeadragon.TileCache.html

Thank you @WilliamCarlos


#873: the accuracy issue of ObjectSampler in --total mode

Hi Andrei
As discussed in #873.
Because JEP 331 doesn't return the actual pseudo-random interval in JVM, the event._total_size is the best we can use now.
I would appreciate it if you can help review this.

Merged. Thank you for the contribution.


remove karroffel from CODEOWNERS file

This PR removes me from the CODEOWNERS file.
I have not been using Godot personally for a while and I will soon be short on free-time because of studies, so I think removing me from the CODEOWNERS file and looking for another person to maintain GDNative is the best solution for both me and the Godot project.
Thanks to all the amazing people in the Godot developer community! :blue_heart:

It's sad for the Godot community to lose such a maintainer, but all good things come to an end and I'm thankful that you make things clear, so that we can encourage new contributors to step up to maintain GDNative.
Thanks a ton for all the work you've done so far on GDNative, GLES2, GDScript and many other areas over the past few years. There's a lot of @karroffel code in this repo and we'll take good care of it :)
Have fun with your new studies, and feel free to drop by anytime for a jam, a small contribution or just a chat :)

@karroffel I hope to see you anyway in Fosdem/GodotCon, for me Godot is first a community to have beers with 😃


[Mime] Remove @thi.ng/api from dependency?

I just looked at the code and am I missing something cause I'm not seeing it being referenced or used at all.
Maybe remove it to trim dependency requirements a bit more? :)

The api package is used/needed during compile/generation time, moved it to devDependencies...


Import components inside setup

Clear and concise description of the problem
Quite often there is a need to specify types for a component's ref variable.
To control behavior, such as a toggle, or receive some data.
Simplest example:
MyComponent.vue

<template>

hi

</template>

App.vue

<template>
  <my-component ref="myComp" />

  <button @click="myComp?.sayHi()">Say Hi</button>
</template>

Current behavior

Project with reproduction here
Suggested solution
https://github.com/antfu/unplugin-vue-components/pull/562
Closed by author 🤷♂
https://github.com/antfu/unplugin-vue-components/issues/527
Seems valid solution
Alternative
No response
Additional context
Related issue
https://github.com/antfu/unplugin-auto-import/issues/118
But it was closed without solution
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

相同的: https://github.com/antfu/unplugin-vue-components/issues/527
我有一个其他的想法：
创建一个 componentInstance.d.ts
import type { GlobalComponents } from '@vue/runtime-core'

declare global {
  type ComponentInstance = {
    [Property in keyof GlobalComponents]: InstanceType<GlobalComponents[Property]>
  }
}

Closed.
Simple and nice solution provided https://github.com/antfu/unplugin-vue-components/issues/601#issuecomment-1437848252

兄弟们真厉害解决了66666


Tweak artifact build and wire to workflow

Issue #, if available:
Description of changes:

Use ldflags to set version natively with Go instead of script
Puts all archives in build/dist instead of mixing
Uses xray for executable name for every archive

This could be considered too breaking to go forward with

Add architecture to zipfile
Swap xray.exe and xray_service.exe, which I think is correct?

This could be considered breaking, but really it looks like the current names are just mistakes right now

In a followup I'll redo the Dockerfile to read built artifacts if they are available instead of rebuilding, so we only build once during a release.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Build folder looks like

One other question actually - so the old binary names (xray_mac and the old windows ones) would be named the same in the old ZIPs, or those would be updated immediately? If the former, I guess we'll have to do some renaming of the binaries after building them to have a duplicate for the old & new names.

@willarmiros Old zips should have old names. So yeah, the repackaging for deprecation process will be doing some copy/rename of zips, other extract-rename-recompess.


Segmentation Fault core dumped after closeing multiple connections

Hello,
I am writing a prgram that connects to multiple plcs at once. For this i have create a class. I wanted to implement a restart functoion in wich i would stop all the plc (.close() ) and then i would reconnect the all (.open()) but after i close the first connection i get a segmentation fault (core dumped).

SPKAAL and GR17A are names of the plcs

This is likely linked to #256 and i think is an issue with the ADS library on linux. You may have to find a workaround or may not be able to use linux to connect to multiple systems.


fix(docs): Begin a "Troubleshooting" section

Troubleshooting issue added based on #97

Pull Request Test Coverage Report for Build 579

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.762%

Totals

Change from base Build 576:
0.0%

Covered Lines:
217

Relevant Lines:
219

💛  - Coveralls

Hey @Corim 👋!
What a great idea of adding the Troubleshooting section. I love your addition. The only thing I would change, if you don't mind, is the last sentence; I wouldn't say the response becomes invalid as I believe this might paint the wrong picture but rather that GraphQL sticks to its definition.
I am not sure whether we want to keep this as short as possible and only present the solution or genuinely acknowledge what happens behind the scenes. Maybe the following comment can better portray what I have in mind;
https://github.com/maticzav/graphql-shield/issues/126#issuecomment-416524581
Tell me what you think! 🙂

I just wanted to start by saying I think you’ve done a great job with this project!
I do agree that the last sentence should be changed.
In regards to the solution length, I think a clear, concise answer helps keep out the noise, but I also think your explanation in #126 is tremendously helpful. I’m not sure how taboo it is, but a temporary solution - resulting in the fastest buildup of a robust troubleshooting section - might be to link to the issues (in this case #126 and #97) that already have explanations and examples. These links could be changed in the future to reference a blog post or document containing a lengthy and detailed answer.
I wouldn’t say this is the “correct” method, but linking to the issues that already have written explanations would definitely be an effective method to utilize work you’ve already done (until more time can be put into writing up comprehensive answers).
Going forward, a few other troubleshooting issues that come to mind relate to caching and "trying to use the permissions system as a filter" (which obviously doesn’t work, but it did take me some time to realize why)
As a newbie, I’ve made all the now-obvious mistakes. I will commit these changes, let me know what you think.
Thanks for all the work you've done!

@Corim, this is such an amazing PR! I’ll merge it right away; thanks for addressing suggestions from above.
I hope you add all the issues you can think of, let me know if you need any help!
Thanks again 🎉

:tada: This PR is included in version 3.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


:triangular_flag_on_post: Handle .no_exist files in try_to_load_from_cache

The presence of a file in .no_exist cache folder means the file does not exist in the repo. We should leverage this information in try_to_load_from_cache helper.
Related: https://github.com/huggingface/transformers/pull/18871 and https://github.com/huggingface/huggingface_hub/pull/986.

Fixed by https://github.com/huggingface/huggingface_hub/pull/1039.


Uf 6991 lift camunda version

Acted on PR-comments

Maybe not 100% in scope, but still:
https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&id=Sundsvallskommun_pw-parking-permit

Maybe not 100% in scope, but still: https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&id=Sundsvallskommun_pw-parking-permit

Fixed!


[BUG]  Not able to scrape Metrics of all instances

Is there an existing issue for this?

[X] I have searched the existing issues

YACE version
0.13.7
Config file
config: |-
apiVersion: v1alpha1
discovery:
jobs:
- type: AWS/RDS
regions: [ap-south-1]
period: 300
length: 300
dimensionNameRequirements: [DBInstanceIdentifier]
metrics:
- name: CPUUtilization
statistics: [Maximum]
- name: DatabaseConnections
statistics: [Sum]
- name: FreeableMemory
statistics: [Average]
- name: FreeStorageSpace
statistics: [Average]
- name: ReadThroughput
statistics: [Average]
- name: WriteThroughput
statistics: [Average]
- name: ReadLatency
statistics: [Maximum]
- name: WriteLatency
statistics: [Maximum]
- name: ReadIOPS
statistics: [Average]
- name: WriteIOPS
statistics: [Average]
- type: AWS/ElastiCache
regions:
- ap-south-1
period: 300
length: 300
metrics:
- name: CPUUtilization
statistics: [Average]
Current Behavior

Expected Behavior
Able to see metrics of only three RDS instances while I have a total of 17 rds instances in my AWS account
Steps To Reproduce
No response
Anything else?
No response

Need Help @alext

@cristiangreco hey could you please help me on this ?

Do the other 14 instances have tags? In the same region? Are they running\producing metrics you see in the AWS Console?

Yes they all are in the same region and I can the see metrics on console , but YACE is scraping metrics for 3 instances .

Hey @NickAdolf , I was able to see all metrics when I was using prometheus exporter but it is making tooo many API calls so switched to YACE.

Same issue on Elaticache, I only get metrics of one redis cluster while we have 4 redis clusters


Match History Issue

I get the following when going to Match History:

Desktop Tracker is running v 1.6.1 Android Tracker is running v 1.1.0

Thank you for your report. We will investigate it and try to find solution as soon as possible.

So I just checked today. I am seeing Match History now. All of my recent ones, except the first one I played yesterday, are not showing data on the cards my opponent played though.

Now this functional should work correctly.


feat: the sample project of large files upload

Fixes #22541, add a .NET 5 sample project of large files upload

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: GadHao sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@GadHao  fantastic!
@guardrex  who in the PU should I ask to review the code?

Barry and Pranav helped with the major updates that we did to the initial release topic.

@pranavkm please review this short but very valuable customer contribution on large file uploads - or recommend a reviewer.

btw @GadHao ... After you get all of @pranavkm's updates addressed, I suggest that you conform the project layout to match the (usual) convention on the repo, especially given that the existing samples for this topic conform to the pattern. It should only take a quick update/commit to resolve this:
Change the path to the solution/project ...
/5x/largeFiles

... to ...
/5.x/LargeFilesSample

... and then take the whole app (the contents of your current LargeFilesSample folder) and move it into that :point_up: 5.x/LargeFilesSample folder. Basically, it's the same layout and naming as you see for the existing 2.x/SampleApp and 3.x/SampleApp project folders here ...

https://github.com/dotnet/AspNetCore.Docs/tree/main/aspnetcore/mvc/models/file-uploads/samples/2.x/SampleApp
https://github.com/dotnet/AspNetCore.Docs/tree/main/aspnetcore/mvc/models/file-uploads/samples/3.x/SampleApp

@Rick-Anderson ...... 💵 ASP.NET Core money 💵 says on it, "In Pranav We Trust! :smile:." WRT the code comments tho: At this time, we aren't apparently going to describe this project in the topic. Later perhaps, a section could be added with text in place of the code comments to explain out those points. That's the plan?

@GadHao Hey - should this work with files over 4GB? I get an unexpected close of stream error with anything over 4GB - I have turned off RequestFiltering in IIS too.,

@Eelesy ... Open an issue using the This page feedback button and form at the bottom of the English-US topic. Use of the This page feedback form adds metadata to cross-link the topic to your issue and automatically pings the author. We don't work off of closed issues and merged PRs.

Nevermind @Eelesy ... I see you opened https://github.com/dotnet/AspNetCore.Docs/issues/22623. This ping was in my Inbox before that one. :smile:

@GadHao Hey - should this work with files over 4GB? I get an unexpected close of stream error with anything over 4GB - I have turned off RequestFiltering in IIS too.,

@Eelesy  I'm afraid this can't be done when you host the app by IIS.
For apps hosted by Kestrel, it can be done and I have tested it. Because when you use Kestrel to host it, you can set the MaxRequestBodySize to lift the maximum request body limit, and the MaxRequestBodySize  has type long
But when you use IIS to host it, the value to limit the maximum request body is maxAllowedContentLength in the web.config file. According to MSDN, it's a uint attribute, so you can never upload a file over 4GB

@GadHao thanks for your expertise and the fantastic sample you wrote.

@GadHao Hey - should this work with files over 4GB? I get an unexpected close of stream error with anything over 4GB - I have turned off RequestFiltering in IIS too.,

@Eelesy I'm afraid this can't be done when you host the app by IIS.
For apps hosted by Kestrel, it can be done and I have tested it. Because when you use Kestrel to host it, you can set the MaxRequestBodySize to lift the maximum request body limit, and the MaxRequestBodySize has type long
But when you use IIS to host it, the value to limit the maximum request body is maxAllowedContentLength in the web.config file. According to MSDN, it's a uint attribute, so you can never upload a file over 4GB

I have turned off IIS Request Filtering in IIS so that the file is not blocked by IIS. The endpoint in the controller starts to process the files, and then the Unexpected End of Stream error occurs. I will try Kestrel and see if IIS is still the issue,

@Rick-Anderson It will be very worthwhile if it can help anyone 😊

@GadHao Hey - should this work with files over 4GB? I get an unexpected close of stream error with anything over 4GB - I have turned off RequestFiltering in IIS too.,

@Eelesy I'm afraid this can't be done when you host the app by IIS.
For apps hosted by Kestrel, it can be done and I have tested it. Because when you use Kestrel to host it, you can set the MaxRequestBodySize to lift the maximum request body limit, and the MaxRequestBodySize has type long
But when you use IIS to host it, the value to limit the maximum request body is maxAllowedContentLength in the web.config file. According to MSDN, it's a uint attribute, so you can never upload a file over 4GB

I have turned off IIS Request Filtering in IIS so that the file is not blocked by IIS. The endpoint in the controller starts to process the files, and then the Unexpected End of Stream error occurs. I will try Kestrel and see if IIS is still the issue,

I also got the 413 status code after turning off the Request Filtering of IIS. I guess this may be some kind of internal limitation of IIS.

@GadHao Hey - should this work with files over 4GB? I get an unexpected close of stream error with anything over 4GB - I have turned off RequestFiltering in IIS too.,

@Eelesy I'm afraid this can't be done when you host the app by IIS.
For apps hosted by Kestrel, it can be done and I have tested it. Because when you use Kestrel to host it, you can set the MaxRequestBodySize to lift the maximum request body limit, and the MaxRequestBodySize has type long
But when you use IIS to host it, the value to limit the maximum request body is maxAllowedContentLength in the web.config file. According to MSDN, it's a uint attribute, so you can never upload a file over 4GB

I have turned off IIS Request Filtering in IIS so that the file is not blocked by IIS. The endpoint in the controller starts to process the files, and then the Unexpected End of Stream error occurs. I will try Kestrel and see if IIS is still the issue,

I also got the 413 status code after turning off the Request Filtering of IIS. I guess this may be some kind of internal limitation of IIS.

Thank you for your reply. I think it is impossible in IIS then, I will try Kestrel. :)

There's been a lot of discussion on this subject in the PU's repo. Search for issues there, and you'll find things like ......
https://github.com/dotnet/AspNetCore/issues/2711
... and there are many cross-linked (mostly closed) issues cross-linked to that one.
The current batch of open issues for "Kestrel" and "file upload" can be found with ...
https://github.com/dotnet/aspnetcore/issues?q=is%3Aissue+is%3Aopen+kestrel+file+upload
... and you might want to broaden that to include closed issues by removing the +is%3Aopen qs param. Just keep in mind over there that they don't work or discuss subjects in closed issues and merged PRs.

I notice this version of the streamed file upload sample doesn't include DisableFormValueModelBindingAttribute seen in previous versions of the sample. Is there a reason for this?


Move completions to new compiler testing infrastructure

In #51  and #55 we moved signatureHelp and hover to a new testing infrastructure. This new infrastructure makes it very productive to polish the output from the presentation compiler using ~metaserver/testOnly -- tests.compiler. This workflow is way more productive than doing publishLocal, open vscode, manually type code and see what it produces.
The completion provider can also benefit from some polishing in how the completions are ordered, for example that fields of a class take prioritity over defaults like toString/finalize

PS. I've started working on this.

Actually, would someone be interested in working on this? I have a WIP diff here https://github.com/scalameta/language-server/compare/master...olafurpg:completions?expand=1 that can be used as a base. There are other issues I can work on instead, for example building semanticdbs on the fly with the presentation compiler.

I can work on this. I've already started to hack around with completion priorities a few days ago, so I have a vague idea of how to tackle this.

Awesome! Just realised why completions started crashing a lot now, I removed the _CURSOR_ trick in the signatureHelp PR. Adding _CURSOR_ avoids many parse errors and enables completions on parts like User.<COMPLETE>
https://github.com/scalameta/language-server/pull/51/commits/fe144332132f40885d2964f3d76a854829b55ea6#diff-be76cbaabee1e6378859da7ffe659ab1L60
We should probably add _CURSOR_  in Compiler.addCompilationUnit, even if that's used for more than completions.
I suspect with this change we can avoid this hacky catch here
https://github.com/scalameta/language-server/blob/b160931177b6d9d0d351ec364cfbca70669fba08/metaserver/src/main/scala/scala/meta/languageserver/compiler/CompilerUtils.scala#L15

I'm not sure I understand the _CURSOR_ trick, but is it just to make sure the parser doesn't fail when completing at the end of a line?

We should probably add _CURSOR_ in Compiler.addCompilationUnit, even if that's used for more than completions.

addCompilationUnit as no offset in input, where should we add the __CURSOR__?

Ah, nvm, got it. I think your intuition is right, I've added it back and it seems to be working. I'll push it to a branch soon.

Scratch that, it seems to have no effect :(


Final Contribution Blog Post

Looks good overall! I notice you didn't include any screenshots or images in your blog post though - would you like me to merge this post without them? 2 of the 10 total points in the rubric for the blog posts are for "appropriate use of images"

Ack! I thought I had. Oops! I'll fix that right now.


feat(card-in-card): add Card in Card (UI Component) and Card Group with Card in Card (Layout Component)

Related Ticket(s)

GitHub Related tickets - https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/5304, https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/5308, https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/5280
JIRA ticket (HC Dev Board) - https://jsw.ibm.com/browse/HC-1647

Description

This PR adds in the "Card in Card" UI component that's designed to be used as standalone or assembled within the "Card Group" layout component/band.

Design Specs

https://ibm.ent.box.com/folder/132489332751?s=jad2fd65p56odcme9aoy1lgmuo1xaifu

Changelog
New

<dds-card-in-card>, a featured card image-like component that's used to highlight important/featured content (eg. Solution Case Study). This component showcases clickable callout image in 16:9 for large/desktop and 4:3 for small/mobile
<dds-card-in-card-image>, a component that extends the "Image" component so that the different aspect-ratio style can be applied.
<dds-card-in-card-image>, a component that extends the Card Footer.
card-in-card style - @import '@carbon/ibmdotcom-styles/scss/components/card-in-card/card-in-card';

New Prop

grid-mode - A select prop that defaults to (collapsed/1px) and an option to define (narrow/16px).

Storybook Previews

Card in Card (UI Component) - /story/components-card-in-card--default
Card Group with Card in Card (Layout Component) - /story/components-card-group--with-card-in-card

Missing

I still need to expose video-cta-container within DDSCardInCard for the variant where there's a play video button that opens up as an overlay.

@jeffchew Thanks for the reminder. I forgot to run yarn lint:styles in my other commits. Hopefully, this time around the Web Components preview will be built correctly. https://github.com/carbon-design-system/carbon-for-ibm-dotcom/pull/5419/commits/4edb75f9a78fa481c6f753f9a5bc2084e63ef8d4

@proeung the card in card with video story doesn't appear to be working for me, can you check? Also might be good to have an example in Docs on how to implement the video version.

@proeung the card in card with video story doesn't appear to be working for me, can you check? Also might be good to have an example in Docs on how to implement the video version.

@jeffchew This is the missing item mentioned in the PR description note above. I'm working with @RaphaelAmadeu to see what's the best way to use the video-cta-container to render the play video button that opens up as a lightbox/overlay.

I added the overlay hover effect to the image, however, we might want to turn this into a utility class if this hover treatment should be applied to other card components that are outside of card in card.

Do you mean this hover image effect should be applied to Card (..or whatever the dependent component is)? I can make a bug ticket if that's a cleaner solution so you don't override a fix here. We are doing a clean up of our components and there might be a bug if this effect isn't there already.

Do you mean this hover image effect should be applied to Card (..or whatever the dependent component is)? I can make a bug ticket if that's a cleaner solution so you don't override a fix here. We are doing a clean up of our components and there might be a bug if this effect isn't there already.

@shixiedesign Yes, I think a separate tech debt/clean-up ticket makes sense for this image hover treatment. I noted that in my other comments as the styling that I added in this PR might be reusable or not depending on how you guys are planning on applying this effect across other image card components.
Regardless, here's the latest preview of the Card in Card component that includes the image hover overlay that you mentioned in the design QA. Let me know if the darkness level corresponds to the design specs. Happy to adjust if needed. https://ibmdotcom-webcomponents.s3.us-east.cloud-object-storage.appdomain.cloud/deploy-previews/5419/index.html?path=/story/components-card-in-card--default

@proeung the card in card with video story doesn't appear to be working for me, can you check? Also might be good to have an example in Docs on how to implement the video version.

@jeffchew This is the missing item mentioned in the PR description note above as well as in this commit. I'm working with @RaphaelAmadeu to see what's the best way to use the video-cta-container to render the play video button that opens up as a lightbox/overlay.

Hi @proeung ,
To make code freeze we are making the decision to split out the video variant work into another PR, and we can move forward with the image variant since that appears to be ready. @RaphaelAmadeu will work with you on the details.

@jeffchew Makes sense and sounds good! I'll work with @RaphaelAmadeu to create a new PR that accounts for the video variant work. In the meantime, I went ahead and remove the "with video" Storybook variant (https://github.com/carbon-design-system/carbon-for-ibm-dotcom/pull/5419/commits/13bc590060990524681990d90911a1b190b2e1c3) so that we're not causing confusion when this PR is merged to master.


[WIP] Refactor HTTP async* operations

Changes:

Use stackless coroutines instead of hand-rolled state machines.
Added the is_continuation() helper to retrieve the state of a stackless coroutine.
Added the prepare_or_error() helper to avoid issues with yield labels crossing initialization of catch blocks.
Remove caching of mutable buffer in read_some_op.
Renamed bytes_transferred to total_bytes_consumed_ in read_some_op since it stored the number of bytes consumed by the parser.

To Do:

[ ] write_some_op
[ ] write_msg_op
[ ] write_op

Resolves: #764

I greatly appreciate your contribution, it is wonderful to see the community getting involved in the project. This is going to really help everyone since it will enable me to get even more done with the new libraries I am building on top of Beast. However, there's a learning curve to the style used to write the composed operations as you can see from my comments. Please don't let that discourage you!
I think that prepare_or_error should be in its own commit (in boost::beast::detail::) and come first, it looks quite the useful function and something that I will be adopting in new code.


Chore: Grenerate data for the questions and answers

We will need to show 10 questions (minimum?) and each of these questions will need 4 answers.
We already have 2 questions and corresponding answers here.
The task is to generate more data. They do not need to be meaningful at this point. it can be anything. Later on when we what to improve it we can make question answers based on the theme.

Done with #7


Second Impact and 3rd Strike music extracted to strange folders

Thank you for this utility.
After extracting, I found the Second Impact music was in the 3rd Strike folder, and the 3rd Strike music was in the Main folder.
A couple of caveats:

I'm on a Mac and changed \\ to / in extract.py to get the script running.
Maybe this is an issue with how the game assets are bundled, not a bug in the extractor.

It's an issue with how the game assets are bundled 👍 And yeah the script was meant to be for windows and coded in less than an hour so i never tested this elsewhere :)


Add absolute tolerance functionality to assert_xydata() and assert_bin_heights()

Currently, assert_xydata() and assert_bin_heights() support relative tolerance but not absolute tolerance. Absolute tolerance should be implemented in these functions. This should be a trivial change given how tolerance is currently implemented with np.testing.assert_allclose().

hey @ryla5068 can you double check that this is addressing tolerance for both histograms and other types of plots? if it does - cool! if it doesn't just note what we DIDN't address here!

@lwasser this can be closed when #234 is merged. As we decided previously, we are removing relative tolerance functionality and adding absolute tolerance in its place (as opposed to in addition).
This PR also addresses tolerance for histograms.


Fixing github Issue https://github.com/Microsoft/vso-agent-tasks/pull…

…/1214. where we select azurewebsite by name that we select after calling get-azurewebsite as that command can return multiple values

Fixes#1214


question about the paper

Hi, thank you for the excellent work. I have one question after reading the WACV 2023 paper.
Though different skeletons have different topologies, they often share similar end-effector definition(say hand and foot), so have you tested supervising all the skeleton's end-effector position with same ground truth while left other joints to work with the proposed autoencoder?
This is just a shallow thought on my part, could there be something wrong with it?

The main benefit of our method is that we don't need to think about such heuristics or special cases. Just put all joints in a big pot and the rest is taken care of, the algorithm finds out how those joints are related without specifying that wrists or ankles are always the same. Also, it's not entirely true that the ankle points are always defined the same way in all datasets.
So overall I have not tried such a thing, and it may be possible to squeeze some extra performance by hand-coding some special cases.

Fine, Thank you for your kind reply.


Gnome Dark Theme

Gnome (at least Gnome 42 & 43) uses the dconf key color-scheme under org/gnome/desktop/interface to select dark mode. Additionally, the key for setting the wallpaper in dark mode is different to light mode (picture-uri-dark vs picture-uri). This is a simple patch for that.
Not sure if there is anything else to do here. Tested on my computer (using impermanence to wipe home so I don't think I was missing anything here). Light mode works correctly (if chosen).

Nautilus uses the GTK theme for me. Perhaps Gnome needs some more settings to make it prefer the Stylix theme over the default.


Add DeletedTeams to BadgeState

depends on https://github.com/keybase/keybase/pull/3870
cc @cjb

@cjb i'll let you know when the server side is deployed to test

@joshblum cool, cjb/DESKTOP-9784-deleted-teams-banner is an untested try at integrating (if you merge this branch into that branch)

@joshblum Ah, I don't think we'd want to badge anything for a team you deleted yourself.


Add Wayland protocol module

The Wayland-Server crate uses the user_data field to keep track of lifetimes, which is incompatible with wlc. We'll have to write a wrapper around wayland_sys so that the unsafety doesn't pollute Way Cooler.

With the release of this update on the wayland-rs crate, it should be much easier to interface with Wayland clients via protocols. We might be able to use the wayland-server module, if not though there is always wayland-sys.


Poor contrast for highlighted items in search

Hey @sdras! Love the theme, thanks for sharing it with all of us!
I've been using it for a couple weeks and I've noticed that the selected item in search results is very difficult to read. This is also true for selecting rows in the "problems" pane (ie TypeScript errors). See below screenshot.
Any way to increase the contrast on these selected items so they become more readable?
Screenshots

Thanks for pointing this out! This should be fixed now :) let me know if you're still having issues. May need a restart.


Add command handler

Is your feature request related to a problem? Please describe.
Dude the code itself looks super unmanged , add a command handler and event handler
Describe the solution you'd like
I'll make a custom command handler and port over every single stuff to it
Describe alternatives you've considered
Use of wok command can quick the process but isnt that credible

I'll like to do this stuff u can assign me this

@apoorvcodes Sure, go ahead

Fixed by #15


fix: adapt to changes in mamba 2.0

QC

[x] The PR contains a test case for the changes or the changes are already covered by an existing test case.
[x] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

Summary by CodeRabbit

New Features

Enhanced flexibility in specifying the package manager for environment creation, allowing the use of either "conda" or "mamba".
Added a new method to dynamically generate command-line arguments for environment creation.
Introduced a property to retrieve the version of the specified frontend package manager.
Updated the compress_vcf rule to include a new parameter for improved VCF compression.
Added a new contig declaration and updated INFO fields in the VCF file for enhanced dataset detail.

Bug Fixes

Improved logging messages for better clarity regarding the frontend being utilized during package installation.
Enhanced test robustness by using a dedicated function to check for the environment setup flag file.

Dependency Updates

Updated the version constraint for the snakemake-interface-executor-plugins dependency to a newer version.
Tightened version constraints for the peppy and paramiko packages in the test environment.

Closing in favor of #3121


SV report fixes

Change snpEff database to GRCh38.86 so that VCF is annotated with Ensembl genes/transcripts
Modify annotate_counts so that if there is no overlap between the sample SVs and the count table (i.e. for BNDs annotated against the Manta MSSNG database, which does not contain BNDs), the column is just populated with '-'. We could instead use a lifted over file from GRCh37 in the future if desired, as this does contain BND counts
The columns associated with the MSSNG count are named after the file name, so for example the Manta SV column becomes "noBNDs_includeHQsamples_mssngSubjetsCounts_hiseqX_mantaSVs_db6_hg38_1based_FIXEDCOLUMNS_sorted_nochr_uppertype"; may want to reconsider renaming these files to something more comprehensible

Thanks for your help on this Madeline! It is now ready to merge. Tested and made changes to the SV workflow.
Also, unrelated to SVs but involving the hg38 pipeline: changed the reference from Illumina to GIAB, and incorporated the C4R exome counts into the vcfanno annotation pathway (other changes for this found in the cre hg38 branch).


feat: integer operations in norm_num

Another major rewrite of norm_num, this time to support returning integers as well as nats. Rational numbers are included but stubbed for now pending the rest of the algebraic typeclasses.

@digama0, I think the doc-string on Result.toInt actually belongs on the missing function deriveInt. I've added this function in https://github.com/leanprover-community/mathlib4/pull/507/commits/e5a9aa8292d65dfabab53b6b7998413e761a6a4b, but perhaps you can scrutinize this commit especially carefully to see that it is what you want.

Looks great to me, and I'm already successfully using the additions in abel1.
bors d+

bors merge


Feat: while statement

Added support for while statement using previously implemented syntax, new token, etc.
Usage in main.pych:
while ( expression ) { print("jeff"); }

Could someone verify the break statement please>

Just changes for readability
Do you also want to add continue? maybe a group decision

ditto on continue one of the most underappreciated statements.


feat: specify flag name in parse error message

We have use cases where we want to use the min and max values for an integer flag (See PR here)
It would be nice if the parsing error message included the flag it pertained to
Before
➜  the-worst-folder git:(main) wow create character --strength=51 --intellect=51 --agility=2
    Error: Expected an integer less than or equal to 50 but received: 51

After
➜  the-best-folder git:(main) wow create character --strength=51 --intellect=51 --agility=2
    Error: Parsing --intellect
        Expected an integer less than or equal to 50 but received: 51

@MichaelGoberling Thanks for the contribution - love the idea! I just made a couple of suggestions to clean up the implementation


TypeError: Elysia is not a constructor when bundling for node (with polyfills) with esbuild with esm

since the node entry in package.json points to the cjs build, that's what's imported, but it seems like for some reason it results in the default export of the Elysia class going under exported.module.default.default, and when marking elysia as external in esbuild, esbuild correctly inserts import Elysia from "elysia" in the resulting bundle, but the actual class still gets exported under Elysia.default, so new Elysia errors out, but editing the bundle manually to do new Elysia.default makes it work properly.
You can find some more info under https://github.com/evanw/esbuild/issues/3580
Importing Elysia as a named import rather than the default import with import { Elysia } from "elysia" works, but the @bogeychan/elysia-logger plugin imports elysia via default import and I have no control of that

Relevant: https://github.com/elysiajs/elysia/issues/396

Elysia now exports esm and cjs separately as follows:
{
	"exports": {
		"./package.json": "./package.json",
		".": {
			"types": "./dist/index.d.ts",
			"bun": "./dist/bun/index.js",
			"import": "./dist/index.mjs",
			"require": "./dist/cjs/index.js"
		}
        }
}

This should have fixed the problem with import and require separately.


statusBar: focusNext and focusPrevious

Fixes https://github.com/microsoft/vscode/issues/95433
This PR introduces:

statusBarFocus context key
workbench.statusBar.focusPrevious and workbench.statusBar.focusNext commands
Uses the status hover background color for showing focus
Exposes focusNextEntry and focusPreviousEntry on the statusBarService
focus goes cyclic, so if you focus next and the last is focused we focus the first.
Tab moves focus out of the status bar.

All with the idea to make the status bar item navigable once the status bar is focused. So the flow is F6 to focus status bar, left and right to go through the status bar items.
Up and down are secondary keybindings for left and right. MS Word behaves like that and I think it makes sense to also navigate like this to be aligned with activity bar and users might not know how status bar looks visually.
Also I decided to put focus on the a element, because that way Orca announces the most concise aria-label.
I tried this with Orca, NVDA and VoiceOver and seems to work nicely.
Two issues I hit:

On Linux: entries are not ordered with how they are displayed. Though I could not repro this on Mac or Win. @bpasero might you know more (I know you use that flax trick but I thought you just use it for the display, not on the model level)
Once an item is focused VoiceOver reads the ariaLabel of the item and the whole ariaLabel of the statusBar. This seems to work fine with Orca and NVDA. So we can file an issue with VoiceOver.

@bpasero let me know what you think and if it looks good to you I will merge. Thanks!

Seems promising. :-) Thanks

@bpasero thanks for the review, I will adress your comments and merge.
We can introduce a new color for focus. Though since this will primarily be used by screen reader users I would only introduce it once users request it.
The original F6 focus on the status bar is not rendered - we should probably change that.
As soon as you press left / right you should see the focus as the hover.

@isidorn I see it working now, but when I press F6 I would expect the first item in the status bar to show focus, but in fact the last one seems to have focus?

@bpasero the behavior is first the whole status bar is focused (so everything first gets read out to the user), then pressing left / right takes you through each item.
I do not see the last item being focus just by pressing F6.

Here is what I see.

@bpasero that's strange. Can not see that. Can you maybe do some
setInterval(() => console.log(document.activeElement), 1000) and check where the focus gets lost

@isidorn there you go:

I think the reason for this is that my first entry "Remote" is always hidden.

@bpasero thanks. I have pushed a fix, that we jupm over hidden items, made the code a bit more complex but works fine now.
I have also changed the initial value of the context key.
Thus I plan to merge this in if it is ok by you.
As follow up work I plan to change the F6 navigation to also focus onto tha activity bar and the panel title area.

For reference here's the commit that tackles that F6 navigates to the activity bar ef14938a46

@bpasero thanks, merging in.
Not so sure about Esc since we already changed ActionBar to not do anything on Escape - since the focus would jump unexpectedly.
Though we should keep the whole "what to do with esc regarding focus" discussion on-going.


chromium: 67.0.3396.87 -> 67.0.3396.99

Motivation for this change
New stable version with some minor fixes is out for quite some time (20 days), so stable bump is long overdue.
Also fix issues with Beta (68) and Dev (69).
I checked out resulting Stable, Beta and Dev binaries by watching some YouTube with them.
Things done

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

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

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

@fpletz Thanks! Could you please merge this in release-18.03 as well?

@bendlas Sorry, forgot to CC you here. I'm still confused as to who to CC on changes to packages...

@YorikSar No worries. When I tried to do the update, last week or so, google has had another case of missing .tar.xz. I should have created a WIP PR right there, but since there didn't seem to be any CVEs, I instead left it in a stash, which I just re-tried. I then pushed that commit to my fork before rebasing (and hence notifying you before noticing your update), because I had already built the system and wanted to install it.
Sorry for spamming you and thanks for keeping an eye on chromium.

@bendlas Ok, good. I though I made you do extra work by not including you in CC here. I actually was gathering my thoughts about fixing beta/dev for quite some time now. I was afraid that again beta will be promoted, Chromium won't be ready in NixOS and I'll have to sit without new Chrome again :)


Use existing serializers

As titled, the goal is to support the reuse of https://github.com/pinojs/pino/blob/master/docs/API.md#stdSerializers

love it - one teeny weeny doc thing then fine for merge


[Bug-FE] [Gantt chart] The Gantt chart does not show the execution status of all components

Search before asking

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

What happened
The Gantt chart does not show the execution status of all components

What you expected to happen
Normal display of component execution status
How to reproduce
Execute workflow definition and view Gantt chart
Anything else
No response
Version
3.0.0-beta-2
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@XuXuClassMate

@XuXuClassMate Hello, please describe exactly what happened to you to create this problem.

I will fix it


test: e2e tests to check that libraries can query new identity claim

Description of changes
Creates e2e tests to test for the new identity claim and ensure each library can still query for owner.
Description of how you validated changes
Ran these tests with the changes from https://github.com/aws-amplify/amplify-cli/pull/10196, https://github.com/aws-amplify/amplify-cli/pull/10189, & more to be added.
There will be a fast follow to deal with linting errors in these test files.
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #10198 (d2f5903) into master (f2ac2b2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #10198   +/-   ##
=======================================
  Coverage   54.27%   54.27%
=======================================
  Files         849      849
  Lines       46933    46933
  Branches    10004    10004
=======================================
  Hits        25471    25471
  Misses      19445    19445
  Partials     2017     2017

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Connection error in windows 10

I cannot connect to my ollama serve, and met this error:

after some digging, I figured out that it was caught by curl from system. When I ran command coped from log, I got this:

curl -sSL -D C:\Users\Eric\AppData\Local\Temp\plenary_curl_cb9c41a5.headers --compressed -X GET http://<IP_ADDRESS>:11434/api/tags
curl: option --compressed: the installed libcurl version doesn't support this

when I removed '--compressed' flag, It works:

curl -sSL -D C:\Users\Eric\AppData\Local\Temp\plenary_curl_cb9c41a5.headers -X GET http://<IP_ADDRESS>:11434/api/tags
{"models":[{"name":"llama2:latest","modified_at":"2024-01-16T10:46:47.812037511Z","size":3826793677,"digest":"78e26419b4469263f75331927a00a0284ef6544c1975b826b15abdaef17bb962","details":{"format":"gguf","family":"llama","families":["llama"],"parameter_size":"7B","quantization_level":"Q4_0"}},{"name":"mistral:latest","modified_at":"2024-01-17T05:39:41.463682696Z","size":4109865159,"digest":"61e88e884507ba5e06c49b40e6226884b2a16e872382c2b44a42f2d119d804a5","details":{"format":"gguf","family":"llama","families":["llama"],"parameter_size":"7B","quantization_level":"Q4_0"}}]}

I am not familiar with Lua language, can you please tell me how to change this behavior? Or I will have to install a new version of curl that support this feature and change PATH environment variable to solve this problem.

Hey,
I've not tested this plugin to work with Windows, but it seems the issue you are facing is related to the dependency plenary, and more specifically, libcurl on Windows.
From my own brief research, it seems you can get around this by installing a version of curl that specifically supports the --compressed flag on Windows, so I would start your search there.
If anyone else has insight on this I'd be glad to know!


adobepass returning 401 or unable to extract post url error

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2020.09.06
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '--get-filename', '--ap-mso', 'Comcast_SSO', '--ap-username', 'PRIVATE', '--ap-password', 'PRIVATE', 'https://disneynow.com/shows/fancy-nancy/season-02/episode-10-nancy-rocks-the-boat-spring-dress-mess/vdka18980859']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dlc version 2020.09.12
[debug] Python version 3.6.9 (CPython) - Linux-4.15.0-112-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg N-50726-g20c5f4d835-static, ffprobe N-50726-g20c5f4d835-static
[debug] Proxy map: {}
ERROR: Unable to download webpage: HTTP Error 401: Unauthorized (caused by <HTTPError 401: 'Unauthorized'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dlc -U  to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.
  File "/usr/local/bin/youtube-dlc/youtube_dlc/extractor/common.py", line 627, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dlc/youtube_dlc/YoutubeDL.py", line 2236, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.6/urllib/request.py", line 532, in open
    response = meth(req, response)
  File "/usr/lib/python3.6/urllib/request.py", line 642, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python3.6/urllib/request.py", line 570, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.6/urllib/request.py", line 650, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)

Description

When using the tool off the providers network, a 401 error is produced. When on their network, a "ERROR: Unable to extract post url; " is generated.
https://github.com/ytdl-org/youtube-dl/issues/26558#issue-696714081

Ditto.

I am not exactly sure where the error is here as I haven't dealt with adobepass so far.

I downloaded 2020.09.30 release. Still has the same issue. (The error appears to be the same with the same verbose responses - the line numbers are different but I am guessing this is just due to code revisions rather than being something new.)

The issue is still there with the 2020.10.09 release.

I started tweaking the code to get the output of the post_url page, and it appears I'm being auto-logged in but nothing is catching it:

You're automatically signed in with Xfinity and ready to watch.

Access to this experience is an added benefit of your Xfinity TV service.

Thanks for taking a look at it. If you just want to scratch the itch of resolving it, please do! However, FWIW, I have gotten around this issue in newer releases of yt-dlc by using cookies. I have no developing ability so I could not tell you what change fixed it, since in the past cookies did not work either.
I log in to whatever site (sometimes need to sign out and sign back in) before generating the cookies file. I then use this syntax (also works with single links):
youtube-dlc --ap-mso=Comcast_SSO --ap-username [comcast username] --ap-password [comcast password] --cookies cookies.txt --batch-file links.txt --ignore-errors
(sufr3ak and I appeared to be the only ones participating on this project as well as OG yt-dl, and we sorted it out over there before the issue was...closed as a duplicate without any interaction from the developers. Surprising I know.)

Sadly that seems to be a common thread over there. :(
I was looking at it earlier but had other stuff I needed to get done. I'll have to familiarize myself with that code to see if I can figure it out. Using cookies works because it just injects the cookie data into request, which bypasses the need to authenticate.


Add a path manipulation library

Goals:

A base for all path-based functionalities in the project.
Stateless, do not depend on the file system (sys/fs will be the extension of sys/path for that purpose).

Problems:

Does the relative vs absolute path distinction matter enough to warrant distinct types?
Does it matter to be able to manipulate POSIX paths on Windows and vice versa?

Depends on #2, we need NullessString as a base for path types.

Does it matter to be able to manipulate POSIX paths on Windows and vice versa?

Since what we are doing is completely lexical, it would be nice to be able to do this, given that POSIX paths are frequently used everywhere (since it is used for URI).
However it is hard to determine a good API strategy for this. Ideas includes:

Provide PosixPath and WindowsPath types with a Path type that aliases to the one corresponding to the native OS.
Provide paths/posix and paths/windows, with paths exporting the native version. However the documentation will have to be duplicated, which is a pain.

Does the relative vs absolute path distinction matter enough to warrant distinct types?

They do matter, but complicate API usage. For example:

Programs taking arbitrary input (say, parameters: foo /foo/x.txt ../bar/x.nim), which might be an absolute path or a relative path, and it's not possible to provide a toPath() that dispatches to the right path type on runtime.
Not all programs/libraries care about "relative" vs "absolute". They might do just fine with either of them. An AnyPath typeclass can solve this.

Solutions:

Absolute and Relative being a distinct version of Path that provide one-way implicit conversion back to Path. Converters have to be investigated to see if they are "safe" enough first. Alternatively an AnyPath typeclass can provide the same functionality, but we would still need the Path type due to the dispatching issue in the first example.
Path being a variant object that might refer to either Absolute or Relative.

Notes for POSIX implementation:

.. is a dangerous atom: a/../b is not necessary to be equal to b, since if a is a symlink, the .. atom will resolves to the parent of whatever a refers to.

Notes for Windows implementation:

Microsoft documentation on path namespaces: https://docs.microsoft.com/en-us/windows/win32/fileio/naming-a-file
It is untested how Windows .. atom behaves compared to POSIX. Windows do have symbolic links, but they might not carry the surprising behavior of POSIX. This has to be tested however.
Regular DOS paths (eg. C:\path\to\something) contain plenty of surprising behaviors:

Certain names are "special" (refer to the linked doc for more information): C:\Windows\System32\NUL is the same as NUL, NUL.txt is the same as NUL, but NULabcd is not special. Not sure if NUL being a directory will work (ie. C:\NUL\a.txt), but it's safe to assume that it is not.
There is a limitation of 260 characters.
Trailing space ( , untested if all form of spaces or just the space character) or period . in each path element are trimmed automagically by the Windows API. This means C:\Windows.\System32 is the same as C:\Windows\System32.
/ is a valid path separator and will be treated in the same way as \ (only if / is used exclusively?).
C:Windows is not the same as C:\Windows. The former is a relative path.
\something is a relative path.

It is unknown how much of DOS path issues applies to UNC paths (eg. \\hostname).
Device paths (paths starting with \\? or \\.) does not share the limitations of DOS paths.

Differs in path semantics between Windows and POSIX makes it hard to determine the characteristic of the Path type. It is unsure if a Path type should be formed or if string-based (to be more precise, strings.Nulless-based) interface should be used instead.

For what it's worth we managed to support both kinds of windows paths in c++'s std::filesystem::path interface at the same time. (NT paths and Windows paths)


added NO_DL build configuration.

If defined NO_DL, the engine will not support using the tcti shared library.
Some system such as VxWorks builds the tcti as static library and links with OS image together.
In this situation, shared library is not supported and should be removed from engine.
Signed-off-by: yfeng<EMAIL_ADDRESS>

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   67.45%   67.45%
=======================================
  Files           9        9
  Lines        1177     1177
=======================================
  Hits          794      794
  Misses        383      383

Impacted Files
Coverage Δ

src/tpm2-tss-engine-tcti.c
56.19% <ø> (ø)
: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 ae7d7e1...8571051. Read the comment docs.


Add network exporter

Addition of the Network Exporter

Thanks for your PR. I aim to avoid duplication in this list, and this network exporter seems to duplicate both the blackbox and especially smokeping exporters which are already listed. If there's ways in which you feel those could be improved, I'd encourage you to talk to their maintainers.

@brian-brazil  Actually is a mix of both blackbox and smokeping but with more metrics (jitter stats) and features but specially the MTR module (traceroute)

At the end of the day it's filling the same base use cases that already listed exporters do, so I don't think it makes sense to list this here. You are free to announce your exporter on the -users list.

@brian-brazil, Ok no problem


Remove codec from google proxy example

Issue Template
Remove codec from google proxy example (v2)
Description:
This is supposed to be rarely used as the default is AUTO, and it specifies AUTO.

This applies to a lot of other files in the configs folder as well.

https://github.com/envoyproxy/envoy/pull/2825


Incorrect License (Packagist)

I'm getting the wrong license for my package on Packagist.  It's showing "proprietary" (and it was at one point) but the latest version (and master) are both published as MIT.

Example image: https://img.shields.io/packagist/l/iodatacenters/oktasdk.svg
Packagist page: https://packagist.org/packages/iodatacenters/oktasdk (json)

This appears to be fixed now.

If this issue recurs, please feel free to comment or reopen.


Documentation on how to install Docker engine offline

Problem description

Two customers have asked us on how to install Docker engine on on-premise server with no network connection.
However, our manual doesn't cover how to install Docker engine on Windows Server.  I am not sure if we should put this on our docs or engage Microsoft to update theirs.
Problem location

Microsoft already hosts install instruction, so this is the most logical place to put.
https://docs.microsoft.com/en-us/virtualization/windowscontainers/quick-start/quick-start-windows-server

Project version(s) affected

17.03
Suggestions for a fix

These steps work.

do this from online server

# on online machine
invoke-webrequest -UseBasicparsing -Outfile docker.zip https://download.docker.com/components/engine/windows-server/17.03/docker-17.03.0-ee.zip

do this from offline server

#— extract the archive —
Expand-Archive docker.zip -DestinationPath $Env:ProgramFiles
#clean up zip
Remove-Item -Force docker.zip
# enable hyper-v if not yet enabled
Enable-WindowsOptionalFeature -Online -FeatureName Microsoft-Hyper-V -All

#adding docker to PATH
$env:path += "$env:ProgramFiles\Docker"

#register docker daemon as a service
dockerd --register-service

#start the daemon
Start-Service docker

Topic is live: https://docs.docker.com/docker-ee-for-windows/install/


Set continuation indent to 4 in IntelliJ explicitly

For some reason IntelliJ keeps occasionally switching between 4 and 8 otherwise.

@bot-gradle test and merge please.

I've triggered a build for you.


Latin segmenter should support word segmentation

We should be able to support splitting words by methods other than the text casing. Libraries like instant-segment exist to do that.

redneckbossryan -> redneck, boss and ryan can  be extracted
massachusetsinstutitute -> massachusetts, institute

This issue came from a discord chat I was having about how search works vs what I thought would happen. So this all comes from the belief that a user should be able to search for key segments within the system. So, if we use the example above

RedneckBossRyan - red, neck, redneck, boss, Ryan

However, this goes deeper for my use hope, I would want someone to be able to search for redneckbo and get the results accordingly. So, instant-segment does make a lot of sense, but we are also at that point having to consider how many languages and other factors that come into play.
Would it maybe make sense to create a solution that would take words that are not dictionary and maybe store them in segments, so RedneckBossRyan could be stored as word segments sure, but could also be stored in 3-4 character chunks

Sorry if my question sounds a bit simple, but, why don't you add spaces when searching in Meilisearch? 🤔

It's a valid question. The issue is usernames. People type these long winded usernames so searching becomes very "challenging"

Mmmmh, I see, unless having the possibility to upload a custom dictionary, your use case can't be solved completely.
However, solving this issue (#129) could enhance your search experience. 🤔
If a contributor solves it, you could expect a better Meilisearch behavior in a future version. 😃

I'm investigating meilisearch for my employer, whose main customers will be Dutch. For our use case word segmentation is very important. Dutch (and German too) uses a lot of compound words, so searching for a part of a word should work correctly. In my testing, meilisearch (and typesense for that matter), was unable to find documents containing "lichtgeel" and "zachtgeel" when searching for "geel". Even though specific types of yellow should come up when searching for yellow in general.
Instant-segment may not work in our case as it is meant for English text, for which corpuses exist which have the bigrams separated. In Dutch the correct word is a combination of two words.

An other german here with the same problem.
At least in German the compound words are also hypenated at the word boundary and iirc that word boundary is to be prefered.
So maybe https://github.com/typst/hypher/tree/main/patterns can serve as a source for these boundaries.


backport: merge bitcoin#26519, #25880, #26982, #26584, #27577, #28419, #28196, #28433, #28489, #28525 (BIP324 backports)

Additional Information

When backporting bitcoin#25880, changes needed to be made to p2p_ibd_stalling.py to help it pass.

DIP3 activation had to be delayed to a block beyond the range of the test. This is to prevent block rejection arising from a missing DIP3-compliant coinbase (done with -dip3params=2000:2000)
Mock time was disabled to ensure nodes in the test do not resort to direct fetching (with mock time enabled, nodes would be considered close apart in time, which would prevent the primary node from fetching in parallel, which is behavior this test relies on) (done with self.disable_mocktime = True)
The nodes connected do not report compressed headers support (the test relies on sending headers messages and reworking it to use compressed headers has little benefit) (done with services = NODE_NETWORK | NODE_BLOOM)

When backporting bitcoin#28196, in the v2transport_test unit test, references to 4000000 were substituted with MAX_PROTOCOL_MESSAGE_LENGTH as Dash messages have a protocol limit of 3MiB (source) while Bitcoin messages have a protocol limit of 4MB (source).

Additionally note that short message IDs as defined in the BIP324 spec (source) have not been changed to include Dash-specific messages, meaning, Dash-specific messages will always take 13 bytes.
As FEEFILTER is not supported by Dash, it has been replaced with a blank string in the short IDs table. It was not removed as doing so would disturb the table's arrangement as specified in spec and require readjustment of tests to account for the change in layout.

Breaking Changes
None expected.
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas (note: N/A)
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have made corresponding changes to the documentation (note: N/A)
[x] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

Additionally note that short message IDs as defined in the BIP324 spec (source) have not been changed to include Dash-specific messages, meaning, Dash-specific messages will always take 13 bytes

message_type ... a one byte ID in range 1..255

only 28 IDs are used in Bitcoin so why don't we use IDs 128+ for Dash-specific messages? Doubt they'll get to these numbers anytime soon. @PastaPastaPasta ?
also pls see da5b56c7ed

only 28 IDs are used in Bitcoin so why don't we use IDs 128+ for Dash-specific messages? Doubt they'll get to these numbers anytime soon. @PastaPastaPasta ?
This is actually the goal in the not so distant future, probably before we actually merge in stuff where it's actually usable. If not, we'll do that via a new protocol version. Excluding that for now was just to expedite the process, as the code doesn't have a trivial way to just start at 128 for example if I remember correctly


Error when using Meteor-React

Do you have any example on using this with Meteor and React?
When I copied lines from the sample code in the client/main.js file, an error came out:
Uncaught ReferenceError: LoggerFile is not defined
Thank you!

Hello @maitrid ,
From the error you've referenced looks like Logger or LoggerFile is not imported.
Not sure what is so different in React from JS, it's still JavaScript, you should import it this way:
import { Logger } from 'meteor/ostrio:logger';
import { LoggerFile } from 'meteor/ostrio:loggerfile';

Thank you, @dr-dimitru!
I missed the 2nd import line. Now trying...After having these lines near the top of a component file:
import { Logger } from 'meteor/ostrio:logger'; import { LoggerFile } from 'meteor/ostrio:loggerfile'; const log = new Logger(); new LoggerFile(log).enable();
And having a log.error("This is logger working"); inside the render() method. There is no output. What did I miss? Do i have to have an adaptor?
Thanks again!

There is no output. What did I miss? Do i have to have an adaptor?

Where you're looking for output?
LoggerFile will put logs into the file
If you want to see logs in a console - use LoggerConsole adapter

I would be looking for output in the browser console from the client code, and in the command console from server code. Will look into adapters. Thanks!

@maitrid Let me know what you've selected and what is working for you.
Keep me updated.

Closed due to silence at issue owner end.
Feel free to reopen it in case if the issue still persists on your end.


[PROPOSAL] Add Bake Time

What/Why
What are you proposing?
This repo documents things like the TSC/GB and should be edited carefully. I propose we add Bake Time like in https://github.com/opensearch-project/.github/blob/main/.github/workflows/bake_time.yml so that changes sit for 48 hours without comment before getting merged.
What problems are you trying to solve?
Avoid rushed changes that may require more discussion to undo.
What is the developer experience going to be?
Every PR will sit for 48 hours before getting merged.

I agree for the TSC/GB pieces, but it will definitely add challenges for the other items we use this repo for - presentations, community docs, etc. Interested in your thoughts on that.

We can probably include/exclude some folders, https://github.com/peternied/bake-time/issues/11.

I like that idea a lot - include the two folders we're building for the governing-board & technical-steering-committee - would only slow down the ability to post meeting minutes, but that is a good balance.

Since we've moved the TSC / GB pieces over to a more appropriate home - this becomes unnecessary.

@krisfreedain If you don't think we need it here please feel free to close


"Export data" from footer wrongly uses pinned tab query

Description
Run a query, pin result tab (tab a).
Run another query, do not pin result tab (tab b).
With tab b active, hit "Export data" in the footer below to export the results of that tab b.
DBeaver exports result of tab a instead.
With (multiple) pinned tabs always the first pinned tab to the left is used.
DBeaver Version
Version <IP_ADDRESS>403031740
Operating System
Linux, Debian/SID
Database and driver
No response
Steps to reproduce

SELECT * FROM a
pin result tab (tab a)
SELECT * FROM b
do NOT pin result tab (tab b)
with tab b active hit Export data in the footer below to export results of tab b
exported file contains results of tab a!

Additional context
No response

Thanks for the bug report


chore: add support for alpha, beta and rc images

The old regexp used to detect the version was only accepting versions
in the format major.minor-revision, now we also accept the format
with the beta major(alphaN|betaN|rcN)-revision
Closes #103
Signed-off-by: Jonathan Gonzalez V<EMAIL_ADDRESS>

/ok-to-merge


Document how to set sample nodes to squares (a la tskit)

I think this is the fastest method to make large ARGs match the tskit standard styling, where samples are squares. Could we put something like this snippet into the docs somewhere?
d3arg.nodes.loc[np.isin(d3arg.nodes["id"], ts.samples()), "symbol"] = 'd3.symbolSquare'

Almost a dupe of https://github.com/kitchensjn/tskit_arg_visualizer/issues/87

Added this example to /docs/tutorial.md! Thank you.


Unpin & upgrade winit to 0.30.5

This updates winit to 0.30.5.
https://github.com/emilk/egui/pull/4849 Had to pin the version to 0.30.2, as a Winit patch changed the behavior of selecting a theme. Winit 0.30.5 reverts this, so we could stick with window.theme(), but the newly added ActiveEventLoop::system_theme is more like what egui wants anyway, as individual windows can have theme overrides.
Also bump smithay-clipboard to prevent some now duplicate dependencies.

Tested and can confirm that this fixes touchscreen release issues (#4895).


Create constant strings

Some strings such as "USER" and "EXIT" are used as constants but not defined in that way.
Find a good location for them.

Is this still an issue? Otherwise it can be closed.


added bonita

I second this: Bonita is a great engine, with a very good BPMN support and a quite powerful UI application and form builder


Getting Started guide needs some help

The "Getting Started" link on crates.io goes to http://doc.crates.io/guide.html, but that page doesn't actually show how to add/use a dependency except via git. The first section of http://doc.crates.io/crates-io.html should probably replace that section of documentation and be moved up the page.

Yeah, I have a general "clean up Cargo docs" thing on my list, this is certainly part of it.

This has since been fixed to show time from crates.io first.


[PLATFORM-454]: [Auth0Ex] Add default middleware for Absinthe

https://prima-assicurazioni-spa.myjetbrains.com/youtrack/issue/PLATFORM-454
This PR adds an Absinthe plug and middlerware developed by @enerdgumen for BBurago.
I am unsure about a few things:

should this plug/middleware live in this repo or in its own repo?
should Absinthe be included as an optional or test-only dependency?

Forgot to answer to

should this plug/middleware live in this repo or in its own repo?

I'd keep it here (if it's not too annoying for dependencies management)

I refactored a bit

absinthe version requirement is now lowered to 1.6
I split the absinthe modules into plug and middleware
I tried to refactor the RequirePermission module


Deployment for goerli

In this PR we add configuration that is needed for Goerli contracts deployment.

Closing in favour of https://github.com/keep-network/coverage-pools/pull/214


Page.Dispose() never completes

Using 0.3.2, if you try to dispose of a Page instance, it awaits forever. Try this, the last Console.WriteLine() call is never reached because it all hangs when the inner using statement calls Dispose on the page instance.
using System;
using System.Collections.Generic;
using System.IO;
using System.Threading.Tasks;
using PuppeteerSharp;

namespace PuppeteerSharpPdfDemo
{
    class MainClass
    {
        public static void Main(string[] args)
        {
            MainAsync().GetAwaiter().GetResult();
        }

        public static async Task MainAsync()
        {
            var options = new LaunchOptions
            {
                Headless = true
            };

            Console.WriteLine("Downloading chromium");
            await Downloader.CreateDefault().DownloadRevisionAsync(Downloader.DefaultRevision);

            Console.WriteLine("Navigating google");
            using (var browser = await Puppeteer.LaunchAsync(options, Downloader.DefaultRevision))
            using (var page = await browser.NewPageAsync())
            {
                await page.GoToAsync("http://www.google.com");

                Console.WriteLine("Generating PDF");
                await page.PdfAsync(Path.Combine(Directory.GetCurrentDirectory(), "google.pdf"));

                Console.WriteLine("Export completed");
                //Console.ReadLine();
            }
            Console.WriteLine("The End"); // this line never reached :(
        }
    }
}

Hey @kemmis. I got the same issue in a benchmark app I'm coding. And as you showed, the demo app is having the same issue.
I was able to get my benchmark working removing the using (var Page
var page = await browser.NewPageAsync();
await page.GoToAsync("http://www.google.com");
Console.WriteLine("Generating PDF");
await page.PdfAsync(Path.Combine(Directory.GetCurrentDirectory(), "google.pdf"));
Console.WriteLine("Export completed");
await page.CloseAsync();

I think there is something related to nested GetAwaiter().GetResult()

I tried calling page.CloseAsync().GetAwaiter().GetResult(); from within my own class' Dispose() method and I have the same problem. Can you confirm that the Target.closeTarget command succeeds?

Did you try await page.CloseAsync(); instead of GetAwaiter()?

Okay that works!

It's something we need to work on, of course. I'm glad you got it running!
(don't close this issue)


[Upload Multi File PHP] Fix "ftbl0cloud.000webhostapp.com" path default when upload on another host and fix code Runtime Error

To fix a folder path issue in GitHub, you can follow these steps:
Open the file that contains the folder path you want to fix.
Locate the folder path in the file and make sure it is correct.
If the folder path contains a backslash ("") instead of a forward slash ("/"), replace it with a forward slash.
Make sure the folder path doesn't contain any spaces or unsupported characters.
If the folder is located in a subdirectory, make sure you include the subdirectory name at the beginning of the folder path, separated by a forward slash.
Save the file with the corrected folder path.
By following these steps, you should be able to fix any folder path issues in your GitHub repository.
Backup Ver.1
bShare-Online-Hosting-main.zip

Avoid upload Shell.php by using this code attached
PHP.txt
`
";
            echo "Tên file: $names[$i] ";
            echo "Lưu tại: 𝐘𝐨𝐮𝐫 𝐩𝐫𝐞-𝐝𝐢𝐫𝐞𝐜𝐭𝐨𝐫𝐲$names[$i] ";
            echo "Cỡ file: $sizes[$i] ";

            move_uploaded_file($tmp_names[$i], 'uploads-cdn/'.$names[$i]);
        } else {
            echo "Upload không thành công: $names[$i] không phải là file ảnh hoặc video hợp lệ";
        }
    }
    echo "Tổng số file upload: " .$numfiles;
}

?>

Chọn file để upload:
      (Cỡ lớn nhất mà PHP đang cấu hình cho phép upload là )
<input name="fileupload[]" type="file" multiple="multiple" />
<input type="submit" value="Đăng ảnh" name="submit">

`


conda-suggest-conda-forge 2020.9.30

Merge only after success.
This pull request was auto-generated by rever

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


SEGSEGV when channel cannot be found

Hi,
I just had a segfault when running matterircd 0.11.4 connected to mattermost team edition 3.9.0.
Here's the error logged by mattermost and matterircd:
Jun 12 15:25:21 mattermost-host platform[30591]: [2017/06/12 15:25:21 UTC] [EROR] /api/v3/teams/p1gqez5gj3frjrxhafsdc45iha/channels/bxm3eth4djfcxggeiofxsk1sdr/:getChannel code=500 rid=fab337h98pf65rnsxsjrenkstr uid=4ux8atkiybrr3c87k63pzu5bgh ip=2a03:b0c0:2:d0::1392:e001 There is no channel with channel_id=bxm3eth4djfcxggeiofxsk1sdr on team with team_id=p1gqez5gj3frjrxhafsdc45iha [details: ]
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: panic: runtime error: invalid memory address or nil pointer dereference
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: [signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x7b238f]
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: goroutine 114 [running]:
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: github.com/42wim/matterircd/vendor/github.com/42wim/matterbridge/matterclient.(*MMClient).parseActionPost(0xc4200779a0, 0xc420500f00)
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: /go/src/github.com/42wim/matterircd/vendor/github.com/42wim/matterbridge/matterclient/matterclient.go:299 +0x4bf
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: github.com/42wim/matterircd/vendor/github.com/42wim/matterbridge/matterclient.(*MMClient).parseMessage(0xc4200779a0, 0xc420500f00)
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: /go/src/github.com/42wim/matterircd/vendor/github.com/42wim/matterbridge/matterclient/matterclient.go:266 +0x7e
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: github.com/42wim/matterircd/vendor/github.com/42wim/matterbridge/matterclient.(*MMClient).WsReceiver(0xc4200779a0)
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: /go/src/github.com/42wim/matterircd/vendor/github.com/42wim/matterbridge/matterclient/matterclient.go:249 +0x662
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: created by github.com/42wim/matterircd/vendor/github.com/42wim/mm-go-irckit.(*User).loginToMattermost
Jun 12 15:25:21 mattermost-host matterircd-0.11.4[29027]: /go/src/github.com/42wim/matterircd/vendor/github.com/42wim/mm-go-irckit/mmuser.go:64 +0x300
Jun 12 15:25:21 mattermost-host systemd[1]: matterircd.service: main process exited, code=exited, status=2/INVALIDARGUMENT
Jun 12 15:25:21 mattermost-host systemd[1]: Unit matterircd.service entered failed state.

I don't know what triggered the channel look up though.
If you need more information, let me know.

This was fixed in master  (see #97).
I just released v0.11.5 (https://github.com/42wim/matterircd/releases/tag/v0.11.5) with the fix in it.