ember-data-1-0-beta-15 breaks record.serialize

running almost anything through ember-fire results in
Uncaught TypeError: undefined is not a function.

the offending line here is:
var serializedRecord = record.serialize({includeId:false});

file/code here
the problem is that record is now in fact a snapshot not a record and accordingly the api has changed.
a quick solution is:
record.record.serialize({includeId: false})

as record.record results in the same object as the pre beta-15 api returned.
I am working on:
EmberFire - 1.4.2
Hope this helps.
notes:
http://emberjs.com/blog/2015/02/14/ember-data-1-0-beta-15-released.html

Hey, thanks for the report, this looks like a dupe of #229.
I wasn't aware of snapshot.record to get back to the original record. We should be able to use something like record = record.record || record to grab the record in beta.15+
I'll try to find some time over the weekend to look at 15+ compat.

oops I meant #226

:+1: thanks.

PR raised #234

Thanks!

On Apr 1, 2015, at 1:03 PM, Tim Stirrat<EMAIL_ADDRESS>wrote:
PR raised #234
—
Reply to this email directly or view it on GitHub.

You can now upgrade emberfire to 1.4.4 in package.json to get this fix.


Batch output does not create folder location if it doesn't exist error

Bug Description
Not sure if this is a bug or a feature request considering its something that is in the original RVC but when you are doing batch inference and your output folder location doesn't exist it'll error out. This feature is something I love using in normal RVC to separate my batch outputs very much need it in Applio as it gets tedious having to create each folder manually.
Steps to Reproduce
Outline the steps to replicate the issue:

run-applio.bat
go to batch tab
put in your input folder location
put in desired output folder location
edit settings if needed
convert

Expected Behavior
In normal RVC it'll create the folder output location if it doesn't exist but in Applio it just errors out instead of making that folder.
Assets

Desktop Details:
Windows 10, Chrome using the web ui (run-applio.bat)

Thank you for reporting this issue. I understand that you're expecting the output folder to be created automatically if it doesn't exist, similar to the behavior in the original RVC. This is a reasonable expectation, and I'll investigate further.
In the meantime, I'd like to confirm that you've checked the create_output_folder setting in the applio_config.json file. According to the documentation (https://docs.applio.org), this setting should be set to true to enable automatic creation of the output folder. Could you please verify if this setting is enabled in your configuration file? If not, please try setting it to true and see if it resolves the issue.

there is no create_output_folder setting in the config.json for applio

hey, we can add that feature

done https://github.com/IAHispano/Applio/commit/f40b82178d5f37f4d02a364c86e47cbd3b3e7d6d


Add missing slash for main entry file

This PR fixes next error (probably because of invalid path to main entry file).
Module not found: Error: Cannot resolve module 'electron-sudo' in ...

Temp solution is to use electron-sudo/dist/index require path instead.

@sergeybekrin Thanks<EMAIL_ADDRESS>


Support double signal handling

Fixes #220

Great writeup @krnowak!

@krnowak It's ready for another review

One thing I missed. The defer hc.queue.Shutdown() will likely need to stop being a defer and should be put between <-ctx.Done() and wg.Wait(). Otherwise it will probably deadlock…

Another thing - likely all the wg.Done() calls should be deferred, but wg.Wait() calls - not.

@krnowak Another one :)

@krnowak Another one 💯

LFAD.


Submission approval: Log in separate private channel

Currently, all submission approval/rejections are logged in a channel that contains several other log messages, causing it to become quite spammed. A separate private channel for this type of content is a much-needed addition.

Create a new #submission-log channel
Any automated actions (such as automatic rejections) are logged in this channel
Any moderator actions (submission approval/rejections) are logged in this channel
User actions (reactions) are NOT logged in this channel

Verified on prod, closing


Add options to change the rotation angle

This adds an angle option to the Flip module to rotate the image 90, 180 and 270 degrees
@ckassab Test this locally and let me know if there are any issues.

Fixed and tested with @ckassab , merging


pull request for e2e test at 12/08/2016 12:26:02

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


Name collision with "require"

filtrex has a global name collision with requirejs's require. Can we please rename all references to require?

This is actually an issue in the jison.js dependency.
A few options:

As part of the build file we find and replace all "require" refs to something else, e.g. "_internal_filterex_require".

We wrap everything in an anonymous function (function() { ... })() so it doesn't pollute the global namespace.

@JonathanWheeler: I'm busy with other projects right now. Want to take a crack at either of these?

Hi,
you only need to wrap the entire content of filtrex.js with a define([],function{
});
that makes require local. Additionally you might return a require-js module:
https://github.com/mc007/xexpression/blob/master/expression.js


Update to glimmer-scoped-css 0.7.0

This doesn’t work, there must be configuration missing or something as boxel-ui no longer has styles/global.css in dist, which causes the host build to fail.
But it does fix that components in boxel-ui can have images in their CSS that gets copied into dist.

I pushed a commit here that I hope helps unblock things:

Upgrade @embroider/addon-dev to the unstable channel. That is where the feature in keepAssets() we need is released right now.
Drop rollup-plugin-import-css. That's not what we want while building a library. Presumably it was added in response to an error message, but that error was due to the lack of the aforementioned keepAssets() feature that makes it interoperate with other CSS-producing plugins.
I fixed a couple unrelated build warnings where rollup didn't understand the way interior imports were being done via the package name.

The thing to know about keepAssets is that it handles files that appear in an import statement somewhere in the code. It ensures that both the import statement and the corresponding file are propagated to the build output. It doesn't do anything with files that nobody is importing. So when this PR does import './hello.png';, that's a clever workaround to trick keepAssets into including hello.png in the build output, so that url('./hello.png') in the CSS down below will probably work during the app build.
keepAssets does not currently look inside CSS files to recursively identify more assets that need to be kept. That would be a good feature to add. That would eliminate the need for this kind of workaround.
Alternatively, one can use the copy plugin to just directly copy various files through to dist. I used that here for the global.css, etc.


Renaming non-abstract AbstractButtonBlock to ButtonBlock

As of 1.20.1, AbstractButtonBlock isn't Abstract.

verified in enigma, will be merged on check completion!


Added Azure.MariaDB.FirewallRuleCount

PR Summary
Fixes #1855
Added Azure.MariaDB.FirewallRuleCount rule.
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] Change is not breaking
[ ] This PR is ready to merge and is not Work in Progress
Rule changes

[x] Unit tests created/ updated
[x] Rule documentation created/ updated
[x] Link to a filed issue
[ ] Change log has been updated with change under unreleased section

Other code changes

[ ] Unit tests created/ updated
[ ] Link to a filed issue
[ ] Change log has been updated with change under unreleased section

Hey @BernieWhite

Could you tell me what you're thinking about using this logic? The rule is not bulletproof as it only runs against child resources declared within the parent resource, but from my experience most customers declare child resources within the parent resource anyways.
I'm waiting for the merge on Azure.MariaDB.AllowAzureAccess that will help with good test cases.
Should we add ARM and Bicep code examples for this one? Where we declare some firewall rules on a MariaDB server?

@BenjaminEngeset I have a few comments below but it looks good so far.

Could you tell me what you're thinking about using this logic? The rule is not bulletproof as it only runs against child resources declared within the parent resource, but from my experience most customers declare child resources within the parent resource anyways.

For counting the number of firewall rules for the server, this logic looks good. PSRule will automatically nest sub-resource into the parent resource if they exist in the same deployment/ template when expanding a template. So declaring sub-resources outside of the parent resource is not a concern.
You could write a template with only one firewall rule in it but since we can't make an assumption about how many rules already exist; it's an edge case we don't need to solve in this rule.

Should we add ARM and Bicep code examples for this one? Where we declare some firewall rules on a MariaDB server?

Because this rule is about the number of firewall rules and reducing them, I don't think we need to an example code.

Thanks @BernieWhite. I had no idea that PSRule was that powerful in terms of automatically nesting sub-resources that are declared outside the parent resource, always taught they would been treated as standalones then.

@BernieWhite
Check the latest updates and let me know you're thinking.


encoding/asn1: slice is lost during marshal/unmarshal

The following program:
package main

import (
    "encoding/asn1"
    "fmt"
    "reflect"
)

func main() {
    data := []byte("\x06\x010")
    var v interface{}
    _, err := asn1.Unmarshal(data, &v)
    if err != nil {
        return
    }
    data1, err := asn1.Marshal(v)
    if err != nil {
        panic(err)
    }
    var v1 interface{}
    _, err = asn1.Unmarshal(data1, &v1)
    if err != nil {
        panic(err)
    }
    if !reflect.DeepEqual(v, v1) {
        fmt.Printf("v0: %#v data=%q\n", v, data)
        fmt.Printf("v1: %#v data=%q\n", v1, data1)
        panic("not equal")
    }
}

prints:
v0: []int{1, 8} data="\x06\x010"
v1: <nil> data="0\x06\x02\x01\x01\x02\x01\b"
panic: not equal

The slice should not get lost during marshal/unmarshal.
on commit b0532a96a850c6c93de2c414f5349562938f0f19

Unmarshal/Marshal/Unmarshal was not idempotent as the Object Identifier type was not returned through the interface. The limit case OID = 0 returns an error. The zero OID is {0.0}
If bytes are transmitted to Marshal, a sequence of bytes is returned and not an OID and this is the intended behavior. Object Identifier is a list of int and the example submitted translates to
\x06\x010 = \x06\x01 + "0" which is OID {1,8} coded as \x06\x01\x30


Infoj Skip Update

It was noticed that the infoj_skip method wasn't working correctly when you tried to use group in the current codebase.
To fix this, the code has been amended to check if each property exists n the infoj_skip array, rather than using OR operator.

@dbauszus-glx Thanks for the comments, I've used that code block above to amend infoj.mjs and working well :)
I've also amended filter.mjs to use the same logic to ensure if skipping group this is also skipped in the filter here too.
Ready for review again

This is all working for me now - thanks @dbauszus-glx for switching the filter check :)


Update apply-tier-4-visa.yml

https://govuk.zendesk.com/agent/tickets/994665 downtime for 5 April to 7 April.
Needs to revert back on the morning of Tues 7 April.
e.g. https://www.gov.uk/apply-tier-4-visa/y/extend_general/V6G1G9F62

:+1:


Upgrade Apache Commons Collections to v3.2.2

Version 3.2.1 has a CVSS 10.0 vulnerability. That's the worst kind of
vulnerability that exists. By merely existing on the classpath, this
library causes the Java serialization parser for the entire JVM process
to go from being a state machine to a turing machine. A turing machine
with an exec() function!
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2015-8103
https://commons.apache.org/proper/commons-collections/security-reports.html
http://foxglovesecurity.com/2015/11/06/what-do-weblogic-websphere-jboss-jenkins-opennms-and-your-application-have-in-common-this-vulnerability/

