Upgrade UI - Some issues related to the Staking feature

Describe the bug

[x] The minimum active value on the pool is incorrect

[x] Do not show List pool of the Kusama chain

[ ] Do not show list collator of the Bifrost Polkadot

Actual: On the old version, do not support staking for Bifrost Polkadot
Expect: If do not support -> Remove Bifrost Polkadot on list

[ ] Do not show list validator of the Polkadex/ Polkadex Testnet/ Ternoa/ Ternoa testnet

[x] Still allow user pool with Amount > Available balance

[x] Error when select KSM, ASTR nominated staking record in all accounts mode

[ ] Re-check the APY of the validator (TZERO)

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./1185-merge-build-1.0.2-1-1681285767530.zip
Please check:

[ ] Re-check minimum stake on some parachain (Moonbase, Moonbeam,...)

[ ] Do not display the Staking detail screen of the CAPS

[ ] Update search criteria

Actual: Search by network name
Expect: Search by token


Enable building with WFA2-lib

This adds support for linking against WFA2-lib in StrobeAlign. I’ve added the C++ demo program from WFA2-lib as src/wfademo.cpp and it works (this should of course be removed again).
I chose to go with the technique of downloading the dependency at build time. This is nicely supported by CMake. That way, we don’t have to copy the WFA2-lib sources into StrobeAlign. I have pinned to the most recent WFA2-lib commit so that we’ll always get the same code.
On the UPPMAX cluster rackham, the default CMake and GCC versions are too old, but more recent, working versions are available. I have been able to compile StrobeAlign like this:
module load cmake gcc
cmake -B build
make -C build

(I recently found out about the -B option to cmake. It replaces the mkdir build && cd build && cmake .. steps.)
I am sure that the way the CMakeLists.txt file is structured can be improved, but I’m still learning CMake best practices.

great, I will test this as soon as i have time!


How to style Typeahead options box

Hi, Now i do realize it might be out of scope of this package but i don't know where else to post this question:
How do i style the Typeahead options box?
Everytime i click in the inspector to inspect its styling it closes and remove the DOM object. I want to make it full length (that of the input box) and probably allow it to open above the input field. Any pointers please?

The way that works best for me to be able to inspect the DOM is using Chrome and the Chrome Dev Tools.  As you have noticed, the minute you move from the page the menu goes away.  Here is the trick:
When the menu is open, hover over one of the menu items (not the input box) and then hit the [F8] key.  This will pause the code preventing the menu from going away and allow you to inspect the DOM.

@1-0-1 thanks man! its a perfect solution :)


KeyError: 'symbol'

Traceback (most recent call last):
File "webservice.py", line 147, in getbalances_mt
result_data = wallet.get_balances(data)
File "./com/morningtech/bts/adapter/wallet.py", line 198, in get_balances
logger.info(acc_int.balances)
File "/usr/lib/python3.4/site-packages/bitshares/account.py", line 94, in balances
for b in balances if int(b["amount"]) > 0
File "/usr/lib/python3.4/site-packages/bitshares/account.py", line 94, in
for b in balances if int(b["amount"]) > 0
File "/usr/lib/python3.4/site-packages/bitshares/amount.py", line 76, in init
self["symbol"] = self["asset"]["symbol"]
File "/usr/lib/python3.4/site-packages/bitshares/blockchainobject.py", line 162, in getitem
return super().getitem(key)
KeyError: 'symbol'
e: 'symbol'
{'code': '400', 'data': '', 'msg': "'symbol'"}
This error occurs when the interface is called many times. What is the reason?

account.balances is a property that contains a list of instances of Amount.
Hence, you'd need to do
b = account.balances
b[0]["asset"]["symbol"]


Version 0.1.3

Proposed new version including the DER length fix.

Your patch is working great @sigmike :)
I think we should merge this into the master branch now.


[WIP] Update forms-and-input-components.md

Some minor improvements to the Blazor Forms and Input components topic

@mkArtakMSFT ... I'll need to break this PR when I perform the .NET 7 release activities this morning. .NET 7 content will rotate to the top of the article to make .NET 7 section links work for the release.

@mkArtakMSFT ... Moving this to ...
https://github.com/dotnet/AspNetCore.Docs/pull/27999
... but don't look yet. It's still at DRAFT stage.
I still need to work out a quick non-DA first example and put that on the PR before you review. I'll ping u ... 🤔 ... probably on Wednesday morning .......... unless I hit the Mega Millions jackpot tonight! 💰💰💰💰🤣


feat: Add request_timeout, a parameter as optional to cluster::request() which allows user to set http_client::timeout and add error message for request timeouts

At now, cluster::request can't accept HTTP requests that require a long time to wait for response, especially which takes over 5 seconds. That's because there's no option to set http_client::timeout as user want. So, I added the optional parameter request_timeout to set it.
When an user calls cluster::request(), It would be passed through cluster::request() -> http_request::request_timeout (what I added in this commit) -> http_client::timeout
And also, http_request::run is printing Malformed HTTP response when request has timed out, so I fix it to print a log "Timed out while waiting for the response". For this, I needed to add a public member variable to http_client, timed_out which would be set by https_client::one_second_timer()
Code change checklist

[x] I have ensured that all methods and functions are fully documented using doxygen style comments.
[x] My code follows the coding style guide.
[x] I tested that my change works before raising the PR.
[x] I have ensured that I did not break any existing API calls.
[x] I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

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


Update WifiCommands.md

GoPro Camera(s): Hero3+Black
Tested on: Hero3+Black
Firmware version: v03.03

Added an example to the camera TM.
Is this ok?

Check line 53 again %10%05%0b%10%2f%31
If your example time is 2016-05-11 16:47:49, the correct format if i'm not mistaken would be %16%05%11%16%47%49

With firmware v03.03 they are using hex instead of integers. So in my case, Hero3+Black v03.03 line 53 is correct. Maybe this info could be added with that note?

In that case, yes a note that hex is used to express the integers in the string

would that latest change be ok?

Can you provide a JSON file showing the output of the time in hex?

Could you elaborate?
Do I get this JSON file from the Gopro? (Maybe not in that format, but where do I get that data?)

As in providing the output of the GoPro after running the CAMERA TM command

I'm testing in powershell. (Don't hit me)
The content received when sending this to the GoPro %10%05%0b%12%0f%38
is
0
16
5
11
18
15
56
Were not sure how to format this to JSON

Okay, I see where the failure point is. The output of the command is decimal, not ASCII / ANSI formatting, so if you were to convert 16 5 11 18 15 56 to hex, the result would be 31 36 20 35 20 31 31 20 31 38 20 31 35 20 35 36. That being said, you'd need to convert decimal to hexadecimal. 16 5 11 18 15 56 expressed in hexadecimal is  10 05 0b 12 0f 38

Be sure to write a note, along the lines of converting decimal to hexadecimal

Not quite sure what you meant, but I have tried to simplify it.
Is this ok?

That will do, but I suggest you refer to https://github.com/silent3rror/goprowifihack/tree/patch-8/HERO3

#38

Also, will add a link to @silent3rror's numeric table.

@KonradIT minor changes to save confusion #39


Documentation is not working on Safari

Hey. I noticed that documentation of v1 is not working properly on Safari (iOS/OS X). There is a scroll issue and documentation is unreadable.
I checked in browser console that one file cannot be downloaded https://davidkpiano.github.io:35729/livereload.js and browser does not display rest of the page.
I made a small investigation and discovered that probably this commit caused a problem.
Can you look at that? I think that the simplest solution would be delete live reload plugin, but decision is up to you ;)

Thanks for pointing that out! I went ahead and removed that plugin.

I just checked and scroll issue still exists. So problem must be with something else than plugin I mention above.

Scroll issue should be fixed now!


Command cli utility

a simple tool to send json requests to the hawkular server. useful to simulate a ui or other client that wants to send commands to the agent

Perhaps send out a few examples of how to use it.


[Analytics] Events around sign up/account creation

I used the Segment debugger tab for this troubleshooting but it is not exactly real-time + it is challenging if other people are on the beta at the moment. Let me know if you think there are better ways to go about figuring out which event fires when exactly. Here are some initial observations

Sign up with email event fires even if there was an error signing up; if consecutively you do sign up successfully, the sign up event is fired again
After a successful sign up, the  Sign up with email event is fired more than once
Created account event does not fire if onboarding is skipped

cc @katarinabatina

@anipetrov Want to dip your toes in some play time with Xcode and the debugger (specifically breakpoints) to be able to track these down yourself?

That would be amazing @alloy, thank you so much for offering!!
On Monday, February 8, 2016, Eloy Durán<EMAIL_ADDRESS>wrote:

@anipetrov https://github.com/anipetrov Want to dip your toes in some
play time with Xcode and the debugger (specifically breakpoints) to be able
to track these down yourself?
—
Reply to this email directly or view it on GitHub
https://github.com/artsy/eigen/issues/1123#issuecomment-181310907.

--
[A.]      Ani Petrov
       Artsy <https://artsy.net/>
<EMAIL_ADDRESS>       +1<PHONE_NUMBER>

We’re gonna take it one step further and write some code: https://github.com/artsy/eigen/issues/1127
Can you make a cal event for when you’d have half a day in your morning (my afternoon) to spend on this? If you can wait till the first week of March, then we can even do it at the office when I’m over.

I could do either -- whatever you think would be the best use of your time. If you'd rather not waist half a day of your time here on this, I will put a cal event for some time next week.
Thank you again for offering to take the time!!

Au contraire, I imagine that pairing whilst I’m there would be more efficient!  So let’s do it then :+1:

Amazing, thanks a million! I will put something on your cal!

:bow: :+1:

Gonna take this I'm afraid, you two should pair on the analytics visual mode :P

[ ] Sign up with email event fires even if there was an error signing up; if consecutively you do sign up successfully, the sign up event is fired again
[ ] After a successful sign up, the Sign up with email event is fired more than once
[ ] Created account event does not fire if onboarding is skipped

I've done the top/bottom pretty easily - the sign up event stuff is complicated by the fact that we use 2 types of events to cover 3 types of cases. We need to cover tapping the buttons, actually signing in and creating a new user. I think some of the analytics around these are mixed up, will double check it all over.

@orta Is that TODO list still up-to-date and the double-fire still needs to be fixed?

I think so, yes

Ok :+1:

The Sign up with email event is currently being fired from 3 different locations:

createUserWithName:email: https://github.com/artsy/eigen/blob/84a274ade261b4e32b97881e0299f4a3ce8ff950/Artsy/Networking/API_Modules/ARUserManager.m#L387
createUserViaFacebookWithToken:email:name: https://github.com/artsy/eigen/blob/84a274ade261b4e32b97881e0299f4a3ce8ff950/Artsy/Networking/API_Modules/ARUserManager.m#L446
createUserViaTwitterWithToken:secret:email:name: https://github.com/artsy/eigen/blob/84a274ade261b4e32b97881e0299f4a3ce8ff950/Artsy/Networking/API_Modules/ARUserManager.m#L480

However, in the first case there’s an additional Created account event but no initial ‘start’ event, whereas in the other two cases there are only initial ‘sign up with facebook/twitter’ events and no ‘created’ event.
In short, I think we need:

‘Sign-up with email’ -> ‘created account’
‘Connect with twitter’ -> ‘created account’
‘Connect with facebook’ -> ‘created account’

Does that make sense?

With regards to the double firing of the sign-up event when successfully creating an account, I haven’t yet figured out where/how that is triggered.

@anipetrov In case the above makes sense, I have this PR #1301.

I had not expected the link from my PR would auto-close this.

@orta if you have info on where the double firing of the event is/was coming from, please let me know.

I can't re-procude the double firings, it's very possible I fixed it but did't tick the box

Ok 👍


install-kubeadm: fix What's Next link text

/assign @chenopis
Signed-off-by: Ahmet Alp Balkan<EMAIL_ADDRESS>

This change is

ping @chenopis


Descriptors for command arguments and return values

This is a consequence of #25 , which makes it possible to increase the level of detail of descriptors and the most obvious detail we are missing at this moment is a richer description of command signatures.
I'm still considering if it is worth the complexity to allow for:

overloading of commands
variable number of arguments and return values

My current take is that those features would seriously damage the simplicity and readability of command signatures and it's not worth it.
Use-cases that really require those can still be implemented in the same manner that the SetPropertyValueCommand is implemented: developers can always implement their own HDC-message generation and parsing and deal with the craziest dynamic requirements they may need to face.

Based on insights gained while refactoring the S/GetPropertyValue commands, I decide that it's preferable to keep the HDC descriptor syntax as simple and straightforward as possible by disregarding overloading and variable arguments/returns.
This would otherwise easily become a complexity trap for unexperienced users. More experienced users can nevertheless implement custom commands with a single BLOB argument and a single BLOB return value and compose/parse those in their custom proxy/service subclasses that are tailored to their application layer's needs. This is in fact how S/GetPropertyValue commands deal with the variable datatype of the argument and return value.


operator== could be faster by checking node identity

Story
When comparing two nodes we can speed up use cases where we reuse node instances partially or totally:
Json my_json = Json::object {
    { "key1", "value1" },
    { "key2", false },
    { "key3", Json::array { 1, 2, 3 } },
};
auto my_other_json = my_json;
if (my_json == my_other_json)
{
    ...
}

Two nodes sharing a portion of nodes:
Json common = Json::object {
    { "key1", "value1" },
    { "key2", false },
    { "key3", Json::array { 1, 2, 3 } },
};
Json a = Son::object {
    { "common", common },
    { "extra value", "value1" }
}
Json b = Son::object {
    { "common", common },
    { "extra value", "value1" }
}
if (a == b) {
    ...
}

It may be useful to apply the same concept also for operator<, in such case would always return false:
bool Json::operator< (const Json &other) const {
    if (m_ptr == other.m_ptr)
        return false;
    if (m_ptr->type() != other.m_ptr->type())
        return m_ptr->type() < other.m_ptr->type();
    return m_ptr->less(other.m_ptr.get());
}

These little changes would open memory + CPU optimisations of this sort:
Json reuse_instances(Json new_node)
{
   static std::set<Json> known_values;

   auto it = used_values.find(new_node);
   if (it != used_values.end()) {
       return it->second;
   }

   // may recursively do this for member nodes of object and arrays
   ...

   known_values.insert(new_node);
   return node;
}

...

Json my_object = reuse_instances(Json(Json::object {
    { "key1", "value1" },
    { "key2", false },
    { "key3", Json::array { 1, 2, 3 } },
}));

Automated message from Dropbox CLA bot
@rressi, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

Automated message from Dropbox CLA bot
@rressi, thanks for signing the CLA!

Looks great - thanks!

I think this operator could be improved with the following implementation:
bool Json::operator== (const Json &other) const {
   return m_ptr == other.m_ptr
              && m_ptr->type() == other.m_ptr->type()
              && m_ptr->equals(other.m_ptr.get());
}

In this way the compiler would do the best in any case.

m_ptr is always non-null, otherwise the dereferences here would crash.
I like your version for clarity, though I think it's equivalent in performance.  The compiler can't reorder short-circuit operators any more than it can reorder ifs.  And JsonValue is a polymorphic object, so the virtual calls can't be inlined in order to let the compiler start eliminating cases.


Autosave overrides currently open file

Enabling the new autosave feature in code brings a strange behaviour in your plugin.

VSCode Version: 1.6
OS Version: macOS Sierra 10.12

Steps to Reproduce:
- Enable autosave with 10sec delay
- Open a file and change something
- Switch to another file without saving
- On auto save the open / active file gets overriden by the file which is currently saving.

Copied from https://github.com/Microsoft/vscode/issues/13646

Another related issue is that if you don't have autosave on, edit two files then select save all, the first edited file gets overwritten by the latest edited file

I think this is related to #27?
Same issue here:
VSCode Version: 1.8.0-insider
OS: Windows 10


ASSERTION FAILURE at RecyclerObjectGraphDumper.cpp, line 45 with gc

hi, the following test case asserts at:
RecyclerObjectGraphDumper.cpp, line 45) dumpObject == nullptr,
(run the debug build with the -collectgarbage flag..):
function f() {
let a = [1,2,3,4];
Promise.all(a);
for (let e in a) {
CollectGarbage(e);
}
}
for (let t = 0; t < 100; t++){
f();
}
platforms:all
version: latest master compiled at: 13/4/2019

this is still relevant..


ouldn't find "libopenalpr-native.so"

java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/com.example.xross.aicar-2/base.apk"],nativeLibraryDirectories=[/data/app/com.example.xross.aicar-2/lib/arm64, /vendor/lib64, /system/lib64]]] couldn't find "libopenalpr-native.so"
I found this error when the app run on Android version 5.0.But it certainly work in Android Version 4.4. How should i solve.

Are you using the demo app?


Button className theme override not working

Describe the bug
Option 1 of theme customization described here will not work sometimes. The custom class is appended to the default class of the rendered button. This will work if the custom class is added to the DOM below the default class, and won't work if the class is in the DOM above the default class, given the "cascading" nature of CSS.
Example: bg-orange-500 will override default bg-cyan-500. But, bg-blue-500 won't override bg-cyan-500.
{/* works, you get an orange button */}
<Button className="bg-orange-600">My button</Button>

{/* doesn't work, uses cyan (default) anyway */}
<Button className="bg-blue-600">My button</Button>

To Reproduce
Steps to reproduce the behavior:

Attempt to override button background with any color where the first name is sorted before "cyan" – for example, blue.
Run your project and inspect in browser.
You will see your custom className attached at the end of the rendered class attribute of the native button.
Go to computed tab of dev tools, look for background-color, you will see cyan is being used.
Attempt again with a color appended after cyan – for example, orange.
Build, inspect, and you will again see orange at the end.
This second time, the browser decides to use orange instead of cyan. Ignoring order inside the class attribute, and using some other order.
When using something like Vite hot reload, if you change the color for bg-blue-700 for example, you will see it appear. I believe because hot-reload is injecting this style at the end of the  element of the DOM. This is, being below in the "cascade" and therefore being the one used by the browser.

Expected behavior
All overrides should work, regardless of compiling/minimizing/rendering/etc.
Screenshots
–
System information:

Device: Macbook Pro
Browser: Edge (should be independent of browser).

Project information:

Tailwind: 3.3.2
Flowbite: 1.6.5
Flowbite React: 0.4.6
Type: Vite React-Typescript

Additional context
I did not test this with other components, but might as well apply there too.
Also, let me know if this is intended behavior. We might need to clarify that in the docs.
Edit: Rewrite with my most recent understanding of why this isn't working.

To summarize the above:
Inline override as described here in Option 1 will not work sometimes.
It will depend on wether or not the custom class used is injected in the DOM above or below the default one. Highly dependent on the stack used to compile/minimize styles.

Forgot to mention a workaround I've been using:

Make custom classes more specific to force preference. For example using sm:bg-blue-600 will achieve the same result, but since it's inside a media query, has higher precedence and therefore guaranteeing the overwrite will work.

@codeams the best thing here is to use ! whenever you want to overwrite an already existing style. But, it seems that the documentation doesn't leave it explicit. @zoltanszogyenyi

@codeams Another option is to create your own theme color for the buttons. So you will be adding a new color, and you can use something like:
<Button color="my-color">My button</Button>

BTW, since we introduce the twMerge this is not an issue anymore.


Assignment instead of comparison

You have an assignment instead of a comparison at MigrateGenerateCommand.php:233
Not causing any problem for me but renders the exception at line 238 pointless..

Haha, great catch. I'm almost too scared to fix it, it might be a feature by now ;)

Fixed in #109


Option to show all day events

Currently, we filter out all day events in the get_closest function
This is done because otherwise, the all day event will always be shown when it's taking place even if other events are more relevant
Option 1:
Make this optional using a flag
Option 2:
Show other event unless the end time of the all day event is before the start time of the closest event
I personally prefer option 2 as it will do things the 'smart' way, only show all day events if there's nothing else more interesting

What about multiple all day events?

agh, good question
maybe just print out "multiple all day events" or something like this, prompting the user to click it and view the full calendar

A better way to put it would be:
If next event is tomorrow and closest all day event is today - show all day event
If more than one all day event, concat them up to N string length, order by the way Google's API provides the events
All day event 1, All day event 2, All...


Add support for LabelTags in scripts that set up whole environment

Expected Behavior
MedTagger should add LabelTags with tools to default ScanCategories that are being added during environment setup (both for development and Docker-Compose).
Keep in mind that it should also work while your setup may already exists and you want to upgrade your dev instance to the latest version of code.

