Master

added hierarchical (flattened) and reentrant locked machines

Hi,
let me know what you think about this (flattening) approach to HSM and thread-safe function access. I had to do some minor adaptions to the main files to ease subclassing but tried to keep the changes as minimal as possible. The only functional change I have done is to remove the New Node from diagram creation and replaced it with the first state in the machines state list. I did this because New is not connected to the state diagram if 'New' is not part of the machine. If you find the approach to be sufficient but have some remarks concerning code style/best practise/philosophy, let me know and I will do by best to make the code 'blending' into transitions as best as possible. If this approach does not fulfil some requirements, I am open to constructive criticism/feedback as well :).
Best regards,
Alex

This looks great, thanks! I'll review this as soon as I can--probably on Monday.

Okay, I've looked through the code... This is really great--thanks so much for the contribution! I only have a few minor stylistic comments (and I'll probably make some minor edits to the README after this is merged):

I'm not sure about using the underscore character to concatenate nested state names. I think the forward slash is probably just as good for purposes of nesting, and would be less likely to cause confusion, since using the underscore in variable names is quite Python and is likely to be used a lot in states that don't involve nesting. I recognize that it probably won't cause any problems functionally if users mix non-nested states with underscores and nested states, but it seems a bit confusing. So unless you have objections, I'd prefer to see 'foo/bar/baz' rather than 'foo_bar_baz'.
In the counter example, the instance name is 'counter' in some cases and 'calculator' in others--this should be made consistent.
Siince LockedMachine and HierarchicalMachine seem to have identical code, could we maybe use use a mixin pattern or something to manage this without the redundancy?
For consistency, we should probably call the LockedHSM class 'LockedHierarchicalMachine'. It's kind of annoying to type, but I think it's good to keep the nomenclature consistent.
To minimize redundancy, I think the 'Stuff' and 'Inherited Stuff' classes in the testing modules should probably be moved into a separate module (probably 'utils') under testing, and then they can be imported in multiple places as needed.

Otherwise everything looks great! If you don't have time to make some of the above changes, just let me know, and I can always merge first and then patch it myself.

Hi,
thanks for the feedback.

In the counter example, the instance name is 'counter' in some cases and 'calculator' in others--this should be made consistent.

fixed

Since LockedMachine and LockedHSM seem to have identical code, could we maybe use use a mixin pattern or something to manage this without the redundancy?

fixed; I had tried multiple inheritance for LockedHierarchicalMachine but could not get it to work. I assumed there were issues with super and __getattribute__ in multi-inheritance scenarios... well, I tried again, now it works \o/... seems these issues were related to previous locking approaches.

For consistency, we should probably call the LockedHSM class 'LockedHierarchicalMachine'. It's kind of annoying to type, but I think it's good to keep the nomenclature consistent.

agree and fixed; Co-worker of mine always says 'code is usually read way more often than it is written' and with code completion it does not really matter.

To minimize redundancy, I think the 'Stuff' and 'Inherited Stuff' classes in the testing modules should probably be moved into a separate module (probably 'utils') under testing, and then they can be imported in multiple places as needed.

fixed; called the module test_utils

I'm not sure about using the underscore character to concatenate nested state names.

That one is quite tricky. I started with dots actually (like domains) but went for underscore later on because dots mess with auto_transitions. E.g. model.to_A.1 does not work and model.to_A/1 wont work either. The characters allowed in Python 2.x keywords are quite limited. Double underscores (to_A__1) would be an alternative or replacing '/' or '.' in function keywords only. State would be called 'A.1' but auto_transition would be to_A_1. The first approach stretches functions calls and second approach is rather inconsistent. Both are quite nasty. What do you think?

Oh, good point re: auto_transitions. I agree that the alternatives are messier, so let's stick with underscore for now and revisit it in future if it becomes an issue. Thanks for making all the other changes!

On Sat, Jan 2, 2016 at 10:51 AM, aleneum<EMAIL_ADDRESS>wrote:

Hi,
thanks for the feedback.
In the counter example, the instance name is 'counter' in some cases and
'calculator' in others--this should be made consistent.
fixed
Since LockedMachine and LockedHSM seem to have identical code, could we
maybe use use a mixin pattern or something to manage this without the
redundancy?
fixed; I had tried multiple inheritance for LockedHierarchicalMachine but
could not get it to work. I assumed there were issues with super and
getattribute in multi-inheritance scenarios... well, I tried again,
now it works \o/... seems these issues were related to previous locking
approaches.

FYI, I was running into this multiple inheritance issue with v0.2.9 as
well. getattribute was doing a locking check to make sure self !=
self.model but that is gone in favor of the inspect module and I haven't
had problems since then.

For consistency, we should probably call the LockedHSM class
'LockedHierarchicalMachine'. It's kind of annoying to type, but I think
it's good to keep the nomenclature consistent.
agree and fixed; Co-worker of mine always says 'code is usually read way
more often than it is written' and with code completion it does not really
matter.
To minimize redundancy, I think the 'Stuff' and 'Inherited Stuff' classes
in the testing modules should probably be moved into a separate module
(probably 'utils') under testing, and then they can be imported in multiple
places as needed.
fixed; called the module test_utils
I'm not sure about using the underscore character to concatenate nested
state names.
That one is quite tricky. I started with dots actually (like domains) but
went for underscore later on because dots mess with auto_transitions. E.g.
model.to_A.1 does not work and model.to_A/1 wont work either. The
characters allowed in Python 2.x keywords
https://docs.python.org/2/reference/lexical_analysis.html#grammar-token-identifier
are quite limited. Double underscores (to_A__1) would be an alternative or
replacing '/' or '.' in function keywords only. State would be called 'A.1'
but auto_transition would be to_A_1. The first approach stretches functions
calls and second approach is rather inconsistent. Both are quite nasty.
What do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/tyarkoni/transitions/pull/52#issuecomment-168427391.

--
~Wilfred Tyler Gee


Can't update some services: no such manifest

Hello
shepherd does not update some services
no such manifest: docker.io/crazymax/swarm-cronjob:latest
Sun Jul  2 00:04:00 +05 2023 Error updating service portainer_swarm-cronjob! Image crazymax/swarm-cronjob:latest does not exist or it is not available
no such manifest: docker.io/openproject/community:12
Sun Jul  2 00:04:04 +05 2023 Error updating service project_project! Image openproject/community:12 does not exist or it is not available
no such manifest: docker.io/gregewing/apt-mirror:latest
Sun Jul  2 00:04:07 +05 2023 Error updating service repo_repo! Image gregewing/apt-mirror:latest does not exist or it is not available
no such manifest: docker.io/zabbix/zabbix-agent2:ubuntu-6.4-latest
Sun Jul  2 00:04:35 +05 2023 Error updating service zabbix_zabbix-agent2! Image zabbix/zabbix-agent2:ubuntu-6.4-latest does not exist or it is not available

But here they are
https://hub.docker.com/layers/crazymax/swarm-cronjob/latest/images/sha256-93b3142c7686d3ad32db7164084fa28239e336f92f557dd3a9ccb3e3b500319f?context=explore
https://hub.docker.com/layers/openproject/community/12/images/sha256-1da78ac275865b21f17d304d0dda375e99b0918d8eb6f8f343cd26793a965c50?context=explore
https://hub.docker.com/layers/gregewing/apt-mirror/latest/images/sha256-1f72138d40ad2b91fa1be1477a6c15b27f72a0b040cf4ad28518ab11a6a42bc5?context=explore
https://hub.docker.com/layers/zabbix/zabbix-agent2/ubuntu-6.4-latest/images/sha256-56fde7922d3b240fa4013c9c9d59629eeebfb9f8b8986291e6bfa94c470fb1a8?context=explore
It's not temporary. Such logs every day. Only these services, other are fine.

Mhh.. Which Docker version is installed on your host?

24.0.2 on every node
docker node ls
ID                            HOSTNAME   STATUS    AVAILABILITY   MANAGER STATUS   ENGINE VERSION
om2y8v8dbxp7t30dbxeqzd4g0 *   docker01   Ready     Active         Reachable        24.0.2
37vo26jcvf4u0v21kw0bxwg3u     docker02   Ready     Active         Leader           24.0.2
vnqc6bxlvapfxhev81vbyd93i     docker03   Ready     Active         Reachable        24.0.2

Do you have any plans to migrate on docker:23 ?

Finally an explanation to the cause of this issue :)
Been seeing this for 6+ months

Same for me, where I mostly see it with GHCR.
Also docker 24.

Do you have any plans to migrate on docker:23 ?

Yes, we should do that of course. I guess there are no downsides to that.
(Sorry for the late reply.)

Well, I was a bit quick. Migrating to the official image means losing armhf support. I would be okay with that. Don't know if anyone still uses this on armhf?
Not sure if that breaking change would need a new major release (2.0) though. Currently thinking about doing a minor one only.

This issue may be related.
https://github.com/orgs/community/discussions/45779


import-grammars should format output using prettier

one option could be:
exec("npx prettier ./src/parser/config/nodeTypesConfig.json --write");
or use the API:
https://prettier.io/docs/en/api.html

[ ] move prettier from devDependencies to dependencies

I just thought about making import-grammars a npm run script instead of including this into the CLI of the parser?

The idea in general is nice. But I would argue that the output doesn't have to be human readable.  Most IDEs and Editors support this out of the box. I would rate this below low priority therefor I close it.


Remove version: Jackett.Jackett version 0.21.771

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.21.771/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


[HUDI-1517] create marker file for every log file

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
Create marker file for every log file. It will improvement perf of rollback of log file when marker file based rollback is enabled. It will also help cleaning partial generated log file which can be deleted. HUDI -3026 depends on this change.
https://issues.apache.org/jira/browse/HUDI-1517
Brief change log

Add a callback to HoodieLogFormatWriter so that we can inject mark file creation when log file is created.

remove previous logic which generate marker use simulating data file name
generate marker for each log file
distinguish markers against log file for new log file from existing log file.

Change reconcile logic in HoodieTable so that partial created log file can be deleted.

for log file created, we can treat them as data file which is different with appending log file.

Add rollback logic in markerFileRollbackStrategy so that we can use markfile to rollback log blocks.