We can no longer merge this pull request as is, however the dependency has been upgraded. Thanks!


Integrate with baidu warp-ctc

I want to integrate mxnet with warp-ctc https://github.com/baidu-research/warp-ctc .
I find that, warp-ctc only provide one interface called:
ctcStatus_t compute_ctc_loss(const float* const activations,
float* gradients,
const int* const flat_labels,
const int* const label_lengths,
const int* const input_lengths,
int alphabet_size,
int minibatch,
float *costs,
void *workspace,
ctcComputeInfo info);
For CTC Loss layer, its forward is same as Softmax. The only difference is calculate grad in backward.
I find in mshadow/tensor_c(g)pu-inl.h , there is SoftmaxGrad. So, I think the easiest way to do this is implement an CTCGrad and use this in backward function to calculate grad.
This is my main idea. But I do not know how to do it in detail. For example, mxnet use expression template, but warp-ctc is different. So, how can I mix them together?

Another way may be write an python layer to use ctypes to call libwarpctc.so according to http://mxnet-tqchen.readthedocs.io/en/latest/how_to/new_op.html
However, I hope I can use mshadow to integrate it.

Since it is from C++, the best way will be integrate from c++.
The expression template is not a must for mxnet, but rather a convenient way to do so. The TBlob support directly fetching out raw pointers, and from there there is not much difference of writing a kernel. See the implementations of calling CuDNN kernels might help

see ROI pooling will be more helpful.
On Wed, Jun 1, 2016 at 09:31 Tianqi Chen<EMAIL_ADDRESS>wrote:

Since it is from C++, the best way will be integrate from c++.
The expression template is not a must for mxnet, but rather a convenient
way to do so. The TBlob support directly fetching out raw pointers, and
from there there is not much difference of writing a kernel. See the
implementations of calling CuDNN kernels might help
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/2305#issuecomment-223049537, or mute
the thread
https://github.com/notifications/unsubscribe/ABM13n_JajIts6S8hcgL7yshvU3FsH9Uks5qHbPkgaJpZM4IrRnq
.

--
Sent from mobile phone

see http://mxnet.dmlc.ml/en/latest/system/operator_util.html

This can be done as a plugin. Check plugin/torch for example

Thanks all, I will try your suggestions today!

When I get int labels, I use following code:
TBlob label = in_data[warpctc_enum::kLabel];
int* flat_labels = static_cast<int*>(label.dptr_);
However, I find label is always a float array, only
float flat_labels = static_cast<float>(label.dptr_)
return right results.
When I pass data in DataIter, I use
label_all = [mx.nd.array(label, dtype = np.int32)]
but it does not work. Label is alwarys a float array...

you need to add a InferType function to your operator property to specify the type of your input/output. and you need to bind your input to the right type

@tqchen @antinucleon @pluskid currently module doesn't support specifying type. adding a data_type is probably not a good solution since there will be data_ctx for model parallel support.
Should we change data_shapes to (name, shape, type, ctx)? However the name data_shapes is miss leading.
Is a noncompatible change to data_info acceptible?

I am personally ok with this change, or maybe keep both fields and if info is available use info first? Although it do feel a little bit strange to ask the data iter to specify the context.

I think we should not change the data_shapes.
Instead, trying to specify the type in a similar way as context, or either via a seperate mapping or attributes in graph might be a better idea


Question: About sudo process migration

Hi, I'm interested in having PAM available in xcodes.
I'll try to make a PR, but I have a few questions before it.

Are there any concerns about using posix_spawn instead of Process when running sudo?
Is it OK to obsolete readSecureLine?

I would like to do the migration without interfering with the current use case.
Thanks

By becoming sudo via PAM, you can do away with your own password input and use a user-specified method (ex. Touch ID).

Hi @noppefoxwolf always open for ways to make Xcodes better!

From what I read and my understanding, there is no current concerns with using posix_spawn
I'm not too familiar with PAM but Xcode CLI can be used in CI environments so as long as there are options to keep the existing functionality (manually type in sudo password security) i'm up to see how it can be improved.

@MattKiazyk
Thank you for feedback!

I'm not too familiar with PAM but Xcode CLI can be used in CI environments so as long as there are options to keep the existing functionality (manually type in sudo password security) i'm up to see how it can be improved.

OK, I wasn't thinking about CI.
I'd like to keep the current behavior and provide the option to use PAM.


Prez UI label display

Simplify the handling of rendering UI items.
As a start, the API should provide an rdfs:label value for all vocab list items, property items, etc.
We may look to use JSON-LD framing in the future for handling UI data and drop the need for RDF parsing libraries.

This appears to be working now. API returns rdfs:label by default and the UI renders the label property defined in the respective profile.


optimize enumerate with start argument

Change looks good to me, thanks. Could you please add a test in tests/run/for_in_string.pyx? You can use the typeof() function to show the inferred type of the loop variable in your example.

I'll revert this change until https://github.com/cython/cython/issues/4171 is resolved.


Bugfix for issue 28222

Please take look at the fix and upload the new version to jenkins plugin store.

Fixes issue #4


Benchmarks?

I've been using imagemin for a while just because of the devs around it, but does anyone have any stats on how it stacks up to other image minifiers? A co-worker is suggesting ImageAlpha and ImageOptim.
If not, could someone write a blog post on this or add a cool graph to the README?

ImageAlpha is a GUI for png only and uses pngquant, pngnq and posterizer
ImageOptim is another GUI which under the hood uses PNGOUT, AdvPNG, Pngcrush, OptiPNG, JpegOptim, MozJPEG, jpegtran, and Gifsicle
The only plugin I don't see for imagemin is pngnq and posterizer and I don't think that really matters with the 5 other png plugins available. So they all pretty much use the same open source compressors, and imagemin has the better coverage IMO as seen on https://www.npmjs.com/browse/keyword/imageminplugin

Thanks for the info. I figured the GUIs were probably using open-source compressors under the hood.
I've been lazily using imagemin-cli's defaults.
Co-workers are claiming these "stacks" offer significantly higher compression than imagemin-cli's defaults:

png: ImageAlpha -> ImageOptim (co-worker claims to have shrank 700kb to 100kb which wow'd my socks off)
jpg: JPGmini -> ImageOptim

I'll do a few of my own tests on various images and post my results this weekend, but if there are huge savings to be had by combining various compressors it might be a worthy endeavor to create some sort of engine that would test various combinations of these plugins to get huge, near-lossless savings.

Imagemin is just a light wrapper around multiple different image optimization tools:
https://www.npmjs.com/browse/keyword/imageminplugin
You'd need to look into benchmarks for the different tools. I don't have any numbers in front of me, but from some limited research a year ago I recall pngquant being the best tool for PNGs. I also did some tests about a week ago and found mozjpeg was the best for JPEG files (~30% smaller than all other tools I tested) although that may be a special case for the files I was optimizing.
Also whenever you hear large stats like a file shrinking from 700kb to 100kb, it's good to investigate and see what's going on. Maybe the 700kb file was the least-optimized possible bitmap and any minification tool would have gotten it around ~100kb. Maybe one of the tools he used was lossy and destroyed the quality. Maybe one of the tools was limiting the image size (in pixels) -- I usually run a gulp script which limits my images to 1600px wide because anything wider than this is just overkill in my opinion.

Thanks @stevendesu
I'll revisit this in the future. I think I'd like to combine https://github.com/uber/image-diff/ with imagemin and just throw a million combinations at it so quality is >= 90% with the most savings.
It'd take a while to process a directory of images using various settings, so I'll just use imagemin defaults and wrap that functionality up into a package to run as a build step. Then run that step overnight or something.
I'll record the winning combinations, and if I end up noticing a particular combination routinely does better I can report back.


how to change the default learning rate ？

Hello, the initial lr is 10-3, how can I change this in the codes?
Thank you.

i have same question


start adding python bindings with pybind11

need to grab a few things for python build:
$ sudo apt update
$ sudo apt install python3-build python3-wheel python3-venv python3-virtualenv python3-pybind11

build c++ with python binding flag
$ mkdir build
$ cd build
$ cmake -DBUILD_PYTHON_BINDINGS=ON ..
$ make

$ cd ../fshd_py
$ python3 -m build --wheel

Then you can either install and use it:
$ pip3 install dist/*.whl
$ fshd_motion_example

Or, you can try it out before installing (this method will also work after installing)
$ python3 -m fshd.examples.motion

added command line options from example as well (use -h to check)
$ fshd_motion_example -h
usage: fshd_motion_example [-h] [-a A] [-t T] [-p P] [-b B]

options:
  -h, --help  show this help message and exit
  -a A        (meters) sets the incident wave amplitude
  -t T        (seconds) sets the incident wave period
  -p P        (degrees) sets the incident wave phase angle
  -b B        (degrees) sets the incident wave direction

@hamilton8415 I finished binding the LinearIncidentWave and FS_HydroDynamics classes. I also ported MotionExample.cpp to python using the new module.
need to grab a few things for python build:
$ sudo apt update
$ sudo apt install python3-build python3-wheel python3-venv python3-virtualenv python3-pybind11

build c++ as usual
$ mkdir build
$ cd build
$ cmake ..
$ make

$ cd ../fshd_py
$ python3 -m build --wheel

Then you can either install and use it:
$ pip3 install dist/*.whl
$ cd ~
$ mkdir fshd_motion_plots
$ cd fshd_motion_plots
$ fshd_motion_example

Or, you can try it out before installing... but it will spit out png's in the fshd_py directory
$ python3 -m fshd.examples.motion

now you can choose to turn python bindings on (off by default) with
cmake -DBUILD_PYTHON_BINDINGS=ON ..

two more examples:
$ python3 -m fshd.examples.incidentwave
$ python3 -m fshd.examples.excitingforce

all examples added

added readme

check this out! Found something I could just drop your incident wave class into (like 10 lines of code)
using the python bindings
$ pip3 install pyglet configparser
$ tar -xvzf pabennett_ocean.tar.gz
$ cd pabennett_ocean
$ python3 main.py

pabennett_ocean.tar.gz

make building the python stuff the default

done!

array of inc_wave objects will need to be passed to the fs-hydro calc routines

Shouldn't be a problem for the bindings.
Glad you like it; it was really fun to do :) I hadn't used pybind11 before (I always used boost.python) but I know it's pretty popular and I can see why. They made it super easy and it doesn't have the bloat of boost (or the really convoluted compiler errors).

Merging


Upgrade to symfony 2.6.4

Fix security

Did you launched the functionnal tests? Everything works? :)

cartouche/app/phpunit.xml.dist
..
Time: 14.22 seconds, Memory: 43.50Mb
OK (2 tests, 16 assertions)
Everything is Ok


Extend webhook usage section

Based on https://t.me/grammyjs/20079

Should be merged after #67 in order to avoid conflicts


HTML code hinting without "<"

Can we have HTML code hinting without starting with the "<".
It makes coding pages quick and easy, it would also be super great
if frameworks like bootstrap could be included in the code hinting feature.
            * CURRENT
    1. Type "ar"     => no code hint
    2. Type "<ar"   => code hint (area, article)

            * IDEAL
    1. Type "ar"     => code hint (area, article)

@mozn-lz Thanks for using brackets.    In it's current form,  Brackets provides html tag hints only when you have pressed '<'. Any character you type after that is just going to provide filtering. This has been done with a purpose, to keep it non intrusive and keep it targeted. You can always seek hints by pressing ctrl+space.

Also you can use Emmet


Router.run and Handler documentation

https://github.com/rackt/react-router/blob/master/UPGRADE_GUIDE.md#rendering
I'm new to React Router and it seems the documentation is out of date. Anywhere I can reference to learn the new API?

If you want to use React Router 0.13.x the documentation is here.
Or for the React Router 1.0.0-rc1 which the most recent release the docs can be found here.
What you pointed out is the Upgrade Guide, which is useful when you want to upgrade from 0.13.x to 1.0.0-rc1

hmm. so the latest release doenst use route handlers anymore?

@ccorcos no it doesn't. please have a look into the docs I pointed you out to.

Yeah. I checked it out. Just wanted to make sure because there was a stack overflow question that said the Route component was deprecated. But it looks like its back!

Hi guys.I am trying to update React from version 0.13.3 to the version 15.6.1., could you point me out what am I doing wrong please? :
Router.run(app.getComponent(), req.originalUrl, async function (Handler, state) {
to
ReactDOM.render(app.getComponent(), req.originalUrl, async function (Handler, state) {
I am getting this error: Target container is not a DOM element.


Two button collapse if we change Language  to اردو

Expected behavior
The Search option and the the other button for opening  slide options shoud work independently without any issu of collapsing.
There should  be a special arrangement for these options in اردو  selection in Language.
Actual behavior
When i change the Language  to اردو then two buttons of option and search option are collapsed and hard to press the main option button instead to search option.
How to reproduce

Download Esteem App Version 1.5.1
Change Language  from setting  to اردو

Change English to اردو

Collapsed Buttons

Operating system:Android Nougat 7.0
Esteem: Version 1.5.1

Posted on Utopian.io -  Rewarding Open Source Contributors

version 1.5.2 resolves this!


Feature recommendation: Going back with backscape/alt+left_key after going to source(ctrl click)

I think it will be make so many people's life easier when working with a lot of files.

And what if I want in file delete something, backspace will not work?
And you have tabs, if you close tab opened by CTRL+Click, you will be back on your file.

Let's track in #27112


[NEW]: Glossary addition request ("sats")

Date
2023-09-26
Description
Can we add a glossary entry for "sats". I don't actually know what this means but see it on r/buttcoin all the time.
Links
n/a
Agreement

[X] I irrevocably agree to release my contribution under the CC-BY 3.0 License

Attribution name
No response
Personal link
No response

https://web3isgoinggreat.com/glossary#sats


change neck joints to revolute and fix some typos

The following patch is an uncommitted change on herb2, which I think corresponds to setting the neck tilt to -30 degrees instead of -40 degrees. @yskim041, can you confirm and bundle this change into this PR?
diff --git a/robots/herb_base.urdf.xacro b/robots/herb_base.urdf.xacro
index 33ac0c7..5d22a87 100644
--- a/robots/herb_base.urdf.xacro
+++ b/robots/herb_base.urdf.xacro
@@ -233,7 +233,7 @@
           velocity="4.18879" />
       <origin
           xyz="0 0 -0.0919"
-          rpy="-0.698132 0 0" />
+          rpy="-0.523599 0 0" />
       <parent
           link="neck_pan" />
       <child

@jslee02 @yskim041  Is this PR still work in progress?

Bump

@brianhou: Ready for review. This allows for control of the neck.


Add php 7.2 support

Cast $this-ips as array before to call count method

@spike31 The failing tests are actually not related to this change but to new deprecations in Symfony 3.4. Can you please consider merging this? Currently the bundle is not compatible with PHP 7.2.

Thank you. 👍

Hi @leofeyer

The failing tests are actually not related to this change but to new deprecations in Symfony 3.4.

It's actually a deprecation of php 7.2 and it will be great to find what causes it.

@aivus I think this is related to phpunit. Phpunit 5 is not still compatible with PHP 7.2
https://phpunit.de/

My previous comment was for the failling tests


Posh-Git prompt is gone

System Details

posh-git version/path: 1.0.0 beta3 ~\Documents\WindowsPowerShell\Modules\posh-git\1.0.0
PowerShell version: 5.1.18362.145
git version 2.25.0.windows.1
OS: Microsoft Windows NT 10.0.18362.0

Issue Description
Posh-Git was working great for months.  Then, it suddenly stopped working; there is no posh-git prompt anymore.  It's like it's not installed.  I see no errors.  I can even see values in $global:GitPromptSettings
I have tried uninstalling, reinstalling, upgrading (I was on 0.7.3 for months and the problem came about before I upgrade to 1.0.0).
I have installed the following in the past few days, any of which could be the culprit, just based on the timing:

SnapComms App (forced installation by my IT)
Postman-win64-7.17.0
DB Browser for SQLite
MS Visual C++ 2013 Redistributable (a dependency installed by Chocolatey)
Node.js
MS Visual Studio Code
Angular CLI

Can you confirm that posh-git has been imported?  Run Get-Module posh-git and post the output here.  If that shows posh-git is loaded, it is likely then that some module is either defining a prompt function before posh-git is imported - in which case posh-git will not overwrite what appears to be the user's custom prompt function.  Or another module is imported after posh-git and it overwrites the prompt function.  We might be able to tell if you dump the prompt function def like so: Get-Command prompt | % Definition.

Ok, I fixed it but I don't really know how/why.  Occasionally, my IT organization will remove my domain account from the local Administrators group.  It's really nefarious because everything looks the same.  I can still run programs elevated and there doesn't appear to be any restriction.  I run Powershell and Visual Studio elevated and the title bar still says "Administrator."  But weird things start happening, like this, which is how I (slowly) begin to realize it's happened.  So, I switch over to another local admin account, re-add my domain account to the Administrators group, and sign back in on my domain account.  Everything then returns to normal.  My posh-git prompt is back.
ps - Thank you, @rkeithhill, for your quick response.  I did run the commands you mentioned before I posted my question.  I confirmed that the module was loaded and the prompt was coming from posh-git, which was why this was such a weird mystery.  Here is the output (same now as it was before I made the Administrators fix):
Output of Get-Module posh-git:
Script     1.0.0      posh-git                            {Add-PoshGitToProfile, Expand-GitCommand, Format-GitBranch...
Output of Get-Command Prompt | % Description:
$settings = $global:GitPromptSettings
if (!$settings) {
    return "<`$GitPromptSettings not found> "
}
if ($settings.DefaultPromptEnableTiming) {
    $sw = [System.Diagnostics.Stopwatch]::StartNew()
}

$origLastExitCode = $global:LASTEXITCODE

if ($settings.SetEnvColumns) {
    # Set COLUMNS so git knows how wide the terminal is
    $Env:COLUMNS = $Host.UI.RawUI.WindowSize.Width
}

# Construct/write the prompt text
$prompt = ''

# Write default prompt prefix
$prompt += Write-Prompt $settings.DefaultPromptPrefix.Expand()

# Get the current path - formatted correctly
$promptPath = $settings.DefaultPromptPath.Expand()

# Write the path and Git status summary information
if ($settings.DefaultPromptWriteStatusFirst) {
    $prompt += Write-VcsStatus
    $prompt += Write-Prompt $promptPath
}
else {
    $prompt += Write-Prompt $promptPath
    $prompt += Write-VcsStatus
}

# Write default prompt before suffix text
$prompt += Write-Prompt $settings.DefaultPromptBeforeSuffix.Expand()

# If stopped in the debugger, the prompt needs to indicate that by writing default propmt debug
if ((Test-Path Variable:/PSDebugContext) -or [runspace]::DefaultRunspace.Debugger.InBreakpoint) {
    $prompt += Write-Prompt $settings.DefaultPromptDebug.Expand()
}

# Get the prompt suffix text
$promptSuffix = $settings.DefaultPromptSuffix.Expand()

# When using Write-Host, we return a single space from this function to prevent PowerShell from displaying "PS>"
# So to avoid two spaces at the end of the suffix, remove one here if it exists
if (!$settings.AnsiConsole -and $promptSuffix.Text.EndsWith(' ')) {
    $promptSuffix.Text = $promptSuffix.Text.Substring(0, $promptSuffix.Text.Length - 1)
}

# This has to be *after* the call to Write-VcsStatus, which populates $global:GitStatus
Set-WindowTitle $global:GitStatus $IsAdmin

# If prompt timing enabled, write elapsed milliseconds
if ($settings.DefaultPromptEnableTiming) {
    $timingInfo = [PoshGitTextSpan]::new($settings.DefaultPromptTimingFormat)
    $sw.Stop()
    $timingInfo.Text = $timingInfo.Text -f $sw.ElapsedMilliseconds
    $prompt += Write-Prompt $timingInfo
}

$prompt += Write-Prompt $promptSuffix

# When using Write-Host, return at least a space to avoid "PS>" being unexpectedly displayed
if (!$settings.AnsiConsole) {
    $prompt += " "
}
else {
    # If using ANSI, set this global to help debug ANSI issues
    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseDeclaredVarsMoreThanAssigments', '')]
    $global:PoshGitLastPrompt = EscapeAnsiString $prompt
}

$global:LASTEXITCODE = $origLastExitCode
$prompt

Glad you got it figured out.  Yeah, your output indicates that it should be displaying the posh-git prompt.  OK to close this issue then?

Hi, I have the same issue and I even know what caused it, but I can't figure out how to fix it. Can someone help me?
My system is Windows 10, posh-git was working fine until I initialized conda (python package distribution software provided by Anaconda) via conda init powershell, which made some modifications to the system. Now the git status info is not displayed in the prompt. Running Get-Command prompt gives:
CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Function        prompt                                             0.0        Conda

I guess, prompt settings were overridden by conda. Is there a chance to fix it and make both posh-git and conda prompts work together? I decided to first ask here hoping that there is a quick fix. If not, I'll open an issue in the conda repo.

@evfro This was just reported the other day for conda. See https://github.com/dahlbyk/posh-git/issues/765 for fix.

@evfro The main issue you are facing here is because Conda will initialize first so that it can activate your environments. This makes some changes to the PowerShell Prompt. post-git will not make any changes if it sees that the Prompt is already customized. So to solve this, open your PowerShell $PROFILE and in that add Import-Module posh-git at the top of the file. Moreover, any other posh-git customization you may have (like colors or date) add them all after the import module line but before the Conda code. So, just add all you posh git stuff before Conda does anything to the Prompt.

@dahlbyk @arafat-ar13 thank you very much for the help and explanations! It's all clear now. I have fixed the problem. Actually, I prefer not to have conda initialized by default, so I removed it from $PROFILE. I then created a special profile dedicated to my conda environment in the Windows Terminal. The settings look like:
"commandline" : "pwsh.exe -ExecutionPolicy ByPass -NoExit -Command (C:/Users/user/Anaconda3/Scripts/conda.exe shell.powershell hook) | Out-String | Invoke-Expression ; (conda activate my_env)",

When I need it I just open a new tab in the terminal using this profile. This way I have more control over which environmental variables are available.


chore(main): release 4.0.2

:robot: I have created a release beep boop
4.0.2 (2024-10-09)
Bug Fixes

groups id length increase (1018cc8)
wrong help message (675af79)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v4.0.2
:sunflower:


Disallow naming a worker as default

Description:
Currently naming an explicit worker as default is allowed. This can be ambiguous as we also refer to the implicit worker as default. We need to disallow explicitly naming a worker as default.
Suggested Labels (optional):
Type/Improvement, Component/BVM
Suggested Assignees (optional):
@lankavitharana

PR https://github.com/ballerina-platform/ballerina-lang/pull/12776 merged, hence closing


enroot-mount: failed to mount   Permission denied

[root@node0 pytorch]# srun --mpi=none  --container-image=./mlperf-nvidia+object_detection.sqsh --container-name=object_detection  --container-mounts=/home/wjy/nv-maskrcnn/dataset_dir/coco2017:/data  ls /data
slurmstepd: error: pyxis: container start failed with error code: 1
slurmstepd: error: pyxis: printing enroot log file:
slurmstepd: error: pyxis:     enroot-mount: failed to mount: /home/wjy/nv-maskrcnn/dataset_dir/coco2017 at /tmp/enroot-data/user-0/pyxis_object_detection/data: Permission denied
slurmstepd: error: pyxis: couldn't start container
slurmstepd: error: spank: required plugin spank_pyxis.so: task_init() failed with rc=-1
slurmstepd: error: Failed to invoke spank plugin stack
srun: error: node0: task 0: Exited with exit code 1

Are you running as root? This looks like NFS root squashing. I suppose you can't do /home/wjy/nv-maskrcnn/dataset_dir/coco2017 as root either.

yes，root user.  change to root directory, there is no permission problem.  But  why is there  permission problem,  root is the highest privilege


Update to rails 6

Update publishing-api to rails 6.
See individual commit messages for more detail.
Trello: https://trello.com/c/6hnEg6wT/1820-5-upgrade-publishing-api-to-rails-6
Major note is that this work is somewhat unfinished, some restructuring needs to be done
for Zeitwerk to work correctly. This will be written into a separate card.

What's in progress with the "WIP Review new framework defaults" commit?


Build failed:  use of unstable library feature "drain"  on matrix.rs

build failed in rust v1.4. use of unstable library feature "drain"
in :: src\linalg\matrix.rs:1059
h_full_data.extend(holder_data.drain(..m-i));

This is because you need to update to the latest stable rust build: 1.6.0. It came out yesterday :)

ok :)


Update coursier to 2.1.0-M5

Updates io.get-coursier:coursier from 2.1.0-M2 to 2.1.0-M5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.get-coursier", artifactId = "coursier" } ]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Closing in favor of https://github.com/com-lihaoyi/mill/pull/1753


Adding additional Property support to ImageryLayer

Related #224
Related #66
While building the ImageryLayer (PR#224), there were a number of properties that are used in the Imagerylayer that are not simple values. (See
Some of these properties have already implemented in prior gissues/PRs.  There are a number of properties that still need to be implemented to clean up the imagery layer support instead of using the current copyValues.
Per feedback from PR #224
The following properties probably don't belong in this copyValues and need to be dealt with individually:
pixelFilter: https://developers.arcgis.com/javascript/latest/api-reference/esri-layers-ImageryLayer.html#pixelFilter
capabilities
customParameters
effect
fields
fieldsIndex
mosaicRule
multidimensionalInfo
rasterFunction
rasterFunctionInfos
serviceRasterInfo
sourceJSON
timeExtent
timeInfo
timeOffset
This gissue is a placeholder for now.  It will get edited as additional items are added to the imageryLayer implementation.

Some of these were added with #313


Pact fails to rerun a request with multipart form data

My tests generate a PACT file with multipart-form data. When I run the Pact file at provider end it fails saying "Stream ended unexpectedly".
If I send exactly same request as that of a PACT file by creating my own HTTP client, it works great. It seems something is messing up the request. Mostly I feel its the this code
Here is my request from recorded interaction:
"request": {
"method": "POST",
"path": "/some/path",
"headers": {
"Content-Type": "multipart/form-data; boundary=----WebKitFormBoundary"
},
"body": "------WebKitFormBoundary\nContent-Disposition: form-data; name="file"; filename="somemodel.txt"\nContent-Type: application/octet-stream\nContent-Transfer-Encoding: binary\n\nsome model\n------WebKitFormBoundary--"
}
Http Client request generated with exactly same body , works fine

There is kind of a deadlock. In above example if I manually edit request body and replace all "\n" with "\r\n" then provider test works.
To have "\r\n" in pact file, I need to provide it in PactDslJsonBody. If i provide body with "\r\n" it fails during consumer pact verification, reason being, by the time request hits mock server, all the "\r" are stripped off from request body and just "\n" exist (I am not sure how that happens)
Bottom line is provider test doesn't work without "\r" where ideally it should , and consumer pact verification doesn't work with "\r\n"

Finally found out the issue after 3 days of intensive debugging. This line of code was the culprit which was replacing "\r\n" with just "\n"
https://github.com/DiUS/pact-jvm/blob/master/pact-jvm-consumer/src/main/scala/au/com/dius/pact/model/unfiltered/Conversions.scala#L71


Bluetooth auto trust not working

bluetooth is discoverable, but when i need to connect a new device i have to open bluetoothctl and trust the new device, otherwise it disconnects after a few seconds. Running Raspbian 9. Tested on a raspberry pi b+, 3b, and 3b+ with the same outcome, any fix?

Having the same issue. I had to manually turn on bluetooth and make it advertise, then add my device as trusted in order to pair... :/

There are two instances of /usr/bin/hciconfig in /etc/systemd/system/bluetooth-agent.service. I changed those to /bin/hciconfig and that fixed this problem for me
sed -i s#/usr/bin/hciconfig#/bin/hciconfig# /etc/systemd/system/bluetooth-agent.service

that almost fixed it, I ended up just using hciconfig instead of bin/hciconfig to get it working

This should be fixed by #24


Spam error message when creating issues/comments is not handled properly

The problem lies in error.rb:62 where the handle_message() method expects a multi-level hash/array construct whereas the spam error comes in as {"error"=>"Spam detected"}. The error message is:
NoMethodError: undefined method `join' for "Spam detected":String
    gitlab (4.6.1) lib/gitlab/error.rb:63:in `block in handle_message'
    gitlab (4.6.1) lib/gitlab/error.rb:62:in `map'
    gitlab (4.6.1) lib/gitlab/error.rb:62:in `handle_message'
    gitlab (4.6.1) lib/gitlab/error.rb:46:in `build_error_message'
    gitlab (4.6.1) lib/gitlab/error.rb:20:in `initialize'
    gitlab (4.6.1) lib/gitlab/request.rb:66:in `exception'
    gitlab (4.6.1) lib/gitlab/request.rb:66:in `raise'
    gitlab (4.6.1) lib/gitlab/request.rb:66:in `validate'
    gitlab (4.6.1) lib/gitlab/request.rb:46:in `block (2 levels) in <class:Request>'
    gitlab (4.6.1) lib/gitlab/client/issues.rb:56:in `create_issue'

In case you wonder how I found this problem: I'm testing our own API layer against Gitlab.com. I probably need to rethink this strategy and need to test it against a separate Gitlab instance :)
I'd be willing to provide a PR.

I'd be willing to provide a PR.

Please do.

Fixed in version 4.7.0


[WIP] for having notification (Slack & Twilio)

I open this PR in order to get feedbacks about the Notifier Interface, Notifier Abstract class and the Observer. Not to have feedbacks about CS or the lack of exceptions ;).
I'm not used to work with Laravel so I'm not sure that I'm doing it right

Can you apply the StyleCI diff please :)

I'll re-open this PR later.


[bug] tauri android init is failed

Describe the bug
$npm create tauri-app -- --alpha
(v3.4.0)
->mobile
$cd XXXXX(project dir)
$npm install
$npm run tauri android init

<EMAIL_ADDRESS>tauri
tauri android init
Error tauri.conf.json error on tauri: Additional properties are not allowed ('allowlist', 'updater' were unexpected)

tauri/tooling/cli
/schema.json
line.462
"additionalProperties": false
is reason?
Reproduction
No response
Expected behavior
No response
Platform and versions
sorry npm run tauri info
is error too.

Stack trace
No response
Additional context
No response

This should be solved by https://github.com/tauri-apps/create-tauri-app/pull/424 , the latest release of tauri alpha brought some breaking changes, thanks for your patience. For now, in your Cargo.toml you can change tauri version to 2.0.0-alpha.8and tauri-build version to 2.0.0-alpha.4 and it should work

thank you amrbashir.
I tried your advise.
but another error occred in not only<EMAIL_ADDRESS>and<EMAIL_ADDRESS>but also<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I give up to use 2.0.0-alpha.
I use<EMAIL_ADDRESS>and<EMAIL_ADDRESS>waiting for v2.0 become stable


Missing a tiny bit of documentation in dart_api.h

This issue was originally filed by<EMAIL_ADDRESS>
I was reading dart_api.h, and I noticed there was a tiny bit of documentation missing. The shutdown parameter is not documented below:
/**
 * Initializes the VM.
 *
 * \param create A function to be called during isolate creation.
 *   See Dart_IsolateCreateCallback.
 * \param interrupt A function to be called when an isolate is interrupted.
 *   See Dart_IsolateInterruptCallback.
 *
 * \return True if initialization is successful.
 */
DART_EXPORT bool Dart_Initialize(Dart_IsolateCreateCallback create,
                                 Dart_IsolateInterruptCallback interrupt,
                                 Dart_IsolateShutdownCallback shutdown);
It's not really a big deal, but I figured I should file a bug just to prove that someone had read all that lovely documentation ;)

This is now fixed.
https://github.com/dart-lang/sdk/blob/master/runtime/include/dart_api.h#L876..L877


Unable to use generator command

Before opening a new issue, please take a moment to review our community guidelines to make the contribution process easy and effective for everyone involved.
Description
Unable to use generator command
Steps to reproduce
Unable to use generator command

Steps
run: yarn run generate component or yarn run generate container
below error appears

Expected behavior
Should run and generate our component / container
Screenshots
If applicable, add screenshots to help explain your problem.
Versions

react-boilerplate-cra-template: 1.0.2
Node/NPM: v14.15.0
Browser: none

See my answer -> https://github.com/plopjs/node-plop/issues/183
Also fixed in the new release


Fixed issue with kitebot sanitising mouser links

As far as I'm aware, mouser never adds tracking data to URLs (?) and this also fixes the issue with kitebot breaking mouser URLs (#93). I've tested this locally using docker, but it should be verified to see that it doesn't break anything else somehow.

seems legit :trollface:

I am kinda busy right now, maybe @diminDDL can look if this works :trollface:


Fix copy_from_slice panic in read_blocking because of different lengths

When reading from the ring buffer I always quickly get a panic at https://github.com/klingtnet/rb/blob/master/src/lib.rs#L425
thread '<unnamed>' panicked at 'destination and source slices have different lengths', /checkout/src/libcore/slice/mod.rs:669:8

We don't always write to the end of data but rather we write cnt bytes, which may be less than data.len()

Thank you for this fix! Can you please also add a test case that triggers the panic?
I also think that this fix can be applied to some other functions, e.g. read because they basically have the same copy logic.

@klingtnet Fixed the other methods and added tests!

Looks fine to me and—again—thank you for your contribution!


callsArgOn and callsArgOnWith

The idea is that you can pass a context to the callsArgOn functions so you can test such constructs:
Y.later(1000, this, function () {
    this.doSomething(someArgument);
 });

I am not sure if it is useful on yield as well, but would be happy to implement it there as well.

Cool, this is definitely useful. I would be nice to include the yield methods too, but how would we name the yieldTo one? yieldToOn? :)

