Corrected f_beta computation by adding METRIC_EPS in denominator

Added METRIC_EPS in the denominator of computation to avoid nan values in f_beta score.

What does this PR do?
Will give the correct f_beta score and prevent outputting nan values for multilabel classification with macro.

Fixes # (issue)
#4187
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃 :+1:

Hello @abhinavg97! Thanks for opening this PR.

In the file pytorch_lightning/metrics/classification/f_beta.py:

Line 132:121: E501 line too long (124 > 120 characters)

Do see the Hitchhiker's guide to code style

We have this function (still used in functional) that took care of this:
https://github.com/PyTorchLightning/pytorch-lightning/blob/130de22fd75253330320473d8081fd60698c3d64/pytorch_lightning/metrics/functional/reduction.py#L40-L78
We should probably reuse it in the new class api

@abhinavg97 is it correct that this happens when both recall and precision is 0 for a particular class?
Could we add a test case for this?

Yes, that's right.
Here is an example:
p = torch.Tensor([[0,0],[0,1]])
t = torch.Tensor([[0,1],[1,0]])

Yeah I think we can add a test for this. Is it okay if I add the above case as a test?

can we add a test to verify that these changes fix the actual master...?

@abhinavg97 can you add the following testcase to tests/metrics/classification/inputs and afterwards add to the corresponding parametrization of the f-beta tests:
# Generate edge multilabel edge case, where nothing matches (scores are undefined)
__temp_preds = torch.randint(high=2, size=(NUM_BATCHES, BATCH_SIZE, NUM_CLASSES))
__temp_target = abs(__temp_preds - 1)

_multilabel_inputs_no_match = Input(
    preds=__temp_preds,
    target=__temp_target
)

@abhinavg97 can you add the following testcase to tests/metrics/classification/inputs and afterwards add to the corresponding parametrization of the f-beta tests:
# Generate edge multilabel edge case, where nothing matches (scores are undefined)
__temp_preds = torch.randint(high=2, size=(NUM_BATCHES, BATCH_SIZE, NUM_CLASSES))
__temp_target = abs(__temp_preds - 1)

_multilabel_inputs_no_match = Input(
    preds=__temp_preds,
    target=__temp_target
)

Yes sure, will do. Thanks for the pointer to the input location :)


official page doesn't work

Hi, I wanted to let you know the official page doesn't show any content beside the tabs navigation..
I'm on Firefox 50, Windows 7

Hey, thanks for reporting. I'll kill that demo page because it is difficult to sync it every time with the releases.

Demo page killed 💀


Support query search in web

I noticed that search in web history page only supports

queryid in exact/whole match (must match all characters)

So, i suggest we should

support querytext search
support partial keyword match

As column filters in the UI initially
maybe also column sorting?

Good idea, but won't there be performance issues when the query log is large?

@ytwp I don't think it will be too much burder to server,
Also I just noticed that in web page, the history size used at all.
I'll fix this


Vagrant fails to start docker in all the VM

The error is:
TASK: [docker | ensure docker is running (and enable it at boot)] *************
failed: [slave1] => {"failed": true}
msg:  * Docker is managed via upstart, try using service docker start

failed: [master1] => {"failed": true}
msg:  * Docker is managed via upstart, try using service docker start

failed: [master2] => {"failed": true}
msg:  * Docker is managed via upstart, try using service docker start

failed: [master3] => {"failed": true}
msg:  * Docker is managed via upstart, try using service docker start

FATAL: all hosts have already failed -- aborting

Looks like related to https://github.com/ansible/ansible-modules-core/issues/1170

Fixed as in https://github.com/ansible/ansible/issues/10675 using the wrong ansible version the correct one is <IP_ADDRESS>


Allow numba.jit to compose with numba.stencil

Today, I think that the way to use stencil functions with numba is to include them in another function that is jitted.
@numba.stencil
def f(x):
    ...

@numba.jit
def g(x):
    return f(x)

I wonder if it might make sense instead to allow numba.jit to be directly composed with numba.stencil
@numba.jit
@numba.stencil
def f(x):
    ...

Or, going further, stencil may just have a jit=True keyword.  When I originally tried to use stencil I assumed that it would jit automatically.  I suspect that most users coming to this function for the first time have a similar expectation, used to as we are with the simple API of numba.jit.
Example

In [1]: import numba

In [2]: import numpy as np

In [3]: x = np.arange(25).reshape((5, 5))

In [4]: x
Out[4]:
array([[ 0,  1,  2,  3,  4],
       [ 5,  6,  7,  8,  9],
       [10, 11, 12, 13, 14],
       [15, 16, 17, 18, 19],
       [20, 21, 22, 23, 24]])

In [5]: @numba.stencil
   ...: def f(x):
   ...:     return (x[-1, -1] + x[-1, 0] + x[-1, 1] +
   ...:             x[ 0, -1] + x[ 0, 0] + x[ 0, 1] +
   ...:             x[ 1, -1] + x[ 1, 0] + x[ 1, 1])
   ...:

In [6]: %timeit f(x)
189 ms ± 1.24 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [7]: @numba.njit
   ...: @numba.stencil
   ...: def f(x):
   ...:     return (x[-1, -1] + x[-1, 0] + x[-1, 1] +
   ...:             x[ 0, -1] + x[ 0, 0] + x[ 0, 1] +
   ...:             x[ 1, -1] + x[ 1, 0] + x[ 1, 1])
   ...:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-7-f2d04f4f5a9e> in <module>
      1 @numba.njit
----> 2 @numba.stencil
      3 def f(x):
      4     return (x[-1, -1] + x[-1, 0] + x[-1, 1] +
      5             x[ 0, -1] + x[ 0, 0] + x[ 0, 1] +

~/miniconda/envs/dev/lib/python3.7/site-packages/numba/decorators.py in njit(*args, **kws)
    234         warnings.warn('forceobj is set for njit and is ignored', RuntimeWarning)
    235     kws.update({'nopython': True})
--> 236     return jit(*args, **kws)
    237
    238

~/miniconda/envs/dev/lib/python3.7/site-packages/numba/decorators.py in jit(signature_or_function, locals, target, cache, pipeline_class, **options)
    171                    targetoptions=options, **dispatcher_args)
    172     if pyfunc is not None:
--> 173         return wrapper(pyfunc)
    174     else:
    175         return wrapper

~/miniconda/envs/dev/lib/python3.7/site-packages/numba/decorators.py in wrapper(func)
    187         disp = dispatcher(py_func=func, locals=locals,
    188                           targetoptions=targetoptions,
--> 189                           **dispatcher_args)
    190         if cache:
    191             disp.enable_caching()

~/miniconda/envs/dev/lib/python3.7/site-packages/numba/dispatcher.py in __init__(self, py_func, locals, targetoptions, impl_kind, pipeline_class)
    542         arg_count = len(pysig.parameters)
    543         can_fallback = not targetoptions.get('nopython', False)
--> 544         _DispatcherBase.__init__(self, arg_count, py_func, pysig, can_fallback)
    545
    546         functools.update_wrapper(self, py_func)

~/miniconda/envs/dev/lib/python3.7/site-packages/numba/dispatcher.py in __init__(self, arg_count, py_func, pysig, can_fallback)
    180
    181         argnames = tuple(pysig.parameters)
--> 182         default_values = self.py_func.__defaults__ or ()
    183         defargs = tuple(OmittedArg(val) for val in default_values)
    184         try:

AttributeError: 'StencilFunc' object has no attribute '__defaults__'

In [8]: @numba.njit
   ...: def g(x):
   ...:     return f(x)
   ...:

In [9]: g(x)
Out[9]:
array([[  0,   0,   0,   0,   0],
       [  0,  54,  63,  72,   0],
       [  0,  99, 108, 117,   0],
       [  0, 144, 153, 162,   0],
       [  0,   0,   0,   0,   0]])

In [10]: %timeit g(x)
569 ns ± 27.5 ns per loop (mean ± std. dev. of 7 runs, 1000000 loops each)

It would be nice also, for this to work with other decorators like guvectorize

Thanks for the suggestion, I think that this would be a good thing to get working too. I'm fairly sure it's technically possible but may require a fair bit of unpicking of the current implementation as it's quite involved (also might be easier if/when more generic function IR inlining is supported).


ReferenceError: DevExpress is not defined

Hello. Can you help me with error:

Exception: Call to Node module failed with error: ReferenceError: DevExpress is not defined

in

node_modules\devextreme-angular2\core\dx.component.js:2:10

I add devextreme-angular2 to project with execute

npm i devextreme-angular2

In project i use webpack.
Thank you.

Here's a similar issue that might help: https://github.com/DevExpress/devextreme-angular2/issues/12

Hello.
Note that devextreme-angular2 has dependence from devextreme. Those scripts should be included to your project.
Take a look at the #12 and #47 threads for more details. Please let us know if that information does not help you.


Switch grafeas to go modules.

Specifically

Make a protodeps folder and check in our proto deps as git submodules go generate commands.
Update the protoc version, since googleapis isn't compatible with
protoc 3.3.0.
Rewrite all the proto dep paths
Remove vendor folder

Could you please pull the latest changes, resolve conflicts and push to the branch again? Happy to review and merge, since 0.1.0 is now out.

@aysylu I rebased the cl - and it's working locally. Should be good for review.

Hey @clrprod, it looks like some tests are failing, could you please TAL?

@aysylu It was a timeout while pulling modules - I've repushed to retrigger circleci and it should be fixed. In go 1.13 (which isn't released), this will go through a go proxy that likely has higher uptime than the module source.

@aysylu It passed


Create a metric that counts total labels per scrape

Proposal
If someone is running Observability as a Service and doesn't have access or knowledge of the labels per team then they can run into an issue where they are blind to the number of labels compared to the "label_limit" set in the scrape config. If there was a metric that counted the number of labels per scrape and that was exposed at every scrape then the central team could understand at a high-level where each user/team sits compared to their rate limits. This would ultimately help the observability team control cardinality and costs per team.

There is the prometheus_target_scrape_pool_exceeded_label_limits_total metric, which counts how many times the limit has been exceeded.

Added a PR which includes the metric in question. We'll see if this gets accepted...

