4141 - Add Hill cipher encryption

I would like to work on this. Can you please assign me to this?

closed - please use Issue-Template!


[BugFix] Fix GSON Util concurrent bug

Fixes SR-17990
Use Object lock to protect its properties from changed by other serialize thread. But this will only take effect when all threads uses GSONUtils to serialize object, because other methods of changing properties do not necessarily require the acquisition of the object lock.
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.0
[x] 2.5
[x] 2.4
[x] 2.3

[FE PR Coverage Check]
:heart_eyes: pass : 5 / 5 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/persist/gson/GsonUtils.java
5
5
100.00%
[]

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.3


[sft_llama2] Add check of arguments

Addressed https://github.com/huggingface/trl/issues/629#issuecomment-1683538661 and fixes #629
cc @lvwerra

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Proposed 1.9.0

Hi @manojsdoshi
Fully approved amendment should go in 190?
https://github.com/ripple/rippled/pull/4097

@RichardAH will add that in the next beta


Update urls.txt

added fix for tpb new URL.

Merged. Thank you!

Glad I could be of help
On 24 May 2016 4:07 a.m., "Someguy123"<EMAIL_ADDRESS>wrote:

Merged. Thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/Someguy123/HiddenEverywhere/pull/9#issuecomment-221019200


Where is the firestore exported data located by default on windows?

I installed the package globally on Windows 11
npm install -g firestore-backfire @google-cloud/firestore
And I didn't notice that I have to use the full path, so I used a relative path instead
backfire export . -p database -k credentials.json
and
backfire export data -p database -k credentials.json
Where are the data located by default in order to delete it?

Hey @elieobeid7, I don't personally use Windows so I can't say I know for sure. But you can try adding the logLevel flag when you run backfire which will output some useful information that might help you track down where your data is.
For example:
backfire export . -p database -k credentials.json --logLevel debug

Hey @elieobeid7, haven't heard back from you, so I'm going to close this issue for now. Feel free to re-open if you have any further questions.


Permit serialization of responses

It makes it easier to store.
Here is a code example with diesel:
use chrono::NaiveDateTime;
use diesel_as_jsonb::AsJsonb;
use geocoding::opencage::Results as OpenCageResult;
use geocoding::openstreetmap::OpenstreetmapResult as OSMResult;
use uuid::Uuid;

use crate::schema::locations;

#[derive(Serialize, Deserialize, Debug, AsJsonb)]
pub enum LocationData {
    OSM(OSMResult<f64>),
    OpenCage(OpenCageResult<f64>),
}

#[derive(Queryable)]
pub struct Location {
    pub id: Uuid,
    pub provider: String,
    pub data: LocationData,
    pub label: String,
    pub lat: f64,
    pub lon: f64,
    pub created_at: NaiveDateTime,
    pub updated_at: NaiveDateTime,
}

#[derive(Insertable)]
#[table_name = "locations"]
pub struct NewLocation {
    pub provider: String,
    pub data: LocationData,
    pub label: String,
    pub lat: f64,
    pub lon: f64,
}

This looks useful and uncontroversial to me. Could you also update CHANGES before we merge?

Sure, should I add an entry 0.1.1 to the file ?

Thanks!

bors r+

bors retry

This ended up as part of 0.2.0 btw – now on crates.io.

Nice, thank you
Le ven. 10 avr. 2020 à 17:57, Stephan Hügel<EMAIL_ADDRESS>a
écrit :

This ended up as part of 0.2.0 btw – now on crates.io.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/georust/geocoding/pull/31#issuecomment-612094028, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOZ5J4HQFNN32G3INLZXDDRL46WRANCNFSM4MBC4AUQ
.


Film viewers are hard to watch

Scene 382005
Pulling down on the vertical sliders too far will trigger the return of the next scene

Closing as this issue has been opened at https://github.com/soap-bubble


critical security flaw

Hi, I'm using the bridge to bridge Telegram <-> IRC, and I discovered that the bridge sends the Telegram API bot token when it receives media, this is a huge security flaw, because with the token I can do (almost) everything I want with the bot...
For example, when I send a photo in a group the bot is in, it bridges it to IRC, it sends a message like <Eli>  https://api.telegram.org/file/bot<token>/photos/file_1.jpg
it just sent the token.
Suggested fix:
Download the file and put it inside the directory that an (internal) http server serves.
,
Eli

This is happening without specifying UseInsecureURL ?
What version of matterbridge ?
https://github.com/42wim/matterbridge/blob/master/matterbridge.toml.sample#L694-L698

Hi, thanks for replying
(btw, I don't manage the bot)
I assume UseInsecureURL was turned on, because it does relay photos and if it's turned off it will not (and now it does), so yeah, I was all hyped up for nothing.
It might be a very good idea to implement my solution (or upload all photos to imgur)

This also already exists, see https://github.com/42wim/matterbridge/wiki/Mediaserver-setup-[advanced]
If you want imgur support please open a feature requests issue

I see, thank you very much, I guess we can close this issue.
(also sorry for me not doing any research lol, I was in a rush)


Image Picker

Can we please have the ability to set the useRootNavigator for Image Picker to false so that it does not cause an issue with other routes? I have an issue using this with the latest version of Auto Router and nested routes.

@rlee1990

I have an issue using this with the latest version of Auto Router and nested routes.

Can you elaborate by providing a minimal code sample which shows current issue and expected behavior you are looking for using the plugin ?

@darshankawar
So I have an app that uses Auto Router with nested routes. One set of nested routes is home and a subroute of Profile and Profile Edit.
While in profile edit I use the Image Picker package to select an image or use the camera. When doing this it creates an issue with the router and removes the stack. I had the same issue with any dialog prompt. I fixed the dialog prompt issue by setting useRootNavigator to false. I suspect that Image Picker uses the root Navigator.

@rlee1990 Would be great if you could show us a minimal code sample. That way, it'll be helpful to make this issue actionable.


Add documentation programtically to WQ-Review GUI using tool tips

This will be a big pain, and not hte best use of resources. Closing


WSL breaks the plugin on Windows

WSL on Windows creates a vEthernet Network Interface which the plugin selects by default as the internal address.
The below image shows the output from  RGetIp.internalIP, RGetIp.networkType, and NetworkInterface.list from dart:io
Sorry if I could'nt explain in detail, I'm still fairly new to Flutter.

Same issue


Add basic functions for membership syncing?

Something where the bridge can dump a list of remote user objects into a function and have the SDK automatically figure out which ones need to get configured in the room.

This is far too specialized for the SDK to do. Though some common cases show up, there's some things like user decorations, mappings, etc that have more varied answers. The SDK should already have enough functions to support someone writing their own membership sync which is going to be more reasonable for their use case.


Hooks cause error

When calling a hook in the demo project, the following error message appears in the console (see screenshot)

Steps to reproduce:

clone the repo and navigate to the demo section
change the directory of the react-pixi-tilemap path to fit the package or install it via npm/yarn
yarn install
insert const mapHook = useMap() just before the return statement and add it to imports
yarn start

Here the modified code from App.js, just for redundancy:
import { Stage, Sprite } from '@inlet/react-pixi';
import { Texture } from '@pixi/core';
import { Tilemap, useTilemapLoader, useMap } from 'react-pixi-tilemap'

const tilemape = process.env.PUBLIC_URL + '/stages/map.tmx'

const App = () => {
    const map = useTilemapLoader(tilemape)
    const mapData = useMap()
    // console.log(mapData)

    return (
        <Stage width={window.innerWidth} height={window.innerHeight} options={{ resizeTo: window }}>
            <Tilemap map={map} scale={0.75}>
                {/* These sprites show off the layering order */}
                <Sprite texture={Texture.WHITE} x={120} y={380} width={64} height={64} />
                <Sprite texture={Texture.WHITE} x={700} y={420} width={64} height={64} />
                <Sprite texture={Texture.WHITE} x={500} y={700} width={64} height={64} />
            </Tilemap>
        </Stage>
    )
}

export default App;

I am using this package for a university project btw.
Thanks in advance for providing such a nice package :)

Aight, I found it out myself:
The Provider of Tilemap is not a parent of this specifi component.
Only children of <Tilemap/>   can get access to the context provider. that is needed in order for the hooks to work. Therefore, any child of tilemap can use the hooks.


Module: Perception how to split Pointpillars model split several separate libtorch models

本地通过pytorch训练出pointpillars的pkl模型文件，怎么通过libtorch将其分割成几个网络呢pts_backbone.zip pts_bbox_head.zip pts_middle_encoder.zip pts_neck.zip pts_voxel_encoder.zip，感谢！

Confused about the same problem, anyone can help? Thanks!!!

same confusion, any explanations in detail? Thanks!


truncate table was not clearing memory from UMM in a PR region. The c…

Changes proposed in this pull request
Truncate table was not clearing memory from UMM in a PR region. The call was only while region being destroyed.
Patch testing
pre-checkin running
ReleaseNotes changes
NA
Other PRs
https://github.com/SnappyDataInc/snappydata/pull/570

Looks good -- see couple of questions.


🐛 Profile picture - Indicates recording on iPad Mini

Cc: @AntPolkanov
Hi
Describe the Bug
When I tested the functionality to take a photo, the green iOS "recording" indication does not switch off after the photo is taken and uploaded.
To Reproduce
Steps to reproduce the behavior:

Tap your profile icon in the top left corner of the app
Tap your profile icon again to open the "Edit Profile Picture" dialog
Tap "Take a new photo"
Take a photo
Tap "Use photo"
Tap "Use this picture" on the "Edit Profile Picture" dialog

After the dialog is dismissed, the "recording" indication remains switched on - see screenshot below.
Expected Behavior
The "recording" indication should switch off after the photo is taken and "Use photo" is tapped.
Tasks

[x] Investigate
[x] Fix

Acceptance Criteria

[x] The "recording" indication is switched off after the photo is taken
[x] Meets Definition of Done - https://github.com/SSWConsulting/SSW.Rewards.Mobile/blob/main/_docs/Definition-of-Done.md

Environment

Device: iPad Mini (5th Generation)
OS: iPadOS 17.4.1

Screenshots

Thanks!

Done - this has been fixed and navigating away on iOS should stop the indicator from showing.

Done - this has been fixed.
We've moved to a new scanning library which doesn't have this issue.


Allow Cardboard UI to be disabled

It'd be nice if there was a pref on WebVRConfig to not create the Cardboard UI (the back arrow, centerline, gear icon for settings).

We're going to mirror what the Chrome UA does. I believe the UA will have both a divider and a back arrow.


Adds the Scale Generator exercise.