Haha in fact that was one of the main reasons why I didn't do it right away ;) yieldToOn sounds stoopid, but is probably the most logical way in terms of consistency… I will not have time today but I think I will find some time tomorrow to do the implementation.

Haha, yeah... At least yieldOn sounds decent. We'll take yieldToOn too, even if the name does kinda stink. At least it's consistent :)

Any news? Also, see this: https://github.com/cjohansen/Sinon.JS/pull/78

Finally found time to do this. I looked at #78 which is basically doing the same thing but I think the form was not perfect so I did it my way ;)
For documentation check https://github.com/cjohansen/sinon-web/pull/6

Added another commit which cleans up some stuff that jumped me while working in the stub tests, hope that's ok.

Thanks!


Value on select lists replaced by label

I have a select list like this:
<select (change)="onChange($event)">
 <option *ngFor="let event of (events$ | async | orderBy:'id') " [selected]="event.id === (currentEvent$ | async)" [value]="event.id">{{event.name}}</option>
</select>
As a value I assign just ids like 1,2,3, etc.
As label I assign name of events like "Great Summer 2018" etc.
In the TS I have nothing more than just regular NGRX selectors:
events$ = this.store.select(getAllEventEntities);
currentEvent$ = this.store.select(getCurrentEventId);
As long as I do not have NgrxFormsModule imported to my module everything works fine. I loaded the module to one of sub modules, not a core module.
After the import for some reasons value on my list are replaced by... labels.
In html seems to be fine, BOS 2018
however the onChange event return the label in the event.target.value:
 public onChange(event): void {     const newVal = event.target.value; //label instead of value
I tested version 2.3.2 and 2.3.1 with the same result
my package json:
 "dependencies": {     "@angular-devkit/schematics": "0.4.3",     "@angular/animations": "5.2.7",     "@angular/common": "5.2.7",     "@angular/compiler": "5.2.7",     "@angular/compiler-cli": "5.2.7",     "@angular/core": "5.2.7",     "@angular/forms": "5.2.7",     "@angular/http": "5.2.7",     "@angular/platform-browser": "5.2.7",     "@angular/platform-browser-dynamic": "5.2.7",     "@angular/platform-server": "5.2.7",     "@angular/router": "5.2.7",     "@aspnet/signalr": "1.0.0-preview1-update1",     "@aspnet/signalr-protocol-msgpack": "1.0.0-preview1-update1",     "@nguniversal/module-map-ngfactory-loader": "5.0.0-beta.6",     "angular-oauth2-oidc": "3.1.4",     "angular2-jwt": "0.2.3",     "aspnet-prerendering": "3.0.1",     "core-js": "2.5.3",     "font-awesome": "4.7.0",     "preboot": "6.0.0-beta.3",     "rxjs": "5.5.6",     "zone.js": "0.8.20",     "@ngrx/effects": "5.2.0",     "@ngrx/router-store": "5.2.0",     "@ngrx/store": "5.2.0",     "@ngrx/entity": "5.2.0",     "ngrx-store-freeze": "0.2.1",     "@ngx-translate/core": "9.1.1",     "ngrx-store-localstorage": "^0.3.0",     "@angular/material": "5.1.1",     "@angular/cdk": "5.1.1",     "lodash-es": "4.17.4"   },   "devDependencies": {     "@angular/cli": "1.7.4",     "@angular/compiler-cli": "5.2.7",     "@angular/language-service": "5.2.7",     "@angular/service-worker": "5.2.7",     "@compodoc/compodoc": "1.0.8",     "@types/jasmine": "2.8.6",     "@types/jasminewd2": "2.0.3",     "@types/node": "9.4.6",     "codelyzer": "4.2.1",     "jasmine-core": "2.8.0",     "jasmine-spec-reporter": "4.2.1",     "karma": "2.0.0",     "karma-chrome-launcher": "2.2.0",     "karma-coverage-istanbul-reporter": "1.4.1",     "karma-jasmine": "1.1.1",     "karma-jasmine-html-reporter": "0.2.2",     "protractor": "5.3.0",     "ts-node": "5.0.1",     "tslint": "5.9.1",     "typescript": "2.6.2",     "webpack-bundle-analyzer": "2.11.1",     "@ngrx/store-devtools": "^4.1.1"

Hmmm, yes, I see what is causing this. The default behaviour of Angular is that if an attribute is bound without a directive that has an @Input defined on that attribute it will simply write its value to the DOM. In your case that means the event's id is correctly written to the value attribute of the option. However, ngrx-forms adds a directive to the option element which has an @Input() binding for value. The binding is specifically written in a way that it does not modify the DOM element if the option is not connected to a ngrx-forms control state. However, what I did not realize is that Angular then stops setting the value as it usually does. Therefore the value attribute will not be in the DOM at all and the browser handles these cases by using the option's content as the value which is why you are seeing your event's label being used as the value.
The fix would be to provide another directive that restores the default behaviour of Angular in case the option is not used in the context of ngrx-forms. However, it is still not a perfect solution since this directive might in turn conflict with other directives that modify the value. However, looking at Angular's source code they are doing something similar. Therefore, even though it is not perfectly safe this solution should be good enough in almost all of situations.
I'll try to create a hotfix package tomorrow.

I have just published version 2.3.3 which contains a fix for this bug.

Thank you so much :-)!


Reordered and improved descriptions for resources in Luxembourg

Hi,
When I created the resources, I misunderstood the property order.
This PR fixes the order and improves the description for our resources :)

Thanks @DavidMoraisFerreira 👍


Add observability support for the existing Vector Stores

Related to https://github.com/spring-projects/spring-ai/issues/1243

Thanks for the suggestions @ThomasVitale
Hopefully last commit address them all.

@tzolov super! It's so great to get observability instrumentation for all vector stores!

merged in 93fa2bf45ad37d0455b3f0d3a11b211f839f454f


Browserify compatibilty error

Hi, i'm try it to use wcjs-player on the browser with browserify, is it possible?
I'm getting this error:
browserify main.js -o bundle.js
Error: Cannot find module 'remote' from '/home/diremar/webtorrentplayer/webstorm-bootstrap/node_modules/wcjs-player'
at /usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:46:17
at process (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:173:43)
at ondir (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:188:17)
at load (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:69:43)
at onex (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:92:31)
at /usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:22:47
at Object.oncomplete (fs.js:108:15)

don't sure it's possible use binary .node modules in regular browsers.

I've used WebTorrent modules on the browser with browserify, but it's pure javascript.

WebChimera.js is binary mode - so it can't be used in regular browsers. You could use WebChimera Plugin but it's support will be dropped at Chrome at September (after full NPAPI removing).

Ok, thanks for your answer.
Regards


[dv,pwrmgr,top] update pwrmgr_deep_sleep_all_wake_ups test

Previously this test only covers wakeup source 1,2, and 3.
and 4 and 5 are covered by other tests.
As we move forward to create two different sleep mode and
randomize them, it would be beeter to have on test to cover
all wakeup sources.
So I added wakeup source 4 and 5 to this test

Signed-off-by: Jaedon Kim<EMAIL_ADDRESS>

ind refactoring all the comm

done


Fix typo in comment

https://www.languageediting.com/how-to-choose-the-indefinite-article-a-or-an-before-acronyms


Add discussion of pytest thread safety

Pytest isn't thread safe and it can be confusing if people expect it to be. We should document this.
Also similarly, the warnings module as a whole isn't thread safe. Temporary directories with named files or directories also have thread safety issues since they're using global state via the file system.

Non-exhaustive list of pytest tools that are inherently not thread-safe:

the capsys fixture usage (shared sys.stdout/sys.stderr);
the monkeypatch fixture (and another monkeypatching tool, e.g. from unittest.mock);
pytest.warns (because the warnings module is not safe by itself).

Usage of the tmpdir and tmp_path fixture is currently not thread-safe. However, they could probably be thread safe if the temporary folder they create for each test would be created and deleted within the same thread as the thread used to run them.


[DeepLearning/Ops/DepthwiseConv2DNhwcHwcOp,Conv2DNhwcFhwcOp,BatchMatMulOp,Conv2DNhwcFhwcInt32Op,BatchMatMulInt32Op,MatMulInt32Op] Update BatchMatMulOp DepthwiseConv2DNhwcHwcOp and Conv2DNhwcFhwcOp the tile+vector benchmark. Update Conv2DNhwcFhwcInt32Op,BatchMatMulInt32Op and MatMulInt32Op for RVV benchmark.

