kube status is not load on init

Is there an existing issue outlining your problem?

[X] I have searched the existing issues and they do not solve my problem.

Describe your problem.
On entering tmux with kube status module, kube context is not show until force refresh tmux.conf file.
Paste your configuration.
# List of plugins
set -g @plugin 'tmux-plugins/tpm'
set -g @plugin 'tmux-plugins/tmux-sensible'

set -g @plugin 'sainnhe/tmux-fzf'
TMUX_FZF_LAUNCH_KEY="C-f"

set -g @plugin 'seebi/tmux-colors-solarized'
set -g @colors-solarized 'dark'

# set -g @plugin 'jonmosco/kube-tmux'
# set -g status-right "#(/bin/bash $HOME/.config/tmux/plugins/kube-tmux/kube.tmux 250 red cyan)"

# theme config
set -g @plugin 'catppuccin/tmux#latest'
set -g @catppuccin_flavor 'frappe' # latte,frappe, macchiato or mocha
set -g @catppuccin_window_right_separator "█ "
set -g @catppuccin_window_number_position "right"
set -g @catppuccin_window_middle_separator " | "

set -g @catppuccin_window_default_fill "none"

set -g @catppuccin_window_current_fill "all"

set -g @catppuccin_status_modules_right "application date_time kube"
set -g @catppuccin_status_left_separator "█"
set -g @catppuccin_status_right_separator "█"

set -g @catppuccin_date_time_text "%Y-%m-%d %H:%M:%S"
set -g @catppuccin_window_default_text "#{window_name}"
# set -g @catppuccin_status_modules_left "directory"

# set prefix
unbind C-b
unbind C-t
set -g prefix C-t
bind C-t send-prefix

# enable mouse
set -g mouse on

# split pane
bind | split-window -h
bind - split-window -v
unbind '"'
unbind %

# switch panes using Alt-arrow without prefix
bind -n M-Left select-pane -L
bind -n M-Right select-pane -R
bind -n M-Up select-pane -U
bind -n M-Down select-pane -D

# Initialize TMUX plugin manager (keep this line at the very bottom of tmux.conf)
run '~/.tmux/plugins/tpm/tpm'

Attach screenshots.
No response
What tmux version are you seeing the issue on?
3.4-1ubuntu0.1
Any additional comments?
If I load the kube module from https://github.com/jonmosco/kube-tmux directly without catppuccin, the kube context is show on tmux load.
After entering tmux and force reload the tmux.conf file. kube status is update after I issue kubectl command.

Hello! Could you try adding this after the run TPM line?
set -g status-right "#{E:@catppuccin_status_application}#{E:@catppuccin_status_date_time}#{E:@catppuccin_status_kube}"

Only date and time are showing up. It is not after that I re-source tmux by tmux source ~/.config/tmux/tmux.conf That everything shows up correctly

I'm not surprised, TPM can be quite fiddly. I'd recommend manually source-ing the catppuccin plugin instead so the options are present in the running tmux server before the status line formats are expanded. If that doesn't work I'll look into it further :)


heapprofiler doesn't work well on arm and 64g x86-64

what condition does heapprofiler works on arm, and x86-64 whose memory large than 32g(i guess) need?
I can get samples and generat calling diagram on 16g x86-64 centos, but when I run heapprofiler on a arm with 192g memory and a centos with 64g memroy. The pprof just says the report has no sample. I tried gperftool with 2.3 2.5 2.10 version, with or without libunwind 1.5 1.6.
I compile the program with fno-omit-frame-pointer and -g.
Anyone knows how to make it work, many thanks!!!
bty, could this be related to the heavy use of coroutines or some 32 bit limits？

Please show output of TCMALLOC_STACKTRACE_METHOD_VERBOSE=t ./stacktrace_unittest
All cases you mention should work.

Closing for now, but do reopen please if you still observe any issues with latest code.


Widget Example

I made a widget using this project and wishing to share with others.

Hi, thanks for the contribution. Looks good!


Timed Holiday Greetings build up

If you have the game paused in singleplayer, they build up. Resulting in an extremely laggy storm of fireworks the second you unpause it.

I was more concerned about server lag than singleplayer when I made this, tbh. I'll have a look at this before next New Year's ;)

I was more concerned about server lag than singleplayer when I made this, tbh. I'll have a look at this before next New Year's ;)

Is it only for new years?

Yes, the fireworks are only for New Year's. Christmas has free energy with snow and multi-language season greetings on the RF bars. There's not yet a Easter egg for Easter ;)

Status?


Allow wait=inf

When using spotlight I'd like to be able to share a spotlight instance via a link. However, since wait=True, the most permissive mode in that regard, will kill the instance when the "last browser" is closed, I cannot reliably do this.
I propose to introduce wait=inf to be able to deploy a long running spotlight instance.

Hey! Thanks for your suggestion. An additional option to keep the Spotlight instance running forever sounds rather useful to me.
How about wait="forever" to be more explicit?
For the time being, you can do something like this:
import threading

spotlight.show(df, wait=False)
threading.Event().wait()

@neindochoh ja! Thanks for the workaround.
For sure setting wait to 'forever' will work too.
I saw the added benefit in not relying on a string

wait = float('inf')
wait > 1
True
wait == 1
False
wait < 1
False

wait = False
wait < 1
True
...


MQTT Discovery Light not being added to Homekit or Google Assistant

Home Assistant release with the issue: 0.80.3

Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.): Docker, based on Debian

Component/platform: Homekit, Google Assistant, MQTT Discovery

Description of problem:
MQTT light is a WS2812 connected to a nodemcu, the setup has been created with esphomeyaml using the fastled component.  The light registers into Home Assistant just fine and can be adjusted without any problems however the light doesnt present itself to HomeKit or Google Assistant despite being exposed in the configuration.yaml
Light i am having issues with is called kitchen_lightstrip, i have attempted to disable autostart on HomeKit and then delay the start after HASS has started by 5 and 10 minutes but still the light doesnt appear.
I have relinked my Google Account with HASS and the light isnt exposed.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

homekit:
  auto_start: False
  filter:
    include_domains:
      - switch
      - remote
      - input_boolean
      - script
    include_entities:
      - light.kitchen_striplight
      - sensor.dining_room_temperature
      - sensor.dining_room_humidity
      - sensor.nodemcu_temperature
      - sensor.nodemcu_humidity
      - light.color_temperature_light_1
    exclude_entities:
      - input_boolean.1_notification_test_input
      - input_boolean.2_notification_test_input
      - input_boolean.gamingpc_online
      - input_boolean.switch_1_timer_input
      - input_boolean.switch_2_timer_input
      - input_boolean.test_input
      - switch.office_light_flux

google_assistant:
  project_id: homeassistant-integratio-e4236
  api_key: !secret google_assistant_api_key
  exposed_domains:
    - switch
    - scene
    - input_boolean
  entity_config:
    light.kitchen_striplight:
      name: KITCHEN_STRIPLIGHT
      room: KITCHEN

Traceback (if applicable):
No errors logged from what i can see.

Additional information:

This is the bug/issue tracker for Home Assistant. Please use the forum for questions about installation, setup, configuration and usage.
If it's a bug, please re-open the issue.

Similar issue here https://github.com/home-assistant/home-assistant/issues/13472


Hosts file editing

prepare an example script that may help with appending to hosts file host names needed to be set up to access dispatcher/httpd vhosts
project/script/hosts.sh
assumptions

should work on Mac, Linux and Windows
could request admin privileges on Mac and Windows automatically; on Linux should be run with sudo

implement it as tasks per os ; use - https://taskfile.dev/usage/#platform-specific-tasks-and-commands

define hosts to be appended in local.env as these domains could be use in other scripts as well


[FEATURE] Add support for custom objectives

Introduction
Cases can have objectives, it would be nice to be able to add custom objectives.
This can be implemented under Lib.CaseObjectives
Description of the feature
A clean and concise way of adding custom objectives and triggered action when the objective is completed.
Features

Easy way to create a new objective and assign a trigger for completion
(have a property "IsCustom" so we can see which events are custom made when triggered in an event)
Provide an event that is invoked when a new objective is created. (including in-game objectives)
Provide an event that is invoked when an objective completes (including in-game objectives)
Provide an event that is invoked when an objective is canceled. (including in-game objectives)
Allow child objectives to be attached to the parent objective that will launch when parent objective is completed
(define if the event should trigger on complete or cancel)
Allow manual completion or cancel of objective.

Development notes

Automatic creation of DDS strings record when objective is created
Automatic cleanup of DDS strings record after completion / cancelation
(make sure that no other objective is using this particular dds record before deletion)

Implemented for v2.0.3


disable activate of objects that don't do anything on activate.

Hi @lalalune
Please check this and give me feedback.

This doesn't fix the case where I, saw, throw a blender cube into the scene. I think we should check if there is a use, grab, etc component on it, and then show the "e" if so. There might even already be a canUse function to test this.

@ultrabot2022 please edit your comment to reference the Github issues this fixes and record a video of it working so QA knows what to look for.

@ultrabot2022 I tested this but it has a few issues:
-I am unable to equip avatars as E doesn't appears
-I am Unable to equip chest for claims
-I am unable to equip npc to follow the character.
Can you look into it? Maybe the logic needs to be changed
https://user-images.githubusercontent.com/35926530/181256600-cf01f1f4-5cbf-4842-bec6-000ff8582092.mp4
https://user-images.githubusercontent.com/35926530/181256735-9c17befe-0f52-43f1-be0e-449e20ac1e44.mp4

I added disable logic to only objects that have active components such as 'wear', 'sit', 'use', 'aim', 'drop' before.
But I knew that there are other type of object such as npc, vrm wear after checking the logic again. And I am afraid of whether what type of will be added in the future.
So I added "nonItem" component to non-active object like following and added disable logic to the game.js
The result can be checked in treehouse.scn

I think we'll need to think deeper about this. We should have some logic that evaluates whether the object is usable or not once, and store and retrieve that information -- like a canUse component or something.
By default, GLBs and FBXs should not be usable.


Alerta not receiving alerts from Zabbix and debug log is not being written to

Hi there,
I saw someone post earlier about a similar issue, but I have DEBUG set to True in the script, so I am expecting logging. I have copied the python script to /usr/lib/zabbix/alertscripts/zabbix_alerta.py on my Zabbix application server (with appropriate permissions and ownership), and configured Zabbix as described in your Readme. Zabbix tells me that the action is being performed successfully, but when I query the Alerta API, I don't see any alerts, and /var/log/zabbix/zabbix_alerta.log has yet to be created. I then tried creating the log file with the appropriate permissions and ownership by touching it but it's still 0 bytes.
Of note, I noticed that logging and requests Python modules weren't on my Zabbix application server, so I installed them (argparse was already installed). Are there any other Python modules I could be missing? If it helps, I'm running Zabbix 2.4.3 and using Python 2.6.
Thanks for your help!