Work towards integrating: https://github.com/exercism/php/pull/297

@neenjaw Ready for review.
By the way, I see a tests.toml file in come of the other folders. Is that something I need to manually generate?

At this time, we aren't using the problem-specs repo (other than for inspiration), so for now, no you don't need the tests.toml for this.

Hey is this still ready to be merged @neenjaw ?

Exercise looks good to go, I need to merge #392 first, but this can be merged after that. :)

Yep!

Thanks for catching this @petemcfarlane, hadn't had a chance yet to catch up with the exercism backlog yet.


User profiles get mixed up after login-with-facebook

After I try to login with my facebook user (Steffen Frank), I am logged in as Nadine Gartner
How to reproduce:

start a freshly installed browser with no caches, history, persistent memory... (Firefox in my case). I have never visited facebook.com in this browser before
go to facebook.com and verify that no Facebook user account information is stored in the browser (Facebook asks for credentials)
log in to Facebook with my user Steffen Frank and verify I am actually logged in as Steffen Frank
go to any website that allows a facebook login (e.g. twoo.com) and verify that my own user account will be used for login
now go to teamradio.herokuapp.com and login with facebook

Expected result:

I am logged in as Steffen Frank

Actual result:

I am logged in as Nadine Gartner

We will have a look, at the moment we still can't find any other similar case. If anyone else meets this problem, please let us know. Thank you.

i got the same issue ;D

I tried to test it in the staging environment, but I can't login with facebook

Verified


Broken on Windows because vim.fs.normalize converts all path separators to forward slash (\ -> /)

Recent versions are using vim.fs.normalize to normalize paths but this replaces all "windows-style" backward slashes to forward ones, which is inconsistent with path-sep, causing make-path and probably more functions to break on Windows.
I set path-sep to / and everything seems to be working correctly now.
For a proper fix I suggest that you should ensure that all paths are normalized, then use / as the path separator for all platforms.

Bummer, sorry about the sharp edges. Thanks for looking into it. Sounds like I can dump all the path-sep code for fs.normalize which is great.

Are you able to test the api-make-remake branch?

Should be fixed in 0.9.0

I think it works for me. Thank you!


Showing 340+ warnings in Swift 4 version

Thanks for the great library. But, it is showing 340+ warnings in my xcode for Swift 4 version.
Can you make compatible if possible to Swift 4?

@AnilkumarRabhasa can you please post a screenshot or log?

PFA.

@AnilkumarRabhasa that's lib geos and not GEOSwift. https://github.com/OSGeo/geos.git


Fix fuse metadata cache outdated issue after chmod/chown

What changes are proposed in this pull request?
Fix fuse metadata cache outdated issue after chmod/chown
Why are the changes needed?
Fix #18523
Does this PR introduce any user facing changes?
No

I think there's a situation here that needs consideration: if the directory is large, caching it in the metadata cache would take a long time. Is it reasonable to completely invalidate the cache of the parent directory when modifications occur in the child nodes, or should we simply replace the corresponding child nodes in the list cache?

@jiacheliu3 can you take a look?

I think there's a situation here that needs consideration: if the directory is large, caching it in the metadata cache would take a long time. Is it reasonable to completely invalidate the cache of the parent directory when modifications occur in the child nodes, or should we simply replace the corresponding child nodes in the list cache?

@humengyu2012 thanks for your reply. Currently MetadataCachingFileSystem only syncs metadata from the cluster but not replace existing ones. So I just keep setAcl/setAttribute consistent with createFile/createDirectory/delete/rename here.


What about misandry identification?

Can't you have a generic model for both misogyny and misandry? Why be sexist?

