DrawText: is right align possible?

I am sorry for asking this question but I have trolled all the Documentation, API, Google etc but can't seem to figure out if this is possible:
Right align text in a DrawText()
I am generating an invoice PDF from order lines and some of the text entries need to be in a right aligned column...
Is this possible at all? I tried setting the width of a text field but could not find parameters for text alignment.
Thanks

Hello @spock123!
This is certainly possible to do. You'll just need to measure how wide the text you wish to draw is and position it based on that. Here's an example:
import { PDFDocument, StandardFonts, rgb } from 'pdf-lib';

(async () => {
  const pdfDoc = await PDFDocument.create();

  const font = await pdfDoc.embedFont(StandardFonts.Helvetica);

  const text = 'Foo bar';
  const fontSize = 35;
  const textWidth = font.widthOfTextAtSize(text, fontSize);
  const boxWidth = 250;
  const boxHeight = 100;
  const boxX = 50;
  const boxY = 50;

  const page = pdfDoc.addPage([350, 350]);
  page.drawRectangle({
    x: boxX,
    y: boxY,
    width: boxWidth,
    height: boxHeight,
    borderColor: rgb(1, 0, 0),
  });
  page.drawText(text, {
    x: boxX + boxWidth - textWidth,
    y: boxY,
    font,
    size: fontSize,
  });

  const pdfBytes = await pdfDoc.save();
})();

And here's what the resulting page looks like:

I hope this helps. Please let me know if you have any additional questions!

@Hopding wow thank you for a very thorough and detailed answer. I'll play around with it. Thank you again!

HI. has anyone figure out how to work with text-alignment on multiline? or at least a snippet that we can improve on
currently looking around at API / samples / issues haven't found one..


Add scala_version function

this is related to #14
We give a function scala_version and a scala_mvn_artifact that can be used in name mangling for scala jars. This can help deal with making WORKSPACES less tied to a particular version of scala.

@dinowernli what do you think of this?

LGTM


s3select: CASE statement used to match conditions

Signed-off-by: Albin Antony<EMAIL_ADDRESS>

the implementation of case-when-then-else , is not correct.
the /case expression/ is value which should used by when-then
the reuse of existing code(case-when-then) is not correct.

the implementation of case-when-then-else , is not correct.
the /case expression/ is value which should used by when-then
the reuse of existing code(case-when-then) is not correct.

#51
please check this PR and merge it

#51
please check this PR and merge it

ok,
as discussed need to migrate all test from s3test.

ok,
as discussed need to migrate all test from s3test.


Invalid Kubernetes API Timed out connecting to server

Hi,
I've tried using both the RBAC and non RBAC config for the graylog configuration of fluentd but I keep getting this error:
2018-07-06 15:05:28 +0000 [info]: parsing config file is succeeded path="/fluentd/etc/fluent.conf" 2018-07-06 15:06:29 +0000 [error]: config error file="/fluentd/etc/fluent.conf" error_class=Fluent::ConfigError error="Invalid Kubernetes API v1 endpoint https://<IP_ADDRESS>:443/api: Timed out connecting to server"
Do you know why fluentd needs to contact the API server and how to enable this? I've tried contacting the API server outside of the POD (on the master) and it works fine but a POD isn't able to contact it.
I'm using kubectl / kubeadm v1.9.1+2.1.5.el7
Thanks,
Adam

Maybe, kubernetes_metadata_filter do it. Try removing it from configuration.

Same here.

Even I am having same problem , Can you please clarify what needs to be done ?

I am having same problem too

Also having the same issue with this. I cannot ping the IP that it is trying to connect to, but don't know where it gets that IP from- anybody got any ideas?

Encountered the same problem, remove kubernetes_metadata_filter from configuration works for me.
I'm using kubernetes v1.11.0

Enabling default port in firewall with sudo ufw allow 5170 on master node worked for me.

I am seeing the same issue in a Windows 1809 SAC container running fluentd.


Verify Google Analytics tracking code

Done when confirmed whether or not google analytics is working on the new site.

Looks like it's working correctly, you can see the URLs are using the new OV ID:


Opaque values in JSON not properly decoded

Hi,
I am trying to bootstrap a wakaama client with DTLS. I can run the example client with DTLS-PSK support, and successfully write server and security objects in JSON.
The problem comes when the client registers to a LwM2M server using DTLS-PSK, it fails. Example of a security object written to client on bootstrap:
{"e":[{"n":"0","sv":"coaps://localhost:3002"},{"n":"1","bv":false},{"n":"10","v":3},{"n":"2","v":0},{"n":"3","sv":"dGVzdGx3bTJtY2xpZW50"},{"n":"5","sv":"OGY3NmE1ZDA4YzM0NTlmYzRmOTIwMDNlNjBjZmI3ZTk="}]}

Let me know if you need more details, thanks.
UPDATE: I can confirm now that this is a bug.
The example client with DTLS enabled works fine on bootstrap (it just unhexifies the PSK from command line). On the other hand the PSK from a security object payloaded into JSON must be base64-decoded before storing it.

Hi,
When received as JSON, the lwm2m_data_t has a type String. The security object should indeed base-64 decode it.

R


SFTP host authenticity

When using SFTP like this:
<EMAIL_ADDRESS>I get:
The authenticity of host '<IP_ADDRESS>' can't be established.
SSH-ED25519 key fingerprint is 152:148:110:150.....113:153:154:199.
Are you sure you want to continue connecting (yes/no)?

I tried to mount a file /docker/volumerize/ssh_config:/etc/ssh/ssh_config with the following content:
Host *
   StrictHostKeyChecking no
   UserKnownHostsFile=/dev/null

but with no luck. I still get the question and the unattended backup does not run.

Solution:
Create a pre start script and mount it: /docker/volumerize/add_known_host.sh:/preexecute/backup/add_known_host.sh:ro:
#!/bin/bash

mkdir -p ~/.ssh/
ssh-keyscan -H <IP_ADDRESS> >> ~/.ssh/known_hosts


Add swift-command extension

Description

Efficiently manage and access Linux command snippets for faster execution:

Easily add and locate commands
Supports custom placeholders within commands
Chinese Pinyin fuzzy search functionality
Sync commands across devices by hosting the data file with WebDAV tools

Screencast

Checklist

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

Thanks for your contribution 🔥
@koinzhang have you seen a similar extension in the store, I think I have but I can't find it (I saw one from you about history which is not the same)

Hello @pernielsentikaer, did you mean this Shell History extension? I’ve tried it, and it’s very useful. However, they serve different purposes. The Shell History extension is designed for quickly searching local command history from zsh or bash, while Swift Command focuses on managing personal command snippets.
Swift Command is ideal for those who need to type long commands or work across multiple machines or environments. Since different environments can’t share their command history, an extension like this is the best choice.
Before I built this extension, I tried many others related to commands, Linux, or snippets available in the store. I never found one that allowed me to create and use commands quickly; they were designed for code snippets rather than commands, which don’t require a lot of input. Less is more. That’s why I decided to create this extension. It has significantly improved my work efficiency, and I hope it can help others who need to operate multiple machines.

Makes sense, a few things to look into:

It looks like you have Empty State Flicker - could you add isLoading={} to your <List />

Add style={Action.Style.Destructive} to delete command

One thing that could be nice would be to use useForm for the form, this gives you validation, states and error handling 🙂

Thanks! I’ve fixed the two issues you mentioned. Could you please review it?

Yes, I initially used useForm because it’s simpler. However, I needed to add or remove TextField parameters dynamically when the command input changes. useForm caused some issues with that, so I switched to using Without Utils Validation.

Makes sense 🙂


Additional filtering options

The current harness offers filtering for type name. It'd be nice if it was a more general string search across type, method, etc. Bonus points for some ability to pass regexes.

Closing prior to archive.

Closing prior to archive.


Compatibility