for input matrix size [3,512,512,512,i32],the comparison:
Benchmark Time CPU Iterations
DL_OPS_BATCH_MATMUL/Scalar 27630 ms 27586 ms 1
DL_OPS_BATCH_MATMUL/AutoVectorization 17898 ms 17896 ms 1
DL_OPS_BATCH_MATMUL/Vectorization 3795 ms 3795 ms 1
DL_OPS_BATCH_MATMUL/RvvVectorization 1255 ms 1254 ms 1
for input matrix size [64,3136,i32], [3136,576,i32], the comparison:
Benchmark Time CPU Iterations
DL_OPS_MATMUL/Scalar 6750 ms 6748 ms 1
DL_OPS_MATMUL/RvvVectorization 378 ms 378 ms 2

Please update benchmarks/DeepLearning/README.md. Also add performance comparison of conv with explanation why our method runs faster in the description of this PR.

I got the following error when running ninja in the build file, could you reproduce it in your local machine?


Make absent() checks aware of binary expressions

Codecov Report

Merging #211 (f08676c) into main (40b7cf8) will increase coverage by 0.36%.
The diff coverage is 90.32%.

@@            Coverage Diff             @@
##             main     #211      +/-   ##
==========================================
+ Coverage   94.80%   95.17%   +0.36%
==========================================
  Files          65       65
  Lines        4617     4618       +1
==========================================
+ Hits         4377     4395      +18
+ Misses        187      172      -15
+ Partials       53       51       -2

Impacted Files
Coverage Δ

internal/parser/utils/absent.go
93.93% <88.23%> (+17.74%)
:arrow_up:

internal/checks/alerts_template.go
96.72% <92.85%> (+3.98%)
: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 40b7cf8...f08676c. Read the comment docs.


Docs: Make sure README has links for slack and meetings

why is the last commit appeared in https://github.com/AcademySoftwareFoundation/OpenImageIO/commits/v<IP_ADDRESS> release branch?
it broke source hash check, e.g. in vcpkg

@kikaxa I don't understand, why would it be a problem to backport this README change to a release?

I am sorry, it isnt really. This commit happened before the breakage but at a similar time.
After further investigating we found the source hash change was caused by this repo migration, not the commit


fix: chatgo extension - Worker terminated due to reaching memory limit: JS heap out of memory

Description

fix issues #7453
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi! Any updates on my comment? Anything I can help you with?

Hi! Any updates on my comment? Anything I can help you with?你好！我的评论有更新吗？有什么我可以帮你的吗？

ok,I will fix it soon thanks

You need to run prettier (or ray lint --fix in order to make it pass the tests :)


[FR] Option to use it selectively

use case: I only want to collapse a particular H1 or H2 or Quote or Code block, not all of them…

you mean you need more commands?
like:

Fold all H1
Fold all H2
Fold all H3
Fold all H4
Fold all H5
Fold all H6

Update it to 1.8.2. Now, you can fold/unfold h1~h6 & plain quoteblock by command


enforce import order

https://github.com/trivago/prettier-plugin-sort-imports

I tried to set this up previously and its still kinda buggy


Added support for input types of records configured with ignored optional properties

This PR adds the ability to respect default values for optional properties in record constructors, allowing HotChocolate to handle ignored fields in input types.

Thanks for your help!


Vickel: Watch craigsbarnes78(?:@gmail.com)?

Vickel requests the watch of the watch_keyword craigsbarnes78(?:@gmail\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

craigsbarnes78(?:@gmail\.com)? has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by Spevacus in Charcoal HQ. Reason: 'We determined that the post this targeted wasn't really spam'


notebook cell complete audio cue didn't play if kernel was selected in same execution action

enable audio cues for notebook cell completion
open a notebook without a kernel selected and run a cell
select the kernel so that the cell runs successfully
🐛 cell completes, but audio cue doesn't play

to be clear, @amunger, you were using run by line here to do this, correct? As you can see here, there's no run button when i haven't selected a kernel.

shift+enter should also trigger the cell to run. Do you have any kernels available in that screenshot? I can still see a run button without a kernel selected.

shift + enter adds a new line to the cell and no I don't have any kernels available by design, as it seems it gets auto selected.
what were your more specific repro steps for the above? I've been disabling the python extension to simulate no kernel selected then pressing run by line

I was hoping one of these would allow me to reset it such that there are no kernels, but no luck

oh figured it out- just creating a new notebook

nice, here's the issue

looks like there's a chrome / maybe chromium flag that allows you to bypass this policy:
--autoplay-policy=no-user-gesture-required


Fix #139 Where-Object

Fix syntax highlighting for Where-Object
https://github.com/SublimeText/PowerShell/issues/139
Along the line I actually fixed highlighting for all commands (application paths, etc) and even prevented highlighting where they shouldn't be highlighted.

I don't understand the test results :-1: I'll look at it again after some sleep ;-)


find/delete duplicated files

tbx file duplicate list
tbx file duplicate delete -keep /PATH/TO/KEEP -target /PATH/TO/DELETE

Closing this as won't fix


invalid literal for float()

I'm trying to train my own data on the Yolo network, but before that I have to convert the bounding boxes co-ordinates to the form it wants.
and i get the error: first it prints out all the file names and the content of the data then
1
29 45 615 251
['1\n29', '45', '615', '251\n']
1
29
(803, 624)
Traceback (most recent call last):
File "convert.py", line 96, in
print(float(xmin), float(xmax), float(ymin), float(ymax))
ValueError: invalid literal for float(): 1
29
I need help please. I don't know what to do

Hi @Aurellie
Please change your python version and run it.

@ManivannanMurugavel  I have the same issue, which python version should I use?

Hi @MuhammadAsadJaved
Please use my latest annotation tool

Hello dear,
thanks for your response,
in the tool you pre-define two classes while labeling images, however i am working on single class problem, i am detecting a screen from the given images, How can i manage it? can i remove classes from the code? @ManivannanMurugavel

Now i am using new tool, i change the classes in the Classes.txt file,     then i labeled 10 images to test it and after successfully labeling them i run process. py,     there was no error buy test.txt and train.txt files are empty.

This is not an issue. Please check the directory which contains the all .txt files.

Bro now i have 10 sample images in the Images/001 folder and 10 corresponding .txt files in the Labels/001 folder.  So how i can start training Yolo2 or Yolo3 model ?  are these images already in the Yolo formate?  How can i create train.txt and test.txt files or it do not require? @ManivannanMurugavel

Hi @MuhammadAsadJaved,
First, you copy all images and it's corresponding .txt files and paste into sample folder. Set your folder path in this line
https://github.com/ManivannanMurugavel/YOLO-Annotation-Tool/blob/master/process.py#L8
and run it.

Thank you very much bro,  This problem is resolved now.  I sent you a message on Faccebook, Please reply if it is appropriate. https://www.facebook.com/M.asad.javed.gujjar
@ManivannanMurugavel


[BUG] Failure during test_step_activation unit test

Describe the bug
When running the provided unit test cases, I am getting a failed test.
FAIL: test_step_activation (test_nn.TestNN.test_step_activation)
Traceback (most recent call last):
File "/Users/xxxxxxx/working/mlx/python/tests/test_nn.py", line 456, in test_step_activation
self.assertTrue(mx.array_equal(y, expected))
AssertionError: array(false, dtype=bool) is not true

Ran 172 tests in 10.101s
FAILED (failures=1)
To Reproduce
Include code snippet

python3 -m unittest discover python/test

Expected behavior
Expect to complete all unit tests without error.
Desktop (please complete the following information):
Apple M3 Pro

OS Version: [e.g. MacOS 14.2.1]
python version 3.11.6
PyTorch 2.2.0

Additional context
Add any other context about the problem here.

Which MLX version are you using?

I am not able to reproduce this on an M1 with 14.2. Could be an M3 thing. @jagrit06 would you mind trying that?

Actually I don't even see this test: test_nn.TestNN.test_step_activation. Looks like you might be using a pretty out of date MLX. I would recommend updating. If you still see an issue, I will reopen this.


feat: migration from httplib2 to requests library

Changes:

feat: migration from httplib2 to requests library
BREAKING CHANGE: the 'build_http_connection function of alert_actions_base.py has been deprecated. Users can use 'send_http_request' function of the same class or use 'requests.request' function.

@sfultariya-crest Can you please create an issue, similar to https://github.com/splunk/addonfactory-ta-library-python/issues/50 in this repository, so all users will know what was changed within this breaking change and how to deal with it?

@sfultariya-crest Can you please create an issue, similar to splunk/addonfactory-ta-library-python#50 in this repository, so all users will know what was changed within this breaking change and how to deal with it?

I have created issue #117. Please let me know your suggestion.

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

v6.0.0
GitHub release

Your semantic-release bot :package::rocket:


moving crowdloans hot fix to dev

Codecov Report

Merging #288 (287b1ff) into develop (7920849) will increase coverage by 1.57%.
The diff coverage is 38.19%.


traefik v2.3  Load balancing drr

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
current version  traefik v2.3.7 ,How to configure DRR load balancing algorithm in this version ；
In version 1.7, the DRR load balancing algorithm can be added to the annotations of the service： traefik.backend.loadbalancer.method=drr

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Traefik!
We dedicate the issue tracker to bug reports and feature requests only. My advanced AI has spotted that your issue might be a configuration problem or relates to something that doesn't look like a bug.
To confirm this, please join our Community Forum and reach out to us on the Traefik section.
In case I'm wrong (well, that would be embarrassing :sweat_smile:), my developers will re-open the issue and fix me!
In the meantime, you can double check Traefik's documentation.

I need help

Hello @xuexue1208
Thanks for your using Traefik and asking the question here.
However, there is no DRR (Dynamic Round Robin) algorithm implemented in Traefik v2.  You can try to use WRR (weighted round-robin) instead of DRR. Is there any specific requirement why you need to configure DRR?


Add table creation from files

This PR makes the following changes:

Adds a function to databases to upload parquets to S3
Adds templates for creating tables from external files
converts vocab study to use database upload

Checklist

[X] Consider if documentation (like in docs/) needs to be updated
[X] Consider if tests should be added
[X] Update template repo if there are changes to study configuration

also noting: file upload means that we're going to need to allow file deletion, in cases where a table needs to be recreated with a new dataset. I don't think it's critical right this second - I'm currently deferring that pending a discussion on the mrconso scope, where i might need to do this. but i'll create a issue regardless if we're ok with ignoring for now.


Ubuntu 20.04 - Issue linking -lgtest

When running make run into:
[ 33%] Linking CXX executable kafka-unit-test
/usr/bin/ld: cannot find -lgtest
/usr/bin/ld: cannot find -lgtest_main
collect2: error: ld returned 1 exit status
make[2]: *** [tests/unit/CMakeFiles/kafka-unit-test.dir/build.make:264: tests/unit/kafka-unit-test] Error 1
make[1]: *** [CMakeFiles/Makefile2:1013: tests/unit/CMakeFiles/kafka-unit-test.dir/all] Error 2
make: *** [Makefile:141: all] Error 2