What happens when you try to run the script manually from the command line?
What error do you get?
On Thu, 1 Sep 2016, 00:44 Raman Gupta<EMAIL_ADDRESS>wrote:

Hi there,
I saw someone post earlier about a similar issue, but I have DEBUG set to
True in the script, so I am expecting logging. I have copied the python
script to /usr/lib/zabbix/alertscripts/zabbix_alerta.py on my Zabbix
application server (with appropriate permissions and ownership), and
configured Zabbix as described in your Readme. Zabbix tells me that the
action is being performed successfully, but when I query the Alerta API, I
don't see any alerts, and /var/log/zabbix/zabbix_alerta.log has yet to be
created. I then tried creating the log file with the appropriate
permissions and ownership by touching it but it's still 0 bytes.
Of note, I noticed that logging and requests Python modules weren't on my
Zabbix application server, so I installed them (argparse was already
installed). Are there any other Python modules I could be missing? If it
helps, I'm running Zabbix 2.4.3 and using Python 2.6.
Thanks for your help!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alerta/zabbix-alerta/issues/9, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAlikdvriTz1y2wz2iE6cBavZOermD-9ks5qlgOzgaJpZM4JyJGX
.

Hmm, didn't think to do that, good idea. Looks like Python can't find argparse:
Traceback (most recent call last):   File "zabbix_alerta.py", line 5, in <module>     import argparse ImportError: No module named argparse
Which is strange because when I try to install argpase, I get:
Requirement already satisfied (use --upgrade to upgrade): argparse in /usr/lib/python2.6/site-packages
Strangely, if I open up python interactively, I can see that the directory listed above is listed as an option:
`>>> import sys

print sys.path
['', '/usr/lib64/python26.zip', '/usr/lib64/python2.6', '/usr/lib64/python2.6/plat-linux2', '/usr/lib64/python2.6/lib-tk', '/usr/lib64/python2.6/lib-old', '/usr/lib64/python2.6/lib-dynload', '/usr/lib64/python2.6/site-packages', '/usr/lib/python2.6/site-packages']`

Any ideas? This appears to be related to my system configuration rather than with the script.

I figured it out... I had a permissions issue with the Python modules which denied access to the user running the script. Once I fixed permissions, the script started working. The failure message the same when a module is missing and when it's inaccessible, and I got stuck going down the wrong path.
Thanks for your help!


Builds are failing due to bevy commit not being tracked

One of the issues with the build at the moment is that there isn't any branch or tag pointing to the commit a58b4936d0e1ccd241dca74bceae78a1b6d4f949 on the current bevy fork (https://github.com/terrarier2111/bevy). I've made a branch called leafish on my fork (https://github.com/nathanruiz/bevy) that points to that commit so we can easily move it across.
@terrarier2111 Would you be able to create that tag/branch?

I was trying to avoid making a PR since the current main branch seems to be a single commit rebased onto bevy's main branch at a point in time.
It would be better for @terrarier2111 to tag this commit directly, so that it won't be lost again in case of another rebase.

Tbh normally i just revert the commit and redo it again as there are a lot of changes going on inside bevy any my change is only a one-liner

@terrarier2111 Can you create that tag on your fork so that the builds will start working again? It doesn't matter what the tag is called, as long as it exists, it should be fine.

We are now using the official latest release of bevy


Better error messages

This should provide somewhat better error messages. In case we actually
reach GitHub (and not fail because of a network error), user will see
the HTTP error code and contents of message field from the response.
It's not perfect - ideally, we should use GitHub's response to show
specific error messages (like 'duplicate filename), but that's a
task for another day.

I haven't found any issues, so I guess I just have to rewrite documentation.

I've check too, everything is good. Thank you for this new feature.


policy: Fix potential nil pointer deref in selectorManager.GetSelections

Please ensure your pull request adheres to the following guidelines:

[X] For first time contributors, read Submitting a pull request
[X] All code is covered by unit and/or runtime tests where feasible.
[X] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[X] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[X] All commits are signed off. See the section Developer’s Certificate of Origin
[X] Provide a title or release-note blurb suitable for the release notes.
[X] Are you a user of Cilium? Please add yourself to the Users doc
[X] Thanks for contributing!

This commit address the case in which selectorManager.GetSelections is called before selectorManager.updateSelections is called, which can cause a panic due to a nil pointer dereference in case the internal selectorManager.selections field is nil. To address this case, if the selectorManager has a nil value for its internal field selections, then emptySelection is returned.
The reason GetSelections cannot modify the internal selections field if it sees that it is nil is because selectorManager.setSelections requires the managing SelectorCache.mutex to be locked. This cannot be guaranteed inside a call to selectorManager.GetSelections, which does not require locking.
This needs to be backported to 1.14, 1.1,3 and 1.12, please. Versions 1.13 and 1.12 will have a merge conflict due to changes made in commit https://github.com/cilium/cilium/commit/9397655c7ce93eb5ecc7d518960f987cc5b0c3f4, which adjusted how the atomic pointer is loaded. The needed change for these versions should look something like this:
diff --git a/pkg/policy/selectorcache.go b/pkg/policy/selectorcache.go
index afcccc054b..a478519cf9 100644
--- a/pkg/policy/selectorcache.go
+++ b/pkg/policy/selectorcache.go
@@ -337,7 +337,12 @@ func (s *selectorManager) Equal(b *selectorManager) bool {
 // of the selections. If the old version is returned, the user is
 // guaranteed to receive a notification including the update.
 func (s *selectorManager) GetSelections() []identity.NumericIdentity {
-       return *(*[]identity.NumericIdentity)(atomic.LoadPointer(&s.selections))
+       selections := (*[]identity.NumericIdentity)(atomic.LoadPointer(&s.selections))
+       if selections == nil {
+               return emptySelection
+       }
+
+       return *selections
 }

 // Selects return 'true' if the CachedSelector selects the given
diff --git a/pkg/policy/selectorcache_test.go b/pkg/policy/selectorcache_test.go
index bc2fe04620..8fb47141db 100644
--- a/pkg/policy/selectorcache_test.go
+++ b/pkg/policy/selectorcache_test.go
@@ -610,6 +610,21 @@ func (ds *SelectorCacheTestSuite) TestIdentityUpdatesMultipleUsers(c *C) {
        c.Assert(len(sc.selectors), Equals, 0)
 }

+func (ds *SelectorCacheTestSuite) TestSelectorManagerCanGetBeforeSet(c *C) {
+       defer func() {
+               r := recover()
+               c.Assert(r, Equals, nil)
+       }()
+
+       selectorManager := selectorManager{
+               key:              "test",
+               users:            make(map[CachedSelectionUser]struct{}),
+       }
+       selections := selectorManager.GetSelections()
+       c.Assert(selections, Not(Equals), nil)
+       c.Assert(len(selections), Equals, 0)
+}
+
 func testNewSelectorCache(ids cache.IdentityCache) *SelectorCache {
        sc := NewSelectorCache(testidentity.NewMockIdentityAllocator(ids), ids)
        sc.SetLocalIdentityNotifier(testidentity.NewDummyIdentityNotifier())

Fix potential nil pointer dereference in SelectorManager implementation

/test

/test


Change slide over position and possible options

Hi @joogps,
I wanted to know how to make the slide to open from different position instead from the default bottom position.
Sorry if I have missed it from the SOCManager, I didn't get it very well.
Regards,

Sorry for the late response! I will be refactoring the codebase and adding that option. :D

Thank for the sarcasm.
unstar and remove from my projects ;)


Seems outdated and not active project anymore?

Hi @victorkvarghese
Quite an impressive project this looks to be.
But seems outdated and not an active project anymore? Do you have any further plans? Can you please update it to v0.62 RN.
This has the potential of becoming as big as this masterpiece (25.1K stars)

Hi @Jayiitb
I got really busy and couldn't update 2 projects simultaneously.  I have updated the project to 0.62.2 with some upgrades over the top. Enjoy!
Fixed with commit


Add Reentry Particle Effect from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of pizzaoverhead, to add Reentry Particle Effect to CKAN.
Mod details:
name = /mod/819/Reentry%20Particle%20Effect
author = pizzaoverhead
abstract = This mod activates an unused stock particle effect for reentry, featuring a plasma trail and sparks.
license = All Rights Reserved
Homepage = http://forum.kerbalspaceprogram.com/index.php?/topic/143040-113-reentry-particle-effect/
description = This mod activates an unused stock particle effect for reentry, featuring a plasma trail and sparks.
Please direct questions about this pull request to pizzaoverhead.

Clear intention by this mod author to add this to CKAN.


Update azure-activedirectory-identitymodel-extensions-for-dotnet to 7.0.3

UPDATE: (I don't know how to convert this to a Draft PR). Please don't merge this until we get confirmation this is the exact version we want to take and not something newer.
@mkArtakMSFT @halter73 @MichaelSimons @mthalman

/backport to release/8.0

@eerhardt this has now been approved by Tactics. This should be good to be merged.


Added test for uninstalling local and installing PyPI version.

Yay, it worked! Not going to cheat on this one, @Chevalant. Please merge :smile:


feat: support view-test-artifacts rather than a single artifact, needs work

Description of changes
Add a utility to make it easier to look at multiple failing test artifacts in parallel.

CDK / CloudFormation Parameters Changed
N/A
Issue #, if available
N/A
Description of how you validated changes
Verified locally.
Checklist

[x] PR description included

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1667 (5c239f1) into main (d14d82f) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main    #1667   +/-   ##
=======================================
  Coverage   63.76%   63.76%
=======================================
  Files         370      370
  Lines       21772    21772
  Branches     5211     5211
=======================================
  Hits        13882    13882
  Misses       7168     7168
  Partials      722      722

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

Looks like the lint job in cb is failing, can you run prettier on the changed file to fix it?


fix: Fixes Codecov (test)

Signed-off-by: Andrii Soluk<EMAIL_ADDRESS>

Codecov Report

Merging #23 into main will decrease coverage by 5.61%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   77.48%   71.87%   -5.62%
==========================================
  Files           3        1       -2
  Lines         382      160     -222
==========================================
- Hits          296      115     -181
+ Misses         49       27      -22
+ Partials       37       18      -19

Impacted Files
Coverage Δ

component/storage/couchdb/store.go
71.87% <ø> (ø)

component/storage/mongodb/mongodb.go

component/storage/mysql/store.go

Continue to review full report at Codecov.

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


launching the debugger in tmux never works first time

I have the setup that was described by João Moreno (@joaomoreno) in his blog post https://medium.com/@joaomoreno/persistent-terminal-sessions-in-vs-code-8fc469ed6b41
Whenever I open a terminal tab in VS Code, i enter a tmux session (often attaching to an already present tmux session)
I love this setup! Among other things, it allows all the window control of tmux and it enables me to restart VSCode without losing my terminal session.
Debugging has worked fine in this setup for a long time. With the setting "console": "integratedTerminal", the debugger starts inside the tmux shell, which is just what I want.
After 1.20.0, this stopped working for me.  When I start debugging by pressing F5, the debug command seems to flash briefly in the terminal tab, then it shows tmux. But the debugging never starts.
This problem has been partially fixed by #43683!
However, even after this fix, the following problem remains in the insiders build:
Debugging never works the first time I try to run the debugger (when the integrated terminal has number “1” and is called “sh”), the only thing that happens then is a quick flash of the debug command and that another terminal is created (“2: Node Debug Console”).
Once the “2:Node Debug Console” exists, I can successfully run the debugger.
Thanks for an otherwise great product!

VSCode Version: 1.21.0-insider 76a4a71434d6cdee69fa1d8bdc7533d09598d53 2018-03-02T11:45:30.531Z
OS Version: macOS 10.13.3

Steps to Reproduce:

Have the tmux setup described in the blog post
Have a launch.config with "console": "integratedTerminal" (which worked previously)
Hit F5
A new terminal is created, but the debugger never starts. After a while, the debugging times out with the dialog Cannot connect to runtime process, timeout after 10000 ms - (reason: Cannot connect to the target.: connect ECONNREFUSED <IP_ADDRESS>:5829).
Dismiss the dialog by hitting `Cancel``
Hit F5
Debugging starts as ususal

Does this issue occur when all extensions are disabled?: Yes

@could you please try the latest Insiders?
BTW, I've filed https://github.com/Microsoft/vscode/issues/45300 for the blank drop down.

@weinand Yes, now it works perfectly!
Thank you!

Thanks for verifying this. I'll add the "verified" label.


Allow for site to site configuration

As title says.
I would love to create a site to site network if I want to attach an entire site to the created network. The use case would for example be if you have a network where you have devices that cannot run wireguard (like a old NAS) and you still want resources from it available. Simply create a route on a router and/or server to the VPN gateway and you get the site connected.
This would also mean that the peers that are allowed to access the sites network should get that routing information.

I would also like this. I was asking about it in the matrix/discord chat room with no response so far.

This would probably require recording subnet route information together with peers, and setting up routes to respective peers' subnets whenever the interface is brought up or more peers are added.

Related to https://github.com/tonarino/innernet/issues/30 since it's a similar functionality request.

We have this exact use case at work. A(n extremely) simplified version of what have/need is:

There is a service running on a server that utilizes innernet
There is a gateway which also uses innernet and can communicate with the service
This gateway is responsible for accepting connections from devices that either cannot use Wireguard or innernet directly, but need to communicate with the service. We'll call those "external clients"

Diagram below:

This is super simple to accomplish if all of the "extenral clients" can be placed on a private LAN that happens to fall somewhere in within the innernet-server's /16 subnet. All that has to be done is add an allowed-ips statement to the services innernet interface for the whole subnet of "external clients" (and the gateway must have IP forwarding enabled).
Enabling IP forwarding is outside the perview of innernet, I wouldn't suggest that to be taken up here. However, a flag/command to add a subnet to a peer is totally possible (and probably not too difficult to add either).
The workflow I'm imagining would look something like this. First the boilerplate to just get the diagram above up and running (without the newly proposed site-2-site):
# On "innernet-server"
innnernet-server new
  create "foo" network
# At an "admin" location
innernet add-cidr foo
  create a CIDR "gw" for all the gateway machines
innernet add-cidr foo
  create a CIDR "svc" for all the service machines
innernet add-peer foo
  create a peer for the service
innernet add-peer foo
  create a peer for the gateway
innernet add-association foo
  associate gw <==> svc

# On the gateway machine
 - [..install and start innernet on the foo network..]
 - [..enable IP forwarding..]

# On the service machine
 - [..install and start innernet on the foo network..]

I see two options to enable this site-2-site idea, both have pros and cons:

Option 1: A command to add "IPs" to a peer which would be added to the allowed-ips section of that peer

For example, the "gateway" peer on the "service" machine would get an extra allowed-ips statement

Option 2: A command to add "IPs" to a CIDR, which would be like option 1, but apply to all peers assigned to that CIDR

Further, the "IPs" mentioned above could be implemented as either:

A manual IP/subnet written out
An existing CIDR on the innernet network

I'd suggest a minimum viable product that is most flexible and could be built-upon later. For me, that's only dealing with peers (i.e. not worrying about all peers in a CIDR), and selecting from existing CIDRs.
i.e. The final steps to get it working would look like:
### These are the new steps from this feature request ###

# Create a CIDR to hold the phantom "external clients"
innernet add-cidr foo
  create a CIDR "ext" for the external clients (no actual peers will be created)

# A new CLI option to enable Site-2-Site
innernet add-peer-ips foo
  Peer? (select Peer) we pick "gateway" peer
  CIDR? (select CIDR) we pick "ext" CIDR
  Confirm?

Now assuming the service machine does a fetch we should see the following:
#
wg show foo

interface: foo
  public key: ...
  private key: (hidden)
  listening port: 38268

peer: ...                                           <---- Gateway
  endpoint: ...:57940
  allowed ips: <IP_ADDRESS>/32, <IP_ADDRESS>/24       <----- Notice the /24 "ext" CIDR
  latest handshake: 1 minute, 16 seconds ago
  transfer: 3.58 KiB received, 5.36 KiB sent
  persistent keepalive: every 25 seconds

peer: ...                                            <--- Innernet Server
  endpoint: ...:51820
  allowed ips: <IP_ADDRESS>/32
  latest handshake: 1 minute, 25 seconds ago
  transfer: 97.11 KiB received, 43.94 KiB sent
  persistent keepalive: every 25 seconds

This could be represented in the SQLite DB as a new table of peer-ips similar to associations, which just lists cidr primary keys to peer primary keys.
For completeness sake, the wg command to do this manually on the "service" peer is:
$ wg set foo peer '...' allowed-ips <IP_ADDRESS>/32,<IP_ADDRESS>/24

Where the 100.1/32 CIDR is the "gateway" and 200.0/24 is the "ext" CIDR.

I would like to express my interest of managing allowed-ips through innernet as well. We currently have a custom Wireguard configuration in place to access documentation via an IP whitelisted server. We do this by adding the destination IP to the allowed-ips of the peer configuration of the whitelisted server. That server is configured to do ip forwarding.
Right now using innernet I can only get it working by temporarily disabling the innernet service (so peer configuration isn't overwritten) and then running the $ wg set foo peer '...' allowed-ips <IP_ADDRESS>/32,<IP_ADDRESS>/24 command as @kbknapp mentioned in combination with adding a custom route ip -4 route add <ip> dev <interface>. But this is less than ideal.
As I have no idea on where to get started on a PR for this I'm hoping someone with more knowledge of the project can get us started.

I think I'm in a similar spot to @wezzle where I need some bespoke Wireguard configuration/local system routes to coexist with the Innernet-managed ones.
I can understand the complexity in implementing such a feature, but am curious of if there's simpler-to-implement options that would enable smoother workarounds, like lifecycle hook scripts that could be invoked to apply the manual configuration needed for the peers with subnets behind them. I'd be fine treating the site-to-site peers as snowflakes with static IPs if they could still communicate with the automatically-managed ones.
Curious if anyone else had figured out workarounds for this gap in functionality which is blocking me from migrating from Tinc.


Adding support for tfgit and tfvc as artifact source

@TingluoHuang any concerns before this is merged?

Before you merge, can you please squash so history remains usable when investigating issues?
I'm still interested to hear if @TingluoHuang has any final concerns. He has a longer history with the agent.

squashed all the last commits. The commits look logical now.

I am good with the changes other than the small comment i left. :)

Merging this change.


Predictive code generation

Adds a feature flag that enables the predictive code generation triggered by user actions like opening a file in the editor, saving the document or closing the file.
Currently as a consequence of any of the above actions only code generation will be triggered by default, later we can add support for automatic documentation generation and or trigger any other actions.

This feature reminds me some other code gen service like tabnine provides the ability to connect to code base, and so some pre-training.

Seems like there has some conflicts


Remove printf statments?

Hi
Are the printf's necessary? They would only slow things down if activated several times, but I guess they are there for debugging purposes at the moment?

I'm not author, but as far as I understand, this printf statement happens only on open() call for /dev/input/event1, which happens only at the start of Remarkable application or sporadically. The heavy lifting happens in read() call.


SNS Number type header throws InvalidParameterValueException exception

Type: Bug
Component: "SNS"
Describe the bug
Using dependency spring-cloud-starter-aws-messaging: 2.4.2
I want to use aws SNS as topic for messages and consume them using SQS subcription.
you can find app sample here https://github.com/maniish-jaiin/spring-cloud-messaging-sample and use it for the reproducing the issue.
Steps:

lauch the app according readme
call - http://localhost:8080/create-order
got com.amazonaws.services.sns.model.InvalidParameterValueException

So comments about investigation.
When I send the message, it forms next headers: id, contentType and timestamp.
Awspring core tries to convert it (class io.awspring.cloud.messaging.core.TopicMessageChannel row 116 for Number type). Then in my case it's set attr dataType = "Number.java.lang.Long" (see row 177) for 'timestamp'.
But mazonHttpClient returns exception. It could not handle this type and throw the exception:
com.amazonaws.services.sns.model.InvalidParameterValueException: The message attribute 'timestamp' has an invalid message attribute type, the set of supported type prefixes is Binary, Number, and String. (Service: AmazonSNS; Status Code: 400; Error Code: ParameterValueInvalid; Request ID: 9K79WVXJT35GUXEQ9PEBY318HEN7L576NBZS1EFV0R4ZOXYJ83ZU; Proxy: null)
When I thied change dataType to Number on debug - everything passed.
Sample
You cand tried it out with app https://github.com/maniish-jaiin/spring-cloud-messaging-sample

I've noticed it was fixed already


Drawtiles color offsets

Rebased version of this: https://github.com/openfl/openfl/pull/992

Adds the ability to specify color offsets (not just color tint) in drawtiles API. I added the extra parameters to the end so that it's just purely extra optional information to add, otherwise the new code should have no effect on existing drawtiles commands.

Quick overview of how this works:
Add 4 color offset floats to the end of the drawtiles data.
Add the TileSheet.TILE_TRANS_COLOR flag.

Each State now always has its own colorTransform that is only ever modified to match what's passed in, rather than passing either null or a pointer to the world transform. This colorTransform is never null, and a new skipColorTransform property covers that "null" case.
This is important because the State object needs to retain the color offset information until rendering, and passing in a global colorTransform object or setting it to null destroys that connection.
SpriteBatch also has its own colorTransform instance that is only ever modified, not replaced. This way it's easier to keep track of where colorTransform objects are winding up. They are never passed around, only used to copy information from one another.
The actual rendering looks at the state and sees if skipColorTransform is true, if so, it passes a null to the currentState pointer, otherwise it passes the actual colorTransform.

merged :smile:


Drink-Order

Hey @nashiruddin47! Thanks for opening an issue.
Could you provide more information on the problem you faced or any suggestions you have.

I'll go ahead and close this, but feel free to comment here or open a new issue if you have any more information or need any help. Thanks!


The repo items in explore page have duplicate icon

Description
As title.
The problem is
https://github.com/go-gitea/gitea/blob/c37f95fc56fa3fc8f51610af6915ac39839d7d19/templates/explore/repo_list.tmpl#L13-L40
and
https://github.com/go-gitea/gitea/blob/c37f95fc56fa3fc8f51610af6915ac39839d7d19/templates/repo/icon.tmpl#L1-L14
have duplicate (but not same) check.
Screenshots

Gitea Version
latest codebase
Can you reproduce the bug on the Gitea demo site?
Yes
Operating System
No response
Browser Version
none

The same thing happens with other pages.


python311Packages.hologram: pull patch to fix tests

Description of changes
The tests were failing due to dataclass changes in Python 3.11 (https://github.com/python/cpython/issues/88840). Pulling in https://github.com/dbt-labs/hologram/pull/58.
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Converting into a draft so that I can merge it with https://github.com/NixOS/nixpkgs/pull/244135/commits/18601b84248ffec5f490d3481f6005c7f83b5bca when it hits master.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1053


fix: handle golang panic to prevent the program shut down

Codecov Report
Attention: Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.

Project coverage is 89.38%. Comparing base (bb4c1c7) to head (ab9fdfd).
Report is 5 commits behind head on main.

Files with missing lines
Patch %
Lines

pyfastexcel.go
71.42%
1 Missing and 1 partial :warning:

:exclamation:  There is a different number of reports uploaded between BASE (bb4c1c7) and HEAD (ab9fdfd). Click for more details.
HEAD has 5 uploads less than BASE

Flag
BASE (bb4c1c7)
HEAD (ab9fdfd)

6
1

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #144      +/-   ##
==========================================
- Coverage   96.97%   89.38%   -7.59%
==========================================
  Files          26        6      -20
  Lines        2872      801    -2071
==========================================
- Hits         2785      716    -2069
+ Misses         64       61       -3
- Partials       23       24       +1

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


‘User Story Split’ option is not working (showing) during sprint planning day at some point of time

We observed that during Sprint planning days at some point of time, ‘User Story Split’ option is not working.
Please find the attached screen shot which was taken on 30th June where the Split option is not seen.
We are facing same issue from long back at sprint planning day the issue happens and at during specific time.
Split
user-story-split-option-not-working-on-sprint-panning-day.PNG
Extension details: -
Extension details
Split a work item to track work that will continue into the next sprint.
Publisher
Microsoft DevLabs
Installed version
0.1.11 (Latest)
Last updated
Apr 13, 2023 at 9:56 AM GMT+5:

Thanks for reporting, can you describe your environment? Can you check if you get any errors in the browser console or network log?

If the issue persists, please inform me.


people: 1.1.2-0 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository people to 1.1.2-0:

upstream repository: https://github.com/wg-perception/people.git
release repository: https://github.com/OSUrobotics/people-release.git
distro file: melodic/distribution.yaml
bloom version: 0.6.6
previous version for package: 1.1.1-0

(waiting on sync https://discourse.ros.org/t/preparing-for-melodic-sync-2018-09-04/5919)


Added security protocol at top level

LGTM -- so let me sum up my understanding of what will be in cdappconfig.json -- we have one of these 3 cases:
1: default: securityProtocol key is present but is an empty string... (or... will the key be entirely absent? Unclear to me) 2: securityProtocol key is set to "SSL" when kafka provider is in app-interface mode and forceTLS is true in the ClowdEnvironment config. 3: securityProtocol key is set to "SASL_SSL" when kafka provider is 'managed', managed_ephem, or strimzi

Correct on all accounts, I believe the key will not exist if empty


Feature/103

알림과 알림에 딸린 힌트 api 구현
⛓ Related Issues

close #103

📋 작업 내용

[x] ~ 알림 api 구현
[x] ~ 힌트 api 구현

Codecov Report
Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (5d5c5e1) 97.95% compared to head (9f7f227) 97.34%.

Files
Patch %
Lines

src/components/alarm/AlarmItemBox.tsx
50.00%
4 Missing and 3 partials :warning:

src/hooks/api/useGetInfiniteAlarms.ts
88.88%
2 Missing :warning:

src/lib/api/ApiService.ts
66.66%
2 Missing :warning:

src/components/alarm/index.tsx
87.50%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
- Coverage   97.95%   97.34%   -0.61%
==========================================
  Files         138      142       +4
  Lines        1663     1735      +72
  Branches      177      185       +8
==========================================
+ Hits         1629     1689      +60
- Misses         34       43       +9
- Partials        0        3       +3

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


Change "how this works" to FAQ and describe the alert mechanism there

Also reference from the alert emails

Then, request a review (not only of this page) to the usual suspects :)

Done, but I'm letting this open since I suspect the text will need improvements. Waiting for feedback on this aspect.

I read the text and I find it very good 👍


Sto ref test v3

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-759

Preview environment: https://ci.pr.harness.io/pr-759


Consider adding float right and left

I have been using bootstrap for two years now , but I discovered lately that allowing the ability to float left and right would make bootstrap easier to use.

We have float utilities already.


Adding derive() macro support to SharedStructs

This PR adds support for #[derive(...)] macros to SharedStructs.
Example
#[swift_bridge::bridge]
mod ffi {
    #[swift_bridge(swift_repr = "struct")]
    #[derive(Copy, Clone)]
    struct Foo {
        field: u8,
    }
}

This will successfully add a derived Copy and Clone trait to the struct declared.

Just missing a codegen test and an integration test (don't need the Swift side .. just a Rust integration test is fine for now and in the future we can emit a clone function on the Swift side...)
See the following files in https://github.com/chinedufn/swift-bridge/pull/194/files for an example that you can copy/paste/modify

crates/swift-bridge-ir/src/codegen/codegen_tests/derive_attribute_codegen_tests.rs
crates/swift-integration-tests/src/enum_attributes/derive.rs

See the following files in https://github.com/chinedufn/swift-bridge/pull/194/files for an example that you can copy/paste/modify

crates/swift-bridge-ir/src/codegen/codegen_tests/derive_attribute_codegen_tests.rs

@chinedufn should I wait for that PR to be merged to add the tests on this file you mentioned, or should I create a separate file like derive_struct_attribute_codegen_tests.rs?

Creating a file in this PR sounds good!

Creating a file in this PR sounds good!

just updated with the tests

Looks good. I'll merge once tests are passing (looks like you need to run cargo fmt)

Thank you!

This would be great to have on enum too


Isn't ideal concurrency number of cpus minus 1?

I see sharp sets concurrency equal to number of CPUs. In a high load system, wouldn't it block event loop? I think it should be set to number of cpus - 1.

If "event loop" in this context refers to the Node.js event loop then the short answer is no, it does not block.
Did you see https://sharp.pixelplumbing.com/api-utility#concurrency which explains a little more about concurrency?

Gets or, when a concurrency is provided, sets the number of threads libvips' should create to process each image. The default value is the number of CPU cores. A value of 0 will reset to this default.

The maximum number of images that can be processed in parallel is limited by libuv's UV_THREADPOOL_SIZE environment variable.

I am talking about a situation in which say all cores are busy processing image, won't the scheduling of nodejs event loop hampered?

Each thread's share of CPU time depends on the OS scheduler, e.g. Linux uses https://en.wikipedia.org/wiki/Completely_Fair_Scheduler

Okay sounds good. Thanks a lot for help.


TS tunnel SDK SSH reconnection

Refactor ts tunnel SDK.
Extract connection flow to relayTunnelConnector. It implements TunnelConnector
and uses TunnelSession interface to abstract tunnel client and host.
Extract common host and client interface into TunnelConnection.
Extract common client and host code into TunnelConnectionBase class.
TunnelConnectionSession extends TunnelConnectionBase and drives
connecting to the tunnel.
Extract relay connectivity into mixin class TunnelRelaySessionClass.
Extract sshSession property and disposal into mixin class TunnelSshSessionClass.
Get status code from failed web socket connection.
Port retrying event changes from CS SDK.

Reconnection unit tests are coming soom.

Still TODO: update the @vs/vs-ssh and @vs/vs-ssh-tcp npm packages version once the new version has been published.


Fix errors and logging

added comment on each commit and please check log_to_discord() function because I don't know  why it works now after I remove one of the if blocks

thanks :)


net: opentread: Implemented otPlatRadioGetRssi

Implemented blocking OpenThread otPlatRadioGetRssi api function using
no-blocking energy scan function from radio driver api interface.
Signed-off-by: Tomasz Konieczny<EMAIL_ADDRESS>

@MarekPorwisz I'm not able to add you as a reviewer, but please feel free to leave a review.


Sanjusruthi

Komalafk

@sanjumn21

Hello, Is there anything I can do to help?

Yes I feel someone is hacking or remote access to my accounts.  What do I do


Harvest Golem Mystery: Incorrect quest item referenced

Please, provide a clear description what the bug is:
On the quest "The Harvest Golem Mystery", the part where you are sent to gather "intact runes" the pfquest-turtle addon seems to display the wrong information as it references "cracked rune" on the quest where it should reference "intact rune" as can be seen in the following screenshot.

Steps to reproduce the behavior with as much detail as possible:

Accept the quest part of "The Harvest Golem Mystery" where you are sent to gather intact runes.
Make sure to have the pfquest-turtle addon installed.
Observe as the cracked runes are referenced on the quest part where intact runes should be references

IDs of affected creatures, items, quests or spells with a link to the relevant page:
Item: https://database.turtle-wow.org/?item=60677
Quest: https://database.turtle-wow.org/?quest=40471
Expected behavior. Describe how it should work:
As can be seen in this screenshot from the database ...

The drop rate for intact runes are 40% but for some reason pfquest-turtle references the cracked runes which have an 80% drop rate. Therefore, what should be happening is that pfquest-turtle should reference the correct item in the database.

(posting here as directed by Shagu)
I'm also seeing the same thing:
for the quest, we're supposed to collect 'Intact Animation Rune':

Mouse hover mob 'Harvest Golem' shows wrong item:

Also shows wrong item for mob 'Harvest Watcher' (which should not show for this quest, but previous quest):


keep ResponseHeaderTimeout value for now

https://github.com/moby/buildkit/pull/3995 will revert our upstream contrib which set this value; it has been reverted, but we don't want to revert it in our fork (until we have a more appropriate fix).

@alexcb as long as we're making a change to buildkit, should we (as we discussed last week) git rm session/auth/authprovider/podman.go (in another commit)?

@alexcb as long as we're making a change to buildkit, should we (as we discussed last week) git rm session/auth/authprovider/podman.go (in another commit)?

We can do that, but that seems unrelated to this PR, which is simply updating a comment, which will intentionally cause a merge conflict rather than auto-merging the revert (which we don't want).
Since this PR doesn't change any code, I wasn't planing on updating the referenced buildkiit version in earthly.

Yeah, either way is fine by me. I just recall that we said we'd include the removal of that file the next time we made a change to our fork. But we could just do it in another PR as well.
Much like this change, that change won't have any impact (nothing is using the podman auth provider), so we don't need to update the referenced buildkit version. :shrug:


Update the status of the "In Progress" spec to be v1.0 (ledger agnostic) and create v0.1 when v1.0 is complete

As discussed at the Working Group meeting on 2022.08.01, we are going to change the version of the specification that we are working on to be version v1.0, since we think we have a very good handle on making AnonCreds ledger agnostic, and expect to have life deployments Real Soon Now.
This changes are original plan of creating the v0.1 "as is today with Hyperledger Indy" version of the spec first, and then creating a v1.0 version that makes the specification ledger agnostic, but changes little else.  Now, we will create the v1.0 version first, and when ready, evolve it (devolve it?) to create and publish the v0.1 "as is today with Hyperledger Indy" version.
In this ticket, we will track the items that have to change to get the v0.1 spec. from the v1.0 spec.

For the v0.1 spec, at minimum we need to replace the "AnonCreds Objects Methods" content with text that references the Hyperledger Indy AnonCreds Objects Method only. This may refer to external documents, but should cover the format of the identifiers, and the registration and resolution specifications.  Since we want to still support this method in the v1.0 specification, it may be that we just change this spec. to only point to that method, and remove references to the registry.

Completed.  The 1.0 spec is both the “ledger agnostic” version and aligned with the current anoncreds-rs implementation.


Delete all sessions when calling Pow.Plug.delete_user/2

Prompted by https://github.com/danschultzer/pow/issues/386#issuecomment-578016442
Currently Pow.Plug.delete_user/2 calls do_delete/2 for the plug, which only deletes the current session. However it should clear all sessions for the user. Only Pow.Store.CredentialsCache knows about related sessions which makes this tricky. Might work if a config value is passed on e.g. delete_all_for: user.
PowPersistentSession might interfere with this. The way to resolve that is to delete all persistent session tokens that has a session fingerprint that's deleted. There could be a method in PowPersistentSession.Store.PersistentSessionCache to do that, but it might also be a good idea to be able to look up all persistent sessions for the user.

May I ask a question about this?
I have a custom controller in an admin panel to manage users (list - delete). I will maybe add new - create actions later since with the Invitation extension it is not possible to set a role along the invited user email.
But let's come back to my question about user deletion. If I understand correctly Pow.Plug.delete_user/2 is not for the case when an admin want to delete another user....
So in that case would just calling something like Repo.delete(user) be enough? I mean can I just delete the user from the database without any issue?

If I understand correctly Pow.Plug.delete_user/2 is not for the case when an admin want to delete another user....

Correct, it's only when the user deletes the current user signed in, and deletes the associated session.

I mean can I just delete the user from the database without any issue?

The session will still be active, so if you don't check if the user exists in the DB the user may be able to continue using the app until their session expire. #563 would resolve this.
To remove all the sessions for the user it's necessary to call Pow.Store.CredentialsCache.sessions/2 and then Pow.Store.CredentialsCache.delete/2 for each of them.
I wonder if permitting a second argument with the user to be deleted in Pow.Plug.delete_user/1 makes sense.

Thanks for this clarification.

I wonder if permitting a second argument with the user to be deleted in Pow.Plug.delete_user/1 makes sense.

That would be great! ^^

If I understand correctly Pow.Plug.delete_user/2 is not for the case when an admin want to delete another user....

Correct, it's only when the user deletes the current user signed in, and deletes the associated session.

I mean can I just delete the user from the database without any issue?

The session will still be active, so if you don't check if the user exists in the DB the user may be able to continue using the app until their session expire. #563 would resolve this.
To remove all the sessions for the user it's necessary to call Pow.Store.CredentialsCache.sessions/2 and then Pow.Store.CredentialsCache.delete/2 for each of them.
I wonder if permitting a second argument with the user to be deleted in Pow.Plug.delete_user/1 makes sense.

Hey @danschultzer ,
I ran into this comment while I was looking for a way to clear up sessions in case a user updates their password.
What config do I need to pass to these to query both the sessions and the credentials?
If I do CredentialsCache.sessions([backend: "backend here"], user)
and I delete, the persistent_session stays around and the user is not logged out.
We have a liveview page and getting the config from the socket was not working.
Thanks

@benonymus I ended up doing something like this to clear out the session + the persistent session. Unfortunately, due to how persistent sessions are stored, there's no way except full enumeration to clear it out:
  def before_respond(PowResetPassword.Phoenix.ResetPasswordController, :update, {:ok, user = %{id: user_id}, conn}, _config) do
    session_key = :erlang.term_to_binary([UserService.User, :user, user.id])

    # Delete all active sessions
    from(
      sesh in UserService.PostgresPowStore,
      where: sesh.original_key == ^session_key
    )
    |> Super.Repo.delete_all()

    # Delete the persistent session entries
    from(
      pow in UserService.PostgresPowStore,
      where: pow.namespace == "persistent_session"
    )
    |> Super.Repo.all()
    |> Enum.each(fn entry ->
      case :erlang.binary_to_term(entry.value) do
        {%UserService.User{id: ^user_id}, _opts} ->
          from(
            pow in UserService.PostgresPowStore,
            where: pow.key == ^entry.key and pow.namespace == ^entry.namespace
          )
          |> Super.Repo.delete_all()

        _ ->
          nil
      end
    end)

    {:ok, user, conn}
  end

I think enumeration that doesn't load all (batch it) and using the actual store functions would make this better. I don't care about being generic though, so just winged it a bit :)


feat: usage keys validation

Fix #989

I don't feel that I know exactly what should be where regarding producing an output. So I just added a placeholder message generator in run.go that I guess is probably the wrong place. I was not sure to put it here or in the table.go.

This is awesome @sinabakh! I think logging a warning is fine as we can still run so no need to error (e.g. for the Plan JSON API, which runs the CLI, the run can succeed with warnings).
Can we use the yellow warning style we use for other warnings? e.g. see the following screenshot for when fields and --format=json are used together. @aliscott I guess we're ok with multiple warnings per run for now? Looks like we'll have a max of 2, we can think about collecting them into 1 warning later on.

Can we also change the warning text to the following so it's 1 comma separated line? We use the word "parameter" in docs.
Warning: the following usage file parameters are invalid and will be ignored: request_duratio, mnthly_requests, typo1

Longer term we can think about validating the values and maybe even the resource paths, but for now this is a great start.

Thanks @sinabakh! I'm closing this since I've merged your commits in https://github.com/infracost/infracost/pull/1011. I've updated this to work with the new way we're structuring hierarchical usage data, and have made the UI changes @alikhajeh1 suggested.


PDFJS renders a blank canvas in node environment, if scale >= 9.49

Attach (recommended) or Link to PDF file here:
c53a656b-551d-4c24-bfb6-f994ef3a6d4b.pdf
Configuration:

Web browser and its version: It works fine in web-browser environments
Operating system and its version: Ubuntu 22.04.2 LTS, node v18.15.0
PDF.js version: 3.11.174
Is a browser extension: no

Steps to reproduce the problem:

I've made a minimalistic repo to replicate the problem. Clone this https://github.com/soadzoor/node-pdfjs
npm install
node index.js
Note that a PNG is getting generated with a section from the PDF (FloorplanSection.png)
Now modify the viewport within index.js (replace the scale value with a number above 9.49, or simply uncomment the section from line 31 to 36)
Note that the generated PNG (FloorplanSection.png) is blank now, it contains only white pixels, but it shouldn't.
If I run the same exact code (meaning, same PDF, same viewportsettings, etc., only the rendering method is different, because node.js doesn't have a built-in renderer and/or canvas) in a web-browser environment, the PDF section gets rendered properly. Here's the web-browser version: https://jsfiddle.net/rwdysagu/ , scroll down to the bottom for the code, the base64 data is kind of huge.

What is the expected behavior? (add screenshot)
PDF.js should work the same within node.js, and a web-browser
What went wrong? (add screenshot)
On the node.js version, PDF.js renders a blank image, while the web-browser renders a visible section from the PDF.
They both use the same PDF, and the same exact settings, viewport, etc:

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):
Render is blank if run from node.js: https://github.com/soadzoor/node-pdfjs
Same thing works fine within a web-browser: https://jsfiddle.net/rwdysagu/
Note, that it works as expected until you change the scale on the node.js version to be above 9.49. Any value lower than that gives you the same exact image as the web-browser version (if everything else is the same, of course).

In Node.js we rely on the node-canvas package for rendering, see https://github.com/Automattic/node-canvas
Hence if this is limited to Node.js that sounds like a bug in that package, rather than in the PDF.js library. (Note also that Firefox is the primary development target here.)

For the record:
Since node-canvas was replaced with @napi-rs/canvas, this problem seems to be solved!
Also, for the record, there are some minor differences in rendering output between the 2 canvas implementations. Here's an example:
node-canvas:

@napi-rs/canvas:

You can see the slight differences if you open up both of these in a new tabs, and switch between the tabs.
I couldn't see which one is "better", but they're a bit different for sure.


Simulated Environment: Shimmy (Farama Foundation API conversion tool)

I am interested in writing a tutorial for using langchain with Shimmy, an API conversion tool allowing many popular reinforcement learning environments to be used natively with PettingZoo and Gymnasium.
Since there are already PettingZoo and Gymnasium tutorials/wrappers (https://python.langchain.com/en/latest/use_cases/agent_simulations/petting_zoo.html), it would not require significant new code, but I think it would be a really helpful feature to show people that they can for example load DM Control/DM Lab environments in the exact same way. OpenSpiel provides text-rendered board games, which would be a great feature to show off with a chatbot.
Making this issue just to confirm it is something that the developers would be interested, but I am planning to work on the tutorial later today and if it's not possible to add directly to this repo I will add it to Shimmy (I am the project manager of Shimmy and PettingZoo).
I would be interested in fleshing out integration on PettingZoo's side as well, if there are any extra features which we could add in order to better support langchain we would love to do so. If there is interest in adding training library support I am also interested in working on tutorials to have langchain load a trained model using standard libraries like stable-baselines3, RLlib, or a lighter library like CleanRL.

cc @mbchang


Add Hover

Description:
I would like to request the addition of Hover effect feature
when the text is too long we need to show like text.... like when user hover it will show full text it is useful,
Thanks.

hope fully this will help a lot of users


Add hotlinking anchor tags to each table header

This allows you to send urls like http://<IP_ADDRESS>:5500/index.html#table.shopping to your friend, and have them pointed directly to the shopping table.
Thanks for making this.
ps, my vscode is trying to make a lot of formatting changes. Perhaps either linting the html or providing a linting config would be appreciated :)

I've updated it to instead use table of contents, rather than hotlinking the <th> tags.
This also moves the <script> to the bottom of the html. This way the renderer will parse the entire html block first, and then the javascript. I've also taken it off the <body onload=""> as this was causing the page to "refresh" when onload finished, everytime.

I don't use a linter since I custom format all the html for readability.
i.e. I'll align columns with extra whitespace, insert <meta/> for alignment, etc.
The sidebar TOC is a great idea!
I'll take a look at this later tonight once I verify the 1-column and 2-column mode work correctly across desktop and mobile.  I test resizing the browser window width on these monitors/resolutions and platforms:

desktop 2560x1440
desktop PC 2160x3840
mobile iOS

Appreciate ya!
The TOC won't support resizing as is :( Mainly because I used a fixed location settings in CSS. I could redo this to support resizing, just would take more time.

I don't use a linter since I custom format all the html for readability.
i.e. I'll align columns with extra whitespace, insert  for alignment, etc.

For me personally, this made it harder to assist :( But, its your project I am more than happy to adhere to your style :)
And as for

The hack of using a timer shouldn't be needed -- that's what onLoad is for but I'll take a look if that provides a better solution.
Yea the only reason I did this is because of how the renderer loads the dom. Since I was injecting hash tags, I needed more control over the order the process was ran in. Since I moved to the Table of Contents view, that became let necessary to do.

Gem table finally got added so I can circle around to this tonight.  I'll probably clone your repo and do some 1 and 2 column testing.  My main concern is making sure a screen width of 1920 looks good.
Only deal breaker right now (trivial as it is) is the brace style.  I use Allman style.  Please fix that.
I'm actually excited about this PR as this make linking SO much more user friendly -- for others and myself! :-)


Get genesis validation checks running in CI.

Prereq for merging #489
The checks for genesis validation require a specific set of tool versions to be installed in the docker image in which the validation script runs - some of these tool versions are known at build time, whereas some other tool versions such as go is derived from the go.mod file  at the commit at which the validation runs. We currently use the image ci-builder:v0.49.0 for running the all jobs in superchain registry. The image is built from this Dockerfile. So far, we have explored two options to install the correct tool versions to get the genesis validation checks to run on CI runner:

Install the required tool versions (nvm, go etc.) as pre-req steps in the Circle CI job that is being executed. With this approach, we are observing that the tool versions/settings aren't taking effect when running the actual validation.

Modify the base image dockerfile to install the required tools in to the base image. I have been unable to build the image locally so far, even without changes (hitting timeout errors and such).

Continuing to explore both options.

The list of requirements are:
foundry
gvm
nvm
just
pnpm
yarn


False positives: Requirement 5 - Authentication and Authorization of Application Resources

Describe the bug
Te tests results says that "One or more endpoints returned a <400 status code without authentication information. This may indicate that your app is not performing authentication and authorization checks".
Example 1:

https://pfc-develop.cloud.deiser.com/page/project_navigator-view
Res Code: 200
Req Method: GET
Auth Header: JWT eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJxc2giOiI3YzM0NWI5MjAyMWU5NGEyMDNkNTc4MzIwZGE5MDI5Yzc2OTY0OWJhNDUzYzMyNWE4N2Q5MjZiZDE4NTdjOWZhIiwiaXNzIjoiY3NydC1mYWtlLXRva2VuLWlnbm9yZSIsImV4cCI6MTYxOTA5ODUxNCwiaWF0IjoxNjE5MDg3NzE0fQ.cDTr3q3330neIu7sUVKEXvsL5FDoJmrMl2wlBEGsfDU

If we use Postman to do that request, we have 401 in the response.
Example 2:

https://pfc-develop.cloud.deiser.com/webhooks/projects/updated/test
Res Code: 200
Req Method: GET
Auth Header: JWT eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJxc2giOiI3YThmNGJjOTZiZDEzNjM5MmRhYjFmNjA1YWZhMDI0ZDgxNzQ1OTUwOWEwNzM3OTY3MjdjNGQ2NWQyYTU4YjQwIiwiaXNzIjoiY3NydC1mYWtlLXRva2VuLWlnbm9yZSIsImV4cCI6MTYxOTA5ODUxNCwiaWF0IjoxNjE5MDg3NzE0fQ.tmw6xQNosMUT47PNqkJcOtRUuihsXP2_OK8l6s0B2Jw

If we use Postman to do that request, we have 405 in the response.
To Reproduce
Run tests to https://profields-cloud.deiser.com/atlassian-connect.json using:
docker run -v $(pwd)/out:/app/out connect-security-req-tester https://profields-cloud.deiser.com/atlassian-connect.json
Expected behavior
Pass Requirement 5 tests.

Hi there,
When reproducing those two examples, I do indeed get a 200 response code. I'm including the cURL commands below to verify.
Example 1
curl https://pfc-develop.cloud.deiser.com/page/project_navigator-view -H 'Authorization: JWT eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJxc2giOiI3YzM0NWI5MjAyMWU5NGEyMDNkNTc4MzIwZGE5MDI5Yzc2OTY0OWJhNDUzYzMyNWE4N2Q5MjZiZDE4NTdjOWZhIiwiaXNzIjoiY3NydC1mYWtlLXRva2VuLWlnbm9yZSIsImV4cCI6MTYxOTA5ODUxNCwiaWF0IjoxNjE5MDg3NzE0fQ.cDTr3q3330neIu7sUVKEXvsL5FDoJmrMl2wlBEGsfDU' -vv
Example 2
curl https://pfc-develop.cloud.deiser.com/webhooks/projects/updated/test -H 'Authorization: JWT eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJxc2giOiI3YThmNGJjOTZiZDEzNjM5MmRhYjFmNjA1YWZhMDI0ZDgxNzQ1OTUwOWEwNzM3OTY3MjdjNGQ2NWQyYTU4YjQwIiwiaXNzIjoiY3NydC1mYWtlLXRva2VuLWlnbm9yZSIsImV4cCI6MTYxOTA5ODUxNCwiaWF0IjoxNjE5MDg3NzE0fQ.tmw6xQNosMUT47PNqkJcOtRUuihsXP2_OK8l6s0B2Jw' -vv
This seems to be working as intended, but let me know if you feel otherwise.

We have fixed the error.
This issue can be closed.
Thank you @seanmarpo!


Strange Warnings (apache-cassandra-clientutil-2.1.3.jar and apache-cassandra-thrift-2.1.3.jar)

I'm working on getting Kundera setup in a Wildfly 8.2 deployment.
I've added the following dependencies in my gradle script:
compile 'com.impetus.kundera.core:kundera-core:2.15.1'
compile 'com.impetus.kundera.client:kundera-cassandra:2.15.1'
compile 'com.impetus.kundera.client:kundera-cassandra-ds-driver:2.15.1'

And when I deploy the EAR, I get the following warnings:
Class Path entry apache-cassandra-clientutil-2.1.3.jar in /content/MyTest.ear/lib/cassandra-all-2.1.3.jar  does not point to a valid jar for a Class-Path reference.
Class Path entry apache-cassandra-thrift-2.1.3.jar in /content/MyTest.ear/lib/cassandra-all-2.1.3.jar  does not point to a valid jar for a Class-Path reference.

How do I resolve these?
Thanks.

@pluttrell
If you check MANIFEST.MF file under META-INF in cassandra-all-2.1.3.jar , you will find :
Class-Path: apache-cassandra-clientutil-2.1.3.jar apache-cassandra-thrift-2.1.3.jar

which means that cassandra-all-2.1.3.jar refers classes in apache-cassandra-clientutil-2.1.3.jar and apache-cassandra-thrift-2.1.3.jar. When cassandra-all-2.1.3.jar loads it will also look for these references in the class-path in local network. These jars are not found in local network that's why these warnings are shown.
Since there is no need of these jars so you may ignore these warnings.
HTH,
Devender

@devender-yadav
Thanks for responding.
Ya, I saw those Class-Path entries, and believe that these are usually automatically resolved by listing the dependences in the .pom file, which when Gradle builds the EAR, will usually include all sub-dependencies, which wasn't happening in this case. I took another look at it and it appears that one, but not both are included as dependencies org.apache.cassandra:cassandra-all:2.1.3, so it's a bit odd that this doesn't appear to be working.
In any event, great to know that these are not actually required/used by Kundera.


Revive freezing

This solves #106 and the regression introduced in #117
@lberki @iirina @cushon
I'd really love your feedback about this change (in the bzl file and in WorkspaceDriver.java).
I'm mainly unclear why these are the flags and their values.
I expected the following to be needed:
        "--host_javabase=@bazel_tools//tools/jdk:absolute_javabase",
        "--javabase=@bazel_tools//tools/jdk:absolute_javabase",
        "--java_toolchain=" + toolchain
        "--host_java_toolchain="  + toolchain,
        "--define=ABSOLUTE_JAVABASE="+ javaHome

But instead the following was sufficient (which I don't understand):
 "--host_javabase=@bazel_tools//tools/jdk:jdk",
 "--java_toolchain=" + javaToolchain

FYI @achew22

@ittaiz can you briefly describe what issue you're trying to solve and why do you expect those flag to be necessary?

Yes, definitely.
What we're trying to do here (as shown in the failure of the first commit) is to run bazel itself inside a sandboxed network-blocked bazel test.
The problem is that bazel 21 and onwards tries to download a remote jdk if you don't explicitly specify it via  --host_javabase flag. This is a problem since I don't want the test to download the JDK.
What I'd like to do is have the starlark rule pass the location of the java runtime and have the test bazel use that for it's compilation.

I mainly don't understand how @bazel_tools//tools/jdk:jdk helps (and will it be supported in the future) and why I need to pass  --java_toolchain but not --host_java_toolchain.
If this is by incident then I'd love to know what are the use-cases that require passing host_java_toolchain so I can write the relevant test for it.

Thanks.
I’m not 100% sure about this and already found a bug case (bazel query
doesn’t accept host_javabase).
I’ll fix that and hopefully get some help from Irina/Liam
On Wed, 3 Apr 2019 at 1:48 Andrew Z Allen<EMAIL_ADDRESS>wrote:

@achew22 approved this pull request.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/bazel-integration-testing/pull/120#pullrequestreview-221938377,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIF-H9h7EcYqV6RYLFmYdfe6j5CQPwks5vc95OgaJpZM4cWnHD
.

If my understanding is correct, you need to use @bazel_tools//tools/jdk:absolute_javabase instead of @bazel_tools//tools/jdk:jdk only if your local JDK is at a location that couldn't be identified by bazel (e.g. by looking at JAVA_HOME) or if you want to use a different JDK if multiple are installed. Are any of these an use case in this test? In your initial comment how was javaHome from "--define=ABSOLUTE_JAVABASE="+ javaHome computed?
Setting --host_javabase is enough for not downloading the remote jdk. I assume you need to set --java_toolchain for being able to use a certain language level (or jvm_opts). I don't see a reason why --host_java_toolchain needs to be set as well. The default host java toolchain uses a javac embeded into bazel [*] which works with the default  jvm_opts.
You may want to use a different --host_java_toolchain if you have host java tools that need to be compiled with a different javac version than the one embeded into bazel. I can't think of another use case.
I hope this clears things a bit.
[*] the javac (modules) may be embeded into bazel or into the remote java tools depending on the bazel version you're using.

Are any of these an use case in this test
I think it can be relevant if someone uses this on a machine that doesn't have JAVA_HOME. I'll reproduce this locally and then add a comment, thanks! (Maybe it should be documented in Bazel also somewhere? Maybe in the build file?)
The javaHome was computed from ctx.attr._java_runtime[java_common.JavaRuntimeInfo].java_home_runfiles_path where _java_runtime is @bazel_tools//tools/jdk:current_java_runtime.
I think I need to pass in specific jvm_opts of 8 because of the change here https://github.com/bazelbuild/bazel/commit/137504e8cacda23efcc4b74f7c1b03e0074da030.
Actually I'll try to see if I can refrain from passing this flag if the jdk is >8.

The following part looks like a cycle to me:
"--javabase=@bazel_tools//tools/jdk:absolute_javabase",
"--define=ABSOLUTE_JAVABASE="+ javaHome",

Because javaHome is looking at @bazel_tools//tools/jdk:current_java_runtime which actually points to the value of --javabase. This is an interesting situation, I have to investigate how we handle this kind of cycle.

In your test setup @bazel_tools//tools/jdk:current_java_runtime is evaluated first to the value of either JAVA_HOME or the --javabase value that was passed to the test.
When actually running bazel inside the test ABSOLUTE_JAVABASE points to the local JDK or the test's --javabase (if it was specified). Are you running the test with a custom javabase? If not, then
"--javabase=@bazel_tools//tools/jdk:absolute_javabase",
"--define=ABSOLUTE_JAVABASE="+ javaHome",

is equivalent to --javabase=@bazel_tools//tools/jdk:jdk, which points to the local JDK, so the default value in bazel.

Thanks for your help!
If my “external” bazel is run with a javabase but the “test” bazel runs
without then I need the absolute approach right?
On Wed, 3 Apr 2019 at 14:38 Irina Iancu<EMAIL_ADDRESS>wrote:

In your test setup @bazel_tools//tools/jdk:current_java_runtime is
evaluated first to the value of either JAVA_HOME or the --javabase value
that was passed to the test.
When actually running bazel inside the test ABSOLUTE_JAVABASE points to
the local JDK or the test's --javabase (if it was specified). Are you
running the test with a custom javabase? If not, then
"--javabase=@bazel_tools//tools/jdk:absolute_javabase",
"--define=ABSOLUTE_JAVABASE="+ javaHome",
is equivalent to --javabase=@bazel_tools//tools/jdk:jdk, which points to
the local JDK, so the default value in bazel.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/bazel-integration-testing/pull/120#issuecomment-479452359,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIF2nIP7yw9KYFj2l-uNbfdEwGseF5ks5vdJKtgaJpZM4cWnHD
.

If my “external” bazel is run with a javabase but the “test” bazel runs without then I need the absolute approach right?

That is correct.

Thanks!
Not sure that would be correct. I agree that’s whAt I’m using it for but
OTOH I’m not using a custom java runtime for the test but rather passing
the real java runtime into the test.
On Wed, 3 Apr 2019 at 16:46 Irina Iancu<EMAIL_ADDRESS>wrote:

@iirina commented on this pull request.
In tools/bazel_java_integration_test.bzl
https://github.com/bazelbuild/bazel-integration-testing/pull/120#discussion_r271750191
:

@@ -36,6 +38,9 @@ bazel_java_integration_test_properties_ = rule(
attrs = {
"bazel_version": attr.string(mandatory = True),
"external_deps": attr.label_list(allow_files = True),

   "_java_runtime": attr.label(

You can rename _java_runtime to _test_java_runtime or something similar
to distinguish between the test external javabase and the internal one...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/bazel-integration-testing/pull/120#pullrequestreview-222219226,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIF42BvTiFhvPPPcRqoQvfWQv7R4jYks5vdLC2gaJpZM4cWnHD
.

@achew22 Major changes since you reviewed:

I'm using absolute javabase + define to support cases where people have multiple JDKs installed.
Supporting opt out of these flags per bazel invocation to support running bazel query.
Removed versions below 0.21 since they don't have absolute javabase

Note: I got this to work on our many internal tests locally but still haven't gotten the new flavor to work with RBE. Might mean more changes to this PR though we can also merge this and iterate further in a new PR.
Would appreciate your review

Update: I got our tests to pass also on RBE :)
This is good to go

@achew22 since this is very close to the original PR I'll merge it on Monday morning Israel time. Would still appreciate your second look if you get the chance before then.

I'm starting to become even more convinced that we should move as much logic into .bzl as possible. Are there any major pitfalls you can see coming from that?
How do you see this working out? What parts?
The main two reasons I'm reluctant to make this kind of change are because there is a lot of value in declaratively controlling the bazel invocation from the test case (it's part of the test story) and because I think that setting up the different permutations outside will be noisy and unclear (for example a test which needs to run query and build and then needs different sets of flags)


protobuf not seem to be recognized but it is installed

(base) D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection>python legacy/train.py --logtostderr --train_dir=training/ --pipeline_config_path=training/ssd_mobilenet_v2_coco.config
Traceback (most recent call last):
File "legacy/train.py", line 49, in
from object_detection.builders import dataset_builder
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\builders\dataset_builder.py", line 27, in
from object_detection.data_decoders import tf_example_decoder
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\data_decoders\tf_example_decoder.py", line 24, in
from object_detection.protos import input_reader_pb2
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\protos\input_reader_pb2.py", line 11, in
from google.protobuf import descriptor_pb2
File "C:\ProgramData\Anaconda3\lib\site-packages\google\protobuf\descriptor_pb2.py", line 1114, in
serialized_options=None, file=DESCRIPTOR),
File "C:\ProgramData\Anaconda3\lib\site-packages\google\protobuf\descriptor.py", line 534, in new
return _message.default_pool.FindFieldByName(full_name)
KeyError: "Couldn't find field google.protobuf.FileOptions.php_metadata_namespace"
(base) D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection>python legacy/train.py --logtostderr --train_dir=training/ --pipeline_config_path=training/ssd_mobilenet_v2_coco.config
Traceback (most recent call last):
File "legacy/train.py", line 49, in
from object_detection.builders import dataset_builder
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\builders\dataset_builder.py", line 27, in
from object_detection.data_decoders import tf_example_decoder
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\data_decoders\tf_example_decoder.py", line 24, in
from object_detection.protos import input_reader_pb2
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\protos\input_reader_pb2.py", line 11, in
from google.protobuf import descriptor_pb2
File "C:\ProgramData\Anaconda3\lib\site-packages\google\protobuf\descriptor_pb2.py", line 1114, in
serialized_options=None, file=DESCRIPTOR),
File "C:\ProgramData\Anaconda3\lib\site-packages\google\protobuf\descriptor.py", line 534, in new
return _message.default_pool.FindFieldByName(full_name)
KeyError: "Couldn't find field google.protobuf.FileOptions.php_metadata_namespace"
Please F1 aka Help

(base) D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection>python legacy/train.py --logtostderr --train_dir=training/ --pipeline_config_path=training/ssd_mobilenet_v2_coco.config
Traceback (most recent call last):
File "legacy/train.py", line 49, in
from object_detection.builders import dataset_builder
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\builders\dataset_builder.py", line 27, in
from object_detection.data_decoders import tf_example_decoder
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\data_decoders\tf_example_decoder.py", line 24, in
from object_detection.protos import input_reader_pb2
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\protos\input_reader_pb2.py", line 11, in
from google.protobuf import descriptor_pb2
File "C:\ProgramData\Anaconda3\lib\site-packages\google\protobuf\descriptor_pb2.py", line 1114, in
serialized_options=None, file=DESCRIPTOR),
File "C:\ProgramData\Anaconda3\lib\site-packages\google\protobuf\descriptor.py", line 534, in new
return _message.default_pool.FindFieldByName(full_name)
KeyError: "Couldn't find field google.protobuf.FileOptions.php_metadata_namespace"
(base) D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection>python legacy/train.py --logtostderr --train_dir=training/ --pipeline_config_path=training/ssd_mobilenet_v2_coco.config
Traceback (most recent call last):
File "legacy/train.py", line 49, in
from object_detection.builders import dataset_builder
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\builders\dataset_builder.py", line 27, in
from object_detection.data_decoders import tf_example_decoder
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\data_decoders\tf_example_decoder.py", line 24, in
from object_detection.protos import input_reader_pb2
File "D:\FINAL YEAR\Final Project Fanally\Project\Workshop\TensorFlow\models\object_detection\protos\input_reader_pb2.py", line 11, in
from google.protobuf import descriptor_pb2
File "C:\ProgramData\Anaconda3\lib\site-packages\google\protobuf\descriptor_pb2.py", line 1114, in
serialized_options=None, file=DESCRIPTOR),
File "C:\ProgramData\Anaconda3\lib\site-packages\google\protobuf\descriptor.py", line 534, in new
return _message.default_pool.FindFieldByName(full_name)
KeyError: "Couldn't find field google.protobuf.FileOptions.php_metadata_namespace"
Please F1 aka Help

found a solution working:
Uninstalled protobuf           => pip3 uninstall protobuf
and Installed protobuf        => pip3 install protobuf

@phishekwa Glad to know that you are able to use it after reinstalling protobuf. Feel free to close this issue :)


Unable to start debbuging scaffolded project.

VSCode Version: 1.28.0-insider
OS Version: Windows 7 SP1
Dotnet Core Version:  2.1.408
C# Extension Version: 8,272,977

Steps to Reproduce:

Generate new .NET Core project with dotnet new console
Open project with VS Code
Generate optional assets files (VS Code asks if I'd like to add them: launch.json and tasks.json).
Try to start debugging

The problem is that Debug Adapter is null and cannot set onError property:

It seems that there is some problem with adapter type because the type is null.:

My launch.json (that what  was generated, removed comments only)
{
   "version": "0.2.0",
   "configurations": [

        {
            "name": ".NET Core Launch (console)",
            "type": "coreclr",
            "request": "launch",
            "preLaunchTask": "build",
            "program": "${workspaceFolder}/bin/Debug/netcoreapp2.1/Tools.FtpServer.dll",
            "args": [],
            "cwd": "${workspaceFolder}",
            "console": "internalConsole",
            "stopAtEntry": false,
            "internalConsoleOptions": "openOnSessionStart"
        },
        {
            "name": ".NET Core Attach",
            "type": "coreclr",
            "request": "attach",
            "processId": "${command:pickProcess}"
        }
    ,]
}

No, 64 bit. Update VSCode to newest version (last was from a week ago) and works. Closing.


Exposed Credentials

Could you hide your credentials?
Sever
https://github.com/pmbanugo/Pusher-Auth0-ChatApp/blob/master/src/server.js#L10-Lundefined
Client
https://github.com/pmbanugo/Pusher-Auth0-ChatApp/blob/master/src/public/index.js#L4-L5

Thanks


HTTPS

I generated certificates through LetsEncrypt, they sit in etc/letsencrypt on the server. Can we deploy these in the application?

The main use case for this is third party web applications wanting to interact with the MapRoulette API, crossdomain restrictions usually don't allow mixed HTTP / HTTPS content any more, and more and more web sites are hosted on HTTPS only.

This is becoming more urgent now that OpenStreetMap.org is moving away from http support.

https://github.com/openstreetmap/operations/issues/117
https://doesmysiteneedhttps.com/
https://scotthelme.co.uk/still-think-you-dont-need-https/
You need HTTPS for geolocation in modern browsers.

This is now completed. MapRoulette is now HTTPS only.


Add authors' url

Features

[x] Add authors' URL
[x] Add placeholder for Day 29

@huytd I think you went to sleep already so I merge this PR myself. lol. Let me know if you want to change anything. Thanks.


There isn't on-change method in Svelte and $: don't need a function

This is almost perfect, you only need to remove the unnecessary import and it's done.
<script>
 // You only need to remove this line
  import onChange from "on-change";

  // To create a prop you need to export de variable
  export let showInput;

  let name = "Steve";
</script>

In Computed
<script>
// You only need to remove this line
  import onChange from "on-change";
  let showInput;

 // Don't need a function
  $: lowerCaseName = () => {
    return name.toLowerCase();
  };

// Use this way
$: lowerCaseName = name.toLowerCase();

  let name = "Steve";
</script>

oops, thanks for catching the mistaken on-change import, just pushed the fix for that
the second piece around the computed value is a bit trickier to solve, so will leave this ticket open until we find a good solution there


Automation of criteria evaluation

Suggestion is to leverage the CSpec and VP to enable automation of criteria evaluation.
Suggested criteria:

BA1 (possibly all the population criteria)
PVS1

SP ticket: https://broadinstitute.atlassian.net/browse/CSP-40

Replaces https://github.com/ClinGen/clincoded/issues/2020 and https://github.com/ClinGen/clincoded/issues/1999

Closing this, this concept will become a v4 feature.


Some Examples in read me would be good

As with every gitthub people love examples show usage and setup and a simple implementation that would be used instead of having to go into source to see if its fit for purpose

Hi @davidbuckleyni,
You might find the article Introducing the EfCore.SoftDeleteServices library to automate soft deletes, which is listed in the About section of this repo.
Also, the EfCore.SoftDeleteServices Wiki has more information (link to this can be found in the readme file.
I hope that helps.


use osmosis-labs/cosmos-sdk instead of cosmos/cosmos-sdk

This PR updates the repo to use the osmosis-labs fork of the cosmos-sdk rather than the cosmos one.
It also uses https for submodules instead of ssh.

LGTM!


'pokemon_data' referenced before assignment in incubate_eggs_worker

Expected Behavior
pokemon_data should be assigned
Actual Behavior
File "/home/john/PokemonGo-Bot-dev/pokemongo_bot/cell_workers/incubate_eggs_worker.py", line 29, in work
self._hatch_eggs()
File "/home/john/PokemonGo-Bot-dev/pokemongo_bot/cell_workers/incubate_eggs_worker.py", line 134, in _hatch_eggs
for pokemon in pokemon_data:
UnboundLocalError: local variable 'pokemon_data' referenced before assignment
Code
        try:
            pokemon_data = self._check_inventory(pokemon_ids)
        except:
            pass  # just proceed with what we have
        for pokemon in pokemon_data:

Maybe pokemon_data wasn't assigned before except
Steps to Reproduce
run with
"hatch_eggs": true,
"longer_eggs_first": true,
for some time
Other Information
OS: Linux 4.4 manjaro kde
Git Commit: sorry downloaded zip, but probably 79152bc65f8194ee9eb591fa657b99423771409e on dev

Happens on Ubuntu 14.04.4 LTS as well. Same configuration
"hatch_eggs": true,
"longer_eggs_first": true,
Traceback
File "pokecli.py", line 464, in
main()
File "pokecli.py", line 68, in main
bot.tick()
File "/root/pokemon/bot01/pokemongo_bot/init.py", line 84, in tick
if worker.work() == WorkerResult.RUNNING:
File "/root/pokemon/bot01/pokemongo_bot/cell_workers/incubate_eggs_worker.py", line 29, in work
self._hatch_eggs()
File "/root/pokemon/bot01/pokemongo_bot/cell_workers/incubate_eggs_worker.py", line 134, in _hatch_eggs
for pokemon in pokemon_data:
UnboundLocalError: local variable 'pokemon_data' referenced before assignment

I also have this issue, trying to run without hatch_eggs at the moment to see if its 100% that

Try replacing the word pass with return, see if that works.


Dispose of locally created stream in WaveFileReader when there is an exception thrown in the constructor

When creating a WaveFileReader the ChunkedReader can throw exceptions.  This is OK when a stream is passed in because the WaveFileReader does not own the stream.  But when a file path is passed in the WaveFileReader does own the stream, and does not properly dispose of it (dispose is not called on objects that failed to construct) leaving it locked by the process running NAudio.  This fixes that behavior.

great, looks good


'Uncaught object' error when previously clicked element is removed from the DOM and it is an  or  element

What is your Scenario?
TestCafe (proxy mode) fails with 'Uncaught object' error on leaveElement when the prevElement is removed from the DOM, and it (or its parent, which is also removed) is an  or <area> element. The issue happens because subsequent getParent hammerhead's code returns "host", which is a string representing the host in case of  and <area> elements, not an element. This causes a failure when attempting to get its parent again using nativeMethods.nodeParentNodeGetter.call(el).
From the user's perspective, it appears that TestCafe attempts to move the mouse very slowly to the next element and then fails with an Uncaught object error.
Issue is not reproducible in native automation mode.
What is the Current behavior?
Test execution fails with error:
  1) Uncaught object

   "{"callsite":{"filename":"C:\\work\\git\\testcafe-callsite-issue\\test.js","lineNum":6,"callsiteFrameIdx":5,"stackFrames":[{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{}],"isV8Frames":true}}"
      was thrown. Throw Error instead.

What is the Expected behavior?
Testcafe should execute test commands without error
What is the public URL of the test page? (attach your complete example)
Test page to reproduce the issue
<!DOCTYPE html>

    <meta charset="UTF-8">
    <title>Link Removal Example</title>

    Click me to remove
    <button id="btn">Button</button>

What is your TestCafe test code?
fixture('TestCafe Callsite Issue')
    .page('./page.html');

test('Test link removal', async t => {
    await t
        .click('#link')
        .click('#btn');
});

Your complete configuration file
No response
Your complete test report
No response
Screenshots
No response
Steps to Reproduce

Clone/download the repository: https://github.com/egorshantur/testcafe-callsite-issue
Run the following command:
npm install
Run the test:
npm run test

TestCafe version
^3.4.0
Node.js version
No response
Command-line arguments
testcafe chrome test.js --disable-native-automation
Browser name(s) and version(s)
No response
Platform(s) and version(s)
No response
Other
Tests running in proxy mode began failing after upgrading to version 3.4.0 or higher. The last version without this issue was 3.3.0.
I found that the issue is reproducible after the following change: DevExpress/testcafe#7995
If add removed code back, then test does not fail
        const prevElementInDocument = prevElement && domUtils.isElementInDocument(prevElement);

        const prevElementInRemovedIframe = prevElement && domUtils.isElementInIframe(prevElement) &&
                                         !domUtils.getIframeByElement(prevElement);

        if (!prevElementInDocument || prevElementInRemovedIframe)
            prevElement = null;

Based on the description "removed IE leftovers from src\client\automation," it should not affect my tests as I run them in Chrome. However, it did affect them, so I reviewed the code that was originally executed for all browsers but was removed in that commit, and identified the issue.

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

Seems like I have the same issue
"{"callsite":{"filename":"../commonScripts.js","lineNum":373,"callsiteFrameIdx":5,"stackFrames":[{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{}],"isV8Frames":true}}"


added support for Spansion S25FL1-K 64Mb

We do not have a S25FL1-K 64Mb available, but I found a 16Mb version to perform some tests.

Unfortunately, it is a SL25FL016K, which does not seem compliant with S25FL1-K. Spansion SPI data flash series are notoriously ugly as they may identify with same IDs but expose incompatible instruction set, not mentioning their non-linear erase page size...

Beware that the proposed patch is invalid: 0x40<<20 means 64MB.
64Mb = 8MB, 128Mb = 16MB; I'm not sure what you meant here.

You are right, it should be 0x40<<17 - for 64Mb (8MB)


#show regex not handling all cases

Description
This code
#show regex("\\(.*\\)"): it => text(red)[#it]

(123)

(...)

(a b c)

(. x .)

(x)

( x)

(x )

( x )

(.)

( .)

(. )

( . )

(qwe)

is being rendered like this:

All the lines should be in red.
Reproduction URL
No response
Operating system
macOS
Typst version

[X] I am using the latest version of Typst

Even without regex, this doesn't work. In the following example, nothing is shown in red.
#show "(...)": it => text(red)[#it]
#show "( . . . )": it => text(red)[#it]
#show "(. . .)": it => text(red)[#it]

(...) ( . . . ) (. . .)

This is a known issue. The exaples that don't work consist of multiple text nodes and those can't be mached. Duplicate of Duplicate of #86.