python3Packages.i3ipc: 2.1.1 -> 2.2.1

Motivation for this change
2.1.1 had some failing tests.
Updating it to the latest version fixed it.
Things done
Updated version and sha256 sum.

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

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

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

@GrahamcOfBorg build python3Packages.i3ipc


(JS-A1002) Audit: Regex range is too permissive

Description
In regular expressions, the - operator can be used to match a range of characters. For example, /a-z/ will match any character that has a character code between 'a' and 'z'. Large ranges, like A-z, are often the result of typos or incorrect refactors. The range A-z will match any …
Occurrences
There are 2 occurrences of this issue in the repository.
See all occurrences on DeepSource → app.deepsource.com/gh/bubonicfred/5minitz/issue/JS-A1002/occurrences/

Closed with #456


ci: upgrade to latest action version

https://github.com/wenytang-ms/apicenter-sample/actions/runs/9850274632/job/27195498051

https://github.com/wenytang-ms/apicenter-sample/actions/runs/9851250895/job/27197796530


Ipm v2

v2:
Sent blob now keeps message id, so it's easier to reference them on receivers side;
Channel numbers are build configurable;
Consumed confirmation callback now per id, not per message;
New sol_blob functions to create blobs that duplicate memory, instead of only wrapping them;
Failure on receiving blobs shall send messages to sender that blob is consumed;
Using uintptr_t;
Hopefully all issues of v1 addressed;
Note: One more commit to fix build when flow is disabled.
Introduces API and node types for Inter-Processor Messaging on Soletta. Currently, it's only for Zephyr.
Samples on soletta-small-oses-samples: solettaproject/soletta-small-oses-samples#18

v3 on #2006


[Activity] Not found page text should be different for my strategies and individual strategy vs the explorer

The current text for when there is no activity found in my strategy, individual strategies and the explorer is:
We couldn't find any activities
Try entering a different wallet address or choose a different token pair.

This text only makes sense in the context of the explorer page.

explorer page:

pair search

We couldn't find any strategies
Try a different token pair or reset your filters.

wallet search

We couldn't find any strategies
Try entering a different wallet address or reset your filters.

my strategies:
strategy page:
Strategy not found
The strategy you are looking for does not exist or has been deleted.

Fixed as part of #1426


Refactor ReactVite Registration Strategy

Asset keys are clunky, using "magical strings".

Addressed in 1.4.5


test: #76 Automate Preview Deployment and Comment Deployment URL on PRs

To make this action work, add the required values from Vercel as secrets in GitHub:

Retrieve your Vercel Access Token
Install the Vercel CLI and run vercel login
Inside your folder, run vercel link to create a new Vercel project
Inside the generated .vercel folder, save the projectId and orgId from the project.json
Inside GitHub, add VERCEL_TOKEN, VERCEL_ORG_ID, and VERCEL_PROJECT_ID as secrets

Please lemme know about the setup you're planning.

The workflow is:

New PR  opens
Build and deploy a preview of the changes
PR is Merged
Build and deploy to the production

For the action to work, in the GitHub repository, go to "Settings" > "Secrets" > "New repository secret", and add the environment secret as mentioned.


template or render function not defined when mounting a component using vue-class-component

Version
2.7.4
Reproduction link
https://github.com/dennybiasiolli/vue27-syncfusion-bug
Steps to reproduce

clone the repo https://github.com/dennybiasiolli/vue27-syncfusion-bug
install dependencies with npm install
launch with npm run serve
notice the error in the console

What is expected?
The page should display a button with "Button" text at the bottom of the page.
What is actually happening?
The component is not displaying and in the console there is an error:
Failed to mount component: template or render function not defined.

Maybe the external component you are using is NOT compatible with<EMAIL_ADDRESS>import { ButtonPlugin } from '@syncfusion/ej2-vue-buttons';
Its package.json only list<EMAIL_ADDRESS>

Oh maybe you are right, but I did think the update from 2.6 to 2.7 was without breaking changes.
Just opened this issue in case it can be useful to analyze the problem

Fundamental problem is that ej2-vue is applying different behavior based on the presence of the global h API: https://github.com/syncfusion/ej2-vue-ui-components/blob/master/components/base/src/component-base.ts#L28
This is wrong because the presence of global h is never guaranteed to be the difference between Vue 2 and Vue 3. It should use Vue.version to check behavior difference instead.

@yyx990803 thank you very much for the explanation! I will try to submit a fix with a PR on the ej2-vue-ui-components repository.


Support for multiple trigger buttons

I would like to use multiple trigger buttons for my JS Offcanvas menu, but it seems that the options limit the triggerButton selector to the first element only:
if (options.triggerButton ) {
	this.$triggerBtn = $(options.triggerButton);
	new window.componentNamespace.OffcanvasTrigger(this.$triggerBtn[0], {"offcanvas": offcanvasID}).init();
}

I'd like for all elements matching the selector used for options.triggerButton to work.
Edit:
I was able to create a workaround, but I do believe that the expected behavior of using a selector that allows class names is to apply the functionality to all instances found.
jQuery( function($) {
	var drawerSelector = "#the-drawer";
	var $drawer = $(drawerSelector).offcanvas({
		triggerButton: '.drawer-toggle' // Button to open offcanvas
	});

	var drawerData = $drawer.data('offcanvas-component');

	// Only the first instance of .drawer-toggle works as a toggler,
	// so add a new class and manually trigger the togglin'
	var $offcanvasMenuToggle = $(".menu-drawer-toggle");
	$offcanvasMenuToggle.find("a").on("click", function(e) {
		e.preventDefault();
		drawerData.toggle();
	});
});

Hi David,
the triggerButton is optional. Check the Demo without triggerButton.
Now for your Problem. Can you try this:
A)
<button data-offcanvas-trigger="left">js-trigger-offcanvas</button>
<button data-offcanvas-trigger="left">js-trigger-offcanvas</button>

<aside class="js-offcanvas" data-offcanvas-options='{"modifiers":"left, overlay"}'  id="left">
...
</aside>

$( document ).trigger( "enhance" );

or B)
<button class="js-btns">js-trigger-offcanvas</button>
<button class="js-btns">js-trigger-offcanvas</button>

<aside id="left">...</aside>

$( '.js-btns' ).offcanvasTrigger( {
  offcanvas: "left"
});
$( document ).trigger( "enhance" );

Can you tell me what works for you?

Thank you for such a quick and helpful response. I went with option B and it works great.


Remove legacy single-function class method import logic

Followup on https://github.com/modal-labs/modal-client/pull/1944
Removes a bunch of code that should be unused in modern apps.
I think this should be safe since this updated container entrypoint should only ever be used by functions deployed using the new style classes

@prbot approve


Update API documentation

@rultor merge

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 2min)


Add harbor chart v0.0.1 for global registry

Related issue: https://github.com/rancher/rancher/issues/1738
Notes:

global registry is available in HA setup.
Expose registry /v2 endpoint to the same domain of Rancher server.
Expose /api/projects endpoint to Harbor core. It may be needed when the global registry is used as another Harbor's replication target. it will try to create a project via this endpoint when it does not exist.
Use /registry endpoint to aggregate Harbor UI and its API/services endpoints.
Add a proxy layer to help Harbor work with path prefix.

LGTM

@thxCode Added unit tests for the chart, PTAL.

LGTM

Updated the chart version to 1.7.5-rancher1 per discussion

LGTM


Implement cheat codes

Add a basic set of cheat codes to revm to use in EVMts

Using js predeploys instead


You may add a bit text stating that this project is no more active

Maybe you can contribuite instead of trolling
Il 10 mag 2017 15:58, "Javier Jerónimo"<EMAIL_ADDRESS>ha
scritto:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/thesp0nge/owasp-orizon/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIvPrWISufZDPaMDVGrRYRWM_T-CX6Iks5r4cJzgaJpZM4NWtGC
.

 I was really interested in this project and tried to run it locally, but found that the execution did nothing. Maybe I checked out the wrong branch or missed something in the setup.
Anyway, the problem is that it was linked from a rencently updated page in OWASP Wiki, but in fact, looking at the project page in OWASP, it is listed as an inactive one.
Thats the reason I asked you to put a note stating it is inactive, just in case it is visited from a search result or linked from anywhere different that its project page in OWASP.
Regards.

