Add Flox as an installation option

tlrc is available and working on Flox.
We're basing the package off nixpkgs at the moment.
$ flox init
✨ Created environment 'tmp.kjM6fKpa47' (aarch64-darwin)

Next:
  $ flox search <package>    <- Search for a package
  $ flox install <package>   <- Install a package into an environment
  $ flox activate            <- Enter the environment
  $ flox edit                <- Add environment variables and shell hooks

$ flox install tlrc
✅ 'tlrc' installed to environment 'tmp.kjM6fKpa47'
$ flox activate -- tldr tar
info: cache is empty, downloading...
info: downloading 'tldr.sha256sums'... 3.27 KiB
info: downloading 'tldr-pages.en.zip'... 2.10 MiB
info: validating sha256sums...  OK
info: extracting 'pages.en'... 5037 pages, 5037 new
info: cache update successful (total: 5037 pages, 5037 new).

  tar

  Archiving utility.
  Often combined with a compression method, such as gzip or bzip2.
  More information: https://www.gnu.org/software/tar.

  [c]reate an archive and write it to a [f]ile:

    tar cf path/to/target.tar path/to/file1 path/to/file2 ...

Hi. What is Flox?

@spageektti Flox is an environment/package manager. Think Python virtual environments, but not just for Python packages.
Imagine you're working on a Rust project and want to make sure your team has the exact version of Rust that you're using.
We can achieve this with Flox as follows.
$ # First, notice we currently don't have Rust installed on my system
$ rustc --version
zsh: command not found: rustc
$ # No problem, let's install it using Flox
$ flox search rust
rustc     A safe, concurrent, practical language
irust     Cross Platform Rust Repl
thrust    Chromium-based cross-platform / cross-language application framework
rustus    TUS protocol implementation in Rust.
rustup    The Rust toolchain installer
mrustc    Mutabah's Rust Compiler
rustfmt   Tool for formatting Rust code according to style guidelines
rustcat   Port listener and reverse shell
ht-rust   Friendly and fast tool for sending HTTP requests
svd2rust  Generate Rust register maps (`struct`s) from SVD files

Showing 10 of 784 results. Use `flox search rust --all` to see the full list.

Use 'flox show <package>' to see available versions

Related search results for 'cargo':
cargo     Downloads your Rust project's dependencies and builds your project
cargo-c   Cargo subcommand to build and install C-ABI compatible dynamic and static libraries
cargo-ui  GUI for Cargo

$ # Cool, it looks like Rust is available, I wonder what versions of Rust are available?
$ flox show rustc
rustc - Safe, concurrent, practical language (wrapper script)
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>(aarch64-linux, x86_64-darwin, x86_64-linux only)
$ # Nice, there's a few versions available
$ # I'm interested in using rustc 1.70.0, let's install that
$ flox install<EMAIL_ADDRESS>❌ ERROR: Did not find an environment in the current directory.

Create an environment with 'flox init' or install to an environment found elsewhere with 'flox install $ flox install<EMAIL_ADDRESS>--dir <PATH>'
$ # Whups, I first have to init just like I have to `git init` when working with git
$ flox init
✨ Created environment 'tmp.MHcfA6tb2O' (aarch64-darwin)

Next:
  $ flox search <package>    <- Search for a package
  $ flox install <package>   <- Install a package into an environment
  $ flox activate            <- Enter the environment
  $ flox edit                <- Add environment variables and shell hooks

$ flox install<EMAIL_ADDRESS>✅ 'rustc' installed to environment 'tmp.MHcfA6tb2O'
$ # Is rust available now?
$ rustc --version
zsh: command not found: rustc
$ # No, I still have to activate the environmnet
$ flox activate
✅ You are now using the environment 'tmp.MHcfA6tb2O'.
 Just use git
$ git init .
Initialized empty Git repository in /private/var/folders/vw/ksgd_r_j51704dss486tzpmc0000gn/T/tmp.MHcfA6tb2O/.git/
$ git add .flox/
$ git commit -m 'Initial commit'
...
$ git push
...
$ # Now when your team `git pull`s this repo
$ # They can `flox activate` and be sure they're working with
$ # the same tools you do

It doesn't only do that, you're also able to set environment variables per project, soon we'll also do some sort of service management, so you can spawn and tear down services when you enter a Flox environment, and you also don't have to use the .flox in your Git repo. We have something called FloxHub that's capable of storing the environment metadata for you, and then it's a simple flox pull away of using that environment.
There's also a whole team behind Flox, trying to make it as good as it can be. So, if you happen to have feedback, I'd love to listen to it. 😊
I hope I answered your question, if I didn't let me know.

Imagine you're working on a Rust project and want to make sure your team has the exact version of Rust that you're using.

For Rust and other stuff needed to build a project, it makes sense. Now why would anyone want to install a tldr client in such a dev environment? I can't think of any scenario where you'd need to make sure everyone on your team is using the same version of tldr - it is not a library, it is not a build tool, it is not something that a project would depend on. It makes way more sense to just use a system-wide package manager for things which are unrelated to the project.
Or am I wrong?

@acuteenvy

Now why would anyone want to install a tldr client in such a dev environment?

E.g. when pair programming, it's straightforward to compose a Flox environment that has everything necessary to get going, and that could include tools like tlrc.
Or, when you'd like to bring your environment across many systems. Let's say that you have many machines, some are Linux, some are macOS, and some others run windows WSL. Flox makes it easy to get the same tools on all those systems.

I can't think of any scenario where you'd need to make sure everyone on your team is using the same version of tldr - it is not a library, it is not a build tool, it is not something that a project would depend on.

I do understand that, but I'm also not really looking at tlrc as a dependency, rather as a nice to have next to your build tools and libraries.

It makes way more sense to just use a system-wide package manager for things which are unrelated to the project.

I think that's also the use-case I'm thinking of. When Flox is used in one's home directory, it almost acts as a user-wide package manager. It just so happens that I can also use it as a project-wide package manager.
The way I'm currently using Flox is by having a "toolbox" environment that I overlay with my project-specific environments. That toolbox environment, includes tlrc and similar tools that aren't really a dependency of your project. This workflow looks as follows.
$ # Let's see what's in the "toolbox" environment.
$ # Note the `-r`, which means that we're accessing a remote environment from floxhub.
$ flox list -r bryanhonof/toolbox
tlrc: tlrc (1.9.0)
_1password: _1password (1password-cli-2.26.1)
fd: fd (9.0.0)
jq: jq (1.7.1)
ripgrep: ripgrep (14.1.0)
upterm: upterm (0.13.2)
$
$ # I currently don't have upterm, but I'd like to start pair programming
$ upterm version
zsh: command not found: upterm
$ # So, let's activate this remote environmnet
$ flox activate -r bryanhonof/toolbox
✅ You are now using the environment 'bryanhonof/toolbox (remote)'.
To stop using this environment, type 'exit'

flox [bryanhonof/toolbox (remote)] $ upterm version
Upterm version v0.13.2
flox [bryanhonof/toolbox (remote)] $
flox [bryanhonof/toolbox (remote)] $ # Now I still need my project specific tools
flox [bryanhonof/toolbox (remote)] $ # Let's see what's in this local Flox environment
flox [bryanhonof/toolbox (remote)] $ pwd
/Users/bryanhonof/Developer/bryanhonof/my-awesome-project
flox [bryanhonof/toolbox (remote)] $ flox list
> List using which environment? current directory [my-awesome-project]
rustc: rustc (wrapper-1.79.0)
flox [bryanhonof/toolbox (remote)] $ # A version of rustc, cool
flox [bryanhonof/toolbox (remote)] $ flox activate
✅ You are now using the environment 'my-awesome-project'.
To stop using this environment, type 'exit'

flox [my-awesome-project bryanhonof/toolbox (remote)] $ rustc --version
rustc 1.79.0 (129f3b996 2024-06-10) (built from a source tarball)

You can imagine how this would continue. But note how both environments merged. This provides me with access to the project-specific dependencies, and my auxiliary tools I use across multiple projects.
Now, I also understand if you'd rather not have a PR like this one merged. I just thought it was cool that tlrc was available in Flox today, and wanted to spread the word. 😊

If Flox pulls its packages from nixpkgs, its users will know that it's possible to install tlrc, since Nix is already mentioned on the readme. I don't think including multiple ways of installing packages from a single repository is a good idea. The most common method of installing from nixpkgs is already mentioned.

I agree with @acuteenvy

Understandable, I'll close the PR. We can always revisit if we ever feel the need to. 😊


Files with upper case extensions are not transformed

Falls back to craft transforms when files have an uppercase extension when useForNativeTransforms: true.
Prob no big deal, unless IMGIX is used, where we're trying to avoid using craft transforms.

Has been merged to 2.1.5. Thanks for contributing!


Windows SSH Server conflict with tmpfs in WSL2

It is about #5703, I have already reproduce it, but I don't know why it was closed by @therealkenc
Environment
windows 10 build 19041.388
WSL2
yuk7/ArchWSL(maybe useful, not test in other repo)
Steps to reproduce

use /etc/fstab to mount tmpfs to /tmp
run tmux
use another machine to ssh into this windows and run wsl to get into this wsl2
try run tmux attach and fail

Expected behavior
get tmux session attached
Actual behavior
show no session
what else
my wsl2 user is the same, no problem here, both process started from ssh and the process starting tmux first are root user process

Try reddit, serverfault, arch forum, etc.
The Windows SSH server does not conflict with tmpfs. [The premise doesn't even make sense.]


User registration and password-based login

@danohuiginn to submit PR :)

hey @danohuiginn -- now that aleph.openoil.net has been born to great acclaim, any chance I can tease you into backporting this into master :)

Yes please!

Note to self: folks at Süddeutsche Zeitung would like to see integration with AD/LDAP here, which should not be too hard once we've got a password prompt in the first place.
/cc @vwormer

This should be ready for a review. I'll add LDAP support in a separate branch.

Both, support for stand-alone users and LDAP support was merged.
I'm closing this.


Feature request: ListView

Feature request
Thank you for your amazing work. I'm having a problem with ListView, as shown in the following code and images. When I use import 'package:flutter/material.dart';, there seems to be a spacing between the images in some window sizes.
import 'package:flutter/gestures.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Tests',
      home: const MyHomePage(),
      debugShowCheckedModeBanner: false,
      scrollBehavior: const MaterialScrollBehavior().copyWith(
        dragDevices: PointerDeviceKind.values.toSet(),
      ),
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.black,
      body: ListView(
        children: List.generate(10, (index) {
          return const Image(
            image: NetworkImage('https://placehold.jp/fff/ccc/3333x317.png'),
          );
        }),
      ),
    );
  }
}