[ ] backup_medtagger.py
[ ] fixutres.py


Need to report error if index names are not unique

Is your feature request related to a problem? Please describe.
All index names within a Postgres database have to be unique. Serverpod does not warn the user currently when index names are accidentally reused. This causes .pgsql files to be created that cannot be loaded into the server.
Describe the solution you'd like
serverpod generate needs to stop with an error if index names are not unique.

@Isakdl, maybe this is a rather easy addition to what you are working on with the analyzer now?

@lukehutch do you mean globally across several Protocol files? In a single protocol file this should already be reported as an error since it is even invalid yaml to specify the same key twice.
@vlidholt Yeah it shouldn't be too hard now with the new analyzer since we now support resolving dependencies between files.

@lukehutch Agreed, I would like to add a unique key on the fields in the future that automatically would create an index, i.e:
class: Example
fields:
  name: String, unique

With that said reporting an error is a good idea anyway, and until we have the unique key this fix will be helpful for this case as well.


TextField Form Enter Submit Triggers EndAdornment

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Pressing enter in a TextField to trigger a form submit triggers the Input Adornment onClick. This happens for both startAdornment and endAdornment.
Expected Behavior 🤔
Input Adornment should not be triggered when submitting for with an enter keypress.
Steps to Reproduce 🕹
https://codesandbox.io/s/basictextfields-material-demo-forked-byh55?file=/demo.js
Steps:

Type something in field
Press enter

Context 🔦
Users experience an odd behavior of trying to submit a form while pressing enter
Your Environment 🌎

`npx @material-ui/envinfo`

  System:
    OS: macOS 11.4
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 116.14 MB / 32.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 12.22.1 - ~/.nvm/versions/node/v12.22.1/bin/node
    npm: 6.14.12 - ~/.nvm/versions/node/v12.22.1/bin/npm
  Managers:
    Homebrew: 3.2.3 - /usr/local/bin/brew
    pip2: 18.1 - /usr/local/bin/pip2
    pip3: 21.1.3 - /usr/local/bin/pip3
    RubyGems: 3.0.3 - /usr/bin/gem
  Utilities:
    Make: 3.81 - /usr/bin/make
    GCC: 4.2.1 - /usr/bin/gcc
    Git: 2.30.1 - /usr/bin/git
    Clang: 12.0.5 - /usr/bin/clang
  Servers:
    Apache: 2.4.46 - /usr/sbin/apachectl
    Nginx: 1.21.0 - /usr/local/bin/nginx
  Virtualization:
    Docker: 20.10.7 - /usr/local/bin/docker
  SDKs:
    iOS SDK:
      Platforms: iOS 14.5, DriverKit 20.4, macOS 11.3, tvOS 14.5, watchOS 7.4
  IDEs:
    Nano: 2.0.6 - /usr/bin/nano
    Vim: 8.2 - /usr/bin/vim
    WebStorm: 2021.2
    Xcode: 12.5.1/12E507 - /usr/bin/xcodebuild
  Languages:
    Bash: 3.2.57 - /bin/bash
    Perl: 5.30.2 - /usr/bin/perl
    PHP: 7.3.24 - /usr/bin/php
    Python: 2.7.16 - /usr/bin/python
    Python3: 3.9.6 - /usr/local/bin/python3
    Ruby: 2.6.3 - /usr/bin/ruby
  Databases:
    PostgreSQL: 13.3 - /usr/local/bin/postgres
    SQLite: 3.32.3 - /usr/bin/sqlite3
  Browsers:
    Chrome: 92.0.4515.131
    Firefox: 90.0.2
    Safari: 14.1.1

Thanks for the feedback.
This took me a while to understand but this is actually expected behavior considering you use <button />. That means that the button is actually equivalent to <button type="submit" />. On submit every submit button receives a click event.
What you actually want is <button type="button" /> in the endAdornment.


Which paper did this method come from?

Thanks for your great work!
I think this pixel-level domain adaptation method may be useful in my research work. So I am curious whether this method comes from any specific paper, which will help me to cite and understand its theory.

Hi @KaiiZhang!
Unfortunately, there was no paper or public benchmarks - just my experiments on private datasets.
Feel free to send me an email to<EMAIL_ADDRESS>if you think I can help you.


Overriding autovalue id with the actual database id?

My id field is actually the database key and when a cell is updated I use the id field to persist the change to the database.
However when a new row is inserted the key is automatically generated due to the autoValue property on the .
Now if the user edits the newly created row, I can't update the database because the id field is not the same as the newly created record in the database.
When the  afterInsertRow() hook is called  I write the new row to the database, which gives me a new id, however the Bootstrap table has already been updated with the autovalue id.
What is the best way to update my data with this the new id field obtained from the database insert?

Actually I did some experimenting and it looks like I can change the id in the afterInsertRow() handler and the changed id appears in the grid.
Feel free to close...

Actually I did some experimenting and it looks like I can change the id in the afterInsertRow() handler and the changed id appears in the grid.

surely, you can change it, the auto id value is just for the application which doesn't have a database or state management used and it prevent the new row insert fail without rowkey.


invalid result on get email

My ios App get this error
Get user email error: PlatformException(UNKNOWN, Invalid result on get email: (Function), null, null)
var login = await facebookLogin.logIn(permissions: [
  FacebookPermission.publicProfile,
  FacebookPermission.email,
]);

if (Platform.isAndroid) {
  login = await facebookLogin.expressLogin();
}
switch (login.status) {
  case FacebookLoginStatus.success:

 await facebookLogin.getUserProfile();
    final token = await facebookLogin.accessToken;
    debugPrint("token $token");
    if(token!.permissions.contains(FacebookPermission.email.name)){
    final _email = await facebookLogin.getUserEmail();
    debugPrint("email $_email");

    }

    break;
  case FacebookLoginStatus.cancel:
    ToastHelper.showShort("Login cancelled by the user.");

    break;
  case FacebookLoginStatus.error:
    ToastHelper.showErrorLong(
        'Something went wrong with the login process.\n'
        'Here\'s the error Facebook gave us: ${login.error}');

    break;

}

Hey! Sorry for the late reply, somehow I missed this issue. Is the problem still relevant?

Closed due to inactivity. If the problem is still relevant, fill free to reopen it.


Transactional Benchmark

Created a benchmark txn_benchmark.cpp similar to benchmark.cpp to measure performance of transactional Anna.

CACHE and WARM commands work the same way; creates a dummy transaction, GETs/PUTs all the keys
LOAD runs a number of txns, with each schedule containing a number of actions (GETs/PUTs). Schedules and actions are randomly generated and randomly interleaved
Added compiler target for txn_benchmark.cpp

LGTM! Next step would be to integrate zipf from the old benchmark.


Call stack & registers are not shown when rsp is changed

I was debugging my own implementation of fibers on Windows, when suddenly I noticed, that changing rsp to a new area breaks call stack & registers shown in the debugger. The problem is present until I change rsp back to original thread's stack.
This behavior is also present when using WinAPI fibers, so to reproduce it you can just create a fiber and immediately switch to it.
Breakpoints are working, but the arrow is missing. You can see it on the screenshot.

I believe this has been addressed by my recent passes over the process control layers. I now get correct behavior when testing with Win32 fibers:

You can test these new changes on the dev branch. Please let me know if these passes did not address this issue, and if you're still experiencing it.


Guard http driver from use wihout root

PR Checklist
Please check if your PR fulfills the following requirements:

