[$500] Distance - Amount shows 0.00 in the confirmation page when tax is enabled on workspace

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

Version Number: 1.4.23-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Precondition: User is an employee of a Collect workspace on Old Dot which has tax enabled

Go to workspace chat
Click + > Request money > Distance
Enter addresses and proceed to confirmation page

Expected Result:
The confirmation page shows the correct Amount
Actual Result:
Amount shows 0.00 in the confirmation page
Workaround:
Unknown
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/9dcc2730-006f-418e-990d-1d0705b20cd9
View all open jobs on GitHub

@tienifr Thanks for the proposal. Your RCA makes sense but I'm not sure if that RCA is for this issue. I tried to reproduce this but we seem to always make an API request OpenDraftWorkspaceRequest to retrieve the rates where in the OP video the rates didn't update as if we didn't make any request. Were you able to reproduce the bug?
https://github.com/Expensify/App/assets/16493223/52e2caf3-cae4-40f6-bb4f-5bd1751e39c6

@tienifr I agree we should fix the briefly $0.00 but let's also make sure we are not leaving any bug behind

Waiting for reproduction confirmation cc @lanitochka17

Issue is not reproducible on the latest build 1.4.25-1
https://github.com/Expensify/App/assets/78819774/a0a2cf0b-91f1-4d8b-b935-3f30afa74d73

Still discussing ^

No replies yet, bumped the slack thread once again

We can close this. The brief $0.00 will be fixed by https://github.com/Expensify/App/pull/34135 cc @NicMendonca


add default buffer initialisation

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (96054ab) 41.67% compared to head (31df23a) 43.43%.
Report is 51 commits behind head on master.

Files
Patch %
Lines

clic/src/openclbackend.cpp
50.00%
0 Missing and 2 partials :warning:

clic/src/array.cpp
0.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #206      +/-   ##
==========================================
+ Coverage   41.67%   43.43%   +1.76%
==========================================
  Files          14       14
  Lines        2702     2703       +1
  Branches     1166     1158       -8
==========================================
+ Hits         1126     1174      +48
+ Misses        830      767      -63
- Partials      746      762      +16

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[Docs] ELI5 Tutorial For Invocations

Work in progress. Just putting it out there to see if its worth pursuing writing indepth detailed guides.
Can be previewed here: https://github.com/blessedcoolant/InvokeAI/blob/docs-invocations/docs/contributing/INVOCATIONS.md

This type of documentation is going to be critical. Thanks for putting it together. I've given it a once over, but will get perspective of @brandonrising next week on how clear it is to fresh eyes, and where we can expand.

This looks great! Definitely valuable and worth investing in. One question for the advanced section probably, are invocations allowed to use InvokeAIAppConfig? Any best practices there?

self.is_intermediate is used but never defined

self.is_intermediate is used but never defined

Yes. I'll update this when I'm refining the PR later. I'm just waiting on all the services to settle in. self.is_intermediate is used to determine an intermediate image. Not applicable in this case.

are invocations allowed to use InvokeAIAppConfig

Basically we'll want the dev to access the config via the service rather than trying to instantiate their own class of InvokeAIAppConfig .. so it should be context.services.configuration.xxxx to access any of the data provided by the app config.

@blessedcoolant - Thoughts on the status/timing of this doc? Have it on the list of things to have ready for 3.0 launch

The docs are good for the backend stuff. They front end stuff is still WIP. But that's a big write up. So don't really have an ETA on that.

The docs are good for the backend stuff. They front end stuff is still WIP. But that's a big write up. So don't really have an ETA on that.

Ok - should we merge this in then? Or can merge over to the doc-updates branch if you want


add plottools documentation

I added the PLOTTools.dataPlotters documentation and ensured that the website can be built locally.
However, I am not sure how to modify the README.rst in the root folder (#3), as the usage instructions in the README are entirely for the analysis tools. If I include the usage for PLOTTools, will it become too lengthy?

Good job

It's a good point. Perhaps we can have a chat about what to put in readme file.
Maybe the github page should only have the introduction and installation and we can update the website homepage with examples or something else.


update the unit test of uniteddeployment

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind enhancement

What this PR does / why we need it:
this pr is used to improve the unit test coverage .
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

/lgtm
/approve


Docker login to use --password-stdin for increased security

What current issue(s) does this address, or what feature is it adding?
For security reasons Docker login must use --password-stdin.
See: https://docs.docker.com/engine/reference/commandline/login/#provide-a-password-using-stdin
How did you solve this problem?
Changed the line to pipe the password to docker login command
How did you make sure your solution works?
Totally tested by many people
Are there any special changes in the code that we should be aware of?
No
Please check the following, if applicable:

[ ] Did you add any tests?
[ ] Did you run make lint?
[ ] Did you run make test?
[ ] Are you making a PR to a feature branch or development rather than master?
[ ] Did you add an entry to CHANGELOG.rst? (if not, please do)

Coverage remained the same at 80.26% when pulling c75039c8726ad79036a6c9b60c5f59aa5ec28d87 on nunojusto:development into 6104077d18bfc48409302d94b0e5d49b17cbca74 on CityOfZion:development.

@metachris this look ok?

Well, maybe it's not so important if you are using it locally on you own pc. But in a shared server it's important. And also, this removes the warning message that docker outputs on Travis. Look at logs


not able to delete printers or add printers from outside local sources

Hi, I am not able to delete printers or add printers from outside local sources. ip and port number is good because I can open it in my browser. Thanks

Hey @psdavidson812 , are you still seeing this issue? What do you mean by local sources? Is it your local network?

Yes, I am not able to delete my printer nor am I able to edit printers.
On Sun, Aug 29, 2021, 2:08 PM Gaston Dombiak @.***>
wrote:

Hey @psdavidson812 https://github.com/psdavidson812 , are you still
seeing this issue? What do you mean by local sources? Is it your local
network?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gdombiak/OctoPod/issues/510#issuecomment-907841025,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATIIJIEHSACTGBT7UQ5R7OLT7JZRTANCNFSM5DAPEYJQ
.
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.

Could you check under Settings -> Security that read-only is disabled?

Bingo that was it!!!    Thanks that’s been driving me crazy!!
On Sun, Aug 29, 2021 at 3:59 PM Gaston Dombiak @.***>
wrote:

Could you check under Settings -> Security that read-only is disabled?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gdombiak/OctoPod/issues/510#issuecomment-907861410,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATIIJIEJVFT3WS5DOIHUHBDT7KGS7ANCNFSM5DAPEYJQ
.
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.

Excellent! Another person just run into the same issue. I will improve the UI to give a hint about what is going on so users can troubleshoot this by themselves. Thanks for helping troubleshoot this.
Gaston


ref(workflow): the value of any key in env must be a string

This is related to the error in #297 where one value was a number
          }, {
            "name": "DOCKERIMAGE",
            "value": "1"
          }, {
            "name": "NODE_ENV",
            "value": "production"
          }, {
            "name": "REDIS_PORT",
            "value": 6381
          }, {
            "name": "REDIS_HOST",
            "value": "<IP_ADDRESS>"
          }],

@mboersma is ok to merge?

ok to merge?

@aledbf yes please. Thanks for the fix!


 The other nodes all seem to be hung. When I try to access I just see a bunch of red herring errors in the log.
2015-05-18 16:34:35:380 WARNING Distributed server is not yet ONLINE (status=STARTING), reject incoming connection from /<IP_ADDRESS>:59393. If you are trying to shutdown the server, please kill the process [OServerNetworkListener]
2015-05-18 16:34:35:484 WARNING Distributed server is not yet ONLINE (status=STARTING), reject incoming connection from /<IP_ADDRESS>:59395. If you are trying to shutdown the server, please kill the p

I've let it sit for quite some time and it never leaves the STARTING state. Then I made the mistake of restarting the first node and it lapsed into that state as well.
Also of interest perhaps:
2015-05-18 16:34:25:757 WARNING [node1431965853488] changing mastership of cluster 'e_node1431965853488' from node 'node1431965853518' to 'node1431965853488' [OHazelcastDistributedDatabase]
2015-05-18 16:34:25:757 WARNING [node1431965853488] changing mastership of cluster 'ouser_node1431965853488' from node 'node1431965853518' to 'node1431965853488' [OHazelcastDistributedDatabase]
2015-05-18 16:34:25:758 WARNING [node1431965853488] changing mastership of cluster 'oschedule_node1431965853488' from node 'node1431965853518' to 'node1431965853488' [OHazelcastDistributedDatabase]
2015-05-18 16:34:25:758 WARNING [node1431965853488] changing mastership of cluster 'ofunction_node1431965853488' from node 'node1431965853518' to 'node1431965853488' [OHazelcastDistributedDatabase]
2015-05-18 16:34:25:758 WARNING [node1431965853488] changing mastership of cluster 'v_node1431965853488' from node 'node1431965853518' to 'node1431965853488' [OHazelcastDistributedDatabase]
2015-05-18 16:34:25:759 WARNING [node1431965853488] updated distributed configuration for database: GratefulDeadConcerts:
----------
{
  "version": 16,
  "autoDeploy": true,
  "hotAlignment": false,
  "executionMode": "undefined",
  "readQuorum": 1,
  "writeQuorum": 2,
  "failureAvailableNodesLessQuorum": false,
  "readYourWrites": true,"clusters":{
    "internal": {
},
    "index": {
},
    "*": {
  "servers": ["node1431965853518","node1431965853488","<NEW_NODE>"]
},
    "e_node1431965853488": {
  "servers": ["node1431965853488","node1431965853518","<NEW_NODE>"]
},
    "ouser_node1431965853488": {
  "servers": ["node1431965853488","node1431965853518","<NEW_NODE>"]
},
    "oschedule_node1431965853488": {
  "servers": ["node1431965853488","node1431965853518","<NEW_NODE>"]
},
    "ofunction_node1431965853488": {
  "servers": ["node1431965853488","node1431965853518","<NEW_NODE>"]
},
    "v_node1431965853488": {
  "servers": ["node1431965853488","node1431965853518","<NEW_NODE>"]
}
    }
}
---------- [OHazelcastPlugin]
2015-05-18 16:34:25:761 INFO  [node1431965853488] Saving distributed configuration file for database 'GratefulDeadConcerts' to: /opt/orientdb-community-2.0.9/databases/GratefulDeadConcerts/distributed-config.json [OHazelcastPlugin]
2015-05-18 16:34:29:760 INFO  [node1431965853488] received updated status node1431965853488.GratefulDeadConcerts=SYNCHRONIZING [OHazelcastPlugin]
2015-05-18 16:34:29:766 WARNING [node1431965853488]->[[node1431965853518]] requesting deploy of database 'GratefulDeadConcerts' on local server... [OHazelcastPlugin]
2

