Something wrong with total bytes column width in symbols

Version and Platform (required):

Binary Ninja Version: 3.6.4756-dev, a10935ab
OS: macos
OS Version: 14.2
CPU Architecture: arm64

Bug Description:
Why does total bytes does not shown in the screenshot below even if there is enough space?

Does it show up if you scroll to the right? Still super annoying, but I suspect it's just wider than the window and the numbers are right-aligned.

Fixed in 3.6.4762-dev.


[Doc] Add document for TRANSLATE

Relates to #22704.
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

thanks for your contribution, we updated this function in pr https://github.com/StarRocks/starrocks/pull/36716/files based on your work, and will close your pr


Grpc request failing with "onStreamError" on enabling ssl and http2

<html>
<body>
<!--StartFragment-->
May 29, 2024 7:30:55 AM io.grpc.netty.NettyServerHandler onStreamError
--
  | WARNING: Stream Error
  | io.netty.handler.codec.http2.Http2Exception$StreamException: Stream closed before write could take place
  | at io.netty.handler.codec.http2.Http2Exception.streamError(Http2Exception.java:173)
  | at io.netty.handler.codec.http2.DefaultHttp2RemoteFlowController$FlowState.cancel(DefaultHttp2RemoteFlowController.java:481)
  | at io.netty.handler.codec.http2.DefaultHttp2RemoteFlowController$1.onStreamClosed(DefaultHttp2RemoteFlowController.java:105)
  | at io.netty.handler.codec.http2.DefaultHttp2Connection.notifyClosed(DefaultHttp2Connection.java:357)

<!--EndFragment-->
</body>
</html>

Closing as figured out the issue was with the gateway from where the request was coming.


Add support for ssl driver flags

Describe the feature
dbt-teradata doesn't support ssl-related config params. This request is to add support for ssl* flags.
Describe alternatives you've considered
Don't use SSL
Additional context
https://github.com/Teradata/python-driver
Who will this benefit?
Users who want to use dbt with Teradata over ssl
Are you interested in contributing this feature?
Yes :)

Let's expand this issue and make sure that any flag documented in the driver https://pypi.org/project/teradatasql/ is also supported by dbt. As part of this story, we should make sure that all the flags that are supported by the dbt adapter are documented on https://docs.getdbt.com/docs/core/connect-data-platform/teradata-setup


Support for timeout property

Codecov Report

Merging #246 (9079ea2) into master (61d8e90) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #246   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           19        19
  Lines          215       216    +1
  Branches        29        30    +1
=========================================
+ Hits           215       216    +1

Impacted Files
Coverage Δ

src/classes/StarCraft2API.ts
100.00% <100.00%> (ø)

src/types.ts
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


fd: 7.2.0 -> 7.3.0

changelog
Things done

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

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

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

@GrahamcOfBorg build fd

The error makes me think we were just being haunted by ghosts earlier. But the ghosts may be gone now, so let's see!
installing
    Updating registry `https://github.com/rust-lang/crates.io-index`
error: failed to sync
Caused by:
  failed to load pkg lockfile
Caused by:
  failed to fetch `https://github.com/rust-lang/crates.io-index`
Caused by:
  [16/-1] unexpected return value from ssl handshake -9808

looks like rustc build succeeded on Hydra in the meantime, so let's try again
@GrahamcOfBorg build fd

Now the timeouts are gone but it's the same Darwin error again:
Caused by:
  failed to fetch `https://github.com/rust-lang/crates.io-index`
Caused by:
  [16/-1] unexpected return value from ssl handshake -9808

Since this doesn't look like an issue specific to fd I suggest we merge it anyway. Darwin can be fixed later.

FWIW I can build it on darwin https://gist.github.com/10a2319157de357940732fe1c30972f5

Thanks again for the darwin testing @marsam :smile:


Create a no-server mode which swaps ipykernel out for Brython

https://github.com/SimonBiggs/scriptedforms/issues/126
Main use will be to allow the writing of the scriptedforms docs as a static site and still be able to use scriptedforms to write the documentation.
Will not be a documented feature in and of itself.

Initially create a JavaScript only build which doesn't have any of jupyterlab services included within it.
See if I can make a slider that simply steps through saves of polynomial order plots which will look like it is running matplotlib, but it is actually just displayed saved pictures.
Use a separate webpack entry that declares "no-server". So that I can create no server for the docs.
Set the cache breaking hashes up within that webpack version so that the browser can cache the large JavaScript files.

Make a variable holder class within JavaScript. To update variables within JavaScript the following needs to be done:
variableHolder['python_variable_reference'].next(newValue)
It will be onerous but this will mean data won't collide with the JavaScript global space and it also means that the changes can be subscribed to by the variable service.

From the get go the JavaScript fences will not be designed for general use. It will be for advanced use only. It likely won't get documented for quite some time.

Won't do this. It doesn't actually solve the documentation problem as webpack will unlikely be able to handle it without a plugin. If I do it via API, the way the API is set up I will need a server.

Still worth making the variable handler which can be called. Then custom code can call that and update the variables. Find a way to make a template within JavaScript, and declare certain code areas to be replaced by JavaScript functions.

Actually... For the sake of making the documentation easy to update it might be worth looking into webpack file loader plugins. I'll have a squiz and see just how hard it is.
I could potentially make it so that I write both Python and JavaScript within the one file next to each other. On the left hand side of the docs it will display the markdown file with all of the JavaScript fences removed. On the right hand side it will display the form, except it will run the JavaScript instead of the python. That way I can write the docs as a ScriptedForm and still not have to have a server.
It seriously depends on how hard it is to make a webpack file loader plugin. Hopefully the benefit of being able to quickly update the docs surpasses the effort required to make a webpack file loader...

Actually........ I think "no-server" mode needs to swap in Brython. This might work... I would still need a webpack file loader. We'll see...

If I'm super keen, there could be a way to indicate to webpack that a particular section needs to have the slider(s) iterateted through and have all of the images saved and converted to a "if this slider position show this image which was seen when building"... could have specific webpack pointers within the files. Those Pinter are removed during the compile so that don't show on the docs. Some webpack pointers would replace the python code with Brython, others would run the python code on a jupyter server process iterating through the slider positions etc.
We'll see how possible this is...

No need for this. Will use mybinder instead:
https://github.com/SimonBiggs/scriptedforms/issues/149


Has one fix

This is how I see has_one is supposed to work. Main difference from the current master branch, is that User.first.profile returns the Profile instance and not an ArrayFinderQuery instance.

This looks good just reviewing the code. There may be some edge cases. The cases I want to understand are:

Fresh object and no has_one relative exists.
a. @obj.haz_one.create(name: 'joe')
b. @obj.haz_one.name = 'joe'  # Is this even reasonable to expect

Object in use and has_one relative exists.
a. @obj.haz_one.create(name: 'joe') # creates new object but doesn't delete old one from its collection, right
b. @obj.haz_one = HazOne.create(name: 'joe')

This may not be all the cases, but you see where I'm going with this? If you could put this in another branch, I can pull it and test it.
Thanks!

I think I have them all now. Can you check?

Looks good. There might be another PR coming in the near future.
--
All the best
Holger Sindbaek
· UI/UX
Designer & Web/iOS Programmer
·

http://holgersindbaek.com/
On Wed, 16 Mar 2016 at 13:59 "s.ross"
<
mailto:

wrote:

I think I have them all now. Can you check?
—
You are receiving this because you authored the thread.
Reply to this email directly or
https://github.com/sxross/MotionModel/pull/142#issuecomment-197459335
https://github.com/sxross/MotionModel/pull/142#issuecomment-197459335


Improve input validation for FASTQ

Description of feature
Validation of input FASTQ files should be improved:

certain unsupported samplesheet FASTQ input rows produce non-descript errors

see here for an example: https://nfcore.slack.com/archives/C04CQ1LK7U1/p1724454431405079

all file inputs including FASTQ should be init as Nextflow.File objects and checked to exist (w/ stub mode exception):

https://github.com/nf-core/oncoanalyser/blob/4227bc1083209d79c8735d32e6c75f762a640006/lib/Utils.groovy#L127-L133

This work may also relate to:

https://github.com/nf-core/oncoanalyser/issues/51

Additionally where a samplesheet has FASTQ input files provided as separate entries rather than the expected single entry, an informative error should be provided. See: https://nfcore.slack.com/archives/C04CQ1LK7U1/p1734010565913889


Added RelatedOnlyDropdownFilter option

Hello!
Thanks for that simple but useful addon.
The only thing that I've been missing is that there was no select for RelatedOnlyFieldListFilter.
I've added this option along with some code prettifying. Hope it helps! ;-)

Thank you :+1: :)! Looks good to me, merging now.

Thanks in return!
It would be nice to propose it to Django team as well, 'cause that's the really useful option but currently unavailable in Django 1.11.


Async version?

Any chance of adding async support or creating an async version of this package?

Are there particular calls that you believe are useful in async version?
For example, adb devices in async form seems pretty useless to me.
I can however see a case for certain other commands to be async

Are there particular calls that you believe are useful in async version?
For example, adb devices in async form seems pretty useless to me.
I can however see a case for certain other commands to be async

My use cases is the Android TV integration in Home Assistant.
Currently, it uses the androidtv package, which uses pure-python-adb as one of its two means of sending ADB commands; the other is adb-shell. This is all sync, and Home Assistant runs all these commands in its thread pool.
I've ported the androidtv and adb-shell packages to async (aio-androidtv and aio-adb-shell). An async version of pure-python-adb would enable the Home Assistant integration to be fully async.
And given that this package is all about socket I/O, converting it to async would yield an improvement in efficiency.


[SPARK-14903][ML][PYTHON] Revert SPARK-14071: MLWritable.write property

What changes were proposed in this pull request?
SPARK-14071 changed MLWritable.write to be a property.  This reverts that change since there was not a good way to make MLReadable.read appear to be a property.
How was this patch tested?
existing unit tests

Test build #56933 has started for PR 12671 at commit de01a53.

Test build #56933 has finished for PR 12671 at commit de01a53.

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

Merged build finished. Test PASSed.

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

I'll go ahead and merge this with master


Fallback on .Date if .PublishDate not set

Fallback on .Date like in opengraph template

 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.

 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.

Hey, thanks for this.
These templates has a history and we may have forgotten to update them as Hugo has evolved, but in my head publishDate will (in its default config) always be set if there is a date:
https://gohugo.io/getting-started/configuration/#configure-dates

Hey, thanks for this.
These templates has a history and we may have forgotten to update them as Hugo has evolved, but in my head publishDate will (in its default config) always be set if there is a date:
https://gohugo.io/getting-started/configuration/#configure-dates

Maybe I need to take a look at all of the internal templates and update them then. The date attribute is what is checked when adding updated time in the open graph template. I am a bit confused why there is Published Date, Date, and Lastmod. Is Published Date the "correct" parameter here?
{{- if not .Date.IsZero }}<meta property="og:updated_time" {{ .Lastmod.Format $iso8601 | printf "content=%q" | safeHTMLAttr }} />

Maybe I need to take a look at all of the internal templates and update them then. The date attribute is what is checked when adding updated time in the open graph template. I am a bit confused why there is Published Date, Date, and Lastmod. Is Published Date the "correct" parameter here?
{{- if not .Date.IsZero }}<meta property="og:updated_time" {{ .Lastmod.Format $iso8601 | printf "content=%q" | safeHTMLAttr }} />

Yes, those templates was built before we improved the date logic in Hugo.
{{- if not .Date.IsZero }}<meta property="og:updated_time" {{ .Lastmod.Format $iso8601 | printf "content=%q" | safeHTMLAttr }} />

So, if you found the above in a template, that obviously isn't right.
I think we can say that, in general:

If you want the published date, use PublishDate -- no need to check if it's set.
If you want the last modified date, use Lastmod -- no need to check if it's set.
If you want a date, use Date -- no need to check if it's set.


features settings are incorrectly overridden

Problem
When a crate that is directly dependent on the codebase is also depended on by another dependency, the features set for this crate in the codebase will be incorrectly overwritten by the features in that indirect dependency.
The premise of this phenomenon is that the version of the crate used is the same, and the source of the code is the same. But if the version is different, or the source of the code is different (for example, one comes from 'crates.io', and the other refers to a local path), this kind of error will not appear.
For example:

crate A depend on crate X, features settings for X is ["json_codec"]
crate A depend on crate B
crate B depend on crate X, features settings for X is ["bincode_codec"]

The end result will be: the features settings for crate X in crate A will be overridden by the settings in crate B, that is ["bincode_codec"], this behavior should be an unexpected bug.
Steps
No response
Possible Solution(s)
Multiple references to the same version of the same crate but with different features should be compiled and referenced separately as different libraries.
Notes
No response
Version
rust 1.65 stable

In Cargo, features get unified during the compilation. In your particular case, assumed all the presences of crate X in your dependency tree are in SemVer-compatible versions. You should get a unified feature set ["json_codec", "bincode_codec"] annd X compiles only once with that feature set.
You can also run cargo build --verbose and search for something like --cfg 'feature="bincode_codec"' and --cfg 'feature="json_codec"' to verify Cargo passing the correct features to rustc.
If this is not the case you encountered. Please provide a minimal reproducible example to help maintainers find the quirky behaviour.


response_transformer: option to not overwrite header

When I try to add a header that already exists, the original is overwritten. This is for a type of header that can appear multiple times.
Perhaps you could add another option, value.concat.headers, where any headers specified would be added without removing any that already exist. That or the original header is rewritten as a comma-separated list of all the values (both forms are synonymous according to the HTTP spec).

@jkarneges we released kong 0.6.0 with the fix. now 'add' only add if header is not already set. we also 'append' to append values to already set header.


Enhancement: Check Search-Strings w/ and w/o quotes

I think this problem is already known, but I can't find any open or closed issue related to this case.
Lately I was wondering why Dexie gave me wrong results. My working example:
idb.opendb().then(function(db){
   db.transaction('rw', ['steam_users', 'users_games'], function(){
     db.users_games.where('app_id').anyOf(summary.appid).each(f => {
       // Only add a user one time to this array if he owns 1 of the game/s
       // This prevents us from sending subs including one of the games
       if(u.indexOf(f.steam_id) === -1) u.push(f.steam_id);
     }).then(function(){
       // Now only add users which does not own the game/s to our summary
       db.steam_users.where('steam_id').noneOf(u).each(j => {
         summary.user.push({steam_id: j.steam_id, username: j.username});
       });
     });
   }).then(function(){
     // We have all the data we need - start processing
     sendResponse('Processed all Data.');

     if(summary.user.length <= 0){
       chrome.tabs.sendMessage(sender.tab.id,{
         msg: 'UpdateProgress',
         percentage: 100,
         message: '....'
       });
     } else {
       processThings(summary, sender.tab.id);
     }
   }).catch(function(err){
     console.log(err);
   }).finally(function(){
     db.close();
   });
 });
});

The problem here is, that summary.appids (array) could have numbers w/ and w/o quotes. So I'm using this function now, to fill the array with quotes and unquoted numbers:
var appids = summary.appids.map(Number);
appids.push(...summary.appids);

Edit: Ok, this only works if the numbers are quoted.
I know one should check for itself if this should be a number and only save it w/ or w/o quotes all the time. But I think it would make Dexie more "fail-resistant" if we check the search-strings w/ and w/o quotes to avoid such mistakes. And yes it would make Dexie slower, especially with a lot of DB-Entrys. :/

I understand the problem. Still, I would like to do those kind of magic fixes for the user. Not just for performance reasons but also because things could tend to work even though the code is wrong. Type aware languages like typescript could be to the rescue here though (because you'd define an interface or class for each table that the compiler would check when calling anyOf() and other methods).

Closing.


Update the doc landing page per Issue #4524

@SMoraisAnsys I've updated the doc landing page, removing the "EDB API reference" card but adding a note before all the cards telling readers that they might also want to view the PyEDB documentation. I've moved the future cheat sheet updates to a different (new) issue: Issue #4574.

Thanks @PipKat !


Big Endian issue

Hello, I have endian issue with save files devilutionX a reversed engineered diablo game.
It uses StormLib to encrypt / decrypt save file.
Here is my Amiga Big Endian branch.
https://github.com/AmigaPorts/devilutionX/tree/fix-saving
I have made save file from Little Endian version to load but can't save -> encrypt file so far.
With this commit
https://github.com/AmigaPorts/devilutionX/commit/bc1d0eb23b64a79728adea97f59c0a0757bc83e8
Can you help at this matter?

I am afraid that there is no support for Amiga in StormLib. If you open StormPort.h, there is (was) support for big endian for Mac, before they switched to Intel processors. Since then no one tested big endian support for StormLib.

I could test on big endian. I still have a PowerPC Linux machine lying around.

@grepwood: If you mean that, please, write me an e-mail. I'll send you link to the testing files that are needed for StormLibTest to be executed.

The problem I had was actually not related to StormLib.
So, StormLib with Big Endian patches used in DevilutionX works good Amiga on BE platform.

Ok, closing the issue again.


Create xSPSearchTopology resource

Create a resource that is able to take in the names of servers that should be running each search topology component for a specific search service application. An example of this could be as below:
xSPSeachTopology FarmTopology
{
    SearchServiceApp = "Search Service Application"
    AdminComponents = @("SharePoint1", "SharePoint2")
    CrawlComponents = @("SharePoint1", "SharePoint2")
    AnalyticsComponents = @("SharePoint1", "SharePoint2")
    ContentProcessingComponents = @("SharePoint1", "SharePoint2")
    QueryComponents = @("SharePoint3", "SharePoint4")
    FirstIndexPartitionComponents = @("SharePoint3", "SharePoint4")
    FirstIndexPartitionDirectory = "I:\SearchPartitions\0"
}

This should detect changes to servers and add/remove components from the topology as required.

My work in progress for this issue can be found in my feature-searchtopology branch in my fork of the repo.

This is included in PR #118 and is in testing. Will close this issue when it is merged to dev.

This has been merged in to dev, closing this off


Build breaks when path contains a space

Functional impact
Does the bug result in any actual functional issue, if so, what?
Can't build the project.
Minimal repro steps
What is the smallest, simplest set of steps to reproduce the issue. If needed, provide a project that demonstrates the issue.

Clone project to a path containing one or more spaces (eg. C:\test folder\Blazor)
Attempt to build project (either dotnet build or using Visual Studio)

Expected result
What would you expect to happen if there wasn't a bug
A built project.
Actual result
What is actually happening
The following error messages:
Microsoft.AspNetCore.Blazor.Test -> C:\Users\elcer\Google Drive\devel\Blazor\test\Microsoft.AspNetCore.Blazor.Test\bin\Debug\netcoreapp2.0\Microsoft.AspNetCore.Blazor.Test.dll
  No executable found matching command "dotnet-C:\Users\elcer\Google"
C:\Users\elcer\Google Drive\devel\Blazor\src\Microsoft.AspNetCore.Blazor.Browser.JS\Microsoft.AspNetCore.Blazor.Browser.JS.csproj(25,5): error MSB3073: The command "dotnet C:\Users\elcer\Google Drive\devel\Blazor\src\Microsoft.AspNetCore.Blazor.BuildTools\tools\Microsoft.AspNetCore.Blazor.BuildTools.dll checknodejs -v 8.3.0" exited with code 1.
  Microsoft.AspNetCore.Blazor.Browser -> C:\Users\elcer\Google Drive\devel\Blazor\src\Microsoft.AspNetCore.Blazor.Browser\bin\Debug\netstandard2.0\Microsoft.AspNetCore.Blazor.Browser.dll
  Microsoft.AspNetCore.Blazor.DevHost -> C:\Users\elcer\Google Drive\devel\Blazor\src\Microsoft.AspNetCore.Blazor.DevHost\bin\Debug\netcoreapp2.0\Microsoft.AspNetCore.Blazor.DevHost.dll
  Microsoft.AspNetCore.Blazor.Mono.Test -> C:\Users\elcer\Google Drive\devel\Blazor\test\Microsoft.AspNetCore.Blazor.Mono.Test\bin\Debug\netcoreapp2.0\Microsoft.AspNetCore.Blazor.Mono.Test.dll

Build FAILED.

C:\Users\elcer\Google Drive\devel\Blazor\src\Microsoft.AspNetCore.Blazor.Browser.JS\Microsoft.AspNetCore.Blazor.Browser.JS.csproj(25,5): error MSB3073: The command "dotnet C:\Users\elcer\Google Drive\devel\Blazor\src\Microsoft.AspNetCore.Blazor.BuildTools\tools\Microsoft.AspNetCore.Blazor.BuildTools.dll checknodejs -v 8.3.0" exited with code 1.
    0 Warning(s)
    1 Error(s)

I have the same issue when trying to build in a folder with a space in it too

Confirmed fixed by merge.


If a merchant has no logo, show their emoji if it exists

For merchants with no logo available, if they have an emoji this will be shown in place of the logo before falling back to shopping-bag.svg.

Great thanks for updating those.


Unable To Delete Inventory Items

Expected behavior: When Item is deleted it should not reappear
Actual behavior:
Item is deleted it does not disappear
Steps to reproduce:
Screenshots (if applicable):
OS and Browser:

@docbby I tried to repro this and I couldn't. Could you please double check that and maybe provide more context (steps, OS, browser etc.)?

I put the label on it until we clarify the details

Closing for lack of activity.


:broom: Check licenses only, but not bundle them

Changes

:broom: Check licenses only, but not bundle them

/kind cleanup
Fixes knative/community#1441
Release Note
License files are no longer bundled with Knative sources, but only checked for validity

/cc @dprotaso

/lgtm
/approve


feat: Fancy new setup UI

Using Svelte and Bootstrap to get something a bit more user-friendly and easy on the eyes.
Also added Content-Security-Policy for extra security.


sys/linux: kernel reboot through tty

It's possible to cause a kernel reboot (detected as lost connection to test machine) through k_spec()->fn_boot_it()->ctrl_alt_del() in drivers/tty/vt/keyboard.c. I've triggered this via a USB connected tty, but based on the coverage I see on syzbot, it should be possible to do the same through evdev.

Is there any simple way to prevent this? Turn off some config?

I don't see one. We could disable CONFIG_VT, but that will affect fuzzing of virtual terminals.

This may be another candidate for the "fuzzing lockdown" feature that Tetsuo proposed on kernel mailing lists. I think we need to start collecting all these cases in a single issue, they are spread across random mailing lists.

Let's re-purpose this as a broader issues since we don't have a simple solution anyway.

Amusingly this was already discovered before and forgotten. You may see this patch from Tetsuo mentions k_spec:
https://lkml.org/lkml/2019/12/16/212

@xairy
Initial patches for this purpose arrived at linux-next-20200428 . Please add CONFIG_TWIST_KERNEL_BEHAVIOR=y and CONFIG_TWIST_FOR_SYZKALLER_TESTING=y to your kernel config, and let's check whether the frequency of "unexpected kernel reboot (3)" and "lost connection to test machine (5)" on linux-next is reduced.


Slow response and missing key strokes on REPL

The interactive repl, seen when running./run, is very slow and sometimes does not register key presses.  Using Haskeline <IP_ADDRESS>

this issue is only showing up when using ubuntu w/ terminator CL application. Problem not existing when using Mac OSX/iTerm

run the Repl directly, not through ./build


Fix valid_cnpj? spec

Hi, I believe the spec of valid_cnpj? was a duplication of the valid_cpf?. Just a small fix.

Thanks @ThiagosLima


Test Kafka JMX check against latest kafka versions

1.0.1 is the latest bugfix release.
1.1.0 is the latest stable release.
NOTE: This fixes https://github.com/DataDog/integrations-core/pull/1415 which unfortunately failed the tests because the latest docker images on https://hub.docker.com/r/wurstmeister/kafka/tags/ do not include the -1 suffix like they used to. I realized last night and pushed an updated commit, but looks like I forgot to amend the commit! So fixing here.

Not sure why the tests failed, it's not the systematic failure seen in #1415, I'm guessing just a flaky test, but can someone kick travis?


Large monitor breakpoint

In src/definitions/collections/grid.less, line 645, it states:
@media only screen and (min-width: @computerBreakpoint) {
I think the max-width is missing isn't it? Should be:
@media only screen and (min-width: @computerBreakpoint) { and (max-width: @largestComputerScreen){

Or rather @largestSmallMonitor instead of @largestComputerScreen

Hi @rtarnaud, sorry for the delay, it intentionally targets PCs in general, while @largeMonitorBreakpoint and @widescreenMonitorBreakpoint target specific computer screen sizes, and will override @computerBreakpoint, so setting a max-width for it is unnecessary.


Query: Fix #6737 - Entities not being released

Fixed two places where compiled queries were inadvertently holding onto scoped QueryCompiler state:

The query expression used in the compiled query cache key via EntityQueryable<> constant expressions.
The compiled query delegate, via a capturing closure in QueryCompiler itself.

@ajcvickers


Fix db_migrator.py script for customer Forseti DB name

Updated the db_migrator.py script by using the FORSETI_DB_NAME environment variable for the Forseti database name. This is set via Terraform and the startup script. Will have some additional PRs for the GKE changes.
Addresses #2938 for GCE installations.

This has been release in Forseti v2.21 release.


HIVE-27012:Remove JavaEWAH dependency from HIVE to fix CVEs.

What changes were proposed in this pull request?
Upgrade JavaEWAH to 1.1.7
Why are the changes needed?
Fix CVEs
Does this PR introduce any user-facing change?
No
How was this patch tested?
Hive was build locally , then javaewah was removed and was successfully built again.

@nrg4878 could you please take a look at this ?

I think we do not use this in HIVE anymore, so should be safe to remove.


CEL import results in different UI configuration than when exported

React Query Builder version
7.3.0
Platform
Windows 11, Node 20.12.2
Description
When using the Not In operator in the query builder

and exporting to CEL format you get a string such as:
!(firstName in ["vai"])
If you re-import this string back into the query builder the query builder layout will show an extra grouping that wasn't there originally along with a negation of the group and the use of the in operator.

Additionally the next export will result in a string with an extra set of brackets around it such as:
!((firstName in ["vai"]))
Reproduction

Go to the https://react-querybuilder.js.org/demo/ page
Create a new expression in the query builder as below

Copy the CEL Return statement
Click Import CEL
Paste the statement

Expected behavior
The resultant layout of the query builder should be the same as when the query was first entered, and the exported CEL statement should be the same as the originally exported statement.

Instead it has an extra negated grouping which is also reflected in the exported CEL statement.

Additional information
No response

This is definitely a bug. Probably related to changes made for #682 in #689. I'll try to get to it soon.

This has been fixed in #700. Test here: https://deploy-preview-700--react-querybuilder.netlify.app/demo.
If you're happy with the fix, I'll release a patch soon.

Hi @jakeboone02 it looks great thank you! When are you planning to release a version containing this fix?

@schultekp I was going to wait until I got the library all the way ready for React Compiler with #705, but I can go ahead and release what's in the queue tomorrow.

@jakeboone02 Thank sounds perfect! Thank you!

@schultekp see v7.4.0


Related References For Alternative Table

Currently we are getting the list of related references from the visible-foreignkey annotation defined on the table regardless of table being alternative. We should change this behavior, to use the main table as the source of this annotation or heuristic.
Let's assume A is an alternative table for M. Calling A.related should return the same list of references as M.related does.

I don't think this makes sense. What .related API returns only depend on the context of the reference. So if we're using the correct contexts, the API behaves as exactly as the example says. I am going to close this issue.


Stand-in AMR-wind uses stop time defined in amr_input.inp

@brookeslawski raised the issue that the stop time is hardcoded in launch_dummy_amr_wind() (in dummy_amr_wind.py). As we are reading in other information from the amr_input file, this change also reads in the stop time and uses that.
Additionally, the while loop in DummyAMRWind.run() now uses sim_time_s as its counter to meet the stopping condition, rather than absolute_helics_time, because absolute_helics_time is counting twice as fast as sim_time_s and therefore ends the simulation at (approximately) half the intended runtime. This is likely related to #42 .
Ready for review and merge.

This PR will likely conflict with develop once #48 is merged---I'll merge develop back once #48 is in.


New version: Xpress v0.10.0

Registering package: Xpress
Repository: https://github.com/JuliaOpt/Xpress.jl
Created by: @joaquimg
Version: v0.10.0
Commit: 78b87775a8fb76cd14cf6c119c688ec8c3b01e09
Reviewed by: @joaquimg
Reference: https://github.com/JuliaOpt/Xpress.jl/commit/78b87775a8fb76cd14cf6c119c688ec8c3b01e09#commitcomment-37295112
Release notes:

Major redevelopment of the interface.
Low level functions are automatically generated using Clang.
MOI wrapper was completely rewritten to be specific for Xpress.
Travis integration added.

I was not able to load the package (i.e. import Xpress failed). See the CI logs for details.

Xpress.jl build process requires having Xpress binaries which depend on commercial agreements. (just like Gurobi.jl and CPLEX.jl). We managed to get that working on xpress's travis, but we cannot do that here now.

[noblock] This package requires a commercial third-party dependency which is the cause of the installation failure.


Fix misspellings

Fix misspellings in synopsis and description of Get-Interface

I had accidentally created this on the wrong fork.
See: https://github.com/NuGet/NuGet.Client/pull/2141


Slowness when loading table from S3

Apache Iceberg version
0.3.0
Please describe the bug 🐞
Hi,
I am trying to read a table schema.  for our use case - we need only field names and types
The operation of load_table is very slow, it takes about 30 seconds.
I checked the metadata file on S3 and it is ~9MB, and the table has ~900 columns.
The slowness happens when opening the metadata file from S3.
is this a known issue?
Is there a faster way to get the table schema?
Thanks a lot!
This is the code we use (we have hive metastore + S3 storage):
from pyiceberg.catalog import load_catalog
catalog = load_catalog("prod")
table = catalog.load_table("stores.checkout")
table

Hi,
The PyIceberg version that we use is 0.5.1..
Sorry I wrote the wrong one when opening the ticket.
BTW Our iceberg version is 1.3.0.
Any other way to make the reading of metadata faster? thanks!

@itaise currently facing the same issue, but on a larger scale (bigger table, bigger metadata).
Did you find a way to speed this up ?
Thanks.

Hi,
Actually i didnt. Will love to hear if there is a way. We would like to get
only table metadata in the fastest way possible (for user facing UIs)
בתאריך יום ו׳, 26 בינו׳ 2024, 12:51, מאת anechii @.***

:

@itaise https://github.com/itaise currently facing the same issue, but
on a larger scale (bigger table, bigger metadata).
Did you find a way to speed this up ?
Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/iceberg-python/issues/220#issuecomment-1911858579,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APN7A4EWSNIXMQAJOFAVWW3YQODDHAVCNFSM6AAAAABAZGPPM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJRHA2TQNJXHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Secrets as VolumeMounts instead of EnvVars

Sorry about the extra long branch name, I have been using the "create branch from issue" feature, and unfortunately this issue had a long title. We will try to keep issue titles shorter in the future.
This PR switches all EnvVars to VolumeMounts as a security precaution.
Secrets in Kubernetes as EnvVars is a CIS Benchmark violation and would not pass an audit. Other reasons include protection from logging leaks, and also possibly being enumerated in /proc/ and seen with a ps.
https://stackoverflow.com/questions/51365355/kubernetes-secrets-volumes-vs-environment-variables/53367504#53367504
https://www.tenable.com/audits/items/CIS_Kubernetes_v1.20_v1.0.0_Level_2_Master.audit:98de3da69271994afb6211cf86ae4c6b
Tested with Minikube.

@orcutt989 please create a new PR under your own name

I can't review since I created the PR but this is currently deployed on teztnets and it works. :+1:


RC > Removed SKY CSS import

Replaced runtime and karma style loaders with those released in @skyux/theme
Removed all references to sky.css.

To test, run against the template's builder-dev-rc branch, here:
https://github.com/blackbaud/skyux-template/tree/builder-dev-rc

@Blackbaud-BobbyEarl We're getting a coverage failure because we were never at 100% code coverage for runtime (due to the Pact stuff). I removed a few things from runtime, which is probably why we're getting a different coverage number. Mind doing a sanity check on that?

@Blackbaud-BobbyEarl This is ready for another look.


Support for KEdit All command

Hello, I would like consideration for re-opening 12331, which in my opinion is the most power editing feature that I have never seen in any other editor since.   So, basically in KEdit, you had a command line where you can type in All / [keyword] and what it would do is collapse all the lines and only show lines that included one or more matches of the keyword, hiding all other lines.  So you can quickly get a global view of all instances of that keyword match.  You could make any line the active row and toggle back & forth to display/hide all lines in the text file.  It was really cool.
Another extremely powerful feature of KEdit was the change command, where on the command line with the syntax ch / [old text] / [new text] * * it would replace all instances of [old text] with [new text].  However, in order to preview the scope of changes, what you could do is first apply the All / [old text] command to see where are all the instances that would be replaced prior to applying the change command.

I'd love to see this too - or at least something similar :)


[QUESTION] Splitting big models over multiple GPUs

When specifying the number of GPUs during inference, is it only for parallelism or is the model loaded piece-wise over multiple GPUs, if it's bigger than individual GPUs? For example I'd like to use XCOMET-XXL and our cluster has many 12GB GPUs.
At first I thought that the model parts will be loaded onto all GPUs, e.g.:
comet-score -s data/xcomet_ennl.src -t data/xcomet_ennl_T1.tgt --gpus 5 --model "Unbabel/XCOMET-XL"

However I'm getting GPU OOM on the first GPU:
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 26.00 MiB. GPU 0 has a total capacity of 10.75 GiB of which 11.62 MiB is free. ...

Is it correct that in the above setting the model is being loaded in full 5 times on all 5 GPUs?
Is there a way to split the model over multiple GPUs?

Thank you!

unbabel-comet 2.2.1
pytorch-lightning 2.2.0.post0
torch 2.2.1

same question here

Last time I check this was not very easy to do with pytorch-lightning.
We actually used a custom made implementation with FSDP to train these larger models (without using pytorch-lightning). I have to double check if the new versions support FSDP better than the currently used pytorch lightning version (2.2.0.post0).
But short answer: model parallelism is not something we are supporting in the current codebase.

idea here. Ctranslate2 just integrated tensor parallelism. It also support XMLRoberta, so just wondering if we could adapt a bit the converter so that we could run the model within CT2 which is very fast.
How different is it from XML Roberta at inference ?

Does it support XLM-R XL? the architecture also differs from XLM-R

It seems like they improved documentation a lot actually: https://lightning.ai/docs/pytorch/stable/advanced/model_parallel/fsdp.html

Does it support XLM-R XL? the architecture also differs from XLM-R

we can adapt if we have the detailed description somewhere.
cc @minhthuc2502


36 Valid Sudoku fail for some invalid Sudoku

https://github.com/neetcode-gh/leetcode/blob/main/36-Valid-Sudoku.py
Is there a valid for
Solution().isValidSudoku(
    [
        ["9","2","3","4","5","6","7","8","."],
        [".",".",".",".",".",".","6","5","2"],
        [".",".",".",".",".",".",".","4","3"],
        [".",".",".",".",".",".","9",".","4"],
        [".",".",".",".",".",".",".",".","5"],
        [".",".",".",".",".",".",".",".","6"],
        [".",".",".",".",".",".",".",".","7"],
        [".",".",".",".",".",".",".",".","8"],
        [".",".",".",".",".",".",".",".","9"],
    ]
)

I have worked on this issue and submitted my pr.

Unless I'm missing something the value returned by the current python solution is correct. If something was broken Kraken-ops claims to have fixed it.


🐛 BUG: An itty-router-openapi app stopped working with TypeError: o3.replace is not a function

Which Cloudflare product(s) does this pertain to?
Uncertain
What version(s) of the tool(s) are you using?
Wrangler 3.22.4
What version of Node are you using?
v20.10.0
What operating system are you using?
Mac
Describe the Bug
An itty-router-openapi application I've been developing for a week stopped working today after upgrading from Wrangler 3.22.2 to 3.22.4.  I keep getting the error message:
[wrangler:err] TypeError: o3.replace is not a function
I've provided a minimal reproduction, which is just the OpenAPI example created using npm create cloudflare@latestwhich I can no longer get to work. A deployed application fails with the same error.
Strangely downgrading to 3.22.2 doesn't work, neither does deleting node_modules and reinstalling, restarting machine etc.
My node installation is vanilla LTS 20.10.

Please provide a link to a minimal reproduction
https://github.com/willtemperley/open-api-test
Please provide any relevant error logs
open-api-test % npx wrangler dev
⛅ wrangler 3.22.4
⎔ Starting local server...
[wrangler:inf] Ready on http://localhost:59737
[wrangler:err] TypeError: o3.replace is not a function
at null. (file:///Users/will/openapi-example/open-api-test/node_modules/@cloudflare/itty-router-openapi/dist/index.mjs:1:6075)
at Object.fetch (file:///Users/will/openapi-example/open-api-test/node_modules/@cloudflare/itty-router-openapi/dist/index.mjs:1:6721)
at facade_modules_fetch (file:///Users/will/openapi-example/open-api-test/.wrangler/tmp/bundle-s5zAaN/middleware-loader.entry.ts:46:16)
at facade_invokeChain (file:///Users/will/openapi-example/open-api-test/node_modules/wrangler/templates/middleware/common.ts:53:9)
at Object.next (file:///Users/will/openapi-example/open-api-test/node_modules/wrangler/templates/middleware/common.ts:50:11)
at jsonError2 (file:///Users/will/openapi-example/open-api-test/node_modules/wrangler/templates/middleware/middleware-miniflare3-json-error.ts:22:30)
at facade_invokeChain (file:///Users/will/openapi-example/open-api-test/node_modules/wrangler/templates/middleware/common.ts:53:9)
at facade_invoke (file:///Users/will/openapi-example/open-api-test/node_modules/wrangler/templates/middleware/common.ts:63:9)
at Object.fetch (file:///Users/will/openapi-example/open-api-test/.wrangler/tmp/bundle-s5zAaN/middleware-loader.entry.ts:114:11)
[wrangler:inf] GET / 500 Internal Server Error (635ms)
[wrangler:err] TypeError: o3.replace is not a function
at null. (file:///Users/will/openapi-example/open-api-test/node_modules/@cloudflare/itty-router-openapi/dist/index.mjs:1:6075)
at Object.fetch (file:///Users/will/openapi-example/open-api-test/node_modules/@cloudflare/itty-router-openapi/dist/index.mjs:1:6721)
at facade_modules_fetch (file:///Users/will/openapi-example/open-api-test/.wrangler/tmp/bundle-s5zAaN/middleware-loader.entry.ts:46:16)
at facade_invokeChain (file:///Users/will/openapi-example/open-api-test/node_modules/wrangler/templates/middleware/common.ts:53:9)
at Object.next (file:///Users/will/openapi-example/open-api-test/node_modules/wrangler/templates/middleware/common.ts:50:11)
at jsonError2 (file:///Users/will/openapi-example/open-api-test/node_modules/wrangler/templates/middleware/middleware-miniflare3-json-error.ts:22:30)
at facade_invokeChain (file:///Users/will/openapi-example/open-api-test/node_modules/wrangler/templates/middleware/common.ts:53:9)
at facade_invoke (file:///Users/will/openapi-example/open-api-test/node_modules/wrangler/templates/middleware/common.ts:63:9)
at Object.fetch (file:///Users/will/openapi-example/open-api-test/.wrangler/tmp/bundle-s5zAaN/middleware-loader.entry.ts:114:11)
[wrangler:inf] GET /favicon.ico 500 Internal Server Error (252ms)

Any updates on this?

Looks like this is an itty-router-openapi issue
https://github.com/cloudflare/itty-router-openapi/issues/112

This issue is fixed in the latest version of itty-router-openapi v1.0.7


add datarobot community provider

Description
Add DataRobot provider
Adding a new package?
If this pull request adds a new package:

[x] The package's schema URL in this PR is correct.
[x] The package metadata file, if present, contains:

[x] a supported category (one of Cloud, Infrastructure, Network, Database, Monitoring, or Utility).
[x] a valid plugin download URL. See Publish Your Package.
[x] a description that explains what the package does.
[x] a valid logo URL that points to a PNG whose dimensions conform to the others in this repo (e.g., 100x100).

[x] The package repo contains an Overview doc (/docs/_index.md) that includes:

[x] a brief explanation of what the package is and what it does.
[x] at least one representative example in all supported languages.
[x] a front-matter property for the layout set to package.

[x] The package repo contains an Installation and Configuration doc (/docs/installation-configuration.md) that includes:

[ ] links to SDKs in all supported languages.
[x] a copyable command for installing the resource plugin if necessary.
[x] an example of configuring the provider with pulumi config set.
[x] an example of configuring the provider with environment variables.

[x] The repository has:

[x] a version tag prefixed with v that corresponds with a valid GitHub release and published package SDKs

[ ] A CODEOWNER has reviewed the PR.
[ ] A member of the @pulumi/docs team has reviewed all documentation.

@NolanJMcCafferty A couple issues I've noted:

There's no .NET SDK: https://github.com/datarobot-community/pulumi-datarobot/blob/main/docs/installation-configuration.md
The link to the Go SDK (from the installation and configuration page linked above) is broken (not sure if this is expected or not): https://pkg.go.dev/github.com/datarobot-community/pulumi-datarobot/sdk
You should add links to SDKs here (but it's not the most common path for Pulumi users to install provider SDKs): https://github.com/datarobot-community/pulumi-datarobot/blob/main/README.md
The TS example has YAML as its language: https://github.com/datarobot-community/pulumi-datarobot/blob/main/docs/_index.md

Posted this here along with the email sent to Josh:
We are currently running into this error when trying to publish the .NET SDK as Pulumi.Datarobot:
error: Response status code does not indicate success: 409 (This package ID has been reserved. Please request access to upload to this reserved namespace from the owner of the reserved prefix, or re-upload the package with a different ID. Go to https://docs.microsoft.com/nuget/reference/id-prefix-reservation learn more about Package ID prefix reservation.).
Do you know why "Pulumi.Datarobot" would already be reserved? I don't see any packages on Nuget with that identifier. Could it be because we don't have permissions to publish to "Pulumi." ?

@ringods Ok I have updated the _index.md file and released the fix.

cc @pulumi/docs @sean1588 for a review & merge as I'm currently out.


⚠ ClamAV has degraded performance

In 9474e71, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 566 ms

Resolved: ClamAV performance has improved in f30d9bf.


Unsupported UA for edge (latest version for Mobile)

Please find the attached screenshot

{
"name": "Chrome Mobile",
"version": "77.0.3865.116",
"layout": "Blink",
"prerelease": null,
"os": "Android 10",
"manufacturer": null,
"product": null,
"description": "Chrome Mobile 77.0.3865.116 on Android 10",
"ua": "Mozilla/5.0 (Linux; Android 10; GM1911) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/77.0.3865.116 Mobile Safari/537.36 EdgA/<IP_ADDRESS>36"
}

Duplicate of #158. This has been fixed, and a new release should be out sometime this week.

Perfect thanks


:sparkles:[#64] add subgroup to generate_envvar_docs

Fixes #64

Do you think this is fine or should I try to implement the automatic help text retriever from setup config?


[5.4] SR-13822: FileHandle closeOnDealloc causes a Stack Overflow on Windows

Setting closeOnDealloc to false seems to cause a stack Overflow on
Windows so disable the functionality of NOT closing the file, but only
on Windows.

(cherry picked from commit 42a5b306e25458784796e606ba83ed4c7ef7d08f)

@swift-ci test

@swift-ci test

@swift-ci test linux


Adjust for typed binops

@bors r+

:pushpin: Commit d7ff7ccbad89c4fe9eb84b3b22f4b994f7f169a0 has been approved by RalfJung

:hourglass: Testing commit d7ff7ccbad89c4fe9eb84b3b22f4b994f7f169a0 with merge b5c70e92f326ba4edd11432920f11730b7f4e05e...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing b5c70e92f326ba4edd11432920f11730b7f4e05e to master...


ExampleBot.php not working

hello,
this is my first time trying this lib,
i set my login information in file settings.json and open url
http://localhost/PHP-SteamCommunity-master/Examples/ExampleBot.php
and i got this message on the web page
Fatal error: Class 'waylaidwanderer\SteamCommunity\SteamCommunity' not found in C:\AppServ\www\PHP-SteamCommunity-master\Examples\ExampleBot.php on line 17
what's the problem ?
and please ... i want to remove this
defined('STDIN') or define('STDIN', fopen("php://stdin","r"));

use waylaidwanderer\SteamCommunity\Enum\LoginResult;
use waylaidwanderer\SteamCommunity\MobileAuth\WgTokenInvalidException;
use waylaidwanderer\SteamCommunity\SteamCommunity;

and replce it with normal files include
i hope you tell what files should be included to examplebot to work

Hello. This library is built by oop style(classes and namespaces). You cannot use this only by including. Try to learn oop and ude IDE(e.g. phpstorm)

i will try and i will post again soon :)


support tree/ftree commands

Support tree command to easily show an overview of a node.

tree
ftree

Thanks for contributing! The feature looks great :)
Would you mind rebasing your branch on top of master (I took this opportunity to revamp the dev environment) and fix the tests/test_runner.py::test_help_general test?
After that, we'll be good to go, and I'll release a new version.
Thanks again!

Thanks for contributing! The feature looks great :)
Would you mind rebasing your branch on top of master (I took this opportunity to revamp the dev environment) and fix the tests/test_runner.py::test_help_general test?
After that, we'll be good to go, and I'll release a new version.
Thanks again!

Travis seems to be very slow to run and then report test cases.
Everything runs OK locally, so I'm 👍 on this patch. Thanks again!

Travis seems to be very slow to run and then report test cases.
Everything runs OK locally, so I'm 👍 on this patch. Thanks again!

ok

ok

izk 0.4.4 has now been published to PyPI, which includes your patch!

izk 0.4.4 has now been published to PyPI, which includes your patch!


Fix warning on newer Clangs

Fixed the warning on the latest version of Xcode. Removed the conditional per your tweet since we can assume everyone will have at least 4.3.

perfect, thanks!


Double Display of hide show password field

Whenever we are including hide show password input package it is taking icon also as a one field it is appearing two input boxes as output.

Hi @divs246, I'm sorry but this is the React Native Website repository. The bugs or issues in the React Native framework should be reported in the facebook/react-native repository.


Use In Staging Environment

Is there a way to use the staging public key with this library?

Hello @OlaAde,
Thanks for reaching out to the Team.
Please pass the test public key from your dashboard on staging mode into config.
I hope this helps.
Regards,


ci: Various workflow improvements

Simplify RTD build
Add documentation checks
Fix release CD
Moved coverage tests to dedicated CI step
Improve CI display
Added schedule runner
Mark experimental tests as green when run on the main branch

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

Comparison is base (d362749) 100.00% compared to head (bf94e61) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #4   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines            5         5
=========================================
  Hits             5         5

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


Path used for inline task reference is not honored

The path specified in a reference is respected during compilation, but not during execution.
Repro Steps: Create an inline task, add a reference to a non-standard assembly and attempt to use it in a code fragment.
This bug has also been reported on StackOverflow.
Code that reproduces the error:
<?xml version="1.0" encoding="utf-8"?>
<Project ToolsVersion="14.0" DefaultTargets="TestBuild" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <UsingTask TaskName="Hello" TaskFactory="CodeTaskFactory" AssemblyFile="$(MSBuildToolsPath)\Microsoft.Build.Tasks.Core.dll" >
    <ParameterGroup />
    <Task>
      <Reference Include="$(MSBuildExtensionsPath)\Microsoft\VisualStudio\v14.0\Web\Microsoft.Web.XmlTransform.dll"/>
      <Reference Include="System.Xml" />
      <Code Type="Fragment" Language="cs">
        <![CDATA[
          var f = new Microsoft.Web.XmlTransform.XmlTransformationException("test");
          Log.LogMessage("test: " + f.Message);
        ]]>
      </Code>
    </Task>
  </UsingTask>
  <Target Name="TestBuild">
    <Hello />
  </Target>
</Project>

This is still an issue if using RoslynCodeTaskFactory.
error MSB4018: System.IO.FileNotFoundException: Could not load file or assembly
But it does work with CodeTaskFactory.


chore(merge-back): 2.37.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 43d3fd78c9868a2c07a881b68cdddf4ed99490b9
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Have Travis CI/AppVeyor build binary releases

Travis/AppVeyor are enabled. Haven't set up binary releases though.

This is done in #9. There will be binaries on the releases page on the next release.


[Streams] Coroutines interop with Publishers

Description
Provide coroutine interop with Publishers
This is a fork of https://github.com/Kotlin/kotlinx.coroutines/blob/master/reactive/kotlinx-coroutines-reactive/ with the following modifications:

It is now published as a KMM module, except for js target where there are some unbound symbols.
Removed JVM specific code and Reactor lib specific code.
Removed deprecated functions

Motivation and Context
This should serve as a bi-directional bridge between our reactive publishers and coroutine flows and channels.
How Has This Been Tested?
It hasn't been tested yet and will be a bit later.
Unit tests from original codebase weren't ported either.
Types of changes

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

Importing unit tests from kotlin.coroutines was really not trivial as they were made for JVM target and has quite a few threading logic tests. Those tests also depend on internal test classes that I had to fork as well and adapt for our MPP structure.
That being said, I got them somewhat working with some work.


triangle crash on resize

Linux Nvidia 470.74
Bug - Crash on window resize (x11).
Every other Sample work without crash on resize, only triangle crash.
triangle -g 1
Fatal : VkResult is "ERROR_OUT_OF_DATE_KHR" in /home/danil/Documents/git/Vulkan/examples/triangle/triangle.cpp at line 363
triangle: /home/danil/Documents/git/Vulkan/examples/triangle/triangle.cpp:363: void VulkanExample::draw(): Assertion `res == VK_SUCCESS' failed.

Crash only when Nvidia GPU used as main GPU(monitor connected to this GPU), when it not main - resize does not crash on this app.

Closing due to inactivity.


oserror (errno 30) read-only file system python docker

when I run test:

@JoseponLee Hi, I am facing the same issue, Can you guide me through this?

@JoseponLee Hi, I am facing the same issue, Can you guide me through this?

Hi, I have encounter the same issue. Did you solve this? Can you offer me some advice. Thanks.

when I run test:

Hi, I have encounter the same issue. Could you kindly give me some advice? Thanks!

Hi，I am also facing the same error. Could you tell me how to solve it?

so are there any solutions now?


HMR not working

At my job, I have several project using the same basic structure, and HMR on the dev server is not working for any of them. I have read the documentation in detail but after several hours of scratching my head I have not yet found the problem.
This is my dev server:
const { FuseBox } = require('fuse-box');
const proxyServer = require('./proxy');
const getPlugins = require('./resources/fusebox-config/plugins');
const aliases = require('./resources/fusebox-config/aliases');
require('dotenv').config();

const config = {
  tsConfig: "./resources/fusebox-config/compiler-options.json",
  homeDir: '../src',
  output: '../dist/$name.js',
  sourceMaps: true,
  target: 'browser@es5',
  useTypescriptCompiler: true,
  allowSyntheticDefaultImports: true,
  log: {
    showBundledFiles: false, // Don't list all the bundled files every time we bundle
    clearTerminalOnBundle: true, // Clear the terminal window every time we bundle
  },
  alias: aliases,
  plugins: getPlugins('dev'),
};

const fuse = FuseBox.init(config);
fuse.dev({ port: 4001, root: false }, server => proxyServer(server.httpServer.app));
fuse
  .bundle('app')
  .instructions('> index.jsx')
  .hmr();
fuse.run();

Currently I have to add .watch() to the chain in order to get the page to reload on each file change. With the above configuration, nothing happens when I save, I see nothing in the logs.
I've seen this message in the docs: To avoid problems with HMR settings (below) set your development server config before you define further bundle configuration. But I'm not sure what it means or if its related to my problem.
If you need additional information please tell me and I will be happy to comply.

hey @amygdaloideum v3 is out scope now. We're focusing on v4.
Here is a valid react example
https://github.com/fuse-box/react-example/blob/master/fuse.ts
Using fuse-box@next Here are the docs
If you have problems migrating join us on slack. We can help you ;-)
https://slack.fuse-box.org/

Closing this for now. Re-open if still relevant


Deflake faster joins tests by leaving rooms as part of test cleanup

Many of the faster joins test flakes are due to the homeserver under
test failing to contact Complement homeservers after they have been
torn down. When this happens, subsequent tests can fail if they use a
Complement homeserver that happens to have the same hostname:port as one
which the homeserver under test has previously marked as offline.
Make the homeserver under test leave rooms at the end of each test, so
that it stops trying to contact the Complement homeservers in the room.
Signed-off-by: Sean Quah<EMAIL_ADDRESS>
Reviewable commit by commit.
NB: you can tease out the flakes by forcing Complement to reuse ports whenever possible:
diff --git a/internal/federation/server.go b/internal/federation/server.go
index ed7974d..4c4ccbe 100644
--- a/internal/federation/server.go
+++ b/internal/federation/server.go
@@ -455,7 +455,10 @@ func (s *Server) Listen() (cancel func()) {
        var wg sync.WaitGroup
        wg.Add(1)

-       ln, err := net.Listen("tcp", ":0") //nolint
+       ln, err := net.Listen("tcp", ":63333") //nolint
+       if err != nil {
+               ln, err = net.Listen("tcp", ":63334") //nolint
+       }
        if err != nil {
                s.t.Fatalf("ListenFederationServer: net.Listen failed: %s", err)
        }

Thanks for the review!


Bump Microsoft.OpenApi

Bump Microsoft.OpenApi packages to 1.6.14.
Resolves #2606.

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

Project coverage is 91.66%. Comparing base (830c5a6) to head (52750a4).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2795   +/-   ##
=======================================
  Coverage   91.66%   91.66%
=======================================
  Files          91       91
  Lines        3010     3010
  Branches      517      517
=======================================
  Hits         2759     2759
  Misses        251      251

Flag
Coverage Δ

Linux
91.66% <ø> (ø)

Windows
?

macOS
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


python3Packages.aioshelly: 1.0.4 -> 1.0.5

Motivation for this change
Update to latest upstream release 1.0.5
Change log: https://github.com/home-assistant-libs/aioshelly/releases/tag/1.0.5
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 146849 run on x86_64-linux 1

3 packages built:

home-assistant
python38Packages.aioshelly
python39Packages.aioshelly


[CircleGraph] Relocate message handlers

This will relocate message handler related codes to CircleGraphCtrl.
ONE-vscode-DCO-1.0-Signed-off-by: SaeHie Park<EMAIL_ADDRESS>

for #641, from draft #647

@hyunsik-yoon , rebased. PTAL again;


Roadmap & Ideas

These are features (or rough ideas for features) that I would like to implement in the future. They are not in any particular order, so there's no guarantee that something at the top of the list will be worked on next.

Other views beyond the combat tracker:

Image Viewer. Being able to display art assets like NPCs or maps.

Combat Tracker:

Allow for setting up multiple combats, and load them in the combat tracker.
Character library to make quickly adding pre-saved characters to a combat.

UI

Overhaul to look nicer and perhaps work on mobile or tablet
Dark mode support

Player view where players can submit their initiative rolls

UI has been rewritten using Material UI. This has allowed for theming and dark mode. Mobile support still isn't truly "supported", but it should at least be usable.


chore(resume): update "work" to match last schema

Seems we were using an old version of JSON Resume for work items. When consulting the 1.0.0 schema, in the work section items, fields company and website were renamed into name and url.
Updating to match schema.
Seems image is not in the schema, so moving it to the end to indicate it's an addition.

wtf?? why it got merged?? checks were failing
https://github.com/davidlj95/website/actions/runs/6485203283/job/17610768588
ahhh wait because it's not targetting main so checks aren't required. adding something...


Fix #33732

Fixes #33732

For what it's worth, our codebase (which exhibited this issue in 3.6+, and seemed to be resolved by merging this PR) no longer triggers this assertion with 3.8.

Yes, it appears that something in 3.8.?? fixes this, but it still breaks in 3.7.?? and below.
Do we want to patch older versions (so we should keep this fix).
If not, I can remove the change but I suggest keeping the test for regressions.

I'm pretty sure @ahejlsberg shipped a fix or two to union assignability in 3.8. He definitely encountered this assert along the way, so I think the improvement in 3.8 is intentional. Let's trim this PR down to the test.
The assertion should happen only in cases where the compiler previously failed to give an error when it should. I guess I'd rather have the assert than a failure to error, and in any case we would only ship an extremely urgent fix to 3.7. I don't think that #33732 is that urgent.


Can't install NodeJS gl-client on Mac M1

Here's the stack trace I get when ran npm install
npm ERR! code 101
npm ERR! path /Users/bilthon/Development/Node/Demo/greenlight-test/node_modules/gl-client
npm ERR! command failed
npm ERR! command sh -c node-pre-gyp install --fallback-to-build=false || npm run build
npm ERR! response status 404 Not Found on https://storage.googleapis.com/greenlight-artifacts/npm/gl-client-js/0.1.9/gl-client-0.1.9-darwin-x64.tar.gz
npm ERR!
npm ERR! ><EMAIL_ADDRESS>build
npm ERR! > cargo-cp-artifact -a cdylib gl_client_js index.node -- cargo build --message-format=json-render-diagnostics --release
npm ERR! node-pre-gyp info it worked if it ends with ok
npm ERR! node-pre-gyp info using<EMAIL_ADDRESS>npm ERR! node-pre-gyp info using<EMAIL_ADDRESS>| darwin | x64
npm ERR! node-pre-gyp info check checked for "/Users/bilthon/Development/Node/Demo/greenlight-test/node_modules/gl-client/index.node" (not found)
npm ERR! node-pre-gyp http GET https://storage.googleapis.com/greenlight-artifacts/npm/gl-client-js/0.1.9/gl-client-0.1.9-darwin-x64.tar.gz
npm ERR! node-pre-gyp ERR! install response status 404 Not Found on https://storage.googleapis.com/greenlight-artifacts/npm/gl-client-js/0.1.9/gl-client-0.1.9-darwin-x64.tar.gz
npm ERR! node-pre-gyp ERR! install error
npm ERR! node-pre-gyp ERR! stack Error: response status 404 Not Found on https://storage.googleapis.com/greenlight-artifacts/npm/gl-client-js/0.1.9/gl-client-0.1.9-darwin-x64.tar.gz
npm ERR! node-pre-gyp ERR! stack     at /Users/bilthon/Development/Node/Demo/greenlight-test/node_modules/@mapbox/node-pre-gyp/lib/install.js:67:15
npm ERR! node-pre-gyp ERR! stack     at processTicksAndRejections (internal/process/task_queues.js:95:5)
npm ERR! node-pre-gyp ERR! System Darwin 21.6.0
npm ERR! node-pre-gyp ERR! command "/Users/bilthon/.fnm/node-versions/v14.20.0/installation/bin/node" "/Users/bilthon/Development/Node/Demo/greenlight-test/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build=false"
npm ERR! node-pre-gyp ERR! cwd /Users/bilthon/Development/Node/Demo/greenlight-test/node_modules/gl-client
npm ERR! node-pre-gyp ERR! node -v v14.20.0
npm ERR! node-pre-gyp ERR! node-pre-gyp -v v1.0.10
npm ERR! node-pre-gyp ERR! not ok
npm ERR! error: failed to parse manifest at `/Users/bilthon/Development/Node/Demo/greenlight-test/node_modules/gl-client/Cargo.toml`
npm ERR!
npm ERR! Caused by:
npm ERR!   can't find library `gl_client_js`, rename file to `src/lib.rs` or specify lib.path
npm ERR! Did not copy "cdylib:gl_client_js"

npm ERR! A complete log of this run can be found in: /Users/bilthon/.npm/_logs/2023-06-16T21_20_48_066Z-debug-0.log

I suppose it's the lack of pre-compiled binaries as mentioned in the README, right? But if so, how do I compile them? The npm update also didn't work.

The problem stems from the fact that we have a multi-crate repository, but only the gl-client-js crate source is being bundled in the JS source distribution. Since that depends on gl-client (the Rust library) and that is not distributed compilation fails.
You can build by checking out the code, and then running npm run build in the libs/gl-client-js directory, and then depending on the built library path from your project.
As for the precompiled library, it's there, but there seems to be a mismatch between architecture tag being searched (gl-client-0.1.9-darwin-x64.tar.gz) and the tag that is there (gl-client-0.1.9-darwin-x86_64.tar.gz) (notice the x64 vs x86_64).
Not being a JS packaging expert, I might have gotten this wrong, but I expected the arch to correspond to console.log(process.arch) when looking for precompiled bundles.

Ok, so we created a copy of the binary, so it should now be found by the downloader script.


feature: add volume driver alias

Ⅰ. Describe what this PR did
Add volume driver alias, pouchd can config the exist volume driver's
alias.
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews
Signed-off-by: Rudy Zhang<EMAIL_ADDRESS>

We may need some test case in daemon to verify Volume Driver alias name feature @rudyfly

Codecov Report

Merging #1224 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1224   +/-   ##
=======================================
  Coverage   15.44%   15.44%
=======================================
  Files         171      171
  Lines       10576    10576
=======================================
  Hits         1633     1633
  Misses       8823     8823
  Partials      120      120

Impacted Files
Coverage Δ

daemon/config/config.go
10% <ø> (ø)
:arrow_up:

LGTM


Augmented PersonDeleteCommand

PersonDeleteCommand now makes use of parallel streams to refresh all the CCAs

Codecov Report

Merging #232 (713c9b7) into master (17be725) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #232      +/-   ##
============================================
+ Coverage     44.89%   44.94%   +0.05%
  Complexity      503      503
============================================
  Files           121      121
  Lines          2557     2554       -3
  Branches        316      315       -1
============================================
  Hits           1148     1148
+ Misses         1346     1344       -2
+ Partials         63       62       -1

Impacted Files
Coverage Δ

...ess/logic/commands/person/PersonDeleteCommand.java
76.47% <100.00%> (+11.47%)
:arrow_up:

src/main/java/seedu/address/model/AddressBook.java
67.74% <0.00%> (ø)

...u/address/logic/commands/cca/CcaColourCommand.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


SyntaxError

would this be getting fixed ?

There are some issues with the package , but the code can be used


[Gallery] Add Material Study app Rally as an example app

Add Material Study app Rally as an example app.
Description
Adds a Flutter sample app based on the Material study Rally (a hypothetical, personal finance app). It showcases custom tabs, custom painted widgets, and custom animations.

Related Issues
N/A
Tests
N/A
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I signed the CLA.
[X] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[X] I updated/added relevant documentation (doc comments with ///).
[X] All existing and new tests are passing.
[X] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[X] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[X] No, this is not a breaking change.

This change contains Apache 2.0 licensed files, which aren't allowed in flutter/flutter.
Reviewers (@rami-a @willlarche) FYI to check newly added files for proper license headers.
I'm going to revert this change.  Feel free to re-send with the license headers fixed.
FYI @Hixie

This change was also missing tests.  Please include tests for all changes.

Thanks @tvolkert for reverting it, was not aware that Apache 2.0 license was not allowed. It was needed for the Roboto font.
(Also all of the gallery example studies have smoke tests by default.)

re: tests -- imagine we have a bot that randomly changes or removes code from the codebase and checks it in if the tests pass.  If you don't want that bot to break you, then you should have tests to ensure that doesn't happen.
Another way to think about it is that you should have tests ensuring the behavior of anything that you care about.  The fact that this was able to be reverted without changing any tests implies that the behavior in question wasn't worth keeping.

If/When you re-land it please include the changes in https://github.com/flutter/flutter/commit/68d1fe58da759c22a8c3ce6f2735c914f7844316. I am reverting it for now.


hybridcompute readme t2 config

readme t2 config

For python, the breaking change is ok

@ArcturusZhang Please help on the failure of GO.

/azp run


Move the metadata into PEP 621-compliant pyproject.toml.

Requires #1145
Also this commit gets rid of the most build-time dependencies (module is no longer imported when the wheel is being built, instead modern versions of setupttols can read vars using static analysis. Requires a newer setuptools. distutils support is dropped.

For now we don't delete setup.py, but it should be done. It may require changes in CI scripts, I have no time to debug that, so leave it up to the maintainers of this repo.


Some Questions Regarding Expansion Strategies

Hi,
Firstly, in previous inquiries, I discovered a reference to the address https://molecularai.github.io/aizynthfinder/training.html for creating a new extension strategy. However, it seems this address is no longer active. Could you please clarify if this content has been intentionally removed? If so, where can I find it now?
Secondly, I've come across mentions of the latest USPTO model, uspto_keras_model. Could you direct me to where I can access this model?
I apologize for my lack of understanding regarding these matters and would greatly appreciate your assistance in providing clarification.
Thank you!

The training was moved to a separate repo called AiZynthTrain: https://github.com/MolecularAI/aizynthtrain
You can download the latest model with the download_public_data tool as explained on the landing page, or downloaded from Zenodo: https://zenodo.org/records/7341155

Hi，
Thank you very much for responding to me so quickly.
I found that the filenames downloaded via the 'download_public_data' command are different from those on the website https://zenodo.org/records/7341155. Does this mean that there are different versions of the models?
All the best!

The name does not matter. The  'download_public_data' downloads the latest version.

Thank you for your prompt reply, I have no further questions.


The action voting for EOS block producers don't run.

			var result = await eos.CreateTransaction(new EosSharp.Api.v1.Transaction() 			{ 				Actions = new List<EosSharp.Api.v1.Action>() 				{ 						new EosSharp.Api.v1.Action() 						{ 							Account = "eosio", 							Authorization = new List<PermissionLevel>() 							{ 								new PermissionLevel() {Actor = "myeosaccount", Permission = "owner" } 							}, 							Name = "voteproducer", 							Data =new 							{ 								voter = "myeosaccount", 								proxy = "", 								producers =new string[] { "eosbeijingbp", "eoscannonchn" } 							} 						} 				} 			});
The above code can run, but if the producers order is changed, it won't run:
			var result = await eos.CreateTransaction(new EosSharp.Api.v1.Transaction() 			{ 				Actions = new List<EosSharp.Api.v1.Action>() 				{ 						new EosSharp.Api.v1.Action() 						{ 							Account = "eosio", 							Authorization = new List<PermissionLevel>() 							{ 								new PermissionLevel() {Actor = "myeosaccount", Permission = "owner" } 							}, 							Name = "voteproducer", 							Data =new 							{ 								voter = "myeosaccount", 								proxy = "", 								producers =new string[] { "eoscannonchn", "eosbeijingbp"} 							} 						} 				} 			});
The EOS mainnet will response "voteproducer eosio_assert_message assertion failure" for this code.

That is normal the smart contract enforces that the order of BP be in alphabetic order.


Is the algorithmic_trading using DDQN or DQN?

Hi
Is the algorithmic_trading using DDQN or DQN? If not, is it possible to convert it to DDQN?

It is DQN, if you want to use DDQN, you need to implement the corresponding agent in trademaster/agents/algorithmic_trading

Hi
Thank you for your response! Do you mean if I follow TradeMaster/agents/high_frequency_trading? I only found DDQN for high_frequency_trading.  For now, I understand algorithmic_trading very well. I need to sit down and comprehend high_frequency_trading.

Yes, our high_frequency_trading is DDQN, you can refer to it https://github.com/TradeMaster-NTU/TradeMaster/blob/1.0.0/trademaster/agents/high_ frequency_trading/ddqn.py


Add bordered prop to Dropdown

#3456

Looks like there was a regression on the HD height

Looks like there was a regression on the HD height

@joshwooding What does this mean? How can it be solved?

Something is weird, opening the dropdown via mouse, press down shows a thicker line than released?

@origami-z  The active indicator is only there for active state. that is why. hmm. I will add it for focus state as well?

Do we need visual snapshot on validation status?

Hi @origami-z , what should be done for this? Not sure what this means.

Do we need visual snapshot on validation status?

Hi @origami-z , what should be done for this? Not sure what this means.

Need to include bordered validation status in .qa.stories file so Chromatic can take visual regression


number axis for main statistics

When viewing the main statistics line chart It is rather inconvenient that a vertical number axis is not shown. That means that a screenshot of the chart for example cannot be used to communicate specific quantitative details apart from volume differences between different time periods.
See example:

Thank you for this request - I will make the vertical axis configurable in the add-on options in the next release.

This feature is implemented now and will soon be published with the next release:

Let me know if everything works as expected then.


save path relative to common directory; on load build absolute path

What does this Pull Request accomplish?
Save relative paths (instead of absolute paths) for DSF components in system definition file. On load, construct the relative path into an absolute path to perform file operations.
Why should this Pull Request be merged?
Absolute paths include the Veristand  version which requires a mutation in the system definition to update the path before the system definition can be used by automated test VIs. Mutation requires a load and resave of the system definition in System Explorer.
What testing has been done?
Using artifacts produced in PR-14/Build 5, verified:

When saved, relative path to component is written to system definition
When loaded, component is found and constructed into absolute path
UDP and GE Reflective Memory plugins deploy and run

Notes to Reviewers
VI changes are entirely focused on writing relative path for component into system definition and then reconstructing the absolute path when this relative path is read from the system definition.


Custom script always into else-statement when executed from Polybar

Script has if-statement always false when launched from Polybar
I have written a simple script that runs correctly when I launch it from the terminal but when I run it from Polybar through the click-* command it only goes into the else-statement. The if-statement results always false.
I am a newbie for what concerns shell scripting but I have checked and debugged the code several times and I cannot find the issue. Please have a look and let me know.
Polybar 3.3.0 on Manjaro with i3-gaps
Here a portion of the Polybar config file
[module/firewall]
type = custom/script
exec = $HOME/.config/polybar/firewall_icon.sh
interval = 1
click-left = $HOME/.config/polybar/firewall.sh
format-background = ${colors.bg}
format-foreground = ${colors.cyan}

And here the script firewall.sh
#!/bin/sh
if [ "$( sudo ufw status | grep -wi status | awk '{print $2}' )" = "active" ]; then
    urxvt -e sudo ufw disable
    echo "echo " > "$HOME"/.config/polybar/firewall_icon.sh
else
    urxvt -e sudo ufw enable
    echo "echo " > "$HOME"/.config/polybar/firewall_icon.sh
fi

Expected behavior:
If the firewall has its status active it is then turned off.
If the firewall has its status inactive it is then turned on.
The command needs sudo privileges.
Actual behavior:
When the code is launched from Polybar the if-statement results always false and the firewall is only turned on.

You use sudo in your script. sudo has no way to get a password from you when running it from polybar. That's likely the issue

You can configure the sudoers file to allow password essentially invocation of ufw.
This will allow all of the sudo commands in your script to run in a non-interactive shell.
I am surprised that your test statement works at all when you use = instead of == to test the equivalence of strings.
I would actually suggest that you run the process substitution that is in your test before the test, and you can stuff it in a variable.
It would look like:
readonly fw_status="$( sudo utw status | grep -wi status | awk '{print $2}' )"
If [ "$fw_status" == 'active' ]; then
# your code here
else
# other code here
fi

Finally, I am not sure why you pass the ufw <enable/disable> commands through a new instance of your terminal program.
Is that to avoid the output of the commands?
If so, you can redirect the output from those commands to dev null with &>/dev/null.
Let me know if this helped., Or if you have any questions.  I teach bash, and I am willing to help you refactor this script.

Thanks @guygastineau!
I will close this, as it seems everything has been said.
By the way = for string comparison is the way to go when using [ .. ] (see man test). You can use == in the bash extension [[ .. ]] though (= works there too).

I see. I have used == for so long, and = is almost always exclusively an assignment operator in other languages.  I forgot and assumed ;)
By the way, is this for older/lighter shells, or is /bin/sh a symbolic link for /bin/bash on your system?

On my system sh is symlinked to bash, yes. But [ is POSIX, so it should be supported in any compliant shell. Even more interesting [ is a built-in in most shells, but there exists an executable /usr/bin/[ which has the same behavior.

You are right. [ is also synonymous with test I believe.
Well, good luck with positive.  My personal opinion is, if you are just making scripts for yourself then use bash if it is what your system has.  If you really need the portability for getting it to a broader audience, then I get it.
I say that without judgement.  I just really hate making tools that can run on sh and dash.  I respect those lighter shells for their speed, but for higher level custom scripts I really need the extra builtins from bash 4.x - like associative arrays (or arrays in general).
Have a good day ;)

Curiously, on my system [ is not symlinked to test or vice-versa.
I try to use bash as well, but I generally use [ just because I can quickly get the manpage for it ;)

Interesting. There may be a change of which I was unaware.
I used [ for a long time with bash too.
Then when I had to do some messy regex tests I fell in love with [[ now I even like || and && inside the braces too.
BTW, if you like working in shell you should check out the bash track at exercism.io.  You can login with your GitHub account, and you don't have to remember new credentials.

That's cool, I'll check it out.


Completely wrong predictions on own wav files

When I try to run my own audio recordings using label_wav.py, I always get random predictions. I trimmed the audio files to only contain the keyword and still receive false outputs. Does anyone experience the same?

Same experience here at first, but I probably solved it by checking the format of my own wav file. Your wav file should have the same format as those in training data.
i.e., the sample rate is 22050 and bit rate is 256kb/s, the length of the audio must be exactly 1 second. You can check the details by looking at Google speech command dataset.


Add Oxford University's Colleges

The University of Oxford is sub-divided into 43 Colleges / Societies / Permanent Private Halls. Every student belongs to one of these, and has a email address whose domain is of the form {college}.ox.ac.uk
This PR updates the repo to include a comprehensive set of domains (previously two of them were included).
This site describes more about how mail servers and domains are handled at Oxford: https://help.it.ox.ac.uk/oxford-email-addresses
An authoritative list of colleges, with links to webpages (same domain as email), is here: https://www.ox.ac.uk/about/colleges
But for completeness, below is listed the set of colleges and domains that have been added:

College
Email Domain

All Souls College
asc.ox.ac.uk

Balliol College
balliol.ox.ac.uk

Blackfriars Hall
bfriars.ox.ac.uk

Brasenose College
bnc.ox.ac.uk

Campion Hall
campion.ox.ac.uk

Christ Church College
chch.ox.ac.uk

Corpus Christi College
ccc.ox.ac.uk

Exeter College
exeter.ox.ac.uk

Green Templeton College
gtc.ox.ac.uk

Harris Manchester College
hmc.ox.ac.uk

Hertford College
hertford.ox.ac.uk

Jesus College
jesus.ox.ac.uk

Keble College
keble.ox.ac.uk

Kellogg College
kellogg.ox.ac.uk

Lady Margaret Hall
lmh.ox.ac.uk

Linacre College
linacre.ox.ac.uk

Lincoln College
lincoln.ox.ac.uk

Magdalen College
magd.ox.ac.uk

Mansfield College
mansfield.ox.ac.uk

Merton College
merton.ox.ac.uk

New College
new.ox.ac.uk

Nuffield College
nuffield.ox.ac.uk

Oriel College
oriel.ox.ac.uk

Pembroke College
pmb.ox.ac.uk

Regent's Park College
rpc.ox.ac.uk

Reuben College
reuben.ox.ac.uk

Somerville College
some.ox.ac.uk

St Anne's College
st-annes.ox.ac.uk

St Antony's College
sant.ox.ac.uk

St Catherine's College
stcatz.ox.ac.uk

St Cross College
stx.ox.ac.uk

St Edmund Hall
seh.ox.ac.uk

St Hilda's College
st-hildas.ox.ac.uk

St Hugh's College
st-hughs.ox.ac.uk

St John's College
sjc.ox.ac.uk

St Peter's College
spc.ox.ac.uk

The Queen's College
queens.ox.ac.uk

Trinity College
trinity.ox.ac.uk

University College
univ.ox.ac.uk

Wadham College
wadham.ox.ac.uk

Wolfson College
wolfson.ox.ac.uk

Worcester College
worc.ox.ac.uk

Wycliffe Hall
wycliffe.ox.ac.uk

@philipto apologies that is entirely my fault for not looking through the readme properly before submitting this - thanks for correcting me!


Saving to the same file leads to data loss

Reproduction

save "old scene"
delete all elements (to start from scratch)
press ctrl + s

Actual result

Backed up "old scene" got just overridden

This is technically by design. We're not making assumptions about what canvas action constitutes an action that should reset the active file handle — besides explicitly clearing canvas using the main menu.
I'm not sure that deleting all elements on canvas should reset the file handle as that'd be non-standard? E.g., if you delete all characters in a saved text file and save it again, it does the same thing.

But, thinking about it, deleting all elements probably should clear the active file handle to be on the save side, as it's not gonna happen too often to be that annoying, and losing data is worse.
Problem is we'd have to be checking each update whether all elements are deleted and resetting the file handle if so. But we also need to track whether previous update there were some non-deleted elements so that we don't clear the file handle just as you save an empty canvas to start a file (or similar cases).

But, thinking about it, deleting all elements probably should clear the active file handle to be on the save side, as it's not gonna happen too often to be that annoying, and losing data is worse.

Yeah, that's the point. Also, clear canvas should do the same (if it does not already, can't check rn).
Still "select all & delete" is a workaround for a missing "create new", which would allow to persist the current scene (to file or E+) or clear it (clear canvas). Either way, it should discard the current file handle.

allow to persist the current scene (to file or E+) or clear it (clear canvas)

Essentially combining "save to" and "clear canvas" into one prominent button. As discussed in https://github.com/excalidraw/excalidraw/issues/1957, as "save to" + "clear canvas" are two steps and not really intuitive.


404 Bad Request but there is {{ form.csrf_token }} and SECRET_KEY in form and config

Yesterday application with this form worked normal, but today - error...
...and flask also swears about g.flask.is_authenticated(). Changed to g.flask.is_authenticated...
...and flashes does not work...
forms.py
class LoginForm(Form):
    email = StringField('email', validators=[DataRequired('...'), Email('...'), Length(3, 50, '...')])
    password = PasswordField('password', validators=[DataRequired('...'), Length(3, 50, '...')])
    submit = SubmitField('submit')

template

    {% with flashes = get_flashed_messages() %}
    {% if flashes %}
    {% for flash in flashes %}

<button type="button" class="close" data-dismiss="alert" aria-hidden="true">×</button>{{ flash }}

    {% endfor %}{% endif %}{% endwith %}
    <form class="form-signin" method="POST" role="form" action="{{ url_for('admin_login') }}">
        {{ form.csrf_token }}
        {{ form.email(class="form-control", id="inputEmail", placeholder="...", type="email", maxlength="30") }}
        {{ form.password(class="form-control", placeholder="...", id="inputPassword", maxlength="50") }}
        {{ form.submit(class="btn btn-lg btn-primary btn-block", value="...") }}
    </form>

same issues here

What happened on your "today"?

It was saying Bad Request.
I was switching to the newest flask-login (0.3.0) where they switched the is_authenticated method to a property...
I reset the libraries but still not working, then I added some logging on the library code, and it worked till this morning, then after another reinstall everything is broken again....
really no ideas, im clueless

@Avrong I guess you are using flask_login? If yes could you check your LoginManager session_protection property?
In my case was set to 'strong' and given that https://github.com/maxcountryman/flask-login/commit/a96de342eae560deec008a02179f593c3799b3ba was introduced and it's clearing the session at every request(the hook seems to be global for each request, so it's not bound to a special login view...that's why it's breaking my other forms)
just taking it out made everything work fine....nothing to blame on flask-wtf

Problem not solved but app works if
{{ form.csrf_token }}

change to
<input type="hidden" name="csrf_token" value="{{ csrf_token() }}">

@Avrong that's weird.
{{ form.csrf_token }}

should be the same as
<input type="hidden" name="csrf_token" value="{{ csrf_token() }}">

Can't reproduce this.  The two statements are indeed equivalent.


haskell: Add arguments for run- and test-time dependencies.

Motivation for this change
This is motivated by https://github.com/NixOS/nixpkgs/issues/17226
I too have the issue that several of my haskell binaries run programs that are in PATH, i.e. not hard-coded.
This requires a rebuild of ghc 7.10 and 8.0, and potentially a mass-rebuild.
Things done

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

[x] NixOS
[ ] OS X
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

I am unhappy with this approach for the following reasons:

Wrapping an executable is going to fix that particular use case, but very often Haskell packages offer similar functionality in a library, too, which defines the actual functions that the executable merely provides a standardized front-end too. For users of the library, this solution is not going to work, so the solution is incomplete.

testtimeDepends doesn't serve any purpose. Instead, the proper attribute testToolDepends should be used.

Haskell builds are auto-generated, but our generator is never going to produce any of these fields because that information is not available from Cabal files. So the only place this kind of information could possibly be specified is in overrides -- which means that adding attributes to the generic Haskell builder is unnecessary. Instead, this should be implemented as an external combinator function that lives in, say, haskell.lib.

There is nothing Haskell-specific about the generation of these wrapper scripts, so the generic Haskell builder is not the right place to host that functionality.

Hm OK - I tried the combinators first but had some trouble making them pick up wrapProgram because that seems to be a setup hook. I'll close and try to find a different approach.

Adding makeWrappers to executableToolDepends should do the trick, but I haven't tested it.

Anyhow, this feature should really be integrated with to the generic builder since there is nothing Haskell-specific about it.


Unsupported device model id: TS0601, manufacturer _TZE200_dfxkcots

Hello Could someone advise the ZigBee device is a new dimmer tuay, but you can use it over my CC2531.
root":[1 item
0:{11 items
"ID":int1
"profileID":int260
"deviceID":int81
"inputClusters":[4 items
0:int0
1:int4
2:int5
3:int61184
]
"outputClusters":[2 items
0:int25
1:int10
]
"deviceNetworkAddress":int43413
"deviceIeeeAddress":string"0x60a423fffeec6b47"
"clusters":{1 item
"genBasic":{1 item
"attributes":{8 items
"modelId":string"TS0601"
"manufacturerName":string"_TZE200_dfxkcots"
"powerSource":int1
"zclVersion":int3
"appVersion":int66
"stackVersion":int0
"hwVersion":int1
"dateCode":string""
}
}
}
"binds":[]0 items
"configuredReportings":[]0 items
"meta":{}0 items
}
]
device code
_ieeeAddr: '0x60a423fffeec6b47',
_networkAddress: 43413,
_manufacturerID: 4098,
_endpoints: [ [Endpoint] ],
_manufacturerName: '_TZE200_dfxkcots',
_powerSource: 'Mains (single phase)',
_modelID: 'TS0601',
_applicationVersion: 66,
_stackVersion: 0,
_zclVersion: 3,
_hardwareVersion: 1,
_dateCode: '',
_softwareBuildID: undefined,
_interviewCompleted: true,
_interviewing: false,
meta: {},
_lastSeen:<PHONE_NUMBER>118,
_linkquality: 21

},
endpoint: Endpoint {
ID: 1,
profileID: 260,
deviceID: 81,
inputClusters: [ 0, 4, 5, 61184 ],
outputClusters: [ 25, 10 ],
deviceNetworkAddress: 43413,
deviceIeeeAddress: '0x60a423fffeec6b47',
clusters: { genBasic: [Object] },
_binds: [],
_configuredReportings: [],
meta: {}
},
data: <Buffer 09 08 11 00 04 40>,
linkquality: 21,
groupID: 0,
cluster: 'manuSpecificTuya',
meta: {}
homebridge code

I also see strange, that my Thermostats are recognized as:
Manufacturer: _TZE200_c88teujp Manufacturer ID: 4098
In my opinion we should not connect devices via Manufacturer, but Manufacturer ID
Can we address this issue ?

Ok, I've just discovered that this TRV is a Saswell SEA802 sold also as a TuYa ...
So is it possible that this valve work with Zigbee NT?
https://www.zigbee2mqtt.io/devices/SEA801-Zigbee_SEA802-Zigbee.html

This will be addressed in the next release, since the device discovery will be automatic. Stay tuned