import error

I installed tensorflow 2.1 and pymc4 on window 10 machine. The packages are install in a new venv environment. I can import tensorflow (with some warning), but when trying to import pymc4 I got this error "AttributeError: module 'tensorflow_core.compat.v2.math' has no attribute 'xlog1py'"
I check and found that tensorflow.math.xlog1py throws the same error.  Please suggest.

import tensorflow
2020-02-11 09:55:38.474276: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'cudart64_101.dll'; dlerror: cudart64_101.dll not found
2020-02-11 09:55:38.482351: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
import pymc4
Traceback (most recent call last):
File "", line 1, in
File "c:\Users\user\pyenv\py36\lib\site-packages\pymc4_init_.py", line 6, in
from . import distributions
File "c:\Users\user\pyenv\py36\lib\site-packages\pymc4\distributions_init_.py", line 1, in
from .continuous import *
File "c:\Users\user\pyenv\py36\lib\site-packages\pymc4\distributions\continuous.py", line 4, in
from tensorflow_probability import distributions as tfd
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability_init_.py", line 76, in
from tensorflow_probability.python import *  # pylint: disable=wildcard-import
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python_init_.py", line 21, in
from tensorflow_probability.python import bijectors
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python\bijectors_init_.py", line 23, in
from tensorflow_probability.python.bijectors.absolute_value import AbsoluteValue
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python\bijectors\absolute_value.py", line 23, in
from tensorflow_probability.python.bijectors import bijector
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python\bijectors\bijector.py", line 33, in
from tensorflow_probability.python.internal import distribution_util
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python\internal\distribution_util.py", line 28, in
from tensorflow_probability.python.internal import prefer_static
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python\internal\prefer_static.py", line 30, in
from tensorflow_probability.python.internal.backend import numpy as nptf
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python\internal\backend\numpy_init_.py", line 22, in
from tensorflow_probability.python.internal.backend.numpy import compat
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python\internal\backend\numpy\compat.py", line 24, in
from tensorflow_probability.python.internal.backend.numpy import v1
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python\internal\backend\numpy\v1.py", line 37, in
from tensorflow_probability.python.internal.backend.numpy.random_generators import set_seed
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python\internal\backend\numpy\random_generators.py", line 27, in
from tensorflow_probability.python.internal.backend.numpy.numpy_math import softmax as _softmax
File "c:\Users\user\pyenv\py36\lib\site-packages\tensorflow_probability\python\internal\backend\numpy\numpy_math.py", line 912, in
tf.math.xlog1py,
AttributeError: module 'tensorflow_core.compat.v2.math' has no attribute 'xlog1py'
tensorflow.math.xlog1py
Traceback (most recent call last):
File "", line 1, in
AttributeError: module 'tensorflow_core._api.v2.math' has no attribute 'xlog1py'

Just force upgrade your package using: pip install --upgrade --force tensorflow.
If it doesn't work reinstalling TensorFlow may help!

I tried upgrade and reinstall, but still does not work.

@worasom, although it will sound strange, the problem you are seeing is caused because you installed the tensorflow package. I'll explain why.
When we started developing pymc4, we made the package depend on tensorflow-nightly, because we wanted to profit from the latest unreleased v2 features. As pymc4 is still quite fluid and not ready for a beta release we haven't switched to depend on the latest official releases on tensorflow and tensorflow_probability.
You may be asking yourself why does this cause an error? The thing is that from pip's (and setuptools) point of view, tensorflow and tensorflow-nightly are two completely unrelated packages. The problem is that in fact they aren't. They both end up populating the exact same directory in your python's site-package directory. This means that if you try to install both tensorflow and tensorflow-nightly, pip will overwrite things incorrectly and you'll end up with a broken installation.
The sequence of things that lead to your error are:

Pip installs tensorflow
Try to pip  install pymc4
Pip looks up the pymc4 requirements and finds it needs to install tensorflow-nightly.
Pip installs tensorflow-nightly and breaks the previous tensorflow installation, but pip thinks that everything went great because from its point of view, tensorflow and tensorflow-nightly are completely unrelated packages.
When you try to import pymc4, python fails to import tensorflow and errors out.

How do you fix things? First you'll have to completely uninstall tensorflow. Then you just have to run pip install pymc4 and requirements will be handled automatically.


Merge imports and items, and sort items within an import statement

Looks for sequential import statements within a block, and merges them together if they share the same stem/module, the discards the second/redundant import statements. Items are sorted using the same case-insensitive sorting used for sorting statements based on module name.
The existing µsort codebase has also been formatted/sorted using this new behavior.

Still need to add more thorough functional tests to validate full behavior, as well as documenting this change in functionality.


Multiple warnings in console: "Failed to parse source map from '[.../browser.ts and others]' file: Error: ENOENT"

Describe the bug
Since the first time I installed the library, the errors popped on the console.
WARNING in ./node_modules/custom-electron-titlebar/lib/menu/menu.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '/.../node_modules/custom-electron-titlebar/src/menu/menu.ts' file: Error: ENOENT: no such file or directory, open '/.../node_modules/custom-electron-titlebar/src/menu/menu.ts'

WARNING in ./node_modules/custom-electron-titlebar/lib/menu/menuitem.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '/.../node_modules/custom-electron-titlebar/src/menu/menuitem.ts' file: Error: ENOENT: no such file or directory, open '/.../node_modules/custom-electron-titlebar/src/menu/menuitem.ts'

WARNING in ./node_modules/custom-electron-titlebar/lib/menubar.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '/.../node_modules/custom-electron-titlebar/src/menubar.ts' file: Error: ENOENT: no such file or directory, open '/.../node_modules/custom-electron-titlebar/src/menubar.ts'

WARNING in ./node_modules/custom-electron-titlebar/lib/themebar.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '/.../node_modules/custom-electron-titlebar/src/themebar.ts' file: Error: ENOENT: no such file or directory, open '/.../node_modules/custom-electron-titlebar/src/themebar.ts'

WARNING in ./node_modules/custom-electron-titlebar/lib/titlebar.js
Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '/.../node_modules/custom-electron-titlebar/src/titlebar.ts' file: Error: ENOENT: no such file or directory, open '/.../node_modules/custom-electron-titlebar/src/titlebar.ts'