using v 2.0.9

did the same thing on 2.0.5, sort of.  First and third nodes came up fine, others did not

If I bring them up in a more controlled manner, i.e. one at a time over the span of about 20 minutes, things work better.

Looks like this is probably same issue as #3986

Could be, I definitely didn't see the same errors though.

Yes, it was the same as #3986. Fixed


CSS Audit

Rancher versions:
rancher/rancher
Goal:
Audit css base for unused or redundant code.
-Maybe consolidate organization.
-Suggestion from @westlywright , check usage of dark theme

closing - part of redesign


release-checklist.md: updates and improvements

Making appropriate changes which are shared from ignition release https://github.com/coreos/ignition/issues/1353

While you're here, could you drop the "submit a fast-track" checklist entries?  Butane isn't in FCOS so this doesn't make sense here.


Validation issue on Canggu

Dear @m-schoettel,
Your first <p> tag element isn't closed. I think, it should be done at the end of the XML line 83 after
<lb n="1r3" break="no"/>śa, nāgakaraṇa, karkkaṭarāśī,

Can you check this please?
Best,

Thanks for spotting this validation issue. I was just working on Canggu and made the necessary change. Will push the file at the end of the work session.

@m-schoettel : good. In the future, when you have done the necessary for an issue, feel free to close it yourself. I am doing do now.


z_arm_platform_init silently fails to be called following #62595

Describe the bug

#62595 has renamed the function z_arm_platform_init soc_reset_hook (https://github.com/zephyrproject-rtos/zephyr/pull/62595/files#diff-da3175d625f6d13072c4f16348168e9b326d011de36fc65724821d3369774628R310) . I think this is breaking change in the kernel's API.
In the Kconfig file at arch/arm/core/Kconfig

The platform specific initialization code (z_arm_platform_init) is executed at the beginning of the startup code (__start)

This is not true: this function is never called by the kernel. Anyone who's z_arm_platform_init is implemented out-of-tree (or, in my case, rebased their changes on top of the current HEAD of main), will (potentially) only find out their function is not called if they've dug into this.
To Reproduce
See steps above.
Expected behavior

I think this API change hasn't gone through the cycle documented here https://docs.zephyrproject.org/latest/develop/api/api_lifecycle.html#api-lifecycle
This API change should be mentioned in the release notes or migration guide (I feel like this should have been a blocker to the PR being approved).
https://docs.zephyrproject.org/latest/develop/api/api_lifecycle.html#deprecated seems to have a clearly defined approach to handling deprecation.

Impact

Annoyance
Unexpected failures at run time unless you've studied the code to find this error.
Out-of-tree users are going to have a bad time.

https://github.com/zephyrproject-rtos/zephyr/pull/77218 got merged after https://github.com/zephyrproject-rtos/zephyr/pull/62595 , and is trying to use the old API https://github.com/zephyrproject-rtos/zephyr/pull/77218/files#diff-60f87748dca24415169890afa327a502fd7cb5f939df043f87ad76f2da05dbeeR22 . It's in the current HEAD of main https://github.com/zephyrproject-rtos/zephyr/blob/fd4f3ce246c9ce1ddbd04b20054ebca3b9bbdc6f/soc/nxp/mcx/mcxa/soc.c#L22 . I don't think this is going to work. cc: @NeilChen93 , have you tried using this new SoC/board with the current HEAD of main?

I think there's more confusion in the logic here

config PLATFORM_SPECIFIC_INIT
	bool "Platform (SOC) specific startup hook [DEPRECATED]"
	default y if SOC_RESET_HOOK
	select DEPRECATED
	help
	  The platform specific initialization code (z_arm_platform_init) is
	  executed at the beginning of the startup code (__start).

	  This option is deprecated, use SOC_RESET_HOOK instead.

If I do use SOC_RESET_HOOK, then the default y if SOC_RESET_HOOK results in PLATFORM_SPECIFIC_INIT being selected, so the build system complains:
...
-- Including generated dts.cmake file: C:/Users/andre/projects/zephyrproject/zephyr/build/rpi_pico2/blinky/zephyr/dts.cmake
Parsing C:/Users/andre/projects/zephyrproject/zephyr/Kconfig
Loaded configuration 'C:/Users/andre/projects/zephyrproject/zephyr/boards/raspberrypi/rpi_pico2/rpi_pico2_defconfig'
Merged configuration 'C:/Users/andre/projects/zephyrproject/zephyr/samples/basic/blinky/prj.conf'

warning: Deprecated symbol PLATFORM_SPECIFIC_INIT is enabled.

Configuration saved to 'C:/Users/andre/projects/zephyrproject/zephyr/build/rpi_pico2/blinky/zephyr/.config'
...

I imagine this is true for any board using SOC_RESET_HOOK.

@nashif Is there a plan to document this API change to the kernel?


[dx11][pre-ll] Partial updates of dynamic resources are broken

In D3D11, dynamic resources can only be mapped with either NO_OVERWRITE or DISCARD flags. The former only works on vertex/index buffer. The latter discards the previous contents, thus not supporting partial updates. This is what we do now, and it results in user data being dropped.
Problem-1: we do not detect/handle this properly. At the very least, we should return an error.
Problem-2: we need to re-evaluate the whole Dynamic usage (only for pre-ll branch). D3D11 doesn't actually allow partial updates to happen to GPU-visible resources (like GL does). You can only do this via staging resources, which can be mapped for writing, then copied to GPU-visible resources (basically, PBO path in GL).
cc @zakorgy @Bastacyclop

Thinking about this more, I think I got all the puzzle pieces. Core thing to realize was that the data buffer carried by Encoder is conceptually (and should be - factually) a PBO:

the encoder maps a PBO for the duration of command encoding
whenever the user issues update_buffer/update_texture:

we'll copy data data into the mapped region
we'll record the underlying copy_buffer/copy_buffer_to_texture call in the command buffer

This way, we don't rely on any discard mechanics in the backend, and we don't need a tricky Dynamic type usage at all. The only problem that comes to mind is that the Encoder doesn't know the size of the PBO it will need, and it doesn't have access to a factory for creating more of them or resizing. Thus, I believe the user will just need to specify the size of associated PBO when creating an encoder.
@msiglreith would you see any problems implementing this scheme in the gfx_render to work with LL?

That being said, we need an easier workaround for pre-ll branch. It's sad to have this broken on DX11.

would you see any problems implementing this scheme in the gfx_render to work with LL?

It's actually also a core issue https://github.com/gfx-rs/gfx/issues/1416#issuecomment-320630776. Depending on if we want support in buffer updates.
For render this sounds reasonable in combination with a small buffer for the whole encoder, but the current GraphicsEncoder in render is only a temporary wrapper/encoder. Therefore we would need some changes here to handle a persitent scratch buffer.

More in-depth investigation here - https://github.com/gpuweb/gpuweb/issues/28

@kvark I never liked the Dynamic usage that was not part of my initial proposal and only reintroduced for GL_DYNAMIC_STORAGE_BIT and D3D11_USAGE_DYNAMIC (the broken part). I remember we already discussed the PBO solution at the time and the buffer size was not the only problem. Maybe it had something to do with the unability to keep the Encoder Send with it ?
I think I would rather have some kind of UploadQueue (a step towards higher level abstractions as discussed for mapping overwrites) in render that is basically the PBO solution without bringing the latter inside the Encoder. It probably requires more fine-grained synchronisation but would reduce waste (multiple fences for a single buffer that can be used by multiple encoders), which shouldn't be a problem with LL (@msiglreith ?). For pre-LL it could just be a ring buffer of upload buffers with more waste.

Let's try to clarify the situation a bit more.
The proposed fix is to keep an Upload buffer inside the encoders, with a fixed size specified at creation.
I don't know what are the multithread constraints for D3D11 but with OpenGL we can't acquire (and release) the Upload mapping outside of the GL context thread.
With pre-LL this is a problem since we want encoders to be Send. I see two solutions:

Ensure that the mapping is always available/acquired when the Encoder  is available (i.e. outside submission).
Keep writing into an intermediary Vec before actually copying to an Upload buffer during submission.

The first solution requires less copies. Releasing the mapping before submission if necessary is not a problem, but we also need to ensure the mapping is available after submission which means eagerly synchronizing. This might not be a problem if this synchronisation is smart enough (e.g. we don't want to wait for a following draw command) and does not cost too much.
The second solution seems easier to implement and does not require an Upload buffer for each Encoder. This is the one I would pick for a simple pre-LL fixing.
We could completly remove the Dynamic usage or keep it if we don't want breaking changes, or to modify less backends. I also fear #1208 might complicate things for D3D11 texture updates.
With LL the encoders will be neither Sync nor Send so there should be no thread issue. However without Send there is little convenience provided by update commands IMO. So I would not even emulate them and instead provide better upload abstractions mentioned in the last comment.
@kvark What do you think about all this ?

@Bastacyclop Thank you for the analysis!
pre-LL
We could also consider solution 3) map-write-unmap directly in the Encoder::copy_* implementation.
Overall, I wouldn't worry too much about pre-LL fixing. We haven't promised a "Long Term Support" kind of service for it anyway.  If there is an easy-ish workaround, let's do it! Otherwise - I prefer to focus on post-LL, but will gladly review the patches and publish new versions.
post-LL

So I would not even emulate them and instead provide better upload abstractions mentioned in the last comment.

Getting Vulkan Portability is one of the strategic goals of LL. If we don't provide update_buffer in core, we'll have to emulate it on all platforms by the portability shim around gfx-rs. This would be less efficient on Vulkan than to call native update methods, so is undesirable.
As to how exactly D3D12/Metal backends are going to do it:

CommandPool implementation will manage a grow-able pool of Upload buffers
when encoder needs to update something, it requests the buffer space from the active command pool
buffer space returns to the pool upon resetting the command buffer

We could also consider solution 3) map-write-unmap directly in the Encoder::copy_* implementation.

This is not possible with OpenGL. I guess it is possible with D3D11 ?

@Bastacyclop yeah, our command buffers are backed by ID3D11DeviceContext, which can do Map/Unmap: https://msdn.microsoft.com/en-us/library/windows/desktop/ff476457(v=vs.85).aspx