Each time series can have a different set of labels, so all you can do really is export a single metric showing what's the maximum number of labels exposed.
I don't think that #11394 works as you expect, if I read it correctly it will set that metric to the number of labels on the last metric in each scrape.
The problem here, from my experience, is that trying this sort of accounting tends to be very expensive with Prometheus.
Plus one could make an argument that no team should be blind to the number of labels they have on metrics - it's produced by code they wrote or manage. Label values might be dynamic but the list of labels and their meaning should be fairly static. I set all limits to high values that no reasonable scrape job would ever exceed since they are mostly aimed to be the last line of defence - to stop Prometheus from ingesting metrics from buggy code that generated 100s of labels of put MBs of error blobs into label values. Any fine tuning and trying to keep those limits close to usage is often very difficult in all the environments I work with.

Hello from the bug-scrub!
There are currently three limits under prometheus_target_scrape_pool_exceeded_label_limits_total.

label_limit exceeded
label_name_length_limit exceeded
label_value_length_limit exceeded

I think you are asking for a metric showing the high-water-mark (the highest number encountered in the whole scrape, to "understand at a high-level where each user/team sits"), therefore would need one per limit.
This would be multiplied by the number of targets.
Note we have a metric scrape_sample_limit, behind feature flag extra-scrape-metrics, which can be compared to scrape_samples_scraped, so perhaps three more metrics for the actual limits.
Nobody has commented in two years, so perhaps it is not too useful, but we wanted to clarify what is under discussion.


Upgrade to 2.0.0

can this be upgraded to vuetify 2.0.0^? it has a lot of breaking changes

https://github.com/Rekryt/nuxt-vuetify-dashboard is a version of Creative Tims dashboard using vuetify 2.0.0

Hi!
There is a lot of changes in Vuetify 2. I don’t have time for this now to maintain this repository but i will in the future 😀

can this be upgraded to vuetify 2.0.0^? it has a lot of breaking changes

If you want to purpose a PR for this to help, it could be help many people to use this project.
Otherwise like Gramlin say, you can use the project of Reykryt.
Thanks a lot 😀


Keep Existing VPX filenames is ignored if updated table is in a zip/rar

Describe the bug
When using Update and Replace mode, "keep existing VPX filenames" are ignored if file is in a zip/rar.
To Reproduce
Pre-reqs:

Have a properly named table. Such as "RollerCoaster Tycoon (Stern 2002).vpx"
Have a zipped/rar'd table update that contains a file "RollerCoster Tycoon (Stern 2002) VPW 1.1.vpx" (note: this is an example)

Steps to reproduce the behavior:

Go to VPX Table upload
Select Upload and Replace, ensure all three sub-options are selected.
Click _Select File and browse for your newly downloaded table update
Select the zip/rar file which contains your table update
Click Upload
On completion, VP Studio will now give a scan error message that backglass / pup is missing. On further investigation, you will find the file name is based on what is in the zip/rar.

Expected behavior
When using the option, the file name of the VPX file should be remain same as the existing as the existing table being updated.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 11