[ x ] The commit message follows our guidelines: CONTRIBUTING.md#commit
[ x ] Tests for the changes have been added (for bug fixes / features)
[ x ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ x ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

HttpDriverModule can be imported directly, which currently doesn't make any sense.
HttpDriverModule.forRoot can be called from multiple injectors.

Issue Number: N/A
What is the new behavior?

Consumers must import using HttpDriverModule.forRoot.
HttpDriverModule.forRoot can only be called from one injector. Hopefully the root injector.

Does this PR introduce a breaking change?
[ ] Yes
[ x ] No

Other information

I added some CI-related changes.

@all-contributors please add @LayZeeDK for infra.

@all-contributors please add @LayZeeDK for infra

@all-contributors please add me for infra.


Suspend a backend

Hi, as always thanks to you and your team for your excellent work! As I have three video cards on this machine, I have three backends running. I understand this is helpful when generating multiple images, but for single generation, the additional backends hog GPU resources from other programs, and don't advantage StableSwarm. Is there a way to suspend an unneeded backend without uninstalling it? Also since many integrated motherboard cards have GPUs, is there any advantage to opening a backend without a dedicated GPU for the motherboard?

Very good idea, I can see a lot of use cases where you'd want to temporarily disable a backend, or only sometimes enable it (in addition to your own points, I imagine a backend that's on another machine that you don't always care to bother turning on first or something like that).
I've added a lil toggle switch on the left of the backend title bar to quickly turn a backend on or off. It inherently persists in server settings, same as all other settings.

This also bonus-solves a standing TODO list entry of making an easy way to reload a backend if it dies unexpectedly (now just slap the disable toggler twice! When in doubt, turn it off and back on again, heh)

Also unrelated but while I was editing backend UI anyway I added the ability to add custom titles to backends, so you might eg have backends labeled "Local RTX 3090", "My 4090", "Living Room PC", "Server Box", etc. whatever to easily tell what they are if it's not obvious from the config. Can literally just click the title while in backend edit mode and change it.

Also since many integrated motherboard cards have GPUs, is there any advantage to opening a backend without a dedicated GPU for the motherboard?
I'm not sure what a motherboard integrated GPU is? Do you mean CPU-integrated graphics (Intel Iris/Xe/etc or AMD APUs)? Or laptop (mobile-class) GPUs?
Either way: SD only runs well on GPUs, and usually dedicated GPUs are strongly preferred. Intel integrated graphics is probably flat out a bad idea, AMD APUs might work but will likely be pretty slow. You can technically run SD on CPU directly, but it's horribly slow (takes tens of minutes to generate, vs. a decent GPU generating an image in a few seconds).
Dedicated laptop GPUs (eg an nvidia mobile class) are slower than their desktop equivalents, but still work well. I've tried a laptop mobile RTX 4080 and it runs SD very quickly (at least when plugged in and thus in performance mode).


Multiple parameters for an http reuqest body.

I'd like to have my POST request's data sent as a body but only one body is allowed per HTTP request. Is it possible to have multiple parameters that get sent as one body for a request?

You should consider posting a JSON or XML payload--that way you can send as much data as needed.
HTTP does not allow multiple BODY entities in a single post.  It does, though, allow complex POST bodies.

Yes I do have an JSON payload set up.
{
"BrokerId": 0,
"Inventories": [
{
"ProductionID": "string",
"Section": "string",
"Row": "string",
"StartSeat": "string",
"EndSeat": "string",
"InHandDate": "string",
"IsPDFAvailable": true,
"IsAllowSplit": true,
"IsSpeculation": true,
"Quantity": 0,
"Barcodes": "string",
"Cost": 0,
"Face": 0,
"Description": "string",
"InventoryStatus": "string",
"SupplierTIcketID": 0
}
]
}
However I would like for each field in Inventory array to enter it as its own separate parameter (as if it was a query) instead of directly into the JSON on the body field.

You can describe the payload, you can't make our tools consider it as separate parameters.


Update steermouse to 5.1

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as 1b1775f9bae9c2ef3f360b582ba649ef13f50adf. Your contribution is still credited to you.


feat: add print structure command

A handy #print command for structures.

Still WIP but comments and suggestions welcome!

This should go in core, replacing the existing result of #print for structures.

@digama0 I finally got around to posting an RFC lean4#3644.


Switch check from !isMac to isWindows for specificity.

Only Windows uses Mod+y, this avoids adding the shortcut to other operating systems (such as Linux or iOS).
This adds better support for spoofed user-agents when testing mobile sites on desktop devices.
You could also add || /Pocket/.test(navigator.platform) but it really is optional

Thank you, Aaron!


Update batch size to not be described in terms of gpus and allow non integer values

Allowing STRING types allows for the valid values of auto and auto:N to be set in addition to integers.
Also adjusting the default to auto to remove the magic number and hopefully have better out of the box tuning.
Corresponding change on the eval side: https://github.com/instructlab/eval/pull/67
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[ ] Changelog updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

Is there any corresponding code in eval that we should reference here?

Updated desc to reference corresponding change in eval.

Is there any corresponding code in eval that we should reference here?

Updated desc to reference corresponding change in eval.

If you rebase for any reason please update the commit message with the eval corresponding change, thanks!


Wrong API Endpoint

Hi,
Ive noticed that when calling this api you are using the wrong hostname of burstsms.com/api, this as per http://support.burstsms.com/hc/en-us/articles/202058793-SMS-API-Fundamentals should be transmitsms.com/api. burstsms.com/api no longer works.

I have just installed the gem, burstsms (0.1.4)
via Gemfile: gem 'burstsms'
I seem to have the same error:
Errno::ECONNREFUSED (Connection refused - connect(2) for "burstsms.com" port 443):

0.1.4 does not contain this patch, im guessing the author has not published a new gem with the fix.

mmm, anyway I can edit or override the endpoint?

Nope thats why i patched in the new url :(
On Mon, Feb 5, 2018 at 3:04 PM Ryan Harnwell<EMAIL_ADDRESS>wrote:

mmm, anyway I can edit or override the endpoint?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/madeindata/Burst-Sms/issues/3#issuecomment-362976306,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACGX3gkRBk-rd-Wyf-QBZ1lBldDULBuCks5tRn3EgaJpZM4J7QoT
.

Im sorry i totally forgot to mention that there is another gem for use with burstms, developed by burstsms.
See if you have more luck with:
https://rubygems.org/gems/ruby-transmitsms/versions/0.1.3


Mockserver module

Sure, I was thinking about restructuring the docs completely since we have
modules in the monorepo anyways :)
On Wed, 16 May 2018 at 09:38, Kevin Wittek<EMAIL_ADDRESS>wrote:

@kiview approved this pull request.
LGTM, we should add some documentation in a later PR ^^
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/testcontainers/testcontainers-java/pull/696#pullrequestreview-120523038,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAIiuw3Ru3cjnb_zxwvp0iRFrJ2vsYgks5ty9d4gaJpZM4UAnvj
.


add property tests for fast division routines

@Bodigrim I made the fixes as suggested by you. I also made an internal library for testing instead of exposing the internal module. Let me know if this looks good.

The coverage build is not working due to internal library. This may be one of the reasons why we did not use internal libraries earlier:
cabal --project-file=../../cabal.project.coveralls v2-build --enable-tests --enable-benchmarks --enable-coverage all

HEAD is now at d9e2017 Bump package version to 1.1.0.

Resolving dependencies...

Error:

    Internal libraries only supported with per-component builds.

    Per-component builds were disabled because program coverage is enabled

    In the inplace package 'unicode-transforms-0.3.6'

I tried a hack to symlink only the internal module that we are trying to test for using in the tests. I could not find a better way to do this without exposing the module.
I tried pulling the module into the test executable right from its current location without symlinking it into a separate directory but that does not work because cabal then starts picking up all the modules in the Data directory directly and not via the unicode-transforms library.
Let me know if there is a better way to do this which does not require disabling coverage.

Ah, symlinks do not work on Windows. So there seems to be no way.

Maybe add {-# OPTIONS_HADDOCK hide #-} to Data.Unicode.Internal.Division? Technically it will remain available, but no haddock documentation should hopefully discourage its use.

Ok, I have exposed the module and hidden the haddock.

Any ideas what's wrong with ICU tests?
https://travis-ci.com/github/composewell/unicode-transforms/jobs/362961313#L708

The test passes on my machine which has libicu65 installed:
Prelude Main> import qualified Data.Text.ICU as ICU
Prelude Main ICU> import qualified Data.Text.Normalize as T
Prelude Main ICU T> import qualified Data.Text as Text
Prelude Main ICU T Text> T.normalize T.NFKD (Text.pack "\127340") == ICU.normalize ICU.NFKD (Text.pack "\127340")
True

The travis test is using libicu55 from Ubuntu xenial. We can try if libicu55 reproduces this error then we can perhaps say its because of old icu lib. If we cannot reproduce with that then something weird is going on.

We can also try using Ubuntu bionic in the CI tests, we will have to find a way to install icu on bionic.

It seems codepoint 1f16c (\127340) was introduced in unicode version 12 (https://www.unicode.org/Public/12.0.0/ucd/UnicodeData.txt) and icu55 is at unicode version 7. We need to use the right version of icu i.e. a version that supports unicode version 13.

I see. As long as we know that it is an obsolete ICU who is wrong and not unicode-transforms, I think we can go ahead with a merge.

We need minimum icu-64 to avoid this issue. I will check where to get it from and how to install it on travis.
This also raises another important question: should we bump the major version of unicode-transforms whenever we add support for a new version of Unicode? Though unicode is upward compatible so I guess there should not be any issues, but I am not very sure as I am not an expert.

ICU official page redirects to its github release page: https://github.com/unicode-org/icu/releases/tag/release-67-1 where I can find a tgz for ubuntu, if we want to install this we may also have to install the icu data package separately. Not sure how easy would it be to work with this.
ubuntu focal seems to have icu66 we can try that: https://packages.ubuntu.com/focal/libicu66

I found a Lauchpad PPA, which claims to provide icu-66 for ubuntu bionic: https://launchpad.net/~pkgcrosswire/+archive/ubuntu/ppa/+packages
Travis seems to support Launchpad PPAs: https://docs.travis-ci.com/user/installing-dependencies/#installing-packages-from-a-custom-apt-repository

LGTM!


Add generic Satoshi chain mediator

MDIP needs a Bitcoin Testnet mediator to facilitate adoption among web3 developers using the Bitcoin Testnet ledger.

Changed the goal to add a mediator that works with any BTC-derived chain (Feathercoin, Litecoin, Dogecoin, etc), mainnet or testnet, specified in configuration.


Hyper-V server is now a windows server, allowing ue4-docker setup

Fixes #119

Closing as per https://github.com/adamrehn/ue4-docker/issues/119#issuecomment-747239912


seo fixes

hi! we are doing some seo cleanup over in the registry, and w are updating the titles to align with those changes. this will also allow us to support breadcrumbs on the page.
after merging, this requires a release for the registry to know about it. thank you!!

Thanks @susanev! We'll try to look at it soon.


Fix linting

This is closes #3

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


chore(token): add symbol functions to print classes as their values

This makes it so you can easily print the value of the IOToken and mIOToken classes without having to use .valueOf() everytime. FWIW - mIOToken had this but IOToken did not. Added unit tests to show the usefulness.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.51%. Comparing base (911b02b) to head (3018049).
Report is 42 commits behind head on alpha.

Additional details and impacted files
@@            Coverage Diff             @@
##            alpha      #91      +/-   ##
==========================================
- Coverage   80.86%   79.51%   -1.35%
==========================================
  Files          19       20       +1
  Lines         465      532      +67
  Branches       95      101       +6
==========================================
+ Hits          376      423      +47
- Misses         85      109      +24
+ Partials        4        0       -4

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

:tada: This PR is included in version 1.0.7-alpha.4 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


README

Add a README

Badges (build status, license)
Intro
How to use
Features
License footer

Completed


MetaData.getImportedKeys fails if table not found

If you try to getImportedKeys from a table with wrong referenced table, the method will fail with table not found exception.
The table definition:
CREATE TABLE "server" (
	"id"	INTEGER NOT NULL,
	"user_id"	INTEGER NOT NULL,
	"servergroup_id"	INTEGER NOT NULL,
	"name"	VARCHAR(128) NOT NULL,
	"host"	VARCHAR(128),
...
	PRIMARY KEY("id"),
	FOREIGN KEY("user_id") REFERENCES "user_old"("id"),
	FOREIGN KEY("servergroup_id") REFERENCES "servergroup"("id")
);

The table user_old doesn't exist.
java.sql.SQLException: Table not found: 'user_old'
	at org.sqlite.jdbc3.JDBC3DatabaseMetaData$PrimaryKeyFinder.<init>(JDBC3DatabaseMetaData.java:2011)
	at org.sqlite.jdbc3.JDBC3DatabaseMetaData.getImportedKeys(JDBC3DatabaseMetaData.java:1598)

The FK is not a primary key.
It's not my table, it's the bundled database of an installed pgAdmin4 6.2 and I can't change the database on my own.

Would you know what's the behaviour on other drivers like PostgreSQL / MySQL ? I can't say whether it's a bug or expected.

First, it's possible to get the same configuration in other database - a FK without FK table. In some databases it's easy to create such configuration if you set "wrong" permissions on tables... means the user doesn't have the right to access the FK table...
I didn't test such situations in other DB systems, but I had a similar problem in PostgreSQL and in this case, the driver returned the "wrong key" but had an exception while accessing metadata with this key.
In my opinion: The JDBC driver should return the configured keys, also if it's invalid or don't return invalid keys, but it should not fail.

I was checking the Javadoc for DatabaseMetaData.getImportedKeys and it says:

Throws: SQLException – if a database access error occurs

I think you are right, it should not throw an exception in that case.


broadcast

Original author: Nikhil Jain
Original issue: https://charm.cs.illinois.edu/redmine/issues/605

No body.

Original author: Harshitha Menon
Original date: 2015-02-09 22:11:53

This test basically does a point to point broadcast and measures the time taken for it.
Array element 0 sends point to point messages to other array elements, waits to receive a ack and then measures the time for point to point broadcast.

Original author: Harshitha Menon
Original date: 2015-02-09 22:26:25

I don't think this test needs to be in the autobuild.

Original date: 2015-02-10 15:14:14

If it's not a useful test, then please delete it.


Allow to trigger card automations on microphone unmuted

Is your feature request related to a problem? Please describe.
For my doorbell, I need to run a Home Assistant service to let it know someone answered it. This also makes it stop ringing on other devices.
I would like to be able to send such command to my doorbell as soon as I unmute microphone in the Frigate Card, to signal that I answered it.
Describe the solution you'd like
Something like this:
automations:
  - conditions:
      microphone_unmuted: true
      camera: doorbell
    actions:
      - action: call-service
        service: script.answer_doorbell

Describe alternatives you've considered
@pergolafabio's idea was to create his own microphone button so that he can call the service before unmuting the card. That's actually not bad at all.
Additional context
N/A

Or perhaps:
automations:
  - conditions:
      microphone: unmuted <-------- or muted
      camera: doorbell
    actions:
      - action: call-service
        service: script.answer_doorbell

One issue that comes to mind is that when the card starts (or the config is edited/reloaded) it will immediately match this criteria, and thus call your service. This is probably not what you want? May need to add something like:
automations:
  - conditions:
      on_config_load: false <----------------
      microphone: unmuted <-------- or muted
      camera: doorbell
    actions:
      - action: call-service
        service: script.answer_doorbell

Interesting, you mean when the card starts with the configuration of always unmuted?
If not, I wonder why would such condition match on card start.

Interesting, you mean when the card starts with the configuration of always unmuted?

Ah, yes, sorry -- this would only be an issue on my alternative version where you could specify "unmuted" or "muted".

This is awesome!


Two fixes related to cross-module-optimization

don't make imported type declarations usable-from-inline.
relax a SILVerifier assertion: allow that a shared_external function references a shared function.

rdar://87930768

@swift-ci smoke test


Enhance LLM-based crash triaging with project function code

1.Modification of regex for extracting crash info
Previously, I used 65 equal signs as the regular expression for crash info extraction. The experimental results show that some crash reports do not contain 65 equal signs. In addition, due to the limitation of propmt token, I decided to trim the crash info and get the content after 'ERROR: ' and before 'SUMMARY'. This is the modified regular expression.
2.Slicing driver code and project function code in triage prompt
Due to the limitation of prompt token, I have to trim function code. I trim based on the number of lines in the crash stack, which I call target_line for convenience. For driver code, I trim from one to target_line lines. Note that this does not solve the problem that in some projects, the driver code is too long and seriously exceeds the prompt token. This requires subsequent discussion and processing. For project function code, for each related project function in the crash stack, I trim 4 lines forward based on target_line. I add 5 lines including target_line to the prompt.
The main added or modified codes include: query_introspector_function_line, FUNC_NAME, LINE_NUMBER, _parse_func_from_stacks, _format_triager_problem, _slice_driver_code, _slice_func_code
In addition, I also merged the latest developments made by other developers.
If you have any questions or need any changes, please let me know. Thank you for your review！

Sorry for too many other developers' commit, closed. Please see #379.


Change Footer version handling, Make compression dynamic

Change Footer version handling
Simplify version handling by switching to JSON instead of binary serialization.
fixes #1058
Make compression dynamic
Instead of choosing the compression during compile time via a feature flag, you can now have multiple compression algorithms enabled and decide during runtime which one to choose via IndexSettings. Changing the compression algorithm on an index is also supported. The information which algorithm was used in the doc store is stored in the DocStoreFooter. The default is the lz4 block format.
fixes #904

Should be fine now, except removing the lz-frame stuff


Matomo Container Documentation missing

At the moment it's copy paste of the Crayfish documentation: https://github.com/Islandora-Devops/isle-buildkit/tree/master/matomo

Resolved by https://github.com/Islandora-Devops/isle-buildkit/pull/31


im getting some critical error when i try to start it

full code thing: System.IO.IOException: The process cannot access the file 'currentlyInstalled.apk' because it is being used by another process.
at System.IO.FileSystem.RemoveDirectoryRecursive(String , WIN32_FIND_DATA& , Boolean )
at System.IO.FileSystem.RemoveDirectory(String , Boolean )
at System.IO.Directory.Delete(String , Boolean )
at QuestPatcher.Core.SpecialFolders.CreateAndDeleteTemp() in D:\a\QuestPatcher\QuestPatcher\QuestPatcher.Core\SpecialFolders.cs:line 69
at QuestPatcher.Core.QuestPatcherService..ctor(IUserPrompter prompter) in D:\a\QuestPatcher\QuestPatcher\QuestPatcher.Core\QuestPatcherService.cs:line 44
at QuestPatcher.Services.QuestPatcherUiService..ctor(IClassicDesktopStyleApplicationLifetime appLifetime) in D:\a\QuestPatcher\QuestPatcher\QuestPatcher\Services\QuestPatcherUiService.cs:line 37
at QuestPatcher.App.OnFrameworkInitializationCompleted() in D:\a\QuestPatcher\QuestPatcher\QuestPatcher\App.axaml.cs:line 77

You already have another instance of QuestPatcher open in the background. Restarting your PC will fix this issue.


realm with flutter not working anyone please help me with demo code

Hi,
I new on flutter.try authentication user application show as below attachment.
and I stack with realm DB implementation with flutter (no document for flutter+realm).
I used this link - "https://github.com/mogol/flutter_realm "for reference to connect the realm DB.
if some give short "DEMO CODE" user registration data save on realm data.
I'm trying to resolve this from the last 5 days.

thank in advance

HI @manishydv
I quite sure realm doesn't support dart [realm issue], if you are experiencing issues w/ the 3rd party plugin flutter_realm you may want to open an issue in the dedicated github;
if you get stack Flutter certainly doesn't lack of noSQL db [hive, sembast etc...]
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Inventor: infusion cannot be ued

Using latest version of the game and the mod.
I have a Vitriolist Inventor is my party. I would like to place a Repeating Shot infusion on a longbow, but it doesn't allow me to place it. I am getting a warning in the items description, that i am not proficient with it, so maybe that is the cause of the problem, but for the best of my knowledge, inventor doesn't need to be proficient in an item to place the infusion on it. The longbow would be going for the Yaoman barbarian anyways.
domino2.zip

Repeating is for weapons that have the loading property only, so crossbows in this case.

Longbows and other ranged weapons would just use the normal Enhance Weapon infusion to get +1.


Support Recursive Table Lookup

Tables may be stored in sub-folders of the "Tables" folder.
The Studio does not check these folders right now.

Added with 2.15.0


add class cache to support android multithreading

cache class for jni multithreading support of android (see
android doc)
Example
cache your jvm and needed classes in the thread before new thread is created:
let jvm = env.cache_java_vm().unwrap();
let _ = env.find_class(CLASS_NAME);

use JniCache::get_jvm() to get cached jvm afterwards
find_class calls afterwards will use the cached class

I'm pretty sure that some of this is not safe to do. For example, caching the JVM pointer in such a way that it can be accessed from different threads could lead to unexpected behavior since the pointer returned by GetJavaVM is specific to the thread in which it's called. The doc you linked even touches on this:

This usually does what you want. You can get into trouble if you create a thread yourself (perhaps by calling pthread_create and then attaching it with AttachCurrentThread). Now there are no stack frames from your application. If you call FindClass from this thread, the JavaVM will start in the "system" class loader instead of the one associated with your application, so attempts to find app-specific classes will fail.

Maybe only keep a hidden jvm pointer around to delete the global refs on Drop? Even then, I'm not sure how safe it is to use another thread's jvm pointer.
Also, rather than enforce caching for every find_class call, add a find_class_with_cache method that accepts a cache to use for the lookup. That way, it's up to library consumers whether or not to cache and where to store the cache.
Side question: What happens if a cached class gets reloaded? Does it reload to the same location pointed to by the cached pointer? Or does that pointer get invalidated?

closing due to lack of activity


Drop gardener-seed-admission-controller

What this PR does / why we need it:
With https://github.com/gardener/gardener/pull/7053, the gardener-seed-admission-controller binary will be dropped from the gardener/gardener code base.

/hold
until https://github.com/gardener/gardener/pull/7053 is merged

/kind enhancement
/lgtm
/approve
/unhold
https://github.com/gardener/gardener/pull/7053 is merged

Thanks @oliver-goetz  :)


Fixes bars not showing in responsive, small histogram

Fixes #400 , related to https://github.com/CartoDB/cartodb/issues/8320
@javisantana

👍


Added VERISIGN_INTERMEDIATE_ parameter

When using the VERISIGN_PRODUCT_ Parameter one also has to add this parameter:
VERISIGN_INTERMEDIATE_ = """ """
While doing this, I also added code, so the external_id field is filled with data from CA-Answer

Hi @hosseinsh,
is the build error a cause of my changes or is it simply a problem of travis usage?
thanks
ferdinand

Coverage decreased (-0.02%) to 58.965% when pulling ed3472d029c88b678e1e9e5104151bd888b207b4 on sirferl:master into 0d76690091104a916477a3b9bb96041c19e37054 on Netflix:master.

Hi @sirferl, not sure about the Travis error, looks like to have resolved.

Merging this one for now. Thank you @sirferl.


ability to pass configuration to /buildtarget

Yep, of course. Cheers!

Got this working but I think I am going to have to do another PR so you can do /p:Platform="Any CPU"

Think you're probably going to need to query for build targets thinking
about it.
On 21 Feb 2015 12:47, "Jonathan Channon"<EMAIL_ADDRESS>wrote:

Got this working but I think I am going to have to do another PR so you
can do /p:Platform="Any CPU"
—
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-server/pull/181#issuecomment-75370195
.


What problem does this feature solve?
目前使用remix插件时，会报错：
[ Farm ] Failed to start the server:
 Error: Vite plugin 'remix' is not compatible with Farm for now. Because it uses viteConfig.build['manifest'] which is not supported by Farm. Current supported keys are: outDir,sourcemap,minify,cssMinify,ssr,watch,rollupOptions,assetsDir

配置如下：
import { defineConfig } from '@farmfe/core';
import { vitePlugin as remix } from "@remix-run/dev";

export default defineConfig({
  vitePlugins:[remix()]
});

What does the proposed API look like?
未来是否会支持remix插件？

我们看看哈


Allow users to cache certain requests in envoy

might be some x-kusk: cache extension. Also we need to think about cache eviction policies and stuff like that.
Some background information:
https://twitter.com/copyconstruct/status/1288996754814193664
https://www.envoyproxy.io/docs/envoy/latest/start/sandboxes/cache
https://docs.google.com/document/d/1WPuim_GzhfdsnIj_tf-fIeutK0jO4aVQfVrLJFoLN3g/view#heading=h.wjxw6fq7wefi

@povilasv How is this going? Any progress on the implementation?

@povilasv Can you add some update here?


Use Hamcrest assertions instead of JUnit in common/reactive (#1749)

Part of #1749
I'll do backport into helidon-2.x after merge.
I've already signed OCA. It's an automation problem.
I followed these instructions

[ ] Create Backport into helidon-2.x

/oca-checked

/trigger


refactor!: remove --workload parameter from run.sh and deprected --workload dlstreamer run

BREAKING CHANGE: we no longer support --workload parameter from run.sh and thus deprected the dlstreamer workload use cases.
Closes: #381
PR Checklist

[x] Added label to the Pull Request for easier discoverability and search
[x] Commit Message meets guidelines as indicated in the URL https://github.com/intel-retail/automated-self-checkout/blob/main/CONTRIBUTING.md
[x] Every commit is a single defect fix and does not mix feature addition or changes
[ ] Unit Tests have been added for new changes
[x] Updated Documentation as relevant to the changes
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with repository license and clearly outlined the added dependency.
[ ] PR change contains code related to security
[ ] PR introduces changes that breaks compatibility with other modules (If YES, please provide details below)

What are you changing?

Issue this PR will close
close: #381
Anything the reviewer should know when reviewing this PR?
Test Instructions if applicable

git clone this PR
make clean-all
test model downloads: make clean-models; ./download_models/getModels.sh  should see models downloaded without problem
unit tests: make unit-test-profile-launcher should see all passed
functional test: you can run any pipeline profile without using --workload and it should work; one example:

$ make run-camera-simulator
$ PIPELINE_PROFILE="object_detection" RENDER_MODE=1 sudo -E ./run.sh --platform core --inputsrc rtsp://<IP_ADDRESS>:8554/camera_0

smoke test: make run-smoke-tests should see ALL PASSED

If the there are associated PRs in other repositories, please link them here (i.e. intel-retail/automated-self-checkout )

Note: will leave the cleanup of DLstreamer related code underneath ./configs/ in a separate code-cleanup PR

Makefile list-profiles example needs to remove --workload ovms

Makefile list-profiles example needs to remove --workload ovms

thanks, good catch!


Fix numerical instability in DuckDB Q15

The revenue calculation was numerically unstable and Q15 could (incorrectly) return no results.

FWIW, I guess that the other libraries have a problem with this as well. @phofl: This kind of instability is something that Dask will become prone to once we disable CSE. If you have a good intuition on how to detect this on query generation, I'd love to flag/handle this.


Hub - Echo Service

I added a simple Echo Service to Hub, but we need return_route to work over TCP transport for the that to be completed
This is required for this example https://github.com/ockam-network/ockam/issues/951

Here's a test program that works with UDP transport:
alias Ockam.Transport.UDPAddress
{:ok, address} = Ockam.Transport.UDP.create_listener [port: 3000, route_outgoing: true]

defmodule Printer do
  use Ockam.Worker

  @impl true
  def handle_message(message, state) do
    IO.inspect(message, label: "printer")
    {:ok, state}
  end
end

Printer.create(address: "printer")

route_to_node_running_echo_service = [%UDPAddress{ip: {127, 0, 0, 1}, port: 5000}]
message = %{
  onward_route: route_to_node_running_echo_service ++ ["echo_service"],
  return_route: ["printer"],
  payload: "hello"
}

Ockam.Router.route(message)

Is it done? @mrinalwadhwa @hairyhum

Yes. It's done.


Remove version: Jackett.Jackett version 0.20.2444

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.20.2444/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


(REQUEST) AT Targets Anti-Aim

the AA will always follow the nearest target or nearest to crosshair, for me this was a game changer in my HvH days in csgo,maybe it could be implemented here?

no? why would you do that you have fake angles on tf2
use them


Warn user that pre-release dependencies need to be explicitly listed in the Gemfile

[x] Add failing spec
[x] Add warning

Closes #4657.
@segiddins @RochesterinNYC

So far, I've managed to recreate the issue in #4340. Will be figuring out when and under which conditions to display the warning.

Added the baseline implementation. So, the output would be:
<EMAIL_ADDRESS>~/dev/test-bundler/4657/bundled_app
$ dbundle install
Fetching source index from file:/Users/boggs/dev/bundler/tmp/gems/remote2/
Resolving dependencies...
Bundler could not find compatible versions for gem "rspec-core":
  In Gemfile:
      guard-rspec (= 4.6) was resolved to 4.6.0, which depends on
        rspec (< 4.0, >= 2.99.0) was resolved to 2.99.0, which depends on
          rspec-core (~> 2.99.0)

      rspec-rails (= 3.5.0.beta1) was resolved to 3.5.0.beta1, which depends on
        rspec-core (= 3.5.0.beta1)

NOTE: Pre-release dependencies need to be explicitly listed in the Gemfile.

Thoughts on this? @segiddins @RochesterinNYC

Ah, I seem to keep breaking unrelated specs here and there.
Failures:
  1) bundle install with git sources gem install hooks runs pre-install hooks
     Failure/Error: expect(err).to eq_err("Ran pre-install hook: foo-1.0")

       expected "" to eq err "Ran pre-install hook: foo-1.0"

       Commands:
       $ /home/travis/.rvm/rubies/ruby-1.8.7-p374/bin/ruby -I/home/travis/build/bundler/bundler/lib:/home/travis/build/bundler/bundler/spec -r/home/travis/build/bundler/bundler/tmp/libs/install_hooks.rb -rsupport/hax /home/travis/build/bundler/bundler/exe/bundle install --no-color
     # ./spec/install/gemfile/git_spec.rb:875
Finished in 9 minutes 0 seconds (files took 2.67 seconds to load)
2036 examples, 1 failure, 9 pending
Failed examples:
rspec ./spec/install/gemfile/git_spec.rb:858 # bundle install with git sources gem install hooks runs pre-install hooks

Maybe a Travis fluke? Could you re-run the specs for this?

Aha! Looks like the specs finally passed haha.

@homu delegate=RochesterinNYC

:v: @RochesterinNYC can now approve this pull request

@homu r+

:pushpin: Commit 2f48243 has been approved by RochesterinNYC

:hourglass: Testing commit 2f48243 with merge bfadfdb...

:broken_heart: Test failed - status

@homu retry

:hourglass: Testing commit 2f48243 with merge b516176...

:broken_heart: Test failed - status

@homu retry

:hourglass: Testing commit 2f48243 with merge aacfbe4...

:broken_heart: Test failed - status

@bundlerbot r+

It turns out the given test for this change is an example of when the warning does not need to be printed, as the conflict has nothing to do with the fact that rspec-core is a pre-release.
Bundler could not find compatible versions for gem "rspec-core":
  In Gemfile:
    guard-rspec (~> 4.6) was resolved to 4.6.0, which depends on
      rspec (< 4.0, >= 2.99.0) was resolved to 2.99.0, which depends on
        rspec-core (~> 2.99.0)

    rspec-rails (~> 3.5.0.beta1) was resolved to 3.5.0.beta1, which depends on
      rspec-core (= 3.5.0.beta1)

NOTE: Pre-release dependencies need to be explicitly listed in the Gemfile.

The original issue, https://github.com/bundler/bundler/issues/4657, got properly fixed in the resolver, so this warning shouldn't be needed. Closing!


histogram result is not writeonly

result is written with atomicAdd(dest.result[bucketIdx].x, 1U);
The intel mesa driver rightly complains about this and does not build
the shader.

Someone reported this the last week and it was fixed just a few days ago :smile:. Can you try on latest master?

Master works fine.


ZoneID Error

After the 4.2.1 (as well as 4.2.2) cloudflare-ddns throws the following error:
FATA[0047] Couldn't get ZoneID   error="ListZonesContext command failed: HTTP request failed: Get \"https://api.cloudflare.com/client/v4/zones?name=mydomain.tld&per_page=50\": dial tcp: lookup api.cloudflare.com on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:52005-><IP_ADDRESS>:53: i/o timeout"

The yaml config file contains:

iface: eth0
token: #### (with zone access)
proxy: enabled
domains:  (list of subdomains)
host-id: hostid

It looks like an issue with tailscale DNS. We need more information here.

Try to curl this domain and see if the issue repeats
If yes, restart the tailscale service or go though their troubleshooting tips
If it doesn't, try an earlier version of the program

Tomorrow I'll also check how it's working on my machines (they run tailscale too).

Tested on my machines with 4.2.2. No issues noticed.

It looks like an issue with tailscale DNS. We need more information here.

Didn't notice that, I'll turn it off and see the outcome.

After turning off Tailscale cloudflareddns runs perfectly! Thanks for pointing out Tailscale

No problem. We too have issues with tailscale DNS from time to time.


Some possible errors in OmegaPRMv2 / omegaprm.py

System Info
same with repo
Who can help?
No response
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the codebase (such as scrips/, ...)
[ ] My own task or dataset (give details below)

Reproduction
Why there exists that in OmegaPRMv2 / omegaprm.py
Perform Monte Carlo estimation for s_new
self.monte_carlo_estimation(s_new)
Perform Monte Carlo estimation for s_new
self.monte_carlo_estimation(s_new)
It occurs two times, Is it supposed to occur only once?
Expected behavior
Why there exists that in OmegaPRMv2 / omegaprm.py
Perform Monte Carlo estimation for s_new
self.monte_carlo_estimation(s_new)
Perform Monte Carlo estimation for s_new
self.monte_carlo_estimation(s_new)
It occurs two times, Is it supposed to occur only once?

Hi,
The duplication has been removed.
Thanks for your contribution!


server properties change does not apply latest revision in the MSK cluster only creates a new revision

Describe the Bug
When we create a new configuration by adding some parameters to the server_properties, the new configuration is created as a new revision in AWS but the same is not applied to the MSK cluster already launched. This is similar to what is reported here.
Expected Behavior
When adding / removing configuration from server properties, the new configuration is created as a new revision but the same is not automatically applied on the MSK cluster.
Steps to Reproduce
Modify properties that are being passed here. Plan the changes and apply. The new revision of the configuration is created but the same is not applied on the MSK cluster.
Screenshots
No response
Environment
No response
Additional Context
No response

+1

+1


can we create pytuple with different types

i have been reading the doc of pyo3, and also the examples. I found that when you are trying to new a pytuple, you usually make a slice in which each element is of same type.
is there a way to new a pytuple of different types?

nvm, found it...
().into_py(py: Python<'p>)


Added MonthPicker functionality.

This is the same commit from #241; I’m not sure why this PR was opened.


Remove prompt driver/tokenizer model defaults

@vasinov I think there's still the discussion of:

Should the source of truth for models exist in Tokenizers?
Should the model variables still be prefixed with DEFAULT_?

@vasinov should we have the conversation of https://github.com/griptape-ai/griptape/issues/242 before merging this in?


OnError for FingerPrint authentication

Hi There,
Just curious, is there a way that i can override onError for FingerPrint authentication. I have to wait for TooMany attempts in order to override onError.

Hello @dhiwakarmani
Yes you can when you extend the class AppLockActivity.
Just override the method.
Cheers,
Olivier


Uncaught Error: error: 126/?~\AppData\Local\atom\app-1.25.1\resources\app.asar.unpacked\node_mo...

[Enter steps to reproduce:]

...
...

Atom: 1.25.1 x64
Electron: 1.7.11
OS: Unknown Windows version
Thrown From: Atom Core
Stack Trace
Uncaught Error: error: 126\?\C:\Users\PROF-LAB01\AppData\Local\atom\app-1.25.1\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
At ELECTRON_ASAR.js:173

Error: error: 126\\?\C:\Users\PROF-LAB01\AppData\Local\atom\app-1.25.1\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:173:20)
    at Object.Module._extensions..node (module.js:598:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:187:18)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (/app.asar/static/index.js:47:45)
    at require (internal/module.js:20:19)
    at customRequire (~/AppData/Local/atom/app-1.25.1/resources/app/static/<embedded>:96:26)
    at get_KeyboardLayoutManager (~/AppData/Local/atom/app-1.25.1/resources/app/node_modules/keyboard-layout/lib/keyboard-layout.js:7:65)
    at get_manager (~/AppData/Local/atom/app-1.25.1/resources/app/node_modules/keyboard-layout/lib/keyboard-layout.js:16:42)
    at Object.getCurrentKeyboardLayout (~/AppData/Local/atom/app-1.25.1/resources/app/node_modules/keyboard-layout/lib/keyboard-layout.js:32:17)
    at exports.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.25.1/resources/app/node_modules/atom-keymap/lib/helpers.js:201:42)
    at KeymapManager.module.exports.KeymapManager.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.25.1/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:467:20)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (~/AppData/Local/atom/app-1.25.1/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:347:30)
    at WindowEventHandler.handleDocumentKeyEvent (~/AppData/Local/atom/app-1.25.1/resources/app/src/window-event-handler.js:110:40)