Aliases with NULL for Source-Adress causes Mailman to stop working

In the current Version of the Tutorial of Thomas Leister is decribed that an Catch-All Adress has to be setup by setting the source_username to null.

This is not possible within mailman. So i descided to set the source_username to @ at first. Then u Updates the database rows within the sql shell.
After changing the souce_username to null the mailman stop working.

What worked for me: directly inserting an empty string as the source_username in the database. Mailman will just display this empty string as, well, an empty string so effectively you'll end up with a source address looking like @yourdomain.tld
NOTE: I use this forked version, but iirc nothing with aliases was changed: https://git.fridaysforfutu.re/websiteag/mailman

Thanks for the Comment but in fact this change will make the collect-all addresses unusable. However i've followed your suggestion and fixed the aliases script that postfix now uses an empty-string instead of null as catch-all identifier


Inspection and type of parameters

Hello Eduardo,
Thanks for the ploomber engine. I integrated it in my projects and it works well.
Is there a way to inspect a notebook to get the parameters needed by the notebook and also their types?
If that is possible, it would be nicer to get the schema definition if they are richer types like dataclasses or pydantic models.
Cheers,
Vijay

hi @vijayvammi,
There's currently no way to do this; I remember seeing something like this in papermill's codebase, so if you have time, we could port the code over here!
Should be pretty straightforward (just remember to add the license at the top of the file)
let me know if you're interested!


One too many in resource in Task resources inputs - tutorial?

In this tutorial
Is there not one too many inputs in this "Task" for deployment. logically there should not be any git in this task - still learning so I am probably wrong but check this:
apiVersion: tekton.dev/v1beta1
kind: Task
metadata:
  name: deploy-using-kubectl
spec:
  params:
    - name: path
      type: string
      description: Path to the manifest to apply
    - name: yamlPathToImage
      type: string
      description: |
        The path to the image to replace in the yaml manifest (arg to yq)
  resources:
    inputs:
      - name: source
        type: git
      - name: image
        type: image
  steps:
    - name: replace-image
      image: mikefarah/yq
      command: ["yq"]
      args:
        - "w"
        - "-i"
        - "$(params.path)"
        - "$(params.yamlPathToImage)"
        - "$(resources.inputs.image.url)"
    - name: run-kubectl
      image: lachlanevenson/k8s-kubectl
      command: ["kubectl"]
      args:
        - "apply"
        - "-f"
        - "$(params.path)"

To me there should not be any:

    inputs:
      - name: source
        type: git

But only:
    inputs:
      - name: image
        type: image

The git repository is needed since it holds the manifest file to be applied by the run-kubectl step.
The path param is used to specify the file path to the manifest to be applied. This file path corresponds the location of the manifest in the git repo.

There are certainly ways to not have to use this git PipelineResource to provide the manifest, but it just happens to be the way the tutorial Task was created.

Right, got it - thanks!


import cocktails programatically

[X] I have read the FAQ.

Describe the bug
There used to be a way to import cocktails from external websites directly from the backend. It has been removed from the README.md recently. However it doesn't work currently. I can import cocktails through the web UI. From the backend, such an error is thrown:
[2023-03-08 20:52:11] production.INFO: [IMAGE_SERVICE] Image created with id: 450
Undefined array key "sort

To Reproduce
docker exec -it bar-assistant php artisan bar:scrape https://tuxedono2.com/coco-no-coco-cocktail-recipe
Versions:
podman v4.4.1

Hi, with the newest version, this command (docker exec -it bar-assistant php artisan bar:scrape {url}) is not available anymore. Is it changed ?


FT-141 # Store templates by form name in template service

new classes to hold templates
tests
documentation

LGTM 👍


Add URL verification information

Under How do I get verified on Mastodon? there is no mention of the profile URLs verification process.
If someone adds a link to their Mastodon profile from the URL added to their profile, the linke will show up in a small green-shadowed box.

Thanks for reminding me. This is a very new feature and I'm reluctant to add information about it until more Mastodon servers have picked up the latest builds.

The "How do I let people know that I am who I say I am?" section covers this now.


[SPARK-48415][PYTHON] Refactor TypeName to support parameterized datatypes

What changes were proposed in this pull request?
1, refactor TypeName to support parameterized datatypes
2, remove redundant simpleString/jsonValue methods, since they are type name by default.
Why are the changes needed?
to be consistent with the Scala side
Does this PR introduce any user-facing change?
1, typeName is no longer a class method
2, type names changes:
Decimal(10, 2): decimal -> decimal(10,2)
DayTimeIntervalType(DAY, HOUR): daytimeinterval -> interval day to hour
YearMonthIntervalType(YEAR, MONTH): yearmonthinterval -> interval year to month
How was this patch tested?
ci
Was this patch authored or co-authored using generative AI tooling?
no

I think we should list all behaviour changes. typeName is sort of an API too

I think we should list all behaviour changes. typeName is sort of an API too

updated the user-facing change part, thanks

on second thought, let me try still keep it classmethod in superclass, but only change to normal method in those 3 types.

thanks @HyukjinKwon
merged to master


bug: gcds-pagination navigation links (next, previous and the different pages) do not work in webkit / safari

Prerequisites | Prérequis

[X] I have read the Contributing Guidelines. | J’ai lu les Lignes directrices sur les contributions.
[X] I agree to follow the Code of Conduct. | J’accepte de me conformer au Code de conduite.
[X] I have searched for existing issues that already report this problem, without success. | J’ai vérifié parmi les tickets (issues) existants que ce sujet n’avait pas déjà été soumis.

GC Design System Components Package and Version | Paquet et version des composants de Système de design GC
0.27.0
Current Behavior | Comportement observé
Clicking on any of the navigation links do not emit an onclick event or result in navigation
Expected Behavior | Comportement attendu
Navigation to the proper page
System Info | Information sur le système
Tested on iphone 14 and 16 with latest Safari and latest webkit on windows

Steps to Reproduce | Étapes pour reproduire le bogue
You can run the repro code below in the repro-webkit branch. No events get emitted on webkit. There is a patch on the branch already that fixes the problem. it can be applied by adding     "postinstall": "patch-package"
to the "scripts" in package.json
It might require nuking node_modules to get rid of the vite cache.
Code Reproduction URL | URL de reproduction du code
https://github.com/theonlyguills/dfo-gcds-vue-testrig-ajcamr/tree/repro-webkit
Additional Information | Informations supplémentaires
WebKit does not seem to generate onclick events for things that don't have a tabindex if their href does not result in navigation. Adding a tabindex to all the links fixes the problem.
Also, without the tabindex, all the links are not keyboard accessible either on webkit.

Also tested on 0.30.0 with the repro link above

https://docs.webkit.org/Ports/WindowsPort.html#downloading-build-artifacts-from-buildbot
The build of webkit I used to test on windows

@theonlyguills Thank you for reporting a bug! We're able to see the fix you have and will look into it further. At the moment we are pausing releases until the new year due to team capacity over the holidays.
Please react with a 👍 on the issue if this is something that affects you, it helps us with prioritization. If you have any use-cases, screenshots, etc to share, please don't hesitate to reach out to us or provide a comment below.

Awesome, no rush, currently we implemented the patch-package script from the repro into our app so we're bundling the patched gcds. I imagine it would have impact on a lot of users given how widespread iOS devices are, but there is a workaround. Thanks!


Database restored failed. Error message: ERROR 1062 (23000) at line 24: Duplicate entry '1' for key 'PRIMARY' -

I have this error when i try to import "gettingStartedQuickly.sql" :
-Database restored failed. Error message: ERROR 1062 (23000) at line 24: Duplicate entry '1' for key 'PRIMARY' -
Actual db version is 10.5.12 and the db version in template is 10.1.38.
Can you help me with this?
Thanks in advance.

Hi @pcariaga. Did you add any other data before? Error message looks like there is a conflict with an Area that area has been created.

Hi @bfaivre thanks for the response, yes that´s correct, problem solved. Im already running it.


An existing connection was forcibly closed by the remote host

Using ASP.NET Core 2.1 RC1 the process is killed after an hour or so and we see these logs:
Application: dotnet.exe
CoreCLR Version: 4.6.26426.2
Description: The process was terminated due to an unhandled exception.
Exception Info: System.AggregateException: One or more errors occurred. (An existing connection was forcibly closed by the remote host) ---> System.Net.Sockets.SocketException: An existing connection was forcibly closed by the remote host
   at System.Net.Sockets.Socket.UpdateStatusAfterSocketErrorAndThrowException(SocketError error, String callerName)
   at System.Net.Sockets.Socket.Shutdown(SocketShutdown how)
   at Microsoft.AspNetCore.Server.Kestrel.Transport.Sockets.Internal.SocketConnection.Shutdown()
   at Microsoft.AspNetCore.Server.Kestrel.Transport.Sockets.Internal.SocketConnection.Abort()
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.Http1OutputProducer.Abort(Exception error)
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.HttpProtocol.Abort(Exception error)
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.HttpConnection.Abort(Exception ex)
   at System.IO.Pipelines.PipeCompletionCallbacks.Execute()
   --- End of inner exception stack trace ---
   at System.IO.Pipelines.PipeCompletionCallbacks.Execute()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ThreadPoolWorkQueue.Dispatch()

Faulting application name: dotnet.exe, version: 2.1.26426.3, time stamp: 0x5ae1770d
Faulting module name: KERNELBASE.dll, version: 10.0.14393.2189, time stamp: 0x5abdad60
Exception code: 0xe0434352
Fault offset: 0x0000000000033c58
Faulting process id: 0x2e90
Faulting application start time: 0x01d3e6aa48e5f5c6
Faulting application path: C:\Program Files\dotnet\dotnet.exe
Faulting module path: C:\Windows\System32\KERNELBASE.dll
Report Id: f10816d3-b0de-49bc-9745-60f9b7dea199
Faulting package full name:
Faulting package-relative application ID:

BTW we're using web sockets without SignalR.

/cc @halter73

We're having this issue on production and relying on memory cache, it's quite frustrating to have the process killed. Is there any workaround we could use until this is fixed?