Hi,
As the AMI task was for misogyny identification, we are not able to look into Misandry. We acknowledge that `Misandry Classification' is an important task, but as we do not have the data for the task, we were not able to build models for the same. As suggested by you, a generic model that could identify both misogyny and misandry will be an invaluable contribution to the research.


Update the README for Linux

Coverage remained the same at 28.151% when pulling e7f93b03665a74db501f6925eb774bc90db04cc7 on Newlifer:master into cd88b01b74f06d12dcf79c901d5294d84589f0e6 on akrennmair:master.

I think the changes proposed in #381 make more sense, so I'll go with that. Thanks for caring, though! :heart:


[Coverity CID :203482]Memory - corruptions in /subsys/net/lib/lwm2m/ipso_push_button.c

Static code scan issues seen in File: /subsys/net/lib/lwm2m/ipso_push_button.c
Category: Memory - corruptions
Function: button_create
Component: Networking
CID: 203482
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

false positive


Fixed about dialog position on first view

Description
When viewing "About JMeter" dialog it was positioned with the top corner in the middle rather than in the centre of the screen because the first time the dialog wasn't pack()'ed so didn't have any size.
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.

Codecov Report

Merging #351 into trunk will increase coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##              trunk     #351      +/-   ##
============================================
+ Coverage     58.28%   58.29%   +<.01%
- Complexity    10217    10219       +2
============================================
  Files          1160     1160
  Lines         74426    74427       +1
  Branches       7343     7343
============================================
+ Hits          43378    43384       +6
+ Misses        28552    28549       -3
+ Partials       2496     2494       -2

Impacted Files
Coverage Δ
Complexity Δ

...ore/org/apache/jmeter/gui/action/AboutCommand.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...apache/jmeter/extractor/TestBoundaryExtractor.java
97.88% <0%> (ø)
17% <0%> (ø)
:arrow_down:

...mpler/hc/JMeterPoolingClientConnectionManager.java
30.95% <0%> (+3.17%)
11% <0%> (+1%)
:arrow_up:

...s/org/apache/jmeter/timers/PoissonRandomTimer.java
78.37% <0%> (+5.4%)
10% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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

Can't see how this changed caused a test failure...


Support Get.isCheckExistenceOnly().

HBase 1.0 had a setting for only checking existence, which is intended to have the same functionality as Table.exists(Get).  GetAdapter should convert Gets with that flag to appropriate Bigtable Filter objects.  In addition, HBase 1 and 2 exist checks should just create new Get objects with setCheckExistenceOnly(true).
This should fix issue #1707.

LGTM


Requests from  dictionary/lookup and dictionary/examples going to the dictionary/lookup link

In the page https://docs.microsoft.com/en-us/azure/cognitive-services/translator/reference/rest-api-guide, while clicking on dictionary/lookup and dictionary/examples, it takes us to https://docs.microsoft.com/en-us/azure/cognitive-services/translator/reference/v3-0-dictionary-lookup, while there is another explicit page for dictionary/examples.

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

ID: c81b4a4f-656e-da26-c90b-c93f35cd56bc
Version Independent ID: df78c6fa-25e4-5600-4b08-0b6760a8056a
Content: Text Translation REST API reference guide - Azure Cognitive Services
Content Source: articles/cognitive-services/Translator/reference/rest-api-guide.md
Service: cognitive-services
Sub-service: translator-text
GitHub Login: @laujan
Microsoft Alias: lajanuar

@jyravi Thanks for spotting this. We are investigating the issue and will update you shortly.

@jyravi The document is now updated. We will go ahead and close the issue. Thanks for spotting this.

@RohitMungi-MSFT Can you please refer the PR link which you have raised for the particular correction?

@jyravi The PRs are automatically referenced on this thread. In this case it is 187142

Thank you very much. @RohitMungi-MSFT


Strange output from DataFrame.apply when applied func creates a dict

Just had something odd come up while trying to come up with something for this SO question.
If we use DataFrame.apply() to try and create dictionaries from the rows of a dataframe, it seems to return the dict.values() method rather than returning the dict itself.
df = pd.DataFrame({'k': ['a', 'b', 'c'], 'v': [1, 2, 3]})
df.apply(lambda row: {row['k']: row['v']}, axis=1)
Out[52]:
0    <built-in method values of dict object at 0x07...
1    <built-in method values of dict object at 0x03...
2    <built-in method values of dict object at 0x07...
dtype: object

Looks like it's probably something to do with trying to grab the values attribute when the output of the applied function is a Series or something similar.
Library versions:
pd.show_versions()
INSTALLED VERSIONS
------------------
commit: None
python: 2.7.6.final.0
python-bits: 32
OS: Windows
OS-release: 7
machine: AMD64
processor: Intel64 Family 6 Model 30 Stepping 5, GenuineIntel
byteorder: little
LC_ALL: None
LANG: None

pandas: 0.15.0
nose: 1.3.4
Cython: 0.21
numpy: 1.9.0
scipy: 0.14.0
statsmodels: 0.5.0
IPython: 2.3.0
sphinx: 1.2.3
patsy: 0.3.0
dateutil: 1.5
pytz: 2014.7
bottleneck: None
tables: 3.1.1
numexpr: 2.3.1
matplotlib: 1.4.2
openpyxl: 1.8.5
xlrd: 0.9.3
xlwt: 0.7.5
xlsxwriter: 0.5.7
lxml: 3.4.0
bs4: 4.3.2
html5lib: 0.999
httplib2: None
apiclient: None
rpy2: None
sqlalchemy: 0.9.7
pymysql: None
psycopg2: None

Hi all, I've run into the same issue myself. I actually need to pass a list of dicts for each row as my output, and it would be nice to be able to do list(df.apply(lambda x: x.to_dict(), 0)). This would require df.apply to return a Series of dtype object, where the elements are dicts.
apply returning a Series of dicts would be consistent with the behavior when the passed function returns an object Pandas doesn't understand (apparently, any non-numeric without a values attribute). I guess I can add a check to _values_from_object to make sure the input is, say, a subclass of PandasObject.

Hey, I still have this issue with my function returing a dict as part of aggregation

well this was fixed in 0.17.0


Fix a bug where protocol violation errors trigger duplicate headers writes

Motivation:
If a protocol violation is triggered after initialization of a request
context:

The associated request is also closed with the violation cause.
https://github.com/line/armeria/blob/117a21e17ec9e30b0c3c2d74d16fdde3cab62434/core/src/main/java/com/linecorp/armeria/server/Http1RequestDecoder.java#L239-L240
The cause closing the request will be converted into a HttpResponse by
the default ServerErrorHandler.
https://github.com/line/armeria/blob/117a21e17ec9e30b0c3c2d74d16fdde3cab62434/core/src/main/java/com/linecorp/armeria/server/DefaultServerErrorHandler.java#L95-L95
An HttpResponseSubscriber subscribes to HttpResponse and tries to
write on a wire even though the response headers for protocol violation
was written already.

In addition to the duplicate headers writes, there are mismatches between
the actually returned headers and the headers logged by LoggingService.
The written status information is lost during handling the exception.
Modifications:

Wrap the cause of protocol violations with HttpStatusException when
closing a request so that LoggingService correctly logs the headers.
Update a closed ID when an error response is written for an HTTP/1 request.
Build response headers from the status of HttpStatusException if no
response headers is specified before when completing a RequestLog

Result:

The status of a protocol violation is now correctly logged by
LoggingService
A response headers is exactly written once for a protocol violation error.

Thanks, @ikhoon ❤


df resetting after adding a row

Good day everyone,
I'm trying to create a new df using AgGrid but every time I add a new row, the df resets without saving the inputs previously typed.
Do you know how to avoid this issue?

It seems more an issue of state management. I think you'll need to hold you dataframe in st.session_state. Could you please share some code?

I had a similar problem, and it is not a problem of session state management as far as I could tell.
I believe it's a potential bug in the behavior depending on whether reload_data is enabled/disabled.
To solve it (or maybe it could be considered a workaround, if the behavior is not as intended), I now conditionally set reload_data.
Basically, I hold the dataframe in a cache (this could be in the streamlit session state) as well as a boolean flag for whether the dataframe has changed outside of editing cells in AgGrid. This flag I pass in as reload_data.
If I don't do that, I am having the same @lalo-ap describes: The dataframe taken from the AgGrid repsonse does not contain the changes made outside of AgGrid. If I now write that dataframe back into the cache, I lose that change.

Here's a (I hope) minimal example that shows the problem:
import streamlit as st
from st_aggrid import AgGrid
import pandas as pd

st.write("press \"add row\" button a few times, then edit a cell.")
st.write("try with reload data enabled and disabled.")

def mkdf(val):
    return pd.DataFrame.from_records([{"val": val}])

df = st.session_state.get("df", mkdf(0))

def add_row():
    i = df.get("val").max()
    new = mkdf(i+1)
    st.session_state.df = pd.concat([df, new], ignore_index=True)

st.button("add row", on_click=add_row)

reload_data = st.checkbox("reload data", value=True, key="reload_data")

response = AgGrid(
    df,
    editable=True,
    reload_data=reload_data,
    key="grid"
)

st.session_state.df = new_df = response.get("data")

c1, c2 = st.columns(2)

c1.write("before:")
c1.dataframe(df)

c2.write("after:")
c2.dataframe(new_df)

@NichtJens I think the best way is to use two different dataframes in cache: the one you pass as input to AG Grid ( for e.g. called df), and the one that comes out of AgGrid.get("data") (for e.g. called changed_df).
The trick here is that, when you add a row, you concat the new row with changed_df, and you assign this concatenation to df. In this way, what was previously typed is saved.
Dealing with True / False for reload_data does not seem to be stable.
You could adapt your code with the following:
import pandas as pd
import streamlit as st
from st_aggrid import AgGrid

def mkdf(val):
    return pd.DataFrame.from_records([{"val": val}])

# Initialize session_state
if "df" not in st.session_state:
    st.session_state["df"] = mkdf(0)

def add_row():
    i = st.session_state.changed_df.get("val").max()
    new_row = mkdf(i + 1)
    st.session_state.df = pd.concat(
        [st.session_state.changed_df, new_row], ignore_index=True
    )

st.button("add row", on_click=add_row)

grid = AgGrid(st.session_state.df, editable=True)

st.session_state.changed_df = grid.get("data")
# Cast correct dtype (st.aggrid sometime mess with casting):
st.session_state.changed_df = st.session_state.changed_df.astype(
    {col: st.session_state.df[col].dtypes for col in st.session_state.df.columns}
)

c1, c2 = st.columns(2)

c1.write("before:")
print(st.session_state.df.dtypes)
c1.dataframe(st.session_state.df)

c2.write("after:")
print(st.session_state.changed_df.dtypes)
c2.dataframe(st.session_state.changed_df)

I am encountering a similar issue where I have two data grids, A and B.  B is a calculated result from A.
When I select a row in grid A, grid B should update, that works for me.
The problem occurs when I try to allow the user to edit grid B, so I cached the updates to the session.
Although the cached data is updated, the grid does not reflect the changes. On the other hand, if I set reload_data=False, the updated values in grid B are visible, but the grid doesn't refresh when I select another row in grid A.


Word mistake

You have a small mistake in display tab, in bottom block on the image

Ah, I got it right once though 😅 , thanks for reporting this issue!


HHH-14526,  HHH-9646, HHH-15932, HHH-15933, HHH-15934, HHH-15935, HHH-15936, JPA-46 omnibus

This much-too-big PR incorporates the following work:

fix @Basic(optional=false) which had no effect
add @DiscriminatorFormula(discriminatorType=WHATEVER)
add @Where(applyInToManyFetch = true)
support @DiscriminatorOptions for JOINED inheritance
allow @ManyToOne with @JoinColumn that references the PK of a @SecondaryTable
allow subclasses to share tables in a JOINED inheritance hierarchy with a @DiscriminatorColumn
tests showing that HHH-9646 and JPA-46 are fixed
disabled test demonstrating issue HHH-15933
various cleanups and refactorings of the surrounding code

Well, this is > 6000 LOC now, and seems good, so I'm going to merge it.


Deploying to FARGATE

Having troubles deploying a task on a Fargate Cluster, it complains Task definition does not support launch_type FARGATE
Also when I specify network_mode to awsvpc it complains Error response from daemon: network awsvpc not found

@MorrisMatrix ok so looks like the last time the aws lib was updated was before Fargate was out. I just moved this project to use golang dep and grabbed the latest aws lib. I haven't really looked much into what else needs to happen but can you try the latest image and see what happens? I imagine there's more needed, but I'd like to verify before I look too much into it :)

i need this functionality as well. we need to expose a few parametes in the RegisterTaskDefinitionInput ill file a PR

@MorrisMatrix and @mlibrodo if you all want to try the latest image, I'll tag a version when it's verified that it works and close this.  Thanks @mlibrodo for the contribution!

peloton/drone-ecs:latest works for my fargate deployment

ok tag 1.0.6 will be out shortly for it. :) I'll close this out and feel free to open new issues for anything relating to Fargate.


import(`./${'foo'}/index.js`) fails

Describe the bug
import(`./${'foo'}/index.js`) fails (it complains that it can't decypher the dynamic import) but import(`./`+'foo'+`/index.js`) is fine. This happens when importing from a replaced environment variable.
Reproduction
see above
Steps to reproduce
use a defined-d variable in an import
System Info
System:
    OS: Linux 6.1 NixOS 24.05 (Uakari) 24.05 (Uakari)
    CPU: (8) x64 Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz
    Memory: 1.85 GB / 15.20 GB
    Container: Yes
    Shell: 5.2.21 - /run/current-system/sw/bin/bash
  Binaries:
    Node: 20.9.0 - /nix/store/a1hckfqzyys4rfgbdy5kmb5w0zdr55i5-nodejs-20.9.0/bin/node
    npm: 10.1.0 - /nix/store/a1hckfqzyys4rfgbdy5kmb5w0zdr55i5-nodejs-20.9.0/bin/npm
    pnpm: 8.15.1 - /nix/store/3kw9mkkwjd69jaqp5la66g10y7wckq2s-corepack-nodejs-20.9.0/bin/pnpm
  npmPackages:
    vite: ^5.0.12 => 5.0.12

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

rollup does not seem to support similar parsing either, you may need to use it as guided by the prompts.

Duplicate of #15864


Merge prTesting_201909180348 branch to master branch

Docs Build status updates of commit 12e244d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

split/updateTestCon.md
:warning:Warning

Details

split/updateTestCon.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.
[Warning] Missing attribute: title. Add a title string to show in search engine results.

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


Conflict when using monadic-html with Scala's scala-xml

I have a project that both needs monadic-html and the standard Scala scala-xml. Since I am using Scala 2.12, scala-xml is a dependency.
As I understand it, you add a new version of the Node class, so that embedding arbitrary data works. However, this makes it impossible to use mhtml's Node and scala-xml's Node at the same time.
So in the dependency load order

scala-xml
monadic-html

I get

overloaded method constructor UnprefixedAttribute with alternatives:
[error]   (key: String,value: Option[Seq[scala.xml.Node]],next: scala.xml.MetaData)scala.xml.UnprefixedAttribute
[error]   (key: String,value: String,next: scala.xml.MetaData)scala.xml.UnprefixedAttribute
[error]   (key: String,value: Seq[scala.xml.Node],next1: scala.xml.MetaData)scala.xml.UnprefixedAttribute
[error]  cannot be applied to (String, org.scalajs.dom.Event => Unit, scala.xml.MetaData)
[error] 		 command.function(e)}>

If the load order is reversed, I get

value \ is not a member of scala.xml.Elem

This is not strictly a bug, I suppose, but pretty unfortunate. If there is no way to fix this in general, I hope you might be able to point me to a workaround.
PS: This library is amazing, thanks for making it! :)

That is unfortunate. Before doing anything drastic like looking into OSGI
or using multiple classloaders (which I don't have experience with, and
maybe there's a better solution), what are you trying to use in scala-xml
(I'm guessing a parser?) that isn't in mhtml's version of it?
On Fri, Sep 1, 2017 at 10:50 AM, Freddex<EMAIL_ADDRESS>wrote:

I have a project that both needs monadic-html and the standard Scala
scala-xml. Since I am using Scala 2.12, scala-xml is a dependency.
As I understand it, you add a new version of the Node class, so that
embedding arbitrary data works. However, this makes it impossible to use
mhtml's Node and scala-xml's Node at the same time.
So in the dependency load order

scala-xml
monadic-html

I get
overloaded method constructor UnprefixedAttribute with alternatives:
[error] (key: String,value: Option[Seq[scala.xml.Node]],next:
scala.xml.MetaData)scala.xml.UnprefixedAttribute
[error] (key: String,value: String,next: scala.xml.MetaData)scala.xml.UnprefixedAttribute
[error] (key: String,value: Seq[scala.xml.Node],next1:
scala.xml.MetaData)scala.xml.UnprefixedAttribute
[error] cannot be applied to (String, org.scalajs.dom.Event => Unit,
scala.xml.MetaData)
[error]  command.function(e)}>
If the load order is reversed, I get
value \ is not a member of scala.xml.Elem
This is not strictly a bug, I suppose, but pretty unfortunate. If there is
no way to fix this in general, I hope you might be able to point me to a
workaround.
PS: This library is amazing, thanks for making it! :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/OlivierBlanvillain/monadic-html/issues/73, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA37jv7CXD1yIMmIfJVJLaFeRYsYpYU9ks5seBmdgaJpZM4PKQKc
.

--
Brandon Barker
<EMAIL_ADDRESS>

To be specific, I'm using these methods:

scala.xml.XML.loadString for the String -> xml.Elem transformation, this works fine, since it doesn't rely on Node.
XPath-like access methods/operators ("/", "//", ...). These break.

My application parses an XML format, which the scala-xml library is well suited for, and it uses mhtml for the UI.

monadic-html is intended to be used from Scala.js, and as far as I can tell scala-xml is still official unpublished for Scala.js, so there might be something wrong with your build.

It looks like we will be able to switch back to mainstream scala-xml shortly:
https://contributors.scala-lang.org/t/upcoming-scala-xml-1-1-0/985

Oh, I see! I'm still a newbie in Scala, and the whole build/cross-compile/dependency-shebang is a bit confusing starting off. I just added "org.scala-lang.modules" %% "scala-xml" % "1.0.6" to my libraryDependencies, and then it at least recognized the loadString method, so I just assumed this was working fine, the only problem being the one described above.
So there might be a release for Scala.js soon, and then this library could be updated to avoid the whole conflicting package thing?

You probably won't ever be able to cross compiler code with  loadString, given than this method relies on a Java XML parser. I'm not sure you will be able to avoid reading some documentation on sbt :P


publish L2-aux files to SOAR

[ ] implement changes
[ ] reprocess for review
[ ] review
[ ] whitelist product and check format with Helen
[ ] update DPDD
[ ] batch release "old" Files
[ ] include into daily publishing

I guess this can be closed inlieu of #396 @nicHoch?


No Cover Art

Describe the bug
When am downloading fanfic from fanficiton.net there is no cover art. Is this a bug or how do I fix it.

Also is there a way to export the fanfic you download as an epub

Also is there a way to export the fanfic you download as an epub

Currently, there is no way to export to an EPUB file. I had it in mind but ultimately put it on low priority because I didn't require the feature myself. If there's enough demand, I will implement it. Please create a separate issue regarding this.
As per the original issue, please also provide the URL of the fanfic where you found the problem.
Thank you
May you have a great day

For some background, fanfiction.net applies a dummy cover art for the fanfics which actually do not have any cover art. This dummy cover is ignored by the app currently and isn't downloaded.
For ones with the cover,  for example
https://www.fanfiction.net/s/13646184/1/The-Witch-and-the-Sorcerer

@kolbdog3333 Please create another issue if you face any problems.


changed default version name to XXX.BetaX

Thank you for submitting this pull request
JIRA: (please edit the JIRA link if it exists)
link
referenced Pull Requests: (please edit the URLs of referenced pullrequests if they exist)

referenced pull request 1
referenced pull request 2
referenced pull request 2
etc.

How to retest a PR or trigger a specific build:

* a pull request please add comment: Jenkins retest this

I think this change is not needed, nonetheless what the version will be, why we need to guide the specific version here, you still can fill whatever you want or not?

I think this change is not needed, nonetheless what the version will be, why we need to guide the specific version here, you still can fill whatever you want or not?

hmm - I think it is better for who ever does this release he has an example of how to call the version. There were so many complains about the t version extension.

there were only complaints from @ge0ffrey but it was related to Optaplanner which we removed from the list of such releases.

there were only complaints from @ge0ffrey but it was related to Optaplanner which we removed from the list of such releases.

up to you - if you find this change not needed feel free to close this - but look first on changes I have also done to the doc:
https://repository.jboss.org/nexus/content/groups/public/org/drools/drools-bom/7.46.0.Beta1/


chore(package): update tap to version 10.2.0

Closes #15
https://greenkeeper.io/

Coverage remained the same at 100.0% when pulling 4d6caf483ca274408b3e2920447c8a186b59828f on greenkeeper/tap-10.2.0 into 85711a4f47453f0edd5b7c28d8d9401e7cc20f49 on master.


⚠ Bofrost.es has degraded performance

In e22f4a9, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1524 ms

Resolved: Bofrost.es performance has improved in a773aac.


Implement precompile mock contracts and foundry mock util test contracts

Description:
This PR integrates foundry into this hardhat project using the official hardhat and foundry documentation guides.
It also implements hedera precompile mock contracts and corresponding foundry util test contracts for easy integration and testing in foundry projects.
Related issue(s):
Fixes
hashgraph/hedera-json-rpc-relay/issues/1172
Notes for reviewer:
As soon as this merged in, the foundry-example project can install hedera-smart-contracts as its only dependency and much of the common code in this PR can be removed, making the foundry-example project much simpler.
Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

@Nana-EC should I add artifacts/ to the .gitignore? It's typical for the folder to be ignored in hardhat projects and I have many changes to files in that folder and am unsure whether not ignoring that folder was deliberate.

@Nana-EC should I add artifacts/ to the .gitignore? It's typical for the folder to be ignored in hardhat projects and I have many changes to files in that folder and am unsure whether not ignoring that folder was deliberate.

No, not the whole folder.
In the .gitignore you'll see we ignore the build-info and dbg.json files under artifacts.
You can commit the .sol files that show up under artifacts/contracts/

@Nana-EC I'm not sure if Github notified you to this comment as you're not a collaborator on my fork?

Thanks for tagging me, yeah I didn't see the notification but have now responded.
Btw I would stay simple and iteratively improve as needed since this PR is getting large.

@Nana-EC I replied to your reply, not sure if GitHub notified you now that you're active on that PR. I've had some spare time lately :) I'll try to limit any further work to the tests.

Hey @Nana-EC this PR has been open for quite a while and I might not have as much time to spare in the coming weeks so I'd appreciate it if at least some of the reviewers assigned could spare some time before then.

Hey @mshakeg, yep I've periodically checked. Since the CI tasks were failing I was waiting for them to pass first.
Sure, will push to get reviewers looking at this in the next week or 2.
Thanks again for this.

Hey @Nana-EC thanks. I did make a few changes that fixed some of the workflow issues pertaining to foundry, however, there's another issue; hardhat seems to be using a gas limit of 21_000 instead of the higher default of ~300k(if gas estimation fails) which is what hardhat uses locally(and hence the test cases pass), so I specified gas in the network config in the hopes that it would fix this, however, the issue persists as can be seen here. I don't yet know what's causing the discrepancy in local execution and Github actions.
I could try to specify a gasLimit for every call and transaction, however, this is non-ideal.

Hey @Nana-EC thanks. I did make a few changes that fixed some of the workflow issues pertaining to foundry, however, there's another issue; hardhat seems to be using a gas limit of 21_000 instead of the higher default of ~300k(if gas estimation fails) which is what hardhat uses locally(and hence the test cases pass), so I specified gas in the network config in the hopes that it would fix this, however, the issue persists as can be seen here. I don't yet know what's causing the discrepancy in local execution and Github actions.
I could try to specify a gasLimit for every call and transaction, however, this is non-ideal.

By the way the team is making progress on the gasEstimation side.
I believe all function not using HTS system contract methods should work. In the current Mirror Node sprint 0.84 we should begin to see some supported HTS system contract functions.
If you were blocked on some of these then they should begin to function in the next few releases of Mirror Node as HIP 584 work progresses.

Even on<EMAIL_ADDRESS>hardhat's auto gas estimation does not work and it uses a fixed 21_000 for gas which is always inadequate.
The hardhat gas network level config override also does not work, possible due to NomicFoundation/hardhat/issues/1228

@Nana-EC finally had some time to make a few updates and get the test ci in a better state. However it still seems like quite a few test cases fail for the following reasons:

if a test case tests a revert, hardhat makes an eth_estimateGas request to the hedera local node which returns the expected revert message, however hardhat doesn't match this revert message or attempt the tx(with an overestimated gas) to get the revert message to match
contracts intermittently fail to deploy, you can confirm this by comparing the jobs in run<PHONE_NUMBER> and run<PHONE_NUMBER>

Hey @mshakeg.
I saw your ping, I can review this this week.
A couple of products have had version bumps including new features such as mirror node and local node so FYI if you want to bump to take this improvements.
Additional note is the relay has a fix to manage the reverts from Mirror Node as 200's instead of 400 HTTP codes to be consistent with other relay products. If this affects foundry then you might want to wait for the next relay version of 0.33 next week.
I'm hoping we can get simplified working version in soon and improve so let's coordinate on this.
If we can break any of this work out into separate smaller tickets feel free to do that and simplify the PR.
Thanks

@Nana-EC thanks, will hopefully allocate some time this weekend.
I just upgraded to the latest hedera local node(i.e. 2.14.0) and the revert tests cases issue as described in (1.) is no longer an issue.
However, a bunch more test cases fail such as the following in test/erc-20-extensions/ERC20Extensions.js. This is really strange since I'm running the tests on hashgraph/hedera-smart-contracts@main:latest so I would expect them all to pass. Curious to know if this is also an issue on your machine.
How would you want to break up this PR? It's already quite minimal imo. The mocks and tests implemented are relatively basic at this point.
  2) ERC20ExtensionsMock tests
       ERC20Burnable tests
         should be able to execute burn(amount):
     Error: Failed to get a different supply value after undefined tries
      at pollForERC20BurnableChangedSupply (utils/helpers.js:75:11)
      at Context.<anonymous> (test/erc-20-extensions/ERC20Extensions.js:93:31)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)

  3) ERC20ExtensionsMock tests
       ERC20Burnable tests
         should be able to execute burnFrom(address, amount):

      AssertionError: expected 800 to equal 900. The numerical values of the given "ethers.BigNumber" and "ethers.BigNumber" inputs were compared, and they differed.
      + expected - actual

      -800
      +900

      at Context.<anonymous> (test/erc-20-extensions/ERC20Extensions.js:119:29)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)

  4) ERC20ExtensionsMock tests
       ERC20Pause tests
         should pause and unpause the token:

      AssertionError: expected false to be true
      + expected - actual

      -false
      +true

      at Context.<anonymous> (test/erc-20-extensions/ERC20Extensions.js:173:54)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)

@Nana-EC thanks, will hopefully allocate some time this weekend.
I just upgraded to the latest hedera local node(i.e. 2.14.0) and the revert tests cases issue as described in (1.) is no longer an issue.
However, a bunch more test cases fail such as the following in test/erc-20-extensions/ERC20Extensions.js. This is really strange since I'm running the tests on hashgraph/hedera-smart-contracts@main:latest so I would expect them all to pass. Curious to know if this is also an issue on your machine.
How would you want to break up this PR? It's already quite minimal imo. The mocks and tests implemented are relatively basic at this point.
  2) ERC20ExtensionsMock tests
       ERC20Burnable tests
         should be able to execute burn(amount):
     Error: Failed to get a different supply value after undefined tries
      at pollForERC20BurnableChangedSupply (utils/helpers.js:75:11)
      at Context.<anonymous> (test/erc-20-extensions/ERC20Extensions.js:93:31)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)

  3) ERC20ExtensionsMock tests
       ERC20Burnable tests
         should be able to execute burnFrom(address, amount):

      AssertionError: expected 800 to equal 900. The numerical values of the given "ethers.BigNumber" and "ethers.BigNumber" inputs were compared, and they differed.
      + expected - actual

      -800
      +900

      at Context.<anonymous> (test/erc-20-extensions/ERC20Extensions.js:119:29)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)

  4) ERC20ExtensionsMock tests
       ERC20Pause tests
         should pause and unpause the token:

      AssertionError: expected false to be true
      + expected - actual

      -false
      +true

      at Context.<anonymous> (test/erc-20-extensions/ERC20Extensions.js:173:54)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)

Just did a quick review.
The breakout note was if you wanted to. I'm not requiring you break out the PR.
Although review would have definitely been faster if you had broken it out with something like the below.

Foundry config files with a single simple files
Initial set of mock contracts for simple HTS tests
More complex mock contracts and tests for more detailed tests

Tests from latest were passing as last recent PRs. We'll confirm

Replaced by https://github.com/hashgraph/hedera-smart-contracts/pull/462


Use player's metadata to get album art

Take a look at this:
https://github.com/BrianValente/MediaNotification/blob/master/app/src/main/java/james/medianotification/services/NotificationService.java#L402
With OnActiveSessionsChangedListener I can get every MediaController running on the device and access it's metadata, including the album art. I'm using it right now in my fork and works great with Spotify. Note that my fork only works with Spotify, this needs to be tested with more music players.
I'm interested in this project, if you need a team or just some help you can count on me.

This looks great, thanks! I wasn't aware of this API before (well, I was, but I didn't know how to use it). I'll test it with a few other music players tomorrow if I get time. And yes, I'm definitely interested in teaming up on this project, I could use a bit of help with it 😀
Already working on making an 'about' page in the app that lists contributors to the project using the github API, should be done tomorrow morning.


publish-wasm job fails due to insufficient permissions

Aim
This happened during performing the 0.3.0 Noir release.
Expected behavior
The publish-wasm job should succeed and link the new commit in noir-lang/noir_wasm to the commit of the release.
Bug
The job fails due to Resource not accessible by personal access token.
https://github.com/noir-lang/noir/actions/runs/4408358082/jobs/7723270706
To reproduce
No response
Installation method
None
Nargo version
No response
@noir-lang/noir_wasm version
No response
@noir-lang/barretenberg version
No response
@noir-lang/aztec_backend version
No response
Additional context
No response

Modified permissions and re-ran the job, so this should now be fixed. The issue was that the token was scoped to only noir-lang and noirup.
See here for job succeeding


Apostrophes (') in search don't work

Adding an apostrophe in the search seems to break it and give no results:
OK: "sorting i am" => many results
http://docs.raneto.com/?search=sorting+i+am
Bad: "sorting i'm" => zero results
http://docs.raneto.com/?search=sorting+i'm

Just tested and this seems to be resolved.
Thanks to @cassiobock for the improvements
Closing.


Implement Header Dashboard & Nav area from Phase 1 Mocks

@spmcbride1201 We may need assistance in integrating Sparklines


feature request: Indent before the code block

There is a known problem in obsidian - this is the design of code blocks inside lists in live preview:
https://forum.obsidian.md/t/live-preview-better-support-of-code-blocks-in-lists/31352/57
Unfortunately, the authors have been dealing with the problem for almost 2 years and without result.
At the same time, you have a wonderful extension that greatly improves the user experience!
If you could add indents in front of your code blocks in the extension, it would be wonderful!
Thanks!

You mean when a codeblock is inside a list? You mean the red vertical line I drew in with red (between 1 and 2)?

I mean the need for indentation before the code block in some cases (for example, inside the list)

ah sorry. You mean indentation. I was thinking about indentation lines. Sorry, my bad.
But I am still confused a little. On the picture the  codeblock is indented, or not? Maybe we are not talking about the same thing. Could you send two pictures. One where it is wrong, and one where I can see what you need?

Of course:

In reading mode it should work.

In edit mode it doesn't. I didn't even notice that. I will look into this

can you name other cases when the code block needs to be intended? You said, inside a list, but what are the other cases?

Here are the cases that come to mind:

If possible, you could set the left indent depending on the number of spaces before the first set of or set it to the option after (example):css indent:4

thank you! I might have an idea how to solve this. I try to solve it until the next release.

Just a quick question. This way would it be ok?

It's not perfect, because the indentation lines are missing, but I didn't find another way to solve this reliably.

Available in version 1.2.4. Please test it, and if there is any problem, just report back.


"Fight gym" should be removed from the ToDo list, and vowed to never be implemented

Hello there.
I am a Pokémon Go player, and have found it increasingly frustrating that some people opt to use bots and thus destroy the game for others. There are already people who level up their account + Pokémon extremely high and then use them to capture + keep arenas, making it extremely hard for honest players who want to enjoy a fair competition.
Bots like these are problematic enough as-is. But even allowing masses of cheaters to fight in gyms would destroy any remaining possibility to enjoy this part of the game in an honest fashion. Because even if I should put the manual effort into the game, I will never be able to compete fairly with a bot who can "run" at superior speeds and will just re-take every arena I take within a short time. Everybody should get a fair shot at the arenas, not just people who cheat.
I know that this is probably a controversial point here, but I firmly believe that in bot software for a publicly available game played by millions of people on live servers, ethics can and should be an issue! Please don't take the fun away... this is such a great, enjoyable game for many people, and certainly the bot can do enough without additionally being able to fight in arenas/gyms automatically...

I actually kind of agree with this....
Coming from someone who was an original developer in the software - but also goes out every night and plays manually - I'll actually talk to the team about this.

I also agree... but if they don't do it someone else will do it...
This is not the only one bot out there.

Well, sure. But perhaps the authors of the other bots can be persuaded as
well. The harder it is to get a bot with this feature, the better...
I guess I find it sad that these bots exist at all, but at least my hope is
that most people using them are lazy and won't walk around
Dima<EMAIL_ADDRESS>schrieb am Do., 28. Juli 2016, 04:39:

I also agree... but if they don't do it someone else will do it...
This is not the only one bot out there.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PokemonGoF/PokemonGo-Bot/issues/1318#issuecomment-235785010,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2J9qTcPD5vQF90ZxVWdjP0QfHLU8O-ks5qaBZUgaJpZM4JWzry
.

i do agree.... no gym bot pls

Agree, doing so will only gather Niantic's attention on spending more resources on bot hunting and account banning.

I would agree as well. I personally use this bot to satisfy the need to catch Pokemon while I am otherwise unable to due to many responsibilities. That said I have my main account which is 100% legit to battle gyms with. Using a bot to combat gyms would be parasitic to the game overall in my opinion, and even more reason for Niantic to start wielding the ban hammer.

This project has attracted my attention and I'm really interested in seeing it develop. I haven't had the time to contribute to it but plan on it once I get the time. My opinion might not throw a lot of weight here, but I'd just like to say that I agree.
I agree that developers should make ethical decisions when writing bots but when they threaten the quality of the game for others it attracts a lot of negative attention. I'm sure other people will make it, but I'd love to see this project take the higher ground.

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1, no one really needs or wants that functionality. Think of the poor legit players in Central Park.

-1 want dat pokemoney so bad to expend that damn inventory

@mehdi-S
Would still ruin the fun for every legit player around you. We should at least take the gyms manually with our botted accounts.

+1 it's already extremely frustrating battling at gyms (tons of glitches) and when you finally take the gym down, someone comes in and steals it. Having bots will basically make it impossible for normal players to take gyms.

I'm one of the very first and most active contributors not among the founders of this bot, and my personal opinion is that we should not implement Gym fights.
We will have an official announcement and documentation update about the whole team's decision with regards to Gym fights published in a few minutes.

The core contributors thank you all very much for your care and concern of this issue.
We are in agreement, adding functionality to this bot (or any other) that will enable people to automatically battle gyms is not in the best interest of the game and thus not something we want to do. The PokemonGoF team and our numerous contributors are big fans of Pokemon Go and strive to enable people to enjoy the game in a new way while not ruining the experience for the rest of the community of players.
We will not now or ever include any gym battling related functionality in this bot and we encourage other bots in the community to join our stance.
We have updated our readme as suggested to remove that line item and have added messaging about our stance. If anyone has additional concerns they believe are not handled with our direction, please reach out.


xpPackageDevel remove PRJ_NAME

CI requires that the devel package name match the name of the repo
for example, if the repo is named NG_GDP, the devel package should be named NG_GDP-<git describe --tags>-<OS>-devel.tar.xz
I was trying to minimize xpFindPkg() changes -- so trying to keep
xpFindPkg(PKGS sdl_ggdp)

so I introduced the PRJ_NAME parameter to xpPackageDevel() so the package could be named differently than the repo
but since CI requires them to be the same there is no reason to give a project the ability to change this

completed with commit to main branch referenced above

the call in VantageServer will need to be updated to
xpFindPkg(PKGS ng_gdp)


Azure ML workspace service fails on deployment when ARM and Auth tenants are different

Azure ML workspace service fails on deployment when ARM and Auth tenants are different.
Steps to reproduce

deploy TRE with AAD_TENANT_ID != ARM_TENANT_ID
create a workspace
assign researchers to the workspace
deploy AML workspace service
it will fail on assigning data scient role for the researches

│ Error: authorization.RoleAssignmentsClient#Create: Failure responding to request: StatusCode=400 -- Original Error: autorest/azure: Service returned an error. Status=400 Code="PrincipalNotFound" Message="Principal 643e0d2b1dac4472b01fd816fe693c65
does not exist in the directory *******. Check that you have the correct principal ID. If you are creating this principal and then immediately assigning a role, this error might be related to a replication delay. In this case, set the role assignmen
t principalType property to a value, such as ServicePrincipal, User, or Group.  See https://aka.ms/docs-principaltype"
│
│   with azurerm_role_assignment.app_role_members_aml_data_scientist["643e0d2b-1dac-4472-b01f-d816fe693c65"],
│   on roles.tf line 22, in resource "azurerm_role_assignment" "app_role_members_aml_data_scientist":
│   22: resource "azurerm_role_assignment" "app_role_members_aml_data_scientist" {

consider adding a warning in the docs that it works only if it is in the same tenant and fail fast with an error message inside get_app_role_members.sh or try to filter out irrelevant users.

As AML relies on the Azure Active Directory in the tenant it is deployed into it is only possible to use Azure ML when the AAD Tenant is the same as the ARM Tenant. Hence this issue cannot be resolved.


Internal Server Error on editing interim meeting

Describe the issue
I was editing an existing SCITT interim meeting request to add additional sessions, and upon trying to save, got an internal server error at https://datatracker.ietf.org/meeting/interim/request/interim-2022-scitt-02/edit.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

I see this error, both on production and on the sandbox - it looks like something went wrong while it was saving the changes, resulting in a partial save. Technically, it created the Session record for each session but did not schedule it into a TimeSlot as it should have. That is what led to the error you're now seeing.
Can you describe what edits you tried to make that led to the error? How many sessions did you add and what dates (roughly) did you specify?
It looks like you canceled the meeting later - guessing that was in the hopes that it would clear this up to try again. That was a reasonable thing to try, but didn't work. Partly that's because of the original problem, but I think there are some bugs to be filed related to canceling the interim...

I think #4846 obviates this issue. If it recurs, reopen this or start a new issue.


envoy: 1.25.1 -> 1.26.1

Description of changes

https://github.com/envoyproxy/envoy/compare/v1.25.1...v1.26.1

v1.25.2 (https://www.envoyproxy.io/docs/envoy/v1.26.1/version_history/v1.25/v1.25.2)

dependency: update Kafka to resolve CVE-2023-25194.
docker: unify published images as tag variants. For example, envoyproxy/envoy-dev is now available as envoyproxy/envoy:dev
ext_authz: deprecated (1.25.0) ext_authz.v3.AuthorizationRequest.allowed_headers in favour of ext_authz.v3.ExtAuthz.allowed_headers.

v1.25.3 (https://www.envoyproxy.io/docs/envoy/v1.26.1/version_history/v1.25/v1.25.3)

dependency: update Wasmtime -> 6.0.1, to resolve CVE-2023-26489, CVE-2023-27477.

v1.25.4 (https://www.envoyproxy.io/docs/envoy/v1.26.1/version_history/v1.25/v1.25.4)

http: Validate upstream request header names and values. The new runtime flag envoy.reloadable_features.validate_upstream_headers can be used for revert this behavior.
grpc: when Envoy was configured to use ext_authz, ext_proc, tap, ratelimit filters, and grpc access log service and an http header with non-UTF-8 data was received, Envoy would generate an invalid protobuf message and send it to the configured service. The receiving service would typically generate an error when decoding the protobuf message. For ext_authz that was configured with failure_mode_allow: true, the request would have been allowed in this case. For the other services, this could have resulted in other unforseen errors such as a lack of visibility into requests (eg request not logged). Envoy will now by default sanitize the values sent in gRPC service calls to be valid UTF-8, replacing data that is not valid UTF-8 with a ‘!’ character. This behavioral change can be temporarily reverted by setting runtime guard envoy.reloadable_features.service_sanitize_non_utf8_strings to false.
http: fixed a bug where x-envoy-original-path was not being sanitized when sent from untrusted users. This behavioral change can be temporarily reverted by setting envoy.reloadable_features.sanitize_original_path to false.
http: stop forwarding :method value which is not a valid token defined in https://www.rfc-editor.org/rfc/rfc9110#section-5.6.2. Also, reject :method and :scheme headers with multiple values.
http3: reject pseudo headers violating RFC 9114. Specifically, pseudo-header fields with more than one value for the :method (non-CONNECT), :scheme, and :path; or pseudo-header fields after regular header fields; or undefined pseudo-headers.
lua: lua coroutine should not execute after local reply is sent.
oauth2: fixed a bug where the oauth2 filter would crash if it received a redirect URL without a state query param set.

v1.25.5 (https://www.envoyproxy.io/docs/envoy/v1.26.1/version_history/v1.25/v1.25.5)

dependency: update Curl -> 8.0.1 to resolve CVE-2023-27535, CVE-2023-27536, CVE-2023-27538.
http: amend the fix for x-envoy-original-path so it removes the header only at edge. Previously this would also remove the header at any Envoy instance upstream of an external request, including an Envoy instance that added the header.

v1.25.6 (https://www.envoyproxy.io/docs/envoy/v1.26.1/version_history/v1.25/v1.25.6)

tls: Fix build FIPS compliance when using both FIPS mode and Wasm extensions (--define boringssl=fips and --define wasm=v8).

v1.26.0 (https://www.envoyproxy.io/docs/envoy/v1.26.1/version_history/v1.26/v1.26.0)

many

v1.26.1 (https://www.envoyproxy.io/docs/envoy/v1.26.1/version_history/v1.26/v1.26.1)

tls: Fix build FIPS compliance when using both FIPS mode and Wasm extensions (--define boringssl=fips and --define wasm=v8).

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build envoy


Compose: bell icon doesn't forward to Sensor Settings and Alerts

Description:
Bell icon doesn't forward to Sensor Settings and Alerts
Broken on both sensor card view and history view

QA passed


Update all branches to remove docs:push from package.json

Remove the postPublish script from package.json across all the branches, for github action to perform release.


DO NOT MERGE: Streaming List Keys, Buckets, MapReduce, and 2i

List keys streaming.

Moved this work to another branch.


MeshOutline applied full object instead of outlining

Describe the bug
Creating new material for outline is applying material color to the entire object.
To reproduce

Open OutlineExamples from below path

Duplicate the Cube on the left-side.

Create new material with a color (Yellow here) with below options

Apply the material

Run the application

Notice the color of the copied cube. Entire cube has Yellow color instead of outline.

Note: If I switch the material to OutlineOrange and other colors available within MRTK, then outline is properly rendered. It appears something silly happening with the newly created material.
Expected behavior
The applied material color should come as outline instead of applying the entire object.
Screenshots
Already provided above.
Your setup (please complete the following information)

Unity Version 2019.4.14f1
MRTK Version v2.5.3

Target platform (please complete the following information)

PC, Mac & Linux Standalone
Android

Additional context
NA

@abhilashca


Feature/support multiple reporters and control log level for logging reporter

Allows adding multiple reporters to a registry and setting the log level logging method that will get used by the Logging reporter

Pull Request Test Coverage Report for Build 203

19 of 22 (86.36%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 88.448%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/measured-signalfx-reporter/lib/registries/SignalFxSelfReportingMetricsRegistry.js
1
4
25.0%

Totals

Change from base Build 198:
-0.3%

Covered Lines:
740

Relevant Lines:
818

💛  - Coveralls


properly fix bug

The bug still appears. This fixes the issue when entering a menu and exiting.

Is there any specific case where this actually fixes anything?
From what i can see, your change immediately disables the focus again after it has been set, leaving only the hover-effect, which makes sliders look a little awkward when they're moved with the cursor outside of the actual widget bounds.

Is there any specific case where this actually fixes anything?
From what i can see, your change immediately disables the focus again after it has been set, leaving only the hover-effect, which makes sliders look a little awkward when they're moved with the cursor outside of the actual widget bounds.

I fixed the sliders issue. I'm testing this on 1.21 btw, and I made this PR because the mod was not working on 1.21 or 1.20.6 for me. Idk if it works properly on 1.20.2.

Also the sliders disabled the focus effect when moving the mouse off of it in 1.19.3.

Interesting... For me, the mod is working 100% as intended on both 1.20.5 and 1.21. Can you confirm that this isn't an issue with a specific mod's widget? (see #4 for more info)

Also the sliders disabled the focus effect when moving the mouse off of it in 1.19.3.
I see. Actually didn't even remember that. If you revert the 2nd and 3rd commit (i honestly have no idea if/how i can do that myself), i can merge it (as those changes won't be needed in that case), and then it should be exactly as it was in 1.19.3.

Interesting... For me, the mod is working 100% as intended on both 1.20.5 and 1.21. Can you confirm that this isn't an issue with a specific mod's widget? (see #4 for more info)

Also the sliders disabled the focus effect when moving the mouse off of it in 1.19.3.

I see. Actually didn't even remember that. If you revert the 2nd and 3rd commit (i honestly have no idea if/how i can do that myself), i can merge it (as those changes won't be needed in that case), and then it should be exactly as it was in 1.19.3.

I just reverted the commits.

Also, it's not a specific mods widgets.

merged, thanks.


Drop Python 3.7 support

Proposed change

Drop Python 3.7 support
Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[ ] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[x] Dependency upgrade (version update)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts and has clean commit history
[x] The code style looks good: make pre-commit
[x] All tests pass locally: make test, make tox (we strongly encourage adding tests to your code)
[x] The related documentation has been added/updated (check off the box for free if no updates is required)

coverage: 100.0%. remained the same when pulling d11101b777e058e2e68ea78eb5bcacd77a2e7875 on arkid15r:deprecate-python-3.7 into 3a44170779ca839ee125746b97082222a6bd9e92 on dr-prodigy:beta.


delete all resource in resource group but not the resource group

Hi
I need to clear a resource group from all the components (resources) but I must leave the resource group untouched, (I gave rights to the service principle  on the resource group)
do you have a way to do that?
Thank you for great scripts
Izia

Hi @itzikorfa - interesting problem. Can you delete the resource group and then recreate it?
In case not, I uploaded a new azurerm release 0.9.6 which includes a get_resource_group_resources() function. You could use this to list the resources in a resource group. You could then do a delete on each resource, and maybe we can add a generic resource delete function that takes a resource id. The only problem would be you can't delete a resource if something depends on it. For example you can't delete a public IP address if it is associated with a NIC. If you know what resources are likely to be in a resource group, then it wouldn't be too hard to order appropriately.


Update challenge UI to support multiple coding panes for JavaScript, HTML, and CSS

Update challenge UI to support multiple coding panes for JavaScript, HTML, and CSS

Update: @BerkeleyTrue says the UI will now support multiple coding panes but we don't have a system in place to execute multiple panes.


Log startup time and config

Description
Log info about the start-up time and configuration used when connections are being opened. The existing logging of such info in RxDocumentClientImpl can't calculate the total time since it is missing few calls before and after in the flow and was not printing all the info. It was also not checking the log level if enabled and thus was putting on stack data and calls that were not needed (like converting connect policy to string) and wasting CPU cycles.
The logged info looks like this (sample taken from local run with this change):
INFO com.azure.cosmos.CosmosClientBuilder - Cosmos Client with (Correlation) ID [00005] started up in [335] ms with the following configuration: serviceEndpoint [https://forklift1-dev-azure-cosmosdb.documents.azure.com:443/], preferredRegions [[]], connectionPolicy [ConnectionPolicy{httpNetworkRequestTimeout=PT1M, tcpNetworkRequestTimeout=PT5S, connectionMode=DIRECT, maxConnectionPoolSize=1000, idleHttpConnectionTimeout=PT1M, idleTcpConnectionTimeout=PT0S, userAgentSuffix='-wmt-azsdk-java-cosmos/4.41.0-beta.1 MacOSX/12.6.2 JRE/1.8.0_333', throttlingRetryOptions=RetryOptions{maxRetryAttemptsOnThrottledRequests=9, maxRetryWaitTime=PT30S}, endpointDiscoveryEnabled=false, preferredRegions=null, multipleWriteRegionsEnabled=false, proxyType=null, inetSocketProxyAddress=null, readRequestsFallbackEnabled=false, connectTimeout=PT5S, idleTcpEndpointTimeout=PT1H, maxConnectionsPerEndpoint=130, maxRequestsPerConnection=30, tcpConnectionEndpointRediscoveryEnabled=true}], consistencyLevel [Session], contentResponseOnWriteEnabled [true], sessionCapturingOverride [false], connectionSharingAcrossClients [false], clientTelemetryEnabled [false].
There are no tests since looks like logging info is not verified anywhere in the SDK (if there are samples let me know) and the above snapshot was the result of the change being running. Also it is not clear if one should use the core SDK ClientLogger or directly use log4j log factory as done in some of the impl classes.

@Gueorgi please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

Contributor License Agreement

I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree [company="Walmart"]

@microsoft-github-policy-service agree company="your company"

@microsoft-github-policy-service agree company=Walmart

@microsoft-github-policy-service agree company=Walmart

The code part has been merged in https://github.com/Azure/azure-sdk-for-java/pull/33267 so this PR is no longer needed.


Add Babel to users 🎉

Motivation
Babel launched their new website, and it's using Docusaurus!
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Netlify.
Related PRs
NA

Just wanted to give a heads up to @hzoo about this.

yep, thanks! Still plenty of stuff we want to do (translation/versioning) but figured we should switch over 😄since were getting a lot of questions about v6 and v7 docs


.tex files cannot be converted to PDF by TeXworks

.tex files cannot be converted to PDF by TeXworks. Always indicate that pdfTex is not configured properly

Ensure that you are copying both your .TEX file and the ../layers directory. The .TEX files include this at line #4:
\subimport{../layers/}{init}

You could edit line #4 if you want to change the location of the layers directory in your project.
I got the .TEX files to work in Overleaf.

@jtquisenberry  how to generate for custom architectures


💬 [Question] [Design] No way to create a new table from table list

💬 Question

This is a question for the Designer.
As user I can see open tables, but I am unable to find a button to open a new Table in the corresponding list.

@ulfgebhardt I understood it like this:
We don't want to have multiple places where we create tables.
So the icon is misleading since we should only find open tables here.
I wrote in our Konzeption channel a request to Hardi to create a Icon without +


Create edit profile feature on profile page

Please do work for this task in a branch called issue-055.

Logged in users can only edit first and last name.
Need to implement more editable features for faculty and office positions.


Open VSX Listing: Signing the Publisher Agreement

Thank you for being part of the Open VSX community by adding your extensions to the Open VSX Registry. Please note that the service was recently transferred to the Eclipse Foundation and urgent action on your part is needed so we can continue to list your extensions. To ensure uninterrupted service, please sign the Eclipse Publisher Agreement on or before January 8, 2021. If not signed by that date, your extensions will be delisted and will no longer appear on the site nor be available via the API. If you sign at a later date, your extensions will then be re-activated. The signing process is explained in the Wiki (steps 1 and 2).
Please also note that all extensions MUST have a license in order to be listed.
More details are in these recent blog posts:
https://blogs.eclipse.org/post/brian-king/open-vsx-registry-under-new-management
https://blogs.eclipse.org/post/brian-king/new-era-open-vsx-registry
Today, there’s growing momentum around open source tools and technologies that support Visual Studio (VS) Code extensions. Leading global organizations are adopting these tools and technologies. This momentum has spurred demand for a marketplace without restrictions and limitations. Thanks for joining us on this journey as we continue to build the Open VSX community. We look forward to continued innovation from you in 2021!

@damonsk Just a reminder for you in case you missed this. Our deadline was extended but the new one is approaching fast, this Friday January 29.

@damonsk Just a reminder for you in case you missed this. Our deadline was extended but the new one is approaching fast, this Friday January 29.


Added a new game Red Light Green Light

Top ball is Player 1, bottom ball is Player 2. Player 1 can press Up Arrow to move and Player 2 can press 'w' to move. Rules are same as it was in Squid Game.
PR Description 📜
My game "Red Light, Green Light" which is inspired by a game in a popular Netflix series "Squid Game", have a really good concept which is you have to avoid the doll's sight otherwise you will die. The participants find themselves on a massive playground with a giant animatronic doll at the opposite end. If any player is caught moving after the command of "Red Light," they are eliminated from the game.
Fixes  #456

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @uzmaansari23 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @uzmaansari23 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Chef backup does not work if it is not /var/chef

Description
I wanted to backup my configuration file to a specific folder directory in my linux workstation prior to the template overwriting it.  I wanted to override the default backup directory by chef
So I tried doing this.
Chef::Config[:file_backup_path] = "/var/my_backup"

template "myconfig.ini" do
  source 'myconfig.ini.erb'
  backup 5
end

It does not work.  Everything is getting back up still at the /var/chef/backup folder.
However changing it to this code, works with no issues.
Chef::Config[:file_backup_path] = "/var/chef/my_backup"

template "myconfig.ini" do
  source 'myconfig.ini.erb'
  backup 5
end

So any folder that is not within the /var/chef directory is not allowed but any folder that is within the /var/chef/ is allowed
Chef Version
Chef-solo 12.6.0
Platform Version
Linux
Replication Case

Create a recipe that creates a template.  A file should be existing on that disk.
On the template resource set the backup property.
Prior to the template being called, set the file_backup_path to a different directory other than the /var/chef/.

Client Output
Backup does not work.  It is still is backing the directory to /var/chef/backup.
Stacktrace
No Stacktrace as there is no exception thrown only that the output is not expected
NOTE: CHEF CLIENT BUGS ONLY
This issue tracker is for the code contained within this repo -- chef-client, base knife functionality (not
plugins), chef-apply, chef-solo, chef-client -z, etc.

Requests for new or alternative functionality should be made to feedback.chef.io
Chef Server issues
ChefDK issues
Cookbook Issues (see the https://github.com/chef-cookbooks repos or search Supermarket or GitHub/Google)

Does /var/my_backup exist?

Yes it exist prior to me running the code.  I am running with root privilege also.

Interesting. Can you try with an up to date chef client, please? And then gist a debug log of the run with chef-solo -l debug and put the link here.

I did a quick yum remove of my chef..then I downloaded my rpm.  My chef-solo version is now
chef-solo -v
Chef: 12.19.36

Retried my code but still the same.  Putting it in subdirectory of /var/chef works though.  But other than that, it is ignored.
Sorry but I am new to chef and I am just trying it out..I am not sure howto give you the logs.  I tried running debug but I am getting some exception

Tried other things but still the same. Could this be a bug?

yes, possibly. Please run chef with -l debug on the end of your normal command line and attach the output to a gist.


Electron: add caveat regarding symlink

Using the symlink will cause the app not to accept user input. See this issue for more details https://github.com/atom/electron/issues/1151#issuecomment-92085514

I’m missing the error, here.

There were extra spaces after the caveats section. This is now fixed.
Cheers.

Thank you for the contribution. It needed some fixes, so they were made in 12f5f6002d7a6b7fdd55db3ba56f5d19d3e6cc01. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Dnscrypt-proxy starts then terminates on boot - no internet - Win 7 Pro 64-bit

Five months ago, we installed Simple DNSCrypt 64-bit on our Win 7 Pro 64-bit machine (16GB RAM, Verizon FIOS 1Gbps), which in turn installed the dnscrypt-proxy process and service on the machine.
Starting just three weeks ago ±, on reboots, our internet connection got spotty on this machine and then started to die.  Not all reboots, but maybe half.  I figured out that the dnscrypt-proxy process and service are starting on reboots but then something is terminating them pretty quickly as booting continues.  When they terminate, the PC cannot connect to the internet.
I tried a number of things, but the only immediate solution I found was to change the dnscrypt-proxy service's start to "Automatic (Delayed Start)", which works but causes every reboot to take longer.
Here's a link that will take you to an xls with the entire Autoruns list of startup items:  https://onedrive.live.com/view.aspx?resid=C405D387ABA56BA!162&ithint=file%2cxlsx&app=Excel&authkey=!ADan0i2LhXcYGyk   (Rows with strikeout text are DISABLED and do not start.)   Do you see anything in this startup list that would cause the dnscrypt-proxy process and service to terminate?
UPDATE - SOME PROGRESS - Oct 14 2018
I have done the following:

I have written this .bat file and saved it to my desktop - and you will see that the next-to-last line is the important one:

cd "C:\Users\[MyUserName]\Desktop\"
echo. >> start-dnscrypt-results.txt
echo -------------------- >> start-dnscrypt-results.txt
echo. >> start-dnscrypt-results.txt
echo %DATE% %TIME% >> start-dnscrypt-results.txt
echo. >> start-dnscrypt-results.txt
sc start dnscrypt-proxy >> start-dnscrypt-results.txt
echo. >> start-dnscrypt-results.txt

I have gone into Task Scheduler and scheduled the above batch file to run on Startup, with a delay of 30 seconds, and with Highest Privileges. (I had to use Task Scheduler with Highest Privileges because I could NOT get my bat file to run as Administrator otherwise - if I run it any other way, I must manually run it "as Administrator" and click Yes on the UAC prompt.)

I configured the Properties of the service dnscrypt-proxy to retry to start three times.

I also needed to change that service to start as "Automatic" and not "Automatic (Delayed Start)".

Well, it is working.
When I reboot, I can see either that the service is starting and staying on, in which case the .bat file leaves an error message in the results file,
or that the service starts and then stops and then starts again, with success codes in the results file.
What do you think?
By the way, that results file looks like this --

[SC] StartService FAILED 1056:
An instance of the service is already running.
Sat 10/13/2018 18:32:52.80
[SC] StartService FAILED 1056:
An instance of the service is already running.
Sat 10/13/2018 18:35:04.10
[SC] StartService FAILED 1056:
An instance of the service is already running.
Sat 10/13/2018 19:11:52.22
SERVICE_NAME: dnscrypt-proxy
TYPE               : 10  WIN32_OWN_PROCESS
STATE              : 2  START_PENDING
(NOT_STOPPABLE, NOT_PAUSABLE, IGNORES_SHUTDOWN)
WIN32_EXIT_CODE    : 0  (0x0)
SERVICE_EXIT_CODE  : 0  (0x0)
CHECKPOINT         : 0x0
WAIT_HINT          : 0x7d0
PID                : 3532
FLAGS              :
Sat 10/13/2018 20:52:48.19
[SC] StartService FAILED 1056:
An instance of the service is already running.
Sun 10/14/2018  7:17:51.31
SERVICE_NAME: dnscrypt-proxy
TYPE               : 10  WIN32_OWN_PROCESS
STATE              : 2  START_PENDING
(NOT_STOPPABLE, NOT_PAUSABLE, IGNORES_SHUTDOWN)
WIN32_EXIT_CODE    : 0  (0x0)
SERVICE_EXIT_CODE  : 0  (0x0)
CHECKPOINT         : 0x0
WAIT_HINT          : 0x0
PID                : 2592
FLAGS              :
Sun 10/14/2018  9:27:55.70
[SC] StartService: OpenService FAILED 5:
Access is denied.
Sun 10/14/2018 10:25:22.13
[SC] StartService FAILED 1056:
An instance of the service is already running.
Sun 10/14/2018 10:27:36.17
SERVICE_NAME: dnscrypt-proxy
TYPE               : 10  WIN32_OWN_PROCESS
STATE              : 2  START_PENDING
(NOT_STOPPABLE, NOT_PAUSABLE, IGNORES_SHUTDOWN)
WIN32_EXIT_CODE    : 0  (0x0)
SERVICE_EXIT_CODE  : 0  (0x0)
CHECKPOINT         : 0x0
WAIT_HINT          : 0x7d0
PID                : 4048
FLAGS              :
Sun 10/14/2018 11:12:03.61
[SC] StartService FAILED 1056:
An instance of the service is already running.

Thanks.

And maybe there's another possibility, although I have not tested it,
Go to C:\Program Files\bitbeans\Simple DNSCrypt x64\dnscrypt-proxy, look for the file dnscrypt-proxy.exe, right-click it, go to Properties, click on the "Compatibility" tab, hit the button at the bottom "Change settings for all users", and now enable "Run this program as an administrator".
Again, I have not yet tested to see whether this (by itself) overcomes the unknown evil force that is stopping the dnscrypt-proxy service on some (not all) of my boot-ups.
Please report back.

stealzy - can you see what is happening at your bootup?
I also (for a very long time) have had Task Manager start automatically at bootup.  It's a separate Task Scheduler item.  If I leave Task Manager on the Services tab, then, when it starts, I see that dnscrypt-proxy starts then stops, and then my bat file kicks in to start it again, dnscrypt-proxy reaches "Running" and my bat task exits.
Before you try my bat file, please make Task Manager start at bootup and let us know what you see.

Also I'm seeing this issue on a computer with Windows 7 and 8.1, both x64 Enterprise. I tested with UAC disabled through secpol but it's still happening, and, as previous reports, on boot and when using for a while. If threre's something I can help to debug or provide additional info, please let me know.

In case it's not clear, my steps 1 - 4 above have been working.

I'm experiencing the same issue. It looks to me as if the service doesn't start properly if it can't download the latest resolvers list, which may be the case e.g. when using a dial-up connection. If I reboot the same day then it works fine. But if I reboot after a few days, it apparently tries to update the list and fails to start at all.

rkwk - I'm not a tech and you may be 100% right in your observation.
However, I'm on 1Gbps FIOS with low latency (always ≤5ms), not dial-up, and I definitely have this issue about half of my reboots.
Maybe on reboot our NIC+network connections don't start quite fast enough or give priority to other processes and services, dnscrypt-proxy stops if it doesn't get an immediate NIC+network connection to the internet, and then with the dnscrypt-proxy service stopped, the PC cannot connect to the internet at all.  Maybe that's why my steps 1) - 4) above work.

It's still a mystery why dnscrypt-proxy service shows as "Stopped" in my Task Manager during bootup.   (My bat file then kicks it into "Starting" and "Running".)
Any ideas what causes this?  Sequence of startup items?
My PC also has as startup items: Avast Free AV, MalwareBytes, OneDrive, SpiderOakOne, PCloud, iReboot (because my PC dual boots 7 and 10, both Pro 64-bit), Start Menu X ...
Of course, a million other things also start on boot.  Please see my link at the start here to an xls printout of ALL my startup items from Autoruns.
Any ideas, anyone?

I just noticed that after running Simple DNSCrypt with no issue of being deactivated, it happened again after reboot. The notorious change made in the system, are updates installed by Windows Update, and perhaps it's likely cleaned by Windows Defender KB915597. (Windows 7 Enterprise x64 SP1). It's worth investigating more in depth, but I didn't found a log or proper way to track Defender activities.
If anyone can spot a matching as consequence after install updates, please report.

dimsdein - you may well be right, but my Win 7 Pro 64-bit system uses Avast AV Free, not Windows Defender, and I have the problem on some bootups.

dimsdein - you may well be right, but my Win 7 Pro 64-bit system uses Avast AV Free, not Windows Defender, and I have the problem on some bootups.

The PC has ESET installed, however, real-time analysis of Windows Defender was disabled. Now I've disabled system-wide Windows Defender through Group Policy,  and I'll keep an eye if it was causing conflicts.

In my case it was the cumulation of 2 problems that I solved.

When my machines boots without any active internet connection (e.g. used non-autoconnect WiFi or unplugged LAN cable) then the dnscrypt-proxy service does not start.
I solved this by creating a simple task in the Task Scheduler that automatically restart the dnscrypt-proxy service when the Internet connection is established (I have added to the post the attachment d3bil0x-DNSCryptRestart.zip in the xml format that must be imported into the Windows Task Scheduler).
But sometimes the above solution failed, fortunately I discovered the culprit - the popular PeerBlock app, which (based on its default lists) blocks access to the default address of SimpleDNSCrypt Advanced Settings->Fallback Resolver: <IP_ADDRESS>:53. if the validity period of the Resolvers List expires (it seems to me determined by the refresh_delay = 72 parameter from the [sources.public-resolvers] section from the dnscrypt-proxy.toml configuration file). The solution is to add the address <IP_ADDRESS> to the exceptions in the PeerBlock or change the address of SimpleDNSCrypt Fallback Resolver to another, e.g. GoogleDNS <IP_ADDRESS>:53 or OpenDNS <IP_ADDRESS>:53

After 2 weeks of testing all works fine.

In my case it was the cumulation of 2 problems that I solved.
1. When my machines boots without any active internet connection (e.g. used non-autoconnect WiFi or unplugged LAN cable) then the `dnscrypt-proxy` service does not start.
   I solved this by creating a simple task in the Task Scheduler that automatically restart the `dnscrypt-proxy` service when the Internet connection is established (I have added to the post the attachment [d3bil0x-DNSCryptRestart.zip](https://github.com/bitbeans/SimpleDnsCrypt/files/2576315/d3bil0x-DNSCryptRestart.zip) in the xml format that must be imported into the Windows Task Scheduler).

2. But sometimes the above solution failed, fortunately I discovered the culprit - the popular PeerBlock app, which (based on its default lists) blocks access to the default address of SimpleDNSCrypt `Advanced Settings->Fallback Resolver: <IP_ADDRESS>:53`. if the validity period of the Resolvers List expires (it seems to me determined by the `refresh_delay = 72` parameter from the `[sources.public-resolvers]` section from the `dnscrypt-proxy.toml` configuration file). The solution is to add the address <IP_ADDRESS> to the exceptions in the PeerBlock or change the address of SimpleDNSCrypt Fallback Resolver to another, e.g. GoogleDNS <IP_ADDRESS>:53 or OpenDNS <IP_ADDRESS>:53

After 2 weeks of testing all works fine.
NOTE: For a connections requiring longer time, eg for performing slower DHCP autoconfiguration, you can enable the parameter Edit Trigger-> Advanced Settings-> Delay task for: x seconds from Task Scheduler.

@d3bil0x You are right, after testing issue was hidden by coincident moves. Nonetheless, I think that dnscrypt-proxy should be aware by waiting for internet connection.

Did you guys try to play with this: https://github.com/jedisct1/dnscrypt-proxy/blob/master/dnscrypt-proxy/example-dnscrypt-proxy.toml#L187 ?

Hi @bitbeans.
With the netprobe_timeout = 0 it looks like it works :)

dimsdein, d3bil0x and Mr. Hermann -
First, a txt copy of my dnscrypt-proxy.toml is here --
https://1drv.ms/t/s!ArpWuno4XUAMgST2ktvO7c4faIGw
Second, I love Simple DNSCrypt on my PC (and DNSCloak on my iPad) but I'm not a tech.  Could the three of you do a little brainstorming and put the final fix into a new version of Simple DNSCrypt itself?
I am NOT convinced it is ONLY one thing or another.  My workaround Task Scheduler+bat is working, but it is not the same as the notes here from dimsdein and d3bil0x - and I do not have the same anti-virus as you guys.  There is a deeper reason for the glitch I think.

dimsdein, d3bil0x and Mr. Hermann -
First, a txt copy of my dnscrypt-proxy.toml is here --
https://1drv.ms/t/s!ArpWuno4XUAMgST2ktvO7c4faIGw
Second, I love Simple DNSCrypt on my PC (and DNSCloak on my iPad), but I'm not a tech and I cannot follow your suggestions. I think dnscrypt-proxy is good tech for good social reasons but ideally should work nicely.
Could the three of you do a little brainstorming and put the final fix into a new version of Simple DNSCrypt itself?
I am NOT convinced it is ONLY one thing or another. My workaround Task Scheduler+bat is working, but it is not the same as the notes here from dimsdein and d3bil0x - and I do not have the same anti-virus as you guys. There is a deeper reason for the glitch I think.

@glnzglnz Change option netprobe_timeout to netprobe_timeout = 0 in your 'dnscrypt-proxy.toml', that is what is working for @d3bil0x, seems that dnscrypt-proxy shuts down when a network is not available after netprobe_timeout has reached. I assume that setting to 0 dnscrypt-proxy will wait until a network is available.

0.6.1 will have this setting in the UI

0.6.1 released with netprobe_timeout.