I've converted the files which didn't use unix line endings using dos2unix. (I'm guessing @zdavis is using windows? If so, please have a look at this: https://help.github.com/articles/dealing-with-line-endings/#platform-windows)
Furthermore I've updated the default flags from npm to yarn. As far as I could see yarn doesn't have a silent option.
And I've removed the prune task, as yarn does not have such a task.
Click here to see the changes without the line ending changes

Hi @koenpunt,
Many thanks for this pull request!


bootloader: Fix image signature validation when OpenOCD is attached (IDFGH-2130)

The code as-is invoked verify_checksum() only if OpenOCD was not attached,
but if SECURE_BOOT_CHECK_SIGNATURE is defined, the invocation of
verify_secure_boot_signature(...) depends on verify_checksum() being run.
Instead, always invoke verify_checksum(), but discard the result if
OpenOCD is attached.  This still allows the software breakpoints per
the original use case of having this check (see commit 439140d26),
but allows the signature validation to still work if there are no
software breakpoints set.  (The signature validation check is always
run despite OpenOCD being attached.)
Signed-off-by: Tim Nordell<EMAIL_ADDRESS>

Note: I ran into this while having a debugger attached while working with FOTA - it would never pass image validation.  In this case, one is trying to run the image validation code without setting breakpoints in what is being validated at all (but might in the actually running image).

@tim-nordell-nimbelink Thanks for reporting.

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

 All committers have signed the CLA.

@tim-nordell-nimbelink Sorry about not providing any update here. This was fixed in https://github.com/espressif/esp-idf/commit/07bf1e23db718f4c5cfd800b5dc1e4ecaf88ecb5. Closing the PR.


Automatic selection of 'best' Socket RV

Is your feature request related to a problem? Please describe.
Currently we ask the person running sshnp which is the RV closest to them. There are two problems with this approach..

People move!
Machines crash

Describe the solution you'd like
A much better solution would be to have a single atSign specified like @ rv_earth and having multiple the RV points listening and responding.
However the sshnp only listens to the first response, meaning the closest/fastest for the location of the sshnp would be selected.
This would also allow redundancy of RV's.
This combined with and RV actually responding with a group of RV processes in region/local provides a scalable architecture.
So what is required...

[ ] sshrvd to be able to request another group of sshrvd's for IP/Ports and respond back (provides scale)
[ ] sshnp to collect the 'fastest' response and ignore duplicates from other ssrvd's
[ ] Stretch goal for sshnp if it hits issues with the fist response try the second as a backup
[ ] Stretch stretch goal, to involve sshnpd to see what is closest for it and to find the best middle location - e.g sshnp gets USA and sshnpd gets AP then pick EU

Describe alternatives you've considered
No alternatives come to mind but happy to discuss
Additional context
No response

Given that all notifications go client-atServer-arServer-client for both the request and the response leg, fastest to respond doesn't mean closest - in some cases it could mean farthest away
However, given a range of IP addresses from many RVDs, and the ability to for sshnpd and sshnp to determine latency to each RVD, then sshnp and sshnpd could figure out the "best" one

https://pub.dev/packages/dart_ping looks like a good candidate to use for determining latency. Actively maintained, well documented, good code quality. Has support on all platforms (including iOS via the https://pub.dev/packages/dart_ping_ios plugin). On Mac / Linux / Windows platforms it runs the OS's ping command

This will be a very nice feature but is P2/P3 since we currently have the ability for the human to choose an appropriate rvd

Making this a P2, I think it is useful to have a solution for this.


rename Drag&Drop plants

[ ] "Drag & Drop plants" -> rename to "select plant"

fixed by @ThorbenStaufer


[Feature Request]: Improve performance of generating successful writes in Java's BigqueryIO connector

What would you like to happen?
The current implementation recreates a TableRow from the protobuf message when withPropagateSuccessfulStorageApiWrites(true) is used. For tables with a large number of rows this becomes a CPU intensive operation. Based on the assumption that the consumers of the resulting PCollection only need to have a single, or just a few, columns from the original row, the following change should significantly improve the CPU and memory cost of pipelines which need to have this optional output.

Create "withPropagateSuccessfulStorageApiWrites(@NonNullable Set columnsToPropagate)" in addition to keeping "withPropagateSuccessfulStorageApiWrites(boolean propagateSuccessfulStorageApiWrites). This will be a non-breaking API change.

Pass the set of the required columns all the way to the DoFn responsible for the generation of the successful writes. There should be no additional memory increases in any of the methods - this is a static DoFn configuration.

Use the set of required columns rather than the meta data from the proto message to create the output TableRow. There are two possible places where the optimization can occur (proto -> DynamicMessage and DynamicMessage -> TableRow). If the former can be changed it will produce the best outcome, but even if we can change the latter it will still reduce a lot of CPU cycles.

This approach will work for all the permutations of inputs and methods - writeProtos()/writeTableRows and AT_LEAST_ONCE/EXACTLY_ONCE.

Issue Priority
Priority: 3 (nice-to-have improvement)
Issue Components

[ ] Component: Python SDK
[x] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[x] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

Fixed by #32312


Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: CONCATENATION, CONV_2D, MAXIMUM, MAX_POOL_2D, MUL, PAD. Here is a list of operators for which you will need custom implementations: ExtractImagePatches.

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (or github SHA if from source):

Provide the text output from tflite_convert
# Copy and paste here

Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Also, please include a link to a GraphDef or the model if possible.
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem.
If including tracebacks, please include the full traceback. Large logs and files
should be attached.

The select tf option can enable tf.ExtractImagePatches op. See https://www.tensorflow.org/lite/guide/ops_select


handle when focus_surface is NULL

Context
Bug fix.
When starting X client, the compositor crushes.
Summary
Fixed the bug.
How to check behavior
Same as https://github.com/zigen-project/zen/pull/107.

I'll merge this without review as it's a small bug-fix change.


loading config returns 404 when site is non-root of domain

When loading Sveltia CMS v.0.3.5 from the following path:
https://example.org/my-actual-site/admin/

It returns 404 for the config.yml as it tries to load from:
https://example.org/admin/

Looks like the current configuration path is written in an absolute path /admin/config.yml. I’ll change it to relative.
In the meantime, you should be able to work around the issue by specifying the config path in the HTML file 🙂

Also fixed!!

Thanks for the verification! 🙌


Add an "install" and a "pull" command

Add a "name" config option, which sets the "name" property in project.mml and functions as the directory name for the installed project in the mapbox projects directory. Defaults to "OSM bright"

Add a "path" config option, which specifies the directory containing the user's mapbox projects. This enables us both to install the project for them, and pull back changes to be committed into the repository.

Unfortunately, adds a lib directory with one file, utils.py containing one function, copy_tree. I didn't want to jam this function into make.py, but if you prefer that organization I will understand as it sucks to add a dir just for this one function. I tried hard to use the stdlib's functions, but none of them worked exactly right.

Moves configure.py to a configure.py.sample file; this will avoid anyone accidentally checking in their credentials.

Adds an "install" command to make.py, which copies build to config[path]/config[name]. It will only overwrite any files in the install directory which match files in the build dir. (i.e. if you add a file to the install dir, it won't get blown away by "install"ing.

"Install" will mangle the name of config[name] so that it is a valid directory. It will turn a name like "My OSM Map" into "MyOSMMap".

Adds a "pull" command to make.py which pulls changes back from the install directory to the osm-bright repo so they can be easily committed.

This is awesome. I made a change to copy_tree in utils.py to add a default empty value to the ignores variable because I was getting an error from the call on line 27 in make.py. Make sense?

Yes indeed, my bad for not hitting that before I submitted.
Awesome!


Patch 1.4.1

Description
Patch for 1.4
Includes #2259
Type of change

[ ] New feature (backwards compatible)
[ ] New feature (breaking API changes)
[x] Bug fix
[ ] Other (Documentation, CI, ...)

Contribution checklist

[x] Add our license to new files.
[x] Check that your code adheres to our coding style.
[ ] Write unittests.
[x] Run the unittests and check that they’re passing.
[x] Run the benchmarks to make sure your change doesn’t introduce a regression.

We still manage to hit the timelimit that was increased in #2249 when running in emulation (not consistently, but it happens on slow agents). I think if we increase the timeout by another factor of 10 (or at least ~3), it should pass consistently.

We still manage to hit the timelimit that was increased in #2249 when running in emulation (not consistently, but it happens on slow agents). I think if we increase the timeout by another factor of 10 (or at least ~3), it should pass consistently.

Increased by a factor of 100, since we're only testing that using the solver option doesn't throw an error.

Redoing with cherry pick for clean commit history.


Make hoovering and deleting parent dir work everywhere for cookbook_artifacts

Right now, there are multiple calls to hoover_delete_checksums but we only override one of them.
This patch also fixes the spot where we delete /cookbook_artifacts/name when the last /cookbook_artifacts/name/identifier is removed.

:+1:


Migrate to Spring Security Authorization Server

Delete the authorization endpoints that are handled by a new and separate module the org.openwms.core.auth

classes deleted, implementation started in new repository


Notification icon not clearing once tab has focus

Use to work fine, so suspect it's a chromium update that has changed something.  But the notification doesn't go away when the tab gets focus.  Only able to clear it with a refresh of the tab.
Using brave browser which is chromium based
Brave is up to date
Version 1.25.68 Chromium: 91.0.4472.77 (Official Build) (64-bit)

Thanks! I'll look into this today or tomorrow.

I am also seeing this as of today

This appears to be Brave-specific, as I can reproduce this in Brave, but not in Chrome or plain Chromium 91.
I'll need to investigate what's different with how Brave works with extensions.

I'm on mainline Chrome (Version 91.0.4472.77 (Official Build) (64-bit)) and also experiencing it. Is there anything I can do to help diagnose?

Oh, interesting. I am on Windows 10; are any of you using macOS or Linux?

Nope, I'm Windows 10 too

For me, it's not happening all the time, about 10% of the time it works. If it helps, in my case, I have a pinned tab for my gmail, and it's on that one that it's not working most of the time. I have the ext enabled on pinned tabs only and set not to flash.

I was finally able to reproduce it tonight on mainline Chrome. After digging into the logs, I found an error that lead me to this SO thread: https://stackoverflow.com/questions/67806779/im-getting-an-error-tabs-cannot-be-edited-right-now-user-may-be-dragging-a-ta.
It appears to be a browser bug, but a workaround is provided in that thread that I will attempt to apply.

I have submitted an update to the Chrome Web Store.

OK - this latest version is live on the Chrome Web Store.


OCPQE-18902 don't fail cmd update-bug-list when got unauthorized error

/test images

@jianzhangbjz the job is pass in new PR


Chymyst5

Edits to two examples, first one is my perceived issue with correctness of Waiting for Godot (who blocks, who gets the reply, etc), the second and last example is about clarity that we need to inject the molecule to resolve the future.
Additionally there are 1-2 minor typos.

giving up on this PR.


Can't run eval_dataset: creating robot failed

🐛 Bug

The ovmm evaluation script fails to run after a fresh installation following the instructions.
Steps to Reproduce
Steps to reproduce the behavior:

Simply run
python projects/habitat_ovmm/eval_dataset.py

The error message is:

pybullet build time: May 20 2022 19:45:31
2023-06-21 11:26:43,692 Initializing dataset OVMMDataset-v0
2023-06-21 11:26:44,468 initializing sim OVMMSim-v0
[11:26:45:367752]:[Core] ManagedContainerBase.h(329)::checkExistsWithMessage : ::getObjectByHandle : Unknown Lighting Layout managed object handle :  . Aborting
2023-06-21 11:26:46,012 Initializing task OVMMNavToObjTask-v0
/home/users/jiaming/ovmm/ovmm_env/lib/python3.9/site-packages/gym/spaces/box.py:84: UserWarning: WARN: Box bound precision lowered by casting to float32
logger.warn(f"Box bound precision lowered by casting to {self.dtype}")
Running eval on [1200] episodes
[11:26:46:852964]:[Core] ManagedContainerBase.h(205)::getObjectHandleByID : Unknown ArticulatedObject managed object ID: -1 . Aborting
[11:26:46:852999]:[Core] ManagedContainerBase.h(329)::checkExistsWithMessage : ::getObjectCopyByID : Unknown ArticulatedObject managed object handle :  . Aborting
Process ForkServerProcess-1:
Traceback (most recent call last):
File "/home/users/jiaming/ovmm/ovmm_env/lib/python3.9/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/home/users/jiaming/ovmm/ovmm_env/lib/python3.9/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "/home/users/jiaming/ovmm/ovmm_env/lib/python3.9/contextlib.py", line 79, in inner
return func(*args, **kwds)
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/core/vector_env.py", line 281, in _worker_env
result = result_or_fn(**function_args)
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/core/gym_env_episode_count_wrapper.py", line 61, in reset
return self.env.reset(**kwargs)
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/core/gym_env_obs_dict_wrapper.py", line 36, in reset
return self.env.reset(**kwargs)
File "/home/users/jiaming/ovmm/home-robot/src/home_robot_sim/home_robot_sim/env/habitat_ovmm_env/habitat_ovmm_env.py", line 107, in reset
habitat_obs = self.habitat_env.reset()
File "/home/users/jiaming/ovmm/ovmm_env/lib/python3.9/site-packages/gym/core.py", line 283, in reset
return self.env.reset(**kwargs)
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/utils/gym_adapter.py", line 300, in reset
obs = self._env.reset()
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/core/environments.py", line 54, in reset
observations = super().reset()
File "/home/users/jiaming/ovmm/ovmm_env/lib/python3.9/contextlib.py", line 79, in inner
return func(*args, **kwds)
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/core/env.py", line 422, in reset
return self._env.reset()
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/core/env.py", line 260, in reset
self.reconfigure(self._config)
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/core/env.py", line 342, in reconfigure
self._sim.reconfigure(self._config.simulator, self.current_episode)
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/tasks/rearrange/rearrange_sim.py", line 260, in reconfigure
self.robots_mgr.reconfigure(new_scene)
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/tasks/rearrange/robot_manager.py", line 102, in reconfigure
robot_data.robot.reconfigure()
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/robots/mobile_manipulator.py", line 143, in reconfigure
Manipulator.reconfigure(self)
File "/home/users/jiaming/ovmm/home-robot/src/third_party/habitat-lab/habitat-lab/habitat/robots/manipulator.py", line 82, in reconfigure
self.sim_obj.auto_clamp_joint_limits = True
AttributeError: 'NoneType' object has no attribute 'auto_clamp_joint_limits'
....

Expected behavior

The evaluation should run successfully.
Additional context

I locate the problem being the following line in "home-robot/src/third_party/habitat-lab/habitat-lab/habitat/robots/manipulator.py"
def reconfigure(self) -> None:
        """Instantiates the robot the scene. Loads the URDF, sets initial state of parameters, joints, motors, etc..."""
        # TODO: The current implementation requires users to define all the components of the robot in a single URDF.
        # The future will support loading multiple URDF files.
        if self.sim_obj is None or not self.sim_obj.is_alive:
            ao_mgr = self._sim.get_articulated_object_manager()
            self.sim_obj = ao_mgr.add_articulated_object_from_urdf(  # <<< THIS FAILS AND RETURNS NONE
                self.urdf_path, fixed_base=self._fixed_base
            )

I confirm the urdf file exists and urdf_path correctly points to the file. I also created a simple URDF and called ao_mgr.add_articulated_object_from_urdf. The fails again and the error message is the same as follows:

[12:15:29:130992]:[Core] ManagedContainerBase.h(205)::getObjectHandleByID : Unknown ArticulatedObject managed object ID: -1 . Aborting
[12:15:29:131056]:[Core] ManagedContainerBase.h(329)::checkExistsWithMessage : ::getObjectCopyByID : Unknown ArticulatedObject managed object handle :  . Aborting

Solved by re-build habitat-sim with the bullet option.

Thanks for figuring this out! We are updating the instructions here to install habtiat-sim withbullet always.


Upgrade org.pitest:pitest-maven 1.11.5 -> 1.11.6

This PR contains the following updates:

Package
Type
Update
Change

org.pitest:pitest-maven (source)
build
patch
1.11.5 -> 1.11.6

Release Notes

hcoles/pitest
v1.11.6
Compare Source
#1171 Prevent double instantiation of features
#1170 Update links to https (thanks @romani)

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade pitest-maven-plugin 1.11.5 -> 1.11.6

See:
- https://github.com/hcoles/pitest/releases/tag/1.11.6
- https://github.com/hcoles/pitest/compare/1.11.5...1.11.6


Failed to allow Update.exe to set foreground window

OS
Windows 11
Programming Language
dotnet 9.0
VPK Version
vpk 0.0.869
Library Version
Velopack 0.0.869
What happened?
I'm seeing "Failed to allow Update.exe to set foreground window" when updating a .net console application). Any ideas? Thanks!
Relevant log output
warn: Platform.UpdateClient.ApplicationUpdater[0]
      Failed to allow Update.exe to set foreground window.
      System.ComponentModel.Win32Exception (5): Access is denied.
         at Velopack.UpdateExe.StartUpdateExe(ILogger logger, IVelopackLocator locator, IEnumerable`1 args)

This is just a warning that can be ignored. Presumably console applications are not allowed to set the foreground window.

OK, such was my hope - thank you for confirming :-)


[SPARK-3444] [core] Restore INFO level after log4j test.

Otherwise other tests don't log anything useful...

I'm reusing the bug number since it feels overkill to file a new bug for this, but let me know if you think otherwise.

Merged build triggered.

Merged build started.

Test build #36220 has started for   PR 7140 at commit 6cff13a.

Test build #36220 has finished for   PR 7140 at commit 6cff13a.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait ExtractValue
abstract class ExtractValueWithStruct extends UnaryExpression with ExtractValue
abstract class ExtractValueWithOrdinal extends BinaryExpression with ExtractValue

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #36226 has started for   PR 7140 at commit de14836.

Test build #36226 has finished for   PR 7140 at commit de14836.

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

Merged build finished. Test PASSed.

LGTM


Add TBE Header Writer and TBE Header Parser

Summary
Fixes Issue #46 and #48. This PR implements a robust TBE header parser and writer, fulfilling the requirements for parsing, manipulating, and exporting TBE file headers while maintaining data integrity and hierarchical relationships.
Key Features
Parsing Improvements:

Support for Metadata Extraction:

Parses BGN, EOT, and TBL sections, along with their respective attributes (ATT lines).
Stores metadata in a C-native data structure with hierarchical relationships intact.

Error and Warning Feedback:

Provides detailed warnings for unexpected or missing fields.
Outputs clear error messages for critical issues like memory allocation failures or malformed lines.

Flexible and Robust Design:

Handles continuation lines in BGN and EOT sections.
Recognizes and organizes TBL sections and their associated ATT attributes.

Writer Functionality:

TBE Format Export:

Writes parsed metadata back into the TBE format, preserving delimiters (BGN and EOT), sections (TBL), and attributes (ATT).
Ensures compatibility for round-trip conversions (parsing and re-exporting yield identical files).

Validation During Export:

Checks for missing critical data and provides appropriate error messages.
Ensures proper alignment and formatting for exported data.

Memory Management:

Proper allocation and deallocation of all dynamically managed memory (attributes, sections, and the overall header structure).

Improved Integration:

Updates main.c to:

Demonstrate both parsing and writing functionality with a sample TBE file.
Print structured metadata to verify correctness.
Export parsed data to a new TBE file to validate round-trip compatibility.

Updated .gitignore:

Adds exclusions for:

Compiled files (*.o, executables).
Build artifacts and editor-specific files.

Testing
Compilation:

Built the program with the following command:

gcc -o main c/src/main.c c/src/functions/isolate_header.c c/src/functions/output_TBE.c

Ran with a sample file:

./main

Sample Output
Warning [Line 12]: Empty attribute name in ATT line.
Warning [Line 12]: Empty attribute name in ATT line.
Warning [Line 12]: Empty attribute name in ATT line.
Warning [Line 12]: Empty attribute name in ATT line.
Warning [Line 12]: Empty attribute name in ATT line.
Warning [Line 12]: Empty attribute name in ATT line.
Parsed TBE Header:
=== Global Metadata (BGN) ===
  Title: Inventory for bgd
  Source: Duke Bluesky Sensors
  Author: bdefoy
  Date: 2023-05-09 18:08:56
  Comment: Automatically generated by saq_inventory_tsdata.R
  InputFile1: ../Dku_data_pull/Our Data/Bangladesh/2021-10-01_2021-11-01/81432123012/Level1.csv
  Bangladesh: 1. Siddeshwari
  Bangladesh: 2. Dhaka University
  Bangladesh: 3. Mohakhali
  Bangladesh: 4. Basabo, Kadamtala
  Bangladesh: 5. Shimrail
  Bangladesh: 6. Mirpur, Pallabi
  Bangladesh: 7. South Nilkhet
  Bangladesh: 8. Vasantek
  Bangladesh: 9. Matuail
  Bangladesh: 10. Dholaipar
  Bangladesh: 11. Cantonment, Balurghat
  Bangladesh: 12. Bashundhara RA
  Bangladesh: 13. Uttara, sector 10
  Bangladesh: 14. Hemayatpur, NAAFCO
  Bangladesh: 15. Mohammadpur
  Bangladesh: 16. Narayangonj, Gognogor
  Bangladesh: 17. Mirpur, Botanical
  Bangladesh: 18. North Badda
  Bangladesh: 20. Damaged
  Bangladesh: 21. Jagannath University
  Bangladesh: 22. Uttar Khan
  Bangladesh: 23. Bhola, Charfasson 1
  Bangladesh: 24. Mirpur Pallabi 2
  Bangladesh: 25. Nakhalpara
  Bangladesh: 26. Basabo, Kadamtala 2
  Bangladesh: 27. Mouchak
  Bangladesh: 28. Rayerbag
  Bangladesh: 29. Cox's Bazar, Ward-4
  Bangladesh: 30. Cox's Bazar, Tekpara
  Bangladesh: 31. Dhaka University
  Bangladesh: 32. Cox's bazar, Kolatoli
  Bangladesh: 33. DU Club
  Bangladesh: 34. Farmgate
  Bangladesh: 35. Mirpur, Pirerbag
  Bangladesh: 36. Sher-E-Bangla Nagar
  Bangladesh: 37. Basila
  Bangladesh: 38. Mirpur, Shah Ali Bag
  Bangladesh: 39. Khilgaon, Goran
  Bangladesh: 40. Anondo Bazar
  Bangladesh: 41. Banani
  Bangladesh: 42. Nakhalpara, bonoful
  Bangladesh: 43. Rajshahi, Thakurmara2
  Bangladesh: 44. Rajshahi, Nowdapara
  Bangladesh: 45. Rajshahi, Thakurmara
  Bangladesh: 46. Maniknagar
  Bangladesh: 48. Rajshahi University
  Bangladesh: 49. Netrokona, Kendua
  Bangladesh: 50. Netrokona, Kendua2
  Bangladesh: 51. Siddheshwari2
  Bangladesh: 52. Panchagarh, Atwari
  Bangladesh: 53. Panchagarh, Atwari2
  Bangladesh: 54. Khilgaon, Chowrasta
  Bangladesh: 55. Cox's Bazar, Sheetal
  Bangladesh: 56. Cox's Bazar, Tekpara2
  Bangladesh: 57. Dhaka university2
  Bangladesh: 58. Shimrail Indoor
  Bangladesh: 59. Mouchak2
  Bangladesh: 60. Uttora, chondalvog
  Bangladesh: 61. Mugda, GMT
  Bangladesh: 62. Curzon hall
  Bangladesh: 63. Khilkhet
  Bangladesh: 64. Satarkul
  Bangladesh: 65. Sylhet, MPU
  Bangladesh: 66. Sylhet, Ambarkhana
  Bangladesh: 67. Sylhet, MPU2
  Bangladesh: 68. Sylhet, Ambarkhana2
  Bangladesh: 69. Kazlarpar School
  Bangladesh: 70. Gazipur
  Bangladesh: 71. Bhola, Charfasson 2
  Bangladesh: 72. Bhola, Charfasson 3
  Bangladesh: 73. Science Lab
  Bangladesh: 74. CARS
  Bangladesh: Dhaka University 19
  Bangladesh: Dhaka university 47

=== Global Metadata (EOT) ===
  InputFile1093: ../Dku_data_pull/Our Data/Bangladesh/2023-04-01_2023-05-01/81432152032/Level1.csv
  Bangladesh: Dhaka University 75

TBL Section: Variable

TBL Section: country
  Attribute: Name = Name
  Attribute: Integer = Integer
  Attribute: degrees N = degrees N
  Attribute: True=inside = True=inside
  Attribute: Number = Number
  Attribute: Time (Etc/UTC) = Time (Etc/UTC)
  Attribute: hours = hours
  Attribute: None = None
  Attribute: ug/m3 =
  Attribute: Unique ID for Aqsebs = Unique ID for Aqsebs
  Attribute: >1: New site location = >1: New site location
  Attribute: Number of records in input file = Number of records in input file
  Attribute: Offset at New Year = Offset at New Year
  Attribute: From External = From External
  Attribute: Placeholder =
  Attribute: sitename (Name) = sitename (Name)
  Attribute: serial_number (Integer) = serial_number (Integer)
  Attribute: latitude (degrees N) = latitude (degrees N)
  Attribute: is_indoors (True=inside) = is_indoors (True=inside)
  Attribute: nrecords (Number) = nrecords (Number)
  Attribute: date_end (Time (Etc/UTC)) = date_end (Time (Etc/UTC))
  Attribute: utc_offset (hours) = utc_offset (hours)
  Attribute: pm25_scale (None) = pm25_scale (None)
  Attribute: pm25_offset (ug/m3) =

Exported TBE data to c/output_data/output_TBE.csv

@mohamdlog I noticed that this PR addresses fixes for two different issues. I would recommend splitting the fixes into separate PRs — one for each issue — so that each change can be reviewed and tested in isolation. This will also help maintain a cleaner commit history and improve collaboration within the team.


Field lists

Design field lists, considering for both the label and the value:

type size
type weight
line height
color
alignment

Here's the default we get with PyData:

Here's the design for field lists in Figma:
https://www.figma.com/file/ahjEJjeaFL6oTjeQaueGDQ/napari-Sphinx-Theme?node-id=33%3A538

A couple of things different in this new design from the PyData default:

now the labels are right-aligned (this makes it easier for the eye to track from the label to its value, as they are always a consistent distance apart (10px))
the labels are Barlow Semibold
All text in the field lists is the same size as text inside of admonitions
line-height within a single list item (e.g. if it runs to the next line) is 1.5, but the spacing between list items vertically is ~18px

There are a couple of things to update for this one, then it will be good!

[ ] The "fields" text should be black, currently is #333333
[ ] The text size for all the text should be 14px (for wide screens) / 9 px for narrow screens (<= 495px wide) (right tnow the field text is correct size, only the non-field text needs updated)

@liaprins-czi I think this one is fixed, could you check pls?

The fixes look good, but now there is something new happening that wasn't before: the spacing between each line is too tall.
Here's how it should look:

Here's how it looks now:

This looks great to me! 🫶🪐


怎么在apollo上部署新的检测模型？比如yolov5的onnx模型如何部署到apollo上去？是否有相关的资料参考

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04):
Apollo installed from (source or binary):
Apollo version (3.5, 5.0, 5.5, 6.0):
Output of apollo.sh config if on master branch:

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:
怎么在apollo上部署新的检测模型？比如yolov5的onnx模型如何部署到apollo上去？是否有相关的资料参考

Supporting materials (screenshots, command lines, code/script snippets):

你好，目前添加新的检测模型，需要用户添加一个新的Detector，在Detector中进行调用模型对应的推理框架及前处理后处理，可以参考  https://github.com/ApolloAuto/apollo/blob/master/docs/howto/how_to_add_a_new_camera_detector_algorithm.md


Feature Request - Add date stamp on gated messages

Feature Request
We are using the gated message feature to check message arrival over Lora radio and its extremely useful.
Please add, date-stamp to each message.
Love your work is a fantastic map and features

^ ^ One up on this one please - Especially when monitoring several nodes

Howdy! Thanks for the feedback :)
I've updated the messages UI to show the full date time instead of the relative time ago when you click on a message in the list.
Hope this helps!

Hi Liam
Thank you.  to explain what we are using your gated messages for see  https://pole1.co.uk/trial/
this is extremely useful data so our local mesh can see who receives a message sent via mesh/lora
I was hoping for each individual message to have a time-stamp so we can see when it received via your MQTT.
Hope that makes sense 🙂
Cheers
Paul

From: Liam Cottle @.>
Sent: 07 November 2024 10:16
To: liamcottle/meshtastic-map @.>
Cc: Paul-Pressure @.>; Author @.>
Subject: Re: [liamcottle/meshtastic-map] Feature Request - Add date stamp on gated messages (Issue #79)
Howdy! Thanks for the feedback :)
I've updated the messages UI to show the full date time instead of the relative time ago when you click on a message in the list.
Hope this helps!
Screenshot.2024-11-07.at.11.15.03.PM.png (view on web)https://github.com/user-attachments/assets/ff133c8d-4e49-4a12-92fc-605b28422901
—
Reply to this email directly, view it on GitHubhttps://github.com/liamcottle/meshtastic-map/issues/79#issuecomment-2461838562, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BHPLAIB6N7U543EADCP2AYLZ7M4WVAVCNFSM6AAAAABRBUMRPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRRHAZTQNJWGI.
You are receiving this because you authored the thread.Message ID: @.***>


[$1000] Icons in VBA flow are very hard to see or white in dark mode

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

Action Performed:

Access staging.new.expensify.com (Web)
Sign into a valid account
Click on Profile > Workspace > Connect bank > Connect manually (First issue here on Routing page)
Enter routing information and proceed to the next page
Scroll down to the "Incorporation date" section and locate the calendar icon on the right of the input field (Second issue here)

Expected Result:

The background of the routing image should match the dark theme
The calendar icon should be a color that is easily seen in the dark theme
When the user taps the dropdown, all text should be clear and readable

Actual Result:

The background of the routing image is white and does not match the current theme
The calendar icon is black and very hard to see, an average user may miss this entirely if they do not know its there
When the user taps the dropdown, all text is black on a dark background and hard to understand

Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.2.33-1
Reproducible in staging?: y
Reproducible in production?: n
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1669696874240429
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01bad5604e9c4055b5
Upwork Job ID:<PHONE_NUMBER>329428480

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Gonna reassign to myself since Maria might be out for the day.

@dhairyasenjaliya solution one looks good! Can you get a PR up soon since this is a deploy blocker?

Hey @luacmartins sure creating PR now

Cool, I'll assign you to this issue and get the process started! Thanks!

Going to speed this one up since it's a deploy blocker

@mvtglobally both are separate issue I logged here https://expensify.slack.com/archives/C049HHMV9SM/p1669787031782639?thread_ts=1669787031.782639&cid=C049HHMV9SM

@mvtglobally please log another issue for https://github.com/Expensify/App/issues/13132#issuecomment-1332452883

@tjferriss Im haven't reviewed anything, so I’m not eligible for compensation

@tjferriss Is this eligible for a 50% bonus since PR merged within an hour

@mollfpr my mistake.
@dhairyasenjaliya yes, you are eligible for the bonus. Can you please apply to the Upworks job?: https://www.upwork.com/jobs/~01bad5604e9c4055b5

Checklist to close the issue now that the PR is merged:

[ ] If this bug was caught on production, add or update the regression tests (SO for How do I get new regular regression QA tests added?) so that the same bug can’t reach production again and link to the updated test here:
[ ] Link to the PR that caused the bug:
[ ] Comment on the offending PR to notify the author and reviewers that the PR caused this bug.
[ ] Reduce the pay of the C+ on the offending PR if they should have caught the regression (ie. they overlooked a step/checkbox that led to the regression.)
[ ] Start a discussion in #contributor-plus about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
[ ] Complete payment to @contrib1, @cplus1, and @issureporter1 after 7 days.

@tjferriss applie along with bug reporting bonus as well

@dhairyasenjaliya you've been hired.

@tjferriss checklist is done on my end.

I've created the regression test issue and paid out @dhairyasenjaliya. I'll close this out once the regression test has been added.

Applause confirmed we don't need an additional regression test for this issue so we're good to close this out. Thanks everybody.


How to include non-python files

Apologies if this has been posted somewhere but I could not find in the documentation or from search the issues.
I am using uv to try to package an ML model along dependencies, I'd like to include some model weights in the form of safetensors into the package however could not find the relevant option or syntax, either from command line or to add to pyproject.toml.
I was wondering if someone could help point me to the right direction.  Ideally I'd prefer to use one suite / everything uv to create instead of jumping to another set of tools.
Thank you.

uv build relies on the build system you specify, so you should refer to the documentation of the build system you are using.
uv's build system, build backend, is still in preview mode, if I'm not mistaken.

Thank you, @FishAlchemist .  That is helpful.


Potential account/mount problem for Windows using Linux containers

If you are running with an AzureAD account, docker cannot mount to the local drive with those credentials, so the certificate will not be shared/accessible to the Docker image. I chased this myself for close to a day, thinking I had entered paths wrong, generated the certificate wrong, etc. Unfortunately, the only workaround I've found for this specific issue is to create a local account (which some folks cannot do and has implications for folks in regulated environments).
Reference: https://github.com/docker/for-win/issues/132

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

ID: 3e087007-7838-e0f9-d713-22182a5d9704
Version Independent ID: 2f32c7c3-8667-0e88-495b-c0c888572964
Content: Hosting ASP.NET Core Images with Docker over HTTPS
Content Source: aspnetcore/security/docker-https.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Here's another reference: https://github.com/docker/for-win/issues/1352

I believe this is resolved with Docker <IP_ADDRESS>.  See release notes https://docs.docker.com/docker-for-windows/release-notes/   and https://www.docker.com/blog/new-filesharing-implementation-in-docker-desktop-windows/
FWIW I also experienced this challenging problem and several other incompatible windows features when using AzureAD accounts.

Closing due to

I believe this is resolved with Docker <IP_ADDRESS>.

For the next person who finds this issue, does azure ad enable this to work?
What was the result?


Add a build reference accessory workflow for cellranger

We ended up with using the Ensembl instead of Gencode data for building the reference, as the 10x documentation showed here, to reproduce the promising result we got from running the cellranger pipeline against their official 1.2.0 reference.
The reference built by this workflow is uploaded to the public bucket now. The testing workflow runs for this PR are:

https://job-manager.mint-dev.broadinstitute.org/jobs/3a7dd7cf-6394-408a-8a79-9d14a792c642
https://job-manager.mint-dev.broadinstitute.org/jobs/bb2d50b2-37ba-4521-8bd9-5486bfe4ca8a

Please ensure the following when opening a PR:

[ ] Added or updated tests
[ ] Updated documentation
[x] Applied style guidelines
[ ] Considered generalizability beyond our own use case

@dshiga @mckinsel I made the changes based on your comments, would you mind taking another look at the PR? Thanks!

Thanks @rexwangcc, looks good.

LGTM!

I'm going to merge this PR!


Storage code migration

Includes source code as it is from Storage Java SDK master branch with history preserved.
Code location is in storage/data-plane/
In this PR, Storage SDK source code is migrated without any changes.
SDK component is not configured in the build.
No code changes on our end are included in this PR.
Follow up PR will include:
Storage SDK build integration to the central java repo.
Live tests Integration to our Azure Dev Ops pipeline.
Code + Pom cleanup

Can one of the admins verify this patch?

@rickle-msft please look at this PR when you are back, we want to make sure that everything we have in our repo moved over here successfully.
@g2vinay please wait for Rick to approve before merging

@g2vinay Hey mate make sure to hit me up when this merges. I'll need to get the ball rolling on documentation once that happens.
Do you also plan to migrate the preview branches? (if any?)

SDK component is not configured in the build.
No code changes on our end are included in this PR.

Is there any reason we don't go ahead and make those changes as part of this PR as well? There seems like there is a lot of unnecessary files and stuff coming in that we could eliminate completely if we just cleaned them up at the same time. Also as I mentioned in another similar PR we have analysis tools like component governance that will start flagging stuff even if this isn't part of the build.


Add registration codes

Not necessarily useful for boat, but for the beta and other deployments it would be great to have a way to add new registration codes later on. Currently the database must be re-seeded to add more codes. Adding an additional migration to load more codes feels far too heavy handed.
Consider seeding more than 10 default codes?

Since we're gonna reset the beta anyway, the need for this has gone away. I'm just gonna seed it with more codes.


CI: remove webdriver e2e for protractor

Related to https://github.com/jhipster/generator-jhipster/pull/12767

Merging this, following https://github.com/jhipster/generator-jhipster/commit/0a6bd4d1b8a6388d82e80b924c6ca8d8ebb04408#commitcomment-43300552


Fix workflow for MacOS

The MacOS CI workflow currently fails at installing dependencies because it doesn't find apt. I think we need to have a different install script for Mac. Maybe outsource the install script to a shell script so that the tests.yml can be the same for all platforms?

Looks like apt has been removed for some time now

Fixed in e88faed


If Flutter Firebase Messaging is support, occur Error "IAP not prepared. Check if Google Play service is available".

Version of flutter_inapp_purchase
2.2.0
Platforms you faced the error (IOS or Android or both?)
Android
Expected behavior
Show IAP dialog.
Actual behavior
Not show IAP dialog and occur Error.
IAP not prepared. Check if Google Play service is available.

Tested environment (Emulator? Real Device?)
Android: Emulator and Real Devices
Steps to reproduce the behavior

FlutterInappPurchase.instance.initConnection
-> result is "Billing client ready"
FlutterInappPurchase.instance.getSubscriptions
FlutterInappPurchase.instance.requestSubscription
-> throw exception.

Enviroment
$ flutter --version
Flutter 1.20.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision bbfbf1770c (3 days ago) • 2020-08-13 08:33:09 -0700
Engine • revision 9d5b21729f
Tools • Dart 2.9.1

Additional Comments
This issue is depended on the Flutter Firebase Messaging.
Version.
firebase_messaging: 6.0.16

If Background Message(Firebase Messaging Plugins) is support, it occur issue.If Background Message not support, it's not occur issue.
https://pub.dev/packages/firebase_messaging
package io.flutter.plugins.firebasemessagingexample

import io.flutter.app.FlutterApplication
import io.flutter.plugin.common.PluginRegistry
import io.flutter.plugin.common.PluginRegistry.PluginRegistrantCallback
import io.flutter.plugins.GeneratedPluginRegistrant
import io.flutter.plugins.firebasemessaging.FlutterFirebaseMessagingService

class Application : FlutterApplication(), PluginRegistrantCallback {
    override fun onCreate() {
        super.onCreate()
        FlutterFirebaseMessagingService.setPluginRegistrant(this)
    }

    override fun registerWith(registry: PluginRegistry?) {
        GeneratedPluginRegistrant.registerWith(registry)
    }
}

AndroidManifest.xml
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="com.xxxx.xxxx.xxx">
    <application
        android:name=".Application"
        android:label="@string/app_name"
        android:icon="@mipmap/ic_launcher">

did someone find a solution for it?


Enable minRuleThickness in rendering options

This PR introduces a rendering option named minRuleThickness. It enables a web site administrator to specify a minimum thickness, in ems, for fraction lines, \sqrt top lines, {array} vertical lines, \hline, \hdashline, \underline, \overline, and the borders of \fbox, \boxed, and \fcolorbox.  Default is 0.04.
That default is a breaking change for {array} vertical lines, \hline, and \hdashline. KaTeX has, until now, rendered them as 0.05em thick. But LaTeX \showthe\arrayrulewidth reveals that the standard is 0.4pt, i.e. 0.04 KaTeX ems.
When no minRuleThickness is specified KaTeX rendering for these functions looks like this:

I plan to try minRuleThickness of 0.05 on my own pages, but if someone gets bold and sets it to 0.06, the resulting rendering looks like this:

This is in pursuit of issue #1848.

Codecov Report

Merging #1964 into master will increase coverage by 0.08%.
The diff coverage is 87.8%.

@@            Coverage Diff             @@
##           master    #1964      +/-   ##
==========================================
+ Coverage   93.18%   93.26%   +0.08%
==========================================
  Files          79       79
  Lines        4843     4858      +15
  Branches      845      845
==========================================
+ Hits         4513     4531      +18
+ Misses        290      287       -3
  Partials       40       40

Flag
Coverage Δ

#screenshotter
89% <85.36%> (+0.07%)
:arrow_up:

#test
84.52% <60.97%> (+0.04%)
:arrow_up:

Impacted Files
Coverage Δ

src/buildTree.js
100% <ø> (ø)
:arrow_up:

src/fontMetrics.js
100% <ø> (ø)
:arrow_up:

src/environments/array.js
98.61% <100%> (ø)
:arrow_up:

src/Settings.js
75.67% <100%> (+0.67%)
:arrow_up:

src/functions/enclose.js
100% <100%> (ø)
:arrow_up:

src/domTree.js
82.6% <100%> (+1.08%)
:arrow_up:

src/functions/underline.js
100% <100%> (ø)
:arrow_up:

src/functions/overline.js
100% <100%> (ø)
:arrow_up:

src/Options.js
100% <100%> (ø)
:arrow_up:

src/buildCommon.js
98.08% <100%> (ø)
:arrow_up:

... and 2 more

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 17bfb24...8625768. Read the comment docs.

@ronkok this is awesome!  I will try to review this sometime this week. 🤞

@kevinbarabash Thank you for the review. There is good stuff there, but there is quite a lot of it. It may take me a couple of days before I get to it.

Okay, comments are picked up to the best of my ability and screenshots are updated. This is ready for back check.

The most recent update improves the SVG paths for the extra viniculum of a \sqrt. Now the corner radius is at the upper corner at all times. This improves the appearance of a radical when the font size is very large. I have temporarily put a 144px example at http://webmathcomparison.net/. In that example, the value of minRuleThickness is set at 0.06. Here's a screenshot of that example.

Now the corner radius is at the upper corner at all times.

Awesome!


buildinfo.h.in two lines removed

As @SebastianSchildt suggested in #68

PR is fine! put there is some Eclipse Red-Tape, before we can merge it....

You need to make an Eclipse Account https://www.eclipse.org , T
-There, in your account  you can electronically agree to the Eclipse Contributor Agreement
You need to set your github username (junh-ki) in Eclipse Account
Finally, your commit message needs to be "signed-off", that means commiting with git commit -s -m "My commit message"

That is a bit annoying, but it is necessary for Eclipse to automatically check whether you did agree to the contirution agreement (that is the check that failed here)
That is a one time task, once you have done this, you can do PRs all the time 🥳  (only all commits need to be with "-s" parameter


Enforce policies type to be either "compliance" or "control"

@lfu Can you verify the validates logic makes sense and if so the tests will need to be updated as part of this change.

Thanks @lfu, @gmcculloug for review!
It turns out, that spec fix is quite easy. Labeling fine/yes as this relates to bz https://bugzilla.redhat.com/show_bug.cgi?id=1435780

👍 Nice!

I was hoping it would be that easy.  Thanks @isimluk.

When backporting to fine, please backport together with https://github.com/ManageIQ/manageiq-ui-classic/pull/851

Fine backport details:
$ git log -1
commit 20ed0debe3e9d78a1feb3a75bf851393ca0aeb57
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Thu Mar 30 09:22:45 2017 -0400

    Merge pull request #14519 from isimluk/rhbz#1435780

    Enforce policies type to be either "compliance" or "control"
    (cherry picked from commit 6b88182314fd5019bf52c2d9d85ef55364299b8b)


CM-359: Remove konflux specific configs

Remove Dockerfiles created specifically for konflux, since its now hosted in https://github.com/openshift/cert-manager-operator-release.

/test e2e-operator

/lgtm
/label docs-approved
/label px-approved
No user facing chanages, this is only a redundancy de-dup effort as same files landed into o/cert-manager-operator-release repo recently.

/retest

/test e2e-operator

/cc @lunarwhite
for qe-approved label

/retest


update build script to just create panes if tmux session is found

gonna stop making issues for build script, i change it everyday it feels like


onReady and onAfter fire multiple times

Hello,
I've got a small problem:
If I click some links, everything works fine.  The more I change pages, the more the events fire after a pageload.

Where is the JS declaration located? Because it sounds like your initialization of $page.smoothState(options) is located inside the #main element.
Each time SmoothState injects the new content, the JS is parsed and executed. This ends up stacking the number of instances added to the DOM.


Update nginx README.md with -p 8080:8080 published port mappings

Related: #452
This updates the nginx README.md to use -p 8080:8080 published ports. Running the examples as they are results in errors like this:
docker run --rm -d -p 8080:80 cgr.dev/chainguard/nginx:latest && curl -I localhost:8080
a9567dde7c0296bcf5e6581784aeceef1d3d7599c1c482dbb35a5af238adc793
curl: (52) Empty reply from server

Whereas with these changes the following works:
docker run --rm -d -p 8080:8080 cgr.dev/chainguard/nginx:latest && curl -I localhost:8080
3cd44f2f42948ad0299fd333c29c6fd84877b594f829588d84c1ac838a35472f
HTTP/1.1 200 OK
. . .

Pre-review Checklist

[X] Documentation. Let's make this excellent. Include usage example.

I don't get it.. when I run the failing monopod command it seems to work?
monopod readme --check --badge-root-url=https://storage.googleapis.com/chainguard-images-build-outputs/badges --summary-root-url=https://storage.googleapis.com/chainguard-images-build-outputs/summary images/nginx/README.md

Check passed. All README.md files are up-to-date.

Got it all sorted and pushed an updated version.


Add SetCookies to use this API without Login.

It might not be suitable, but i'd like to use it personally. I don't mind if it'll be rejected. @ahmdrz

Can you explain more why you want this ?

hi @ahmdrz
Now i'm preparing a web app which is like auto-liker for my friends.
They need to input their Instagram password on the app, but I don't want to keep their Instagram password cuz of security. I'd like to store only

cookies
goinsta.Instagram.Informations

so that, I can access&like instead of them.
GetSessions() enabled us to get cookies, so I can get cookies tho,,, I was not able to find a way to set cookies. so I've prepared SetCookies().

Hi
Seems good , one of my TODOs is make backup from current informations.
Please note to encrypt sessions and save them.
By the way
Thank you :)

👍


Compatibility with storage blocks from different mods

Describe the Suggestion
This has been a known problem for a long time with forge Create.
However, this is a fabric port so it needs to have different ways of being compatible with different mods?
I'm having a problem with these two:
EnderChests [Fabric]

normal hoppers can input and output items
create doesn't interact with these chests at all

Extended Drawers

create can input and output items reliably when in a block form
when part of a moving contraption (like a drill or harvester), Create doesn't interact with these drawers (unlike forge Create with Storage Drawers mod)

Screenshots and Videos
No response
Additional Context
Would it be possible to make all kinds of storage blocks work with Create, no matter what mod it is?
Do I have to make issues at corresponding mods repos to make them compatible?

Was wondering this as well, after unsuccessfully trying to use chests from Expanded Storage and Tokenable Furnaces and Storage mods in Create machinery. Both of these mods extend vanilla ChestBlock and are normally compatible with every other mod, but not Create.

mysterious

Create has strict limits on what can be used on contraptions since it has to be serialized and saved. I want to look into this further at some point, but other issues are higher priority.


translate: take advantage of Rust Option & Result types

This commit introduces OptionToGlib and TryFromGlib to help with translations from and to Glib types which encode a value with a sentinel to indicate an undefined value and/or for which a range of values is not valid.
These traits allow auto-implementing Option<T>::to_glib, Option<T>::from_glib, Result::<T, _>::from_glib or Result::<Option<T>, _>::from_glib from dependent crates when appropriate despite the orphan rule.
Required for https://gitlab.freedesktop.org/gstreamer/gstreamer-rs/-/issues/234.
Note: I removed the existing fallible implementations of FromGlib for Option<T> (e.g. i32 to u32). I believe we should now use the auto-implemented Result<T, InvalidError> and update the caller where appropriate instead of silently returning None, which is not the same IMHO. See also the comment for impl FromGlib<u32> for char. If you agree, I'll update code in the affected crates in the gtk-rs and gstreamer organizations. Meanwhile, CI will fail.

Looks like a nice improvement overall!

Additional proposal: for signed to unsigned conversions, we could use TryFromIntError instead of InvalidError. Maybe InvalidError is too broad and not necessary since it doesn't allow for auto-implementations anyway.

Additional proposal: for signed to unsigned conversions, we could use TryFromIntError instead of InvalidError. Maybe InvalidError is too broad and not necessary since it doesn't allow for auto-implementations anyway.

Why do we have from_glib() conversions for those anyway (and I doubt they're used widely)? I would remove those and only keep from_glib() / to_glib() for actual GLib types where this makes sense.

Why do we have from_glib() conversions for those anyway (and I doubt they're used widely)? I would remove those and only keep from_glib() / to_glib() for actual GLib types where this makes sense.

I guess that was implemented to handle the cases where the glib type was a signed so that negatives would indicate that the value is invalid (e.g. an error code in a return value) but the actual usable value was positive.

I guess that was implemented to handle the cases where the glib type was a signed so that negatives would indicate that the value is invalid (e.g. an error code in a return value) but the actual usable value was positive.

I believe those can go away. I can't remember a single usage of that :)

I believe those can go away. I can't remember a single usage of that :)

Good! :)

I'm considering removing the InvalidError type and let the user define their own type for TryFromGlib in this particular cases (e.g.: TryFromIntError, CharTryFromError, ...). I would also remove UndefineOrInvalidError in favor of an Either<NoneError, E> where E: Error.
(More context on the above: initially, I thought of InvalidError to auto-implement a Result<T, InvalidError> (in a similar way as NoneError (formerly UndefinedError)), but it didn't make sense since try_from_glib already returns Result<T, InvalidError>. IMO auto-implementing FromGlib for Result<Option<T>, E> when TryFromGlib::<T>::Error matches Either<NoneError, E: Error> is still valuable since that is different from try_from_glib which returns Result<T, Either<NoneError, E>>.)
For the u32 to char conversion, there is TryFrom for char in std since rustc 1.34.0. There's also a nightly from_u32(i: u32) -> Option<char>, but I don't really see the point in returning an Option rather than a Result. So in glib::translate I would remove:

impl TryFromGlib<u32> for char.
impl FromGlib<u32> for char, which either forces functions which use this to return a Result<_, CharTryFromError> (or an Option, but I don't like it) or expect in the wrapper body, which is the current one size fit all behaviour.

What do you think?

Sounds good to me but @GuillaumeGomez will probably be unhappy if you add either as a dependency (fine by me though).

@GuillaumeGomez will probably be unhappy if you add either as a dependency (fine by me though).

In that case, I can make a new enum for this.

I confirm: not happy! :< (but debatable depending on the amount of required code)

I confirm: not happy! :< (but debatable depending on the amount of required code)

Ok! I'll push an update shortly with a local enum so you can tell.

No need for Either: NoneOrInvalidError seems simple enough IMHO.
I'll update gir and take a look at the affected crates tomorrow: Pango & GTK showed up from a rapid check.

Hey @fengalin ! We migrated the glib crate into gtk-rs/gtk-rs repository. Could you please reopen your PR there? :)

Transferred to https://github.com/gtk-rs/gtk-rs/pull/11


Can't access grid layout ref in responsive grid layout

Hi,
The ResponsiveReactGridLayout do not allow to forward a ref to the underlying ReactGridLayout.
This prevents keeping track of the number of columns and size of the layout for example.
To give a use case, I wan't to display a placeholder grid behind the layout during edition, so I need to know precisely it's size. I could compute it from a parent component, but the most straightforward solution is using a "ref" to access the rowHeight, cols and width props.
I've implemented a working version for RRGL 1.16 but I should probably open a PR for new versions with forwardRef.
I will need to implement this sooner or later anyway when I update my project to React 16.
Thanks again for your work on RRGL, it's been running strong for years in our project and provides an excellent user experience.

Stalebot is a bit too eager to close issues btw

The same issue.
@eric-burel Did you solve this problem?


Convert Flag Arguments to a Flag Class

Will reduce the number of arguments in the init methods, and clean up the code.

This will get cumbersome, cleaner to use arguments.


Tools for CamLOS closest points

Create two methods with easy / explicit names in Rays:

one to compute the geometrically closest point of each LOS to an axis (using self.set_dsino() and returning only the distance, theta and pts
one to compute the points of minimum rho from a Plasma2D instance and returning rho, theta and the pts

Fixed in devel by PR #212

Create two methods with easy / explicit names in Rays:
* one to compute the geometrically closest point of each LOS to an axis (using self.set_dsino() and returning only the distance, theta and pts

What do you mean by axis?
In any case if an axis is a line, then there is already a function that is similar to this probably.

Yes, the functions in _GG are all already existing and operational, I just added extra methods to the Rays class with more explicit names from a tokamak point of view (axis = magnetic axis = circle in 3d)
Fixed


Review and request for clarification

My review point sequentially through the doc. But first: it's a remarkable piece of art / work that extends all prior work and design around KERI based or KERI invoked technologies. Well done, Philip. I can grasp it. Here are my two cents about the things I think deserve clarification.

First paragraph: 'This document specifies a design for public VCs only.' -> Explain briefly why private TELs are different and why this is a different topic.

TEL paragraph
This explanation begs for a schematic infographic, but I am afraid that's not done in IETF specs?

VCR paragraph: 'A Public Verifiable Credential Registry (Registry) is a form of a Verifiable Data Registry' -> Me: where is the sugar? Shop owner: next to the salt! ...

VCR paragraph:  'Two types of TELs will be used for this purpose.' -> Be more specific, which two types do you refer back to? I think you mean iception vs. tracking, but could also be issuance vs. revocation, but it's not clear AT THIS POINT in the text. Later it becomes clear that you mean management vs. VC TELs.

management TEL paragraph: pls check definition https://github.com/trustoverip/acdc/wiki/registrar

Self Addressing Identifiers / Derivation paragraphs:
a.. use the abbrev SAID too. Why? Because people might be confused: is this the same Self Addressing Identifier (SAID!) that's all over the place in ACDC?
b. reference the more detailed IETF description of SAIDs and shorten these paragraphs. You don't need to repeat. If it's no repetition then tell me what it is and why you do it. At least it's not clear why SAIDs and its derivation come out of the blue in this IETF spec AT THIS POINT in the text.

Self-Addressing Identifiers in a TEL paragraph: typos in the first sentence makes it gibberish. What is 'wrt'?

Use Case paragraph:
' a lightweight, easy to understand use case ' Don't even go there :) for several reasons:
a. it's not lightweight, it's not easy to understand
b. New acronyms drop in (vLEI, you mean what? Should I know GLEIF?) without any explanation
c. you introduce a mechanism "proof presentation" that hasn't been addressed so far.

Security Considerations paragraph:
What is an Endorser?
TEL events need to be placed in escrow -> first mention, what is it? (I know, but the reader doesn't)
until an anchoring KEL event is seen for the TEL identifier. 'Is seen', do you mean 'first seen' or is first seen not relevant in countering DDOS? (afaik it is).

This spec is clearly still 'Under construction'. Hope you find the time to finish the spec and have some guidance from this review.

If Philip wants to use these remarks to create a new version of the spec one day, it can be kept open, if not it can be closed. These aren't remarks that I could offer in PR unfortunately


Added Covid Tracker in ReactJs

Description
Added Covid Tracker in ReactJs under GSSOC'22 Level 3.
Fixes #743
Screenshots

Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] This change requires a documentation update
[X] Documentation Update

Checklist:

[X] My code follows the style guidelines(Clean Code) of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works

@Ayush7614 please review my PR.


[HOLD] Blazor WASM configuration

Fixes #17419
Internal Review Topic

HostingEnvironment defaults to the Development environment when using the Blazor development server. In non-ASP.NET Core hosts, the Production environment is used by default.

Is that correct? In the past, we've always described the default environment change from Development to Production on the basis of publishing (or explicitly configuring the value), not on the basis of a "server" or "host." Even if it is correct, what is a "Blazor development server"? As far as the docs go, that's a new phrase for something.

@guardrex could we document a special case for PWA - The config JSON file will be cached for offline use. Note that with PWAs specifically, since the configs are cached, you can only edit it when making a new deployment. There's no point editing it between deployments because users will already have a cached version of the file which would continue to be used.

@pranavkm Need a clarification on that: PWAs update in the background and will pull down the new file when? ...

When merely the updated config file alone is updated on the server (i.e., not the whole deployment).
When the whole app is redeployed with the updated config file.

When the whole app is redeployed with the updated config file.

would be the right answer. There's no mechanism to update it in PWAs if the config file is independently updated.

I see the service worker assets manifest doesn't get updated until the app is recompiled.

@pranavkm Try it now. My concern about "Blazor development server" is just a nit, but it sounds like it might be something new/different based on all of our other docs.


[Autocomplete] add reason prop for resetInputValue of useAutocomplete

fixes: https://github.com/mui/material-ui/issues/36085

[x] I have followed (at least) the PR section of the contributing guide.

Demo:
Screencast from 25-04-2023 16:39:41.webm

I am terribly sorry for the number of commits, it's my first time contributing and my vscode on ubuntu is really struggling rn :pray:

I am terribly sorry for the number of commits, it's my first time contributing and my vscode on ubuntu is really struggling rn 🙏

No worries, we squash when merging anyway, thanks for working on this 🙌

Is there a chance this could introduce a breaking change for developers that relied on testing reason === 'reset' but now the values are different?

Is there a chance this could introduce a breaking change for developers that relied on testing reason === 'reset' but now the values are different?

Can you be more specific? which case would be a breaking change?

hi @oliviertassinari @siriwatknp So does that mean that this will never get implemented in MUI 5 or is there anything I can do?

@binh1298 Sorry I overlooked the breaking change – you can re-open the PR (and I'll label it as  breaking change), it can be merged when we are ready to bump the minor version (5.13.x to 5.14.0)!

hi @mj12albert , would you mind help me add the labels for this one?
https://github.com/mui/material-ui/pull/37301


gnome: further changes 3.20 -> 3.24

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Thanks. Can you look into the comments https://github.com/NixOS/nixpkgs/pull/28053#issuecomment-322913930 or should I make a PR later?


Light2D/LightOccluder2D broken after beta5 update

Godot version:
3.1 beta5
OS/device including version:
Wi10 x64
Issue description:
After the beta5 upgrade, the shadows went crazy. Please see the picture:
https://ibb.co/34WZj0X
Steps to reproduce:
Minimal reproduction project:

The lshades look as if they were turned by -90 degrees.

you can attach image directly on github issue though... :)

Could you provide more information about what you did to get those results? I am having trouble reproducing any issues.

@martys71 Are you using a light shader to modify LIGH_VEC by any chance?

@clayjohn: please see this project: https://github.com/DiscoveringGodot/4_Heist_Meisters/tree/5dfb43013cad82f7b21de2b811cb5fd78f279b54
Scenes/Levels/Level1

Regression from  #25779. I now remember why I changed  LIGHT_VEC to not affect shadows... it makes no sense and it breaks with rotated lights.
I just put together a hack to allow both rotated lights and changing shadows with LIGHT_VEC but I still think we should not let LIGHT_VEC affect shadows...


Cannot satisfy requirement with environment marker

I'm trying to build a pex file that includes sanic==19.6.3. In its setup.py file, it requires uvloop>=0.5.3; sys_platform != "win32" and implementation_name == "cpython".
If I build the pex like this everything works:
pex sanic==19.6.3 -o pex.out
But in my build process, I want to first download and store wheels, so that I can then build offline (without PyPI). Normally the following method works with other packages, but it doesn't with sanic. I guess it's because of the environment markers.
$ pip download sanic==19.6.3 -d wheels
[...]
$ pex --disable-cache --no-pypi -f wheels/ sanic==19.6.3 -o pex.out
Could not satisfy all requirements for uvloop>=0.5.3; sys_platform != "win32" and implementation_name == "cpython":
    uvloop>=0.5.3; sys_platform != "win32" and implementation_name == "cpython"(from: sanic==19.6.3)

I guess this is a bug in pex but I'm not even sure. What do you think?
Pex 1.6.11
Python 3.7.4

The downloaded uvloop wheel is named uvloop-0.13.0-cp37-cp37m-manylinux2010_x86_64.whl, and if I simply rename it to uvloop-0.13.0-cp37-cp37m-manylinux1_x86_64.whl it can now be found and the pex file is built without error.
So I guess this is the same issue as #763


Add support for C++ concept symbol kind to index store DB

@swift-ci please test

I haven't been able to find a clang change corresponding to this, is there a PR I missed?

Turns out I don't need this.

The upstream PR is at https://reviews.llvm.org/D126031


Added kitten-tracker plugin

I've added a plugin that allows you to track your kitten status. It will give notifications when anything changes with your kitten (time to feed, time to play with it, grown, overgrown, ran away...). It also shows timers for all of these items. This plugin is based on pull request #2894 that was never finished/merged because the original creator is not on Github anymore. There are some improvements compared to his version. For example, the plugin uses overlays instead of infoboxes to show the timers because they can't show any durations longer than 60 minutes correctly. Also quite a lot of refactoring.
This plugin is especially useful for Iron Men that want to trade in their cats for death runes in Ardougne, that's what I use it for.

In case you didn't notice, your plugin didn't build. (gradle line > 120 characters long, you can see it by clicking details in the failed check).

Hey @geheur,
I totally forgot about this. I did make a change some time ago but I did not change the commit hash in the plugin-hub repo fork. The error should now be fixed. Can you restart the workflow?


chore: Bump fsnotify/fsnotify to v1.6.0

The fsnotify release notes mentions a change that might fix #33:  inotify: don't ignore events for files that don't exist (https://github.com/fsnotify/fsnotify/pull/260, https://github.com/fsnotify/fsnotify/pull/470).
This PR bumps fsnotify version and makes event checking more readable.

Thanks, applied


test questions

Why the detected filter types are all AddressAndPortDependentFiltering, but the map detection has EndpointIndependentMapping and AddressAndPortDependentMapping types. Is there anything to pay attention to in filter detection?

The Behavior test measures how predictable the port mapping is with respect
to source address  (ip and port) to destination address.  Most well behaved
NATs will have Endpoint Independent Mapping - meaning that the same
external port is used to map to the internal ip:port pair regardless of
what remote address is used.
Some NAT environments (whether a result of a NAT device or the ISP) are
Address and Port dependent mapping - which means the port mapping is
difficult to predict. Subsequent P2P or VOIP calls may need a relay server.
The Filtering test measures how open the device can be reached from
addresses and ports it did not originally send to.  Given that most NATs
act as firewalls in addition to the PC having its own local firewall,
incoming packets are simply dropped if there was not any corresponding
outbound packet sent to the remote address it originated from.
Does that make sense? I can elaborate more if needed.
For what it's worth, I don't find the Behavior and Filtering tests
particularly useful for establishing connectivity with another
endpoint. They are best used to simply log the NAT type so you can debug
the reasons for a failed connectivity failure later.
On Thu, Jan 27, 2022 at 12:50 AM gengfeipeng @.***>
wrote:

Why the detected filter types are all AddressAndPortDependentFiltering,
but the map detection has EndpointIndependentMapping and
AddressAndPortDependentMapping types. Is there anything to pay attention to
in filter detection?
—
Reply to this email directly, view it on GitHub
https://github.com/jselbie/stunserver/issues/46, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHNSF2QZPITS4OYZ2I5RXTUYEBONANCNFSM5M5IJQ4Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


no-unused-modules does not work with shebangs

index.js:
#!/usr/bin/env node
import { example } from './foo.js'
export { example }

foo.js:
export const example = true

.eslintrc.yml:
parserOptions:
  ecmaVersion: 2019
  sourceType: module
plugins: [import]
rules:
  import/no-unused-modules: [2, {unusedExports: true}]

Then:
$ eslint
/home/user/example/foo.js
  1:1  error  exported declaration 'example' not used within other modules  import/no-unused-modules

✖ 1 problem (1 error, 0 warnings)

However this works when removing #!/usr/bin/env node.
eslint: 5.16.0
eslint-plugin-import: 2.17.3
node: 12.3.1
OS: Ubuntu 19.04

It doesn't really make any sense to have an export in the same file as a shebang; overwhelming convention is to separate the library part from the bin part of a package.
That said, we should be able to strip the shebang.

Yes I agree. I only added an export in index.js so it does not trigger a warning, to make the issue simpler.
However even without that export, the bug is still present. I.e. foo.js example is reported as not imported, even though index.js imports it.

It's not the rule having a problem with the shebang, it's the default parser used by ESLint:

Switching to another parser (e.g. babel-eslint or @typescript-eslint/parser solves this problem in my test case. Can you please verify it also fixes the error in your project, @ehmicky

Yes it does, thanks! However I think users might not consider switching ESLint parser to get this rule to work.

@ehmicky great to hear, thanks for the feedback.
I just had a look at import/named, which also parses the code to do some analysis. This rule returns a lint error (Parse errors in imported module './foo.js': Unexpected character '#') in case the file couldn't be parsed (see here).
We should do the same for import/no-unused-modules. I will prepare a PR for this later this week.

Shebangs will soon become a regular part of the language, at which point eslint will parse then.
If the file doesn’t parse, these rules should probably bail out.

Bailing out is what already happens. The rule basically fails silently, unfortunately for the wrong file, namely the importing file, not the exporting file.
I think, telling the user that a parsing error occurred at least lets the user know that something unexpected happened. In addition to returning the parsing error as a lint error, I would also explain this error on the readme. What do you think about that?

As long as there's a workaround besides "remove the shebang" or "change the parser".

I don't think this will be a good workaround, as it only tells the user that something unexpected happened. Technically seen there is no way of catching this issue and still reporting the correct results.
The only good workaround would include listening for that parsing error, programmatically and temporarily remove the shebang and re-parse the file. According to how the parsing works, this might be complex, as we can only pass file names to the parser, but no file content (in this case it would be far more easy to just strip out the shebang).

That might actually be tenable if the parser can be made to accept string content, but if not, i'm not really sure what could be done.

I did some quick tests on this. The parsers do accept a string (at least espree does, and the other parsers have to be compatible), and eslint-plugin-import is also already supplying this string.
I did a simple .replace('#!/usr/bin/env node', '') on the supplied string and got the expected linting result (no report about the export from foo.js), without parser errors.
Technically seen, we could strip out the shebang here (only until it is an official part of the language, of course). I can't evaluate, if this is a clean solution or just some quick-and-dirty solution, though.

@ljharb what do you think about my last comment? Would that be a clean solution for this issue?

@rfermann unless node itself exposes a helper function to strip the shebang (which it actually might), i don't think a string replace is reliable, especially because the shebang doesn't have to call into env, doesn't have to be called "node" (even "iojs" would work when installed), etc.

It seems the rules import/default and import/no-named-as-default can also not deal with shebangs in imported files. They error out with Unexpected character '#' (1:1). Interestingly this issue seems only present when importing from ESM files, not from CJS ones.

Would it not be possible to do what eslint does?
const textToParse = stripUnicodeBOM(text).replace(astUtils.shebangPattern, (match, captured) => `//${captured}`);

That's from the parse function in linters/linter.js in the eslint package.

if eslint itself does it, then perhaps this is no longer an issue?

if eslint itself does it, then perhaps this is no longer an issue?

Bug is still valid using latest versions of eslint and this plugin. I've set up a test repo at https://github.com/silverwind/eslint-hashbang-bug which you can clone and run npm install && npm test to reproduce the bug.

Issue is probably that espree is still not able to parse shebangs. I think the fix will be to just do what eslint does and replace it with a line comment before handing it to the parser.
Maybe it's possible to even re-use the linked parse function.

https://github.com/import-js/eslint-plugin-import/pull/2431


Server to write to writer direct

Direct write to io.Writer is about 4-times faster than writing though buffered IO

implemented


Fix bug in Velocity Verlet

Description
@pwborthwick recently pointed out that the Velocity Verlet code had "0*5" instead of "0.5", which led to incorrect behavior. The print statement also was not consistent with Python3.
What are your new additions? Please provide a brief list.

Changes

[ ] Fix typo in Velocity Verlet code
[ ] Change print statements to be python3 consistent

Status

[ ] Click when ready for review-and-merge

Codecov Report

Merging #103 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

Changes look good but, for the record, that's not the version I wrote; my version's in Tutorial 12a


Issue on docs

Path: /sdk/php/guides/laravel
I really want to use this, but install does not work and there are many issues. Does not feel production-ready.

curl --proto '=https' --tlsv1.2 -sSf "https://darkterminal.github.io/turso-php-installer/dist/turso-php-installer.phar" should be curl --proto '=https' --tlsv1.2 -sSf "https://darkterminal.github.io/turso-php-installer/dist/turso-php-installer.phar" -O

php --m | grep libsql should be php -m | grep libsql

After running turso-php-installer install it says "Turso Client is Ready!". Then I run php -m | grep libsql and libsql is not installed :(

The install script should be non-interactive. Currently that is not possible.

@darkterminal are you able to help out here and see what's going on?

I will work for that... thank you for reporting this issue...


Better indication of kernel having pending cells

For #7661

Codecov Report

Merging #7808 (c8d87c0) into main (1f7fcf5) will decrease coverage by 0%.
The diff coverage is 92%.

@@          Coverage Diff          @@
##            main   #7808   +/-   ##
=====================================
- Coverage     69%     69%   -1%
=====================================
  Files        363     363
  Lines      22285   22298   +13
  Branches    3375    3377    +2
=====================================
- Hits       15521   15512    -9
- Misses      5423    5441   +18
- Partials    1341    1345    +4

Impacted Files
Coverage Δ

src/client/datascience/jupyter/kernels/types.ts
100% <ø> (ø)

...ent/datascience/jupyter/kernels/kernelExecution.ts
72% <88%> (+3%)
:arrow_up:

...client/datascience/commands/activeEditorContext.ts
91% <100%> (+<1%)
:arrow_up:

src/client/datascience/jupyter/kernels/kernel.ts
73% <100%> (-1%)
:arrow_down:

.../client/datascience/debugLocationTrackerFactory.ts
42% <0%> (-8%)
:arrow_down:

src/client/datascience/jupyter/jupyterDebugger.ts
19% <0%> (-7%)
:arrow_down:

...datascience/editor-integration/codelensprovider.ts
69% <0%> (-5%)
:arrow_down:

...datascience/editor-integration/cellhashprovider.ts
69% <0%> (-3%)
:arrow_down:

...client/datascience/kernel-launcher/kernelDaemon.ts
56% <0%> (-2%)
:arrow_down:

...lient/datascience/variablesView/notebookWatcher.ts
88% <0%> (+1%)
:arrow_up:


Password manager not storing passwords

OS/Platform
macOS
Installed
Other (please use 'Additional context' field below to mention how exactly)
Version
126.0.6478.126
Have you tested that this is not an upstream issue or an issue with your configuration?

[X] I have tried reproducing this issue in Chrome and it could not be reproduced there
[X] I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there
[X] I have tried reproducing this issue in ungoogled-chromium with a new and empty profile using --user-data-dir command line argument and it could not be reproduced there

Description
Passwords will not store in password manager
How to Reproduce?
1: create password / attempt to import from csv
2: browse to chrome://password-manager/passwords
3: passwords are not listed
Actual behaviour
Despite having a seemingly normal Login Data database, I cannot get chromium to save passwords nor can I import a passwords.csv or have it access the Login Data database. However, it seems cookies will retain login status.
Expected behaviour
Passwords normally appear in the interface.
Relevant log output
N/A

Additional context
I remember when upgrading chromium versions in the past (from release builds via homebrew) I would have to give access to the macos keychain, im assuming so it could read ~/Library/Application Support/Chromium/Default/Login Data. I noticed on this upgrade that prompt did not appear even after reinstall. Could this have to do with the new iCloud password feature? Login Data database is still populated with what seems to be the correct data however I have not attempted to decrypt the blobs. Running MacOS 14.5.
Please let me know how I can assist in finding a solution to this. Thank you.

Unfortunately, I, following these steps, cannot reproduce this issue.
Steps Followed:

Import passwords from a CSV.
Browse to chrome://password-manager/passwords

Everything is retained even after restarts.

I noticed on this upgrade that prompt did not appear even after reinstall.

That prompt should reappear every time a new version is reinstalled, and I assume that this is where the problem comes from. I suggest that you find a way to do a complete fresh install and test again.

Thanks for the response.
Yeah most likely is a MacOS keychain issue or something. Weird though because I've been on 14.5 without modifying any settings for quite a while now and have had previous versions of chromium upgraded on 14.5 with no issue. I do not have time to troubleshoot further today but will report back when I find a solution in case someone else runs into this problem.

The issue I encountered is the same as the one you described.
130.0.6723.116（Official Build, ungoogled-chromium） (arm64)
OS: 15.1 (24B2083)


[bugfix] Respect default values for module objects in the configuration

Fixes #1943

@jg can you check that this works for you?


How to use proxy v1.1

Hello, i am using v1.1 api. My connect method

const client = new TwitterApi({
appKey: 'CjulERsDeqhhjSme66ECg',
appSecret: 'IQWdVyqFxghAtURHGeGiWAsmCAGmdW3WmbEx6Hck',
accessToken: 'accestoken',
accessSecret: 'secret',
});
How can use all request w/ proxy
Note: For questions about how to use an endpoint, or problems related to Twitter API than the lib itself, please use the GitHub Discussions instead of opening a new issue.

Closed? Why?

As explicitly described in issue template, issues are not a place to ask questions on this repository, but for reporting bugs. Discussions are meant to seek help.
Furthermore, documentation about proxies is available on the first page of docs: https://github.com/PLhery/node-twitter-api-v2/blob/master/doc/basics.md#make-requests-behind-a-proxy
And related issue + pull can be found in a 5 secs search on GitHub: #157 + #149.