Adjust upgrade and downgrade UT to adapt this change

Verify this pull request
Callback logic: it is minor and few thing we can test. So no specific UT for it.
reconcile logic of log file : Add a UT for this.
rollback logic: covered by existing UT.
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Hi @yihua
This PR is ready for review. Could you have a look when you have spare time.

sorry. will review in a day or two.

@nsivabalan By the way, the following case may be handled in a more efficient way if we have marker for each log. Looking forward to your opinion.
In one commit,
task attempt 1 create a log file .fileId_instantTime.log.2_0-0-1. But this task attempt failed.
task attempt 2 create a new log file .fileId_instantTime.log.2_0-0-2. This task attempt succeeds and .fileId_instantTime.log.2_0-0-2 is returned in writeStat.
Before: log file is fail-safe. So we preserve both of them and it should be idempotent to handle the content twice.
Now: we can delete .fileId_instantTime.log.2_0-0-1 in 'reconcileAgainstMarkers' phase because it is an invalid file, which improves perf little. But this way no longer treat log file fail-safe totally.

hi @guanziyue can rebase this pr?

@hudi-bot run azure

hi @guanziyue can rebase this pr?

Done...

Overall looks good！

hi @guanziyue Overall looks good！

@XuQianJin-Stars Hi XuQianJin, as we discussed offline, I just revert changes which change existing methods signature.

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

hi @guanziyue any chance you would rebase and resolve the conflicts? let's land this soon

hi @guanziyue any chance you would rebase and resolve the conflicts? let's land this soon
Done. Could u approve CI running?

hi @guanziyue any chance you would rebase and resolve the conflicts? let's land this soon

Hi @xushiyan , I have it rebased on latest master branch. Could you kindly have a look on this?

took a quick pass and lgtm! will take another tmr. we should be landing it soon

I cherrypick this PR to the internal HUDI. And find a FileNotFoundException during read latest snapshot of a mor table.

The exception would happen if enable spark speculative feature under the following case.

Job1 is writing to a MOR table and not finished yet. It enables spark speculative feature.
Job2 is reading the latest snapshot from the MOR table, when it call getLatestMergedFileSlicesBeforeOrOn, it might list the log files which are written by speculative attempt task in Job1.
Job1 is finished, deletes the log files which are written by slow speculative tasks.
Job2 throws the FileNotFoundException when it read the log file which is already deleted in step3.

Revert this PR because of severe regression~


Allow transparency to be set for room colorcondition

This will allow transparency to be specified in the colorcondition. If no state is matched, the default transparency will be used. Using the color name of transparent throws unnecessary console errors.
Example configuration:
entities:
  - entity: <an entity>
    type3d: room
    object_id: <a room object (generally the floor) with a name containing "room". >
    room:
      elevation: 254
      transparency: 100 #default if no transparency is specified in colorcondition.
      attribute: state
    colorcondition:
      - color: yellow
        transparency: 80
        state: 'on'

Hello. I really woul dlike to use this feature for presence detection, is there an updated working version with this code? Thaks in advance!


Rev. column of History tab should not show I-D version for RFC items