On the upcoming Owasp Summit, there will be a session to help people rebooting the project (http://owaspsummit.org/Working-Sessions/Project-Summit/Owasp-Orizon-Reboot.html).
Owasp Orizon was marked as Inactive because I didn't update a Wiki but this is, IMHO, a very bottom priority.
The most, and difficult in the cybersecurity world, concern is to build an ecosystem of people working on it. The same, it was very easy to me with dawnscanner project since ruby guys love writing code. Unfortunately, very few cybersec guys love writing stuff.
So the project is trying to reboot, very slowly, but it's far from being inactive

What is the state of the project actually? I'm a developer studying a cybersecurity master, and I'd like to contribute to some open-source projects related to cybersecurity, but of course, I'd like if it has a community behind and I'm not the only one hehehehe


Fixed typos in linear algebra tutorial

This is the fix to two small typos in linear algebra notebook.

Good catches, thanks @edwinhung 👍


Change README.md

I made a change!  Feel free to include it if you like it!!!!

Thank you for coming to the workshop! What is your name, so I can mark you as complete?

Hi Kevin!  I'm Greg Irwin, and I attended the Codefellows Unic and Git for Everyone course. Cheers!

Excellent, thank you! Thank you for coming to the workshop, as well!
On Thu, Sep 3, 2015 at 4:16 PM, branespace<EMAIL_ADDRESS>wrote:

Hi Kevin! I'm Greg Irwin, and I attended the Codefellows Unic and Git for
Everyone course. Cheers!
—
Reply to this email directly or view it on GitHub
https://github.com/codefellows/sea-w32-uge/pull/1#issuecomment-137598940
.


Implement timeout cancellation on Chain base class

Description: Implement base Chain cancellation policy (execute a Chain’s cancellation after n milliseconds), similar to the implementation in the JS repository

Closing because the PR wouldn't line up with the current directory structure of the library (would need to be in /libs/langchain/langchain instead of /langchain). Feel free to reopen against the current head if it's still relevant!


Create plot_regression_fromoutput_Hbb.py

Change signal label to only Hbb

@ikaul00 I don't see the need to duplicate the script, can you comment out the other strings in the dictionary and add the bb? It can also be an option too. Closing this but maybe you can make another PR to modify the original script


GPL-2.0 in test tests/php

Hi,
thank you very much for providing this package.
I am currentliy doing a license analysis of markdown and came across the GPL-2.0 license text in tests/php.
How is the relation of the GPL-2.0 to the files in this repo, does it apply to any of the files, if yes it would be very useful that those files contain a reference to the GPL-2.0 otherwise no one can determine to which files it applies and so it will be very hard to comply to the license.

Well, I had no idea that file was there. In fact, the php tests were added before I joined the project, so we are not likely to get a definitive answer. My best guess would be that the license was added with the php test files. As I understand it, back when those tests were added. they were copied from the php markdown project as-is. Presumably, this license is the license which applies to them (that is all files in the php directory and any subdirectories).
Curiously, the current license for PHP Markdown does not match at all (BSD 3 clause), so that makes my answer suspect. I also checked PHP Markdown classic and that license doesn't match either (I downloaded the distribution and checked the included license file). In fact, that license goes back 17 years.
I have no idea what the GPL license applies to.

it would be very useful that those files contain a reference to the GPL-2.0

The problem is that the testing framework doesn't allow for that. We have had a strict policy that we do not edit these files (same for the pl tests). If a specific test fails and we have no intention of making it pass, then we skip the test (as defined in tests/test_legacy.py), we do not edit it.
It is not clear what the right path forward is here.

Obviously, GPL 2.0 is not compatible with BSD, but I question whether it matters in this case. And what I mean by this, do we need to keep the license around for using the test cases as spec. Or does following a spec under a license make your code need to follow that license? I suspect not. Google copied the API of Java in Android 🤷🏻.
While I think we definitely should reference where the test cases came from, I question whether the Markdown test cases are something copyrightable as they are just Markdown prose. Do I have mark my documentation GPL 2.0 if I use <HTTP://WWW.SOMEURL.COM> in my docs because it was in a test? I doubt it.
I'm not sure what the best way to represent this is, but I don't think this is something that puts us in danger, but this may be misleading.

Obviously, text/prose is copywritable, but we are essentially monkeying the behavior of PHP Markdown , not copying their code.
Maybe for safety, we recopy the tests cases from PHP Markdown and throw out those that we don't want to follow and update the license? I guess you can re-evaluate which test cases have changed sense, and write your own that test the same thing...

Here's a discussion on the matter: https://opensource.stackexchange.com/questions/7503/implications-of-using-gpl-licenced-code-only-during-testing. Granted, I don't think anyone taking in that thread is a lawyer, and I'm not either. So, 🤷🏻.

Maybe for safety, we recopy the tests cases from PHP Markdown and throw out those that we don't want to follow and update the license? I guess you can re-evaluate which test cases have changed since, and write your own that test the same thing...

That is were I'm going with this. For that matter, maybe we don't even need any of the PHP tests at all. Back when those tests were new, PHP was one of the few leading Markdown implementations. Today, it is one of many. So unless there are some important tests in there that don't exist elsewhere, I'm inclined to just delete the entire directory. And if there are some important tests, then we should replace them with new tests using our new framework and delete the directory.

@waylan and @facelessuser thank you for picking this up.
The most clear solution would be to follow your proposal @waylan to delete the entire directory, if it is not of value anymore.

I should point out that according to test coverage reports, we are relying on the PHP and/or Perl tests for some stuff. With all tests run (1038 tests, 24 skipped) , the coverage report looks like this:
Name                                  Stmts   Miss  Cover   Missing
-------------------------------------------------------------------
markdown/__init__.py                      5      0   100%
markdown/__main__.py                     42      0   100%
markdown/__meta__.py                     14      0   100%
markdown/blockparser.py                  30      0   100%
markdown/blockprocessors.py             310      7    98%   214, 219-220, 229, 281, 629, 635
markdown/core.py                        151      0   100%
markdown/extensions/__init__.py          29      0   100%
markdown/extensions/abbr.py             102      7    93%   64, 177-178, 181-184
markdown/extensions/admonition.py        86      0   100%
markdown/extensions/attr_list.py         97      0   100%
markdown/extensions/codehilite.py       115     18    84%   152-173, 237
markdown/extensions/def_list.py          58      2    97%   105-106
markdown/extensions/extra.py              9      0   100%
markdown/extensions/fenced_code.py       98      1    99%   126
markdown/extensions/footnotes.py        227      6    97%   139-140, 160, 167, 398-399
markdown/extensions/legacy_attrs.py      23      0   100%
markdown/extensions/legacy_em.py         12      0   100%
markdown/extensions/md_in_html.py       207      0   100%
markdown/extensions/meta.py              43      0   100%
markdown/extensions/nl2br.py              8      0   100%
markdown/extensions/sane_lists.py        23      0   100%
markdown/extensions/smarty.py            86      0   100%
markdown/extensions/tables.py           145      0   100%
markdown/extensions/toc.py              256     14    95%   72-78, 84-96
markdown/extensions/wikilinks.py         48      0   100%
markdown/htmlparser.py                  139      0   100%
markdown/inlinepatterns.py              407      2    99%   175, 852
markdown/postprocessors.py               56      2    96%   140, 143
markdown/preprocessors.py                27      0   100%
markdown/serializers.py                  95      0   100%
markdown/treeprocessors.py              227      5    98%   52, 112, 240-242
markdown/util.py                        151      7    95%   108, 128-133, 256-261
-------------------------------------------------------------------
TOTAL                                  3326     71    98%

However, on a local machine which doesn't have all of the necessary dependencies installed and the PHP and Perl tests are automatically skipped (1038 tests, 132 skipped), the coverage report looks like this:
Name                                  Stmts   Miss  Cover   Missing
-------------------------------------------------------------------
markdown\__init__.py                      6      0   100%
markdown\__main__.py                     42      0   100%
markdown\__meta__.py                     14      0   100%
markdown\blockparser.py                  31      0   100%
markdown\blockprocessors.py             304      8    97%   214, 219-220, 229, 247, 281, 629, 635
markdown\core.py                        148      0   100%
markdown\extensions\__init__.py          29      1    97%   112
markdown\extensions\abbr.py             102      7    93%   64, 177-178, 181-184
markdown\extensions\admonition.py        87      0   100%
markdown\extensions\attr_list.py         98      0   100%
markdown\extensions\codehilite.py       115     97    16%   48-52, 116-133, 146-188, 212-247, 260-265, 269-288, 296-335, 339-343
markdown\extensions\def_list.py          57      2    96%   105-106
markdown\extensions\extra.py              9      0   100%
markdown\extensions\fenced_code.py       97     10    90%   90, 115, 120-137, 162, 176
markdown\extensions\footnotes.py        227      7    97%   139-140, 160, 167, 290, 398-399
markdown\extensions\legacy_attrs.py      24      0   100%
markdown\extensions\legacy_em.py         13      0   100%
markdown\extensions\md_in_html.py       208      0   100%
markdown\extensions\meta.py              44      0   100%
markdown\extensions\nl2br.py              9      0   100%
markdown\extensions\sane_lists.py        21      0   100%
markdown\extensions\smarty.py            86      0   100%
markdown\extensions\tables.py           145      0   100%
markdown\extensions\toc.py              256     14    95%   72-78, 84-96
markdown\extensions\wikilinks.py         48      0   100%
markdown\htmlparser.py                  140      0   100%
markdown\inlinepatterns.py              384      2    99%   175, 852
markdown\postprocessors.py               57      2    96%   140, 143
markdown\preprocessors.py                28      0   100%
markdown\serializers.py                  96      0   100%
markdown\treeprocessors.py              228      5    98%   52, 112, 240-242
markdown\util.py                        143      7    95%   106, 128-133, 256-261
-------------------------------------------------------------------
TOTAL                                  3296    162    95%

It is clear the those reports are different. It is possible that those differences are all attributed to the Perl tests and removing the PHP make no difference, but we would need to confirm that. If any of the PHP tests are contributing to coverage, those need to be identified and replicated in the new test framework before we delete them. For that matter, we should also be replicating the relevant Perl tests as well, but that is not necessary to remove the PHP tests.


CouchDB: add support for key that starts with underscore

Currently, we are not able to use keys that start with an underscore. e.g _somekey.
CouchDB will raise an error Only reserved document ids may start with underscore.
This is a big problem for us. For example in KMSimplementation we have something like this
base64.RawURLEncoding.EncodeToString(random.GetRandomBytes(uint32(keySetIDLength)))
Since _ (underscore) is a character that can be used for base64 there is no guarantee that _  will not be the first character.
Requirements:
The problem should be resolved in CouchDB implementation.

this is duplicate of #2137
currently the framework has base58Wrapper to fix this, it will be replaced with prefixWrapper soon (upcoming PR I'm working on)


fix typo.

What It Does 🔎
Fix a typo in docs.
Documentation Updates 📘
Yep.
How To Test ✔

N/A.
Linked Issues 🎫

Related Pull Requests 🔀

Thank you for that 🙏  Deploying as an hotfix !


System Error when running Ganache 2.5.4 on win32

I have installed 2.6x to test Filecoin feature but I had so many problem on returns and stability, effects the tests negatively then I decide to downgrade by uninstalling 2.6x then installing 2.5.4 but while opening new app reports this unfamiliar data which remains at C:\Program Files\WindowsApp folder which can make anyone crazy cos it's a mistery to get there and remove somewthing, it's like someone else's computer! I can't clean up the old app data in bloody Windows, so I can't use 2.5.4 stable version now. what do you suggest?

PLATFORM: win32
GANACHE VERSION: 2.5.4
EXCEPTION:
Error: Invalid flavor: filecoin
    at new WorkspaceSettings_WorkspaceSettings (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\types\settings\WorkspaceSettings.js:20:15)
    at new Workspace_Workspace (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\types\workspaces\Workspace.js:16:21)
    at flatMap (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\types\workspaces\WorkspaceManager.js:67:19)
    at Array.flatMap (<anonymous>)
    at WorkspaceManager_WorkspaceManager.enumerateWorkspaces (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\types\workspaces\WorkspaceManager.js:17:10)
    at WorkspaceManager_WorkspaceManager.bootstrap (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\types\workspaces\WorkspaceManager.js:76:10)
    at IpcMainImpl.<anonymous> (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\index.js:404:22)

Found one solution

I did backup the %APPDATA%\Ganache  folder
then removed it from there (C:\Users\<USERNAME>\AppData\Roaming\Ganache\workspaces)
started fresh-installed v2.5.4,

it works. ✔

stopped the app, copied one of my 'workspace' folder from my backup to the new %APPDATA%\Ganache folder which was generated by the fresh-installed v2.5.4
started the app,

I can see my old chain/workspace, works. ✔


1.29.0 Regression - CTRL + ` does not close terminal unless terminal is in focus

VSCode Version: 1.29.0
OS Version: Windows 10

Steps to Reproduce:

Open integrated terminal - I have it set to Git Bash in this case.
Place focus in the editor
CTRL + ` doesn't close terminal (this worked previously).
Put focus on terminal.
CTRL + ` closes the terminal.

Can reproduce.

https://github.com/Microsoft/vscode/issues/19400#issuecomment-423074454 ?

Yes this is by design


ItemStack.getNbt()

in my script I tried to use player.getHeldItem().getNbt(), but It can not find the function getNbt()
PS. player = getMinecraftClient().getPlayer()

Some of the clientscript features are only in the new beta, you can find these beta mods on my discord server: https://discord.gg/7R9SfktZxH


depriciated torch.utils.ffi used

Traceback (most recent call last):
File "build.py", line 3, in
from torch.utils.ffi import create_extension
File "/usr/local/lib/python3.6/dist-packages/torch/utils/ffi/init.py", line 1, in
raise ImportError("torch.utils.ffi is deprecated. Please use cpp extensions instead.")
ImportError: torch.utils.ffi is deprecated. Please use cpp extensions instead.
Traceback (most recent call last):
File "build_modulated.py", line 3, in
from torch.utils.ffi import create_extension
File "/usr/local/lib/python3.6/dist-packages/torch/utils/ffi/init.py", line 1, in
raise ImportError("torch.utils.ffi is deprecated. Please use cpp extensions instead.")
ImportError: torch.utils.ffi is deprecated. Please use cpp extensions instead.

Have you solved this problem？ @research-boy

@JamesZhan107 Not yet, let me know if you can help me with this

@JamesZhan107 Not yet, let me know if you can help me with this

https://github.com/ruinmessi/ASFF/tree/09f558863ae297663ec476f52a90c0cf6881415e/utils/DCN
This repo used dcnv2 and has modified the module. It can be complied.

If you guys are using more up to date pytorch, you can checkout the deformable conv module in mmdetection repo.

inputs = Variable(torch.randn(N, inC, inH, inW).cuda(), requires_grad=True) offset = conv(**inputs**) #offset = Variable(torch.randn(N, num_deformable_groups * 2 * kH * kW, inH, inW).cuda(), requires_grad=True) output = conv_offset2d(**inputs**, offset)
As is shown in test.py, Are these two inputs the same ？

‘’‘
’inputs = Variable(torch.randn(N, inC, inH, inW).cuda(), requires_grad=True)
offset = conv(inputs)
#offset = Variable(torch.randn(N, num_deformable_groups * 2 * kH * kW, inH, inW).cuda(), requires_grad=True)
output = conv_offset2d(inputs, offset)
‘’’
As is shown in test.py, are these two inputs the same?

‘’‘
inputs = Variable(torch.randn(N, inC, inH, inW).cuda(), requires_grad=True)
offset = conv(inputs)
#offset = Variable(torch.randn(N, num_deformable_groups * 2 * kH * kW, inH, inW).cuda(), requires_grad=True)
output = conv_offset2d(inputs, offset)
‘’’
As is shown in test.py, are these two inputs the same? @chengdazhi

Yep

Yep

OK, Thanks you !

Hi @chengdazhi,
I tried the DCN from here: https://github.com/chengdazhi/Deformable-Convolution-V2-PyTorch/tree/mmdetection/mmdet/ops/dcn. It still gives the depreciation error. I then went to the official mmdetection repo https://github.com/open-mmlab/mmdetection, but couldn't find the DCN module. Could you point me to the right location?
Thanks!


Update install instructions for Openstack VIM Driver

Update install instructions for Openstack VIM Driver

PR has been merged.


Facebook detection

This bug with android login is still not solved, so only works on pc.
Also facebook detects this man in the middle before and he dont log you to account, instead he warns you to change pwd if you typed it before.
Why gmail is removed?

Not really.


chore: use published or unpublished DTO based on input

Description

[!TIP]
Update the generateActionCollectionViewDTO to take into account the viewMode.

Fixes [Task - Server]: Update generateActionCollectionViewDTO to check for viewMode
Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!WARNING]
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8326337566
Commit: f2c67c2c3153c5240891be6216db174bcd22c5c1
Cypress dashboard url: Click here!
It seems like no tests ran 😔. We are not able to recognize it, please check workflow here.

Summary by CodeRabbit

New Features

Enhanced the application's ability to switch between published and unpublished collections based on the view mode, providing a more dynamic user experience.

/ok-to-test<EMAIL_ADDRESS>

/ok-to-test

/ok-to-test<EMAIL_ADDRESS>


Integrate Howdy Unlock

Description
Hi :)
Hyprlock is fantastic! I like it very much!
What would make it even more awesome would be if it would allow me to unlock with Howdy's Face-Unlock (much like the fprintd integration).
What would make it even more awesome would be the option to only allow Face-Unlock before the Machine suspends - I'd like to enter my password the first time after resume from suspend and then use face-unlock until I suspend my machine again. I think that would be a good compromise between security and ease of use. :)

Howdy already works when pressing enter. Or do you want hyprlock to automatically start howdy?

yeah - sorry I saw the posts after I created this one... Howdy stable didn't work for me - howdy-beta-git works - but it out unlocks even if I cover my camera... will tinker some more - and close this :)
thanks for the fast reply!

@fufexan I mean an option to automatically start howdy, just like what we can currently do with hyprlock, would be nice


Cant load image using URL while offline with cache - hits failure block every time (NSURLErrorDomain Code=-1009)

I am trying to allow an image to be loaded on a view controller that is loaded when the app is first opened. When I load the application with internet connection, the image loads fine.
If i run the app with no internet, the kf.setImage fails and the following is returned:

A URL session error happened. The underlying error: Error
Domain=NSURLErrorDomain Code=-1009 "The Internet connection appears to
be offline." UserInfo={_kCFStreamErrorCodeKey=50,
NSUnderlyingError=0x281639620 {Error Domain=kCFErrorDomainCFNetwork
Code=-1009 "(null)" UserInfo={_kCFStreamErrorCodeKey=50,
_kCFStreamErrorDomainKey=1}}, _NSURLErrorFailingURLSessionTaskErrorKey=LocalDataTask .<10>,
_NSURLErrorRelatedURLSessionTaskErrorKey=(
"LocalDataTask .<10>" ), NSLocalizedDescription=The Internet connection appears to be offline.,
NSErrorFailingURLStringKey=https://athlete-series-media-bucket-development.s3.eu-west-1.amazonaws.com/user/c6990e17-48cd-4197-870b-ca8236e48c69/avatar/c6990e17-48cd-4197-870b-ca8236e48c69.png,
NSErrorFailingURLKey=https://athlete-series-media-bucket-development.s3.eu-west-1.amazonaws.com/user/c6990e17-48cd-4197-870b-ca8236e48c69/avatar/c6990e17-48cd-4197-870b-ca8236e48c69.png,
_kCFStreamErrorDomainKey=1}

I have used this code for loading images offline before and used the code below and it works fine. If I terminate the app, switch internet off, reload the app, the picture should display. Currently, it just displays the placeholder.
Code to set UIImageView:
if let url = user.profileImage?.imageUrl {
                         let url = URL(string: url)
                         let processor = DownsamplingImageProcessor(size: self.userImg.bounds.size)
                             |> RoundCornerImageProcessor(cornerRadius: 20)
                         self.userImg.kf.indicatorType = .activity
                         self.userImg.kf.setImage(
                              with: url,
                              placeholder: UIImage(named: "Avatar"),
                              options: [
                                  .processor(processor),
                                  .scaleFactor(UIScreen.main.scale),
                                  .transition(.fade(1)),
                                  .cacheOriginalImage
                              ])
                          {
                              result in
                              switch result {
                              case .success(let value):
                                  print("Task done for: \(value.source.url?.absoluteString ?? "")")

                                 // From where the image was retrieved:
                                 // - .none - Just downloaded.
                                 // - .memory - Got from memory cache.
                                 // - .disk - Got from disk cache.

                                 // The source object which

                              case .failure(let error):
                                  print("Job failed: \(error.localizedDescription)")
                          }
                     }

The code enters the failure block when the app is ran with no internet. I have used this code on previous projects and it successfully displays with no internet, however something is wrong on this instance. Any help would be appreciated.
Reproduce
Open application with internet, image loads fine.
Close application & turn internet off.
Open application.
Note image does not display (only placeholder will)

.

I cannot reproduce this issue, or at least cannot reproduce it 100%. Did you try it on all your devices/simulators and are they all the same?
In Kingfisher, the download images will be sent to a disk cache. As its name indicates, it is just a cache version of the image. That means, there is no guarantee that these files are always there and prepared to use. Kingfisher puts these downloaded image data to your app's cache folder, which managed by the system. So besides the disk cache rules defined by Kingfisher (maximum size and expiration duration), the system also reserves the right to purge it at the proper time (that usually happens when you do not have enough storage on the disk). In this case, the iOS system would just delete these cached files without notifying your app or Kingfisher. The only way to retrieve them back is by loading from the Internet.
If you can reproduce it on all your devices, including on those devices with enough disk storage. That might be related to the cache key you are using. By default, Kingfisher determines the cache key by composing the URL and the processor it is using. If you are sure that the URLs are not changing during different app sessions, you can try to print out the processor.identifier before setting the image, to see if it is the same.

@onevcat I found the problem. When first loading online, the image is never cached / stored on the disk or memory.. When debugging, before loading the image initially, the cache memory storage contains 9 items, disk storage "maybeCached" is the same.
Is there a way to increase the default size for this so that when loading this image initially it can be stored? Or is that the upper limit reached for this cache system? Thanks

Yes. There is no size limitation of disk cache by default. And it (sizeLimit = 0) is also no limitation in according to your config.
I want to declare some fact (and implementation detail) for you:

The maybeCached is a shortcut to check image in disk cache, to reduce disk I/O. As its name suggested, it does not mean exactly the count of images in the cache.
If you are setting the break point and show the cache result in the completion handler for a single image, it is possible that you cannot get the correct result. Writing to disk cache happens in a background thread and the setting image completion handler would be called earlier than the image actually written to cache. To inspect a more correct result, try to add the .waitForCache option when setting the image.
For debugging purpose, you can try to use the imageCachedType(forKey:processorIdentifier:) method to check the cache state of an image. As my last reply, I strongly suspect that you are having a different URL or processor identifier every time so the cache is not retrieved. So please make sure to compare the URL and processors to make sure the cache keys are the same on each load for the image.

if let url = user.profileImage?.imageUrl {
    let url = URL(string: url)
    let processor = DownsamplingImageProcessor(size: self.userImg.bounds.size)
        |> RoundCornerImageProcessor(cornerRadius: 20)
    print("URL: \(url.absoluteString)")
    print("Processor Id: \(processor.identifier)")

If these are not identical, it means you are not trying to seek the image under the same key.
And to confirm the cache type of an image:
ImageCache.default.imageCachedType(forKey: url.absoluteString, processorIdentifier: processor.identifier)

Before you load it for the first time, it should be .none. After loading, it should be .memory. And after a reboot of the app (and before loading it again), it should be .disk.

For adjust the cache config (although I don't think it is your case), check the wiki page below, and all related classes.
https://github.com/onevcat/Kingfisher/wiki/Cheat-Sheet#set-limit-for-cache

@onevcat I can narrow it down to the limiting issue because this cache implementation with the user profile image works fine on fresh/new accounts on the application, I.E. accounts with not much "Sessions" with images stored on the cache..
I debugged thoroughly and the processor ID & url absolute string are the same each time. I added the .waitForCache option and when confirming the type of cache for the image using
ImageCache.default.imageCachedType(forKey: url.absoluteString, processorIdentifier: processor.identifier)

On first initial load, before setting image, cacheType is none
In completion block of setImage, cacheType is memory
After reboot of app, before setting image again, cacheType is none again
In completion block of loading the second time, cacheType is memory

**Note that, sometimes after rebooting of the app, within the completion block of setting the image successfully, cacheType is occasionally none
And so on.. It never reaches the disk, and even sometimes fails to write to memory -> which I am attributing to storage issues..
Therefore, how would you go about debugging / attempting to increase the size of memory / disk cache, beyond the default no limit that is configured automatically?
Thanks

Hmmm,
That is interesting. I tried exactly your code, and it runs well and I can get the desired result:

The cache itself is totally optional when setting the image, so it does not give back the error directly. So if you want to debug this, you may need to set a breakpoint on the line to see if there is a disk error or not. It is easy if you are using CocoaPods or SPM, but might be tricky if you are using Carthage.
https://github.com/onevcat/Kingfisher/blob/master/Sources/Cache/ImageCache.swift#L398
Another (maybe) easier way would be manually calling the store method, and check the cache result which it returns. Maybe do it in the setting image completion handler:
imageView.kf.setImage(/*...*/) { result in
    switch result {
    case .success(let value):
        ImageCache.default.store(value.image, forKey: url.absoluteString, options: .init([.processor(processor)])) { r in
            print(r)
        }
    //...
}

to check if both the memory cache and disk cache is giving you the .success result.
By the way, can I get one of the image URL or the image itself which you are using? I want to also make sure it is not related to some of the data issues or so.

By default, the memory cache size limit is set to 25% of your device total memory, and the disk cache can take unlimited space. So it should be a really rare case that your disk cache fails unless the device is about to be out of disk storage itself.
Adjusting the config like that might not help IMO.

@onevcat Yes, that code works for me also, just not on this one specific account / instance of the iOS application, due to what I believe is being caused by the number of stored images in the cache already due to synced "sessions"..
Using
ImageCache.default.store(value.image, forKey: url.absoluteString, options: .init([.processor(processor)])) { r in
            print(r)
        }

In the success block returns:

On first initial load, r = **CacheStoreResult(memoryCacheResult: Swift.Result<(), Swift.Never>.success(), diskCacheResult: Swift.Result<(), Kingfisher.KingfisherError>.success())**

In completion block, **cacheType is **none****

After reboot of app, r = **CacheStoreResult(memoryCacheResult: Swift.Result<(), Swift.Never>.success(), diskCacheResult: Swift.Result<(), Kingfisher.KingfisherError>.success())**

In completion block of loading the second time, cacheType is memory

After reboot of app the third time, r = CacheStoreResult(memoryCacheResult: Swift.Result<(), Swift.Never>.success(), diskCacheResult: Swift.Result<(), Kingfisher.KingfisherError>.success())

cacheType is memory

Image being used can be seen here, although like I said before, this functionality works fine on this application when I login with an account that has no previous cached images (i.e. no synced session) ->
I have attempted this on numerous devices with 10GB+ free and even on fresh iOS simulators and the same thing occurs. The issue definitely lies within the limit on the cache / the image not being stored in memory/disk to enable it to be loaded offline, due to some sort of restriction / constraint / blocker..
Do you have any other recommendations on ways to debug? It is clear from that "r" result that it is never being cached?
Thanks

No idea about what is happening now.
I need a reproducible sample to dig into but currently, there seems to be no way for me to reproduce it, so it is quite hard for me to say what can be wrong.
The image seems quite normal and can be cached on disk without a problem. So it is definitely related to a certain use case.
You mentioned some "no synced sessions" and "synced sessions". What do you mean by that?
Or is it possible that you can create a minimal sample project in which this can be reproduced? So I can start to look at it sooner.
Thanks!

@onevcat
What could you take from the cache store results? I.E. the print statements of r? It is clearly failing to save here right?
Basically the application is for syncing sport sessions, and viewing the sessions of that. Therefore, an account with many sessions, each having an image attached to them, will have more cache used up.
When I test with a fresh account, I.E. no session images or only a few session images taking up space in the cache, it functions fine as it should. Saves to disk and then works offline. However, when I test with an account that has many sessions (I.E. 30+), each having a session image stored in the cache, the cache saving function does not work.
I may be able to produce a sample for you to see the problem but I will have another play about with config settings / freeing up some storage in the cache / not storing all session images here and see if I can resolve this..
Thanks for looking into it!

What could you take from the cache store results? I.E. the print statements of r? It is clearly failing to save here right?

If failing, the ImageCacheResult value will contain the error information. But in your case, it is always printing success, so the disk cache was done without a problem.
For the session, I now know what you were talking about. If you are on the simulator which can also reproduce it, I guess you can just open the Cache folder of the app (You can find the device on your mac in the folder of ImageCache.default.diskStorage.directoryURL). Open it with Finder, and inspect how the disk cache file changes. Maybe there is a case that the cache gets purged by the system.
After all, as my first reply indicated, the cache DOES NOT guarantee the persistence of images (as the AppStore review guideline, these content which can be redownloaded should be put as cache), and your logic should not rely on the images always existing.
If you can ensure that the issue is due to the purging behavior of the system, but you still need the offline access, a possible way might be using this init method to create your own cache in the Document folder, and then assign it to the KingfisherManager.default.cache for future use. But I personally do not recommend doing so, which leads to a risk of rejecting in the review.

@onevcat I was eventually able to find the issue.. The cached images were being wiped after being stored initially as there was a call to cache.clearMemoryCache() and cache.clearDiskCache() hidden deep within the code.. This was a project I inherited so apologies for the misunderstandings!
Thanks for all your help, the library is great! This can be closed now.

Never mind! Glad to hear that you have found what is going on!


Update etcdjs version and bump major version

etcdjs introduces a breaking change, where options.refresh defaults to false.

Released in 2.0.0


[alembic:x64-osx] build failure

Host Environment

OS: [osx]
Compiler: AppleClang <IP_ADDRESS>30032

To Reproduce
Steps to reproduce the behavior:
./vcpkg install alembic
Failure logs
Call Stack (most recent call first):
scripts/cmake/vcpkg_build_cmake.cmake:91 (vcpkg_execute_build_process)
scripts/cmake/vcpkg_install_cmake.cmake:24 (vcpkg_build_cmake)
ports/alembic/portfile.cmake:36 (vcpkg_install_cmake)
scripts/ports.cmake:79 (include)
Error: Building package alembic:x64-osx failed with: BUILD_FAILED
-- Note: alembic only supports dynamic library linkage. Building dynamic library.
-- Downloading https://github.com/alembic/alembic/archive/cfe114639ef7ad084d61e71ab86a17e708d838ae.tar.gz...
-- Extracting source /Users/DMotor/Documents/dev/vcpkg/vcpkg/downloads/alembic-alembic-cfe114639ef7ad084d61e71ab86a17e708d838ae.tar.gz
-- Applying patch fix-find-openexr-ilmbase.patch
-- Using source at /Users/DMotor/Documents/dev/vcpkg/vcpkg/buildtrees/alembic/src/e708d838ae-dc6119ffed.clean
-- Configuring x64-osx-dbg
-- Configuring x64-osx-rel
-- Building x64-osx-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:139 (message):
Command failed: /Users/DMotor/Documents/dev/vcpkg/vcpkg/downloads/tools/cmake-3.17.2-osx/cmake-3.17.2-Darwin-x86_64/CMake.app/Contents/bin/cmake --build . --config Debug --target install -- -v

Duplicated to https://github.com/microsoft/vcpkg/issues/12830, which has build logs.


NuGet packages being reported as updated despite not existing in the commit

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
nuget
Package manager version
No response
Language version
No response
Manifest location and content before the Dependabot update

https://github.com/xt0rted/dotnet-startup-projects/pull/134
https://github.com/xt0rted/dotnet-startup-projects/pull/135
https://github.com/xt0rted/dotnet-startup-projects/network/updates/762957402

Here's more in a different repo

https://github.com/xt0rted/dotnet-rimraf/pull/268
https://github.com/xt0rted/dotnet-rimraf/pull/270
https://github.com/xt0rted/dotnet-rimraf/pull/271

dependabot.yml content
No response
Updated dependency
No response
What you expected to see, versus what you actually saw
There should have been 4 packages updated in the first PR, and 5 in the second, which the title & body of the PR says, but there's only 1 package updated in each. The same is true in other repos, both public and private. It seems to be any nuget update group with more than 1 package update results in only 1 package being updated while the title & body have the correct information.
I have dependabot logs in all my repos that have a TON of errors about being run on a read-only file system but I'm not sure if that's related to this issue. Both things seem to only be happening for nuget packages though.
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

The same thing started to happen in my private repo since last week, which has group update configured. It only updates one dependency in each group.
For me, there are no errors about read-only file system though, it only has errors like
  proxy | 2023/12/28 16:22:02 [377] 400 https://dc.services.visualstudio.com:443/v2/track
  proxy | 2023/12/28 16:22:02 [377] Remote response: {"itemsReceived":1,"itemsAccepted":0,"errors":[{"index":0,"statusCode":400,"message":"103: Field 'time' on type 'Envelope' is older than the allowed min date. Expected: now - 172800000ms, Actual: now - 612414003ms"}]}

I just noticed a couple of the PRs we got at work this week are not only not updating all the packages the PR says it is, but they're also duplicating the packages in the PR description.

We are also experiencing this (in a private repo). For us, the PR description is correct (no duplicates), but it only actually bumps the version of the last dependency listed.

Similar / same error pattern as #8576

also seeing this across multiple repos

This is a duplicate of https://github.com/dependabot/dependabot-core/issues/8576


Handle Column groups

Columns and their term source references and/or associated other columns (such as units) have to be grouped or at least created as groups when inserting them.
Current plan:

[ ] Create Term Source REF and  Term Accession Number Columns for every annotation column
[ ] Make these columns Hidden and read-only

i just noticed this issue and this was implemented with PR #34, but the additonal columns are not read-only yet. Will add this with the next PR.

Hidden should be enough


[DNM] modules: Add the Arduino Zephyr API

Add the Arduino Core API for zephyr as an optional module

I don't know why it's not assigning any reviewers, hence tagging manually
CC @nashif  @carlescufi @soburi

@DhruvaG2000
First of all, I think it's too early since there is no SPI implementation yet.
SPI should be included when writing a device overlay, but I'm not in a position to test it.
(Most of all, most people who use it will think SPI works!)
This should be implemented soon.
Also, as I previously commented, I think there will be a need to discuss the GPL license. I think, ultimately, it will need to be agreed upon by TSC, but the PR needs to include the information necessary to make the problem known.

Hi @soburi ,

SPI should be included when writing a device overlay,

Yes, I agree to those points, but as you can see in this PR I am not adding any code as such to upstream. This is just an effort to ensure that people are able to seamlessly pull in the arduino core module after they've cloned zephyr without having to dig around much or needing to make manual changes to the main yml file just to pull in the arduino module.
I have referred to the Documentation here: https://docs.zephyrproject.org/latest/contribute/external.html#integration-as-optional-modules
It states:

Optional projects that provide value to users directly and through a Zephyr subsystem or platform shall be added to an optional manifest file that is filtered by default. (submanifests/optional.yml).

It was my understanding that the Arduino core API is one such optional project directly under zephyr and users of zephyr rtos should be made aware within optional.yml if atall they're interested to pull it in and use it (Ofcourse subject to limitations of whatever the project supports at that point of time)
The other option is also this: https://docs.zephyrproject.org/latest/contribute/external.html#integration-as-external-modules
If "as external modules" sounds better to everyone I am open to that as well, let me know. It is to be noted that

I think there will be a need to discuss the GPL license

This too has been a long standing issue, and I get the part that integrating code in a single repo can be an issue, but in this PR I plan on bringing no such code directly inside zephyr-rtos anywhere. This is still completely external as a lib/module.
What I should probably do now is create the issue as told in the documentation https://github.com/zephyrproject-rtos/zephyr/issues/new?assignees=&labels=RFC&template=007_ext-source.md&title= to get reviewed by the Technical Steering Committee (TSC) .

Hi, I think there needs to be a maintainer entry for this module.

Good catch, I think I am also missing the changes to be made in modules/ like Kconfig and CMakelists atleast (if I were to look at for eg. https://github.com/zephyrproject-rtos/zephyr/tree/main/modules/lz4 )
But I will let the TSC comment on the issue first and see if they're okay with the idea in the first place.

@DhruvaG2000 please follow the instructions here:
https://docs.zephyrproject.org/latest/contribute/external.html
although in this case it may be considered an "internal" project, but it still needs discussion in the TSC

@DhruvaG2000 @carlescufi
The premise of my opinion is that if we are going to merge features, they need to be easily available and useful enough for general users.
I believe that there are still issues to be resolved at present.
Me and @DhruvaG2000 may have different opinions on this point.
It is true that the module is not part of the GPL, but the method of "downloading GPL part at the user's responsibility" is incomplete, and it is necessary to think about how to properly incorporate it into the module.
(In other words, We need to seriously think about how to handle GPL within your zephyr structure.)
Dialogue with TSC is necessary to resolve this issue, and it is best to start this dialogue early. I think it's great that you created this issue.
I believe there are two issues that need to be resolved as soon as possible. I think these issues need to be discussed in parallel with licensing issues for the merge.

Developing user usage methods
It is necessary to be able to easily use this function from west command.
(As an example of integration, I considered a method using the shield structure. Currently, snippet can be used, so I think it is preferable to use it. https://github.com/zephyrproject-rtos/zephyr/pull/64051)

Implementation of required APIs (As I mention in above)

Digital (GPIO), Analog (ADC), PWM, and I2C are currently being implemented. Since SPI requires changes to dts overlay, I think it need to implement this before publishing.

Any updates on this? I am using this for MicroBlocks support for Zephyr and would love to see it become an official module.
@soburi SPI library support PR is open now: https://github.com/zephyrproject-rtos/gsoc-2022-arduino-core/pull/103 . Please provide feedback

Just to provide an update, SPI and all the other Arduino APIs required for MicroBlocks to work have been merged.


use ansible to deploy ibm blockchain using OpenShift cluster

Hello,
I am following  these documentations :

OpenShift Configuration on AWS.
Deploy IBM blockchain using ansible.

All command ran successfully without any error. but the routes for ibm blockchain platfomr  keeps loading forever

All pods are working

I  am unable to trace the error, my questions are:

How to solve the route issue?
How to trace the error?
Would reloading and installing IBM Blockchain platform would solve the issue?
Appreciate your replay.

@yehia67 please can you run the following command in the ibm-platform project: oc get ibpconsoles,ibpcas,ibppeers,ibporderers,pods,deployments,services,routes,pvc

@yehia67 thanks, the console appears to be running, so suspect that Ansible cannot access it for some reason. Please run this command now: curl -kvvv https://ibmconsole-ibp-console-console.awa.valoro.xyz

I got this output
* Rebuilt URL to: https://ibmconsole-ibp-console-console.awa.valoro.xyz/
* Could not resolve host: ibmconsole-ibp-console-console.awa.valoro.xyz
* Closing connection 0
curl: (6) Could not resolve host: ibmconsole-ibp-console-console.awa.valoro.xyz

I tried to change url to curl -kvvv https://ibmconsole-ibp-console.awa.valoro.xyz
but it gave me the same output

@yehia67 okay, that's what I expected. The system running Ansible cannot resolve DNS entries that should map to your Red Hat OpenShift cluster. Is awa.valoro.xyz definitely the right domain to use? Do any *.awa.valoro.xyz hostnames resolve correctly on this system?

Yes the Openshift dashboard resolve but with unsecured connection

@sstone1  is following this guide could solve my issue

We don't have any ssl on our cluster could this be the problem? is cert-manager a good solution for this>

@yehia67 this is a DNS problem; your system cannot resolve the hostname ibmconsole-ibp-console-console.awa.valoro.xyz to the IP address of your OpenShift cluster. What is the hostname of the OpenShift dashboard?
I don't believe it's possible to have an OpenShift cluster without SSL. You also do not need to enable SSL passthrough on OpenShift, only Kubernetes.


Cursor behaves strange

Hi, i just had the following bug but after closing and re-opening the file it is gone... I still wanted to report it, maybe somebody else also saw it. I never saw this before 0.206.0.

Since its not reproducible, i haven't tested it safemode.
My custom packages:

linter
linter-rubocop
color-picker
file-icons
pigments

I can reproduce this by the following actions:

Split screen
Move the cursor to line number bar (or somewhere in the editor where the cursor changes)
Move fast and click to another window
Finally, the cursor won't be updated

Packages:

linter
file-icons

Cool that im not the only one. For me it happened without split screen.
/cc @as-cii
Could this be (in some way) related to the tiling stuff? :)

Hmmm, seems you can't reproduce it by the steps before the editor not starts to bug.
So the only clean workaround is to reopen the file.

@philipgiuliani: yep, this regression was likely generated by the new tiling mechanism. I am truly sorry this is causing headaches to you guys, I am already investigating what is causing the issue.
Unfortunately, however, I am not able to reproduce it locally (I had no luck with the steps provided by @textedit). I am afraid this may have been caused by a serialized state which doesn't play well with tiling, but I have no way to confirm that.
I'd be super grateful if you guys could help me to reproduce this issue consistently, so that I can, hopefully, find and fix the bug. Thank you!

@as-cii No problem, it really happened only once. Maybe its just with the serialized state of 0.205.0? It happened when i opened atom for the first time today (after the update). After restarting it now serval times and trying a lot to reproduce it, im not able to.. :(
I also can't reproduce it with the steps of @textedit
I'll report it here when i find something.

I have edited my comment.
Atom is running with the version 0.205.0 and i think i have the issue since then.

@as-cii This could also be due to my PR that landed in 0.205. I have not yet been able to reproduce this.

I haven't seen this so far for the last days... Maybe it was really an upgrade issue?

I haven't seen this so far for the last days... Maybe it was really an upgrade issue?

@philipgiuliani: yeah, I suspect that the issues you and @textedit were having weren't quite the same and I have a gut feeling that yours might have been related to some corrupted status.
If @textedit and @maxbrunsfeld agree, I'd say to close this for now. Thoughts? :thought_balloon:

I think we should close this if nobody's experiencing it currently. We can always reopen it.


Using Valid Count to Compute Mean/StdDev in SPDTI

In Shifu 0.2.6, total count is used for mean. Try to use valid count (total count - missing value count).

Done in 0.2.7 branch


Adjust elapsed_time type in UserResponse struct to be a float representing milliseconds

This is a time.Duration in our Results struct(1), but we represent this as a string to the user(2). We've received a request to represent this as a JSON number, and more specifically, milliseconds as a float.
(1): https://github.com/redhat-openshift-ecosystem/openshift-preflight/blob/main/certification/runtime/runtime.go#L20
(2): https://github.com/redhat-openshift-ecosystem/openshift-preflight/blob/ff73edd14f29ba48379feb8ce760886fafda7183/certification/formatters/util.go#L81-L89

/assign edreyer-1


[EDA] - HH & Admin Account Naming Format is Incorrect for Spanish & French

Do you want to request a feature or report a bug? : BUG

What is the current behavior? : When the running User's language is in Spanish or in French, the "Administrative Account Name Format" and "Household Account Name Format" is duplicating some words.

What is the expected behavior? : The Name Formats follow the convention (and do not duplicate words.)

What is the motivation / use case for changing the behavior? : User experience

Steps to Reproduce :

Change running user language to French (Setup > Users > Edit > Change Language > Save.)
Go to Parameters EDA > Systeme > Modifier > [This should change the drop-downs at the bottom to Compte administratif {!LastName}] > Enregistrer
Create a Contact "Fox Mulder"
Look at the Account Name

Expected Result : Compte administratif Mulder
Actual Result : Compte administratif Compte administratif Mulder

Go to Parameters EDA > Systeme > Modifier > Change to HH Account Model & Change Naming Format to Foyer {!{!FirstName}} {!LastName}

Expected Result : Foyer Dana Scully
Actual Result : Foyer Foyer Dana Scully
Note that this also happens in Spanish. It does not happen in English.

**lurch: add

Tracking W-038618


Missing dot background color value?

I want to create this moc but I can't find how to change the dots to be with background color and using FixedTimeline.tileBuilder(
This is the moc

The dots have a border color and different background color.
This is the closest that I have reached

Values that I think are missing:

Background color for outlined indicator (for border like effect).
Option to control the number of lines in the dashedLine.
A way to choose the currently selected position in the timeline and automatically all outlined indicators before that will have selected properties (like a purple background) and the rest will have different indicator properties (like a white background).

So in my pr I have added the option to insert a widget as the dot indicator, this way we can insert everything that we want without limitation.


texts from describe and it blocks are not printed when using browser nightmare

Hi, after update to meteor from 1.8.1 to 1.10.2 and updates:
<EMAIL_ADDRESS>-><EMAIL_ADDRESS>and internal update:
<EMAIL_ADDRESS><EMAIL_ADDRESS>The output of the client-side tests is not visible if TEST_BROWSER_DRIVER=nightmare
I was able to reproduce this issue in this repository if you do meteor npm t
https://github.com/jirikrepl/meteortesting-mocha-client-reporter-bug-reproduction
Here's the screenshot from the terminal output:
My test file runs both on the server and client-side. You can see a lot of prints on the server-side ('some test', 'can do something'), but only %s%s on client-side:
The error and warnings:
[ERROR] There is no route for the path: /
%cElectron Security Warning (Insecure Resources)
%cElectron Security Warning (Insecure Content-Security-Policy)

shouldn't be related to the reporter "print" issue. In my app, I don't see these errors, but still cannot see the content of "describe" and "it" block strings

Thanks for the detailed report. Please run the tests again using Puppeteer (https://github.com/meteortesting/meteor-browser-tests#puppeteer). If that works, this bug needs to be reported to the https://github.com/meteortesting/meteor-browser-tests repository.

Great! I did npm i --save-dev puppeteer and added TEST_BROWSER_DRIVER=puppeteer to my package.json test scripts. Client reporter string are now back and tests are passing. Thanks!

Please test with the latest version. meteortesting:browser-tests has just been released as 1.3.4 which should fix this bug. If it doesn't, please open a ticket in https://github.com/meteortesting/meteor-browser-tests.


Make IP Lease available to deployment through ENV variable

Problem Statement
Akash Network supports leasing IP addresses from providers that offer them but the leased IP is not easily accessible by the deployment. Having access to the IP lease from the deployment is important in cases where the deployment needs to use the IP address for communication once the application comes up. A good example of this is Kadena Nodes that need this info for communication.
Workaround
The current workaround (in case of Kadena) is to read the IP address manually and build the application (Kadena node) by manually inserting the IP address at build time.
Suggested Solution
The suggested solution is to improve user experience for tenants by exposing the IP lease to the deployment through an environment variable.

Hey guys, I started to read through the ipOperator code and cluser code. Giving the context I currently see this place where I can potentially add the new env variable for the IP.
https://github.com/akash-network/provider/blob/e7aa0b5b81957a130f1dc584f335c6f9e41db6b1/cluster/kube/builder/workload.go#L99
If this is the correct place, then what I'll need to figure out is to find a way to get the IP operators data, specifically the IP address string in the context of this function, so I can add it as part of the params to the addEnvVarsForDeployment .
Does the workload object has any way to currently access this data? If you have any specific files that you suggest me to read more in detail I'll happily go though them.
Thanks a lot :)

Thanks a lot @boz . I'll get back to you if I have any further questions.🙏

Some related information regarding updating env variables on an already running deployment. Thanks to @andy108369 for sharing
Working POC to change a configmap without triggering a redeploy:
apiVersion: v1
kind: ConfigMap
metadata:
  name: cm1
data:
  VAR1: "value123"
---
apiVersion: v1
kind: Pod
metadata:
  name: pod1
spec:
  containers:
  - name: container1
    image: nginx
    volumeMounts:
    - name: cm1-vol
      mountPath: /etc/config
  volumes:
  - name: cm1-vol
    configMap:
      name: cm1

Shell excution:
root@node1:~# kubectl get cm cm1 -o json | jq -r '.data'
{
  "VAR1": "value123"
}

root@node1:~# kubectl exec -ti pod1 -- sh -c "cat /etc/config/VAR1" ; echo
value123

root@node1:~# kubectl edit cm cm1
configmap/cm1 edited

root@node1:~# kubectl get cm cm1 -o json | jq -r '.data'
{
  "VAR1": "value456"
}

root@node1:~# kubectl exec -ti pod1 -- sh -c "cat /etc/config/VAR1" ; echo
value123

root@node1:~# sleep 1m

root@node1:~# kubectl exec -ti pod1 -- sh -c "cat /etc/config/VAR1" ; echo
value456

In terms of where to add the configmap creation, based on what I read so far, I believe we can add it on the handling of the events:
ctypes.ProviderResourceAdd, ctypes.ProviderResourceUpdate:

https://github.com/akash-network/provider/blob/e7aa0b5b81957a130f1dc584f335c6f9e41db6b1/operator/ipoperator/ip_operator.go#L237
I see we have access to the kube client from the ipoperator so probably the way we could implement this is potentially calling the kube client to create the configmap.
A couple questions I have:

The event has the service name available, however I believe we might need to attach the configmap to the deployments which are using the requested IP lease. Is this something we need to do beforehand, and is would it be OK to create a "default" configmap when creating deployments that require an IP lease?
If the configmap still does not exists, what would be the best place to create and attach this confimap as a volume? On the deployment creation?
Does the kube client offer any functionality as of now to update a configmap?

Thanks for the help :)

Is it possible to not support updating the value for now?

the schedulingGates that have been mentioned during today's call https://kubernetes.io/docs/concepts/scheduling-eviction/pod-scheduling-readiness/
and the readinessProbe https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/
my idea with the readinessProbe is that we could use something like this:
readinessProbe:
  exec:
    command:
    - <a shell command that would make a query against the akash-service to check whether the IP has been allocated for this particular deployment>
  initialDelaySeconds: 5
  periodSeconds: 5

From discussions on call:
Possible Options for Solving this

One option could be waiting for the IP address before starting the Pod.

Implement and endpoint that returns "metadata" about the workload. Some concerns on permissions, and getting lease information from other leases that are not. The endpoint can talk directly to kube, probably a sidecar.

Using a readinnes probe, you can specific a simple query on the endpoint.

Using a configmap to update the value of the ip addres

New kuberenetes version 1.27. Has a new feature called schedule gate. Similar to a readiness probe.

Notes and Discussion

If we use schedule gate, still going to close the lease if IP address location takes longer that the healthchecks.

To be able to wait for the IP we'll need to make the healthchecks time a bit longer since IP addresses can take several minutes to be available.

Having a service that you could query would also solve other issues like monitoring. But having a query service will require dealing with authentication and authorization with K8s, and also opens vector of attacks for the providers' cluster + we are using extra resources from the provider

Proposed solution for now

Deployment manager needs to have an extra parameter called ip address for example.
The provider monitor then extends the time for the lease to get the IP addres and wail until the addres is ready if it sees these parameters describe on #1 on the workload.
We also need to somehow show it in the status of the deployment, that the lease is waiting for an IP and not unhealthy.

Hey everybody. I'm continuing the fix for the issue of the env variables adition. Based on previous conversations. I wanted to validate if this plan makes sense to you all of you as this is my first time touching the codebase and I want to make sure I'm understanding the code structure.

What we need to do is to wait until the IPOperator has the IP Lease ready for a given OrderID. For that we can use the  GetIPAddressStatus  located at ip_operator_client.go in the cluster folder.

func (ipoc *ipOperatorClient) GetIPAddressStatus(ctx context.Context, orderID mtypes.OrderID) ([]ipoptypes.LeaseIPStatus, error) {
	path := fmt.Sprintf("ip-lease-status/%s/%d/%d/%d", orderID.GetOwner(), orderID.GetDSeq(), orderID.GetGSeq(), orderID.GetOSeq())
	req, err := ipoc.newRequest(ctx, http.MethodGet, path, nil)
	if err != nil {
		return nil, err
	}

	ipoc.log.Debug("asking for IP address status", "method", req.Method, "url", req.URL)
	response, err := ipoc.client.DoRequest(req)
	if err != nil {
		return nil, err
	}
	ipoc.log.Debug("ip address status request result", "status", response.StatusCode)

	if response.StatusCode == http.StatusNoContent {
		return nil, nil // No data for this lease
	}

	if response.StatusCode != http.StatusOK {
		return nil, extractRemoteError(response)
	}

	var result []ipoptypes.LeaseIPStatus

	decoder := json.NewDecoder(response.Body)
	err = decoder.Decode(&result)
	if err != nil {
		return nil, err
	}

	return result, nil
}

I can use a waitgroup until the status for the IP is ready. (ie no errors on the above function right?)
2.Once that happens I can proceed with the deployment as normal. The waiting logic will be placed on the Deploy() function inside cluster/kube/client.go.

All of the above needs to be conditioned on an attribute in the SDL. I planning to name this attribute something like waitForIps.

One the wait logic is finished, I can add  the logic to inject the Ip Addresses on the container() function in the kube/builder/workload.go file so that it can be addded when the addEnvVarsForDeployment() is called. At this point, I should have the guarantees that the IP addresses from the ipOperator are already available so I can use the same function GetIPAddressStatus to get the actual IP address string.

Some questions:

Is there anything wrong with the above plan? Are there any considerations I need to keep in mind when implementing this?
How can I access the attributes flag from inside the cluster manager in cluser/manager.go I see the deploymentManager struct has a mgroup *manifest.Group but I'm not quite sure on what section of the SDL would be more appropriate to add this new attribute.
The GetIPAddressStatus() function returns a LeaseIPStatus array. However looking at the LeaseIPStatus struct, I don't quite see how to identify if the IP is ready or not. Can you help me understand how to determine if the IP is ready?

Thanks again for the help.

@boz could you please review @PJEstrada questions above at your convenience.

Hi @PJEstrada, great write-up!  Sorry for the delay.
In general, we try to avoid blocking.  An event-based approach would be preferable.

Emit an event when the IP Operator has an IP ready.
You can listen for these events in cluster/service.go and pass the event to the manager as needed from there.
Don't start the deployment right away if it should wait for an IP.

It's not clear to me why you need to add a new attribute - shouldn't all leases that have an IP wait for the IP to deploy?

The GetIPAddressStatus() function returns a LeaseIPStatus array. However looking at the LeaseIPStatus struct, I don't quite see how to identify if the IP is ready or not. Can you help me understand how to determine if the IP is ready?

It looks to me like that function only returns LeaseIPStatuses that are ready to use.
Again, sorry for the delay.  I'll try to watch this more closely.

Hi @PJEstrada, great write-up! Sorry for the delay.
In general, we try to avoid blocking. An event-based approach would be preferable.

Emit an event when the IP Operator has an IP ready.
You can listen for these events in cluster/service.go and pass the event to the manager as needed from there.
Don't start the deployment right away if it should wait for an IP.

It's not clear to me why you need to add a new attribute - shouldn't all leases that have an IP wait for the IP to deploy?

The GetIPAddressStatus() function returns a LeaseIPStatus array. However looking at the LeaseIPStatus struct, I don't quite see how to identify if the IP is ready or not. Can you help me understand how to determine if the IP is ready?

It looks to me like that function only returns LeaseIPStatuses that are ready to use.
Again, sorry for the delay. I'll try to watch this more closely.

Hi @boz  thanks a lot for your comments! I'll keep them in mind. A couple followup questions:

What is the recommended mechanism to emit events and read events on the provider codebase? Can you point me to some examples on where an event is emmited and readed?

Thanks again!

Sure, here's a few:

Publishing that a lease was won
Manifest received
Publishing the status of a deployment

@PJEstrada - checking in here. Are you still working on this issue?

Removing @PJEstrada from assignee since he doesn't seem to be working on this

@sfxworks - given that you worked on a couple smaller PRs for provider - is this something you're interested in working on?

@sfxworks - given that you worked on a couple smaller PRs for provider - is this something you're interested in working on?

Good morning,
Sure, ill take a look at this this week. Thanks!

@anilmurty @sfxworks let's pause work on this until the following is resolved.

this issue alone brings up post-configuration of the deployment and there could be other things along with lease IP to be added post-deployment.
The issue itself needs a deep understanding of the codebase and interactions of various parts between each other.
The refactor of the deploy service is coming next month and it will undoubtedly break implementation.
The provider-services split is not yet finished, which is definitive for features like this, as split will define communication API between various operators and services.

The best temporary solution is to add detection of external IP into the container's entrypoint. this solution has quirks, but it is still better than nothing.
# ensure curl is installed
AKASH_LEASE_IP=$(curl ifconfig.me)


Mount fails in Debian 10

Works on Ubuntu 20.04.6 5.4.0-152-generic
Works on Ubuntu 23.04 6.2.0-23-generic
Works on Rocky 9 5.14.0-284.11.1.el9_2
Fails on Debian 10 5.10.0-21-amd64
Fails on Debian 10 4.19.0-24-amd64
Fails on Debian 11 5.10.0-23-amd64
Works on Debian 12 6.1.0-9-amd64
Note: On debian 10, we needed to do sysctl -w kernel.unprivileged_userns_clone=1 to let unshare --user work. We do not have to do that for debian 11 or 12.

Changed the repo on the Debian 11 machine to testing, upgraded Linux to 6.1.0-9-amd64, seems to have fixed it.

Fails on Centos 8 Stream 4.18.0-496.el8

Fails on alpine and arch where /tmp is tmpfs, so the inner unshare could not see the temp script to execute.

Thanks a lot Eric! Could we add some of the different passing distros in CI? That would be great to guard against regressions


[SPARK-12873][SQL] Add more comment in HiveTypeCoercion for type widening

I was reading this part of the analyzer code again and got confused by the difference between findWiderTypeForTwo and findTightestCommonTypeOfTwo.
I also simplified WidenSetOperationTypes to make it a lot simpler. The easiest way to review this one is to just read the original code, and the new code. The logic is super simple.

cc @cloud-fan

Test build #49580 has started for PR 10802 at commit 9232ba2.

Test build #49580 has finished for PR 10802 at commit 9232ba2.

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

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

Merged build finished. Test PASSed.

LGTM.  Should we wait for https://github.com/apache/spark/pull/10577 ？ That one also modifies WidenSetOperationTypes

I'm going to merge this. Sorry @gatorsmile you will need to rebase a little bit (changes should be small though). This version is simpler, and hopefully you are using a simpler version too rather than basing off the old, more complicated code.

Sure, will do. : )


fix: resource aws_cloudwatch_log_group should respect the context.enabled flag

what

Check the context flag as well as the cloudwatch_log_group_enabled flag to provision CW log group.
Additionally, add the same check before calls to coalesce functions that rely on null-label module. If the context is disabled, the label module will return null and call to coalesce will fail.

why

To destroy/disable this module we have to set two flags when only one should be enough

references

#181

/test all

@bartelemi can you run the following and push your changes?
make init
make github/init
make readme

/test all

/test all


feat(trace): SpanStartOption and WithLinks

What this PR does / why we need it
Clerk needs to link producer spans with consumer ones cross VM boundaries. In order to do so, we need gobox's trace support to start a span with a link. Clerk already propagates traceHeaders as part of the event, so this PR is to take the same headers (map[string][]string - same as existing To/FromHeaders methods) and generate a link out of them + attach the link as a new StartSpanOption. StartSpanOption abstracts out otel implementation ...

Jira ID
CDT-102

Notes for your reviewers

Pull Request Test Coverage Report for Build 6e784f0c-a13e-448b-83ba-4c03cef53302

41 of 43 (95.35%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 61.985%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/trace/trace.go
5
7
71.43%

Totals

Change from base Build 55acdb60-78a1-44ff-9b21-e0ac02efe82e:
0.3%

Covered Lines:
2710

Relevant Lines:
4372

💛  - Coveralls


Application is misspelled in the box.psm1 file.

Multiple bad requests:
Invoke-RestMethod -Uri $uri -Method Post -Headers $headers -Body $json -ContentType "applicaiton/json"
Please update from 'applicaiton' to 'application'.

Fixed, thanks @pdubblas!


tree shaking donot work for es6 class

I import some es6 class,but not use it,and then build it with webpack,but this es6 class is all in my code,the tree shaking isnot work! but import functions it work well.

It looks like you just deleted our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


Update action-groups.md

As far as I'm aware, it's not always necessary to configure notification. Sometimes, just having an alert suffice. But the current text gives me an impression that alert and notification are always together. I added that sentence to remove ambiguity.

@azarboon - Can you respond to the requested changes?
@AbbyMSFT - Sign off or approve when this PR is ready to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Improve the Mixer perf testing framework.

Add support for writing tests directly against runtime.Dispatcher interface.
Make the perf test execution type (i.e inprocess/coprocess/dispatcher) a flag.
Rename perf.Env to perf.Settings, both to avoid conflict with adapter.Env, and to put additional, generic settings.

This depends on: https://github.com/istio/istio/pull/2323 Also renames dispatcherForTesting => dispatcher.

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
36828a433cc33cb22ac4428a9ce14e582d465cb3
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-presubmit

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
36828a433cc33cb22ac4428a9ce14e582d465cb3
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-presubmit

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
36828a433cc33cb22ac4428a9ce14e582d465cb3
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-presubmit

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
36828a433cc33cb22ac4428a9ce14e582d465cb3
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
90862cfbb45961efdbb5a322987f720bec43bc2a
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Accomodated most feedback. PTAL.

/test istio-presubmit


Question: Best Practice for Arrays of Reducers

I often find I have multiple copies of a Component/Reducer pair one a page that exist in an array. This often happens on forms with lists. To differentiate actions on different components, I have to manually attach some sort of Id or Index parameter to all action creators. This just ends up polluting my action creators and reducers
Simple Duck
const FOO = 'FOO'
export function foo(bar) {
  return {type: FOO, bar}
}

export function fooReducer(state={bar: null}, action) {
  switch(action.type) {
    case FOO:
      return {...state, bar: action.bar}
    default:
      return state
  }
}

Duck with ID
const FOO = 'FOO'
export function foo(bar, id) {
  return {type: FOO, bar, id}
}

export function fooReducer(state={bar: null, id: null}, action) {
  switch(action.type) {
    case FOO:
    	if(action.id === state.id) {
    		return {...state, bar: action.bar}
    	}
    	else {
    		return state;
    	}
    default:
    	if(state)
      return state
  }
}

While this isn't difficult, it adds a lot of boilerplate, especially when the Id ends up being something like form.fooList[1]. It essentially means all action creators and reducers need to be aware that they are in a list.
One option would be to wrap action creators and reducers with functions that add the Id to the action, and reads it back, respectively. But then you are just repeatedly serializing and deserializing paths in the state tree.
Another option, and one that would to attach a hash to the reducer state, and wrap all action creators with a function that inserts the hash into the action. But then you loose the strength of a structured tree.
I looked at redux-forms, and it goes with the first option. I didn't know if this was common across all redux apps, or if there was another method. I feel like there should be some way to leverage to the state tree without having to serialize into a string.

You might be interested in https://github.com/erikras/multireducer.


Modularize CartoDB.js (AMD/UMD, CommonJS)

We’ve developed a flexible mapping stack which allows us to combine maps and services from various providers, where CartoDB.js is an important component. Our mapping stack is AMD-based, which allows us to only load the parts we need to draw a specific map. What we would like to achieve is something like:
define([ 'bower/leaflet/dist/leaflet', 'bower/cartodb.js/cartodb_noleaflet' ], function (L, cartodb) {
  var map = L.map(…);
  var layer = cartodb.createLayer(map, ...);
});

Any chance you will add support for common JavaScript module patterns?

There has been no activity on this issue for several months and that's why we are closing it (automatically). If you think this still needs to be addressed please reopen it. The good news is that we are now working on the next version of CARTO.js, stay tuned.


TS-4550 Adds X-Transaction-Id to xdebug

This makes a header value that matches the new log tag, %.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/354/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/248/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/384/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/280/ for details.


Consider adding local-exec example for NerdGraph automation to reduce manual step

Summary
I might understand that we might not want to automate that by default but would be nice to have a local-exec example.
Desired Behavior
Ex of local-exec (run only one time) for connecting with nerd-graph

NVM local exec. https://registry.terraform.io/providers/newrelic/newrelic/latest/docs/resources/cloud_aws_link_account feels more apropriate.

It was simple as:
resource "newrelic_cloud_aws_link_account" "aws_link" {
  account_id             = var.newrelic_trusted_account_id
  arn                    = module.cloudwatch_metrics_forwarder_stack.newrelic-one-external-role-arn
  metric_collection_mode = "PUSH"
  name                   = var.aws_account_name
}


Document with no abstract fails

The example at https://typst.app/project/wGseT9HAhk-HHQ7oVBJ-pw contains
#import "lapreprint.typ": template

#show: template.with(
  title: "A beautiful preprint template"
)

and fails with "Type none has no method map (line 278)"
This is related to #4 so suggestion is to do some kind of if-else logic to handle metadata that potentially does not exist.

Fixed here https://github.com/MitchellAcoustics/LaPreprint-typst/pull/1


TEST PROBOT #3

A clear and concise description of what the bug is.
Expected results
what you expected to happen.
Actual results
what actually happens.
Screenshots
If applicable, add screenshots to help explain your problem.
How to reproduce the bug

Go to '...'
Click on '....'
Scroll down to '....'
See error

Environment
(please complete the following information):

superset version: superset version
python version: python --version
node.js version: node -v
npm version: npm -v

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

Yippee! Thanks @superset-github-bot :) 🔥 🔥 🔥 🔥 🔥 🔥 🔥 🔥 🔥

For reference, we should be able to start doing all sorts of fun things using probot now
https://github.com/apache-superset/superset-github-bot


Update sanitize-html type definitions.

Last week, I wrote type definitions for the sanitize-html library that we are now using and I submitted a PR to the DefinitelyTyped project. Before merging it, they gave me some feedback, so I'm updating the JupyterLab version to be up-to-date with the DefinitelyTyped version.

👍


Roll back to a previous version (updated and/or reported) record

A record can be "rolled back" to a former version, and the last values must be saved as another part of the array.

Issue moved to LBAB-Humboldt/biomodelos_db_api #7 via ZenHub


Missing "tmpdir" option in downloads

Describe the bug
As discussed in #51 there should have been a parameter in order to specify the temporary directory. Did the support to this feature change or it's not supported anymore?
the current version seems like it creates a temporary file (inside download.py)
                with tempfile.NamedTemporaryFile() as file:
                    # Server might not send all bytes specified by the
                    # Content-Length header before closing connection.
                    # Log as a warning and try again.
                    try:
                        for chunk in response.iter_content(chunk_size=1024 * 1024 * 5):

Expected behavior
list(
    download_features(
        features,
        PATH_WHERE_TO_SAVE,
        {
            "tmpdir": PATH_WHERE_TO_SAVE, ##this not supported?
            "concurrency": 4,
            "monitor": StatusMonitor(),
            "credentials": Credentials(USER_SENTINEL_ESA, PASSWORD_SENTINEL_ESA),
        },
    )
)

the tempdir looks like it's not supported anymore.
Environment

OS: Ubuntu 23.10
Package version: [cdsetool==0.2.11]
Python version: [3.11.6]

There was a change to the tmpdir option in https://github.com/CDSETool/CDSETool/pull/124, which may the issues you are experiencing. As you note, a fix is proposed in #51 (and https://github.com/CDSETool/CDSETool/issues/132), which is still in the feature request stage
I will try to prioritize this feature, as i imagine users downloading cross-filesystem might experience highly reduced IO

Perfect. I thought it was already available and for some reason it was not available anymore. Then I shell wait.
Thanks

i will notify you in #51 when the feature has been released :+1:

Hello any news on the feature #51?


Enable subresource status for VPA

What this PR does / why we need it:
The gardener reconciles the /status field in all vpa objects. The recommendations are stored in the /status field. This should not happen and if the vpa were to change an object and the recommendations are not there then the pod would get its normal requests (defeating the purpose of the vpa). This enables the subresource status for the VPA and prevents this behavior.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Gardener no longer reconciles `status` in VPA objects.

Just doubled checked how this works and it prevents vpa-recommender from writing to the status field. I'll reopen with workaround.


Configfile fixes

This PR introduces the following changes to the configfile generation:

The categories are no longer numbered (fixes #53 )
The category name decorations are removed
fixed category typos

Before:

After:

All looks good, Appreciate the help.


feat : remove duplicate secp256r1

closes #42

.github/workflows/dependencies.yml:55:15: the runner of "peter-evans/create-pull-request@v5" action is too old to run on GitHub Actions. update the action's version to fix this issue [action]
|
55 |         uses: peter-evans/create-pull-request@v5
|               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Error: Process completed with exit code 1.
@mattsse


Don't use BigtableDataClient directly

All of these references need to be removed.

Dataflow

[x] ./bigtable-dataflow-parent/bigtable-hbase-beam/src/main/java/com/google/cloud/bigtable/beam/CloudBigtableIO.java.  Fixed in #2070

One offs

[x] ./bigtable-test/bigtable-emulator-maven-plugin/src/it/simple-it/src/test/java/SimpleIT.java.  This was fixed in #2078.
[ ] ./bigtable-client-core-parent/bigtable-hbase/src/test/java/com/google/cloud/bigtable/hbase/PutMicroBenchmark.java.  This class should be removed.

Client Core

[ ] ./bigtable-client-core-parent/bigtable-client-core/src/test/java/com/google/cloud/bigtable/grpc/async/TestBulkMutation.java
[ ] ./bigtable-client-core-parent/bigtable-client-core/src/test/java/com/google/cloud/bigtable/grpc/async/TestBulkMutationAwaitCompletion.java
[x] ./bigtable-client-core-parent/bigtable-client-core/src/test/java/com/google/cloud/bigtable/grpc/async/TestAsyncExecutor.java. This was removed
[x] ./bigtable-client-core-parent/bigtable-client-core/src/test/java/com/google/cloud/bigtable/grpc/async/TestBulkRead.java.  Fixed in #2064
[x] ./bigtable-client-core-parent/bigtable-client-core/src/main/java/com/google/cloud/bigtable/grpc/async/RetryingMutateRowsOperation.java NOTE: This is only a reference via a comment, which is appropriate to keep.
[x] `./bigtable-client-core-parent/bigtable-client-core/src/main/java/com/google/cloud/bigtable/grpc/async/BulkRead.java
[x] ./bigtable-client-core-parent/bigtable-client-core/src/main/java/com/google/cloud/bigtable/grpc/async/AsyncExecutor.java.  This class was removed
[ ] ./bigtable-client-core-parent/bigtable-client-core/src/main/java/com/google/cloud/bigtable/grpc/async/BulkMutation.java
[x] ./bigtable-client-core-parent/bigtable-client-core/src/main/java/com/google/cloud/bigtable/grpc/async/MutateRowsRequestManager.java NOTE: This is only a reference via a comment, which is appropriate to keep, since BigtableDataClient uses this class.

HBase 1

[x] ./bigtable-client-core-parent/bigtable-hbase/src/test/java/com/google/cloud/bigtable/hbase/TestBigtableTable.java.
[x] ./bigtable-client-core-parent/bigtable-hbase/src/main/java/com/google/cloud/bigtable/hbase/BigtableRegionLocator.java.
[x] ./bigtable-client-core-parent/bigtable-hbase/src/main/java/com/google/cloud/bigtable/hbase/AbstractBigtableTable.java.
[x] ./bigtable-client-core-parent/bigtable-hbase/src/main/java/com/google/cloud/bigtable/hbase/AbstractBigtableRegionLocator.java.

HBase 2

[x] ./bigtable-hbase-2.x-parent/bigtable-hbase-2.x/src/main/java/com/google/cloud/bigtable/hbase2_x/BigtableAsyncTableRegionLocator.java.
[x] ./bigtable-hbase-2.x-parent/bigtable-hbase-2.x/src/main/java/com/google/cloud/bigtable/hbase2_x/BigtableDataClient.java

There are two major issues left:

Bulk mutations - We need a way to cheaply convert MutateRowsRequet.Entry protos to model.Mutation.  That needs a change in google-cloud-java
ReadModifyWrite - We need a way to convert the proto Row response to model.Row. That was implemented here: https://github.com/googleapis/google-cloud-java/pull/4458.  Once there's a new google-cloud-bigtable release, we can convert ReadModifyWrite to the wrapper.

ReadModifyWrite is done.  BulkMutations is fine as is, and we use IBulkMutation for HBase, which meets our core objective.


feat(context): set request as context for serializer

@lob/api-engineering-team

LGTM. Might want to add something to the README to explain this feature

Please add to README and re ping

@lob/api-engineering-team updated README
Does it make sense to bump Joi to 6.x.x here too?

Yeah - I would bump in a separate PR though

LGTM

LGTM


WindowCovering in intermediate state (unless 0/100)

Expected Behavior
When having a WindowCovering device, it shouldn't say "opening..." or "closing..." when in intermediate state.
Current Behavior
It shows in the Home app as closed if the covering is closed (0) or show opened when it is fully open (100), but when it is in any intermediate state (1-99) it always shows "opening..." or "closing..." - not only when opening and closing. The blinds in the image are all open to that percentage for hours now.

Steps to Reproduce (for bugs)
Service code:
let service = accessory.getService(globals.Service.WindowCovering);
service.getCharacteristic(globals.Characteristic.CurrentPosition).on("get", this.getState.bind(this));

getState code:
getState(callback) {
      callback(null, 50);
}

Versions

Homebridge: 0.4.44
Node:       v6.9.4
OS:         raspbian

This happens when you don't have a get for the TargetPosition characteristic.
I think that the default behavior when not having it is making the target position like the current position, in my use case I don't have an API for target position, so I just update the target position every 10 seconds with the current position, or if the user manually sets a position it will be valid for 10 seconds.
If you don't think there should be a default behavior, feel free to close this.


Fix link to caniuse.com in README

Link works (previously was relative to this page)
Link display (capitalisation) is consistent with other links in this document

Thanks!