Edited CmakeLists.txt manually to googletest folder:
#---------------------------
# googletest
#---------------------------
if (DEFINED ENV{GTEST_ROOT})
    set(GTEST_ROOT $ENV{GTEST_ROOT})
else ()
    set(GTEST_ROOT /usr/src/googletest/googletest)
endif ()

message(STATUS "googletest root directory: ${GTEST_ROOT}")

include_directories(SYSTEM ${GTEST_ROOT}/include)
link_directories(${GTEST_ROOT}/lib ${GTEST_ROOT}/bin)

You saw the scripts here, why not setting the env var GTEST_ROOT (to /usr/src/googletest/googletest) instead?

@kenneth-jia setting the env var does not change the error:
adam@adam-GE66-Raider-10SFS:~/dev/modern-cpp-kafka/build$ export GTEST_ROOT=/usr/src/googletest/googletest
adam@adam-GE66-Raider-10SFS:~/dev/modern-cpp-kafka/build$ cmake ..
-- librdkafka include directory: /usr/include
-- librdkafka library directory: /usr/lib/x86_64-linux-gnu/
-- pthread library: /usr/lib/x86_64-linux-gnu/libpthread.so
-- With NO clang-tidy build option
-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.71.0/BoostConfig.cmake (found version "1.71.0")
-- boost include directory: /usr/include
-- boost library directory:
-- googletest root directory: /usr/src/googletest/googletest
-- rapidjson include directory: /usr/include
-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.71.0/BoostConfig.cmake (found version "1.71.0") found components: program_options
-- boost include directory: /usr/include
-- boost library directory: /usr/lib/x86_64-linux-gnu
-- Configuring done
-- Generating done
-- Build files have been written to: /home/adam/dev/modern-cpp-kafka/build
adam@adam-GE66-Raider-10SFS:~/dev/modern-cpp-kafka/build$ make
[  2%] Linking CXX executable kafka-unit-test
/usr/bin/ld: cannot find -lgtest
/usr/bin/ld: cannot find -lgtest_main
collect2: error: ld returned 1 exit status
make[2]: *** [tests/unit/CMakeFiles/kafka-unit-test.dir/build.make:264: tests/unit/kafka-unit-test] Error 1
make[1]: *** [CMakeFiles/Makefile2:1013: tests/unit/CMakeFiles/kafka-unit-test.dir/all] Error 2
make: *** [Makefile:141: all] Error 2

It seems like something in cmake/make is off and not allowing me to build

Poor user error on my part, apologize for the hassle. I incorrectly installed gtest.


Major Reservoir Data Visual

