Dropout keep_prop for training with VGGFace2, should be 0.6 rather than 0.4.

I noticed davidsandberg is reproducing ArcFace. The training parameter is set to be identical to Arcface.
But in Arcface, the author claimed "In this paper, the dropout parameter is set as 0.4". The author is using MXNet, and the dropout parameter in MXNet is the probability to drop.
So in Tensorflow, the keep_prop for dropout should be 0.6, rather than 0.4.

@dragonasc2 But i cant find arcface code in david`s project?

@ZvcXoyo yeah, no arcface softmax loss in david's project, but the embedding size, fixed standardization, weight decay are identical to arcface.

OK，thanks：D
发送自 Windows 10 版邮件应用
发件人: Dragon_n
发送时间: 2018年4月27日 10:06
收件人: davidsandberg/facenet
抄送: ZvcXoyo; Mention
主题: Re: [davidsandberg/facenet] Dropout keep_prop for training withVGGFace2, should be 0.6 rather than 0.4. (#723)
@ZvcXoyo yeah, no arcface softmax loss in david's project, but the embedding size, fixed standardization, weight decay are identical to arcface.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@ZvcXoyo I got accuracy=99.50% ~ 99.63% with m=0.35. But it seems to be a little bit overfit at end.
Parameters:
embedding_size=512
softmax_margin=0.35
softmax_scale_factor=64
weight_decay=5e-4
optimization_method : momentum, momentum=0.9
learning_rate =
1e-2 for epoch[0, 16)
1e-3 for epoch[16, 22)
1e-4 for epoch[22, 26)
1e-5 for epoch[26, 32)

Thank you for sharing！

@dragonasc2 I tried almost the same parameters, and at final stage, update the 0=0.35 as m=0.5 and continue training, but still find it is a bit overfit and generalize not well very in my test dataset,  have you tried other hyperparamters then?

I got accuracy = 99.60% ~ 99.58% with arcface, m=0.35.
validation=99.04% @ false accept rate 0.001
This is the best performance I achieved by now.
Early stopping reduces overfiting.
Parameters:
embedding_size=512
softmax_margin=0.35
softmax_scale_factor=64
weight_decay=5e-4
keep_prop=0.6
optimization_method : momentum, momentum=0.9
learning_rate =
1e-2 for epoch[0, 8)
1e-3 for epoch[8, 12)
1e-4 for epoch[12, 14)
training takes 49 hours @ 1 * GTX 1080TI
Besides, I tried keep_prop=0.4. It reaches accuracy=99.45%, validation=98.31% @ false accept rate 0.001

I got accuracy = 99.60% ~ 99.58% with arcface, m=0.35.
validation=99.04% @ false accept rate 0.001
dataset=VGGFace2
This is the best performance I achieved by now.
Early stopping reduces overfiting.
Parameters:
embedding_size=512
softmax_margin=0.35
softmax_scale_factor=64
weight_decay=5e-4
keep_prop=0.6
optimization_method : momentum, momentum=0.9
learning_rate =
1e-2 for epoch[0, 8)
1e-3 for epoch[8, 12)
1e-4 for epoch[12, 14)
training takes 49 hours @ 1 * GTX 1080TI
Besides, I tried keep_prop=0.4. It reaches accuracy=99.45%, validation=98.31% @ false accept rate 0.001

Hi, did u get the results by finetuning the pretrained softmax model or just train using arcface loss from scratch?


res.json() doesn't always return the response to the client

I am using angular to talk to express and sometimes the route doesn't return the response.
Here is what I have
Express code
router
    .route('/usage/:id/:query/:type')
        .get(function (req, res) {

            // count all records from customer by customer ID
            var sqlQuery = "select count(*) from Tracking where CustID='"+req.params.id+"'";

           // filter count by column name. Example where: Browser = Chrome.
            if (req.params.query != 'undefined')
                sqlQuery += "and "+req.params.query+" Like '%"+req.params.type+"%'";

            sdb.select(sqlQuery,function( error, result, meta ){
                    res.json(result); // sometimes doesn't return back to client? But the code always reaches here
            });
    });

Angular controller code
    .controller('CustomerUsageController', function ($scope, $rootScope, $location, Customer, $routeParams, $http) {
        function getUsage(itemName, custID, query, type){
            $http.get("/api/usage/"+custID+"/"+query+"/"+type)
            .then(function(res) {

               var usage = res.data;
                if (typeof usage[0] != 'undefined') {){
                    $scope.usage[itemName] = usage[0].Count;
                }
                else{ // database failed so try again
                    getUsage(itemName, custID, query, type); // Problem: This will call getUsage correctly  except it will never get back inside the .then function of $http.get() above.
                }
            },
            function(err) {
               // no errors
            });
        }

        getUsage("Chrome",$routeParams.id, "Browser", "Chrome");
        getUsage("Safari",$routeParams.id, "Browser", "Safari");
        getUsage("Netscape",$routeParams.id, "Browser", "Netscape");
        getUsage("Firefox",$routeParams.id, "Browser", "Firefox");
        getUsage("Windows",$routeParams.id, "OS", "Win");
        getUsage("Mac",$routeParams.id, "OS", "Mac");
        getUsage("Chromebook",$routeParams.id, "OS", "CrOS");
        getUsage("iPad",$routeParams.id, "OS", "iPad");
        getUsage("Surface",$routeParams.id, "OS", "Platform");
        getUsage("Total",$routeParams.id);
}

The bottom getUsages in CustomerUsageController code all work.  But it appears querying the database that many times this fast causes the database to fail sometimes.  The failed query(s) return back to Angular a NextToken.   My solution is to recursively call getUsage again inside the http.get().then.  This will tell Express to run the same query again.  And that works but the response is never returned back to the client.
Debugging express it does call .route('/usage/:id/:query/:type') and all the code inside will work.  But nothing will ever get passed back to angular.  Other things to note: error = null inside sdb.select(sqlQuery,function( error, result, meta ){ and result variable is correctly populated with the data I want passed back.   It just never is.

@essentialskills I think you should debug your response in the client through developer console and get the response code you receive from the server. If you receive 304 status for the browser the response is not changed and you cannot display updated data.
Just in case try to set the response status as below:
res.status(200).json(yourResponse);

In the client side i personally do not call recursively getUsage function
I hope I was helpful
Nick

@essentialskills we have two external factors in the observation - sdb and Angular. It is likely that the behavior is because of them.
It is very unlikely that it is an issue with res.json. To recognize it as an issue with it, please submit a bare-bones Express app reproducing the behavior.
You might want to join us at https://gitter.im/expressjs/express, for general discussions on Express and Express apps.

Thanks for the help guys!  Looks like Express is doing its job perfectly.  Debugging the network I can clearly see Express returned status 200 with the correct json data that I queried from the database.  I am at a loss to why angular is ignoring it still.  But at least I know Express is not the problem.

Solved!! Now that I know its something to do with the client side, I decided to call the recursive getUsage from another thread.
setTimeout(function(){getUsage(itemName, custID, query, type)}, 100);
Now it will try over and over and over until it gets the data it needs!

Glad you manage to solve it :+1:

server.get('/',(req,result)=>{
result.status(200).json(jsonObj);
result.end();
});
this is my code where jsonObj contains the json file. But it doesn't display in client side.

So we don't resurrect an old resolved issue, I am going to lock this.  @jaison10, if you have an issue you think is something broken in express, feel free to open a new issue.  If you are looking for support I would start with the Express gitter or Stack Overflow.


Touch to create a new file and press enter will open the file by OS default editor

NVIM v0.6.0-dev+1518-g02bf251bb
Build type: Release
LuaJIT 2.1.0-beta3
Compilation: clang -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNVIM_TS_HAS_SET_MATCH_LIMIT -O2 -DNDEBUG -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/tmp/neovim-20210714-39663-lqn3eb/build/config -I/tmp/neovim-20210714-39663-lqn3eb/src -I/usr/local/include -I/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include -I/usr/local/opt/gettext/include -I/tmp/neovim-20210714-39663-lqn3eb/build/src/nvim/auto -I/tmp/neovim-20210714-39663-lqn3eb/build/include
编译者<EMAIL_ADDRESS>
Features: +acl +iconv +tui
See ":help feature-compile"

     系统 vimrc 文件: "$VIM/sysinit.vim"
         $VIM 预设值: "/usr/local/Cellar/neovim/HEAD-02bf251/share/nvim"

Run :checkhealth for more info

Describe the bug

Open ranger inside neovim
:touch init.go to create a file named init.go
Press enter on init.go
OS default editor will be launched.

Expected behavior
init.go will be opened by neovim and ranger is closed.

FAQ
Q: Couldn't open some special types of files by using Enter or l in Ranger.

@kevinhwang91 Thing is, I could open a existing go file normally by pressing enter. This issue only happens for file just created by touch command.

What is r say?

change your rifle.conf


Introduce update state on CV implementation API

What this PR does / why we need it
Extend the CV implementation API to provide information, whether an Update of the descriptor has been done.
This information is then used to trigger the pubsub handler.
If no update has been done, the query of the repository's pubsub info is omitted.
Which issue(s) this PR fixes

are there tests for this behaviour?

Will this fix https://github.com/open-component-model/ocm/issues/872?

just needs a rebase and a rereview

Will this fix #872?

no @frewilhelm


GeoStyler demo fails to load GeoServer "population" demo style

Bug
Describe the bug
Trying to upload and parse the Geoserver population style (used in topp:states) makes the GeoStyler demo wait indefinitely, with the browser eventually suggesting to kill the process
To Reproduce
Steps to reproduce the behavior:

Go to https://geostyler.github.io/geostyler-demo/
Click on "Load style", "SLD style parser"
Upload the style found at https://raw.githubusercontent.com/geoserver/geoserver/main/data/release/styles/popshade.sld
The GUI never finishes loading, eventually the browser suggests killing the process

Expected behavior
It's a simple categorical style, should be loaded without issues
Desktop (please complete the following information):

OS: Linux Mint
Browser Chrome
Version 104

Tried to reproduce this issue, but while the small loadingmask keeps rotating infinetly (which is a bug), the style itself loads quickly without errors and is displayed correctly.
Chromium wanted me to save the file as a "txt", i changed that to sld. Not sure if that may be related?

Interesting, this morning it loaded instantly, it would not yesterday...
I've kept poking it with other styles, here are a few show various types of issues:

https://raw.githubusercontent.com/geoserver/geoserver/main/data/release/styles/pophatch.sld (says "loaded successfully" and then the page turns completely white, besides the "docs npm" and "imprint" bars at the bottom)
https://raw.githubusercontent.com/geoserver/geoserver/main/data/release/styles/dem.sld (loads correctly, but the UI does not give much impression that it actually happened, the rule row is all blank)
https://raw.githubusercontent.com/geoserver/geoserver/main/data/release/styles/giant_polygon.sld (loads, the symbol icon is a "broken document", fill-color is gone, fill-opacity is "NaN"... the same happens with Lakes and poly_landmarks, all from the GeoServer release data directory)

Thanks for testing and reporting. Here is what i can say about the issues:

https://raw.githubusercontent.com/geoserver/geoserver/main/data/release/styles/pophatch.sld (says "loaded successfully" and then the page turns completely white, besides the "docs npm" and "imprint" bars at the bottom)

I could reproduce it and will investigate what is going on here. The white page is typically for buggy react code and i guess we missed some try catch stuff somewhere.

https://raw.githubusercontent.com/geoserver/geoserver/main/data/release/styles/dem.sld (loads correctly, but the UI does not give much impression that it actually happened, the rule row is all blank)

The style was loaded correctly, but currently there is no preview for RasterSymbolizers. But if you click on the (empty) symbolizer of the first rule and then on "ColorMap" you will find what you where looking for. :slightly_smiling_face:

https://raw.githubusercontent.com/geoserver/geoserver/main/data/release/styles/giant_polygon.sld (loads, the symbol icon is a "broken document", fill-color is gone, fill-opacity is "NaN"... the same happens with Lakes and poly_landmarks, all from the GeoServer release data directory)

This seems to be a bug in the geostlyer-sld-parser.

This is fixed now, as the demo uses the latest geostyler-sld-parser.


Cannot Drop into the first item occasionally

Hi,
There is already an issue item in the list, but I think my situation is a little different from that one. When I drag a new item into the list, sometimes I can drag an item into the first place, sometimes I cannot. It's very wired. My example as following (modified from the GitHub source example). I tested the code on ChromeV37 and FirefoxV33.
The example can be found in the following link:
https://gist.github.com/anonymous/960c0b7b447ab89ae465
Thanks in advance!

@huskier does this issue persist with the current version? If so could you please create a live example on jsfiddle or codepen?

If this is still valid, please send a live demo with the latest version / a PR with a failing test or/and a bugfix.


Linker cannot find "boost_python-vc140-mt-x64-1_66.lib" on Windows.

This relates directly to #572.
So first it seems like irrespective of whether I use -DOPENEXR_BUILD_PYTHON_LIBS=0, the Python bindings are being built and then I get a few linker errors:
LINK : fatal error LNK1104: cannot open file 'boost_python-vc140-mt-x64-1_66.lib' [***\openexr\build\platform-windows\arch-AMD64\openexr\PyIlmBase\PyIex\iex_python2.vcxproj]
[...]
LINK : fatal error LNK1104: cannot open file 'boost_python-vc140-mt-x64-1_66.lib' [***\openexr\build\platform-windows\arch-AMD64\openexr\PyIlmBase\PyImath\imath_python2.vcxproj]

I'm not sure where it tries to get the library from but I can see it here along with the shared version and a name variation:

***\boost\1.66.0.rf1\platform-windows\arch-AMD64\python-2.7.14\boost\stage\lib\boost_python-vc140-mt-x64-1_66.lib
***\boost\1.66.0.rf1\platform-windows\arch-AMD64\python-2.7.14\boost\stage\lib\boost_python-vc140-mt-x64-1_66.dll
***\boost\1.66.0.rf1\platform-windows\arch-AMD64\python-2.7.14\boost\stage\lib\libboost_python-vc140-mt-x64-1_66.lib

Here is a doctored configuration log if that helps:
-- Selecting Windows SDK version  to target Windows 10.0.17763.
-- The C compiler identification is MSVC 19.0.24210.0
-- The CXX compiler identification is MSVC 19.0.24210.0
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configure ILMBASE Version: 2.4.0 Lib API: 24.0.0
-- Looking for pthread.h
-- Looking for pthread.h - not found
-- Found Threads: TRUE
-- Looking for include file ucontext.h
-- Looking for include file ucontext.h - not found
NOTICE-- WARNING pkg-config generation disabled
-- Configure OpenEXR Version: 2.4.0 Lib API: 24.0.0
CMake Warning (dev) at CMakeLists.txt:29 (_find_package):
  Policy CMP0074 is not set: find_package uses <PackageName>_ROOT variables.
  Run "cmake --help-policy CMP0074" for policy details.  Use the cmake_policy
  command to set the policy and suppress this warning.

  CMake variable ZLIB_ROOT is set to:

    ***\zlib\1.2.11.rf1\platform-windows\arch-AMD64\zlib

  For compatibility, CMake is ignoring the variable.
Call Stack (most recent call first):
  OpenEXR/config/OpenEXRSetup.cmake:136 (find_package)
  OpenEXR/CMakeLists.txt:30 (include)
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Found ZLIB: ***/zlib/1.2.11.rf1/platform-windows/arch-AMD64/zlib/lib/zlib.lib (found version "1.2.11")
-- Performing Test OPENEXR_IMF_HAVE_SYSCONF_NPROCESSORS_ONLN
-- Performing Test OPENEXR_IMF_HAVE_SYSCONF_NPROCESSORS_ONLN - Failed
-- Performing Test OPENEXR_IMF_HAVE_GCC_INLINE_ASM_AVX
-- Performing Test OPENEXR_IMF_HAVE_GCC_INLINE_ASM_AVX - Failed
-- Configure PyIlmBase Version: 2.4.0 Lib API: 24.0.2
CMake Warning (dev) at CMakeLists.txt:29 (_find_package):
  Policy CMP0074 is not set: find_package uses <PackageName>_ROOT variables.
  Run "cmake --help-policy CMP0074" for policy details.  Use the cmake_policy
  command to set the policy and suppress this warning.

  Environment variable Python_ROOT is set to:

    C:\Python27

  For compatibility, CMake is ignoring the variable.
Call Stack (most recent call first):
  PyIlmBase/CMakeLists.txt:34 (find_package)
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Found Python: C:/Python27/python.exe (found version "2.7.14") found components:  Interpreter Development
-- Found Python2: C:/Python27/python.exe (found version "2.7.14") found components:  Interpreter Development
-- Could NOT find Python3: Found unsuitable major version "2", but required major version is exact version "3"
NOTICE: Found Python 2.7.14
-- Found Python2 libraries: 27
-- Boost version: 1.66.0
-- Found the following Boost libraries:
--   python
CMake Warning at PyIlmBase/CMakeLists.txt:104 (message):
  Ambiguous boost python module found, assuming python 2

-- Found NumPy 1.13.3 for Python 2.7.14: ***/numpy/1.13.3/numpy/Lib/site-packages/numpy/core/include
-- Configure OpenEXR Viewers Version: 2.4.0 Lib API: 24.0.1
-- Found OpenGL: opengl32
-- Could NOT find FLTK (missing: FLTK_LIBRARIES FLTK_INCLUDE_DIR)
CMake Warning at OpenEXR_Viewers/CMakeLists.txt:36 (message):
  FLTK not found, exrdisplay will not be built

NOTICE: Disabling playexr, not yet ported to WIN32 platforms
-- Configuring done
-- Generating done
-- Build files have been written to: ***/openexr/build/platform-windows/arch-AMD64/openexr

Random related thoughts, what about using pybind11? :)

oh, it's happening (pybind11), but we are going to do a bunch of c++11
cleanups as well, and so wanted a stable release before we potentially
break the ABI
Kimball
On Fri, Sep 27, 2019 at 11:05 AM Thomas Mansencal<EMAIL_ADDRESS>wrote:

Random related thoughts, what about using pybind11? :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openexr/openexr/issues/573?email_source=notifications&email_token=AAHM3IH75CKJSOEMD4IAEY3QLU53FA5CNFSM4I27Q54KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7XGQAA#issuecomment-535717888,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHM3IFRQWZGEYDSS7HEWP3QLU53FANCNFSM4I27Q54A
.

cmake, boost, and python are having a contest to thwart each other's naming conventions recently. Have you tried setting, from the command line
-DBOOST_ROOT=c:/blah/.../blah/boost/1.66.0.rf1/platform-windows/arch-AMD64/python-2.7.14/boost/stage

and if that isn't good enough, setting instead the boost trifecta of BOOST_DIR, BOOST_LIBRARYDIR, and BOOST_INCLUDE_DIR should work or it might be BOOST_INCLUDEDIR, depending on alternate Thursdays. The FindBoost.cmake script is frightening.

Hi @meshula / Nick,
I don't think I changed our DBOOST_ROOT value at all, my assumption was that if it works for OIIO it should work for OpenEXR but maybe it is an assumption I should not make :] I will try!
Cheers,
Thomas

FindBoost.cmake is complex and savage to tame. Let us know how your experiment goes.

Checking back on this - any update? Given that the longer-term direction of the project is to move to pybind11, the technical steering group has limited bandwidth to invest in the current approach. If you or someone else from the community would possibly investigate further and propose a patch, we'd be happy to review it.

The issue seems to be that Boost was forgotten in PyImx/CMakeLists.txt and PyImath/CMakeLists.txt when PyIlmBase was refactored here https://github.com/openexr/openexr/commit/4d97270c6ce0916483c1aff5b1f77846cfff11a0
PyImathNumpy was however updated and that builds nicely.

On further inspection, here's (https://github.com/openexr/openexr/commit/0eff97241f495027021b54978028475f0b2459dd) where all code got removed from PyImx/CMakeLists.txt and PyImath/CMakeLists.txt.
Commit message doesn't say why...

Ok upon further investigation I think I've found the real error.
PyIex and PyImath is being built twice. One is a .dll and the other is a .pyd Python module.
Only the .dll has Boost added. The python module is missing it.
I'll create a PR asap.


Fix for NPCs stop attacking after being poisoned NPC (Issue #487) and Fixweight not working properly (Issue #502).

-Fixed: NPCs stop attacking after being poisoned NPC (Issue #487).
The bug happens when a NPC get damaged while the player is not reachable and it was caused by a missing assignement in the check in method CCharFight::OnTakeDamage
 if (bAttackerExists == false)
  {
   ....
   attacker.ignore = false; //Attacking.ignore was not set causing it to be set at 246
   m_lastAttackers.emplace_back(std::move(attacker));
  }

When the player goes away, the player is properly removed from the NPC's attacker list,  but if in the meantime the NPC suffers damage from the player by a command/item that
calls directly OnTakeDamage the NPC will add back the player but without setting the attacker.ignore property properly (It was set to 246 in my case).
-Fixed: Fixweight not working properly (Issue #502).
The problem was caused by the  method call:
 m_totalweight += pCont->GetWeight();
...
int CItemContainer::GetWeight(word amount) const
{ // true weight == container item + contents.
 return( CItem::GetWeight(amount) + CContainer::GetTotalWeight());
}

CContainer::GetTotalWeight would return m_totalweight of the current container but at the top of the Fixweight method we set m_totalweight value to 0 with the result
that Fixweight returned only the weight of the single item containers.
Minor problem i found:
When a player login, it seems that the Bank box weight it's added to is own weight, so a naked player should only weights 3 stones (backpack) but instead it weights 6 stones (3 backpack + 3 bank box)
I can't manage to track it.
@drk84
Forgot to remove the //

I think it's the issue https://github.com/Sphereserver/Source-X/issues/485 #485 Not 487


My first branch

you learned to create a branch, created md file and made a commit.

merged first branch


Early use cases

Let's focus on setting up three HPC scale use cases

single large simulation, e.g. G2D on cortical surface 515k vertices
parameter sweep, RWW+BOLD on HCP 100 connectomes
parameter inference, Epileptor w/ simulated sEEG data or MEG?

Case 1 runs into parallelization difficulties in the history evaluation, probably requiring all the strategies listed on the relevant part of the wiki.
While implementing HMC or ADVI may take too much time for this milestone, 2 & 3 both have for computational core parallel stepping full simulation.  We can focus on this as a goal; i.e. how to do one full step of a whole simulation, applying in parallel over parameters (case 2) or time (case 3). In case 2, step on state in place, while in case 3, difference between expected and computed next states is relevant (and we'd be computing partial derivatives).

It's still worth considering focusing on case (3) even if it requires simplifying other aspects: parameter inference w/ ADVI could be tractable & performant, and not too tricky. If we assume Gaussian priors (so that we can avoid transformations), focus on computing log p & gradient, main change to code is adding

partial derivative calculations -> gradient
full state variable trace ([-max delay, T]) is stored

Is it worth testing different models? ADVI is fairly involved, but we could test with just an optimization first: hill-climb log p using gradient.

Nearly done with basic code generation in b118134, with a little more work, should be possible to run simulations.

This issue was high level and #19 #20 cover the remaining work.


Ammo.js triggers do not work

Ammo.js triggers stopped registering entities after an upgrading engine from 0.0.18 to 0.0.20

Probably broken by this commit

Fixed in https://github.com/AndyGura/gg-web-engine/releases/tag/0.0.24


feat: send emails on success/failure

This adds the capability to send emails to the releaser. Notifications that are actually helpful will go into another PR.

Updated this to use any SMTP service rather than assuming Gmail/Oauth.


Hide windows-sys's documentation behind a feature flag?

Suggestion
Hi! I've been building a program that uses the bevy game engine crate, and in the midst of this, I've found it useful to render documentation for everything via cargo doc.
However, I noticed that the windows-sys crate generates quite a lot of documentation. Unfortunately, cargo doc has no way to exclude certain dependencies without excluding all dependencies (the --exclude flag only seems to work on packages that are within the current workspace).
I don't believe this is an issue with the windows-sys crate, per se - it's doing its job just fine. However, unless a user is using windows-sys directly, they may not need to see windows-sys's documentation anyways - meaning windows-sys's documentation is simply wasting disk space, and worse, making cargo clean take a lot longer due to the mass amount of files generated.
Still, some users may need windows-sys's docs, so hiding them behind a blanket #[doc(hidden)] may not be a good idea. Perhaps windows-sys could attach #[doc(hidden)] to most stuff, unless a certain feature flag (let's say windows-sys/render_docs) is specified?
I just think it may be useful to avoid generating so much documentation unless a user needs it.

So you'd like to generate docs for your crate and all dependencies except windows-sys, right? (Just making sure I understand, thanks!)

Yes. windows-sys generates a lot of documentation, and because I'm using the bevy game engine crate, I have no need to look at the windows-sys documentation ever. So I'd like a way to avoid generating windows-sys documentation.
There is https://github.com/rust-lang/cargo/issues/4049, however that was opened in 2017 with no plans to add it yet. Thus, as a stopgap, I'm hoping windows-sys can provide a method to skip generating so much documentation.

Hey, sorry for the delay. Been off the grid camping. 🏕
Just catching up here but this is what I do. Is that sufficient?
cargo doc -p package --all-features --no-deps

@kennykerr It could work, but given the large number of bevy_* crates that I'd actually want to look at in the resulting documentation, plus the fact that I have other dependencies Id want the documentation for, specifying each wanted package isn't really something I'd be keen on doing.
I'd much rather just ask for a way to exclude certain crates (i.e. windows-sys) than need to include every crate I want as a command line parameter.

Fair enough. I've been meaning to reduce the overhead of doc comments for other reasons, so I'll keep this in mind for that effort.

Mainly just for me: if we add #![doc(hidden)] to the crate's lib.rs file then all docs are effectively removed, rather than having to add the attribute to each and every item individually.

@kennykerr If it isn't too much effort, you may still want to leave a note on the crate root so as to avoid confusion. For example, "Documentation is disabled. To view documentation, pass --features "windows-sys/render_docs" to cargo." or similar. That way, folks aren't confused when the documentation suddenly isn't visible.

Not a fan of tying docs to a feature. It's not part of the standard cargo doc experience and folks will overlook it causing confusion. We should really try to lean on https://github.com/rust-lang/cargo/issues/4049 more.

@riverar I agree - but the reason I suggested a feature anyways is solely for the interim. Plus, given that it's been over six years since that issue was opened, and there doesn't appear to be many comments, I'm not exactly hopeful that it'll be added soon.

Sure, I wouldn't exclude docs automatically. If provided, that would be something that you would have to opt in to.

There are also cfg options.
https://github.com/microsoft/windows-rs/blob/master/.cargo/config.toml

Features are meant to be additive so that doesn't seem like a natural fit. I experimented with a cfg option but rustdoc seems to fail to handle cfg_attr correctly.
// lib.rs
#![cfg_attr(windows_doc_hidden, doc(hidden))]

This would seem like the simplest and most natural way to skip doc generation for the crate, if it worked.

Features are meant to be additive so that doesn't seem like a natural fit.

I'm confused. With what I suggested, using the feature adds the documentation - and not using the feature removes it. Thus, it seems like a windows-sys/render_docs feature would be additive.
Could you perhaps elaborate?

I was assuming the docs feature would need to be enabled by default, but I don't suppose that is necessary.

Ah, okay that makes sense.
Yeah, my thought is that the vast majority of things that will rely on windows-sys, will rely on it indirectly - usually due to a crate somewhere in their dependency tree, which likely abstracts away stuff from windows-sys.
Thus, the vast majority of Rust crates will have little need for the windows-sys docs during development. This means that documentation could be disabled by default - only being enabled when someone actually needs the documentation, using a feature flag passed to cargo doc or cargo-watch.


Created task_04.txt

Issue: #5

Hey @23abdul23
Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm.
Thanks for contributing in OpenCode'24 ✨✨!

Hey @23abdul23
Your PR has been merged 🥳🥳 and you have earned 10 points.
Thanks for contributing in OpenCode'24 ✨✨


Update TLS cert

Hi!
Your TLS certificate is invalid :/
/vanten

sorry for the late response!
currently this web has gone unmaintained for a while, I have to ask my web admin to update on that since I can't access the server panel for a while. you might need to wait for a few days until i can resolve the problem
anyways, thanks for noticing! :) I really appreciate it.

Resolved


adding estimated-usage.yml

Description
Closes https://github.com/terraform-ibm-modules/terraform-ibm-observability-da/issues/78
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

:tada: This PR is included in version 1.7.0 :tada:
The release is available on:

GitHub release
v1.7.0

Your semantic-release bot :package::rocket:


feat: Update appVersion to 0.4.1

Summary of changes:
Update appVersion in Chart.yaml to 0.4.1
Checklist:

[ x ] I updated the artifacthub.io/changes annotation in Chart.yml according to the documentation
[ ] Optional: I updated in README.md the Helm Values

Thansl @0xThresh


[Event Request] Report 5753 "Get Source Documents" – Existing Event “OnPurchaseLineOnAfterGetRecordOnBeforeCreateShptHeader”

Could we please have the existing publisher “OnPurchaseLineOnAfterGetRecordOnBeforeCreateShptHeader” in trigger “OnAfterGetRecord” (DataItem “Purchase Line”), Report "Get Source Documents”, be expanded by an additional parameter, including a handle pattern?
            LineCreated := true;
        end;
    RequestType::Ship:
        if WhseActivityCreate.CheckIfFromPurchLine2ShptLine("Purchase Line") then begin
*** start old code ***
            //OnPurchaseLineOnAfterGetRecordOnBeforeCreateShptHeader(
            //  "Purchase Line", "Warehouse Request", WhseShptHeader, WhseHeaderCreated, OneHeaderCreated);
*** stop old code ***
*** start new code ***
            IsHandled := false;
            OnPurchaseLineOnAfterGetRecordOnBeforeCreateShptHeader(
              "Purchase Line", "Warehouse Request", WhseShptHeader, WhseHeaderCreated, OneHeaderCreated, IsHandled);
            if not (IsHandled) then begin
*** stop new code ***
                if not OneHeaderCreated and not WhseHeaderCreated then begin
                    CreateShptHeader;
                    OnPurchaseLineOnAfterCreateShptHeader(WhseShptHeader, WhseHeaderCreated, "Purchase Header", "Purchase Line", "Warehouse Request");
                end;
                if not WhseActivityCreate.FromPurchLine2ShptLine(WhseShptHeader, "Purchase Line") then
                    ErrorOccured := true;
                LineCreated := true;
*** start new code ***
            end;
*** stop new code ***
        end;
…
    [IntegrationEvent(false, false)]
*** start old code ***
    //local procedure OnPurchaseLineOnAfterGetRecordOnBeforeCreateShptHeader(PurchaseLine: Record "Purchase Line"; var
    //WarehouseRequest: Record "Warehouse Request"; var WarehouseShipmentHeader: Record "Warehouse Shipment
    //Header"; var WhseHeaderCreated: Boolean; var OneHeaderCreated: Boolean)
*** stop new code ***
*** start new code ***
    local procedure OnPurchaseLineOnAfterGetRecordOnBeforeCreateShptHeader(PurchaseLine: Record "Purchase Line"; var WarehouseRequest: Record "Warehouse Request"; var WarehouseShipmentHeader: Record "Warehouse Shipment Header"; var WhseHeaderCreated: Boolean; var OneHeaderCreated: Boolean; var IsHandled: Boolean)
*** stop new code ***
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


feat(nic2008): add new standard and concordance

Description
Adding National Industrial Classification from India and concordance to ISIC4.
Types of changes

What types of changes does your code introduce:

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Further comments

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

Project coverage is 100.00%. Comparing base (f0d34fe) to head (c32b969).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #84   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           83        86    +3
  Lines          330       341   +11
=========================================
+ Hits           330       341   +11

Files with missing lines
Coverage Δ

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

pyisic/_standards/nic2008/__init__.py
100.00% <100.00%> (ø)

pyisic/_standards/nic2008/nic2008.py
100.00% <ø> (ø)

pyisic/_standards/nic2008/nic2008_to_isic4.py
100.00% <100.00%> (ø)

pyisic/types.py
100.00% <ø> (ø)


Add little more river and lake and some sugar cane

Adding, Removing, or Changing
Adding
Type of Modification
Biome / Region
What are you trying to modify
add little more river
add lake (size small and large)
add some sugar cane around water
Minecraft Version
1.17.1
Iris Version
1.9
Checklist

[X] I am using an unmodified version of the overworld. (If you modified the overworld but know the content suggestion is valid)
[X] I am using the latest version of the overworld OR I have made sure this feature was not already implemented in the latest version.

There should already be sugarcane by waterbeds, and if not ill look into reimplementing them / upping the rarity


Deploy fix for scoreboard length

This adds a way bigger max size to the scoreboard to allow the placeholders to show up if they're too big, here is how it looks like on my server (ignore the transparent scoreboard, that's not included in this PR)

Cool PR. Should add the feature to the "Implements section" of the readme.


Modifier Bucket not consistently working after installing 0.14.0 and Nordlond Bestiary (or 0.14.1 and NB <IP_ADDRESS>)

It's very weird. Sometimes it works, sometimes not. It might be an issue with Token Action HUD--I didn't try that much with using skills right from the sheet, but it definitely wasn't working with Token Action HUD and the Berserker from the Berserker's sheet or the drop down from Token Action HUD.

When you say "didn't work" what do you mean? It doesn't display? Or the modifiers don't affect the dice roll? Or modifiers can't be added to the bucket?

Just tried it with Aim for a weapon via Token HUB for one of the Bestiary characters and it didn't add to the mod bucket.  Also tried from the Nordlond character sheet, and also didn't add to the bucket.  Just tried opening the mod bucket and clicking on any modifier does nothing.
Tried some other tokens and actors, same thing.  Basically, seems like the mod bucket doesn't show anything.
However, when I think click to roll something, all the mods work (whether clicked from the Token HUD, Nordlond sheet, GCS sheet, or via opening the mod bucket itself.

Question: are you self hosting or using Forge? I saw some weirdness with Forge during development.

Forge
On Sun, May 22, 2022 at 8:14 AM Nick Coffin, PI @.***>
wrote:

Question: are you self hosting or using Forge? I saw some weirdness with
Forge during development.
—
Reply to this email directly, view it on GitHub
https://github.com/crnormand/gurps/issues/1423#issuecomment-1133882934,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALDN5UWJCRIZP23FIF335BTVLIQMBANCNFSM5WSMMMRQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Dictated and sent from my iPad

When you say "didn't work" what do you mean? It doesn't display? Or the modifiers don't affect the dice roll? Or modifiers can't be added to the bucket?

Mine is scattered: Sometimes it won’t display in the bucket. Other times it displays in the bucket but doesn’t apply it to the roll. Let me see if I can get info today with the console.

Just clicked on -2 deceptive attack twice, modifier bucket still shows +0. It did apply it though.
And then I applied Shocked -4 to DX/IQ checks and it does apply to the bucket, but when I rolled DX for that character, it did not apply it and the bucket still shows -4. No errors on the console.

Not sure if this helps, but this is what I see when I click on the modifier bucket and click -2 Deceptive Attack once. The modifier bucket does not update, still shows +0.

Confirming also that without the Bestiary, there's no problem.

Updated: Still having an issue with GGA 0.14.1 and NB <IP_ADDRESS>.
Here's a test sequence: With NB inactive, I take the :"Orc Warrior" token and roll an attack, no problem. Using Token Action HUD for these. Then I apply "-2 to Hit (Deceptive Attack)," it shows up in the MB, roll the attack, no problem, MB clears. Then I apply shock -4, it shows up in the MB, I roll DX, MB clears, no problem.
Next, I take an Acolyte from NB in my actors directory but I do not have NB loaded..

I do the same thing with the Acolyte. No problems,. Using Token Action HUD to make the rolls.

I delete those two from the map. I go to manage modules, click on NB <IP_ADDRESS>. System logs me back in. I navigate back to the test map. I bring the Orc Warrior token on to the scene. I repeat the sequence. After the first attack, I go to insert "-2 to hit (Deceptive Attack)." It does not show up in the MB. But I roll and it is applied. I try the shock -4, it does show up in the MB, I roll DX, it does not apply the modifier and -4 still shows in the MB.

I bring the Acolyte on to the scene, repeat, and get the same exact behavior.

Some interesting console messages:

I delete the actors and try disabling Token Action HUD for good measure, rolling from the sheet instead. Same behavior.

Console:

I can give you some idea of what is probably going on, although I don't know how to solve it yet. Hopefully the instructions above allow me to recreate it so that when I have a proposed solution, I can verify it fixes the problem without you guys being my guinea pigs.
@TheFabulousIronChef , @Farmeroz -- confirm that the problem appears in Forge, or does it appear when self-hosting?

I have not tried it self hosting yet, but I will see if I can give it a
shot either tonight or sometime tomorrow.
On Sat, May 28, 2022 at 12:53 PM Nick Coffin, PI @.***>
wrote:

I can give you some idea of what is probably going on, although I don't
know how to solve it yet. Hopefully the instructions above allow me to
recreate it so that when I have a proposed solution, I can verify it fixes
the problem without you guys being my guinea pigs.
@TheFabulousIronChef https://github.com/TheFabulousIronChef , @Farmeroz
https://github.com/Farmeroz -- confirm that the problem appears in
Forge, or does it appear when self-hosting?
—
Reply to this email directly, view it on GitHub
https://github.com/crnormand/gurps/issues/1423#issuecomment-1140296435,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALDN5UXJIDRGWYAP43VDUSDVMJFQJANCNFSM5WSMMMRQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Dictated and sent from my iPad

Can confirm: It must be a Forge thing. I am not having this problem locally
hosted. Or at least, I could not replicate it. I think I have all the same
mods running.
On Sat, May 28, 2022 at 2:51 PM Vic LaPira @.***> wrote:

I have not tried it self hosting yet, but I will see if I can give it a
shot either tonight or sometime tomorrow.
On Sat, May 28, 2022 at 12:53 PM Nick Coffin, PI @.***>
wrote:

I can give you some idea of what is probably going on, although I don't
know how to solve it yet. Hopefully the instructions above allow me to
recreate it so that when I have a proposed solution, I can verify it fixes
the problem without you guys being my guinea pigs.
@TheFabulousIronChef https://github.com/TheFabulousIronChef , @Farmeroz
https://github.com/Farmeroz -- confirm that the problem appears in
Forge, or does it appear when self-hosting?
—
Reply to this email directly, view it on GitHub
https://github.com/crnormand/gurps/issues/1423#issuecomment-1140296435,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALDN5UXJIDRGWYAP43VDUSDVMJFQJANCNFSM5WSMMMRQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Dictated and sent from my iPad


.atom folder support

Please add a check for the .atom-build.x file in the .atom directory
All Editors/IDEs have a folder, .idea, .eclipse, .vscode, and .atom
All plugins should have a secondary check in the .atom folder for their files that are normally in root.
it's just nice to have one directory to gitignore rather than 20 files.

The package is dead, closing.


edits to rst_files/testing.rst

I haven't completely finished this but I wasn't sure what we are doing with indentation of lists? There was a lot of text under dotpoints that wasn't indented? Should we be indenting this?

Could we also discuss why we are focusing on unit tests specifically? And not the other types of testing?

Thanks @natashawatkins, all great comments. Will iterate on them and get back to you.

I think it would be really useful to have someone (new to Julia) actually run through the commands and everything carefully. I don't think I have the time but we might have an RA do that soon.
Also again I thought the screenshots were a little hard to read/see what's going on.

@natashawatkins I acted on your feedback in the last set of commits. Will merge this now, and you can open a new PR from this branch (or a new one) whenever you're ready.

Thanks @arnavs !


Line IRQ set for line 0 interrupts twice

It appears that around the rollover point from line 511 to line 512, there is an opportunity for the IRQ signal to glitch and generate an interrupt in the middle of the back porch, ahead of line 0.

As seen in the above sequence of images, the unexpected line IRQ happens at roughly line 480 + 10 + 2 + 20 = 512. This erroneous IRQ appears to happen during every frame while a line IRQ is set for line 0.
It isn't obvious to me why this is happening. y_counter and line_irq are both registers in composer.v; they should be safe. Needs further investigation.

After more debug, this happens on every line 0-10, possibly 11 and 12 also. It appears to be due to the y counter in the composer being 9 bits instead of the 10 needed to count all 525 lines of the frame. Fixing this line in composer.v might solve the issue:
    reg  [8:0] y_counter_r, y_counter_rr;


add comment check result

add some comment

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
no

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

add comment for easy to understand code

It's obvious from method names.


Error: "iap is not defined"

When i visit some extensions the app shows a popup with this error message

(1.8.6 from fdroid)

That's because that's not available in your country

That's because that's not available in your country

Then, mind of adding an warning instead of showing error message, will ya?

Konuyu kapatıyor birde :D


[Snyk] Security upgrade lerna from 7.1.4 to 8.0.1

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

build_scripts/npm_windows/package.json
build_scripts/npm_windows/package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Server-Side Request Forgery (SSRF) SNYK-JS-IP-7148531
  718

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Server-Side Request Forgery (SSRF)

sny


Append data set api call works now, added a button to run api tests against localhost

@stowellm

Good to pull :hurtrealbad:


Server side rendered open modal

Issue description

components: Modal
reactstrap version 5.0.0, but the problematic code still exists in latest version
import method: es with Webpack
react version 16.2.0
bootstrap version 4.0.0

What is happening?
When an open modal is server side rendered, then it is failing with "document is not defined". The exception is thrown in Modal's init method.
What should be happening?
Render as it should. Probably move checking of isOpen, and calling of init to componentDidMount from constructor.
Steps to reproduce issue

Create a modal with isOpen true.
Try render it with React's renderToString.

Error message in console
ReferenceError: document is not defined
    at Modal.init (...)
    at new Modal (...)
    at resolve (...)
    at ReactDOMServerRenderer.render (...)
    at ReactDOMServerRenderer.read (...)
    at renderToString (...)

Code
import {renderToString} from "react-dom/server";

renderToString(
    <Modal isOpen>
        <ModalBody>Lorem ipsum</ModalBody>
    </Modal>);

Same issue here! It would be great to have a possibility of opened modal on load/init.

Just need feedback ... everyone okay with it opening immediately via componentDidMount ???
Only downside is that the true HTML sent down from server wouldn't match the opened HTML (not really a downside just if someone cared 100% about showing the modal content being open/in correct DOM location in source -- who needs that/cares?)
It would not be noticeable and look better as when loading the page it would slide the modal down instead of already be open (no transition).

@TheSharpieOne That library is one of many techniques as mentioned in my above comment. Essentially, those libraries are JavaScript backend specific which wouldn't work for our case which is a library (we shouldn't provide/expect a specific backend).
Backends = NodeJs via Express, PHP via V8JS, etc.
We would have to mimic that functionality and provide some escape hatches to render the Modal portals separately  (which we could do -- my thought is that most people that are running SSR are less likely simply executing static code and more likely hydrating on the client and thus , the simplest solution would be to perform in componentDidMount()).
Otherwise, we would have to store all the portals specifically rendered via Reactstrap and then provide a separate hook to retrieve those ... such that the backend provider could inject as necessary.

Yeah, that makes sense. Lets go with the lighter hydrating method

Sounds good. Will reference this issue with PR (hopefully this weekend)

@virgofx Is there any update on this?

Sorry not yet ... been super busy should only be a quick fix to migrate the opening portion to trigger client side ... give me another week should have it done unless you want to submit a PR ... always welcome as well.

any update?

resolved at https://github.com/reactstrap/reactstrap/pull/1495

This problem is still here (reactstrap 8.0.0) but the fix is quite simple :
<Modal isOpen={typeof(document) !== 'undefined' ? true : false}>
</Modal>

And the error with isOpen set to true is :
ReferenceError: document is not defined
    at Modal.init (/home/user/project/node_modules/reactstrap/lib/Modal.js:272:23)

hi @boutdemousse, version 8.0.0 is not included PR's fix this problem. Should you update to version 8.0.1 just released by @TheSharpieOne  at https://github.com/reactstrap/reactstrap/commit/ce2a9a080dd3ac655900865acb98c88124b267ac

@TheSharpieOne should close this issue?

Using Next.js, this issue crashes the whole app when loading a Tooltip that has isOpen={true} on the server.


Drab dislike HTML comments when they appears inside a do..end eex block in a child file

Sorry, I'm here again with something that appears as a little bug. It seems that in certain conditions Drab has problem in compiling .drab files when it encounter an HTML comment ( i.e.  ) inside a do .. end block, exiting the compiler with the error:
    The following arguments were given to Drab.Live.EExEngine.ampered_tag?/1:

        # 1
        {:comment, "FOO"}

The following is an example where I was able to reproduce the problem I encountered in a much more complex code:
# templates/page/home.html.eex

<%= render( MyApp.PageView, "_drab_test_child.html", data: [1,2,3]) %>

# templates/page/_drab_test_child.html
<%= if @data do %>

<% end %>

15:25:40.345 [info]  Compiling Drab partial: lib/my_app_web/templates/page/_drab_test_child.html.drab (geydgnrsheydqmbu)

== Compilation error in file lib/my_app_web/views/page_view.ex ==
** (FunctionClauseError) no function clause matching in Drab.Live.EExEngine.ampered_tag?/1

    The following arguments were given to Drab.Live.EExEngine.ampered_tag?/1:

        # 1
        {:comment, "FOO"}

    Attempted function clauses (showing 2 out of 2):

        defp ampered_tag?({_, attributes, _})
        defp ampered_tag?(string) when is_binary(string)

    lib/drab/live/eex_engine.ex:347: Drab.Live.EExEngine.ampered_tag?/1
    (elixir) lib/enum.ex:2955: Enum.reject_list/2
    (elixir) lib/enum.ex:2958: Enum.reject_list/2
    lib/drab/live/eex_engine.ex:322: Drab.Live.EExEngine.assigns_and_parents_from_pattern/1
    lib/drab/live/eex_engine.ex:201: anonymous fn/2 in Drab.Live.EExEngine.handle_body/1
    (elixir) lib/enum.ex:1899: Enum."-reduce/3-lists^foldl/2-0-"/3
    lib/drab/live/eex_engine.ex:195: anonymous fn/3 in Drab.Live.EExEngine.handle_body/1
    (stdlib) lists.erl:1263: :lists.foldl/3
$

Instead, if the HTML comment is outside the do .. end block:
# templates/page/_drab_test_child.html
<%= if @data do %>
<% end %>

it compiles without any error.
--EDIT--
It seems that the problem arise when the HTML comment is the first thing inside the do..end  block, in fact this works:
# templates/page/_drab_test_child.html
<%= if @data do %>

<% end %>

Thanks a lot for reporting! This is a bug in a custom eex engine


Add options to gather stdout and stderr from browser

Attempt at adding feature in #347.  Exposes both stdout or stderr through an io.Writer. Should not affect users who do not use the added ExecAllocatorOptions Stdout() and Stderr().

Implemented all the requested changes, with the exception of combining stdout and stderr.

May I suggest that someone definitively tests whether or not stdout and stderr are both used by Chrome before making a high-level API change?

We know it prints most things to stderr. I have an existence proof of the vanilla version of Chromium printing a line ( Using PPAPI flash. ) to stdout. This page also seems to confirm that it uses both.

PPAPI and NaCL have been long long long deprecated.

Absolutely. Just saying that, if the question is "Does regular Chrome/Chromium print stuff to both stdout and stderr?", the answer is yes. Clearly stdout is not used a lot, and as I admitted in #348, I don't think this feature will be widely used, but I need it for my use case, and I could see some other people wanting to grab at least stderr as well.
If you think it's better to combine them into a single output stream, I'm happy to do that - just let me know.

If you think it's better to combine them into a single output stream, I'm happy to do that - just let me know.

I think that's a good first step. In practically all cases, we'll only have stderr, so users won't be affected. And if anyone wants to capture something extra from stdout, they can also do that.

Alright, made that change. Let me know your thoughts.

Thanks for the helpful feedback. Here's another possible solution using a TeeReader so that we gather all output, including the websocket address. Added a test case as well.

I'll squash-merge, since there are many commits in this PR.

@pmurley I've pushed a few commits to master. If you have any thoughts, feel free to send another PR.


Bug when changing output and input prompts in the IPython console

From<EMAIL_ADDRESS>on 2014-08-02T16:30:34Z
Spyder Version:  2.3.0rc
Python Version:  3.4.1
Qt Version    :  4.8.6, PyQt4 (API v2) 4.10.4 on Windows
pyflakes >=0.6.0:  0.8.1 (OK)
pep8 >=0.6      :  None (NOK)
IPython >=0.13  :  2.1.0 (OK)
pygments >=1.6  :  1.6 (OK)
sphinx >=0.6.6  :  1.2.2 (OK)
psutil >=0.3    :  2.1.1 (OK)
rope >=0.9.2    :  0.9.4-1 (OK)
matplotlib >=1.0:  1.3.1 (OK)
sympy >=0.7.0   :  0.7.5 (OK)
pylint >=0.25   :  None (NOK)
What steps will reproduce the problem?

go to preferences
go to console preference
change input and output display
I type:
>>> for input
and
<<< for output

Restart a new console (or spyder)
>>> appears now instead of in[1]
type a number after >>>
EX:
>>>0.1
A error appears in internal console:
Inspect Spyder internals:
spy.app, spy.window, dir(spy)
>>> Traceback (most recent call last):
File "C:\cpge\Anaconda3\lib\site-packages\IPython\qt\base_frontend_mixin.py", line 138, in _dispatch
handler(msg)
File "C:\cpge\Anaconda3\lib\site-packages\IPython\qt\console\rich_ipython_widget.py", line 138, in _handle_pyout
return super(RichIPythonWidget, self)._handle_pyout(msg)
File "C:\cpge\Anaconda3\lib\site-packages\IPython\qt\console\ipython_widget.py", line 237, in _handle_pyout
self._append_html(self._make_out_prompt(prompt_number), True)
File "C:\cpge\Anaconda3\lib\site-packages\IPython\qt\console\ipython_widget.py", line 544, in _make_out_prompt
body = self.out_prompt % number
TypeError: not all arguments converted during string formatting
What is the expected output? What do you see instead?
the output expected was just the typed number....
Instead, nothing in python console.
Please provide any additional information below
. Thank you
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1923

From ccordoba12 on 2014-08-17T10:04:37Z
I confirm the bug. Thanks for reporting.
Summary: Bug when changing output and input prompts in the IPython console  <span class="oldvalue"> (was: console bug when change output (and input display)) </span>
Status: Accepted
Labels: Cat-IPythonConsole MS-v2.3.2

From ccordoba12 on 2014-08-17T10:18:00Z
This is a bug in IPython itself. I'll send them a PR to fix it, so you'll have to wait until they release a new version.
Labels: -MS-v2.3.2 MS-v2.4

From ccordoba12 on 2014-09-04T15:22:20Z
My pull request was merged in IPython master, so it'll be released with its 3.0 version (in a couple of months).
Status: Fixed

From ccordoba12 on 2014-08-17T13:23:43Z
This is the corresponding IPython pull request https://github.com/ipython/ipython/pull/6328


Vodafone- Drobeta-Turnu Severin is not recognized as a city in Romania

ENVIROMENT: production
URL: https://scrapers.peviitor.ro/src/vodafone/index.html
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open URL in browser
Click on 'Run Scraper' button
Check location for 'Sales Advisor' where Drobet-Turnu Severin is not recognized as a city in Romania

EXPECTED RESULTS:
Drobeta-Turnu Severin should be recognized as a city in Romania
ACTUAL RESULTS:
The Scraper does not recognize 'Drobeta-Turnu Severin' as a city in Romania and displays 'Drobeta-Turnu Severin is not a city in Romania'

Fixed

After retesting Drobeta-Turnu Severin is recognized as a city in Romania


(changes made) - added python book

What does this PR do?
Add resource(s)
For resources
Python 3 Patterns, Recipes and Idioms
Why is this valuable (or not)?
It is a good book to learn python
How do we know it's really free?
this book is published under a Creative Commons Attribution-Share Alike 3.0 license.
For book lists, is it a book? For course lists, is it a course? etc.
book
Checklist:

[ ] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Sentry django integration not catching all 500

In continuation of this thread:
There were a couple of exceptions in DRF serializers which were not caught by Sentry. It looks like a known issue.

How to resolve the issue quickly? Write my own middleware? Any examples?
How to resolve the issue in the long term, so other users won't hit it?

@untitaker, could you take a look please, as I'm following your advice and posting it as an issue. Thanks!

I'm suspecting I have the same problem and if it's true, then it's extremely critical that we fix this.
    sentry_sdk.init(
        dsn=os.environ.get("SENTRY_DSN"),
        integrations=[DjangoIntegration(), CeleryIntegration(), RedisIntegration()],
        send_default_pii=True,
    )

We recently switched from Rollbar to Sentry, so this is kind of disappointing. :(

So there are multiple issues here:

In the original thread somebody said their errors from requests made by Slack are not showing up. Please go to your project settings and disable Inbound Filters, specifically the web crawlers filters.

The SDK may have crashed internally. Please run init() with debug=True as keyword argument, maybe this starts printing out errors.

My issue was resolved. Our sentry was a bit misconfigured and it was crashing in post processing when getting exceptions. Feel free to close the issue.

ok @denizdogan please open a new issue with more information


Activating Open Collective

Hi, I'm making updates for Open Collective. Either you or a supporter signed this repo up for Open Collective. This pull request adds backers and sponsors from your Open Collective https://opencollective.com/Parsley.js ❤
It adds two badges at the top to show the latest number of backers and sponsors. It also adds placeholders so that the avatar/logo of new backers/sponsors can automatically be shown without having to update your README.md. [more info]. See how it looks on this repo.
We have also added a postinstall script to let people know after npm|yarn install that you are welcoming donations (optional). [More info]
You can also add a "Donate" button to your website and automatically show your backers and sponsors there with our widgets. Have a look here: https://opencollective.com/widgets
P.S: As with any pull request, feel free to comment or suggest changes. The only thing "required" are the placeholders on the README because we believe it's important to acknowledge the people in your community that are contributing (financially or with code!).
Thank you for your great contribution to the open source community. You are awesome! 🙌
And welcome to the open collective community! 😊
Come chat with us in the #opensource channel on https://slack.opencollective.com - great place to ask questions and share best practices with other open source sustainers!

Wow, super :-)
Haven't investigated, but the build fails on Travis though...

I think the issue is that I wrote the dependency as opencollective instead of opencollective-postinstall. The ppst install was recently changed to be more lightweight, and I did not update the npm install code for a few days - I apologize. I made the change.

Awesome, thanks! I'll have a deeper look soon


Bitfinex issue. Margin trading support.

System information

zenbot default
MacOS 10.14.3
v4.1.0 (commit ref, or version)
unstable
v10.13.0
Python 2.7.15
zenbot trade --paper
pasted API keys, and changed trading settings

Describe the problem

My strategy is short oriented, based on bollinger bands, so my paper trading sim balance is 2 BTC and 0 USD. Bot should work with margin trading, so i choose Bitfinex as exchange, but it doesn't work properly, for comparison I run same strategy with Poloniex, and console output seems good, but not with Bitfinex.

Does anybody use zenbot for margin trading? What exchange do you use? Does zenbot support Kraken margin trading?

Source code / Error logs
conf.js
https://gist.github.com/stepandra/bf4632518025651e682dc2d1c0fed156
conf-sample.js
https://gist.github.com/stepandra/a7d91eb8552081264612f86a906a50c0
Poloniex console output
https://i.imgur.com/IK9NF0v.png
Bitfinex console output
https://i.imgur.com/FONVfJy.png
As we can see, Bitfinex tickers( period is 5m in my strategy ) wasn't updated, also no balance, and no trades.

I can confirm. Margin trading doesn’t work.
Margin is the best way to setup a bot, if stoploss and takeprofit would work IN REALTIME (not now).
M.

System information
zenbot default
MacOS 10.14.3
v4.1.0 (commit ref, or version)
unstable
v10.13.0
Python 2.7.15
zenbot trade --paper
pasted API keys, and changed trading settings
Describe the problem
My strategy is short oriented, based on bollinger bands, so my paper trading sim balance is 2 BTC and 0 USD. Bot should work with margin trading, so i choose Bitfinex as exchange, but it doesn't work properly, for comparison I run same strategy with Poloniex, and console output seems good, but not with Bitfinex.
Does anybody use zenbot for margin trading? What exchange do you use? Does zenbot support Kraken margin trading?
Source code / Error logs
conf.js
https://gist.github.com/stepandra/bf4632518025651e682dc2d1c0fed156
conf-sample.js
https://gist.github.com/stepandra/a7d91eb8552081264612f86a906a50c0
Poloniex console output
https://i.imgur.com/IK9NF0v.png
Bitfinex console output
https://i.imgur.com/FONVfJy.png
As we can see, Bitfinex tickers( period is 5m in my strategy ) wasn't updated, also no balance, and no trades.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I can confirm. Margin trading doesn’t work. Margin is the best way to setup a bot, if stoploss and takeprofit would work IN REALTIME (not now). M.
…

I thought, that zenbot work properly with realtime SL, isn't it?
Did you find some bot solution for margin trading?
I found this bot https://github.com/Haehnchen/crypto-trading-bot, but I didn't test it yet.


bitwarden: 2023.10.1 -> 2023.12.0

Description of changes
Diff: https://github.com/bitwarden/clients/compare/desktop-v2023.10.1...desktop-v2023.12.0
Changelog: https://github.com/bitwarden/clients/releases/tag/desktop-v2023.12.0

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

I am not sure if the following errors are related to using electron 26
01:28:13.617 › TypeError: Cannot read properties of undefined (reading 'deleteBiometricKey')
    at BiometricsService.<anonymous> (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48310:55)
    at Generator.next (<anonymous>)
    at /nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48218:71
    at new Promise (<anonymous>)
    at biometrics_service_awaiter (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48214:12)
    at BiometricsService.deleteBiometricKey (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48308:16)
    at DesktopCredentialStorageListener.<anonymous> (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48517:45)
    at Generator.next (<anonymous>)
    at /nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48394:71
    at new Promise (<anonymous>)
01:28:13.618 › TypeError: Cannot read properties of undefined (reading 'deleteBiometricKey')
    at BiometricsService.<anonymous> (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48310:55)
    at Generator.next (<anonymous>)
    at /nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48218:71
    at new Promise (<anonymous>)
    at biometrics_service_awaiter (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48214:12)
    at BiometricsService.deleteBiometricKey (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48308:16)
    at DesktopCredentialStorageListener.<anonymous> (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48517:45)
    at Generator.next (<anonymous>)
    at /nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:48394:71
    at new Promise (<anonymous>)
(node:893169) UnhandledPromiseRejectionWarning: TypeError: Cannot read properties of undefined (reading 'setContextMenu')
    at TrayMain.updateContextMenu (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:47357:23)
    at MessagingMain.updateTrayMenu (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:46857:28)
    at MessagingMain.onMessage (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:46783:22)
    at MessagingMain.<anonymous> (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:46774:151)
    at Generator.next (<anonymous>)
    at /nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:46753:71
    at new Promise (<anonymous>)
    at messaging_main_awaiter (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:46749:12)
    at IpcMainImpl.<anonymous> (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:46774:79)
    at IpcMainImpl.emit (node:events:513:28)
(node:893169) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 22)

and when exiting:
01:28:25.591 › TypeError: Object has been destroyed
    at WindowMain.<anonymous> (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:47777:36)
    at Generator.next (<anonymous>)
    at /nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:47547:71
    at new Promise (<anonymous>)
    at window_main_awaiter (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:47543:12)
    at WindowMain.updateWindowState (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:47772:16)
    at WindowMain.<anonymous> (/nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:47706:28)
    at Generator.next (<anonymous>)
    at /nix/store/n501n5zszk113sq0wiyv47cxlqavhqgp-bitwarden-2023.12.0/opt/Bitwarden/resources/app.asar/main.js:47547:71
    at new Promise (<anonymous>)
Segmentation fault (core dumped)

TypeError: Cannot read properties of undefined (reading 'deleteBiometricKey')

Reading through the bitwarden/clients code this simply doesn't seem to be implemented for Linux.


preserveAllComments does not place comment close to original position

Describe the bug
I understand that preserveAllComments follow a heuristic but it seems quite wrong in this case and leads to some weird behavior in our application that relies on comments to be close to the AST node that they were placed against. We were hoping you might be able to do something about this for us.
Input code
export function a() {
    globalThis //?
}

Config
{
  "jsc": {
    "parser": {
      "syntax": "typescript",
      "tsx": true,
      "decorators": false,
      "dynamicImport": true
    },
    "transform": {
      "react": {
          "runtime": "automatic"
      }
    },
    "target": "es2018",
    "preserveAllComments": true
  },
  "sourceMaps": true
}

Playground link
https://play.swc.rs/?version=1.2.185&code=H4sIAAAAAAAAA0utKMgvKlFIK81LLsnMz1NI1NBUqOZSAIL0nPykxJyQjMxiBX19e65aAI2m0qAqAAAA&config=H4sIAAAAAAAAAz2OMQ7CMAxF7%2BK5AzChboiJgUNYxkVBdRzZDqKqencSEGxfT8%2F%2Fe4WHE4wrFDRn68mXHPiCEWIp7GSpBAwQ3lBY5QFuTGoYag7jhLN3tGSURBcpavH1tnZjmH1Sk15rjBSfUHMk4TaANVQwEsHWbbQ7NwPYD7v9sW0W4%2FbTk0%2FzfFYRzuH%2FbtdqxFcsP%2FQGyaRUqsoAAAA%3D
Expected behavior
export function a() {
    globalThis; //?
}

//# sourceMappingURL=file.js.map

Actual behavior
export function a() {
    globalThis;
} //?

//# sourceMappingURL=file.js.map

Version
<EMAIL_ADDRESS><EMAIL_ADDRESS>Additional context
No response

And that's probably the reason /* istanbul ignore next */ and similar comments do not work


Memory managment and perfomance

Hi!
I wanted to make a comment about excessive memory allocation.
Looked at version 5, saw some mentions of pools.
By redundant allocation of memory, I mean frequent calls to new and clear instead of using the allocated memory earlier. You can also write immediately in memory (file), instead of pre-recording in the buffer.
I would like to understand what we are going to in version 5.
Keeping keys, using buffers, etc.
I wanted to debug an upsert in version 5, so to speak, to feel the difference, but inside notImplemented.

Hi @lobster2012-user, now upsert already implemented. New v5 works complete different from all other version about how use memory buffers and re-allocate and share same memory spaces.
In v5 I implement some like virtual file mapping, in memory, in pages blocks. Each page buffer can be shared across threads when are read-only but has a own page buffer when need write. This avoid lot's off byte[] creation...

Hi @mbdavid .
Thank you for answer.


pdfx reports mailto: links as an error ('nonnumeric port')

Hi there,
Thanks for creating a really useful tool.
When running pdfx -c across a PDF with mailto: links (e.g<EMAIL_ADDRESS>it reports these as errors:
nonnumeric port<EMAIL_ADDRESS><EMAIL_ADDRESS>Is there a way to suppress this, or disable checking non-HTTP(S) links?
Also, the line output colouring doesn't seem to play nicely on Windows. Is there a way to disable this?
Thanks,
Graeme

Thanks for the feedback! On sunday ill release an update to fix mailto and
color output on win.
On Jul 22, 2016 11:36, "Graeme West"<EMAIL_ADDRESS>wrote:

Hi there,
Thanks for creating a really useful tool.
When running pdfx -c across a PDF with mailto: links (e.g. mailto:
<EMAIL_ADDRESS>it reports these as errors:
nonnumeric port<EMAIL_ADDRESS><EMAIL_ADDRESS>Is there a way to suppress this, or disable checking non-HTTP(S) links?
Also, the line output colouring doesn't seem to play nicely on Windows. Is
there a way to disable this?
Thanks,
Graeme
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/metachris/pdfx/issues/17, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHIy1apNOK5lDtmdtWwvJbWajAkvVX9ks5qYI8ogaJpZM4JSmNQ
.

Thank you!


Zypper_package prevents upgrade package on chef client > 12.6

It appears that changes were made to zypper.rb that now prevents upgrades. This seems to be a fundamental problem. Would anyone happen to know if there is a possible work around for this issue or if there exists a patch? We see that even the latest available chef client versions also exhinit the same problem”.
Can you please update on this issue. Thanks in advance.

Description
It appears that changes were made to zypper.rb that now prevents upgrades. This seems to be a fundamental problem. Would anyone happen to know if there is a possible work around for this issue or if there exists a patch? We see that even the latest available chef client versions also exist the same problem”.

Chef Version
chef client > 12.6

Platform Version
SLES12 and SLES15

Replication Case

Client Output
package is uptodate even there is new package is available in repo.

Stacktrace

Do we have any latest update on this issue. Thanks

There's not enough information in this issue

@rjnkmr04 can you provide a reproducer for this? ie. what are you executing and what the result is vs. expected result?

I'm going to close this out at this point. If you want to open a new issue that provides the necessary examples of what you're trying to accomplish we can dig in and see what we can do. There's not enough information in this ticket though to investigate.


[HOLD for #26747] [$1000] Request money - Amount is partially covered by top selection menu

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

issue found when executing PR https://github.com/Expensify/App/pull/23698
Action Performed:

Go to https://staging.new.expensify.com/
Log in with an expensifail account
Tap the green plus button
Tap on "Request money"

Expected Result:
Amount should be fully visible
Actual Result:
The amount is partially covered by top selection menu (because of the banner on top of the screen)
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[x] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/0576f922-c093-44c2-9016-e6184fe3ec34

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0152c32e49ad0a8a1b
Upwork Job ID:<PHONE_NUMBER>116150784
Last Price Increase: 2023-08-16
Automatic offers:

tienifr | Contributor | 26249176

@parasharrajat Apple iPhone SE (2020) / Safari

Will comment in the thread

We are doing something similar to 1. We will have to add logic to apply the smaller buttons when the screen dpi/height is smaller(link iPhone SE devices). For the rest of the screens, the normal button will show. This will only be applied to keypad buttons.

Can't we remove the BigNumberPad when screen is small and use normal keyboard?

Let's first try the smaller button approach. We won't be changing any percentage-based heights, we'll just see how small the viewport is and swap out our big buttons for small buttons.

Let's try both our regular buttons (40px) and our small buttons (28px) for the numbers and see how it goes. The bottom green button will be unchanged.

Yup, medium works for me!
On Mon, Aug 21, 2023 at 1:33 AM Rajat Parashar @.***>
wrote:

regular buttons(40px).
LGTM.
—
Reply to this email directly, view it on GitHub
https://github.com/Expensify/App/issues/24603#issuecomment-1685669984,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARWH5QBQU2IIG5YQYQQRRTXWLXJHANCNFSM6AAAAAA3RR36YQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Agreed, that proposal looks good to me

Hi, the PR create this regression

Removing Hold for payment, will update once #26747 is merged.

https://github.com/Expensify/App/pull/26747 still open

FYI ^that is incorrect. I did not review this on time because we were on a merge hold, so this is eligible for a bonus

Ah 🤦 thanks for the clarification

Summarizing payment on this issue:

Issue reporter: @wildan-m $250 via Upwork (offer extended in Upwork)
Contributor: @tienifr $1000 (paid in Upwork)
Contributor+: @parasharrajat $1000 via Newdot

Upwork job is here, no bonus

Hi @stephanieelliott thanks for the offer. I wanted to confirm the amount. My bug report is valid but it's not exported to github yet. If we refer to this link, how much I should receive? $50 or $250? (Of course I'll be happy with the last one 😁)
Note: my bug report posted one day before bonus recalibration published.

@wildan-m I'm basing off the date this GH issue was created, so it would be $250. Thanks for checking!

All paid!

@stephanieelliott Thanks, you are the best! 😺

$1,000 payment approved for @parasharrajat based on BZ summary.


Change Discovery API

I tried to download images from a change discovery Ordered Collection. My code fails because the label in Manifest -> metadata has been set to be of type String. Actually the object is supposed to be either a String or an array of a HashMap as defined here - https://iiif.io/api/presentation/3.0/#44-language-of-property-values
Fix this and the code should download all the images in all the manifests in all the pages of a given ordered collection


Fix broken --fail-fast

rubocop --fail-fast was broken at 43a157f1a674da16b45a13f69e242565382f7ac5
I fixed this and add minimum test for it.

Thanks for the first time contribution.
It seems that CI has failure.
https://travis-ci.org/bbatsov/rubocop/jobs/350648219#L4210
That will be resolved by adding your account at the end of CHANGELOG.md.
Refer https://github.com/bbatsov/rubocop/commit/8ec6cf6dcf70b2cb796ab26d9193c3a4eb9c3eaf#diff-4ac32a78649ca5bdd8e0ba38b7006a1eR3232
When you are finished, please squash multiple commits in PR into 1 commit.

@koic I fixed the CHANGELOG.md and rebased the commits into a single one :+1: Thank you for your help :raised_hands:

👍

👍

Are there any plans to release it?


[Task] Implement streaming latency report to Analytic API

Description

Implement streaming latency report to Analytic API

Impacted Area

None

Ticket URL

https://uizaio.atlassian.net/browse/EBC-417

@deathemperor This PR is not fully tested. So, please consider before merge it !
Thanks !

Currently, the old trackingMuiza implementation works only for VOD, I will check and fix for livestream soon.

Will update later


chore: bdd-spring-1576227768 to 0.0.1

chore: Promote bdd-spring-1576227768 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[BugFix] Solve issues-3470: Yarn webui fails to obtain task status when submitting a Flink task after turning on Kerberos authentication.

解决yarn webui开启Kerberos认证后递交Flink任务无法获取任务状态的问题，增加调用Flink HTTP API时候进行Kerberos认证，
相关issues: https://github.com/DataLinkDC/dinky/issues/3470
Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

请使用spotless 格式化代码,详细查看此文：https://www.dinky.org.cn/docs/next/developer_guide/contribution/pull_request#pull-request-code-style

Hi. Run 'mvn spotless:apply' to fix these violations.


Documentation improvements

Include the necessity to provide a heading or caption to tocs when show_nav is false.
first sentence of how to configure is superfluous
option lists instead of definition lists?
the description of the samp directive also reads very odd...

.. option:: is meant for command-line programs. Therefore skipping it for now.


test(cldloader): add test cases for cldLoader

Codecov Report

Merging #7 into master will increase coverage by 79.1%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master     #7      +/-   ##
========================================
+ Coverage   20.89%   100%   +79.1%
========================================
  Files           8      7       -1
  Lines          67     50      -17
  Branches        9      8       -1
========================================
+ Hits           14     50      +36
+ Misses         53      0      -53

Impacted Files
Coverage Δ

src/cldLoader.ts
100% <100%> (+100%)
:arrow_up:

src/cldFactory.ts

src/cldAsmModule.ts
100% <0%> (+100%)
:arrow_up:

src/loadModule.ts
100% <0%> (+100%)
:arrow_up:

src/index.ts
100% <0%> (+100%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix nestjs-zod peer dependency version

Problem:
Getting a warning from Yarn:
nestjs-zod is listed by your project with version 3.0.0, which doesn't satisfy what nestjs-zod-prisma (p2e4d0) requests (^1.1.0).

Solution:
nestjs-zod is currently at 3.0.0 and it seems the intention is to align these versions.

Any progress regarding this?


How to return value in async testing via done?

I am testing my apis by using mocha. Since calling to my apis needs authorization, I call the authorization api to get the access token in the beforeEach hook. I wrap the doFakeAuth function in the utils.js file and export it, which is denoted as follows.
exports.doFakeAuth = function (done) {
  var route = '/v1/auth/weibo';
  var body = {
    weibo_uid: 'test_weibo_uid_full_data',
    avatarurl: 'http://fake_weibo',
    birth_year: 1995,
    birth_month: 5,
    birth_day: 8,
    gender: 2,
    nickname: 'cw',
    signature: 'i will win'
  }

  request(url)
    .post(route)
    .send(body)
    .expect('Content-Type', /json/)
    .expect(200)
    .end(function (err, res) {
      if (err) throw err;
      //
      res.body.should.have.property('user');
      res.body.should.have.property('token');
      res.body.user.should.have.length(24);
      return res.body.token;
      done();
    });
}

However, I cannot get the res.body.token in the async call. I also tried
done(res.body.token);

But it said Error: done() invoked with non-Error: <token detail>.
So how can I get the token in the caller.

return res.body.token;
done();

The above is a basic misunderstanding of programming.  Once you return from a function, any subsequent lines within the function are unreachable.
I'm not sure really what I'm looking at here.  Is utils.js code under test, a fixture, or what?  If you need to call some async function and get its return value:
describe('foo', function() {
  var response;
  beforeEach(function(done) {
    someAsyncCall(function(err, data) {
      response = data;
      done();
    });
  });
  it('should blah blah', function() {
    // use response here
  });
});

@boneskull How would you do this if beforeEach is defined in a helper file to be used across multiple specs?

@schatekar do what..?

If I have defined beforeEach in a separate file then how do I return value of response variable set inside beforeEach back to my specs?


[ New Button ] :  Neumorphic-neon-button

This is a(Neumorphic-neon-button):

[ ] Update to an existing Animated Button
[ ] Any Error
[x] Proposal to the Repository
[ ] Documentation / README.md changes

**Details:
Neumorphic-neon-button **

@PoorviShetty @Spyware007 could you look in this matter and assign me this issue.

Could you plz assing me this issue???


Add CI workflow against xmonad latest commit

Description
Add CI against xmonad latest commit
Checklist

[ ] I've read CONTRIBUTING.md

[ ] I tested my changes with xmonad-testing

[ ] I updated the CHANGES.md file

[ ] I updated the XMonad.Doc.Extending file (if appropriate)

@liskin I have updated the MR and the cache seems to be restored properly (that was the only concern I had with introducing the cabal hashes). If you can give another quick review, I would be happy to get this merged.

@liskin Done. BTW, don't you have access to merge it yourself ? If not, I think we should fix that.

@psibi No, I'm only in the "Contributors" team, not in the core.

@byorgey Can we add @liskin as part of the core team ?

I've sent the invite.

Thanks @geekosaur!
I'll try to use my best judgement but I'd still appreciate if the expectations were said explicitly, like where's the boundary between "just merge it", "seek review from anyone" and "seek review from more senior members". ;-)

Given how people keep disappearing, I'm not sure we can have boundaries there. One may find oneself the most senior member available.


RAG Implementation with Elastic search cloud

@pranaydeep139 these changes seem out of date. If they're no longer required, please close the PR and delete the branch.


Match string variables with literal information in oneOf proptype

This could be way off-base, but it seems like we'll often have literal information with a StrT. This makes oneOf a lot more powerful/useful.

This change isn't as foolproof as I'd hoped.
var s = "hello";
var f = () => React.createClass({ propTypes: { s: React.PropTypes.oneOf([s]) } });
s = "oops";
f() // runtime check and static type disagree


fix(swoole extension) | workspace

fix when skip install extension, build fail.

Hello @xx, any date for this adjustment? I'm experiencing this problem (https://github.com/laradock/laradock/issues/2910#issuecomment-804422479 )... this solution worked for me..

@garbinmarcelo Hi!

@garbinmarcelo Hi!

Sorry my friend, I ended up marking the wrong user in this post. :laughing: :coffee:


chef-client::config fails on Russian version of Windows

https://github.com/chef/ohai/issues/1233

Closed by https://github.com/chef/ohai/pull/1240


[Bug] robocop fails with AttributeError: 'NoneType' object has no attribute 'data_tokens'

What happened?
Tried out robocop on some of our test suites, and I see a crash examining our test suites
root@7f1836e96e29:/home/cisco/cxta# robocop --exclude wrong-case-in-keyword-name tests/integration/robot_files/
[..many warnings removed..]

Traceback (most recent call last):
  File "/venv/bin/robocop", line 8, in <module>
    sys.exit(run_robocop())
  File "/venv/lib/python3.8/site-packages/robocop/run.py", line 280, in run_robocop
    raise err
  File "/venv/lib/python3.8/site-packages/robocop/run.py", line 270, in run_robocop
    linter.run()
  File "/venv/lib/python3.8/site-packages/robocop/run.py", line 81, in run
    self.run_checks()
  File "/venv/lib/python3.8/site-packages/robocop/run.py", line 125, in run_checks
    found_issues = self.run_check(model, str(file))
  File "/venv/lib/python3.8/site-packages/robocop/run.py", line 143, in run_check
    for issue in checker.scan_file(ast_model, filename, source, templated)
  File "/venv/lib/python3.8/site-packages/robocop/checkers/__init__.py", line 103, in scan_file
    self.visit_File(ast_model)
  File "/venv/lib/python3.8/site-packages/robocop/checkers/spacing.py", line 367, in visit_File
    self.check_empty_lines_after_section(section)
  File "/venv/lib/python3.8/site-packages/robocop/checkers/spacing.py", line 405, in check_empty_lines_after_section
    section_name=get_section_name(section),
  File "/venv/lib/python3.8/site-packages/robocop/utils/misc.py", line 320, in get_section_name
    for token in node.header.data_tokens:
AttributeError: 'NoneType' object has no attribute 'data_tokens'
Fatal exception occurred. You can create an issue at https://github.com/MarketSquare/robotframework-robocop/issues/new/choose . Thanks!

I am able to isolate the issue to a specific test suite:
## Unit test file for cxta_ccdiff.py keywords
#
*** Settings ***

Library    BuiltIn
Library    OperatingSystem
Library    CXTA
Resource   cxta.robot
Force Tags  diff

*** Variables ***

# Directory where configs are stored
${dir}      tests/datafiles/cxta_ccdiff-configs

*** Test Cases ***

ios-diff
    @{result}=   use ciscoconfdiff to compare ios configs "${dir}/ios-1.txt" and "${dir}/ios-2.txt"
    verify result   ${result}[0]   ${dir}/ios-expected-diff1.txt
    verify result   ${result}[1]   ${dir}/ios-expected-diff2.txt

but when I take the file and put it elsewhere, robocop processes it just fine.. so you might actually have challenges reproducing this in your environment..
When I add a breakpoint before the failure, I see
319  	def get_section_name(node):
320  	    if node.header is None:
321  	        breakpoint()
(Pdb) p node
<robot.parsing.model.blocks.CommentSection object at 0x7f2b9a736820>
(Pdb) p node.body
[EmptyLine(tokens=[Token(EOL, '\n', 1, 0)]), Comment(tokens=[Token(COMMENT, '## Unit test file for cxta_ccdiff.py keywords', 2, 0), Token(EOL, '\n', 2, 45)]), Comment(tokens=[Token(COMMENT, '#', 3, 0), Token(EOL, '\n', 3, 1)])]
(Pdb) p node.header
None
(Pdb)

Operating System
Linux
Robocop version
2.0.0

Nevermind, I see it -> "Token(EOL, '\n', 1, 0)]), Comment..". So it starts with empty line that's why it tries to report the issue - but it fails due to section not having header ("hidden" *** Comments ***).

For workaround you can try to exclude this rule (-e empty-line-after-section)


select leaf and display details

#5
changes in this pr:

removed pico.css
implemented initial sizing & layout as discussed
additional percy snapshot sizes for selected leaf

@rlskoeser thank you!! It is so helpful to see the various sizes!

Since you're saying there won't be that much more data I'm confident that the experience on the 1024px will be better if it's similar to the 1280. So we can make the mobile threshold at 768px and below instead.
It would be ok to keep the panel/svg ratio as is for now till we add the labels and see how much space is left for us but I think the svg panel could easily be reduced to 60% rather than the 65% – that way we could have more relaxed margin on the right for the panel.


Generated forms are not ADA compliant

I was playing with your library and realized that generated forms have labels that are not tied to the corresponding input controls through IDs. This results in screen readers not finding the information to read out to the disabled users.

Thanks for the feedback! Feel free to fix it with a pull request, otherwise I'll put this on the backlog.

Does your tagging mean you have completed this item for the 0.6.0 version? OR does it mean you plan to have it done for that version?
I'd like to help if you can use a newbie.


Use constant value supplier to measure ingestion delay metrics

Problem
In multiple Pinot deployments, we have seen the REALTIME_INGESTION_DELAY_MS and END_TO_END_REALTIME_INGESTION_DELAY_MS metrics monotonically grow even though there were no active Kafka consumers.
How did I figure out that there are no active Kafka consumers?
consumingSegmentsInfo API returned the below response:
{
  "_segmentToConsumingInfoMap": {
    "telemetry__0__2399__20240415T1650Z": [],
    "telemetry__1__2435__20240415T1600Z": [],
    "telemetry__2__757__20240415T1632Z": [],
    "telemetry__3__2413__20240415T1642Z": []
  }
}

The above response shows that consumers are not connected. I also checked the thread dump, and there are no Kafka consumer threads.
Image showing REALTIME_INGESTION_DELAY_MS grow monotonically

Zoomed out image showing REALTIME_INGESTION_DELAY_MS grow monotonically every minute

Solution
Changed the supplier to provide constant value to publish REALTIME_INGESTION_DELAY_MS and END_TO_END_REALTIME_INGESTION_DELAY_MS metrics.
Ingestion delay metrics are published for every batch of messages fetched off Kafka HERE, and this will continue to work when Kafka consumers are active.
bugfix

I don't think this is the correct fix. We rely on the callback to track the ingestion delay. I feel the real issue is this metric is not removed after the consumer is destroyed.

There are two issues:

Why metric is not removed when consumer is destroyed?
Why metric keep increasing when no consumer exists?
This PR address #2. In fact, the happy path publishes the metric after every batch at https://github.com/apache/pinot/blob/7f87cd3a9ffd7f73c14502d9457a037926aab75f/pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java#L647 .

IMO, we need to fix both the issues.

It is under the if (previousMeasure == null) branch, which means this PR will completely turn off the gauge...
I don't see this as a problem because we want to get the delay dynamically. The really issue is why the metric is not removed