buildah-commit: return value is not image ID of image created

Description
From the documentation for buildah commit...:

RETURN VALUE
The image ID of the image that was created. On error, 1 is returned and errno is returned.

Steps to reproduce the issue:

OCI_ID=$(${BUILDAH} commit ${OCI_NAME} ${OCI_NAME})
echo $OCI_ID

Describe the results you received:

Describe the results you expected:
<some-signature or 1>

Output of apt list buildah:
buildah/xenial 0.16-1~ubuntu16.04.2~ppa4 amd64

Output of buildah version:
Version:       0.16
Go Version:    go1.8.7
Image Spec:    1.0.0
Runtime Spec:  1.0.0
Git Commit:    f5a76e0
Built:         Fri Apr  6 15:33:05 2018
OS/Arch:       linux/amd64

Output of cat /etc/*release:
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.04
DISTRIB_CODENAME=xenial
DISTRIB_DESCRIPTION="Ubuntu 16.04.4 LTS"
NAME="Ubuntu"
VERSION="16.04.4 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.4 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

Output of uname -a:
Linux <redacted> 4.4.0-119-generic #143-Ubuntu SMP Mon Apr 2 16:08:24 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Output of cat /etc/containers/storage.conf:
# storage.conf is the configuration file for all tools
# that share the containers/storage libraries
# See man 5 containers-storage.conf for more information

# The "container storage" table contains all of the server options.
[storage]

# Default Storage Driver
driver = "overlay"

# Temporary storage location
runroot = "/var/run/containers/storage"

# Primary read-write location of container storage
graphroot = "/var/lib/containers/storage"

[storage.options]
# AdditionalImageStores is used to pass paths to additional read-only image stores
# Must be comma separated list.
additionalimagestores = [
]

# Size is used to set a maximum size of the container image.  Only supported by
# certain container storage drivers (currently overlay, zfs, vfs, btrfs)
size = ""

# OverrideKernelCheck tells the driver to ignore kernel checks based on kernel version
override_kernel_check = "true"

@taqtiqa-mark I think (hope) this was fixed very recently with #566 which was merged just a few days ago.  If you get a chance can you get the absolute latest/greatest and retest please?

Thanks, it is fixed in the latest.


Generate SQL queries with comments for find operations

Description

Closes #3573.
Readiness checklist

[x] I added/updated unit tests (and they pass).
[x] I added/updated integration/compatibility tests (and they pass).
[x] I added/updated comments and checked rendering.
[ ] I made spot refactorings.
[ ] I updated user documentation.
[x] I ran task all, and it passed.
[x] I ensured that PR title is good enough for the changelog.
[ ] (for maintainers only) I set Reviewers (@FerretDB/core), Milestone (Next), Labels, Project and project's Sprint fields.
[ ] I marked all done items in this checklist.

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

@chumaumenze please leave a comment on #3573 so we could assign the issue

@chumaumenze https://github.com/FerretDB/FerretDB/blob/1f18df076dc2c7bc804e9358d3b933ad2f9992ec/CONTRIBUTING.md?plain=1#L365
It makes our review process harder


[off/2] [discussion] Using library without access to site with iframe hosting

Info from manual:

The first (iframeResizer.min.js) is for the page hosting the iFrames. It can be called with native JavaScript;
The second file (iframeResizer.contentWindow.min.js) is a native JavaScript file that needs placing in the page contained within your iFrame. This file is designed to be a guest on someone else's system, so has no dependencies and won't do anything until it's activated by a message from the containing page

Is there possible to use library in this case:

<form enctype="application/x-www-form-urlencoded" method="post" action="..">

You would have to detect that the iFrame contents have loaded, inject the script into the iFramed page. Detect when it has finished loading and then call iFrameResize on the parent page.

Also you would need to be able to access copIFrame.contentWindow

Also you would need to be able to access copIFrame.contentWindow

Here I am getting problems with
var getIFrameDocument = function getIFrameDocument(iframe) {
    // SecurityError: Permission denied to access property "document" on cross-origin object
    var doc = iframe.contentDocument ? iframe.contentDocument : (iframe.contentWindow ?  iframe.contentWindow.document : iframe.document);

     return doc;
}

var x = getIFrameDocument(copIFrame);

You have to get the other site to give you permission using CORS.


Gatsby + Autoplay

I'm unable to get the Autoplay feature working in Gatsby. Looking through the issues, it looks like I have to include it as a module, but import { Autoplay } from "swiper/dist/js/swiper.esm"; throws an error. My current params are below, which is based on the current docs:
 const sliderParams = {
    direction: 'vertical',
    slidesPerView: 1,
    loop: true,
    centeredSlides: true,
    autoHeight: true,
    autoplay: {
      delay: 2500,
      disableOnInteraction: false
    },
  }

Also having an issue here. Nothing I do seems to get the autoplay feature to work.

@mister-cairns
Not sure if you are still having this issue but doing this worked for me.
import { Autoplay, Swiper as RealSwiper } from 'swiper';
Then inside the component call:
RealSwiper.use(Autoplay);
Hope this helps.

try to use this https://swiperjs.com/react
import SwiperCore, { Autoplay } from "swiper/core"
SwiperCore.use([Autoplay])


⚠ RPC server - Figment Solana has degraded performance

In c2a2b6c, RPC server - Figment Solana ($FIGMENT_SOLANA_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 102 ms

Resolved: RPC server - Figment Solana performance has improved in 1627cfc.


Enable ansi colors for tests results in github action console

This is already implemented, now it has to be styled better


Add recursion support to the def get_files_by_extension, change tests

Add support to the --no-recursion argument.
Which version of puremagic is used? (requirements.txt)


Import the almond-spark dependency

When I tried;
import $ivy.org.apache.spark::spark-sql:2.4.0
import $ivy.sh.almond::almond-spark:@VERSION@
I got the following errors:
Failed to resolve ivy dependencies:Error downloading sh.almond:almond-spark_2.12:@VERSION@   not found: /Users/.ivy2/local/sh.almond/almond-spark_2.12/<EMAIL_ADDRESS> not found: https://repo1.maven.org/maven2/sh/almond/almond-spark_2.12/@VERSION@/almond-spark_2.12-<EMAIL_ADDRESS>I have tried several methods and even reinstall almond but the errors won't go away.

@VERSION@ is a template that is replaced with the current almond version during doc generation.
If you look at the spark docs through the website, you should see a real version you can use (if you've installed the latest kernel version):
import $ivy.`org.apache.spark::spark-sql:2.4.0` // Or use any other 2.x version here
import $ivy.`sh.almond::almond-spark:0.5.0`

Thank you. This works! It was an oversight on my part.


Script Editor UX

Normal Editor view: I added the Open Script Editor button in middle center (a good place to add general editor toolbar in future)

After opening Script Editor: has a back button & save, reset button on header. Also a option to change code editor theme in bottom right corner

After clicking reset button:

I removed the hotkey Shift + backtick as it was creating confusion with normal editor cmd

Looks great, nice work! Love the addition of the light/dark mode toggle


Question: Where should relationship errors point to?

Say I have some relationship named friend that has raised an error.  What should the pointer value be?
{
    "pointer": "/data/relationships/friend", // to the field itself
    "pointer": "/data/relationships/friend/data", // to the data envelope within the friend relationship object
    "pointer": "/data/relationships/friend/data/id" // to the id field that is actually in error
}

Is this left to the implementer's discretion or is there an agreed upon pattern?  The specification is unclear on relationship errors.

I think there's a bit of discretion here, but generally it makes sense for the pointer to refer to the most specific value possible responsible for the error. So, if you know for sure that the id value caused the error, I'd point to that.


[Feature request] Add lightweight logging capability

Hey @shaps80! Thanks for an amazing library, it's 💯💯!
Context
I would like to propose new feature - a simple logging framework. Currently there are few in-app logging frameworks like Dotzu but most of them seems to be inactive for a quite long time now (Dotzu since Sep 2017).
This feature would add possibility to preview & report logs collected during app runtime. These would be categorised using its severity (error, warning, info, ...) with possibility to add meta data (like network call response).
Library design impact
I am not sure if this propose fits design goals of Peek since the library is more related to the app UI. It would be nice if could add more context to reported screens though.
I would like to hear your opinion on that. Also, I am open on getting involved into implementation. Thank you!

@josefdolezal thanks for getting in touch.
I have considered something like this.
Could you elaborate on the type of logging you would expect? Also I'm not sure I understand what you mean by "add more context to reported screens"

Thank you for quick response!
I've updated my first post so it's all together.

Ahh ok I think understand what you mean.
So Peek is primarily an inspection tool, rather than a debugging tool. I have considered previously whether or not Peek could be more of a dashboard for various tools but my main concern is keeping Peek lightweight, focused and with a very small runtime-footprint.
This kinda sounds more like a debugging and bug-reporting tool – in which case there are various other frameworks you can integrate with that even have server side logging for consolidated logs, etc... which IMHO is a lot more useful than on device.
I can see a little bit of value, so if you want to try and convince me further I'm totally open to it, but I wouldn't want to waste any of your time on something that personally I feel is a little out of scope for what Peek is as a product.

Hey @shaps80, sorry for late reply!
No problemo, thats why I asked in the first place! I agree with you that the runtime-footprint should be as small as possible. This could be done using some plugins API (maybe  related to #29?) though so Peek would still be "just" inspection tool but extendible to do more if user needs it.
I will close the issue for now and will let you know if I come up with solution that will fit Peek's API design better.
Thank you for your time & review!


Scope technical expertise & resources

Step 1: Complete scoping of technical expertise and resources needed to build and host interactive conceptual models (including platform, equipment and staffing requirements for setting up and maintaining these interactive tools). Report due.

Step 1. Scope Technical Expertise & Resources.pdf
or Step 1. Scope Technical Expertise & Resources - Google Docs


adding set algebra to models and fixtures

Closes https://github.com/donejs/place-my-order/issues/131.

@justinbmeyer can you take a look at this one also?


Fixnum deprecated in Ruby 2.4.0

I'm getting deprecation warnings such as the following:
/cache/ruby/2.4.0/gems/mail-2.6.4/lib/mail/attachments_list.rb:33: warning: constant ::Fixnum is deprecated
Fixnum and Bignum have been deprecated in favor of Integer:
http://blog.bigbinary.com/2016/11/18/ruby-2-4-unifies-fixnum-and-bignum-into-integer.html

Thanks @Jacotb. This was addressed in #1053.


Make "getInitialSize()" method package scoped

Methods on ConnectionPoolConfiguration should be package private.

Thanks a lot. That's merged now.


🐛 responsivity by width (not touch)

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[x] Refactoring

Context

[x] Closes #<issue_number>
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[x] I've tested it at </bsx/collection>
[x] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

The issue was that we were relying on the touch aspect of the device instead of the resolution.
The remaining issue is, that if the navbar on the tablet should be the same as on the desktop, the Explore button will always lead the user to the explore part of the app ->  there's no hover on the tablet (Also, the requirement from @exezbcz was to have click/tap on Explore to lead to the explore part of the app).
Should we disable the rerouting from the navbar to Explore on mobile devices?

The issue was that we were relying on the touch aspect of the device instead of the resolution.
The remaining issue is that if the navbar on the tablet should be the same as on the desktop, the Explore button will always lead the user to the explore part of the app -> there's no hover on the tablet (Also, the requirement from @exezbcz was to have click/tap on Explore to lead to the explore part of the app).
Should we disable the rerouting from the navbar to Explore on mobile devices? // I did this; currently, the link to Explore by clicking/tapping on it works only on a desktop; let me know if you want something else.

yup, in case of tablet, there wont be any selection displayed, after clicking on explore, it will take you to explore page.

the dropdown still shows for a sec on tablet when clicking

hmm that's weird, can't reproduce it from devtools tablet but I guess there's a momentary hover state activated even on tablet, can be disabled tho.
followup question: what about the chevron? If the Explore button on small screen works only as link to explore section, shouldn't we hide it as well?

the dropdown still shows for a sec on tablet when clicking

hmm that's weird, can't reproduce it from devtools tablet but I guess there's a momentary hover state activated even on tablet, can be disabled tho.
followup question: what about the chevron? If the Explore button on small screen works only as link to explore section, shouldn't we hide it as well?

good q, if its possible and not that hard, i would hide it on touch devices and leave it only for devices that have option to hover.

good q, if its possible and not that hard, i would hide it on touch devices and leave it only for devices that have option to hover.

🚀

pay 30 usd

😍 Perfect, I’ve sent the payout
💵 $30 @ 24.85 USD/KSM ~ 1.207 $KSM
🧗 FNyt7T1xdbhN7dagf7yGYCRJuE3R45VmTCE3tjzy1rKxa7y
🔗 0xfbd7ceab99536bb4c4b8de2bbf30ade58b0b233da6b9c79ab4bc78c7fe0aefe8
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


dc-configurator findById doesn't work if dcId is a string

The findById call in /src/service/dc-configurator.js line 50 (latest master 7084e5e181b4ee565ebde56edb8f3aac66c69daa) will return undefined if dcId is a string. My testing showed that sometimes it was a string, sometimes it was an integer.
When it returns undefined, url becomes false further down the stack and requests stop working.
I'm making a fix to force dcId to be an integer when being passed to findById(). This fixes my issue locally.
Pull request #134

Recent (2.x & 3.x) branches of the project work slightly different in this place but thanks 👍

Merged #134


Variable isn't visible inside include if it is not used in extended template

Assume I have this structure:
<!-- nav.html -->
<div id="{{ nav_id|default('normal') }}">
...some html...
</div>

<!-- base.html -->
...skip...
{% include "nav.html" %}

and finally
{% extends "base.html" %}
{% set nav_id="index" %}

nav_id will not be visible in nav.html unless I access this variable in base.html. For example this way make nav_id work again in nav.html:
<!-- base.html -->
...skip...
<!-- {{ nav_id }} -->
{% include "nav.html" %}

As long as there is workaround it is not big problem, but still looks inconsistent.

I can confirm this bug with jinja 2.8, but in my case the suggested workaround above does not work at all. It doesn't matter if the variable is used or not before the include statement. Evenmore, my case is even simpler as I do not use template inheritance. I have just a simple template with a for loop and an include statement inside the loop that includes another simple template that tries to access the variable set with each iteration in the loop. This seems to be a critical regression as things do not work as explained in the Note at http://jinja.pocoo.org/docs/dev/templates/#import-visibility

I can confirm this bug with jinja 2.8, but in my case the suggested workaround above does not work at all. It doesn't matter if the variable is used or not before the include statement. Evenmore, my case is even simpler as I do not use template inheritance. I have just a simple template with a for loop and an include statement inside the loop that includes another simple template that tries to access the variable set with each iteration in the loop. This seems to be a critical regression as things do not work as explained in the Note at http://jinja.pocoo.org/docs/dev/templates/#import-visibility

False alarm. I was using jinja through Ansible and it is Ansible's fault (it uses jinja in a way that included templates are not passed the context). It is fixed for the upcoming version (2.0) of Ansible.

+1 on this, I'm experiencing the same with a block and an extends:
layout.jinja2
Hello, {% block world %}{% endblock %}

my_page.jinja2
{% set text = 'Pizza' %}
{% extends "layout.jinja2" %}
{% block world %}
  {% include "world_partial.jinja2" %}
{% endblock %}

world_partial.jinja2
{{ text }}!

Here is what I'd expect the output to be upon rendering my_page.jinja2:
Hello, Pizza!

But instead I get this:
Hello, {{ text }}!

You can "fix" it using @xen's method of using the variable inside the block:
my_page.jinja2
{% set text = 'Pizza' %}
{% extends "layout.jinja2" %}
{% block world %}
  <!-- text is {{ text }} -->
  {% include "world_partial.jinja2" %}
{% endblock %}

But this just feels like a hack to me.

Oops, I guess that'll teach me to force push!

This is not reproducible in version 2.10. It looks to have been solved with #619.
Can this be closed?


crate won't compile

With new 0.19.1 version I got build fail:
   Compiling garde v0.19.1
error[E0432]: unresolved import `idna`
  --> /home/ghashy/.cargo/registry/src/index.crates.io-6f17d22bba15001f/garde-0.19.1/src/rules/email.rs:18:5
   |
18 | use idna::AsciiDenyList;
   |     ^^^^ use of undeclared crate or module `idna`

I use this feature set:
garde = { version = "0.19.1", features = ["derive", "email"] }

But if I add email-idna feature it compiles:
garde = { version = "0.19.1", features = ["derive", "email", "email-idna"] }

Fixed in v0.19.2


Radarr files not converting, using wrong folder

I can't work this one out, for some reason whenever I try and have a file converted for Radarr its trying to make a temp file in C:\Windows so ends up failing Sonarr works files work fine, and by the looks of it its just via qbittorent
sma.log

PermissionError: [WinError 5] Access is denied
Nothing to figure out, its a permission error
No idea why you'd be using C:\Windows as your output directory but there you go, either change your output directory of change your permissions

There is no output directory listed in the .ini, cos if that was the case then anything tag for sonarr would do the same, if you look at the log, the sonarr conversion worked fine (Bigsky) but the radarr one did not (All star superman)

[Converter]
ffmpeg = D:\PlexBackend\FFmpeg\bin\ffmpeg.exe
ffprobe = D:\PlexBackend\FFmpeg\bin\ffprobe.exe
threads = 0
hwaccels = cuda, dxva2, qsv, d3d11va, opencl, vulkan, h265_nvenc
hwaccel-decoders =
hwdevices =
hwaccel-output-format =
output-directory =
output-directory-space-ratio = 0.0
output-format = mp4
output-extension = mp4
temp-extension =
minimum-size = 0
ignored-extensions = nfo, ds_store
copy-to =
move-to = \Plex\To Sort
delete-original = True
process-same-extensions = True
bypass-if-copying-all = False
force-convert = True
post-process = False
wait-post-process = False
detailed-progress = False
opts-separator = ,
preopts =
postopts =
regex-directory-replace = [^\w-_. ]
[Permissions]
chmod = 0644
uid = -1
gid = -1
[Metadata]
relocate-moov = True
full-path-guess = True
tag = False
tag-language = eng
download-artwork = poster
sanitize-disposition =
strip-metadata = True
keep-titles = False
[Video]
codec = h265_nvenc
max-bitrate = 2500
bitrate-ratio = hevc:0.8
crf = -1
crf-profiles =
preset =
codec-parameters =
dynamic-parameters = False
max-width = 0
profile =
max-level = 5.2
pix-fmt =
prioritize-source-pix-fmt = True
filter =
force-filter = False
[HDR]
codec =
pix-fmt =
space = bt2020nc
transfer = smpte2084
primaries = bt2020
preset =
codec-parameters =
filter =
force-filter = False
profile =
[Audio]
codec = aac, aac
languages = jpn, eng
default-language = eng
include-original-language = True
first-stream-of-language = False
channel-bitrate = 128
variable-bitrate = 0
max-bitrate = 0
max-channels = 0
filter =
profile =
force-filter = False
sample-rates =
sample-format =
copy-original = False
aac-adtstoasc = False
ignored-dispositions =
force-default = False
unique-dispositions = False
stream-codec-combinations =
[Audio.Sorting]
sorting = language, channels.d, map, d.comment
default-sorting = channels.d, map, d.comment
codecs =
[Universal Audio]
codec = aac
channel-bitrate = 128
variable-bitrate = 0
first-stream-only = False
filter =
profile =
force-filter = False
[Audio.ChannelFilters]
6-2 = pan=stereo|FL=0.5FC+0.707FL+0.707BL+0.5LFE|FR=0.5FC+0.707FR+0.707BR+0.5LFE
[Subtitle]
codec = mov_text
codec-image-based =
languages = eng
default-language = eng
include-original-language = False
first-stream-of-language = False
encoding =
burn-subtitles = False
burn-dispositions = Forced
embed-subs = True
embed-image-subs = False
embed-only-internal-subs = False
filename-dispositions = forced
ignore-embedded-subs = False
ignored-dispositions =
force-default = False
unique-dispositions = False
attachment-codec =
remove-bitstream-subs = False
[Subtitle.Sorting]
sorting = language, d.comment, d.default.d, d.forced.d
burn-sorting = language, d.comment, d.default.d, d.forced.d
codecs =
[Subtitle.CleanIt]
enabled = False
config-path =
tags = default, no-style
[Subtitle.FFSubsync]
enabled = False
[Subtitle.Subliminal]
download-subs = False
download-forced-subs = False
include-hearing-impaired-subs = False
providers =
[Subtitle.Subliminal.Auth]
opensubtitles =
tvsubtitles =
[Sonarr]
host = <IP_ADDRESS>
port = 8989
apikey = 5ee115a0e19e40f799228c008c544204
ssl = False
webroot =
force-rename = False
rescan = True
block-reprocess = False
in-progress-check = True
[Radarr]
host = <IP_ADDRESS>
port = 38084
apikey = 9dae3f8534b7485fb549b9f77bb6a85a
ssl = False
webroot =
force-rename = False
rescan = True
block-reprocess = False
in-progress-check = True
[Sickbeard]
host = localhost
port = 8081
ssl = False
apikey =
webroot =
username =
password =
[Sickrage]
host = localhost
port = 8081
ssl = False
apikey =
webroot =
username =
password =
[SABNZBD]
convert = True
sickbeard-category = sickbeard
sickrage-category = sickrage
sonarr-category = sonarr
radarr-category = radarr
bypass-category = bypass
output-directory =
path-mapping =
[Deluge]
sickbeard-label = sickbeard
sickrage-label = sickrage
sonarr-label = sonarr
radarr-label = radarr
bypass-label = bypass
convert = True
host = localhost
port = 58846
username =
password =
output-directory =
remove = False
path-mapping =
[qBittorrent]
sickbeard-label = sickbeard
sickrage-label = sickrage
sonarr-label = sonarr
radarr-label = radarr
bypass-label = bypass
convert = True
action-before =
action-after =
host = <IP_ADDRESS>
port = 38090
ssl = False
username = Morpheus2n2
password = Oliver150107!
output-directory =
path-mapping =
[uTorrent]
sickbeard-label = sickbeard
sickrage-label = sickrage
sonarr-label = sonarr
radarr-label = radarr
bypass-label = bypass
convert = True
webui = False
action-before =
action-after =
host = localhost
ssl = False
port = 8080
username =
password =
output-directory =
path-mapping =
[Plex]
host = localhost
port = 32400
refresh = False
token =
username =
password =
servername =
ssl = True
ignore-certs = False
path-mapping =

Ok, Debug all set so should kick off when next file hits in a bit :D
Thats what I cant work out the Root folders have always been the same for all my clients for years, its only been happening for last 2 weeks when I pulled a new download of the script lol

Ok debug log here, SABNZbd works file with Radarr tags, qbittorrent doesn't lol
sma.log

QBT is reporting your root path as "."
No idea why but that's the issue
Are your command line parameters set correctly?

Hm was there an update that broke something to QBT? It shouldn't be returning a . for your root path. That's causing the problem though

a2ea0e4c55cd3051a3814bb29a73b8e8f24643c9
See if that works around it

sma.log
Still doing it unfortunately, its so odd as its only Films this happens to.

9e3541e790a65e6cf50cfa6de3386b12e1b1d10a
Try that

sma.log
Yep that seems have done the trick, its in progress right now, will keep eye on it over next couple of days before closing this thread :D
Thank you so so much.

Ok that failed but it was a different issue which I have identified and fixed.
It seems FFMPEG 6.0 doesn't like HW Decoding and encoding, took it back down to 5.0 and its working like a dream again


Performance isn't improving

Hi,
I'v written  a little bit of code that I think could benefit performance-wise from using rayon, but when I tried it, I discovered it didn't.
the code does this:
I'v got an md5 of a number between 1 and 10000000, and I need to find what it is by checking each number in this range.
#![feature(test)]
extern crate rayon;
extern crate test;

use std::time::SystemTime;

use md5;
use md5::Digest;
use rayon::prelude::*;

fn is_same(checked: f64, digest: Digest) -> bool {
    md5::compute(checked.to_string().into_bytes()) == digest
}

fn main() {

    let timeit = SystemTime::now();
    println!("counting in parallel:");
    println!("result: {:?}", (1..10000000).into_iter()
        .find(|num: &i32| {
            is_same((*num) as f64, md5::compute(1234567.to_string().into_bytes()))
        }));
    println!("it took {:?} seconds.", SystemTime::now().duration_since(timeit).unwrap());

    let timeit = SystemTime::now();
    println!("result: {:?}", (1..10000000).into_par_iter()
        .find_any(|num: &i32| {
            is_same((*num) as f64, md5::compute(1234567.to_string().into_bytes()))
        }));
    println!("it took {:?} seconds.", SystemTime::now().duration_since(timeit).unwrap());
}

For some reason, trying this on my computer give worse results when using parallel.
can someone please explain this?

When you search for a match on 1234567, the serial version only computes exactly 1234567 items and stops. The parallel version will divide the range into smaller ranges, so all the computation it does on the higher portion of the range is extra work that the serial version skips. While it will still stop as soon as a match is found anywhere, that's still some extra used CPU resources. If your machine has SMT, like Intel hyper-threading, then that extra work on one thread can actively slow down other threads competing for resources on the same core.
However, if I change your search value to 7654321, the parallel version is consistently faster. Now it's a benefit to have some threads working on the high end of the range, whereas the serial version has to process all the way from the beginning.
Another option is to use par_bridge() on the range instead, so items are basically distributed in parallel from the beginning of the iterator. The bridge adds some overhead compared to into_par_iter(), but it's small compared to the md5 work you're doing, so that's OK. I find that this is indeed faster than the serial version at finding 1234567. It's also faster for 7654321, but not as fast as the pure parallel version.
Results on my laptop i7-7600U (2 cores, 4 logical threads):
finding with iter():
result: Some(1234567)
it took 514.543215ms seconds.

finding with into_par_iter():
result: Some(1234567)
it took 980.925067ms seconds.

finding with par_bridge():
result: Some(1234567)
it took 274.061157ms seconds.

finding with iter():
result: Some(7654321)
it took 3.165408314s seconds.

finding with into_par_iter():
result: Some(7654321)
it took 123.41093ms seconds.

finding with par_bridge():
result: Some(7654321)
it took 1.743722778s seconds.

Here's another result searching for 0, so they have to compute everything and won't find it:
finding with iter():
result: None
it took 4.124399588s seconds.

finding with into_par_iter():
result: None
it took 2.014241121s seconds.

finding with par_bridge():
result: None
it took 2.217921687s seconds.

you can do a sequential iterator on blocks of ranges (using step by for example) and then map each block to a parallel find and do a sequential find on the mapped iterator

@yardenfi - please feel free to re-open if you have further questions.


Exterimental auth

Introduce context-aware versions of 'runWarpServer*' functions and implement generalised authentication support via an AuthHandler.
For more details, see here: https://hackage.haskell.org/package/servant-server-0.19.2/docs/Servant-Server-Experimental-Auth.html
And here: https://docs.servant.dev/en/stable/tutorial/Authentication.html

ping?


add label before page size dropdown

Is it possible write like this 'Rows per page : ' before than page size dropdown ?

@udenizdogan please read pagination docs, here we show you how to custom the pagination.
this online demo is might be your top answer.
please tag me if the issue still remain, thanks


suppress security lint warnings for a while.

Those suppression should be removed once #1446 is fixed.
What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

@jmuk: Adding do-not-merge/release-note-label-needed because the release note process has not been followed.

One of the following labels is required "release-note", "release-note-action-required", or "release-note-none".
Please see: https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md#write-release-notes-if-needed.
Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/release-note-none

@jmuk: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
0d1b0335693ed7fca142a46cb94ffb7bbd6b990e
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

/lgtm

@jmuk: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
a4ed2c73483d741e12ac68977e36cd33ae0a74f1
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmuk: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
b068d5b25c923b03d63040009eb7463d924df2e8
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/assign @ldemailly

/lgtm
(but unfortunately you need to resolve a conflict so you'll loose the lgtm again)

rebased, and removed the suppression of mixer/pkg/il since the related PRs ( #1828) seems to be reverted.

/lgtm


Dedicated server crash fatal error

[2021-12-25 01:21:32.481] [info] [DEDICATED PRINT] [574.383]  [2021-12-25 01:21:32.482] [info] [DEDICATED PRINT] FatalError:  [2021-12-25 01:21:32.482] [info] [DEDICATED PRINT] Tried to remove entity #209:npc_dropship [209]:models/vehicle/crow_dropship/crow_dropship_hero.mdl while a player (#6, 'PlayerSomm', lifestate #0) was attached.

Same issue as https://github.com/R2Northstar/NorthstarMods/issues/13

this is fixed as of current commit to NorthstarMods, will be in next release which should ideally be out today


Add a flag to print specs errors while running.

It can be very long to wait for other specs to pass in order to see results, and it's not very convenient to indicate only specs we want to be runned in a file.
I think this can improve workflow:

before: code -> run specs -> wait -> fix errors -> repeat.
after: code -> run specs -> fix progressively errors -> repeat.

Right now you can run specs with --fail-fast, but what you are suggesting is printing errors as they appear, but continue running everything. Does rspec have something like that, so we can get an inspiration for the output format? Or do you have a suggestion for it?

This might be something similar, as an rspec plugin: https://github.com/grosser/rspec-instafail

There hasn't been any interest for this in the past four years, but it seems this could be useful and shouldn't be hard to do. A custom Spec::Formatter or adjustment of existing one can be integrated with spec. A nice feet would be to include (optional) diagnostics into TAP formatter.
The focus feature certainly helps to conveniently indicate which specs to run.


ui creator

Describe the feature
it would be nice to have a way to customize the cad:s ui with somekind of app maybe you could change the button places and stuff
Additional Context
No response

This is pretty much impossible for a one-man maintainer. Probably will not happen in the near future.

i thouhg that would be the case if it were to hapen it doesnt need to be a app html would be nice too

You're always free to create one yourself and contribute to SnailyCAD ;)!

ill maybe try but im not good at code exept lua anddd maybe c++


Install codacy to repo

Codacy needs to be installed in the repo, more on Codacy

@dikshashukla455 go for it, let us know if we need to help or enable anything!

Sure, I will work on it

@ikayz Codacy wants $15 for getting started after the 14 day free trial. So, what do you suggest that I do?

Yeah, for personal use you have to subscribe. It's free for open source projects. It requires admin rights I think to a repo. I'll take care of this issue since I have admin rights

Okay ! so, I will look for another issue?


lite: tiny typo-like update to ResizeBilinear opt op

@jackwish Thanks for the PR , Here we can process only merge comit to tensorflow:master , please update it .

oops, pinged too many people, sorry... will open another.

@jackwish Thanks for the PR , Here we can process only merge comit to tensorflow:master , please update it .

@rthadur check #24514 instead please.


graphics/pdcurs34: Fix compile error

Summary
Fix a compile error when CONFIG_SYSTEM_TERMCURSES=y.
Fix a compile warning when CONFIG_PDCURSES_MULTITHREAD=n.
Impact
None
Testing
Run example/pdcurses.

Please ignore the nxstyle warnings which have nothing to do with this fix.


Navigate between screens with Compose: Android Basics with Compose Step 5

On Step 5

I cannot go furhter then Flavor.
The "onNextButtonClicked from Flavor to Pickup is not working.
I already copied the whole NavHost from the solution and the issue is still the same.
I cannot find the fault.

Ich already tried to figure out, what happens on the Event with Log.d
On SelectedOptionScreen the onNextButtonClicked is received, but in the Flavor Screen, nothing happens...??? And the Log.d is not carried out at all.
I can not go to the PickupScreen. I am really confused and dont know, where to look after the Problem anymore...
Since nobody else seems to have this problem....it is probably my own fault (or nobody had tried the navigation at this time of the codlab??)...but I dont know where I coud have made it...
And funny thing is, that i can got to Pickup directly from Start...... Where is the fault here?

AND...the "Cancel" Button is not working either...

You gotta show more of your code than that, but note that both the Flavor and Pickup routes should both be using the SelectOptionScreen composable, BUT they obviously have their respective arguments for that composable (options, onSelectionChanged) so make sure you don't have the same arguments for both routes' SelectOptionScreen by mistake. I would then verify if your routes are correct. I hope this helps.

You gotta show more of your code than that, but note that both the Flavor and Pickup routes should both be using the SelectOptionScreen composable, BUT they obviously have their respective arguments for that composable (options, onSelectionChanged) so make sure you don't have the same arguments for both routes' SelectOptionScreen by mistake. I would then verify if your routes are correct. I hope this helps.
Also the cancel button isn't supposed to work at this point of the tutorial.

Thanks for your concern! But yes, I had that in mind. My Code for the Route in the NavHost is exactly the same, as in the solution but doesn't work. I left it this way and moved on in the pathway... 😊😊

I do not wonder anymore and moved on. ....

I know you said you moved on, but I must say that I was wrong, and this is actually very important to know regarding Kotlin syntax... in the SelectOptionScreen composable, make sure you do not have curly braces around the onNextButtonClicked parameter you are using, and if you are, you need to have opening and closing parentheses with onNextButtonClicked. so like this:
onClick = onNextButtonClicked
or
onClick = { onNextButtonClicked() }

@brendanbdev
😊
Exactly this was the problem! I figured it out, as I tried to build the app in unit 4. There were curly braces for an empty Lambda on 2 "onNextButtonClicked" events, i had missed in following the instructions. So nothing happend :-)..In my CityApp, I got it right and the navigation is functional as it should. Thanks for the further instructions!


refactor: update dev-server example

Updating the dev-server example so that it:

Reloads the server on changes
Reloads the client
Shows server errors on the client

P.S. i am not entirely sure whats the advertised use case with this example, but serve-handler seems to be a very "strict" static server middleware - it directly outputs a 404 if it fails to find a file for any request - instead of giving other middlewares the chance to do something better :D, so this makes it imho hard to use along with tinyhttp.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #29   +/-   ##
=======================================
  Coverage   61.35%   61.35%
=======================================
  Files          10       10
  Lines         339      339
  Branches       77       77
=======================================
  Hits          208      208
  Misses        130      130
  Partials        1        1

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 aba04b0...9784a9b. Read the comment docs.

LGTM, merging

@all-contributors please add katywings for example


phpPackages.phpstan: 1.10.58 -> 1.10.59

Description of changes
release notes: https://github.com/phpstan/phpstan/releases/tag/1.10.59

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build phpPackages.phpstan


Wrong bower.json

Hi
I updated angular-audio and my main project bower.json shows the following version info
"angular-audio": "^1.7.3",

However the bower.json file for angular-audio in my bower libs directory shows the following version
"name": "angular-audio",
"version": "1.7.0",
I have tried removed the library and reinstalling via the console which shows the following
bower angular-audio#*           cached https://github.com/danielstern/ngAudio.git#1.7.3
bower angular-audio#*         validate 1.7.3 against https://github.com/danielstern/ngAudio.git#*
but the bower.json file still shows 1.7.0.
However... The bower.json in the github source shows 1.7.2 ???
The main issue I'm having is the same problem as issue 85 that was reported as fixed according to the release notes and the issue remarks
Release Notes v1.7.1
Fixed https://github.com/danielstern/ngAudio/issues/85
Here is what I get on my console
angular.audio.js:128 Uncaught TypeError: Cannot read property 'play' of undefinedtwiddle @ angular.audio.js:128
Then if I do anything that goes near another sound file I get this which is obviously the same error as reported in the issue
undefined:1 Uncaught (in promise) DOMException: The play() request was interrupted by a call to pause().
Any idea whats going on with downloading the latest version i.e. 1.7.3 ????
Steve

Hi All
Any idea when the npm and bower installs will be updated to 1.7.3 as this version fixes the issue

@smcardle If you would like to handle it I'd be happy to make you a contributor

Please use NPM from now on https://github.com/bower/bower/issues/2298


DRILL-4359: Adds equals/hashCode methods to EndpointAffinity

Adds equals/hashCode methods to EndpointAffinity to allow for comparison in tests.

+1


Master tests fail

Recent changes in the imports in qiskit/ignis/verification/__init__.py make many (maybe all) verification tests fail. I wonder how in the first place it was technically possible to merge code with failing tests.

The current state of master passed all the unit tests prior to merging. The branch protection settings on master prevent merging a PR unless travis succeeds. I am also not able to reproduce this locally. I expect this is a local issue in your environment caused by having a stale directory or pycache from before import changes. Try running git clean -fdX in the repo prior to running the tests.

Solved when I ran git clean -fdX in the Terra clone. Thanks @mtreinish !


Feature request: line numbers

Thank you for the great and very lightweight product!
It would be really useful if the app had line numbers in the table. Sometimes you refer to a line in a csv file by its number.

Maybe a total number of lines could be great in the status bar.

Please propose an UI mock up of how the status bar should be displayed

To show row number just click on menu
Window -> Show line number

The App Store version doesn't have that menu item.

any update on this?

+++


Ignore draft/WIP PRs

Ignore draft PRs
Ignore WIP PRs (without reviewers?)

Ideally create a new column for draft/WIP PRs and ignore them in Total column

The condition without reviewers is not valid criteria for the the wip status. The author might have announced the PR in some channel and await the possible reviewers to add themselves to the PR, according to their workload. Just the draft and WIP will be used as criteria.


Delete gh-pages branch?

Since we are using master to generate the website, and gh-pages is out-of-sync by now, I plan to remove it. Objections?

Hi @libcce! I can't view the settings, see above. Can you please check from which branch the site is generated?

Let's try to work this out via Zoom @katrinleinweber.


feature/#1128 - faster MapTileCache management

Removed class:

LRUMapTileCache: already deprecated class, removed en route

Impacted classes:

IConfigurationProvider: impacted the removal of LRUMapTileCache in comments; gently refactored
MapTileCache: new boolean mAutoEnsureCapacity and its getter; optimized method garbageCollection using mAutoEnsureCapacity; method ensureCapacity now returns a boolean ("actually changed the capacity?"); optimized method shouldKeepTile
MapTilePreCache: added a little optimization in method fill; fixed typos
MapTileProviderBasic: set the new MapTileCache.mAutoEnsureCapacity to true

aside from the one comment, nothing pops out as a potential issue. I'll pull down the branch and give it a whirl.

aside from the one comment

@spyhunter99 ???


Update command not working correctly

Update command only updates once. When trying to update an item more than once, the update does not happen or reflect.
Example of how to run the command is given below:
Update <class name> <id> <attribute name> <attribute value>
Update BaseModel 1234-1234-1234 email<EMAIL_ADDRESS>

Fixed.


fixes #472: Fix race condition on startup 4.2.x

Fixes #472

Meriging this as the Java CI is green


Add generation of travis via haskell-ci adding tested-with.

I put on hold the other changes you made until this will be solved.


[Fix] : Improve the Portal revocation function

What does this PR do?
Improved the code avoiding loop when match is found.
Related ticket
Fixes #654
Type of change

[ ] Chore
[x] Bug fix
[ ] New feature
[ ] Documentation update

Check list

[x] Unit tests for any smart contract change
[ ] Contracts and functions are documented

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

Project coverage is 94.90%. Comparing base (e11d134) to head (88e6152).

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #657      +/-   ##
==========================================
+ Coverage   92.66%   94.90%   +2.24%
==========================================
  Files          30       30
  Lines         491      491
  Branches      114      114
==========================================
+ Hits          455      466      +11
  Misses         24       24
+ Partials       12        1      -11

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


Fixes for SNAP-3147 :

Changes proposed in this pull request
Problem: When cluster restarted and if user do not refresh the page "Uptime:" field is not showing the correct Uptime of cluster.
Fixes:

Adding instance creation time and its getter.

Patch testing

Tested manually

Is precheckin with -Pstore clean?
ReleaseNotes changes
(Does this change require an entry in ReleaseNotes? If yes, has it been added to it?)
Other PRs
(Does this change require changes in other projects- snappydata, spark, spark-jobserver, aqp? Add the links of PR of the other subprojects that are related to this change)

Merging.


Vary storage by host

If we set the descriptive TEST_OUTPUT_HEROKU_COM_STORAGE_BASE_URL configuration variable, this will allow @heroku/devex to retire their instance.

@dmathieu, done!

LGTM

For us to use this, we'd need to

Grant your busl user access to our bucket
Set the TEST_OUTPUT_HEROKU_COM_STORAGE_URL
Configure DNS/SSL for test-output.heroku.com to point to busl
Switch our services to point use test-output.heroku.com

Does that sound about right?

@joshwlewis almost, yes. Busl has no IAM user. You need to pass presigned credentials to busl when publishing and creating a stream.
Vacuole does that here.

@tt any news on this? We'd love to move over.


GPL-608 Q. Do we have for all plates in this box all the plate map information for all samples?

User story
GPL-608 | As heron stakeholders (Rich C, Danni, Sonia, John S) I woudl like for a box to know all plates have complet plate maps to having to revist a box only some plate  have plate maps
Who are the primary contacts for this story
Rich C
Danni W
Preamble
What we expected:
A LH lab would send to SFTP plate maps (in night) for plates shipping to Sanger the next day.
On the next day we would receive box and box manifest.
SSR pick all the positives from a given box with this information.
What we observe:
When we don't have plate map information for all the plates in a box, the LH +ves on site report suggests that these plates have only -ves, and we dont pick from them.
LH lab updates SFTP with plate map information
SM see that picked from boxes suddenly have new positives we can pick
Logistics become difficult as we were expecting to pick all the +ves from a box on a day, not over several days.
Root issue: If for a given box on a given day we have plate maps for some of the plates, but not all, and SM start picking from the box.
Options

Have a new report that show for a given box all plates have complete plate map information. (Complete plate map information = root sample id, RNA plate barcode , result, result date.
Add to existing LH positive on site report new column ' All my sibling samples are in a plate that has a all the plate map information for our box'
List the +ves on site report by box barcode, show which have plate maps or not.
Something more elegant.

Acceptance criteria
To be considered successful the solution must allow:

[ ] TBD
[ ] ...

Dependencies
This story is blocked by the following dependencies:

#<issue_no.>
sanger/#<issue_no.>

Additional context
Add any other context or screenshots about the feature request here.

Discussed with Rich C & James M on 20th October agreed to deprecate/archive low value given lens of Beckman robotics


Add reindent utility.

The intention is to use this tool in dlang.org to generate docs with narrower indentation when targeting narrow-screen media like Kindle.

There already is a detab utility, maybe there's some value in uniting them?

when targeting narrow-screen media like Kindle

Do you have a plan to implement the usage of this utility in the dlang.org makefiles as well?

win32.mak too please.

It's messy to unite it with detab, because this code uses ranges whereas detab uses a manual char array loop.

Then maybe reimplement detab on top of this code? :P detab is probably D1-era.

Isn't detab just a simple matter of running std.regex.replaceAll("\t", "    "); on every line?

Not if you have non-leading tabs (or mixed tabs and spaces). Then you need to track columns.

Oh, right. Darn. :-D
Well, it would have no code in common with reindent then. Although I can see some benefit in having a single program with subcommands that you can select, e.g., reformat reindent abc.d, reformat detab abc.d, etc.. On Linux, you could even symlink reindent, detab, etc., to the executable, and use argv[0] to switch on what action to run.

OK, let's leave detab alone then.

Do you have a plan to implement the usage of this utility in the dlang.org makefiles as well?

Well, I was planning to take a look to see how nasty it would get. "Planning to implement" sounds a bit too strong. :-P

Well, I was planning to take a look to see how nasty it would get.

OK. I think it might get very messy, especially in the case of win32.mak.

That is an awful lot of code for a "simple" replacement:
perl -p -e 's/^(  )+/$& =~ s,  ,    ,gr/e' filename.d

Why no go the whole mile and have a reindent that knows D source, looks for braces and such? Maybe dfmt is the proper place?
Still, I like the small script for educational purposes.

Hmm... ^s/(  )+\1/\1/g

You mean ^s/( +)\1/\1/g?

Err, yes.

But this only correctly performs reindentation of 2n spaces to n spaces, whereas the current code is more flexible, e.g., reindenting 4-space indentation to 3-space.

Well, yes, but do we need N-to-N reindent?

I think the harder problem is integrating this tool with the makefiles.

Yes. :-)

Are we going ahead with this, or should I just close this and forget about it? It's been a looong time.

The intention is to use this tool in dlang.org to generate docs with narrower indentation when targeting narrow-screen media like Kindle.

Can't you do that with CSS?

Code is usually rendered in blocks with a fixed-width font, so I don't see how CSS would help solve this problem.

But if there's not that much interest in this, maybe this PR should just be closed. It is just a trivial utility after all.


[QUESTION] How to use Middleware in FastAPI?

Hi,
I was trying to figure out whether FastAPI makes sense in the following application:

C++ Application on Server A (CA)<--middleware over TCP--> FastAPI Application on Server B <--HTTP(S)/Websocket-->mobile web browser/desktop web browser
C++ Application on Server B (CB)<--middleware shared memory--> FastAPI Application on Server B <--HTTP(S)/Websocket-->mobile web browser/desktop web browser

The usage cases are:

Application CA and application CB want to push notifications to be displayed to all connected (logged in) browser clients.  But server A's CPU cache is very precious that the middleware has to be as light as possible and so HTTP isn't an option.  For application CB, I could find some C++ http library to interact with FastAPI application over HTTP probably but shared memory seems better as both applications live on the same server.
The browser clients want to send some commands/instructions to Application CA to do something on CA.

I did read someone post a similar question.  However, the ticket was closed as the original poster didn't provide use cases.
I believe that I can replace "FastAPI Application on Server B <--HTTP(S)/Websocket-->mobile web browser/desktop web browser" with "Java Application on Server B<--TCP-->Java GUI/Android App/iOS App" in the above diagrams.
However, it seems like the development and maintenance cost are going to be quite high given I can limit a subset of browsers as the users are internal.

@hc128168 Have you considered using some message queue like RabbitMQ? When your server consumes message sent from server you cant emit messages via WebSocket?

@pyropy you mean something like these?

C++ Application on Server A (CA)<--RabbitMQ over TCP--> FastAPI Application on Server B <--HTTP(S)/Websocket-->mobile web browser/desktop web browser
C++ Application on Server B (CB)<--RabbitMQ's shared memory--> FastAPI Application on Server B <--HTTP(S)/Websocket-->mobile web browser/desktop web browser

@hc128168 Yes something like that. You can find more info at the docs: https://www.rabbitmq.com/
But do you still have questions around how to use middleware for FastAPI? I don't see usecase for middleware to solve your problem :)
Here's like to the FastAPI and Starlette docs regarding middlewares:

https://www.starlette.io/middleware/
https://fastapi.tiangolo.com/tutorial/middleware/

Hi @pyropy , thanks for your suggestion and links.
I think FastAPI's and Starlette's docs define "middleware":

A "middleware" is a function that works with every request before it is processed by any specific path operation. And also with every response before returning it.

And the middleware I referred to in this post is more like Zeromq, RabbitMQ, 29west, etc (i.e. a messaging layer).
Is there a tutorial or document somewhere that demonstrates how I can run a dedicated thread within FastAPI to process the messages from the middleware (e.g. RabbitMQ) and then push through websocket(s)?

@hc128168 I didn't manage to find any tutorials that implement fastapi, rabbitmq and websocket but as from my experience you would run webserver using uvicorn or something similar and start Rabbitmq (or other messaging services) consumers running in the same event loop.
Upon recieving the message you would emit it to whatever room or topic you choose to via websocket.

I'm not sure I understand the idea with the arrows and the shared applications, but I guess something like ZeroMQ would give you the lowest latency if that's what you are trying to optimize. Otherwise, something based on Redis (or RabbitMQ) could work.
Maybe this could help: https://github.com/encode/broadcaster
If with something like ZeroMQ, I guess you could have to run an infinite loop in an async function started at a startup event that updates the data, but not sure. :shrug:


Fix Editable(Text) shortcuts to respect read-only on desktop

Description
This PR fixes EditableText (or RenderEditable under the hood) to respect read-only mode when it comes to cut/paste shortcuts and backspace/delete keys on desktop.
Related Issues
Pointed out by @rydmike in #48099. This PR does not fix enableInteractiveSelection though, only readOnly.
Tests
I added the following tests in packages/flutter/test/widgets/editable_text_test.dart:

keyboard shortcuts respect read-only on android
keyboard shortcuts respect read-only on fuchsia
keyboard shortcuts respect read-only on linux
keyboard shortcuts respect read-only on macos
keyboard shortcuts respect read-only on windows

$ flutter test test/widgets/editable_text_test.dart
Running "flutter pub get" in flutter...                            275ms
00:11 +151: All tests passed!

$ flutter analyze --flutter-repo
Building flutter tool...
Running "flutter pub get" in flutter...                            287ms
Analyzing 3 directories...
No issues found! (ran in 46.9s)

Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

/cc @gspencergoog

The Google testing failure appears to be unrelated to this change, so I'll land this when we clear that out.


No login-prompt in step 5

First of all, thanks for the very good deployment guide. It's really easy to understand.
I'm at the moment trying to deploy this FAQ-plus bot within my organiazation. I've got a problem in the deployment guide at step 5, where we should be able to finish the configuration app. When I go to the redirected URL at https://[BaseResourceName]-configuration.azure.net there is no prompt to login. As you can see in the picture below, the url is just just a normal web-app page.

I've done the deployment guide multiple times within different resourcegroups and with different KB in QnAmaker. Every time I get to step 5, where I can't go further due to no login prompt.
As a global administrator, I've also tried to grant admin consent in the API permissions within the configuration app as I saw that proposed in another troubleshoot. That didnt help either.
I can't figure out where I've done something wrong. Hopefully one of you guys can help :-)
Best regards

@Fledelius The full URL can either be https://[BaseResourceName]-configuration.azurewebsites.net or https://[BaseResourceName]-config.azurewebsites.net, and if the URL was not written as how I have written it, then I'll make sure that this update is made for the deployment guide

@coderkrishna Thanks for your reply. I think the deployment guide is fine, and thats not where my issue lies.
As far as I understood, the right URL can always we be found in the App Service for the [BaseResourceName]-configuration/config app, right?
In the App Registration I've redirected the configuration/config (either name is appropriate, right?) app's URL to my App Services URL - and thats https://[BaseResourceName]-config.azurewebsites.net
My issue is that when I go to that URL (https://[BaseResourceName]-config.azurewebsites.net), I dont get the prompt to login as it says in the deployment guide. It just shows the standard welcome webpage as I posted in my OP.
It seems like are is no transition (lack of better words) from the App Registration-config to the URL, and I cant figure out where I went wrong.
Thanks for your time.

@Fledelius - Have you set up the authentication correctly? I am going through the deployment guide. Please share the screenshot of the configuration app registration, especially the authentication piece. I'm thinking you have not specified the Redirect URI there

@coderkrishna here is screenshot of the configuration app registration at the authentication piece. I redirected the URI to https://[BaseResorceName]config.azurewebsites.net as told in the guide.

Appreciate your time.

@Fledelius That looks good to me, can you also send me a screenshot of your resource group? I want to see the resources as well

Of course. Here you go

We are not able to repro this issue at our end.
Let us know if issue is not resolved yet.


Update Public-Themes-List.md

Added RPC Authority theme and updated BRS links.
Thank you : D

based


Replace md5 with sha256 on UI

Since PyPI is being rewritten, it's a good opportunity to replace md5 hashes with sha256.
NOTE: there is also #681 which talks about the upload functionality related to md5

Related #681

It's planned to do that yes. It requires a bit of work to migrate everything because we have to loop over all of the current packages and recompute their hashes.


Missing alias America/Montreal

Could you please update the iana database, America/Montreal is present in the windows map but not in the iana map.

Hopefully fixed now (https://github.com/rubenillodo/windows-iana/pull/19). Thanks for reporting this!

Thanks for being so reactive, I actually rewrote your findWindows so that cant happen again.
I feel it should still try to search even if it can't find aliases.
import { intersection as _intersection } from 'lodash'
import { findAlias } from 'windows-iana'
import { IanaName, WindowsZoneName } from 'windows-iana/dist/enums'
import { map as timeZoneMap } from 'windows-iana/dist/time-zone-map'

// Note: We had to rewrite that function because the default doesn't even
// try to search in the timezone map is it can't find the aliases.
// The problem is some "less used" aliases like America/Montreal, are not
// in their list of aliases.
export const findWindows = (
  ianaTimeZone: IanaName | string
): WindowsZoneName | undefined => {
  const aliases = findAlias(ianaTimeZone) || [ianaTimeZone as IanaName]
  const timeZone = timeZoneMap.find(({ iana }) => _intersection(iana, aliases).length > 0)
  return timeZone?.windowsName
}


Update README.md to reflect current gradient checkpointing support

Previously the readme stated gradient checkpointing was incompatible with 4-bit lora in the current implementation however this is no longer the case. I have replaced the warning with a link to the hugging face documentation on gradient checkpointing.

Could you swap the link to the main branch's one for future ref? I think this is the new link?
https://huggingface.co/docs/transformers/main/en/perf_train_gpu_one#gradient-checkpointing


Bug: Folder opens when any button is clicked in list view

What happened?
When the folders page is open in the list view format and you try to click on any of the 5 buttons to manage it, the folder preview showing all of its files will also open, along with the actual request button content (i.e prompt to delete it, notification to make it public/private, etc).

Version
upstream (ghcr.io/diced/zipline:trunk)
What browser(s) are you seeing the problem on?
Chromium-based (Chrome, Edge, Brave, Opera, mobile chrome/chromium based, etc)
Zipline Logs
No response
Browser Logs
No response
Additional Info
No response

It looks like this also happens in the files list (on the home screen), so I think this is a bug across all the UI where you display lists of things.


EGE debugging features

Module to auto debug engine

[x] hex dump
[x] ~shared_ptr tracker~ we have UBSAN now
[x] EventLoop etc. inspector

SharedPtr tracker is done.


Include needed modules on the example code

The example code on README.md requires the use line to have the modules available when compiling.
Without it the example can't compile.

Good catch! Thanks.

You are welcome (:


Support UUIDs for Target Type

For Xcake I had to support UUIDs for Target types, as when I was building a project scaffolder based on Xctemplates the templates used UUIDs rather than nice names like 'application'. It was easier to pass this straight in and pass it to xcodeproj than try to treat it like a symbol and look it up in xcodeproj's constants.
I would very much like to restart my scaffolder on Struct rather than Xcake so this will be useful :)

@jcampbell05 Could you provide some examples? I'm happy to do it if you can provide more detail on what changes would be made 👍

So you can see what I did here https://github.com/jcampbell05/xcake/blob/master/lib/xcake/xcode/project.rb#L86
When the type is a symbol i.e ':application' I find the UUID and use that. If the type is a string then I assume it's a UUID and pass it directly in :)

Ah I see! We could definitely support that, potentially by extending DSL functions to look for another parameter, or maybe a hash e.g.:
target 'My app' do
  type :uuid => 'XXXX'
end

Then we can do a simple type check, .is_a?(Hash) then pass through the :uuid value

@lyptt yup that would work :) will be good to see if I can swap it out for Struct :)  I'm guessing Struct has a serializer for the Spec, Yaml and Json too (For the project recreation)

@jcampbell05 Yup! StructCore::Specwriter can generate JSON and YAML spec files.


Add windows exif

Adding Windows metadata: title, author, comments..

People need to check out mattiasw/ExifReader. exif-js is dead.

People need to check out mattiasw/ExifReader. exif-js is dead.

I don't understand. What do you need?

I am saying your PR will never be merged. The project is not maintained and the last NPM release was over two years ago.


AttributeError: 'OpenProject' object has no attribute 'get_user_services'

Hi,
I've just copy/paste the code bloc of readme and i'v got error :
AttributeError: 'OpenProject' object has no attribute 'get_user_services'
Some module changed ? is really hard to follow with test case, have you a documentation ?
or simple exemple for beginning ?
regards,

I think it should be get_user_service, without an 's' at he end.


Develop JSON Schema for Event Profile 'Fishing Event'

Dear @Aravinda93 , dear @albert-verdeny ,
This is an event profile for fish traceability, loosely aligned with existing solutions and guidelines, but deliberately not a 1:1 copy to not overcomplicate things. It contains a couple of elements that, from my POV, our tool should be able to cover, incl. support for 'legacy' EPC URIs, quantities, specific unit codes, ILMD data (even though the latter is likely to be discouraged/deprecated in the upcoming future), hierarchical data structures in extensions, code values, etc. So, I think that developing the corresponding JSON schema for this is a valuable and important exercise.
Event Profile 'Fishing Event'
Set of rules

The EPCIS Event is an ObjectEvent.
The action field is populated with the value "ADD".
The bizStep field is populated with the value "commissioning".
The readPoint is populated with an SGLN EPC URI (e.g. "urn:epc:id:sgln:4012345.00001.0")
The quantityList includes exactly one ID.
The ID in the quantityList is an LGTIN, expressed as an EPC Class URI (e.g. "urn:epc:class:lgtin:4012345.099914.LOT1").
The quantity field as part of the QuantityElement must be populated with a positive decimal value.
The uom field as part of the QuantityElement must be populated with a UN/CEFACT Rec 20 value for mass (FYC, see https://github.com/gs1/UnitConverterUNECERec20/blob/master/src/UnitConverterUNECERec20.js - type "mass"), e.g. "KGM"
The event contains an ilmd field.
All ILMD field names are prefixed with "cbvmda".
The event contains a proper declaration for "cbvmda" ("urn:epcglobal:cbv:mda").
Within the ILMD element, there is a cbvmda:bestBeforeDate field.
The cbvmda:bestBeforeDate field is filled with a proper date value.
Within the ILMD element, there is a cbvmda:storageStateCode field.
The cbvmda:storageStateCode field is filled with a corresponding code value according to https://navigator.gs1.org/gdsn/class-details?name=StorageStateCode&version=12 (e.g. "PREVIOUSLY_FROZEN")
Within the ILMD element, there is a cbvmda:vesselCatchInformationList field.
Within the cbvmda:vesselCatchInformationList field, there are two sub-fields: cbvmda:vesselID and cbvmda:vesselFlagState.
The cbvmda:vesselID field is filled with a string value.
The cbvmda:vesselFlagState field is filled with a ISO 3166-1 alpha-2 code value (e.g. "DE").

Are all of the above rules understandable/make sense for you?
Very much looking forward to seeing the corresponding JSON schema come alive - many thanks in advance for your support in this!
Kind regards;
Ralph
@dakbhavesh: FYI.

Dear @RalphTro
Just wanted to update you that @albert-verdeny and I are in contact and started to develop the JSON Schema based on your detailed description. We will keep you posted on the same and request for your feedback as we have the draft version of the same.
Best regards,
Aravinda

Dear @Aravinda93  and @albert-verdeny - MANY THANKS!

Dear @RalphTro
I have one small confusion associated with point 16 of your description. It has been mentioned as the field cbvmda:vesselCatchInformationList so should this be of "type": "array" or "type": "object"?
Because the field key contains list I thought it should be array but the later points 17, 18 and 19 of the description do not seem to support list but rather as object so thought of clarifying the same.
Can you please confirm if cbvmda:vesselCatchInformationList is array or object? As per the description in (CBV Standard doc)[https://ref.gs1.org/standards/cbv/#page=111] it should be a List
Example 1 with "type": "array":
"cbvmda:vesselCatchInformationList": [
  {
    "cbvmda:vesselFlagState": "DE",
    "cbvmda:vesselID": "1234"
  }
]

Example 2 with "type": "object":
"cbvmda:vesselCatchInformationList": {
  "cbvmda:vesselFlagState": "DE",
  "cbvmda:vesselID": "1234"
}

Dear @Aravinda93 ,
Good question. I just double-checked CBV section 9.2.3, and given that the CBV states "The value of vesselCatchInformationList consists of one or more elements named vesselCatchInformation, which contains the following subelements (...)".
Even though implementations I am aware of did not repeat fields such as e.g. vesselID etc., I think the CBV specification (at this time, we only had an XML mapping) was intended to cater for situations to indicate several ones. So, if we apply/map this to the JSON/JSON-LD world, I think it should be an array of objects; i.e. option 1.
WDYT?
Kind regards;
Ralph

@RalphTro
Thanks a lot for the quick clarification. Yes even I thought it should be an option-1 with "type": "array" but just wanted to double-check. I will proceed with option1 in JSON Schema.

 I think this happened becase I was looking at an old XML event when drafting this. ;-)
So, please disregard this requirement.

Dear @RalphTro
Surely we can provide you with the first draft of the JSON profile before Friday. It has gone through initial testing from @albert-verdeny and my side. There are some minor changes and final validation needs to be done from our end as soon as it's done we will provide you with the same.

Super. Thanks for the heads-up and looking forward to testing it!

Dear @RalphTro,
Another point I would like to clarify: in your requirement

Within the cbvmda:vesselCatchInformationList field, there are two sub-fields: cbvmda:vesselID and cbvmda:vesselFlagState.

I understand the two fields are required, correct?
Now, in the comments above we see that cbvmda:vesselCatchInformationList needs to be an array.
Do we want to ensure that there is at least one element on this array or any similar condition?
If not, this means cbvmda:vesselCatchInformationList can then technically be empty, so the event wouldn't necessarily have cbvmda:vesselID and cbvmda:vesselFlagState.

Dear @albert-verdeny ,
Thanks for your follow-up questions.
As to "I understand the two fields are required, correct?" --> Correct. Both fields need to be present. (For instance, to ensure that an event contains data that is legally required.)
As to "Do we want to ensure that there is at least one element on this array or any similar condition?" --> I would like to see that the schema validates/ensures that both sub-fields are present, non-empty, and filled with a proper value (i.e. string + ISO 3166-1 alpha-2 code).
Does this answer your question?

Dear @RalphTro,
Thanks for the reply and apologies I am not clear on the second point yet.
I understand that the attribute cbvmda:vesselCatchInformationList must be an array of items cbvmda:VesselCatchInformation.
I also understand that the items cbvmda:VesselCatchInformation must contain cbvmda:vesselID and cbvmda:vesselFlagState, that's clear now, thanks.
What I'm still unsure about: is there any restriction in the number of items that the array cbvmda:vesselCatchInformationList contains? Should we enforce at least one element in the list?

Basically I want to avoid the situation where we have an event with "cbvmda:vesselCatchInformationList": [] that passes the JSON schema – of course provided that this is something we want to fail.

Ah, I see now what you meant. Yes, I think avoiding an empty array as you pointed out is a good idea/helpful furter quality check! In this case, defining that this property needs to contain at least one sub-property would do the trick from my POV.

Dear @RalphTro
@albert-verdeny  and I have created the initial version of the JSON Schema based on your requirements.
Please find the attached schema file and code for your review. We look forward to your feedback and any observations you may have.
Event Profile Issue#7 Schema.json
Example

JSON Schema
{
  "$schema": "http://json-schema.org/draft-07/schema#",
  "allOf": [
    {
      "$ref": "https://ref.gs1.org/standards/epcis/epcis-json-schema.json"
    },
    {
      "type": "object",
      "properties": {
        "@context": {
          "type": "array",
          "items": {
            "type": "string",
            "const": "https://ref.gs1.org/standards/epcis/2.0.0/epcis-context.jsonld"
          },
          "minItems": 1,
          "maxItems": 1
        },
        "type": {
          "type": "string",
          "enum": ["ObjectEvent"]
        },
        "action": {
          "type": "string",
          "enum": ["ADD"]
        },
        "bizStep": {
          "type": "string",
          "enum": ["commissioning"]
        },
        "readPoint": {
          "type": "object",
          "properties": {
            "id": {
               "$ref": "#/definitions/readPointId"
            }
          },
          "required": ["id"]
        },
        "quantityList": {
          "type": "array",
          "items": {
            "$ref": "#/definitions/quantityElement"
          },
          "minItems": 1,
          "maxItems": 1
        },
        "ilmd": {
          "type": "object",
          "properties": {
            "cbvmda:bestBeforeDate": {
              "type": "string",
              "format": "date"
            },
            "cbvmda:storageStateCode": {
              "type": "string",
              "enum": ["NOT_PREVIOUSLY_FROZEN", "PREVIOUSLY_FROZEN"]
            },
            "cbvmda:vesselCatchInformationList": {
              "type": "array",
              "items": {
                "type": "object",
                "properties": {
                  "cbvmda:vesselID": {
                    "type": "string"
                  },
                  "cbvmda:vesselFlagState": {
                    "$ref": "#/definitions/vesselFlagStateEnum",
                  }
                },
                "required": [
                  "cbvmda:vesselID",
                  "cbvmda:vesselFlagState"
                ],
                "additionalProperties": false
              },
              "minItems": 1
            }
          },
          "patternProperties": {
            "^cbvmda:": {}
          },
          "required":[
            "cbvmda:bestBeforeDate",
            "cbvmda:storageStateCode",
            "cbvmda:vesselCatchInformationList"
          ],
          "additionalProperties": false
        }
      },
      "required": [
        "@context",
        "type",
        "action",
        "bizStep",
        "readPoint",
        "quantityList",
        "ilmd"
      ],
      "additionalProperties": true
    }
  ],
  "definitions": {
    "readPointId":{
      "type": "string",
      "format": "uri",
      "pattern":"^urn:epc:id:sgln:((\\d{6}\\.\\d{6})|(\\d{7}\\.\\d{5})|(\\d{8}\\.\\d{4})|(\\d{9}\\.\\d{3})|(\\d{10}\\.\\d{2})|(\\d{11}\\.\\d{1})|(\\d{12}\\.))\\.(\\%2[125-9A-Fa-f]|\\%3[0-9A-Fa-f]|\\%4[1-9A-Fa-f]|\\%5[0-9AaFf]|\\%6[1-9A-Fa-f]|\\%7[0-9Aa]|[!\\')(*+,.0-9:;=A-Za-z_-]){1,20}$"
    },
    "quantityElement": {
      "type": "object",
      "properties": {
        "epcClass": {
          "$ref": "#/definitions/quantityElementURI"
        },
        "quantity": {
          "$ref": "#/definitions/quantityElementDecimal"
        },
        "uom": {
          "$ref": "#/definitions/quantityUOM"
        }
      },
      "required": [
        "epcClass",
        "quantity",
        "uom"
      ],
      "additionalProperties": false
    },
    "quantityElementURI": {
      "type": "string",
      "format": "uri",
      "pattern": "^urn:epc:class:lgtin:(([\\d]{6}\\.[\\d]{7})|([\\d]{7}\\.[\\d]{6})|([\\d]{8}\\.[\\d]{5})|([\\d]{9}\\.[\\d]{4})|([\\d]{10}\\.[\\d]{3})|([\\d]{11}\\.[\\d]{2})|([\\d]{12}\\.[\\d]{1}))\\.(\\%2[125-9A-Fa-f]|\\%3[0-9A-Fa-f]|\\%4[1-9A-Fa-f]|\\%5[0-9AaFf]|\\%6[1-9A-Fa-f]|\\%7[0-9Aa]|[!\\')(*+,.0-9:;=A-Za-z_-]){1,20}$"
    },
    "quantityElementDecimal": {
      "type": "number",
      "exclusiveMinimum": 0
    },
    "quantityUOM": {
      "type": "string",
      "enum": ["KGM", "GRM", "ONZ", "LBR", "MGM", "MC", "STI", "TNE"]
    },
    "vesselFlagStateEnum": {
      "type": "string",
      "enum": [
        "AD", "AE", "AF", "AG", "AI", "AL", "AM", "AO", "AQ", "AR", "AS", "AT", "AU", "AW", "AX", "AZ",
        "BA", "BB", "BD", "BE", "BF", "BG", "BH", "BI", "BJ", "BL", "BM", "BN", "BO", "BQ", "BR", "BS", "BT", "BV", "BW", "BY", "BZ",
        "CA", "CC", "CD", "CF", "CG", "CH", "CI", "CK", "CL", "CM", "CN", "CO", "CR", "CU", "CV", "CW", "CX", "CY", "CZ",
        "DE", "DJ", "DK", "DM", "DO", "DZ",
        "EC", "EE", "EG", "EH", "ER", "ES", "ET",
        "FI", "FJ", "FK", "FM", "FO", "FR",
        "GA", "GB", "GD", "GE", "GF", "GG", "GH", "GI", "GL", "GM", "GN", "GP", "GQ", "GR", "GT", "GU", "GW", "GY",
        "HK", "HM", "HN", "HR", "HT", "HU",
        "ID", "IE", "IL", "IM", "IN", "IO", "IQ", "IR", "IS", "IT",
        "JE", "JM", "JO", "JP",
        "KE", "KG", "KH", "KI", "KM", "KN", "KP", "KR", "KW", "KY", "KZ",
        "LA", "LB", "LC", "LI", "LK", "LR", "LS", "LT", "LU", "LV", "LY",
        "MA", "MC", "MD", "ME", "MF", "MG", "MH", "MK", "ML", "MM", "MN", "MO", "MP", "MQ", "MR", "MS", "MT", "MU", "MV", "MW", "MX", "MY", "MZ",
        "NA", "NC", "NE", "NF", "NG", "NI", "NL", "NO", "NP", "NR", "NU", "NZ",
        "OM",
        "PA", "PE", "PF", "PG", "PH", "PK", "PL", "PM", "PN", "PR", "PT", "PW", "PY",
        "QA",
        "RE", "RO", "RS", "RU", "RW",
        "SA", "SB", "SC", "SD", "SE", "SG", "SH", "SI", "SJ", "SK", "SL", "SM", "SN", "SO", "SR", "SS", "ST", "SV", "SX", "SY", "SZ",
        "TC", "TD", "TF", "TG", "TH", "TJ", "TK", "TL", "TM", "TN", "TO", "TR", "TT", "TV", "TW", "TZ",
        "UA", "UG", "UM", "US", "UY", "UZ",
        "VA", "VC", "VE", "VG", "VI", "VN", "VU",
        "WF", "WS",
        "YE", "YT",
        "ZA", "ZM", "ZW"
      ]
    }
  }
}

Dear @Aravinda93 and @albert-verdeny ,
MANY THANKS! And congratulations: happy to report that my first tests were successful. Great work!
One observation: Initially, I tried an EPCIS document (not just the event itself), and the validation failed. So, the interesting question is: do we also want to support our tool to be able to validate EPCIS documents embedding one or several events? If the answer is yes (which I am inclined to advocate), the JSON schema needs to be tweaked a bit so that a document is not invalidated from my POV.
WDYT?
FYC, I attached both of my test files.
EPCISDocWithFishingEvent_valid.json
EPCISFishingEvent_valid.json
Kind regards;
Ralph
PS: In case you are wondering about the file extension: GitHub does not seem to allow uploading JSON-LD files. ;-)

Dear @RalphTro
Thanks a lot for the quick testing and feedback.
Yes, we have currently taken into consideration only the individual event but we will make the changes to the schema so it can be validated by both the event and the document.

Dear @RalphTro, The intention of this exercise is to learn about real profiles from users. So at this stage, I feel it is okay to keep the schema validating the individual events.
We are yet to discuss the API support we want to provide. Therefore, it will be good to come up with ideas at that time whether we want to support the whole document individual event, or both.
My suggestion is to defer schema supporting the EPCIS document (with multiple events) for the time being and keep it limited to event JSON/JSON-LD only.
What do you think?

Dear @dakbhavesh ,
Thanks for sharing your thoughts. My spontaneous reaction: if we want our tool to be compatible with exising ones such as https://ref.gs1.org/tools/epcis-sandbox/create-event-data, I think it's almost inevitable to support EPCIS documents as e.g. the output of the 'Event Data Generator' is an EPCIS document.
So, how do you like the following idea/solution approach: our library/tool always comes with some kind of 'boilerplate' which inherently supports EPCIS documents (i.e. at least does not invalidate an event if it is embedded in an EPCIS document)?
Happy to discuss/reach consensus on this subject in the group.

Dear @dakbhavesh  and @sboeckelmann ,
Got your point. Agreed - let's table EPCIS documents for the time being.


cargo check no longer shows warnings unconditionally

In the past, cargo check would always show every warning that a cargo clean && cargo build would show, even if there were no errors and I'd already run cargo check. For example, if I had an unused variable, cargo check would print something like this every time I ran it, regardless of file timestamps or build artifacts:
$ cargo check
   Compiling asht v0.1.0 ([redacted])
warning: unused variable: `a`
 --> src/main.rs:2:9
  |
2 |     let a = 3;
  |         ^ help: consider using `_a` instead
  |
  = note: #[warn(unused_variables)] on by default

    Finished dev [unoptimized + debuginfo] target(s) in 0.21 secs

Recently the behavior changed, and now cargo check only shows warnings if I touch the source code or delete the build artifacts. Otherwise, it just acts the same as cargo build does when the build is up to date:
$ cargo check
    Finished dev [unoptimized + debuginfo] target(s) in 0.12 secs

The first nightly release with the new behavior is nightly-2018-04-15. The change seems to have been caused by rustc, and a bisect of the rust-lang/rust repo showed commit 2a5f3ee0c5b5 as the culprit. I'm not familiar with the Rust build system, but I did notice that for a binary crate, deleting any of the following files will make cargo check print warnings the next time I run it:

target/debug/lib{name}.rmeta
target/debug/deps/lib{name}-{id}.rmeta
target/debug/.fingerprint/{name}-{id}/bin-{name}-{id}
target/debug/.fingerprint/{name}-{id}/dep-bin-{name}-{id}

I don't know if this is technically a bug report or a feature request. Let me know if I should file it against rust instead of Cargo.
versions
The last Rust release with the old behavior is nightly-2018-04-12, which has Cargo version 1.26.0-nightly (b70ab13b3 2018-04-04).
The first Rust release with the new behavior is nightly-2018-04-15, which has Cargo version 1.26.0-nightly (008c36908 2018-04-13).
I got both of those releases through rustup. The specific version of Cargo doesn't seem to matter, though, since (as I mentioned) it was a rustc change that caused the behavior change.

I believe this is a consequence of rust-lang/rust#49289 which should unify the behavior of libraries and binaries

My apologies, I forgot to include closed issues when I looked for existing issues about this, so I missed the prior discussions linked from #5236. You can close this one as a dupe if you want.

Ok and no worries! I'll go ahead and close this


Changes to request parser to accommodate spaces

This pr contains a major change to the parser to all for spaces in the url of an HttpUrlNode. This pr is a draft until further design considerations are established.

@bleaphar Can we abandon this PR based on the outcome of our discussion offline? I think our conclusion was that allowing whitespace in the grammar could be bug prone and that a better approach may be to introduce a refactoring in the IDE to encode URLs with whitespace correctly instead.


Avoid relying on bitflags internals

This will allow to use a bitflags 2-derived bitflags 1 implementation in mozilla-central.

Ironically, bitflags 1 makes clippy unhappy now.


use with Yarn's PnP mode

Yarn has an alternative modules layout system it calls Plug'n'Play.
For this to work in ESM mode, it uses a loader. However, node can only load one loader. In this case, first the pnp loader must be called to obtain the module source and then the esbuild loader.
There's node-loader-core which allows chaining loaders, but I wonder if there would be a more direct way possible?
There's an official esbuild plugin to make esbuild use pnp. Maybe that could be used?

actually another problem is that using PnP means that this module is not available directly on the filesystem. So this should really be implemented in the PnP loader itself or via a future proposal of ambient loaders.
Closing since there's nothing that can be done right now.


overhead in getPropertySamples with 'visual'

I recently realized that getPropertySamples is very slow (43s for 2k sample iterations) in the conversionReaction example.
The largest portion (29s) of time goes into the 21 calls of plotPropertyUncertainty which updates the legend and histogram bars each time. Indeed, it updates every 100 MCMC iterations as coded in line 119 of getPropertySamples:
    if (mod(j,100) == 0) || (j == length(properties.S.logPost))

I suggest to change it to 10%, 20%, ... of the iterations instead or removing the sequential drawing entirely.

Sounds good. Maybe add it as an option with 10% as default?

Agreed.
At some point one might include an adaptive strategy based on the expected computation time.

Ok, I changed the code to a percentage heuristic for now and gonna merge it into development in near future. An adaptive strategy sounds good. I'd suggest we implement it together with the modularization / expansion of the visularization tools as mentioned in issue #49


Do not follow into symlinked directories when traversing directories

Do not follow into symlinked directories when traversing directories
and :dont_follow and :recursive are in effect.

Thanks!


🐛 Bug |  cmd+z does not undo previous action.

Description
As a blogging platform user I would expect that with CMD+Z I would be able to undo the previous or few previous actions, but currently it is not working like that.
Device

OS: macOS Catalina
Chrome
Version 93.

Fixed in the new editor


Fixing NGL not updating state in sync with ContactMap

Codecov Report

Merging #44 into master will increase coverage by 0.17%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #44      +/-   ##
==========================================
+ Coverage   83.89%   84.07%   +0.17%
==========================================
  Files          22       23       +1
  Lines         913      923      +10
  Branches      123      112      -11
==========================================
+ Hits          766      776      +10
  Misses        146      146
  Partials        1        1

Impacted Files
Coverage Δ

src/helper/NGLHelper.ts
100% <100%> (ø)

src/component/NGLComponent.tsx
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Generate default .rules.verible_lint from verible-verilog-lint

Summary
Generating a complete '.rules.verible_lint' file is a bit tedious to do by hand. It would be better to be able to get a default  '.rules.verible_lint' file out of the lint tool as a starting point.
Test cases and examples
Execute 'verible-verilog-lint --print_rules_file' to print the default rules to stdout. Users can then pipe/copy&paste the output to a file for latter use ('.rules.verible_lint' for example).
Example output:
~/git/verible/bazel-bin/verilog/tools/lint$ ./verible-verilog-lint --print_rules_file
+always-comb
+always-comb-blocking
+always-ff-non-blocking=catch_modifying_assignments:false;waive_for_locals:false
-banned-declared-name-patterns
+case-missing-default
+constraint-name-style
+create-object-name-match
-disable-statement
-endif-comment
...
Proposal
I've implemented the above in a fork, mainly to see if I could implement the feature, with the view to start a pull request. Before I do is there is any appetite for such a feature?
Additional context
None

I like that, please send a pull request!

Closing, thanks for the PR!


Defaults for GOMAXPROCS

/cc @cyx

LGTM. It seems a bit conservative, but I'm not sure about good tuning defaults for it either.

Note this is likely to change in Go 1.5 http://golang.org/s/go15gomaxprocs

@kr it will, but I don't think you'd actually want to set GOMAXPROCS to the cpu count for 1x / 2x dynos and the other types are == to the cpu counts of the underlying systems.

I will also likely modulate this based on actual tests and feedback at some point in the future, but this is likely better than the default of 1 (or the new 1.5 defaults) on Heroku.


Get guides made for Competitors

I'd like to have guides made for how Airflow differs from

[x] Oozie
[ ] Luigi
[ ] Askaban

Already have one for AWS Glue.

One Down: https://www.astronomer.io/guides/airflow-vs-oozie/

Got these made- not gonna bother with Askaban because I can't remember a single prospect that has considered using it...


[Wait for #2566] [ Layers ] Update Layers to support FP16

In this PR
This PR enables the FP16 support for the layers below:
. input layer
. mse loss layer
Resolves:
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2567. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @jijoongmoon, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2567-202405071338460.26453304290771-f7c1785523c684874bce9a45ac1186ac39f3572d/.


Drop or vendor dj_database_url

Hi, I was just searched for django pydantic settings and found this.
The reason I searched was that many django projects reading environment variables are not maintained or did not release the changes.
dj_database_url is version 0.5 and renders wrong db engine. postgresql_psycopg2 jacobian/dj-database-url#131 - which according to comments had been fixed in the master but not released.
I abandoned dj_database_url for django-environ which I found out suffering from the same problem (not releasing) joke2k/django-environ#282

Huh, I'll have to take a look at that. I'm using postgis for basically everything, which is probably why I hadn't caught that.


[fix] rework tera manifest evaluation

only pipe manifest through tera, when it will be executed -> after checking labels -> after checking where

I'm submitting a

[x] bug fix
[ ] feature
[ ] documentation addition

What is the current behaviour?
Closes #283.

Codecov Report
Base: 46.64% // Head: 46.64% // No change to project coverage :thumbsup:

Coverage data is based on head (d76f6cb) compared to base (7e6eace).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #286   +/-   ##
=======================================
  Coverage   46.64%   46.64%
=======================================
  Files          79       79
  Lines        2742     2742
=======================================
  Hits         1279     1279
  Misses       1463     1463

Impacted Files
Coverage Δ

app/src/commands/apply.rs
0.00% <0.00%> (ø)

lib/src/manifests/load.rs
0.00% <0.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

The solution works, but we have to reconsider piping everything through tera.

Labels and where conditions can use templating. So this likely would cause some problems?

Nah, the main problem is the way I solved the problem:

First I deserialize the YAML files into the Manifest struct
then serialize it back to YAML to do the tera templating
and then deserialize it back to the Manifest struct

Serde adds the defaults to the resulting YAML before piping it through tera, which resulted in some bad side effects, because we don't have a proper serialize_with - method e.g. here.
But let's talk on thursday about it :)


!arturdebil not working

Arturdebil does not work.
Bot has admin permissions
Cmd output:
Error [GUILD_MEMBERS_TIMEOUT]: Members didn't arrive in time.
at C:\Users\Maciek\Desktop\PossumBot-master\node_modules\discord.js\src\managers\GuildMemberManager.js:317:16
at Timeout._onTimeout (C:\Users\Maciek\Desktop\PossumBot-master\node_modules\discord.js\src\client\BaseClient.js:83:7)
at listOnTimeout (internal/timers.js:549:17)
at processTimers (internal/timers.js:492:7)
Error [GUILD_MEMBERS_TIMEOUT]: Members didn't arrive in time.
at C:\Users\Maciek\Desktop\PossumBot-master\node_modules\discord.js\src\managers\GuildMemberManager.js:317:16
at Timeout._onTimeout (C:\Users\Maciek\Desktop\PossumBot-master\node_modules\discord.js\src\client\BaseClient.js:83:7)
at listOnTimeout (internal/timers.js:549:17)
at processTimers (internal/timers.js:492:7)

@Kojfer13 do you have intents enabled?

@Barsik008 how do we enable intents?

@YilmazBrkT read the guide