Would be awesome if computations could be chained in a single command

    new_table = table.compute([
        ('rank2014', agate.Rank('value2014'),
        ('rank2015', agate.Rank('value2015')
    ])

    new_table2 = new_table.compute([
        ('rank_change', agate.Change('rank2014', 'rank2015')
    ])

could become:
    new_table = table.compute([
        ('rank2014', agate.Rank('value2014'),
        ('rank2015', agate.Rank('value2015'),
        ('rank_change', agate.Change('rank2014', 'rank2015')
    ])

But I can't see any way to do this without instantiating every intermediate table which would be sloooooow.

Nope, the implications of this are too messy. Not doing it.


SAK-44529 revert the change to set a new tool_consumer_instance_guid

+1 as this change will break existing installs if they didn't manually set that property.


137170159-Notify-integration

Notify integration:

Adds class DMNotifyClient with send_email
Adds notifications-python-client.git to requirements.txt
Standardise strings in tests/test_email.py
Add py.test fixtures in tests/test_email.py
dm_notify_client supplier initialised DMNotifyClient instance
dm_notify_example_success supplies an example send_email response
Tests

Questions for tomorrow:
Hashing email. How to get this back?
client_class a paradigm worth considering?
personalisation and making sure we can find a unique instance of an email.
Really; what should the reference be?

For posterity:

Hashing email. How to get this back?
Dont need to
client_class a paradigm worth considering?
Paradigm is fine but indicate the attribute is private with _
Really; what should the reference be?
Ref is email_address, template and context
notify.py or email.py?
Neither; subdir

Also: hash emails and reference.

Alright everyone, thanks for the comments, I've addressed everything and as discussed with @allait on Friday I'm going to go ahead and merge. 👍


Masthead: L0 dropdown first item focus state

The focus state of the first item in the L0 dropdowns is only showing 1px of the 2px border on the top side . Is it possible to fix this? I looks like the rule at the bottom of the L0 is on top of the focus state.

@RobertaJHahn Does this need to be closed? It looks like it got solved?

@photodow Thanks for noticing.  Closing.


Update attributes.md to add Examples

Update attributes.md to add the Analog Chip Example:
https://wokwi.com/projects/330112801381024338

Thanks!


Todoist integration broken by 1.0.4

In version 1.0.4 the button isn't shown in the Todoist web app anymore.
Reported in case 335164, checked and verified.

Re-enabling the integration form Settings permissions tab should fix this issue. I've answered the user as well.


test this please

chore(filters): change the development API version

Build FAILED. See http://<IP_ADDRESS>:8080/job/pull-request/1/ for more details.

Build FAILED. See http://<IP_ADDRESS>:8080/job/pull-request/2/ for more details.


polybar-msgand pacman hook

Checklist

[X] I have read the appropriate section in the contributing guidelines
[X] I believe this issue is a problem with polybar itself and not a misconfiguration on my part
[X] I have searched for other open and closed issues that may have already reported this problem
[X] I have checked the known issues page for this problem.
[X] I have followed the debugging guide to narrow down the problem to a minimal config.

Steps to reproduce

Two bars are launched through a launch.shscript and its pids are saved to /tmp/polybar.pidsfor later use:

#!/bin/bash

# Terminate already running bar instances
killall -q polybar
rm /tmp/polybar.pids

# Wait until the processes have been shut down
while pgrep -u $UID -x polybar >/dev/null; do sleep 1; done

# Launch Polybar, using default config location ~/.config/polybar/config
polybar -r mybar &
echo "$!" >> /tmp/polybar.pids
polybar -r tray &
echo "$!" >> /tmp/polybar.pids

sleep 2

polybar-msg -p $! cmd hide

echo "Polybar launched..."

Polybar launches successfully and all modules work correctly, including updates-arch-combined (shown below). I am able to manually run

polybar-msg action updates-arch-combined reset

and
polybar-msg -p $pid action updates-arch-combined reset

with the correct $pid specified.

A Pacman hook is created to automatically reset said module after an update:

[Trigger]
Operation=Install
Operation=Upgrade
Operation=Remove
Type=Package
Target=*

[Action]
Description=Change update module on Polybar
Depends=polybar
When=PostTransaction
Exec=sh -c '/home/nicolas/.config/polybar/scripts/updates_hook.sh'

which calls the following script:
#!/bin/bash
input="/tmp/polybar.pids"
pid=$(head -n 1 "$input")
polybar-msg -p $pid action updates-arch-combined reset

Can confirm that it works when manually executed.

Run a Pacman update:

sudo pacman -Syu

Minimal config
[bar/mybar]
monitor = ${env:MONITOR:HDMI-0}
width = 100%
height = 27
radius = 0.0
fixed-center = true

background = ${colors.background}
foreground = ${colors.foreground}

line-size = 3
line-color = #f00

border-size = 0
border-color = #00000000

padding-left = 0
padding-right = 0

module-margin-left = 0
module-margin-right = 1

font-0 = "Deja Vu Sans:fontformat=book:size=11;1"
font-1 = "Jetbrains Mono Nerd Font:size=12;1"

modules-right = updates-arch-combined

cursor-click = pointer
cursor-scroll = ns-resize

enable-ipc = true

[module/updates-arch-combined]
type = custom/ipc
hook-0 = ~/.config/polybar/scripts/updates-arch-combined.sh
initial = 1
interval = 600
click-left = ~/.config/polybar/scripts/updates-arch-combined.sh
format-underline = ${colors.secondary}

Polybar log
No response
Expected behavior
The Pacman hook should run successfully. It worked on previous version of Polybar, although using
polybar-msg -p $pid hook updates-arch-combined 1

instead of
polybar-msg -p $pid action updates-arch-combined reset

Actual behavior
Instead Pacman returns the following error:
(4/5) Change update module on Polybar
polybar-msg: No channel available for pid 788
error: command failed to execute correctly

Window Manager and Version
i3-gaps 4.20.1
Linux Distribution
Arrch
Polybar version
polybar 3.6.0

Features: +alsa +curl +i3 +mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) +composite +xkb +xrm +xcursor

Build type: Release
Compiler: /usr/bin/c++
Compiler flags: -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions         -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security         -fstack-clash-protection -fcf-protection -Wp,-D_GLIBCXX_ASSERTIONS -O3 -DNDEBUG -Wall -Wextra -Wpedantic -Wsuggest-override
Linker flags: -Wall -Wextra -Wpedantic -Wsuggest-override  -Wall -Wextra -Wpedantic -Wsuggest-override

Additional Context / Screenshots
No response

Since version 3.6, polybar-msg has to run under the same user as the target bar instance. Under which user does the pacman hook run?

Oh. I was thinking it might be a user related problem, yes. Pacman is executed with sudoso I guess the hook would be running as root. Can confirm that the behavior is the same when executing directly polybar-msgas root.
I've been trying to find a way to execute pacman hooks as user instead of root, with no luck so far. I guess this might not be the right place to ask, though.
I've tried, for example, prepending sudo -u $(logname), but I keep getting the same error.

Not completely the wrong place to ask. polybar-msg (and polybar itself) use $XDG_RUNTIME_DIR to look for the IPC socket. So when you run polybar-msg, it needs to have this variable set correctly.
Apparently machinectl can help here: https://unix.stackexchange.com/questions/346841/why-does-sudo-i-not-set-xdg-runtime-dir-for-the-target-user

I'll look into it. Thank you very much.

Is it possible to run just one command via machinectl method? It also asks for password, I'm not even sure if it could be used in scripts.

@pvonmoradi This works for me:
machinectl shell "patrick96@" /usr/bin/bash -c "echo hello world"

It also asks for password, I'm not even sure if it could be used in scripts.

You really only need machinectl when you want to run polybar-msg as a different user. Otherwise, you can directly use polybar-msg. If you want to run polybar-msg as a different user, you have to run machinectl as root, in which case it shouldn't ask for a password.

@patrick96 Thanks.
That @ is really needed. Also the binaries in front of bash -c should be in full path.
In machinectl manpages it is written:

Note that systemd-run(1) with its --machine=
switch may be used in place of the machinectl
shell command, and allows non-interactive
operation, more detailed and low-level
configuration of the invoked unit, as well as
access to runtime and exit code/status
information of the invoked shell process. In
particular, use systemd-run's --wait switch to
propagate exit status information of the
invoked process. Use systemd-run's --pty switch
for acquiring an interactive shell, similar to
machinectl shell. In general, systemd-run is
preferable for scripting purposes. However,
note that systemd-run might require higher
privileges than machinectl shell.

Does it mean systemd-run is another method in this problem?

Also the binaries in front of bash -c should be in full path.

Only if there is no PATH set. Everything inside of the quotes after bash -c is resolved according to your $PATH variable, so absolute paths generally aren't needed.

I wonder if it means systemd-run is another method in this problem...

Sounds like it. Best to try it.

Exec = /usr/bin/sudo -u nikmosi /bin/zsh -c "echo hook:module/updatesc-trigger1 > /tmp/polybar_mqueue* || true"
1 is hook-id in updatesc-trigger1


Add exercise: two-fer

Add the Two-Fer practice exercise.

@simisc Thanks! I've added @pclausen as a reviewer

Just to weigh in:

This passes the CI [1]
The "quality" of the example file doesn't really matter as it's a practice exercise
The tests look really clean

So let's give @pclausen a day or two to reply, but I'm in favour of taking a relatively optimistic merging approach to this if he doesn't get round to it! :)
Thanks @simisc!

[1]

Absolutely. But I did want to give @pclausen the chance to review :)

Thanks!


macro/definition preview

What do you want to see added to the extension?
Preview of macro/definition replacements.
Screenshots (taken from clion):

How would it help you?
Сurrent version gives little useful information about what I did wrong:

How often would you use this feature?
Always, I'm a macro lover.

This is definitely on my radar! The c/cpp extension does that as well in VSCode.
The information is already there, it just has to be implemented.
This will be added in https://github.com/Sarrus1/sourcepawn-vscode/issues/362.

A sneak peek of what I was able to do in the feat/salsa branch.


Allow setting ListenerInvocationErrorHandler for SagaTestFixture

Either I am missing something, or there is no way to configure a ListenerInvocationErrorHandler for the saga manager used by SagaTestFixture. The construction of the saga manager is hardcoded, and I have not been able to find a way to override it.
            sagaManager = AnnotatedSagaManager.<T>builder()
                    .sagaRepository(sagaRepository)
                    .sagaType(sagaType)
                    .parameterResolverFactory(parameterResolverFactory)
                    .handlerDefinition(handlerDefinition)
                    .build();

The default is nowLoggingErrorHandler, which results in any handler exceptions being ignored our tests. This is inconvenient because, for example, assertions like expectNoDispatchedCommands now sometimes pass because an exception occurred in a saga event handler

Good suggestion @martijnvanderwoud, I resolved this quickly in commit 7806af7, which will make it part of 4.3.

Great, thanks Steven!


Give the SDK packages a little love

We should not put code into index.js. Node uses the index.js file to look up files in a folder.
The "index.js" file has the purpose to expose & prepare an interface for a folder/unit.
Furthermore, all the code lives mostly in one file (in the index.js). Would be great to split up little units.
Both enhancements will help other (SDK) developers to follow & understand the code.
This has lower priority for now, but this is a perfect post cleanup task 👍

Closing as duplicate of https://github.com/TryGhost/Ghost-SDK/issues/74


kayvee request logging

Here's a prototype of node request middleware, extending morgan. We can decide whether to continue to extend morgan or remove all unnnecessary parts, later.
To add this middleware, currently it is invoked via app.use(morgan('clever'))

Start the express server
npm install
node server.js

Calling curl "localhost:3000/hello/world?a=1&b=2" results in this log being produced:
{"method":"GET","path":"/hello/world","params":"?a=1&b=2","response-size":"13","response-time":792955,"status-code":200}
```

Sorry, didn't mean to open against upstream. thanks!


Code of Conduct

Let's include a code of conduct that we can link to from other repos and use as a social contract across our work.

Incredible background, thanks @eth-r!


Help with installation through termux on Android.

Luckily, I have an android. And termux is available to me. However, when I try to install the git package it completely fails. It is unable to locate the package "git". I am assuming I downloaded the wrong version maybe?
Options for Termux installation:
Termux
Termux:Styling
Termux:Float
Termux:API
Termux:Widget
Termux:Tasker
Termux:Boot
I don't know which one is the unrooted or even how to unroot termux if thats the case and I have to. I've never worked with this application before and would like further help.
$ pkg install git
Checking availability of current mirror: ok
Reading package lists... Done
Building dependency tree... Done
E: Unable to locate package git

"git" is available in Termux, but you have to update your package list by running apt update command first.  then you can run apt install git.


[Don't Merge] V5 performance wins

Removing inefficient array methods for vanilla loops, some blocks will need to get new comments since some of the operations are a bit alien - for a new contributor -
Notes

remove unneeded object creation
use loops instead of reduce, filter, forEach

Some jsperf links:

for vs foreach https://jsperf.com/for-vs-foreach/75

Nice to have

a way to actually test for perf improvement, while these should be safe changes, it would be great to have any kind of benchmarking in v5

Some of these changes are so small that I almost wonder if it is worth the hit in readability

I was skeptic but it's not THAT bad, maybe I'm just used to reading for loops

Reassignment mutation for variables like keys + i is clever, but it is definitely more prone to mistakes.

in most cases we could even NOT assign, it's just way more readable when using an alias.
For the loop indexes i I'm not that concerned, there are some nested loops later in the renderer but nothing too bad, and there doesn't seem to be intricate-async code or anything, it's easier to understand than I initially thought. I'd still love to add some meaningful comments to some critical blocks.
BTW The Montage example seems to be a good test case, I'm still researching for a way to compare the two versions.

It's not that for loops are hard to read, but there is objectively more noise involved within each expression (initial, condition, increment) as well as with referencing via bracket notation. Like I mentioned though, I definitely agree that performance is more important here.

I think threejs did the same in hot paths, using for loops, I would be fine with it.

@gsimone does the "Don't Merge" still apply? It would be good to merge this in considering that we want to rename reconciler.tsx. We can drop the latest commit on the v5 branch - dec14b23dfbe1b1d3d3265ea24fcfb2e7cb92c08 - then merge this in and then commit the rename.

Sure, @drcmda said it's ok to merge. How do we go about > drop the latest commit on the v5 branch - dec14b2 - then merge this in and then commit the rename ?

@gsimone try just renaming the file in this branch, it might be easier

I'll fix the conflicts tomorrow 👍

@gsimone I would try to fix it, but I don't have access to your branch :P
I believe this will work though since it will put your changes on top
git pull -r origin v5

Thanks @stephencorwin 🥇


Syscall tracing very very very slow

On current master, the following command takes a very long time:
❯ time cargo run --release --example syscalls -- ls
    Finished release [optimized] target(s) in 0.01s
     Running `target/release/examples/syscalls ls`
pid = 60232, pc = 7f33dc5e0c50: [execve], SyscallExit
pid = 60232, pc = 7f33dc5e5ccb: [brk], SyscallEnter
pid = 60232, pc = 7f33dc5e5ccb: [brk], SyscallExit
...
pid = 60232, pc = 7f33dbec4b06: Exiting { exit_code: 0 }

real	0m43.973s
user	0m0.054s
sys	0m0.049s

strace ls only takes 17 ms. Any idea what the difference is? I was under the impression that strace uses ptrace in the same way to Pete. So how is it 2500x faster?

First i don't know this project, i was looking at it and i seen your issue!
In src/ptracer.rs at 363 is:
const DEFAULT_POLL_DELAY: Duration = Duration::from_millis(100);

You can add this line after 36 in examples/syscalls.rs
*ptracer.poll_delay_mut() = std::time::Duration::ZERO;

The delay i think was added to be more readable!

Interesting. Surely there's a way to do this without polling?
Also btw @ranweiler you can slightly simplify the loop that does the poll because in Rust loops can return values.

@Timmmm, thanks for reporting, the default poll delay is definitely too high. I'll try to at least push a perf bugfix for that tonight, but as noted above, you can set it yourself in the meantime.
I'm experimenting with a few different strategies that would let the kernel notify us, instead of manual polling. There are some tradeoffs, assumptions about tracee behavior, and version-dependent Linux-isms at play here, so I'm trying to be very careful in how I implement it. I'll probably preserve the polling as a fallback mode, but that's part of what I'm trying to sort out. Please stand by!


feat: adjust event filters

And one more thing. Now that we have vector counts for stage as well, we could display it beneath the radio buttons.


InputDate component  can't be disabled

Current behaviour
InputDate  component can't be disabled when i put the property disabled=true.
it seems that this property is ignored.
Versions
Focus-core: 0.15
Focus-components: 0.15.6

PR https://github.com/KleeGroup/focus-components/pull/1214


Improve image pull check

How to categorize this PR?

/area testing
/kind enhancement
What this PR does / why we need it:
Improves image pull check with following:

extract the image manifest for Node architecture from the index file
extract all image layers from the manifest
ensure that all image layers data files exist in the registry storage

Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
N/A
Release note:

NONE

@dimitar-kostadinov can you rebase and force push so that the concourse-ci/helmcharts check gets removed on its own?

/test pull-gardener-extension-registry-cache-e2e-kind

/hold

/remove-lifecycle stale

/test pull-gardener-extension-registry-cache-e2e-kind

/unhold


[fluent2-tokens] Remove dynamic color collections from control token sets

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
Binary change:

File
Before
After
Delta

libFluentUI.a
32,361,536 bytes
32,311,512 bytes
-50,024 bytes

Removed ButtonDynamicColors and PillButtonDynamicColors from the possible control tokens, and update the controls using them.
Verification

Before
After

Pull request checklist
This PR has considered:

[x] Light and Dark appearances
[ ] iOS supported versions (all major versions greater than or equal current target deployment version)
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[ ] iPad Pointer interaction
[ ] SwiftUI consumption (validation or new demo scenarios needed)
[ ] Objective-C exposure (provide it only if needed)

Microsoft Reviewers: Open in CodeFlow

is the order of operation, fix main, merge main -> token branch, then apply this fix?

is the order of operation, fix main, merge main -> token branch, then apply this fix?

Either order should work. If I get this in, the merge from main should be easier, since the missing class wouldn't cause a break.


Mark your GPS location

Many apps put a blue dot at your GPS location. That should appear at the center of the search if the search is based on a GPS location. But users expect the blue dot to periodically update their geolocation.
First: Put some icon (flag?) at the spot where the search is centered but don't move it if the user moves.
Second: Make it the blue dot and periodically update.

Leaflet custom icon tutorial may be helpful here: http://leaflet.cloudmade.com/examples/custom-icons.html


add giphy-api

https://hackage.haskell.org/package/giphy-api-<IP_ADDRESS>

Looks like I need to relax the aeson bounds. Will reopen once that is done.


implement restrictions on edit template surveys

https://app.zenhub.com/workspaces/met-ops-6254279dcfa60b0010ea5a0d/issues/gh/bcgov/met-public/1367
Description of changes:

Added a new role to check user access while editing a template survey
Logic to restrict users without a role to edit template surveys

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #1496 (b5be614) into main (d1f7b82) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1496      +/-   ##
==========================================
+ Coverage   72.04%   72.09%   +0.05%
==========================================
  Files         331      331
  Lines        9711     9724      +13
  Branches      696      696
==========================================
+ Hits         6996     7011      +15
+ Misses       2598     2596       -2
  Partials      117      117

Flag
Coverage Δ

metapi
79.30% <100.00%> (+0.10%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

met-api/src/met_api/resources/survey.py
75.86% <100.00%> (+2.41%)
:arrow_up:

met-api/src/met_api/services/survey_service.py
73.78% <100.00%> (+2.50%)
:arrow_up:

met-api/src/met_api/utils/roles.py
100.00% <100.00%> (ø)


Plugin is not working

Hi, sorry for the bad title but not sure how to put it.
I'm using OS X 11.4 Big sur with iterm 2 and a patched font
I cannot get the plugin to work, it doesn't highlight and it doesn't show todos in trouble or telescope
Note: When I say todos I mean everything, hack, perf...

I have even created a branch on my config disabling every plugin that's not todo-comments
I'm using NVIM v0.5.0-dev+7da86f55a
 - INFO: key_backspace (kbs) terminfo entry: key_backspace=^H
 - INFO: key_dc (kdch1) terminfo entry: key_dc=\E[3~
 - INFO: $TERM_PROGRAM='iTerm.app'
 - INFO: $COLORTERM='truecolor'

The default regexes expect a colon after the keyword, as in
TODO: test

@kevinpita It was actually not working for me just now as well. I updated my neovim install and it's functioning fine again

The default regexes expect a colon after the keyword, as in
TODO: test

I'm so sorry didn't know it worked like that, thanks! awesome plugin


Lumen support?

Why not make it available for Lumen as well? Thanks.

This is now obsolete. You are better of using the built in Authentication and Authorization.


Github icons at the bottom of the page on website

I am trying to change the icons at the

We should also try to change the nighthawk logo at the top to our own logo.

ok I'll do that later when I have the time

We should also try to change the nighthawk logo at the top to our own logo.

done, check it out when you have the time


[BEAM-877] Allow an option to not flatten record types in BQ

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Add a flag to BQ source so that we can allow for record type to not be flattened
Change the BQReader so that DirectRunner parses the dictionaries correctly
Add tests to make sure that dict parsing is working correctly

Future work:

Write an integration test so that can check the cleanup of temporary datasets correctly, for now verified this manually

R: @chamikaramj PTAL

R:  @chamikaramj PTAL

Looks like this needs a rebase.

R @chamikaramj PTAL :)

R: @chamikaramj PTAL
With #1501 we've upgraded apitools so we can unblock this PR now.

R: @chamikaramj PTAL

LGTM. Thanks.

R: @robertwb for review and merge


[BoostDay] Notepad

Description
There should be a notepad for customers to save their favorite products to, so they have easy access to them and can quickly buy these products over and over again. The feature already existed in Shopware 5 and is really a basic in ecommerce that should be available in Shopware 6 as well. While we're at it, I propose we make the notepad actually good and not only existing.

A notepad should be available for everyone, not just logged in customers. So we need the possibility to link a notepad to a session or to a customer.
A user should be able to have multiple notepads, name them and move products between them.
Users should be able to make their notepad public or leave them private (default). Public notepads should be accessible with a link.
The price of a product should be saved for every entry on every notepad for that point in time. The idea is to be able to calculate price changes since the product was added to the notepad and display them to the user. ("Product is 10 % cheaper than it was when it was added to the list.")
A cronjob should send mails for discounted products on notepads. When a product's price is temporarily discounted, all users with this product in one of their notepads should receive a notification mail.

Example
Let's say a customer would like to save some products he really likes into a list of his favorites. Now let's say, this were to happen in a Shopware 6 shop. See the problem?

related existing ticket https://issues.shopware.com/issues/NEXT-4900

HI @jkrzefski
please do not forget, that the boostday is one-day only. Tasks should be somewhat completable within the 8 Hours of this day.
The mentioned issue is a little more complex and should be developed by our core developers.
Anyway as @mitelg mentioned there is already an existing ticket. If this ticket gets enough votes we'll consider implementing this functionality.
If you'd like to I would add your ideas to the ticket. This way people would bet a little more details. What do you think?
Sunny Regards
Pantrtxp

Hey @pantrtxp you underestimate what @jkrzefski is able to do within 8 hours 😬  For real this is a bigger task, yes. And maybe just the first more common parts from shopware 5 are a more realistic approach but 8 hours is just one worker. Just increase the amount of workers and you can make 8 hours to 80 hours 😉  That is how business administration works, doesn't it?

@JoshuaBehrens Just how nice women can deliver a baby in one month, right?

I believe, this is a feature request that can fairly easily be split into multiple parts. If we split the requirements into the basics and some advanced features, we would implement as much of it as we can in the given timespan and have the rest of it still open for future pull requests.
@pantrtxp But I can also understand, that you would rather have your core developers on this and have the feature thoroughly tested before releasing it. And for that, I would like it very much, if you add my suggestions to the existing ticket.

my suggestion is to implement the whole thing at once and do not implement the first parts or something like that.

While we're at it, I propose we make the notepad actually good and not only existing.

you said it by yourself :wink:

I have added your Ideas as a comment :)
If we implement this feature may be some of your wishes will remain open for another BoostDay.
If you are fine with my comment I'd like this issue to be closed since it is a duplicate
Sunny regards
Pantrtxp

@pantrtxp Looks good to me. Just one question: Is there a reason for the changes from notepad to wishlist in the feature list? Is there a difference between the two that I missed?

naming things... we are good at that :smile:

No, there is not, but since the ticket is called "Wishlist functionality missing" I thought I was keeping it consistent

Alright, I can live with that.


how to init a layer weights from before layer before model compile

I want to implement paper:CNN-RNN: A Unified Framework for Multi-label Image Classification https://arxiv.org/pdf/1604.04573v1.pdf.
The net work structure can simple understand by below figture

My code can be see below

So, the problem coming to me is that how to decoder sequence embed label to label index.
By the way , anyone knows better way to implement the net work.

Hi @yanyuxiaoxiao,
Did you happen to solve the problem?


Won't install on Windows

Python 3.6 (x86-64)
pip == 9.0.1
setuptools == 34.3.2
(no36) D:\Git\project>pip install --force --upgrade markupsafe
Collecting markupsafe
  Using cached MarkupSafe-1.0.tar.gz
Building wheels for collected packages: markupsafe
  Running setup.py bdist_wheel for markupsafe ... error
  Failed building wheel for markupsafe
  Running setup.py clean for markupsafe
Failed to build markupsafe
Installing collected packages: markupsafe
  Found existing installation: markupsafe 1.0
    Uninstalling markupsafe-1.0:
      Successfully uninstalled markupsafe-1.0
  Running setup.py install for markupsafe ... error
  Rolling back uninstall of markupsafe
Exception:
Traceback (most recent call last):
  File "d:\python\no36\lib\site-packages\pip\compat\__init__.py", line 73, in console_to_str
    return s.decode(sys.__stdout__.encoding)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe9 in position 66: invalid continuation byte

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "d:\python\no36\lib\site-packages\pip\basecommand.py", line 215, in main
    status = self.run(options, args)
  File "d:\python\no36\lib\site-packages\pip\commands\install.py", line 342, in run
    prefix=options.prefix_path,
  File "d:\python\no36\lib\site-packages\pip\req\req_set.py", line 784, in install
    **kwargs
  File "d:\python\no36\lib\site-packages\pip\req\req_install.py", line 878, in install
    spinner=spinner,
  File "d:\python\no36\lib\site-packages\pip\utils\__init__.py", line 676, in call_subprocess
    line = console_to_str(proc.stdout.readline())
  File "d:\python\no36\lib\site-packages\pip\compat\__init__.py", line 75, in console_to_str
    return s.decode('utf_8')
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe9 in position 66: invalid continuation byte

I have the same issue here, but since I was using MarkupSafe without issue on windows when I worked with Python 3.5, I assume the error comes from Python 3.6. Maybe the title of this issue should be updated ?

Actually, the problem seems to come from missing wheel for MarkupSafe on Pypi. Creating it manually solved the issue for me. Here is the workaround:
git clone git@github.com:pallets/markupsafe.git /tmp/markupsafe
cd /tmp/markupsafe
./setup.py bdist_wheel

The result is a valid wheel: /tmp/markupsafe/dist/MarkupSafe-1.0-cp36-cp36m-win_amd64.whl.
In your other project where you need to use it, simply call
pip install /tmp/markupsafe/dist/MarkupSafe-1.0-cp36-cp36m-win_amd64.whl

And you're done

Exception:
Traceback (most recent call last):
File "c:\users\ray\appdata\local\programs\python\python36\lib\site-packages\pip\compat_init_.py", line 76, in console_to_str
return s.decode(sys.stdout.encoding)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xb6 in position 33: invalid start byte
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "c:\users\ray\appdata\local\programs\python\python36\lib\site-packages\pip\basecommand.py", line 215, in main
status = self.run(options, args)
File "c:\users\ray\appdata\local\programs\python\python36\lib\site-packages\pip\commands\install.py", line 342, in run
prefix=options.prefix_path,
File "c:\users\ray\appdata\local\programs\python\python36\lib\site-packages\pip\req\req_set.py", line 784, in install
**kwargs
File "c:\users\ray\appdata\local\programs\python\python36\lib\site-packages\pip\req\req_install.py", line 878, in install
spinner=spinner,
File "c:\users\ray\appdata\local\programs\python\python36\lib\site-packages\pip\utils_init_.py", line 676, in call_subprocess
line = console_to_str(proc.stdout.readline())
File "c:\users\ray\appdata\local\programs\python\python36\lib\site-packages\pip\compat_init_.py", line 78, in console_to_str
return s.decode('utf_8')
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xb6 in position 33: invalid start byte

I met the same error with Python 3.6 on windows 10 and solved by @alorence 's solution.

I also meet this question, I found this is just a decode error. In my environment， the console's coding is 'gbk', and this  return s.decode('utf-8'). so it make error. when I modify the source code to s.decode('gbk'). It works.

I investigated using the git bisect command.
It was found that it was corrected by the following modified commit of pip.
https://github.com/pypa/pip/commit/8a101321f6aaa24dd796241221f81a92bf116e9f

In case someone do have my issue :
Traceback (most recent call last):   File "run.py", line 2, in <module>     from app import app   File "G:\Tommy\Programming\Websites\Python_Websites\flaskHW\app\__init__.py", line 1, in <module>     from flask import Flask   File "C:\Program Files\Python36\lib\site-packages\flask\__init__.py", line 19, in <module>     from jinja2 import Markup, escape   File "C:\Program Files\Python36\lib\site-packages\jinja2\__init__.py", line 33, in <module>     from jinja2.environment import Environment, Template   File "C:\Program Files\Python36\lib\site-packages\jinja2\environment.py", line 15, in <module>     from jinja2 import nodes   File "C:\Program Files\Python36\lib\site-packages\jinja2\nodes.py", line 19, in <module>     from jinja2.utils import Markup   File "C:\Program Files\Python36\lib\site-packages\jinja2\utils.py", line 647, in <module>     from markupsafe import Markup, escape, soft_unicode   File "C:\Program Files\Python36\lib\site-packages\markupsafe\__init__.py", line 14, in <module>     from markupsafe._compat import text_type, string_types, int_types, \ ModuleNotFoundError: No module named 'markupsafe._compat'
@alorence 's solution fixed it all. kudos to him for fixing an hours long issue

for those who have this issue i was able to solve this by typing chcp 65001(which change the caracter code) in cmd before pip install,
hope it will help

Here is a simple solution. Since it is a decode error, it can be solved by changing the console's coding to utf-8. Just run
chcp 65001
pip install markupsafe

and it works.

no. it's didn't help.
(myvenv) d:\work\Python\2018\hook\app>main.py
Traceback (most recent call last):
  File "D:\work\Python\2018\hook\app\main.py", line 1, in <module>
    from flask import Flask
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\flask\__init__.py", line 19, in <module>
    from jinja2 import Markup, escape
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\jinja2\__init__.py", line 33, in <module>
    from jinja2.environment import Environment, Template
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\jinja2\environment.py", line 15, in <module>
    from jinja2 import nodes
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\jinja2\nodes.py", line 19, in <module>
    from jinja2.utils import Markup
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\jinja2\utils.py", line 647, in <module>
    from markupsafe import Markup, escape, soft_unicode
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\markupsafe\__init__.py", line 14, in <module>
    from markupsafe._compat import text_type, string_types, int_types, \
ModuleNotFoundError: No module named 'markupsafe._compat'

(myvenv) d:\work\Python\2018\hook\app>chcp 65001
Active code page: 65001

(myvenv) d:\work\Python\2018\hook\app>pip install markupsafe
Requirement already satisfied: markupsafe in d:\work\python\2018\hook\myvenv\lib\site-packages

(myvenv) d:\work\Python\2018\hook\app>main.py
Traceback (most recent call last):
  File "D:\work\Python\2018\hook\app\main.py", line 1, in <module>
    from flask import Flask
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\flask\__init__.py", line 19, in <module>
    from jinja2 import Markup, escape
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\jinja2\__init__.py", line 33, in <module>
    from jinja2.environment import Environment, Template
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\jinja2\environment.py", line 15, in <module>
    from jinja2 import nodes
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\jinja2\nodes.py", line 19, in <module>
    from jinja2.utils import Markup
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\jinja2\utils.py", line 647, in <module>
    from markupsafe import Markup, escape, soft_unicode
  File "d:\work\Python\2018\hook\myvenv\lib\site-packages\markupsafe\__init__.py", line 14, in <module>
    from markupsafe._compat import text_type, string_types, int_types, \
ModuleNotFoundError: No module named 'markupsafe._compat'

did I do something wrong?

before this I had tried install flask:
(myvenv) d:\work\Python\2018\hook\app>pip install flask
Collecting flask
  Downloading Flask-0.12.2-py2.py3-none-any.whl (83kB)
    100% |████████████████████████████████| 92kB 287kB/s
Collecting Werkzeug>=0.7 (from flask)
  Downloading Werkzeug-0.14.1-py2.py3-none-any.whl (322kB)
    100% |████████████████████████████████| 327kB 613kB/s
Collecting itsdangerous>=0.21 (from flask)
  Downloading itsdangerous-0.24.tar.gz (46kB)
    100% |████████████████████████████████| 51kB 1.7MB/s
Collecting click>=2.0 (from flask)
  Downloading click-6.7-py2.py3-none-any.whl (71kB)
    100% |████████████████████████████████| 71kB 1.8MB/s
Collecting Jinja2>=2.4 (from flask)
  Downloading Jinja2-2.10-py2.py3-none-any.whl (126kB)
    100% |████████████████████████████████| 133kB 1.2MB/s
Collecting MarkupSafe>=0.23 (from Jinja2>=2.4->flask)
  Downloading MarkupSafe-1.0.tar.gz
Installing collected packages: Werkzeug, itsdangerous, click, MarkupSafe, Jinja2, flask
  Running setup.py install for itsdangerous ... done
  Running setup.py install for MarkupSafe ... error
Exception:
Traceback (most recent call last):
  File "d:\work\python\2018\hook\myvenv\lib\site-packages\pip\compat\__init__.py", line 73, in console_to_str
    return s.decode(sys.__stdout__.encoding)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xd1 in position 3: invalid continuation byte

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "d:\work\python\2018\hook\myvenv\lib\site-packages\pip\basecommand.py", line 215, in main
    status = self.run(options, args)
  File "d:\work\python\2018\hook\myvenv\lib\site-packages\pip\commands\install.py", line 342, in run
    prefix=options.prefix_path,
  File "d:\work\python\2018\hook\myvenv\lib\site-packages\pip\req\req_set.py", line 784, in install
    **kwargs
  File "d:\work\python\2018\hook\myvenv\lib\site-packages\pip\req\req_install.py", line 878, in install
    spinner=spinner,
  File "d:\work\python\2018\hook\myvenv\lib\site-packages\pip\utils\__init__.py", line 676, in call_subprocess
    line = console_to_str(proc.stdout.readline())
  File "d:\work\python\2018\hook\myvenv\lib\site-packages\pip\compat\__init__.py", line 75, in console_to_str
    return s.decode('utf_8')
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xd1 in position 3: invalid continuation byte

(myvenv) d:\work\Python\2018\hook\app>pip install flask
Collecting flask
  Using cached Flask-0.12.2-py2.py3-none-any.whl
Collecting Jinja2>=2.4 (from flask)
  Using cached Jinja2-2.10-py2.py3-none-any.whl
Requirement already satisfied: itsdangerous>=0.21 in d:\work\python\2018\hook\myvenv\lib\site-packages (from flask)
Requirement already satisfied: click>=2.0 in d:\work\python\2018\hook\myvenv\lib\site-packages (from flask)
Requirement already satisfied: Werkzeug>=0.7 in d:\work\python\2018\hook\myvenv\lib\site-packages (from flask)
Requirement already satisfied: MarkupSafe>=0.23 in d:\work\python\2018\hook\myvenv\lib\site-packages (from Jinja2>=2.4->flask)
Installing collected packages: Jinja2, flask
Successfully installed Jinja2-2.10 flask-0.12.2

and code of programm:
from flask import Flask
app = Flask(__name__)
if __name__ == '__main__':
    app.run()

@OlegBrony  It seems that Markupsafe is failed to build and install, but pip thinks it is already installed after the unsuccessful install. Maybe this issue only appears in Virtualenv environment. Please try to uninstall, 'chcp 65001',  and install it again.

@gKevinK Yep, reinstall worked. Thank you a lot.

Hi , Today I have the same problem on win10 , please dont't try to solved it .
Download the best whl for your python version  .
https://www.lfd.uci.edu/~gohlke/pythonlibs/#markupsafe
pip install MarkupSafe-1.0-cp36-cp36m-win_amd64.whl
thanks for this amazing project

I also saw this error. But I upgraded my pip to 10.0.1 and resolved problem.
My pip verson is pip 10.0.1 from c:\anaconda3\envs\my-rdkit-env\lib\site-packages\pip (python 3.6)
I used WIndows 10.

@maxtortime solution solved it for me

Closing, it looks like upgraded pip solved it, but it will also be solved by #45.

PyPI has wheels for MarkupSafe 1.1.0.


Implement IUdonEventReceiver interface on UdonSharpBehaviour

Feature Description:
This one's pretty simple, I want UdonSharpBehaviour to implement the interface IUdonEventReceiver to make it work with the new image downloader class.
Additional context:
When trying to pass this into VRCImageDownloader.DownloadImage() I get a compile error. I tried adding IUdonEventReceiver to the list of interfaces to the UdonSharpBehaviour class myself and it required a bunch of internal vrchat jargon we didn't need, but was necessary for the interface to work. As it stands right now, unless using hacky methods, we cannot pass the current behaviour into the DownloadImage() method. As such this may be an upstream problem I'm willing to file a canny for. Either that or I'm just doing it wrong.

you can cast UdonSharpBehaviour to IUdonEventReceiver directly, with (IUdonEventReceiver)this

It's a great workaround, thanks!
That being said, I'd expect to be able to use an UdonSharpBehaviour anywhere an UdonBehaviour is used directly since an UdonSharpBehaviour essentially is an UdonBehaviour. This would be good to note on the documentation. It would also be good to note on the documentation that Material and UdonBehaviour parameters are not optional, unlike what is described with the Udon Graph.


Eng-6073, ENG-6076, ENG-6077 - Testing & Workflow Updates

Stories:
ENG-6073 - UIClass Extension Tests - only a few were applicable because our swizzling methods just call other functions that have tests
ENG-6076 - Tracker Events Tests - Tests for a couple stand alone functions in our TrackerEvents folder
ENG-6077 - Simplified the NeuroIDClass file and removed outdated attributes and methods. Also updated isSDKStarted to have internal variable and not allowed to be set by user
Misc: I also updated the workflow files so that the notifications failure to slack doesn't run every time when the job is just skipped, updated the cache keys as it was failing to cache and pod install was still taking forever

nitpicks, pulled and tests ran. I say address the comments and 🚢 it.


[BEAM-2405] Override to sink interface in batch dataflow

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

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

R: @chamikaramj  PTAL
We can deprecate the sink interface to not be user facing now as this is a unified model.

On Tue, Jun 6, 2017 at 4:07 PM, Sourabh Bajaj<EMAIL_ADDRESS>wrote:

@sb2nov commented on this pull request.
In sdks/python/apache_beam/runners/dataflow/dataflow_runner.py
https://github.com/apache/beam/pull/3306#discussion_r120503749:

@@ -378,6 +379,23 @@ def run_Flatten(self, transform_node):
PropertyNames.ENCODING: step.encoding,
PropertyNames.OUTPUT_NAME: PropertyNames.OUT}])

def apply_WriteToBigQuery(self, transform, pcoll):

I'm a bit confused. This is the dataflow_runner.py file so we're only
overriding the one in dataflow runner. We have always used the streaming
api for the pipelines in the direct runner.

Oh, yes, I was thinking this was direct runner for some reason. This is
good.

PTAL again

LGTM. Will merge after tests pass.

Coverage decreased (-0.01%) to 70.617% when pulling e0f6076d31960636732384004871a806ba809831 on sb2nov:BEAM-BQ-SINK-MERGER into 1d2000d8c570ce2a2cdb63b5a208201526c394d8 on apache:master.

Should be ready to merge now.


Not working

My launch.json file is as follow:
{
  "version": "0.2.0",
  "configurations": [
    {
      "type": "node",
      "request": "launch",
      "name": "Jasmine Current File",
      "program": "${workspaceFolder}/node_modules/.bin/jasmine-ts",
      "args": [
        "--config=${workspaceFolder}/jasmine.json",
        "--project=${workspaceFolder}/tsconfig.test.json"
      ],
      "console": "integratedTerminal",
      "internalConsoleOptions": "neverOpen",
      "useWSL": true,
      "localRoot": "${workspaceFolder}",
      "remoteRoot": "${command:extension.vscode-wsl-workspaceFolder}"
    }
  ]
}

But I'm getting the following output:
cd c:\Users\phili\Documents.{90F8C996-7C70-4331-9D70-FB357D559FD5}\workspace\server-
translator && C:\Windows\System32\bash.exe -ic "node --inspect-brk=28807
node_modules/.bin/jasmine-ts --config=C:\Users\phili\Documents.{90F8C996-7C70-4331-9D70-
FB357D559FD5}\workspace\server-translator/jasmine.json --project=C:\Users\phili\Documents.
{90F8C996-7C70-4331-9D70-FB357D559FD5}\workspace\server-translator/tsconfig.test.json"-
bash: cd: c:UsersphiliDocuments.{90F8C996-7C70-4331-9D70-FB357D559FD5}workspaceserver-
translator: No such file or directory

Look at config option for example from the output above:
--config=C:\Users\phili\Documents.{90F8C996-7C70-4331-9D70-
FB357D559FD5}\workspace\server-translator/jasmine.json

Is there is something I've misconfigured?

Your output complains about the config and project args. If you want to pass the MS Windows path try using backslashes, i.e.
{
  "version": "0.2.0",
  "configurations": [
    {
      "type": "node",
      "request": "launch",
      "name": "Jasmine Current File",
      "program": "${workspaceFolder}/node_modules/.bin/jasmine-ts",
      "args": [
        "--config=${workspaceFolder}\jasmine.json",
        "--project=${workspaceFolder}\tsconfig.test.json"
      ],
      "console": "integratedTerminal",
      "internalConsoleOptions": "neverOpen",
      "useWSL": true,
      "localRoot": "${workspaceFolder}",
      "remoteRoot": "${command:extension.vscode-wsl-workspaceFolder}"
    }
  ]
}

Otherwise you can pass the WSL paths with;
{
  "version": "0.2.0",
  "configurations": [
    {
      "type": "node",
      "request": "launch",
      "name": "Jasmine Current File",
      "program": "${workspaceFolder}/node_modules/.bin/jasmine-ts",
      "args": [
        "--config=${command:extension.vscode-wsl-workspaceFolder}/jasmine.json",
        "--project=${command:extension.vscode-wsl-workspaceFolder}/tsconfig.test.json"
      ],
      "console": "integratedTerminal",
      "internalConsoleOptions": "neverOpen",
      "useWSL": true,
      "localRoot": "${workspaceFolder}",
      "remoteRoot": "${command:extension.vscode-wsl-workspaceFolder}"
    }
  ]
}

Ok, I made you suggestion:
"args": [
  "--config=${command:extension.vscode-wsl-workspaceFolder}/jasmine.json",
  "--project=${command:extension.vscode-wsl-workspaceFolder}/tsconfig.test.json"
]

But there is still this last line in my output that is problematic:
bash: cd: c:UsersphiliDocuments.{90F8C996-7C70-4331-9D70-FB357D559FD5}workspaceserver-
translator: No such file or directory

It looks like a path without slashes. Have you any idea why this?

The output missing slashes appears to be the same as in your first comment.
Looks like the issue occurs before the args are being processed, otherwise the output would have been in WSL format, although perhaps with slashes missing too.

I haven't found a solution for this yet. I pause this problem for now and will post here once make it work. Anyway it looks like not a vscode-wsl-workspacefolder issue. Thanks for your responses.

Thanks for the update @kalitine.
Yes, please do post the fix to your issue when you discover it.


SOL-3: Create a github action that releases the SDK to pypi when a new tag is created

Going to close this in favour of: https://github.com/common-fate/common-fate-sdk-python/pull/3


Textarea no autoresize on multiline initial value

Textarea doesn't seem to expand when you load it with initial value. It only fixes itself when you start typing in it.
Codepen: http://codepen.io/anon/pen/wJZxYa

https://github.com/marcosmoura/vue-material/issues/492


export tsdb.Iterator

To be able to consume the exported Map Reduce functions we need to also export the tsdb.Iterator

+1 on green build and changelog entry.

Got thumbs up from @corylanou  merging.


Documentation: platform support

So, you know, I looked through the readme again, to see how things were going, but I saw something strange.
Here's the section that kind-off bothered me, under 'dependencies':

RCG was created and tested on linux x86_64 based OS, install the [...]

Huhhh.. what's linux x86_64 based OS??
Let's think over it, what information does these kinds of sentences tend to convey:

The OS it was made on (optionally)
What platforms it supports
OS version supported, or, in the case of Linux and potentially Unix, distribution name and version

The reason why some people call Linux -- GNU/Linux, is because it's the kernel that empowers the environment, not the other way around, most of the time, so the stuff you see on the screen, the environment and such, is a vital part of the user experience. It can, and often does introduce bugs and vulnerabilities.
So what works on one distro, might not work on the other, simply because, for example, Debian doesn't have an up-to-date compiler, so it doesn't even build. That's important information, even just mentioning on what distro it was made would help.
Some people basically hand-craft their Linux environments, like people following the Linux From Scratch handbook. If they were, they're probably vastly more knowledgeable than either one of us, but even the little things help.
(tldr: distro's important)

If it makes kernel API calls, I wouldn't say it's needed, but it's really important to mention the latest supported kernel, or, at least, the latest kernel version it was known to work on, even more so if it uses new, bleeding edge security APIs, like, maybe, randomisation. Your program may expect behavior from an API call that might have changed throughout the kernel's lifetime, for example. Of course, you don't need to do all that homework, but it'd be amazing both for educational purposes and for people who your program, simply breaks, and they want to help fixing it.

And, btw, the architectures not important, for end-users, it's basically a buzzword, if it runs Linux, it abstracts it all away, unless you're doing really low level operations. At least, it's not important to the end-user. If they're running some really exotic architecture, that's doesn't have complete, or official support yet, like ARM, they're likely aware of it and it's they're job to report it in the bug report.
Whoo! I think.. that's about all? Quite the sprint there
But, as you might have noticed, while 'linux x86_64 based OS' sounds funny, it lacks a bit of substance, if nothing else, I hope you learned something.

Wanting to clarify, there's absolutely no pressure to do any of these things, take it at your own pace and see what works for you.
I make this Saturday afternoon completely in my free time because I think it's interesting and I like writing huge blocks of text no one will ever read in what I'm interested, so, no pressure, it's a pleasure/hobby project, you don't have any expectations to answer to :P

The README simply states that you just need ncurses to build (though it does erroneously refer to GCC and gmake as dependencies). Most people should be able to figure out that any environment with ncurses is a supported platform. All of the other information you described would be largely irrelevant for such a small and simple codebase.
I also see no issue with mentioning what sort of system the program was created on; this information is separate to what systems are supported.

I admit, I went a bit overboard on the information.
Though I believe my point still stands, 'linux x86_64 based OS', while it kind-off makes sense, in the sense that a distribution could be described as an environment built around the Linux kernel, mentioning the architecture has more use as a buzzword than anything else. And 'linux based OS' doesn't say much more either, might as well just say Linux instead.
It all comes from the fact that, to me, it just sounds like an unnecessarily long-winded way to say Linux, I even had to double-read it just to make sure I didn't read it wrong.
Linux, plain and simple, would inform just as much, without confusion.

Okay I read here some of the messages, I'm going to simplify some of it actually, I think for now only on Linux that it works and maybe on Unix based OS that will work too, but it depends.

I will do more tests and gonna tell if it works on other platforms too.

Check it out to see the new changes guys


Content-Type not being set on server responses

I've been trying to use grpc-health-probe as a health check (livenessProbe) on my gRPC pods. Everything is great, except for one thing: tower-grpc is not setting the Content-Type header on server responses, thus grpc-health-probe fails like so:
grpc-health-probe -v -addr localhost:10011 -service MyService -tls -tls-ca-cert test/ca.crt -tls-client-cert test/client.crt -tls-client-key test/client.key
parsed options:
> addr=localhost:10011 conn_timeout=1s rpc_timeout=1s
> tls=true
  > no-verify=false
  > ca-cert=test/ca.crt
  > client-cert=test/client.crt
  > client-key=test/client.key
  > server-name=
establishing connection
connection establisted (took 14.765746ms)
error: health rpc failed: rpc error: code = Unknown desc = OK: HTTP status code 200; transport: missing content-type field

Seems like the change in #70 got rolled back by accident? Was that intentional? Maybe there should be some integration tests to catch these interop things.

By the way, here's the relevant snippets of code:
syntax = "proto3";

package myservice;

service MyService {
  rpc Check(HealthCheckRequest) returns (HealthCheckResponse);
}

message HealthCheckRequest { string service = 1; }

message HealthCheckResponse {
  enum ServingStatus {
    UNKNOWN = 0;
    SERVING = 1;
    NOT_SERVING = 2;
  }
  ServingStatus status = 1;
}

impl server::MyService for MyService {
    type CheckFuture =
        future::FutureResult<Response<HealthCheckResponse>, myservice_grpc::tower_grpc::Status>;
    fn check(&mut self, _request: Request<HealthCheckRequest>) -> Self::CheckFuture {
        future::ok(Response::new(HealthCheckResponse {
            status: health_check_response::ServingStatus::Serving as i32,
        }))
    }
}

It would seem like an unexpected bug, since it looks likek the content-type should be set here: https://github.com/tower-rs/tower-grpc/blob/master/tower-grpc/src/generic/server/streaming.rs#L55

I did some more digging and I've figured out what's going on here. In short:

tower-grpc doesn't respond with a 404 when there's no matching RPC (this seems like a bug, no?)
the grpc health checking protocol doc specifies that:

A client can query the server’s health status by calling the Check method, and a deadline should be set on the rpc. The client can optionally set the service name it wants to query for health status. The suggested format of service name is package_names.ServiceName, such as grpc.health.v1.Health.

however, the grpc-health-probe tool doesn't modify the request path when you pass the -service flag (which I assumed it did), it merely populates the service parameter on the HealthCheckRequest proto with whatever you pass.
One more thing: it's unclear to be if there's a way to use 2 separate protos with the same server. From what I can tell, it's only possible to have one server, so I can't actually use the grpc-health-probe tool with tower-grpc unless I fork everything and change the definitions. I wanted to just stick everything in one proto, with one server for simplicity, but I guess that's not practical.

Ah, OK. When a gRPC server doesn't recognize a service path, it returns an "unimplemented` response. That still has a 200 OK http status code, because gRPC says so. ;_;
Looks like this is missing setting the header: https://github.com/tower-rs/tower-grpc/blob/master/tower-grpc/src/server/unimplemented.rs

It looks like UNIMPLEMENTED is supposed to be 501? https://github.com/grpc/grpc/blob/master/doc/statuscodes.md

Nope: https://github.com/grpc/grpc/blob/master/doc/PROTOCOL-HTTP2.md#responses
Notice it shows :status 200 as constant. And then claims some broken implementations may send non-200 HTTP statuses.
That status code mapping is meant to try to translate when response doesn't conform to gRPC, such as if a proxy is in-between.

For the record, for anyone else wanting to do this, I ended up just making a small binary that implements the same functionality as part of my crate.
Here's the source: https://gist.github.com/brndnmtthws/21a0aa51e21d8b9ae2a1d4dce22cdcbb


[FIX] return value should be false if timeString is null

Could you please make this pull request against the dev branch? I've also changed the default branch now, so this kind of confusion shouldn't happen again :)
PR looks good otherwise, I couldn't detect any unwanted behavior.


prerequisite of az cli not documented

I installed this in a powershell version that didn't have the the Az module installed and i got an error when trying to run Get-MgPolicyTokenIssuancePolicy about some class from the Az module not being found.  Maybe it was because i had AzureRM module installed in that powershell..not sure but the fact that this depends on the Az module should be called out i think.

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

ID: cfa0d6a2-8795-2d70-8282-4ab8169c8b70
Version Independent ID: 80d8d30e-9bc3-87c1-1f8f-70850a787b3c
Content: Install the Microsoft Graph PowerShell SDK
Content Source: microsoftgraph/docs-conceptual/installation.md
Product: non-product-specific
Technology: microsoft-graph-powershell
GitHub Login: @msewaweru
Microsoft Alias: eunicewaweru

@drdamour - please share the error you got here. We shouldn't have any dependency on the Az module that I'm aware of.

circled 1 is the error about unable to find Microsoft.WindowsAzure.Commands.Utilities.Common.AzurePSCmdlet
ignore my typo crossed off
2 is me importing Az module
3 is shows i no longer received the error about missing Microsoft.WindowsAzure.Commands.Utilities.Common.AzurePSCmdlet
thus it appears there is a dependency

Hi @SteveMutungi254 could you please confirm that we shouldn’t have any dependency on Azure CLI when running the Get-MgPolicyTokenIssuancePolicy cmdlet? Thank you.

Hi @SteveMutungi254 could you please confirm that we shouldn’t have any dependency on Azure CLI when running the Get-MgPolicyTokenIssuancePolicy cmdlet? Thank you.

We don't/shouldn't have any dependency on Azure CLI. From the error, it is clear that the user needs to issue Connect-MgGraph to authenticate.

@SteveMutungi254 i think you're looking at the wrong thing/not close enough, check the one labeled 1, that's the depednecy...once i forced the import the error changed, that's what i am showing

@drdamour : Kindly shoot me an email<EMAIL_ADDRESS>for further follow up. We will update this issue with comments after the offline investigation. cc: @csmulligan .

@drdamour thanks for reaching out! Throughout 2024 we will be phasing out GitHub Issues as the feedback mechanism for content and replacing it with a new feedback system. For more information see: https://aka.ms/ContentUserFeedback.
I’ll go ahead and close this issue for now, but please don’t hesitate to reach out if you require any further assistance.
Thanks again!

such a bad decision, my fav thing about docs was github issues driving them


findNextView() function that also takes the name of a view

What exactly should this function do?
Only changing the currentViewCounter to the id of the view which name was specified and then calling findNextView()?
Just trying to render the view which name was specified (regardless of the currentViewCounter)?
What should happen to variables as currentViewCounter, currentTrialInViewCounter, CT ?
If the only use case of this function is to go to an error/apologize end view, that is a dead end, then the implementation details do not really matter. Example: in an interactive experiment one participant leaves and the other participant gets redirected to this end view, with no possibility to change to another view. The participant now closes the experiment or reloads the page. This view is the last one in the views_seq and normally cannot get reached, because the thanks view does not call findNextView().
Other possible use cases, where implementation details matter:

After N incorrect trials the participant should see the instructions again. The participant reads the instructions and presses ok, then the experiment continues from that point in time or that part of the experiment gets reset (delete the data? or rename) and the participant has to redo that part of the experiment.
"Complex" experiments, where the order, type or number of iterations of trials depend on the choices of the participant. E.g. the first trial asks if the participant has experience with topic X, if he chooses yes, then he will see trials A and if he chooses no, then he will see trials B.

The question is, which use cases do we want to support and how flexible our approach should be.

I think this issue is from your suggestions from the email on 11.12.2018.

If the experiment ends for some reason (participant leaves), it could look
like this:
... customInteractiveView -> gameAborted -> (lobby|thanks) ...
The user can decide to go back to the lobby to start again or to end the
experiment.

For this the front-end would need a modified version of findNextView(),
which takes a name of a view and jumps to that view. This could also be
interesting for other non-interactive experiments, to show some trials, if
the participant chooses a particular answer and to show some other trials,
if the participant chooses another answer. The function should also have
another parameter to decide, whether to delete the (trial)data or not. The
first option would be useful, if the user goes back to the lobby the other
one otherwise.

I think it wouldn't hurt to make it as flexible as possible from the beginning, if we're going to add this function anyways.


khal: init at git today

A powerful command line tool to manage calendars in the terminal. To be used in conjunction with vdirsyncher for remote calendars. Today's git version.

We already have khal at version 0.7.0. Why do you want to add the latest version from git? We usually prefer to use stable releases.

I completely overlooked that it could also be in all-packages.nix, I just queried pythonpackages which I had open in my editor. There are plenty of commits since, so I'll keep my updated version - but indeed no need to add these to the repository.


docs: adding information on cis benchmarks and config audit scans

This is based on the following Slack questions: https://aquasecurity.slack.com/archives/C03L2H45L3H/p1692294989987299?thread_ts=1692293975.132919&cid=C03L2H45L3H

@AnaisUrlichs thank you 🙏


Final Pull Request :cry:

Checklist

[ ] I have updated the changelog.txt, if applicable.
[x] I have bumped the software version in _version.py, if applicable.
[x] I have checked that the unit tests passed.
[x] I have checked that Minnie and/or RoboMinnie passed.
[x] I have resolved merge conflicts.

Excellent work!


how to disable progress-bar on some special request.

Hi @hooklife
I'll look into it ;-)

Would love to enjoy this feature as well 👍

That would be interesting.
for example, i would hide the bar on certain pages/ route of my application.

You could use two Axios instances:
import axios from 'axios'
import { loadProgressBar } from 'axios-progress-bar'

const axiosWithProgress = axios.create()
const axiosWithoutProgress = axios.create()

loadProgressBar({}, axiosWithProgress)

axiosWithProgress.get()
axiosWithoutProgress.get()

I would really like to see this feature as well, love the package.
having two instances is dirty, as soon as you get in to maintaining tokens and interceptors for both of em. using fetch api not to trigger the progress bar faces the same issue.
This functionality is very useful when you want to limit the progress loader to navigation loading, and disable it for searching, uploading files, validation etc...

Up pls

This would be great to set some flag on the axios request to not show the loading bar. Until then, there is always CSS :)


[webapp up] when updating, show success message and version in output

If the issue is to do with Azure CLI 2.0 in-particular, create an issue here at Azure/azure-cli

Extension name (the extension in question)
az webapp up
Description of issue (in as much detail as possible)
When I updated there was no console output. I wasnt expecting to see "Updated successfully to version x.y.z". Can we do that?
Output of current extension
▶az webapp up -n papa-node-hello-003
Resource group 'appsvc_rg_Linux_CentralUS' already exists.
App service plan 'appsvc_asp_Linux_CentralUS' already exists.
Creating app 'papa-node-hello-003' ....
Webapp creation complete
Updating app settings to enable build after deployment
Creating zip with contents of dir /Users/papa/_git/node-hello ...
Preparing to deploy and build contents to app.
Fetching changes.
Fetching changes.
All done.
{
  "app_url": "https://papa-node-hello-003.azurewebsites.net",
  "location": "Central US",
  "name": "papa-node-hello-003",
  "os": "Linux",
  "resourcegroup": "appsvc_rg_Linux_CentralUS ",
  "serverfarm": "appsvc_asp_Linux_CentralUS",
  "sku": "BASIC",
  "src_path": "/Users/papa/_git/node-hello ",
  "version_detected": "0.0",
  "version_to_create": "node|8.9"
}

we made a few changes in this space & moved the command to core - please file new issue on azure-cli if there is more feedback. Thank you.


adds id text to each command

While using crictl @estesp reported a nit with the UI.. particularly that it would be nice if we told users POD-ID was needed. As is it says POD, which is confusing.
I checked and we're just not consistent across the UI so here's a PR to fix that.
@Random-Liu
address issue #334
Signed-off-by: Mike Brown<EMAIL_ADDRESS>

/lgtm Let's wait for the test result

lgtm


Only hash

Tested things locally, outstanding issues are related to https://github.com/ipfs/js-ipfs-api/issues/713

@disdavid Woohoo! Thanks a ton for the merge! 🎆 I'll go ahead and update any related work.


Lighten background in dark theme

Black is a bit too hard on the eyes. I think a dark grey type colour would look a lot nicer

It's easy to extend the base theme to something like a dark grey color scheme just like how the dark theme did.
You can make a PR if you truly want this happen 😉

Black is perfect for nighttime reading. Plus it's l33t.


Publish Docker Image

Summary
Add a CI step to build and publish the docker image for this repo to the GitHub package registry.
Details
Use GitHub Actions to define a build/deploy step that runs on each commit to the main branch. The CI step should build the docker image and publish it to the GitHub package repository for this repo under both a tag with the short commit hash, as well as the latest tag.
This would allow users to pull an already-built docker image from the registry and enable easier use and integration of new changes in hosted docker environments.

Would you be open to a PR that adds this functionality?

I built one here
version: "3.9"
services:
  web:
    image: itsnoted/chatpad:latest
    container_name: chatpad
    ports:
      - "8200:80"

https://hub.docker.com/r/itsnoted/chatpad

Any instructions on how to use this?


Form gets $dirty by default with debounce = false

This is a follow up of issue #238 which should have fixed incorrect $dirty reports.
Basically what happens is that a form just loaded will report the field as being dirty, even if it hasn't been touched yet - IF debounce is set to false. When debounce is set to true, the warnings works kind-of correctly, but only after a certain delay - meaning the user must pause before hitting save.

Please post a plunker reproducing the issue so people can look into fixing it.
A PR fixing the issue would be nice if you have found a reproducible bug.

sorry for seeming daft, but I've never made a plunkr. Do you have a good sample using angular-ui-tinymce unto which I could then build upon?

If debounce is set to 'false', no dirty-check is made before updating the view. Therefore, the editor is considered dirty by default (take a look at 'debouncedUpdate', which performs a dirty-check).
https://github.com/angular-ui/ui-tinymce/blob/master/src/tinymce.js#L95-L102
A tested patch:
// Update model when:
// - a button has been clicked [ExecCommand]
// - the editor content has been modified [change]
// - the node has changed [NodeChange]
// - an object has been resized (table, image) [ObjectResized]
ed.on('ExecCommand change NodeChange ObjectResized', function () {
    if (options.debounce) debouncedUpdate(ed);
    else if (ed.isDirty()) {
        ed.save();
        updateView(ed);
    }
});


Is it possible to add badges as well as hide bnb on scroll?

Looking at https://github.com/pocheshire/BottomNavigationBar, it seems more feature rich. Just wondering if the XF version can do those things?

Sure, would be great to add additional features over time. PR's very welcome!


Future support

This is the start of work to support future versions of Python + QT + databroker

we can pip install pyqt6, do you think it is worth an extra CI job?
Testing both in one process is between a very bad idea (because it might crash) and impossible (because it will crash).

I think we can have an extra job in CI for that case (for Python 3.7 only probably as https://pypi.org/project/PyQt6/6.3.0/#files has the wheels only for that version). Do we want to maintain a separate requirements-dev.txt with PyQt6? Right now we have PyQt5 listed there.

I added pyqt6 to py310 and if pyqt6 is installed mpl 3.5 and up will pick it automatically.

@klauer, are you OK with this update? Let's have it in if there are no objections.

Thanks, Ken! I'll go ahead and will merge it then.
Thanks for the future support fix, @tacaswell!


Telefe.ar has malformed stream

Describe your issue
The stream added in iptv-org/iptv#16169 has malformed url

Stream URL (required)
(https://mitelefe.com/Api/Videos/GetSourceUrl/694564/0/HLS / https://ssl.cloud.telefe.com/Api/Videos/GetSourceUrl/694564/0/HLS)

Which makes the json in streams.json has no url
{"channel":"Telefe.ar","timeshift":null,"http_referrer":null,"user_agent":null}
Contributing Guide

[x] I have read Contributing Guide

The reason for this bug is that the url field in the request contained two url's in brackets at once:

https://github.com/iptv-org/iptv/issues/16169
The bot did not understand what to do with it and apparently saved the record without the mandatory url field.


revolver: Add missing frame-time to the `noise' style

Hello,
the noise style was missing the frame time, causing errors about   being an invalid interval for sleep, e.g. during the demo.

Could the PR be merged? It would simplify the Hacktoberfest verification.


Update dependencies and code

Closes #35
This will also drop the new-ledger-namespace compat flag.

@MangoIV Do you know how to obtain the fourmolu etc tools pre-commit-hook uses in the shell?
Having fourmolu from haskell-nix's shell tools may cause version mismatches between what pre-commit-hook uses and what is available in the haskell nix dev shell, right?

I'm not aware how to get them from pre-commit-hooks.nix themselves, but there are two workarounds:

don't include fourmolu in the build-depends at all and just let people use pre-commit run (which imho is the cleanest solution)
defined the tools you plan to use in the buildInputs for the shell and set the tools attrset in the attrset you pass to the pre-commit-hooks to the tools you're planning to use.


The type or namespace name 'Raylib_cs' could not be found (are you missing a using directive or an assembly reference?) [Chess-Challenge]

kind of new to programming, so sorry if this is obvious, but I have no clue how to fix this issue. could someone please help me? did i do something wrong when downloading these files to VS? thanks in advance!

I'm newer as well and have this same issue.

@persoest @ashtonsprunger Please check the closed issues before creating a new one: #2

I have the nuget.org API endpoint as referenced by the package already in Visual Studio 2022 Community, so that isn't the issue for all people. As others who have mentioned this error also mentioned ARM, I assume there is no raylib build for ARM, so that might be another culprit if your NuGet settings are already correct.

@LKS90 Still, the error message indicates that the package can't be found (not that it isn't supported on the platform). While I'm not an expert with C# and NuGet, I would expect the error message to be different if a package is not support on a specific platform.

Well, it's the error message I get with my ARM device. I just switched to my desktop and the error is gone, so...


Describe the bug ✍
The image is not getting displayed in the README file, instead it is displaying code.
Here is the image of it:

Provide step by step information reproduce the bug 📄

Go through the Adventure_Game README file.
Scroll down you can find the bug.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

I want to work on this issue, please assign this issue to me.

Hey @Bindusree1515 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

i solved #3151 sir
please check and approve sir.

Hey @Bindusree1515 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Fix scoped plugin dependencies

Plugins declared dependencies on packages in the @hapi scope, but dependency resolution works only with the unscoped package name.
I'm curious if it works on your system as-is. I'm trying to debug it further down the stack:

https://stackoverflow.com/questions/61084503
https://github.com/hapijs/hapi/issues/4088

what version of node.js is the docker package installing?  v8.x??  This might be a v8.x vs v12 issue... I was working on updating the code to work with v12 since v8.x is no longer supported and I may have accidentally updated the v8.x branch (master)

The Docker container uses node  12.9.1 (and will automatically use the latest release from the 12.x branch whenever it is rebuilt).

Related commit: 3d6a2e2

ok... I think I found the issue.  It wasn't nodeJS it was hapi.  The latest version of hapi (v19.x) requires the @hapi scope AND nodeJS v12.x.  These changes were applied to the master branch by mistake.  I've reverted those changes and will now merge the pull request.  Thank you for helping diagnose the issue.


Add generate subcommand to client to create certs

This lets users create certs without having to go through the OIDC flow!
used like this:
./client generate --public-key-file <> --gcp-private-ca-parent <> --uri <> --ct-log-url <>

I used it to create a fulcio signed cert for the rekor public key (https://github.com/sigstore/rekor/issues/256).

closing in favor of https://github.com/sigstore/fulcio/pull/87


error while ionic run iOS

ionic-app-scripts build

[00:02:21]  ionic-app-scripts 1.0.0
[00:02:21]  build dev started ...
[00:02:21]  clean started ...
[00:02:21]  clean finished in 6 ms
[00:02:21]  copy started ...
[00:02:21]  transpile started ...
[00:02:25]  typescript: src/pages/home/home.ts, line: 19
Property 'scan' does not exist on type 'BarcodeScanner'.
  L18:    scanBTN() {
  L19:       this.barcodeScanner.scan().then((barcodeData) => {
  L20:  // Success! Barcode data is here

[00:02:25]  transpile failed
[00:02:25]  ionic-app-script task: "build"
[00:02:25]  Error: Error
npm ERR! Darwin 16.4.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "ionic:build" "--"
npm ERR! node v6.2.2
npm ERR! npm  v3.9.5
npm ERR! code ELIFECYCLE
npm ERR! ionic-hello-world@ ionic:build: ionic-app-scripts build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the ionic-hello-world@ ionic:build script 'ionic-app-scripts build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the ionic-hello-world package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ionic-app-scripts build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs ionic-hello-world
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls ionic-hello-world
npm ERR! There is likely additional logging output above.
Apples-MacBook-Pro-4:BarcodeProject BhuminB$ ionic plugin update

Dependency warning - for the CLI to run correctly,
it is highly recommended to install/upgrade the following:
Install ios-sim to deploy iOS applications.npm install -g ios-sim (may require sudo)
Install ios-deploy to deploy iOS applications to devices.  npm install -g ios-deploy (may require sudo)

cordova-plugin-compat 1.1.0 "Compat"
cordova-plugin-console 1.0.5 "Console"
cordova-plugin-device 1.1.4 "Device"
cordova-plugin-splashscreen 4.0.1 "Splashscreen"
cordova-plugin-statusbar 2.2.1 "StatusBar"
cordova-plugin-whitelist 1.3.1 "Whitelist"
ionic-plugin-keyboard 2.2.1 "Keyboard"
phonegap-plugin-barcodescanner 6.0.5 "BarcodeScanner"
CODE:
home.ts file
i

mport { Component } from '@angular/core';
import { BarcodeScanner } from 'ionic-native';
@Component({
selector: 'page-home',
templateUrl: 'home.html'
})
export class HomePage {
constructor(public barcodeScanner:BarcodeScanner) {
};
scanBTN() {
this.barcodeScanner.scan().then((barcodeData) => {
// Success! Barcode data is here
alert('barcodeData > '+JSON.stringify(barcodeData));
}, (err) => {
// An error occurred
alert('barcodeData err >'+JSON.stringify(err));
});
};
}

@bhandaribhumin this is an Ionic issue. You should be asking them why the build fails.


Test GPU code

Test GPU code on the CI server.
Use eigen-CUDA for instance. (https://github.com/NLESC-JCER/EigenCuda)
Make an action in the eigen-CUDA repo and make a remark that we tested it with eigen-Cuda and that we had a successful action.

We decided to go for less complicated example.
See:
https://github.com/ci-for-science/example-gpu-houston


build_simple_timeline.py example is broken

Build Problem
python build_simple_timeline.py /mnt/e/yt/ex.xml
Traceback (most recent call last):   File "test.py", line 87, in <module>     main()   File "test.py", line 83, in main     otio.adapters.write_to_file(tl, args.filepath)   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/__init__.py", line 191, in write_to_file     return adapter.write_to_file(   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/adapter.py", line 188, in write_to_file     result = self.write_to_string(input_otio, **adapter_argument_map)   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/adapter.py", line 255, in write_to_string     return self._execute_function(   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/plugins/python_plugin.py", line 174, in _execute_function     return (getattr(self.module(), func_name)(**kwargs))   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/fcp_xml.py", line 1967, in write_to_string     _build_sequence_for_timeline(   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/fcp_xml.py", line 1276, in wrapper     elem = func(item, *args, **kwargs)   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/fcp_xml.py", line 1851, in _build_sequence_for_timeline     _add_stack_elements_to_sequence(   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/fcp_xml.py", line 1905, in _add_stack_elements_to_sequence     track_elements = _build_top_level_track(track, track_rate, br_map)   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/fcp_xml.py", line 1793, in _build_top_level_track     _build_item(item, timeline_range, transition_offsets, br_map)   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/fcp_xml.py", line 1747, in _build_item     return _build_clip_item(   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/fcp_xml.py", line 1276, in wrapper     elem = func(item, *args, **kwargs)   File "/home/cypher/miniconda3/envs/edit/lib/python3.8/site-packages/opentimelineio/adapters/fcp_xml.py", line 1637, in _build_clip_item     _build_rate(clip_item.source_range.start_time.rate) AttributeError: 'NoneType' object has no attribute 'start_time'
Incorrect Functionality and General Questions
Describe the issue here.
To Reproduce

Operating System: WSL2 ubuntu
Python version: 3.8
OpenTimelineIO release version or commit hash:OpenTimelineIO==0.13.0

Expected Behavior
I should get an exported fcpx compatible xml
Logs
If applicable, attach a conplete console log, that show a reproduction of the problem entirely.
Additional Context
Add any other context about the problem here.

I'll fire an issue about the FCPXML and EDL adapters.  For now I'm going to make a PR that adds a source_range call to build_simple_timeline.py so that it works as advertised.


[MOD] fixed bug, checking image extension for sending image messages

fixed bug, checking image extension for sending image messages

Thnx :))


hashCode() for classes overriding equals(). fixes #1185

adding missing hashCode() for classes overriding equals().

@dmikurube sure. Btw, what is the specific reason not to include it into the next release? Do we make a distinction between types of fixes to include to some types of releases?

We've confirmed that v0.9.20 fixed #1151, and #1196 has been merged. Let's go merging this.
As ed17c5680fbb6df5997c9adb94e2a55cee4b1b35 was not compliant with the CI, we may want to squash the commits so that .

(I usually did local squash and force push for such a case so far so that we have a "merge commit" in the commit history. But now, we may want to change the custom in this OSS embulk repo.
Ping me if you want to talk about it, or please go ahead with merging. But at least, please keep the PR number (#1186) in the squashed commit message so that we can find this PR later from Git commit history.)

@dmikurube +1 for squashing with pr number.


Feature/add test

Just a simple smoke test

Since the smoke-test code in this PR is pretty much identical to previously reviewed and accepted code in dwave-examples, I will just merge.


No connection to Copilot was found

``I wanted to try this package but I am getting a NoConnectionException error on running the following code.
  import asyncio
  import os

  from sydney import SydneyClient

  async def query_sydney():
      sydney = SydneyClient()

      async for response in sydney.ask_stream(
          "test", citations=True
      ):
          print(response, end="", flush=True)
  asyncio.run(query_sydney())

I am running this on python 3.11 and here the details regarding the package which I am using.

Name: sydney-py
Version: 0.20.1
Summary: Python Client for Copilot (formerly named Bing Chat), also known as Sydney.
Home-page:
Author: vsakkas
Author-email<EMAIL_ADDRESS>License: MIT
Location: C:\Python311\Lib\site-packages
Requires: aiohttp, brotli, websockets
Required-by:

don't know how to resolve this issue. If anyone can help much appreciated

Hi @SleepyOwL00
Looks like you forgot to add:
await sydney.start_conversation()

or do this instead:
async with SydneyClient() as sydney:
   # your code


Bring over content from Wix site

This PR brings most the content from the Wix site over to Solara. I think there are some tweaks that we'll want to do to the styling, but this gets the basics down. The main things added here:

I added the history burb from the Wix site to the about page. Per a discussion with @patudom and @johnarban today, I've omitted the Objectives section. Maybe we want something else to flesh this out?
The Data Stories page has been populated with all of the content from the Wix page. The filtering via a chip group has been ported over as well. The layout is auto-populated from a list of dictionaries, so to add a story, just add a new entry to the list with the same structure. The tags field of the dictionaries are used for the filtering.
The Team page has also been brought over, and is auto-populated similarly to the Data Stories page. To add a new team member, just add a new entry to the relevant list.
I brought over the Contact page as well. If someone fills out the form, we'll get an email to the<EMAIL_ADDRESS>email (from itself) with the person's message and a subject line containing their name and email. The method that we're using here is the same as what we're using in the backend server to send emails

For all of the images, I just downloaded them from the Wix page. If someone has the original images and they're of higher quality, feel free to replace.

Also worth noting - I've already added the environment variables needed for the email functionality here to the EBS environment.

This looks really good. The filtering on the data stories works great. There is just one issue I see: in light mode, the name text of the team should still be white:


Dataloader.Ecto not compiled with Ecto 2.x

Since commit https://github.com/absinthe-graphql/dataloader/commit/8a53e31342893d72a0096d0aee7b6b03992dd92a changes the optional dependency on ecto to ecto_sql, since my project is still on Ecto 2.x the optional dependency on ecto_sql has no effect. This means that dataloader may be compiled before ecto, which in turns means that the if Code.ensure_loaded?(Ecto) check fails in dataloader_ecto.ex and then my code breaks since it uses Dataloader.Ecto.
I'm not sure what the best course is for this. Some options/thoughts:

List a breaking change that only Ecto 3.x is supported
Split Dataloader.Ecto into a separate project that has a direct dependency on ecto
a. This isn't ideal for me but it would've caused the error to appear earlier since I would have noticed ecto_sql being pulled in (or gotten a mix conflict)
Revert the dependency change (this should be okay since everything that Dataloader.Ecto uses is actually from ecto and not ecto_sql

@axelson yeah I think the only reason we did ecto_sql was for tests I think, can you do a PR where you move it to only: :test ?

@benwilson512 created! https://github.com/absinthe-graphql/dataloader/pull/68


Readd implicit package references section

For customers targeting older versions, we still need this info as it's linked from the product using https://aka.ms/sdkimplicitrefs. This info used to live in csproj.md.
Preview link.

@dsplaisted This looks good to me. But I think you should have better memory on the framework reference of 1.x 2.x time

@wli3 @dsplaisted @adegeo Thanks for reviewing!

I also updated the aka.ms link to point to https://docs.microsoft.com/dotnet/core/project-sdk/overview#implicit-package-references.


[opentelemetry-cpp] Update to v1.0.0

Update opentelemetry-cpp library to v1.0.0

Which triplets are supported/not supported?

Yes

/azp run

Thanks for the version update!


JSON.parse: unexpected character at line 1 column 1 of the JSON data

Uncaught runtime errors:
ERROR
JSON.parse: unexpected character at line 1 column 1 of the JSON data

@Abhijrathod which API? and can you paste the Go backend panic log?


Make archive links absolute

The archive is only available on docs.docker.com, and not when doing a "make docs", so relative links will not resolve.
Making these links absolute, so that they always point to the live "docs.docker.com" domain.
/cc @sfsmithcha

LGTM

author LGTM


using vue-form-generator with other css frameworks

Currently vue-form-generator and some of the field types are bound to bootstrap.css. We're using another css framework in our application. It may be a good idea to set a theme in the options of vue-form-generator or a callback that allows for switching components between different frameworks
When rendering a form field this would allow for a resolution of what component to use to render the field (eg first look if field-{theme}-{name} is available before using field-{name})

thanks for the response. In our case we use http://materializecss.com/forms.html - in this case we for example had to replace the datepicker. I don't think it's the job of vfg to provide for all different types of css frameworks. What I think may be a good idea to be a bit more css framework agnostic in the basic components and allow 3rd parties to develop for different css frameworks.
While the approach of adding classes works it does not seem to be the best approach if you have a large set of schemas as you would have to add the classes to every schema resulting in a lot of duplicate work.
If a theme is introduced the vfg options could also contain the basic custom classes per field to be added as a more global approach.

I'm able to do that, which isn't too bad.
As for the date picker, there is no real solution.
Have you tried Pikaday or DateTimePicker ?
It is also fairly easy to create your own fields. You can use the core version, and add your own fields, with your own framework in mind. We encourage you to share your work with the community of course.
We will never be able to satisfy everyone and continue to add new feature at the same time.
Good luck !

thank you for the example. We intend to share our custom components in the future. Will keep you posted

Hi! I'm trying to use some tool like this one but it seems to be too opinionated. I'm trying to find out how I should use it with another ui framework (i.e. bulma). The only reference I found in the docs is styleClasses property, can you give a clue where should I look for?

Found the answer here, seems that online manual is outdated:


feat: update favicons

Update text and favicons.

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


llama: Decouple patching script from submodule

The current patching script relies on the existing llama.cpp submodule, which we plan to remove in favor of the vendored code soon, so this updates the patching script to no longer rely on that.  It still uses the git commit from the submodule to ensure we don't drift while the submodule is still present.  In the commit that removes the submodule in the future, this would be switched to a manifest file recording the git hash we're sync'd to.
This also adjusts the paths of the patches we carry to be aligned with our file layout instead of upstream to make it easier to generate and keep them updated.
Replaces #7124 on main

The script currently checks out llama.cpp in an ephemeral directory, but it seems github clones are sporadically slow these days, so it might make sense to move it to a top-level well known directory and .gitignore that so folks who run this ~regularly aren't having to re-checkout the tree multiple times.


Add NodeSource to nodejs-security-overview.md

Adds NodeSource to the "Companies helping with Node.js security" section of nodejs-security-overview.md

Just wanted to point out something that was confusing in your express example:
app.get('/', async (request, response) => {
  // awaiting Promises
  const result = await getContent()
  return result
})

In this example, nothing gets sent, you still need to call response.send(result). The other thing is that if getContent() returns a rejected Promise, express won't handle the error, and the request will just hang. If you want to use async/await, this is what your middleware would have to look like:
app.get('/', async (request, response, next) => {
  try {
    // awaiting Promises
    const result = await getContent()
    response.send(result)
  } catch (err) {
    next(err)
  }
})

Or you could write a higher-order-function to wrap your middleware to make it work like you presented (which isn't built into express out of the box):
const wrapAsync = (middleware) => {
  return async (req, res, next) => {
    try {
      // Note that we don't `res.send` in here, the middleware should probably still
      // control whether or not it responds to the client.
      await middleware(req, res, next)
    } catch (err) {
      next(err)
    }
  }
}

app.get('/', wrapAsync(async (request, response) => {
  // awaiting Promises
  const result = await getContent()
  response.send(result)
}))

I put this whenever I just need to simulate, or actually want to delay execution.
const delay = ms => new Promise(r => setTimeout(r, ms));
const timeout = (ms, message) => new Promise((_, rej) => reject(new Error(message)));
...
    // timing out (doesn't cancel the original whatIWant() though
    Promise.race([whatIWant(), timeout(1000, "too long dude!")]);

Another thing, although request-promise has a simpler interface, I really wish more articles would use node-fetch (or isomorphic-fetch) for their request examples.


Write fields query differently to avoid namespace issues

Aim to work around an issue with the fields query that failed for a customer, but which we cannot reproduce. Using fields=id;type involves namespaces for the type field, which may be causing RESTCONF to get confused somehow. So hopefully using a depth query will prevent the issue:
curl "http://localhost:6749/admin/api/restconf/data/base:profiles/profile?depth=2" \
-u "admin:Password1" \
-H "accept: application/yang-data+json"

This should return a couple more fields as follows, but remains an efficient query, that brings back the id and type fields that the rest of the migration logic needs. We will ask the customer to verify that this query works before merging.
{
  "base:profile": [
    {
      "id": "authentication-service",
      "type": "profile-authentication:authentication-service",
      "settings": {
      },
      "token-issuers": {
      }
    },
    {
      "id": "token-service",
      "type": "profile-oauth:oauth-service",
      "settings": {
      },
      "token-issuers": {
      }
    },
    {
      "id": "user-management",
      "type": "profile-user-management:user-management-service",
      "settings": {
      }
    }
  ]
}

Works like a charm! :-)


Read from indexing queue during crash recovery.

Change #704 introduced a regression that started reading the queue only
after potential crash recovery. When more than the queue capacity was
indexed, Prometheus deadlocked.
@beorn7 @juliusv
This attempts to preserve that you don't have to call Storage.Stop() if Storage.Start() failed.
#721

Done

:+1:


--silent should make no logs come through at all

Component
Anvil
Describe the feature you would like
When using --silent, I know it says in docs it only silents the startup, but it would be nice to silence the JSON-RPC method names being called to avoid spamming the logs if you do not care for it.
The use case is if you are trying to run a local node in a child_process, you have to override the max buffer to infinity for it not to bail out after it's been running for x time over just being able to do --silent and make it run in the background without any noise.
Additional context
No response

good point, I consider this a bug. fixing

awesome thanks a lot


RDBS-92 Make Term & TermInfo arrays GC friendly

make termInfo a struct
clear Term array

https://issues.hibernatingrhinos.com/issue/RDBS-92#focus=Comments-67-54286.0-0

https://issues.hibernatingrhinos.com/issue/RDBS-92#focus=Comments-67-54286.0-0

LGTM

LGTM


Edit Changelog for 0.16

Generated function for extensible record are removed in 0.16
see https://github.com/elm-lang/elm-compiler/issues/1088

Ah, I get what you meant about the changelog now. I want to mess around with the formatting, but I'm working on adding this right now.


ASP.NET MVC fails with 400 when Chinese characters are in cookies

@netnr opened this here: https://github.com/dotnet/core/issues/1049
Issue Title
asp.net core 2.0 mvc
Development project, cookies has Chinese, access site return error code: 400
General

@JunTaoLuo ?

Chinese：
调试过程中，我只是改变 cookie是否含有中文，没中文一切正常，含有中文，页面空白，也没有异常抛出，只看到控制台400错误，或许是内部问题。
English：
In the process of debugging, I just change the cookie whether it contains Chinese, no Chinese is normal, containing Chinese, blank pages, and no exception thrown, only see console 400 error, perhaps internal problems.

Chinese characters are not allowed in any http header (including cookies) without additional encoding.

See https://github.com/aspnet/KestrelHttpServer/issues/1144

If that's the case, it's unscientific

This need to be closed?