I am getting the EXACT same Unhandled Exception trace on my development machine using Core 2.1 rc1 and a freshly created MVC project.  The Exception can be triggered reliably within a minute by rapidly clicking between "Home", "About", and "Contacts" from multiple browsers (Multiple browsers aren't required, but I find that it makes the problem happen quicker).
This development machine formerly ran 2.1 preview2 (without Kestrel crashes).

This issue was moved to aspnet/KestrelHttpServer#2554

Troubleshooting Update:
I attempted to recreate this bug with a fresh install of DotNet-2.1-RC1 on a fresh Windows 1709 VM, and I was unable to do so.  All of my projects ran just fine without crashing.
I returned to my original dev machine (also Windows 1709) and attempted to recreate a fresh environment by uninstalling all versions of DotNet Core and manually removing the NuGetFallbackFolder before reinstalling RC1.  This did not resolve the issue.
Then I realized that the fresh VM was on a network that bypasses Web-Filtering.  I put my original dev machine on that same network, and suddenly I was no longer able to trigger the Uncaught Exception error.  Fortunately/Unfortunately, returning the device to the original filtered network did not cause the bug to return.  I'm also unable to correlate the Uncaught Exceptions with any events logged by the Web Filter, but that doesn't mean much, because the kinds of rules that telemetry reporting would trigger don't show up in the log.
Question:
Is there any chance that the development WebHost is attempting to communicate out to the internet and crashing when the communication is cut off (possibly mid-stream) by a web-filter?


Support supervision for chat messages

Regular chat messages can also need supervising, not just the tool calls inside those chat messages.
To support this, I suggest that we reuse the tool paradigm that we already support. We can consider the case where the LLM returns text as calling a 'tool'.
Then, when we observe an LLM message return from the client, we should check:
a) does the message have text but 0 tool calls?
b) && is there a chat_tool registered?
If so, we should submit supervision requests/results for the registered chain just as we do normally.

Regarding chat message + tool calls -  I think here the tool calls take precedence so I would treat it the same as tool calls only. I wouldn't try to support some esoteric case of supervising chat messages and tool calls at the same time.

Regarding chat message + tool calls - I think here the tool calls take precedence so I would treat it the same as tool calls only. I wouldn't try to support some esoteric case of supervising chat messages and tool calls at the same time.

Support could probably be added down the line if needed anyway although I haven't thought too hard about it

@joehewett
So I believe on the client side:

I will add chat supervisor role: ✅

class SupervisorType(str, Enum):
    CLIENT_SUPERVISOR = "client_supervisor"
    HUMAN_SUPERVISOR = "human_supervisor"
    NO_SUPERVISOR = "no_supervisor"
    CHAT_SUPERVISOR = "chat_supervisor"

Use CHAT_SUPERVISOR role when registering new supervisors  ✅
I will not assign these supervisors to any tool? Or will I have some tool_id that corresponds to chat_tool? ❓
The next question is how to register chain of supervisors - that one is tied to a tool_id, so @joehewett what should I send there instead? This depends on the point 3.
❓

also

To run chat supervisors in the supervisor chains the same as with tool supervisors, i need to create tool_call_id - What should I create here
 ❓

Then I can run submit requests/results normally on the supervisor chains  ✅

I suggest that we register a tool call the chat_tool. If you think about the LLM as just outputting a message as being a tool call, then this makes some sense conceptually. So regsiter a chat_tool, and if you notice that the LLM message has no tool calls then we can consider it to be a call of the chat_tool.

Ok makes sense, I will try to implement it this way on the client side. You just need to support the CHAT_SUPERVISOR type and I can start testing it.


Adding requires_grad to kwargs when registering empty parameters.

What does this PR do?

Fixes #2361
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[x] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@younesbelkada  Looks like no problems there:
CUDA_VISIBLE_DEVICES=7 RUN_SLOW=1 pytest tests/quantization/bnb/test_4bit.py
==================================================================================================================================== test session starts =====================================================================================================================================
platform linux -- Python 3.11.5, pytest-7.4.4, pluggy-1.4.0
rootdir: /home/blacksamorez/transformers
configfile: pyproject.toml
plugins: anyio-3.7.1
collected 37 items

tests/quantization/bnb/test_4bit.py ..............s......................                                                                                [100%]

====================================================================================================================================== warnings summary ======================================================================================================================================
../quip-sharp/.conda/lib/python3.11/site-packages/_pytest/config/__init__.py:1373
  /home/blacksamorez/quip-sharp/.conda/lib/python3.11/site-packages/_pytest/config/__init__.py:1373: PytestConfigWarning: Unknown config option: doctest_glob

    self._warn_or_fail_if_strict(f"Unknown config option: {key}\n")

tests/quantization/bnb/test_4bit.py: 10 warnings
  /home/blacksamorez/quip-sharp/.conda/lib/python3.11/site-packages/bitsandbytes/nn/modules.py:226: UserWarning: Input type into Linear4bit is torch.float16, but bnb_4bit_compute_dtype=torch.float32 (default). This will lead to slow inference or training speed.
    warnings.warn(f'Input type into Linear4bit is torch.float16, but bnb_4bit_compute_dtype=torch.float32 (default). This will lead to slow inference or training speed.')

tests/quantization/bnb/test_4bit.py::Bnb4BitTest::test_rwkv_4bit
  /home/blacksamorez/quip-sharp/.conda/lib/python3.11/site-packages/torch/_utils.py:831: UserWarning: TypedStorage is deprecated. It will be removed in the future and UntypedStorage will be the only storage class. This should only matter to you if you are using storages directly.  To access UntypedStorage directly, use tensor.untyped_storage() instead of tensor.storage()
    return self.fget.__get__(instance, owner)()

tests/quantization/bnb/test_4bit.py::Bnb4BitTest::test_rwkv_4bit
  /home/blacksamorez/quip-sharp/.conda/lib/python3.11/site-packages/torch/utils/cpp_extension.py:28: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
    from pkg_resources import packaging  # type: ignore[attr-defined]