Additional context
(Also minor text error with word "insteaf" for If enabled, the display name of the table is kept insteaf...

Thanks for reporting. Yes, I'm aware of this bug but forgot to write a ticket for this.
2.19.0 will come with a re-implementation of uploads and drag-n drop features where I will also take care of this.

Fixed with 2.20.0.


chore: add Sonarqube XML exporter

This adds an additional exporter for Sonarqube's generic coverage data XML format, as defined at https://docs.sonarqube.org/latest/analysis/generic-test/

Thanks a lot @thedavidharris, I will review it ASAP.

Thanks a lot @thedavidharris, I will review it ASAP.


opnode: Add debug tracer to geth in e2e test

Codecov Report

Merging #418 (c4098b5) into main (3a1f6ff) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #418      +/-   ##
==========================================
+ Coverage   51.68%   51.74%   +0.05%
==========================================
  Files          57       57
  Lines        5764     5767       +3
==========================================
+ Hits         2979     2984       +5
+ Misses       2415     2414       -1
+ Partials      370      369       -1

Impacted Files
Coverage Δ

opnode/test/geth.go
64.80% <100.00%> (+0.86%)
:arrow_up:

bss/sequencer/driver.go
72.05% <0.00%> (-2.21%)
:arrow_down:

opnode/node/api.go
58.64% <0.00%> (-0.76%)
:arrow_down:

opnode/rollup/driver/state.go
76.19% <0.00%> (-0.40%)
:arrow_down:

l2os/bindings/l2oo/l2_output_oracle.go
9.50% <0.00%> (+0.61%)
:arrow_up:

l2os/drivers/l2output/driver.go
69.36% <0.00%> (+2.31%)
: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 3a1f6ff...c4098b5. Read the comment docs.


Timeout Code Comment for v1alpha2 TaskRunSpec Says Default Timeout is 10 Minutes

The code comment for Timeout as part of the TaskRunSpec appears to state the default time of 1 hour incorrectly.
Expected
The default timeout is 1 hour as documented here.
Actual
The comment states the timeout default is 10 minutes.
/kind documentation

:sweat_smile:


Use CanIUse data and browserslist to enable/disable plugins

Stylecow vs. PostCSS text:

The plugin of the previous example (postcss-vmin) only makes sense if we want to add support for explorer 9. In postcss, you have to decide which plugins you have to use, according with the browser support you need (with the exception of some plugins like autoprefixer). In stylecow, the plugins can have the browser support, so, if you use the vmin plugin but your minimum suport is explorer 10, this plugin wont be executed.

If cssnext will use Can I Use as Autoprefixer we will bit this feature.

It's already possible since 1.x.
https://github.com/cssnext/cssnext#nodejs-options
https://github.com/cssnext/cssnext/blob/78832083bb0b293294853d9957e1c21e8d539a4d/index.js#L12

Awesome!
Second question: does plugins order matter in postcss-import?

postcss-import is executed before anything else, so I don't see any possible issue.


Scrum 1516

Modularized update of pubmed nlm resources to only create new resources, but not update existing ones.  This should be run before doing any new paper creation, from pubmed searches, dqm updates, pubmed paper updates, but probably not before creating one-of papers via PMID through the UI, because it would take too long.

Thanks @valearna .
@sweng66 Yeah, I think that's right.  When we process papers from pubmed, we connect them via NLM, and the primary key from the J_Medline is the NLM, so as long as we get the resources that have new NLMs, we can connect them to papers, otherwise we'd have to go back later to look at all the paper info to see if we can connect them to resource.  So this was more of a last minute thing so that references can get connected to resources.  We'd like to also update the resource information, and sort_dqm_json_resource_updates.py does that for DQM resources, but hasn't been tested that much and would have to be modified for pubmed resources.  It'll probably be easy to do, but Ceri hasn't prioritized us doing that yet.  The reference->resource connection is more complicated than just NLM, but essentially the NLM is the primary non-Alliance-key that we need to resolve to, so we just need to get those in.
PubMed doesn't keep papers after they delete them, so if they remove a PMID after we created it, we can't find it at PubMed later, so we lose the source of truth.  For that reason we're storing the original pubmed xml and later will store the latest pubmed xml (so that we have a record of what we used to create something for later debugging).  Similarly we don't know whether when they update the J_Medline.txt we'd lose source of truth, so we're keeping a copy of the latest file we used.  Possibly we should keep copies of all files processed, but that seemed like too much.   It's mostly for debugging though, if something's wrong with our data later it might be easier to look at the source file before trying to track through all the code.  I think it's a good point that it doesn't help us much here as with the paper XML since we'll be updating this file over and over.
I didn't know about BioPython when I wrote this.  I wrote the WormBase xml processing some 21 years ago, and found it easy to redo it in python instead of looking at new software.  We tried using some xmltodict at some point, but there was html markup in the xml values that were getting lost and the curators wanted to keep.  When Paulo was trying to refactor the xml processing, I think he tried to use BioPython, but he also started using pandas and other stuff that was making it more complicated, and the xml_to_json parser was already working well enough, so we're leaving it as-is, more for convenience and to avoid extra work.

@sweng66 Yeah, we'll probably take the code from the dqm one and convert it to a module that can be used by the pubmed one.  The pubmed one so far is just a light way to get new resources before creating papers.  Depending on how fast it is to update the pubmed resources, we can make that part of the before-paper-updates script.
Yeah, I think SGD has a lot more PubMed papers than WormBase.  Even now, with Olin asking us to change the s3 storage from standard to glacierIR, aws is having issues with converting so many files.  But at WormBase, sometimes (not often) something wouldn't look right with the data for a paper, and Kimberly would see that it was correct at PubMed, and we'd find that the original PubMed file has different data, and that was the issue, which was faster than trying to debug the code.
Cool cool.  If you think importing Bio later will be good for new functionality, we totally can.

OK all sound good! Thanks for the details. @azurebrd


NameError: name 'page' is not defined

File MYDIRECTORY/venv\lib\site-packages\characterai\pyasynccai.py", line 16, in goto
await page.goto(f'https://beta.character.ai/{link}')
NameError: name 'page' is not defined
This is the error when I try sending a message to the api with my code below: (not all of the code is shown below because of the other project i'm working on, a discord bot)
from characterai import PyAsyncCAI
caiToken = "CENSORED"
cclient = PyAsyncCAI(caiToken)
print(f"{char}: {msg}")
data = await cclient.chat.send_message(char, msg)

Oh btw I did await inside a function

Didn't you read the docs? You have to use the .start() function, here:
from characterai import PyAsyncCAI

caiToken = "YOUR_TOKEN"

cclient = PyAsyncCAI(caiToken)
cclient.start()
print(f"{char}: {msg}")
data = await cclient.chat.send_message(char, msg)

ok thank you!

Ok i tested it, it still says the same error, whether I add the cclient.start() or not.

That shouldn't happen, try adding page = None at the top of the PyAsyncCAI file

It gives me a different error when I add the code page = None to the file.
AttributeError: 'NoneType' object has no attribute 'goto'

Try re-installing the library, use pip uninstall characterai and then pip install characterai


Limit size of modulus for BN_mod_exp_mont_consttime()

Otherwise the powerbufLen can overflow.
Issue reported by Jiayi Lin.

Although not a security issue it is kind of hardening so we might want to consider this for 1.1.1 as well.

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

Merged to master, 3.0, and 3.1 branches. Thank you for the reviews. 1.1.1 needs tweaks so created a separate PR.


Add Bubble Sort Algorithm

Ground rules:

Whoever claims the Issue first, should send a Pull Request within 24 hours else it'll be assigned to another claimant and further Pull Requests regarding that issue will not be entertained.
Pull Requests sent without claiming and referencing the Issue will not be entertained.
Make sure your Pull Requests have just one commit in them. Squash multiple commits into one if you have to.

I want to claim this

i want to claim this

I want to claim this

I WANT TO CLAIM THIS

claiming this

I want to claim.

i claim

I want to claim this

i claim this


No module Error

Traceback (most recent call last):
File "psse_decrypt.py", line 10, in
from cryptography.hazmat.primitives.serialization import pkcs12
ModuleNotFoundError: No module named 'cryptography'

Windows 11 25281.1000
Python Version - 3.10.9
pycryptodome  - 3.17
pycryptodomex - 3.17
🐛

I just found the solution.
When I read "Requires: pycryptodome cryptography", I thought cryptography was a word, not another library.
I think it would be more user friendly if it was rephrased as:
Requires: pycryptodome (python -m pip install pycryptodome ) and Cryptography (python -m pip install cryptography).
I am sorry for the misunderstanding


SFX-159: Add Search Driver plugin

The search driver is responsible for listening for events, making search requests when those events are received, and dispatching the results through other events.

@jmbtutor Currently getting two typescript build errors when building locally. It looks like this needs to be added to the CircleCI config too.


Power BI Action: Publish error- One or more errors occured

Hi Sir,
From today only facing this error,before reports were getting deployed to destination workspace. Don't  know what is the issue.everything is same only.I am selecting single pbix file to deploy. Error is also not descriptive.Waiting for your reply as soon as possible

Closed

Hello, may I know how did you fix this issue?
Thanks.


Add "fickle" to whitelist

@jo3-l there's also something called "ficks" (https://www.ficksdrink.com/). maybe we should exclude that as well

Yes, excluding those would also be good but I'm afraid they will have to wait for the next release. You're welcome to send a PR with those additions; I will add them later this month along with the other larger queued changes if not.
(The current whitelist is created from semi-automatic review of a large corpus of English words in the scripts/ directory. It does contain fickle, but not {fick, ficks}. I wonder if there is a more comprehensive list available somewhere... something for me to look into.)

@jo3-l Can that process be made fully automatic? I imagine you can run the matcher on all words. If there is a match, you add it into the whitelist of the phrase that matched. Then you pretty-print the result back into the source.


Cannot start app

With the most recent changes to nodenet branch, the application fails start using the npm start script

Fixed the immediate problem, should find a way to clean the dist directory as a pre-commit hook to prevent path issues going unnoticed for so long

Added the hook


seabios: Update src URL

https://github.com/coreboot/coreboot.git endorses seabios.org as
the official seabios website since Stefan Reinauer's 2011 commit
f58b63d6dc30e9400002fd0e791948a6be4384ef.
(Content hash is unchanged because content is identical.)
Motivation for this change
Fix "hash mismatch" fetch error.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@chkno Please also make a P-R for the version bump to 1.13.0 and think about adding yourself as maintainer.

This was fixed in #79880.


Manual Control of Robot Arm By Human/User

Is it possible or is there some way to control the robots manually in the panda gym environment in order to capture recordings of demonstrations.
By manually I mean, instead of an agent predicting actions at every step, is it possible or is there some way for a human/user to control the robot using keyboard mappings or something like that.

Hi, @Vidharth
Yes, you can manually control the robot by passing your own action into the step() method. Here is a simple example:
import gym
import panda_gym

env = gym.make('PandaPickAndPlace-v1')
env.reset()

env.step([1, 0, 0, 0]) # Go FORWARD
env.step([-1, 0, 0, 0]) # Go BACKWARD
env.step([0, 1, 0, 0]) # Go LEFT
env.step([0, -1, 0, 0]) # Go RIGHT
env.step([0, 0, 1, 0]) # Go UP
env.step([0, 0, -1, 0]) # Go DOWN
env.step([0, 0, 0, 1]) # OPEN fingers
env.step([0, 0, 0, -1]) # CLOSE fingers

Another example, where action depends on the observation:
import gym
import panda_gym

env = gym.make('PandaPush-v1')
obs = env.reset()

# Get specific observations
ee_position = obs['observation'][0:3]
object_position = obs['observation'][7:10]

# Choose action according to object position and end-effector position
action = object_position - ee_position # move in the direction of the object

# Step
obs, reward, done, info = env.step(action)

Each envrionment has its own observation and action space. Read the following for more details.
You should be able to adapt this code to do keyboard-control, if that's what you want to do (using the keyboard module for example).


Discord unban

I got banned a while ago from the discord, because of personal issues between me and Ras last year. It's been a while now and I don't think the ban should still stand. I apologize.
DC: 하나#1111

There is no ban on our Discord with that username. Are you sure you have the correct one?

Closing due to no reply. Open a new issue if needed.


you guys have to change input-control-coding like unity

    float translation = Input.GetAxis("Vertical") * speed;
    float rotation = Input.GetAxis("Horizontal") * rotationSpeed;

like this,
look at.
how beautiful?

Godot handles axis control pretty well. Using action system enables devs to have all the inputs handled in the same way. https://godotengine.org/article/handling-axis-godot

As @novhack said, axis handling was revised in Godot 3.1, making controller support easy to implement.


[BEAM-12123] Proactively reject unsupported types in Java UDFs.

R: @apilloud

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Tests are still failing?

Tests are still failing?

Fixed


swag init  generates swagger.yaml file in reverse order

Describe the bug
swag init generates the swagger.yaml file in reverse order. The swagger.json file of the same spec is in correct order.
To Reproduce
Steps to reproduce the behavior:

Run the following:

swag init \
    --generalInfo "${some_dir}/router.go" \
    --output "${some_dir}i/docs" \
    --outputTypes json,yaml

Go to "${some_dir}" and open the  swagger.yaml and swagger.json files
See the contents of both files

Expected behavior
Both swagger.yaml and swagger.json files are in correct order.
Screenshots
swagger.yaml file in reverse order but swagger.json it's not.
For example, for the YAML file the description is last, for the JSON file the description is first and so on.
Your swag version
e.g. 1.16.3
Your go version
e.g. 1.21.6
Desktop (please complete the following information):

OS: Arch Linux
Browser: Firefox
Version: 122.0.1

Additional context
Add any other context about the problem here.

yaml file is converted from json and will lost orders as a result of using map[string]interface{} during json decoding.
if we marshal spec.Swagger directly into yaml, the generated file may be huge with many useless lines, because spec.Swagger haven't set omitempty for yaml encoding.

yaml file is converted from json and will lose orders as a result of using map[string]interface{} during json decoding. if we marshal spec.Swagger directly into yaml, the generated file may be huge with many useless lines, because spec.Swagger haven't set omitempty for yaml encoding.
go-openapi/spec has not considered to marshal into yaml, I think, refer to https://github.com/go-openapi/spec/blob/master/swagger.go

OK, so I guess this could be improved by implementing a YAML marshaller in go-openapi.
As I imagine, that's no small task. I'll consider looking at the code and see if I can do a PR there.


Marshal/Unmarshal skips newline characters CR LF

Recently we discover that after marshal/unmarshal the result string is not identical with the original one and skips new lines (CR, LF or CRLF).
There are very similar issues reported in the past (#226, #449, #959, #1030) for different versions and for some of them, it is marked as fixed. The interesting part is that in previous version that we used – 2.2.11, there was not such an issue, but in current – 2.3.0 there is a problem.
Affected Versions
[2.3.0]

@galyanikolova There were some changes to new line escape handling in 2.3.0 and later.
Bugs in question were:
https://bugs.openjdk.java.net/browse/JDK-8196491
https://bugs.openjdk.java.net/browse/JDK-8172297
https://bugs.openjdk.java.net/browse/JDK-8171195
The PR referenced is quite an old one. In order to not reintroduce any regression, lets nail down the problem first.
First, issue you are reporting is due to XML parsers normalization. It should relate only to escaping \r (carriage return), not the \n (line feed).
I have tested both 2.3.0 and 2.3.2 in most basic JAXB configuration and get no issues with CR/LF not being preserved after a JAXB marshall/unmarshall roundtrip.
https://github.com/bravehorsie/jaxb-newline-test
Please provide an example demonstrating the issue (ideally on last released version 2.3.2).

works fine with 2.3.6/3+; reproducible in "old" version included in SE 8 b311


Slider should support step values less than one

Is your feature request related to a problem? Please describe.
The Slider component currently doesn't support step values less than one.
Describe the solution you'd like
The Slider component should accept the same values that the native  element does. E.g. accept any positive number (integer/float) or "any"
Additional context
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input#attr-step

Long overdue with reply here, but thanks for requesting this feature. We'll hopefully have a look at this in the not to long future and get back to you 😅  🤞


Feature new icons

Fixes #406, #409
Changes proposed:

[x] Add
[ ] Prepare
[ ] Delete
[ ] Fix

Things I've done:

[x] I've pulled the latest master branch
[x] I've run npm install to install all the dependenies
[x] I've checked my code, it is clean.
[x] I've run ESLint.
[x] My pull request fixes an issue, I referenced the issue.
[x] My pull request is not too big, otherwise I've already squashed it.
[x] I've run npm run build to build the extension. (If I had done something with the extension.)

Simply Beautiful :)

It should be working... at least it's working for me. @jens1o can you confirm that you see the changes?
@kristianmandrup you have the icons correctly set? I mean, the file icon theme that it's selected it's vscode-icons, isn't it?

@kristianmandrup which version of VSCode are you using?

Just a moment...

Working for me.

I have no idea what may be happening. Try reinstalling the extension. Is it the extension disabled?

Works now! Turns out I had installed multiple different icon plugins and was using the Seti themes.
Now it looks beautiful :) Thanks a lot


How can I do a counter when iterating over results?

I want to output something like:
#1 10,000
#2 12,000
#3 56,000
But I have no idea how to do a counter with my loop.  Right now I have
        {% for sale in sales %}
        <p>{{sale.amount}}</p>
        {% endfor %}

{% for sale in sales %}
{{loop.index}} {{sale.amount}}
{% endfor %}


[Docs] Rest API styles and navigation improvement

Signed-off-by: Lee Calcote<EMAIL_ADDRESS>Notes for Reviewers
This PR improves the readability and navigation of the REST APIs.
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 16f6bc2f47b28541c8e8053e06a65fe75adcb9f4 at: https://65c30059eed0262d8753e54f--meshery-docs-preview.netlify.app

@senali-d @vishalvivekm @sandramsc, thoughts?

The new design looks good.

@senali-d here is an example of why the text is indented under DESCRIPTION. The same indentation has been added to all heading/sections.

From your screenshot, it looks like the parameters section needs to be indented.

🚀 Preview for commit 094a332d9b8b025d085d1e1b6a7a83f2d81bc53c at: https://65c5ad81d060d2582dd05c80--meshery-docs-preview.netlify.app

I added an indent for the rest of the description details.


Hot reload not working?

I cloned the repo (commit 1bd7ae5687548d8635ed0ca897ceba2d396adf41)
Ran npm run dev
Pointed my browser to http://localhost:3000/
I modified components (e.g. CounterButton.js) and I expected that modifications would be visible in browser right after I save the file. Server notices that files have changed and builds them but browser won't update until I reload the page.
I get this in my Chrome console:

I am doing something wrong?
(index):1 EventSource's response has a MIME type ("text/html") that is not "text/event-stream". Aborting the connection.
polling-xhr.js:232 GET http://localhost:3000/api/ws/?EIO=3&transport=polling&t=1443974856241-3&sid=ONhEJqbtod8EEPTfAAAD net::ERR_EMPTY_RESPONSE
polling-xhr.js:232 POST http://localhost:3000/api/ws/?EIO=3&transport=polling&t=1443974879508-4&sid=ONhEJqbtod8EEPTfAAAD net::ERR_CONNECTION_REFUSED
polling-xhr.js:232 GET http://localhost:3000/api/ws/?EIO=3&transport=polling&t=1443974880288-5 net::ERR_CONNECTION_REFUSED

I guess that the browser is trying to access URL http://localhost:3000/__webpack_hmr, but the router catches this url and renders a page with NotFound component.

@jsyrjala
Yea, looks like this is a problem in webpack-hot-middleware, I've made a pull request glenjamin/webpack-hot-middleware#32

Thanks @Blitz2145 . It fixes the hot reloading.

Just remove the .development() from the /bin/server.js
var WebpackIsomorphicTools = require('webpack-isomorphic-tools');
global.webpackIsomorphicTools = new WebpackIsomorphicTools(require('../webpack/webpack-isomorphic-tools'))
// .development(DEVELOPMENT)   //  <-------------------
.server(rootDir, function() {
require('../src/server');
});


bump contracts and geth

Latest contracts, latest geth! Onwards and upwards!

Coverage decreased (-16.7%) to 69.827% when pulling 7574bbafb855dd72b2980e65ccf6500e4ece299e on inomurko/bump-contracts-7c3f79-geth-1-9-11 into e127775203c0ed2e3fdd2fad3e7611898186b023 on master.


Failed to install on windows with Ruby-2.2.4 & 2.3.3

Hi,
I have been trying this for 2 days, but was never successful installing on windows 7
I tried with Ruby-2.2.4 and 2.3.3 as well.  However, I do not see any issue with Ruby-1.9.3 & taglib-ruby-0.6.0 on the same machine.
OS: Window-7-x64
Ruby:- rubyinstaller-2.3.3.exe
Ruby- DevKit-mingw64-32-4.7.2-20130224-1151-sfx.exe
c:\certification_test_suite>gem install taglib-ruby --platform ruby
Temporarily enhancing PATH to include DevKit...
Building native extensions.  This could take a while...
ERROR:  Error installing taglib-ruby:
        ERROR: Failed to build gem native extension.

    current directory: C:/Ruby23/lib/ruby/gems/2.3.0/gems/taglib-ruby-0.7.1/ext/taglib_base
C:/Ruby23/bin/ruby.exe -r ./siteconf20161221-3256-19l00zu.rb extconf.rb
checking for main() in -lstdc++... yes
checking for main() in -ltag... no
You must have taglib installed in order to use taglib-ruby.

Debian/Ubuntu: sudo apt-get install libtag1-dev
Fedora/RHEL: sudo yum install taglib-devel
Brew: brew install taglib
MacPorts: sudo port install taglib

*** extconf.rb failed ***
Could not create Makefile due to some reason, probably lack of necessary
libraries and/or headers.  Check the mkmf.log file for more details.  You may
need configuration options.

Provided configuration options:
        --with-opt-dir
        --with-opt-include
        --without-opt-include=${opt-dir}/include
        --with-opt-lib
        --without-opt-lib=${opt-dir}/lib
        --with-make-prog
        --without-make-prog
        --srcdir=.
        --curdir
        --ruby=C:/Ruby23/bin/$(RUBY_BASE_NAME)
        --with-tag-dir
        --without-tag-dir
        --with-tag-include
        --without-tag-include=${tag-dir}/include
        --with-tag-lib
        --without-tag-lib=${tag-dir}/lib
        --with-stdc++lib
        --without-stdc++lib
        --with-taglib
        --without-taglib

To see why this extension failed to compile, please check the mkmf.log which can be found here:

  C:/Ruby23/lib/ruby/gems/2.3.0/extensions/x86-mingw32/2.3.0/taglib-ruby-0.7.1/mkmf.log

extconf failed, exit code 1

Gem files will remain installed in C:/Ruby23/lib/ruby/gems/2.3.0/gems/taglib-ruby-0.7.1 for inspection.
Results logged to C:/Ruby23/lib/ruby/gems/2.3.0/extensions/x86-mingw32/2.3.0/taglib-ruby-0.7.1/gem_make.out

Attached the logs and screenshots

gem_make.out.txt
mkmf.log.txt

Hey. Sorry about that, I don't run Windows anymore and I'm not going to investigate this. Windows is not really friendly to Ruby extensions.
The output shows that it can't find taglib:
checking for main() in -ltag... no

So that's the problem. Not sure how to correctly install it either. I'm gonna close this in favor of #62.


Please add my cool code.

[!CAUTION]
It will be worth it, trust.

Reasons why you should add my code:

It's cool
I made it
please

I will think about it.

I will think about it.

u aint thinking hard enough


Fix moonhighway.com domain redirect

This should not point to herokuapp.com...

Set up a www record to point to the domain.


java-library init template should use java-library plugin

Expected Behavior
As a user, I want Gradle to encourage best practices by leveraging the hot new java-library plugin in project bootstrap templates
Current Behavior
gradle init --type java-library still generates a project that uses the java plugin.
Context
It is unclear how many users this will help, but presumably thousands will try this out.
Your Environment
N/A

Here's another one for ya if you want @mkobit ;)

@eriwen From what I can see this is already done in 3.4.

Yes I just tried it, the generated build.gradle uses java-library plugin.

My mistake. I tried this init template in one of my projects, but perhaps I wasn't on Gradle 3.4. Closing as invalid.


zip file is 0 bytes on download 0.7.5

0.7.5 downloads with a 0byte file size in: /tmp/packer-vpacker_0.7.5.zip
using the hack from #17:
  class {'packer':
    version => 'packer_0.7.5', # prefix is a workaround: https://github.com/boxen/puppet-packer/issues/17
  }

error:
Error: rm -rf /tmp/packer* /tmp/packer_0 && curl http://dl.bintray.com/mitchellh/packer/packer_0.7.5_darwin_amd64.zip?direct > /tmp/packer-vpacker_0.7.5.zip && mkdir /tmp/packer && unzip -o /tmp/packer-vpacker_0.7.5.zip -d /tmp/packer && rm -rf /opt/boxen/packer && mv /tmp/packer /opt/boxen/packer && chown -R bswift /opt/boxen/packer returned 9 instead of one of [0]
Error: /Stage[main]/Packer/Exec[install packer vpacker_0.7.5]/returns: change from notrun to 0 failed: rm -rf /tmp/packer* /tmp/packer_0 && curl http://dl.bintray.com/mitchellh/packer/packer_0.7.5_darwin_amd64.zip?direct > /tmp/packer-vpacker_0.7.5.zip && mkdir /tmp/packer && unzip -o /tmp/packer-vpacker_0.7.5.zip -d /tmp/packer && rm -rf /opt/boxen/packer && mv /tmp/packer /opt/boxen/packer && chown -R bswift /opt/boxen/packer returned 9 instead of one of [0]

0B puppet-packer download size,  and 83M  manual download size:

Looks like the download link is giving a 302 response and redirecting to CloudFront. Adding -L to the curl command should fix it. I'll submit a pull request.

Hi there,
Yes its, adding -L options works. I use this:
curl -L0k http://dl.bintray.com/mitchellh/packer/packer_0.7.5_darwin_amd64.zip -o /tmp/packer-v0.7.5.zip


Gcc warning cleanup -Wwrite-strings

Simple warning cleanup.

With mjheagle8's blessing, I'm taking over maintenance of this project. If you still care about this PR, please reopen it over at my fork.


Update incident-response-playbook-dart-ransomware-approach.md

Defender for Cloud Apps (previously known as Microsoft Defender for Cloud Apps)  should be Defender for Cloud Apps (previously known as Microsoft Cloud App Security)

#please-close


Snapshot selection

It seems to me that the couple last versions ignore the snapshot flag. I already tried using 1, 2 and -1 as arguments. Can someone else confirm this please? Also thanks @horrorho for the awesome job.

I've noticed this same issue.
On sex, 22 de jul de 2016 at 00:14 Aliasr<EMAIL_ADDRESS>wrote:

It seems to me that the couple last versions ignore the snapshot flag. I
already tried using 1, 2 and -1 as arguments. Can someone else confirm this
please? Also thanks @horrorho https://github.com/horrorho for the
awesome job.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/horrorho/InflatableDonkey/issues/13, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AHf-iN5diaKWdrXbkLra0--gXAt1BB-6ks5qYDV8gaJpZM4JSaAD
.

Thank you for the ticket. I'm currently porting over some of LiquidDonkey's command line argument methods and re-factoring other code. So this issue should hopefully be resolved by the end of the weekend.

Merge pull request #15 from mattandersen/master resolved this issue for snapshots, but not for devices

Device filtering works just fine. You have to specify the device's UUID, displayed to the left of the model number when listing snapshots.
Example:
java -jar InflatableDonkey.jar <username> <password> -d<PHONE_NUMBER>ABCDF1234567890ABCDF1234567890 -s 2 --extension sms.db
That would download the Messages database from the 2nd snapshot of device<PHONE_NUMBER>ABCDF1234567890ABCDF1234567890

Awesome! I misread the readme and was going for --device 1
Thanks!


Update example RedisCacheHandler to use a prefix

It is conventional in Redis to prefix keys to effectively "namespace" related keys. This change adds the ability to specify a key prefix, and defaults to 'ngx-isr'.
This makes it much easier to deal with when using tools such as Redis Commander - see the screenshot comparing un-prefixed entries (orange) with prefixed ones (green):

Hello @michaelbromley
Thanks for this 🎉
Just merged!
Will do some other fixes and will make a release.


[SPARK-44461][3.5][SS][PYTHON][CONNECT] Verify Python Version for spark connect streaming workers

What changes were proposed in this pull request?

Backport https://github.com/apache/spark/pull/42421 into 3.5. Also partially backport https://github.com/apache/spark/pull/42135
Add python version check for spark connect streaming foreachBatch_worker and listener_worker
Why are the changes needed?

Necessary check
Does this PR introduce any user-facing change?

No
How was this patch tested?

I believe it can be skipped here

Local test build fails for some weird error. Let's see if the test job complains

Merged to branch-3.5.


proposal: encoding/hex: use SIMD instructions for large blobs

Proposal Details
Currently the hex package uses a rather trivial implementation for encoding and decoding hex strings (well, ok, it's not exactly a complex thing). However, due to processing the input byte-by-byte, it can only run so fast. If the caller only wants to convert a few bytes to hex, then there's not much to improve. However, if the caller has big blobs of hex data (e.g. KBs to MBs), then using SIMD instructions could speed things up by at least an order of magnitude.
(Yes, I'm fully aware that having megabytes of hex text feels like solving the wrong problem in the first place. Unfortunately that's how a certain API encodes all its binary data in JSON, so I try to work with what I have.)
I have found some older projects doing similar things (e.g. go-hex), and was wondering whether there would be an appetite to upstream a newer SIMD implementation into mainline Go if we were to contribute it? We'd probably focus on amd64 and arm64.
(Again, yes, we could just use that library or something we write ourselves and leave Go stdlib out of it, but it seems like an improvement to the standard libs that doesn't really have any relevant drawbacks.)

The current policy for accepting assembly is https://go.dev/wiki/AssemblyPolicy

I wonder what has changed since Brad offered his rationale (in 2018):

Sorry, this is a nice speed-up, but this isn't a package where we want assembly.
We reserve assembly for the deep, gross, dark corners of Go.
A package like encoding/hex is supposed to be a high-level and readable package.
If these speed-ups matter for read applications, we need to achieve this somewhere else: moving some optimized functions into a gross package and using them from encoding/hex, or probably modifying the compiler to recognize more patterns and generate better code.
But this package should remain pure Go with no unsafe.

If anything, the objection is stronger today as Go supports even more machine targets now, and production machines are increasingly using arm64 where amd64 used to be the norm.
The third-party go-hex package seems like a reasonable solution for the (I hope) vanishingly small number of applications where encoding large blobs of data as hex is a significant bottleneck.

I cannot say if anything changes, but the reason I brought it up is because in 2018 there was a very strong push (if I remember correctly) on making the Go assembler super smart. My SSE work on SHA3 was pending for 2 years because of it. However, after the optimizations landed, it turned out that the Go assembler isn't as good as people hoped and hand assembly still outperformed it so far that my SAH3 got merged.
Due to that, I'm wondering if in the light of those experiences, there would be more willingness to accept optimizations into the standard lib (the answer might be no, and that's ok, just figured I'd ask).
Point in case, hex encoding a fairly large blob (5MB say) takes immensely more with the stdlib than the SIMD one.

On the top chart, RLP is the binary size and JSON is the hex encoded size (weird enveloping, bear with me)
On the middle and bottom charts are the encoding and decoding speed, where the green part (RLP) is contained in all cases, on top of which yellow is stdlib hex encoding and blue is simd hex encoding (i.e. you want the diff between color X and green).

The difference is immense.

If anything, the objection is stronger today as Go supports even more machine targets now, and production machines are increasingly using arm64 where amd64 used to be the norm.

My point exactly, that's why I'd like something upstream that can do both amd and arm.

The third-party go-hex package seems like a reasonable solution for the (I hope) vanishingly small number of applications where encoding large blobs of data as hex is a significant bottleneck.

My counter question would be why reject an obvious immense performance improvement from the standard lib? My reasoning (tongue in cheek, no offence meant) would be that if Go ships hex, it should be as fast as possible. If Go doens't want to optimize it's hex library, then it should be removed from the stdlib because it's not production ready any more.

Has anyone measured a portable pure-Go implementation that uses as 64KiB lookup table?

That's actually a nice idea, will try it.

The problem with adding assembly code to the standard library is maintenance.  Is encoding/hex a significant enough package that there is a benefit that is worth this cost?  I understand that for your particular program there is a big benefit.  But for your particular program you can presumably use your own optimized encoding/hex packages.  Are there are other real programs out there where the performance of encoding/hex matters?

Ugh, turns out my issue is even messier. Hex encoding is half my problem, but the forced JSON escaper is the other part of the issue. I can make my hex strings generate/parse fast, but Go's json package will still iterate every hex character one by one, trying to see if it needs escaping or not. Diverging here a bit, but isn't there a way to avoid doing that (or... ha... doing it via SIMD to make it meaningfully fast)?

For what it's worth, Go's JSON string encoding overhead is 110x compared to simply copying the data into the output buffer. Opened a new issue for that https://github.com/golang/go/issues/68203

Has anyone measured a portable pure-Go implementation that uses as 64KiB lookup table?

I've rolled one yesterday, initialized a string with 256x256x2 bytes to use as a lookup and then tried to iterate my source byte slice two bytes at a time and look up the encodings. Myeah, 2x slower than stdlib :) Haven't spent time further on it since it was so far underperforming. Just a memo.

After investigating our issues a lot more, it seems it's a combination of hex being slow(-er than we'd like) and json also doing a lot of post-processing on strings, the combo of the two hitting us out of latency requirements. I've opened a bunch of issues across the board on various json packages, but decided that there are too many philosophical roadblocks in place to meaningfully reduce the latencies for us, so we'll bite the bullet and roll out a binary protocol across 9 orgs/teams.

Before you switch to a binary protocol, you should evaluate using a different JSON encoder/decoder. A number of alternative ones exist, some motivated by greater control over the encoding (for example, a hundred details such as whether empty slices should be emitted as null or []) and others strictly by performance, including, for example, whether maps should be iterated in key order. We are considering a large collection of such concerns together as the basis for a future v2 implementation of encoding/json; I don't expect that process to conclude any time soon, but in the meantime you may find that one of the existing codecs implements the standard semantics with sufficiently improved performance that it could be a viable drop-in replacement for your needs.


Volume projection bounds

Purpose

This PR adds the capability to set parametric bounds in the embedding volume of each FFD box. This is useful to control the behavior of overlapping FFDs, where the user wants to avoid embedding any point in one portion of one or both of the FFDs.
This PR needs changes in https://github.com/mdolab/pyspline/pull/63 to work.
Expected time until merged

After https://github.com/mdolab/pyspline/pull/63 is merged.
Type of change

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (non-backwards-compatible fix or feature)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Documentation update
[ ] Maintenance update
[ ] Other (please describe)

Testing

Checklist

[ ] I have run flake8 and black to make sure the Python code adheres to PEP-8 and is consistently formatted
[ ] I have formatted the Fortran code with fprettify or C/C++ code with clang-format as applicable
[ ] I have run unit and regression tests which pass locally with my changes
[ ] I have added new tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation

I can't merge it with the failures, @anilyil can you do it?

Did anyone actually run this on Azure? The tests are failing

Did anyone actually run this on Azure? The tests are failing

We tried to manually run an Azure and accidentally ran the wrong PR and didn't realize it.

Ah okay, I see that manual run. For future reference, the checks in PRs should be updated even if you manually run the pipeline (as long as it's the correct commit).


Pluggable Streamer Prototype in Java

The following pull request implements the pluggable streamer written in Java. It is used to demonstrate what the concept is and how one would implement a streamer using only the uTransport interface. The intention of this PR is to provide example code and NOT be used for implementing an actual streamer as that would be written in C++ or rust to be useable everywhere. .

@tamarafischer
While you're offering Steven critique feel free to check out the attempt in Rust 😉


ask HC re 'discoverability'

already have appt to redirect old site.

site is discoverable, so closing.


cant seem to use video returned by getInternalPlayer, to draw on html5 canvas context

I am following this tutorial that shows how to use ReactPlayer:
I get through most of the tutorial properly.
I see the following error when I try to create bookmark images:

My playerRef is set as follows:

I tried using several techniques, and as you see in the image above, I even tried to use the captureVideoFrame function that is referenced by cookpete here.
getInternalPlayer does not seem to return something that can be passed to the drawImage method of an html5 canvas context.
This code seems like it should work:

However the error I see is as follows:

Note, in the console, I have displayed the video object and it can be seen here:

Looking at your documentation, it seems like getInternalPlayer can only be used in this way if its referencing a FilePlayer

How can I get the "current video frame" in a standardized fashion, so that I can then render to an html5 canvas?

The clue is here:

It looks like you are playing a Youtube URL, which renders an <iframe>, which is what gets returned by getInternalPlayer(). The problem is that iframes can't be used in drawImage – only images, videos or SVGs are supported, as the error describes.


Use the required AI key in settings

Here's something that ought to be documented a little more clearly, it caught me out for a while.
Somewhere in MS docs it mentions that you can use an app settings key of either ApplicationInsights:InstrumentationKey or APPINSIGHTS_INSTRUMENTATIONKEY and it says that in the Azure environment it tends to be the latter. In fact that's not the case. You must use APPINSIGHTS_INSTRUMENTATIONKEY in Azure otherwise server Application Insights simply doesn't work.

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

ID: 59b9aed0-b6a7-d3e9-d616-85e865966444
Version Independent ID: df2e545f-7f65-82b7-c793-2d04d91d7d9e
Content: Troubleshooting no data - Application Insights for .NET
Content Source: articles/azure-monitor/app/asp-net-troubleshoot-no-data.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@stuarthallows , Thanks for brining this to our attention. We will investigate it further and update you shortly.

@stuarthallows I think you are probably referring to this part of the ASP.NET Core doc: https://docs.microsoft.com/azure/azure-monitor/app/asp-net-core#enable-application-insights-server-side-telemetry-no-visual-studio.
specifically this section:
Typically, APPINSIGHTS_INSTRUMENTATIONKEY specifies the instrumentation key for applications deployed to Azure Web Apps.
@cijothomas did something change/break on the App Service/Azure Web App side or should the "typically" be dropped and this be made explicitly clear to use APPINSIGHTS_INSTRUMENTATIONKEY if setting the ikey via environment variable with an Azure Web App?

I don't see any doc stating ApplicationInsights:InstrumentationKey will work for Azure webapps - it won't work. It only works if using ApplicationInsights SDK for Asp.Net Core (https://docs.microsoft.com/en-us/azure/azure-monitor/app/asp-net-core)

@cjthompson I know that ApplicationInsights:InstrumentationKey won't work, but as the doc uses the word Typically, APPINSIGHTS_INSTRUMENTATIONKEY that implies that you may be able to use either, but that one is preferable to use one over another. My suggestion is that the documentation could explicitly state the value required in Azure.

@mrbullwinkle feel free to change the doc if you agree there is confusion.
As the doc for WebApps clearly states how ikey can be setup, there shouldn't be . But I may be wrong.

I think it is possible someone could read the ASP.NET Core doc in isolation and never read the Azure Web App doc and therefore could end up confused. I agree if you have read the Azure Web Apps doc it is clear what to use, but the ASP.NET Core doc gets more traffic and does better from an SEO perspective. I will add a sentence to clarify, that also cross links to the Azure Web App doc to avoid any confusion that might be caused by lack of discovery of the dedicated Azure web app doc.

@stuarthallows I updated the doc to help clarify that section and cross link to the Azure App Service monitoring article to make things a little more clear. The change is merged internally and will go-live in the public docs between 10 AM - 11 AM tomorrow (UTC-7). Thank you for helping us to improve our docs!

#please-close


Add Disk Sensei latest

A decent disk analysis app.

Thank you for the contribution. It was merged directly as da56225d1d4500482b59545a00096befd7690058 to keep commit history cleaner. Your contribution is still credited to you.


Would like to access a grandparent value for the prefix or the suffix.

Not really an "Issue" more like a feature request, I need to have a prefix on an entity which uses a value from a grandparent entity.
Here is how the hierarchy works: Proposal -> Opportunity (lookup) -> Department (lookup) [new_Suffix]
{opportunity.new_department.new_suffix|00}{createdon:yy}-
Let me know your thoughts.
-Stephen

This can be done with the existing functionality.  You would just have to add a calculated or autonumber field on Opportunity that gets the value from the grandparent.  That way there's no need to pull directly from the grandparent to the child.
Some thoughts on this request:

When this was initially built, it intentionally only supported immediate parent lookups.  Partly for simplicity, and also because it follows precedent (OOB calculated fields only support parent as well).
Once you get to grandparent, it could introduce performance issues, because multiple retrieve calls will be needed to get the final value
If grandparent is supported, where should the line be drawn?  Why not great-grandparent or beyond?
An issue with a grandparent lookup, is that it introduces the possibility of a circular reference.  Think of Account/Contact:  You can have Parent Account on Contact, and you can have Primary Contact on Account, and in some instances that would be an infinite loop.

On the other hand...  OOB calculated fields only supports immediate parent.  So a project like this could differentiate itself by being better than OOB.
Just my $.02

That is correct.
On Fri, Jul 14, 2017 at 12:57 PM, Travis Sharp<EMAIL_ADDRESS>wrote:

Just for clarification:
Opportunity is a lookup on a proposal, and you need to look at a the
department field suffix that is specified on the opportunity.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BESDev/Celedon-Autonumber-RTM/issues/1#issuecomment-315425112,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADUICQW1HPaZMxeOF3bM_UaVoMKkK4u2ks5sN6vxgaJpZM4MlWsW
.

--
"Every day is a journey, and the journey itself is home." -- Matsuo Basho

Thank you for the response, I had planned on attempting this myself and am
still willing to; I had tried earlier but as others had pointed out there
was an issue with it and i needed something "right then" so i moved on to
write my own.
Since i can now open the project. i can take a crack at it. I'll be more
than happy to contribute... or at least try to.

It is simple and elegant, and it works which is the best part
If the performance hit is know upfront and if the functionality is
need as in my case I could see it as a valid trade off, I would much prefer
one place to manage my auto numbers, than 2+
I could see if being infinite if they are okay with the performance
hit, by using the [child].[parent].[GP].[GGP] and so on.
Circular References - Also a an issue, I had planned on adding a check
for that, then throwing an error.

Any other thoughts?
On Fri, Jul 14, 2017 at 3:03 PM, Matt Barnes<EMAIL_ADDRESS>wrote:

This can be done with the existing functionality. You would just have to
add a calculated or autonumber field on Opportunity that gets the value
from the grandparent. That way there's no need to pull directly from the
grandparent to the child.
Some thoughts on this request:

When this was initially built, it intentionally only supported
immediate parent lookups. Partly for simplicity, and also because it
follows precedent (OOB calculated fields only support parent as well).
Once you get to grandparent, it could introduce performance issues,
because multiple retrieve calls will be needed to get the final value
If grandparent is supported, where should the line be drawn? Why not
great-grandparent or beyond?
An issue with a grandparent lookup, is that it introduces the
possibility of a circular reference. Think of Account/Contact: You can have
Parent Account on Contact, and you can have Primary Contact on Account, and
in some instances that would be an infinite loop.

On the other hand... OOB calculated fields only supports immediate parent.
So a project like this could differentiate itself by being better than
OOB.
Just my $.02
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BESDev/Celedon-Autonumber-RTM/issues/1#issuecomment-315454205,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADUICao4CSXMyiNlawcGxUX4QAEoE3ofks5sN8mNgaJpZM4MlWsW
.

--
"Every day is a journey, and the journey itself is home." -- Matsuo Basho


Traefik middleware transform headers

Feature
Add a middleware that can be used to alter/transform headers.
Use case:
When using something like forwardAuth, it seems that traefik allows the forwardAuth to trustForwarded headers. I couldn't test the way I understand is that if HTTP_X_FORWARDED_HOST is defined, then it will pass it to the authentication server. As long as the authentication server is available without passing through a proxy server this will possibly work.  The headers will be overriden and inside your Authentication server you'll see only the HTTP_X_FORWARDED_HOST which is the actual domain of the auth server. The information about the origin of the forwarded authentication seems to be gone from there.
Renaming headers would allow to rename the header to something like HTTP_REFERER which would point to the server that originated the forward auth request, Header for paths could be renamed too so we know where to make the redirect to. Modifications could be done in batch to rename all the defined X-Forwarded-.* to X-Auth-Forwarded-.*
traefik.http.middlewares.<middleware>.headers.transform = op1, op2, op3

Transforming operations:

Rename: Rename(header, new_header)
HeaderRewriteRule(regex, replace): Rewrite header name using a regex with a match/capture
ValueRewriteRule(header, regex, replace): Same as HeaderRewriteRule but for values
Join(header, sep, value/header) : Possibility to join multiple headers inside one. This can be useful when you're hoping through multiple proxy servers. For example HTTP_X_FORWARDED_FOR can be use to track a list of all the proxies

Here https://en.wikipedia.org/wiki/X-Forwarded-For
From my experience, it doesn't seem like Traefik comply with this. But with a way to transform an header. When I look at the result of whoami.example.com, I can see that traefik doesn't append the ip of each proxy being passed. With a middleware like that it could be as simple as
Join(X-Forwarded-For, ',' REMOTE_ADDR)
X-Forwarded-For: <IP_ADDRESS>
REMOTE_ADDR: <IP_ADDRESS>
=
X-Forwarded-For: <IP_ADDRESS>,<IP_ADDRESS>

At the same time, this would solve this issue #3919
Those are suggestion to come up with a solution for forwardAuth and if done correctly, it could be used internally to define generic rules like the X-Forwarded-For that could be implemented directly with such rule by default.

As for X-Forwarded-For, it would allow things like tracing path of request in a multi proxy setup. Right now, it doesn't seem like the endpoint has any knowledge of the complete path it takes. Having something in the line of what I'm proposing would enable couple of useful usecases and not just fixing forwardAuth implementations.
That said, if someone comes with better syntax for transformation considering that we may want to create multiple rules but having to create 1 middleware for each rule may seem a bit overkill. Being able to set something like that using labels could be done by adding multiple rules sequentially while being part of the headers middleware.
Something in the line of
traefik.http.middlewares.<middleware>.headers.transform = op1, op2, op3
Where modifications made by op1 impact on op2 and op3.

From my experience, it doesn't seem like Traefik comply with this. But with a way to transform an header. When I look at the result of whoami.example.com, I can see that traefik doesn't append the ip of each proxy being passed.

By default, Traefik v2 don't trust external X-Forwarded-* headers, but you trust them:
https://docs.traefik.io/v2.1/routing/entrypoints/#forwarded-header

@ldez Thank you, with that setting I can finally get my headers down to the auth server. That's one problem solved and I guess it would be a good enough solution for many people. But... I see a few issues.

This is a list of static IPs, what are my guarantee that my ips won't change on my services and I'll be trusting IPs that shouldn't be trusted and IPs that should be trusted won't be trusted anymore.
While having the forwarded headers down to the auth server is definitely some progress, my application server requires the forwarded-host to correctly find which db is associated to the request. It's a design that allow one server to serve multiple domains. So you can share the same code with different database selected by a rule computed from forwarded headers. In this case, the auth server would require both headers from the client to the auth server but I only receive 1 or the other set.

In the meantime, I can fix the database name but being able to get both sets would be a better solution. Considering that it would always work regardless of the trustedIPs list.
Thanks again, I'll be able to finish this.

We will close this issue, because the main topic of the issue is a question around  X-Forwarded-* headers.
But the part of this issue about a middleware to interact with header, is a good subject for a proposal.
So we propose to open another issue to keep the issue clean.
Thank you.

@ldez ok I'll create a new issue for the proposal alone


refactor: SDPD case num

Description

Take existing case information being used for the scrape method to organize assets and create a new metadata key/value field case_num
Use case_num to create the file tree structure for saving assets

Goal
Facilitate clean-prefect scrapes while removing complexity from clean-scraper
How to review

Does case_num for SDPD look correct? (Note: other agency case_nums may look different depending upon the agency).

> python -m clean.cli scrape-meta ca_san_diego_pd
sb16-sb1421-ab748_page=4/04-30-2015_3200_Hancock_Stree
sb16-sb1421-ab748_page=4/09-28-2015_1100_West_San_Ysidro_Boulevard
sb16-sb1421-ab748_page=4/04-01-2014_IA_2014-185

@newsroomdev afaict the listing of the documents (date + address) is not directly related to the actual case number. For example:

Case num (from police document): 24300923
Case ID (from URL for the case): 02-23-2024 4800 Savannah Street
Page title: 02-23-2024 4800 Savannah Street
File base name: February_23_2024_4800_Savannah_St_Documents_Redacted_NN.pdf

In this case, if the actual Case Number (ie the number used for internal tracking by the police department) is not readily available from the metadata on the site or the name of the file itself, best we can do is what you appear to be doing (ie using the case ID or page title).
The actual determination of the case number then becomes a task for AutoFolio or other downstream processes that examine the file contents to determine case number.


False errors about Parenthesis ( ) characters in x500 Addresses

Category

[ ] Enhancement
[X] Bug
[ ] Question
[ ] Documentation gap/issue

Version
Please specify what version of IdFix you are using: [<IP_ADDRESS>]
Expected Behavior
x500 addresses are not flagged up as invalid proxy addresses
We previously used version 1.11 and that version has the expected behaviour.
Observed Behavior
x500 addresses are flagged up as invalid proxy addresses with a Character error, causing thousands of false errors.
The suggested correction by the tool is to remove Parenthesis  "(" and ")" characters.
This is incorrect and sync is unaffected.  Our valid x500 addresses contain these characters and are simply repopulated from Azure during the next sync if the address is changed or removed.
Steps to Reproduce
Add an x500 proxy address to an AD object that contains the characters Parenthesis "(" or ")", such as the following:
x500:/o=ExchangeLabs/ou=Exchange Administrative Group (FYDIBOHF23SPDLT)/cn=Recipients/cn=f7a6b595bd894d3ca3d5f1cfae62d4fb-FirstLastna
Run a query in IDFix and review the results.

Having the same problem in two of my clients.

Thanks for letting us know - we have a proposed fix in and just need to do some further testing before we make the next release.

What is the status of this fix? The issue is still open.

Any update for when <IP_ADDRESS> or newer version will be published?
Thanks.

I am having the same exact same issues with <IP_ADDRESS>. I am currently using Azure AD Connect to upload a subset of users into Azure and was cleaning up problems reported by a previous version of IDFix. I was down to just a handful of issues and then noticed I was using an older version of IDFix. I upgraded to <IP_ADDRESS> and then reran it and it added over 150 additional accounts in for needing charcter edit(needing to removing Parentheses from the x500 addreses). I was not paying attention to the subset of users and made a couple of the edits in IDFix, but then noticed that something was fighting me and changing the attribute in AD back. I did it again and confirmed the attributes in AD had a parenthesis in x500 address, confired that IDFix edit removed that parenthesis in AD attributes, and then confirmed that Azure AD Connect was syncing and re-adding the parenthesis back to the value that causes IDFix to report as a problem. The Service Sync Manager on Azure AD Connect is not reporting any issues. My version of Azure AD Connect is 1.4 and a few months old, so I am going to update it to 1.5, but assuming this is a bug with IDFix <IP_ADDRESS>

Just released <IP_ADDRESS> including a fix for this issue. Please try out the latest version and you should no longer see this issue. Thank you for your patience and for reporting the issue allowing us to get it fixed. Thank you!


linting fails on a fresh project

Hi folks
We were excited to see a lint command being added recently and are eager to include that in our CI pipelines.
However, I noticed that linting fails on a fresh tanka project.
 Remember to configure the API endpoint:
`tk env set environments/default --server=https://<IP_ADDRESS>:6443`

# setup sample app using default k.libsonnet
cat > environments/default/main.jsonnet <<EOF
local k = import "k.libsonnet";
{
    podinfo: k.apps.v1.deployment.new(
        name="podinfo",
        replicas=1,
        containers=[k.core.v1.container.new(
            name="podinfo",
            image="stefanprodan/podinfo"
        )]
    )
}
EOF

# run lint:
$ tk lint environments/default/main.jsonnet
...
/path-to/tk-sample/vendor/github.com/jsonnet-libs/k8s-libsonnet/1.20/_custom/volumeMounts.libsonnet:1:11-43 couldn't open import "doc-util/main.libsonnet": no match locally or in the Jsonnet library paths

local d = import 'doc-util/main.libsonnet';

Error: Linting has failed for at least one file

If i add the docsonnet library to the project, the error disappears:
$ jb install https://github.com/jsonnet-libs/docsonnet/doc-util
GET https://github.com/jsonnet-libs/docsonnet/archive/fedfb4920fae931a27190df0aeec5ef858daf3b5.tar.gz 200

$ tk lint environments/default/main.jsonnet
echo $?

What's your recommendation - should we add that library to the jsonnet projects?
docsonnet's README explicitly states that this should not be required:

Do my projects need to have doc-util installed to vendor/?
No! The docsonnet binary comes included with it, and during normal Jsonnet use the docsonnet keys will never be accessed, so your Jsonnet runs just fine without.

https://github.com/jsonnet-libs/docsonnet#do-my-projects-need-to-have-doc-util-installed-to-vendor
If yes, it might be a good idea to add it to the default tk init setup, so that future users can use tk lint right out of the box.

Might be nice to add a caveat in the docsonnet repo indeed. Linting cannot work with inaccessible imports

@julienduchesne I'll happily create a PR to add a notice into the docsonnet repo README.
For our projects, we solved the linting issue by adding docsonnet manually.
Should tanka install this dependency automatically when creating new projects with tk init? If yes, I'd create a PR.

@julienduchesne I'll happily create a PR to add a notice into the docsonnet repo README.
For our projects, we solved the linting issue by adding docsonnet manually.
Should tanka install this dependency automatically when creating new projects with tk init? If yes, I'd create a PR.

I think both a notice and installing the dependency automatically with tk init would be reasonable. This has caused other issues in the past


npm audit alert being triggered - alchemy-sdk tied to 3-year-old @ethersproject/providers 5.7.2 - with ws 7.x

Sorry if this has been addressed or if I'm missing something obvious.
[REQUIRED] Environment
Win10

Browser version:  none
Alchemy SDK version:  3.4.7

[REQUIRED] Describe the problem
npm audit shows

ws  7.0.0 - 7.5.9
Severity: high
ws affected by a DoS when handling a request with many HTTP headers - https://github.com/advisories/GHSA-3h5v-q93c-6h6q
No fix available
node_modules/@ethersproject/providers/node_modules/ws
@ethersproject/providers  <=5.7.2
Depends on vulnerable versions of ws
node_modules/@ethersproject/providers
alchemy-sdk  *
Depends on vulnerable versions of @ethersproject/providers
node_modules/alchemy-sdk

How to reproduce:
npm audit
Relevant code or sample repro:
all my Alchemy code works fine (thanks)

This has been an ongoing issue. We previously fixed by manually updating the package-lock to the proper versions of ws and elliptic but with the 3.5.0 release it seems like the problem has returned and we have not been able to find the correct versions to override with yet.
This was supposedly fixed in 3.4.3 but we still had to do the manual override. You can see a closed issue here where we raised the problem.
https://github.com/alchemyplatform/alchemy-sdk-js/issues/457


Some issues to discuss

Hey, I really like your project and it looks good to our team. There are some issues though I want to discuss here:
1) Loop variables are never read
for key, val of someObject
  doSomethingWith(val)

gives me:
✗ #1: Variable "key" is never read ( first defined on line 1).

Is there an option to ignore this? I know I could write this in another way but it's nice to use the for ... of syntax here.
2) Undefined identifier if used before defined
foo()

foo = ->
  return 1

gives:
✗ #1: Undefined identifier "foo".
✗ #3: Variable "foo" is never read ( first defined on line 3).

This cannot be correct because foo is defined within the scope.
3) Variable never read
describe 'some spec', ->

  foo = null

  beforeEach ->
    foo = 1

gives:
✗ #3: Variable "foo" is never read ( first defined on line 3).
✗ #6: Variable "foo" is never read ( first defined on line 3).

Same:
intervalId = setInterval ->
  clearInterval intervalId
, 50

-> Variable "intervalId" is never read

Thanks for the reports!
To ease communication and tracking, I'm going to split all mentioned issues in separate requests!


[BUG]: Pointer should be disabled on mobile device

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When using the website on a mobile device, the pointer is not very useful and it remains stuck on the last position.
This makes the website look buggy
Thus, I want to disable the pointer on mobile devices.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

/assign-me


Update Makefile for ppc64le kustomize change

Kustomize version is updated  for ppc64le.

Why are these changes needed?

Related issue number

Checks

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

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

Raised the PR to build on Power(ppc64le) as kustomize version =<EMAIL_ADDRESS>is supported on ppc64le


add RBAC used by pvmigrate

Closes https://github.com/replicatedhq/pvmigrate/issues/177

A suggested followup (from slack) would be to use this in our integration tests somehow


Profile - Update - Error:  Display Name Already In Use

Description of bug
Provide a clear and concise description of the bug.
Steps to reproduce
List the precise steps to reproduce the bug:

Go to www.dnncommunity.org
Create an account if you do not have one
Log into account
Go to "edit profile"
Click "update" button

Current behavior
Message Error:  Display Name Already In Use
Things get saved
I still use my display name
Expected behavior
no error message
Screenshots

Error information
Provide any error information (console errors, error logs, etc.) related to this bug.
Additional context
Provide any additional context that may be helpful in understanding and/or resolving the bug.
Affected version

[ ] 10.00.00 alpha build
[ ] 09.10.02 release candidate
[ ] 09.10.01 latest supported release

Affected browser

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Internet Explorer 11
[ ] Microsoft Edge (Classic)
[ ] Microsoft Edge Chromium

I cannot reproduce this on a couple of recent sites. My guess is this is specific to that site and maybe because "required unique display names" was turned on after you created your account and there was already a duplicate. Can you post this in the forums there so an admin can take a look at the display names. Unfortunatelly even though I am an admin there, cloudflare does not like me and I can't access the persona bar on that site.


Why, as a browser developed from scratch, is it not developed using Rust?

Nowadays, there's a trend of migrating from C++ to Rust, so why does Ladybird still choose to develop in C++ instead of Rust?

There is a FAQ on https://ladybird.org/ where this exact question is answered

There is a FAQ on https://ladybird.org/ where this exact question is answered

Thanks! If it's a browser built from the ground up in Rust, that would be even more promising.

To be fair the disclaimer does say it was in part due to the constraints of Serenity and Andreas's initial preference as the project is starting to grow faster with a larger community it might make sense to look into binding rust with CXX to the existing code base and swapping out certain sections as time goes on

Wish it was all rust, C++ tooling ecosystem is a nightmare, but a full port would be unlikely and a Rust + C++ hybrid would still require native tooling, and bindings to things like Obj-C / Swift for macOS UI still usually requires native tooling see Zed for a good example of that.

Nowadays, there's a trend of migrating from C++ to Rust

Honestly, "it's the trend" is the worst possible reason to use a specific programming language.

Nowadays, there's a trend of migrating from C++ to Rust, so why does Ladybird still choose to develop in C++ instead of Rust?

serenity is written in C++ from the ground up and so for the entire system and apps. if you want an entire OS in your favorite language, you can have a look at redox.
"new" projects do not require to be written in rust.

There is a FAQ on ladybird.org where this exact question is answered

Thanks! If it's a browser built from the ground up in Rust, that would be even more promising.

As some good web tech has been built with Zig people wonder about that language as well for building a web browser and/or engine.
There's still the Servo web engine project and I wouldn't be surprised if Ladybird takes some inspiration from that. Though Servo is more modular so projects like Tauri want it to make building apps even leaner and work faster.

If it were written in Rust, it wouldn't be worth touching.

Too bad C++ is crusty. 0-days? No thanks.


adjust to django 1.9

userprofile.Activity.ip_addr: (fields.W900) IPAddressField has been deprecated. Support for it (except in historical migrations) will be removed in Django 1.9.
HINT: Use GenericIPAddressField instead.

userprofile.Activity.ip_addr: (fields.W900) IPAddressField has been deprecated. Support for it (except in historical migrations) will be removed in Django 1.9.
HINT: Use GenericIPAddressField instead.

fixed


Corrected inconsistent listener notification for resumeAll() and pauseAll()

resumeAll() is invoking SchedulerListenerSupport.triggerResumed(TriggerKey triggerKey) with triggerKey as null
But as documentation, resumeAll() is equivalent of calling resumeTriggerGroup(group)on every group, not on a single trigger
So changed it to invoke SchedulerListenerSupport.triggersResumed(String triggerGroup) also to be consistent with pauseAll() method

Hello! Thank you very much for your contribution and interest in helping improve the Quartz community.
After a period of dormancy, the Quartz project is back under steady maintenance by multiple volunteers, who are working to once again handle contributions such as yours.
We notice that your contribution was made without use of the DCO feature (the sign-off feature on commits via the -s option). Can you please update your PR with commits that use the -s option, agreeing to assign copyright ownership and other terms as described at the contributor agreement referenced here: https://github.com/quartz-scheduler/contributing/blob/main/CONTRIBUTING.md
You can easily add signoff to your previous commits by running (on your PR branch):
git commit --amend --signoff --no-edit
git push -f

Or (for multiple commits):
# change 5 to the number of commits
git rebase HEAD~5 --signoff
git push -f

Signed off my commits, thank you

yes this is still relevant


enable/disable floats configuration

Hi @yurydelendik
I've added a new operator config option called enable_floats (defaults to true) under the OperatorValidatorConfig.
I believe this PR will be very useful for most (if not all) Blockchain projects using wasm for running smart-contracts.
Blockchain project using wasm avoid floats in order to stay deterministic across different hardware / operating-systems.
Thank you!
Yaron

This patch appears to disable float operators, but not float types, for example in function signatures, locals, or globals. Is that intended?
If you do intend to disable float types, disabling float operators would then be unnecessary and the patch could be simpler, because it's not possible to use float operators if there are no float types (except in unreachable code, but there they can be ignored).
Also, while I don't object to the patch, I do want to mention for what it's worth that WebAssembly's floating-point is deterministic in all respects except for NaN bits. Some platforms tolerate some floating-point rounding nondeterminism, but WebAssembly does not.

@sunfishcode Hi!
Basically, I wanted to make sure no floating operators will appear in the code.
I guess that floating types could still appear. I wasn't sure how it the best way to disable the float types in the source code.
Thank you for the clarification regarding the deterministic nature of floats in wasm.
Nonetheless, I know that in my company we'd like to avoid them feature-wise (and as said, I believe more Blockchain projects will find that feature relevant).

I believe more Blockchain projects will find that feature relevant

Except float, which features will be beneficial? The reason why I'm asking is that in might be beneficial to move it to the feature level and use #[cfg(feature =  for blockchain-related subset. Perhaps even for deterministic stuff two?

I think that having #[cfg(feature = deterministic)] could be interesting.
Right now, I don't have anything else that stops me besides avoiding floats.
On the other hand, I guess that different Blockchain projects will have differentiations regarding what they would like to have. I'm afraid this feature might be a bit immature right now... (IMHO)
Personally, I love the freedom to enable/disable each feature in isolation as it's right now (and there aren't many flags right now).
Thank you!

