Update build.gradle to use react native 0.20.+

I was running in to the issue talked about here: https://github.com/facebook/react-native/issues/5963
The fix, posted lower down was pretty simple, it seems that the react native version in the package.json and in Android's build.grade were out of sync. This should fix the issue.

@arao456 Thanks. I'm working on the upgrade to the Forever 21 release as we speak. I'll look this over as I'm wrapping up.

Scratch that. I'll take this now. Yoink! Thanks, @arao456

Sure thing, happy to help!

Looks like with v0.21.0 installed globally running react-native upgrade will do this:
compile "com.facebook.react:react-native:+"  // From node_modules

And so it seems the updates will occur automatically going forward.


:CocInstall coc-tsserver fails

I open vim and try to install coc-json and coc-tsserver extensions. The later fails.
...
Downloading from https://registry.npmjs.org/coc-tsserver/-/coc-tsserver-1.8.6.tgz
...
Extension download at /var/folders/hv/m15wmy656f5_863_02dq185r0000gn/T/v9NSu21/coc-tsserver-ZVGN0v
Installing dependencies by: /usr/local/bin/npm install --ignore-scripts --no-lockfile --production
npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.
npm ERR!      /Users/teh/.npm/_logs/2021-09-02T21_15_13_857Z-debug.log
/usr/local/bin/npm install exited with 1

My npm --version is 7.21.0 w/ NodeJS v16.8.0 and vim --version says:
VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Aug 15 2021 13:04:41)
macOS version - x86_64
Included patches: 1-3350
Compiled by Homebrew
Huge version without GUI.  Features included (+) or not (-):
+acl               +file_in_path      +mouse_urxvt       -tag_any_white
+arabic            +find_in_path      +mouse_xterm       -tcl
+autocmd           +float             +multi_byte        +termguicolors
+autochdir         +folding           +multi_lang        +terminal
-autoservername    -footer            -mzscheme          +terminfo
-balloon_eval      +fork()            +netbeans_intg     +termresponse
+balloon_eval_term +gettext           +num64             +textobjects
-browse            -hangul_input      +packages          +textprop
++builtin_terms    +iconv             +path_extra        +timers
+byte_offset       +insert_expand     +perl              +title
+channel           +ipv6              +persistent_undo   -toolbar
+cindent           +job               +popupwin          +user_commands
-clientserver      +jumplist          +postscript        +vartabs
+clipboard         +keymap            +printer           +vertsplit
+cmdline_compl     +lambda            +profile           +virtualedit
+cmdline_hist      +langmap           -python            +visual
+cmdline_info      +libcall           +python3           +visualextra
+comments          +linebreak         +quickfix          +viminfo
+conceal           +lispindent        +reltime           +vreplace
+cryptv            +listcmds          +rightleft         +wildignore
+cscope            +localmap          +ruby              +wildmenu
+cursorbind        +lua               +scrollbind        +windows
+cursorshape       +menu              +signs             +writebackup
+dialog_con        +mksession         +smartindent       -X11
+diff              +modify_fname      -sodium            -xfontset
+digraphs          +mouse             -sound             -xim
-dnd               -mouseshape        +spell             -xpm
-ebcdic            +mouse_dec         +startuptime       -xsmp
+emacs_tags        -mouse_gpm         +statusline        -xterm_clipboard
+eval              -mouse_jsbterm     -sun_workshop      -xterm_save
+ex_extra          +mouse_netterm     +syntax
+extra_search      +mouse_sgr         +tag_binary
-farsi             -mouse_sysmouse    -tag_old_static
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
  fall-back for $VIM: "/usr/local/share/vim"
Compilation: clang -c -I. -Iproto -DHAVE_CONFIG_H -DMACOS_X -DMACOS_X_DARWIN -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: clang -L. -fstack-protector-strong -L/usr/local/lib -L/usr/local/opt/libyaml/lib<EMAIL_ADDRESS>-L/usr/local/opt/readline/lib -L/usr/local/lib -o vim -lm -lncurses -liconv -lintl -framework AppKit -L/usr/local/opt/lua/lib -llua5.4 -mmacosx-version-min=11.3 -fstack-protector-strong -L/usr/local/lib -L/usr/local/Cellar/perl/5.34.0/lib/perl5/5.34.0/darwin-thread-multi-2level/CORE -lperl<EMAIL_ADDRESS>-lpython3.9 -framework CoreFoundation -lruby.3.0 -L/usr/local/Cellar/ruby/3.0.2/lib

Should be something wrong with your npm (bad repository), you can configure npm.binPath to yarn in coc-settings.json

I started with a fresh ~/.npm and didn't change anything else.
That worked. Odd, but effective. Thanks.


⚠ Processors has degraded performance

In 811ca57, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 73 ms

Resolved: Processors performance has improved in c82314b.


Fixing VT check for Windows

Was previously checking against a string when BASH can do a bool check as wmic cpu get VirtualizationFirmwareEnabled | sed -n 2p is a boolean command. Changed the check of $ve against a bool of true/false and echos if Virtualization is enabled for debugging.

Thanks!  I should have double checked before pushing was in a bit of a rush this morning.


Create report from multiple folders

Is it possible to create a report from different folders?
When I have a big project with submodules with theyr dependencies I have to run multiple times npm audit, with the result that I have many html reports.
I'd like to tell npm audit a way (ex. npm audit --merge ./project/*/package.json - where * is a wildcard for every subfolder to check) to watch every submodules and merge the reports into one.

Interesting use case. I can see how that would be super useful, especially with microservices.
Although, what you're looking for seems to be more like an npm feature. I'm not sure how npm-audit-html could support that without actually wrapping npm instead of piping from it.

Thank you very much @nprail for your quick reply.
So do you think it could be better ask directly to NPM?
Meanwhile I've found this (https://gist.github.com/aravindkumarsvg/6169b8812b6e96f54737128a6f24fe4c), It's not perfect (it doesn't report errors), maybe it could be helpful to someone

@b82 Yeah, either create an issue with either npm or maybe one of the mono-repo helpers like Lerna. Seems like it might be more in the scope of the latter.
If npm or Lerna do add support for this use case, I'd gladly make any modifications needed for npm-audit-html to better support it.


Quantile loss

Roger Koenker has mentioned POGS as a good choice for solving large scale quantile regression problems. In one paper he mentions that he has ran some experiments with the GPU implementation of POGS.
How can one apply POGS to quantile regression? Were Koenker's experiments using an unpublished POGS implementation of quantile loss? Or is it possible to to define a quantile regression problem using the 16 available functions in prox_lib.sh?
Otherwise, I'd love a bit of direction on how to implement quantile loss in POGS.

Hi Howard, this answer surely comes too late, but I figured I'd leave it here for others that might be interested:
The quantile regression problem can be defined as
minimize_{x} \sum_i (1/2) |a_i x - b_i| + (\tau  - 1/2) (a_i x - b_i)

where a_i and b_i are your data.
This fits nicely with POGS if you use f as the absolute value and g as zero. The expression becomes  f = (1/2) |y_i - b_i| + (\tau - 1/2) y_i and g is zero, where y_i = a_i x. Note that the term (\tau - 1/2) b_i has been left out since it's not relevant to the minimization since it's constant.
Here is an example of how you'd set it up in C++
  const T tau = static_cast<T>(0.2);

  f.reserve(m);
  for (unsigned int i = 0; i < m; ++i)
    f.emplace_back(
      kAbs,
      static_cast<T>(1),
      b[i],
      static_cast<T>(1),
      static_cast<T>(tau - static_cast<T>(0.5)));

  g.reserve(n);
  for (unsigned int i = 0; i < n; ++i)
    g.emplace_back(kZero);

I'll add an example to the C++ examples directory.


NullPointerException when use GetText command

In the org.openqa.selenium.internal.seleniumemulation.GetText#handleSeleneseCommand() method, if the WebDriverException happened, it will use finder.findElement() to receive the element again, but in OperaDriver, it will close the connection when the exception happened.
The exception will cause at com.opera.core.systems.ScopeServices#executeCommand(),
For example,
connection.send(commandBuilder.build());
Note that the connection in this case is null.

Would you be able to provide me with a reproducible test case for this?  I've never used Selenium RC styled locators before.  Can you share the customized locator you have that fails with OperaDriver with me?

is this not good enough?
Selenium selenium = new WebDriverBackedSelenium(driver, baseUrl);
// Perform actions with selenium
selenium.open(url);
selenium.getText("some locator we customized!"); // causes an exception.


create modal component

The modal must:

have header, footer and content slots
allow disable the header
open with animation, and close with animation
allow modal inside modal
close with "esc" and close on click out

In GitLab by @victoragcosta on Feb 15, 2022, 14:21
mentioned in commit 1eb279927188b0c27db9700295892e527ae85b71


If a plugin yields no output, the menu bar should be hidden

I like the situation where a menu would go away when a script no longer produces an output. For example, one of the scripts I use must be—because of the BitBar limitation—written as follows:
#!/usr/local/bin/zsh
  if  [[ -z `ifconfig | grep tun0` ]]
  	then
  	echo "🔓 VPN"
  else
  	echo -e "\b"
  fi

The problem is that it returns a blank space in the menu bar when things are all right. Instead of this, I would like no space at all and hence, no menu at all.

Unfortunately, the script can't be written as…
  if  [[ -z `ifconfig | grep tun0` ]]
  	then
  	echo "🔓 VPN"
  fi

The above script returns nothing under normal conditions. The issue then is that it makes non-BitBar menu icons sitting on its left moving back and forth continuously. Simply, because no output is returned.

As a result, it would be nice to have the related BitBar menu item gone when a plugin output nothing.

Thanks matryer. I confirmed the issue is now solved with later versions.


crc8: Use explicit error type

Even if just one error can happen, it may be beneficial to be able to
have a separate error type to allow for From<Error> implementations.

👍🏻

👍🏻

Thanks @barafael for the suggestion for this change!

Thanks @barafael for the suggestion for this change!


Fix for update transformer api

https://github.com/vespa-engine/vespa/issues/20893
I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

This app could be using the pre-converted onnx model from the msmarco-ranking sample app?

@jobergum Well, the idea was that with the export script, the users could experiment with various Transformer models themselves.


Add a pass to specialize opaque type archetypes.

Clones functions containing opaque type archetypes replacing the opaque
type by a concrete type.
rdar://46140751

This does not check for resilience.
Resilience is currently not respected by type replacement: https://github.com/apple/swift/blob/master/lib/AST/Type.cpp#L2459

@swift-ci Please test

Ah, so basically all opaque types are eliminated by this pass today?

Yes until we fix https://github.com/apple/swift/blob/master/lib/AST/Type.cpp#L2459

I don't know the right answer yet how to do resilience properly: Maybe don't specialize if the namingDeclaration is resilient.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - dbf7f64baaa0b24bd404161473d43cd14491f679

Build failed
Swift Test OS X Platform
Git Sha - dbf7f64baaa0b24bd404161473d43cd14491f679

I think we should just say the opaque type is resilient if the module it’s in is resilient.
Making it depend on the resilience of the underlying nominal seems like an unnecessary complication. Also the underlying nominal could be something like Array which is fragile, but you might be returning a ‘some Sequence’ still expecting it to be resilient.
Finally inlinable functions would be an exception because obviously if you can see the body of the function the opaque type is not resilient.
What do you think @jckarter?

We should also add a test to validation-test/Evolution which changes an opaque type resiliently, once this is implemented.

I meant we would call ‘’opaquedecl->getNamingDecl()’’ which returns either a abstractstoragedecl or a func decl. From there we would figure out if that decl is resilient or not in our context: here is where I am more fuzzy and  would need to look at how our APIs work. But we could apply the rules you mentioned if the decl is in a different resilient module it is resilient except if the funcdecl is inline able. Or maybe I can get from the naming decl to the sil function and use SIL apis. I will look into this.

@swift-ci Please test linux

Build failed
Swift Test Linux Platform
Git Sha - 70f5fd0b69cd4e4a0a7967aff9028f9d3a82fd35

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 70f5fd0b69cd4e4a0a7967aff9028f9d3a82fd35

Build failed
Swift Test OS X Platform
Git Sha - 70f5fd0b69cd4e4a0a7967aff9028f9d3a82fd35

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 3beb095bf2302fd70115a38d9d1df4effaaddcb6

Build failed
Swift Test Linux Platform
Git Sha - 3beb095bf2302fd70115a38d9d1df4effaaddcb6

@swift-ci Please test linux

@swift-ci Please smoke test

@swift-ci Please smoke test

@slavapestov I had hoped to get some feedback from you on how best to deal with resilience. It looks like Arnold has the basic criteria implemented—the underlying type is resilient if the module is resilient, or if the body of the NamingDecl is inlinable. However, should we pass down a ResilienceExpansion so that we do the right thing when specializing in the bodies of methods that are themselves inlinable and things like that?

For instance, if you had:
public func foo() -> some P { ... }

@inlinable public func bar() { _ = foo() }

you wouldn't necessarily want to expose foo()'s underlying return type to other modules that inline bar.

@jckarter @aschwaighofer Yeah, I think you should look at SILFunction::getResilienceExpansion(). If a function's resilience expansion is Minimal, you should not specialize any opaque types except those from other inlinable functions.

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - cfc34b16b6f5df19349d64374973d0327c22bda7

Build failed
Swift Test Linux Platform
Git Sha - cfc34b16b6f5df19349d64374973d0327c22bda7

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - cfc34b16b6f5df19349d64374973d0327c22bda7

Build failed
Swift Test OS X Platform
Git Sha - cfc34b16b6f5df19349d64374973d0327c22bda7

This broke the windows build! https://dev.azure.com/compnerd/windows-swift/_build/results?buildId=1963

I will fix.

@compnerd I hope that is the circular dependency issue I introduced. Should be fixed by #24443


metaDE

For metaDE, there is nothing show up for individual study method.

Never mind, I didn't select an active dataset...


Updated preview portal AKA.MS link.

Updated preview portal link with full URL from later in document.

@lwsKCMO : Thanks for your contribution! The author, @alkohli, has been notified to review your proposed change.

Hi @lwsKCMO Thanks for your feedback. The proposed change looks good. Signing off.
#sign-off


Not able to use a S3 repository with base_path

Hello,
Thanks for the plugin !
We are already using successfuly S3 plugin for S3 snapshots (following this documentation https://opensearch.org/docs/2.0/opensearch/snapshot-restore/#amazon-s3 with and access and secret keys) and would like to use your plugin to include client side encryption . We have successfully created private/public RSA keys, installed your plugin (in addition to S3 plugin already in place) and pushed keys to opensearch keystore following your documentation .
Using the same bucket that we use with for current S3 snaphot, we create a new repository have an error with base path
It seems the base path is duplicated somewhere

We don't know where to use this setting
encrypted.security_provider - fully qualified class name for the third party security provider, e.g. com.amazon.corretto.crypto.provider.AmazonCorrettoCryptoProvider

Hi @pmarjou22, thank you for reporting the issue. I fixed the problem, so today evening/tomorrow morning will prepare a new release. Regarding:
encrypted.security_provider - this property needs to be set in opensearch.yml configuration aka static configuration. By default the plugin uses Bouncy Castle security provider, which is not as performant as it could be. So If you want to replace it you need to add an additional security plugin jar file manually in the plugin installation directory. The reason why I did not add e.g. Amazon Corretto security provider is the lack of support for MS Window, Arm64 etc. Be careful this feature is experimental and I haven't finished all testing yet.

So the issue was fixed the new version is: <IP_ADDRESS>

Hello, thanks a lot it works now !


Update code to be more maintainable

closes #17, closes #14
This is a start to help the code maintainability. It also fixes the page responsiveness as a side effect.

I updated the pull request to fix a bug I found where if you click a mouse that has already been uncovered the mouse would be counted again.

Hello @Owen-Dechow, Thank you so much for your contribution, I am merging this pull request Please give this repository a star 🌟

Hey I just noticed, you approved the PR but, you didn't actually merge it.


langchain[minor]: Date Output Parser

Closes #2873
Added the Date Output parser, parsing LLM output into a Date object via converting it to an ISO8601 date time string.
Willing to provide documentation once its been merged.
Please leave feedback below, this is my first contribution.
This feature is similar to the Python version with docs here.
Tested manually.

Relevant issue

Hey sorry meant to get to this earlier, but I spent the past two days resting because of COVID, just curious though, from Jacob's commits it seems like he kept the parser in langchain/, and did not move it into langchain-core/, is there a reason for that?

Ah hope you're feeling ok :(
Yes, Brace was mistaken. Some things still belong in LangChain.

Thank you!


[CROSSDATA-257][CD] Flattened arrays

With this, flattenedCollect performs a bi-dimensional flattening thus removing structures as fields as well as arrays.  The former kind of flattening is the horizontal one whereas the latter is done vertically, that is: Each array element makes the row to be duplicated in a new row where the array is replaced by the element itself.

LGTM

:+1:

:+1:


Enhance code readability

Hello, I hope you are well
Thank you for helping open source and building this project for the .NET community
Out of curiosity, I was reviewing the codes of this repository and I found some points that might be good to correct.
One of the performance improvements was where you used lists
The next thing is that you can use the primary constructor according to the latest version of .NET
In some cases, we could have written the code in a more concise and readable manner and used expressions
There were several typo problems that I corrected
I hope these things are helpful and I will be happy to contribute to the progress of this project
Please check and let me know the result

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

Comparison is base (b725b9e) 94.10% compared to head (bac3515) 90.78%.

Files
Patch %
Lines

src/NRedisStack/Bloom/BloomCommandBuilder.cs
77.27%
5 Missing and 5 partials :warning:

...rc/NRedisStack/CountMinSketch/CmsCommandBuilder.cs
83.78%
3 Missing and 3 partials :warning:

src/NRedisStack/Json/JsonCommandsAsync.cs
84.21%
2 Missing and 4 partials :warning:

src/NRedisStack/CoreCommands/CoreCommands.cs
50.00%
1 Missing and 1 partial :warning:

src/NRedisStack/Json/JsonCommandBuilder.cs
90.90%
0 Missing and 2 partials :warning:

src/NRedisStack/Json/DataTypes/KeyValuePath.cs
75.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #241      +/-   ##
==========================================
- Coverage   94.10%   90.78%   -3.33%
==========================================
  Files          85       85
  Lines        5178     5120      -58
  Branches      487      478       -9
==========================================
- Hits         4873     4648     -225
- Misses        178      345     +167
  Partials      127      127

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

Hey @nimanikoo, Some of the changes you made are only supported in .NET 8 and not in .NET 6 and .NET 7.
We are just making some changes in CI and how we perform tests against all .NET versions (this PR).
When we're done we can merge your PR (we may need to make some changes and adjustments to support all .NET versions).

Hey @nimanikoo, Some of the changes you made are only supported in .NET 8 and not in .NET 6 and .NET 7. We are just making some changes in CI and how we perform tests against all .NET versions (this PR). When we're done we can merge your PR (we may need to make some changes and adjustments to support all .NET versions).

Thank you very much for taking the time to do this
I have completed my changes up to this step and I am eagerly waiting for the result of this process
Thank you for your time and effort to help the open source community @shacharPash @gerzse

Hey @nimanikoo, we're done.
You can now fix the tests that fail, probably most/all of them fail because of .NET 8 syntax that is not supported in .NET 6 or 7.

@nimanikoo The screenshot you have uploaded is not visible, for some reason. Can you, please, upload it again?

Hello again
Sorry for not showing the previous photo, I think the problem is with the GitHub upload process
I ran the Redis container and checked the project several times with different computers, but the tests did not run, and the error I get is for this issue in Stack Overflow:
https://stackoverflow.com/questions/64749385/predefined-type-system-runtime-compilerservices-isexternalinit-is-not-defined
Just to be sure, this time I uploaded the error photo in another storage:
https://fastupload.io/en/xoEJ4WYu5BRNGyk/preview

Hello again,
I hope you are doing well.
I have successfully resolved the issue with building the dock tests and applied the latest changes. The tests have run successfully on my end, so please review them.
One issue I encountered was that some tests were designed for Redis version 7.1.242, which rendered them incompatible with the latest version of Redis. I attempted to install the required version, but unfortunately, it was not available in the Redis archive. I searched on https://github.com/redis/redis/releases but couldn't find it.
In my opinion, to address this issue, we could define a global variable or read this configuration directly from the system's test configuration. Alternatively, we could set it by default in the appsettings.json . This way, everyone can easily configure the version of Redis installed on their local machine and contribute to the project.
Additionally, during testing, I noticed that several Redis features were tested, but certain modules were not installed on my Redis instance. It was necessary to install these modules, such as the redisbloom module, for comprehensive testing.
I believe documenting these experiences thoroughly will greatly benefit future contributors, as they may encounter similar challenges and need to install additional Redis modules. This documentation will facilitate faster development for subsequent contributors
252 tests have been skipped during the testing  process due to incompatibility with the Redis version. The tests were configured for version 7.1.242.
Thank you for the code review.

Hi @nimanikoo how are you?, regarding the missing modules - if you run the docker in the README (docker run -p 6379:6379 --name redis-stack redis/redis-stack:latest) you should get all the modules. I would appreciate it if you could specify what exactly you ran/downloaded so that we can understand why you were missing modules.
Secondly, the tests that run on .NET 6 and 7 still fail... because of a syntax that is only supported by .NET 8, for example: List<object> args = [key];

Hi, I came across this pull request, I was trying to fix the issue with testing problems, First I tried to run the tests from Fixup branch, i got the error for the Search section tests as mentioned above. then i switched to master branch to run the tests to see if all tests pass without a problem, it still throw System.IndexOutOfRangeException exception. i was comparing the code changes between master and fixup branches, the part of code that throws the exception didn't have any changes.

can you help how to solve this issue?
@shacharPash
@gerzse

Hi @nimanikoo how are you?, regarding the missing modules - if you run the docker in the README (docker run -p 6379:6379 --name redis-stack redis/redis-stack:latest) you should get all the modules. I would appreciate it if you could specify what exactly you ran/downloaded so that we can understand why you were missing modules.
Secondly, the tests that run on .NET 6 and 7 still fail... because of a syntax that is only supported by .NET 8, for example: List<object> args = [key];

@nimanikoo, did you see my comment?

Hi dear @shacharPash  , how are you? I read your comment and this conflict was on my local machine. Redis was running locally on my machine on port 6379 and it did not have a number of Redis stack modules and it was causing problems, I fixed this local conflict and the problem was solved.Currently, all the tests are passed correctly, except the tests related to the search, which @omidpakdel mentioned, and I have the same problem in branch Master for tests.

@nimanikoo You can see the problematic sections in what failed the .NET 6 and 7 tests

Hello @shacharPash , How are you doing?
I've fixed all the compiler errors that were there, and I'm confident that we won't have any more issues with the .NET version or syntax.
I would have really loved to be able to fully utilize the new features of .NET 8 in this project, but unfortunately, it's not possible.
Please let me know if there's anything else I can do to help us get closer to merging.

Hi @shacharPash , how are you?
After implementing recent code changes, 17 out of 18 tests pass  successfully.However, one test is encountering a build error.
I've thoroughly reviewed the logs but haven't been able to identify the exact error, and it doesn't appear that any changes have been made compared to the master branch.
Furthermore, the line causing the error hasn't undergone any significant modifications.
Would you be able to assist me in resolving this issue or provide guidance towards a solution?


policy / k8s integration: ensure that status update does not change content of CNP

Depending on the K8s version we are using, we update the CNP itself when updating its status - ensure that we are not accidentally updating the content of the CNP in this case as a safety check so that policy is not modified errantly by cilium-agent.

@ianvernon Can you provide more info how could this happen?

@aanm I saw a case with K8s 1.9 where a policy's status appeared to get updated by a single cilium-agent in a cluster, but that status update appeared to corrupt the policy, as the entire object was being updated in K8s. I am unable to conclusively say whether it did due to lack of visibility / logs, but there was no policy added by a user of the cluster, nor by a program / automation which adds policies to a cluster. The only event that occurred when the CNP got 'corrupted' was right after a status update from a node. I looked at the UpdateStatus code for a CNP when running in cases where we have to update the object itself in K8s, but could not see how this could happen. However, I was unable to come up with a better hypothesis as to how the policy got corrupted.
However, since we do not support K8s 1.9 anymore, I think we should just remove the capability of Updating objects when updating their status entirely to remove even the possibility of this happening.


Skin Mesh From Mesh in UI should automatically work on a group

Maybe also work when multiple meshes selected.  First selection to others.

should be fixed as of this commit
https://github.com/louisVottero/vtool/commit/b866b8b5b3e285ff80e868ac355fdf18c7ef9865


Sphinx-copybutton is not visible until code block is hovered or focused

I want to leave this one up to design whether we should change the user experience here, and if so, how.

I'm going to have what might be a controversial opinion, but IIRC the goal of sphinx-copy-button was to simplify copying when >>> and ... were present in the cell and thus annoying to remove for large block of code when you actually did not wanted to copy "all".
With css user-select we've already made the prompt and other line numbers non-selectionable, so there should be fewer, if no need at all for sphinx copy-button.
I think there is a better way to achieve this than the copy button by using already existing user interface. I think we likely are looking for something like user-select:contain, or user-select:all.
Right now you still have to drag to select a full cell.
Maybe making a quadruple click in a cell select the whole text (triple click is select the whole line by default)
        document.querySelectorAll('pre').forEach(pre => {
            pre.addEventListener('click', (function() {
                let clickCount = 0;
                let timer;

                return function(event) {
                    clickCount++;
                    clearTimeout(timer);

                    if (clickCount === 4) {
                        let range = document.createRange();
                        range.selectNodeContents(this);
                        let selection = window.getSelection();
                        selection.removeAllRanges();
                        selection.addRange(range);
                        clickCount = 0; // Reset the count after quadruple click
                    }

                    timer = setTimeout(() => clickCount = 0, 400); // Reset click count after 400ms
                };
            })());
        });

Does the trick.
Now for accessibility, I believe we "just" need the copy action of the browser to copy the content of the cell when focused.
And with that we'll just have no need for sphinx copy button, and then decide if we want to phase it out.

Does the Copy Button Stay or Go?: From an accessibility perspective, I think it is important to keep the copy button. It provides a simple and clear one-click action for copying code, which is useful for users with motor impairments or those using assistive tech. The button will ensure that all users have an accessible and straightforward way to copy code.

You seem to have missed :

Now for accessibility, I believe we "just" need the copy action of the browser to copy the content of the cell when focused.

If the cell is focusable and has tabindex=..., and when focused the native 'copy' function works be copying the full content, why would we need an extra copy button ? I mean from an accessibility perspective wouldn't edit > copy, or contextual menu > copy be better alternatives ? Plus keyboard shortcuts would work as well.
I don't think it is easier for a user with impairment to managed to either point the mouse as a small target, or use keyboard navigation to move away from the current selection to find the copy button. That at least my experience with people I know have accessibility issues are more likely to use keyboard shortcuts.
Likely i'm miss-understanding how assistive tech might work, I'm representing something like keyboard like navigation where you focus each item in turn, and so would likely have already a code block focused and then want to "copy" it's content.
I'm also curious about what it give as screen readers, having copy buttons mean that while navigating the page every other other will read "copy button" which seem to me like a worse experience.
Now I agree that it's more discoverable and easier for some categories of users, mostly novices that don't know keyboard shortcuts, or contextual menus. I can also be convinced it's
it's more convenient for typical users without accessibility issues that can easily reach for the copy-button. But I don't believe that this copy button is better for people with accessibility issues in general.
One thing I'm afraid is wether we are just trying to improve the copy button because it is there, which is a sunk-cost fallacy. Instead of asking "if there was no copy-button what would we do ?". I'm also aware of chesterton's fence, but I believe in this case we clearly know why sphinx copy-button was added : to allow copying without getting the prompts in the clipboard – reason which definitely does ont exists anymore as the prompt are not selectable.

On the if we keep it how to improve it.:
When the code block has a header bar I think the button could/should be in the header bar and always be visible. For example in a case like this:

I think a clipboard icon would be also more recognizable.
Another question if we keep the button, why have it only on multiline code blocks ? I'm assuming pre tags (and even whole paragraphs) also deserved to have copy button ? like in:

banner by prepending the following HTML to all pages (be sure to replace URL_OF_STABLE_VERSION_OF_PROJECT with a valid URL, and adjust styling as desired):

I can see how URL_OF_STABLE_VERSION_OF_PROJECT could be a copy target.

I like these arguments for getting rid of the extension. However, even if we get rid of it in the PyData Theme docs, there's still the question of whether theme adopters will still put the extension on their site even if we make the changes @Carreau suggests.
My understanding is that part of the value proposition of the PyData Theme is that it plays nicely with certain extensions and even carries over some desirable properties (such as more accessible color contrasts) to some of these extensions (for example, Sphinx Design).
In the short term, we may want to make a few changes that only apply to the copy button extension when it's used in conjunction with the theme. In the long term, we may want to work with the extension authors to implement accessibility best practices.

Alternatively, we could take a stronger stance and tell theme adopters that they shouldn't use the copy button extension.

I don't think we should take a strong stance right now. I would much better explore what we can do without, and do a 3 step removal:

opt-out
opt-in
remove compatibility

This let us explore wether the copy button is actually (un)necessary.

3 step removal

Love that

With the step-0 improve the theme html/js to make copy button unnecessary :-)

Could I get an update on this?
My understanding is two-fold:

We will make the copy button always visible in the short term
Over the long term we will climb @Carreau's three-step removal ladder, evaluating at each rung whether we move to the next.

Is that correct?
If so, we can move this issue to dev and call it done when the button is always visible. We should open a separate issue to track removing the extension.

I'll add myself as assignee and will look into making it always visible.

@gabalafou yes your summary is correct, that is the plan

Bump on this. If there isn't current work underway on this, could I take it over?

Switched the assignee from me to you.


fix: NotificationSettingsClient.delete now returns None for 204 No Content response

Changed

NotificationSettingsClient.delete now returns None for 204 No Content response

Included in https://github.com/PaddleHQ/paddle-python-sdk/pull/41


MNT: http:// → https://

Hello @DimitriPapadopoulos, thank you for submitting the Pull Request!

In the file niworkflows/__about__.py:

Line 26:80: E501 line too long (81 > 79 characters)

In the file niworkflows/utils/bids.py:

Line 109:80: E501 line too long (91 > 79 characters)

To test for issues locally, pip install flake8 and then run flake8 niworkflows.


Deprecate edgeConfigId in favor of datastreamId

Description

As mentioned in #995, we should rename edgeConfigId to datastreamId. This adds the deprecated() validator (as created in #1000, so merge this PR after that one) to edgeConfigId. It adds a warning whenever edgeConfigId is used in the configure command.
I did not use this validator on datasetId because it is deprecated in favor of a deeply nested value, nor did I deprecate eventMergeId because it does not have a replacement.
Related Issue

https://jira.corp.adobe.com/browse/PDCL-10587
Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Improvement (non-breaking change which does not add functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA or I'm an Adobe employee.
[x] I have made any necessary test changes and all tests pass.
[x] I have run the Sandbox successfully.

I will merge this after #1000 is merged.


Upgrade TypeSpec to 0.43.0

Description
Fix https://github.com/Azure/autorest.csharp/issues/3272

upgrade typespec library to 0.43.0
resolve the breaking changes
update cadl-ranch projects

Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

@chunyu3 , do you have the SDK Regen PR?  I think all typespec files are updated so it should not block the regen.

@chunyu3 , do you have the SDK Regen PR? I think all typespec files are updated so it should not block the regen.

@lirenhe regen PR is generated.

@chunyu3 this PR looks good but I am holding off merging it because the regen PR is having issues with finding the alpha package in the public feed.  I reached out to @hallipr but we weren't able to fully resolve the problem.
I believe its fine to merge this in as the resulting regen is visible and we can update to the beta packages once this is merged but I will let you do that tonight.


MotionEvent: Fix calculation of z values in scale_for_screen method

MotionEvent attributes z, oz, pz, dz will be set to 0 when p is None. Follow-up on https://github.com/kivy/kivy/pull/7659.

Maintainer merge checklist

[x] Title is descriptive/clear for inclusion in release notes.
[x] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[x] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

LGTM, if possible adding a test for what was broken before that change would be nice.

@tshirtman This change is to reset z values for some extreme case where p argument changes between scale_for_screen calls. So nothing is broken, we are just covering this extreme case. Merge if you agree.


Gradle files unchanged after linking

After linking with: "rnpm link " I see added to MainActivity.java, but I don't see the necessary changed to build.gradle or settings.gradle. My apps package.json file is below. The output of the link command is successful. Also, this might be a related issue, but when i run "rnpm install " I get the following output:
rnpm-link info Module react-native-fbsdk has been successfully installed & linked
But in this case the linking doesn't work at all. Seems to ignore the MainActivity and gradle files, where at least the "rnpm link " makes the changes to MainActivity.
Please let me know what other info you need.
{
  "name": "event_photo",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start",
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "alt": "^0.18.4",
    "native-base": "^0.4.7",
    "react": "15.0.2",
    "react-native": "^0.26.3",
    "react-native-contacts": "^0.2.8",
    "react-native-fbsdk": "^0.2.2",
    "react-native-image-picker": "^0.19.4",
    "react-native-router-flux": "^3.31.0",
    "react-native-vector-icons": "^2.0.3"
  }
}

I'm trying to link react-native-fbsdk. I can do it manually, but I seem to continuously have this problem even with other packages.

That's quite strange. Just tried to generate a blank project using<EMAIL_ADDRESS>and it worked. Did you apply any changes to your gradle files?

Yes, there are many other changes to the gradle file. Actually my project was originally a non react native android project, then it evolved into react native. During the evolution there were a number of issue I ran into. Although it has been working fine for a while now, I'm thinking of remaking the project from scratch. Perhaps this cleanup will get rid of this strange problem. Will update the thread in the next few days with the result.

it's sensitive to indentation in gradle files, maybe you changed it for key-lines (we use find-replace approach)

After creating a fresh react native project all the android files seem to be updated accordingly when running rnpm link with the exception of the MainApplication.java file. Rnpm should be adding the package names in the array defined in getPackages overriden function, but I end up having to do it manually with each package I install.
Also, I find that rnpm install does not install the package or link the package, but it gives the following output:
rnpm-link info Module react-native-fbsdk has been successfully installed & linked

Thanks for your feedback, @joshg111!
We fixed MainApplication patch issue in react-native (rnpm is going to be deprecated soon as we merged it into react-native core). About fbsdk: it's a very tricky module. I think it'll be great if you can join our discussion here https://github.com/facebook/react-native/issues/8623.


搭建调试环境

What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.04%) to 50.562% when pulling d13397ab93e92aeeb5895a9017921f0d9e104470 on<PHONE_NUMBER>:master into 247e0bfbe4f17cd757a595f580f1801bb1934468 on apache:master.

It seems that you resolved two issues in this pr: added the windows boost script and polished the example, so could you split this pr into 2 Independent PR, it will be better if you can propose two issues first then releated with 2 issues. So I will close this pr first, and wait for your next PR.


SqlDataReader.GetXmlReader(int) returns unusable reader rather than throwing on DBNull

Describe the bug
Most GetXYZ(int) (e. g. GetInt32, GetString, etc) methods on SqlDataReader will throw if they encounter a DBNull.Value and cannot return it. The exception is GetValue which can just return DBNull.Value directly.
GetXmlReader is another exception but with (in my opinion) less reasonable behavior. When called on a column whose value is SQL NULL, it returns a non-null XmlReader object which fails when read from.
To reproduce
Include a complete code listing (or project/solution) that we can run to reproduce the issue.
var connectionString = "...";
using var conn = new Microsoft.Data.SqlClient.SqlConnection(connectionString);
conn.Open();
using var cmd = conn.CreateCommand();
cmd.CommandText = "select cast(null as xml)";
var reader = cmd.ExecuteReader();
reader.Read();

Console.WriteLine(reader.IsDBNull(0)); // true
using var xmlReader = reader.GetXmlReader(0); // works, does not return null
Console.WriteLine(XDocument.Load(xmlReader)); // throws InvalidOperationException "The XmlReader state should be interactive"
//Console.WriteLine(reader.GetString(0)); // throws SqlNullValueException "Data is Null. This method or property cannot be called on Null values."

Expected behavior
For consistency with the other get methods, I think GetXmlReader should throw SqlNullValueException when called on a DBNull column.
This behavior would be useful when processing query results because any errors due to expected non-null values fail fast early rather than manifesting later as a different-seeming exception.
Further technical details
Microsoft.Data.SqlClient version: 4.0.0
.NET target: .NET 5
SQL Server version: SQL Server 2017
Operating system: Windows 10

I found this behaviour when I was working to add GetFieldValue<TextReader> etc. I think the original reason was so that you could always use the returned reader as part of a using statement without throwing in the using code but that means that the user should always call IsDBNull beforehand or check the state of the reader once they've asked for and received it.
Unfortunately I think that changing the default behaviour of the existing methods to throw on null value is likely to break a lot of old code. The pattern I recommend is to always check IsDBNull before you ask for the reader because the null check is part of the row metadata it's essentially free. Programming assuming that null will throw exceptions is a poor choice because it's using exception for control flow which limits scalability.

I think the original reason was so that you could always use the returned reader as part of a using statement without throwing

Maybe this wasn't always true, but using blocks are null-safe, right? E. g. you can do:
using var y = SomeFunctionThatMightReturnNull();

And it will just noop when it gets to the implicit finally block.

Unfortunately I think that changing the default behaviour of the existing methods to throw on null value is likely to break a lot of old code

Perhaps, but:
(1) Microsoft.Data.SqlClient is relatively new compared to System.Data.SqlClient and it feels like stuff like this ought to be able to be fixed. The ability to be versioned independently of .NET gives consumers more control around upgrading as well.
(2) given that using blocks won't break and actually trying to do anything with the returned XmlReader throws an exception, I struggle to imagine that there's much code working well today under the old behavior that would be broken under the proposed new behavior. Obviously something can be contrived but it doesn't seem too likely.

The pattern I recommend is to always check IsDBNull before you ask for the reader because the null check is part of the row metadata it's essentially free

I agree with this if the data might actually be null. However, in most cases you know whether the data coming back ought to be null or not based on the underlying column types, check constraints, business rules, etc. If you don't think it should ever be null, then I think it is the right move to just call GetInt32() and rely on the internal check to fail fast if your assumption was invalid (likely due to a bug elsewhere in the system) rather than having to insert an explicit if-null-throw check before every column access. It's like the cases where you'd use .Single() or .First() instead of the OrDefault() versions of those methods.

@madelson we will look into this and will decide about the breaking change internally and will update you soon.

Thanks @JRahnama . Even if it isn’t possible to make the change, it would be great to call out the inconsistency in the documentation.

Perhaps changing the default behaviour to throw and using an AppContext switch to turn back the previous behaviour like we have in other places would work here.

We would support an API doc change. However, the implementation of the breaking change is not likely to happen.


configuration cleanup

Major files : configuration

[ ] #10
[ ] #11
[ ] #12

Major files : scripts

[ ] #13
[ ] #14

https://github.com/cykomaniacs/cyko/commit/82cee6e5a89e6e149e5fc8e18a0b0cf9131f4e0e


Two different .bin file plotting color wonder

Question
Hi guys,
I just used vegeta tool to create two different bin files, and plot them by
$vegeta plot result1.bin result2.bin > plot.html
The colors generated by plot look similar which make it hard to tell. Is there any change I could make to adjust the line color distribution? The illustrated figure looks like distinct and easy to tell by simply looking at the color.
Also, the number in bottom left corner means moving average window size (in data point), I do not understand why we just simply modify the values and the latency will change respectively.
Sorry if I asked the dumb questions...

It looks like line colors are hardcoded in the script. See:
https://github.com/tsenart/vegeta/blob/19b74586217105bbde8ded6077c70095e97146bf/lib/plot/plot.go#L273-L276
https://github.com/tsenart/vegeta/blob/19b74586217105bbde8ded6077c70095e97146bf/lib/plot/plot.go#L253-L261
Maybe it could be a parameter passed to plot command? @tsenart what do you think?

This has been fixed in 60c2d660882aa630a4dd08a110955de94beb4cb6


Changing between topologies feels slow

the load indicator might take too long to show up?
if a topo has been seen before, and i switch back to it, i should be able to see it right away.

Loading indicator only show for initial load at the moment, shouldn't be hard to change..

Closed by #2232


Allocation failed, memory limit is very restricted

Seems memory limit is very restricted not exceeding let say around 1gb
(node v9.5.0)
% geojson-merge DE5.geojson DE6.geojson DE7.geojson > DE.geojson
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [/usr/local/bin/node]
 2: node::FatalTryCatch::~FatalTryCatch() [/usr/local/bin/node]
 3: v8::Utils::ReportOOMFailure(char const*, bool) [/usr/local/bin/node]
 4: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [/usr/local/bin/node]
 5: v8::internal::Factory::NewRawOneByteString(int, v8::internal::PretenureFlag) [/usr/local/bin/node]
 6: v8::internal::String::SlowFlatten(v8::internal::Handle<v8::internal::ConsString>, v8::internal::PretenureFlag) [/usr/local/bin/node]
 7: v8::String::WriteUtf8(char*, int, int*, int) const [/usr/local/bin/node]
 8: node::StringBytes::Write(v8::Isolate*, char*, unsigned long, v8::Local<v8::Value>, node::encoding, int*) [/usr/local/bin/node]
 9: node::Buffer::New(v8::Isolate*, v8::Local<v8::String>, node::encoding) [/usr/local/bin/node]
10: node::Buffer::(anonymous namespace)::CreateFromString(v8::FunctionCallbackInfo<v8::Value> const&) [/usr/local/bin/node]
11: v8::internal::FunctionCallbackArguments::Call(void (*)(v8::FunctionCallbackInfo<v8::Value> const&)) [/usr/local/bin/node]
12: v8::internal::MaybeHandle<v8::internal::Object> v8::internal::(anonymous namespace)::HandleApiCallHelper<false>(v8::internal::Isolate*, v8::internal::Handle<v8::internal::HeapObject>, v8::internal::Handle<v8::internal::HeapObject>, v8::internal::Handle<v8::internal::FunctionTemplateInfo>, v8::internal::Handle<v8::internal::Object>, v8::internal::BuiltinArguments) [/usr/local/bin/node]
13: v8::internal::Builtin_Impl_HandleApiCall(v8::internal::BuiltinArguments, v8::internal::Isolate*) [/usr/local/bin/node]
14: 0x9f4e81842fd
15: 0x9f4e823d196
fish: 'geojson-merge DE5.geojson DE6.g…' terminated by signal SIGABRT (Abort)

probably has to do with https://stackoverflow.com/questions/38558989/node-js-heap-out-of-memory

export NODE_OPTIONS=--max_old_space_size=4096 fixed it


Black/NodeBrowserCICD

Complete sdk for node and browser with parameterized tests for the CI/CD pipeline. Also several edits to the documentation

@MichaelEdwardBlack Is it clear why CI is failing or is this expected?


Fix #40 Change named parameter 'Precision' to 'precision'

Fixes the follow error:
TypeError: init() got an unexpected keyword argument 'Precision'

Do you think we should keep the float precision? I have removed it in the develop branch some days ago.

Hi @msander
You are probably right, I was a bit too quick with the fix. It solved my compilation issue but caused issues further down the line (64 is too much for the precision value).
Removing it works for me as well.
Thanks
Dieter


Migrate from GitLab CI to GitHub Actions

Create a GitHub Actions workflow based on .gitlab-ci.yml. Name the workflow .github/workflows/workflow.yml. The workflow should run on pushes to the master branch, and on pull requests. Here's an example snippet:
on:
  push:
    branches: [ master ]
  pull_request:

Delete the GitLab CI file.

If you'd like to work on this, leave a comment saying something like "I'll work on this.", and you can begin without waiting for me to OK it. The point of writing the comment is to let others know that this issue has already been taken up.

I'll work on this

Looks like there is some issue in the yml file. Others can pitch in.

@garg-saurav Hey hi, are you still working on this issue?

@neelkamath Can I take up this issue if @garg-saurav is not currently working on it?


GDPR & Data Protection + compliance

@decbays has considered GDPR and suggests that we need to be able to offer for people to remain anonymous, and only offer public information if they feel like doing so

We've agreed on the project that for the purposes of CollabW22 we'll create mock data #4 to avoid holding personal data.
We would also have to be extremely aware of data security, and @decbays and @FlicAnderson agreed that protected characteristics (ie gender) will be left out of data collection by design as we don't believe it would be beneficial for the project.


ODPI-10 Publish distro artifacts to Nexus (Hadoop)

Adding the deploy target for hadoop.

@c0s  Can you please review the changes.

@c0s  Can you please try now.

All good, as per https://jira.odpi.org/browse/ODPI-10?focusedCommentId=10335&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-10335

@c0s  Add the zookeeper also along with hadoop, for testing of zookeeper, you have to make a small change or tell me where should I make the change in gradle.
The change is:-

Copy the "deploy/zookeeper-nexus-deploy-build.xml" in the zookeeper root folder parallel to "build.xml", before the ant is triggered.
With "deploy_arg=yes", deploy target should use the "zookeeper-nexus-deploy-build.xml" file for starting the build.
With "deploy_arg=no", mvn-install will be used.

Pleas let me know your feedback, and guide me to incorporate the "point-1" part.


[5.10] Substring boundaries during matching

Cherry pick of #675 for the 5.10 branch.

@swift-ci Please test


Closure should capture the narrowed type when closure is created with in a narrowed environment

Description:
import ballerina/io;
public function main() {
    string? x = "hello";

    if (x is string) {
        var f = function() {
                    x = (); // static type of `x` should be `string` but it's `string?` now.
                };
        f();
        string kk = "--" + x + "--"; // x is no longer a string
        io:println(kk);
    }
}

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Related spec issue - https://github.com/ballerina-platform/ballerina-spec/issues/1043.


Parse prep time into total time expressed in minutes

Prep times are scraped in several formats:

PT#H
PT#M
PT#H#M
Any of the above, with trailing garbage text

This change converts this scraped format into total minutes of prep time:
prepTime = (hours * 60) + minutes
Resolves: #2

Looks good - I'd move this to the twopeas spider, since this function is pretty specific to that one specific website.

As I was writing the spider for simplyrecipes, I noticed that their website also used a similar format for cooking time. It turns out that it's a time duration format specified by ISO 8601. I've used the  1c3a8f3, I've used the isodate library to convert this to minutes.
Either way, thanks for looking in to this!


Can't insert books in SMP

Me and my wife are playing on a LAN SMP map on 1.7.2 and we cant craft the Enchiridion but any time we go to place a book in it it won't let us.

Enchirdion 2 removes this anyways. (Different way of accessing books).


Fix two 'PRISMA' vulnerabilities

Pull Request Motivation
Our enterprise customers complain whenever there are vulnerabilities found in the images we ship, and don't care whether the vulnerabilities are really exploitable or not.  To that end, we maintain an internal fork of cert-manager just to apply CVE fixes.  I suspect that we might not be the only ones who have customers who are this demanding, so I wanted to submit these fixes upstream since others might find it as helpful to have them as we do.
Kind

Release Note

NONE

Added sign-off

Sorry to annoy you with this, but the CI wants both commits to have a signoff message. @pmerrison

Sorry to annoy you with this, but the CI wants both commits to have a signoff message. @pmerrison

Ah, no worries, sorry I missed one.  Will get it fixed up

/ok-to-test

/retest

Not sure why tests are failing. I tried to include the upgrades in https://github.com/cert-manager/cert-manager/pull/6332 together will a bunch of other upgrades.
I added you as a co-author in that commit.

resolved in https://github.com/cert-manager/cert-manager/pull/6332


add pkg-config support

First, thanks for creating this system.  I need a C++17 RabbitMQ client in a systemd daemon using sd-event and the contortions I was going through to use rabbitmq-c were disheartening.  I have high hopes for this alternative.
I also need pkg-config support, but have been unable to extract the relevant commits from pull request #170.  I'm creating this issue for tracking, and will immediately be submitting a PR that provides only that feature.
The only "invasive" change is to add CMake variables for the version number, and to increment the patch version to 3.0.2.  The rationale for the increment now is that anything built from current master is no longer 3.0.1 so the versioning should be different.  This can be mooted in the PR or here.

Wow, fast.  Thanks.  Apparently I haven't managed to learn to close github issues in commit comments correctly yet.


Fix iOS deployment target for FMDB

It's defaulting to 4.3 but then we are getting this warning:
⚠  /Users/travis/build/snowplow/snowplow-objc-tracker/Pods/FMDB/src/fmdb/FMDatabaseQueue.m:83:9: 'dispatch_queue_set_specific' is only available on iOS 5.0 or newer [-Wunguarded-availability]

        dispatch_queue_set_specific(_queue, kDispatchQueueSpecificKey, (__bridge void *)self, NULL);

        ^~~~~~~~~~~~~~~~~~~~~~~~~~~

⚠  /Users/travis/build/snowplow/snowplow-objc-tracker/Pods/FMDB/src/fmdb/FMDatabaseQueue.m:152:54: 'dispatch_get_specific' is only available on iOS 5.0 or newer [-Wunguarded-availability]

    FMDatabaseQueue *currentSyncQueue = (__bridge id)dispatch_get_specific(kDispatchQueueSpecificKey);

        ^~~~~~~~~~~~~~~~~~~~~~~~~~~

Closing as fixed (probably in version 0.7.0)


Documentation updates

The majority of these changes are to the README file, but there are some other small documentation updates included as well.
For now, I've temporarily removed all links to projects that aren't compatible with Irmin 1.0 but am interesting in adding links to related projects that work with the current version at some point. Having the links to the incomaptible libraries is very confusing.
I think the README still needs quite a bit more work, especially in regards to how we describe what Irmin is. This is something I will continue to think about, but if anyone has any opinions on this I would love to hear them.

That looks great, thanks!


Review dandihub Ansible config

Should some of this be moved to Terraform?

I think it was moved to Terraform https://github.com/dandi/dandi-infrastructure/tree/master/terraform but this was never closed

dandihub still uses ansible. with 2i2c and our functioning script this would be more of a consolidation project rather than a necessity.


Need binary for python 2.5

I need help installing zmq in python 2.5. I see that since 2.1.x you have removed support for py25, but installing those previous versions gives problems. Even configuring the python setup.py configure --zmq=/lib/site-packages/zmq. I try with pip and with easy_install. Different versions, and also getting installers from:
ZeroMQ downloads
Please need if someone has the executable or msi for python 2.5
Thx!

Supporting Python 2.5 may be difficult, but if you download the source for the old version and install with --zmq=/usr/local, it should work. Note that it will require old libzmq as well (2.x, I believe).

Thx!


Unable to install ruby red-adbc on Mac M1

When I attempt to install the red-adbc gem , it fails.
gem install red-adbc
Building native extensions. This could take a while...
ERROR:  Error installing red-adbc:
	ERROR: Failed to build gem native extension.

    current directory<EMAIL_ADDRESS>/Users/pstalcup/.rvm/rubies/ruby-3.2.2/bin/ruby -rrubygems /Users/pstalcup/.rvm/rubies/ruby-3.2.2/lib/ruby/gems/3.2.0/gems/rake-13.0.6/exe/rake<EMAIL_ADDRESS><EMAIL_ADDRESS>
rake failed, exit code 1

Gem files will remain installed in<EMAIL_ADDRESS>for inspection.
Results logged to<EMAIL_ADDRESS>
I had similar failures for trying to install red-arrow, and was able to install it following a suggestion from https://github.com/apache/arrow/issues/14945#issuecomment-1352598282
However, when I attempt to apply the same technique it says
gem install red-adbc -- --with-ldflags="-Wl,-undefined,dynamic_lookup"
Building native extensions with: '--with-ldflags=-Wl,-undefined,dynamic_lookup'
This could take a while...
ERROR:  Error installing red-adbc:
	ERROR: Failed to build gem native extension.

    current directory<EMAIL_ADDRESS>/Users/pstalcup/.rvm/rubies/ruby-3.2.2/bin/ruby -rrubygems /Users/pstalcup/.rvm/rubies/ruby-3.2.2/lib/ruby/gems/3.2.0/gems/rake-13.0.6/exe/rake<EMAIL_ADDRESS><EMAIL_ADDRESS>--with-ldflags\=-Wl,-undefined,dynamic_lookup
invalid option: --with-ldflags=-Wl,-undefined,dynamic_lookup

rake failed, exit code 1

Gem files will remain installed in<EMAIL_ADDRESS>for inspection.
Results logged to<EMAIL_ADDRESS>
Is there any way I can install this gem?

Is it possible to attach that logfile?

Contents of the respective files:
current directory<EMAIL_ADDRESS>/Users/pstalcup/.rvm/rubies/ruby-3.2.2/bin/ruby -rrubygems /Users/pstalcup/.rvm/rubies/ruby-3.2.2/lib/ruby/gems/3.2.0/gems/rake-13.0.6/exe/rake<EMAIL_ADDRESS><EMAIL_ADDRESS>
rake failed, exit code 1

and
/Users/pstalcup/.rvm/rubies/ruby-3.2.2/bin/ruby -rrubygems /Users/pstalcup/.rvm/rubies/ruby-3.2.2/lib/ruby/gems/3.2.0/gems/rake-13.0.6/exe/rake<EMAIL_ADDRESS><EMAIL_ADDRESS>--with-ldflags\=-Wl,-undefined,dynamic_lookup
invalid option: --with-ldflags=-Wl,-undefined,dynamic_lookup

rake failed, exit code 1

Hmm, guess that doesn't say much.
The Rakefile it's running is trying to determine if the native packages are installed. Are they? I guess if you're on macOS, those aren't necessarily distributed through Homebrew or anything, so you may have to build from source.


It's not a issue report

requireComposerPackages using this method I got the following problem: "The HOME or COMPOSER_HOME environment variable must be set for composer to run correctly" and found the solution here https://github.com/statamic/cms/issues/3383#issuecomment-846323885 . Others may have the same problem. In order not to repeat that problem, I suggest adding this to the code.
From:
(new Process($command, base_path(), ['COMPOSER_MEMORY_LIMIT' => '-1']))
             ->setTimeout(null)
             ->run(function ($type, $output) {
                 $this->output->write($output);
             });

To:
(new Process($command, base_path(), ['HOME' => getenv ('HOME'), 'COMPOSER_MEMORY_LIMIT' => '-1']))
             ->setTimeout(null)
             ->run(function ($type, $output) {
                 $this->output->write($output);
             });

@ArtMin96 thank you for your comment, but since it's not a real issue, and it's not even our package problem, it's related to how composer works, I will close this issue on Github.


Add file attachment support for pages

Links to attachments are enabled but file upload and attachment to pages not.

Added in: https://github.com/olafrv/mdtocf/releases/tag/1.1.0


Update external go packages

This PR contains the following updates:

Package
Type
Update
Change

github.com/onsi/gomega
require
minor
v1.24.1 -> v1.25.0

k8s.io/client-go
require
minor
v0.25.4 -> v0.26.0

sigs.k8s.io/controller-runtime
require
minor
v0.13.1 -> v0.14.1

Release Notes

onsi/gomega
v1.25.0
Compare Source
1.25.0
Features