I narrowed the issue to the source maps, which point to non-existing files in node_modules/custom-electron-titlebar/src/[file].ts. Other packages don't do such things, rather they point to .d.ts definitions.
Fixing it requires the source maps to be changed from this:
{"version":3,"file":"themebar.js","sourceRoot":"","sources":["../src/themebar.ts"],

To this (since sources aren't included):
{"version":3,"file":"themebar.js","sourceRoot":"","sources":["./themebar.d.ts"],

To Reproduce
Steps to reproduce the behavior:

Install the library on an Angular 10 app
Open Dev tools
See error

Expected behavior
It shouldn't
Screenshots

Desktop (please complete the following information):

OS: Mac
Electron version: 11
Framework: Angular

I can confirm this issue. Started to happen since latest update.

I have the same issue

me too!


Add ability to override SEO description in docs pages

This PR adds the ability to use description and title frontmatter fields to override the SEO description and title of any docs page, in the same way as the Blog pages.
Since the Blog and Docs use the same SEO component, all the same SEO tags are used. If either is not provided, the page will fall back on the same SEO title and description it had before.
Fixes #1785
Can be verified on the deploy preview with test data here. I'll remove it before merge.

So, can we and should we auto-generate these descriptions? We should also check how other tools, search engines do this.

It's possible to auto-generate descriptions, and even make auto-generation manually overridable, but I don't know what template or rules would work to make a description out of here. There isn't much metadata to work with in these docs pages other than the page name.

@rogermparent btw, why don't we see the link check after the deployment? No changes to content files?

@rogermparent

but I don't know what template or rules would work to make a description out of here. There isn't much metadata to work with in these docs pages other than the page name.

that's a good task to figure this out indeed. I would first check existing libs, existing doc systems.

@rogermparent btw, why don't we see the link check after the deployment? No changes to content files?

Good catch, I'll have to figure that out.

There isn't much metadata to work with in these docs

There's the title (H1 header), usually the first paragraph is a good intro (we could truncate it into a preview), and there are <abbr> terms in case we also want to add keywords (cc @jeremydesroches WDYT?)
All could be overwritten manually with this frontmatter.

There isn't much metadata to work with in these docs

There's the title (H1 header), usually the first paragraph is a good intro (we could truncate it into a description), and there are <abbr> terms in case we also want to add keywords (cc @jeremydesroches WDYT?)
All could be overwritten manually with this frontmatter.

The behaviour as described in OP and as tested looks good.
I don't think generating descriptions is needed. I think it's okay to leave the standard fallback if a custom description isn't added.
For SEO purposes, leaving the description blank is actually better than generating one (if there isn't a custom description available). When the description is left blank, Google and other search engines will do their own generation based on search query and document content — likely to be better than what we can generate.

For SEO purposes, leaving the description blank is actually better than generating one (if there isn't a custom description available). When the description is left blank, Google and other search engines will do their own generation based on search query and document content — likely to be better than what we can generate.
Also worth clarifying: the description doesn't directly contribute to how a page is indexed as it's not being checked for keywords, etc. It contributes indirectly through its effect on clickthrough and bounce, which are key engagement metrics that affect the index position (rank). A good description can increase likelihood of clickthrough and, if well matched to the content, lower the bounce rate.

Interesting, I never knew this. I could tweak our current setup to do this either everywhere or just on docs pages if that's desirable.

leaving the description blank is actually better... search engines will do their own generation

Great! Let's leave the heavy lifting to the SEOs then!

I could tweak our current setup to use blank descriptions either on docs pages or everywhere

WDYT @shcheklein? If yes, let's create a separate issue though. Thanks!


consider using the system UI font for feed & article list

Is your feature request related to a problem? Please describe.
By default the Feed list and Article list panes use “Lucida Grande” as a font. This feels quite out of place on systems that use the “SF-**” font families for any lists.
Describe the solution you would like
Choose the San Fransisco system font for Article and Feed list at the default font-size used by the OS.
MacOS 12, Vienna 3.8.1

With Vienna 3.9b, this seems to have been fixed along the lines of https://github.com/ViennaRSS/vienna-rss/issues/1603#issuecomment-1222797345
Closing.


Export CSV

Description
Adds a basic CSV export feature

TODOs
Read the Gruntwork contribution guidelines.

[x] Update the docs.
[ ] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.
[ ] Attention Grunts - if this PR adds support for a new resource, ensure the nuke_sandbox and nuke_phxdevops jobs in .circleci/config.yml have been updated with appropriate exclusions (either directly in the job or via the .circleci/nuke_config.yml file) to prevent nuking IAM roles, groups, resources, etc that are important for the test accounts.

Release Notes (draft)

Added CSV export feature to aws and inspect-aws
Migration Guide
No breaking changes

Friendly ping on this PR.

CLosing this PR. We can reopen it later.


Uncaught TypeError: this.opts.extensions.join is not a function

http://cl.ly/image/0K0a0a2J3u2m
This happens about 50% of the time when I refresh the page, which of course breaks the menu.
Any ideas? I moved the script to load at in the head but same results.

Here's a live URL.
Go to https://attorney-group.com/colorado/ scroll to the bottom of the page. Refresh your browser a few times, you'll see an error in console and the links will show at the bottom of the page.
I'm using
	$menu.mmenu({
		"offCanvas": {
			"position": "right"
		},
		"extensions": [
			"theme-dark"
		]
		}, {
		"classNames" : {
			"selected" : "current-menu-item"
		}
	});

I can't figure out why it works on some loads but errors on others.
Any insight?

It's something with the extensions opts.
I cleaned things up, per the documentation, but as long as I provide an extension (or multiple) the errors still occur frequently
	$menu.mmenu({
		"extensions": [
          "theme-dark",
          "pageshadow"
        ],
		"offCanvas": {
			"position": "right"
		},
		"classNames" : {
			"selected" : "current-menu-item"
		}
	});

If I remove the extensions it works fine....

That's odd, if the extensions option always is an array, I can't think of any reason why join would not be a function.

I know, I can't figure it out for the life of me, it's driving me nuts :grin: .
If you sit there and refresh that page several times you'll see sometimes it loads without issue, other times that error triggers and the menu fails.
There are no other console errors, so I don't see where anything else would be conflicting.
I'm trying to think that the scenario could be? Menu sometimes loads too quick or too late? Code is below. I thought maybe in some cases things are trying to initialize too fast maybe?
	var $menu = $(".nav-primary").clone().appendTo('body');
	$menu.removeAttr( "id" ).removeAttr( "class" ).find( 'ul' ).removeClass( "genesis-nav-menu menu-primary" );
	$menu.mmenu({
		"extensions": [
          "theme-dark",
          "pageshadow"
        ],
		"offCanvas": {
			"position": "right"
		},
		"classNames" : {
			"selected" : "current-menu-item"
		}
	});
	var API = $menu.data( "mmenu" );
	$(".mobile-menu-toggle").click(function() {
		API.open();
	});

Somehow, I still can't reproduce the issue, I've refreshed the page about 20 times but am not getting the error.
Completely irrelevant to this issue, but your classNames.selected should be in the second argument (the "configuration" object).

Yeah it's weird to me too. Never figured it out, I ended up just not loading the extensions and it works fine. I'm using this script on probably half a dozen seperate client sites (love it!) and this is the only one that has encountered this error - so even though the console log doesn't show it, I'm going to assume there is some script or edge case at play here just on this site. Because of that I'm not going to worry about it any further - when I posted this I thought maybe it was something obvious.
So you're saying the correct syntax would be what I have below? Let me know. That and go ahead and close this.
$menu.mmenu({
        "extensions": [
          "theme-dark",
          "pageshadow"
        ],
        "offCanvas": {
            "position": "right"
        },
      }, {
        "classNames" : {
            "selected" : "current-menu-item"
        }
    });

That ought to do it, check the mmenu + WordPress tutorial:
http://mmenu.frebsite.nl/tutorials/frameworks/wordpress.html

I've run across this as well, you can kind of see what's going on here:

In this case, some overly chatty PHP caused multiple elements with the [data-ref="mobileMenu"] selector to get passed to the constructor. The exception is not raised when passed a constructor that matches a single element. Maybe that will help you narrow it down.
  $(function() {
    // mobile nav
    $('[data-ref="mobileMenu"]').mmenu({
       "offCanvas": {
          "position": "top"
       },
       "extensions": [
          "theme-dark"
       ],
       "autoHeight": true,
       "navbars": [
          {
             "position": "top",
             "content": [
                "prev",
                '',
                '',
             ]
          },
          {
             "position": "bottom",
             "content": [
                'REGISTER'
             ]
          }
       ]
    });
  });


Added monitoring function for link022

Added a goroutine to periodically collect AP & system states.
Added a Prometheus exporter for link022 to export states.

Codecov Report

Merging #81 into master will decrease coverage by 3.35%.
The diff coverage is 54.41%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage      63%   59.65%   -3.36%
==========================================
  Files          11       14       +3
  Lines         638     1046     +408
==========================================
+ Hits          402      624     +222
- Misses        167      326     +159
- Partials       69       96      +27

Impacted Files
Coverage Δ

agent/service/hostapd.go
69.33% <ø> (ø)
:arrow_up:

agent/syscmd/states.go
0% <0%> (ø)

agent/monitoring/prometheus/exposition_server.go
0% <0%> (ø)

...nt/monitoring/prometheus/openconfig_ap_exporter.go
56.34% <56.34%> (ø)

Continue to review full report at Codecov.

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


Interview Question - Why promises are preferred over callbacks?

Can I take this?

Can I take this?

Sure


Build docker image in travis-ci

docker cloud queue time is too long

Disable auto build in docker cloud, replace with travis-ci when any pr merged into master


Add support for loading Testplatform.fakes in Dev 17

We ship v 17.0 of the Testplatform.Fakes assembly in Dev 17. This change accounts for it.

@cvpoienaru @AbhitejJohn Ready for review and merge. Testing it locally with the changes, the Fakes tests were able to be run with the TestPlatform.Fakes 17.0 and 16.0 versions.


feat(CategoryTheory/Monoidal): replace 𝟙 X ⊗ f with X ◁ f

We set id_tensorHom and tensorHom_id as simp lemmas. Partially extracted from #6307.

[x] depends on: #10898

Thanks!
bors merge


move build command into variable

Feature Request
The Problem
If my build command differs from the one hardcoded in the action, we have a problem.
The Solution
Change to yarn deploy
Alternatives
Move into variable.
Additional Context

I arrive late.


fix: change to freeze the selected package while installing or uninstalling

Description

Changes to freeze the selected package while installing or uninstalling.
Related Issue

fixes #111
Motivation and Context

Reference #111
How Has This Been Tested?

OS: Windows 10 Home
Version: 21H1
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

It looks a good fix.


Regarding the screenshot

This is not an issue, just a question.
How are you achieving the titlebar free / outlined iTerm windows? Is it simply a mod of iTerm itself, or did you write a plugin to do it?

It's in Preferences > Profiles > Window > 'Style' drop-down > No Title Bar.
(I'm not sure if it's still only feature of nightly builds)

Ah thanks, I was also curious about that. The beta has it now (just grabbed it).
I have another question though: What about that menu bar?

The Apple menubar is hidden using the option that was added with El Capitan. I'm using the program nerdTool to create the illusion of a bar that you see in the screenshot.

@koekeishiya care to share/export that bar configuration? It looks great.

nerdTool-Bar.zip
Scripts have to be customized for your system according to resolution.
The actual bar is just an image file that is displayed below the printed info.

Thanks for the explanation.
(Side note: when I was writing http://github.com/fjolnir/xnomad I achieved focus follows mouse via SIMBL(squeezed into http://github.com/fjolnir/afloat))

Oh right, uh the titlebar free iTerm is done with the iTerm setting as rastasheep explained.
If you are wondering about the green border, I wrote a script that basically draws a transparent rectangle using (hammerspoon) hs.drawing extension. This can be found here:  https://gist.github.com/koekeishiya/dc48db74f4fdbfbf5648

Really liking kwm so far, especially now that keybindings aren't compiled but set in .kwmrc.
Quick question, what is the name of the web browser in the two panes on the left in the new screenshot?
https://cloud.githubusercontent.com/assets/6175959/12092967/8d8853d8-b300-11e5-8a44-ec1245efdc74.png

I'm using firefox with stylish for custom css

Is that "archey" you're running there? I really like how the apple logo is looking. Possible to share?
side note: I love kwm, can't live without it. keep up the good work :+1:

The logo and information is printed using the program screenfetch.

Just wanted to add that I recreated @koekeishiya's NerdBar configuration for use in Übersicht:
https://github.com/herrbischoff/nerdbar.widget
Maybe that's of interest for someone else but me. ;)

That looks great, I'll be using it for sure. Übersicht seems a lot easier to main and change than nerdTool is. Thanks.
I may put it up as an example in the readme.md for how to get a decent bar if you don't mind?

@koekeishiya: Sure, go ahead. License is MIT btw, just as kwm. Please also feel free to send PRs in case you adjust something like distances, etc. Those may be off as I didn't spend a lot of time optimizing them. The colors were mostly grabbed through a color picker, so feel free to adjust them where necessary as well.

I have now added a some description / link regarding your widget to the readme.md.
Thanks again : >

You're very welcome! The least I can do for my now favorite window manager. ;)


Download Stories function broken

Lately when I try to download a person's story which has videos in it, the download fails with various 403 errors. Oddly enough, the videos load fine when viewing the stories, this only happens when trying to initiate a story download. Also, this only happens on some stories with videos, not all of them.

GET https://scontent-lga3-1.cdninstagram.com/v/-token-removed-/-videofilename-.mp4 403 ()

Failed to load https://scontent-lga3-1.cdninstagram.com/v/-token-removed-/-videofilename-.mp4: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'chrome-extension://bojgejgifofondahckoaahkilneffhmf' is therefore not allowed access. The response had HTTP status code 403.

content.js:108717 Uncaught (in promise) Error: InvalidStateError: Failed to read the 'responseText' property from 'XMLHttpRequest': The value is only accessible if the object's 'responseType' is '' or 'text' (was 'arraybuffer').
at XMLHttpRequest.xhr.onreadystatechange (content.js:108717)

The "downloading pending" spinning circle keeps looping and the download never happens.

same.

@prKassad's change fixed this issue. Thanks!


Use /bin/bash in pre/post scripts

Looks fine, but was it a different shell other than bash that it was using? (zsh?)

I think this is to make it match Xcode defaults when creating a new run script but could be wrong

It was using /bin/sh before. This change makes it so you don't have to wrap more complicated things in another script.

No one has asked for it, and my example isn't needed anymore, so I could just drop this. Or I could make it an option.


Refactor, Cleanup Code and use Generics

Refactored handler to use generics instead of "podController" wrapper type
Deduplicated Getter Types
Deduplicated Controller logic and entrypoints
Deduplicated Webhook entrypoints
Deduplicated Controller Tests (using generics)

Tests are now exactly the same for all three objects (there were slightly diverged before)
Deduplicated podTemplate in Test Data

Overall we lost about 1k lines of code. Also extending/maintaining controllers and controller tests became easier. Before you basically had to make the same change in three files each. That is now in one place. Only downside is that you have to deal with generic types in tests now.

I will resolve the conflict. I also found why my tests for #167 failed. I will improve the test setup to make those easier to add.


Cleanup unneeded requirements

python-distribute conflicts with setuptools and is deprecated.
Moreover we don't run unit test at build time

Can one of the admins verify this patch?

@hguemar could you please create an issue in the tracker for this fix and try to go over the guide at https://github.com/ceph/ceph-deploy/blob/master/CONTRIBUTING.rst#contributing-to-ceph-deploy
Some of your changes would break for Python 2.6 (e.g. it removes the python-argparse requirement). This PR also removes python-tox but doesn't remove python-mock. I think that the removal of those test dependencies is good, but can you remove all of the test dependencies?

LGTM. We can always add test deps back if/when we can ever get the tests to pass during rpmbuild.

@alfredodeza I'll rework that patch as per your suggestion.

@hguemar ping?

Bump...this would be real nice to have, as it's not possible to install the Luminous version of ceph-deploy on Fedora - it's provided by another package now:
$ rpm -qf /usr/lib64/python2.7/argparse.py
python2-libs-2.7.14-2.fc26.x86_64

@mcronce would you be willing to take over this PR and update @hguemar commits to follow the contributing guidelines? This needs to be rebased as well. I don't mind a new PR and crediting the initial work


Added act-table data-controller in satis

Reverted count menu to count dropdown hence closing this PR.


[localhost] [bug] Running gives AssetNotFound("index.html")

Tested on a fresh Vanilla JS initialization.
Here's my Cargo.toml, I just added the plugin and portpicker:
[package]
name = "testing"
version = "0.0.0"
description = "A Tauri App"
authors = ["you"]
license = ""
repository = ""
edition = "2021"

[build-dependencies]
tauri-build = { version = "1.3", features = [] }

[dependencies]
tauri = { version = "1.3", features = ["shell-open"] }
serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0"
tauri-plugin-localhost = { git = "https://github.com/tauri-apps/plugins-workspace", branch = "v1" }
portpicker = "0.1" # used in the example to pick a random free port

[features]
custom-protocol = ["tauri/custom-protocol"]

And here is my main.rs
#![cfg_attr(not(debug_assertions), windows_subsystem = "windows")]

use tauri::{utils::config::AppUrl, window::WindowBuilder, WindowUrl};

fn main() {
  let port = portpicker::pick_unused_port().expect("failed to find unused port");

  let mut context = tauri::generate_context!();
  let url = format!("http://localhost:{}", port).parse().unwrap();
  let window_url = WindowUrl::External(url);
  // rewrite the config so the IPC is enabled on this URL
  context.config_mut().build.dist_dir = AppUrl::Url(window_url.clone());
  context.config_mut().build.dev_path = AppUrl::Url(window_url.clone());

  tauri::Builder::default()
    .plugin(tauri_plugin_localhost::Builder::new(port).build())
    .setup(move |app| {
      WindowBuilder::new(app, "main".to_string(), window_url)
        .title("Localhost Example")
        .build()?;
      Ok(())
    })
    .run(context)
    .expect("error while running tauri application");
}

And it gives this:

Can you share your tauri.conf.json? You also need to clear the windows array.

You also need to clear the windows array.

ok, but the same error happens:
{
  "build": {
    "beforeDevCommand": "",
    "beforeBuildCommand": "",
    "devPath": "../src",
    "distDir": "../src",
    "withGlobalTauri": true
  },
  "package": {
    "productName": "testing",
    "version": "0.0.0"
  },
  "tauri": {
    "allowlist": {
      "all": false,
      "shell": {
        "all": false,
        "open": true
      }
    },
    "bundle": {
      "active": true,
      "icon": [
        "icons/32x32.png",
        "icons/128x128.png",
<EMAIL_ADDRESS>        "icons/icon.icns",
        "icons/icon.ico"
      ],
      "identifier": "com.tauri.dev",
      "targets": "all"
    },
    "security": {
      "csp": null
    },
    "updater": {
      "active": false
    },
    "windows": []
  }
}

Ok the docs is wrong, it only works on production. Use it like this instead:
#![cfg_attr(not(debug_assertions), windows_subsystem = "windows")]

use tauri::{utils::config::AppUrl, window::WindowBuilder, WindowUrl};

fn main() {
  let port = portpicker::pick_unused_port().expect("failed to find unused port");

  let mut context = tauri::generate_context!();
  let url = format!("[http://localhost:](http://localhost/){}", port).parse().unwrap();
  let window_url = WindowUrl::External(url);
  // rewrite the config so the IPC is enabled on this URL
  context.config_mut().build.dist_dir = AppUrl::Url(window_url.clone());

  tauri::Builder::default()
    .plugin(tauri_plugin_localhost::Builder::new(port).build())
    .setup(move |app| {
      WindowBuilder::new(app, "main".to_string(), if cfg!(dev) { Default::default() } else { window_url })
        .title("Localhost Example")
        .build()?;
      Ok(())
    })
    .run(context)
    .expect("error while running tauri application");
}

If you want to open the app on the browser you need to check out the tauri-invoke-http: https://github.com/tauri-apps/tauri-invoke-http

Also i'm not sure what you meant, the app localhost URL should be accessible in the browser on dev too. You just won't be able to use any Tauri API without tauri-invoke-http.

Also i'm not sure what you meant, the app localhost URL should be accessible in the browser on dev too. You just won't be able to use any Tauri API without tauri-invoke-http.

Nor be able to access index.html unless I'm on production.

Sorry I confused you.
What I want is to be able to access the app through the classic window way, but also be able to access it through the browser.

Then you need to use tauri-invoke-http.


[Feature]: ADDING A FEEDBACK SECTION [GSSOC'23]

[Y ] Would you like to take up this task?
[Y ] Are you from GSSoC'23?

✨
Feature request

Is your feature request related to a problem?
Kindly provide an explanation of the problem that your feature aims to resolve.

Describe the solution you'd like
Please present a concise and precise description of the solution you envision.

Describe alternatives you've considered
Elaborate on any alternative solutions or features that you have already evaluated as potential approaches.

Additional context
Include any supplementary information, contextual details, relevant screenshots, or supporting materials that can aid in better understanding and supporting your feature request.

I want to contribute, Can your Please assign me this task?

@sourabhsikarwar Please provide your email address, so that i can add it on feedback section, so that you will be able to get the feedbacks

@kharshita590 My mail is<EMAIL_ADDRESS>

Okay @Harshu467


Tower placement does not work

Describe the bug
Since the redesign, when spawning new Towers into the game, the Towers are not placed correctly. Instead they are placed far off to the side of the board barely visible.
To Reproduce
Steps to reproduce the behavior:

Start the application.
Click on "Select Level".
Start a Level.
Hold a Tower card into the camera.
Click on "Build Tower".
Select a tower to build.
The tower will be barely visible in the distance when the game board is displayed.

Expected behavior
The tower should be built at the location of the Tower card.
Bug Observed on the Following Devices
Check applicable devices and platform by putting an x in-between the square brackets.

[x] Unity editor
[ ] HoloLens 1
[ ] HoloLens 2
[ ] HTC VIVE
[x] Smartphone
[ ] Other:

Some missing references caused the problem to occur and has now been resolved.


No dropzone exists in clone use case when data source is an ng-repeat and not fixed object

So most of the examples we use are fixed, hardcoded objects.  My data consists of a variable number of ui-sortable lists which must be able to clone to a single ui-sortable.  Currently, in the clone use case, attempting to drag from one of these other ui-sortables into the target will render a dropzone of undefined and as a result fail to move anything properly.
Sorry if I fail to tag this properly, not the most experienced with github's issue tracker.

Hi there!
Can you fork one of the examples found in README and post it to showcase you use case?

Any feedback on this?

Sorry, I'll try to get some data soon but I've been forced to move on in my
project
On May 28, 2017 4:13 PM, "Thodoris Greasidis"<EMAIL_ADDRESS>wrote:

Any feedback on this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angular-ui/ui-sortable/issues/519#issuecomment-304537405,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AauxE_22VkRPJ-cG8_KuLcew6XZ3fjb3ks5r-dVxgaJpZM4NfNug
.

Updated the cloning example found in README to work with a dynamic number of sources.
Closing for now, feel free to re-open in any case.


Track changes made to ORM fields since fetch/save

I have some use cases that require me to keep track of which fields I've changed. This branch adds a new public property to each model, Model.changes, which will record any changes made to the model's fields since the last time it was fetched or persisted. The implementation relies on each field telling the model instance about a change as it happens.
This could probably be a lot more robust and featureful, but I didn't want to overcomplicate things before writing a few tests for the basics and opening a branch.

Is this something like this you are looking for ? https://docs.djangoproject.com/en/4.1/ref/models/instances/#django.db.models.Model.refresh_from_db
I suggest the same keyword then. 😀

Is this something like this you are looking for ? https://docs.djangoproject.com/en/4.1/ref/models/instances/#django.db.models.Model.refresh_from_db

Not exactly. I'm more thinking of "dirty fields", similar to the django-dirtyfields plugin. I could've mimicked that API more closely but wanted to get some feedback on the idea before building out any more functionality.

This is my own branch and it's kinda outdated. I'll close for now and we can come back to this later.


some renaming and removal

Pull Request Test Coverage Report for Build 82746

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 78.229%

Totals

Change from base Build 82739:
0.0%

Covered Lines:
5088

Relevant Lines:
6504

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.5.3

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.5.3


How to capture the video on neo

In [7]: vd = open('/dev/video0', 'rw')
In [8]: cp = v4l2.v4l2_capability()
In [11]: fcntl.ioctl(vd, v4l2.VIDIOC_QUERYCAP, cp)
IOError                                  Traceback (most recent call last)
 in ()
----> 1 fcntl.ioctl(vd, v4l2.VIDIOC_QUERYCAP, cp)
IOError: [Errno 25] Not a typewriter
Does anybody know how to capture the video stream on neo?

Opp  0 ppl
On Sun, Mar 25, 2018, 10:01 AM qmind<EMAIL_ADDRESS>wrote:

In [7]: vd = open('/dev/video0', 'rw')
In [8]: cp = v4l2.v4l2_capability()
In [11]: fcntl.ioctl(vd, v4l2.VIDIOC_QUERYCAP, cp)
IOError Traceback (most recent call last)
in ()
----> 1 fcntl.ioctl(vd, v4l2.VIDIOC_QUERYCAP, cp)
IOError: [Errno 25] Not a typewriter
Does anybody know how to capture the video stream on neo?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/issues/225, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABfUYYuNQw_SVa-JXoyQMJG-IFpoe89iks5th6MfgaJpZM4S6Jmy
.

I'm not sure what you're trying to do. Feel free to reopen with a more elaborate explanation of what you're trying to achieve and what doesn't work.


added umami and matomo

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Improvement (refactoring and improving code)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

Description
added Umami and Matomo tracking codes
Umami docs with example of tracking code: https://umami.is/docs/collect-data
Matomo docs with example of tracking code: https://developer.matomo.org/guides/tracking-javascript-guide
Additional context
Fixes #1645

Thank you so much!
Can you please provide an reference to the official documentation to prove the accuracy of the PR changes?

what do you mean? Sorry, I didn't understand

I mean examples of official docs on integrating tracking code into a website from Umami and Matomo respectively.

sure! I've updated the PR description

I was trying out migrating to this Umami Integration in config but even though the include script is there, I'm not sure if it's actually being included in the rendered html. The script tag is not in the build html. Am I missing something?
I added domain and id to the config, and that's it

@PriyavKaneria Please note that web analytics is only included in the production build. Please double check as it seems to be working as expected. If you are still having problems, please post a separate discussion to provide more details.
JEKYLL_ENV=production bundle exec jekyll s


[BEAM-2344] Re-rename fat jar

Signed-off-by: Jason Kuster<EMAIL_ADDRESS>Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify.
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @davorbonaci @dhalperi

Coverage remained the same at 70.667% when pulling 94cdd31e1e9a386dfcc7964aecf14ffc1c4f5850 on jasonkuster:jar-name into b633abe2c8efd3ac607a0be14980315e5f0d9fa6 on apache:master.


[Improvement][master] TaskPriorityQueueConsumer No need to sleep for one second

Describe the question
TaskPriorityQueueConsumer.run() call taskPriorityQueue.size() and if size is 0,then thread will b sleep for 1 second.
I think there is no need to do this, code is show here:

I think these three lines removed will be better，because  taskPriorityQueue.take() is a blocking method.
Removed sleep... will improve the performance of scheduler.
  if (taskPriorityQueue.size() <= 0) {                         Thread.sleep(Constants.SLEEP_TIME_MILLIS);                         continue;    }

TaskPriorityQueueConsumer's logic is very easy. just cosume the task waiting to run in  priority queue. I think that there is no need a determine statement if(queue.size() <= 0) Thread.sleep(1 second). because code below is queue.take() which is a blocking method if queue is empty.

See for specific reasons #4140

See for specific reasons #4140

anybody who want to do this feature please refer the PR #4140, then re-submit a new PR

I can solve this problem, assign it to me!

I can solve this problem, assign it to me!

good, happy new year,  I have assigned to you, looking forward to your PR

I can try it, assign it to me!

I can try it, assign it to me!

done


Remove your poe token from git history

...

Did I leak it?

Could you post the commit/file link?

Could you post the commit/file link?

https://github.com/mak448a/AI-Drawing-ChatBot/commit/1fbb2ac
https://github.com/mak448a/AI-Drawing-ChatBot/commit/ad9abcf

Ok, thanks. I'll just change my password.

Lol I realized that this was an example key (notice that the pattern is very QWERTY-like) 😅

Lol I realized that this was an example key (notice that the pattern is very QWERTY-like) 😅

lol


Industry Data Collection

[ ] Census Bureau: ASM, Econ Census: @calmc provided basic methods.

[  ] Change API key method to match EIA API methods
[  ] Verify that relevant variables are being returned.

[ ] BEA for chain quantity indexes:

[ ]  read in historical data (1969-1997)
[x]  append new data from BEA API: CM completed with commit 994c4fa

[ ] Bureau of Labor Statistics (BLS)
[ ] Create csvs of mining energy data for electricity and fuels from NEA data (NEA_data tab in Minin energy_....xlsx).

@iisabeller , I'll finish the methods for BEA data, as well as for BLS.


[Bug]: Overdiding Toaster component WARNING

Reproduction
https://nuxt.new
Describe the bug
I am getting the following ERROR with Nuxt:
[8:13:13 AM]  WARN  Overriding Toaster component. You can specify a priority option when calling addComponent to avoid this warning.

And I don't know how to fix it. Because, Toaster is not working. I removed all the toaster related code where I used it. But it is giving error even now. Only it exists in components/ui/toast folder.
System Info
System:
    OS: Linux 6.8 Ubuntu 24.04.1 LTS 24.04.1 LTS (Noble Numbat)
    CPU: (8) x64 Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz
    Memory: 4.31 GB / 15.27 GB
    Container: Yes
    Shell: 5.2.21 - /bin/bash
  Binaries:
    Node: 23.1.0 - ~/.nvm/versions/node/v23.1.0/bin/node
    npm: 10.9.0 - ~/.nvm/versions/node/v23.1.0/bin/npm
    pnpm: 9.12.2 - ~/.local/share/pnpm/pnpm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 130.0.6723.69
  npmPackages:
    @vueuse/core: ^11.1.0 => 11.1.0
    nuxt: ^3.13.2 => 3.13.2
    radix-vue: ^1.9.7 => 1.9.7
    shadcn-nuxt: ^0.10.4 => 0.10.4
    vue: latest => 3.5.12

Contributes

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests

Oh, BTW, I found what's causing the error. I had installed sonner and also toast components at the same time. So, it was giving this error. I removed sonner component and this resolved the issue.
But, I couldn't find a way to work with both at the same time in different view/page .vue components. Pls, help...

Also encountered this issue, would be nice if the cli could give a warning on which one to choose for the calendar and toast if you accidently select both

I fixed this by changing default name of Toaster to Sonner or anything else that is not Toaster, example:
// components/ui/toast/index.ts
export { default as Sonner } from "./Sonner.vue";


support multiple gettext domains

Instead of 'site' as the default gettext domain, use a configurable default domain. Laddr's default domain could be "laddr" with a laddr.po file containing all the translations for the app itself.
Then when a brigade wants to translate some of their brochure content, they could create a separate .po file and not have to deal with merging upstream translation changes

+1


add --color option

I am having problems with the color output of the cli runnnig it with docker-compose.
Would be posible to force the color output in the same way it is posible to disabled it with --no-color?

For color support I currently use chalk. This library supports forced colors if the environment variable FORCE_COLOR=2 is set (see https://www.npmjs.com/package/chalk#supportsColor)

docker-compose.yml

....
  environment:
      FORCE_COLOR: 2
...

Test with the following Dockerfile would produce colored output: (added in root of httpyac repo)
# syntax=docker/dockerfile:1

FROM node:16.14.0-alpine3.14
COPY . ./app
WORKDIR app

RUN pwd && npm install && \
 npm run compile

ENV FORCE_COLOR=2
VOLUME /app/http

docker run -v /home/weber/src/httpyac/httpyac.github.io/examples/request:/app/http anweber/httpyac:0.0.1 -- "**/*.http"

Thank you, i will try it.
By the way, i am loving the project :D

For me it worked with FORCE_COLOR. But in my use case I don't need the color support.
I will deploy a managed Docker image and keep that update to date. Only my first attempt with pack had failed. But maybe until then I will just use a Dockerfile.

It worked perfectly!
FROM node:17-alpine
RUN npm i -g httpyac
WORKDIR /test
ENV NODE_ENV=production
ENV FORCE_COLOR=2
COPY . .
CMD ["httpyac","send", "**/*.http", "--all","-e", "production", "-o", "exchange"]

thx for sharing your Dockerfile.


location based reminders

Hi this is cool but it'd be nice to be able to do something like
r get milk when leaving home
possible?

Probably not but will keep this open.

Adding a note to myself here to see if I can get this going.


[Audio]: Victory Sound Effect!

Check if this audio is already added to RetroStudio.

[X] The audio is not already in game.

Is this for the main-menu playlist?

[X] Add to main-menu playlist.

Is this song from an old Roblox era or a sound effect?

[X] This song is from old Roblox or is a sound effect.

A link to the audio on Roblox
https://www.roblox.com/library/9171226692/Victory
A link to the audio file to download.
https://files.fm/f/uqx3g93uv
Notes
This sound effect was used in old roblox games like deathrun.

your download link is set to private access :(


export(Enum) properties are missing the PROPERTY_USAGE_SCRIPT_VARIABLE flag

Godot version:

OS/device including version:
Windows 10 64
Issue description:
When you add exported properties, they get flagged with the SCRIPT_VARIABLE flag. This flag is missing when you export an enum, making such property impossible to distinguish with the built-in ones.
Steps to reproduce:
See the reproduction project below.
The test_int exported property gets correctly flagged with PROPERTY_USAGE_SCRIPT_VARIABLE - 8192 (+ 7 for PROPERTY_USAGE_DEFAULT):
{class_name:, hint:0, hint_string:, name:test_int, type:2, usage:8199}
The enum1 exported property does not:
{class_name:, hint:3, hint_string:Enum 1:0,Enum 2:1, name:enum1, type:2, usage:7}
This makes it impossible for instance to distinguish from a built-in mouse_filter property:
{class_name:, hint:3, hint_string:Stop,Pass,Ignore, name:mouse_filter, type:2, usage:7}
Minimal reproduction project:
https://gist.github.com/PLyczkowski/fc0ffcebcd22f8421798c1a510aa9ce5

Thanks!


local import fails in >= 0.0.16 but worked in 0.0.8

First, thanks for pytest-homeassistant-custom-component, which I adopted a couple of months ago for pyscript, which has quite a few tests.
I am currently using 0.0.8, and everything worked fine.  I just tried 0.0.20, but for reasons I don't understand my test files are unable to import from custom_components.pyscript.  For example, if I run this from the top-level a clone of the pyscript directory:
pytest tests/test_unit_eval.py

I get the error:
ImportError while importing test module '/Users/craig/git/pyscript/tests/test_unit_eval.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
../../.pyenv/versions/3.8.3/lib/python3.8/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/test_unit_eval.py:3: in <module>
    from custom_components.pyscript.const import CONF_ALLOW_ALL_IMPORTS, CONFIG_ENTRY, DOMAIN
E   ModuleNotFoundError: No module named 'custom_components.pyscript'

I'm doing something wrong, but can't readily figure it out.
I'm running python 3.8.3 with:
homeassistant-custom-component==0.0.20
pytest==6.1.2
homeassistant==0.118.2

I created a clean venv and got the same issue.
I also tried cloning pytest-homeassistant-custom-component with its own venv, and after installing its dependencies, I used that venv and it does succeed in the import, although it gets a different error:
E       fixture 'hass' not found
>       available fixtures: aiohttp_client, aiohttp_raw_server, aiohttp_server, aiohttp_unused_port, cache, capfd, capfdbinary, caplog, capsys, capsysbinary, cov, doctest_namespace, fast, loop, loop_debug, monkeypatch, no_cover, proactor_loop, pytestconfig, raw_test_server, record_property, record_testsuite_property, record_xml_attribute, recwarn, requests_mock, test_client, test_server, testrun_uid, tmp_path, tmp_path_factory, tmpdir, tmpdir_factory, unused_port, worker_id
>       use 'pytest --fixtures [testpath]' for help on them.

But after I install pytest-homeassistant-custom-component==0.0.20 in that 2nd working venv, the import fails.
Do you have some ideas I can try?

Have you tried adding a custom_components/__init__.py?
I haven't fully wrapped my head around pytest and why things get weird in this case, but see https://stackoverflow.com/questions/50174130/how-do-i-pytest-a-project-using-pep-420-namespace-packages
Im not sure why it would work in one version and boy the next though.

That did it!  Wow, thanks!

I originally had this disclaimer in the README but then somehow convinced myself that it was unneeded by testing it (probably in a version around 0.0.8).  The other question is how home assistant is able to get around this, I haven't figured that out yet either :)

It sounds like this is fixed.  I will close, but feel free to reopen if there are still problems


Update documentation to only utilize extensions that are natively part of ebpf-for-windows repo

Describe the feature you'd like supported
Once issue #2974 is completed, the XDP hook will be removed from this repo. We should update our relevant documentation/samples (such as dbeugging.md) to use a program type that is part of native ebpf-for-windows instead.
Proposed solution
Update the documentation to use an ebpf-for-windows 'native' program type (such as cgroup_sock, or even the sample_ext).
Additional context
No response

I'm going to close this task as it is no longer needed. I will instead make a note in issue #2974 to also update the documentation when xdp_test is removed.


Does this Library remove listeners automatically?

@kshivang
According to the Firebase docs
When you are no longer interested in listening to query realtime updates, you must detach your listener so that event callbacks stop. This allows the client to stop using bandwidth to receive updates.

So I'm curious how is it that CombineFirebase handles this as I can't find any code referencing the removal or detachment of these listeners. I'm concerned with bandwidth usage which can get expensive.
Is CombineFirebase doing this automatically somehow or are we responsible for it?

CombineFirebase is exactly for this. Purpose of combineFirebase is to give firebase listener api lifecycle of a publisher, which should be the default lifecycle for any api in new Combine Framework paradigm. Default lifecycle of a publisher is data is being listened after at-least one subscriber is subscribed, this is done via sink and assign api. Both sink and assign api in turn return Cancelable, which you can use to stop listener by using cancel() method of cancellable. Basically all the api of combineFirebase remove listeners on cancel method invocation. Hope that answer your concern.

@kshivang Ahh very informative piece. I always store in my cancelbag because compiler demands it, didn't care to understand it until now.
I saw cancel() method in your code but couldn't figure out why it wasnt called but I see it in the Cancellable protocol.
Thanks very educational explanation.


Rename function for jumping

QuicktexJump -> quicktex#DoJump

What's the purpose of renaming this function?
If you want to autoload the function, it needs to be under the autoload folder.

The purpose is to mark this function as "exported".
This way, when you write a  mapping for keyword dictionary, it's obvious where does this function come from.

By the way, doc mentions g:quicktex_always_tex variable, which is actually g:quicktex_always_latex

I like the idea. And thanks for letting me know about g:quicktex_always_tex. I'll fix the documentation soon.

Fixed. Thanks for your help!


Based on Zheng's commit 68cb0c6

The major steps in workflow
-- Verify that no deployment failure using latest generated image
-- Verify that IBM environment will be removed after deployment with untitled IBM account
-- Generate SAS urls for creating image offer
-- Send notification to Teams channel
Update mainTemplate.json
-- To adopt workflow changes, allow passed in Storage Account and Vm name for visibility
Update README
-- To describe steps of creating image offer using SAS urls generated by workflow and the post actions after it.
Based on https://github.com/WASdev/azure.websphere-traditional.image/pull/16/commits/68cb0c6d5c9a94740565b01dab218c42f70781be
new file:   .github/workflows/build.yml

Add comments to address @git4rk concerns.

Remove

userEmail: ${{ secrets.USER_EMAIL }}

make vmAdminId and vmAdminPassword come from secrets.

modified:   README.md
modified:   src/main/arm/mainTemplate.json
new file:   test/mainTemplate.json
Identical to 68cb0c6.
Signed-off-by: Ed Burns<EMAIL_ADDRESS>

This PR obsoletes Zheng's PR.

Hello @git4rk  as far as we know, this commit is ready to merge. Please give your final review and merge if desired.


How to Get Long Lived Access Token?

Creating an app which would allow users to make posts to their Google My Business listings, I've successfully ran some tests using: https://github.com/googleapis/google-api-php-client/blob/master/examples/idtoken.php
But I am seeing that the access token expires in 1 hour?

How do I get an access token that would have a longer life span?
LinkedIn doesn't create long-lived access tokens like Facebook, but LinkedIn tokens expire every 90 days, that's not a big bother on users since it's just basically reauthenticating their account with an app once every 3 months, but I can't imagine having to reauthenticate every hour being practical.
How would I go about getting an access token with a longer life span?

What you're looking for is a refresh token. Once you have a refresh token, you can use Google_Client::fetchAccessTokenWithRefreshToken() to get a new access token as needed.

@jdpedrie Thank you for the response. I just added $client->setAccessType("offline"); and I see i get a refresh token and can use it to get a new access token using the fetchAccessTokenWithRefreshToken() method.
Questions:

Does this refresh token expire?
So is it fine to simply always call this method whenever I'm about to do a call to the API? Basically(PHP):

$refresh_token = '...';
$access_token =  $client->fetchAccessTokenWithRefreshToken($refresh_token)['access_token'];
// Use this access_token (which would always be unique) for API call

They don’t have a scheduled expiration, but you should write your app with the expectation that refresh tokens will periodically become invalidated.
Rather than using the refresh token all the time, it would be much better to cache the token along with its expiration and only issue a refresh once the previous access token is expired.

@jdpedrie Another thing..I think i'm misunderstanding google's implementation here...
Think of the product I am working on as a piece of software that lets users create posts on their Google My Business listings.
The users authenticate on my website, through a my Google console project which has been approved to use the Google My Business API.
So:

User has piece of software on their own computer(or other independent environments that I don't have control of.)
On software dashboard, user clicks "Authenticate"
Software sends user to my website here: https://example.com/gmb_auth
On that path I have the GMB library redirecting the user to the google auth endpoint
Once the user has authenticated, google will send the user back to  https://example.com/gmb_auth because I have that URL set as the authorized redirect uri
Once  https://example.com/gmb_auth receives the response from google's auth, it will send the user's access token, refresh token etc. back to the software dashboard and the process is complete.

This is all fine...and would work just fine, except, the access token isn't long-lived, and even though I send back the refresh token to the user...I'd also need to send back my Client ID and Client Secret if I'm to use the fetchAccessTokenWithRefreshToken() method
On the distributed software, if I don't set the Client ID and Client Secret, I get the error:
Array ( [error] => invalid_request [error_description] => Could not determine client ID from request. )
My question is, how can I refresh access tokens without having to send the Client Secret to the application? (seems like a security flaw)
Or how else can I go about this so the user can create posts on their GMB locations without authenticating every hour?

If I'm understanding correctly, couldn't your application make a call to, say, example.com/refresh_token with the client's refresh token? Then example.com could make the auth call with your ID and secret, get an access token, and pass it back to the app.
It's difficult to answer in my position since I don't have enough information and I'm wary of giving advice which may not lead in a totally secure direction. I definitely advise seeing more specific help from someone who has a good understanding of OAuth and who is in a position to more fully understand your use case.

@jdpedrie Thanks again for your response. Your workflow makes much more sense than sending the client secret back to the distributed app.
It just seems like having to do this hourly doesn't seem ideal. Is there no other way with Google to receive longer-lived access tokens for such distributed applications?
If not then I'll have to go with your workflow

do make
сб, 20 июн. 2020 г. в 20:15, Uriahs Victor<EMAIL_ADDRESS>
@jdpedrie https://github.com/jdpedrie Thanks again for your response.
Your workflow makes much more sense than sending the client secret back to
the distributed app.
It just seems like having to do this hourly doesn't seem ideal. Is there
no other way with Google to receive longer-lived access tokens for such
distributed applications?
If not then I'll have to go with your workflow
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googleapis/google-api-php-client/issues/1853#issuecomment-647001196,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APNECNOBVALR3JZGO5N5OW3RXS765ANCNFSM4OCFW7GA
.

@UVLabs To the best of my knowledge, you cannot extend the lifetime of an access token, only renew credentials using the refresh token.

Thank you @jdpedrie

create folder how active directory as services and as registry
пн, 22 июн. 2020 г. в 19:23, John Pedrie<EMAIL_ADDRESS>
@UVLabs https://github.com/UVLabs To the best of my knowledge, you
cannot extend the lifetime of an access token, only renew credentials using
the refresh token.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/googleapis/google-api-php-client/issues/1853#issuecomment-647516175,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APNECNMSWX22ERS7H27WDBLRX5LLLANCNFSM4OCFW7GA
.

Creating an app which would allow users to make posts to their Google My Business listings, I've successfully ran some tests using: https://github.com/googleapis/google-api-php-client/blob/master/examples/idtoken.php
But I am seeing that the access token expires in 1 hour?

How do I get an access token that would have a longer life span?
LinkedIn doesn't create long-lived access tokens like Facebook, but LinkedIn tokens expire every 90 days, that's not a big bother on users since it's just basically reauthenticating their account with an app once every 3 months, but I can't imagine having to reauthenticate every hour being practical.
How would I go about getting an access token with a longer life span?

It's not practical, it is an absolute joke navigating Google's apis.


feat(api): partial schema insertion

Problem Statement
As development teams regularly roll out new API endpoints, features each addition often necessitates corresponding updates to the schema. The current procedure, which requires comprehensive schema re-uploads for any modification, has proven to be inefficient and not conducive to rapid development cycles. There is an essential need for a more refined update mechanism that supports iterative and incremental schema modifications, allowing teams to make precise changes with minimal disruption.
Request Payload Structure
PATCH /v1/{tenant_id}/schemas/partial-write
{
  "metadata": {
    "schema_version": ""
  },
  "entities": {
    "entity-name": {
        "write": [],
        "delete": [],
        "update": [],
    }
}

Behavior Description
When the schema_version in the request metadata is left empty, the system will default to using the latest(head) schema version as the base for updates.

name (string): The name of the entity to be changed.
write (string array): Conditions to be added. If a relation or permission/action already exists, it should return an error.
delete (string array): Names (permissions/actions) to be deleted. If the relation/permission/action name does not exist, it should return an error. Note: specifying the name is enough as relation/permission/action names should be unique.
update (string array): Conditions to be updated.

If schema_version is specified, the endpoint will perform the same update process on the given version and generate a new version thereafter.

Any update?
I think usecase update schema is very important in real world.

Hi @lekhanhtoan37, We are working on this on priority. We realised that this approach was not the best and causing quite a few anti patterns hence its taking some time. Should be done by next week hopefully.


Port package to ROS iron

Hey,
This PR ports this package to ROS iron because the function g2o::make_unique was removed. The changes should be backwards compatible

Thank you for the PR! I've merged it.


Fix JSON tag for Name field

What this PR does / why we need it:
Adds a JSON tag to the Name field.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #438
Release note:
Add JSON tag to the Name field

/assign @kron4eg

/cherrypick release/v0.6

/retest


Can you post more of the output?  There should be additional details that correspond with Failed: @medplum/core#build

Yes i run npm ci first
*after npm run build:fast on medplum root

after npm run dev on packages/server
PS C:\Users\user1\medplum\packages\server> npm run dev

<EMAIL_ADDRESS>dev
ts-node-dev --poll --respawn --transpile-only src/index.ts

[INFO] 08:49:55 ts-node-dev ver. 2.0.0 (using ts-node ver. 10.9.1, typescript ver. 5.3.2)
{"level":"INFO","timestamp":"2024-01-03T06:49:59.688Z","msg":"Starting Medplum Server...","configName":"file:medplum.config.json"}
{"level":"INFO","timestamp":"2024-01-03T06:50:00.332Z","msg":"Already seeded"}
{"level":"INFO","timestamp":"2024-01-03T06:50:00.342Z","msg":"Loaded 1 key(s) from the database"}
{"level":"INFO","timestamp":"2024-01-03T06:50:00.350Z","msg":"Server started","port":8103}
after npm run dev on packages/app

Hi @leah-committed - Our Discord Forum is a better place to get help with stuff like these kinds of issues. I'm closing this ticket, but feel free to ask more questions in our support forum!

Hi, I still haven't received a reply. If possible please open again.

just update your node.js to v20
https://discord.com/channels/905144809105260605/1271069319349932133/1271096517976916090


If TURN is activated with ports 10000-11000, how to set JVB_PORT in .env?

Hi!
I'd really like to try your TURN server modification. When I configure .env as stated in the example, I get an error when the videobridge is startet, that port 10000 is already in use. I understand that if I use ports 10000-11000 for TURN that setting JVB_PORT to 10000 is not possible.
But what should I do? Set JVB_PORT to 9999 or TURN port range to 10001-11000?
Hope you can help?
Many thanks and keep up the good work!
Regards,
Holger

Hi Holger,
You are free to use any ports what you want ) Just remember that ports have not be crossed. And, of cource, it must be allowed on your in-firewall.

From: Holger Pandel<EMAIL_ADDRESS>Sent: Friday, March 20, 2020 3:52 PM
To: netaskd/docker-jitsi-meet
Cc: Subscribed
Subject: [netaskd/docker-jitsi-meet] If turn activated with ports 10000-11000, how to set JVB_PORT in .env (#2)
Hi!
I'd really like to try your TURN server modification. When I configure .env as stated in the example, I get an error when the videobridge is startet, that port 10000 is already in use. I understand that if I use ports 10000-11000 for TURN that setting JVB_PORT to 10000 is not possible.
But what should I do? Set JVB_PORT to 9999 or TURN port range to 10001-11000?
Hope you can help?
Many thanks and keep up the good work!
Regards,
Holger
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi!
Thanks. I know that ports should not cross :-) - that's why I asked, because your env.example does not take that into account and I wasn't sure, i there is any special setting I overlooked.
I'll give it a try.
Regards
Holger

I've changed an example in aa52b36


fix(DCOS-16363): Content in table cells overflowing

This PR wraps content in table cells.
Before:
https://cl.ly/3Z0V3v260Q31
After:
https://cl.ly/3d2h0h2r3X3n
Checklist

[ ] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

Thoughts @leemunroe?

Spoke offline with @tnguyen0 and in the case of the permissions I agree it makes sense to wrap.
For other instances I would hold off until we learn if is an issue or not e.g. for IDs truncation works


Bug/jenkins 42059 dropdown wrong position

Description
Fix an issue where scroll offsets of dropdown containers weren't all being taken into account.

See JENKINS-42059.

Submitter checklist

[X] Link to JIRA ticket in description, if appropriate.
[X] Change is code complete and matches issue description
[ ] Appropriate unit or acceptance tests or explanation to why this change has no tests

Reviewer checklist

[ ] Run the changes and verified the change matches the issue description
[ ] Reviewed the code
[ ] Verified that the appropriate tests have been written or valid explanation given

@reviewbybees

:bee: nice one

Nice 👍


allow port number to be configured by DNS

@DavesPlanet Please do not use a string for the port number.

@yankejustin per the commit message I have changed the port from being a pure number to also being configurable via DNS. A DNS address is a string. You can still put a number in it if you like.

There was no message in neither the description region of this pull request nor the commit description. They are both blank.

My point is that there are many better ways to using a string and, if configurable DNS is the intention, knowingly and willingly cause an exception in order for it to work correctly.

My first git push, I'll do better next time, I should have filled out more info than just the commit message but the commit message is "allow port number to be configured by DNS ". Thanks for the feedback on the int.Parse, good catch.

Hmmm, I made a new commit with completely new features and it seems like github merged my new commit into the pending pull request because I was just committing to master. Not exactly what I was after, I guess in the future I should use a branch for each pull request. The commits were both related to improving the ability to recover from loss of a service. The port number lookup is critical in that xRat requires a VPN to support either remote port forwarding or a static IP. If you switch to a new VPN then the remote port you were using would certainly not be available. The second commit increases the number of host names and ports from 1 to 3. 90% of the time it tries to connect to the primary, 9% of the time it tries the secondary, and the remaining 1% of the time it will try the tertiary host and port. Got one more pull request queued up for this evening and I will be sure to make a "feature branch" for that pull before I commit.

Yes. Read up on exactly how to use it.  x)
You can look at my network graph.
Here is a good reference.

As for the first part of that paragraph: Github automagically will add any commits made on a branch that is currently is in an open pull request to the pull request.
As for the other part, if you use Github for Windows, it is significantly easier to add comments to each of your commits. ;)


^^ MORGAN roadmap ^^

Here is a roadmap for the second named release:
features:

[ ] edit album page
[ ] view album page
[ ] delayed albums search
[ ] genre subscriptions
[ ] ability to create a personal albums collection
[ ] bring back quality indicators (?)
[ ] display a playing track in a window title

gears:

[ ] delete album.artist field from instance schema, it is a derived data that can be calculated from album.tracks[].artist fields
[ ] add files to IPFS without copying
[ ] publish multiple albums at once as array

rest:

[ ] test on a large amount of albums added (10GB+)
[ ] investigate on a distribution via package managers (dnf / apt / aur)

The roadmap is subject to change, therefore feel free to share your ideas below!

Maybe adding a repo for ubuntu & debian would be an idea. ref https://github.com/Siderus/Orion/pull/154

These aren't tasks, they are full-blown issues. I suggest you create a separate issue for every one of them so we can see what's planned before creating an issue!
You can also group them in the milestone so we can see the progress of that roadmap shard.

@NatoBoram seems reasonable.


Address Issue #229

Changed database name from scratch to schema.
I have looked at the published documentation and there is no reference
to the database name

While I'm testing this tool - I can verify this

@adgiles - I guess this PR needs to be redone (or updated from dev) because of conflicts from previous PR ( #244 ) - might have been better to combine these GDB changes into one PR

you can probably ignore this one and just merge the other, this had the same changes in it


Default AUTH_MECHANISM

Sets default AUTH_MECHANISM values when using non URI configuration method.
Without this, trying to configure pymongo using a configuration prefix causes a KeyError
(Now with added tests)

I don't quite follow why this didn't work with a custom prefix, do you understand that?

If you specify any auth params(user/pass) it tries to get the
auth_mechanism from the app config, but if you haven't set it then you get
a key error as there is no default.
I think it's also the case when you don't have a custom prefix, haven't
actually tested that case.
On Mon, 22 May 2017 at 22:27, Dan Crosta<EMAIL_ADDRESS>wrote:

I don't quite follow why this didn't work with a custom prefix, do you
understand that?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dcrosta/flask-pymongo/pull/93#issuecomment-303224796,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAILOT5piq7YpWDFmjMkJgfPiqQdjTgzks5r8f25gaJpZM4NilT8
.

If you specify any auth params(user/pass) it tries to get the
auth_mechanism from the app config, but if you haven't set it then you get
a key error as there is no default.
It's also the case when you don't use a custom prefix.
Error
Traceback (most recent call last):
  File "/usr/lib/python2.7/unittest/case.py", line 329, in run
    testMethod()
  File "/home/mark/development/flask-pymongo/tests/test_config.py", line 171, in test_missing_auth_mechanism_in_nonprefixed_config
    mongo = flask_pymongo.PyMongo(self.app)
  File "/home/mark/development/flask-pymongo/flask_pymongo/__init__.py", line 97, in __init__
    self.init_app(app, config_prefix)
  File "/home/mark/development/flask-pymongo/flask_pymongo/__init__.py", line 279, in init_app
    auth_mechanism = app.config[key('AUTH_MECHANISM')]
KeyError: 'MONGO_AUTH_MECHANISM'

Thanks for the fix here!


Bug with useGlobal hook

Describe the bug
useGlobal hook behaves unexpectedly when you trying to use it in derived hook function. The initial state turns out fine, but updates is not appearing, until component is reset to get initial state again.
To Reproduce
Steps to reproduce the behavior:

Write some global state picker like:

useGlobal(global => global);

Bring it to derived hook like:

const useGlobalState = () => useGlobal(global => global);

Use your derived hook in functional component like:

const ExampleComponent = () => {
  const globalState = useGlobalState();
  return <pre>{globalState}</pre>;
};

See the bug

Expected behavior
Expected behavior to be like a first pick of initial state.
Screenshots
For example, play button is diabled because host connection is broken (according to current hook state). But actually connection has already been established a long time ago.

Fixed


Autocomplete does not always intelligently suggest expected items

From @matej-zajacik on April 18, 2017 8:35

VSCode Version: Code 1.10.1 (653f8733dd5a5c43d66d7168b4701f94d72b62e5, 2017-03-02T00:33:15.706Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

toml
be5invis
0.0.2

dart-code
DanTup
1.0.2

csharp
ms-vscode
1.3.0

Autocomplete does not show up when an enumerator is expected as a function parameter, or when you type the "new" keyword after declaring a class member.
Steps to repro:

Just write the following code and observe what happens when you reach the "..." mark.
 public class SomeClass
 {
 	private Dictionary<int, int> myDict = new ... // <- In Visual Studio, the autocomplete suggests "Dictionary<int, int>()". In Code, nothing shows up; you have to type it again.

 	public enum SomeType
 	{
 		Type1,
 		Type2
 	}

 	public void SetType(SomeType type)
 	{
 	}

 	public void Start()
 	{
 		SetType(... // <- In Visual Studio, the autocomplete options show up with the SomeType enum. In Code, nothing shows up.
 	}
 }

Expected:

Similar behavior as in Visual Studio 2015/2017. These issues are among the few Code lacks from VS.

Copied from original issue: Microsoft/vscode#24939

From the description of the bug, I believe this is related to the completion providers that the C# extension registers. Please let me know if you believe the issue is with VSCode itself

This is a duplicate of https://github.com/OmniSharp/omnisharp-vscode/issues/146.


Overflow in pxy2zxy

By running carmen-cache with -fsanitze=integer I see we only have one test that hits the if (zDist == 0) condition at https://github.com/mapbox/carmen-cache/blob/cd20c47ed2bd1ecc8081b22e14ac85e8ea22c47a/src/binding.cpp#L934.
It is the test/geocode-unit.proximity.test.js (specifically # forward province - multilayer) in carmen as of https://github.com/mapbox/carmen/commit/dbe167d609a6cc5374a20ddd7accb81ba62edf34.
Before the if (zDist == 0) condition the zMult variable overflows. This is harmless because it is not used, but it still ends up being<PHONE_NUMBER> because target_z and z are both 6 and so zDist because 0 at https://github.com/mapbox/carmen-cache/blob/cd20c47ed2bd1ecc8081b22e14ac85e8ea22c47a/src/binding.cpp#L932
Tasks:

[ ] Ensure we have test coverage in carmen-cache directly that triggers this condition (to protect from a regression if the if (zDist == 0) condition were every accidentally removed
[ ] Fix the harmless overflow

Proposed fix:
diff --git a/src/binding.cpp b/src/binding.cpp
index b6bb856..a424111b 100644
--- a/src/binding.cpp
+++ b/src/binding.cpp
@@ -903,12 +903,12 @@ ZXY pxy2zxy(unsigned z, unsigned x, unsigned y, unsigned target_z) {

     // Interval between parent and target zoom level
     unsigned zDist = target_z - z;
-    unsigned zMult = zDist - 1;
     if (zDist == 0) {
         zxy.x = x;
         zxy.y = y;
         return zxy;
     }
+    unsigned zMult = zDist - 1;

     // Midpoint length @ z for a tile at parent zoom level
     unsigned pMid_d = static_cast<unsigned>(std::pow(2,zDist) / 2);

Just tested again and I'm also seeing a similar potential overflow in bxy2zxy (https://github.com/mapbox/carmen-cache/blob/64af63ce274bb518951ee2151af4efaa1d1fb43e/src/binding.cpp#L956)
# coalesceMulti bbox
../src/binding.cpp:956:29: runtime error: unsigned integer overflow: 1 - 2 cannot be represented in type 'unsigned int'
SUMMARY: AddressSanitizer: undefined-behavior ../src/binding.cpp:956:29 in

Note: these problems are still present. They are minor, but should ideally still be fixed. They are not seen in the carmen-cache tests because of lacking coverage. They only show up when carmen-cache is instrumented with the address sanitizer and the carmen tests are run with this version of carmen-cache.
I don't have the bandwidth to keep pushing here, so I will lay out the remaining todos and offer to spirit guide someone who picks this up:

[ ] merge https://github.com/mapbox/carmen-cache/pull/95 will starts publishing sanitized binaries for linux
[ ] optionally: test those binaries downstream in carmen (npm install --toolset=asan will install them)
[ ] See if the overflow errors still exits
[ ] Add unit tests in carmen-cache that catch them, then the tests should start failing in carmen-cache
[ ] Fix the overflows

presume this was fixed by #116


Support the datetime type from MSSQL Server 2008

I was using this library to work with an old MSSQL server for a few days, and I got the following error: not implemented makeDecl for type 61.
It looked like https://github.com/denisenkom/go-mssqldb/issues/279
I no longer have a Microsoft Server I can work with, but it might be related to this specific version of MSSQL (MSSQL Server 2008 R2).

This is already covered by #303.


lib: Update login instructions for PATs, not OAuth

Please be sure to read the contributor's guide to the GitHub package before submitting any pull requests.
Requirements (from template, click to expand):

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.

Description of the Change

Update login instructions to direct users to generate a Personal Access Token via github.com/settings/tokens, instead of via the unavailable github.atom.io/login (OAuth token generator) page.
Also: Add written directions about which scopes are needed, as users will now have to manually select the scopes they need on their own tokens in order for the github package to function right (and not reject the token for having insufficient permissions/scopes).
(The OAuth flow via github.atom.io/login used to specify the scopes without user input or control, other than the choice to grant or not grant access. Now we must advise users of which scopes are needed.)
Screenshot or Gif

Before (click to expand):

After (click to expand):

Applicable Issues

Fixes #7
See also: (essentially fixes): https://github.com/atom/github/issues/2686
NOTE: I included the workflow scope in these new instructions. The old github.atomio/login OAuth token did not have this scope. The workflow scope was separated out from the general repo scope after maintenance of the official Atom github package had already diminished, so it was never fixed there. Setting the workflow scope on a PAT and using that instead of the OAuth token was the very same best workaround as was available for some years, so I rolled that into this PR. (See again issue https://github.com/atom/github/issues/2686).
workflow scope is not needed unless you need to push updated GitHub Actions YAML files up to your remote, however.
So these instructions go beyond bare minimum permissions/scopes needed to get up and running and do things :tm: with the github package, but it's just really confusing when [only] pushes that touch your GitHub Actions workflow YML files fail, potentially with a confusing or inaccurate error message as to why. And explaining it is harder than telling folks to set the permission.
So rather than delve into these fine details that are hard to explain and which people may not wish to think hard about when they don't have to... I just instructed folks to set the workflow scope whether they need to or not.
Much like the package could easily be changed to only require public_repo scope for repo access instead of the catch-all repo scope.
TO-DOs
Maybe explain when/why the workflow scope is needed in these instructions. (I dunno, man.)
Maybe update the app to require public_repo rather than full-on repo in terms of repo access scopes, and explain when/why you need full-on repo (for private repo access, basically).
NOTE: I consider both of these OUT OF SCOPE for the current PR. I respectfully ask that reviewers not ask that these be added to this PR, in order to avoid "scope creep" (pun not intended, but I'll take it) of this PR, review complexity, haggling over finer details, please and thank you. This is the targeted fix aimed at enabling folks to get authed in the github package without knowing ahead of time these precise instructions, or having to find them on the Discord somewhere, etc. etc.

A quick reminder that the testing on this repo is not testing the code, it is rather b0rked.

So this is flying blind in terms of CI, though you can attempt to run the tests locally I think the generous timeouts for the CI environment may come in handy. I forget how to activate those, though... Hm.
Testing Notes
Anyhow, IMO the best way to verify these changes is first off to see that they are valid HTML... and beyond that, I think manual testing is pretty appropriate here. Clone this PR branch, cd into it and do ppm install (wait for it to finish), then ppm link, make sure you find and rename or delete your existing saved github package token (see this comment https://github.com/atom/github/issues/2686#issuecomment-851136891 for how to view/edit tokens on each OS) so it shows the login screen, reload (or quit/relaunch) Pulsar, verify that the package doesn't crash when you try to click through to the login screen, an lastly verify that the login screen matches my "After" screenshot from above).

Okay, I got two approves (a UI one and a verbal one), and after I got around to reviewing the feedback (sorry for the delay!), it seems this is in a good place to merge.
Merging now!
Thank you for reviews/feedback!

Now to actually bump the github version in Pulsar


Release v0.17.2

(PR #309, 2022-03-31) fix(rut): AttributeError for GeneralName object without type_id

@fpinto-cdd don't forget to explicitly push the tags:
git push --tags

I think it is already pushed here https://github.com/fyntex/lib-cl-sii-python/tags

Codecov Report

Merging #310 (4382182) into master (3b747a1) will not change coverage.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##           master     #310   +/-   ##
=======================================
  Coverage   84.53%   84.53%
=======================================
  Files          34       34
  Lines        2573     2573
  Branches      349      349
=======================================
  Hits         2175     2175
- Misses        258      259    +1
+ Partials      140      139    -1

Impacted Files
Coverage Δ

cl_sii/rut/crypto_utils.py
80.00% <0.00%> (ø)

cl_sii/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


PYTHON-2613 Build self-contained manylinux{2010, 2014} wheels

While this is working now, it raises a few open questions and things to investigate:

Why could g++ not handle the extra whitespace returned by pkg-config --libs-only-l /path/to/libbson-1.0.pc? Does this point to an underlying problem?
Should we really be exporting the output of pkg-config --libs /path/to/libsson-1.0.pc to the LDFLAGS environment variable? As per GNU Make docs LDFLAGS should only contain what we intend to pass using -L to the linker. Library names themselves should be exported as LDLIBS.
Should we also be exporting the CXXFLAGS environment variable? As per GNU Make docs this is the variable used to pass extra args to the C++ compiler not CFLAGS.

Why could g++ not handle the extra whitespace returned by pkg-config --libs-only-l /path/to/libbson-1.0.pc? Does this point to an underlying problem?

The pkg-config output contained plain whitespace as confirmed by the following test:
>>>  lnames = query_pkgconfig("pkg-config --libs-only-l libbson/lib/libbson-1.0.pc")
>>> for k in enumerate(lnames):
...     print(k)
...
(0, '-')
(1, 'l')
(2, 'b')
(3, 's')
(4, 'o')
(5, 'n')
(6, '-')
(7, '1')
(8, '.')
(9, '0')
(10, ' ')
(11, ' ')
>>>

This seems to be a bug in x86_64-linux-gnu-g++ but I am not. certain.


Handling aliased modules (via Babel plugins)

Hi, my project has quite a few modules imported as aliases, which we're handling via babel-plugin-module-resolver.  Specifically, we're using it to simplify otherwise deeply nested import paths.
Unfortunately, I'm not quite clear on how to get such dependencies to show up in Madge.
Here's a simplified example:
Given this folder structure:
-src
  -components
    -component-a
    -component-b
      -stuff
        -more-stuff
          -my-view.js

You might see the following in src/components/component-b/stuff/more-stuff/my-view.js:
import componentA from 'components/component-a'

instead of
import componentA from `../../../component-a`

When I change the componentA import to use a relative path (like the 2nd part of the example), that dependency shows up just fine in Madge.  The first approach, however, gets ignored entirely.
Can anyone suggest a way to correctly apply the babel plugin to get Madge to use the proper import/require paths?  I suspect it will involve the requireConfig setting, but I haven't been able to find the secret sauce just yet, since adding even an empty requireConfig significantly changes the tree generated by Madge.

Hi @jrnail23. Thanks for contributing!
If you try setting the baseUrl of the require config to src/, that should resolve components/component-a.
Can you provide the command and configuration that you're using to invoke Madge? Once you provide that, I can give you more detailed instructions.

@pahen Can we close this until we get more info?

Yes, I agree. I'll close it.


Revert "UI/UX Improvements in visuals and dev cycle"

Reverts luxonis/depthai#95
This does not work with -bb on depth_sipp:
python3 test.py -s metaout previewout depth_sipp,5 -bb

So just reverting for now until we get it compatible there.

Here's after the revert.  Going to just merge this real quick as we are needing this overlay for a demo.

Huh- why are we defining the bounding box differently when using depth_sipp? Should've tested, my bad.

You’re good man!  I knew about this feature and totally forgot to mention it test it etc. even though I fully knew about it.
So the why is with pulling depth to calculate xyz information of the object there’s this variable padding_factor which lets you decide what portion of the object detector bounding box is used to calculate object depth.  And it’s useful to display that inner box so you can see if it’s working correctly for your object of interest.
I totally forgot about this when discussing the changes to the UI.  My bad.


Docs for value formatting

The docs for value formatting do not contain any of the value formats, making it difficult to see which are supported if you are not currently running Evidence (eg you are exploring)

It would be good if we could autogenerate these docs from the current list of value tags somehow so it stays up to date.

On researching this issue, it's pretty tricky using docusaurus. They don't have simple support for scripting.
Think it's a copy / paste job for now, and then can address making it easier when we move over to Evidence for docs

Closed by #696, see https://docs.evidence.dev/core-concepts/formatting/


Dump individual recipe contents

Opening regarding @vmiheer's comment in #701:

On other hand, is there way to keep the generated shebang script for later inspection? I see a file created in /tmp/justJOLujt/maestro but it gets deleted when the just execution stops. Maybe printing the path at the beginning so that one can put sleep 100 to inspect the content would be useful.

This is definitely possible, but an easier way of doing it might just be to add the ability to just print the generated shebang script to stdout. Would that work?

Oh yeah, I was just thinking of something really simple. 😊
I just tried -v.
It says:
just -v maestro
===> Running recipe `maestro`...

Might be great if it prints the filename here 😊. If dumping is going to take a lot of time to implement.
Also kudos for always prompt replies. 👏

I just remembered that if you use the --verbose flag twice, it will print shebang recipes to stdout, so try just -vv maestro. That would let you see the contents of the shebang recipe, although wouldn't let you actually get the file, unless you redirected stderr to a file.
Also, one thing you might notice is that shebang recipes have a ton of  blank lines. This is actually intentional, so that if there's an error in a shebang recipe that is caught at runtime by the interpreter, the line numbers in the error messages will correspond to the line number in the Justfile.

Might be great if it prints the filename here 😊. If dumping is going to take a lot of time to implement.

I think the easiest thing to do would be to add a flag, maybe --keep-tempfiles, which prevents just from deleting tempfiles. Then, change it so that using the --verbose flag twice prints the path to shebang scripts. You could use both of these to both get the path of scripts and keep them around to inspect them.

Also kudos for always prompt replies. 👏

Thanks you, it's my pleasure!

@casey  Would it make sense to overload the --dump flag for this? I.e. just --dump with no additional arguments has the current behavior, but just --dump recipe_name would print just the text of that recipe?

@casey Would it make sense to overload the --dump flag for this? I.e. just --dump with no additional arguments has the current behavior, but just --dump recipe_name would print just the text of that recipe?

I like this idea a lot. You could do just --dump > script to extract a recipe to a script. (Modulo the fact that {{…}} interpolations wouldn't be valid syntax.)


Outdated installation instructions.

The installation instructions are outdated:

[ ] 2x failed job tables end up being created leading to errors.
[ ] The foreign keys are causing issues (perhaps just on MySQL, needs research).
[ ] Forgot to mention to create the JWT key.

for the foreign key issues, change bigInteger to unsignedBigInteger on course_id and level_id inside migrations that set foreign references
        Schema::create('words', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->string('source');
            $table->string('target');
            $table->unsignedBigInteger('level_id');
            $table->unsignedBigInteger('course_id');
            $table->jsonb('data');
            $table->timestamps();
        });

        Schema::table('words', function (Blueprint $table) {
            $table->foreign('level_id')->references('id')->on('levels');
            $table->foreign('course_id')->references('id')->on('courses');
        });

        Schema::create('levels', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->unsignedBigInteger('course_id');
            $table->string('title');
            $table->timestamps();
        });

        Schema::table('levels', function (Blueprint $table) {
            $table
                ->foreign('course_id')
                ->references('id')->on('courses')
                ->onDelete('cascade');
        });

And you also need to add ->nullable() to these two:
            $table->timestamp('started_at')->nullable();
            $table->timestamp('finished_at')->nullable();

Then it migrates and seeds fine on mysql


Feature Request: Add arbitrary  attributes to ng-repeat.

I'd like to add arbitrary attributes to ng-repeat elements based on scope.  The attributes are injected into the elements created by ng-repeat without requiring a changes to existing templates.
I'm assuming this can be easily achieved injecting a property into scope the compiler recognizes.

It is not clear what the exact usecase is (or feature) is. From what I can tell, the idea to basically be able to magically modify a template, by assigning some properties on the scope.
This sounds like a bad idea. Manipulating the DOM from a controller is considered an anti-pattern.
If you want to change the template, you could use a custom directive wrapped around the ngRepeat - I can't be more specific as I don't fully understand your usecase and requirements.
--
I'm going to close this, because in its current state it is not something we would add to the core.
In case I misunderstood the nature of the requested feature, please open a new issue with a concise explanation including how this feature would be used (e.g. a code snippet using the proposed syntax) and what its behavior would be (e.g. showing the resulting HTML).
Thx


Inner class builder inheritance

Hi, you should consider the possibility to make the inner class builder implement an interface. In my specific case I need the builder to implement an interface with generics. This could be defined in configurations.
My proposal is to add two properties:
builderImplements: `fully.qualified.java.type` // could be also a generic type ex. `fully.qualified.java.type<T>`
builderExtends: `fully.qualified.java.type` // same as above

Obviously when one of these has a value, the builder generated will implement or extend the type specified.

Hi. Thanks for the suggestion.
Sorry, but it's not the goal of this plugin to produce any possible desired Java output. The problem is that every time we add features like this, they require support and maintenance forever. Everyone has some change they would like to make to Java output because they have some idea about extending the code to fit some other requirements in their system. We simply can't meet all these demands.
Of course, you're free to fork and extend this project to your own needs.