Hi @yurydelendik @sunfishcode
I'll need this feature pretty soon, and I'd rather not work with a forked version.
Please let me know if there's anything I can help with, in order to push this feature/similar forward.
Thank you!

Another feature with nondeterminism is shared memory.

Hi @yurydelendik
I've made changes according to your guidance
Thank you!

@yurydelendik my tests failed, I'm checking why.
@sunfishcode I'll need to add the deterministic feature to check_shared_memarg_wo_align too right?

@YaronWittenstein I believe check_shared_memarg_wo_align is only called when check_threads_enabled returns true, so it shouldn't need an extra check.

tests pass now :)

Thank you for the patch.

Thank you!


increase test timeouts

increase test timeouts to account for 3 failing tests (https://build.chromium.org/p/client.flutter/builders/Windows/builds/3297/steps/run test.dart for tests shard/logs/stdio)

(follow up to https://github.com/flutter/flutter/pull/16979)

Do we know why they're taking longer now?

The PRs do have some info about why; @Hixie filed a bug on the sdk repo - https://github.com/dart-lang/sdk/issues/33039 - which is probably the best tracking issue.
We've set up a benchmark on the dart benchmarking system to track this - flutter analyze --flutter-repo - using combined commits from the dart and flutter repos, and are working on a longer term plan.

I still see 2 failures -TimeoutException after 0:00:30.000000: Test timed out after 30 seconds. - after this PR. I'm not certain that timeout: const Timeout.factor(2.0) doubled anything?

Hmm, https://build.chromium.org/p/client.flutter/builders/Windows/builds/3299 is passing, so perhaps things are working after all.


ref shortcode doesn't work with org files

What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.69.0/extended linux/amd64 BuildDate: unknow

Does this issue reproduce with the latest release?
didn't tested, i'm using a system wide installation for hugo
https://packages.gentoo.org/packages/www-apps/hugo
if I have a content/path/to/file.org and use {{< ref "path/to/file.org" >}} it raises with a REF_NOT_FOUND error.

Try to prepend it with a "/".

it doesn't work too... i've already tried with no path/to/file, /path/to/file, path/to/file.md(i don't know if hugo outputs a md file from the org one so i thought it worth the try) and /path/to/file.md.

I'm  not a big org users, so I'll have to ask: Do shortcodes in general work fine?

they work, i've only tested the gist one, but since it worked i didn't tested the others. i can test all of them if it helps.

No, I was just after a confirmation on a binary level.

OK, I will eventually check. Maybe @niklasfasching can chime in. I don't see why org files should behave differently in this situation.

As a ugly hack until this is fixed you can refer to the org-file by using .md as extension instead of .org. (Tested with v0.71.1).

No idea :/

It is working for me on v0.81.
So for {{ ref . "post/dir1/dir2/b.org" }} I've got: http://localhost:1313/post/dir1/dir2/b/.
Please let me know if I understood the problem incorrectly.

It is working for me on v0.81.
So for {{ ref . "post/dir1/dir2/b.org" }} I've got: http://localhost:1313/post/dir1/dir2/b/.
Please let me know if I understood the problem incorrectly.

Unable to reproduce. This works as expected:
[[{{< relref "tests/org-2.org" >}}][Org 2]]


chore: Move Telemetry class files to a separate  /Telemetry folder

Description
The Telemetry partial class consists of 27 files, which were all located at root level in Core/Features/.
This change simply moves them to a /Telemetry subfolder, purely for organisational purposes. The namespace remains unchanged.
Related Issue(s)

N/A

Verification

[x] Your code builds clean without any errors or warnings
[x] Manual testing done (required)
[ ] Relevant automated test added (if you find this hard, leave it and we'll help out)
[x] All tests run green

Documentation

[ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

Quality gate failed
😂


[SPARK-20955][Core]Intern "executorId" to reduce the memory usage

What changes were proposed in this pull request?
In this line, it uses the executorId string received from executors and finally it will go into TaskUIData. As deserializing the executorId string will always create a new instance, we have a lot of duplicated string instances.
This PR does a String interning for TaskUIData to reduce the memory usage.
How was this patch tested?
Manually test using bin/spark-shell --master local-cluster[6,1,1024]. Test codes:
for (_ <- 1 to 10) { sc.makeRDD(1 to 1000, 1000).count() }
Thread.sleep(2000)
val l = sc.getClass.getMethod("jobProgressListener").invoke(sc).asInstanceOf[org.apache.spark.ui.jobs.JobProgressListener]
org.apache.spark.util.SizeEstimator.estimate(l.stageIdToData)

This PR reduces the size of stageIdToData from 3487280 to 3009744 (86.3%) in the above case.

cc @JoshRosen

Test build #77654 has started for PR 18177 at commit 38a4e31.

It seems reasonable but there are likely many more similar opportunities - is this a significantly large one? Or are there other easy wins?
Interning strings isn't as problematic as it used to be. Ideally the JVM string deduplication would take care of this even better and automatically though it is still an experimental flag.

@srowen Considering Spark keeps 1000 stages in UI, if each stage has 10000 tasks, the duplicated strings will be a lot. I observed this, about 150MB in a heap dump. Of cause, it's not significantly large but still worth to eliminate them. I agree that JVM string deduplication is better but it 's not enabled by default.

Test build #77654 has finished for PR 18177 at commit 38a4e31.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77654/
Test PASSed.

LGTM. This seems like an easy win based on our empirical observations of high memory usage. I'm sure that we'll continue to find and fix others as we profile and heap dump more.

Thanks! Merging to master and 2.2.


Add some codegen related terminology to glossary

cc #48
EDIT: also remove duplicate MIR def...

I would appreciate some confirmation on these definitions.

They all sounded right to me.

Needs rebase tho

Rebased :)
Also, a heads up: I might be really slow to answer for the next few days...


add SendGrid logo at the top of README.md

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.