When I try to use import 'package:pixel_snap/material.dart';, the behavior changes to always having one pixel between images at some window sizes.

Although this looks slightly better, it's still not "pixel-perfect". Could you please provide support for ListView, or give me any other suggestions to try?
If you think this is not a feature of the current plugin and that this is a problem with Flutter itself, I will close this feature request. Again, thank you for your work.
Environments

Flutter version: 3.3.3
Windows scale: 125%
Windows version: 21H2
Test application size: 1584 x 892 px (Default)

The problem here is actually that the image itself is not pixel-snapped and to pixel-snap it, 0.0.1 wraps it in another widget. But that leaves some space around.
Can you try 0.0.2 and let me know if it works better?

I added a forked version of FittedBox to 0.0.3, which itself should be pixel aligned (so it doesn't break parent layout pixel alignment), but I don't think it's always possible to have pixel aligned child, especially when it uses a scale transform. If you have example of FittedBox misbehaving you can provide one and I'll see if I can improve on it. But it general using FittedBox for part of app that needs to be pixel aligned is probably bad idea.
As for third-party packages, it really depends. There is no silver bullet here I'm afraid. If the widget lets you specify size, border, decoration, child, etc. you can .pixelSnap() those. If not but the widget is fully constrained by your layout then it may not render pixel snapped, but it shouldn't break your layout.
If the widget is not constrained and it sizes itself, you can wrap it with PixelSnapSize. It will not magically make the widget pixel-snapped, but will add enough space around it so that it doesn't break the rest of your layout.

Thank you very much. I mainly want to solve the pixel gap problem with the ListView and other widgets mentioned above. So I may need to fork or copy the code for custom modification, such as changing the default ceil mode of FittedBox to snap mode to fit more window sizes.

I implemented my snap mode FittedBox widget, which has additional scale handling and works perfectly. I'll close the issue for now.


Need for function that takes in Date object & returns formatted string

Date should be returned as a string in the form:  YYYY-MM-DD hh:mm
Single digit months, days, hours, and minutes should all have a leading zero. Also add one to return value of Date.getMonth() because 0 indexing.

I will be addressing this on an email notifications feature branch.


Added new performance test - 1KB message (description), 50 threads

is it perhaps worth converting https://github.com/almighty/almighty-jobs/blob/master/devtools-ci-index.yaml#L421 into a template segment ? and add this to match that ?

Hi KB,
Yes - that's a great idea - will do it on Thursday - we plan on creating
multiple related jobs soon.
Thanks!,
Len
On Wed, Mar 8, 2017 at 9:17 PM, Karanbir Singh<EMAIL_ADDRESS>wrote:

is it perhaps worth converting https://github.com/almighty/
almighty-jobs/blob/master/devtools-ci-index.yaml#L421 into a template
segment ? and add this to match that ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/almighty/almighty-jobs/pull/107#issuecomment-285234248,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnOPYg2ylpsP7hUy6nVKfS6AcOOgpLtks5rj2FTgaJpZM4MV181
.

--
Len DiMaggio<EMAIL_ADDRESS>JBoss by Red Hat
314 Littleton Road
Westford, MA 01886  USA
tel:<PHONE_NUMBER>
cell:<PHONE_NUMBER>
http://www.redhat.com
http://community.jboss.org/people/ldimaggio


feat: add support for 3rd party resolver

This PR closes Custom Resolvers #156.
Created this so someone can create an Angular resolver now we have Angular working with vite

I think we need some new interface for resolver, because you still need to handle the other parts like stringify dynamic import for different frameworks.

Looking at the work from Angular. It seems to work ok, However, I do agree a new interface would be better as I notice Vue is different from the others.
I'll work on the new interface and update

@hannoeru
I was thinking Maybe the resolver doesn't need a different interface it makes sense as a simple resolver. My thinking is to add a Page controller (taken from page context)
In this regard people can extend from the current PageContext and override the current functionality. This way people don't have to implement everything unless it's required.
Option 1:
export interface IPageControl {
  setLogger(logger: Logger): void
  setUserOptions(userOptions: UserOptions): void
  setViteInfo(viteRoot: string): void

  addPage(path: string | string[], pageDir: PageOptions): void | Promise<void>
  removePage(path: string | string[]): void
  checkCustomBlockChange(path: string): void | Promise<void>
  resolveRoutes(): Promise<string>
  onUpdate?(): void
  customBlockMap: Map<string, CustomBlock>
  pageRouteMap: Map<string, PageRoute>
  debug: Record<string, Debug.Debugger>
}

export interface ResolverConstructor {
  // eslint-disable-next-line @typescript-eslint/no-misused-new
  new(userOptions: UserOptions, viteRoot: string): Resolver
}
export interface Resolver {
  setLogger(logger: Logger): void
  setUserOptions(userOptions: UserOptions): void
  setViteInfo(viteRoot: string): void
  resolveRoutes(ctx: IPageControl): Promise<string>
}

Option 2: make the Resolver class have the IPageController
export interface ResolverConstructor {
  // eslint-disable-next-line @typescript-eslint/no-misused-new
  new(userOptions: UserOptions, viteRoot: string): Resolver
}
export interface Resolver {
  setLogger(logger: Logger): void
  setUserOptions(userOptions: UserOptions): void
  setViteInfo(viteRoot: string): void
  resolveRoutes(): Promise<string>

  addPage(path: string | string[], pageDir: PageOptions): void | Promise<void>
  removePage(path: string | string[]): void
  checkCustomBlockChange(path: string): void | Promise<void>
  onUpdate?(): void
  customBlockMap: Map<string, CustomBlock>
  pageRouteMap: Map<string, PageRoute>
  debug: Record<string, Debug.Debugger>
}

In option two, I'll create a base class which is an abstraction of PageContext. We remove the functions intro another class which then allows people to extend  the original class and overwrite the functionality


Implement Num.toF64

I'm going to work on adding this builtin function.

This function exists now.


Apt repositories do not make SDK available on Ubuntu Linux

This has been broken for quite some time. Developers are forced to build from source. There is no option (at least on Ubuntu) to install from your repository. We have tried on multiple systems to install the SDK via apt. Here are the steps for Ubuntu 20.04 LTS:

curl -sSL https://packages.microsoft.com/keys/microsoft.asc | sudo apt-key add -
sudo apt-add-repository https://packages.microsoft.com/ubuntu/20.04/prod
sudo apt-get update
sudo apt install k4a-tools

Message received is: E: Unable to locate package k4a-tools
Please fix, as it is inconvenient to build from source every time we need the tools on our dev machines.

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

ID: cf93cee5-3ed0-7500-aee5-da8e0f6941b6
Version Independent ID: 89db53d9-22b9-35cf-1d98-22f6cdaeaef9
Content: Azure Kinect Sensor SDK download
Content Source: articles/kinect-dk/sensor-sdk-download.md
Product: kinect-dk
GitHub Login: @Brent-A
Microsoft Alias: brenta

@Brent-A Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@VisionaryMind I believe you are using ARM. If that is the case you need to use https://packages.microsoft.com/ubuntu/18.04/multiarch/prod instead of https://packages.microsoft.com/ubuntu/18.04/prod. This is documented here https://github.com/microsoft/Azure-Kinect-Sensor-SDK/blob/develop/docs/usage.md#debian-package.

@VisionaryMind I believe you are using ARM. If that is the case you need to use https://packages.microsoft.com/ubuntu/18.04/multiarch/prod instead of https://packages.microsoft.com/ubuntu/18.04/prod. This is documented here https://github.com/microsoft/Azure-Kinect-Sensor-SDK/blob/develop/docs/usage.md#debian-package.

@VisionaryMind Did you get a chance to see Quentin response? We hope that answers your query.
Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.

@VisionaryMind Did you get a chance to see Quentin response? We hope that answers your query.
Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.


Don't transmit typing events from ourselves

Typing events from the bridge results in an "infinite loop" of typing events

LGTM, thx


index error may be caused by non-standard fields or variant type

hello professor. @mdbarnesUCSD
I rerun the code in the Untubu Linux.
I find that only one vcf file of 5 in my test can be run correctly.
I ask copilot for answers.
He tells me that"If your VCF file contains non-standard fields or variant types, or if the variant types include insertions (INS), deletions (DEL), and replacements (RPL), the SigProfilerMatrixGeneratorFunc function might not be able to correctly process the file.  Therefore, to ensure that your VCF file can be correctly read, you should make sure that your VCF file meets the format and content requirements of a standard VCF file."
Is it really true?
Waiting for your help

the  non-standard fields or variant type are like  HOMLEN，SVLEN，TYPEOFSV answered by copilot


Fix the broken Sdop.__eq__, and add the missing __ne__ operators.

ne is needed by python 2 (https://stackoverflow.com/q/4352244/102441), else it defaults to x is not y
Also:

move some tests that ended up in the wrong place after a merge/rebase
use the helper alias methods in Ga to increase coverage

Rebased since codecov got confused somehow

Introduced codebeat issue is a feature, raising that Sdop and Dop are extemely similar - waiting for a response on slack at https://clifforddev.slack.com/archives/CGKEFNRD0/p1576754956000500.

^ That was what I was aiming for


IndexOptimize @Indexes excluded table included in output

Including a space after an index name when excluding indexes results in the excluded index being included in the results.
Here is an example of the @Indexes parameter with a space before the comma after the first excluded index. This results in Table1 being included instead of excluded.
@Indexes = 'ALL_INDEXES, -MyDB.dbo.Table1 , -MyDB.dbo.Table2'
Removing the space before the comma results in the expected behavior of excluding the table.
@Indexes = 'ALL_INDEXES, -MyDB.dbo.Table1, -MyDB.dbo.Table2'
The execution results for both scenarios is attached.
ExecutionOutput.txt

I tested the change and it is working. Thanks, Ola!


Adding parentelement

ParentElement.tk is a springboard that identifies the browser
and serves targeted malware. Managed to crash my iPad

@ddalex thx for sharing the info :smiley:

I'll take it.
Thanks Alex @ddalex.
Merging.

Thank you!
On Mon, 15 Apr 2019 at 14:57, Steven Black<EMAIL_ADDRESS>wrote:

I'll take it.
Thanks Alex @ddalex https://github.com/ddalex.
Merging.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/StevenBlack/hosts/pull/952#issuecomment-483262306,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABk_PM2IZJ7yzXpQPkwjFTwQJXSgvUpNks5vhIVJgaJpZM4cq_aR
.

--
Alex Damian | Technical Lead | Time Out Media
77 Wicklow Street | London | WC1X 9JY
T: +44<PHONE_NUMBER>
timeout.com

https://www.timeoutmarket.com/*

--
Time Out is the trusted global platform that inspires and enables people to
experience the best of the city.
The information contained in the e-mail
(and any attachments) may be confidential and/or legally privileged and is
solely for the attention of those to whom it was intended to be sent. If
you are not the intended recipient, you are notified that any review,
distribution or copying of this e-mail (and any attachments) is prohibited
and that legal privilege has not been waived. If you have received the
e-mail in error please let us know by e-mail (to the sender of the email)
and delete the e-mail (and any attachments) immediately from your system.
Time Out Group plc: reg no 07440171 (England and Wales) VAT No: 135615325
registered at 77 Wicklow Street, London WC1X 9JY. Time Out England Limited:
reg no 01782049 (England and Wales) VAT No: 135615325 registered at 77
Wicklow Street, London WC1X 9JY.


Tech Onboarding GDX Analytics to AWS Prod

Describe the issue
Search in the onboarding app for "gdx analytics" for the details
Additional context

Deciding which app to use will be key

Definition of done

Create the project-set and perform the onboarding training
Schedule a meeting with their tech contact Mark W (see the app entry)
The team says they want VMs, this one might be a first instance which could affect if we have a dedicated meeting only for this onboarding or if you batch them with the APS team (current though is better to separate but team decision)

Not sure who will be assigned to this task (Max?). but please include me. I would like to shadow you in the process to create the project set to be able to document the steps (with the goal of automating them if possible)


Minimum absolute difference

Coverage increased (+0.1%) to 88.535% when pulling 2b72e53ca54196b25d26839c3439820debffcbe1 on minimum-absolute-difference into f2e8605acd0eb2a6401485fc20dffccee9f77936 on master.

Coverage increased (+0.1%) to 88.535% when pulling 4f82016489cea7f052775a3c974f19f3122efddb on minimum-absolute-difference into f2e8605acd0eb2a6401485fc20dffccee9f77936 on master.


Exif support for image upload?

I am uploading image but it is getting rotated. Are exif meta tags supported in this library?

Please look at #374


Dracord ODD does not permit use of docImprint or byline on titlePage

I am building a corpus in which title pages are quite extensively marked up. In particular I use the standard TEI elements <docImprint> and <byline> to distinguish the relevant parts of the title page. These elements have (for some reason) been removed from the Dracor ODD: why? and how do other projects markup these components in their absence ?

Ooops, sorry : I see I posted this in the wrong place. Plz ignore!


Fail code on build

Please help if possible.
Win10 Visual Basic C++
error[E0308]: mismatched types
   --> C:\Users\Buddy\.cargo\registry\src\github.com-1ecc6299db9ec823\autopilot-0.1.4\src\bitmap.rs:475:20
    |
475 |         bmiColors: [],
    |                    ^^ expected an array with a fixed size of 1 elements, found one with 0 elements
    |
    = note: expected type `[winapi::um::wingdi::RGBQUAD; 1]`
               found type `[_; 0]`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0308`.
error: Could not compile `autopilot`.

To learn more, run the command again with --verbose.
error: cargo failed with code: 101

Thanks for the report, this is now fixed on the latest master.


Fixed bug in Slide.vue for IE Compatibility

Changed use of array.includes(...) to array.indexOf(...) for IE compatibility
Has been tested in production on IE11.

Coverage remained the same at 69.528% when pulling c560a021232679b9e0fed99021d59f0bddf505eb on ENAndrew:internet-explorer-fix into c98e676c920b0d1e227fe96724d7a77db6e39f07 on SSENSE:master.

Hey! Thanks @ENAndrew, this pairs well with another fix going out in this patch so I'll merge this in as-well.  Really happy to have this issue squashed, thx for the contribution :~) :octocat:


Provide platform "Copy As..." submenu

Provide a editor/context/copy core submenu, in the editor context menu.
cc @alexr00, @eamodio (gitlens) for adoption once this is in.

Quick question, once we have this sub menu. Should "Copy as Plain Text" and "Copy as Rich Text" (w/ syntax highlighting) as part of the menu? Also these two are builtin commands, do they always show at the very top?

Yes, I think so. cc @egamma

I'd think the Copy Relative Path should too

@alexr00 I forgot to update the original post in this issue: there's also the menuBar/edit/copy submenu, in case you're interested in being there as well.

@joaomoreno  👍 already adopted both menu entries in GHPRI.

This will be verified via @alexr00's feature. @alexr00 can you mark this as verified?


Feature Request

Missing if block, could you add this and its variations? There is no feature request template available.

Could you elaborate a bit more ? As far as I can see, an if block is highlighted just fine. Maybe post some sample code blocks so I can understand.

Sorry I did not distinguish between snippet and syntax. It is the snippet that is missing, Highlighting works fine.

Snippets are more of a user thing (meaning snippets are not generic, some users may like them and some don't) than a package providing them (& you can very easily create your own snippets & use them). This package only ships with a few snippets for basic blocks & I am not interested in increasing them. You can find more information on how to create snippets here https://www.sublimetext.com/docs/completions.html#snippets
I am going to close this as a wont fix.


Fix archive-precomputed-blocks

This PR fixes the parsing of some precomputed blocks that were received over GraphQL. Tested on
this block, which fails without this change.
Checklist:

[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

Is this the right branch to make this change on? Should it be compatible instead?

Is this the right branch to make this change on? Should it be compatible instead?


GH-475 Fix variable property info init for class types

Fixes #475

Applied to main (=2.1) and branch 2.0.


Buttons for "Skip" or "Done" actions

I feel little bit frustrated to select "Skip" or "Done" task from the action menu. Especially when skipping multiple tasks at the same time to seek for something that is doable within a task list.
Every time you have to skip, you need two clicks in the timer view.
Having shortcut buttons to click directly above/below/corner of the task would make it way faster and intuitive to use.
Using shortcut keys would perhaps work also.

Agreed, I also want to take a look at adding a quick button for toggling done to the list

as there is already delete and edit.

I started to use again after changing jobs. First improvement idea came into my mind again, and apparently I did wrote an issue about this. So, I'm +1 my own issue after forgetting it 😆
It looks that the source code is somewhere at https://github.com/zidoro/pomatez/blob/d802264f6e01f1b6ba4d4477d09a1e746d4b7f97/app/renderer/src/routes/Timer/PriorityCard.tsx#L80C9-L80C32


KTable as input binding doesn't work if @input is not specified at parameter level

For KStream we can specify binding as
@StreamListener("input")
public void process (KStream inputStream)
However, this doesnt work for KTable. Throws Illegal Argument Exception "Method must be declarative". I had to explicitly specify as follows to make the binding work.
@StreamListener
public void process(@input("input") KTable inputTable)
is this expected behavior?
Code I am attempting is something like this:
This is working fine:
`
@EnableBinding(TradeMsgReaderTopics.class)
public class TradeMsgReaderProcessor
{
private static final Logger logger = LoggerFactory.getLogger(TradeMsgReaderProcessor.class);
@StreamListener
public void process(@Input(TradeMsgReaderTopics.INPUT) KTable<String, TradeMessageEvent> inputTable)
{
inputTable.toStream().foreach((key, value) -> logger.debug("key: {} :: value: {}", key, value));
}
}`
This is not working:
`@EnableBinding(TradeMsgReaderTopics.class)
public class TradeMsgReaderProcessor
{
private static final Logger logger = LoggerFactory.getLogger(TradeMsgReaderProcessor.class);
@StreamListener(TradeMsgReaderTopics.INPUT)
public void process(KTable<String, TradeMessageEvent> inputTable)
{
inputTable.toStream().foreach((key, value) -> logger.debug("key: {} :: value: {}", key, value));
}
}`
The binding interface is:
`@configuration
public interface TradeMsgReaderTopics
{
String INPUT = "tradeMsgEventStore";
@Input("tradeMsgEventStore")
KTable<String, TradeMessageEvent> tradeMsgEventStore();
}`

@tambre The fix for this issue is pushed to master and backported to 2.1.x.


#690 choices can be null look at OpenAiApi.java:606

Thank you for taking time to contribute this pull request!
You might have already read the [contributor guide][1], but as a reminder, please make sure to:

Sign the contributor license agreement
Rebase your changes on the latest main branch and squash your commits
Add/Update unit tests as needed
Run a build and make sure all tests pass prior to submission

thanks, merged as dea7ab67c3c3312e26cedf8c17e7439579314bc8


Windows position and size, with taskbar in Win11

Hello. I found this program a great companion. Using Janet, I made my Win11 more ergonomic than my hand-crafted River on Wayland setup on Linux.
One thing with which I need help, though, is the size of the windows. They do not take the Windows Taskbar into account, and the bottom of the window is obscured. It is not always, and I still need to understand the pattern.
I would like to ask you for guidance in debugging so that I can understand and fix the behavior.

They do not take the Windows Taskbar into account, and the bottom of the window is obscured.

Did this always happen to specific windows? All I can think of right now is that maybe those windows have minimum size limits, and they "overflowed" from the frames at the bottom. Can you take a screenshot when this happens, and see if those windows are of their minimum sizes? If that's the case, the :frame-to-window-size command may be able to fix it.
Windows (the OS) calls the desktop space excluding the taskbar "work areas", Jwno should have accounted for the taskbar here and in this function.
That said, I run Windows 10 on my machine, and Jwno is only tested in Win 11 VMs. Since Microsoft recently removed download links to their free VM images, I didn't really test Jwno on Win 11 lately. Maybe it's a Win 11 thing. I'll try to get a Win 11 setup ASAP.

Thank you very much! I am happy to test Win11 every day. After writing about this issue, I set the taskbar to hide, and the problem did not manifest. But after setting it to always show, the problem is back. Please see the screenshot of the Zulip client.

One more thing. Please, no hurry; with the taskbar hiding, I am fine.

... I set the taskbar to hide, and the problem did not manifest. But after setting it to always show, the problem is back.

I see. When the space reserved by the taskbar is changed, Windows (the OS) would send some message to Jwno IIRC, but Jwno doesn't handle it currently 😅.
In this case, can you try Update Monitor Layout from Jwno's notification icon menu, and see if the work area can be resized properly, thus fixing the problem?

I wonder if this is the same problem you described in the original issue though.

They do not take the Windows Taskbar into account, and the bottom of the window is obscured.

Does this still happen when you're not changing the taskbar settings at all?

Update Monitor Layout command fixes it.
It happened before I started to do anything with the taskbar settings.
I will investigate more later. Thank you.

Great, thank you for the feedback!
You can start Jwno like this, to have its logs written to a file:
jwno.exe --log-level debug --log-file <file name> --no-console

So that we may be able to do a postmortem if it happens again.
And I should indeed test it more on Win 11. I just found a bug when playing with Win 11's Paint app, but that's another story 😅

After some more investigation, I have found that it happens when I switch between one and two monitors on the notebook. I have not yet switched debugging on, but when I have more time, I will.
Anyway, I am so used to jwno after that couple of weeks that I cannot imagine using Windows without it :-D.

I found that it happens when I switch between one and two monitors on the notebook.

Do you mean that it happens when connecting/disconnecting to an external monitor?
Do the monitors have different scale settings? You can check the scale in Settings app -> System -> Display -> Scale & layout section:

There may be edge cases when connecting/disconnecting monitors, that I'm not aware of, if your scale settings for each monitor are different.
I played with my Virtualbox VM a bit, emulating monitor connection/disconnection by enabling/disabling the second virtual screen, but could not reproduce the issue locally. Steps to reproduce it would be of great help 🙏.

I'm closing this issue then. Feel free to re-open it if you have more info, or would like to tackle it again.


How can i add the props (if i want to use custom props) to the banner ?

Hello !
How can i add the props object to the banner component in case i want to use custom props.
When i add the component like described in the readme:

how can i change the cookie settings via the prop object ?
(i am asking because this is not shown)

Same question here... SvelteKit Noob...

In your component :
<GdprBanner {...props} on:analytics={initAnalytics} />

<script>
    const props = {
  /**
   * You must set the cookie name.
   **/
  cookieName: 'beyonk_gdpr',

Not sure what you mean by props - do you mean different options in the list?


Fixing broken links to Dockerfile locations (#1293)

Fixing broken links to Dockerfile locations.  Removing jupyterhub link since the code for jupyterhub was previously removed from the repo.

This change is

/assign @ironpan

/assign

This change can be previewed at:
https://deploy-preview-1635--competent-brattain-de2d6d.netlify.com/docs/reference/images/

/ok-to-test

/lgtm
/approve


Herostratus not asking for elevation on Windows

This happens when packaged for Windows. The app doesn't ask for elevation nor does anything. Completely broken.

Notes:

The app works if right-click, open as administrator.
The app is completely packaging from Windows.

It seems to be working with a just packaged app from Windows 10 with v0.35.2.

We fixed the elevation dialog not appearing, but the app still fails because it can't find a script called invisible.vbs in the asar package called from within windosu.

Works beautifully after inlining invisible.vbs in the *.cmd files.

Closing this issue since we moved from windosu to elevator, which doesn't have this issue: https://github.com/resin-io/herostratus/issues/29.


Problem found by using openblas-0.2.19-1.3-linux-x86_64.jar with docker container

I use openblas with java(openblas-0.2.19-1.3-linux-x86_64.jar). When I run my app in docker container which, the app start 7 threads like the following:
Thread 465 (Thread 0x7f2e875da700 (LWP 138126)):
#0  0x00007f2fda5e6705 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib64/libpthread.so.0
#1  0x00007f2e879164cb in blas_thread_server () from /home/sankuai/.javacpp/cache/openblas-0.2.19-1.3-linux-x86_64.jar/org/bytedeco/javacpp/linux-x86_64/libopenblas.so.0
#2  0x00007f2fda5e2df3 in start_thread () from /lib64/libpthread.so.0
#3  0x00007f2fd9ef52cd in clone () from /lib64/libc.so.6

which I set OMP_NUM_THREADS=8.

when I unset OMP_NUM_THREADS,  it start 7 blas_thread_server threads
when I set OMP_NUM_THREADS=1, no blas_thread_server thread start

The problems is that these 7 blas_thread_server threads consume a lot of cpu time
y.  but the same code in the virtual machine has no problem.
Any ideas?
@xianyi Can you give me some tips?
PS： I found the openblas code https://github.com/xianyi/OpenBLAS/blob/48f075cfd52b73bf77d7627fcb425392c4d6bf24/driver/others/init.c#L96
and in my container:
#  cat /sys/devices/system/node/node0/cpumap
00000000,00000000,00000000,00000000,00005555,55555555
#  cat /sys/devices/system/node/node1/cpumap
00000000,00000000,00000000,00000000,0000aaaa,aaaaaaaa

but in vm
$  cat /sys/devices/system/node/node0/cpumap
00000000,00000000,00000000,000000ff

Is it related to this?

Can you run a dynamic profiler like perf to understand which functions you call from BLAS?
Those threads do their part of numeric processing, in single thread case same work is done on main thread and one cpu only , N times slower.

@brada4 thanks for you reply. this is my flame-graph, which shows the unknown function.

any ideas? thanks!

you have to profile native library behind java jni. Like perf record ; perf report
We do not ship java wrappers, if they are made they work with any BLAS interface.

Not sure what kind of effort goes into creating the .jar, but perhaps it would already help if a newer one was available. 0.2.19 was released a good two years ago and I recall there were a few problems with cpu enumeration, thread locking efficiency and cpu affinity around that time.

The "unknown" ir some OpenBLAS assembly kernel that goes unidentified because of missing debug symbols in library.
Probably worth getting in touch with middleman making that JAR package to keep debug symbols in the OpenBLAS native library (and hoping down the road they update to 0.3.7)
Currently given scheduling operations it boils down to hundred or so possible entrances into BLAS.

Actually there is a newer version already available:
https://repo.maven.apache.org/maven2/org/bytedeco/javacpp-presets/openblas/0.3.5-1.4.4/

@0x0916 maybe by chance you know which BLAS or LAPACK function is called, though with 2yo product version you wont get more help than list of fixes in that function during last years

Actual source of java components found here (at v0.3.7 as of today):
https://github.com/bytedeco/javacpp-presets/tree/master/openblas
It calls CBLAS and LAPACKE generating java API from included C header files.

Actual source of java components found here (at v0.3.7 as of today):
https://github.com/bytedeco/javacpp-presets/tree/master/openblas
It calls CBLAS and LAPACKE generating java API from included C header files.

Thank you! I will update the openblas to 0.3.5-1.4.4 to see if the problem exists.
If it still EXISTS, how about to compile openblas jar with debuginfo?

Probably you use "install" target in place of "strip-install" when building bytedeco jar file. Thats outside OpenBLAS, we dont undermine debugging for obvious need stated above ;-)

What is the host system driving docker containers?

What is the host system driving docker containers?

CentOS 7.5

I just suspected different problem, but CentOS not at fault.


Conversion of dates

Hi,
When selecting the data timestamps are properly converted to Base.Dates.DateTime but dates are fetched as strings. Is there any reason not to convert it to Base.Dates.Date?
result = LibPQ.execute(connection, "SELECT d, dt FROM public.test;")
data = DataStreams.Data.stream!(result, DataStreams.Data.Table)
dump(data)

_NT_d_dt{Array{Union{Missings.Missing, String},1},Array{Union{DateTime, Missings.Missing},1}}
  d: Array{Union{Missings.Missing, String}}((1,))
    1: String "2018-01-01"
  dt: Array{Union{DateTime, Missings.Missing}}((1,))
    1: DateTime
      instant: Base.Dates.UTInstant{Base.Dates.Millisecond}
        periods: Base.Dates.Millisecond
          value: Int64<PHONE_NUMBER>0000

Thanks,
Piotr

This was an oversight! There is no reason to not have this. Thanks for the report.

Thank you for a lovely LibPQ, it's hugely appreciated.  Anyway, this oversight just bit me as well.

How do we add the LibPQ.LIBPQ_TYPE_MAP and LibPQ.LIBPQ_CONVERSIONS to automatically convert to Base.Dates.Date type?


Flow Diagram: verify user has permissions and diagram exists (if upating)

re-use lineage editor role.

@kamransaleem


Fix vignette aspect ratio when not using postProcessing

If you're not using the post processing rendering pipeline (i.e. rendering directly to the canvas drawing buffer) then the vignette aspect ratio will be wrong.
This happens because when binding the imageProcessingConfiguration in the material, no value is provided for the aspectRatio parameter, whereas when binding via the post processing pipeline, the correct aspect is provided. This change resolves this by using the current render target (or canvas drawing buffer) aspect ratio when the aspect ratio isn't supplied. And semantically, the aspect ratio parameter has been changed to overrideAspectRatio to clarify

In the gif, the circular vignette is correct

Hey @haxiomic it is fun to see you there again :-) Thanks for the fix

It's been a little while @sebavan! :)

@sebavan It's been great to see all the new stuff in Babylon! I'm really loving the editor. Man that makes life so much easier!


Blog App: validations and model specs

In this [PR] I have:

Add validation for the following ( name, posts_counter ) in User model.
Add validation for the following ( title, comments_counter, likes_counter) in Post model.
Add tests for the all models (User, Post, Comment, and Like).

Thank you @AshakaE for the code review.


[google_maps_flutter] Fix UIKit availability and pod lint warnings

Description
Fix google_maps_flutter UIKit availability warnings introduced in https://github.com/flutter/plugins/pull/2607.  Also fix pod lib lint warnings:
 -> google_maps_flutter (0.0.1)
    - WARN  | license: Missing license type.
    - WARN  | description: The description is equal to the summary.
    - WARN  | [iOS] keys: Missing primary key for `source` attribute. The acceptable ones are: `git, hg, http, svn`.
    - WARN  | [iOS] xcodebuild:  /Users/m/Projects/plugins/packages/google_maps_flutter/google_maps_flutter/ios/Classes/GoogleMapController.m:176:7: warning: 'UIGraphicsImageRendererFormat' is only available on iOS 10.0 or newer [-Wunguarded-availability]
    - WARN  | [iOS] xcodebuild:  /Users/m/Projects/plugins/packages/google_maps_flutter/google_maps_flutter/ios/Classes/GoogleMapController.m:176:78: warning: 'defaultFormat' is only available on iOS 10.0 or newer [-Wunguarded-availability]
    - WARN  | [iOS] xcodebuild:  /Users/m/Projects/plugins/packages/google_maps_flutter/google_maps_flutter/ios/Classes/GoogleMapController.m:176:48: warning: 'UIGraphicsImageRendererFormat' is only available on iOS 10.0 or newer [-Wunguarded-availability]
    - WARN  | [iOS] xcodebuild:  /Users/m/Projects/plugins/packages/google_maps_flutter/google_maps_flutter/ios/Classes/GoogleMapController.m:178:7: warning: 'UIGraphicsImageRenderer' is only available on iOS 10.0 or newer [-Wunguarded-availability]
    - WARN  | [iOS] xcodebuild:  /Users/m/Projects/plugins/packages/google_maps_flutter/google_maps_flutter/ios/Classes/GoogleMapController.m:179:13: warning: 'UIGraphicsImageRenderer' is only available on iOS 10.0 or newer [-Wunguarded-availability]
    - WARN  | [iOS] xcodebuild:  /Users/m/Projects/plugins/packages/google_maps_flutter/google_maps_flutter/ios/Classes/GoogleMapController.m:181:53: warning: 'UIGraphicsImageRendererContext' is only available on iOS 10.0 or newer [-Wunguarded-availability]
...
[!] google_maps_flutter did not pass validation, due to 9 warnings (but you can use `--allow-warnings` to ignore them).

Related Issues
Fixes https://github.com/flutter/flutter/issues/55258
https://github.com/flutter/flutter/issues/55245
Dependency to merge https://github.com/flutter/plugin_tools/pull/97
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the [Flutter Style Guide].
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

since this isnt a breaking change would it be worthy of cherry picking it to a more stable branch? Not sure how that process works exactly.

@andrewescutia This change is already available in google_maps_flutter-v0.5.26+2 or greater, if you update your app's pubspec version range.

noice! I see that now within the pub changelog. For some reason I was reading this as an update to Flutter itself. Do the PRs themselves notate what package version they are a part of?

noice! I see that now within the pub changelog. For some reason I was reading this as an update to Flutter itself. Do the PRs themselves notate what package version they are a part of?

Yes, each PR should update the CHANGELOG and version: https://github.com/flutter/plugins/pull/2681/files#diff-40608137945b7a665e28d032c6e04916R1


Adding toko icf

Just mention I'm seeing the in memory passing of geometry from cadquery to gmsh is not working. I'm waiting for the cadquery release on conda forge to happen the I shall try again. I suspect cq and gmsh have different ocp versions somehow

If that doesn't work them I shall shift to Exporting step file and importing it again

done in pr #36


cheetah-flake pre-commit hook fails with AttributeError: 'FlakesChecker' object has no attribute 'JOINEDSTR' when run on .tmpl file containing py3.6 f-string

From running pre-commit on a .tmpl file with py3.6 f-string (f'abcd {somevariable} xyz') with .pre-commit-config.yaml cheetah_lint sha v0.3.4.
Traceback:
Traceback (most recent call last):
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/bin/cheetah-flake", line 11, in <module>
    load_entry_point('cheetah-lint==0.3.4', 'console_scripts', 'cheetah-flake')()
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/cheetah_lint/flake.py", line 408, in main
    retv |= flake(filename)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/cheetah_lint/flake.py", line 394, in flake
    flakes = get_flakes(file_contents)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/cheetah_lint/flake.py", line 387, in get_flakes
    data += get_from_py(file_contents)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/cheetah_lint/flake.py", line 306, in get_from_py
    check(py_lines), py_lines, cheetah_lines,
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/cheetah_lint/flake.py", line 261, in check_flake8
    checker.input_file('<compiled cheetah>', py_lines)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/flake8/engine.py", line 195, in input_file
    line_offset=line_offset,
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/flake8/engine.py", line 172, in _retry_serial
    return func(*args, **kwargs)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/flake8/engine.py", line 126, in input_file
    return fchecker.check_all(expected=expected, line_offset=line_offset)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/pycodestyle.py", line 1608, in check_all
    self.check_ast()
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/pycodestyle.py", line 1554, in check_ast
    checker = cls(tree, self.filename)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/flake8/_pyflakes.py", line 63, in __init__
    withDoctest=withDoctest)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/pyflakes/checker.py", line 426, in __init__
    self.runDeferred(self._deferredFunctions)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/pyflakes/checker.py", line 463, in runDeferred
    handler()
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/pyflakes/checker.py", line 1055, in runFunction
    self.handleNode(stmt, node)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/pyflakes/checker.py", line 794, in handleNode
    handler(node)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/pyflakes/checker.py", line 747, in handleChildren
    self.handleNode(node, tree)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/pyflakes/checker.py", line 793, in handleNode
    handler = self.getNodeHandler(node.__class__)
  File "/nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/pyflakes/checker.py", line 627, in getNodeHandler
    self._nodeHandlers[node_class] = handler = getattr(self, nodeType)
AttributeError: 'FlakesChecker' object has no attribute 'JOINEDSTR'

I was hesitant to post this here since it's not actually code in cheetah-flake that's the problem, exactly.
The error is being thrown by pyflakes. Googling the error, turns out this was an error in older versions of pyflakes and is fixed in newer versions. I checked which version of pyflakes is the one running:
(virtualenv_run) $ cat /nail/home/daniellg/.cache/pre-commit/repo55sxfmr_/py_env-python3.6/lib/python3.6/site-packages/pyflakes/__init__.py
__version__ = '1.2.3'

Which is old enough it still has lack of py36 support.
cheetah-lint is calling flake8, which itself is making this call to the old version of pyflakes. I quickly looked at repos for pyflakes/flake8 and didn't see old versions pinned, nor are they pinned in this cheetah_lint repo. So I'm not sure yet why it's using v1.2.3 of pyflakes.
As a sanity check, I did nuke /nail/home/daniellg/.cache/pre-commit/ completely and reinstall hooks but no change in behavior.

Unfortunately there's not really much that can be done here due to https://github.com/asottile/cheetah_lint/issues/14 :'(

@asottile thanks for the quick response, despite the sad news.

@dan98765 if you want to try and fix it, the suggestion in #14 will probably work (and won't be too difficult) -- the code hint is here: https://github.com/asottile/cheetah_lint/blob/d4e9471b3647d1ccfb8e5070dc7358b7f95f4a36/cheetah_lint/flake.py#L257-L264


Sometimes slow upload of jars

DEBUG 2014-05-13 17:13:43,556 [pool-1-thread-2] com.hazelcast.stabilizer.provisioner.Bash: Executing bash command: scp -r -o UserKnownHostsFile=/dev/null -o CheckHostIP=no -o StrictHostKeyChecking=no /Users/danny/IdeaProjects/hazelcast-stabilizer2/dist/target/hazelcast-stabilizer-0.3-SNAPSHOT/lib danny@<IP_ADDRESS>:hazelcast-stabilizer-0.3-SNAPSHOT/lib
DEBUG 2014-05-13 17:24:02,353 [pool-1-thread-3] com.hazelcast.stabilizer.provisioner.Bash: Bash output:
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.

I guess we can close this one as well (it already has the wontfix label).
The only thing I have in mind would be a checksum comparison for each JAR so we don't upload files again, which are already there. But this should be a new feature, not a bug.

indeed


[SPARK-17270][SQL] Move object optimization rules into its own file

What changes were proposed in this pull request?
As part of breaking Optimizer.scala apart, this patch moves various Dataset object optimization rules into a single file. I'm submitting separate pull requests so we can more easily merge this in branch-2.0 to simplify optimizer backports.
How was this patch tested?
This should be covered by existing tests.

Test build #64510 has started for PR 14839 at commit 39d8a9e.

Test build #64510 has finished for PR 14839 at commit 39d8a9e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64510/
Test PASSed.

Merged build finished. Test PASSed.

LGTM. Merging to master and branch 2.0.

hmm. There are conflicts in branch 2.0.

Seems the conflicts are caused by https://github.com/apache/spark/commit/d063898bebaaf4ec2aad24c3ac70aabdbf97a190#diff-a636a87d8843eeccca90140be91d4faf.

OK I will submit a separate one.


Would like to wrap my library using this code

Summary
Hi, congratulations for your work. This is an amazing repository. I was checking the source code and I was wondering if it fits to a library I'm developing. It is used to run asynchronous requests against WebDrivers/Winium. I didn't find any unit test to help me to check it. This is the library. Let me know if you think it is possible to adapt/extend your code to use it, please. If it is possible, I'd be happy to play around and send some PRs related to it. Thank you!
Motivation
Why are we doing this? What use cases does it support? What is the expected outcome?
I have a fresh library to interact with WebDrivers and Wnium, but the code I quite verbose. I think your code may make my library simpler to be used.
Describe alternatives you've considered
Use other repository to wrap my code or develop it by myself.

What problem does the library solves?

Hi, here are the answers

What you like about Bose?

I liked that way it simplifies the boring task to write Selenium tests/crawlers. For example, to click on elements we always need to get these elements, so, for me the click method in Selenium would be smart enough to get and click. I would make the developer interface less verbose and we wouldn't need to wrap the things in Page Objects or other structures. Your library does it. It condenses the related methods in less lines of code, which is perfect, because it reduces the responsibility of the tester. It reduces the maintenance and errors. As people use Bose more and more, they will find issues, that are common for many testers, so the fix is shared across the community.

What problem does your library solves?

It is intended to command executions against Drivers synchronously and asynchronously. Launch the Driver as a server and send requests to it. The intention is that the user does not worry about which Driver he/she is using. It can be WebDrivers like Selenium, MobileDrivers like Appium, or DesktopDrivers like Winium (this was copied from my README :)
So, the basic idea is have one single library to interact with many UI controllers. Selenium is just intended to be used with Web pages, Winium with Windoes Descktop and Appium in mobile tests. Each time we change the stack we need to change the library/methods we use. I've tested my library with ChromeDriver, Firefox and Winuim, but I have plans to extend it.
Other key feature is the possibility of run asynchronous and remote calls to these webdrivers. Asynchronous make the tests faster and it can be executed with many driver instances, so no need to use Selenium Grid. The Remotpart is related to infrastructure. The team can have just one server with a controlled WebDriver running, hence the tester/developer does not need to build the local environment by herself.

How will it help Bose user's?
I think it helps Bose user to use an async/remote/agnostic library using a friendly interface. As I say, may library is quite verbose, so the user will find herself doing things like "open session", "get element", "click", "close session". I have to admit it is boring

Let me know you have more questions. :)

@douglasdcm Honestly, I do think it is a common use case. So, I won't be adding this functionality. Please feel free to fork it, and modify it as per your library.


Delete unnecessary character in v0.2.7

when I install version 0.2.7 there is a "+" character at the beginning of line 171:

+             <ha-icon icon="[[getWindDirIcon(windBearing)]]"></ha-icon> [[getWindDir(windBearing)]]

              <ha-icon icon="hass:weather-windy"></ha-icon> [[computeWind(weatherObj.attributes.wind_speed)]] [[ll('uSpeed')]]

Thanks a lot for your work on this great card. Love it.

Shoud be solved with #46


Selecting Users when creating game

in our happening, some people don't want to play betrayal, causing the game to lock up when others want to play, leaving no winner. I would like to suggest a feature to select which users in the happening will be going to play the game, kind of like chess, or checkers

Yep, we should have that!
It may be some time before I'll get around to it. (Or I'd be more than happy to merge a pull request! :))

I'd have to look into writing Coffeescript ;), but If I have time, I'll definatly try

Hello @vanviegen,
First of all, thanks for this nice game. We really like to play it in our happening. However, there are some people that don't understand how to play this game or just don't want to play it.. So it would be really nice if you can make this piece of code that allows us to choose people who play :)
Kind Regards,
Jérémy


Days of buffering rework

Fixed bug with transactions types check.
Fixed bug with AoM hide option. (rework hide aom logic)
Add tooltip with statistics.

Tested the last commit, works for me now on both of my budgets (even with scheduled split transactions), thank you.

Was using visibility: hidden instead of display: none so the alignment in the header didn't change when you hid AoM, but honestly nobody will notice and when you enable the other option life is good. :+1:

Thank you!
The only feedback I've got is an English thing. Instead of, "Here you are this new shiny metric!" you'd say something like "Here's this shiny new metric!" or "Here you go, have this new shiny metric!"
In terms of what we're doing here I'd say, "Here, use this instead!" :smile: I'll go ahead and reword it a bit there.

@blargity thanks for correction!


Fail safe flag without valid country code class

A potential fix for #3260. Render blank instead of showing the flag of Andorra when the country code class name is missing or invalid.

Thanks this seems clear.

I have created a new PR on the next branch:
https://github.com/Semantic-Org/Semantic-UI/pull/3333


Failing to run with io.js or Node.js 0.12

After trying npm i -g node-inspector and node-debug index.js where index.js only contains console.log('test') I get a blank source and the script appear to fail to run:

I'm on OSX 10.10.3 using Chrome 43.0.2357.81
I tried with io.js 2.1.0 and 1.8.2 and Node.js 0.12.4 (and re-installing node-inspector each switch).
I tried node-inspector 0.10.1 and 0.8.3
I also tried with --target doing: npm i -g node-inspector --target=1.8.2 (which installed with --target=1.8.2 but still had the same issue; it did not even install using --target=2.1.0).

The only way I was able to get it working was by installing Node.js 0.10.38 and node-inspector 0.10.1.
Thanks for your help!

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/18227523-failing-to-run-with-io-js-or-node-js-0-12?utm_campaign=plugin&utm_content=tracker%2F195817&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F195817&utm_medium=issues&utm_source=github).

Hello @shama , you can do the following:

Run node-inspector in first shell and node --debug-brk test in second.
Is there any errors in first shell?
If there is no error in first step, can you please follow this instruction
How long app starts up on 0.10.38?

@3y3 Thanks for your help!

Ran both of those and no errors in the first shell (running node-inspector)
Here is the output I get running DEBUG=node-inspector:protocol:* node-inspector: https://gist.github.com/shama/ebc5063878499422b273
It's not immediate but the source appears ~ 2 seconds.

Hm... This looks like a problem with connection to debugged app.
I receive the same trace log, if I run node-inspector without debugged app.
I don't know, what else I need at current time for debugging...
Are you ready to modify node-inspector sources?

@3y3 Sure! I'll dive into the source and see if I can find out where it's failing.

I don't know for sure if I'm having the same problem, but it seems similar. node-inspector used to work great but now it's doing this thing most of the time where the interface loads but does nothing, remains completely blank. My environment:
node 0.10.32 / node-inspector 10.0.0 / 10.0.1 running on Ubuntu 12.04.
Debugging remotely from Windows 7 x64 / Chrome 42 on different physical machine on same local network.
node-debug --debug-brk --web-host=example.com script.js

Since I updated Chrome to v41 it started exhibiting frequent DNS failures and I recently ran Fiddler (debugging proxy) to see if it could reveal anything, and just running it seems to sidestep the Chrome DNS failures, so I've been running it since. I mention it in case it's relevant somehow, but as I said the interface loads, just nothing within it. And I believe I've had it function normally at least once with Fiddler running.

I don't know if it's relevant, but I'm also getting this in Chrome console:
Assertion failed: Unknown experiment canvasInspection
  Runtime._assert @ Runtime.js:358
  Runtime.ExperimentsSupport._checkExperiment @ Runtime.js:953
  Runtime.ExperimentsSupport.isEnabled @ Runtime.js:894
  WebInspector.ProfileTypeRegistry @ ProfileTypeRegistry.js:19
  (anonymous function) @ ProfileTypeRegistry.js:43
  evaluateScript @ Runtime.js:153
  scriptSourceLoaded @ Runtime.js:136

@jmm , do you have fiddler enabled permanently?
I know one shadow issue about fiddler + web sockets.

@3y3 Thanks for the reply. Lately I've been running fiddler continuously. I'm not sure if that's what you mean by permanently. Chrome is so messed up that if I disable fiddler and try to open the node-inspector URL, I immediately get this net::ERR_NAME_NOT_RESOLVED error that I get rampantly (since moving from Chrome v36 [I think] to v41) when fiddler isn't running.

I know one shadow issue about fiddler + web sockets.

Do you know of something I can / should do to configure fiddler to work with node-inspector? Would following those instructions you linked to shed more light on my problem? I believe I've had it work at least once with fiddler running, but I can't say for certain.

I immediately get this net::ERR_NAME_NOT_RESOLVED error that I get rampantly (since moving from Chrome v36 [I think] to v41)

Can you give me a screenshot with this error.

Do you know of something I can / should do to configure fiddler to work with node-inspector?

Unfortunately I don't know any solution here.
I also wonder to see debug log . It will be helpful.

@3y3 Thanks very much for your help. It sounds like the same symptoms, but in case the cause of my problem is different I don't want to squat on @shama's issue, so I opened #671 to pursue mine further. I started with a recap of the basic info I posted here and I'll be posting additional information.

Closing as this seems to be fixed with node-inspector 0.12.3 and node.js 4.0.0. Thanks!


RFC 8707: Resource Indicators for OAuth 2.0

The RFC8707 details how a client can indicate to the authorization server on which resource server(s) the token will be used. This allows the authorization server to adapt the token claims depending on which resource server(s) it is intended, like for instance the aud and scope claims.
I suggest implementing this spec in authlib.
@lepture what do you think? Would you have implementation suggestions to tackle this?

This should be implemented together with #427


Update complete-basic-operations-using-sharepoint-rest-endpoints.md

Category

[x ] Content fix
[ ] New article
Related issues: fixes #2716

What's in this Pull Request?
Fix content error.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/sp-add-ins/complete-basic-operations-using-sharepoint-rest-endpoints.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

thx @Seiyasu, appreciated.


plot tree with high cardinality feature

When I try to plot a tree that splits on a feature with high cardinality the overall diagram gets missed up no matter how I tried to adjust the width and height

There must be a way to ignore writing all the values that are used on the split within the nodes

Hi @moziada, thanks for using LightGBM. I think a possible solution would be to check the number of categories in the split and if there are too many we should collapse them and show them in the tooltip instead, which would look like this:

The problem is that at least for me (firefox + jupyterlab) the graph isn't automatically shown as SVG, so I had to save it first and then load it and show as HTML which is cumbersome. We should find a way to display it as SVG by default, I can look into that later, or if you're interested you can investigate and contribute the change yourself. The logic for showing it in the tooltip is very simple, here's a sample diff:
diff --git a/python-package/lightgbm/plotting.py b/python-package/lightgbm/plotting.py
index d2a57209..87c287c0 100644
--- a/python-package/lightgbm/plotting.py
+++ b/python-package/lightgbm/plotting.py
@@ -493,7 +493,13 @@ def _to_graphviz(
                     direction = _determine_direction_for_numeric_split(
                         example_case[split_feature], root['threshold'], root['missing_type'], root['default_left']
                     )
-            label += f"{_float2str(root['threshold'], precision)}"
+            if is_categorical_split and len(root['threshold'].split('||')) > 5:
+                threshold = '...'
+                tooltip = root['threshold']
+            else:
+                threshold = root['threshold']
+                tooltip = None
+            label += f"{_float2str(threshold, precision)}"
             for info in ['split_gain', 'internal_value', 'internal_weight', "internal_count", "data_percentage"]:
                 if info in show_info:
                     output = info.split('_')[-1]
@@ -525,7 +531,8 @@ def _to_graphviz(
             if "data_percentage" in show_info:
                 label += f"
{_float2str(root['leaf_count'] / total_count * 100, 2)}% of data"
             label = f"<{label}>"
-        graph.node(name, label=label, shape=shape, style=style, fillcolor=fillcolor, color=color, penwidth=penwidth)
+            tooltip = None
+        graph.node(name, label=label, shape=shape, style=style, fillcolor=fillcolor, color=color, penwidth=penwidth, tooltip=tooltip)
         if parent is not None:
             graph.edge(parent, name, decision, color=color, penwidth=penwidth)

Hey @jameslamb sorry I did not receive your response earlier, I will look into it and try to make a contribution.
I will keep you with any updates

After searching I have found that this is a common issue with JupyterLab and people have opened issues on it (like this issuse).
The workaround to display svg images on JupyterLab is to wrap the svg image content with IPython.display.HTML then the tooltip works properly

Also I have tested the tooltip on Jupyter Notebook it works fine no workarounds are needed

Both screenshots are taken on Firefox browser

@jmoralez sorry I forgot to mention you with my results

Sure I am working on a PR

I am having difficulties on testing the changes I have made, is there any systematic way to make changes and install the new version to the current python environment?

Since you're only touching Python code, compile the library just one time by running the following from the root of the repo.
rm -r ./build
mkdir ./build
cd ./build
cmake ..
make -j2

Then every time you change the Python package, run this from the root of the repo.
cd ./python-package
python setup.py install --precompile

Then every time you want to run the tests, point pytest at any of the ddirectories and files in the tests directory. For example, by runming the following from the root of the repo.
pytest tests/python_package_test/test_plotting.py

I have made the changes but I noticed that all .rst doc pages under Plotting section are not available on github

That page is automatically generared from the docstrings in python-package/lightgbm/plotting.py.

You can also render the docs locally following this.


Calico ippool job is failing with the overlap CIDR error message for the pod CIDR

Calico ippool job is failing with the overlap CIDR error message for the pod CIDR
Expected Behavior
ippool job should run successfully
Current Behavior
ippool job fails with the below error message.
Failed to apply 'IPPool' resource: error with field IPPool.Spec.CIDR = '<IP_ADDRESS>/16' (IPPool(ippool-cs-1) CIDR overlaps with IPPool(default-ipv4-ippool) CIDR <IP_ADDRESS>/16)

I tried with different CIDR as well (<IP_ADDRESS>/16) same error. If i configure some other CIDR other than cluster POD CIDR job runs success.
Possible Solution
job should run if it matches the POD CIDR and there is no overlapping with <IP_ADDRESS>/16 or fe80::/10.
Steps to Reproduce (for bugs)
Kube-Proxy has argument as --cluster-cidr=<IP_ADDRESS>/16 and there is no  - --masquerade-all flag
Run the cluster in AWS Environment with cloud-provider flag enabled. Node names are registered with private DNS name of an EC2 instance.

I did verify that NodeNames are matching the workloadendpoint names and
BGP peer status looks fine. I could see that
We have network manager changes done on this file /etc/NetworkManager/conf.d/calico.conf as
below. [keyfile]          unmanaged-devices=interface-name:cali*;interface-name:tunl*
Each node got set of cidr blocks from <IP_ADDRESS>./16 range

Context
Communication between pod to pod is not happening , some pods are able to communicate with other pods. Some pods are not able to communicate with other pods.
Your Environment

Operating System and version: Ubuntu
calico/node:v3.0.1
calico/ctl:v2.0.0-2-g58eac998
calico/cni:v2.0.0
AWS Environment
Kubernetes 1.8.3 with cloud provider flag enabled.

NAME   CIDR   NAT   IPIPMODE   DISABLED

@caseydavenport Thanks for your help. Closing this issue as it was misunderstanding and configuration from my side.


Improve path parameters detection

This PR improves the detection of path parameters by PathParamNames.

Endpoint
Before
After

/endpoint/:param1/:param2/rest
"param1" | "param2"
"param1" | "param2"

/endpoint/:param1:param2/rest
"param1:param2"
"param1" | "param2"

/endpoint/:param1%3Aaction
"param1%3Aaction"
"param1"

endpoint/:param1%3Aaction:param2
"param1%3Aaction:param2"
"param1" | "param2"

This PR is required by https://github.com/astahmer/openapi-zod-client/pull/180

Thank you very much. I'll review it asap

Can you also add runtime checks in unit tests in zodios.test.ts ?
does my current regexp to replace url parameters works out of the box with those patterns ?

Sure, I’ll add it.
Which regex do you mean?

Sure, I’ll add it.
Which regex do you mean?

This one https://github.com/ecyrbe/zodios/blob/main/src/utils.ts#L54

Sure, I’ll add it.
Which regex do you mean?

This one https://github.com/ecyrbe/zodios/blob/main/src/utils.ts#L54

@Blejwi do you still have time to work on this ?

closing, i made another solution. available in zodios v10.9.3


guidance

I am using NANO IOT, SAMD board
all I want is a timer2 isr routine to count so I can replace millis() which uses timer 0 and I have to use low power sleep which kills timer 0.
but I am missing some terminology, and haven't found a place to get it.
from the sample
// You can select SAMD Hardware Timer  from SAMD_TIMER_1 or SAMD_TIMER_3

// Depending on the board, you can select SAMD21 Hardware Timer from TC3-TCC
// SAMD21 Hardware Timer from TC3 or TCC
// SAMD51 Hardware Timer only TC3

// Init SAMD timer TIMER_TC3
SAMDTimer ITimer(TIMER_TC3);

what is the difference between SAMD21 or SAMD51
and what is TCC and TC3
and why would I pick one of the other
and how does all this relate to timer2?
I have written ISR in just about every language on just about every hardware type,
so all of the process is not the problem
your library looks like it encapsulates all the prescaler and other technical goop I don't want to know about, which is wonderful.  just need to get over the hump somehow.
(and my name is Sam D... which is pretty coincidental)

and then theres this
#define HW_TIMER_INTERVAL_MS      1L     // so I think MS means Mili-seconds as Micro is usually codes uS
// Interval in microsecs  ...   then the comment says micro
if (ITimer.attachInterruptInterval(HW_TIMER_INTERVAL_MS * 1000, TimerHandler))  // but this seems to imply 1 second
or it means you want Millisecond interval, but the timer is microseconds
in which case I would change the constant to
#define HW_TIMER_INTERVAL_MS      1000L
and take the multiply out
just trying to understand the words

using either TCC or TC3 both work

Thanks for your interest in the library, and I'm glad to have chance to work with a SAMD-related person here ;-)
You're using NANO_33_IOT, which is using SAMD21 MCU and you can use both TC3 or TCC timer. You have to pick the timer not being used by the core or other libraries, you have to read or try to know.

#define HW_TIMER_INTERVAL_MS 1L // so I think MS means Mili-seconds as Micro is usually codes uS
// Interval in microsecs ... then the comment says micro
if (ITimer.attachInterruptInterval(HW_TIMER_INTERVAL_MS * 1000, TimerHandler)) // but this seems to imply 1 second

The timer count unit in
ITimer.attachInterruptInterval(HW_TIMER_INTERVAL_MS * 1000, TimerHandler);

is in microseconds (which in HW or SW, we use uS, while ms is for miilisecond).
So if you're using HW_TIMER_INTERVAL_MS in ms, you have to multiply to 1000 to convert to uS.
Or if you're defining in uS, you can use it directly
ITimer.attachInterruptInterval(HW_TIMER_INTERVAL_US, TimerHandler);

Thanks and happy exploring,
Best Regards,


Check whether a chaincode has been installed and instantiated

Signed-off-by: feihujiang<EMAIL_ADDRESS>Check whether a chaincode has been installed and instantiated, if it has, not throw an error, to enable repeated tests.

Checklist

[ ]  A link to the issue/user story that the pull request relates to
[ ]  How to recreate the problem without the fix
[ ]  Design of the fix
[ ]  How to prove that the fix works
[ ]  Automated tests that prove the fix keeps on working
[ ]  Documentation - any JSDoc, website, or Stackoverflow answers?

Issue/User story

Steps to Reproduce

Existing issues

[ ] Stack Overflow issues
[ ] GitHub Issues
[ ] Rocket Chat history

Design of the fix

Validation of the fix

Automated Tests

What documentation has been provided for this pull request

LGTM :)


Allow setting the job service account

What this PR does / why we need it:
The ability to set the job service account. Fixes #749
If applicable:

[X] this PR has been tested for backwards compatibility

@radu-matei What do you think about adding a flag to only allow service accounts that start with a certain prefix or a regex.
Not really sure how we would validate that during project creation though since it doesn't go through the brigade api

Well, we're in almost the same situation now, aren't we?
The user creating a Brigade project wants to use a specific service account for jobs. That information is unused until a build is actually created for that project - in which case, if the SA exists everything is OK - if not, it fails because the missing SA.
The same would be true if we validate with a regex - you try to use a SA, but until a build is scheduled, you don't know if a) the SA even exists and b) if the you are allowed to use it.
I like this, and the more I think about it, the more it makes sense - however, we should have the a sensible default here.
cc @technosophos

Yea that is true it will error during the build process either way.
I didn't want to break backwards compatibility so I left the default service account as brigade-worker but on my own deployments I will just be setting it to default.
If we don't really care I can change the default to just default as that service account typically has no permissions to do anything.

Also for the regex checking, should that be done in the controller or in the worker? If it's done in the worker we could print out some logs with error messages about it. I am not too sure how to implement that in the javascript though so I will need some help with it.

Before changing the default service account we should also keep in mind lots of people use Brigade for CD inside the cluster, for example with a Helm step. (While we're still pre-v1.0, and technically we could break backwards compatibility, this would be pretty significant - on the other hand, if we want to do it, we should do it pre-v1.0).
There are implications to where to check as well - if the check is done in the controller, the worker wouldn't get started, so there'd be no logs for the build. Meaning that someone who doesn't have access to the controller logs would have no idea what happened.

Ping @rmb938.

@radu-matei I've added tests for the controller and rebased. I am not too sure how to go about adding tests for the regex validation since it's an environment variable (can't seem to change the environment during a test) and the check exists the process.

@rmb938 - awesome, thanks!
I'll have a look early tomorrow at this.

Ping @radu-matei for a final approval.


added goreleaser config

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
203

Relevant Lines:
388

💛  - Coveralls


Check whether the task finishes before deferring the task for DataprocSubmitJobOperatorAsync

Like the SnowflakeOperatorAsync, we need to verify if the task has already completed before deferring it to prevent unnecessary deferring. This way, we can skip deferring the task if it has already been finished.

@Lee-W Did we consider the suggestion I gave earlier. If not, can you please a comment in response on what your thoughts are?

@pankajkoti Sorry, I think I missed this one. Let me check how I can improve it

PR has some conflicts with main.


vscode RPM for RHEL9

Please could you make available vscode RPM for RHEL9
[https://code.visualstudio.com/docs/setup/linux#_rhel-fedora-and-centos-based-distributions]
Many thanks in advance

Closing in favour of #162853


Onnx support?

Hi, really like this work!
Given its advantage on faster inference, have you considered adding support functions, like the example below, to compile SetFitTrainer into the onnx format for production-wise usage?
If that sounds promising, I will be happy to make this feature work!
Example:
# Train
trainer.train()

# Compile to onnx
onnx_path = "path/to/store/compiled/model.onnx"
trainer.to_onnx(onnx_path, **onnx_related_kwargs)

Hi @blakechi thanks for your interest in our work! Instead of compiling SetFitTrainer into ONNX, I think it would be better to have something like an onnx_export() function that lives inside a setfit.onnx module.
Ideally, this function would take a SetFitModel as input and output model.onnx, similar to what was done for Stable Diffusion here: https://github.com/huggingface/diffusers/blob/main/scripts/convert_stable_diffusion_checkpoint_to_onnx.py#L32
Since the current implementation of SetFitModel uses scikit-learn estimators for the classification head, it might be best to wait until we implement a pure PyTorch version in #8

Hi @lewtun,
Thanks for your reply! onnx_export for SetFitModel sounds good to me.
Sure, happy to lend a hand on onnx after #8. :grinning:

I'm not sure if this is helpful, but I was working on deploying some of these models using ONNX and this is what I came up with so far.  If others are looking for a place to start here is some code that will convert the base model and the head and then you can run them separately. I haven't been able to merge them into one graph yet but hopefully it's a start while we wait for #8 :).
`
from skl2onnx import convert_sklearn
from skl2onnx.common.data_types import FloatTensorType
from onnxruntime import InferenceSession
from pathlib import Path
from setfit import SetFitModel
from transformers import AutoTokenizer
from transformers.convert_graph_to_onnx import convert
import torch
import numpy as np
import sys
def mean_pooling_np(model_output: np.array, attention_mask: np.array):
token_embeddings = model_output[0]
input_mask_expanded = np.broadcast_to(
np.expand_dims(attention_mask, axis=2), token_embeddings.shape
)
sum_embeddings = np.sum(input_mask_expanded * token_embeddings, axis=1)
sum_mask = np.clip(input_mask_expanded.sum(1), 1e-9, sys.maxint)
return sum_embeddings / sum_mask
trained_model_path = "path/to/your/trained/setfit/model"
onnx_sentence_model_path = "/path/to/save/onnx/to"
onnx_head_model_path = "/path/to/save/onnx/to"
model = SetFitModel.from_pretrained(trained_model_path)
Convert the sentence transformer model to onnx
convert(
"pt",
trained_model_path,
Path(onnx_sentence_model_path).absolute(),
15,
trained_model_path,
)
Convert sklearn head into ONNX format
initial_type = [("model_head", FloatTensorType([None, 768]))]
onx = convert_sklearn(model.model_head, initial_types=initial_type, target_opset=15)
with open(onnx_head_model_path, "wb") as f:
f.write(onx.SerializeToString())
Load and use the models
text = ["some text to do stuff with"]
tokenizer = AutoTokenizer.from_pretrained(
"sentence-transformers/paraphrase-mpnet-base-v2"
)
session = InferenceSession(onnx_sentence_model_path)
head_session = InferenceSession(onnx_head_model_path)
tokens = tokenizer(text, truncation=True, return_tensors="np")
preds = session.run(None, dict(tokens))
pooled_preds = mean_pooling_np(preds, tokens["attention_mask"])
print(head_session.run(None, {"model_head": pooled_preds}))
`

Thank you, @nbertagnolli !
A few issues that I had with the script and how I resolved them:

For the paths to store the onnx models, the convert script expects that the directory that will contain them will be empty. Just create an empty onnx directory and put them in there.
Be careful with the initial_type -- it should have the right shape for your LM.
You should also use the appropriate tokenizer for your model.
My model-head was expecting np.float32, not np.float64.
For python 3+, I would change sys.maxint with sys.maxsize.

Sorry for the waiting time and good new is Issue #8 has been resolved!
Just a suggestion. Maybe @nbertagnolli can work on converting the sklearn head and I can work on the PyTorch version? So we can work on the part we are familiar with.  :)

Happy to help @blakechi do you have a branch you're currently working on? Want me to create one and put an initial ONNX script together? Nice work on #8!

Thanks, @nbertagnolli!
Or maybe we can open 2 PR separately? Seems like you are almost ready to open one, so I don't want to hold your back. 😅 But I'm fine on either way :)
Maybe you can give us some suggestion? Which way is more convenient for you to manage? @lewtun

Sounds good @blakechi I'll open a PR soon with an initial script for onnx conversion and we can go from there! : )

okay, sounds good to me :)
Make sure you follow @lewtun 's suggestion as below:

Hi @blakechi thanks for your interest in our work! Instead of compiling SetFitTrainer into ONNX, I think it would be better to have something like an onnx_export() function that lives inside a setfit.onnx module.
Ideally, this function would take a SetFitModel as input and output model.onnx, similar to what was done for Stable Diffusion here: https://github.com/huggingface/diffusers/blob/main/scripts/convert_stable_diffusion_checkpoint_to_onnx.py#L32
Since the current implementation of SetFitModel uses scikit-learn estimators for the classification head, it might be best to wait until we implement a pure PyTorch version in #8

Hey @nbertagnolli and @blakechi super cool that you're excited to work on the ONNX export for the two heads 🔥 !!
I agree it's best to have an initial PR first so we can hone the design and then iterate from there.

Do we also need a Python API for ONNX models?
Found this pretty helpful also from  Diffusion

I like that idea, I think that could make working with the sklearn heads easier. : )

tokenizer = AutoTokenizer.from_pretrained(
    "sentence-transformers/paraphrase-mpnet-base-v2"
)

@nbertagnolli I tried this approach but I'm getting an issue regarding the inputs. I trained a multi-label model using text inputs

@AnshulP10 please take a look at the PR we've been working on #156. @kgourgou  pointed out the above script has some things that you need to modify for some models. This PR hopefully addresses those concerns.  In the PR there is a function called export_onnx which should do what you want. Let me know if you still have trouble.

With #156 merged, this feature request has been implemented :)


Server log does not contain information about queries and responses since 0.13

Till Selendroid 0.12 there were some useful messages in a log, describing queries to server on device and responses to tests. It was like: INFO: Payload? {"using":"id","value":"login_button"}
Since Selendroid 0.13 they are gone. I cannot see them even using -verbose and -loglevel VERBOSE. Could you please return them back, it was very useful. Or, if they just moved to another place, could you, please tell me where to find those logs?

This is a question rather than an issue. Please send questions to the selendroid user group.
For issues please provide a concise reproducible test case and describe what results you are seeing and what results you expect.


Possibility to access prometheus via data source

Hi,
is it possible to configure and access your prometheus data on existing grafana installations?
I have already an instance running and it will be great if I can only add another dashboard.
Thanks JO!

Hi!
I changed the configuration so that the prometheus endpoint is publicly available under https://covid19.spiessknafl.at/prometheus. You should be able to add this URL as a datasource for your Grafana dashboards.

Also, if you have a cool dashboard you would like to share, I can easily add it to my instance.

Hi!
Thanks! The data source is working!
Sure, I will start to build a dashboard now which I can also share with you ;)
I'll keep you posted.
Thanks JO!

Hi,
Is it possible to add localization data (LAT/LONG) to the Prometheus DB?
This data is needed to implement a map view into the dashboard.
I have seen that the Data Repository by Johns Hopkins CSSE can deliver it.
I think this data would also be a great source because the update frequency is higher.
https://github.com/CSSEGISandData/COVID-19
BR/JO!

I just had a look at the data. It lacks details for Austria (Bundesland) and is also older than the information from BMSGPK.
The lat/long thing is a good idea.

Jep, You are right. Maybe you can merge the data to get the coordinates for the map?

...and another Feature Request: a timestamp to track the update times :) Thanks JO!

lat/long is included now.


Custom serdes are not working

Hi,
I have created a custom serde to work against a graph stored in HBase (using HGraphDB) so that I could integrate SHC with Spark GraphFrames.  I wrote a blog about it here.  I ran into a couple of issues for which I have filed pull requests (#113, #114, #116).
Other than those minor issues, the integration of SHC with GraphFrames worked very well.
Thanks!

Hi, @rayokota. Thank you very much for help improving SHC. Great PRs. I've merged them.
Could you please also update your blog so people know about this and the issues are gone? Thanks again.

Thanks @weiqingy !  I've updated the blog to say the issues have been merged.  Thanks for your quick response!


migrate to https://github.com/cosmos/gogoproto

From sig-support: this does not block the upcoming 0.24 network upgrade


Update Mapbox style background color

It is currently hsl(55, 11%, 96%). This causes an ugly gray flash as the map loads.
Let's update it to #cad2d3 to match the color of the ocean, so the map loads smoothly.
We need to make this change in Mapbox studio (which I don't have access to).

Tried but it is actually trickier than that as most of the layers are transparent... 😞

Looking at the style definition Mapbox sends, there should be base layer called "background" that has a single property "background-color" defined.

Yep, but the layers on top of the background are transparent

Happy to give you access if you want to have a look

Happy to take a look if you give me access :)

It may take a while to propagate but I think I figured it out using a landcover layer. @bcherny FYI

@ericboucher Close but not quite. Think you can revert your change and I'll give it a shot later?


Interval

@noCharger can you please add the original PPL command here ? additional context is needed
Maybe its is related to the adddate command ?

Let's close until there's a proper description

[Catch All Triage - 1, 2, 3]


Minimizing counterexample traces

I think one thing that could be very useful to have in the simulator would be an option to have it attempt to minimize invariant violations.
When I get counterexample traces, I will very often get long counterexamples.
These counterexamples would often still be counterexamplen when simply omitting some part of the trace.
I would like to have an optional flag like --minimize-counterexample. When it is given, once an invariant is found, Quint tries to minimize the trace while still having it violate the invariant.
A downside is that this might be computationally nontrivial for long traces. I am fine with this; some sort of best-effort approach at minimizing away the most trivial cases would be sufficient for me. The cases I have are usually very benign, where for many transactions, it is possible to remove them without affecting either enabledness of the trace, nor the invariant violation at the end.

Note: I think it could be even cooler to allow this to facilitate a loop with an external oracle (an MBT driver, specifically).
Specifically, maybe this could help facilitate a loop where there is some discrepancy in some trace between the model and the system. Then we would like to check if a smaller trace also is a counterexample, to find a small one that surfaces the difference. So the loop would be: take a trace that surfaces the system fault. make it smaller. check if it is still a viable trace AND if it still surfaces the system fault. repeat


Request for missing videos

hi， thanks for your datasets.
I have submitted the form to request the downloading of the missing video, but there is currently no response.
Is it still available for downloading?

您好，您发给我的信件已经收到，我会尽快查看。


Dataset for Similarity and Category and Results for method and step level inference?

Hi thank you for your work! I want to ask about two things:

Would it be possible to release the similarity and category test set? I only see the random strategy in the GDrive.
Could you report the exact accuracy for method-level and step-level that you briefly mentioned in figure 3? Having the results for the best performing ones (the triangle ones) and the human performance (the circles) would be great!

Thank you in advance.

Hi,

I just uploaded the test set in google drive.

model: method: 0.6972, 0.7431, 0.5316; step: 0.7848, 0.7465, 0.6607
human: method: 0.905, 0.727, 0.74; step: 0.92, 0.8920, 0.86
the order is random, similarity, category.
let me know if you have more questions, thank you!
Yue

Thank you for your response! Just one more thing that I forgot to ask. What is the exact accuracy for the goal (on random, similarity, category)?

Please see the results in the table 2 of the paper.

Oh, I missed that. Thank you!


Move reactive programming example to TP

The code using reactive programming examples in the project DataProvider.TextReader
Scope

[x] create branch Maintenance-3-100
[x] upgrade NuGet packages
[x] code refactoring (file headers, formatting, etc)
[x] transfer to TP
[x] integrate with the TP

Next steps:

[ ] fix recent modifications in the origin project
[ ] final review the code


how would i go about transporting a binary rblm to a xml rblm?

for a game im working on

Something like this would work:
public static XmlRobloxFile ToXmlFile(BinaryRobloxFile bin)
{
    var xml = new XmlRobloxFile();

     foreach (var child in bin.GetChildren())
        child.Parent = xml;

    return xml;
}