Allow binderhub-service's pod running binderhub to be configured with extraVolumes and extraVolumeMounts

This can enable us to trial mounting the build pods docker config also to the binderhub pod, allowing it to use that to extract permissions to read from the docker registry. This can be generally useful, but also provides one simple approach to resolving or testing a resolution #4315.
This can be considered done, if the binderhub-service chart gets extraVolumes and extraVolumeMounts config added to influence its deployment pod and the container running binderhub.
Practical input

doing this is very similar to doing the changes in https://github.com/2i2c-org/binderhub-service/pull/96

in https://github.com/2i2c-org/binderhub-service/pull/96 the tpl function was used, it makes us able to provide configuration that is itself something the helm template can render, and that can be useful - so mimic its inclusion also for extraVolumes and extraVolumeMounts

to test and develop changes, make use of commands below - the tools/ commands are available in
z2jh / binderhub as well btw
# generates a .json version of the values.schema.yaml which is respected by the helm CLI
# (the YAML version is just to make work with the schema file, which must be .json, easier)
tools/generate-json-schema.py

# this script validates the chart's default values against the schema but also the
# example config values in tools/templates/lint-and-validate-values.yaml
tools/validate-against-schema.py

# test render the chart locally, showing specifically the rendered deployment.yaml template
# to modify developing extraVolumes and extraVolumeMounts
helm template binderhub-service --show-only templates/deployment.yaml

# use this command to render the chart, focusing on whats rendered after
# `volumes:` string in the deployment resource, updating every second.
# This is useful when modifying the template file to verify changes quickly
watch -n1 'helm template binderhub-service --show-only templates/deployment.yaml | grep -A10 "volumes:"'

This has become much lower priority due to https://github.com/2i2c-org/infrastructure/issues/4343. I think I'm mostly ok with this not being done as well, as it no longer necessarily serves any product function.


Access to @ui-devtools/tailwind after sponsorship

Hi @siddharthkp, can I have access to @ui-devtools/tailwind?

Hi!
my bad, the automated job failed 🙃
Added you to the team now, you should have an invitation in your emails/notifications


InvalidArgumentError (see above for traceback): Incompatible shapes: [1,35,63,36] vs. [128,116]

# to train resnet101 problem:
Traceback (most recent call last):
File "./faster_rcnn/train_net.py", line 109, in
restore=bool(float(args.restore)))
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 400, in train_net
sw.train_model(sess, max_iters, restore=restore)
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 255, in train_model
cls_prob, bbox_pred, rois =  sess.run(fetches=fetch_list, feed_dict=feed_dict)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 895, in run
run_metadata_ptr)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1124, in _run
feed_dict_tensor, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1321, in _do_run
options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1340, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Incompatible shapes: [1,35,63,36] vs. [128,116]
[[Node: gradients/sub_1_grad/BroadcastGradientArgs = BroadcastGradientArgs[T=DT_INT32, _device="/job:localhost/replica:0/task:0/gpu:0"](gradients/sub_1_grad/Shape, gradients/sub_1_grad/Shape_1/_1071)]]
[[Node: Mean_2/_1093 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/cpu:0", send_device="/job:localhost/replica:0/task:0/gpu:0", send_device_incarnation=1, tensor_name="edge_4286_Mean_2", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"]]
Caused by op u'gradients/sub_1_grad/BroadcastGradientArgs', defined at:
File "./faster_rcnn/train_net.py", line 109, in
restore=bool(float(args.restore)))
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 400, in train_net
sw.train_model(sess, max_iters, restore=restore)
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 142, in train_model
grads, norm = tf.clip_by_global_norm(tf.gradients(loss, tvars), 10.0)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gradients_impl.py", line 542, in gradients
grad_scope, op, func_call, lambda: grad_fn(op, *out_grads))
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gradients_impl.py", line 348, in _MaybeCompile
return grad_fn()  # Exit early
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gradients_impl.py", line 542, in
grad_scope, op, func_call, lambda: grad_fn(op, *out_grads))
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/math_grad.py", line 700, in _SubGrad
rx, ry = gen_array_ops._broadcast_gradient_args(sx, sy)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_array_ops.py", line 393, in _broadcast_gradient_args
name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 767, in apply_op
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2630, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1204, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
...which was originally created as op u'sub_1', defined at:
File "./faster_rcnn/train_net.py", line 109, in
restore=bool(float(args.restore)))
[elided 0 identical lines from previous traceback]
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 400, in train_net
sw.train_model(sess, max_iters, restore=restore)
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 112, in train_model
self.net.build_loss()
File "./faster_rcnn/../lib/networks/network.py", line 469, in build_loss
deltas = bbox_outside_weights * self.smooth_l1_dist(bbox_inside_weights * (bbox_pred - bbox_targets))
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/math_ops.py", line 865, in binary_op_wrapper
return func(x, y, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_math_ops.py", line 2629, in _sub
result = _op_def_lib.apply_op("Sub", x=x, y=y, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 767, in apply_op
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2630, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1204, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
InvalidArgumentError (see above for traceback): Incompatible shapes: [1,35,63,36] vs. [128,116]
[[Node: gradients/sub_1_grad/BroadcastGradientArgs = BroadcastGradientArgs[T=DT_INT32, _device="/job:localhost/replica:0/task:0/gpu:0"](gradients/sub_1_grad/Shape, gradients/sub_1_grad/Shape_1/_1071)]]
[[Node: Mean_2/_1093 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/cpu:0", send_device="/job:localhost/replica:0/task:0/gpu:0", send_device_incarnation=1, tensor_name="edge_4286_Mean_2", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/cpu:0"]]

Could anyone help on this? thanks a lot


Voice error

An error encountered: DisTubeError [VOICE_CONNECT_FAILED]: Cannot connect to the voice channel after 30 seconds
Version lasts

0.8.0

Try using discordjs/voice to join voice channel instead of distube

I did not understand

Use @discordjs/voice package to joim the voice channel. Follow discord.js guide/documentation

I already do it

Do it join the voice channel? If not, it is due to @discordjs/voice, not related to distube.

well I don't know because I noticed that from my console and I don't log when he joins the vocals
and it only does it once or twice on my users

I mean don't use DisTube#play. Just use @discordjs/voice method to join a voice channel and see if it is worked.
But I thought it happens every time you play music. If it just happens rarely, it is due to your connection to discord voice server at that time


Firefox for Android jumps forward one extra step on typing

Tested on Sony Xperia 10II with Android 10 and Firefox 82.1.1 and 83.0.0 beta

Please see https://github.com/viresh-ratnakar/exolve/issues/8
This is a known issue, and unfortunately I do not have any good ideas on how to fix without adding browser-specific workaround code that I'm reluctant to do.
The ideal thing to do would be to track down the issue in Firefox code itself and get it fixed there :-). If/when I get the time, I'll try to do a bit more research.

Closing out old bugs that I did take some action on. If I haven't fixed every aspect as expected, apologies, but please open a new bug (hard for me to parse out what remains to be fixed).


[Bug] A large number of seconds flashes before loading is shown

Insomnia Version: 6.3.2

Operating System: Linux

Details
When request is triggered, a large number flashes for a moment before loading is shown.
Animation

Frame

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

yeah? well, which activity do u need, lol. the issue is still there.

Sorry folks! This was incorrectly tagged as a feature request along with other bugs. I'm going back through and cleaning up, sorry for the inconvenience.

I was in this area of the codebase recently and I believe this bug doesn't exist anymore.  At least, I was working quite closely with it and I never saw this occur.
It's been a few years.  I'm more than happy to take another look, but I'll need some reproduction steps.
For example, just now I fired off a request to https://mockbin.org/delay/3000 and didn't see the above issue in the timer.


[B03985] Enable protection of project setting values

Coverage decreased (-0.6%) to 79.834% when pulling 021ce420e2250daf755e92226bc3d9af7fe2a759 on sprint12-b03985-pass into 9f8a1a31525d4b288217c45ed9947dc18fcdde58 on sprint12-staging.


bacalhau config set --config=config.yaml <key> <value> fails if config.yaml doesn't exist

Bug Description
This appears to be a regression introduced via https://github.com/bacalhau-project/bacalhau/pull/4466
bacalhau config set --config=config.yaml datadir /some/path
invalid argument "config.yaml" for "-c, --config" flag: the specified configuration file "config.yaml" doesn't exist

Wasn't regression but intentional. if the user miss typed a config file name I would prefer it fails than assuming it wrote to the correct file and create a new one instead

I thought the behavior we wanted was to create a config file if one didn't exist? Right now, the current behavior is that config set will create a config file in the repo is one doesn't exist. But, for example re-suing the above command, bacalhau will not create the config.yaml file.


Content Security Policy Header Extensibility

I was wondering if there is any plans for CSP Header extensibility.
I'm experiencing a minor errors in chrome where my favicon doesn't load. When my client and IDP are on different URLs, I'm seeing an error:

Refused to load the image 'http://mysite.com/favicon.ico' because it violates the following Content Security Policy directive: "default-src 'none'". Note that 'img-src' was not explicitly set, so 'default-src' is used as a fallback

The current implementation with CSP headers defined in the various IEndpoint classes is fairly rigid. There is a potential work around by writing some middleware to add the CSP header earlier in the request, but that doesn't seem ideal.  It be nice to have more control within the endpoint classes themselves, so that the existing directives (especially the script-src with the sha256), doesn't need to be recreated explicitly in middleware.
I was having another issue (more of an edge case) where my client, immediately upon authentication, redirects to another client. This fails because the redirected URL wasn't set within the form-action directive.

Refused to send form data to 'http://otherclient.com/' because it violates the following Content Security Policy directive: "form-action http://originalclient.com".

Ultimately, it be great if there was some sort of "CSP Directive Service" that could be extended to give more control with the headers.

The code for this is all there for you to modify: https://github.com/IdentityServer/IdentityServer4.Quickstart.UI/blob/release/Quickstart/SecurityHeadersAttribute.cs#L26

Ah... I missed that. Thanks!

I spoke too soon here. The SecurityHeadersAttribute gives me the control of the views, but it does not give control of the response of the AuthorizeResult endpoint. I'm able to reproduce the above problem running the quickstart Identity Server with the SecurityHeadersAttribute. The only change I made was that I added a client with an implicit flow and disabled consent for simplicity.
The client (localhost:5000) challenges and redirects to identity server (localhost:1941). I enter local valid credentials into the form and the page is post to http://localhost:1941/account/login?returnUrl=....
The /account/login action calls SecurityHeadersAttribute, but nothing happens because it is RedirectResult and not ViewResult. This is not an issue. The redirect sends the browser to http://localhost:1941/connect/authorize/login?client_id...XXX.
SecurityHeadersAttribute does not impact http://localhost:1941/connect/authorize/login (its not within MVC), so that endpoint responds with a 200 and includes the default CSP header:

default-src 'none'; frame-ancestors http://localhost:5000; form-action http://localhost:5000; script-src 'unsafe-inline' 'sha256-VuNUSJ59bpCpw62HM2JG/hCyGiqoPN3NqGvNXQPU+rY='

Because http://localhost:1941/connect/authorize/login responds with a 200, Chrome automatically tried to fetch http://localhost:1941/favicon.ico. The CSP policy doesn't allow for this and you can see in the Chrome network trace where the status there is "(blocked:csp)".
At a minimum, I'd suggest adding to img-src self directives anywhere where IdentityServer sets "default-src 'none',
That would resolve the fav icon issue, but the only way to get control of the CSP headers in the AuthorizeResult response is to set up middleware to watch for the paths and write the CSP header (replicating the logic where origin of redirect uri is added) before AuthorizeResult has a chance to set it.

A less important detail, but from what I can tell, Chrome does not write the fav icon error to the console (it only shows up in network trace), unless there are additional CSP errors. When there are additional network errors, the favicon issue is written to the console with the following message:

Refused to load the image 'http://localhost:1941/favicon.ico' because it violates the following Content Security Policy directive: "default-src 'none'". Note that 'img-src' was not explicitly set, so 'default-src' is used as a fallback

So what's the actual impact of this? Seems like a chrome issue more than anything...

Fair point on the favicon issue. Adding img-src {formOrigin} to the header string would mitigate it, but in the end, there is no real user pain and Chrome probably shouldn't even make that request when CSP forbids it.
The genesis of this for me was the other CSP issue I had. When my Identity Server redirects to client A, the endpoint designated to be redirected to performs an action (merging accounts), and upon success immediately redirects to Client B. The "secured area" of Client A has no UI.
I'm finding that when it redirects from Identity Server to Client B via Client A (with no user intervention at Client A),  the browser rejects this because the AuthorizeResult CSP header doesn't allow form-action to client B (even though the actual form post is to Client A). My guess is this could also be solved with a hidden form post from Client A, but I've yet to try that yet.
With these two specific issues, one may be more of a Chrome issue and the other is perhaps an edge case. I was just originally wondering that given these sort of cases may arise, if there were plans for extensiblity of CSP headers created by AuthorizeResult. But at the same time, I could also see how the complexity of adding extensiblity there may not be justified by the amount of value added in return, especially when there are workarounds for everything I've identified.

Yes, for this other issue I think it's the same one we saw and I addressed it today: https://github.com/IdentityServer/IdentityServer4/issues/659

Yes, that looks like that will solve it. I should be all set with the next release. Thanks

Hi,
I'm using IdentityServer4 v1.5.2 and IdentityServer4.AspNetIdentity v1.0.1 (the latest from nuget) and I am still seeing 'unsafe-inline' when hitting the /connect/authorize/login URL. Was this actually solved and if so what could I be missing?
I need to resolve as it's being picked up by our veracode security scan.
Thanks
Mike

We are using IdentityServer4 v2.1.1 and are getting the "Refused inline script..." on the '/connect/authorize/callback' page on the Ipad's Safari and Chrome.  Therefore it just sits on the callback page.
I've tried intercepting/modifing the CSP header at various places without any luck. Do we need to download the source and reference the project so that we can customize the CSP header? etc?  Any advice is greatly appreciated!

Thanks,
Jay

I would also like CSP header extensibility of some kind. I would like to add an allowed source to the frame-ancestors policy, because I have the following scenario:

Two apps that authenticate against IS4 -- let's called them 'A' and 'B'.
B must be framed within A so they appear to be the same application to the end user.
B is not accessible until user authenticates to A, which creates a session on the IS4 server with cookie authentication to ASP.NET Core Identity.
B should then (ideally) always be connecting to IS4's Authorize endpoint under circumstances where the user has a valid cookie-based session/auth. If the cookie-based session on the IS4 is not valid, I want B to tell A to force re-authentication and redirect to IS4 login. I do not want to embed the IS4 server's login screen into application A in this case.

As far as I can tell, there used to be a nice option for this in IS3, but there is no simple way to accomplish what I need in IS4:
https://identityserver.github.io/Documentation/docsv2/advanced/csp.html
Is that correct? Do I have a valid need here?

Shoot -- I just updated IS4 to the latest version and everything is working. It appears that you've relaxed the frame-ancestors policy for the Authorize endpoint. Sorry!


[ROCKETMQ-19] Synchronize LinkedList.add() in multi-thread

[ROCKETMQ-19] Synchronize LinkedList.add() in multi-thread
JIRRA ISSUE: https://issues.apache.org/jira/browse/ROCKETMQ-19

@Zhang-Ke Thanks for your PR, Could you please verify mentioned problem, no matter using unit or other ways. At last, please fix the conflict problem.

Duplicated with  [https://github.com/apache/incubator-rocketmq/pull/22]PR #22 (url)

Thanks for your ISSUE review before PR @Zhang-Ke


set uncheckedNullType correctly for iterators of Java collections/arrays

Currently, iteration of a Java collection or object array results in a crash if the for loop encounters a null element.  It should use uncheckedNullType.

Looks like the fix for this involves rolling back the previous fix to #4162, which looks to have been rubbish.

Done, I think. That only took me all day :-(


[REFACTOR] Build Game Status Query

Use a pretty set and not strings to generate status query

join would be a good idea for this

@emilyseibert is there any way for me to test this?

nope, just as long as the games come back for you


[FEATURE] Seek to detection should also autozoom.

"Clicking on the detection number highlights the detection but does not jump to zooming in to this detection. It would be helpful to have the zoom to detection option through one of these methods, or another. "
While some users want this, it may confuse others who need to see full frame context.  Perhaps this should be a hotkey and bound to a track list item button, but not the default behavior.
For example: arrow keys select track, but shift-arrow selects track with zoom?

I think this should be a user setting instead of an extra/special keypress


🧑💻 : 🧑💻 : Challenges  for each topic

:red_circle: Title :
:red_circle: Enhancement Aim :
:red_circle: Changes :
🧑💻 : Challenges  for each topic
Screenshots 📷

✅ Details to Include When Taking the Issue:
Name :
Bhuvaneswari kapuluru gssoc extd

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

can you describe more

Conduct  challenges with questions and options and show result
On Mon, Oct 21, 2024, 2:29 PM Rahul Mahato @.***> wrote:

can you describe more
—
Reply to this email directly, view it on GitHub
https://github.com/iamrahulmahato/master-web-development/issues/1527#issuecomment-2426045151,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHOWNHDR6MMKCJJBXHENA3LZ4S66FAVCNFSM6AAAAABQJKN6YWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRWGA2DKMJVGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

in this format sir


About map coodinates and scale

Hi everybody.
I am trying to work with the movement of players extracted from replays, but allthe coodinates seems to be positives and none under 9k (I was told that coord = CBodyComponent.m_cell[XY] * 128 + CBodyComponent.m_vec[XY]).
So maybe the scale of the map is diferent than the used by the getLiveLeagueGames API or I am doing something wrong.

m_cell[XY] is a positive value, from 0 to 255 iirc.
m_vec[XY] seem to be in the range from 0 to 255.999, so it seems the "times 128" is incorrect.
Now if you would do  CBodyComponent.m_cell[XY] * 256 + CBodyComponent.m_vec[XY]), you would get values ranging from 0 to 64k on both axes. If there are none under 9k, you are definitely doing something wrong.
I think the rest must be a scaling / translation issue. What is the range of values in getLiveLeagueGames()?

I implemented a basic map view in clarity analyzer today, to check this once and for all:
CBodyComponent.m_cell[XY] * 256 + CBodyComponent.m_vec[XY] - 32768.0
calculates everything correctly and sets the origin (0/0) to the middle of the map.
Dota only uses around a quarter of the available space, it's map is around 29000 units high/wide.

Thanks for your help.
Also, the scale in getliveLeague games the map seem to be around 16000 units high and wide

Then I think halving what you get from my calculation should give you what you want.

Hey, quick heads up: Cells are only 128 units wide/high, even if the vector is sometimes more than 128^2 in length, so the computation is:
CBodyComponent.m_cell[XY] * 128 + CBodyComponent.m_vec[XY]
with the origin being in the bottom left of the map.

So,everything is going to be in positive range right?

Yes, but you can also translate the coords to have the origin in the middle of the map, no problem.
There's a CWorld entity, which contains properties for the min/max extent of the world, which goes from approximately -9000 to 8200 (for X), so I guess Valve also do translate.


1.1.0rc PyPi wheels missing

Describe the bug
The publish to Pypi run failed
https://github.com/executablebooks/sphinx-book-theme/actions/runs/6870341669
Reproduce the bug

List your environment

So it did! Thank you for reporting.

Ugh, it's two failing tests that arguably aren't important for the release to be useful to people.
One is a bug in the sphinx-theme-builder that makes it impossible to build the wheel on windows machines (but we use Linux to build wheels)
One is a bug in the documentation build somewhere (link here). I think we should just build and push it. I'll try to resolve the docs build and remove the dependency on the windows build.

One is a bug in the sphinx-theme-builder that makes it impossible to build the wheel on windows machines (but we use Linux to build wheels)

I'm reluctant to put out an sdist that won't build on Windows, so I'm holding until the upstream bug here is fixed. Let me chase that up ...

Here is the issue I opened in the stb repo:

https://github.com/pradyunsg/sphinx-theme-builder/issues/46

There is a giant pile of buggy old releases being pulled in by 1.0, I’m pretty desperate for having a 1.1.0rc wheel on PyPI.

@agoose77 doesn't this build failure just mean that people won't be able to develop this theme on windows, not that they won't be able to use it? If that's the case I suggest we just cut a release as an RC. We can fix the windows but in an RC2.

OK - I was hoping we could just get a proper release done by fixing
upstream so that our sdist's aren't buggy. However, we can make an rc, to
unblock workflows.
On Thu, 23 Nov 2023, 16:21 Chris Holdgraf, @.***> wrote:

@agoose77 https://github.com/agoose77 doesn't this build failure just
mean that people won't be able to develop this theme on windows, not
that they won't be able to use it? If that's the case I suggest we just
cut a release as an RC. We can fix the windows but in an RC2.
—
Reply to this email directly, view it on GitHub
https://github.com/executablebooks/sphinx-book-theme/issues/782#issuecomment-1824686020,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJQZHJ7OZ4GFMRXMYV6BWDYF5ZXVAVCNFSM6AAAAAA7NNVOR2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRUGY4DMMBSGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


write operation not permitted

Error on Android device：write udp <IP_ADDRESS>:40200-><IP_ADDRESS>:53: write: operation not permitted
Here is the code snippet：
var conn net.Conn
if conn, err = d.dialer.Dial("udp", server); err != nil {
log.Error(err)
return
}
defer conn.Close()
var rtt time.Duration
r, rtt, err = d.localDnsClient.ExchangeWithConn(msg, &dns.Conn{Conn: conn})

This is OS limitation problem. You should check your Android device network permissions.

/close
On Tue, 21 Jul 2020, 08:11 Yasar Alev<EMAIL_ADDRESS>wrote:

This is OS limitation problem. You should check your Android device
network permissions.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/miekg/dns/issues/1137#issuecomment-661656062, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACWIWYZD647SKDGUEHNFW3R4UWSTANCNFSM4PDBGJEA
.


added docs for near api doc

Created docs and example to help demonstrate how to use the call and view methods in NEAR scocial

@doriancrutcher the Near API, but the file is empty. 🤔   I think something happened when you committed your changes.


app crashes on launch

The app has been crashing on launch since I updated from 1.1.59 to 1.1.65. I have attached a log from 1.1.70.
log.txt

Can't replicate this but

Also thanks for making this ticket so quickly

1.1.71 fixed the issue for me. Thanks

Damn... Just updated to 1.1.65 before seeing this.
Regardless, I'm liking the app.


Adding gatsby-plugin keyword in package.json

Hello 👋
As part of https://github.com/gatsbyjs/gatsby/issues/14013, I would like to add the keyword gatsby-plugin to the package.json file of your source plugin.
It is documented in https://www.gatsbyjs.org/contributing/submit-to-plugin-library/ if you would like to know more about it.
If you accept to merge this PR, could you also publish a patch version of your source plugin on NPM? Let me know if I can assist in any way with this. 🙂
Thank you very much!

Oh! Merging this now.  I'll update the NPM package as soon as I can.

I updated NPM,

Awesome, that was quick and efficient! Thank you very much @ldd! 💯


Printed labels are wastefully big

Hi
I have finally found a solution how to talk to my USB Brother QL-700 from docker over tcp.
Unfortunately the labels are too big and I have not found a way how to modify it.
See the attached photo.
The smaller label is the optimal size for me, drawn in Gimp and sent to the printer directly via
brother_ql -m QL-700 -b network -p localhost print -l 62 stitek.png

It must have width of 696 px to not be resized by the software automaticaly.
The larger one is printed from inventree.
Can anyone help me with the configuration, please?
Thanks in advance

@ladmanj are the labels printed from the same template? They appear to be different labels

Hi, I mentioned it above. The smaller one is "hand-drawn" in gimp, exported to png and sent to brother_ql outside the InvenTree completely.
That is only for size comparison. That's the size i would like.
The bigger one is the smallest I can ever print out of InvenTree itself.
My English is far from perfect, so I don't know if it's described well.
Cheers
Jakub

Maybe it's worth mentioning, that I'm completely new to InvenTree. It's my first weekend of experience what so ever.
I have no items in the database yet to print about. Only the stock locations.
J.

Have you tried adjusting the label "width" and "height" properties in InvenTree?
For example, on the demo server:

Have you tried adjusting the label "width" and "height" properties in InvenTree?
For example, on the demo server:

Yes, I tried that. The printout size is always the same.
The same result as if i sent a png of 75x75 px directly to the brother_ql.
If the width in pixels is smaller than the physical width of the paper, then it's automatically expanded to fit.
Unfortunately it means that the length of the final label gets also expanded (In 1:1 ratio of course).
That's unwanted so the png file must be as wide as the paper itself. 62 mm in my case, which translates to 696 px.
J.

Yes, I tried that. The printout size is always the same.

Hmm, so the printed size is independent of any width / height setting you set?
What version of InvenTree are you running?
Unfortunately I do not have access to a printer to run any tests on, but perhaps there is an obvious bug

What version of InvenTree are you running?

# Version Information:
InvenTree-Version: 0.15.4
Django Version: 4.2.12
Commit Hash: fa1a9da
Commit Date: 2024-06-16

Database: postgresql
Debug-Mode: False
Deployed using Docker: True
Platform: Linux-6.6.30-gentoo-x86_64-x86_64-with
Installer: DOC

Active plugins: [{'name': 'InvenTreeBarcode', 'slug': 'inventreebarcode', 'version': '2.0.0'}, {'name': 'InvenTreeCoreNotificationsPlugin', 'slug': 'inventreecorenotificationsplugin', 'version': '1.0.0'}, {'name': 'InvenTreeCurrencyExchange', 'slug': 'inventreecurrencyexchange', 'version': '1.0.0'}, {'name': 'InvenTreeLabel', 'slug': 'inventreelabel', 'version': '1.0.0'}, {'name': 'InvenTreeLabelMachine', 'slug': 'inventreelabelmachine', 'version': '1.0.0'}, {'name': 'InvenTreeLabelSheet', 'slug': 'inventreelabelsheet', 'version': '1.0.0'}, {'name': 'DigiKeyPlugin', 'slug': 'digikeyplugin', 'version': '1.0.0'}, {'name': 'LCSCPlugin', 'slug': 'lcscplugin', 'version': '1.0.0'}, {'name': 'MouserPlugin', 'slug': 'mouserplugin', 'version': '1.0.0'}, {'name': 'TMEPlugin', 'slug': 'tmeplugin', 'version': '1.0.0'}, {'name': 'Brother Labels', 'slug': 'brother', 'version': '0.9.0'}]

Unfortunately I do not have access to a printer to run any tests on
I will be glad to test it for you. Just my paper supply isn't infinite.
How can I help you?

Now I printed it again and suddenly it's ok.
There is probably a combination of any/all of the following:

Default values are not used unless changed back and forth
The settings are not applied synchronously (the next immediate print doesn't respect the settings)
The resize mechanism is triggered in a unfortunate way

In any case I will be continuing testing and reporting.

@ladmanj ok glad you got it working, please keep an eye out for any clues towards making this issue repeatable (so we can fix it).
For now I'll close this out, but feel free to check back in if you have any further information.


Circular crop mask

Hey,
Thanks for the package and the new null safety update.
There is a PR that adds a circular crop mask and it would be great if you could merge it to the package as well. #59
Thanks again.

Check this https://www.youtube.com/watch?v=1aRwxnX5nms


.github/workflows/update-terraform-providers.yml: re-enable

Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

dry-run; https://github.com/NixOS/nixpkgs/actions/runs/3203347859

Traceback (most recent call last):
  File "/nix/store/w67169jnq7c64x2kbk5xibqh9figky4k-update.py", line 244, in <module>
    main(args.max_workers, args.keep_going, args.commit, args.packages)
  File "/nix/store/w67169jnq7c64x2kbk5xibqh9figky4k-update.py", line 220, in main
    confirm = input('Press Enter key to continue...')
EOFError: EOF when reading a line
Press Enter key to continue...
Error: Process completed with exit code 1.

oops, not sure how I missed that.

now it's working: https://github.com/NixOS/nixpkgs/pull/194984


build: switch to rollup for bundling

Luis -
First of all, love the utility; thank you so much!
Many "libraries", utilities, etc. (most notably React) have begun to switch to Rollup for bundling their assets into a small, neat module that can be easily consumed by other dependents.
This PR simply adds in Rollup support, and bundles as a UMD build. Practically, there won't be a difference in the output (yet... !) because you were already distributing as a UMD, but this does allow us to modularize the code base, and have utilities (e.g. the async forEach) externalized and not contained within a single monolithic file.
Let me know if there's anything else I can explain, or help figure out!
Note, this article (by the author of Rollup) may be interesting to you.
Thanks!

@luisvinicius167 any interest in getting this merged in? Thanks!

@DSchau my apologizes for long response time! Your do an awesome work, thanks!

Awesome news @luisvinicius167; thanks so much! I may do a PR this weekend with some of those changes/improvements (e.g. modularizing the code base) I was referring to earlier. Have a great day!

@DSchau Great!


Move #removeFromConnectionsAndListeners into the finally block.

As mentioned here, a http request should be removed from the connections and listeners lists even if HttpResponseListener#handleHttpResponse(clientResponse) throws an exception. This mirrors the behavior for HttpResponseListener#failed(e) in lines 239 and 248 and is important because those list are checked  to see whether the request is still pending.

@crykn Good catch!


🐛 Cannot extends a jsonc configuration file

Environment information
CLI:
  Version:                      1.6.3
  Color support:                true

Platform:
  CPU Architecture:             aarch64
  OS:                           macos

Environment:
  BIOME_LOG_DIR:                unset
  NO_COLOR:                     unset
  TERM:                         "xterm-256color"
  JS_RUNTIME_VERSION:           "v18.16.1"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "yarn/4.1.1"

What happened?

Define a biome.jsonc configuration file.
Define a second configuration file extending from the first.
Witness an error when loading that second file.

Expected result
If jsonc is supported to define configuration files, it should be possible to extend jsonc-defined configuration files, otherwise it’s a bit weird.
Code of Conduct

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

Thank you, @KittyGiraudel. Can you share a snippet of the configuration you're using?

I updated my original comment to reflect the output from biome rage ran on this reproducible test case I just created: https://github.com/KittyGiraudel/biome-2279
I noticed that the problem only occurs when the JSONC configuration file contains comments. So it‘s not the format per se that is the problem, but the presence of comments that make the extends choke.
I hope this helps. :)


Add :reproducible_resource_limit attribute

This doesn't multiply by 1000, unlike :rlimit. The corresponding CLI flag is /reproducibleResourceLimit.
I'm a little unsure about the name. It's a bit of a mouthful. Maybe leaving out "reproducible" would be better.
Fixes #795.

I simplified things by removing the backward-compatibility layer. We'll want one in Dafny, but it's not necessarily needed in Boogie, given the very few users of the feature.


Add Tasmanian Mismatch tool to usegalaxy.org

Adds the Tasmanian Mismatch tool to usegalaxy.org. Requested by @bwlang.
Installation sequence for tool-installers

[ ] Test using @galaxybot test this
[ ] Inspect CI output for expected changes
[ ] Deploy using @galaxybot deploy this if test install was successful
[ ] Merge this PR

@galaxybot test this

@galaxybot deploy this


Add CHD code list link

https://github.com/nhs-r-community/chd-codes

It's a code list https://health-inequalities.nhsrcommunity.com/data_sources.html#code-lists


Change field order in api NetworkPolicy from float to interge

Expected Behavior
The field order in networkpolicy should be interge not float64
Context

I am developing with calico and introducing some domain-related crds to make my user better working with calico. However in k8s, it doesn't allow me to use float as a field in spec.
refer to https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/api-conventions.md#primitive-types
I have to make it be a interge and convert it to float before using calico api.
How do you think about using float or interge as the type of field order?

I do not think we would want to use an integer because that would be problematic for upgrades.
What you linked to suggested to not use any float, so I don't see why switching from float64 to float would be any better.
Is there actually something that is blocking you from using a float? The NetworkPolicy.projectcalico is being stored as a K8s CRD so I don't believe there is actually anything preventing you from using a float64 other than the guidelines you've linked to.

Yes, I agree. We maybe should have used an integer to start with based on those recommendations, but I don't think we can change it now since we can't upgrade a float64 to an int.

@tmjd user can choose etcd or k8s as backend of calico, and we decide to support both of them. So we use a CRD to offer a new layer of abstraction and depending on user's choice, we create a calico CRD or use clientv3. However, when we wanted to create new-level api, we found the field type of order is not supported in k8s. Right now we just use integer because it is convertable from float64 to an integer

So we use a CRD to offer a new layer of abstraction and depending on user's choice,

The calicoctl tool already provides an abstraction on top of the different data store backends. I don't think we can change this field since it would break existing users.


Allow you to get the layername you are on.

Basically, if you are on chest screen the layer name will tell you "chest_screen", basically this is just a useful utilit.y

This method only works for UIScene derived screens (which actually contain a std::unique_ptr<ScreenView>).
A much simpler approach is to call the AbstractScene::getScreenName virtual function.

Thanks, as a general rule, this project has been defining memory layouts of classes, because it makes it easier to understand the class, and easier to preview the structures size and layout with tooling. Im not completely against having a build access macro for testing, but I don't think it should be in final committed code if that makes sense

This is the standard of code that I am using in this project, updating isn't a thing this project will do, except for major version releases. It takes an insignificant amount of time to just write a the member variables, there are even tools you can use to help you do this too.

no offense but asm is right, then you gotta updated the comments too and all the padding

and like not even just updating but if you wanna add a member like in between 2 members that already exist u gotta recalculate the padding

I have to be honest, its not a particularly hard thing to do, nor does it take more than a minute. You can use tools like: https://github.com/Viladoman/StructLayout which allow you to preview the sizes and positions.

thoughts on making it as easy as possible?

Forced to use Visual Studio

"screenview->visualtree->root->layername" this is dumb.
May I suggest"screenview->visualtree->chickenbutt"?

🤬


pex does not respect absense of --resolve-local-platforms on Mac M1, builds using local platform anyway

On an M1 Mac, when I don't specify --resolve-local-platforms, pex will still build dependencies using the local interpreter matching the --python flag, even when binary wheels exist.
When using rosetta Python (x86 emulation), this issue does not exist.
Reproduction - I have two venvs, one for m1 native (arm) and one for rosetta (x86)):
Note the binary wheel exists for grpcio for requested --platform. Using --complete-platform does not help.
# x86 works as expected - pulls the binary wheel and makes the pex
% pex --platform=manylinux2014_x86_64-cp-38-cp38 --python=python3.8 --pip-version=22.2.2 --resolver-version=pip-legacy-resolver grpcio==1.47.2 -o grpcio.pex
(x86-python) ~/src/issues/pex-python-flag
% unzip -l grpcio.pex | grep 'whl/$'
        0  01-01-1980 00:00   .deps/grpcio-1.47.2-cp38-cp38-macosx_10_10_x86_64.whl/
        0  01-01-1980 00:00   .deps/grpcio-1.47.2-cp38-cp38-manylinux_2_17_x86_64.manylinux2014_x86_64.whl/
        0  01-01-1980 00:00   .deps/six-1.16.0-py2.py3-none-any.whl/

# same command in arm just starts building a wheel
 pex --platform=manylinux2014_x86_64-cp-38-cp38 --python=python3.8 --pip-version=22.2.2 --resolver-version=pip-legacy-resolver grpcio==1.47.2 -o grpcio.pex -v
pex: Building pex :: Resolving distributions (grpcio==1.47.2) :: Resolving requirements. :: Resolving for:
  /Users/shalabh/src/issues/pex-python-flag/arm-python/bin/python3.8
pex: Hashing pex                                                                                pex: Hashing pex: 90.3ms
pex: Isolating pex: 0.1ms
pex: Building pex :: Resolving distributions (grpcio==1.47.2) :: Resolving requirements. :: Building distributions for:
  BuildRequest(target=LocalInterpreter(id='Users.shalabh.src.issues.pex-python-flag.arm-python.bin.python3.8', platform=Platform(platform='macosx_12_0_arm64', impl='cp', version='3.8.16', version_info=(3, 8, 16), abi='cp38'), marker_environment=MarkerEnvironment(implementation_name='cpython', implementation_version='3.8.16', os_name='posix', platform_machine='arm64', platform_python_implementation='CPython', platform_release='21.6.0', platform_system='Darwin', platform_version='Darwin Kernel Version 21.6.0: Mon Aug 22 20:20:05 PDT 2022; root:xnu-8020.140.49~2/RELEASE_ARM64_T8101', python_full_version='3.8.16', python_version='3.8', sys_platform='darwin'), interpreter=PythonInterpreter('/Users/shalabh/src/issues/pex-python-flag/arm-python/bin/python3.8', PythonIdentity('/Users/shalabh/src/issues/pex-python-flag/arm-python/bin/python3.8', 'cp38', 'cp38', 'macosx_12_0_arm64', (3, 8, 16)))), source_path='/Users/shalabh/.pex/downloads/resolver_download.gmefuolk/Users.spex: Building /Users/shalabh/.pex/downloads/resolver_download.gmefuolk/Users.shalabh.src.issues.pex-python-flag.arm-python.bin.python3.8/grpcio-1.47.2.tar.gz to /Users/shalabh/.pex/built_wheels/sdists/grpcio-1.47.2.tar.gz/c7f888b067f2b1833a5670f20eeac2c6063039a23a99b799929de065c1b18692/cp38-cp38-macosx_12_0_arm64

<CTRL-C>

This has something to do with the --python flag because if I remove that, even arm quickly builds the pex from the pypi wheel.

@shalabhc this is by design but obviously not what you expect. Each --python, --interpreter-constraint, --platform and --complete-platform adds to a list of targets resolved for; i.e. they each add a "platform" to a multi-platform PEX.

The question then is what you are trying to do by adding --python?

More spin: --resolve-local-platforms is a modifier for --platform and --complete-platformand that is all. It has no bearing on either--pythonor--interpreter-constraint` which select 1 and 1+ local interpreters unconditionally. After all that selection you are left with 1 or more real local interpreters and 1 or more foreign platforms. For each real local interpreter if there is no wheel, that interpreter will be used to build an sdist into a wheel for that real local interpreter.
Does that make sense?

Ah I did not realize this. I assumed --python=python3.8 selected the interpreter on the target (it's under "PEX target environment options:"). Specifically I thought it meant "when this pex is run, ensure it is run with a binary called python3.8 and fail if python3.8 is not present in the target environment". I did not think it affected how the build itself was done.
I can exclude --python=python3.8 but how do I tell it run the pex with python3.8 in case the target has multiple interpreters present?
Here I think it makes sense because grpcio has a wheel for x86 but no wheel for arm. The only confusion that remains is: what is the point of building the wheel for arm when it is not going to be used? IIUC it will build the arm wheel but then discard it because the --platform specifies that only the manylinux wheel should be included in the pex.

The point is Pex thinks you're asking it to build a multiplatform PEX file that works on Linux x86_64 and Mac arm.

To require the PEX runs using python3.8 you need not do anything. Pex gained the ability to probe compatibility on boot a while back. That can take time though, so you can steer by setting the --python-shebang option. More direct, if you control the PEX execution command, change it from my.pex ... to /this/exact/python my.pex ....

Indeed I just noticed it is a fat pex on macos x86 (two wheels for grpcio):
 unzip -l grpcio.pex | grep 'whl/$'
        0  01-01-1980 00:00   .deps/grpcio-1.47.2-cp38-cp38-macosx_10_10_x86_64.whl/
        0  01-01-1980 00:00   .deps/grpcio-1.47.2-cp38-cp38-manylinux_2_17_x86_64.manylinux2014_x86_64.whl/

I knew about multi platform pexes and created them intentionally with multiple --platform flags, but did not realize --python was also adding the local platform.
Thanks for the explanation!

You're welcome. I'll close this as answered, but please re-open if you have more questions.


Cannot use images from private/secure registries

Hi - this is likely not a bug, rather my just not knowing how, but I couldn't find the answer elsewhere.
We are running Tekton through the Redhat Pipelines operator (1.8.2) on Openshift 4.10.39 (Kubernetes Version: v1.23.12+7566c4d), which uses Tekton Pipelines: v0.37.5
We have the need to use custom images from our Artifactory instance in tasks.  I couldn't get it to load those images, so tried to take Artifactory out of the problem space and simply pull from the internal Openshift registry, but received a similar error.
The error message pulling the image is:

Failed to pull image "default-route-openshift-image-registry.apps.cluster.fqdn/openshift/pipeline-tools:latest": rpc error: code = Unknown desc = pinging container registry default-route-openshift-image-registry.apps.cluster.fqdn: Get "https://default-route-openshift-image-registry.apps.cluster.fqdn/v2/": x509: certificate signed by unknown authority

I tried the solution mentioned at https://github.com/tektoncd/pipeline/issues/1171, but the operator overrides any attempted changes to the deployment.
Appreciate any insight.  Thank you.

Hi @elfner
Currently, the solution for insecure registries in Pipeline involves mounting the certificate through a ConfigMap and envrionment(SSL_CERT_FILE).
Additionally, the Tekton Operator project also provides support for this feature. You can refer to this pull request for more information: https://github.com/tektoncd/operator/pull/246.
Since you are using the Red Hat-provided Pipeline Operator, I cannot guarantee its compatibility with this feature. I recommend reaching out to the relevant individuals in the OpenShift community for assistance and clarification on this matter.

Thank you.  In the meantime, I was able to get it to work.
It all boiled down to defining the service account correctly to reference the secret with the .dockerconfigjson.
Before it just referenced:
secrets:
  - name: xxx

but it needed to reference it as:
imagePullSecrets:
  - name: xxx

You can use a podTemplate to set the corresponding serviceAccountName, and then associate the imagePullSecrets with the serviceAccountName in the configuration to achieve your desired outcome.

/close


Command create doesn't set Authorization header when token is informed.

According with the Dynamic Client Registration spec (http://openid.net/specs/openid-connect-registration-1_0.html), the registration endpoint could need an initial access token to protect it.
In the usage, this feature was planned but it does not work.
Usage: openid-client-create [options] <registration_endpoint> [properties...]

Options:

-t, --token [value]  use as value initial access token

When I set the token, like:
$ openid-client create -t foo https://op.example.com/client/register redirect_uris=https://lvh.me:3000/cb client_name=Foobar

the Authorization header is not setted.

Thank you for noticing, i merged your PR and published a patch version.


[sshshim] turn on gate

Summary
Turns on this feature for all devbox cloud shell users.
How was it tested?

started a fly VM
started a devbox cloud shell to that VM
inspected ~/.config/devbox/ssh/shimsto see files created andlogs.txt` to see operations running well
MUTAGEN_DATA_DIRECTORY=~/.config/devbox/mutagen sync list to inspect active sync sessions.
stop VM
MUTAGEN_DATA_DIRECTORY=~/.config/devbox/mutagen sync list to confirm no more active sync sessions.

Current dependencies on/for this PR:

main

PR #376   👈

This comment was auto-generated by Graphite.

I just want it there for a few days, while y'all use it. In case it blocks someone, then the switch can unblock them. I'll remove the code in due course.


Add a table to track organizational salaries over time

Basically, the equivalent of https://issarice.com/givewell-executive-compensation but more broadly for more than just "executives"?
I need to brainstorm this more with @riceissa and @sebastiansanchez123.

This sounds like a good idea, but I think most orgs are pretty secretive about salaries so I am worried we wouldn't be able to collect enough data for the table to really be useful.

Salaries could also be associated to positions, like in the Holden/Elie case (which could show up when one goes to Holden's or Elie's page).

there's that h1b visa site that has salary info for various orgs


Enable logging of conda env creation

Closes  https://github.com/theacodes/nox/issues/345.

Thanks, don't think nox.options has the runtime options, that value should be passed down to the ProcessEnv class from here
https://github.com/theacodes/nox/blob/cdb391fa24a766138e596ee6b819ae49add5ec4f/nox/sessions.py#L490-L490
(there you can use sefl.global_config.verbose)
Also, maybe good to enable this for virtualenv as well https://github.com/theacodes/nox/blob/cdb391fa24a766138e596ee6b819ae49add5ec4f/nox/virtualenv.py#L404-L404

Hi @Tolker-KU, I'd love to merge this. Do you think you can address the comments from @stsewd above? No rush, of course!

Hey folks, It looks like this PR has gone stale. If you're still interested in getting it merged, let me know and we can get this started again.


Questions about the paper

Hey guys, great work! Thank you for publishing the paper. Very impressed with your results, especially for 250M and 780M models - they look super cool!
I've got several questions:

Am I right, that in your opinion, you got better results than GoLLIE because:

Your prompt is better than GoLLIE and you found out that is's important to generate an entire output (aka full contextual lengths, instead of what they are doing), and you made this entire algorithm actually viable thanks to LCS, etc
You resolve the problem of Back Tokenization (which also adds some additional performance)
Did I forget something???

When reading the paper, I got the impression that the Hierarchical Matching algorithm can be replaced with teacher forcing - you just make the model generate the correct word (if it's time to generate the next word of the sentence) or you force the model to make entity prediction aka generate "(" then some entity and then ")". Why did you do the "Hierarchical Matching Algorithm", am I missing something?

Thank you very much for your response
P.S. I am also working on NER-pretraining using artificial data. But I am primarily interested in pre-training BERT-like encoders with great token-level embeddings so mostly Feature Extraction task. Recently we released our models and got the SOTA few-shot results for NER. You can find them and the paper here: https://huggingface.co/collections/numind/paper-65e1f6e14639e2a465af823b

Thank you for your appreciation of our contributions.
Your grasp of our methodology aligns closely with our intentions. To put it more directly, in terms of the final $F_1$ score, the improvements over previous approaches are primarily reflected in:

Recall: The model is guided to make judgments on every token in a sentence (including those in unlabeled texts), which helps recall more entities.

Precision: The context of an entity often plays a crucial role in its identification; for example, words like "go to", "travel to" or "run to" are often followed by a location name. Including contextual information into training is also beneficial.

We observed improvements in both recall and precision, which in turn led to an increase in the $F_1$ score.

The teacher-forcing method you mentioned is indeed a good approach. Similarly, during our experiment, we attempted to construct a rule-based constraint decoding method, where only the correct word would be considered during inference. In this process, we made the following observations about the rule-based constraint decoding method:

Its effectiveness is comparable to that of the Hierarchical Matching algorithm, as reflected by the similar $F_1$ score.
The efficiency of inference is lower, and it's challenging to implement an efficient batched inference algorithm.

Given that auto-regressive generation itself is not very efficient, we prefer an offline and faster method, which is why we ultimately chose such a optimized matching algorithm.

I'm quite interested in your work, and gain more insights in dataset creation after reading your paper. NuNER also provides better pre-trained models for NER applications to choose from. Thank you for your contributions in terms of data, methods and pre-trained models!

@yyDing1 Thank you very much for sharing your intuition about your work :)
Usually, papers are so formal and crammed with information, that it's difficult to get the intuition behind some actions. Very happy I was able to understand yours from your paper :)
Good luck with your future research! Looking forward to it

Btw feel free to use the dataset from our paper - it's like Pile but like 15 times bigger and contains slightly other domains and texts (as it's based on C4). Maybe you can even merge it with Pile and train the model on it
https://huggingface.co/datasets/numind/NuNER

It's really rewarding to hear that the intuition behind our research was clear and accessible. That's always a goal we strive for and your feedback means a lot.
We are excited about the prospect of developing a stronger generative Named Entity Recognition (NER) model based on the larger dataset that covers a broader range of domains. This is already under progress, and this model will be released soon.
Best wishes for your ongoing and future projects as well.

Hello, we have trained a more powerful model by mixing the data from Pile-NER and NuNER, achieving an $F_1$ score of 64.12 in a zero-shot setting, an improvement of 0.6 compared to using Pile-NER alone. Moreover, benefiting from the broader coverage of NuNER, we notice that the model can recoginize more entity categories accurately.
GNER-T5-large-v2 are released here. More models will be released soon.

I combined all the data and performed some data processing work, primarily focusing on external entity sampling: I sampled categories of entities that were not present in the sentences as additional queries. Thanks for your advice; I will optimize the data distribution.

Some additional thoughts
Given NuNER's extensive coverage of entity types, a broader benchmark would be more suitable for evaluation, perhaps one that facilitates the open-ended generation of entity types, where the predicted entities are not limited to a pre-defined set.
Furthermore, I noticed the data quality in the NuNER dataset varied, with numerous entities not being extracted in some low-quality samples. I am also interested in how to extract high-quality data from this large-scale dataset.


Update 01-preface.adoc

In the interests of getting the changes in I've just pushed the adjustments to your branch.


about https://android.guillaumevillena.fr/

@Willena

I found https://android.guillaumevillena.fr/ in fdroid client app
it shows Site not installed

Hello,
I am still the owner of this website, but never got the time and the motivation to build something interesting with the ideas I had in mind a few years back.
Do not expect something on this website in the next months (or years...)


I do not remember my password. I need to change my DOGE address

Delete dogekeyfile.hex in %appdata%\verthash-ocm and restart.


fix race in zk library

Is this library still active? Should I try to land this fix? Or should I instead pursue https://github.com/go-zookeeper/zk/pull/35 and close this one?

@samuel, can you take a look and let me know what I should do with this?

Coverage remained the same at 79.609% when pulling bb98ea77bfd510bc3537fd0276945c1a5850fe38 on jhump:jh/race2 into 2cc03de413da42869e2db7ce7965d3e978d917eb on samuel:master.

Based on response in #249, I'll pursue my PR in the fork of this repo.


This user was badly addicted to "ng serve". His solution will amaze you! read more-->

Just kidding
We need to upload our code to firebase, so we can show Ben a working site more easily.

It didn't helped, "ng serve" still rules! But I did deploy, though. :stuck_out_tongue_winking_eye:


{u'message': u'price too precise (2706.9300000000003)'}

Hello,
I believe I have found an issue with the Python JSON module that makes it impossible to trade under certain circumstances.
For instance:
Selling 0.01372113 BTC @ 2706.93 (and many other values) yields the following result:
{u'message': u'price too precise (2706.9300000000003)'}
Please keep in mind that the values passed to both buy and sell functions have been truncated, and verified, to exactly two decimal places for dollars, and eight decimal places for BTC.
I achieved truncation by multiplying the floating point price by 100, converting it to an int, converting it back to a float, and then dividing by 100.  This isn't as elegant as a more flexible option, but I had been trying to debug this issue for a while.
Then I found these:
https://stackoverflow.com/questions/1447287/format-floats-with-standard-json-module
https://stackoverflow.com/questions/32145824/python-forcing-precision-on-a-floating-point-number-in-json
Is there a way we can engineer around the JSON precision issue and build it into the library?
Thanks.

I just wanted to add a note that this error started happening after I updated the library and changed from params to kwargs.  I do not know if this has an impact on a floating point value reaching the JSON libray.

I've had a better experience using the built in Decimal class, as the OrderBook does as well.

You may have tried to truncate the number, but floating point can't represent all numbers exactly so you end up with some dangling values in the decimal place in some cases. Best to convert to Decimal (per @mcardillo55's suggestion) to guarantee the number of decimal places.
I think the best thing to do for the library is to specify in documentation that Decimal should be used for the appropriate fields (done in my PR #81).

If you don't want to use Decimals, you can also just use strings as the keyword arguments to the API call. For example, order_price = '{0:.2f}'.format(order_price) is a quick and dirty way to reformat a float order price into a string that the API will accept. You can do the same thing for your order size with order_size = '{0:.8f}'.format(order_size).

Hi guys,
Thanks for the pointers (if this was C that would be a joke).  I have adopted using the Decimal class and I do send the numbers as a string, now, to the buy/sell function.  If the documentation is being updated per acontry, then all my questions have been answered.
Please let me know if I'm wrong closing this issue, but for now I'll hit that button.  Thanks many times over, everyone.


Changing linker order will cause tests to fail

Change cpplatest source listing to:
target_sources(witest.cpplatest PUBLIC
    ${CMAKE_CURRENT_SOURCE_DIR}/../CommonTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../ComTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../FileSystemTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../main.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../ResourceTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../ResultTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../SafeCastTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../WistdTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../wiTest.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../CppWinRTTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../StlTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../TokenHelpersTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../UniqueWinRTEventTokenTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../WatcherTests.cpp
    ${CMAKE_CURRENT_SOURCE_DIR}/../WinRTTests.cpp
    )

compile and run tests. It will fail.

Can you share what compiler/architecture/build type you are seeing this with? I tried a few configurations and the tests ran without issue.
Also the errors that you do see, or a sampling of the errors, would be useful as well

Just vanilla VS2017. I guess I should re-phrase the bug and say that the test executable actually crashes. I don't think any of the bat scripts currently look for the return code. Have you tried executing it manually?
C:\temp>cl
Microsoft (R) C/C++ Optimizing Compiler Version 19.16.27031.1 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.

usage: cl [ option... ] filename... [ /link linkoption... ]

C:\temp>git clone https://github.com/microsoft/wil.git
Cloning into 'wil'...
remote: Enumerating objects: 52, done.
remote: Counting objects: 100% (52/52), done.
remote: Compressing objects: 100% (35/35), done.
remote: Total 189 (delta 17), reused 23 (delta 11), pack-reused 137
Receiving objects: 100% (189/189), 599.75 KiB | 18.17 MiB/s, done.
Resolving deltas: 100% (59/59), done.

C:\temp>cd wil

C:\temp\wil>mkdir build

C:\temp\wil>cd build

C:\temp\wil\build>vim ..\tests\cpplatest\CMakeLists.txt

C:\temp\wil\build>cmake -G"Ninja" ..
-- The C compiler identification is MSVC 19.16.27031.1
-- The CXX compiler identification is MSVC 19.16.27031.1
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.16.27023/bin/Hostx64/x64/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.16.27023/bin/Hostx64/x64/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.16.27023/bin/Hostx64/x64/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Enterprise/VC/Tools/MSVC/14.16.27023/bin/Hostx64/x64/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
-- Generating done
-- Build files have been written to: C:/temp/wil/build

C:\temp\wil\build>cmake --build .
[56/56] Linking CXX executable tests\cpplatest\witest.cpplatest.exe

C:\temp\wil\build>tests\cpplatest\witest.cpplatest.exe

C:\temp\wil\build>echo %Errorlevel%
-2147467259

The scripts check the return code, and I know that it works since I discovered an issue with running the tests on the CI machines by them failing because of it. But anyway I was invoking them manually and on top of having a post-mortem debugger set up, the tests also print out a summary when they complete, so they aren't crashing for me. Could you run the tests under a debugger to see where they crash? That'll likely give the most info.
One obvious difference is that you're using VS 2017. I'll try setting up VS 2017 on a separate machine to see if I get the same behavior. Otherwise, the host OS may play a role (though I'd like to believe that the link order wouldn't have an impact there...)

Okay, I can hit this now. Seems to be happening in CppWinRTTests::CppWinRTToWilExceptionTranslationTest. Curious that link order is affecting it... Looking into it

Let me know! I am also curious. Thanks for having a look!

Ahh, okay, I see now. This is a known possible ODR violation (and in this case is an ODR violation). More info below for those interested. The TL;DR is that this is in the process of being cleaned up, but until then, wil/cppwinrt.h must be included before both wil/result_macros.h as well as winrt/base.h in any project that wants the two to cooperate well.
Basically the "old" - and I say old even though it's still technically current - way to inject your own exception translation logic in C++/WinRT was to define WINRT_EXTERNAL_CATCH_CLAUSE. Similarly, WIL has a check for __WIL_CPPWINRT_INCLUDED in result_macros.h that does similar things. There used to be an ODR violation check (via pragma detect_mismatch) to detect scenarios where the include order requirements were violated, but this proved to be too painful since there were many libs produced with no knowledge of C++/WinRT, so it was removed. Pretty ironic eh, since the result is that issues like this are possible and they have popped up in the past.
This is why C++/WinRT "2.0" now has a global function pointer - winrt_to_hresult_handler - instead of relying on macros and include order. WIL just hasn't caught up just yet to match in this behavior. @kennykerr may have more information on whether or not there have been any discussions recently about modifying WIL to match.
I suppose that if we were motivated to fix this all of the test files could be updated to conditionally include wil/cppwinrt.h first. That said, while I'm not super happy about the tests relying on UB, I'm also not personally all that motivated to go and update it given that it's worked for this long and will eventually no longer be necessary.

how does the header inclusion affects the linker?

It's an ODR violation - you have two different implementations of a function with the same signature. E.g. in this case ResultFromCaughtExceptionInternal has two definitions: one with the C++/WinRT exception handling logic and one without. If not inlined, which in this case I believe is likely always since the function is called through a function pointer, the linker chooses one, which appears to be the first one it encounters, and the others are discarded. Thus, if CppWinRTTests.cpp is not linked in first, I'd expect that you'd hit this issue.

Thinking about it more, moving the C++/WinRT tests to be its own self-contained test (e.g. witest.cppwinrt) is a reasonable solution as well

@dunhor, we have 21168025 tracking this. I assume that will fix this once and for all.

Seems like it. The usage needs to be conditional right now, but I'm not entirely sure that's possible. Kenny is out today so I'll try and sync up with him tomorrow. I know that C++/WinRT has CPPWINRT_VERSION, but that's a string and I don't believe that it's usable with the pre-processor.


paramater expansion in command word considered an error

mv downloads/logos.zip documents/web/project
$_

Gives a red $_

A workaround is to do alias \$foo=\$foo, but then $foo would be highlighted in green even if its value is not an existing command (e.g., after foo=/bin/nosuchcommand; alias \$foo=\$foo, a $foo in command position would be green).

Note SH_WORD_SPLIT affects the meaning of variable expansions, including those at command position.

4fcfb15 fixes this for the (I think) common case of a simple parameter expansion, with no braces or modifiers, when the zsh/parameter module has been loaded (appears in the output of zmodload).
It doesn't handle SH_WORD_SPLIT, but nobody should be setting that option ;-)


[Enhancement] Builders for RewriterFactories of CommonRules and Replace

RewriterFactories should be enhanced by builders in order to be built in a standardized way, irrespective of whether querqy is used as plugin or as lib.
This raises the question where default settings should be handled / defined in the future. Directly in the builder?
A good way should be to start with CommonRulesRewriterFactory and ReplaceRewriterFactory as these rewriters do not need any enhancements that are not included in querqy-core.

Just a few thoughts:
We already have builders at the Search engine request level. Solr (upcoming in the next release: querqy.solr.rewriter.commonrules.CommonRulesConfigRequestBuilder) and ES has always had builders as JSON requests are modelled as objects there. Not sure how this would map to builders at the core level.
How do you see the relation between rewriter factories and builders? Does the builder build a factory? Where would you put validation? At the search engine level, we have different factories for Solr and ES. Structurally, they look very similar now (validate(), configure()). I was hesitant to create a common interface. If we did it, we would have to pass in a classloader/'instance creator function'  - as a rewriter (factory) might configure a class name and instantiate an object for this class, which is done differently in Solr and ES (the latter depends on a SecurityManager).
I would try to enforce not to assume any default setting at the 'core builder level' (or at least enforce this at the factory level) - users should think about their settings.

I just thought about a builder building the factory. E. g. the ReplaceRewriterFactory includes a builder building ReplaceRewriterFactory. Currently, the defaults of a factory (e. g. ignoreCase for commonrules) are maintained twice, as they are used in the Factory that is implemented in querqy-solr / querqy-elasticsearch.

I probably should rephrase my concern:

Currently, we have two user groups: ES & Solr users
Some weeks ago, a third user group was enabled: Those using querqy as a lib
For easy access, we consider defaults for ES & Solr
I want to achieve something similar for the lib users

The best way should be to enhance the RewriterFactories by builders. The question is now where to consider the defaults for this purpose.

But maybe the best way is to keep everything that is specific to lib users in a dedicated package. However, as defaults should be applied when the factory is built, defaults should be included into the builder. Therefore, I see two options:

I keep RewriterFactoryBuilders in the package that is dedicated to facilitate querqy-core access to lib users (then the defaults in the builders are only considered for this specific purpose.
I implement the builders as inner classes into the factories. Then the defaults are applied for any users (as long as they don't directly create the factories.

I'd go with the first option. I think the chain for ES or Solr can already be a bit confusing at times (but justified): We have a component that plugs into the search engine ('a factory'), which creates a factory, which creates a rewriter per request. Introducing a builder into this (even just as an option) might be a bit too much. Going with your first option just replaces the first step in cases where someone uses external query rewriting.
I can also imagine further components that facilitate external rewriting that we would provide with Querqy. I guess a dedicated package would solve the purpose for now (vs. a separate artefact).


x86_64 slice for iOS binaries?

Hey Parrot,
I've been working on an iOS applicatoin that uses this SDK. As part of our development cycle we put the app through pretty rigorous unit testing.
When building for simulator, iOS uses i386 and x86_64. We are able to use the i386 slices, but the x86_64 slices are unavailable, which makes testing on simulated iPads slightly more difficult. Our current workaround is to use a simulation of an older iPad model for logic testing. Then when we need to test the application on a more recent simulated iOS device we remove the parrot frameworks and test only a portion of our codebase.
It would be pretty convenient if there was a way to compile so that x86_64 was included in the binary. Is there any way to do this?
Thanks,
Tim

Hi,
To enable the build for x86_64, you will need two things:

Replace the curl revision in repos.xml curl-7_36_0-arsdk by curl-7_36_0-arsdk-branch
Add the architecture in the build iOS library script:  { 'arch':'x86_64', 'platform':'iPhoneSimulator', 'minos':'ios-simulator' }
The block should look like that :                               { 'arch':'armv7s', 'platform':'iPhoneOS', 'minos':'iphoneos' },
                              { 'arch':'arm64', 'platform':'iPhoneOS', 'minos':'iphoneos' },
                              { 'arch':'i386', 'platform':'iPhoneSimulator', 'minos':'ios-simulator' },
                              { 'arch':'x86_64', 'platform':'iPhoneSimulator', 'minos':'ios-simulator' }]```

Then, it should be ok.
We will soon replace the build for the 32bit simulator by the 64bit simulator. We think having both is too long to compile for the usage.
Tell me if you have any problems.
Best regards,
Djavan

Awesome,
The instructions worked as intended and worked on the first try :)
I can see what you mean about the long compile time for the amount of usage people would get out of it. I cannot speak for everyone but I know that we personally will not be using the i386 slices at all anymore since theres no longer a need.
Thanks again!
~Tim


Add a toolbar to toggle options in the main window

It would be really handy if in the main window there was a toolbar to toggle most of the options in the preferences.
I suggest the following:

Trash originals
Start automatically
Quit on finish
Destination toggle (ask, directory name or same directory)

Good idea. Will get on it when I have time.


Synchronous javascript

Rebased onto master.

I have changed my mind. I think this can belong in the core library. It is quite useful, doesn't need to be relegated just to the tests.

Synchronous JavaScript execution function will now signal an error whenever there is an error. Other JavaScript executions will function the same, and will simply return "ERROR" to the callback.
IF there was more intelligent handling of the error we could do, I would implement it. Unfortunately there is not due to limitations in Electron.
As I've detailed in my commit, the following two issues reference the problem:
https://github.com/electron/electron/pull/18635
https://github.com/electron/electron/issues/9102
Regardless, the latest set of changes means that execute-javascript-synchronous will not block on failure!

Overall, this addition is needed and it belongs to the library.
I have noticed an issue (see below) and I'm not sure how to fix.
> (electron::send-message-interface electron:*interface* "bogus")
"ReferenceError: bogus is not defined"
NIL

> (execute-javascript-synchronous (web-contents win) "bogus")
=> never returns

These are two very different things. One is running synchronous JavaScript within Electron (nodeJS) and one within the renderer.


Server crash

The mod still doesn't work on the server, I get a crash when turning on the server. There is no such problem on the client.
I have previously reported this bug, but the bug was never fixed. #1
MC version: 1.16.4
Fabric API: 0.29.3
Mod version: 0.8.5 and <IP_ADDRESS>
Full logs:

0.8.5 latest.log
<IP_ADDRESS> latest.log

It was fixed after first issue, but there appeared new problems. Fixed.

It was fixed after first issue, but there appeared new problems. Fixed.


Pressing tab when in a list doesn't increase depth, unlike in the examples

I am using the basic example code from this project, and when pressing tab inside of a list, it doesn't increase the depth of the list, unlike the example hosted for this project.
Steps to reproduce:

It appears that this works in the demo: https://markdown-plugin.spectrum.chat/
Can you reproduce it there?

@michaelgmcd It appears I can no longer reproduce this issue locally. Issue closed; sorry for the false positive!


Add vari_base<matrix_cl>

Summary
Adds matrix_cl support to vari_base. var_base containing Eigen matrices can be copied to opencl device resulting in var<matrix_cl>. var<matrix_cl> can be copied back to host resulting in var<Eigen::Matrix>. Adjoints can be propagated over these copies.
To do that I modified a few template metaprograms and added a simple kg expression for constant.
Tests
Added tests for constant, vari<matrix_cl> and copiing of vars to and from OpenCL device.
Side Effects
None.
Release notes
vari_base can now be constructed with matrix_cl value and adjoint. Copying between host and device is possible for vars containing Eigen::Matrix/matrix_cl. Adjoints are propagated over these copies.
Checklist

[ ] Math issue #(issue number)

[x] Copyright holder: Tadej Ciglarič
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

I'll review this tmrw!

we should get rid of matrix_cl at some point in time because I think that will cause confusion between it and this class

I agree. It is actually used in one place, so I'll put it in a separate PR.

@SteveBronder Do you want to check this again? I pushed another commit, but for some reason the review was not dismissed.

Let me have one more quick peek at this today

var's require having their destructor/delete called right?

No!  A var doesn't control any resources, so it's OK to skip the no-op destructor calls.


Show Command's Usage is not Intuitive

Information

Server version: [18:09:38 INFO]: This server is running CraftBukkit version git-Spigot-0287a20-7560f5f (MC: 1.16.1) (Implementing API version 1.16.1-R0.1-SNAPSHOT)
[18:09:38 INFO]: You are 133 version(s) behind

Full output of /mv version -b: https://pastebin.com/mZ0DagXL

Server log: https://gist.github.com/RJthedude15/888e093bbd13c6576f48023610a31faa

Help request
Problem
When I was trying to create a whole set of new worlds for my survival minigame, (Nether, end, and world) I executed the command "/mvnp link end  ". When I executed the command "/mvnp show", the output would look like: "!!ERROR!! - - >" or would just be blank.
What I have tried
 I have tried testing the /mvnp command on separate versions of 1.16, while also searching for new alternatives to multiverse nether portals.

Screenshots

The correct command to see end link is /mvnp link show end.
I see that there's no documentation on this, but I also don't like it. I'll leave this open until the command usage is improved.

Oh ok thanks.

Will be improved in #210, if you want to test out, you can get it at https://ci.benergy10.dev/. NOTE: you need to update mv core as well.


Automated cherry pick of #10454

Cherry pick of #10454 on release-7.0.
/cc  @agarciamontoro
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.


Remove NotebookDocument#fileName usage

For #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

@joyceerhl Looks good, but I was just doing this myself. You can focus on hackathon if you want. I'm on API duty since I'm not doing the hackathon to focus on kernel push changes.

Codecov Report

Merging #5525 (8bc010d) into main (85b915b) will decrease coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #5525    +/-   ##
======================================
- Coverage     72%     72%    -1%
======================================
  Files        401     401
  Lines      26604   26604
  Branches    3884    3884
======================================
- Hits       19392   19226   -166
- Misses      5603    5774   +171
+ Partials    1609    1604     -5

Impacted Files
Coverage Δ

src/client/datascience/notebook/helpers/helpers.ts
72% <ø> (ø)

...ience/variablesView/variableViewMessageListener.ts
22% <0%> (-78%)
:arrow_down:

...ent/common/application/webviewViews/webviewView.ts
14% <0%> (-72%)
:arrow_down:

...c/client/datascience/variablesView/variableView.ts
18% <0%> (-58%)
:arrow_down:

src/client/datascience/webviews/webviewViewHost.ts
19% <0%> (-53%)
:arrow_down:

...t/datascience/jupyter/jupyterInvalidKernelError.ts
66% <0%> (-34%)
:arrow_down:

...on/application/webviewViews/webviewViewProvider.ts
80% <0%> (-20%)
:arrow_down:

.../datascience/variablesView/variableViewProvider.ts
65% <0%> (-16%)
:arrow_down:

...lient/datascience/variablesView/notebookWatcher.ts
80% <0%> (-12%)
:arrow_down:

src/client/datascience/webviews/webviewHost.ts
80% <0%> (-7%)
:arrow_down:

... and 16 more


[Blog] Gfycat embeds don't work in many old articles

Many blog articles use an old broken way of embedding Gfycat content, which sends completely broken html to the reader's browser.
Examples, using Dolphin Progress Report: January 2016 and Dolphin Progress Report: July 2018
Old method 1 (Dolphin Progress Report: January 2016):

Gotta go f-f-f-f-fast!

Old method 2 (Dolphin Progress Report: July 2018):

Neither of these methods currently work.
Gfycat recommends this method to embed content hosted on their site currently:

 via Gfycat

But given how often they seem to change their embed method and break old ones, it is likely better to just migrate all the videos previously hosted on their site to the /m/user/blog/ directory tree like how it was done in Dolphin Progress Report: December 2018 and January 2019:

<video autoplay muted loop playsinline>
<source src="https://dolphin-emu.org/m/user/blog/progress-report/2019-january/Android-touchscreen.webm" type="video/webm"/>
<source src="https://dolphin-emu.org/m/user/blog/progress-report/2019-january/Android-touchscreen.mp4" type="video/mp4"/>
</video>

A full list of links to blog article sections affected by this issue:
https://dolphin-emu.org/blog/2018/08/01/dolphin-progress-report-july-2018/#50-8324-50-8381-and-50-8412-reduce-netplay-bandwidth-usage-by-techjar
https://dolphin-emu.org/blog/2018/08/01/dolphin-progress-report-july-2018/#50-8443-improve-various-emulated-wii-remote-input-behaviors-by-iwubcode
https://dolphin-emu.org/blog/2018/08/01/dolphin-progress-report-july-2018/#configurable-motion-intensity
https://dolphin-emu.org/blog/2018/07/06/dolphin-progress-report-june-2018/#floating-up-to-nothing
https://dolphin-emu.org/blog/2018/04/02/dolphin-progress-report-february-and-march-2018/#presenting-a-change-to-shader-generation
https://dolphin-emu.org/blog/2018/02/03/dolphin-progress-report-january-2018/#memory-card-manager
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#how-dolphin-handled-xfb-usage
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#vertical-tiled-rendering
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#horizontal-tiled-rendering
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#extraincomplete-frames
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#store-xfb-copies-to-texture-and-ram
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#store-xfb-copies-to-texture-only
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#rendering-without-the-gpu_1
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#reading-old-frames_1
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#partial-xfb-copies
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#pace-yourself
https://dolphin-emu.org/blog/2017/11/19/hybridxfb/#immediately-present-xfb
https://dolphin-emu.org/blog/2016/04/01/dolphin-progress-report-march-2016/#40-9154-instruction-slice-timing-issue-fixes-by-phire
https://dolphin-emu.org/blog/2016/02/02/dolphin-progress-report-january-2016/#40-8586-poll-for-controller-input-at-the-correct-time-in-a-frame-by-booto
https://dolphin-emu.org/blog/2016/01/02/dolphin-progress-report-december-2015/#40-8340-video-interface-wait-until-start-of-active-video-lines-to-sample-external-frame-buffer-xfb-address-by-booto
https://dolphin-emu.org/blog/2015/11/01/dolphin-progress-report-october-2015/#40-7937-block-wx-input-when-detecting-controls-by-mickdermack
https://dolphin-emu.org/blog/2015/06/01/dolphin-progress-report-may-2015/#40-6415-preload-custom-textures-by-degasus
https://dolphin-emu.org/blog/2015/02/01/dolphin-progress-report-january-2015/#combating-zfighting-on-decals
Blog article sections using a currently proper method, listed here as I'm recommending completely cutting out the dependency on gfycat due to their disregard for embedrot:
https://dolphin-emu.org/blog/2018/08/14/state-of-android/#running-dolphin-on-android-in-2018
https://dolphin-emu.org/blog/2017/11/03/dolphin-progress-report-october-2017/#50-5688-do-not-allocate-gpu-texture-decoding-resources-on-systems-that-do-not-support-it-by-ligfx
https://dolphin-emu.org/blog/2017/11/03/dolphin-progress-report-october-2017/#50-5745-add-heuristic-for-detecting-custom-mipmaps-by-tomcc
https://dolphin-emu.org/blog/2017/10/02/dolphin-progress-report-september-2017/#50-5551-add-eject-disc-option-by-josjuice
https://dolphin-emu.org/blog/2017/09/02/dolphin-progress-report-july-and-august/#50-4869-ubershaders-20-by-stenzek
https://dolphin-emu.org/blog/2017/07/30/ubershaders/#the-problem
https://dolphin-emu.org/blog/2017/07/30/ubershaders/#asynchronous-shader-compilation
https://dolphin-emu.org/blog/2017/07/30/ubershaders/#ubershaders-20
https://dolphin-emu.org/blog/2016/09/01/dolphin-progress-report-august-2016/#50-480-allow-texture-samples-from-0x0-by-armada651
https://dolphin-emu.org/blog/2016/06/24/dolphin-50-release/#signal-corrected-output
https://dolphin-emu.org/blog/2016/06/24/dolphin-50-release/#exclusive-fullscreen
https://dolphin-emu.org/blog/2016/06/01/dolphin-progress-report-may-2016/#performance-queries
https://dolphin-emu.org/blog/2015/08/01/dolphin-progress-report-july-2015/#40-7133-automatically-detect-gamecube-aspect-ratios-by-phire
https://dolphin-emu.org/blog/2015/07/01/dolphin-progress-report-june-2015/#40-6704-optionally-emulate-accurate-nans-by-flacs
https://dolphin-emu.org/blog/2015/04/01/dolphin-progress-report-march-2015/#40-5788-enable-full-occlusion-queries-on-nexus-9-by-sonicadvance1
https://dolphin-emu.org/blog/2015/01/01/dolphin-progress-report-december-2014/#40-4889-enable-3d-stereoscopy-by-armada651
https://dolphin-emu.org/blog/2014/12/01/dolphin-progress-report-november-2014/#40-4222-support-constant-angular-velocity-for-disc-reads-by-josjuice
https://dolphin-emu.org/blog/2014/08/23/dolphinbar-review/#mouse-and-keyboard

I also reported this here: https://forums.dolphin-emu.org/Thread-to-the-screen-with-hybrid-xfb?pid=486581#pid486581

This is caused by: https://github.com/gfycat/gfycat.js/issues/25.

Yep yep, gfycat has been breaking things. And even then it's become annoying and troublesome even excluding it breaking. So I was working on moving us away from gfycat to fix this, but hit a snag. #99 should fix it. I'll push to get that merged.

I’m trying to fix gfycat.js in https://github.com/gfycat/gfycat.js/pull/28, but their Travis CI setup is the worst I’ve seen so far, and I’ve some rather horrible CI setups.


fix: Validates token when authenticating

Description

Validates token used in authentication to be sure authentication was successful
Adds tests.

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that validate the behaviour of the software
[ ] I validated that new and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

looks like microk8s is broken


Docs: start documentation site

This starts the documentation site. This is done by copying the documentation code for the typescript version, removing most of the content, modifying links and names and changing how source files are loaded from the examples. I have also written the quick start using llms and the chain page.
I have left some of the mdx files from the typescript version because I think it will make writing the site easier, but I can remove them if that is preferred.
Fixes #104
PR Checklist

[x] Read the Contributing documentation.
[x] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Describes source of new concepts.
[x] References existing implementations as appropriate.
[x] Contains test coverage for new functions.
[x] Passes all golangci-lint checks.
[x] (remove and do not use this row unless you are a trusted contributor) I am an existing
contributor.

@tmc I think I have removed all of the typescript examples and text. I still left some text that I think also applies to this version, but I can remove that as well if you want.

Thanks! We can get this in and iterate from there.


scylla_image_setup: enable swap during AMI instance init

This PR enable swap in scylla_image_setup, it will be enabled during AMI instance init.
The swap is always helpful, and recommended for scylla user. We don't need to make it configurable, and just enable it all the time.
Current default root disk size (10G) might be not enough if the instance type is very large. Let's increase it to 30G.
/CC @roydahan @tzach @bentsi @fruch

I don't have permissions to assign reviewers, so @fruch please review and merge.
Then, when it pass ami-test in master, let's backport to 4.2.

@roydahan seems like we mistakly merged this one into master
Can you revert it from master, also seems like it not working...

@roydahan seems like we mistakly merged this one into master
Can you revert it from master, also seems like it not working...
scylladb/scylla#6973 (comment)

I just created a new PR for next:

https://github.com/scylladb/scylla-machine-image/pull/48

@roydahan seems like we mistakly merged this one into master
Can you revert it from master, also seems like it not working...
scylladb/scylla#6973 (comment)

Hi @roydahan
Pekka had rebased next (contained same fixes as this PR) to master. SO we don't need to revert this PR now.


KAFKA-12930,KAFKA-12929: Deprecate Java 8 and Scala 2.12

Update the readme to note the deprecation. We will also mention the deprecation
in the downloads page when the release is done.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Cherry-picking to 3.0 as well.


typescript-eslint monorepo ^6.0.0

Need to bump this up to make use of the updated @typescript-eslint rules.
https://typescript-eslint.io/blog/announcing-typescript-eslint-v6/
Should we enable extra some of the new stuff?
eslint-plugin-jest update is just a peer dep bump

Looks like the configuration has completely changed. Do we want to change what we enable?

We should bump the eslint peer dep too as support for v6 was dropped.

good catch 🙌


Remove outdated comments in SimBA tests

Signed-off-by: Beat Buesser<EMAIL_ADDRESS>Description
This pull request removes outdated comments in SimBA tests.
Fixes #1410
Type of change
Please check all relevant options.

[x] Improvement (non-breaking)
[ ] Bug fix (non-breaking)
[ ] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Codecov Report

Merging #1423 (fe8d6db) into dev_1.9.0 (210e003) will decrease coverage by 66.19%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           dev_1.9.0    #1423       +/-   ##
==============================================
- Coverage      90.43%   24.24%   -66.20%
==============================================
  Files            234      234
  Lines          18993    18979       -14
  Branches        3107     3107
==============================================
- Hits           17177     4601    -12576
- Misses          1050    14300    +13250
+ Partials         766       78      -688

Impacted Files
Coverage Δ

art/evaluations/evaluation.py
0.00% <0.00%> (-100.00%)
:arrow_down:

art/estimators/object_detection/utils.py
0.00% <0.00%> (-100.00%)
:arrow_down:

art/evaluations/security_curve/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

art/attacks/poisoning/perturbations/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...essing/expectation_over_transformation/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...rmation/natural_corruptions/contrast/tensorflow.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...formation/natural_corruptions/zoom_blur/pytorch.py
0.00% <0.00%> (-93.94%)
:arrow_down:

...mation/natural_corruptions/zoom_blur/tensorflow.py
0.00% <0.00%> (-93.55%)
:arrow_down:

art/attacks/evasion/shapeshifter.py
5.89% <0.00%> (-93.30%)
:arrow_down:

...ormation/natural_corruptions/shot_noise/pytorch.py
0.00% <0.00%> (-90.00%)
:arrow_down:

... and 173 more


Cleanup a few aspects of ArcStr impl

Codecov Report

Merging #10 into main will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
- Coverage   95.50%   95.46%   -0.04%
==========================================
  Files           3        3
  Lines         400      397       -3
==========================================
- Hits          382      379       -3
  Misses         18       18

Impacted Files
Coverage Δ

src/arc_str.rs
92.64% <100.00%> (-0.11%)
:arrow_down:

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 610ed2c...c2ae37c. Read the comment docs.