Add support of cookies to the Rest Client module

Describe the issue
Version v24 adds support for cookies to the HTTP calls. The Rest Client module currently doesn't support this.
Expected behavior
The Rest Client module should also support adding and reading cookies.
Steps to reproduce
The Rest Client module has no functions to add or read cookies.
Additional context
The Rest Client module supports working directly with the underlying HttpClient, and the HttpRequestMessage and HttpResponseMessage. In that way, cookies are supported. But it would be more developer-friendly if the module itself exposes functions to work with cookies.
I will provide a fix for a bug

[X] I will provide a fix for a bug

Of course approved!


DataTranslator registry

Is this request specific to one implementation of Sponge?
No
Sponge Version
1.16.5-36.2.5-8.1.0-RC1153
What are you requesting?
https://github.com/SpongePowered/SpongeAPI/blob/9684a0ccc22507688626698c80d9c5e5f2a6ceda/src/main/java/org/spongepowered/api/data/DataManager.java#L159
DataManager needs a way to register additional data translators which are returned by that method.
Otherwise a plugin wanting to serialize an object that does not implement DataSerializeable cannot do so

Found out why this is:
DataManager#registerTranslator was deprecated in API7 recommending to use GameRegistryEvent.Register<DataTranslator<?>>
But in API8 DataTranslator is no longer a RegistryType and as such we have no registration anymore :(


Support for host and port based policies

Description
This PR introduces support for host and port based activation. One can associate Trireme with the base host namespace as a default action. In addition, one can associate Trireme with specific services listening to specific ports.
This also cleans up the systemdutils with a more structured implementation.
Test plan
Requires additional test for the new functionality.

Codecov Report

Merging #376 into master will decrease coverage by 0.55%.
The diff coverage is 33.57%.

@@            Coverage Diff             @@
##           master     #376      +/-   ##
==========================================
- Coverage   56.32%   55.77%   -0.56%
==========================================
  Files          51       51
  Lines        6088     6152      +64
==========================================
+ Hits         3429     3431       +2
- Misses       2369     2414      +45
- Partials      290      307      +17

Impacted Files
Coverage Δ

trireme.go
47.97% <0%> (-0.49%)
:arrow_down:

configurator/configurator.go
40.38% <0%> (-0.35%)
:arrow_down:

monitor/contextstore/contextstore.go
61.81% <100%> (ø)
:arrow_up:

monitor/linuxmonitor/linuxMonitor.go
48.11% <21.73%> (-11.67%)
:arrow_down:

monitor/linuxmonitor/processor.go
41.31% <31.88%> (-6.02%)
:arrow_down:

monitor/rpcmonitor/rpc.go
67.25% <47.36%> (-11.82%)
:arrow_down:

Continue to review full report at Codecov.

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


Vagrant linux image requires cmake for git branch

In order to run bundle install successfully on the git branch, the Vagrant linux image (or any linux install) requires that the cmake library is installed or otherwise the rugged 0.21.4 gem will fail to install.
Perhaps add this library to the markusproject/ubuntu image or just add to the instructions that cmake is required separately... by simply running sudo apt-get install cmake?

@oneohtrix Can you please add cmake to the image?

@oneohtrix @david-yz-liu It's also going to require gitolite.
sudo apt-get install gitolite should get the files required if one chooses to use Git functionality later on. The package will install but won't get setup unless the user chooses to run sudo dpkg-reconfigure gitolite after.. but I'll provide instructions on that somewhere on the Wiki.

@oneohtrix Sorry couple more dependencies: sudo apt-get install libssh2-1 libssh2-1-dev libgpg-error-dev

Oh crap, sorry I didn't see this! I'll add it ASAP.

@oneohtrix Its actually okay! I edited the Vagrant instructions to account for this.

Well if @oneohtrix adds it to the image, that's better in the long run since it removes an installation step.

@oneohtrix @david-yz-liu Okay, Lawrence if you look at this wiki page https://github.com/MarkUsProject/Wiki/blob/master/SettingUpVagrantEnvironment.md (not sure why its not on the main MarkUs wiki yet) you can see the additional setup I had to do to get Gitolite working.

hey @david-yz-liu @SoftwareDev -- can this be closed?

Yes, thanks.


Apply Second Reduction Rule inside of Builder

Right now, the bdd_builder and zdd_builder do not account for the user generating the same node twice.

[ ] Create a hash value of each node (Maybe drawing inspiration from the hash function in FUTURE_WORK.md)
[ ] Implement a hash-table on-top of tpie::array without array-doubling (or at least an upper bound on its allowed size).
[ ] We can ask the user for a cache-size (which is probably somehow related to the max-width) of what they want to create. This sets the size of the hash-table.

[ ] When the user adds a new node, compute the hash and look up, whether it was already created earlier. If the user generates the same node twice, then reuse the one from the hash-table.
[ ] When doing a lookup in the hash-table, then if one visits a node, that the user has no references to anymore (check the reference count of the std::shared_ptr) then it can be freed. This probably means, that one needs to reinsert all of the collisions.

If the cache-size is set to 0, then it works just as it always did. I'm not sure of yet, whether this or the maximum possible should be the default value.

In many ways, this makes the builder on-par with the streaming BDD algorithms of Shin-ichi Minato and Shinya Ishihara. So, it is probably not that bad an idea.

Question is: what should the default size be? (A) 0, i.e. no cache (same behaviour as now) or (B) as large as possible?


(feat) Add rust to codegen and add support for scalar type

Changes

 Adds rust visitor to codegen
 updates codegen snapshot with rust snapshot to be able to generate rust code
 Adds support for scalar field
 replaces single quote with double quote for consistency fixes test against this change

Author Checklist

[x] Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
[x] Vital features and changes captured in unit and/or integration tests
[x] Commits messages follow AP format
[x] Extend the documentation, if necessary

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 99.216%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
2018

Relevant Lines:
2026

💛  - Coveralls


Obj file not viewable in blender

The obj file that is created from dem2tin is not view able in blender.  Is there some reason that this would be the case? The original tif file depth map imports just fine as a DEM in BlenderGIS.

I am unable to reproduce this, the generated .obj imports into blender 2.79 for me, but I think there is some problem with the file, faces are not showing correctly:

Set the import option 'Z up' and your viewport clip distances appropriately.
In my test the imported obj was rotated -90 about Z, and the origin was very far away from the geometry, so the resulting object is translated away from the GeoTIFF's origin... :-/


Backwards compatibility for ConwayWdrlNotDelegatedToDRep

Can we get increased clarity on the exact semantics of the ConwayWdrlNotDelegatedToDRep rule?
In particular, how does it interact with script withdrawal credentials, and in particular those from previous eras?
Many dApps depend on the ability to "withdraw" 0 assets to delegate logic to a staking script and save on execution units. There is some worry that the new "must be delegated to a dRep" could break Plutus v1 and Plutus v2 scripts, as they are not able to delegate to a dRep, and thus would be immediately unable to execute a withdrawal, locking potentially millions of dollars of user funds with the Conway hard fork.
Will delegatedAddrs from here include registered script credentials? If not, I believe the proposed implementation suffers from the concern above.
I believe we need to make this condition either:

Only apply to public key credentials
Automatically delegate script credentials to the Abstain dRep
Automatically delegate Plutus v1 and Plutus v2 script credentials to the Abstain dRep

To avoid breaking backwards compatibility.

(as a side note, I'm very proud of myself for navigating the Haskell code base well enough to find where this condition was implemented LOL)

well spotted @Quantumplation !

@JaredCorduan It was actually someone else, Riley (@ILikeCardano) on Twitter, afaik.  I'm just the one who thought to bring it to Github to ask about it, rather than arguing in a twitter space :P

Oof, I wasn't aware of this trick/abuse (classic https://xkcd.com/1172/). I'll need to think a bit about this, but obviously we'll have to do something about it. Here's another hacky way to solve this: just always allow withdrawals of 0 coins. Automated delegation seems bad, because you can't delegate without having a credential registered first, so adding this would potentially impact other parts of the system as well.

heh, yea, it turns out this trick can save an absurd amount of execution units, so it's been heavily used at this point.
Exempting withdrawal 0 is a partial solution, but might still catch some people unawares; With the Sundae contracts, for example, we explicitly fail when attempting to delegate so that this staking address can never earn rewards, but others may not have done the same thing.
We were discussing it on twitter, and probably the simplest solution is to filter script credentials out of wdrlCreds here.
That would allow script credentials to withdrawal without delegating, without undermining the goal of this rule: If the goal is to get an acknowledgment of governance from people, or an explicit opting out, then creating a proxy stake script to avoid delegating to the abstain DRep seems to serve the same role of "explicit opt out", and will only be an option for the most dedicated of technical users heh.

@jmhrpr Good to know! We hadn't thought of that case in particular, but we did think of this for MIR transfers by allowing a withdrawal of non-zero rewards in our scripts; it's just pretty inconvenient because most transaction builders aren't going to monitor the reward amount and build the correct tx, so at worst it'd cause a small hiccup until someone did withdraw the rewards.

Yeah, there's an even easier way in Conway: when proposing a GA, you can specify any stake address to return the deposit to. It's probably worth building a system that doesn't suffer when there's money in the reward account.
We could just exclude scripts in general, I don't think there'd be an issue with this. However, it's a bit inconsistent and I'm not sure people even care about this, so maybe we can just get rid of it entirely.

It sounds like there are only two sensible solutions here:

either prevent withdrawals for only for KeyHash stake credentials that have not delegated to a DRep that are. This would essentially preserve the current behavior for PlutusV1 and PlutusV2 scripts
or drop the feature completely

After some discussions with various parties it seems like the former is the best path forward, since that still takes care of the common case of a regular user being forced to delegate to a DRep, while not affecting power users that rely on scripts for staking.
Any objections in going with this solution?

Nope!
That would effectively equate to just filtering ScriptCredentials out of wdrlCreds here right?

That would effectively equate to just filtering ScriptCredentials out of wdrlCreds here right?

Yep, that's exactly it

It sounds like there are only two sensible solutions here:

either prevent withdrawals for only for KeyHash stake credentials that have not delegated to a DRep that are. This would essentially preserve the current behavior for PlutusV1 and PlutusV2 scripts
or drop the feature completely

After some discussions with various parties it seems like the former is the best path forward, since that still takes care of the common case of a regular user being forced to delegate to a DRep, while not affecting power users that rely on scripts for staking.
Any objections in going with this solution?

Would this feature be retained in PlutusV3? That would be desired.

Would this feature be retained in PlutusV3? That would be desired.

@zygomeb None of the scripts will be affected by this feature, that includes PlutusV3 onwards

@lehins confirming that upcoming node 9.0 will have have this feature included?
https://x.com/IOHK_Charles/status/1799817186204057922
Need compatibility or we risk bricking all of DeFi

Thus feature does not affect 9.0 but it will 10.0, because that is when we leave the bootstrap phase.And yes, I can confirm that plutus scripts will not be affected. So, no worries, no bricking of DeFi will be allowed.


revamp the top right "x" and arrow

🦄 Select your Profile

[ ] Streamer
[ ] User

📍 Where would be the Improvement?
None
ℹ Describe your Idea
I'm not sure how to improve this, but multiple users have stated that the "x" and the arrow at the top right are very confusing. At first, some thought that it was to minimize the chat so that they could see the full screen. Or if they are clicking around on some of the games to close the popup for those games or features they thought it was to remove it of the screen. Right now its to close the the stream and go back to the main site.

⚡ Do you have an example?
No response
😇 Anything else you would like to share?
No response

We should re-think the ex as we are still getting confusing feedback. Here is a video to elaborate
https://www.loom.com/share/e827b2a56ae5457d90e9268fee84828e?sid=bc58ac37-aa4b-4e47-aadc-23c78cc86fbd


docs: add missing documentation for indentation example

Description
This PR adds the missing documentation for the indentation example.

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 remained the same at 93.197%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2298

Relevant Lines:
2348

💛  - Coveralls

sorry! I was on vacation and didn't see this in time 🙏🏽❤


[WARN] : topic 'odom_comb' is not published by MIR

Hi,
I have an issue that this is similar to #17 although I am using the newest version mir_robot on MIR100 with 2.0.18.
In my case, though, when I launch mir.launch there are two warning messages for the unpublished topics:

mir_status
odom_comb

I am not sure if this is related to the other issue I have that is also related to another issue #7.
Although I update the system time and synchronize the MIR time from the device I have huge delays in tf_monitor.
And, because of that I can't generate the map in rviz, etc. I would appreciate any help. Thanks!
Best,
Tarik

Hi,
I am also facing the same issue. The system runs perfectly on MiR web interface but mir_driver is missing both of the topics as you mentioned above. When generating map, it will generate some data occasionally but mostly it would just drop messages for the tf transform.
I also realized while try to map, it would occassionally show a perfect map exactly the same as the web interface and then fall back to a totally irrelevant map.


Finish new orphaned blocks functionality

In #8991 @bitterdev worked on #3234 . The solution that that PR came to is really close, but needs some updates.

When you add blocks from this it reloads the page. This doesn’t happen whenever we add anything else from this panel. We don’t want a full page reload. I know you're aware of this because it was in your comment earlier, I just think we need to have time to fix this.
In the original pull request, if you had no orphaned areas, it shows all the blocks in the sidebar. I think this was just a logic error and I fixed it (See: https://github.com/concrete5/concrete5/pull/9076/commits/46096d8e9a9a7ab4124817c735aa74ab1ce77f19#diff-bf126826b8eca421801edc03e59fb652R78)
We need to modify my new showOrphanedBlocks method to detect whether the option should be shown. If that method returns true we show the option in the dropdown, otherwise not.
If you drag an orphaned block into the page not at the top of the area, right now it reloads and the block is at the top of the area. It doesn’t respect you adding the blocks into the middle of the area after other blocks. Probably related to number 1.
I wasn’t able to delete an orphaned block from the panel, I think the z-index of the delete link needs to be higher? Or maybe something else is happening?

Because of point number 3: The problem is that it is only possible on client side to determinate the used blocks. The only thing i can do is to pass the area names to the query params when fetching the sidebar content. But a server-side-only method will not work. Only with dirty hacks like making a curl request in a background and parsing the DOM, analyzing the code of the theme (which is not 100% working too if you remember the old comments) .. So this method needs to be removed. But i can implement such an query to the controller based on the query params.

Here's what I'd do:
when users open the panel we don't show the Orphaned Blocks menu item. Instead, we detect the list area names via JavaScript, and perform an ajax request to check if we need to show that menu item.
This ajax request should send area names via POST (pages may contain a lot of area names, and it's safer to avoid long querystrings).
The PHP endpoint of that ajax request should return true as soon as it find the first orphaned block; in case no orphaned block is found, it returns false.
If the client javascript receives that true response, it shows the menu item, otherwise not.
I think it's the only sane way to do that.

@mlocati Yes post is better of course because of the url length limitation. But thats not the point.

User click click on the plus icon in the toolbar
The controller use the active tab that is stored in the session.
The controller returns the view of the active tab

If you want to hide the menu item, or display the first tab (in case that orphaned blocks is selected but not available for this page) you can do so server-side.
To calculate if there are orphaned blocks available you can use the existing algorithm which also takes use of the given area names that are passed by the get/post params.
But what you can't do is: Implement a server-side-only-query to check if orphaned blocks are available without having the area names first from client side.
Therefore i have created this comment to get andrew's approval to remove the showOrphanedBlocks method.
An AJAX request to check if orphaned blocks are available is redundant because you can query that also in the same request when loading the sidebar view.
And if you mean to hide the orphaned blocks menu item on client side think about that when a tab is selected it will be also the start view when a user is re-opening the sidebar. And if there are no orphaned blocks available for the current page you need to display the first tab instead. And because of the current implementation and also because of performance reasons only the view of the selected tab that is stored in the session will be returned and not all views at once. So just hiding the elements in CSS is not a solution in this case.
So conclusion: change to post, remove the method from the view and execute the query in the controller and all good.

This makes sense, @bitterdev . I think that's a reasonable solution: we're going to pass the area names to the panel (hopefully by POST, as we can all agree it'll be less likely to break that way) from the browser to the backend. And then check the area names within the controller like you've said.
This makes sense to me. I approve!

This is complete!


ostree.config: add aboot (Android) bootloader config option

We want to add aboot to the list of possible bootloaders so we can distinguish if we are using aboot or one of the other bootloaders.
Signed-off-by: Eric Curtin<EMAIL_ADDRESS>

Thanks guys!


Octave Transpose for MIDI/CV Track

It would be great if the MIDI/CV track had a octave +/- transpose option. I currently use a Beast's Chalkboard between my per|former cv out and my oscillators but it seems like this could be done in the software (and save me 8hp!).

Now that you've incorporated new midi filtering capabilities, it would be great to be able to drop the lowest note of a chord a few octaves to be used as a bass voice (and vice versa)

depending on the oscillator I'm using, the arpeggiator tops out while arpeggiating upwards. This way I'd be easily able to knock the arp pattern down a few octaves if I hear this happening in a live setting.

Will be part of 0.1.31 release.

Oh just realized yoy where asking for octave and not transpose parameter.


need help, how can i fix this

  File "C:\AI\google-tts-and-lm-studio-local-server\kitt_g_voice.py", line 5, in <module>
    import whisper           # Import whisper library for audio transcription
  File "C:\AI\google-tts-and-lm-studio-local-server\venv\lib\site-packages\whisper.py", line 69, in <module>
    libc = ctypes.CDLL(libc_name)
  File "C:\Users\nelson\AppData\Local\Programs\Python\Python310\lib\ctypes\__init__.py", line 364, in __init__
    if '/' in name or '\\' in name:
TypeError: argument of type 'NoneType' is not iterable

I am not totally available right now but I wanted to respond.  Here is how I deal with errors.  Paste the same exact error into gpt 3.5 (free) or GPT 4.  Gpt will immediately tell you what is wrong and offer the fix.  I do it all the time.  That is how I got this repo working.  Keep doing that until it fixes the issue for you.  I don't know if you tried that already but that is what I can offer at the moment.  I will check back in the morning.  I hope this helps.

thanks for your fast response,
i did tired ask gpt, it just ask me (venv) pip install --force-reinstall whisper ,
and import ctypes
but the error still there.

I just added more Info above.

Still not able to fix it.


ENGINE in OPENSSL

What is the concept of an engine in the context of OPENSSL?
What are its specific use cases?

You can read a great deal of information about ENGINEs in the project documentation : https://github.com/openssl/openssl/blob/OpenSSL_1_1_1-stable/doc/man3/ENGINE_add.pod
Consider that the current development of the next major release of OpenSSL is moving that concept even further, into a new architecture based on "providers".
You can read more about the design of the upcoming 3.0.0 release at: https://www.openssl.org/docs/OpenSSL300Design.html
In 3.0.0 ENGINEs will be deprecated in favor of providers, but there should still be at least limited support for the existing ENGINE API.
Another great source of info about differences between previous release and the upcoming 3.0.0 can be found in the wiki: https://wiki.openssl.org/index.php/OpenSSL_3.0

The links you shared were of great help.
Btw, how do i list the supported engines on my hardware?

The links you shared were of great help.
So, as far as i understand it OpenSSL has built-in support for cryptographic acceleration using the ENGINE object type. if i have a hardware device with a super fast implementation of ECDSA, then i can set the engine to that hardware device (instead of NULL), which means that the operations are now computed by the hardware device instead of the default OpenSSL software layer. Am i correct?

Yes, ENGINEs were originally designed exactly for supporting hw implementations, but their flexibility allowed them to be generally used as "bags" of alternative or new implementations, even in software.
E.g. the GOST engine is comprised of software implementations that enrich the capabilities of libcrypto by allowing users to dinamically load support for cryptographic algorithms mandated by the Russian government.
In other cases they have been used as research tools to, e.g., test and validate alternative implementations of already supported algorithms.

Also, the implementations of the engines are done by specific hardware vendors and later, depending on the need, the application needs to load it dynamically?

Depending on the version, OpenSSL has historically provided a set of built-in engines, some of which are built statically into the library at compile-time and others result in "dynamic" engines that are loadable at run-time.
The details of how an application can make use of a specific set of dynamic engines depend on a few factors.
In general the recommended way is to edit the system-wide or application-specific OpenSSL config file, so that it has an "engines" section to provide all the information necessary to load the desired set of functionality.
Applications that always fully initialize libcrypto by loading the config file don't need any further change, and enabling or disabling this or that engine is completely transparent.
Unfortunately, up to 1.1.1, while applications linking against libssl are always required to fully load the config file, applications that only link against libcrypto are not strictly required to load the configuration file, so your mileage about transparent ENGINE support in libcrypto-only applications might vary: some of these applications might need to either be modified to properly load the config file on init or to explicitly try to load an engine programmatically.
Beware that the upcoming 3.0 release will introduce the concept of providers, that will supersede the ENGINEs. While the ENGINE API will still be supported, it will be considered legacy, deprecated, and there might be usability issues in trying to use advanced features of the provider architecture and legacy ENGINEs at the same time.

Btw, how do i list the supported engines on my hardware?

https://www.openssl.org/docs/man1.1.1/man1/engine.html

Btw, how do i list the supported engines on my hardware?

I should probably expand a little bit more about this, but the previous comment was already quite long.
Anyway: as mentioned before the openssl engine CLI tool can be used to inspect engine objects and capabilities.
On my Ubuntu 18.04.4 LTS development machine, using the distribution installation of OpenSSL, I can observe the following:
; openssl engine -t -c -vvvv
(rdrand) Intel RDRAND engine
 [RAND]
     [ available ]
(dynamic) Dynamic engine loading support
     [ unavailable ]
     SO_PATH: Specifies the path to the new ENGINE shared library
          (input flags): STRING
     NO_VCHECK: Specifies to continue even if version checking fails (boolean)
          (input flags): NUMERIC
     ID: Specifies an ENGINE id name for loading
          (input flags): STRING
     LIST_ADD: Whether to add a loaded ENGINE to the internal list (0=no,1=yes,2=mandatory)
          (input flags): NUMERIC
     DIR_LOAD: Specifies whether to load from 'DIR_ADD' directories (0=no,1=yes,2=mandatory)
          (input flags): NUMERIC
     DIR_ADD: Adds a directory from which ENGINEs can be loaded
          (input flags): STRING
     LOAD: Load up the ENGINE specified by other settings
          (input flags): NO_INPUT

I am using the default configuration provided by the distribution maintainers here, which is not loading any extra dynamic ENGINE, while the openssl CLI apps all try to enable all available builtin engines.
The two engines listed are two built-in static ENGINEs compiled inside my libcrypto:

rdrand provides optional support for Intel RDRAND
dynamic is the static engine that allows libcrypto to load dynamic engines

To check what other dynamic engines are available in my system:
; openssl version -a
OpenSSL 1.1.1  11 Sep 2018
built on: Tue Nov 12 16:58:35 2019 UTC
platform: debian-amd64
options:  bn(64,64) rc4(16x,int) des(int) blowfish(ptr)
compiler: gcc -fPIC -pthread -m64 -Wa,--noexecstack -Wall -Wa,--noexecstack -g -O2 -fdebug-prefix-map=/build/openssl-kxN_24/openssl-1.1.1=. -fstack-protector-strong -Wformat -Werror=format-security -DOPENSSL_USE_NODELETE -DL_ENDIAN -DOPENSSL_PIC -DOPENSSL_CPUID_OBJ -DOPENSSL_IA32_SSE2 -DOPENSSL_BN_ASM_MONT -DOPENSSL_BN_ASM_MONT5 -DOPENSSL_BN_ASM_GF2m -DSHA1_ASM -DSHA256_ASM -DSHA512_ASM -DKECCAK1600_ASM -DRC4_ASM -DMD5_ASM -DAES_ASM -DVPAES_ASM -DBSAES_ASM -DGHASH_ASM -DECP_NISTZ256_ASM -DX25519_ASM -DPADLOCK_ASM -DPOLY1305_ASM -DNDEBUG -Wdate-time -D_FORTIFY_SOURCE=2
OPENSSLDIR: "/usr/lib/ssl"
ENGINESDIR: "/usr/lib/x86_64-linux-gnu/engines-1.1"
Seeding source: os-specific
; openssl version -a |grep ENGINESDIR
ENGINESDIR: "/usr/lib/x86_64-linux-gnu/engines-1.1"
; ls /usr/lib/x86_64-linux-gnu/engines-1.1
afalg.so  capi.so  padlock.so

In the default system-wide ENGINE directory I have three engines available. For each of them we could use the openssl engine CLI to try to load them and get more information.
; openssl engine -vvvv -c -tt afalg
(afalg) AFALG engine support
 [AES-128-CBC, AES-192-CBC, AES-256-CBC]
     [ available ]
; openssl engine -vvvv -c -tt capi
140255949046208:error:260B606D:engine routines:dynamic_load:init failed:../crypto/engine/eng_dyn.c:485:
140255949046208:error:2606A074:engine routines:ENGINE_by_id:no such engine:../crypto/engine/eng_list.c:335:id=capi
; openssl engine -vvvv -c -tt padlock
140358876758464:error:260B606D:engine routines:dynamic_load:init failed:../crypto/engine/eng_dyn.c:485:
140358876758464:error:2606A074:engine routines:ENGINE_by_id:no such engine:../crypto/engine/eng_list.c:335:id=padlock

Apparently the afalg ENGINE is the only one that successfully loads: it is meant to support (part of) the Linux Kernel AF_ALG userspace Crypto API (requires Linux >= 4.1.0).
The capi engine should provide support for the Microsoft CryptoAPI, while padlock should support the VIA PadLock Instruction Set on supported VIA CPUs.
3rd party engines can be installed in the default location or somewhere else, depending on the openssl config file and on the use of the OPENSSL_ENGINES env var.
; OPENSSL_ENGINES=/tmp openssl engine -vvvv -c -tt afalg
140552048570816:error:25066067:DSO support routines:dlfcn_load:could not load the shared library:../crypto/dso/dso_dlfcn.c:117:filename(/tmp/afalg.so): /tmp/afalg.so: cannot open shared object file: No such file or directory
140552048570816:error:25070067:DSO support routines:DSO_load:could not load the shared library:../crypto/dso/dso_lib.c:162:
140552048570816:error:260B6084:engine routines:dynamic_load:dso not found:../crypto/engine/eng_dyn.c:414:
140552048570816:error:2606A074:engine routines:ENGINE_by_id:no such engine:../crypto/engine/eng_list.c:335:id=afalg

Thanks for your prompt replies.
This information was really helpful so as to connect the dots.


pex command failing with version error for sendgrid on macos

Discussed in https://github.com/pantsbuild/pex/discussions/1529

Originally posted by suriya92 December  7, 2021
Hi There,
pex command is failing on macos(version:11.6,intel chip)with the following error when attempted to include sendgrid(target platform is linux)
ERROR: Could not find a version that satisfies the requirement starkbank-ecdsa>=2.0.1 (from sendgrid)
ERROR: No matching distribution found for starkbank-ecdsa>=2.0.1
pex command: pex.venv/bin/pex -vvvvvvvv --output-file output.zip  --manylinux=manylinux2014 --resolve-local-platforms --no-pypi --index=https://pypi.org/simple/ --resolver-version pip-2020-resolver --platform linux-x86_64-cp-38-cp38 "sendgrid==6.9.1"
The same command works fine on Linux. Appreciate any help on resolving this.
Thanks !

Hey @suriya92, it looks like the library starkbank-ecdsa only is distributed as an sdist (source distribution), i.e. a .tar.gz file: https://pypi.org/project/starkbank-ecdsa/#files.
When you install something with Pip or Pex, it needs to convert those into .whl files. The easiest way to do that is by the library author distributing a pre-built wheel (a bdist). Otherwise, you have to build the wheel locally. When you build a wheel locally, you can only build for your current platform (e.g. macOS vs. Linux).
Here, you are trying to build a linux-x86_64-cp-38-cp38 wheel for starkbank-ecdsa but you're doing it from macOS, which is impossible. The only reason this is working when you run from Linux is that you have the option --resolve-local-platforms set so that you can try to build that wheel locally - if you remove that flag and run from Linux, the build will fail.
Some of your options are:

Ask the library author to release wheels: https://pythonwheels.com
Pre-build the wheel yourself and internally host it with a server or your filesystem, then have Pex look there.
Give up on building from macOS

Does this make sense?

Hey @Eric-Arellano Thank you for your explanation. We are building this PEX via Pants.

@suriya92 I'm going to close this as an answered question since Eric's advice is accurate. If you need more explanation of those 3 alternatives, please speak up though.


Java Widget IntSlider Documentation?

How do i observe an IntSlider widget? When the slider value changes i want a function to be called.
Where can i find a good documentation of the beakerx java?
Thanks.

Thank you for your question.
We have not got as far as implementing "observe": https://github.com/twosigma/beakerx/issues/6562
There is JavaDoc in the Help menu.

Then how can i react to a widget change?

You can use EasyForm or interact.

`import static com.twosigma.beakerx.groovy.widgets.Interactive.interact
import ipyvolume.PyLab;
class DataHolder{
static double[] output = null;
}
public static Object f(a,b,c,d){
if (DataHolder.output == null){
    DataHolder.output = new double[3];
}

DataHolder.output[0] = a;
DataHolder.output[1] = b;
DataHolder.output[2] = c + d;

return DataHolder.output;

}
interact(this.&f, 0d, 4d, 0d, 0d);`
Why do the output values not update depending on the slider values?

interact() only works with widgets.

How do you use the Output widget to print the text? I see no documentation for the java/groovy version. Iam walking in the dark.

Is this what you need? https://github.com/twosigma/beakerx/blob/master/doc/groovy/OutputWidget.ipynb

How can i add a description to the slider?


Add usage event to track when a iOS network device is used

We recently added wireless debugging for iOS network devices https://github.com/flutter/flutter/pull/118895 and want to track it's usage.
Fixes https://github.com/flutter/flutter/issues/118479.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@jmagman You mentioned to me maybe using an existing tracking event such as when a user runs flutter run and adding data to it. I found this, which I believe is what you were referring to.
https://github.com/flutter/flutter/blob/a07e8a6ac43d4582c86174592ebe45edc37d115a/packages/flutter_tools/lib/src/runner/flutter_command.dart#L1323-L1375
Unfortunately the handler is registered before the device is selected, though, which is why I decided to go with a new event.

Spent some more time looking at this, maybe it's more appropriate to add this as a RunCommand.usageValues() CustomDimensions instead?
https://github.com/flutter/flutter/blob/224e6aa18cfd7ce5f839f8910d85f6cb5d4e455c/packages/flutter_tools/lib/src/commands/run.dart#L445
@christopherfujino can you weigh in on the best way to do this?  Or maybe @eliasyishak working on analytics right now may have a suggestion?

Spent some more time looking at this, maybe it's more appropriate to add this as a RunCommand.usageValues() CustomDimensions instead?
https://github.com/flutter/flutter/blob/224e6aa18cfd7ce5f839f8910d85f6cb5d4e455c/packages/flutter_tools/lib/src/commands/run.dart#L445
@christopherfujino can you weigh in on the best way to do this? Or maybe @eliasyishak working on analytics right now may have a suggestion?

I'll defer to Elias on this one :)

@jmagman this is actually something I have been thinking about to standardize the usage events we send.
Does your RunCommand.usageValues() essentially resolve all the necessary data you want to send for collection?

Does your RunCommand.usageValues() essentially resolve all the necessary data you want to send for collection?

I guess that's my question, should @vashworth move her new usage dimension to that method?  It seems like that's the right place for it.

Codecov Report

Merging #118915 (224e6aa) into master (bb73121) will decrease coverage by 0.06%.
The diff coverage is 96.27%.

:exclamation: Current head 224e6aa differs from pull request most recent head 2ed679a. Consider uploading reports for the commit 2ed679a to get more accurate results

@@             Coverage Diff             @@
##           master   #118915      +/-   ##
===========================================
- Coverage   91.39%    91.34%   -0.06%
===========================================
  Files         548       548
  Lines       99585     98473    -1112
===========================================
- Hits        91013     89946    -1067
+ Misses       8572      8527      -45

Impacted Files
Coverage Δ

packages/flutter/lib/src/animation/animation.dart
91.66% <ø> (-0.34%)
:arrow_down:

packages/flutter/lib/src/gestures/arena.dart
84.90% <ø> (-0.15%)
:arrow_down:

packages/flutter/lib/src/gestures/binding.dart
86.52% <ø> (-0.38%)
:arrow_down:

packages/flutter/lib/src/gestures/converter.dart
92.25% <ø> (-0.10%)
:arrow_down:

...ackages/flutter/lib/src/gestures/drag_details.dart
100.00% <ø> (ø)

packages/flutter/lib/src/gestures/hit_test.dart
94.64% <ø> (-0.19%)
:arrow_down:

packages/flutter/lib/src/gestures/long_press.dart
87.15% <ø> (-0.35%)
:arrow_down:

packages/flutter/lib/src/gestures/monodrag.dart
96.37% <ø> (-0.06%)
:arrow_down:

...tter/lib/src/gestures/pointer_signal_resolver.dart
70.00% <ø> (-2.73%)
:arrow_down:

packages/flutter/lib/src/gestures/recognizer.dart
93.49% <ø> (-0.04%)
:arrow_down:

... and 93 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I think that would make sense to add it to the CustomDimensions class that gets returned from the RunCommand.usageValues getter instead of sending it yourself through UsageEvents.send(...). That way you can just piggy back on an event that is already being sent.
But do you need to add a new custom dimension?

But do you need to add a new custom dimension?

Yes, I believe so. I see that it would need to be added in https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/reporting/custom_dimensions.dart. Looks like I also need to add it to Google Analytics (https://support.google.com/analytics/answer/2709829#zippy=%2Cin-this-article)?

maybe it's more appropriate to add this as a RunCommand.usageValues() CustomDimensions instead?

Also DriveCommand, so the RunCommandBase commands.


Update to url 2

The travis failure comes from travis setting an enviroment variable TRAVIS_APT_PROXY that is unexpected in the test case for reading the enviroment variables.


Custom canvas positioning

I am creating a canvas before calling html2canvas in order to achieve better quality (found this solution in one of the answers here), but the problem that I am now facing is that there is a gap at the top of my image. Seems like the scanned div is correct, but the image is shifted like its taking the header into the consideration.
`        var canvas          = createCanvas();
    html2canvas(elementToExport, {

        canvas: canvas,
        useCORS: true,
        allowTaint: true,
        letterRendering: true

    }).then(function(canvas) {

        // do the rest

   });

`

This is the top of my image. The idea is to not get the black gap. When I dont use the custom canvas it works fine, but the image is low quality


Issue13 make conditionDiagnosisUse multivalued

Signed-off-by: Brian Cragun<EMAIL_ADDRESS>Addresses #13

change Condition/model from three effectively boolean values ( conditionPrincipalDiagnosis conditionRoleIsAdmitting &
conditionRoleIsChiefComplaint) to a single conditionDiagnosisUse, which has possible values: AD, DD, CC, CM, pre-op, post-op, and billing. (See https://terminology.hl7.org/3.1.0/CodeSystem-diagnosis-role.html)
Add display names for these values in fhir_constants.EncounterResource
Create processing in Condition/ffhirs based on the conditionDiagnosisUse
Remove decoration of use code + Id. Use the straight ID

Updates and adds unit tests and documentation.

Also updated tasks.py documentation for nulling a field using replace_text.


[feature] parameter transformation

Use case:

Entities have UUIDs as PK.
Within the application domain UUIDs are stored in their default string representations (32 chars hex + 4 dashes)
to get shorter URLs the UUIDs should be represented as base64url

Would this be a feature that is possible to integrate in this bundle? If we can find a viable solution, i can also provide a PR.
Some thoughts regarding implementation:

The transformation to use must be specified on a per parameter basis within the route definition. (requirements? defaults?)
Should it be allowed to specify multiple transformations (a chain) or should the transformation chain be built externally and each parameter can only use exactly one transformation
Reverse transformation must occur very early so things like ParamConverter gets the "canonical" representation

I personally have never used UUID before so my knowledge about them is limited. However, the way you describe it, it sounds simple enough. You can register it with a higher priority than the doctrine EntityIdResolver.

The transformation to use must be specified on a per parameter basis within the route definition. (requirements? defaults?)

In the end it does nothing more than transforming the Entity into a string representation, which at the moment is simply the getId() method. The router will still handle the generation so that's your feature set.

Should it be allowed to specify multiple transformations (a chain) or should the transformation chain be built externally and each parameter can only use exactly one transformation

It depends on how you want to represent it. Is there a default way of showing the UUID in the url?

Reverse transformation must occur very early so things like ParamConverter gets the "canonical" representation

That's pretty much up to how you implement the ParameterConverter. I wasn't planning on adding ParamConverters in this package and I don't think the reverse transformation belongs anywhere else. A good example would be that a parameter converter checks if it can be decoded back into the UUID and if there's an entity with that given ID.
I can see the need for a UUID converter; I've seen questions coming by on Freenode#symfony how people would represent that in the URL. If there's no standard, maybe a configuration node could provide you the option to define a transformer which can do both ways. This would make it fairly easy to implement in the ParamConverter as well.

well the base64url encoding can be used for any binary data. it is not limited to UUIDs. there are also other encodings like base36 (not case sensitive) that could be useful. thats why i want this feature to be transparent. you just configure the transformation to apply to a certain parameter in the route definition. and the rest is done by the route generator and the route matcher (?).

Feel free to make a PR so we can check if this is really beneficial

@backbone87 are you still interested in making a PR for this or shall I close the issue?

Closing as it seems the feature is not wanted anymore

sorry for not replying, but the feature isnt currently on the top of the list of the project i work for. maybe i will come back to it, another time.


Update symbols.js

added bitcoin symbol ₿

Hey @TheButterZone, thanks for your contribution! I'm not keen on adding the Bitcoin symbol to this project, cryptocurrency is a very seedy and exploitative world and I want to stay about as far away from it as I possibly can. Sorry.


Run Background Task Only When Internet Connection Is Available

My Background  job is calling irrespective of internet connection state.
Current Behaviour:
--> scheduled task is working when app is background state.( Even when internet is not-connected )
Expected Bahaviour:
-->  Scheduled Task Should Execute When Device Connected To Internet Irrespective of App State( both Background And Foreground State).
**### I tried with this code :+1: **
BackgroundJob.schedule({
                jobKey: regularJobKey,
                period: 10 * 1000,
                networkType: NETWORK_TYPE_ANY
            });

---> But i Am getting An ERROR: expected dynamic type double', but had type string' (constructing arguments for BackgroundJob.schedule at argument index 5)
@vikeri @hkairi  guys could you help me, i've been searching for the solution since a week.
### Thanks In Advance.
**### More Info About My Project Specification :+1: **
Dependency Version : "react-native-background-job": "^2.1.1",
version of RN is: 0.50.3
react-native-cli: 2.0.1
My Device : windows-7, 64-bit , 8GB RAM

Try this way:
networkType: BackgroundJob.NETWORK_TYPE_ANY
Because React native don't know what is NETWORK_TYPE_ANY.
NETWORK_TYPE_ANY is a const given in this react-native-background-job package.

Tq @imran1992 . it's Working.

Thanks @imran1992 for the help. Closing this.


Could not broadcast input array from shape (128) into shape (96)

Rasa NLU version:
14.0a1
Operating system (windows, osx, ...):
Image built with docker, and running on docker.
Content of model configuration file:
here is the pipeline config:
pipeline:
- name: "nlp_spacy"
- name: "tokenizer_spacy"
- name: "ner_crf"
- name: "ner_synonyms"
- name: "intent_featurizer_count_vectors"
- name: "intent_classifier_tensorflow_embedding"
  intent_tokenization_flag: true
  intent_split_symbol: "+"

Issue:
The training sent me an error:
{
    "error": "could not broadcast input array from shape (128) into shape (96)"
}

If I change the pipeline and don't use spacy anymore, it works fine.
Is there a reason why this pipeline is not working fine with both spacy and tensorflow_embedding?

Thanks for raising this issue, @wochinge  will get back to you about it soon.

Which docker image are you using
Can you share the training example where you used the intent_split_symbol?

I build my own image.
I don't have any training example with several intent.

I will try without the several intent_split_symbol in the config file. I let you know.
Do you want me to share my dockerfile?

It works well without the several intents option.

@bguivarch Sharing the dockerfile would be great 👍
You can also use the image rasa/rasa_nlu:full if you don't want to build your own image.


[ALLUXIO-1884] Allow mount table to create UFS objects during resolution

https://alluxio.atlassian.net/browse/ALLUXIO-1884

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9340/
Test PASSed.

LGTM, thank you for this useful feature

LGTM


Update README.md to use connect URL to add a layer to a function

Hi,
I'm making this pull request to propose to use AWS Lambda Console connect URL to associate a layer ARN to a function.
Basically, you can go to <AWS Lambda Console Home>#/connect/layer?layer=arn:aws:lambda:us-east-1:553035198032:layer:nodejs10:1 to land on the "Add layer to function" page with the ARN pasted in the "Provide a layer version ARN" input box.
Then you could pick any function you want to add that layer. Please see the screenshot for the result.

Thanks.

Awesome, thank you! :shipit:

No problem. Thank you. 😆


Validate that the scope k8s demo works as expected with 1.0.0

Do all pods start with no issue
Does anything crash
Do we get console logs
Do we get live data from workers in the appscope source
$ cd scope-k8s-demo/
$ ./start.sh cribl

pull master, edit as needed, make all, docker build -t cribl/scope:0.7.0-rc5 -f docker/base/Dockerfile .,
then over in the scope-k8s-demo directory:
$ SCOPE_VER=0.7.0-rc5 ./start.sh cribl
no need to docker push anything.
when doing docker build use the same version as 'scope version' uses

The demo works with 1.0 as defined above. Closing this issue. A PR has been created to update the AppScope version used with the k8s demo.


The Service type should be define separately

Currently, a Service is a principal,  but in order to generalize over getters and constructors,
it would be handy if it is defined  in its own class, a simple solution would be:
class Service: Principal {      ... }
The service is used interchangeably as a Principal.
these characteristics appeal to an inheritance approach from the Principal class.

:link: zboto Link

@chcmedeiros any advice here, is there a structural difference from a Service and a Principal? any invariant we can base on to define a concrete class for principals?

From the candid point of view the Principal variant represents a principal value, which is used to identify canisters. A principal is a unique identifier assigned to a canister and is often used for authorization and access control purposes.
On the other hand, the Service variant represents a service type value. A service type used to represent the interface of a canister. As we see on the .did files it defines the methods that can be called on the canister and the types of arguments and return values for those methods.
In the end the Service variant is used only to indicate that a value is of a service type.
So I would say
class Service: Principal { ... }
Seems to be a good idea

It also seems to be how this is represented by candid https://docs.rs/candid/latest/candid/parser/value/enum.IDLValue.html#variant.Service

Using inheritance is quite complex in this scenario, to ensure the Service is never constructed by its own methods but using the parent class interface, it is necessary to make base class constructors/destructors private plus minor details in deriving class definition. this might be in conflict with how the Principal base class is currently used in the code, in order to continue with this approach we would need to do something like:

define a PrincipalBase class that would have the same constructors/destructor and members as defined by the current Principal class.
define a class Principal : protected PrincipalBase{}
define class Service: protected PrincipalBase{}

Even with this approach, there are corner cases.
So it is better to follow Rust approach and define Service class as a simple wrapper, which will also be move-only.

fixed via #24


Rename "Add Signature" to "Re-sign"

Description:
Rename "Add Signature" to "Re-sign"
Related issue(s):
Fixes #242
Notes for reviewer:
Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

[ ] confirm with @lbaird

I had asked that it remain “Add signature” rather than “Re-sign”.  Let’s
discuss this before making the change.
On Tue, Feb 8, 2022 at 11:47 AM Patrick Staton @.***>
wrote:

Description:
Rename "Add Signature" to "Re-sign"
Related issue(s):
Fixes #242
https://github.com/hashgraph/hedera-transaction-tool-demo/issues/242
Notes for reviewer:
Checklist

Documented (Code comments, README, etc.)
Tested (unit, integration, etc.)

You can view, comment on, or merge this pull request online at:
https://github.com/hashgraph/hedera-transaction-tool-demo/pull/252
Commit Summary

31643e1
https://github.com/hashgraph/hedera-transaction-tool-demo/pull/252/commits/31643e1d5bc0b7a977fed94fed70439044182988
fix(#242): Rename Add Signature to Re-sign

File Changes
(2 files
https://github.com/hashgraph/hedera-transaction-tool-demo/pull/252/files
)

M
tools-integration/src/test/java/com/hedera/hashgraph/client/integration/BatchTransactionEndToEndTest.java
https://github.com/hashgraph/hedera-transaction-tool-demo/pull/252/files#diff-a4e5ced766dd0739f561d4acaaa851e390ec1783ff5a46bde60b4377c6261daa
(8)
M
tools-ui/src/main/java/com/hedera/hashgraph/client/ui/HomePaneController.java
https://github.com/hashgraph/hedera-transaction-tool-demo/pull/252/files#diff-7ac1bebccbd6737177a466ceb71cbc876ac4b62088bc7e94239c4b387fc10a40
(2)

Patch Links:

https://github.com/hashgraph/hedera-transaction-tool-demo/pull/252.patch
https://github.com/hashgraph/hedera-transaction-tool-demo/pull/252.diff
—
Reply to this email directly, view it on GitHub
https://github.com/hashgraph/hedera-transaction-tool-demo/pull/252, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDBYWZZN6ZZACF7RGQBLNDU2FJL7ANCNFSM5N3FJHSQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Add the sampleapp and hostwebsite iot test cases.

Impacted tests(approved): new 9, update 0, delete 0
Unit test platform: Crosswalk for Android 20.49.525.0
Unit test result summary: pass 8, fail 0, N/A 1
Unit block reason: No device to test.

LGTM


Single modal views for proposals in treasury, registry, nft

Description (what needs to be done) / Expected Behavior
On the explorer
Hand off single modal views for proposals in treasury, registry, nft to Fabiola for code changes. Or just a text explanation of which ones to not show on each. I.e. instead of showing all three sets of proposals in the modal for each, show only one to reduce confusion.
Acceptance Criteria
Mockup or description of what to change so frontend can execute.
Supporting information (Figma link, notes, etc)
When finished, please add Link to the PR or completed Figma / design asset, as a comment to the ticket.

Here's how the modals should look like:
Image
Image
Image
For the NFT modal,  the preview image of the NFT that shows after an NFT ID is chosen shouldn't expand/be part of the NFT ID text box. It's a separate item.
https://www.figma.com/file/gXLZPoh1BchigOB5y9YkQZ/Tezos-Homebase---v2-UI?node-id=395%3A1&t=zDEfmpQlXxgV85mt-0

Closed on #448


ImageMagick "area" resource limit should be in pixels ?

it is currently defined in bytes
https://github.com/heroku/stack-images/blob/e65a994d5540840ebf173528875db4bad5b29467/heroku-18/setup.sh#L213
the documentation is not really clear https://imagemagick.org/script/resources.php
but it looks like that memory/disk resources expect "bytes" and height/width/area expect "pixels"

Define arguments for the MAGICK_MEMORY_LIMIT, MAGICK_DISK_LIMIT, and MAGICK_MEMORY_LIMIT environment variables with SI prefixes (.e.g 100MB). MAGICK_WIDTH_LIMIT, MAGICK_HEIGHT_LIMIT and MAGICK_AREA_LIMIT accepts pixel suffixes such as MP for mega-pixels (e.g. 100MP)

which would make more sense

Fixed in #209 for heroku-22

See:
https://github.com/heroku/stack-images/pull/185#issuecomment-1143478370


#2161 - New spo site chrome set command

This PR contains the new spo site chrome set related to #2161

@waldekmastykarz this seems to be a lot better.

Awesome! Appreciate your help. We'll go through it asap 👏

Thank you for a super quick turnaround @estruyf. We'll get through it asap 👏

Merged manually. Awesome! 👏


Fix an error with optionnal props for islands

Currently with the "hydrate" feature this code
#[island]
pub fn Test(#[prop(optional)] _test: Option<bool>) -> impl IntoView {}

produce this error:
error[E0308]: mismatched types
 --> src\app.rs:6:31
  |
6 | pub fn Test(#[prop(optional)] _test: Option<bool>) -> impl IntoView {}
  |                               ^^^^^
  |                               |
  |                               expected `bool`, found `Option<bool>`
  |                               arguments to this method are incorrect
  |
  = note: expected type `bool`
             found enum `std::option::Option<bool>`
note: method defined here
 --> src\app.rs:6:31
  |
6 | pub fn Test(#[prop(optional)] _test: Option<bool>) -> impl IntoView {}
  |                               ^^^^^-------------
help: consider using `Option::expect` to unwrap the `std::option::Option<bool>` value, panicking if the value is an `Option::None`
  |
6 | pub fn Test(#[prop(optional)] _test.expect("REASON"): Option<bool>) -> impl IntoView {}

This PR fix the usage of optionnal props with islands

Same issue exist in 0.7 too

Interesting, not sure I've tried this before. Thanks for the PR!

@Baptistemontan you'll probably want to reproduce this against main, as that's the branch that will be used for 0.7 and future updates


Too Many Options

Describe the flaw
I think the number of options for conversion process is too many.
Expected behavior
It will be good if the entire first column (from 'GPU Conversion' to 'Model Test mode') could be removed from GUI.
Redistribute the remaining options in the resulting freed-up space.
Screenshots
If applicable, add screenshots to help explain your problem and annotate the images.
Additional context

Add any other context about the problem here.

They're not going to remove them. It's part of the AI and they're actually useful. If you don't know what they mean, then don't mess with them. But they leave remained for those who do, like me.

The new V5 will have a completely new GUI. I'm closing this Issue, ok?


Fixing Blob bindings

These changes fix all our broken Blob samples. There were core issues in the binding pipeline which I've addressed. I've fixed all the samples and tested each - they are now successfully triggered on new blobs, and their out bindings also work (with parameter binding).
There is one ugly hack required (private reflection) until we decide what to do in the core SDK. We might need to make the data we need public in some way.

Ah, so that's why we could never get Blob to work? Sweet!

I just pushed this, but I'll address any feedback anyone has. I wanted to get this in.

Just happy I got to use :smiling_imp: and :angel: on the same PR


Uncaught (in promise) DOMException: Failed to execute 'postMessage' on 'BroadcastChannel': function (dispatch)

This error will trigger once I access datas in my redux.
Uncaught (in promise) DOMException: Failed to execute 'postMessage' on 'BroadcastChannel': function (dispatch)

https://github.com/AOHUA/redux-state-sync/issues/49#issuecomment-570147813 please refer to this issue, I will give more meaningful error message for this when I got time. Thanks.

@AOHUA  much appreciated! But what are the common causes of this?

@AOHUA @vincegerald it seems to be caused by redux-persist persist/PERSIST action
the error is gone when adding it to the blacklist.
createStateSyncMiddleware({
    blacklist: ['persist/PERSIST'],
  })

but then this error appears, which at least seems to be nicer.

despite having
 const persistConfig = {
    ...
    blacklist: ['_persist'],
  }

This issue is most probably caused by non-serializable data in the store. Not sure if it is caused by misuse of redux-persist. I will try to replicate the issue with redux-persist when I got some time.

Here is a basic redux-persist example to which I added this package.
The error on the console is there.
I've hosted it here for today for u to see it without the need to run it locally.
The error is still happening no matter if the rootReducer is first wrapped with persistReducer or withReduxStateSync

Thanks @aschenkel appreciate that. but the site is no longer available. I will try to find sometime during the weekend to reproduce the issue.

I found the issue, there is an easy solution to remove the error, however, I'm not sure if this will cause any other issues. I hope it helps.
import { PERSIST } from 'redux-persist/es/constants';

const persistedReducer = persistReducer(persistConfig, rootReducer);

const middlewares = [
    // disable PERSIST action.
    createStateSyncMiddleware({
        blacklist: [PERSIST],
    }),
];
const store = createStore(persistedReducer, {}, applyMiddleware(...middlewares));

const persistor = persistStore(store);

https://github.com/rt2zz/redux-persist/blob/d7efde9115a0bd2d6a0309ac6fb1c018bf06dc30/src/persistStore.js#L120
Seems that redux-persist is dispatching a function when persisting the state.

@aschenkel Actually it's not breaking the persist functionality, it only stops sending this persist function to other tabs which make sense in such case. Thank you for your help.

It actually makes sense to prevent all the redux-persist action in redux-state-sync

@vincegerald If it solves your problem, please close this issue. Thanks

@AOHUA that works.
I also have included the PURGE action to the blacklist, since on some cases was also generating the error.
import { PERSIST, PURGE } from 'redux-persist/es/constants';

const middlewares = [
    createStateSyncMiddleware({
        blacklist: [PERSIST, PURGE],
    }),
];

I´m having the same issue using axios and redux-thunk.
so instead of a serializable object (with the nested type parameter that you can use to filter).
I have a function being sent, with several lines of code.
I think that should be wise to filter by type

// Function filter
if (typeof v === 'function') {
    // do something
}

Solved with predicate use.
const reduxStateSyncConfig = {
  predicate: (action) => {
    const blacklist = [PERSIST, PURGE, REHYDRATE];
    if (typeof action !== "function") {
      if (Array.isArray(blacklist)) {
        return blacklist.indexOf(action.type) < 0;
      }
    }
    return false;
  },
};

@leopucci Thank you for your help! It is worked for me!
My environment is redux with redux-persist, redux-thunk. 😄

Anytime!


Private variable warning in lazy evaluation

The following statement will produce a warning "Local variable ;_view' is not from this scope.":
private _view = cameraView;

if (!isNull objectParent player && {_view == "GUNNER"}) then {
};

What's strange however is that a warning is not produced in the following statement:
private _view = cameraView;

if (true && {_view == "GUNNER"}) then {
};

Did you verify on latest commit? Should have been fixed with #3.

Yes I've pulled and installed the latest commit (0b952c1b08514ed3ca5086e0c2aa9dc78a44ba08) using
pip install -e .

Yes, pulled the latest commit (0b952c1b08514ed3ca5086e0c2aa9dc78a44ba08) and install using pip install -e ..


Highlight "TODO" word with some different color

As currently, we can see that "code documentation" highlights the params with some color (like @ param) and it was just a comment. So may be we should provide for TODO (comment) something like that so it will be easy to find.

/duplicate of #9899


Add more visible link to OrbitDB GitHub

After looking at https://orbitdb.org/ front page a few times now, I think it is missing a visible link to https://github.com/orbitdb. Similar to IPFS and IPFS pubsub. Today the only GH link I can find is in the GH icon at the bottom of the page.
Do you think it would be beneficial to link the first "OrbitDB" in the first paragraph to https://orbitdb.org/?

@sirfumblestone agreed :+1: A simple solution would be to link the first "OrbitDB" in the text to the repo. A slightly more involved solution would be to add widgets for GH fork/star and make very explicit that the visitor should go to Github.
Want to PR a fix with the first approach?

Sure. I'll make a PR :+1:


We want to use an up-to-date pip on travis

By the look of things pip 9.0.1 and above should handle
the IPython-5 / IPython-6 choice for python 2.7 / python 3.5.
This adds an "upgrade pip" step to the travis build.

This fixes #18. Let's merge this onto master and rebase other branches on top so the tests are useful!


Select filled with ajax re-render

Hi,
I'm trying to fill the select with id=aBrandsSelect  with options loaded from ajax call.
Basically I insert items with this pies of code:
            $.each(data.ResultSet.Result, function(index, option) {
                $('#aBrandsSelect').append(
                    $('<option></option>').val(index).html(option)
                );

            });

How can I re-render this particular select to show the new options?
I'm sorry to bother you with this, but I didn't find answer on your page...
Thanks

Sorry for duplicated issue.
I find answer in closed issues.

Did you try

Oops, I just saw that this method is not in the documentation.
I will add it soon.


Using springfox with Thymeleaf templating

Please take the time to search the repository, if your question has already been asked or answered.

[2.9.2] What version of the library are you using? Is it the latest version? The latest released version is

What kind of issue is this?

[X] Question. Is this a question about how to do a certain thing?

What must be done to get swagger-ui.html request mapping when there is also Thymeleaf template resolving enabled in the app?
It seems that swagger-ui request handling is overwritten.
The v2/api url is accessible.

Also the swagger-resource and e.g. /webjars/springfox-swagger-ui/springfox.css?v=2.9.2 is working.
It's only the swagger-ui.html which is not found (404).

I've added now a
registry.addResourceHandler("/swagger-ui.html").addResourceLocations("/swagger-ui.html");
and it is found.
This looks for me like a workaround.
So would be nice to know why the handling is lost.


[e2e] Ignore cluster delete failures

What type of PR is this?
/kind bug
/kind flake

What this PR does / why we need it:
Ignore the failure of cluster deletion in the e2e test's cleanup.
Which issue(s) this PR fixes:

Fixes #2738
Special notes for your reviewer:
Does this PR introduce a user-facing change?

None

/cc @tenzen-y

/close
I don't think we have enough evidence to justify ignoring cluster deletion errors in Kueue based on a single observed failure (see comment). I'm concerned that doing so could sweep issues under the carpet, leading to unnoticed cluster leaks by the team. A similar sentiment was expressed in this comment.
For now, I'm open to exploring solutions on another layer, possibly as suggested in this comment, but I'll leave the decision to @BenTheElder.
I'm also open to revisiting the idea of ignoring errors in Kueue if this issue becomes more frequent, we rule out that the errors are related to Kueue, and we've exhausted options for resolving it at other layers.
We can keep the issue open for further discussion.

/reopen
I synced in @trasc who suggests we should continue the PR open for discussion and possibly merging.
I would still be leaning to a fix in Kind, but let's keep both options open.

/reopen
retrying

/reopen

/close
we haven't seen the issue in the last couple of months. We can re-evaluate options when this repeats, as also suggested here: https://github.com/kubernetes-sigs/kueue/issues/2738#issuecomment-2538418306


Why the Sample Project APK is removed from play store ?

Hi Pierfrancesco,
Why the Sample Project APK is removed from play store ?
is there any policy violation issue ?

Hi, no policy violation.
The sample is temporarily down because the name of the app didn't explicitly stated that it was not affiliated with YouTube (the company) and therefore got removed. Nothing to do with the player.
The name should be something like: "{name of the app} for YouTube"
I will re upload it soon, with a different name.

Thanks for clarification 👍

Sorry to barge in.
Should my final app, which is using your library, state that it is not
affiliated with YouTube in, say, the description?
Thanks
sob., 16 cze 2018, 07:44 użytkownik Pierfrancesco Soffritti <
<EMAIL_ADDRESS>napisał:

Hi, no policy violation.
The sample is temporarily down because the name of the app didn't
explicitly stated that it was not affiliated with YouTube (the company) and
therefore got removed. Nothing to do with the player.
The name should be something like: "{name of the app} for YouTube"
I will re upload it soon, with a different name.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PierfrancescoSoffritti/Android-YouTube-Player/issues/150#issuecomment-397789040,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASXFFmbWn3X5UeGijoM642Ta-DOQxknjks5t9JtUgaJpZM4UqXgk
.

Just to clarify: this has nothing to do with this library, you are required to do the same if you're using the official YouTube Player API.
To answer your question: you have to say it in the description, but most importantly in the name of the app. As I'm doing here.
This page may be useful.

New sample app published here: https://play.google.com/store/apps/details?id=com.pierfrancescosoffritti.aytplayersample

Do i have to add youtube to end of the name.Can my app name be just "mountain" for example?


chore: Add use cases to developer guide

Added the following use cases to the developer guide:

UC C1 List Customers
UC C2 View Customer
UC C3 Delete Customer
UC C4 Update Customer
UC C5 Bookmark or Unbookmark Customer
UC C6 Assign Individual to Enterprise
UC C7 Create Customer Note
UC C8 Append Customer Note ➕
UC C9 Delete Customer Note
UC O1 List Orders
UC O2 Create Order ➕
UC O3 View Order
UC O4 Delete Order
UC O5 Update Order
UC O6 Advance Order Status
UC O6 Tag Order to Customer
UC R1 List Rewards
UC R2 View Reward
UC R3 Delete Reward
UC R4 Update Reward
UC R5 Set/Add Customer Points ➕
UC R6 Give Reward to Customer

Closes #14

Added the full stops, also updated the list of use cases:

+ UC C8 Append Customer Note
+ UC O2 Create Order
- UC O6 Tag Order to Customer
+ UC R5 Set/Add Customer Points


Remove HSM v1 API

CASMHMS-5539
Summary and Scope
Update HSM image version for CASMHMS-5539 - removal of HSM's v1 API
Issues and Related PRs

Resolves CASMHMS-5539

Testing
For testing, see https://github.com/Cray-HPE/hms-smd/pull/80
Risks and Mitigations
There may be services still referencing the v1 API. These will break.
HMS services and documentation referencing the v1 API will be all migrated to use HSM's v2 API before merging this mod.
Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable

@alvarez3-hpe need to update this file: https://github.com/Cray-HPE/hms-smd-charts/blob/main/cray-hms-smd.compatibility.yaml#L3-L7 by adding the following:
">=3.0.0": "~1.3.0"


Button access and layout adjustment when keyboard is active

There are two problems with my use of KeyboardAwareView.

When the keyboard is activated, the layout is not being adjusted, bringing the action button close to the form.

When the keyboard is active, it is not possible to directly access the button following the flow. I can only access the button action when the keyboard is closed.

The structure of my code looks like this:
<Minimal>
      <Minimal.Content>
        <Minimal.Header />
        <Minimal.Title>Test Title</Minimal.Title>
        <Minimal.Subtitle>Test Subtitle</Minimal.Subtitle>

        <Formik
          initialValues={{ info1: '', info2: '' }}
          style={style.wrappedForm}
          validationSchema={validationSchema}
          onSubmit={handleSubmit}>
          {({
            handleChange,
            values,
            handleSubmit,
            errors,
            touched,
            handleBlur,
            isValid,
          }) => (
            <View style={style.wrappedChildren}>
              <KeyboardAwareScrollView
                scrollEnabled={false}
                contentContainerStyle={style.keyboardContainer}>
                <View>
                  <Picker
                    hasError={Boolean(errors.ddd)}
                    mode="dropdown"
                    placeholder="info1"
                    iosTitleHeader="DDD"
                    name="ddd"
                    selectedValue={values.ddd}
                    firstLabel="info1"
                    onValueChange={handleChange('ddd')}
                    onBlur={handleBlur('ddd')}>
                    {missDDDs.map(renderOptions)}
                  </Picker>
                  {errors.ddd ? (
                    <Text style={style.firstTextError}>{errors.ddd}</Text>
                  ) : null}

                  <Item
                    stackedLabel
                    style={style.input}
                    error={Boolean(errors.phone && touched.phone)}>
                    <Label>Info2</Label>
                    <TextInputMask
                      type="cel-phone"
                      options={{
                        maskType: 'BRL',
                        withDDD: true,
                        dddMask: '(99) ',
                      }}
                      name="phone"
                      customTextInput={Input}
                      value={values.phone}
                      style={style.inputMask}
                      maxLength={15}
                      onChangeText={handleChange('phone')}
                      onBlur={handleBlur('phone')}
                    />
                  </Item>
                  {errors.phone && touched.phone ? (
                    <Text style={style.textError}>{errors.phone}</Text>
                  ) : null}
                </View>

                <PrimaryBlockButton isValid={isValid} onPress={handleSubmit}>
                  Button Action
                </PrimaryBlockButton>
              </KeyboardAwareScrollView>
            </View>
          )}
        </Formik>
      </Minimal.Content>
    </Minimal>

const style = StyleSheet.create({
  keyboardContainer: {
    flex: 1,
    flexDirection: 'column',
    justifyContent: 'space-between',
  },
  wrappedChildren: {
    flex: 1,
    flexDirection: 'column',
  },
});

Here, have a video with the example:
Gravação de Tela 2019-09-26 às 14.53.17.zip
How could I configure KeyboardAwareView to solve the above two issues?
I am using native-base (UI component library for React Native).
Important
When the keyboard is not active, the button action must always be at the bottom of the screen.

Did you figure out a resolution? We're unfortunately still using Native Base. This library doesn't seem to be terribly well supported (no hate against the maintainer).
We've been seeing an issue where we can get the Button to show while scrolling when keyboard is open and an input is focused. However, when first going to a form, if you select a field toward the bottom, the top of the form and our header disappears...

Was solved with onKeyboardWillShow and onKeyboardWillHide props, adjusting my layout and scroll.


[Feature Request] Release Azure CLI as ZIP packages

Is your feature request related to a problem? Please describe.
Currently Azure CLI on Windows is released as an MSI package. This has several problems:

The MSI requires admin permission (https://github.com/Azure/azure-cli/issues/15381), which makes it impossible to install Azure CLI as a normal user.
The installation location is fixed to C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2, making it impossible to customize the installation location.
ZIP allows installing multiple versions side-by-side
Some corporate firewall/policy may block downloading .exe, .msi (https://github.com/Azure/azure-cli/issues/22447)
Updating with MSI is slow and takes several minutes.

Describe the solution you'd like
We could release Azure CLI as a portable ZIP packages, like PowerShell releases its ZIP packages at https://github.com/PowerShell/PowerShell/releases.
Additional context
As there is no official way to sign a ZIP, we can only use SHA256 to compute the hash of the ZIP and publish the hash along with the ZIP, but there is no way to sign it with Microsoft certificate.

Just curious if this has been prioritized in the backlog? It would greatly ease the onboarding pain for organizational users which often have limited access to their environment. Thanks!

Hello--is there any update on this issue?


Disable UAS for a SATA interface.

Hi. I bought this particular case that wasn't working on the USB3 interface.
https://www.amazon.es/dp/B077XVTTJC/?coliid=I2A06MFLQUYY95&colid=1QF557D5V5LOG
After adding the idVendor and idProduct to this list it started working.
I believe there might be countless people with this particular problem. Specially international users that can't find the particular recommended umbrel hardware.

Thank you for your contribution, but we'll actually change Umbrel to automatically apply these quirks in the next release instead of having a builtin blacklist in the next release, so you can close this PR.


Mono support

Support Mono on linux/macos.. maybe usefull if netcore is unacceptable (wcf)
native kafka libraries (librdkafka) should be placed in bin folder manually according platform version

It looks like @4garbage hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement here.
Once you've signed reply with [clabot:check] to prove it.
Appreciation of efforts,
clabot

thanks for the contribution @4garbage - inclusion of this is on the radar for after 1.0 - we also want to do some other changes related to library loading and this can be included/tested at the same time. Note that we won't back-port to 0.11.x.

continuing in #1443


[SMALLFIX] Remove extra new lines.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16179/
Test PASSed.


Implement support for calling different diff tools.

This is a first, rough attempt, at implementing support for different diff utilities. Current options are:

Internal
Pe "diff mode" (needs Pe's PR #85)
mcdiff
kdiff3 (untested)
vimdiff ~(untested)~

Defaults to Internal ~for now, until saving/restore settings gets implemented.~, but remembers last selected choice.
Opening this as a Draft, as to gather some feedback, to see if this sounds more or less OK, or I'm missing the mark by too much.
I guess we could allow adding custom diff tools by reading "tool_name=cmd_line %file1 %file2" or similar from a settings file. For now, mostly due to my limited skills (and short attention spans)... that will be left out as an "exercise for the reader" :-)
Preview (from before mcdiff was added):

At the moment, we can only be sure that "Internal" and "Pe diff" is available (Pe being bundled with Haiku, may change in the future, so better check on that, too).
I think the app should:
a) check which of the diff tools are installed and add only those to the menu.
b) show all diff tools in the menu and pop up an alert if a non-installed diff tool is selected. The alert could offer to install the missing tool via pkgman.

I once very briefly looked at changing this app to use layout management, but gave up for one reason or another.
If you look into creating a simple diff tool, it would be nice to be also able to compare a file against itself on another git branch. If that is possible...

👍


Removed no longer applicable note

@dimitri-furman : Thanks for your contribution! The author, @CarlRabeler, has been notified to review your proposed change.

#sign-off
@dimitri-furman thank you


DOC: update Ubuntu quickstart; conda compilers now work!

The wait is over; conda compilers now seem to build SciPy.
Tested on fresh instances of Ubuntu 16.04 and 18.04  EC2 instance and 20.04 WSL locally. That's all I can do, but please test on your system if possible.

Thanks for updating the docs. always nice when they are fresh and up to date :+1:

@rgommers I saw you mentioned the Ubuntu instructions in gh-12900; they will be even simpler once this is merged. I think I addressed all your comments.


pilot/pkg/xds: optimize the push queue using linked list

The push queue buffer is currently based on slice which may grow its size redundantly:
	newcap := old.cap
	doublecap := newcap + newcap
	if cap > doublecap {
		newcap = cap
	} else {
		if old.cap < 1024 {
			newcap = doublecap
		} else {
			// Check 0 < newcap to detect overflow
			// and prevent an infinite loop.
			for 0 < newcap && newcap < cap {
				newcap += newcap / 4
			}
			// Set newcap to the requested cap when
			// the newcap calculation overflowed.
			if newcap <= 0 {
				newcap = cap
			}
		}
	}

Either it doubling up the capacity (when less than 1024) or growing by a factor of 1.25 (when greater than 1024).
Since the push queue is not accessed by index, I suggest that we replace the slice with a linked list which only takes up space when it's needed, and release the element when it has been popped up.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[x] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any changes that may affect Istio users.

Hi @panjf2000. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

Did you observe memory growth with this?  What are the typical queue sizes? Wondering, is there a real problem here?

Did you observe memory growth with this? What are the typical queue sizes? Wondering, is there a real problem here?

I believe that it's not a fatal issue, just a little concern here, and try to make a tiny improvement.

I would expect a benchmark comparison between slice and linked list.


BatchModeTest: show UI on EDT

Closes https://github.com/imagej/imagej-legacy/issues/306

So the problem is that in the test we call service.showUI on the main thread, which of course then deadlocks with the EDT since we're using graphical components off the EDT.
But I'm not sure that we don't want to make this dispatch to the EDT elsewhere in the UIService stack..?

Closing in favor of https://github.com/scijava/scijava-common/pull/481


[<SDK/ UI/ UI-REACT / PROTOCOL>]: Base64 error

After updating typescript, react-script and all eslint libraries and updating sdk, ui, protocol to the latest version, when starting the server,  I am getting the following error:
Failed to compile.
<EMAIL_ADDRESS>Attempted import error: 'Base64' is not exported from '@tonconnect/protocol'.

In order to help you effectively, could you please create a small project that reproduces this error and share it? This will help to better understand the problem and find a solution more quickly.
Also, please make sure that you have the latest versions of all packages installed. You can check the installed versions using the npm ls command.
The latest versions of the packages are as follows:

@tonconnect/protocol: 2.2.5
@tonconnect/sdk: 3.0.0-beta.1
@tonconnect/ui: 2.0.0-beta.4
@tonconnect/ui-react: 2.0.0-beta.4


Dataset iterator is stalled indefinitely with corrupt TFRecord despite using ignore_errors

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: n/a
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v1.12.0-0-ga6d8ffae09 1.12.0
Python version: 3.6
Bazel version (if compiling from source): n/a
GCC/Compiler version (if compiling from source): n/a
CUDA/cuDNN version: 9/7
GPU model and memory: n/a

Creating this issue per @mrry's suggestion with minimal code to reproduce the issue.
cc @guillaumekln You're right, what I'm seeing may not have to do with OutofRangeError, but the execution stalls indefinitely when it encounters corrupt data within a TFRecord, despite using tf.data.experimental.ignore_errors().
Example Data (for snippets below)

clean.tfrecord
corrupt.tfrecord

With clean.tfrecord:
import tensorflow as tf

filenames = ['/data/clean.tfrecord']
#filenames = ['/data/corrupt.tfrecord']

dataset = tf.data.TFRecordDataset(filenames)
dataset = dataset.apply(tf.data.experimental.ignore_errors())
dataset = dataset.batch(64)
dataset = dataset.repeat(1)

iterator = dataset.make_one_shot_iterator()
next_element = iterator.get_next()

with tf.Session() as sess:
    i = 0
    while True:
        try:
            print(i, sess.run(next_element).shape)
            i = i + 1
        except tf.errors.OutOfRangeError:
            print("Dataset complete")
            break

Output:
0 (64,)
1 (64,)
2 (64,)
3 (64,)
4 (64,)
5 (64,)
6 (64,)
7 (64,)
8 (64,)
9 (64,)
10 (64,)
11 (64,)
12 (64,)
13 (64,)
14 (64,)
15 (64,)
16 (64,)
17 (64,)
18 (64,)
19 (35,)
Dataset complete

With corrupt.tfrecord:
import tensorflow as tf

#filenames = ['/data/clean.tfrecord']
filenames = ['/data/corrupt.tfrecord']

dataset = tf.data.TFRecordDataset(filenames)
dataset = dataset.apply(tf.data.experimental.ignore_errors())
dataset = dataset.batch(64)
dataset = dataset.repeat(1)

iterator = dataset.make_one_shot_iterator()
next_element = iterator.get_next()

with tf.Session() as sess:
    i = 0
    while True:
        try:
            print(i, sess.run(next_element).shape)
            i = i + 1
        except tf.errors.OutOfRangeError:
            print("Dataset complete")
            break

Output:
0 (64,)
1 (64,)
2 (64,)
3 (64,)
4 (64,)
5 (64,)
6 (64,)
7 (64,)
8 (64,)
9 (64,)
10 (64,)
11 (64,)
12 (64,)
(execution stalled indefinitely here)

Not sure why the corrupt files within the TFRecord aren't being ignored with tf.data.experimental.ignore_errors(). @mrry any idea?

I think the issue was that, when ignore_errors is used, the same file will repeat as the file_index is not moved forward to completion.
Created a PR #25705 for a shortcut fix.
Not very familiar with this part of the code, maybe there are some better ways of fixing it? /cc @mrry

Thanks @yongtang!

Thanks @yongtang. @sjain-stanford If your issue was resolved by @yongtang , then please close it or let me know. Thanks

@jvishnuvardhan Seems like this will close automatically when #25705 is merged. I can let you know once I test the fix though (need to build from source).


Why 1.8.2 is not marked latest in rocm-terminal?

See https://hub.docker.com/r/rocm/rocm-terminal/tags/

Fixed, thanks for reporting it.
https://hub.docker.com/r/rocm/rocm-terminal/tags/


WIP Create sd_desktop.md

SD desktop chapter

There has been no development in this fork for more than a month so I'm closing it. Also, the only change was adding one title. I propose starting SD Desktop documentation from a fresh branch made from the latest master and as a branch and not as a fork. If there are any questions, please let me know!


Using .cu files to contain RawKernel code

Often with RawKernel code, the pattern used in this codebase (and other CuPy using codebases) is to have an inline str that is then used in the RawKernel construction
Admittedly this has some benefits:

The CUDA code is next to the RawKernel
Python strs can be interned speeding up some operations with them

However the drawback is we are now maintaining CUDA code in a Python str. IOW we lack

Syntax highlighting with editors
Code completion
C++/CUDA Linters
Ability to use the CUDA code itself
Easier for C++/CUDA developers to get up to speed on

All of these mean it is easier for bugs to slip in undetected and harder to catch/fix them
Not to mention there can be a tendency to use Python str formatting with the inline CUDA code, which can hinder code readability and comprehension
Wonder if there would be some benefits to splitting out this CUDA code into .cu files. That may improve the maintenance experience by addressing the points above. Admittedly it does mean reading in a sidecar file (though that could be cached and possibly done at import time)
Curious what others think of this idea 🙂

Sounds good to me! We already have a path argument for RawModule, so it's just doing the same for RawKernel?

On a second thought:

Often we use string manipulation (replacing a string within the kernel code, concatenate preamble and body, etc.) and such code may be difficult to utilize syntax highlighting etc.
Essentially path option is not mandatory (we can just read and pass it to RawKernel)

For concrete code that can be compiled standalone by either NVRTC or NVCC and therefore can be encapsulated into a single .cu file, I prefer to just use RawModule's path argument to load it and not touch RawKernel. A .cu file can have multiple kernels and it'd blur the line between Raw{Kernel,Module} if we also add path to RawKernel.


Restore parse fix for JSON responses

Resolves #55 - @cdepillabout thanks, it wasn't changed intentionally, I'm not sure how it got lost
Resolves #54 - @teh the code you noticed went missing was replaced with this that also went missing at some point, I think this is a better fix as it will automatically apply the fix for any JSONResponse rather than having to handle it in every instance that expects it.

Thanks! Closing #54.


VSCode not saving file.

Version info from VSCode:
Version: 1.29.1
Commit: bc24f98b5f70467bc689abf41cc5550ca637088e
Date: 2018-11-15T19:07:43.495Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64
Operating System: Linux Mint 19
Kernel: Linux 4.15.0-39-generic
I can open the file (a python file) in VSCode. When it opens it already shows as unsaved (the dot next to the filename, rather than the x to close). I can make changes and hit save, then immediately reopen the file and the saved changes are gone.
Just to be certain this was a problem with VSCode, I opened the file in Sublime, made changes, saved, reopened and the changes are recorded. Oddly enough, these changes still don't show up in VSCode. I can open the file from the VSCode, or directly from a file folder on the OS, neither makes a difference. I noticed there was another thread with a similar issue that had to do with a Git repo. Running git status inside the directory with the .py file inside of it returns:
fatal: not a git repository (or any of the parent directories): .git
I had left the file open in VSCode for sometime, through several reboots and program restarts after having had working save and close, reopen functionality. Then edited the file, exited and restarted and lost all my changes. Now I'm here.
I have the standard python extension and pylint installed.
Let me know if you need more information.
***EDIT: If I make changes in VSCode, then open the file in Sublime the changes ARE THERE. BUT, if I reopen the file in VSCode again afterwards, the changes disappears and the same version of the file always appears. Then, saving the file will overwrite the previous changes. It seems that VSCode is constantly opening a previously cached version of the file no matter what.

Sorry... I jumped over to the wrong Github before I posted... Obviously. Sorry!


Backspace key deletes prompt in lua

LibTerm version 5.1.4 (2)

run lua with no args
hit backspace a few times

This also happens with the Python interpreter
On Wed, Nov 13, 2019 at 2:14 PM Nathan Korth<EMAIL_ADDRESS>wrote:

LibTerm version 5.1.4 (2)

run lua with no args
hit backspace a few times

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ColdGrub1384/LibTerm/issues/42?email_source=notifications&email_token=ALL7CZ4DWB66MRQB6BEOAPLQTRUZ3A5CNFSM4JNBWDYKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HZELLXA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALL7CZZXK2HBBJR5LO447KDQTRUZ3ANCNFSM4JNBWDYA
.


Add command to restart ts server

This plugin has been working great for the most part, but sometimes it does strange things, such as not finding a file within typings/ that is clearly there.
Could you add a command (accessible via cmd + shift + p) that restarts the ts server? I think this would help mitigate this and many other issues that other devs experience.

The tsserver simply needs to be fixed. See #585

Can you try the local fix I created at the comment, which fixes a bug in the project system. Let me know if that fixes your issue

How about running tsserver with something like nodemon?

Any developments on this issue, or the potential fixes to the tsserver? This seems to happen to me sometimes when I leave TS files open and switch between git branches.

There are many things that cause tsserver to fail (adding a module, changing the component in a module from a class component to a function component, etc).
Just have a look at the following:

https://github.com/Microsoft/TypeScript-Sublime-Plugin/issues/585
https://github.com/Microsoft/TypeScript-Sublime-Plugin/issues/539

These and other bugs in this plugin will likely never be fixed. This is a Sublime Text plugin after all, and Microsoft has few reasons to dedicate resources to making it really solid when most devs have already switched over to their very own VS Code =(
That said, this plugin, https://github.com/randy3k/AutomaticPackageReloader, has a command for reloading any Sublime Text plugin. There's currently a small, trivial to fix bug in this plugin that makes this impossible, but I've got a PR open to fix it.
Once this PR is merged, you can just add something like the following to User/Default.sublime-commands:
{ "caption": "Reload TypeScript Package", "command": "package_reloader_reload", "args": {"pkg_name": "TypeScript"} },

Or, to your sublime-keymap file:
{ "keys": ["super+ctrl+alt+t"], "command": "package_reloader_reload", "args": {"pkg_name": "TypeScript"} },

This reloads the TypeScript package, and in most every case I've seen makes it actually start working again. This is a lot easier and faster than restarting Sublime Text.


fix(cli): fix EINVAL running clean on Windows with latest Node

Description
Fix EINVAL error when running clean on Windows with Node 18.20.2+ or 20.12.2+
Test plan
cd packages/test-app
yarn build --dependencies
yarn react-native rnx-clean --include android

Thanks for working that out.  I'd hit this a few times it wasn't obvious what broke.