tests/quantization/bnb/test_4bit.py::Bnb4BitT5Test::test_inference_with_keep_in_fp32
tests/quantization/bnb/test_4bit.py::Bnb4BitT5Test::test_inference_with_keep_in_fp32
tests/quantization/bnb/test_4bit.py::Bnb4BitT5Test::test_inference_without_keep_in_fp32
tests/quantization/bnb/test_4bit.py::Bnb4BitT5Test::test_inference_without_keep_in_fp32
  /home/blacksamorez/transformers/src/transformers/generation/utils.py:1133: UserWarning: Using the model-agnostic default `max_length` (=20) to control the generation length. We recommend setting `max_new_tokens` to control the maximum length of the generation.
    warnings.warn(

tests/quantization/bnb/test_4bit.py::Bnb4BitT5Test::test_inference_with_keep_in_fp32
tests/quantization/bnb/test_4bit.py::Bnb4BitT5Test::test_inference_without_keep_in_fp32
  /home/blacksamorez/quip-sharp/.conda/lib/python3.11/site-packages/bitsandbytes/nn/modules.py:223: UserWarning: Input type into Linear4bit is torch.float16, but bnb_4bit_compute_dtype=torch.float32 (default). This will lead to slow inference.
    warnings.warn(f'Input type into Linear4bit is torch.float16, but bnb_4bit_compute_dtype=torch.float32 (default). This will lead to slow inference.')

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=================================================================================================================== 36 passed, 1 skipped, 19 warnings in 344.18s (0:05:44) ===================================================================================================================

Setup:
Name: accelerate
Version: 0.27.0.dev0
Summary: Accelerate
Home-page: https://github.com/huggingface/accelerate
Author: The HuggingFace team
Author-email<EMAIL_ADDRESS>License: Apache
Location: /home/blacksamorez/quip-sharp/.conda/lib/python3.11/site-packages
Editable project location: /home/blacksamorez/accelerate
Requires: huggingface-hub, numpy, packaging, psutil, pyyaml, safetensors, torch
Required-by:
---
Name: transformers
Version: 4.38.0.dev0
Summary: State-of-the-art Machine Learning for JAX, PyTorch and TensorFlow
Home-page: https://github.com/huggingface/transformers
Author: The Hugging Face team (past and future) with the help of all our contributors (https://github.com/huggingface/transformers/graphs/contributors)
Author-email<EMAIL_ADDRESS>License: Apache 2.0 License
Location: /home/blacksamorez/quip-sharp/.conda/lib/python3.11/site-packages
Editable project location: /home/blacksamorez/transformers
Requires: filelock, huggingface-hub, numpy, packaging, pyyaml, regex, requests, safetensors, tokenizers, tqdm
Required-by: lm-eval
---
Name: bitsandbytes
Version: 0.42.0
Summary: k-bit optimizers and matrix multiplication routines.
Home-page: https://github.com/TimDettmers/bitsandbytes
Author: Tim Dettmers
Author-email<EMAIL_ADDRESS>License: MIT
Location: /home/blacksamorez/quip-sharp/.conda/lib/python3.11/site-packages
Requires: scipy
Required-by:


Worked on the navbar to include dropdown menu and logout button

Also I have Updated dependencies to include bootstrap, popper js and fontawesome.
Closses: #67

Done the same changes to all the media engagement pages

@DanielMwendwa - could you address the couple of outstanding comments that @as2388 mentioned and then ask me to take a look?

Thanks @DanielMwendwa


Update Resources.resw

Update Bahasa Malaysia translations
Fixes #.
A few of the translation strings for Bahasa Malaysia can be updated to reflect new changes in the language or can be shorten.
Description of the changes:

Updated Resources.resw for ms_MY

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

Howdy @webhaikal, we don't accept pull-requests directly to the translations on Uno calculator as they should be done upstream at https://github.com/microsoft/calculator.


Use HTTPS instead of HTTP

More secure, plus it doesn't break if you have the force-HTTPS login setting enabled.

Looks good, thank you!


ui: use different icon labe for releaseIpAddress action

Description
In publicip view, actions disassociateIpAddress and releaseIpAddress actions are shown same which could cause confusion for user.
Added a different icon and label for releaseIpAddress action
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
Before:

After:

How Has This Been Tested?

@blueorangutan ui

@Pearl1594 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6235 (SL-JID-1353)


Refactor build logic

Description
Improve build logic

This isn't a great approach. Closing in favor of other options


[export] “Multiple exports” false positive on merged class and namespace

// foo.ts
export class Foo {
  bar: string;
  constructor(options: Foo.Options) {
    this.bar = options.bar;
  }
}

export namespace Foo {
  export interface Options {
    bar: string;
  }
}

{
  "parser": "@typescript-eslint/parser",
  "plugins": ["import"],
  "rules": {
    "import/export": "error"
  }
}

/tmp/test/foo.ts
  1:14  error  Multiple exports of name 'Foo'  import/export
  8:18  error  Multiple exports of name 'Foo'  import/export

✖ 2 problems (2 errors, 0 warnings)

TypeScript namespace merging should not be an error. This is similar to #1549, but that issue seems to be specific to default exports.

For me, the problem comes from doing something like:
export enum Sex {
    MALE,
    FEMALE,
}

export namespace Sex {
    export function parse(sex: string): Sex {
        switch (sex.toLowerCase()) {
            case 'male':
                return Sex.MALE;
            case 'female':
                return Sex.FEMALE
            default:
                return null;
        }
    }
}

The error is on the parse function inside the namespace even though it is being used in some other parts of the code.

Setting aside that your example is biologically inaccurate, thanks for the additional reproduction code. I'd be happy to review a PR if someone has a solution.

@httpete if you're on the latest version of the plugin and still having problems, please file a new issue.


Strikethrough text is reformatted by format()

It is expected that if I parse and then format a markdown document I should be able to get a rendered string that is identical to the input string, i.e. it should be possible to satisfy the condition markdownString == (parse(markdownString).format())
However parsing the following document:
~~line 1~~
~~line 2~~

and immediately calling .format()
results in the following string:
~line 1~
~line 2~

ExtendedMark defines strikethrough using ~~ https://www.markdownguide.org/extended-syntax/#strikethrough
Commonmark does not specify Strikethrough elements and all the specifications I can find call for ~~.
The library is able to parse the strikethrough element, e.g.
├─ Paragraph
│  ├─ Strikethrough
│  │  └─ Text "line 1"
│  ├─ SoftBreak
│  └─ Strikethrough
│     └─ Text "line 2"

Suggestion: Rendering of the Strikethrough ADT element should be configurable or changed to render as ~~.

Swift-Markdown is based the GitHub-Flavored Markdown specification, which allows one or two tildes for strikethrough. The formatting is only meant to be semantically equivalent to the input, not textually equivalent. We can use this issue to track configurable strikethrough markers, though, so that your example can still format as desired.


Adds docs for custom document models and get_document_model

Documents custom documents as described in #3782.

Thans @emilyhorsman!
Merged.


samples: sockets: http_get: More configurability and logging

Make it easy to override HTTP host/port/path. Print URL which we
request and make few other adjustment to the output for clarity.
Signed-off-by: Paul Sokolovsky<EMAIL_ADDRESS>

I'd like to use this sample as a guinea ping for https://github.com/zephyrproject-rtos/zephyr/issues/5900 , makes sense to clean it up first.

Codecov Report

Merging #5981 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #5981      +/-   ##
==========================================
- Coverage   43.77%   43.76%   -0.01%
==========================================
  Files         485      485
  Lines       54658    54659       +1
  Branches    10690    10690
==========================================
- Hits        23927    23924       -3
- Misses      30591    30595       +4
  Partials      140      140

Impacted Files
Coverage Δ

samples/net/sockets/http_get/src/http_get.c
0% <0%> (ø)
:arrow_up:

tests/kernel/obj_tracing/src/philosopher.c
72.22% <0%> (-16.67%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7623a8...828866c. Read the comment docs.


Allow TLS CA File

First up, thanks for a great gem.
Second, I have a requirement to enable TLS LDAP authentication, although unless we can supply a trusted CA cert, its not terribly secure as it will just accept any cert its given.
I had a dig through the Gems code and I don't see anyway to supply a trusted CA cert.  It just defaults to simple TLS https://github.com/cschiewek/devise_ldap_authenticatable/blob/master/lib/devise_ldap_authenticatable/ldap/connection.rb#L13
Checking the Ruby docs for the ruby-ldap lib shows that it's probably not that difficult to implement (see: http://www.rubydoc.info/github/ruby-ldap/ruby-net-ldap/Net/LDAP:encryption)
Is there any chance this may be included in the future?

I believe you can already achieve this with:
ldap_options[:encryption] = {       method: ldap_config["ssl"],       ca_cert: '/path/to/cert' }


Error with compile Twitter Bootstrap 3.2

Hi,
within compiling process I've got an error:
ParseError: Unexpected input in bootstrap_f6727c3f.less on line 1, column 36244
Seems problem with "line-height: 1.42857143 \0"
It's happened only if
$lessc->setFormatter( 'compressed' );

set.

This appears to work now, can somebody else confirm?

I'm getting the error without using the "compressed" option.
Unexpected input in anonymous-file-0.less on line 1, column 73
I'm using these settings in composer
"twbs/bootstrap": "^3.3",
"oyejorge/less.php": "v<IP_ADDRESS>"

I just tried parsing an empty .less file and it still threw the error... so nothing to do with Bootstrap.


Increase table cell padding, use icons in admin table headers

Increase table cell horizonzal padding from 3px to 5px
Add icons to user and repo list in admin tables

Fixes: https://github.com/go-gitea/gitea/issues/25939

Each icon has a tooltip:

Sorting works as usual:

Could be backported. Open for better ideas for icons.

Could backport I suppose.

As a site admin (and do the security manager's job....), I have objection for using the "icons", especially for the "user table"
I can't know the user's status clearly with that change.

I also don't think it's a good idea. I cannot diff different icons and have to put mouse on that icons every time.

I think you will get used and remember the meanings of the icons.

Does not look like anyone likes the icons.


Add mock module to forklift-console-plugin

Issue:
When mocking Rest API requests using msw we need to inject the worker code in openshift web console
FIx:

[] Add an optional openshift console extention that will inject the worker
[] Add a msw handler that will use the mock package method for fetching mock data using request information to simulate API responses see #412 and use it to build the worker.

Note:
@sjd78 @rszwajko there should be only one handler, so if we leave the server in the mock package, the mock module
should use the same handler

#392 adds MSW support as a context provider console extension.
#427 is the mocks I used to verify that the approach actually will work.
Going forward, a MSW rest endpoint handler will need to be created for each base URL that will want mock data.  They can be generated on the fly or not.  Please remember that MSW rest handlers are setup 1 per base URL ... the search string is not included.  Differentiation per search string needs to be done in the rest handler itself.  An example of pulling the parameters from the URL is: https://github.com/kubev2v/forklift-console-plugin/blob/03dbcdd87df93efcd4990ad03e02e4c3d7cf1d51/packages/mocks/src/console-handlers/providers.ts#L58-L63


Refine the scoring system

Currently, the scoring system is not ideal. Just getting more than one match at once is really difficult, let alone 3 or 4 simultaneously, and there isn't enough of a reward for doing so. It's not even particularly clear that you've made multiple matches.
What we need is a system that better rewards players for skillful matching - I'm considering a system that gives you extra points for matching a target color, which is newly chosen on every match (and perhaps also with its own independent timer). Then you would get extra points, and perhaps also extra bonus time, for making these in sequence.

OK, I've implemented the bonus color. BUT:

It doesn't give you a time bonus
It doesn't have its own independent timer
It doesn't change the bonus color after a match
There's no reward for consecutive matches
There should probably be a separate sound for getting the proper color


ComboBox doesn't work

When I use this code nothing changes when the ComboBox is selected.

@override
Widget build(BuildContext context) {
    String? selectedDivergence = '1';
    const divergenceList = <String>[
      '0',
      '1',
      '2',
    ];
    // Typography typography = FluentTheme.of(context).typography;
    return ComboBox<String>(
      value: selectedDivergence,
      //   isExpanded: false,
      items: divergenceList.map<ComboBoxItem<String>>((e) {
        // print(e);
        return ComboBoxItem(
          value: e,
          child: Text(e),
        );
      }).toList(),
      onChanged: (selected) {
        setState(() {
          selectedDivergence = selected;
          print('selectedDivergence is now $selectedDivergence');
        });
      },
    );
}

Thank your answer

console output:
flutter: selectedDivergence is now 0
flutter: selectedDivergence is now 2
flutter: selectedDivergence is now 1
flutter: selectedDivergence is now 0

Move divergenceList, selectedDivergence out of build.


Thank you!


Floating window background transparent

Hi, thanks for providing such a great plugin.
Can I set a transparent background for floating windows?

Same question, have been searching around but haven't succeeded in setting a window like telescope's to a transparent bg. Neovim noob, probably just not understanding something. Tried setting NormalFloat, TelescopeNormal, TelescopeBorder to $none, cfg.transparent and $cfg.transparent (later of which gives an error, but that's probably obvious) in the highlights in setup().

You might get the desired result using vim.api.nvim_set_hl with the blend option. I don't know what the name of the hl group for floating windows is though.

This looks well for me:
      onedark.setup{
        highlights = {
          -- make floating windows transparent
          NormalFloat = { bg = "none" },
          FloatBorder = { bg = "none" },
          FloatTitle = { bg = "none" },
          LazyNormal = { fg = '$fg', bg = '$bg1' },
          MasonNormal = { fg = '$fg', bg = '$bg1' },
        }
      }


chore(release): 2.34.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: e63c8657e58997db0cfd24b7c99ac3be3bb8779a
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: e0385536181eaf6864fe3774756ed25b2e9a7bf6
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 452b2f65f0ee49dab618090960d945b5df51d6a9
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Two questions about the Baysor pipeline

Hi,
I am running Sopa Baysor segmentation via Snakemake as shown in the tutorial.
It works quite well and is much more convenient than writing my own code for running Baysor on chunks.
I noted that the output .zarr does not contain a cell x gene matrix but only points, images and shapes.
It is not a big deal as there is an anndata in the .explorer, but shouldn't this also be in the zarr by default?
Another question I have is about the segmented shapes. For Baysor I assume this in the coordinate system of µm as that are the input coordinates or is this converted to pixel to match the image in the zarr?

Hello @pakiessling,
Normally, the .zarr should contain the table. It should be inside DATA_DIR.zarr/tables/table. You don't have such a directory? Even when running the toy dataset from the tutorial?
Yes, the intrinsic coordinate system is in microns if you use Baysor. Since we use SpatialData, you can convert it to any coordinate system using the spatialdata.transform API. This way, you can transform the shape coordinates into pixel coordinates if desired!

Ty!
It turns out something with my Spatialdata installation was messed up.
In a fresh enviroment the table loads just fine.

Nice, good to hear!

Not really related to Sopa but do you know how to go about using spatialdata.transform?
Do I first have to attach an image to spatialdata like:
Image2DModel.parse(image, dims=("y", "x", "c"))
and this will bring its pixel coordinate system with it?

You need to provide the name of the coordinate system on which you want to transform your shapes
For instance, if the pixel coordinate system is called "global":
element = sdata["baysor_boundaries"]
baysor_boundaries_pixel = spatialdata.transform(element, to_coordinate_system="global")

In case you don't have a specific coordinate system for your image, you can either add it (using an Identity transformation + the set_transformation API). But the easiest way to do that is probably to use the to_intrinsic function from sopa.


Tabs with an expression for the 'active' attribute break

With v0.10.0 (via bower) I am trying to obtain a tab's active state from an expression, such as <tab active="{{foo() == bar}}">...</tab>", however it breaks with 'undefined is not a function'
It seems that a simply wrapping the setActive call with an if statement does the trick:
      getActive = $parse(attrs.active);
      setActive = getActive.assign;
    //...
    scope.$watch('active', function(active) {
      // Note this watcher also initializes and assigns scope.active to the
      // attrs.active expression.
      if(setActive) { // <--- Add this
          setActive(scope.$parent, active); // <-- This breaks because an expression's value can't be set
      } // <--- Add this

Continuing in #1496


Add core UI 'smoke' end-to-end functional tests

[x] Error list populated on solution analysis
[x] Error list populated on project analysis
[x] Error list populated on file open.
[x] Background analysis on user edit.
[x] Error list empty after file close.

This issue is about implementing our core UI tests

Error list empty after file close. this is verified in UI tests

Error list empty after file close. this is verified in UI tests

@yongyan-gh, create the missing test since we merged the pr

@yongyan-gh, create the missing test since we merged the pr

background analysis on user edit was implemented by #349
UI automation tests was added @ https://github.com/microsoft/sarif-internal/pull/12


[SPARK-43410][SQL] Improve vectorized loop for Packed skipValues

What changes were proposed in this pull request?
Improve vectorized loop for Packed skipValues
Why are the changes needed?
Performance improvement
Does this PR introduce any user-facing change?
no
How was this patch tested?
Existing tests

Welcome to the Apache Spark community, @clownxc .
@sunchao added you to the Apache Spark contributor group and assigned SPARK-43410 to you.


📖  Fixed the path projects.txt

correct path to the file that lists the projects checked each night

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] PR title follows the guidelines defined in  https://github.com/ossf/scorecard/blob/main/CONTRIBUTING.md#pr-process

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

/ok-to-test sha=bc72923


[SD Correction] Some evaluation activities related to audit records are impossible to perform.

Provide the location of the issue
FAU_STG_EXT.1 Test 4 and FAU_GEN.1
What is the correction request for the cPP? Please describe.
PP language in FAU_STG_EXT.1 allows TOE developers option not to provide users with ability to view or export audit records created on the TOE. That creates problems when performing certain evaluation activities:

Log records generated when testing failing connection to the remote audit server will not be accessible:

Log records for FTP_ITC.1 Failure of the trusted channel functions.
Log records for FIA_X509_EXT.1/Rev Unsuccessful attempt to validate a certificate, when this certificate is provided by remote audit server.

Log records generated when connection to the remote server is not yet established:

Start/stop audit function record
Startup self-test and its outcome (mandatory log record in VPNGW PP-Module)

Evaluation activity for FAU_STG_EXT.1 Test 4 is also problematic – evaluator has no way to verify existing audit data is overwritten, dropped or other action is performed.

Describe the solution you'd like
Add PP language to require at least some form of access to locally stored audit records.
Alternative would be to add clarification in the SD on how to perform FAU_STG_EXT.1 Test 4 and FAU_GEN.1 activities in this case.
Describe alternatives you've considered
Alternative solution was to make retransmission of local audit records to the remote server mandatory, that would solve 1 and 2 but not 3 as evaluators can easily revert remote audit server to a good state and check transmitted audit records,
But that solution still leaves TOE troubleshooting problematic when connection to the remote server fails and cannot be re-established for some reason - misconfiguration, certificate expiration or revocation, etc. This will affect TOE customers as well as evaluators.
Additional context
See related #290

Related issue: https://github.com/ND-iTC/Documents/issues/117


Speed up LowRankFun constructor

I'm happy to do this one, but would like to put some ideas on the table first. The solver is fast enough now that you cannot see a lag in @manipulate, but for SIEs with non-trivial kernels, creating the LowRankFuns is now a bottleneck.

Probably the easiest thing that will help is to have a constructor for Funs that recycles the function samples. I think this would have an asymptotic savings of 1/2 and therefore 1/4 for LowRankFuns?
Write some parallel code using the CUDA or OpenCL Julia packages to evaluate the function at all the points in parallel.

@ajt60gaibb implemented some improvements to the 1D constructor to avoid over-resolving functions in the branch
https://github.com/ApproxFun/ApproxFun.jl/tree/fix-ApproxFun-scalarconstructor
but this never got merged because it broke some JacobiWeight tests.  Probably not the problem here, but maybe its worthwhile finishing this merge?

Also, have you compared the timings between LowRankFun and chebfun2?  I'm not sure LowRankFun is actually faster, if it's not then there's probably some more basic improvements needed.  (I put almost zero thought into LowRankFun, it was just hacked together and I'm always surprised when it works.)  I don't have a working Matlab so can't check.

And for SIE kernels there might be an inherent diagonal dominance that can avoid the "findmax" step.  Assuming the diagonal dominance is not just an artifact of the singular kernel (which is factored out)

Is there a convolution-type formula to represent T_n(y-x) as a sum of products of Chebyshev polynomials in both variables individually? Then, a 1D constructor could be used for diagonal SIE kernels K(y-x) and transformed into a LowRankFun or TensorFun.

There is for Laguerre and Hermite polynomials, see http://dlmf.nist.gov/18.18#E38 and http://dlmf.nist.gov/18.18#E39.  But for Tn(y-x) I do not know exactly what you want since the arguments to Chebyshev polynomials are usually on [-1,1].  What intervals do y and x live on?

The closest thing I can think of looks like this: http://dlmf.nist.gov/18.18#E8

We could just precompute the expansion for each T_k, though maybe that's not so elegant or fast
Sent from my iPad

On 9 Jan 2015, at 4:49 am, Richard Mikael Slevinsky<EMAIL_ADDRESS>wrote:
Is there a convolution-type formula to represent T_n(y-x) as a sum of products of Chebyshev polynomials in both variables individually? Then, a 1D constructor could be used for diagonal SIE kernels K(y-x) and transformed into a LowRankFun or TensorFun.
—
Reply to this email directly or view it on GitHub.

Can't you just build it up from three term recurrence?
Sent from my iPad

On 9 Jan 2015, at 4:49 am, Richard Mikael Slevinsky<EMAIL_ADDRESS>wrote:
Is there a convolution-type formula to represent T_n(y-x) as a sum of products of Chebyshev polynomials in both variables individually? Then, a 1D constructor could be used for diagonal SIE kernels K(y-x) and transformed into a LowRankFun or TensorFun.
—
Reply to this email directly or view it on GitHub.

Good point @ajt60gaibb and sorry for the confusion. Since both y and x are in [-1,1], a kernel that is a function of the difference would be on [-2,2] and therefore would be expanded in T_n((y-x)/2).

Also, have you compared the timings between LowRankFun and chebfun2? I'm not sure LowRankFun is actually faster, if it's not then there's probably some more basic improvements needed. (I put almost zero thought into LowRankFun, it was just hacked together and I'm always surprised when it works.) I don't have a working Matlab so can't check.
I found that the chebfun2 is much more accurate for functions of higher rank, but slower.
Anyways, my uses for this issue are mostly mitigated by the new ProductFun constructor. Perhaps we can aim for nested Chebyshev points and recycled function samples to get the simple savings.

I didn't really consider this before because I was convinced the fastest way would be to use an addition theorem in coefficient space, but perhaps the implied symmetry between source and field points of a Green's function can reduce the search space for approximate maxima in the low rank construction and preserves the symmetry through complete pivoting. Will look into this shortly.

Usually, complete pivoting only preserves symmetry if the underlying Green's function is positive definite so the pivots are always on the diagonal.  If this isn't the case, then one can sometimes preserve symmetry by using 2x2 matrices as pivots (but still not sure how to reduce the search space).

Thanks Alex!

For Hermitian positive-definite "cmatrices," the LowRankPositiveDefiniteFun constructor introduces a significant speed-up:

julia> using ApproxFun, SIE

julia> sp1,sp2 = Chebyshev(),Chebyshev()
(Chebyshev(【-1.0,1.0】),Chebyshev(【-1.0,1.0】))

julia> f(x,y) = besselj0(30abs(y-x))
f (generic function with 1 method)

julia> @time F = LowRankFun(f,sp1,sp2);
elapsed time: 0.565386061 seconds (119788788 bytes allocated, 19.01% gc time)

julia> @time G = LowRankPositiveDefiniteFun(f,sp1,sp2);
elapsed time: 0.044882095 seconds (11273888 bytes allocated)

julia> @time H = ProductFun(f,sp1,sp2;method=:convolution);
elapsed time: 0.020821332 seconds (8847984 bytes allocated)

For convolution-type kernels, the ConvolutionProductFun also introduces a significant speed-up:

julia> using ApproxFun, SIE

julia> sp1,sp2 = Chebyshev([-1.0-0.5im,1.0+0.5im]),Laurent(Circle(-0.5+0.5im,0.5))
(Chebyshev(【-1.0 - 0.5im,1.0 + 0.5im】),Laurent(Circle{Complex{Float64},Float64}(-0.5 + 0.5im,0.5)))

julia> f(x,y) = besselj0(30abs(y-x))
f (generic function with 1 method)

julia> @time F = LowRankFun(f,sp1,sp2);
elapsed time: 17.878112127 seconds<PHONE_NUMBER>4 bytes allocated, 57.94% gc time)

julia> @time H = ProductFun(f,sp1,sp2;method=:convolution);
elapsed time: 0.034836927 seconds (15682372 bytes allocated)

Cool.
Sent from my iPhone

On 17 Mar 2015, at 11:14, Richard Mikael Slevinsky<EMAIL_ADDRESS>wrote:
For Hermitian positive-definite "cmatrices," the LowRankPositiveDefiniteFun constructor introduces a significant speed-up:
julia> using ApproxFun, SIE
julia> sp1,sp2 = Chebyshev(),Chebyshev()
(Chebyshev(¡¾-1.0,1.0¡¿),Chebyshev(¡¾-1.0,1.0¡¿))
julia> f(x,y) = besselj0(30abs(y-x))
f (generic function with 1 method)
julia> @time F = LowRankFun(f,sp1,sp2);
elapsed time: 0.565386061 seconds (119788788 bytes allocated, 19.01% gc time)
julia> @time G = LowRankPositiveDefiniteFun(f,sp1,sp2);
elapsed time: 0.044882095 seconds (11273888 bytes allocated)
julia> @time H = ProductFun(f,sp1,sp2;method=:convolution);
elapsed time: 0.020821332 seconds (8847984 bytes allocated)
For convolution-type kernels, the ConvolutionProductFun also introduces a significant speed-up:
julia> using ApproxFun, SIE
julia> sp1,sp2 = Chebyshev([-1.0-0.5im,1.0+0.5im]),Laurent(Circle(-0.5+0.5im,0.5))
(Chebyshev(¡¾-1.0 - 0.5im,1.0 + 0.5im¡¿),Laurent(Circle{Complex{Float64},Float64}(-0.5 + 0.5im,0.5)))
julia> f(x,y) = besselj0(30abs(y-x))
f (generic function with 1 method)
julia> @time F = LowRankFun(f,sp1,sp2);
elapsed time: 17.878112127 seconds<PHONE_NUMBER>4 bytes allocated, 57.94% gc time)
julia> @time H = ProductFun(f,sp1,sp2;method=:convolution);
elapsed time: 0.034836927 seconds (15682372 bytes allocated)
¡ª
Reply to this email directly or view it on GitHub.


Can't begin evict leader when TLS between components is enabled

Please answer these questions before submitting your issue. Thanks!

What did you do?
If possible, provide a recipe for reproducing the error.

I created a TiDB cluster with TLS enabled between components in tidb-operator.
but i failed to upgrade the TiKV instances, the upgrade process will evict the leaders before restart the TiKV instance.

What did you expect to see?

evict leader succeed.

What did you see instead?

failed to evict leader:
Post https://db-pd-0.db-pd-peer.tidb1252078906480005120.svc:2379/pd/api/v1/scheduler-config/evict-leader-scheduler/config: x509: certificate signed by unknown authority

What version of PD are you using (pd-server -V)?

nightly

Fix in https://github.com/pingcap/pd/pull/2368


Free Resource: Highlight.js languages as HTML Select and JSON

Howdy! I needed the list of languages as an HTML select and figured other people might benefit from that, too. Here's a Codepen with the HTML as well as a little script to generate the list as an array of JSON objects: https://codepen.io/command-the-scripter/pen/JjoyKQE

I'm not quite sure what to do with this.  It's not an issue, so it can't live here... and we don't really use the wiki for stuff like this...

Probably it could be helpful if we supply such list of languages for each bundle?.. Yet I'm not sure too.

Don't we already have this name in the grammar file header? Language class in tools/lib/language.js works with it.

Use hljs.listLanguages() and let external I18N system to convert language key into human readable name?

@egor-rogov Oh sure you can already do this if you have the RAW source, but not the library - I'd like to see that changed so that name is actually a key of each language.

Creating #2394 to follow up on this.  Closing this as a duplicate issue of that.


Rebasing in changes #1340

Documentation added related how to rebase in branches in contributing/git.

@programmer290399 is this fine?

I tried to add the image using this link  but it appeared as link under the preview bar. Any suggestions to these changes @programmer290399 @pdxjohnny .

Referenced from - https://docs.github.com/en/get-started/using-git/about-git-rebase

@pdxjohnny is everything resolved for this PR or are there any changes required?


Couldn't find Modifier.preferredWidth and others

In the last 153 build I couldn't find functions Modifier.preferredHeight, Modifier.preferredWidth and other preferred*.
Are those available for desktop ?

https://android-review.googlesource.com/c/platform/frameworks/support/+/1572682

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Move auth to JWTs

Move auth to JWTs + fingerprint

Don't merge until #128 gets merged, I don't want to merge more CommonJS code

ready to merge since #128 has been merged


⚠ Bofrost.lu has degraded performance

In ca5bb38, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1265 ms

Resolved: Bofrost.lu performance has improved in cb592d9 after 8 minutes.


Take split_dataset out from fit

Take out the split_dataset call from the fit function to remove randomness from that, which will make the tests more reliable.

@miuraken Hello! Could you review this PR when you have time?


The video player size is all jacked up

https://gist.github.com/anonymous/4fdc8c8db98cd1b83edd
http://aboutbrowser.com/view/8gDZZ

Update YTC, you are still using version 406 when the most recent version is 520, this bug has been fixed already.

Huh?

from the git hub page

from my extensions page

Correct, 2.1.7 has been obsolete for a long time. @YePpHa has not yet published a new "stable" version. Until that time, please use the Developer Version.


Test multi-repo model

Refs: https://github.com/microsoft/vscode-pull-request-github/issues/1936

[ ] macOS @deepak1556
[ ] linux @roblourens
[x] windows @jrieken

Complexity: 4
Create Issue

The whole extension has moved over to a multi-repo model. Previously, there was the concept of a "currently active repo" when you had multiple repos open in a multi-root workspace. Now, all repos are shown in the Pull Requests and Issues views. In some places, the correct repo is inferred by where the action is taken (ex: if you create an issue from a file, then the correct repo for that file should be used). Some areas verify, both with one repo open and with a multi-root workspace:

Verify that the Pull Requests view shows PRs from all repos.
Verify that the Issues view shows issues from all repos.
Try creating some PRs and issues, and verify that they get created in the expected repos.
Verify that # issue suggestions and @ user suggestions come from the right repos.
Check out pull requests, both from one repo and multiple repos. Verify that the Changes in Pull Request view makes sense.
Verify that comments/reviews on PRs (both when you have a PR checked out and when don't have it checked out) work as expected.
Verify that you can change the issues and PRs queries using the corresponding settings.

👏 works really well


mask mems about reduce_masked_mean

Thansk to the excellent work! I have a question about reduce_masked_mean.
in oder to fuse every camera feature, you get "mask_mems" by mask_mems = (torch.abs(feat_mems) > 0).float(),  and get in reduce_masked_mean,  but in vox_util.unproject_image_to_mem function , there is  a  "valid_mem"  from valid_mem = (x_valid & y_valid & z_valid).reshape(B, 1, Z, Y, X).float(),  and adapt to values by values = values * valid_mem , I'm confused about this, why not using  "valid_mem" from vox_util.unproject_image_to_mem ,but get a new "mask_mems" by mask_mems = (torch.abs(feat_mems) > 0).float()? seems not make sense to me.

guess I understand now ..  sorry for the interrupting  ; )


Where to store IP addresses in Network Connection Info object?

Based on discussion at https://opencybersecu-lz97379.slack.com/archives/C03C2QPSBPB/p1727259577114689
The field for an IP address to be stored in a network connection info object is unclear. The discussion identified the likely suspect of adding an IP address to the uid field which makes sense given the context. Some examples would be helpful

https://opencybersecu-lz97379.slack.com/archives/C03C2QPSBPB/p1727259577114689

I have a question regarding the Network Remediation Activity class. This class requires the connection_info attribute, which corresponds to the Network Connection Information object. However, this object seems to lack an attribute for storing an IP address
As a producer, when I detect a remote ransomware attack on my shared files, I isolate the remote endpoint and intend to generate a corresponding Network Remediation Activity event. Where within the object can I record the IP address of the remote endpoint? (edited)


ENH: Normalise the inverse DCT/DST in scipy.fft

Closes #3677
As described in the issue, this adds the correct normalization factor to the inverse dct/dst such that the identity idct(dct(x)) == x is true for all normalization modes. This is consistent with the other FFT functions in scipy.fft
~I will fix the merge conflicts with #10383 after one is merged.~ ✔

I've added to the release notes:

the inverse real to real transforms (idct and idst) are normalized for norm=None in the same way as ifft. This means the identity idct(dct(x)) == x is now true for all norm modes.


Added a function in order to get the message body summary

Basically I never understood why this function was added in RequestException class https://github.com/guzzle/guzzle/blob/master/src/Exception/RequestException.php#L123 and why only for the responses.
I thought that this is a helpful function that could be added in the functions file.

Thanks @gmponos


[WIP]: Update core notebooks for 0.6 functionality

This is an update of the core notebooks in:

reference/algorithms
reference/games
reference/qis
reference/tools

to terra-0.6 and the added functionality included in it.  reference/tools/quantum_gates_and_linear_algebra.ipynb has not been updated as there is currently a bug holding it back (https://github.com/Qiskit/qiskit-terra/issues/946). I have not updated the registration method since work on this feature set is still ongoing (see https://github.com/Qiskit/qiskit-terra/issues/925).
This does not include the qcvv stuff, as that needs to be cleaned up (see #251), and I will let the aqua people update their own notebooks.

Thanks. To my understanding, as the qiskit-terra is still officially at 0.5.7, these updates will be merged right after qiskit-terra has been updated first, right?

Correct.  However getting working tutorials is part of the release requirements, so these must be complete before 0.6 is confirmed for release.  They can of course be merged after the fact.

That is an ipywidget that is returned by the job status jupyter magic. It automatically checks job status for you.

Made this into a WIP so that additional stuff can be added.

@diego-plan9, when the new registration system is finished, please modify the notebooks in this PR.  All the other changes for 0.6 have already been included.


Webhook Delay?

I have been noticing an odd behavior and wondering if it is just s fluke or if it's related to the Webhook change.
Recently I made the shift from polling to webhook.
Due to the device limits, I had a bit of a struggle completing the shift.
However, as far as I know, I am good to go.
But I have noticed some odd behavior - especially with my door locks.
I will receive notices that a door was unlocked or locked hours after I know that door was opened.
This morning I noticed I immediately got an alert my door was unlocked when I opened the Homebridge and navigated to the Accessories page.  However that door was last opened an hour prior.
I don't really have any other "alerts" configured, I'll try testing with other devices....
Functionality in the "Home" app seems perfectly fine.
This, if its an issue, would seem to impact devices where under "Settings" and "Status and Notifications" -> "Activity Notifications" has been set.

Put homebridge into debug mode.  You can see incoming events from the web hook service.  Go and unlock a door and see if you get the event showing in the logs.

On Mar 3, 2023, at 9:42 AM, coryabooth @.***> wrote:
I have been noticing an odd behavior and wondering if it is just s fluke or if it's related to the Webhook change.
Recently I made the shift from polling to webhook.
Due to the device limits, I had a bit of a struggle completing the shift.
However, as far as I know, I am good to go.
But I have noticed some odd behavior - especially with my door locks.
I will receive notices that a door was unlocked or locked hours after I know that door was opened.
This morning I noticed I immediately got an alert my door was unlocked when I opened the Homebridge and navigated to the Accessories page. However that door was last opened an hour prior.
I don't really have any other "alerts" configured, I'll try testing with other devices....
Functionality in the "Home" app seems perfectly fine.
This, if its an issue, would seem to impact devices where under "Settings" and "Status and Notifications" -> "Activity Notifications" has been set.
—
Reply to this email directly, view it on GitHub https://github.com/iklein99/homebridge-smartthings/issues/147, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABG55EVQSEF336Y3IBWEP3DW2H7MLANCNFSM6AAAAAAVOWHCOQ.
You are receiving this because you are subscribed to this thread.

I found an issue for people that did the SmartApp install more than once.  If you did, you may not have been receiveing events.  That should be fixed now.


Drop Markitup Editor support

it would also probably be best practice to re-create the assets folder and create a .gitkeep file, as this folder is still used

Are we absolutely sure that CKEditor is implemented everywhere?
If yes, then this seems all good to me.

Yes, everything should be including the src/modules/Wysiwyg/html_admin/mod_wysiwyg_js.html.twig file, which was used to switch between the different WYSIWYG editors


Streaming setting linux

### Anything between here and the comment below will become the contents of the drop-in file

[Service]
Environment="OLLAMA_ORIGINS=*"

### Edits below this comment will be discarded

1 systemctl edit ollama.service
2 [Service]
Environment="OLLAMA_ORIGINS=*"
3 systemctl daemon-reload
4 systemctl restart ollama

FYI: If you are using systemctl edit to modify a unit file (or an override for one), you do not need to use systemctl daemon-reload.
PS: Sorry for the fly-by


Cron is running, but laravel tasks are not (solved)

Figured I'd post this problem and the solution here, I can't be the only one having this issue.
I could see in the Docker logs (when you don't run -d you see them) that cron is running, but tasks in laravel are not running, which seems strange because CRON is running the correct artisan command:
workspace_1            | Jun 11 12:31:01 fc6f62756576 CRON[990]: (laradock) CMD (/usr/bin/php /var/www/artisan schedule:run >> /dev/null 2>&1^M)
If you look closely, you'll see ^M at the end of the line. This is because I edited the cron table file in Windows, and the Windows new-line character (carriage return) was added. This causes a syntax error:
bash: 1^M: ambiguous redirect
so this command isn't run, and because MTA isn't installed the output of this command (an error) is discarded.
The solution is to configure git/your windows text editor to use Linux line endings.

solved by using linux line endings


Stop early if randomize_stat_freqs won't do anything

randomize_stat_freqs can't have any effect in the case where all the symbols in beststats have the same frequency, nor does lz77_optimal have any way to improve the compression if that's the case and the last iteration had no effect (and I doubt any improvement is possible since this means they follow a maximum-entropy distribution). So this PR ends the deflation of the block early if that happens.
As well, since the above check means it can no longer get stuck in an infinite loop, this PR changes randomize_stat_freqs to run again if it hasn't made an actual change, which eliminates redundant LZ77 runs. Dead-code elimination could not have done this, because calling a PRNG mutates its state.
This PR also postpones running add_weighed_stat_freqs until we know its result won't be overwritten from best_stats before it's used.

Have you confirmed with Trace-level logging that the further LZ77 runs are making the difference, and not just the slightly different PRNG state from having been invoked fewer times?

I didn't check whether the root cause for the regressions was the different PRNG state or the fact that less LZ77 runs are done, but our results show that even if the cause is a different PRNG state, on average it's slightly worse, and thus the behavior change is a good candidate to be toggleable.

It seems like the optimality is very sensitive to the PRNG state. (Compare the last two commits on eeyore.png!) I think instead of trying to micro-optimize Zopfli, I'll fork it and experiment with other methods of optimizing stats such as genetic algorithms.


Test jar contains resources at its root

The test jar contains resource files that are at the root of the jar (not in a package). This can create ambiguity if these resources exist in multiple jars on the class path or elsewhere on the class path. Resources can, and should, also be scoped to a package, if they are included in a jar. This probably doesn't matter a lot right now, but it does affect the way we need to load them as class path resources, and may be antithetical to Java 9's new module system, if not actually preventing the test jar from working in a modular environment.
$ jar tf test/target/accumulo-test-2.1.0-SNAPSHOT.jar | grep -v ^org/ | grep -v ^META-INF/
FooConstraint.jar
FooConstraint_2_1.jar
FooFilter.jar
ShellServerIT-iterators.jar
TestCombinerX.jar
TestCombinerY.jar
TestCompactionStrat.jar
log4j2-test.properties
shellit.shellit

These test resources were included in the jar because they exist in src/main/resources/. They can probably be relocated to src/main/resources/org/apache/accumulo/test/, but the tests that use them (as well as log4j) would have to be updated to find them at their new location.

Are there issues with these files being in the jar?

log4j2-test.properties
shellit.shellit

I have no idea what that second file is so not sure what to do with it.

Are there issues with these files being in the jar?
log4j2-test.properties

This one might be tricky to move. I'm not sure. We're probably relying on it being in a default location, for log4j to pick it up automatically. However, that's probably also controllable with a system property we can set in the surefire/failsafe config.

shellit.shellit
I have no idea what that second file is so not sure what to do with it.

This can be moved. It actually looks like another one that doesn't actually need to be put into the jar. The ShellIT reads it directly from src/main/resources , and it looks like it does that in order to test executing a file with Accumulo shell commands.


Getting 500 error response for client errors in SCIM/Groups PATCH

Describe the issue:
Getting 500 error response for the below listed client errors

Add a member to a group with only display name
Add a member to a group with only id
Add a member to a group with invalid display name
Add a member to a group with invalid id
Add a user with mismatching user name and id
Add user with empty id
Remove a non-existent member from a group - Only happens with LDAP user stores.
Remove a non-existent user from a group

More details about each of these scenarios can be found at https://docs.google.com/spreadsheets/d/1h8UTae7LnqWx0WYYGOUGkv3mT7Cu5HtX4kM2pOmlbLw/edit#gid=646167061
How to reproduce:
Try the scenarios listed above and observe the responses received
Expected behavior:
Should respond with 400 with the correct error type. Need to decide on the correct error type for each scenario according to the spec.

When fixing these issues need to pay attention to the following case as well.
According to the specification performing multiple operations in one PATCH request should happen as below.
Each PATCH operation represents a single action to be applied to the
same SCIM resource specified by the request URI.  Operations are
applied sequentially in the order they appear in the array.  Each
operation in the sequence is applied to the target resource; the
resulting resource becomes the target of the next operation.
Evaluation continues until all operations are successfully applied or
until an error condition is encountered.

Can test this with the following scenarios

Add and remove a non-existing member of a group in one request - user should not be added
Remove and add a non-existing member of a group in one request - user should be added
Add and remove an existing member of a group in one request - user should be removed
Remove an add an existing member of a group in one request - user should not be removed

Following issues are addressed in this PR

Change the error responses when adding a member to a group with only a display name.
Change the error responses when adding a member to a group with only an ID.
Change the error responses where a user is added with an empty ID.

Addressed Issue Scenarios and PRs
The following issue scenarios have been thoroughly addressed through the respective pull requests (PRs):

Adding a member to a group with only the display name - PR
Adding a member to a group with only the ID - PR
Adding a member to a group with an invalid display name - PR
Adding a member to a group with an invalid ID - PR
Adding a user with mismatching user name and ID - PR
Adding a user with an empty ID - PR
Removing a non-existent member from a group (observed in LDAP user stores) - PR1, PR2
Removing a non-existent user from a group - PR

In the process of evaluating the above issue scenarios across various operational contexts, a specific scenario of interest emerged:
Removal and Addition of a Non-Existing Member
When attempting to both remove and add a non-existing member of a group in a single request, a specific outcome was expected: the user should be added. The JSON snippet below depicts this scenario:
{
    "schemas": [],
    "Operations": [
        {
            "op": "remove",
            "path": "members[display eq user1]"
        },
        {
            "op": "add",
            "value": {
                "members": [
                    {
                        "display": "user1",
                        "value": "db2c7659-1078-4afc-9bc3-f6fba6adc351"
                    }
                ]
            }
        }
    ]
}

This scenario operates as anticipated when using the username to remove the user. However, when utilizing the user ID for removal, a 400 response is encountered. The corresponding JSON snippet is provided below:
{
    "schemas": [],
    "Operations": [
        {
             "op": "remove",
             "path": "members[value eq db2c7659-1078-4afc-9bc3-f6fba6adc351]"
        },
        {
            "op": "add",
            "value": {
                "members": [
                    {
                        "display": "user1",
                        "value": "db2c7659-1078-4afc-9bc3-f6fba6adc351"
                    }
                ]
            }
        }
    ]
}

This issue also has been fixed in the aforementioned PRs.


ONTAP NAS Volume name is limited to 64 characters #2

This is a follow up to:
https://github.com/NetApp/trident/issues/174
For E-Series and CVS you generate a random internal name. One should think it should be possible to do this for the ontap-nas-economy-driver as well?
I've had issues with deployments using the operator and Ansible Service Broker frameworks, and the workarounds are not nice.

This should not be an issue with the 19.07 CSI driver, as long as your storage prefix is not longer than 24 characters. Does that work for you?

According to the change log:

Kubernetes: Implemented CSI Trident (optional for Kubernetes 1.13, exclusive for Kubernetes 1.14+).

We currently run Kubernetes 1.11 (OpenShift 3.11) on all our environments, so that will not work for us at the moment.


Allow Set-OSServerConfig.ps1 to install Service Center

Added parameter to set-OSServerConfig to enable the install of Service Center to be execute d during the Set of the OSServer.
This flag is supported on both O10 and O11.

Please also take a look in:
https://github.com/OutSystems/OutSystems.SetupTools/blob/dev/src/Outsystems.SetupTools/Functions/Install-OSPlatformServiceCenter.ps1
SetSCCompiledVersion -SCVersion $osVersion
For idempotent