Describe the issue
On the History tab of a published RFC, there are lines in the history that apply to the RFC, but the last version of the I-D is given in the Rev column instead. For instance, for RFC 9019:
Date       | Rev. | By       | Action
2022-12-20 | 16   | (System) | Received changes through RFC Editor sync (changed pages to 24)
2021-04-30 | 16   | (System) | Received changes through RFC Editor sync (created alias [RFC 9019](https://datatracker.ietf.org/doc/rfc9019/), changed abstract to 'Vulnerabilities in Internet of Things (IoT) devices have raised the need …
2021-04-30 | 16   | (System) | RFC published

RFC9019 is not draft-ietf-suit-architecture-16.  Instead of the Rev column showing the I-D version number ("16"), perhaps it should show "(RFC)" or the actual RFC number ("RFC 9019").
Code of Conduct

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

This is something to address when we refactor RFCs to be of their own Document type in the datatracker. See #2917.
We could look at a short term fix to the history page that overrides the text in the Rev column (instead of showing the rev listed in the docevent) at the cost of extra computation (slower rendering, but maybe not enough to matter).


Child list unique key issue

For some reason using the CSS prop we get:
App.tsx?t=1719036648473:43 Each child in a list should have a unique "key" prop.

We are using a Fragment to render the style tag... but fragments should not give this error? Maybe it is an issue with React 19? Or we'll have to add a key here

I thought the isStaticChildren boolean flag would fix this, but it did not... 🤔
So problem is that with createElement you spread out children, but you can not do that with jsxDev.

I tried adding keys to the children, but I still get the error... I am stumped 😮

Okay, it actually worked with isStaticChildren... it happens if the css prop element has children

Okay, this is fixed


Add a bootstrap pod to configure provisioner

Second part of https://github.com/kubernetes-incubator/external-storage/issues/131. Do not merge before https://github.com/kubernetes-incubator/external-storage/pull/156 is merged.
How it works:
kubectl create -f deployment/kubernetes/example-config.yaml
kubectl create -f deployment/kubernetes/admin-account.yaml
kubectl create -f deployment/kubernetes/bootstrapper.yaml

first command creates a configmap for local volume discovery config
second command creates a service account for bootstrapper, and bind cluster-admin role to it. This allows bootstrapper to read configmap, as well as creating rolebindings for provisioner.
third command creates bootstrapper.  Bootstrapper will read the configmap, creates service account and rolebindings for provisioner (rolebindings are system:persistent-volume-provisioner and system:node).  It then creates provisioner daemonset based on volume config and pass the config information to provisioner.

Using a bootstrapper, user is not supposed to directly interact with provisioner.
Work items for future PRs:

Make provisioner parameter (like image name) configurable
Clean up resources upon error
Volume config parameter mountDir can be auto-generated ?
Update local volume docs

@msau42

rebased and updated

@msau42 updated, PTAL.  I'll add test in later PRs.

Rebased, PTAL

Updated @msau42

lgtm
The verification on the makefile is failing though:
repo-infra/verify/verify-boilerplate.sh
Boilerplate header is wrong for: /home/travis/gopath/src/github.com/kubernetes-incubator/external-storage/local-volume/provisioner/Makefile


Tooltip issue

Forge version: <IP_ADDRESS>55
Mod version: distinctdamagedescriptions-1.3-beta
There is only DDD mod.

I have tried to tweak client config. There still is the problem.

I have tested to change language. Only English is correct to display.

DDD doesn't have any updated localization that supports all of 1.3's features, so even if the tooltips didn't come out as a garbled mess, the text wouldn't be fully translated. Translation strings for the type names aren't actually provided by external lang files (although they could and probably should be). Some languages may be very challenging to support, given how DDD uses a custom font renderer to display damage type colours, and it may not behave very well for other languages. ("may not behave very well" is an understatement, it clearly doesn't from the screenshot you've provided)
While trying to tweak or patch the font renderer so it behaves correctly with other languages is certainly a noble cause, I expect that to be a little... challenging, given the difficulty I had implementing it to begin with 😅
I'll attempt to at least look into why the text comes out completely unsalvageable and see if I can fix that problem. Outside of that, the rest of this issue just boils down to the fact that localization for other languages doesn't exist for DDD and I'm a predominately English speaker so I can't write those lang files with any hopes of them being accurate.

I guess you break something in 1.3-beta.
There is not any problem in 1.2.1-beta.


[DOC] Add tshoot section with TLS/HTTP mixed datasource

Description
Introducing a troubleshooting section using the initial article Using Perses with TLS and Prometheus with plain HTTP.
Reviewer Hints
I've created a directory collecting troubleshooting articles. I have added the screenshots in the same directory. There is an "images" directory, I wasn't sure if it's a good idea to get all images in a similar spot. Just thinking out loud, do you want screenshots here? They come with a maintenance cost and they get outdated soon.
Checklist

[x] Pull request has a descriptive title and context useful to a reviewer.
[x] Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
[x] All commits have DCO signoffs.

@Nexucis here is a proposal for the troubleshooting section. Feel free ripping it apart :)

Yeah that's ok for me. Thank you for doing it @indigo423.
Regarding the images, I would prefer to use the github storage for that. A tips is to open an issue, then copy and past the image that will give you a link, and then use this link in the mardown file. Like that we don't store any images in git

Regarding the images, I would prefer to use the github storage for that. A tips is to open an issue, then copy and past the image that will give you a link, and then use this link in the mardown file. Like that we don't store any images in git

I've removed the images from the repository, created an issue with the screenshots for reference and used the links instead.

nice thanks @indigo423. Actually you didn't need to create an issue. Just going to the template to be able to copy and past the image to generate the link. And then you can forgot the issue once you have the links. But no worries it's just a trick.
Thank you for doing it !


DEVX-1576 Updates dependency libs to CP 5.4.1

Also updates the README instructions as well as the default values in the Makefile
After a 0.3.1 version is published to Confluent-Hub, we can build docker images from 5.4.1 for kafka-connect-datagen.
Of note: CP Operator has not yet released a 5.4.1 patch for their images, so we cannot yet build and publish datagen images based on cp-server-connect-operator for > 5.4.0

is my understanding correct that you'd like us to publish version 0.3.1 after this PR is merged?

Yes @C0urante if possible.  I'm assuming here we'd want to build a new patch version of this library after updating the dependent library, so I assumed we'd go to 0.3.1.    After a new version is built I can build docker images for our demos.

Version 0.3.1 released!


add --config-files option to fpm command

This is to avoid overwritten of config file when update package.
Signed-off-by: Kai Zhang<EMAIL_ADDRESS>

What exactly does this problem solve?

@otoolep you could find something here: https://github.com/jordansissel/fpm/issues/463
Let met show an example in rpm.
In short, if I update a package under Centos by rpm -U influxdb-xxx.rpm, the config file which was in /etc/opt/influxdb/influxdb.conf will be replaced by the config file in the new rpm. It means modifications on old config file will be lost, which is not expected.
If we add --config-files option to fpm command, the old config file will be remained after the package is updated.
A more sophisticated rule could be find here http://www-uxsup.csx.cam.ac.uk/~jw35/docs/rpm_config.html
However, I'm not quite familiar with deb package. But I think it should have a similar effect.

OK, this is a real issue -- thanks for bringing that option to our attention I'll need to test it out.
@toddboom -- what do you think we want to happen? I think not overwriting installed config files is probably best.

OK, I tested this with debian package creation, and it works well. Thanks again @kylezh .


Rename watcher.go to confl.go

Changes Unknown when pulling 2321da78778d1ce80023306e3df9af574586b1ad on xusss-patch-3 into ** on master**.


Bump to v0.15.0

Superseded by https://github.com/rinsed-org/administrate/pull/7 which takes us to v0.16.0


https://docs.microsoft.com/en-us/azure/aks/devops-pipeline?pivots=pipelines-yaml&tryIt=true&source=docs#code-try-1

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks create Extension Name: aks-preview. Version: 0.5.51.
Errors:
The command failed with an unexpected error. Here is the traceback:
can't set attribute 'nat_gateway_models'
Traceback (most recent call last):
  File "/opt/az/lib/python3.10/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/rolando/.azure/cliextensions/aks-preview/azext_aks_preview/custom.py", line 770, in aks_create
    aks_create_decorator = AKSPreviewCreateDecorator(
  File "/home/rolando/.azure/cliextensions/aks-preview/azext_aks_preview/decorator.py", line 1541, in __init__
    self.models = AKSPreviewModels(cmd, resource_type)
  File "/home/rolando/.azure/cliextensions/aks-preview/azext_aks_preview/decorator.py", line 117, in __init__
    self.init_nat_gateway_models()
  File "/home/rolando/.azure/cliextensions/aks-preview/azext_aks_preview/decorator.py", line 145, in init_nat_gateway_models
    self.nat_gateway_models = nat_gateway_models
AttributeError: can't set attribute 'nat_gateway_models'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks create --resource-group {} --name {} --node-count {} --enable-addons {} --generate-ssh-keys

Expected Behavior
Environment Summary
Linux-5.4.0-1083-azure-x86_64-with-glibc2.28 (Cloud Shell), Common Base Linux Delridge (quinault)
Python 3.10.4
Installer: DEB

azure-cli 2.37.0

Extensions:
containerapp 0.3.5
application-insights 0.1.15
aks-preview 0.5.51
azure-firewall 0.13.0
ai-examples 0.2.5
ssh 1.1.1

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

route to CXP team

@rotorr Thanks for reaching out to us and sharing this feedback. We are looking into this. We will provide an update once we have more details.

@rotorr I see that you are using an older version of aks-preview . Could you please update it to the most recent version using the below command and then test again ?

az extension update -n aks-preview

Hope this helps. Awaiting your reply.

@rotorr I wanted to do quick follow-up to check if you had a chance to look at my above comment. Please let us know if you  are still facing the issue after following the above plan. Awaiting your reply.


Fix Issue: #147 - Making the navbar on Audio page responsive

Added responsiveness to navbar.
Added smooth scroll and hamburger menu with toggle functionality with ("x") svg.

Added responsiveness to navbar.
Added smooth scroll and hamburger menu with toggle functionality with ("x") svg.

Can uh link the issue ?

@Susmita-Dey chk if its properly done

Added responsiveness to navbar.
Added smooth scroll and hamburger menu with toggle functionality with ("x") svg.

and yes,plz mention issue

Added responsiveness to navbar.
Added smooth scroll and hamburger menu with toggle functionality with ("x") svg.

Can uh link the issue ?

As mentioned in the title its issue no: #147

Hey can u merge the PR, I don't it has any issues.

@SahilDahekar Your brach has merge conflicts. Kindly solve them other I can't merge your PR.
__
  Summary: :hammer_and_wrench: This code needs a fix  |  Tags: Not reusable

Hey I just fixed the merge conflicts now you can merge my PR

@SahilDahekar fix merge conflicts
__
  Summary: :hammer_and_wrench: This code needs a fix  |  Tags: Not reusable

okay i'll fix it asap

LGTM @Susmita-Dey


Add byte viewer Float and Double

Add basic Float and Double view in byte viewer.

Need to think more how edit could be implemented.  Having trouble wrapping my head around things like pos<->neg, changing the exponent, sig figs... basically all of it.

Editing the raw byte data as an integer would be the easiest.
Actually to make a very professional elegant double/float editor would be a great tool.
But you would have to deal with nuances of:
positive/negative
normalized/denormalized numbers
infinity/NaN (not a number)
So if exponent is 0 then denormal number, exponent 0xFF then +/- infinity or NaN
I would suggest radio buttons for Normalized, Denormalized, infinity, NaN
Then an input box for the exponent (if its normalized/denormalized)
And an input box for the significand (if its normalized/denormalized)
Finally a radio button for positive and negative (if its normalized/denormalized/infinity - maybe for NaN - normally the sign bit here is nothing more than a custom signal - for completeness though)
See here for more: https://en.wikipedia.org/wiki/Single-precision_floating-point_format
I still feel the 4 control input style would give a very quick easy and flexible way to deal with the internals of floating point values both double and single precision.

I did not try the PR, but about ongoing discussion: I always thought https://www.h-schmidt.net/FloatConverter/IEEE754.html was a pretty good float editor.
For changes which affect the UI, please include a screenshot in the first post.

The inclusion of this feature would solve a significant problem I've been dealing with recently. Although personally reading is the higher priority, I agree that for writing handling it as raw bytes would be the best approach, at least until someone expresses the need to be able to do the conversion within the tool itself. I can't think of a way to elegantly express all the possibilities without it feeling like bloat.


Tests running twice on each PR

Noticed this has been happening lately. I wonder if it's because both PRs and Pushes are listed as instigating events, and maybe only Pushes are needed? @kaitj

Is this happening when the PR is merged? Whenever it is happening, it is likely because both PRs and Pushes are listed as you said. I think we can be more explicit in the actions workflow about the triggers for each by using types. I think what may be happening right now is both the push event, as well as the synchronize event for PRs are getting triggered.

I'm not totally sure what the pattern is, tbh. It's not just when it's merged, though. Check the current open PR for an example. It's not a massive issue, just something that came up recently. We can observer longer for a pattern if necessary

Just to close this off properly, I now understand why this issue occurs. It happens, e.g. when making a change directly on Github and making a PR. Github creates a new branch and pushes your changes to that branch, triggering one instance of tests. Then, it creates a PR from your branch, triggering the second instance of tests. Because both of these things happen "simultaneously", they all get grouped and summarized under the PR results panel, but in reality, both sets of tests always occur. The "push" tests are just usually in the background.
Long story short: nothing to be concerned about here


deleted.

deleted.

If it "Sometimes also doesn't work after device reboot." Then this is a faulty camera module. Not a "jailbreak" issue.


test: add test in TestResponseReadBodyStream in the case of ErrBodyTooLarge

What type of PR is this?

test
Check the PR title.

[x] This PR title match the format: <type>(optional scope): <description>
[x] The description of this PR title is user-oriented and clear enough for others to understand.
[ ] Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.
在TestResponseReadBodySteam中添加了一个body过大的测试用例

 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.shenfc seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add nano to Breeze environment

Make sure to mark the boxes below before creating PR: [x]

[X] Description above provides context of the change
[X] Unit tests coverage for changes (not needed for documentation changes)
[X] Target Github ISSUE in description if exists
[X] Commits follow "How to write a good git commit message"
[X] Relevant documentation is updated including usage instructions.
[X] I will engage committers as explained in Contribution Workflow Example.

In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.

Oh, I hoped to learn vim :<

There is still a chance :) . We are not removing vim ... Not while I am on the watch.
Besides vim is easy: https://vim.rtorr.com/  :bookmark:


[FEATURE] Specify default value for num_workers

🚨🚨 Feature Request

[ ] Related to an existing Issue
[ ] A new implementation (Improvement, Extension)

If your feature will improve HUB
For methods that take a num_workers argument for parallelizing, specify a default value like None that sets it to the max value (num cores total).
1.pytorch
2.tensorflow
3.compute
Difficulty: Easy

Hi, this seems easy. I am new to open source and would like to work on this. Can you clarify what you mean by:
1.pytorch
2.tensorflow
3.compute

Can I take this up .
Just to clarify my approach , some of the methods and functions do not have a num_workers default size set for example https://github.com/activeloopai/Hub/blob/7ffb209c3166d18ba837a94f940ac228834edd4a/hub/core/storage/prefetch_lru_cache.py#L39  .
The num_workers have to be given a default value which could be None (as mentioned in the issue )or can be set to 2 (which is the minimum present in every system).
Does this sound good?

@DiptoChakrabarty, greetings, and welcome to Hub! Yep, your approach sounds good, and looking forward to seeing you open a PR solving this issue!

hey @dhiganthrao I guess the previous method I said was incorrect just wanted to clarify that methods and functions associated with pytorch tensorflow and compute , in there num_workers has to be as default None to pickup thr max num cores.
for example here
https://github.com/activeloopai/Hub/blob/f1b487d646d2e738af4636a8bce10bf66299cc11/hub/core/dataset.py#L474-L478
Is this approach correct?

Hey, I am new to opensource and would like to work on this issue, could someone guide me as to how I would go about accomplishing this ?

Hello @DiptoChakrabarty, I am new here. Just saw this issue. I am just starting off with opensource . Maybe you can guide a bit and we can fix the issue together.

Hey @vaibhav-35. This feature has already been completed. I'm sorry about our poor management of Github issues. I will close this one.


HDDS-7755. Ensure that acquired locks are always released.

What changes were proposed in this pull request?
There occur some possibilities that acquired locks are not released due to exceptions. This change corrects them so that locks are always released theoretically.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7755
How was this patch tested?
Standard CI. https://github.com/duongkame/ozone/actions/runs/3954812858

Thanks @duongkame for the fix.


S3:GetBucketLocation throws AuthorizationHeaderMalformed when using cross-account roles

Description
We have an application that assumes a role in other AWS accounts, and performs actions on those accounts such as getting information about buckets in S3.
When calling GetBucketLocationAsync, if the bucket is in a different region than the client, the client will throw an exception:
AuthorizationHeaderMalformed: The authorization header is malformed; the region 'us-east-1' is wrong; expecting 'us-west-2' status code: 400
Where in the above, 'us-east-1' is the region the client is for, and 'us-west-2' is the region of the bucket. It is unclear as to whether this always happens, or happens periodically. We saw it happen in the past, and then it seemed to have stopped, and then started happening again last week.
This also seems to have been an issue with the Go SDK in the past, as seen here:
https://github.com/aws/aws-sdk-go/issues/380
The same exact workaround, setting ForcePathStyle = true on the client, makes this exception go away. However, I'm not sure if there's any possibly unintended side effects of doing that, and also it doesn't seem like that should be required in the first place.
Reproduction Steps

Make role in Account A that gives access to s3:GetBucketLocation, and allow that role to be assumed by Account B
Running a C# application In Account B, assume that role, and create an IAmazonS3 client using those credentials
Perform the GetBucketLocationAsync call on the client, directed at a bucket in Account A
See exception that occurs.
Make another IAmazonS3 client with those credentials, also specifying ForcePathStyle = true
Perform the same GetBucketLocationAsync call on the client, directed at a bucket in Account A
See that no exception occurs.

Environment
We are currently on a slightly outdated version of the SDK, but it doesn't appear that anything has changed that would have fixed this. I very well could be wrong with that statement, though, so please correct me if it has been.

SDK Version: AWSSDK.Core <IP_ADDRESS>
Package Version: AWSSDK.S3 3.5.4
OS Info: AmazonLinux
Build Environment: dotnet command line inside AmazonLinux as part of Dockerfile
Targeted .NET Platform:  Linux x64

This is a :bug: bug-report

Further information -- it definitely appears to be a periodic issue, as it isn't happening right now, but was happening Friday afternoon. Same exact code in our application.

Hi @Dreyerized,
Good afternoon.
Would it be possible for you to share a sample code to reproduce the issue? Also confirm if this is still an issue.
Thanks,
Ashish

I am having the same issue. The task is to filter the buckets based on selected region (from user). It works fine for the default region: us-east-1 but when I change the region, I get the error: The authorization header is malformed; the region 'us-east-1' is wrong; expecting 'eu-north-1'. This is the sample code.
var client = new AmazonS3Client(accessKey, secretKey, RegionEndpoint.GetBySystemName(selectedRegion))
var buckets = client.ListBuckets().Buckets;

buckets.ForEach(item => {
    var location = GetBucketLocation(client, item.BucketName);
    ...
    if (location == selectedRegion)
    {
        result.Add(item);
    }
    ...
});

private string GetBucketLocation(IAmazonS3 client, string bucketName)
{
    GetBucketLocationRequest request = new GetBucketLocationRequest { BucketName = bucketName };
    GetBucketLocationResponse response = client.GetBucketLocation(request);
    return response.Location;
}

Environment
SDK Version: AWSSDK.Core 3.7.6
Package Version: AWSSDK.S3 <IP_ADDRESS>
OS Info: Windows 2019 Server

Seeing exactly same issue as @NJ261 and @david-dreyer mentioned above using code snippet that @NJ261 provided.
Works fine for default region of us-east-1 and actually works fine for some other region, but only the very first time i call GetBucketLocation for the run of the application. if i call it on subsequent times with a mismatching region, then it will fail as mentioned above.


Container div prevents using sortable for table rows

Because sortable wraps the decorated component with a <div> element, sortable cannot be used for table rows or list items. We've fixed this in our project by modifying react-sortable to render the decorated component directly as the root component with no containing element. This requires that the wrapped component be aware of and render the props generated by sortable. In our case, the wrapped component uses the object spread syntax to ensure all sortable-generated props are rendered.
<tr {...this.props} />

We're open to better alternatives, since I understand this pattern leads to new "unknown props" warnings in 0.15.2. https://facebook.github.io/react/warnings/unknown-prop.html
Any ideas?
cc @ryanswanson

Fixed in current master https://github.com/danielstocks/react-sortable#changing-the-wrapping-tag

That strategy fixes the issue for list items, but I'm not sure how to use it with table rows. The wrapped component is rendered as the immediate child of sortable, but in the case of table rows children needs to be an array of td or td components, not a single component.

I see, and what would you suggest?

Alright, I did another refactoring. You can find working table in https://github.com/danielstocks/react-sortable/tree/master/example-table.

Thanks, that should work for us. Closing this issue.


Appending empty type to a struct increases its size

What version of Go are you using (go version)?

$ go version
go version go1.17.1 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/kirr/.cache/go-build"
GOENV="/home/kirr/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/kirr/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/kirr/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/kirr/src/tools/go/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/kirr/src/tools/go/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.17.1"
GCCGO="/usr/bin/gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3103072093=/tmp/go-build -gno-record-gcc-switches"
GOROOT/bin/go version: go version go1.17.1 linux/amd64
GOROOT/bin/go tool compile -V: compile version go1.17.1
uname -sr: Linux 5.10.0-8-amd64
Distributor ID:	Debian
Description:	Debian GNU/Linux 11 (bullseye)
Release:	11
Codename:	bullseye
/lib/x86_64-linux-gnu/libc.so.6: GNU C Library (Debian GLIBC 2.31-13) stable release version 2.31.
gdb --version: GNU gdb (Debian 10.1-1.7) <IP_ADDRESS>10103-git

What did you do?

Hello up there. I was implementing custom RangedSet class via instantiaion of custom RangedMap class with void value type. Unfortunately having that - even empty - value type in the map entry structure increases its size, as the following program demonstrates:
---- 8< ---- (https://play.golang.org/p/Ed358IA13Fd)
package main

import (
        "fmt"
        "unsafe"
)

type KeyRange struct {  // sizeof(KeyRange) == 16
        Lo int64
        Hi int64
}

type void struct{}      // sizeof(void) == 0

type Entry1 struct {    // sizeof(Entry1) == 16  == 0 + 16
        void
        KeyRange
}

type Entry2 struct {    // sizeof(Entry2) == 24  != 16 + 0
        KeyRange
        void
}

func main() {
        fmt.Printf("sizeof(KeyRange):  %d\n", unsafe.Sizeof(KeyRange{}))
        fmt.Printf("sizeof(void):      %d\n", unsafe.Sizeof(void{}))
        fmt.Printf("sizeof(Entry1):    %d\n", unsafe.Sizeof(Entry1{}))
        fmt.Printf("sizeof(Entry2):    %d\n", unsafe.Sizeof(Entry2{}))
}

kirr@deca:~/tmp/trashme$ go run zero.go
sizeof(KeyRange):  16
sizeof(void):      0
sizeof(Entry1):    16
sizeof(Entry2):    24

What did you expect to see?
I expect that sizeof(Entry1) == sizeof(Entry2) == 16.
What did you see instead?
sizeof(Entry1) == 16, but sizeof(Entry2) == 24  for (imho) no good reason.
The issue is present in all Go versions starting from at least Go1.11, but nevertheless I believe this is a bug.

For the reference: in https://github.com/golang/go/issues/40322 the topic of stride alignment being included into sizeof was raised. However in hereby case there is no any extra stride as the KeyRange type is already well aligned to be strided well out of the box.
Thanks beforehand,
Kirill

Though maybe it is due to GC, so that if &Entry2.void pointer is taken, it still always points to inside of Entry2 object...

Yes, exactly.  This is intended behavior.  See #9401.

Thanks for the pointer, @ianlancetaylor.


Does SuperLogin work with a PouchDB server?

Although the docs hint towards this, I wasn't sure. The reason why I ask is because I am leaning towards the PouchDB-find project for easier in-browser queries compared to the PouchDB views. It sadly only works (for remote instances) with  PouchDB server or CouchDB 2.0 preview. I have decided to use PouchDB server and hence this question.
Thanks in advance!
Sukant

Great question @SukantGujar ... I haven't tested it but it should work. PouchDB-Find also works with Cloudant, which I have tested against extensively with SuperLogin.

Okay, I will validate it with PDB-server as well as CouchDB 2.0. WIll let you know how it goes. Cloudant won't work for me as I am looking for a self-hosted CDB :).

So here's the result of test specs run against pouchdb-server -

As you would notice, only one test fails. Would it cause issues with superlogin?

Update - I traced the test in Visual Code and found that pouchdb-server deletes the testkey correctly, but then returns the reason of querying the deleted document as "missing", whereas the test is expecting "deleted". This causes this test to fail.

Still not sure if this check will cause supperlogin to glitch or not. Please comment.

This shouldn't cause any problems in SuperLogin. The issue is that PouchDB-Server is emitting a different reason code than CouchDB. This is safe to ignore for now. I can update the test to check for 'deleted' OR 'missing'.

Awesome, thanks!
On Apr 20, 2016 12:35 AM, Colin Skow<EMAIL_ADDRESS>wrote:This shouldn't cause any problems in SuperLogin. The issue is that PouchDB-Server is emitting a different reason code than CouchDB. This is safe to ignore for now. I can update the test to check for 'deleted' OR 'missing'.
—You are receiving this because you were mentioned.Reply to this email directly or view it on GitHub


discovery: tests: use collect instead of t in EventuallyWithT calls

What does this PR do?
Some tests of the service discovery test suite use t instead of the provided assert.CollectT argument, causing flakiness in tests. This PR fixes that.
Motivation
Reduce test flakiness.
Describe how you validated your changes

Possible Drawbacks / Trade-offs
Additional Notes

/merge


Left-align feature text

Awright, enough people seem to want this that we'll go with it for now.  Can always tweak it again later.


[Feature Request] Add support for M204

Thank you very much for developing this analyzer. I think it is really awesome.
Could you add support for M204 gcode? Slicers use this to set the actual acceleration: https://marlinfw.org/docs/gcode/M204.html and https://reprap.org/wiki/G-code#M204:_Set_default_acceleration

Yeah it could be useful, but the issue is that if you don't set acceleration settings in your slicer, then no M204 gcode is added to the file


Coweta County, GA

E-mailed 02/28/2017 7:00 a.m. EST
Clint Richmond
GIS Analyst, Information Technology
22 E Broad Street, Suite 252
Newnan, GA 30263
<EMAIL_ADDRESS>678-854-0029
website:http://www.coweta.ga.us/index.aspx?page=685

Justin,
We can provide this data for free since it has to do with a governmental project or database; however we do need you to fill out our annual data exchange agreement.  Please sign, scan, and return.
Then I can put this data together for you.
Thanks,
Adam Ray
GIS Technician, Information Technology
Coweta County Board of Commissioners
22 East Broad Street, Suite 250
Newnan, GA 30263
<EMAIL_ADDRESS>http://www.coweta.ga.us
(770)254-3716 x8213
CowetaCountyDataExchangeAgreement v080516.pdf

not sure how to handle this!?

@iandees or @nvkelso thoughts? Thanks!

I think @iandees needs to do the honors with signature.

Looks like Coweta has an ESRI server: http://gis.coweta.ga.us/arcgis/rest/services/Parcels/MapServer/0
Is this usable for our purposes?

@msirenko Yes, looks like it! Here's a preview:

http://www.arcgis.com/home/webmap/viewer.html?url=http%3A%2F%2Fgis.coweta.ga.us%2Farcgis%2Frest%2Fservices%2FParcels%2FMapServer&source=sd

Lovely render; surprising to see something look so nice out of an automated tool.

Points are here:  http://gis.coweta.ga.us/arcgis/rest/services/AddressPoints/MapServer/0


Plan creation throws error if there are no operations

It would be preferrable if an empty array was initialize for operations when creating a plan.

I believe this is no longer an error.


Adding Scroll Bottom to Top navigation button

can you Add  GSoc ext and hacktoberfest Label to this improvement feature......................

Can you assign me under with some labels

I would like to work on this issue , as I have a strong background in CSS and HTML. Please assign me under GSSOC'24 extended

@Kavish-Paraswar stop spamming in other peoples issue

@Kavish-Paraswar create your own issue i saw you have not create any issue of your own but have commented of every other person's issue

@Kavish-Paraswar this is not good open source practice if i find you doing this again i will block you from contributing in this repo

@Kavish-Paraswar atleast check once that's already added to the website

@devxMani today i will reorganizing all the issue which means closing a lot of issues as people have made the issue for the same problem again and again

everyone here join the discord server for communication i am active there

yes, JOIN DISCORD...............

this issue has been solved, so we are closing this issue.


[computer use] Counting tokens with new count_tokens() method?

In trying to get a better picture of how many tokens each message that Claude sends needs during computer use I am trying to use the new token count endpoint which is right now in beta.
This works fine for regular text messages and responses, but I am struggling with the tool use:
According to the docs for computer use (https://docs.anthropic.com/en/docs/build-with-claude/computer-use) this is how a regular request gets sent:
import anthropic

client = anthropic.Anthropic()

response = client.beta.messages.create(
    model="claude-3-5-sonnet-20241022",
    max_tokens=1024,
    tools=[
        {
          "type": "computer_20241022",
          "name": "computer",
          "display_width_px": 1024,
          "display_height_px": 768,
          "display_number": 1,
        },
        {
          "type": "text_editor_20241022",
          "name": "str_replace_editor"
        },
        {
          "type": "bash_20241022",
          "name": "bash"
        }
    ],
    messages=[{"role": "user", "content": "Save a picture of a cat to my desktop."}],
    betas=["computer-use-2024-10-22"],
)
print(response)

The docs for the token counting endpoint (https://docs.anthropic.com/en/docs/build-with-claude/token-counting) give this example for tool use:
import anthropic

client = anthropic.Anthropic()

response = client.beta.messages.count_tokens(
    betas=["token-counting-2024-11-01"],
    model="claude-3-5-sonnet-20241022",
    tools=[
        {
            "name": "get_weather",
            "description": "Get the current weather in a given location",
            "input_schema": {
                "type": "object",
                "properties": {
                    "location": {
                        "type": "string",
                        "description": "The city and state, e.g. San Francisco, CA",
                    }
                },
                "required": ["location"],
            },
        }
    ],
    messages=[{"role": "user", "content": "What's the weather like in San Francisco?"}]
)

print(response.json())

The thing that I have not figured out yet is how to translate the computer-use tools into the correct format to query the token count, since this approach does not work:
import anthropic

client = anthropic.Anthropic()
response = client.beta.messages.count_tokens(
        betas=["token-counting-2024-11-01"],
        model="claude-3-5-sonnet-20241022",
        tools=[
            {
              "type": "computer_20241022",
              "name": "computer",
              "display_width_px": 1024,
              "display_height_px": 768,
              "display_number": 1,
            },
            {
              "type": "text_editor_20241022",
              "name": "str_replace_editor"
            },
            {
              "type": "bash_20241022",
              "name": "bash"
            }
        ],
        messages=[{"role": "user", "content": "Save a picture of a cat to my desktop."}],
    )

    print(response.json())

anthropic.BadRequestError: Error code: 400 - {'type': 'error', 'error': {'type': 'invalid_request_error', 'message': 'tools.0.type: Extra inputs are not permitted'}}

Duh - I realize my mistake - both beta flages must be added:
import anthropic

client = anthropic.Anthropic()
response = client.beta.messages.count_tokens(
        betas=["token-counting-2024-11-01","computer-use-2024-10-22"],
        model="claude-3-5-sonnet-20241022",
        tools=[
            {
              "type": "computer_20241022",
              "name": "computer",
              "display_width_px": 1024,
              "display_height_px": 768,
              "display_number": 1,
            },
            {
              "type": "text_editor_20241022",
              "name": "str_replace_editor"
            },
            {
              "type": "bash_20241022",
              "name": "bash"
            }
        ],
        messages=[{"role": "user", "content": "Save a picture of a cat to my desktop."}],
    )

    print(response.json())

{"input_tokens":1953}

Well, maybe this will be useful for someone else.

Sorry for reopening this, but I realize that it is not really clear to me how to correctly count tokens for tool useage in the context of computer use:
This is a typical entry from the requests that uses the bash tool
[
  {
    "role": "assistant",
    "content": [
      {
        "type": "text",
        "text": "I'll use the bash tool to echo \"hello\":"
      },
      {
        "type": "tool_use",
        "name": "bash",
        "input": {
          "command": "echo \"hello\""
        }
      }
    ]
  }
]

This is the result of that tools use:
{
  "content": [
    {
      "type": "tool_result",
      "content": [
        {
          "type": "text",
          "text": "hello"
        }
      ],
      "tool_use_id": "toolu_01AUudSxQG4DytRGBbD8guvJ",
      "is_error": false
    }
  ],
  "role": "user"
}

It is not really clear to me which of these values go where in the client.beta.messages.count_tokens() call - do I put them into the messages part? Do I pass certain parameters or encode them all into the "content" string? Do the tool_use and the tool_result go into two separate calls? A simple code example would be really very nice to have - thank you!

After a lot of try and error I figured it out - The "secret" is that for tools one has to pass in the assistants "tool_use" and the users(!) "tool_result" together in the message. Also the "tool_use_id" has to be passed as "id":
response = client.beta.messages.count_tokens(
        betas=["token-counting-2024-11-01","computer-use-2024-10-22"],
        model="claude-3-5-sonnet-20241022",
        tools=[
            {
              "type": "computer_20241022",
              "name": "computer",
              "display_width_px": 1024,
              "display_height_px": 768,
              "display_number": 1,
            },
            {
              "type": "text_editor_20241022",
              "name": "str_replace_editor"
            },
            {
              "type": "bash_20241022",
              "name": "bash"
            }
        ],
        messages=[{"role": "assistant", "content": [
           {
            "type": "text",
            "text": "I'll use the bash tool to echo \"hello\":"
           },
           {
           "type": "tool_use",
           "name": "bash",
           "id": "toolu_01AUudSxQG4DytRGBbD8guvJ",
           "input": {"command": "echo \"hello\""}}]},

           {"role": "user",
            "content": [{
              "type": "tool_result",
              "content": [
                {
                  "type": "text",
                  "text": "hello"
                }
              ],
              "tool_use_id": "toolu_01AUudSxQG4DytRGBbD8guvJ",
              "is_error": False
            }]}])

{"input_tokens":2030}

One more observation - removing tools that are not used in that call from the "tool" list gives a different count:
response = client.beta.messages.count_tokens(
        betas=["token-counting-2024-11-01","computer-use-2024-10-22"],
        model="claude-3-5-sonnet-20241022",
        tools=[
           {
              "type": "bash_20241022",
              "name": "bash"
            }
        ],
        messages=[{"role": "assistant", "content": [{"type": "text","text": "I'll use the bash tool to echo \"hello\":"},
           {"type": "tool_use","name": "bash","id": "toolu_01AUudSxQG4DytRGBbD8guvJ","input": {"command": "echo \"hello\""}}]},

           {"role": "user", "content": [{
              "type": "tool_result",
              "content": [
                {
                  "type": "text",
                  "text": "hello"
                }
              ],
              "tool_use_id": "toolu_01AUudSxQG4DytRGBbD8guvJ",
              "is_error": False
            }]}

        ])

{"input_tokens":651}

For counting the tokens used in the context of the actual request of an ongoing thread one probably has to subtract the "tax" for the "tools" object since it will probably only be tallied once for all (e.g. when you use bash 3x it only costs once)


Automatically start/stop Sauce-Connect.jar

I wrote this wrapper into my helper and test suite files and thought it might be of use to some other folks:
# helper.rb
# Add this in addition to your normal helper code
require 'net/http'
require 'zipruby'

ready_file = 'sauce_ready'
sauce_file = 'Sauce-Connect.jar'

unless File.exists?(sauce_file)
  puts "Fetching Sauce-Connect.jar"
  url = "http://saucelabs.com/downloads/Sauce-Connect-latest.zip"

  Zip::Archive.open_buffer(Net::HTTP.get(URI.parse(url))) do |zf|
    zf.fopen(sauce_file) do |f|
      open(sauce_file,'wb') do |file|
        file.write(f.read)
      end
    end
  end
end

# Get the Sauce configuration information
cfg = Sauce::Config.new()
# Create a command to spawn the jar file
cmd = "java -jar #{sauce_file} -f #{ready_file} #{cfg.opts[:username]} #{cfg.opts[:access_key]}"

# Run the command
`#{cmd} > /dev/null 2>&1 &`
@sauce_connect_pid = $?.pid
puts "Connect PID: #{@sauce_connect_pid}"

# Wait for Sauce to be ready
until File.exists?(ready_file)
  puts "Waiting for Sauce Connect"
  sleep 5
end

#testsuite.rb
require "test/unit"
require "test/unit/testsuite"
require "test/unit/ui/console/testrunner"
require 'helper'
require 'tc_home'
require 'tc_ajax'

# Need to load the tests manualls so we can
#   override at_exit
class TS_web
  def self.suite
    suite = Test::Unit::TestSuite.new('Sauce Tests')
    suite << AJAX.suite
    suite << Home.suite
    return suite
  end
end

# Need to do this for now in order to kill the
#   process when we're done
Test::Unit::UI::Console::TestRunner.run(TS_web);

# HACK: pid is of the `cmd`, which spawns java
at_exit{ Process.kill('TERM',@sauce_connect_pid+1);}

I have some code for automatically doing this, replacing the Python v1 connector, that I'm working on merging in, mind taking a look here

I [created a pull request] with some clean-ups that @santiycr and I hacked on at @SauceLabs' office, if you feel like taking it for a test-drive.


Introduce httpclient for handling requests in DorisStreamLoad

Proposed changes

Refactor DorisStreamLoad to use httpclient for handling HTTP requests and repsonses

Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

cc @hf200012 @JNSimba


⚠ Bofrost.at has degraded performance

In c49426e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2161 ms

Resolved: Bofrost.at performance has improved in 27a6a73.


I din't find the two-stage association part in the dev-branch's code, is this part not pushed yet ?

Thanks for replying

Thank you for your interest! Sorry for not being able to provide a detailed explanation, as I am still cleaning the code and writing documentation (should be finished in January).
The part you are looking for should be in mot_3d/redundancy/redundancy.py, the function motion_model_redundancy. Please let me know if you have further questions.

Thank you for your interest! Sorry for not being able to provide a detailed explanation, as I am still cleaning the code and writing documentation (should be finished in January).
The part you are looking for should be in mot_3d/redundancy/redundancy.py, the function motion_model_redundancy. Please let me know if you have further questions.

@a954217436 Sorry for not @ your before, could you check if this is helpful for you?

Thanks for replying. I'll study the code and then reply you~
And I'm looking forward to your cleaning code.

OK, I understand the RedundancyModule's function now. This is what I'm looking for, thanks a lot!

@a954217436 Thanks! By the way, I have updated a version of clean code to the main branch. Feel free to check it out!


[Feature] Don't store redundant primary key columns

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
I found that Paimon will redundantly store the primary key columns in files with a name prefixed with _KEY_, this increase the query overhead and storage size. Especially when there are several primary key fields, like 5 or even more.
Is it possible to directly use the original column values to be projected as the primary key during query? I think that can reduce the load of reading and writing, as well as the storage volume.
Solution
Generate primary key values directly from original columns instead of use redundant primary key columns.
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Search before asking

[x]  I searched in the issues and found nothing similar.

Motivation
I found that Paimon will redundantly store the primary key columns in files with a name prefixed with _KEY_, this increase the query overhead and storage size. Especially when there are several primary key fields, like 5 or even more.
Is it possible to directly use the original column values to be projected as the primary key during query? I think that can reduce the load of reading and writing, as well as the storage volume.
Solution
Generate primary key values directly from original columns instead of use redundant primary key columns.
Anything else?
No response
Are you willing to submit a PR?

[ ]  I'm willing to submit a PR!

Yes, of course, there is a draft: https://github.com/apache/incubator-paimon/pull/1825 but he have no time to move forward.

Thanks for the reply, good to know that we’re already working on this!发自我的 iPhone在 2024年2月26日，17:22，Jingsong Lee @.***> 写道：
Search before asking
I searched in the issues and found nothing similar.
Motivation
I found that Paimon will redundantly store the primary key columns in files with a name prefixed with KEY, this increase the query overhead and storage size. Especially when there are several primary key fields, like 5 or even more.
Is it possible to directly use the original column values to be projected as the primary key during query? I think that can reduce the load of reading and writing, as well as the storage volume.
Solution
Generate primary key values directly from original columns instead of use redundant primary key columns.
Anything else?
No response
Are you willing to submit a PR?
I'm willing to submit a PR!
Yes, of course, there is a draft: #1825 but he have no time to move forward.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

https://github.com/apache/paimon/pull/4651

https://github.com/apache/paimon/pull/4666

We are introducing this optimization!


Fix for Example 2

The example code was to add "HBI" but the comment said it would result in adding "For Your Eyes Only".

Docs Build status updates of commit f12916f:
:clock10: Full build: the average full build time is 17 min(s) 39 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit f12916f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-pnp/Add-PnPSiteClassification.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 1b3d866:
:clock10: Full build: the average full build time is 17 min(s) 38 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 1b3d866:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-pnp/Add-PnPSiteClassification.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit edd942a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-pnp/Add-PnPSiteClassification.md
:warning:Warning
View (sharepoint-ps)
Details

sharepoint/sharepoint-ps/sharepoint-pnp/Add-PnPSiteClassification.md

Line 0, Column 0: [Warning-ms-prod-or-service-missing] One of the following attributes is required: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


chore: Add a nightly workflow

You could also just download the libwasmer file, and run the tests with go test -v -tags custom_wasmer_runtime, see https://github.com/wasmerio/wasmer-go#supported-platforms (unfold the What to do if your platform is missing? Section).

Wiaint for #211 as we will use the same logic here.


Add example without decorators

Not everyone uses them
As an example here's how we use redux-async-connect (0.1) in our app:
class DashboardRoute extends React.Component<IDashboardRouteProps, {}> {
  static reduxAsyncConnect(params, store) {
    return Promise.all([store.dispatch(getDashboardData())]);
  }
}

What would be the equivalent in redux-connect? getDashboardData is a redux action here so we need the dispatch as well

Decorator is a function that creates a wrapper component. So like with any other thing what you would do is asyncConnect(connect(component))
That is pseudocode, but I hope you get the example. There might be no connect part, etc.
Just understand that decorator is a syntactic sugar for creating a wrapped smth, ie class or fn
decorator(Mixed) returns WrappedMixed

I'm aware of that but was the feature of just using a static method removed since the version we're using?

Its not a static method anymore, but a static property with an array in it
Code is here: https://github.com/makeomatic/redux-connect/blob/master/modules/containers/decorator.js
This is a wrapper:
https://github.com/makeomatic/redux-connect/blob/master/modules/containers/decorator.js#L10-L38
And then it adds a redux-connect wrapper, which maps all asyncItems with 'key' to components props:
https://github.com/makeomatic/redux-connect/blob/master/modules/containers/decorator.js#L44-L55


[QUESTION] Handle "keyup" event to compare with loaded elements

I want to show to the user a message like "it will be stored as a new type" if he writes a new option in the searchbox.
First of all (before using select2) I had a jQuery Autocomplete function to do this with the keyup event, but I don't know how to do this in the better way with select2.
Thanks in advance

Are you talking about the tagging feature?

Ups! Sorry. The issue is solved since I read the docs :)
This can be closed.

No problem, the docs only recently became actually usable 😄


Metrics

In your paper,  you have tested on Img-based and Txt-based datasets. However, in the leaderboard, there are not so clear. Does the result come from an average?

Yes, the leaderboard presents an averaged score.
On Mon, Dec 6, 2021 at 23:12 SakerLiu @.***> wrote:

In your paper, you have tested on Img-based and Txt-based datasets.
However, in the leaderboard, there are not so clear. Does the result come
from an average?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/WebQnA/WebQA/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT2LVR7U53RBDOS5UDBQWKDUPWCQ5ANCNFSM5JQIJ5LA
.
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.

Many thanks...


calibration

Hi, I get instinct parameters from Matlab Calibration App, and each projection error is less than 0.3.
Then I use the instinct params to do the extrinsic calibration. I getpts 14 points in the same checkboard pattern between all cameras. Then I can get the rotation matrices and translation vectors after running the following code.
the code is based on the dannce/calibration/multi-camera calibration/calibration_extrinsic_Lframe.m.
Is it already enough to do hand-labeled markers then? In my experience, the result of hand-labeled is fine.
But I just found in your public script dannce/calibration/acquire_calibration_3cam_mouse_clean.m,
you also performed STEP 6 - refine with checkerboard and  STEP 7 compute the corrected world coordinates.
I am confused why you perform steps 6 and 7.
Thank you very much @davidhildebrand .

Steps 6 & 7 are optional and only need to be used if your calibration fit is subpar.


Upgrade to clojure future spec beta4

Upgrades baton's dependency on clojure-future-spec to 1.9.0-beta4.
The main thing is to migrate from referring to the old clojure.spec namespace to the new
clojure.spec.alpha namespace.
I also took the liberty of fixing a broken test 😉

I also took the liberty of fixing a broken test

This is why CI is so important, clearly.


Add the original SharedMap to experimental

Add the original SharedMap to @fluid-experimental package and remove the contents related to SharedDirectory, will add the attribution-related logic in next step.

I suggest rewording the PR title. It confused me a little before I looked at it closer. Maybe something like, "move SharedMap attribution to experimental scope"?


TWW worldquests planned ?

Is TWW content planned for Broker_WorldQuests ?

Added zone IDs for first four TWW zones is already there, but testing is limited for a few more days ;)

Thank you. I reached lev 80 but it still tells me "you need to unlock The War Within World Quests on one of your characters"
Do you know if there's another requirement to unlock it ?

if (expansion == CONSTANTS.EXPANSIONS.THEWARWITHIN) then
			hasUnlockedWorldQuests = UnitLevel("player") >= 78 and IsQuestFlaggedCompleted(84022) and IsQuestFlaggedCompleted(80592)	-- https://www.wowhead.com/beta/quest=84022/uniting-severed-threads and https://www.wowhead.com/quest=80592/forge-a-pact

guess we have a quest to do again :)

Well guessed ! 1st quest is done but not the 2nd one. It's supposed to be a weekly meta quest and I'm pretty sure it's not available during the early access.

I just unlocked world quests tonight myself, so I was able to update for basic support.   I will add zones and currencies as I unlock them.


Fix Raspberry Pi freeze by adding alpha > 0 check in shader

Somehow the Raspberry Pi freezes if we don't check for color.a > 0.0  in the colorTransform part of the fragmentShader. This is weird because it even happens when there is no colorTransform in use.
Setting gl_FragColor = vec4 (0.0, 0.0, 0.0, 0.0) in an else statement doesn't seem to be necessary.

Sorry, I couldn't commit from my Pi, and made changes on my Mac but in the wrong spot...
Making a new pull request ...


Add orchestrator and support for distributed outsourced computation

Adds support for an intermediate RPC orchestrator
Adds support for rabit.init() and rabit.finalize() over RPC
Enables distributed outsourced computation over RPC

Closing this PR as it's not ready yet. Will open a new one when ready.


Issue with encoding

I'm trying to use lua-toml on Alpine Linux and have issues with encoding:
local toml = require('toml')

local foo = {}
foo.test = { "one", "two", "three" }
local bar = toml.encode(foo)
print(bar)

results in
test = [
]

Besides this problem, lua-toml can't encode nested sequences either it seems. As in:
toml.encode {x = { a = {{1,2}, {3,4}} } }


Force rpc_address to use ws as the protocol to avoid using ICE

Explain the changes

Force rpc_address to use ws as the protocol to avoid using ICE
Fixing the agent port to 60101

As decided we will leave the ICE code in for now and we will do a cleanup once we are 100% about it.
We want to clean the code in a later version, probably at 4.11
Signed-off-by: liranmauda<EMAIL_ADDRESS>

Moving to draft.
2 topics here:

We need to decide what to do with the agent authentication. we will have a talk regarding @guymguym suggestion
Unit tests are failing as we scaling the agent locally and with this change it is failing on rpc_adress which was hardcoded into the same port (60101)

let's have a discussion on that and see how we want to advance with this PR with regards to also moving to node 16

Open a Jira story: https://issues.redhat.com/browse/MCGI-126
Closing.


Tooltips to buttons

Add tooltips with short explanations to these buttons so people understand easier what they're about.

Was added in alpha 5.


Provide a consistent interface for specifying nullary constraints

Closes #473
Implements the following api for specifying nullary constraints (no additional argument needed, e.g., :odd) and unary constraints (additional argument needed, e.g., gt: 18).
type.constrained(:odd) # new variant
type.constrained(:odd, :even) # new variant
type.constrained(:odd, gt: 200, lt: 300) # new variant

The above is covered by new tests and updated yardocs. If this api is acceptable, I will update the docsite accordingly.
Addenda
The api also accepts the following for backward compatibility. However, I suggest deprecating this behavior as non-intuitive and dependent on a quirk of the former implementation (#map). Deprecating will allow for the removal of an otherwise unnecessary application of Array#flatten in processing arguments.
type.constrained([:odd, :even]) # worked before, continue to support
type.constrained([:odd, :even], gt: 200, lt: 300) # new variant, supported as a side-effect

The following also work, although I don't believe they are covered by tests. The last two are no-ops.
type.constrained # error, as before
type.constrained([]) # worked before, continue to support
type.constrained({}) # worked before, continue to support

Finally, the following also work as undocumented behavior.
type.constrained({gteq: 0, lt: 100}) # undocumented, not encouraged
type.constrained({gteq: 0}, lt: 100) # undocumented, not encouraged

Potential Issues

Dry::Types::Sum.constrained (on line 89) does not double-splat the options Hash used to call Constrained#constrained, which means that it relies on the "undocumented" behavior above. Not a big deal, but I would consider adding ** for clarity or, even better, using argument forwarding (...).

Dry::Types::Builder.constrained does not implement the above api. Although this does not break current tests, this should probably be modified (with tests) to be consistent. Maybe just use argument forwarding here as well?

type.constrained([]) # worked before, continue to support
type.constrained({}) # worked before, continue to support

I think we should tweak it to raise an exception in 2.0

Dry::Types::Sum.constrained (on line 89) does not double-splat the options Hash used to call Constrained#constrained, which means that it relies on the "undocumented" behavior above. Not a big deal, but I would consider adding ** for clarity or, even better, using argument forwarding (...).
Dry::Types::Builder.constrained does not implement the above api. Although this does not break current tests, this should probably be modified (with tests) to be consistent. Maybe just use argument forwarding here as well?

It should be safe to use ... there and it would be good to do it already in this PR :)

I think we should tweak it to raise an exception in 2.0

Such calls can occur when dry-types is wrapped inside another lib.

type.constrained([]) # worked before, continue to support
type.constrained({}) # worked before, continue to support

I don't have an opinion either way on keeping this, but if it's important it should be covered by tests. I don't think it is.

I have changed Dry::Types::Builder#constrained and Dry::Types::Sum#constrained to use argument forwarding. All tests remain green.
Issues

I don't know how to represent argument forwarding in Yardoc, so I didn't touch those.
Passing an empty Array or Hash to #constrained doesn't really work (before or after the proposed changes).

Yardoc
I found this example of argument forwarding in the codebase (for another invocation of #constrained no less!), but I wasn't sure if this is the canonical way to represent argument forwarding in Yardoc (as an [Array]). It looks suspect to me. :monocle_face:
      # Build a constrained type
      #
      # @param [Array] args see {Dry::Types::Builder#constrained}
      #
      # @return [Default]
      #
      # @api public
      def constrained(...)
        type.constrained(...).default(value)
      end

Empty Array/Hash argument
Passing an empty Array or Hash to #constrained does not cause an explicit error either before or after the proposed changes. However, they both actually contain a subtle bug.
Entering Types::Integer in IRB produces:
=> #<Dry::Types[Constrained<Nominal<Integer> rule=[type?(Integer)]>]>

Entering Types::Integer.constrained([]) or Types::Integer.constrained({}) should logically produce the same result. However, they actually both produce the following:
=> #<Dry::Types[Constrained<Nominal<Integer> rule=[type?(Integer) AND ]>]>

That dangling AND does not seem to effect the functionality of the Type object, but it does break the AST parser. That is, the following works:
irb(main):005:0> Types::Integer.to_ast
=> [:constrained, [[:nominal, [Integer, {}]], [:predicate, [:type?, [[:type, Integer], [:input, Undefined]]]]]]

But the following causes an error:
irb(main):005:0> Types::Integer.constrained([]).to_ast

This bug stopped me from adding tests for empty Arrays/Hashes, since the test suite invokes #to_ast and then errors out.
Path forward
Please let me know how you would like for me to proceed. I would be happy to update the Yardocs for the two methods I changed in whatever way you recommend.
As for the bug, do you want to address that in this PR, or separately? It might be as easy as adding a guard clause that checks for rules.empty? before calling #with, but I don't know the proper value to return from the guard clause.
Guidance is appreciated if you would like for me to investigate further.

@dcr8898 this looks good overall, I'll now merge it as it is and run my apps against it. If everything's fine, I'll return to the YARD nuances. Thank you for working on this! ❤

So far, it works well, I'll look into YARD issues soon


Viewport fails to clip its AssetImage child

We were getting confused about our layer tree offsets, which caused us
to set an incorrect cull rect for the recording inside a viewport.
This patch does away with layer tree offsets almost entirely. We now use
them only at repaint boundaries, which is where we want the layer tree
to be mutable.
Fixes #1460

@HansMuller

I'm not claiming to completely understand the code that was changed, but LGTM.


Added --preserve-host option to retain host header in upstream request.

Useful for upstream webapps that rely on the host header, for instance to generate weblinks that match the host originally requested.
Similar to the Apache mod_proxy ProxyPreserveHost option: https://httpd.apache.org/docs/trunk/mod/mod_proxy.html#proxypreservehost
If a host header is set in the headers option, --preserve-host is ignored.

Rebased from master to resolve merge conflicts (and keep this PR to a single commit).


chore: make artificially shortened executions count as successes

Currently, successful validations are reported as reverts, which is super confusing.
Why this is safe:

TracerExecutionStopReason::Finished is only emitted on success, Errors always emit Abort instead
Some of the removed code was just dead code. self.result was checked twice.

I'd rather fix the strange behaviour than emulate it to make ShadowVm line up.

Turns out that TracerExecutionStopReason::Finished is produced when a transaction ends even though the tracer can't know at that point if the bootloader ran out of gas before finishing the whole thing. In that case the old code that reports "maybe out of gas" is correct but we need to run it only in that special case.
I'm not sure what the best way to accomplish that is. We could maybe change TracerExecutionStopReason to have a success, failure and tx ended variant. I think I prefer giving DefaultExecutionTracer enough info to be able to detect failure.
Also, it would be nice if the bootloader had "did not finish" as the default value of the result instead of failure.

I ended up with a pretty hacky fix. I would have preferred to make the validation tracer tell the result tracer that there has been no out of gas but that isn't done as easily.


Add BIOS/BMC versions to redfish

Signed-off-by: Micah Hausler<EMAIL_ADDRESS>What does this PR implement/change/remove?
Adds BMC and BIOS version APIs to Redfish provider
Checklist

[ ] Tests added - gofish is not easily testable
[x] Similar commits squashed

The HW vendor this change applies to (if applicable)
Redfish
The HW model number, product name this change applies to (if applicable)
N/A
The BMC firmware and/or BIOS versions that this change applies to (if applicable)
N/A
What version of tooling - vendor specific or opensource does this change depend on (if applicable)
Description for changelog/release notes
Added redfish BIOS/BMC versions

hey @micahhausler in https://github.com/bmc-toolbox/bmclib/pull/261/commits/785f24d9be431117c7c72494600a4450feb1dda7 the GetBMCVersion(), GetBIOSVersion() methods are replaced by the Inventory() method https://github.com/bmc-toolbox/bmclib/pull/261/commits/5846f9219a760663c270022fe02f1e208d213685
If we think the GetBIOSVersion() GetBMCVersion() methods are useful to have just by themselves,
we can add those back in.

ah makes sense, I'll just drop this PR then


Clang tidy

Codecov Report

Merging #23 (164ce4e) into master (64a8df1) will increase coverage by 0.03%.
The diff coverage is 66.00%.

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
+ Coverage   54.04%   54.06%   +0.03%
==========================================
  Files          80       80
  Lines        7692     7699       +7
==========================================
+ Hits         4156     4162       +6
- Misses       3536     3537       +1

Impacted Files
Coverage Δ

...oveit/task_constructor/solvers/planner_interface.h
100.00% <ø> (ø)

core/include/moveit/task_constructor/storage.h
87.92% <0.00%> (-0.97%)
:arrow_down:

core/src/solvers/cartesian_path.cpp
82.93% <0.00%> (-2.07%)
:arrow_down:

core/src/solvers/pipeline_planner.cpp
67.75% <0.00%> (ø)

core/src/storage.cpp
84.32% <0.00%> (ø)

...alization/motion_planning_tasks/src/task_panel.cpp
0.00% <0.00%> (ø)

...sualization/motion_planning_tasks/src/task_panel.h
0.00% <ø> (ø)

...on_planning_tasks/utils/flat_merge_proxy_model.cpp
70.87% <ø> (ø)

...on_planning_tasks/utils/tree_merge_proxy_model.cpp
70.08% <ø> (ø)

core/src/stage.cpp
82.64% <40.00%> (ø)

... and 8 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[1.18.2] Crash: Exception Ticking World

I do not know what happened but I do have a crash report:

crash-2023-04-08_20.46.33-server.txt

Mod version:

DynamicTrees-1.18.2-0.11.0-Beta9
DynamicTreesBOP-1.18.2-3.0.0-Beta3

Modloader:

Forge 40.2.1

I'm glad I am not the only one facing this problem. It seems the issue is caused by:
java.lang.NoSuchMethodError: 'boolean com.ferreusveritas.dynamictrees.block.rooty.SoilHelper.isSoilAcceptable(net.minecraft.world.level.block.state.BlockState, int, boolean)'


Undefined Offset in bodystructure.php

After update to debian 12, rainloop clients stuck after log in because message list cannot be created.
Error message: "Can't get message list" is shown instead.
New mails can be created and sent.
In apache error log, an undefined offset error in function getKeyValueListFromArrayList in  bodystructure.php is shown.
Steps done so far:
Re-Install PHP 7.4 (Sury)
Update Rainloop to 1.17
RainLoop version, browser, OS:
rainloop 1.17
debian 12
php 7.4
any browser
Expected behavior and actual behavior:
show message list is expected
error "Can't get message list" is shown instead
Steps to reproduce the problem:
log in on any browser
Logs or screenshots:
From apache log:
[Mon Aug 07 09:40:30.622987 2023] [php7:notice] [pid 3371736] [client <IP_ADDRESS>:61712] PHP Notice:  Undefined offset: 12185306 in /var/www/rainloop/rainloop/v/1.17.0/app/libraries/MailSo/Imap/BodyStructure.php on line 954
[Mon Aug 07 09:40:30.623015 2023] [php7:error] [pid 3371736] [client <IP_ADDRESS>:61712] PHP Fatal error:  Maximum execution time of 30 seconds exceeded in /var/www/rainloop/rainloop/v/1.17.0/app/libraries/MailSo/Imap/BodyStructure.php on line 952

After complete de- and re-install of php7.4, the problem doesn't occure anymore.


PODs no deleted after finished

Finished pods are not deleted.
Example:
k -n airpod get pods
NAME                    READY     STATUS        RESTARTS   AGE
ovc-798fae72            0/1       Completed     0          1h
ovc-88b69c86            0/1       Error         0          1h
ovc-8dcc9b6e            0/1       Error         0          2h
ovc-9dc87852            0/1       Error         0          1h
ovc-bc31f0f2            0/1       Error         0          1h
ovc-bf22340e            0/1       Error         0          2h
ovc-e57b7b00            0/1       Error         0          2h
passing-test-0025e10c   0/1       Completed     0          18h
passing-test-358d9656   0/1       Completed     0          18h
passing-test-391387f4   0/1       Completed     0          3h

wrong repo


Develop

Coverage increased (+0.005%) to 83.192% when pulling 23323ae66534735c7ff6c78fba64158009b24cd7 on develop into f0899fd50931d2f3d18fd64c51f5630ec117cef2 on qa.


cgdb: fix compilation with GCC14

This pull request adapts cgdb to compile with GCC14.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@fpletz seems like the merge could've waited till finished reviewing.

@paparodeo Sorry, the reponse of @pks-t sounded reasonable to me and at least the fixes the build. We have lots of these small gcc14 fixes and IMHO it doesn't make sense to waste time on searching for the perfect solution (i.e. upstream patch) for every package.


Get CompanyID and Manufacturer Data on discovery callback

Hello! First of all I'd like to thank you for the awesome work you've done with this bluez_inc library. Thank you so much!
Ok, here is the issue I'm struggling with and hopefully you can help me with: I'd like to parse as a byte array the manufacturer data that is included within the advertisement packet from a peripheral device, but the binc_device_get_manufacturer_data method returns a pointer to a GHashTable which honestly I don't know how to process. What is the key that I should look for in that hash table so I can check the Company ID that is advertised? How can I parse the actual manufacture data that is included, is it a key or a value in the hash table? How is the data inserted in that hash table: as string or as byte array?
Thank you in advance

https://docs.gtk.org/glib/struct.HashTable.html
Use one of the GHastable functions to get the keys

The key is a *int and the value is a GByteArray


[BUG] StatefulSet schemahero has too much RBAC permission which may leads the whole cluster being hijacked

Hi community! Our team just found a possible security issue when reading the code. We would like to discuss whether it is a real vulnerability.
Description
The bug is that the StatefulSet schemahero in the charts has too much RBAC permission than it needs, which may cause some security problems, and the worse one leads cluster being hijacked. The problem is that the schemahero doesn't specify a ServiceAccount, so it uses the default ServiceAccount, which is bound to a clusterrole(cluster-role.yaml#L4) with the following sensitive permissions:

create/patch/update verb of the deployments resource (ClusterRole)
patch verb of the statefulsets resource (ClusterRole)

After reading the source code of schemahero, I didn't find any Kubernetes API usages using these permissions. However, these unused permissions may have some potential risks:

create/patch/update verb of the deployments resource

A malicious user can create a privileged container with a malicious container image capable of container escape. This would allow him/she gaining ROOT privilege of the worker node the created container deployed. Since the malicious user can control the pod scheduling parameters (e.g., replicas number, node affinity, …), he/she should be able to deploy the malicious container to every (or almost every) worker node. This means the malicious user will be able to control every (or almost every) worker node in the cluster.
The other permissions of deployments/statefulsets also share the same risks.

The malicious users only need to get the service account token to perform the above attacks. There are several ways have already been reported in the real world to achieve this:

Supply chain attacks: Like the xz backdoor in the recent. The attacker only needs to read /var/run/secrets/kubernetes.io/serviceaccount/token.
RCE vulnerability in the APP: Any remote-code-execution vulnerability that can read local files can achieve this.

Mitigation Suggestion

Remove all the unnecessary permissions
Write Kyverno or OPA/Gatekeeper policy to:

Limit the container image, entrypoint and commands of newly created pods. This would effectively avoid the creation of malicious containers.
Restrict the securityContext of newly created pods, especially enforcing the securityContext.privileged and securityContext.allowPrivilegeEscalation to false. This would prevent the attacker from escaping the malicious container. In old Kubernetes versions, PodSecurityPolicy can also be used to achive this (it is deprecated in v1.21).

Few Questions

Are these permissions really unused for schemahero?
Would these mitigation suggestions be applicable for the schemahero?
Our team have also found other unneccessary permissions (which is not that sensitive as above, but could also cause some security issues). Please tell us if you are interested in it. We’ll be happy to share it or PR a fix.

References
Several CVEs had already been assigned in other projects for similar issues:

2023-30512
2023-30617
2023-26484
2023-30840

Hi community:
I noticed that this issue hasn't been answered yet, please feel free to let me know if you need more information. Our team is happy to cooperate and do what we can to resolve this issue.


Feature Request: Add Nix syntax highlighting

I often share nix configurations and would like to have syntax highlighting for them. Any chance the language could be added to the list?
Looked a bit at how to add languages but it wasn't obvious to me, if you give me any pointers how to do it I'll try to submit a PR myself

Nix should be in the list for a while now. https://bin.bloerg.net runs a slightly older version at the moment but it has nix language support.

My bad, I was looking at an instance with an older version.


Adding Metrics as organization

In the last meeting we agreed to start the discussion about Metrics

I will implement code climate just to get some basic source code state metrics

This initiative has been completed.


refactor: move from deprecated ioutil to os and io packages

Since gopkg and CI are currently using Go 1.16, so it is safe to remove deprecated ioutil package, ref https://golang.org/doc/go1.16#ioutil.

Not sure if anybody using Go (version < 1.16) + bytedance/gopkg, in which case this PR may result in incompatibility. Guess bytedance/gopkg may want to define a minimum version level of supported Go SDKs?


Add haddock-library, haddock-api and haddock

This hopefully enables stack users to benefit from newer haddock releases.

Thanks, merged


Sidebar Tabs & Buttons Hide & Sort Order Sync

💡 Downloading the latest settings doesn't seem to persist my hidden sidebar buttons, sidebar hidden/shown tabs, and the sidebar buttons hidden or shown.
Given I initiate a sync
And I hide a sidebar <tab|button>
When I sync to another machine
Then the sidebar <tab|button> hides the selected tab

Given I initiate a sync
And I reorder the sidebar <tab|button>
When I sync to another machine
Then the sidebar <tab|button> reorders the tabs

Have you tried on restarting the code once the download is complete?


Add trimmed to all multilines blocktrans tags

Following what @ewjoachim said in #4471,

Now I think most of these problems are pretty much stemming from the usage of blocktrans without trimmed, and that makes the translation quite more complex to write. There are some other issues, though.

I updated all our templates to add trimmed to all multilines blocktrans tag.
This PR should be merged before #4478 I suppose, so we don't override the resources twice.

This PR was done by running these commands:
$ cd readthedocs/
$ find . -iname "*.html" -type f -exec sed -i -e 's/blocktrans trimmed/blocktrans/gi' {} \;

$ find . -iname "*.html" -type f -exec sed -i -e 's/{% blocktrans/{% blocktrans trimmed/gi' {} \;

:+1: