Absent alignment requirements for the CONTEXT struct

Which crate is this about?
windows, windows-sys
Crate version
No response
Summary
According to the definition in the Windows.h, the CONTEXT struct must be aligned by 16 bytes, but it is not neither in windows_rs, nor in windows_sys (it's just a #[repr(C)] here).
It causes an ERROR_NOACCESS errors in the GetThreadContext()/SetThreadContext() functions.
Toolchain version/configuration
No response
Reproducible example
No response
Crate manifest
No response
Expected behavior
No response
Actual behavior
No response
Additional comments
No response

Agree, there's a higher level packing/architecture bug here in metadata, transferring to that repo.
Related:

https://github.com/microsoft/win32metadata/issues/460
https://github.com/microsoft/win32metadata/issues/870


Fix toggleMark bug for selections with depth zero

When a selection range starts at depth zero, allow toggleMark commands
to succeed only if the top node both allows the mark and has inline
content. These restrictions align with the restrictions for the command
to succeed on other selections, fixing an edge case that 5692407 did not
catch.

I think it's also possible that this could be implemented maybe simply by using the parent argument to the traversal function, making it unnecessary to handle the top node in any special way.

I think it's also possible that this could be implemented maybe simply by using the parent argument to the traversal function, making it unnecessary to handle the top node in any special way.

The way it is already may be better, because it can avoid a level of recursion that the other approach might take.

There may also be a related bug here, where $to.depth is zero but $from.depth is not. I don't know if that's realistic for a document with a top node that has inline content or not.

In that vein, I pushed an additional commit that I think avoids some other unnecessary recursion. If that's wrong, or you'd prefer it as a separate PR, please let me know.

I've merged the first patch as 9a187ec. The second doesn't seem worthwhile.

Thanks for the quick response!

If it isn't too much trouble and there's no other planned work that you'd want to do here beforehand, would it be possible to make a release with this change? I'm shipping a local copy of toggleMark to work around this currently.

No, definitely no trouble. I've tagged 1.3.1

🙏🏻 Thank you!


Rotating dashboards

We've setup lots of dashing dashboards - different departments need different data. This all works great out of the box.
What we'd really like to do is have our management to rotate between all dashboards - is there anyway to do this? Would this be something to make the browser do instead? Not sure on the best approach for this...
Would love your thoughts!

There are a number of options in this list: https://github.com/Shopify/dashing/wiki/Additional-Widgets#other-third-party-tools
Please close this issue.

@protomouse Did you document anywhere how to run dashboards as slim? That would be very useful.
I tried the obvious, adding slim to the Gemfile and renaming the dashbaords to .html.slim or just .slim,and it does not seem to work. I see that you include the parts of a dashboard, however, how does the layout and the main entry point for a dashboard?


Feature: Optimistic rendering

To ensure a smooth experience for players we want to optimistically render Egg cards. When buying or breeding, we can use call then send on buyEgg and hatchEgg. The call method will let us return the token in advance of the send transaction being confirmed. We should show a timer (similar to cool off) while transaction confirms.
For Hatching, lets have the cards animate back and forth until they are ready to be hatched, then a 1.3x enlarge animation + glow to indicate that it's ready to be clicked, which will show animation and reveal the hatched animal.
Example of a new egg bought that loads in as 50% opacity while waiting for chain confirmation:

In the example above, the hatch prompt should not appear until confirmed on chain and ready to hatch.


Add scroll ability relative to caret position when keyboard is open

To fix:
https://github.com/wordpress-mobile/gutenberg-mobile/issues/436
https://github.com/wordpress-mobile/gutenberg-mobile/issues/432
This PR adds the ability to scroll to caret position when keyboard is open and adds ability to check if caret will be already visible once keyboard is open, thus, we don't need to scroll. Until this PR the automatic scrolling was only scrolling to the end of the focused component and that was causing problems when we have a long text.
After this PR the scrolling decision is given this way:

Scrolling to caret is only available for components who conform to UITextInput, this includes RNTAztecView. React native's text input doesn't provide us the caret position and hides the inner TextView from us so we can't apply scroll to caret behavior in React Native's text input, they will continue to scroll to the bottom of the component. In the future we can replace that with RNTAztecView to achieve same thing.
Why did I add the check for "Is caret at the last line of text input?"
Because automatic scroll didn't look consistent enough otherwise. When you tap some of the lines it was scrolling to caret and caret stayed just above the keyboard as expected, but, when you tap 1 line below for example, the auto scroll could move the caret to position at the top of the page because it performed scroll to the end of component this time.
Review recommendation
I recommend checking some commits separately because some of the changes are just extracting the existing code to a method so those can be differentiated from new code.
To Test

Testing instructions are at the parent PR

That's a perfectly valid concern @etoledom it is best to keep the Javascript code together with its native side and contribute back to the community. The thing is I already changed this file a lot for our needs and there are other changes coming. On some point we can consider moving the javascript code to gutenberg-mobile considering it is just 1 file and a super small interface file. I'll investigate what needs to be done to put the iOS code here, it'll be the first native code in this repo I don't know the effort to be honest.

It was easier than I though @etoledom I'll update the gutenberg-mobile PR accordingly

late note: I remembered why I put this is to our bridge in the first place @etoledom because in the first solution I was importing RNTAztecView, then I had updated it to depend on UIKit only.

Being self-contained looks better and is good for peace of mind. 😛

So true! Thanks for the detailed review @etoledom 👍


Cannot build on Centos 7

Build fails with
"[ 15%] Building CXX object tdutils/CMakeFiles/tdutils.dir/td/utils/StringBuilder.cpp.o
/home/scortopassi/td/tdutils/td/utils/StringBuilder.cpp: In member function ‘td::StringBuilder& td::StringBuilder::operator<<(const void*)’:
/home/scortopassi/td/tdutils/td/utils/StringBuilder.cpp:210:19: error: ‘snprintf’ is not a member of ‘std’
   current_ptr_ += std::snprintf(current_ptr_, reserved_size, "%p", ptr);"

Following the TDLib build instructions generator with Java > Linu> Distro: Other.
[scortopassi@buildlib-test-serena-2 build]$ export CXXFLAGS=""
[scortopassi@buildlib-test-serena-2 build]$ cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX:PATH=../example/java/td -DTD_ENABLE_JNI=ON ..
-- The CXX compiler identification is GNU 5.3.0
-- The C compiler identification is GNU 5.3.0
-- Check for working CXX compiler: /usr/local/bin/g++
-- Check for working CXX compiler: /usr/local/bin/g++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Check for working C compiler: /usr/local/bin/gcc
-- Check for working C compiler: /usr/local/bin/gcc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Could NOT find ccache (this is NOT an error)
-- Found OpenSSL: /usr/lib64/libcrypto.so (found version "1.0.2k")
-- Found OpenSSL: /usr/include /usr/lib64/libssl.so;/usr/lib64/libcrypto.so
-- Performing Test HAVE_STD14
-- Performing Test HAVE_STD14 - Success
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Check if compiler accepts -pthread
-- Check if compiler accepts -pthread - yes
-- Found Threads: TRUE
-- Performing Test HAVE_CXX_FLAG_WALL
-- Performing Test HAVE_CXX_FLAG_WALL - Success
-- Performing Test HAVE_CXX_FLAG_WEXTRA
-- Performing Test HAVE_CXX_FLAG_WEXTRA - Success
-- Performing Test HAVE_CXX_FLAG_WIMPLICIT_FALLTHROUGH_2
-- Performing Test HAVE_CXX_FLAG_WIMPLICIT_FALLTHROUGH_2 - Failed
-- Performing Test HAVE_CXX_FLAG_WPOINTER_ARITH
-- Performing Test HAVE_CXX_FLAG_WPOINTER_ARITH - Success
-- Performing Test HAVE_CXX_FLAG_WCAST_QUAL
-- Performing Test HAVE_CXX_FLAG_WCAST_QUAL - Success
-- Performing Test HAVE_CXX_FLAG_WSIGN_COMPARE
-- Performing Test HAVE_CXX_FLAG_WSIGN_COMPARE - Success
-- Performing Test HAVE_CXX_FLAG_WDUPLICATED_BRANCHES
-- Performing Test HAVE_CXX_FLAG_WDUPLICATED_BRANCHES - Failed
-- Performing Test HAVE_CXX_FLAG_WDUPLICATED_COND
-- Performing Test HAVE_CXX_FLAG_WDUPLICATED_COND - Failed
-- Performing Test HAVE_CXX_FLAG_WALLOC_ZERO
-- Performing Test HAVE_CXX_FLAG_WALLOC_ZERO - Failed
-- Performing Test HAVE_CXX_FLAG_WLOGICAL_OP
-- Performing Test HAVE_CXX_FLAG_WLOGICAL_OP - Success
-- Performing Test HAVE_CXX_FLAG_WTAUTOLOGICAL_COMPARE
-- Performing Test HAVE_CXX_FLAG_WTAUTOLOGICAL_COMPARE - Failed
-- Performing Test HAVE_CXX_FLAG_WVLA
-- Performing Test HAVE_CXX_FLAG_WVLA - Success
-- Performing Test HAVE_CXX_FLAG_WNON_VIRTUAL_DTOR
-- Performing Test HAVE_CXX_FLAG_WNON_VIRTUAL_DTOR - Success
-- Performing Test HAVE_CXX_FLAG_WUNUSED_PARAMETER
-- Performing Test HAVE_CXX_FLAG_WUNUSED_PARAMETER - Success
-- Performing Test HAVE_CXX_FLAG_WCONVERSION
-- Performing Test HAVE_CXX_FLAG_WCONVERSION - Success
-- Performing Test HAVE_CXX_FLAG_WSIGN_CONVERSION
-- Performing Test HAVE_CXX_FLAG_WSIGN_CONVERSION - Success
-- Performing Test HAVE_CXX_FLAG_WCXX14_COMPAT_PEDANTIC
-- Performing Test HAVE_CXX_FLAG_WCXX14_COMPAT_PEDANTIC - Failed
-- Performing Test HAVE_CXX_FLAG_WDEPRECATED
-- Performing Test HAVE_CXX_FLAG_WDEPRECATED - Success
-- Performing Test HAVE_CXX_FLAG_QUNUSED_ARGUMENTS
-- Performing Test HAVE_CXX_FLAG_QUNUSED_ARGUMENTS - Failed
-- Performing Test HAVE_CXX_FLAG_WODR
-- Performing Test HAVE_CXX_FLAG_WODR - Success
-- Performing Test HAVE_CXX_FLAG_FLTO_ODR_TYPE_MERGING
-- Performing Test HAVE_CXX_FLAG_FLTO_ODR_TYPE_MERGING - Success
-- Found ZLIB: /usr/lib64/libz.so (found version "1.2.7")
-- Found ZLIB: /usr/include /usr/lib64/libz.so
-- Found JNI: /usr/lib/jvm/java/jre/lib/amd64/libjawt.so
-- Found JNI: /usr/lib/jvm/java/include;/usr/lib/jvm/java/include/linux;/usr/lib/jvm/java/include /usr/lib/jvm/java/jre/lib/amd64/libjawt.so;/usr/lib/jvm/java/jre/lib/amd64/server/libjvm.so
-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY - Success
-- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY - Success
-- Performing Test COMPILER_HAS_DEPRECATED_ATTR
-- Performing Test COMPILER_HAS_DEPRECATED_ATTR - Success
-- Could NOT find Readline (missing: READLINE_INCLUDE_DIR READLINE_LIBRARY)
-- Could NOT find Readline (this is NOT an error)
-- Configuring done
-- Generating done
-- Build files have been written to: /home/scortopassi/td/build

Any idea?

cmake3 version 3.14.6
g++ (GCC) 5.3.0
gcc (GCC) 5.3.0

snprintf must be a part of a standard С++ library shipped by default with g++ 5.3.0. I expect that something is wrong with the installed libstdc++. The g++ binary is found in /usr/local/bin/g++, but it is very likely that compiler still uses libstdc++ from /usr/include instead of the shipped with the compiler.
Could you paste output of next commands on your system:
grep -r snprintf /usr
find /usr | grep stdio

Thanks for your answer, here you are outputs:
$ sudo find /usr | grep stdio
/usr/bin/systemd-stdio-bridge
/usr/lib64/perl5/CORE/nostdio.h
/usr/include/bits/stdio-lock.h
/usr/include/bits/stdio-ldbl.h
/usr/include/bits/stdio.h
/usr/include/bits/stdio2.h
/usr/include/bits/stdio_lim.h
/usr/include/stdio.h
/usr/include/stdio_ext.h
/usr/include/c++/4.8.2/cstdio
/usr/include/c++/4.8.2/ext/stdio_filebuf.h
/usr/include/c++/4.8.2/ext/stdio_sync_filebuf.h
/usr/include/c++/4.8.2/tr1/cstdio
/usr/include/c++/4.8.2/tr1/stdio.h
/usr/local/gcc53/include/c++/5.3.0/ext/stdio_filebuf.h
/usr/local/gcc53/include/c++/5.3.0/ext/stdio_sync_filebuf.h
/usr/local/gcc53/include/c++/5.3.0/tr1/cstdio
/usr/local/gcc53/include/c++/5.3.0/tr1/stdio.h
/usr/local/gcc53/include/c++/5.3.0/cstdio
/usr/local/gcc53/lib/gcc/x86_64-unknown-linux-gnu/5.3.0/include/ssp/stdio.h

Maybe should I tell compiler to use libraries in /usr/local/gcc53/include/ instead of /usr/include/ ?
The output of grep -r snprintf /usr is really weird, a huge list of "binary file xxx matches". What would you like to know?
Thanks a lot

There is no reason in grep -r snprintf /usr anymore.
The problem is definitely that you have GCC 4.8.2 libstdc++ headers in /usr/include. There is no way g++ could find right headers in /usr/local/gcc53/include by itself. You can try to add line
  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -isystem /usr/local/gcc53/include/")

before
https://github.com/tdlib/td/blob/a0fa744735e524956c6d55287121da69e85c8532/CMakeLists.txt#L160
But after that you are likely to get some link errors instead.
It would be better to completely replace default GCC instalation with GCC 5.3 or newer. There are instructions on how to do that in https://stackoverflow.com/questions/36327805/how-to-install-gcc-5-3-with-yum-on-centos-7-2.


active?

Is this project still active?
Is the django-notifications still merged?
Thanks

I am not actively developing this project, but I am still accepting bug issues and pull requests.
django-notifications is a different project, this project cannot merged with django-notifications because there are major code differences which might introduce breaking changes.
I recommend you to use django-notifications if you are looking for an actively developed project. That being said, the current release of django-notify-x is stable too and can be used in production.

Nice to know.
In the advent of falling back to django-notifications.  Was the API kept close?


Add support for callback to retrieve result from reporters

Currently, each of the reporters bundled with nodeunit keeps the test results to itself; there's no way to get them out. That means that if you want to add any functionality to a reporter (such as a growl notification), you have to make your own fork of the reporter (which is especially cumbersome since most rely on nodeunit's internal utils).
With this pull request, each reporter accepts a done callback to its options hash that runs at the end of the reporter's own done method. That way, instead of implementing an entire reporter to add growl notifications, you can just use the default reporter with a small extra callback:
reporters.default.run(['test'], {done: function(assertions) {
    var msg = 'FAILURES: ' + assertions.failures() + '/' + assertions.length + ' assertions failed (' + assertions.duration + 'ms)';
    exec("growlnotify -m '" + msg + "'");
}});

+1 but I'd rather see assertions as an event parameter for #85..

Hopefully #85 covers this

Excellent, thanks.


Bad auto update

I run fuzzwork.co.uk, and every so often, I get someone downloading the database once per minute. This isn't a small amount of bandwidth, and is entirely unnecessary.
I'm seeing them download the md5, then download the database, at lease once per minute.
This leads to me banning the IP, which is less than ideal for everyone involved.

New (updated in the past 2 months) versions have a catch to prevent this. I'll see about improving this but if people don't update there's not much I can do


GH-674 Fix Await signal to work with parameterized signals

Fixes #674

Applied to main (=2.1) and branch 2.0.


HTTP/2 client

:httpc only support HTTP/1.1, but I think it would be wise to have HTTP/2 client support too.
I'll set up an adapter for gun and update the readme with instructions. Gun seems to be most appropriate alternative to :httpc.

I've been working with :gun, but after testing it out I think it's better to go with Mint.


Feature/au 558 logout

UHF-0000

Re-enable auto logout
This works once well. After continuing dialog won't appear second time.
Do we have text/translations for this feature?
What was done

This thing was fixed

How to install

Make sure your instance is up and running on correct branch.

git checkout UHF-0000_insert_correct_branch
make fresh

Run make drush-cr

How to test

[ ] Check that this feature works
[ ] Check that code follows our standards

Designers review

[ ] This PR does not need designers review
[ ] This PR has been visually reviewed by a designer (Name of the designer)

Other PRs

Link to other PR

This is ready for review. Patch is provided and it fixes the issue with session continuing. Also lates changes from develop is merged to this branch.


If backend returns null for addresses the Transaction cannot be executed

If gasToken or refundReceiver are null the interface cannot create the transaction hash and errors (see errors below). Therefore it is not possible to execute the transaction
Safe with example transactions:
https://rinkeby.gnosis-safe.io/safes/0xd9395aeE9141a3Efeb6d16057c8f67fBE296734c/transactions
Errors shown on console:
Error: “invalid address (arg=“gasToken”, coderType=“address”, value=null)”
Error: “invalid address (arg=“refundReceiver”, coderType=“address”, value=null)”

This does not affect users that do only create transactions through the interface, correct?

Correct

just a note: we need to set them to theirs default value if they're null. Probably using makeTransaction RecordFactory would help with other undesired scenarios.

@rmeissner, This bug was reported a long time ago. Now I'm checking the transactions of the reported Safe, and none of them have a null for those values.
Seems like they defaulted to 0x0000000000000000000000000000000000000000 (@Uxio0 is this due to a change in the back-end?)
Anyway, I'll do what I said in the previous comment (https://github.com/gnosis/safe-react/issues/573#issuecomment-597109363).

Ok, I'm not being able to reproduce this.
Tried even forcing the values to null, but no errors.

Seems like they defaulted to 0x0000000000000000000000000000000000000000 (@Uxio0 is this due to a change in the back-end?)

I'm not aware of any change, but the backend allows null for some fields (like refund_receiver or gas_token) when you use the endpoint POST, but will use 0x0000...00 when indexed from blockchain.
That url you are using was previously shown using POST info, but after a reindex has now the 0x000..000 fields because it was updated from blockchain

This does not affect users that do only create transactions through the interface, correct?

@rmeissner, can you provide the steps to reproduce this? How to create transactions from outside the interface.

This does not affect users that do only create transactions through the interface, correct?

@rmeissner, can you provide the steps to reproduce this? How to create transactions from outside the interface.

Just go to the swagger https://safe-transaction.staging.gnosisdev.com/ and POST a tx setting some values like refund_receiver  or gasToken null instead of 0x000...000


I just upgrade numpy

I got ImportError: numpy.core.multiarray failed to import, so I add pip install numpy --upgrade

@SunSki Thank you!


Error on path /modules/everblock//views/js/custom-compressed1.js

Hi,
It's look like that there is an error with path for views/js/custom-compressed1.js which have to slashes.

Prestashop version <IP_ADDRESS>.
PHP : 7.4
Module Version : v4.7.1
Regards,

Hello
A new release has been published, could you try it and tell me if it solves your problem ?
Best regards

Hi CySSoO,
First of all, thanks for your hard work.
Sorry for my late reply because i was busy. I did the update but i have the same error unfortunately.
Best regards,

Could you try to delete views/js/custom-compressed1.js and then clear cache ?
Best regards

H did it but results are same.
Steps :

delete views/js/custom-compressed1.js
Clean cache via everblockbutton
Clean cache prestashop
Reinstall the new version of the module
delete views/js/custom-compressed1.js
Clean cache via everblockbutton
Clean cache prestashop

Best regards,

Good morning
Does the latest release solve your problem?
https://github.com/TeamEver/everblock/releases/tag/V4.9.0
You will need to empty the cache of your store for the changes to be taken into account.

Hello, I have the same problem. I tried deleting the file and clearing the cache, but it's the same.
I have module version 4.9.6, do you have a solution? (ps 8.1.1)

Hello @CySSoO , I have the same problem. I tried deleting the file and clearing the cache, but it's the same.
I have module version 4.9.6, do you have a solution? (ps 8.1.1)

hi @CySSoO , I have the impression that despite the latest update, we have the same problem.

Hello @A2P2R
Have you tried latest module version ?
https://github.com/TeamEver/everblock/releases/tag/V4.10.2
Best regards

Hello @A2P2R
Have you tried latest module version ?
https://github.com/TeamEver/everblock/releases/tag/V4.10.2
Best regards

Hi @CySSoO,
Yes ! And I am the same problems.
Thank you for your help

Ok, if the file is empty, in fact the internet browser can output a 404
This commit
https://github.com/TeamEver/everblock/commit/7a5a9ca1833a13abe261cf83b8d5fe034ae93341
Can solve your problem. This checks if the file is present on your server, and if it is not empty
Can you try by just updating the following file @A2P2R  ?
https://github.com/TeamEver/everblock/blob/master/everblock.php
This will be on the next release

Hi @CySSoO
thank you very much for your feedback! it works now, I can confirm.
I'd like to take this opportunity to thank you for your work on this module.
thank you

Thanks for helping and supporting us !


Install with go install

Is it possible to get the module name updated to be able to install with the following go command:
go install github.com/AKSarav/Kube-Node-Usage@latest

Error:
go: github.com/AKSarav/Kube-Node-Usage@latest: module github.com/AKSarav/Kube-Node-Usage@latest found (v2.0.0+incompatible), but does not contain package github.com/AKSarav/Kube-Node-Usage

Thanks for the idea. I will fix it soon

Happy to help.... 💖


Bug - Manager - Length of Job Term appears incorrectly in Job Builder in IE

Description
The Length of Term appears inccorectly in the preview modal of the Details step of the Job Builder, when viewed in IE.
Technical Info
Device: Desktop
Operating System: Windows 10
Browser: IE11
Steps to Reproduce
Steps to reproduce the behavior. Example:

Go to manager portal
Go to Jobs
Create new job
Continue in English
Fill in Details page
Click Save & Preview, see the error

Images

I wasn't able to reproduce this error either @tristan-orourke.

Still can't reproduce this error, so I'm going to close it.


Setting platform=darwin along with arch=ia32 fails silently, should warn

I'm not sure whose bug this is (upstream or here), but since I hit it in nativefier, I'll start by filing it here and you can advise me.  I wanted to package a 32-bit OS X Electron app, and set arch='ia32' and platform='darwin'.  Nativefier gets 61% into the process (i.e., in packaging) and then just stops, with no error, and no built output.
In the debugger I can see it silently skipping my combination here in electron-packager:
  var combinations = []
  archs.forEach(function (arch) {
    platforms.forEach(function (platform) {
      // Electron does not have 32-bit releases for Mac OS X, so skip that combination
      if (common.isPlatformMac(platform) && arch === 'ia32') return // <--------- here
      combinations.push(common.createDownloadOpts(opts, platform, arch))
    })
  })

The comment above that code is really useful, and it would be nice to up-level it to the user in the form of a warning, perhaps responding to the darwin + ia32 combo even before the app runs.  It's not obvious that it just has nothing to do, since there is no Electron support for this combo.

@humphd thanks for the report 👍, good catch! We're resuming maintenance of nativefier and we have lots to do. So if you have the time for that, a PR will be most welcome.

I'm looking through older issues, and am retesting this one (over 4 years later), and we must be doing something better, because I not see we show this when trying to do an invalid combination (such as Mac + ia32):

Packaging... This will take a few seconds, maybe minutes if the requested Electron isn't cached yet...
WARNING: The platform/arch combination darwin/ia32 is not currently supported by Electron Packager

Is this along the lines of what you were originally looking for @humphd ?

@TheCleric that sounds good, yes.  If that's the case now, then this is closable.


Snowflake create schema with managed access

Describe the feature
Snowflake managed schemas provide a different behaviour for managing permissions on objects within the schema.   For example future grants without the need of account level managed grants.
https://docs.snowflake.com/en/sql-reference/sql/create-schema.html
Additional context
snowflake specific
Who will this benefit?
Can simplify management of role grants upon objects within the schema, in particular future grants, or if schema child objects where created with a different Role to which dbt is running.

@damianslee Neat! Is this something we should think about recommending across the board, similar to the transient config (which we set to true by default)? This in particular is quite compelling:

For example future grants without the need of account level managed grants.

Are there any drawbacks to using managed schemas? Do you ever find a need for mixing schema-level and object-level privilege management?
A challenge here is that dbt doesn't have a good abstraction today for configuring databases/schemas of objects it plans to create, apart from configuring those objects (models, seeds, snapshots, etc). So if users want to enable this for some schemas but not others, I don't think there's a good place to set that config.
I was hopeful that you could accomplish this yourself today using on-run-end project hooks, via alter <schema> enable managed access as a lead-in to a future grant. Unfortunately, turning on managed access as an alter requires global manage grants (docs, exactly the kind of requirement we're trying to get away from.

After thinking about this for a few more minutes, I realized an obvious way you can achieve this behavior in your own project: reimplement the create_schema macro.
The standard way to do this is by including a macro named create_schema in your root project macros/ directory:
-- must be in root project
{% macro create_schema(relation) %}
{%- call statement('create_schema') -%}
    create schema if not exists {{ relation.without_identifier() }} with managed access
  {% endcall %}
{% endmacro %}

As of v0.21, there's another way to do this, leveraging dispatch, that also enables you to redefine this macro from within a package:
-- defined in root project or package
{% macro snowflake__create_schema(relation) %}
{%- call statement('create_schema') -%}
    create schema if not exists {{ relation.without_identifier() }} with managed access
  {% endcall %}
{% endmacro %}

# root dbt_project.yml
dispatch:
  - macro_namespace: dbt
    search_order ['<root_project_or_package>', 'dbt']

In either case, this will adjust the built-in schema creation at the start of dbt run to include with managed access, thereby allowing you to include future grants in an on-run-end hook.
So the real question is: Should we turn this on for everyone, always?

For my workplace we always use managed schema now.  Have not had a reason to switch back.
So i think default should be fine.
We are currently using Flyway and managed schema was the first thing i was looking for when reading the DBT docs.   and then raised this shortly after reading some of the examples.


Improved color temperature in script editor

Some points to defend this PR, currently the default colors make it hard to even see the name of scripts, and on top of that the colors are intrusive, it is even hard to check which of the scripts is the current one, so with this PR, the current script can be easily identified, the most used scripts are more visible than the less used ones (which is the point of the heat function) and the names are clearly visible.

This is great!
I always hated the huge areas in colors that just didn't fit at all!!!


Hampton, VA

Hi, thanks for your interest in adding your city to the website. Emails on defund12.org are all written by community members. To add your city, you must provide:

Email addresses (at least one) of the government officials you’d like to receive the message
Detailed message concerning the budgeting decisions at stake in your community

Until you provide this content, we cannot post your city.
Messages should follow our email style guidelines. Remember that messages must be specific to your locale and focused on defunding the police. If your message is copied from an external source, please provide a link to the original source.
Your request will be closed until you reply with the necessary content. Thank you!

Dear Mayor Tuck and City Council Members,
My name is [YOUR NAME], and I am a resident of Hampton. This past week, our nation has been gripped by protests calling for rapid and meaningful reevaluation of the role of policing in our communities and an end to institutionalized racism. Accordingly, it has come to my attention that the budget for 2021 is being decided as these protests continue.
Hampton PD receives a disproportionate amount of our city's financial resources. In FY 2020, the Hampton Police Division has a budget of almost $25 million.   While we’ve been spending extraordinary amounts on policing, we have not seen significant improvements to safety, homelessness, mental health, or affordability in our city.
I call on you to meaningfully restrict the Hampton PD budget and instead use those extraordinary resources towards, for example, solving homelessness, which is felt most by our Black neighbors and veterans. Research shows social programs and education to be much more effective at promoting safety and social equity than policing and incarceration.
We can be a beacon for other cities to follow if only we have the courage to change.
Sincerely, [YOUR NAME][YOUR ADDRESS] [YOUR EMAIL][YOUR PHONE NUMBER]

Email Addresses:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Thank you for submitting your city to defund12.org! Your request is now in the queue for review and publishing. We’ll reply here if we have any questions.

Changelog

"This past week" --> "In recent weeks" (since this is no longer technically true)
FY 2020 --> FY20 (per formatting on the city's website)

Edited Email Copy
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Dear Mayor Tuck and City Council Members,
My name is [YOUR NAME], and I am a resident of Hampton. In recent weeks, our nation has been gripped by protests calling for rapid and meaningful reevaluation of the role of policing in our communities and an end to institutionalized racism. Accordingly, it has come to my attention that the budget for 2021 is being decided as these protests continue.
Hampton PD receives a disproportionate amount of our city's financial resources. In FY20, the Hampton Police Division has a budget of almost $25 million. While we’ve been spending extraordinary amounts on policing, we have not seen significant improvements to safety, homelessness, mental health, or affordability in our city.
I call on you to meaningfully restrict the Hampton PD budget and instead use those extraordinary resources towards, for example, solving homelessness, which is felt most by our Black neighbors and veterans. Research shows social programs and education to be much more effective at promoting safety and social equity than policing and incarceration.
We can be a beacon for other cities to follow if only we have the courage to change.
Sincerely,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

This is ready for PR!

on it


Media query for masthead

I would like to serve different images as background in the <header class="masthead"> element. The actual image would be chosen with media query on the device's size (basically, always the same figure but with different file size).
I was wondering if it's something you've already done, or if you have suggestions/templates.
Cheers

That is something that should be easy enough to accomplish with a little bit of custom CSS. Just use media queries in a CSS file targeting the masthead and use different background images in each media query.


Added fix to doctrine.yaml (couldnt find app or default connection for orm)

Added app and default connection
-> problem on migration

Reviewed, OK


Instructor selection: Breaks between events in admin view

In the admin view, the breaks between each event listings are thin gray lines.  Consider what we can do to make the breaks more prominent (darker/heavier break lines; more prominent headers, etc)

This is how it looks without horizontal line and with slightly bigger vertical spacing between entries:

This is how it looks without horizontal line and with 100px vertical spacing between entries:

(Spacing between header 'Recruitment processes' and the first entry can be reduced)
This is how it looks with darker (about 3 times darker) horizontal line:

This is how it looks with darker (3 times darker) horizontal line and increased spacing (same spacing as in first screenshot):

Finally this is how it looks with default horizontal line and slightly bigger spacing:

(This is the closest to current look)

Moving this from the instructor selection project to the accessibility/UI project.


Make English vehicle names more consistent

Fixes #50.
Vehicles are renamed according to some rules:

Vehicles must have a name that describes the vehicle itself rather than the ride/rollercoaster as a whole
Exception: Dodgems, Flying Saucers and all their variants introduced in WW/TT. The term “Dodgems” instead of “Dodgem Cars” is used because all variants appear as unique ride in the ride list, using the vehicle name as the ride name
Use plural form unless the ride can only have 1 vehicle
Consistent capitalization
Problems reported in #50 are fixed

Affects only en-GB.

Well, why are the flower power thingies called “dinghies” then? What do you mean by this word?
Wikipedia tells me a dinghy is a boat: https://en.wikipedia.org/wiki/Dinghy
Which is obviously not the case here. No water. So it doesn't make sense to me.
About cyclops: How do you know they are flying saucers? Sure, the base ride is called flying saucers, but that doesn't mean the cyclops eyes are floating, too. Graphically, they look like vehicles on wheels. Something which the description also suggests:

Riders drive the Eye of a Giant Cyclops

You can't “drive” something that is hovering above the ground. It also sounds weird to say that these giant things actually hover above the ground like the flying saucers.
Or is “Cyclops Ride” actually the correct term? Do I have to read the word “Ride” as in “colloqial term for car” here? Or is “ride” referring to the whole building.
Sorry, not a native English speaker here. :(

I have pushed an update now.
I have reverted the bboat change. Now I see the error here. I thought this was for the flower power ride, but it's actually for the bumper boats … I was completely confused.
I have changed “Cyclops Ride” to “Cyclops Cars”.
The reason why I find it odd to call the Cyclops things “flying saucer” is because these look very big, clunky and therefore heavy. I find it hard to believe they actually float above the ground like flying saucer. The name “Cyclops Flying Saucers” would be weird. Also, the current description of this vehicle doesn't say these vehicles are floating as well.
If you still object, I will revert. But I dislike “Cyclops Ride” as being too generic of a term and also not be in plural like most vehicle names are.

Major update!
I have now added many changes to the vehicle descriptions as well, as many of these were descriptions for the ride rather than the vehicle.
Needs a fresh review (sorry).

It seems the problem runs deeper.
The problem is that the same string is used for both vehicle and ride at the same time. Which is not the case for the other rides. This seems like a strange exception.
If we use “Cyclops Ride”, then the name is correct for the ride but odd for the vehicle.
If we use “Cyclops Cars”, then the name is correct (I hope) for the vehicle, but completely wrong for the ride. And it's not only the name. It's the description, too. The description has to match for the ride, not the vehicle. Which is very weird because most JSONs in the ride directory are for vehicles.
The real solution would be to use separate strings for both ride and vehicle. The vehicle could be read from JSON while the ride name is read from vanilla RCT2 (I hope this is possible).
I agree it's more convenient to have the different car types at least for dodgems and flying saucer-type listed as different rides. It's an exception to the rule atm, but in this case it makes sense. Fewer clicks. This can stay.
AAAAAAAaaaaaaaanyway …
I agree the name “Cyclops Cars” as of now doesn't make sense in the ride window, therefore, I have now reverted this to “Cyclops Ride” in the latest force-push (no other change). But I see this as a temporary solution and I will soon post a new issue on the main repo.

BTW, I really love that OpenRCT2 has lately completely thrown away the brain-dead “one vehicle = one ride” nonsense from vanilla RCT2 with stuff like “ostrich roller coaster” which is actually just a new vehicle. This was so annoying and it was definitely the right choice to completely nuke it. :)
But I see this change still needs some work; a lot of the vehicle descriptions are still wrong/outdated as they describing the ride rather the vehicle. Hence, this PR. :)

@Gymnasiast:

They're not dodgems, but flying saucers.

But how do you know? Just because the ride class is “flying saucers” does not mean the vehicles are hovering. This particular vehicle type, just like the cyclops eye, looks quite bulky and heavy. Yeah, so my same rationale as before …
Interestingly, the JSON file is called rct2.ww.dragdodg.json. dodg sounds like “dodgems”.
Note that there's another vehicle/ride type, the “Skidoo Dodgems” (rct2.ww.skidoo.json) but is of class “flying saucers”. This seems to be the original vanilla RCT2 WW name, not altered. So not even RCT2 was so strict about this. Even in RCT2 being the ride type of “flying saucers” does not mean the vehicles can never be dodgem cars. Which makes sense, actually. “Flying saucers” as ride type is basically just dodgems but without roof.

Since this is the description that appears in the New Ride menu, I think it should mention the three long rotating arms.

This doesn't make sense because this ride simply does not have any arms. At least I don't see any.
I fixed the 2 hypen things.

Latest force-push changes:

Name of Flower Power Ride restored
Description of Flower Power Ride changed
Name of Chinese Dragonhead Ride restored

These changes were too controversial apparently. shrug
I'm not 100% happy because the vehicle includes the word “ride” but this actually seems to be a different issue, see https://github.com/OpenRCT2/OpenRCT2/issues/10194
This PR has dragged on too long for now, and I don't wanna delay it any further.

@Gymnasiast: Is anything else wrong here?

Can you reply to the line notes themselves next time? Makes it easier to keep track of changes/discussion.

Unfortunately, I am not able to reply to the line notes themselves. But I can write normal comments.

You must be mistaken. Every regular user can reply to line notes. You can add replies in the input fields marked Reply....

@Gymnasiast: Done.


[test] Add test utilities needed by integration test

Includes utilities needed to add a test to the swift-integration-tests repo:

--sync option to make requests synchronous
Support for using the index polling similar to our unit tests

@swift-ci please test

@swift-ci please test

@swift-ci please test


Add lis2mdl sensor support in SensorTile.box

Add lis2mdl sensor support in SensorTile.box.
This PR depends to #19430

@MaureenHelm
I would like to get this PR merged for v2.1.0.
Can you take a look if all ur requests have been properly addressed?


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1795 ms

Resolved: Bofrost.at performance has improved in 29a2994 after 58 minutes.


Specify requirements for action-importer gitlab

Why:
We sat for about 30 minutes trying different inputs for the command due to vague requirements for the arguments. This PR clarifies what inputs the --namespace and --project should be.
What's being changed (if available, include any code snippets, screenshots, or gifs):

--namespace clarifies that using the full path for a subgroup is needed
--project specify project slug instead of project URL

Project slug is standard GitLab lingo when creating projects in GitLab.

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@h3rmanj Thanks so much for opening a PR! I'll get this triaged for review ✨

@jfine Thank you for the review! ✨
@h3rmanj Thank you very much for this update! ✨ This repo is currently in a code freeze, but we will get this merged as soon as the freeze ends on 10/7! 🍏


Feature request: command-not-found implementation

First of all, thank you @replit for creating rippkgs and releasing it as an open-source project :)
In your release blog post, it is mentioned that "you’ve incorporated rippkgs into your command-not-found hook".
Given the scope of rippkgs (and the comparison to nix-index), would it be possible to provide a minimal command-not-found handler implementation that makes use of rippkgs (assuming Replit's implementation is too specific to be shared)?
This would be very useful to users of Nixpkgs/devShells in general, but particularly to NixOS users in flake-only (i.e. no channels) configurations [1], who usually resort to nix-index or a custom programs.command-not-found.dbPath [2].
Once again, thank you for rippkgs!

would love to share our command not found implementation! Is it enough to add documentation for this? I'd rather avoid adding the extra nix-index flake input if possible...

@cdmistman First of all, thank you for being open to that :)
To be honest, I didn't imagine it would depend on nix-index, because I thought rippkgs was a complete replacement of nix-index (although I admittedly don't fully grasp the implementation details to understand how they can complement each other).
This request (along with #30) was basically an attempt at mimicking the "ergonomics" of nix-index as a command-not-found, since I envisioned rippkgs almost being a drop-in replacement, one NixOS/Home Manager module option away.
Being in documentation is already a great improvement, which would allow building all the other things upon it (maybe externally, if they don't fit this repo). So please do share :)


Add GCP Workload Identity Federation Auth Strategy

Summary
Adding GCP Workload Identity Federation Auth Strategy. This can be used for connecting to BigQuery.
Legend Engine PR: https://github.com/finos/legend-engine/pull/589

How did you test this change?

[x] Test(s) added
[ ] Manual testing (please provide screenshots/recordings)
[ ] No testing (please provide an explanation)

Could we rebase to latest as last time that caused an issue.

Had merged master changes in this latest commit


FAQ content and styling

Is your feature request related to a problem? Please describe.
In FAQ dark mode, there is not smooth transition when we click on a question.
In light mode, no content is visible at all initially.

Its only visible after clicking on it.

Describe the solution you'd like

Adding smooth transition to faq when active and closed.
making content visible in light mode.

Additional context
Also, the home with blue color does'nt seem good with dark theme or even light theme.  The color can be changed to a better one.

Also,in the navbar,
1) the blue color does'nt really match here also. Can implement better styling on it.
2) Nothing is visible inside the blue box, a new user coming to the website will never now that they can switch between light and dark themes.
Do let me know if i will be assigned any of the tasks

GO AHEAD @saumyayadav25


DataFrame.index changes type after download

another very, very annoying bug:

task.upload_artifact and pass a DataFrame with index of type datetime64
another task downloads this artifact, data is the same, but index changed type to string!!!!

@iirekm could you provide a toy sample code to reproduce the issue ?

another very, very annoying bug:

Well... all bugs are annoying when they happen to you 😄


Unable to add BCC with $personalizationIndex

Issue Summary
It's not possible to add CC/BCC when passing a $personalizationIndex.
The issue seems to be caused by the missing $substitutions parameter when calling addRecipientEmail in addCc and addBcc in the Mail class.
$this->addRecipientEmail(
	"Bcc",
	$bcc,
	$name,
	// substitutions are missing here
	$personalizationIndex,
	$personalization
);

Error
Fatal error: Uncaught SendGrid\Mail\TypeException: $substitutions must be an array. in ...\vendor\sendgrid\sendgrid\lib\mail\EmailAddress.php on line 162
Technical details:

sendgrid-php Version: 7.2.1
PHP Version: 7.1.16

@MarvinWue, thank you for your reply.
It looks like it missed the variable after tracing the line of code :+1: .


HTTPCLIENT-1625 Completely overhaul GSS-API-based authentication backend

Thanks for the initial review @michael-o .
Regarding the renames and changing the defaults, I have specifically tried to touch as little as possible because much of the GSS/Kerberos code was not removed from codebase when disabling SPNEGO, which I assumed was for brackwards compatibility reasons.
I'm fine with your proposed changes, but those may theoretically cause issues with pre-5.3 apps, and will cause the api compare plugin to go wild.

Thanks for the initial review @michael-o .
Regarding the renames and changing the defaults, I have specifically tried to touch as little as possible because much of the GSS/Kerberos code was not removed from codebase when disabling SPNEGO, which I assumed was for brackwards compatibility reasons.
I'm fine with your proposed changes, but those may theoretically cause issues with pre-5.3 apps, and will cause the api compare plugin to go wild.

Will engage next week again.

Can you take another look at the current patch @michael-o ?
I have implemented most of your suggestions, but I had a few more questions comments regarding KerberosConfig.

Can you take another look at the current patch @michael-o ? I have implemented most of your suggestions, but I had a few more questions comments regarding KerberosConfig.

Will pick end of this week or next week.

Rebased to current master

@ok2c @stoty I gave this another conceptual thought: We said that we are going to remove the current code for good, but this PR reuses it and makes it partially undeprecated. For me, this is not straight forward. My gut feeling says that we should remove completely first  and the @stoty can readd the code he thinks is required to satisfy this PR. WDYT?

@michael-o We cannot remove old implementation unless we are willing to go to the major release cycle and release these changes as 6.0. I suggest that the old classes are left deprecated as there might still be users that rely on them and new implementation is added in parallel (with different names or in a different package).

@michael-o We cannot remove old implementation unless we are willing to go to the major release cycle and release these changes as 6.0. I suggest that the old classes are left deprecated as there might still be users that rely on them and new implementation is added in parallel (with different names or in a different package).

I see, than we need new packages names. Agreed.

I have reverted the removals.
I don't see what alternative package name we could use instead of "org.apache.hc.client5.http.auth"
Using another package might also break package level visibility.
Maybe I could prefix the new classes with "Mutual"  ?
i.e
MutualGssSchemeBase
MutualSpengoScheme
The other classes are strictly backwards compatible, like KerberosCredentials, and KerberosConfig.
Should I duplicate those as well ?

I have reverted the removals.
I don't see what alternative package name we could use instead of "org.apache.hc.client5.http.auth" Using another package might also break package level visibility.
Maybe I could prefix the new classes with "Mutual" ? i.e MutualGssSchemeBase MutualSpengoScheme
The other classes are strictly backwards compatible, like KerberosCredentials, and KerberosConfig. Should I duplicate those as well ?

I'll get back to this question this week, have some serious trouble at $work I need to solve first.

I don't think that removing deprecation is problem, especially as these classes were deprecated without providing a replacement, not because they were replaced by something.

I don't think that removing deprecation is problem, especially as these classes were deprecated without providing a replacement, not because they were replaced by something.

@stoty Un-deprecate those classes that can be kept fully backward compatible and create new classes for those that cannot

I have made the discussed changes.

@stoty @michael-o Folks, what is the reason this PR is stuck? @stoty Please rebase the latest off master to pick up Docker based compatibility tests. Feel free to disable japicmp but all integration must pass with your changes.

I am busy, can only peek once in a while. Hopefully the upcoming weeks will be better.

@stoty If you want to expedite the review process we could do the following. Once you are more or less happy with the state of things, submit changes to HttpAutheticator and AuthScheme in a separate PR and will happily review them. Kerberos specific code will have to wait until @michael-o has time to review it.

Thank you @ok2c.
I have rebased the patch and temporarily disabled japcmp.
I have also added and @Internal annotation to HttpAuthenticator, I cannot imagine realistic user code calling that directly.
(though it does not seem to apply retroactively)
I will try to make the time to split the PR.

I don't think that duplicating HttpAuthenticator is a good idea.
For the authentication shemes, a case could be made for duplicating them, as those are pluggable components.
However, HttpAuthenticator is a purely internal class, and no sane external code is expected to call it.
The only way I could think that HttpAuthenticator would be called by external code, is if someone fully re-implemented a new (Closable...)Client class (and the corresponding ...Exec class ), which is not something that I think should be covered by backwards compatibility.
The end result would be replacing HttpAuthenticator with HttpAuthenticator2, and having a dangling HttpAuthenticator, which doesn't work with the other classes anyway (and it would possibly have to be modified anyway just to compile).

@stoty Please disable japicmp for now and make changes to HttpAuthenticator  you deem necessary. I will take it from there.

Thank you.
I have already done that.


Suggestions for upcoming contest & leaderboard

The upcoming contest feature and common leaderboard is quite good.
But you can add some classification on the types of contest present, like Hackathons, Competitive, Data Science etc. This feature is present in "Challenge hunt" chrome extension as well.
It is just a suggestion from my side which can help other users also join Stopstalk.

Hey @likecs : Thanks a lot for your input. We can now crawl from the endpoint that Challenge hunt uses internally to get those tags - http://testchallengehunt.appspot.com/v1/all . Feel free to send a PR :)

The endpoint http://testchallengehunt.appspot.com/v1/all does not have any benefit as we want to show the contests in CodeChef, Codeforces, Spoj, HackerEarth, HackerRank and the endpoint classifies them as hiring or algorithmic nothing else.


Removed transport port

Current coverage is 81.21% (diff: 100%)

Merging #4326 into remove-transport-client will decrease coverage by 0.25%

@@           remove-transport-client      #4326   diff @@
=========================================================
  Files                         1312       1310     -2
  Lines                        44707      44649    -58
  Methods                          0          0
  Messages                         0          0
  Branches                      7116       7115     -1
=========================================================
- Hits                         36421      36262   -159
- Misses                        5618       5701    +83
- Partials                      2668       2686    +18

Powered by Codecov. Last update dc7bdde...8b90ce8


Updated the docs in README.md file

What changes does this PR do?

It tells the committer to click on the "Commit Changes" button, add a commit message and select the correct branch while doing this.

So is it going to marge with the main repo??

My Pleasure.


agent windows

i cant generte an agent for windows always linux help thanks

In order to generate an agent for Windows machines, use "-a windows" parameter, which will change default destination platform of your agent and provide you with a Powershell script instead of Linux command.
Cheers


Any plan to open source translaions?

Hi.
Since en-us is open-sourced, I'd like to know whether it's possible to open source translations?

Just noticed it's done.


Add trigger comment to environment variables

As discussed in issues https://github.com/janinko/ghprb/issues/229 and https://github.com/janinko/ghprb/issues/223, this change will provide the jenkins job with the pull request comment that was used to trigger the build. An example of how this could be used would be to provide the name of a test scheme in a GitHub comment - retest scheme1 please - for use by a build script.

:+1:

:+1:

Closing in favor of #313

:+1:


cs/phpdoc fix Commands

Hello,
Thank you but the builds are failing.
I'm also not a fan of adding these just for the sake of adding them.
I would prefer adding proper return types over these doc blocks.
Also on the command properties, they were removed on purpose.


il-repack doesn't work Akka.Net / Fsharp

I tried to write a basic F# lib based on akka (http://getakka.net/docs/FSharp API) and merged all the Akka dlls (and dependencies) with my lib.
The merged dll seems ok (no error in peverify).
But when I write a console app to test this lib, the Akka spawn function crashes with the following exception:
{System.ArgumentException: Expression of type 'Microsoft.FSharp.Core.Unit' cannot be used for parameter of type 'Microsoft.FSharp.Core.Unit' of method 'FunActor2 Invoke(Microsoft.FSharp.Core.Unit)'    at System.Linq.Expressions.Expression.ValidateOneArgument(MethodBase method, ExpressionType nodeKind, Expression arg, ParameterInfo pi)    at System.Linq.Expressions.Expression.ValidateArgumentTypes(MethodBase method, ExpressionType nodeKind, ReadOnlyCollection1& arguments)
at System.Linq.Expressions.Expression.Call(Expression instance, MethodInfo method, IEnumerable1 arguments)    at System.Linq.Expressions.Expression.Call(Expression instance, MethodInfo method, Expression[] arguments)    at Microsoft.FSharp.Linq.RuntimeHelpers.LeafExpressionConverter.ConvExprToLinqInContext(ConvEnv env, FSharpExpr inp)    at Microsoft.FSharp.Linq.RuntimeHelpers.LeafExpressionConverter.ConvExprToLinqInContext(ConvEnv env, FSharpExpr inp)    at Microsoft.FSharp.Linq.RuntimeHelpers.LeafExpressionConverter.QuotationToLambdaExpression[T](FSharpExpr1 e)
at Akka.FSharp.Spawn.spawnOpt[Message,Returned](IActorRefFactory actorFactory, String name, FSharpFunc2 f, FSharpList1 options)
My lib is just a basic function that creates an agent and posts a message to it (based on F# akka doc):
`
let testing() =
    let testAgent = fun () ->
            spawn system (Guid.NewGuid().ToString())
                (fun mailbox ->
                    let rec loop() = actor {
                       let! msg = mailbox.Receive()
                       mailbox.Sender() <! true
                       return! loop()
                    }
                    loop()
                    )
    let ag = testAgent()
    let res = postAndReply<bool> ag 1000 "test"
    match res with
    | Some(t) when t = true -> printfn "ok"
    | None -> ()

`
Without dll merging, the previous code is working well.
Any ideas ?
Thanks,

You get this error because you merged FSharp core dll (containing Microsoft.FSharp.Core.Unit).
From this point on, code in your assembly is referencing the merged types, and are incompatible if you try to use this library together with code referencing the FSharp dll.
Usual course of action is to only merge "implementation detail assemblies", not the ones which participate (method argument, base type, method return type...) in the public API of your assembly.

Thanks for your quick answer. Indeed, without merging FSharp.Core.dll, it's working :)


diff page rendering, stop showing diffs after first 10k?

on the diff page, suggest stop rendering diffs after 10k lines have been rendered
it hangs the window, and the user will not have any value of showing a mega diff

It's the complexity of the HTML. Too many DOM nodes. It's probably better to turn off syntax highlighting if the diff page becomes too large. But thousands of diffs will do the same.

just stop after a maximum of something, could limit the DOM nodes, but what if there are 100k lines of diff, without syntax highlighting, it would give the same problem

Another option is to render the diffs collapsed, but you still have to transfer the html

might help frontend rendering, but the user also waits for backend rendering
alternatively we need to add events for uncollapse, which is complexity

okay, not super important, closing for now


pimped out readme

Coverage remained the same at 100.0% when pulling c7c1c1b45832d2f6ea816e72edd40d907c4271ae on pimp-out-readme into e9deb984e740739a2ff84b1fc04929340f45660e on master.

Coverage remained the same at 100.0% when pulling c7c1c1b45832d2f6ea816e72edd40d907c4271ae on pimp-out-readme into e9deb984e740739a2ff84b1fc04929340f45660e on master.

Coverage remained the same at 100.0% when pulling c7c1c1b45832d2f6ea816e72edd40d907c4271ae on pimp-out-readme into e9deb984e740739a2ff84b1fc04929340f45660e on master.


rkt: initial user namespace support test by doing the uidshift at extract time

This is just a test PR to show some progress on the user namespaces feature.
The original issue can be found here "Investigating user namespaces" https://github.com/coreos/rkt/issues/986
This PR is using the uidshift at extract time method that is explained in https://github.com/coreos/rkt/issues/986.  This is only for testing so we can identify the blockers and other hidden problems, the uidbase and the uidrange are hardcoded to "10000:65536"
If you want to test it:

$ sudo ./bin/rkt -debug -insecure-skip-verify run docker://busybox -interactive -no-overlay --private-users="10000:65536"

rkt: fetching image from file:///home/tixxdz/code/endocode/rkt/bin/stage1.aci
rkt: found image in local store, skipping fetching from docker://busybox
2015/06/10 15:42:04 Preparing stage1
2015/06/10 15:42:06 Loading image sha512-a4ddd6bfb5ebf9338b02887fa1b42d95617c678f1622ad46e14bac7ea28b7d0b
2015/06/10 15:42:06 Writing pod manifest
2015/06/10 15:42:06 Setting up stage1
2015/06/10 15:42:06 Wrote filesystem to /var/lib/rkt/pods/run/4f5c61ff-0d37-447b-af27-41d6627d4494
2015/06/10 15:42:06 Pivoting to filesystem /var/lib/rkt/pods/run/4f5c61ff-0d37-447b-af27-41d6627d4494
2015/06/10 15:42:06 Execing /init
Spawning container rkt-4f5c61ff-0d37-447b-af27-41d6627d4494 on /var/lib/rkt/pods/run/4f5c61ff-0d37-447b-af27-41d6627d4494/stage1/rootfs.
Press ^] three times within 1s to kill container.
Using user namespaces with base 10000 and range 65536.
systemd 220 running in system mode. (-PAM -AUDIT -SELINUX +IMA -APPARMOR -SMACK +SYSVINIT +UTMP -LIBCRYPTSETUP -GCRYPT -GNUTLS -ACL +XZ -LZ4 +SECCOMP -BLKID -ELFUTILS -KMOD -IDN)
Detected virtualization systemd-nspawn.
Detected architecture x86-64.
Running with unpopulated /etc.

Welcome to Linux!

Initializing machine ID from container UUID.
Failed to populate /etc with preset unit settings, ignoring: File exists
[  OK  ] Created slice Root Slice.
[  OK  ] Reached target rkt apps target.
[  OK  ] Created slice System Slice.
[  OK  ] Created slice system-prepare\x2dapp.slice.
Starting Prepare minimum environment for chrooted applications...
[  OK  ] Started Graceful exit watcher.
Starting Graceful exit watcher...
Failed to set up mount unit: Invalid argument
Failed to set up mount unit: Invalid argument
Failed to set up mount unit: Invalid argument
Failed to open directory /usr/etc/systemd/system/opt-stage2-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95-rootfs-sys-fs-cgroup-memory-machine.slice-machine\x2drkt\x2d4f5c61ff\x2d0d37\x2d447b\x2daf27\x2d41d6627d4494.scope-system.slice-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95.service-cgroup.procs.mount.wants: File name too long
Failed to open directory /usr/etc/systemd/system/opt-stage2-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95-rootfs-sys-fs-cgroup-memory-machine.slice-machine\x2drkt\x2d4f5c61ff\x2d0d37\x2d447b\x2daf27\x2d41d6627d4494.scope-system.slice-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95.service-cgroup.procs.mount.requires: File name too long
Failed to open directory /run/systemd/system/opt-stage2-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95-rootfs-sys-fs-cgroup-memory-machine.slice-machine\x2drkt\x2d4f5c61ff\x2d0d37\x2d447b\x2daf27\x2d41d6627d4494.scope-system.slice-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95.service-cgroup.procs.mount.wants: File name too long
Failed to open directory /run/systemd/system/opt-stage2-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95-rootfs-sys-fs-cgroup-memory-machine.slice-machine\x2drkt\x2d4f5c61ff\x2d0d37\x2d447b\x2daf27\x2d41d6627d4494.scope-system.slice-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95.service-cgroup.procs.mount.requires: File name too long
Failed to open directory /run/systemd/generator/opt-stage2-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95-rootfs-sys-fs-cgroup-memory-machine.slice-machine\x2drkt\x2d4f5c61ff\x2d0d37\x2d447b\x2daf27\x2d41d6627d4494.scope-system.slice-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95.service-cgroup.procs.mount.wants: File name too long
Failed to open directory /run/systemd/generator/opt-stage2-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95-rootfs-sys-fs-cgroup-memory-machine.slice-machine\x2drkt\x2d4f5c61ff\x2d0d37\x2d447b\x2daf27\x2d41d6627d4494.scope-system.slice-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95.service-cgroup.procs.mount.requires: File name too long
Failed to open directory /usr/lib/systemd/system/opt-stage2-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95-rootfs-sys-fs-cgroup-memory-machine.slice-machine\x2drkt\x2d4f5c61ff\x2d0d37\x2d447b\x2daf27\x2d41d6627d4494.scope-system.slice-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95.service-cgroup.procs.mount.wants: File name too long
Failed to open directory /usr/lib/systemd/system/opt-stage2-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95-rootfs-sys-fs-cgroup-memory-machine.slice-machine\x2drkt\x2d4f5c61ff\x2d0d37\x2d447b\x2daf27\x2d41d6627d4494.scope-system.slice-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95.service-cgroup.procs.mount.requires: File name too long
[  OK  ] Started Prepare minimum environment for chrooted applications.

Startup finished in 171ms.
/ #  cat /proc/self/uid_map
0      10000      65536

I tried to build flavor=coreos and it fails on:
stage1/rootfs/usr/bin/systemd-nspawn: unrecognized option '--private-users=10000:65536'

Because the coreos flavor has systemd-nspawn-v215. But this problem will go away soon when #845 get fixed :)
I tried on flavor=usr-from-host (with systemd-v220) and it works, I don't have your errors about "File name too long" and "Failed to set up mount unit: Invalid argument". I guess you are trying with systemd-git-master and something changed in systemd? I will test.

Yes I am trying with systemd git HEAD, I have identified the call site that fail in src/core/mount.c:mount_setup_unit(), but still don't know why...

The patch with chown is not complete: root still belong to nobody (65534)
/ # touch /foo
touch: /foo: Permission denied
/ # ls -ld /
drwxr-xr-x   17 65534    65534         4096 Jun 10 15:35 /
/ # id
uid=0(root) gid=0(root)

@jonboulle hi before we go further in this, it would be awesome if some of you guys could comment on the "Investigating user namespaces" issue:  https://github.com/coreos/rkt/issues/986
So when we get back to this we do it the right way. Please see the last comment, section "2) rkt and uid-shifts for rootfs", points 2.2 and 2.3.
Thanks

I rebased this branch on master, fixed the conflicts and completed the patch with the chown without hardcoding the numbers:
https://github.com/endocode/rkt/commits/alban/rkt-userns
With that, it works both in the rkt run test and in the rkt prepare && rkt run-prepared test. The root directory is also chowned correctly so touch /foo also works.
I think my next steps should be to reorganise the chown patch better, starting with a pull request for the pkg/* changes.

Failed to open directory /usr/etc/systemd/system/opt-stage2-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95-rootfs-sys-fs-cgroup-memory-machine.slice-machine\x2drkt\x2d4f5c61ff\x2d0d37\x2d447b\x2daf27\x2d41d6627d4494.scope-system.slice-sha512\x2da4ddd6bfb5ebf9338b02887fa1b42d95.service-cgroup.procs.mount.wants: File name too long

I couldn't reproduce this error because the memory cgroup is disabled on my Debian (my /proc/cgroups has memory enabled=0). @iaguis could reproduce it on Arch. After merging the changes from master, the error is gone. It might be because of #1178 (reducing the path length) or some other fixes.
So I guess the first step is to rebase this branch on git master.

An update on this PR, the work is currently done on this branch https://github.com/endocode/rkt/tree/tixxdz/rkt-userns-03-08-2015
Some patches and bug fixes are already pushed in master and when finishing that branch we will open a new PR. Thanks

An update on this one, running branch https://github.com/endocode/rkt/tree/tixxdz/rkt-userns-03-08-2015
sudo ./build-rkt-0.7.0+git/bin/rkt --debug=true --insecure-skip-verify run --mds-register=false --private-users=true docker://busybox --interactive
rkt: fetching image from docker://busybox
Downloading cf2616975b4a: [====================================] 32 B/32 B
Downloading 6ce2e90b0bc7: [====================================] 1.15 MB/1.15 MB
Downloading 8c2e06607696: [====================================] 32 B/32 B
2015/08/05 16:50:05 Preparing stage1
2015/08/05 16:50:05 Writing image manifest
2015/08/05 16:50:06 Loading image sha512-9d710100ce6769569b12a39100318bfed5b6b98115ee6315b724c11658db3751
2015/08/05 16:50:06 Writing pod manifest
2015/08/05 16:50:06 Setting up stage1
2015/08/05 16:50:06 Wrote filesystem to /var/lib/rkt/pods/run/29b28cd6-adbc-4d8f-bea4-8b3e437c3f75
2015/08/05 16:50:06 Pivoting to filesystem /var/lib/rkt/pods/run/29b28cd6-adbc-4d8f-bea4-8b3e437c3f75
2015/08/05 16:50:06 Execing /init
Directory /var/lib/rkt/pods/run/29b28cd6-adbc-4d8f-bea4-8b3e437c3f75 doesn't look like an OS root directory (os-release file is missing). Refusing.

Hi, so I failed to rebase all the patches and didn't want to lose more time. Sorry but the new PR is here: https://github.com/coreos/rkt/pull/1250

Closing, the new PR is in #1250


stellar-hd-wallet accounts command not working on Windows 10

I have tried multiple times to use the stellar-hd-wallet on various Windows 10 systems and every attempt gives errors when using the accounts command.
The "new" command allows me to create a 24 word seed. If I then try to generate the associated keys by running the "accounts" command, it always asks for an input of "How many words?" No matter what value is entered (supposedly it should be 24 in my case) it returns an error message with the date, time and "Invalid value"

I have figured out what the problem is, and how to correct it. The problem is on line 18 of commands/io.go. That line strips of the newlines from the command line entry. On windows, however, stripping off the "\n" is not sufficient, since the newlines on windows consist of "\r\n".
The basic fix is to import the runtime package and then determine the operation system using runtime.GOOS. If the operating system is windows, then it can strip off "\r\n" from the command line entry, otherwise the code will remain unchanged and strip off only the "\n".
I am more than happy to submit a pull request for the updated code. Would it be best that this be done against the master branch or the separate stellar-hd-wallet branch, which appears to be 100 days behind the master branch? I welcome any feedback.

Closed in https://github.com/stellar/go/pull/345


storage: barrier manager panicked at Hummock error: SST {} is invalid

Found when benchmarking with nexmark source. Since the connector source reader is busy generating messages, the heartbeat begins to timeout, and then the meta service deletes expired workers and finally panics. Is this caused by worker removal?
The configuration is the same as #4553.
2022-08-10T06:29:24.840569Z  WARN risingwave_meta::cluster: Deleted expired worker Worker {
    worker_node: WorkerNode {
        id: 1,
        r#type: ComputeNode,
        ...

thread 'tokio-runtime-worker' panicked at 'failed to execute barrier: Hummock error: SST 22 is invalid
  backtrace of `MetaError`:
   0: std::backtrace_rs::backtrace::libunwind::trace
             at /rustc/9067d5277d10f0f32a49ec9c125a33828e26a32b/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
   1: std::backtrace_rs::backtrace::trace_unsynchronized
             at /rustc/9067d5277d10f0f32a49ec9c125a33828e26a32b/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
   2: std::backtrace::Backtrace::create
             at /rustc/9067d5277d10f0f32a49ec9c125a33828e26a32b/library/std/src/backtrace.rs:328:13
   3: <risingwave_meta::error::MetaError as core::convert::From<risingwave_meta::error::MetaErrorInner>>::from
             at ./src/meta/src/error.rs:52:33
   4: <T as core::convert::Into<U>>::into
             at /rustc/9067d5277d10f0f32a49ec9c125a33828e26a32b/library/core/src/convert/mod.rs:550:9
   5: <risingwave_meta::error::MetaError as core::convert::From<risingwave_meta::hummock::error::Error>>::from
             at ./src/meta/src/error.rs:94:9
   6: <core::result::Result<T,F> as core::ops::try_trait::FromResidual<core::result::Result<core::convert::Infallible,E>>>::from_residual
             at /rustc/9067d5277d10f0f32a49ec9c125a33828e26a32b/library/core/src/result.rs:2134:27
   7: risingwave_meta::barrier::GlobalBarrierManager<S>::complete_barriers::{{closure}}
             at ./src/meta/src/barrier/mod.rs:823:21
   8: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
             at /rustc/9067d5277d10f0f32a49ec9c125a33828e26a32b/library/core/src/future/mod.rs:91:19
   9: risingwave_meta::barrier::GlobalBarrierManager<S>::barrier_complete_and_commit::{{closure}}
             at ./src/meta/src/barrier/mod.rs:742:68
  10: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
             at /rustc/9067d5277d10f0f32a49ec9c125a33828e26a32b/library/core/src/future/mod.rs:91:19
  11: risingwave_meta::barrier::GlobalBarrierManager<S>::run::{{closure}}
             at ./src/meta/src/barrier/mod.rs:563:22
  12: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
             at /rustc/9067d5277d10f0f32a49ec9c125a33828e26a32b/library/core/src/future/mod.rs:91:19
  13: risingwave_meta::barrier::GlobalBarrierManager<S>::start::{{closure}}::{{closure}}
             at ./src/meta/src/barrier/mod.rs:511:45

Is this caused by worker removal?

Yes. Hummock meta will reject any "invalid SST" that is from a removed worker.

Further more I'd like to add generation id for worker and reject any "invalid SST" from older generation.
That is, if a worker is restarted before its heartbeat timeout, its worker id remains unchanged, but its generation id will increase.

It panics only because we don't enable recovery https://github.com/singularity-data/risingwave/blob/8135a39585d6f5bfe9d58697165a9a3b1fd9ea1a/src/meta/src/barrier/mod.rs#L762
So is it OK to close this issue?

Got it. Sounds okay to close this.


Update protocol 408

I cant use my own version.
So it's just to inform you that you have to add compatibility to the protocol 408
Thanks!

It will be updated tonight (3-4 hours later) in JST! Thank you for the PR!


Update smoke test sample app to use AGIC 1.5

The Java sample app that is used in the CI/CD pipelines needs to be updated to use the latest release which is compatible with AGIC 1.5 and Kubernetes 1.22

Azure Vote and the Java app have both been updated and tested with AGIC 1.5 / K8S edge.


Whats the lates version of kodi I can install on my Boxee Box???

Just wondering if I can apply an update???

You'd probably get a better response if you listed some details about
your Boxee -- current firmware version and any hacks you've done to it.

Just wondering if I can apply an update???

ok well, Ive done the easy install from "http://boxeed.in/forums/viewtopic.php?f=5&t=1216"
so Im not sure what version it gave me...

Out of my depth. I haven't done Kodi / XBMC. See if anyone else replies.

ok well, Ive done the easy install from
"http://boxeed.in/forums/viewtopic.php?f=5&t=1216http://boxeed.in/forums/viewtopic.php?f=5&t=1216"
so Im not sure what version it gave me...

Reply to this email directly or
https://github.com/boxeehacks/boxeehack/issues/626#issuecomment-187378973view
it on GitHub.

Alright thx for the reply tho. Whats the latest version that u installed???

can I install Isengard on my Boxee Box


[JENKINS-60316] Fix escaping single quotes in Unix file paths

JENKINS-60316
Points addressed

a single quote is not allowed between single quotes, even if escaped
a backslash requires escaping if to be used literally in a reqular expression

This escaping has previously already been used in quoteUnixCredentials which was added in commit 62872bccaf3f0e02c663acfdd85be2b38bddf1e8 - see here.
Also see https://www.gnu.org/software/bash/manual/html_node/Single-Quotes.html.
I tested it manually on Linux, with a SSH key and a username/password credential, with a job inside a folder that contains spaces and single quotes.
There is no test infrastructure available yet to test such things automatically - right?

@MarkEWaite do you want to consider this for the next release?

@darxriggs I made a first attempt to duplicate the problem from a Jenkins installation last week before I released 3.1.0-beta.  I was unable to duplicate the problem.
Since I could not duplicate the problem, I was unwilling to include a fix that I could not confirm actually fixed the issue.
This fix will need to wait for a release of git client plugin after 3.1.0.
I would love to have an automated test that shows the problem or an acceptance test on a branch in my jenkins-bugs repository or an acceptance test in my docker-lfs repository.

I tested it manually on Linux, with a SSH key and a username/password credential, with a job inside a folder that contains spaces and single quotes.

@MarkEWaite This info was not helpful to reproduce it? It could reproduce it that way.


add a recent happy-eyeballs

let's see whether the syntax works (for >=0.6.0).
PR checklist:

[x] PR comment includes a reproducible test plan
[x] Change has no security implications (otherwise ping the security team)

I think I am gonna let @mjambon or @aryx handle this one


[#129847885] Use native git resource instead of our fork

🚨 DO NOT MERGE:
What
Concourse 2.1.0 has included our pull request to add GPG signature verification into the git resource, therefore we no longer need to use our fork.
Remove the custom resource type from the pipelines used by the deployer Concourse. As stated in the commit message, we will have to keep using the fork in our bootstrap Concourse as the version is fixed in the AMI.
How to review

Create a test branch off of this branch
Remove SKIP_COMMIT_VERIFICATION=true from the dev section of the Makefile (line 89)
Run the pipeline against a signed commit:

It can be any signed commit, doesn't have to be a merge commit: git commit -S -m 'foo'
Check the paas-cf resource is fetched in successfully. You only need to run the pipeline-lock job at the start.

Run the pipeline against an unsigned commit. Repeat as above, but asserting its failure.

Who can review
Anyone but @alext or me.

This works. Previous issues were due to switching branch and resource type at the same time. What happens still is that you might get the same version twice when switching the resource type. I guess this is OK. This also alleviates issue where resource doesn't pick the latest commit, but one before - but which is in this case the same version.


Running ansible-navigator from a non-root account causes error

ISSUE TYPE

Bug Report

SUMMARY
There is an issue with ansible-navigator or possibly podman when using the su or su - switching from the root user to an unprivileged user and attempting to run the ansible-navigator command.
ANSIBLE-NAVIGATOR VERSION
ansible-navigator 2.2.0

CONFIGURATION
ansible-navigator:
  execution-environment:
    image: ansible-automation-platform-22/ee-supported-rhel8:latest
    pull:
      policy: missing
  playbook-artifact:
    enable: false

LOG FILE
https://gist.github.com/noelmiller/410131a2ffc51e756bef00dcd8e9af0f
STEPS TO REPRODUCE

Connect to ansible control node server using root account using ssh: ssh root@control-node
Do one of the following to switch to the unprivileged user account:

su user
su - user
su --login user

Navigate to directory with relevant ansible-navigator.yml config
Run ansible-navigator (or any subcommand)
You will receive the error I listed above

EXPECTED RESULTS
Expect the interactive ansible-navigator tui to appear, but instead I get the error below.
ACTUAL RESULTS
-------------------------------------------------------------------------------------------
Execution environment image and pull policy overview
-------------------------------------------------------------------------------------------
Execution environment image name:     ansible-automation-platform-22/ee-supported-rhel8:latest
Execution environment image tag:      latest
Execution environment pull arguments: None
Execution environment pull policy:    missing
Execution environment pull needed:    True
-------------------------------------------------------------------------------------------
Updating the execution environment
-------------------------------------------------------------------------------------------
Running the command: podman pull ansible-automation-platform-22/ee-supported-rhel8:latest
WARN[0000] RunRoot is pointing to a path (/run/user/1000/containers) which is not writable. Most likely podman will fail.
Error: default OCI runtime "crun" not found: invalid argument
  Error: Execution environment pull failed
   Hint: Check the execution environment image name, connectivity to and
         permissions for the registry, and try again

ADDITIONAL INFORMATION
This could be an issue with podman itself, I did find another resolved github issue talking about switching user from root to unprivileged user here: https://github.com/containers/podman/issues/8052

One more additional thing I will add.
When you login directly over SSH as the unprivileged user, it does map the XDG_RUNTIME_DIR correctly and the container will pull or run properly.
echo $XDG_RUNTIME_DIR
/run/user/1000

When doing a su user from the root user, it does not map the XDG_RUNTIME_DIR correctly. It is mapped to the root user's value.
echo $XDG_RUNTIME_DIR
/run/user/0

When doing a su - user from the root user, it does not map the XDG_RUNTIME_DIR at all, it's completely blank
echo $XDG_RUNTIME_DIR

When doing a su --login user from the root user, it does not map the XDG_RUNTIME_DIR at all, it's completely blank
echo $XDG_RUNTIME_DIR

If you attempt to map the XDG_RUNTIME_DIR manually, it still gives a similar error
XDG_RUNTIME_DIR=/run/user/1000

echo $XDG_RUNTIME_DIR
/run/user/1000

podman run docker.io/hello-world
WARN[0000] Failed to get rootless runtime dir for DefaultAPIAddress: lstat /run/user/1000: no such file or directory
WARN[0000] RunRoot is pointing to a path (/run/user/1000/containers) which is not writable. Most likely podman will fail.
Error: default OCI runtime "crun" not found: invalid argument

It appears the podman team is aware of this bug and looking for a better way to warn issues of the potential pitfall of using su or sudo to run rootless podman: https://github.com/containers/podman/issues/14603
I personally would like to understand why there can't be a permanent fix for this issue or what is preventing su working properly with podman


test_grow9: for parent->child position propagation

PR https://github.com/xamarin/flex/pull/37 adds a failing test.  I'm not sure if that's expected -- it doesn't seem so for me, though -- but maybe I misunderstand something:
test_grow9 (test_grow.c:232): failed test `flex_item_get_frame_y(_item) == 50.0' (is 0.0)
test_grow9 (test_grow.c:233): failed test `flex_item_get_frame_y(_item) == 50.0' (is 0.0)

To me, this looks like, while the position of the second intermediate node (child2) is computed properly, it fails to be propagated to its children (child21, child22).

.and then I see test_children4, and I realize that children coordinates are probably parent-relative.


Update the sql dump to latest version

When looking at the sql dump, the fields created and modified are still there, while their names in production contains _at at the end of the fields. We should probably try to keep this dump up to date!

completed with https://github.com/lbryio/chainquery/commit/75c046d3e9798e78dbcbf715a41ebc53048bdb15


[QUESTION] Is there any curator helm chart for Open Search !

Hi Team,
We are searching for open search curator helm chart cause the old one for elastic search is not compatible with open search cluster v2.x, if something like that is available rather than inventing the wheel from the first , thanks in advance

[Triage] Could you help here? @prudhvigodithi Thanks.

Hey @sambo2021 for OpenSearch to solve the purpose of curator you can leverage OpenSearch Index Management to handle your index and snapshot management for snapshots. For now there is no direct helm chart for this but a cron job can be installed that can use  API's for index and snapshot management. The OpenSearch operator has some functionality for this though, please check https://github.com/Opster/opensearch-k8s-operator/issues/278.
@sambo2021 can you please contribute if you think its useful to have Index Management and snapshot management operations controlled via helm?
@bbarani @peterzhuamazon @TheAlgo please add your thoughts as well.

Hey @sambo2021 for OpenSearch to solve the purpose of curator you can leverage OpenSearch Index Management to handle your index and snapshot management for snapshots. For now there is no direct helm chart for this but a cron job can be installed that can use API's for index and snapshot management. The OpenSearch operator has some functionality for this though, please check Opster/opensearch-k8s-operator#278. @sambo2021 can you please contribute if you think its useful to have Index Management and snapshot management operations controlled via helm? @bbarani @peterzhuamazon @TheAlgo please add your thoughts as well.

hi prudhvigodithi :
i already tried to do simple code to manage index based on number of days and index prefix
https://github.com/sambo2021/opensearch-curator
and honestly i wanted to simplify that on did by
https://github.com/isaackuang/docker-opensearch-curator

Thanks @sambo2021 do you think you can create a helm chart with that image? and yes we can add it under https://github.com/opensearch-project/helm-charts/tree/main/charts.
Adding @isaackuang @xaocon

I have a workable helm-chart inspired from elastic-curator.
Let me prepare with it.


add KubeArmor ARM support

ref: #614
Tested on Raspberry Pi 4
Kernel: Linux 5.15.0-1005-raspi
Distribution: Ubuntu 22.04 LTS

[x] Support enforcement
[ ] Support observability

[x] accept and connect syscall
[ ] support other syscalls

Codecov Report

Merging #802 (3d769f5) into main (b373341) will decrease coverage by 1.93%.
The diff coverage is n/a.

:exclamation: Current head 3d769f5 differs from pull request most recent head a79b20d. Consider uploading reports for the commit a79b20d to get more accurate results

@@            Coverage Diff             @@
##             main     #802      +/-   ##
==========================================
- Coverage   41.02%   39.08%   -1.94%
==========================================
  Files          30       30
  Lines        9041     9041
==========================================
- Hits         3709     3534     -175
- Misses       4858     5054     +196
+ Partials      474      453      -21

Impacted Files
Coverage Δ

KubeArmor/core/dockerHandler.go
0.00% <0.00%> (-53.34%)
:arrow_down:

KubeArmor/common/common.go
38.61% <0.00%> (-4.07%)
:arrow_down:

KubeArmor/core/kubeArmor.go
50.11% <0.00%> (-2.63%)
:arrow_down:

KubeArmor/enforcer/runtimeEnforcer.go
31.54% <0.00%> (-1.35%)
:arrow_down:

KubeArmor/monitor/processTree.go
94.62% <0.00%> (-1.08%)
:arrow_down:

KubeArmor/monitor/logUpdate.go
72.88% <0.00%> (-1.02%)
:arrow_down:

KubeArmor/monitor/syscallParser.go
62.53% <0.00%> (-0.76%)
:arrow_down:

KubeArmor/core/kubeUpdate.go
48.47% <0.00%> (-0.63%)
:arrow_down:

KubeArmor/feeder/policyMatcher.go
49.42% <0.00%> (-0.42%)
:arrow_down:

KubeArmor/core/crioHandler.go
68.90% <0.00%> (+5.04%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.

CI failing issue will possibly be fixed by https://github.com/kubearmor/KubeArmor/pull/778

CI failing issue will possibly be fixed by #778

The CI was randomly failing, re-ran the test and it just passed.


customMerge on constructor

I want to merge a @js-joda/core type, these apparently can't be cloned simply by value... so I'm looking at customMerge, but I realize I'll more or less have to write it for every key...  What I'd really like to do, is have some kind of a check function that could work on the constructor, and then run the custom merge.
maybe something like
mergers:[ {
    should( key, x, y ): boolean,
    merge( x, y ): unknown
}]

this way the algorithm will just continue if should returns false
in this case all I really want is to return right if defined, and left if not, since these are immutable value objects no need to try to do a special copy.

This sounds like it overlaps with https://github.com/TehShrike/deepmerge/issues/184 and https://github.com/TehShrike/deepmerge/issues/176 – how do the solutions proposed in those threads sound to you?


session file driver bug in windows enviroment

Hi,
I am using CI3 and session driver is "file". Every time when I want to logout then following error showing. Please see attachment.
This is happening only when I work on windows environment (XAMPP) but its ok in production server.
Previously I opened a topic on this but eventually the bug exist so far. This should be fixed.
More Clue::
** Strange behavior is sometimes it works. Most of the time its happened when long time pass as logged in.

After several time logout its works sudden.
I have used $this->session->sess_destroy(); in logout method
after that my redirect is : redirect('abc/controller', 'refresh');
I have huge ajax on page

problem solved


Use matrix rotation when rotating by a multiple of 90 degrees

Jimp has recently merged a change to the rotate algorithm where rotating by 90 degrees uses a more optimal matrix rotation algorithm. (https://github.com/jimp-dev/jimp/pull/1209) This is causing one of our tests to fail. We should implement this shortcut as well as rotating a matrix is a lot easier than actually doing the math on where a pixel should be relocated to.

Fixed in 0.1.0!


Fix opening new untitled query by double click actions

For https://github.com/microsoft/azuredatastudio/issues/16074
Reverting logic back to original method and just changing the input created to be our own (which from talking with the VS Code person writing the override stuff is likely our best choice here)
Changed the query editor service interface to specify returning full UntitledQueryEditorInput for use here. Using interfaces is generally preferred, but we needed these to be full EditorInput objects and given the low use of this service and specialized nature of it I don't see this as being a huge deal here. We can always change it easily enough in the future if needed since it's just an internal service.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 42.762%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
26232

Relevant Lines:
55689

💛  - Coveralls


[Functions] Close InputStreams properly

Motivation
InputStreams should always be closed properly. There were two locations related to function
upload/download where streams weren't closed.
Modifications
try-with-resources block is used to handle an InputStream

/pulsarbot run-failure-checks

@zymap Can you review this?

/pulsarbot run-failure-checks

I found a few more InputStream leaks. I was focusing on Pulsar Functions upload/download related streams, but I happened to find a few others when I did a quick search.


Better version declarations.

It would be good to provide the version number in some better way, e.g. commit tags. Currently, if I wanted to make a script to package RIFE automatically , I'd have to parse the first line of README.md for the version, but of course there's a big chance it will break in the future. There might be also better ways to do it than the tags, I don't use git often, I just found it after quick searching.

It might be easier to manually package it instead of cloning the repo and hoping that nothing breaks.
Alternatively, you could fork it.

I've written a simple PGKBUILD (bascially a bash script) for arch linux (I don't think I'm going to publish it though, I don't want the responsibilty of maintaining it), which packages RIFE from this repo already. The versioning and the models are the only problems. I could resolve them with rclone and maybe parsing #41 now that I think about it, but surely I'm not the only one, who would benefit from those changes, so I though about bringing those issues up.

There should be no major changes in the future. At present, the infrastructure has been basically completed, and generally only the model structure or parameters will be changed.

I think you could still use tags for updating the version, every time you update the model you post a comment in #41 and also update README.md, so you could attach the version number to the commit. It's your choice, so I'll not close this issue just yet, feel free to close it or I can do it, after hearing your decision.
Also, thank you for a quick reply, at least I'll be able to start implementing the workarounds quickly.

Nice, I'll keep updating #41.


Version bump: 2.6.0

This isn't really my commit "to make", but its missing and this is open source after all :)

Made irrelevant by bee4817ebea0eaa592143c7825f5c96f040c84e4.


SyntaxWarning: "is" with a literal. Did you mean "=="?

/content/ee-LandsatLinkr/landsatlinkr.py:726: SyntaxWarning: "is" with a literal. Did you mean "=="?  if i is 0:
/content/ee-LandsatLinkr/landsatlinkr.py:1010: SyntaxWarning: "is" with a literal. Did you mean "=="?  if i is 0:
these warnings appear when importing the landsatlinkr library in python. should I change these to ==? Seems appropriate here!

Hi Annie! Yes, you're right those would be better as ==. It looks like the pattern i is  shows up a few times in the script. They should probably all be switched - I don't think identity check is ever needed, just the equality. Yes, please update!

Done! Thank you!

Thanks, Annie!


Review calculate_max_long and calculate_max_short for accuracy

A lot has changed in Hyperdrive over the last few months. We should make sure that the math used in these functions is actually correct and fix any discrepancies that we find. This should address any of the lingering known issues in the SDK.

While doing this it would be good to verify that #797 and #732 are resolved. This is also related to #894.


Ingestion of samples into the in-memory OOO Chunk

@Dieterbe I'm marking this as done, please correct me if I'm wrong.


Winch aarch64 jmp

Hey 👋
This PR implements jmp, jmp_table and branch instructions for winch targeting aarch64.
#8321

Thanks for reviewing this work 🙂
Should I push the tests ? They are obviously large (13 and 12Mb).
Please tell me if there is anything else that I can improve

Should I push the tests ? They are obviously large (13 and 12Mb).

We have tests of the underlying parts (MachBuffer and its island-insertion logic); that is probably fine for coverage here, vs. checking in a very large test, IMHO. If we wanted to do better in the future we could add the equivalent of Cranelift's chaos mode choice to sometimes artificially lower the island threshold, to avoid the need for very large inputs to trigger this; but no need to do that in this PR.


Missing .meta file generates error

The LICENSE.md has no meta file and it generates the following error in the Unity editor:
Asset Packages/net.tnrd.humanizr/LICENSE.md has no meta file, but it's in an immutable folder. The asset will be ignored.

+1 still an issue


Logging compilation errors in invocation path

Resolves #924
With this changes, when a function with compilation failures is invokec, user logs will change from this:
2016-12-12T23:26:44.824 Function started (Id=d49d6446-c7b7-4222-a005-9d770b24dd9a)
2016-12-12T23:26:44.824 Function completed (Failure, Id=d49d6446-c7b7-4222-a005-9d770b24dd9a)
2016-12-12T23:26:44.839 Exception while executing function: Functions.ManualTriggerCSharp4. Microsoft.Azure.WebJobs.Script: Script compilation failed.

To this:
2016-12-12T23:20:55.128 Function started (Id=3a28d9bc-f933-428e-a789-147d11460f73)
2016-12-12T23:20:55.191 Function compilation error
2016-12-12T23:20:55.191 (5,9): error CS1061: 'TraceWriter' does not contain a definition for 'Test' and no extension method 'Test' accepting a first argument of type 'TraceWriter' could be found (are you missing a using directive or an assembly reference?)
2016-12-12T23:20:55.191 Function completed (Failure, Id=3a28d9bc-f933-428e-a789-147d11460f73)
2016-12-12T23:20:55.206 Exception while executing function: Functions.ManualTriggerCSharp4. Microsoft.Azure.WebJobs.Script: Script compilation failed.

The changes also reduce the number compilation related logs against system logs. Limiting them to function target creation only (when first compiling the function for execution).

@paulbatum can you take a look at the before and after output and let me know if there's anything else you think we should output?

Much better. LGTM.


Add delete feature modal

Issue: https://github.com/zalando-incubator/pazuzu-ui/issues/71

92c4afe - added notice if there are no features to list view
ebac189 - fixed form label animations and search bar
5a2b319 - added materialize modal to confirm feature delete
21c27d5 - added angular2-materialize module, added some tooltips and wave effects
c8dd575 - added simple delete trigger button for demo

Please also point this PR to a corresponding issue on GitHub

👍

Looks good to me too. 👍

👍


refactor: [M3-6287] - MUI v5 Features > Users

Description 📝
MUI v5 migration for Users
Major Changes 🔄

Ran tss-react codemod on src > features > Users.
Replaced default export with named export.
Removed index.tsx file.

Preview 📷
The work on this PR is contained to only migrate the styling to the newer API.
How to test 🧪

Verify the functionality around the Users feature works as expected.
Look for any UI defects or regressions in the Users landing page.
Visit account/users and validate adding new users and changing the User Permissions.

Found no new defects on the Account users page
It would be nice if we could fix the positioning of these buttons that broke due to #9341 but this may be out of scope of this ticket

I went ahead and took care of fixing this UI regression within this PR.

Looks good! only found one small visual regression

Nice catch! This has been fixed with my latest changes.

merging this on behalf of @carrillo-erik


Flashy messages don't disappear

This is a regression that can be reproduced at the master branch. Basically, when you click the cross of a flashy message to make it disappear, it just erases the contents of it but it doesn't disappear. Like this:

This is going to be fixed once I merge the 'UI' branch into master


about change the proteins' length to a same lenth

Hi, author,Good job for this code!
But I have a problem.
If I have some proteins, such as ['VRWFPFDVQHCKLK', 'PFDVQHC',...]
As you can see, they have different length, but I want to deal them to a same length.
So can I use the padding method in NLP? If so, which token should I pick as the pad character?

Hi @mheinzinger. Could you please tell me how this model handles different length inputs?

Hi @xinformatics , we are using the tf-bilm implementation of ELMo to train SeqVec. The Batcher class handles the padding: https://github.com/allenai/bilm-tf/blob/master/bilm/data.py#L193
From a quick glance (did not check this in a while), I saw that the length of the longest sequence within a batch is determined and that all sequences shorter than this length are padded using a special token:
https://github.com/allenai/bilm-tf/blob/master/bilm/data.py#L126