User/job story:
When I am interested in learning about our water storage situation,
I want to obtain high-level and simple information about our reservoirs,
so I can be [concerned/relieved] about our human-made water storage.
Background: One of the data elements we want to show is major reservoir level relative to the average.
To do so, we need need to obtain the daily reservoir level for each major reservoir and its average. We then sum the levels across the major reservoirs and sum the averages. Then we take the sum of levels and divide by average to calculate the delta from average.
This data should update daily from CDEC updates. Refer to prior investigation of where to access data: https://github.com/cagov/drought.ca.gov/issues/229
When considering how to implement, err on low maintainability and robustness as a "feasibility" consideration. Consider how to build in soft fail overs (eg if data doesn't update, it uses last known good data) vs screwing up the entire data visual display). Or there isn't data for all major reservoirs? No value in the JSON might be -9999 (or blank).
Design: Figma Mock

The gray bar is the total capacity of major reservoirs
The blue bar is the current level of major reservoirs
The dotted line is the average of major reservoirs
the red XX% is the difference from average (below or above)
the subtitle of [below/above] should be driven by whether current level is less than or greater than the average.
Hover on the gray bar should show tooltip with total capacity in thousand acre-ft (TAF)
Hover on the blue bar should show tooltip with current level in thousand acre-ft (TAF)
Hover on the dotted line should show tooltip with average capacity in thousand acre-ft (TAF)
Learn more link should go to https://cdec.water.ca.gov/resapp/RescondMain

Additional Requirements:

Text elements should be able to be updated by content editors (eg subtitle and header)
Chart should update daily based on latest information available from CDEC
Consider a last known successful update date should troubleshooting be need (not visible to website visitors)
Placement of the chart should be modular (doesn't have to be changed by content editor, just a factor should we want to reorganize the layout). Similarly, we should be able to embed this chart on other pages of the drought.ca.gov site.

Data Source
https://cdec.water.ca.gov/resapp/service/res/conditions?date=2022-02-15&stationIds=SHA
Query with input date and station ID(s). If you put in a future date, it returns back the last date in which there is data.
Returns back info of interest:

storage
avg
cap

The numbers are in acre-feet so we would want to simplify it by dividing by 1,000 to arrive at thousand acre-feet (TAF).
Example output:
[{     "stationId": "SHA",     "disabled": false,     "detailed": true,     "tags": "1_154 res Daily Report",     "reservoirName": "Lake Shasta",     "drinkingWater": false,     "stationName": "SHASTA DAM  (USBR)",     "latitude": 40.718,     "longitude": -122.42,     "elevation": 1067,     "valDate": "2022-02-17 00:00:00.0",     "actDate": "2022-02-17",     "day": 17,     "month": 2,     "storage": 1681973,     "avg": 3143357,     "cap": 4552000,     "pctAvg": 53.51,     "pctCap": 36.95 }]
Major Reservoir station codes

SHA
ORO
BUL
FOL
CMN
CLE
WRS
SNL
CCH
CSI
CAS
DMV
NML
DNP
EXC
MIL
PNF

Let's review next week

@jaredwyee My one concern here is about how to handle the various on-hover interactions on mobile, since hovers can be clunky on phones. Do we still need to display that more detailed info on touchscreens, and if so, would the design look different?

@adam-little @kmbrlygl thoughts on how to handle for mobile?
Big picture, I do not think the hover-over details are critical information. They are useful "supporting" pieces of data that underscore the validity of the chart (through the precision of using real numbers and allowing someone to vet our math).
But if you had no clue about acre-feet capacity of our reservoirs, it would be totally arbitrary to the lay person.

Looking at covid charts with data notes for reference, the "hover" data note on the chart reveals if you tap on a part of a line. Given this drought data viz is not a line and is a basic bar, I think it's sufficient to show the data note upon tap and then revert to default state when you tap in negative space to the data note.
Assuming responsive resizing occurs there might not be any special handling needed for mobile. But would want to see an example before deciding that completely. Still, not a critical priority as I commented above given it's nice to have info.

From our convo with stakeholders, we will simply state the % of average vs relative comparison of above/below average.

Currently working through this on branch 268-prototype-reservoir-levels. I've followed the latest mocks in Figma. Always open to changing it as needed.

Looking good!


also build with clang

and fix a few errors clang spotted

Adding clang doubles the amount of builds and subsequent testing... but it's not that there are commits to unshield very often...


TCL file for FPGA build

Hello,
You have done great work!
Could you please share with us the TCL script used/generated from the FPGA run?
And maybe more details on building the image and FPGA run flow?
Thanks a lot,
Alaa

Hello,
Has the FPGA been able to run?,
second, how have you loaded initialized memory from verilog file?
Thanks,
Emilio

Hello,
You have done great work!
Could you please share with us the TCL script used/generated from the FPGA run?
And maybe more details on building the image and FPGA run flow?
Thanks a lot,
Alaa


Use CDN copy of workbox-sw in webpack plugin output

R: @goldhand @addyosmani @gauntface
The Webpack plugin now will use a CDN copy of workbox-sw instead of making a local copy. (As a side effect, it will stop including it in the precache manifest, which fixes #1044.)
The ergonomics around building your own local copy of a Workbox bundle via webpack, and then using that for importScripts rather than the CDN copy is still open, and tracked in both #933 and #934.

Changes Unknown when pulling 8f6b30e56f42be28c5f38341587cf8d15b18f4e3 on webpack-use-cdn into ** on v3**.


Introduce requests per LB pool metric & fix LB pool health status metric

This PR introduces the following changes:

fix: Report LB pool health properly
The cloudflare_zone_pool_health_status metric introduced with goal
to report the actual pools health status during the time, regardless
if this pool is the selected one or not. The initial implementation
uses the loadBalancingRequestsAdaptiveGroups type which returns
aggregated Load Balancing origin requests with adaptive sampling and
contains information only about the selectedPool (Name, Health,
AvgRttMs, etc), rather than for all LB pools. This leads on reporthing
back the status only of the selectedPool (which normally is always 1)
and not the overall status of all LB's pools.
According to Cloudflare's documentation about LoadBalancing GraphQL
Analytics API[1], the loadBalancingRequestsAdaptive schema can be
used to fetch analytics about the Raw Load Balancing origin requests
with adaptive sampling. These analytics exposes infromation about the
selected pool like:

Selected Pool - Name/Health/ID/Average RTT
Session Affinity - Type/Status

But it also exposes information about LB's pools and origins regardless
the selection decision, like:

Pools ID/Name/Health/Average RTT/Selected
Origins Name/FQDN/Ipv4/Ipv6/Selected

The latter metrics can be used to report the health of all pools
associated with account's LBs.
For this reason, this commit adds the required loadBalancingRequestsAdaptive
schema struct on the lbResp struct and configures it as the source for the
cloudflare_zone_pool_health_status metric.

https://developers.cloudflare.com/load-balancing/reference/load-balancing-analytics/#graphql-analytics

feat: Add poolRequestsTotal metric
This commit introduces a prometheus counter metric about the total
number of requests per pool.

Hey @martinhaus, any updates on this PR?

LGTM, sorry for the late response


Process inline code generated by another template if annotation's suffix is one of the templates in force-parse argument

This PR is based on @ranvirp work here https://github.com/krzysztofzablocki/Sourcery/pull/801
Just re-created based on his test with latest master changes

🙇


[HOLD for payment 2023-07-14] [$1000] Web - Country field does not autofill for certain addresses.

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

Action Performed:
1.Open your profile.
2.Click on "Settings".
3.Click on "Personal Details".
4.Click on "Home Address".
5.Notice that when selecting an address from the options menu, the country field is not automatically filled.
Expected Result:
The Country field should be auto filled, similar to when the user selects other addresses.
Actual Result:
The Country field is not auto filled for certain addresses.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

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

Version Number: 1.3.32-5
Reproducible in staging?: y
Reproducible in production?: y
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/43996225/93cce2c0-8f32-41f4-a7f3-450f8bab7905
https://github.com/Expensify/App/assets/43996225/490d72a4-0258-4855-b857-f17eeacafda7
Expensify/Expensify Issue URL:
Issue reported by: @usmantariq96
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687237650003769
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014d5d43c97434907d
Upwork Job ID:<PHONE_NUMBER>109485568
Last Price Increase: 2023-06-26

I can reproduce this. But it seems like it only happens when you type in the specific address bangladesh border road into the search box and then select it. I can't get this to not work for any other address (and I tried a bunch).
Part of me wonders if this has something to do with the data we're getting from the API for this address, which prevents auto-filling? Regardless, it is reproducible, so we should fix (as long as this isn't an issue with the address API that we have no control over).

@joekaufmanexpensify  This seems like address API response data issue. Adresses are fetched from /place/autocomplete/json, the response from API contains an array address_components which contains address auto fll data.
First recommendation for bangladesh only contains addressLine1 thats why counry isn't being autofilled. Maybe some entries in adress json need to be updated.

@nishancx Thanks for confirming! Is that something we can update on our end. Or just an issue related to the API we use (over which we have no control).

Hello, @joekaufmanexpensify. I think this is an issue in google place data. So we have to find a solution not to show such addresses in google place autocomplete component.

Hm, got it. If this is an issue with the google API data, then I'm not sure we should do anything. I think we'd rather have an address not auto-complete, than remove an address that exists from the search results. Curious if you agree @mollfpr ?

@joekaufmanexpensify The response from Google API only returns the type of route we use for address line 1, but we still can get the country-long name from the autocomplete data.
https://github.com/Expensify/App/blob/52a18912a42567fdc25cb6c64676516959a16781/src/components/AddressSearch/index.js#L143-L145

We can get the country name from the autocomplete fallback and get the country's short-name by searching it on CONST.ALL_COUNTRIES.

@mollfpr Got it, thanks! In that case, it sounds like this is something we can fix. Is that right? If so, then let's keep this open for proposals.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Country field does not auto-fill for certain addresses (eg. Bangladesh Border Road)
What is the root cause of that problem?
This seems like address API response data issue. Addresses are fetched from /place/autocomplete/json, the response contains an array address_components which contains address auto-fill data.
First recommendation for Bangladesh only contains addressLine1 that's why country isn't being auto-filled.
What changes do you think we should make in order to solve the problem?
We are fetching country with getAddressComponents().
And we are already fetching state and city with getPlaceAutocompleteTerms() and it also returns country which we can use as a fallback.
country fetched with getPlaceAutocompleteTerms() contains full name of country, which needs to be converted to country code in order to perform checks later down in the code. This conversion can be done by using lodash to findKey from CONST.ALL_COUNTRIES.
Now that we have fallback for country code fetched from getPlaceAutocompleteTerms(), it can be used as a fallback for country fetched from getAddressComponents() to solve this issue.

Thanks for the proposal @nishancx
It is similar to what I have in mind, and the proposal looks good!
🎀 👀 🎀 C+ reviewed!

Looks good to me!

Offer accepted.
Ill try to make a PR within 24 hours.

I have finished everything except testing with high traffic account, I have requested for my account to be converted to high traffic account in #expensify-open-source and I'm waiting for the request to go through.

I have made a draft PR, I'll update it as soon as I have a high traffic account to test with.

I have tested changes with high traffic account and it works as expected, the PR is ready for review.

@kavimuru please update the "Issue reported by:" with my GH Handle
Thanks

Not overdue. PR is in review.

Not overdue. PR is in review.

The PR is already deployed to production.

Ah, the automation didn't work because the pull request description doesn't follow the proper format.

I'm going to remove the Reviewing label like it was supposed to do after the production deploy. For the next PRs, make sure to use the format $ https://github.com/Expensify/App/issues/XXXX 🙂

Sure @francoisl, thank you.

When should I submit work in upwork right now or after 7 days regression  period?
Also, if no regressions happen, this should be eligible for bonus right? Since PR got merged within 3 days, 23 hours with weekend in between and without counting weekend it got merded within 1 day, 23 hours of accepting proposal.

Yep, we'll send an offer in upwork soon. We'll issue payment on 2023-07-14. And then this does qualify for a speed bonus. That will be added when we issue payment!

Manually creating BZ checklist, since the automation did not work here:
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
[@mollfpr ] The PR that introduced the bug has been identified. Link to the PR:
[@mollfpr ] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[@mollfpr ] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[@mollfpr ] Determine if we should create a regression test for this bug.
[@mollfpr ] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
@joekaufmanexpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

@mollfpr could you please complete your portion of the BZ checklist when you have a sec? Thanks!

@mollfpr offer sent for $1,000 (We'll issue $500 as bonus during payment)!

@usmantariq96 offer sent for $250!

@joekaufmanexpensify  offer accepted.
Thanks

Bumped BZ checklist in Slack. On track to issue payment tomorrow!

[@mollfpr ] The PR that introduced the bug has been identified. Link to the PR:
 [@mollfpr ] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

No offending PR is causing the issue. We were unaware there was a result from Google API that doesn't have a country value. So this is an improvement we made to handle that.

[@mollfpr ] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

This is a one-time bug, so the regression test should be enough.

[@mollfpr ] Determine if we should create a regression test for this bug.
 [@mollfpr ] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Sign in to the App
Go to the Home Address page from Settings > Profile > Personal Details > Home Address
Enter Bangladesh in the Address line 1 field
Select an option (In this case, Bangladesh Border Road , which was causing the issue)
Verify that the Country field is filled with Bangladesh
👍 or 👎

👍 for the regression steps (and the rest too)

Great, thanks! I updated the BZ checklist with that information. I also created a regression test issue and linked it above.

@nishancx $1,500 sent and contract ended!

@mollfpr $1,500 sent and contract ended!

@usmantariq96 $250 sent and contract ended!

Upwork job closed.

Bug is fixed, BZ checklist complete, and all payment issued. Closing as this is all set. Thank you everyone!


Context menus on Mac?

Hi Guys,
I noticed there was a new release with saving of data via a context menu.
Is this meant to be a right click menu as I am getting nothing on the Mac?
Cheers
Andy

Github going a bit strange so double, closed.


suggestion: include license URL for RSC articles

Observation for RSC articles:

publisher includes CC license info in their Crossref metadata - but instead of the canonical tag "license" they include it in a tag "assertion"
publisher includes RDFa tags on the article pages to expose the CC license URL (though not as META dc.rights in HTML header)

Suggestion: retrieve license URL from either Crossref metadata or from RDFa element on article page - and include license URL in oaDOI output

Example 1

RSC article page -> check HTML source...
Crossref metadata -> check tag "assertion"
oaDOI result ->  result for best_oa_location -> "license": "implied-oa"

Example 2

RSC article page -> check HTML source...
Crossref metadata -> check tag "assertion"
oaDOI result ->  result for best_oa_location -> "license": "implied-oa"

(FYI @ebunge)

fyi: We contacted RSC some months ago and asked them to move the license information from assertion to license_ref. According to my understanding RSC is working on that.


How to upload a file? like a images to server

You can do this in 2 ways:

Use newest version from nightly (github actions) and then add your file to websk/files and just navigate to this file in HTML
Use latest release and in Skript write:

if event-path is "file.PNG"
 return file.PNG

Or something like that... And in HTML write just `file.png|

Can you close this? Do you have any more questions?

@SkyCraft78 can you close this issue?


2.0.0 release?

Hello,
Are there plans for a 2.0.0 release?

Shall release later tonight! - with support only for Java 8

Has this already been released? If not, can it be released?


remove installation of packages in nb sync

What changes have been made
remove line doing CF-SDK installation just lock with new dependency
Verification steps

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

Just realized - without this, the Pipfile.lock isn't updated, is it? :thinking:  Didn't check whether there is some call elsewhere in the file now.


Reflecting data back to local socket

Hi,
I have written some simple code for the tunnelling:
import os
import socket

import threading
import time

SERVER_PORT = 12345

class Server:
    _sock: socket.socket
    _ssh_address: str

    def __init__(self):
        self._sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
        self._sock.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1)
        self._sock.bind(("", SERVER_PORT))
        self._sock.listen(10)

        self._ssh_address = ""
        if os.path.exists("tmp"):
            os.remove("tmp")

        print(f"Server started at {self._sock.getsockname()}")
        threading.Thread(target=self._listen).start()
        threading.Thread(target=self._setup_ssh_tunnel).start()

        while not os.path.exists("tmp"):
            time.sleep(2)
        self._ssh_address = open("tmp").read().split(" ")[-1].strip()
        print(f"SSH tunnelled url: {self._ssh_address}")

    def _setup_ssh_tunnel(self) -> None:
        os.system(f"lt port {SERVER_PORT} --subdomain my-example-subdomain-9843 > tmp")

    def _listen(self) -> None:
        try:
            while True:
                print("Waiting for connection...")
                conn, addr = self._sock.accept()
                print(f"Connection from {addr}")
                message = conn.recv(1024)
                if not message:
                    conn.close()
                    continue

                print(f"Received: {message}")
                conn.sendall(f"HTTP/1.1 200 OK\r\nContent-Type: text/plain\r\n\r\n{message}".encode())
                conn.close()

        except Exception as e:
            self._sock.close()
            print(f"Server stopped {e}")

class Client:
    _sock: socket.socket

    def __init__(self, server_ip):
        server_ip = server_ip.replace("https://", "")

        self._sock = socket.socket()
        self._sock.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1)
        self._sock.settimeout(5)
        self._sock.connect((server_ip, 80))

        ...

    def __del__(self):
        self._sock.close()

def main():
    server = Server()
    time.sleep(3)
    client = Client(server._ssh_address)

if __name__ == "__main__":
    main()

In the Client class, I can connect to port 80, send a GET request and receive the website contents, but I'd like to be able to send commands to my-example-subdomain-9843.loca.lt, which are then sent back to the internal Server._sock socket so that I can handle them internally. If I access "localhost:12345" from this machine, the Server._sock receives the message (GET request).
Is it possible to send commands to the external URL, that are then forwarded to the Server._sock socket, and if so, what might I need to change? I've attempted sending things to port 22, which begins an SSH handshake - do I need to manually complete this or is this something else? Thanks.
Edit
I have been using Wireshark and noticed that there are lots of re-transmissions happening:
This one

This one multiple times (at least 3 sets of these)

And finally this one mixed with the previous one)

This may link with:

When I start the code 1 client joins immediately, but causes these transmission errors. Is it expecting some response from the server socket on startup?

you can not use localtunnel for tcp connections .


How to change the backbone network to mobilenetv2

I found some difficulties when using the code of mobilenetv2 in the file  named backbone, it seems some params went wrong? How can I fix it?

How to do it my dear friend, can you share it?

@BruceLoveDecimal  can you pls share the changes which you have made for mobilenetv2 architecture