Commands
  2x -1:50.3.0 core:undo (input.hidden-input)
     -1:14.5.0 core:save (input.hidden-input)
     -0:11.3.0 core:close (input.hidden-input)

Non-Core Packages

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/14461. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


⚠ UdeS website (prod) has degraded performance

In 4345122, UdeS website (prod) (https://www.usherbrooke.ca/) experienced degraded performance:

HTTP code: 200
Response time: 1241 ms

Resolved: UdeS website (prod) performance has improved in b6860c8 after 12 minutes.


user12986714: Watch lead(?:ing)?\W*+providers?\W*+of

user12986714 requests the watch of the watch_keyword lead(?:ing)?\W*+providers?\W*+of. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

lead(?:ing)?\W*+providers?\W*+of has been seen in 18 true positives, 3 false positives, and 0 NAAs.

Approved by Daniil in Charcoal HQ

Approved by Daniil in Charcoal HQ

Approved by Daniil in Charcoal HQ

Approved by Daniil in Charcoal HQ


Add 1-minute delays between crates

When a new version of a crate is published, it can take a while for crates.io to update the index. Publishing a dependent crate would fail in the meantime. This adds 1-minute delays between cargo publish commands to work around the problem.

Merging this before publishing new version.
bors r+


Enhancement: Set environment variable for collection code coverage

This PR

[x] uses an environment variable COLLECT_COVERAGE to determine whether coverage should be collected and sets it in the build matrix, rather than duplicating the actual command

This was merged manually

A quick message to let you know that League\Url v4 has been moved to another repo

Thank you, @nyamsprod!


log_config parameter to apiDockerCall improperly documented

The docstring at https://github.com/BD2KGenomics/toil/blob/8abd2826f45f1f79984e39ce2ab3fa8b4d112bd9/src/toil/lib/docker.py#L245 says log_config should be a string, but it is passed right on to the Docker API, which according to https://docker-py.readthedocs.io/en/stable/containers.html expects log_config to be a dict.
The documentation for Toil should say that log_config should be a dict.

@adamnovak That's a fair argument.  The original subprocessDockerCall() is mainly used by @glennhickey (who recently added stderr) and @jvivian as far as I'm aware.  The API was implemented due to better error messages, a cleaner interface, and having features that allowed for easier inputs for WDL support.
The docstring and unused stderr + dockerParameters removal are a quick fix.  dockerParameters is completely replaced by using the kwargs.  stderr is not implemented unfortunately.
Eventually I'd like to address the stdout and stderr kwargs in a similar way, which would likely be done with the attach(container, stdout=True, stderr=True, stream=True, logs=False) API call to the container documented here: http://docker-py.readthedocs.io/en/stable/api.html as I believe this use-case is the only current difference in terms of functionality.


Specify stricter dependencies in Pipfile

Is your feature request related to a problem? Please describe.
This isn't necessary a bug (hence why I chose feature request) but also not much of a feature, just a suggestion.  My team currently uses Poetry to manage our library builds, and we've noticed adding red-panda increases the tool's dependency's resolution time exponentially (literally hours, actually have not been able to make it work so far). Looking a bit deeper, I see that it's probably due to the fact your Pipfile dependencies are all wildcards.
Describe the solution you'd like
If possible, and not too big of a lift, could you provide some more meaningful restrictions on dependency versions? Anything but * would likely be helpful.
More generally, even though adding restrictions might seem an "extra", in my experience it's helpful down the line when you might encounter annoying conflicts (you know what you have, and what you can and/or should change).
Describe alternatives you've considered

Not including this in our build: Works but it's obviously not what we'd want!
Not using Poetry:  Have only tried running with classic setuptools but we lose a good deal of sane safeguards without a tool like this. Have not tried Pipenv yet (may consider it and working that possibility separately), but my above point still holds.

Thanks for your time, and for the work put into this, quite helpful!

@m1hawkgsm thanks for opening this. I realize it is also still taking quite some time (~ 2 minutes) to resolve dependencies when installed with pipenv. I'm trying to pinpoint the exact issue. Once this is fixed, I am going to release a 1.0.0b1 version with other improvements.

@m1hawkgsm if you have time, would you mind trying the dev branch in your project build? I tried with pipenv and it took under three minutes to install package and resolve/lock dependencies.

Apologies, my notifications didn't work for this.  I just tried and can confirm this is working beautifully well. Looks good!

Thank you. I will release this once I complete improvements in testing and docs.


Remove thekindkeptn

Removes thekindkeptn from docs since we have zero-install option now.
Signed-off-by: agardnerit<EMAIL_ADDRESS>

Merge please?

I clearly don't know git well enough. I've totally messed this branch up thanks to DCO.
Someone please help to fix. At this point I'm doing more damage than good.

@johannes-b can you review please? Last thekindkeptn build was 0.16.0 so quite old.

@StackScribe I've updated doc links to 0.18. Please merge. Once done I'll raise another PR to remove the first FAQ entry as it mentions keptn install which is no longer supported. I didn't want to do two changes inside this PR.

@StackScribe fixed all links and overrode the DCO as it provides zero value.


Add xmlstarlet to handle JENKINS_PASSWORD properly

/test images

ART needs to fix this: https://issues.redhat.com/browse/ART-3680

/retest

/retest

/test images
@sosiouxme, as this one is fixed  Now that  , this build should succeed...but is it only in brew or the relase has been done already in ubi8/rhel8 ?

/close

closed to force the execution of the make plugins-list job, now reopening
/reopen

/retest

/cherry-pick release-4.10

needs https://issues.redhat.com/browse/CLOUDBLD-8771 to be fixed for 4.11 .
For 4.10 this same build works

/retest

/retest

/assign @jkhelil

/lgtm


Planner should have way to view very long timelines

Example
When planning shorter duration efforts over longer time periods (think weeks in months), the bars get condensed to the point of not being legible at a glance:

Expected behavior
It would be useful to see a monthly or weekly view with a "map" of the greater whole to one side.
Mural has a feature like this, as an example of what I'm describing:

This issue is also related to #25 (zoom and pan support).

I’m on a train today with no wifi, so I can’t demo this yet- but I got horizontal pan and zoom working.

Fixed via #37


macOS: system menu icon does not fade on inactive desktops

The icon should become slightly faded-out on a system menu that is displayed in a desktop/monitor that doesn't currently have focus. Here's how the current icon looks next to other icons that implement this behavior:

This sounds like a Qt bug, but I'll have to investigate further. I only recently obtained the necessary adapter to connect a second monitor to my testing Mac.

Let me know if there's anything I can help with :)

This looks relavant, https://stackoverflow.com/a/41903980/1487176
Looks like they have a icon file that can change with a few settings?

I made the change to call QIcon::setIsMask(). If someone can grab the DMG from the CI server and try it out, that would be great.

Hmm... the icon needs to have a transparent center.

I think it will be good once you have it transparent - it changes with the theme as well - I have not tried the second screen, but I bet that is similar and will change with the other icons, I can confirm it is changing color with


Version issue

Hello,
It appears there is an issue with the current release. This file did not work for me https://github.com/hammerjs/hammer.js/blob/master/hammer.js but this one does http://hammerjs.github.io/dist/hammer.js
I am a bit confused because the dist file is dated 2016-04-23 and the one in the GH repo is 2016-09-30 but both have the same version.
I did not get any errors from the 2016-09-30 file, it just didn't work. I will try to provide more details but maybe this is something obvious that went wrong with the build.
Thanks

@oste the file was simply built later and represents incremental updates since the official release that will become the next release. Master should be in working order but if not and you can figure out why not please open an issue :) We recently reconfigured our build setup.
If you check, you'll notice this file with the tag for 2.0.8 is the file you were looking for: https://github.com/hammerjs/hammer.js/blob/v2.0.8/hammer.js