add MustPassRepeatedly(int) to asyncAssertion (#619) [4509f72]
compare unwrapped errors using DeepEqual (#617) [aaeaa5d]

Maintenance

Bump golang.org/x/net from 0.4.0 to 0.5.0 (#614) [c7cfea4]
Bump github.com/onsi/ginkgo/v2 from 2.6.1 to 2.7.0 (#615) [71b8adb]
Docs: Fix typo "MUltiple" -> "Multiple" (#616) [9351dda]
clean up go.sum [cd1dc1d]

v1.24.2
Compare Source
1.24.2
Fixes

Correctly handle assertion failure panics for eventually/consistnetly "g Gomega"s in a goroutine [78f1660]
docs:Fix typo "you an" -> "you can" (#607) [3187c1f]
fixes issue #600 (#606) [808d192]

Maintenance

Bump golang.org/x/net from 0.2.0 to 0.4.0 (#611) [6ebc0bf]
Bump nokogiri from 1.13.9 to 1.13.10 in /docs (#612) [258cfc8]
Bump github.com/onsi/ginkgo/v2 from 2.5.0 to 2.5.1 (#609) [e6c3eb9]

kubernetes/client-go
v0.26.0
Compare Source
v0.25.5
Compare Source

kubernetes-sigs/controller-runtime
v0.14.1
Compare Source
Changes since v0.14.0
:bug: Bug Fixes

🐛 controllers without For() fail to start by @k8s-infra-cherrypick-robot in https://github.com/kubernetes-sigs/controller-runtime/pull/2108

Full Changelog: https://github.com/kubernetes-sigs/controller-runtime/compare/v0.14.0...v0.14.1
v0.14.0
Compare Source
Changes since v0.13.1
:warning: Breaking Changes

Add Get functionality to SubResourceClient (#2094)
Allow configuring RecoverPanic for controllers globally (#2093)
Add client.SubResourceWriter (#2072)
Support registration and removal of event handler (#2046)
Update Kubernetes dependencies to v0.26 (#2043, #2087)
Zap log: Default to RFC3339 time encoding (#2029)
cache.BuilderWithOptions inherit options from caller (#1980)
Migrate to ginkgo v2 (https://github.com/kubernetes-sigs/controller-runtime/pull/1977), this removes the pkg/envtest/printer package

:sparkles: New Features

Builder: Do not require For (#2091)
support disable deepcopy on list funcion (#2076)
Add cluster.NewClientFunc with options (#2054)
Tidy up startup logging of kindWithCache source (#2057)
Add function to get reconcileID from context (#2056)
feat: add NOT predicate (#2031)
Allow to provide a custom lock interface to manager (#2027)
Add tls options to manager.Options (#2023)
Update Go version to 1.19 (#1986)

:bug: Bug Fixes

Prevent manager from getting started a second time (#2090)
Missing error log for in-cluster config (#2051)
Skip custom mutation handler when delete a CR (#2049)
fix: improve semantics of combining cache selectorsByObject (#2039)
Conversion webhook should not panic when conversion request is nil (#1970)

:seedling: Others

Prepare for release 0.14 (#2100)
Generate files and update modules (#2096)
Bump github.com/onsi/ginkgo/v2 from 2.5.1 to 2.6.0 (#2097)
Bump golang.org/x/time (#2089)
Update OWNERS: remove inactive members, promote fillzpp sbueringer (#2088, #2092)
Default ENVTEST version to a working one (1.24.2) (#2081)
Update golangci-lint to v1.50.1 (#2080)
Bump go.uber.org/zap from 1.23.0 to 1.24.0 (#2077)
Bump golang.org/x/sys from 0.2.0 to 0.3.0 (#2078)
Ignore Kubernetes Dependencies in Dependabot (#2071)
Bump github.com/prometheus/client_golang from 1.13.0 to 1.14.0 (#2070)
Bump github.com/onsi/ginkgo/v2 from 2.5.0 to 2.5.1 (#2066)
seedling to dependabot (#2065)
Bump github.com/onsi/gomega from 1.23.0 to 1.24.1 (#2062)
Bump go.uber.org/zap from 1.21.0 to 1.23.0 (#2060)
Bump actions/checkout from 2 to 3 (#2059)
Bump golangci/golangci-lint-action from 2 to 3 (#2058)
Add Dependabot for dependency management (#2050)
Deprecate Webhook Server TLSMinVersion (#2041)
Bump kubebuilder-release-tools to v0.2.0 (#2012)
grammar: removed doubles of the word "the" (#2006)
Refactor typed_client and unstructured_client to be consistent (#1795)

:book: Additionally, there have been 4 contributions to our documentation and book. (#2034, #2030, #2017, #1992)
Thanks to all our contributors! 😊
New Contributors

@laxmikantbpandhare made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/1986
@Icarus9913 made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/1992
@Tomy2e made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/1970
@hyschumi made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/1795
@oscr made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2006
@matteoolivi made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2017
@johscheuer made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2027
@larsks made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2030
@johannesfrey made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/1999
@Eileen-Yu made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2034
@iiiceoo made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2049
@kannon92 made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2050
@zbindenren made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2056
@dependabot made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2058
@Miciah made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2057
@maciej-karas-form3 made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2051
@qiankunli made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2076
@lucacome made their first contribution in https://github.com/kubernetes-sigs/controller-runtime/pull/2087

Full Changelog: https://github.com/kubernetes-sigs/controller-runtime/compare/v0.13.1...v0.14.0

Configuration
📅 Schedule: Branch creation - "before 7am on Monday" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: go.sum
Command failed: go get -d -t ./...
go: downloading k8s.io/api v0.26.0
go: downloading k8s.io/apimachinery v0.26.0
go: downloading sigs.k8s.io/controller-runtime v0.14.1
go: downloading k8s.io/client-go v0.26.0
go: downloading github.com/onsi/ginkgo v1.16.5
go: downloading github.com/onsi/gomega v1.25.0
go: downloading github.com/google/gofuzz v1.2.0
go: downloading github.com/evanphx/json-patch/v5 v5.6.0
go: downloading github.com/evanphx/json-patch v4.12.0+incompatible
go: downloading k8s.io/klog/v2 v2.80.1
go: downloading sigs.k8s.io/structured-merge-diff/v4 v4.2.3
go: downloading github.com/go-logr/zapr v1.2.3
go: downloading k8s.io/apiextensions-apiserver v0.26.0
go: downloading k8s.io/utils v0.0.0-20221128185143-99ec85e7a448
go: downloading sigs.k8s.io/yaml v1.3.0
go: downloading gopkg.in/inf.v0 v0.9.1
go: downloading sigs.k8s.io/json v0.0.0-20220713155537-f223a00ba0e2
go: downloading k8s.io/kube-openapi v0.0.0-20221012153701-172d655c2280
go: downloading go.uber.org/atomic v1.7.0
go: downloading go.uber.org/multierr v1.6.0
go: downloading github.com/nxadm/tail v1.4.8
go: downloading github.com/google/gnostic v0.5.7-v3refs
go: downloading golang.org/x/oauth2 v0.0.0-20220223155221-ee480838109b
go: downloading gomodules.xyz/jsonpatch/v2 v2.2.0
go: downloading github.com/imdario/mergo v0.3.12
go: downloading k8s.io/component-base v0.26.0
go: downloading github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da
go: downloading gopkg.in/tomb.v1 v1.0.0-20141024135613-dd632973f1e7
go: downloading github.com/matttproud/golang_protobuf_extensions v1.0.2
go: downloading github.com/google/uuid v1.2.0
go: downloading github.com/go-openapi/swag v0.19.14
go: downloading github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822
go: downloading github.com/emicklei/go-restful/v3 v3.9.0
go: downloading github.com/mailru/easyjson v0.7.6
github.com/lunarway/cluster-identity-controller/controllers/core tested by
	github.com/lunarway/cluster-identity-controller/controllers/core.test imports
	sigs.k8s.io/controller-runtime/pkg/envtest/printer: cannot find module providing package sigs.k8s.io/controller-runtime/pkg/envtest/printer


[BUG] Rename "Type" to "Project Type"

Describe the bug
Not really a bug, more of a nitpick.
DDEV uses the term "Project Type" to refer to framework being served. Eg. Drupal10, Laravel, Typo3.
For consistency with DDEV online documention and CLI messages, I suggest it be change in the tree view.
To Reproduce

View a project in the extension

Expected behavior
Label as "Project Type"
Screenshots
(screenshot copied from README)

System (please complete the following information):

OS: Win10 & WSL2
DDEV version: 1.22.1
Docker provider: Docker Desktop
VSCode version: 1.81.1

Additional context
Add any other context about the problem here.

Sure, this change will be included in the next update.

A new update has been published including this change.


Monitor-workspace assignment not loading correctly?

Hyprland Version
 System/Version info
[muffinjets@archlinux ~]$ hyprctl systeminfo
Hyprland, built from branch v0.38.1-b at commit 360ede79d124ffdeebbe8401f1ac4bc0dbec2c91  (props: bump ver to 0.38.1).
Date: Sat Apr 6 14:24:13 2024
Tag: v0.38.1

flags: (if any)

System Information:
System name: Linux
Node name: archlinux
Release: 6.8.2-arch2-1
Version: #1 SMP PREEMPT_DYNAMIC Thu, 28 Mar 2024 17:06:35 +0000

GPU information:
01:00.0 VGA compatible controller [0300]: NVIDIA Corporation GP107 [GeForce GTX 1050 Ti] [10de:1c82] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

[muffinjets@archlinux ~]$

Bug or Regression?
Bug
Description
I have two monitors.  Ideally, I'd like to set my primary monitor to have only workspace 1 on it, and my secondary monitor to be the one that swaps workspaces.
To force this into happening, I have this outlined in my hyprland.conf:
# WORKSPACE CONFIG
# workspace=WORKSPACE,RULES
workspace=1,monitor:DP-1 default:true persistent:true
workspace=2,monitor:HDMI-A-1 default:true
workspace=3,monitor:HDMI-A-1 default:true
workspace=4,monitor:HDMI-A-1 default:true

# AUTOSTART
exec-once = hyprpaper & dunst & waybar
exec-once = [workspace 2 silent] flatpak run dev.vencord.Vesktop --disable-gpu
exec-once = [workspace 1] firefox

(And before it's said, hyprctl monitors reports the monitors with those identifications)
This not only does not lock workspaces 2-4 to HDMI-A-1, but when starting Hyprland from the display manager my firefox window starts in workplace 6 (for some reason??) on DP-1 and workspace 1 autostarted to HDMI-A-1.
For shits and grins, I tried booting with all of the workspace rules commented out, just to have hyprctl monitors report that now my monitor identifications have changed to DP-2 and HDMI-A-3.
How to reproduce
Unsure of how it can be replicated on another machine, but here's my steps:
With the hyprland.txt that I've included: (change filetype to .conf because GitHub didn't like me uploading that kind of file)

Have a DP and HDMI monitor?
Restart Hyprland/reboot to find workspace 6 open for some reason and your DP monitor not have workspace 1 on it
Comment/remove all of the lines in the "WORKSPACE CONFIG" section
Reboot to find that monitor assignment has changed
Replace WORKSPACE CONFIG to adjust to new layout
Reboot/reload to find yourself back to step 2

Crash reports, logs, images, videos
No response

To add onto this, adding more defined workspaces with workspace rules alters the number that the actual first workspace generates on in an n+2 pattern.
In the included config file, I have 4 workspaces defined with rules, and the system starts on 6.
If I were to have 12 workspaces defined, it would put workspace 14 on DP-X.

use monitor description instead of port if they keep changing
if a program forks itself when starting workspace exec rules won't work

That's a lack of knowledge on my part, but the problem where workspace 1 isn't defined to the correct monitor persists, no matter what the monitor is named

you can also only have 1 default workspace per monitor
your syntax is also missing , between each rule

That's fine, but even after correcting that and having the monitors now being assigned by the description instead of the port, the issue persists.  Now workspace 1 is getting assigned to the HDMI monitor.  I can verify that the workspace isn't coincidentally showing up there via usage, it's reliably getting asssigned to the wrong monitor.  Will reply to this with new conf to confirm.

# MONITOR CONFIG
# monitor=name, resolution, position, scale
monitor=desc:ASUSTek COMPUTER INC ASUS VP278 J9LMTF140754,     1920x1080, 0x0,  1
monitor=desc:Sanyo Electric Co.Ltd. LCD TV, 1920x1080, 1920x0, 1

# WORKSPACE CONFIG
# workspace=WORKSPACE,RULES
workspace=1,monitor:ASUSTek COMPUTER INC ASUS VP278 J9LMTF140754, default:true, persistent:true
workspace=2,monitor:Sanyo Electric Co.Ltd. LCD TV,default:true

[muffinjets@archlinux ~]$ hyprctl monitors
Monitor HDMI-A-1 (ID 0):
<EMAIL_ADDRESS>at 1920x0
	description: Sanyo Electric Co.Ltd. LCD TV
<lines skipped for brevity>
Monitor DP-1 (ID 1):
<EMAIL_ADDRESS>at 0x0
	description: ASUSTek COMPUTER INC ASUS VP278 J9LMTF140754

My current (uneducated) is that my HDMI monitor is being initialized first, so there's nowhere else for workspace 1 to go.

You confident about that one?  When I adjust my conf to account for that, the HDMI monitor (Sanyo) is now loading the incorrect resolution/position as if it's not seeing the config line at all.
# MONITOR CONFIG
# monitor=name, resolution, position, scale
monitor=monitor:desc:ASUSTek COMPUTER INC ASUS VP278 J9LMTF140754, 1920x1080, 0x0,  1
monitor=monitor:desc:Sanyo Electric Co.Ltd. LCD TV, 1920x1080, 1920x0, 1

Where?  In the workspace rules or the monitor configuration?

workspace rule

Are you confident in that?  Because when I adjust my conf to account for that, I still get the same issue.
With this configuration:

# MONITOR CONFIG
# monitor=name, resolution, position, scale
monitor=desc:ASUSTek COMPUTER INC ASUS VP278 J9LMTF140754, 1920x1080, 0x0,  1
monitor=desc:Sanyo Electric Co.Ltd. LCD TV, 1920x1080, 1920x0, 1

# WORKSPACE CONFIG
# workspace=WORKSPACE,RULES
# workspace=1,monitor:DP-1, default=true
workspace=2,monitor:desc:Sanyo Electric Co.Ltd. LCD TV

# AUTOSTART
exec-once = hyprpaper & dunst & waybar
exec-once = [workspace 2 silent] flatpak run dev.vencord.Vesktop --disable-gpu
exec-once = [workspace 1] firefox

Workspace 1 is still being defaulted onto the wrong monitor.

workspace=1, monitor:desc:ASUSTek COMPUTER INC ASUS VP278 J9LMTF140754, default:true
workspace=2, monitor:desc:Sanyo Electric Co.Ltd. LCD TV, default:true

# MONITOR CONFIG
# monitor=name, resolution, position, scale
monitor=desc:ASUSTek COMPUTER INC ASUS VP278 J9LMTF140754, 1920x1080, 0x0,  1
monitor=desc:Sanyo Electric Co.Ltd. LCD TV, 1920x1080, 1920x0, 1

# WORKSPACE CONFIG
# workspace=WORKSPACE,RULES
workspace=1,monitor:desc:ASUSTek COMPUTER INC ASUS VP278 J9LMTF140754, default=true
workspace=2,monitor:desc:Sanyo Electric Co.Ltd. LCD TV, default=true

This configuration still spawns workspace 3 on the HDMI (Sanyo) monitor when Hyprland starts.

that's not what I wrote
default:true not default=true


Feedback for "HTTP Communication via Communication Arrangements"

https://help.sap.com/docs/btp/sap-business-technology-platform/http-communication-via-communication-arrangements
the following sentence is not permitted in Eclipse at all
 out->write( lx_http_dest_provider_error->get_text( ) ).

Thanks for your feedback!
It is intended that the exception is handled within the ADT classrunner (if_adt_oo_classrun). The exception would then be output in the console. We decided to leave the definition of the class out of the code example to avoid too long examples. However, you are right that this can lead to confusion.
You can of course use your own exception handling here. It is not mandatory to output the text in the console.
I will update the documentation accordingly.
Best
Felix


serialize and display a structure as a primitive type (String)

Hello,
First of all, I'd like to thank you for your hard work. It's really the only web framework available in the Rust universe that integrates the OpenAPI documentation natively.
i'm migrating a project from actix-web to poem, and i'm stuck on one part.
I have a structure that contains two member values, but I'd like to serialize it as a String and display it as a String on the Doc (swagger) too.
#[derive(Serialize, Deserialize, Default, Clone, Debug)]
struct ID {
     salt : String,
     value : String,
}

#[derive(Object,Serialize, Deserialize, Default, Clone, Debug)]
struct TestDTO {

    #[serde(rename = "id")]
    #[oai(rename = "id")]
    id : ID,

    #[serde(rename = "name")]
    #[oai(rename = "name")]
    name : String,
}

I want it to generate this :
 "TestDTO": {
        "type": "object",
        "required": [
          "id",
          "name"
        ],
        "properties": {
          "id": {
            "type": "string"   <------------------------------
          },
         "name": {
            "type": "string"
          },

I want to force the value of "type" to "string",  and how to define your own way of serializing and deserializing the object? (before I used serde "serialize_with" )
( for example, on the utoipa crate, there's a macro that allows you to force the type of a field:   #[schema(value_type = String)] )
I tried to implement ToJSON trait, but the compiler tells me it's already implemented ( "Object" derive )
I've also tried to use the example (openapi/custom-payload), but it's not at all comprehensible to me.
if you can give me a code that shows how to do it, I'd be extremely grateful.
I'm really blocked on this part, thanks in advance for your help.

Hello again,
I took inspiration from the code here:
https://github.com/poem-web/poem/blob/master/poem-openapi/src/types/external/string.rs
and it works well, can you confirm if this is the right technique to follow or is there something simpler?

Yes, you should implement Type trait for ID.

#[oai(rename = "id")] is unnecessary, unless you want to change the name.

what I meant by my question is that "aoi" replaces "serde", is the use of serde macros useless with "poem/open-api" ?
#[serde(rename = "id")]  VS  #[serde(rename = "id")]

To avoid confusion, poem-openapi does not support parsing serde attributes. 🙂

thank you very much, that answers my question :+1:


Adds troubleshooting, delivery spec customizations to broker/trigger docs

Adds conditions and their reasons
Move prefetch count and autoscaling under Customizations
Adds deliverySpec info

/retest


Estimate the time for step 4

This is related to issue #
What is the problem/goal?
No time
What is the solution?
Yes time

@MarkSomogyvari Please merge!


Add Holidays Check function

Hi, i've found the valentines function (very funny), i decided to extends them with holidays check.
This func check find a match with month - day and write a image.
I know this commit it's only for fun, but after all doesn't exist only Valentine's day <3

I found a test line of code.
Fix it and after i open another pr.
Sorry


update function integration tests

Codecov Report

Merging #270 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #270   +/-   ##
========================================
  Coverage    55.33%   55.33%
========================================
  Files           55       55
  Lines         1688     1688
  Branches       191      191
========================================
  Hits           934      934
  Misses         691      691
  Partials        63       63

Continue to review full report at Codecov.

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


TabError: inconsistent use of tabs and spaces in indentation

Python 3.7.2 (default, Jan 10 2019, 23:51:51)
[GCC 8.2.1 20181127] on linux
Type "help", "copyright", "credits" or "license" for more information.

from openvas_lib import VulnscanManager
Traceback (most recent call last):
File "", line 1, in
File "/home/oguzbey/Desktop/virtualEnviroments/openvasEnv/lib/python3.7/site-packages/openvas_lib/init.py", line 373
elif l_tag == "severity":
^
TabError: inconsistent use of tabs and spaces in indentation

Check last commit
https://github.com/golismero/openvas_lib/commit/21b2b11fec437acc0f27dc324219c6e4be2f18a3


Receiver example not working with Arduino MKR WAN 1310

I'm trying to send basic Lora packets to the LoStik using the radio receiver example using an Arduino running the example radio sender code from https://github.com/sandeepmistry/arduino-LoRa/blob/master/examples/LoRaSender/LoRaSender.ino, but I'm not seeing any data show up on the receiving end. I'm not really sure what I'm doing wrong.
Arduino serial monitor:
14:34:36.035 -> LoRa Sender 14:34:36.461 -> Sending packet: 0 14:34:41.505 -> Sending packet: 1 14:34:46.534 -> Sending packet: 2 14:34:51.560 -> Sending packet: 3 14:34:56.607 -> Sending packet: 4 14:35:01.643 -> Sending packet: 5 ...
radio_receiver.py output:
connection made RN2903 1.0.5 Nov 06 2018 10:45:27<PHONE_NUMBER>

Based on that example code there seem to be a lot of things missing. Do you need to specify which pins the SPI is hooked up to on the LoRa module? Spi frequency? Set the coding rate? Spread factor?
On Wed, Apr 29, 2020, at 12:46 PM, Bork wrote:

I'm trying to send basic Lora packets to the LoStik using the radio receiver example using an Arduino running the example radio sender code from https://github.com/sandeepmistry/arduino-LoRa/blob/master/examples/LoRaSender/LoRaSender.ino https://www.fastmail.com/mail/compose/url, but I'm not seeing any data show up on the receiving end. I'm not really sure what I'm doing wrong.

Arduino serial monitor:
14:34:36.035 -> LoRa Sender 14:34:36.461 -> Sending packet: 0 14:34:41.505 -> Sending packet: 1 14:34:46.534 -> Sending packet: 2 14:34:51.560 -> Sending packet: 3 14:34:56.607 -> Sending packet: 4 14:35:01.643 -> Sending packet: 5 ...

radio_receiver.py output:
connection made RN2903 1.0.5 Nov 06 2018 10:45:27<PHONE_NUMBER>

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/ronoth/LoStik/issues/26, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAOZBCQDCH6FLHWHPHFHPLRPB7ZDANCNFSM4MUALWKA.

I did some research and finally found the solution to this. The issue wasn't anything to do with SPI since the LoRa module is built into the MKR WAN 1310. I had to set the spread factor on the LoStik to 7 since that's what the Arduino defaults to. I also had to set the sync word to 12 to match the default used by the Arduino. I think that's all it took to get communication up. I am still getting radio_errors for the actual data, but that's a different issue I have yet to look into.


Added a per-partition TimeBasedLimiter to KafkaExtractor

Added a per-partition TimeBasedLimiter to KafkaExtractor to allow it to limit how long it spends on any one partition, since a long-running partition will make all other partitions wait for its completion. This required moving the Limiters into the gobblin-core module.

@zliu41 Can you review? I'm not 100% sure about the new location of the Limiter classes that had to be moved so let me know if they're incorrect

I just realized that we can simply turn-on a per-mapper time limiter by setting the following properties
extract.limit.enabled
extract.limit.type
extract.limit.time.limit
extract.limit.time.limit.timeunit
So I'll see how well the per-mapper time limiter works. If it's more desirable to use a per-partition time limiter, then I'll revisit this PR.


Set image format per datastore

Author Name: Javi Fontan (Javi Fontan)
Original Redmine Issue: 3867, https://dev.opennebula.org/issues/3867
Original Date: 2015-07-08

Some datastores only accept one image format. For example, LVM and Ceph use RAW. Add the option to automatically set this format for each datastore.
This can be done setting an inherited parameter in the OpenNebula configuration file:
INHERIT_DATASTORE_ATTR = "DRIVER"

And then adding a @DRIVER@ value to the datastore template.

Original Redmine Comment
Author Name: EOLE Team (EOLE Team)
Original Date: 2015-07-10T07:39:29Z

+1
This could be related to part of #1764#note-5.
Regards.

Fixed