although, I did just notice that in 2.0.8 tagged we forgot to bump the version from 2.0.7 :/ apparently we got it right for the dist but not in that commit

@runspired I see, yeah not too sure what the issue was with the master build. I thought that would be the same as the dist but apparently its actually a build of master.
Thanks. I will see if I can find some time to track down the issue but kind of unfortunate that it seemed to just return and do nada.


Inconsistent path matching between append and parallel

I want to use Coverage to report across multiple Tox envs. Since each Tox env uses a separate virtualenv, I specify paths in Coverage's config:
[coverage:paths]
source =
    src
    .tox/*/lib/python*/site-packages
    .tox/*/site-packages

My package is under the src directory. The first tox path applies to CPython envs, the second to PyPy envs, which use a simpler path to site-packages. Specifying the actual package name at the end of each of the paths, or changing the order of the tox lines, or explicitly spelling out each tox path rather than using globs, doesn't affect the issue.
If I use the command coverage run -p -m pytest (parallel mode), then add an extra env at the end to do coverage combine, I get the correct report, with all tox paths merged into the correct source path. However, if I only want to run one env, like tox -e py36, then I end up with stray parallel files since the env that runs combine isn't specified.
Name                           Stmts   Miss Branch BrPart  Cover
----------------------------------------------------------------
src/itsdangerous/__init__.py     420     68     98     19    82%
tests/test_itsdangerous.py       233      3     27      0    99%
----------------------------------------------------------------
TOTAL                            653     71    125     19    88%

I thought I'd use append mode instead of parallel mode: coverage run -a -m pytest. This almost works, except the pypy path is always listed separately. The pypy3 path is merged just fine, as are all the CPython paths.
Name                                               Stmts   Miss Branch BrPart  Cover
------------------------------------------------------------------------------------
.tox/pypy/site-packages/itsdangerous/__init__.py     420     73     98     23    81%
src/itsdangerous/__init__.py                         420     68     98     19    82%
tests/test_itsdangerous.py                           233      3     27      0    99%
------------------------------------------------------------------------------------
TOTAL                                               1073    144    223     42    85%

I expected that paths would be appended to a single file in the same way they're combined from parallel files.

@davidism Thanks, can you provide a way for me to reproduce the failure, even if it's a commit hash in your repo, and commands to run?

This was from itsdangerous, but it's consistent across the projects I work on. Here's a minimal example project: example.tar.gz

I use Linux, not sure if Windows is different. Need CPython 3.7, CPython 2.7, PyPy 3, PyPy.
Extract, create a venv, pip install -e .
tox -c tox-p.ini will run coverate in parallel mode, combines and reports files correctly.
tox -c tox-a.ini will run in append mode, pypy env is reported as separate files.

@nedbat was there something else you needed before removing the question tag?

(Sorry, i just haven't come back to look at the answer yet!)

Sorry I've let this sit for so long...
When I run the example (thanks!), I see all the Python interpreters split out separately for tox-a.ini:
Name                                                        Stmts   Miss Branch BrPart  Cover
---------------------------------------------------------------------------------------------
.tox/py27/lib/python2.7/site-packages/example/__init__.py       0      0      0      0   100%
.tox/py27/lib/python2.7/site-packages/example/encoding.py      21      1      4      1    92%
.tox/py37/lib/python3.7/site-packages/example/__init__.py       0      0      0      0   100%
.tox/py37/lib/python3.7/site-packages/example/encoding.py      21      1      4      1    92%
.tox/pypy3/site-packages/example/__init__.py                    0      0      0      0   100%
.tox/pypy3/site-packages/example/encoding.py                   21      1      4      1    92%
src/example/__init__.py                                         0      0      0      0   100%
src/example/encoding.py                                        21      0      4      0   100%
tests/test_example.py                                          13      0      0      0   100%
---------------------------------------------------------------------------------------------
TOTAL                                                          97      3     16      3    95%

This is because only coverage combine re-writes paths.  If you move the .coverage file and run combine, it re-maps the paths to produce the right result.
Issue #1212 proposes to re-map paths without coverage combine, which would solve this problem.  I'll close this issue as a duplicate of that one.

This is now fixed in commit 0818611c.

This is now released as part of coverage 7.0.0b1.


Bug 🐛: Link to Components under Quick start in docs TOC is broken

In the Docs Table of Contents, the link to Components under Quick start is broken (https://github.com/helpscout/hsds-react/blob/master/docs/README.md)

I tried to figure out what the correct link should be to open a PR with a fix, but I couldn't 😊.
Note: I've added "Bug 🐛" to the title of the issue until a core team member adds a label 🙈

Thank you, the quick start link has been removed from the doc in d8dbcd7230be202909becd999dbccf6569989eef


Bump webhook and csp-adapter

Issue:

Problem

Solution

Testing

Engineering Testing
Manual Testing

Automated Testing

QA Testing Considerations

Regressions Considerations

Looks like ci failed due to a missing . in the webhook charts.  Fixed by https://github.com/rancher/webhook/pull/161 and new RC has been cut.

Thank you!!


[BUG] "New user failure" "Failed adding user to mautic" E8: Failed adding user to mautic

Description: ""New user failure" "Failed adding user to mautic" E8: Failed adding user to mautic" error is displayed  in the console. The user is not able to sign in.
Steps to reproduce:

Open the https://goodqa.netlify.app/
Sign in with the Google/Facebook
Pay attention to the error message in the console

Actual result:  ""New user failure" "Failed adding user to mautic" E8: Failed adding user to mautic" error is displayed  in the console.
Expected result:  The user should be signed up without any errors
Environment: PROD 1.23.3 //  QA 1.23.1-0
Devices list: Desktop // Windows 10 // Chrome 88.0.4324.150
Attachment:
https://www.dropbox.com/s/zd5c5kpb8v34ne9/The user is not unable to registration on the Prod QA.mp4?dl=0

@sirpy The issue is not reproduced. The user is signed up with Google/Facebook/Passwordless without errors.
Checked on the PROD 1.23.3 // QA 1.23.1-0
Devices list: Desktop // Windows 10 // Chrome 88.0.4324.150
Attachment: https://www.dropbox.com/s/ggkno39zoc1ce76/Sign up works without issues.mp4?dl=0


Mac OS X Smoketest preset

@swift-ci Please smoke test

@swift-ci Please test Linux Platform

@swift-ci Please smoke test

Lets try this one more time.
@swift-ci Please smoke test

@swift-ci Please test Mac OS X Platform

@swift-ci Please test Mac OS Platform

@swift-ci Please test OS X platform

I am testing OS X platform to make sure that the normal build works fine. The normal build is not using these presets.

I just checked on the full OS X platform test. It is building the iOS/tvOS simulator targets (you can tell from the amount of things to build). So this shouldn't affect other people's builds. I am going to merge this once I get the time/correctness for the smoke test.

@swift-ci Please smoke test OS X

@swift-ci Please smoke test OS X


mainModulePath undefined for autocomplete-plus

Any body else noticed this? Our package fails to activate because of this.
@park9140 might know something about this

removing autocomplete-plus and letting apd reinstall it fixes it. Investigating

@basarat, can you document what changes you made to apd that required including it in the project?

Just for future reference it would be better to fork apd then have us npm install a branch directly from github rather than merge it into our repo.

@park9140 will document it. Sorry about including it. I see the advantage of forking (better commits and diffs).
Found the reason. I ran "update all" last night but before it completed I shut down atom (was too tired and assumed something in background would compete it). This results in an empty directory with just folders:

Perhaps I should add a guard against this case but its no longer high priority.

@park9140 I forked apd primarily because I needed a callback on install finish ... however its not a good callback and doesn't handle error cases gracefully. I'd be tempted to rewrite apd using promises.


TheGrandTree.java free inventory slots

I'll create a PR with changes after some testing.
I don't think 4 free inventory slots are needed, only 3, when talking to the king at the end as he takes the plans.
It would have also been helpful to have a starting quest notice that you need free inventory slots.
Thanks for the great tool :)


Testing pivot_table in v1.1.1-dev302 (preparation for v1.1.1) show test is not deterministic

I triggered a round of tests on DuckDB's community extensions, and there is a failure on MacOS run (but not on the linux one ??) for pivot_table.
See here: https://github.com/duckdb/community-extensions/actions/runs/10973205947/job/30470351815#step:17:102.
Unsure if this needs to be solved at the implementation level or adding a rowsort attribute to the test results.

Oh, I think I fixed this on main. I'll update the commit sha!

Actually I am not sure, it might also be a problem in main, even though I can see where this would come from.

It was a logical miss on my part! I needed to add one more column to the order by in some specific cases. That has been fixed, so CI is running again now with a new SHA!

I think this works as expected, thanks!