Add support for DIB files and large data sets -- WIP

DIB files are added to the list of files converted to TIF.
Assuming masks in the DIB format also have the o1 suffix in their filename, these can be included or ignored too.
After converting ~2000 files, I was getting a 'Too many open files' error. See below:

JavaException: /media/starcalibre/New Volume/HT29/NIRHTa-001/AS_09125_050116110001_E16f01d0.DIB (Too many open files)
This was because the Bioformats reader was not being closed after a file was read. This is now fixed! However I think it might be worth doing the file reading in batches. The size of the batches could be dependent upon the amount of memory allocated to the JVM, or just be a user defined parameter. This would reduce the extra overhead of having to load and close the reader each time a file is read. Any thoughts?
Still outstanding:
Updated tests comprising test DIB files.
Updated documentation.

Thanks for this!

it might be worth doing the file reading in batches.

I'd want a benchmark to show 2x improvement before going down this route, because it would add complexity both to the interface and to the code. But yes, could be worthwhile.

Updated tests comprising test DIB files.

This would be great! Just a matter of adding a single .DIB to the test data directories, and the corresponding result TIFF?

Updated documentation.

Just adding ".DIB" to various places, right?

Benchmarking sounds good -- The HT29 dataset has 28 plates in total, and each plate comprises 6912 images. How about doing 14 plates with a batch size of say 500, 14 without a batch. Each plate conversion can then be timed and averaged over each set of 14. Benchmark results and the images will be converted to TIFF in the end. Two birds with one script.. :P

That's a nice benchmark! =) If you're going to be running other stuff at the same time though, you'll have to randomly interleave the benchmark type, so computer load is not a factor... =)

@starcalibre oops! Turns out this PR still needs some work! Can you make this a top priority? I'd like to merge ASAP and cut a release (so I can "pip install cellom2tif" on microscopium.io =).

Is the coveralls failure just because of the fall in coverage? So this should just be a matter of increasing the coverage a little right?

 I need to change the setting in coveralls so tiny drops don't matter.

I'll just bump the coverage a little. There's some easy coverage to be had in there :)

@jni there we go!

@starcalibre but don't forget the comments yet to be addressed above (rename fn_has_ext to has_extension)! That's what I was talking about when I said there was still work to do here!

@jni Ack, of course.. OK, take 2. :P

Boom, done!


java16运行乱码

java version "16.0.2"
大佬，我不太懂java哈，配置了javafx环境，环境如下
export PATH_TO_FX=/Library/Java/JavaVirtualMachines/jdk-16.0.2.jdk/javafx-sdk-11.0.2/lib

运行 shiro_attack-4.5-SNAPSHOT-all.jar 出现乱码，还有报错提示信息如下
java --module-path $PATH_TO_FX --add-modules=javafx.controls --add-modules=javafx.fxml --add-modules=javafx.base --add-modules=javafx.graphics --add-modules=javafx.web -jar shiro_attack-4.5-SNAPSHOT-all.jar

2021-12-16 15:49:41.187 java[89289:289238] CoreText note: Client requested name ".SFNS-Regular", it will get Times-Roman rather than the intended font. All system UI font access should be through proper APIs such as CTFontCreateUIFontForLanguage() or +[NSFont systemFontOfSize:].
2021-12-16 15:49:41.187 java[89289:289238] CoreText note: Set a breakpoint on CTFontLogSystemFontNameRequest to debug.
2021-12-16 15:49:41.199 java[89289:289238] CoreText note: Client requested name ".SFNS-Regular", it will get Times-Roman rather than the intended font. All system UI font access should be through proper APIs such as CTFontCreateUIFontForLanguage() or +[NSFont systemFontOfSize:].
2021-12-16 15:49:41.601 java[89289:289278] CoreText note: Client requested name ".SFNS-Regular", it will get Times-Roman rather than the intended font. All system UI font access should be through proper APIs such as CTFontCreateUIFontForLanguage() or +[NSFont systemFontOfSize:].
2021-12-16 15:49:41.657 java[89289:289278] CoreText note: Client requested name ".SFNS-Regular", it will get Times-Roman rather than the intended font. All system UI font access should be through proper APIs such as CTFontCreateUIFontForLanguage() or +[NSFont systemFontOfSize:].
2021-12-16 15:49:41.657 java[89289:289278] CoreText note: Client requested name ".SFNS-Regular", it will get Times-Roman rather than the intended font. All system UI font access should be through proper APIs such as CTFontCreateUIFontForLanguage() or +[NSFont systemFontOfSize:].
2021-12-16 15:49:41.726 java[89289:289278] CoreText note: Client requested name ".SFNS-Regular", it will get Times-Roman rather than the intended font. All system UI font access should be through proper APIs such as CTFontCreateUIFontForLanguage() or +[NSFont systemFontOfSize:].

使用JDK8 运行

使用JDK8运行

jdk8没问题


Eligibility Questions

Eligibility screen discovery (see Service Blueprint) revealed that eligibility will be assessed based up info in UBF form & templates


ImportError: cannot import name 'web_screenshot' from 'pywhatkit.misc'

Description
ImportError: cannot import name 'web_screenshot' from 'pywhatkit.misc'
Screenshots

Version Information

OS: [Windows 10]
Browser: [Google Chrome]
Python: [3.10.6]
PyWhatKit: [5.3]

error solved in version 5.4

@FabinhoSchramm web_screenshot has been removed


Consider open sourcing the Player SDK

At the moment, the player SDK feel very akward to use, especially in modern Js frameworks.
For now I gave up using this library and I'll stick to the iframe version of the player for now that suits our basic needs. Well, in the end, I'm still using this library, but in an iframe, which I have no control over.
Here are the pros of open-sourcing this library :

Can be published on npm and install as a normal project dependency
Can be built and shipped with the Js bundle of the project
Can be adapted to suit any Js environments (see my foot note on using this with React)
Trust. I don't trust a minified Js library.
The community can do many things for you -> reducing cost
...

There are plenty of other reasons to do so, but I'll let you, other fellow developers do the rest.
As a foot note, here's my twisted take of using this SDK in React :
import { useCallback } from "react";

export default function Video({ id }) {
  const playerRef = useCallback((node) => {
    if (node !== null) {
      try {
        const myNode = document.getElementById(node.id);
        myNode.textContent = "";
        apiVideoSdk.create(`#${node.id}`, {
          id,
        });
      } catch (e) {
        console.log(
          "You must activate third party cookies."
        );
      }
    }
  }, []);

  return (
      <div id="player" ref={playerRef} />
  );
}

Feel free to contact me if you want to discuss this further.

Hi @lutangar :)
The player sdk has been rewritten using Typescript and is now open source.

Hi @lutangar :)
The player sdk has been rewritten using Typescript and is now open source.

Good to know! Couldn't find it though, could you add a link to the source?

Good to know! Couldn't find it though, could you add a link to the source?

@lutangar https://github.com/apivideo/player-sdk/blob/master/index.ts

@lutangar https://github.com/apivideo/player-sdk/blob/master/index.ts

@lutangar https://github.com/apivideo/player-sdk/blob/master/index.ts

Alright, great thanks :+1:

@lutangar https://github.com/apivideo/player-sdk/blob/master/index.ts

Alright, great thanks :+1:


Add support for pxefilename for pools.

This adds the possibility to override the filename for pxe booting per pool.
The filename is not an option, so use of $options is not possible. While $raw_append could be used to achieve the same goal, it's nice to have a separate argument for it. This also mirrors the argument list for init.pp more closely.

Merged, thanks @yaunj.

Thanks for merging. This will make things easier for us when we update the module later.


Changelog

Had some problems switching from v3 to v4 because didn't noticed all the api changes in code. Would be nice to have at least sort changelog message when updating the package version.

This is a nice idea, thank you very much @kirillku.
I added the CHANGELOG.


ODS-5186 - Update Dependencies to include Updates operation and ensure resources are output in correct operational order

Previously everything was output in the correct order, but with only a "Create" operation and no "Updates". Now this fixes it so that the graph results will include both possible operations, and then when needed, split out "Create" and "Update" based on any resources that have to be created before an update can be performed. With these changes the following order of resources should be observed in the output from this endpoint:

Student Create / Parent Create
StudentSchoolAssociation Create/Update
StudentParentAssociation Create/Update
Student Update / Parent Update
StaffEducationOrganizationEmploymentAssociations Create/Update
StaffEducationOrganizationAssignmentAssociations Create/Update

For additional information about this endpoint, please see this Tech Docs page.

Not sure if this is working still... I guess this might still be open. Just checked the API, it only lists Create order!
https://api.ed-fi.org/v5.3/api/metadata/data/v3/dependencies


Textarea: validation is not happened correctly when maximum value is set even ValidateOnInitialRender

Textarea: validation is not happened correctly when maximum value is set even ValidateOnInitialRender
I have text area with maxLength="6". But when we have longer text more than 6 in value(in code) or set value to the field with longer text then still all values are getting displayed . validation is happened only when clicking on text area field and backspace is pressed.
Expected: 1. Either only max length of 6 letter should display.
OR
2. validation should throw when we try to type some value in text area and tab out.
Please find the attached sample project

Hi, I'm not seeing any attached example.
Can you make one on the lit.dev playground?

Hi,
Please find the attached html input for text area if we set max length=6 and set the initial value the validation is not happening on inital render even if we set  ValidateOnInitialRender.
So if we have empty value and type in some text validation is correct.
<mwc-text-area
label="Relation Code"
helper="Helper text"
helperPersistent
type="number"
maxLength="6"
charCounter
value="testing in new text area"

Thanks for filing this bug. This is the native behavior for HTMLInput elements: value set programmatically is not restricted by maxlength (see https://jsbin.com/xupadusivo/edit?html,output) while user input is. For that reason, I'm going to close this. Let me know if you have any questions!


New (soft and little) bug in the Character Sheet :).

Hello :) ! It's me again ;) !
Thank's for the new version ; the problem of "Energy Shield Recharge per Second" and "Life Regeneration per Second", in the Character Sheet, are succesful solved :) !!!
But unfortunately, another bug is appeared : the "Estimated Physical Damage reduction" is wrong (it was OK yesterday, before the last update).

I join the passive skill here, and the equipement :

Thanks for reading ;) !
Elgin

This one I won't be fixed. As a community there is a lot of information we just don't have. Basically we are missing average monster damage, so I updated the damage formula to the last one we were told.

I understand :).
Thank's for reading :).


Refactor SKY130 Digital Cells to be in a Seperate Namespace

Including the digital cells alongside the primitives has a few annoying side-effects:

Long module loading time
Confusion between primitives and cells
Unnecessarily large/difficult to navigate PDK data files
Very long and redundant cell names

It'd be better to have standard cell libraries in different namespaces for these reasons and make it something as slick as #114, where we can just use:
from gf180 import digital7track as d
simple_and_gate = d.and2_1

Instead of:
from sky130 import modules as s
# Now you wait for 5 whole seconds
simple_and_gate = s.sky130_fd_sc_hdll__and2_1

Sounds great, totally agree.

So getting into the details of this issue, it seems that with the SimpleNamespace approach:
from sky130.digital import high_density as hd

Is actually logically equivalent to:
import sky130.digital as _tmp
hd = _tmp.high_density
del _tmp

Which is... horrible. This stems from the fact that SimpleNamespace is a variable type rather than a submodule. To really get fine-grained control of imports and ensure that we're only importing the cells we're interested in, we'd have to split up the files into individual submodules which are populated without the use of SimpleNamespace.
I've tried this out and works pretty well on my local machine but the call changes to:
import sky130.digital.high_density as hd

And teasing apart the primitives to mirror this change, would require effort that may not be worth it... hey, you might have 4s of unnecessary overhead when loading in your digital cells, but at least you don't have that overhead when loading in the base PDK module and the primitives (the core functionality of Hdl21).
So, going to leave this open for now, change the name to reflect the state of affairs, and mark it a "good first issue" for newcomers.

Okay was just being lazy here this got resolved in latest push of #114


Can't connect more than one array to a Node signal

Tested versions
4.3.stable
System information
Windows 11 - Godot 4.3.stable - Forward+
Issue description
I want to connect a signal of a node to the erase function of many arrays, when I try to do that an error appears telling me that the signal is already connected
Steps to reproduce
play the MRP and you'll see, the project is very simple, it has only 2 scripts
Minimal reproduction project (MRP)
minimal-reproduction-project.zip

Maybe using  "CONNECT_REFERENCE_COUNTED" flag will help:
parent.died.connect(arr_2.erase.bind(parent), Object.CONNECT_REFERENCE_COUNTED)

But that's because the signal "died" is already connected to the array method, a signal can only be connect to a method once, that's why you are getting the error, if you want you "died" signal to call the method twice, each time with a different arr I recommend you this code:
func _ready(): 	parent.died.connect(func () -> void: 		arr.erase(parent) 		arr_2.erase(parent) 		)
I don't think this is a bug...

Not a bug indeed, when connecting the bound arguments are ignored and only the method and object are considered, so you can't connect the same method of the same object to the same signal without the flag
Closing as not a bug, please ask for more support in the other community channels


load the plugin only on dev mode

If you run composer with the no-dev switch on your production server like
composer install --no-dev --no-interaction --optimize-autoloader
You will run into this error:
Exception: Plugin CakephpFixtureFactories could not be found.
In [/xx/vendor/cakephp/cakephp/src/Core/PluginCollection.php, line 143]

This is because the plugin is installed as dev requirement. So in your Application.php file you should load accordingly, for example like this.
protected function bootstrapCli(): void
{
    // some code here
    if (Configure::read('debug')) {
        $this->addPlugin('CakephpFixtureFactories');
    }
}

Maybe this is obvious for others, but I did not understand at the first glance why my deploy script fails.
So maybe it can be mentioned in the docs.

Yes, good point! I'll amend the docs.

Done.


how to predict using the model saved in seq2seq example?

Hi,
I have run the model in lstm_seq2seq.py, and predict the sequence the with the mdoel online. Now I want to predict the sequence using the model saved in the disk, how can I implement with it?
I have tried the load_model function, but find that the result is not true(just like I have not trained)
Any suggestion would be  appreciated
Thanks,
Zhang

The problem has been solved and save both the encoder_model and decoder_model would be work(not just save the model trained).

but it seems like the state is not saved and the model is not returning anything good @scut-salmon


[iOS][Adaptive Cards)] [Flow Bot Adaptive Card with button isn't getting displayed in Teams for iOS]

Platform
What platform is your issue or question related to? (Delete other platforms).

iOS

Author or host
Author, sending cards from Flow to Teams using the "post your own adaptive card as the Flow bot to a channel" step in my flow.
Version of SDK
Tried formatting my message in adaptive card version 1.0 and 1.2, but that didn't make any difference.
Details
Please have a look at the attached screenshots. The pictures are of the same message sent by the Flow bot. As you can see, on desktop, the message with the "approve" button looks like you would expect. On iOS, the message isn't displayed at all. You just see the Flow bot icon and the timestamp.

Please look into this, as we have people working on iPads who need to interact with these messages.

@menbren do you happen to have the sample JSON that is attempted to be rendered here at all? Thanks.. @jwoo-msft FYI.

In addition, I have another message in the same flow (for when a quote is approved). This one is showing up on iOS, but the button isn't working. It remains "pressed" for about two seconds, but then nothing happens. It works as expected on desktop. Here's the JSON for that:
{     "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",     "type": "AdaptiveCard",     "version": "1.2",     "body": [         {             "type": "ColumnSet",             "columns": [                 {                     "type": "Column",                     "width": 2,                     "items": [                         {                             "type": "TextBlock",                             "text": "Quote ready to be send"                         },                         {                             "type": "TextBlock",                             "text": "@{variables('quote_number')} Approved",                             "weight": "Bolder",                             "size": "ExtraLarge",                             "spacing": "None",                             "separator": true,                             "horizontalAlignment": "Left"                         },                         {                             "type": "RichTextBlock",                             "inlines": [                                 {                                     "type": "TextRun",                                     "text": "@{variables('quote_number')} ",                                     "weight": "Bolder"                                 },                                 {                                     "type": "TextRun",                                     "text": "for "                                 },                                 {                                     "type": "TextRun",                                     "text": "@{variables('account_name')} ",                                     "weight": "Bolder"                                 },                                 {                                     "type": "TextRun",                                     "text": "with a total of "                                 },                                 {                                     "type": "TextRun",                                     "text": "@{variables('currency')} @{variables('amount')} ",                                     "weight": "Bolder"                                 },                                 {                                     "type": "TextRun",                                     "text": "is approved and ready to be send."                                 }                             ]                         }                     ]                 }             ]         }     ], "actions": [{         "type": "Action.OpenUrl",         "title": "Send Quote",         "url": "Removed on purpose",         "style": "positive"     }] }

@menbren,
It's because of malformed json for actions.

selectAction is missing url property
Action.OpenUrl contains malformed url, you could just leave as "", instead of "Removed on purpose"

Desktop version is based on JS, so it's more tolerant.

Thanks, will give this a try. The formatting of the cards I used are straight from adaptivecard.io, so that might be worth looking into.

Thanks, will give this a try. The formatting of the cards I used are straight from adaptivecard.io, so that might be worth looking into.

Yes, that is one of the inconsistency issues we are looking into.

@nesalang, do we have a bug tracking the inconsistency on the JS side we can reference here? Thanks

@jwoo-msft, got the message card to work, but upon clicking on the button in the iOS Teams app, my URL gets escaped automatically, so the "#" in the url turns into "%23", making the URL useless. (On desktop the URL isn't escaped)

cc @shalinijoshi19

@menbren, can you share the url you used?

https://app.pandadoc.com/a/#/documents/4Nw3YxHpiNeypvzk2CzVR2 turned into https://app.pandadoc.com/a/%23/documents/4Nw3YxHpiNeypvzk2CzVR2

@jwoo-msft, what's the status here? Thanks :)

@jwoo-msft, what's the status here? Thanks :)

@menbren, the conversion doesn't happen at iOS SDK.
@shalinijoshi19, can you help me forward this issue to Teams mobile?
- (void)didFetchUserResponses:(ACOAdaptiveCard *)card action:(ACOBaseActionElement *)action
{
    if (action.type == ACROpenUrl) {
        NSURL *url = [NSURL URLWithString:[action url]];
       /// [action url] returned url in string unchanged.

Payload used
{
    "type": "AdaptiveCard",
    "version": "1.0",
    "body": [],
    "actions": [
        {
            "type": "Action.OpenUrl",
            "title": "Action.OpenUrl",
            "url": "https://app.pandadoc.com/a/#/documents/4Nw3YxHpiNeypvzk2CzVR2 "
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json"
}

This issue has been copied over to Microsoft Teams repo here: https://github.com/MicrosoftDocs/msteams-docs/issues/4738


TODO:  _model_module_version and _view_module_version should be anywidget.__version__

          TODO: Right now the anywidget frontend code and the python package are published together with the same version. So `_model_module_version` and `_view_module_version` should be `anywidget.__version__`

Originally posted by @manzt in https://github.com/manzt/anywidget/pull/46#discussion_r1096560803

oh I actually think this this got added to _descriptor.py already!


Minimum Viable Product for tree-tremble

After a Hadrian's-Wall-long discussion in #232, it looks that it will be best to postpone the Great Merge (the tree-tremble milestone) until Hadrian meets the Minimum Viable Product requirements.
Let us agree on what this MVP is. The current list includes the following (I will keep updating it as we go):

[ ] Binary and source distributions, #219. Estimated effort: 2-5 weeks (I don't know much about this).
[x] Implement dynamic way, #4. Estimated effort: 1-2 weeks.
[x] Support most standard build flavours, #188, #268. Estimated effort: 1 week.
[x] Cross-compilation, #177. Estimated effort: 3-5 weeks (I don't know much about this).

Please let me know if the above list needs any changes.
The list of current limitations is here: https://github.com/snowleopard/hadrian#current-limitations.
Note, the above effort estimates are based on my abilities and availability. I'm sure other people could do it faster, and if they volunteer to help I will amend the estimates.
Items removed from the list:

Add more documentation, #55, #56. Estimated effort: 2-3 weeks. Removed because we have a paper about overall principles and can add more comments on demand after merging.

I would add cross-compilation to the list (probably not hugely difficult, but if it shows up weaknesses in the overall architecture you'll want to get them addressed before the merge).
I would remove the documentation requirement. The overall "principles" are covered in the paper we wrote. The details are covered in the code. I'm sure additional docs will help everyone, but having them produced on-demand even after merging seems fine. The old system required docs precisely because it wasn't compositional, the new one is just a bunch of disjointed stuff, none of which is too confusing in isolation.

@ndmitchell Thanks, I agree. I've amended the list.

This goes very slowly, but now we've got source distribution rules, #219.

Posting here as requested on ghc-devs: A really useful feature of the build system today is that I can make 2 in the ghc subdirectory and get a very fast rebuild. In particular, no dependency-checking is done (which takes several seconds). I would be quite disappointed to see this (admittedly unsafe) feature be dropped.
It is also convenient to tell the build system to target just one specific file (e.g., libraries/base/dist-build/.../Data/Type/Equality.o), which I've done when building that file with stage-1 fails and I'm debugging stage-1. I could probably live without this feature though.

Thanks @goldfirere!
Freezing Stage1 is a popular feature and has already been requested -- see #250. We do intend to provide an equivalent in Hadrian.
Rebuilding a single file is already possible by specifying its name as the build target, i.e. hadrian target, but at the moment this will also trigger rebuilds in Stage1.

This isn't just about freezing stage 1. It's about re-building just the compiler (no libraries) and skipping the time-consuming dependency analysis.

@goldfirere I see, thanks. If I understand correctly, the freeze feature should be flexible enough to freeze different sets of build artefacts: whole stages, packages, etc.
One simple way to support such custom freezing is via predicates, e.g. having a user setting freeze:
freeze :: Predicate
freeze = return False -- default setting: freeze nothing

freezeStage1 :: Predicate
freezeStage1 = stage0 -- freeze all artefacts built by the bootstrapping (Stage0) compiler

freezeAllButCompiler :: Predicate
freezeAllButCompiler = not <$> package compiler -- freeze all packages expect compiler

These predicates can be composed as explained here:
https://github.com/snowleopard/hadrian/blob/master/doc/user-settings.md#verbose-command-lines

make 2 is a combination of freezing stage 1 for the duration of this build and building a single target (the compiler).  @snowleopard we need to be able to specify what to freeze from the command line, in addition to having a user setting.

@simonmar How critical is the ability to specify such settings via the command line? Isn't the ability to select a user build flavour from the command line sufficient?

Hadrian has been merged, so I think we can close this issue.


feat: support korean

Support Korea

Thank you for your pull request! :heart:

I had the same issue with the link below.
https://stackoverflow.com/questions/73509984/eslint-typeerror-this-liboptions-parse-is-not-a-function

I don't think that specific ide issues should change to codebase this way.
Published as v0.7.0 :tada:


[DS-3992] Added multiple custom exceptions and removed most direct uses of RuntimeException.

This is a work in progress.
The best practice for exceptions is to refrain from using "RuntimeException" directly, and instead look to use Spring Data Rest Exceptions (like "DataAccessException") which extend RuntimeException.  If we find we need a new exception type, we should create it.
For the most part I have created new custom exceptions.
In most cases, how to apply a Spring NestedRuntimeException and specifically a DataAccessException is opaque (to me). So advice is welcome.
Here is how I have done the mapping.  In cases where an SQLException is thrown while attempting to retrieve items (e.g. findOne and findAll) I have thrown DataRetrievalFailureExceptions. https://docs.spring.io/spring/docs/current/javadoc-api/org/springframework/dao/DataRetrievalFailureException.html
Since the cause of other SQLExections is less clear, I created a custom exception that extends DataAccessException:  https://github.com/DSpace/DSpace/blob/214eefb8470cef3620d9f2836730a79d9abbe295/dspace-spring-rest/src/main/java/org/dspace/app/rest/exception/RESTSQLException.java#L17
Spring Data has a UncategorizedSQLException with a single 3-parameter constructor that didn't seem suited to our needs, but perhaps I am wrong and this is an alternative. https://docs.spring.io/spring/docs/current/javadoc-api/org/springframework/jdbc/UncategorizedSQLException.html
For other exceptions, I created custom exceptions that extend RuntimeException.
I made a lot of changes in exception catch blocks to throw the new exceptions. This too is a work in progress since we need to rebase with master after other PRs have been merged.

Still a work in progress.  The last commit includes the following changes:

Implemented more custom exceptions.
Where it makes sense, I am rethrowing the custom exceptions in DSpaceRestRepository to reduce the number of try/catch blocks that need to be maintained elsewhere in the code.
Added only a few calls to Spring NextedRuntimeExceptions (so far only using DataRetrieveException as described earlier).
Updated controller advice to handle the new custom exceptions.

Quick note. I see that https://jira.duraspace.org/browse/DS-3825 is a related issue claimed by @lap82.
I saw there were unhandled exceptions in the submission workflow (and might have started work on a few of these). I'm happy to wait on this PR until that work is completed, then see what is left for me to do.


Feature Request: close button for destroy suggestion list container

I think there is no close button for destroy suggestion list container. Have any option to close manually?

I think there is no close button for destroy suggestion list container. Have any option to close manually?

@xeronce Thanks for your comment. I will update as soon as possible.

@xeronce

You can activate/deactivated the suggestion box by changing the value of the variable activateSuggestionBox.
Please update to the latest version 0.0.2 at pub.dev to use this feature.


#582 VS 2013 Crash NullReferenceException

it was sometimes possible for one of the children in moduleTree to be
null. Check for null before mod.Name string comparison.

fix #582

I couldn't actually reproduce this issue, but as described in https://github.com/Microsoft/nodejstools/issues/582#issuecomment-156053390, this is the only way that a NRE could be thrown with the provided call stack.
Still haven't conclusively narrowed down how the tree could be null, but I believe it may be happening here:
https://github.com/Microsoft/nodejstools/blob/master/Nodejs/Product/Analysis/Analysis/ProjectEntry.cs#L106

@DinoV Any chance you could review this? It's not entirely clear to me in what scenarios mod might be null here

This was the same defensive code I was considering adding, but still couldn't figure out why this would ever occur :-/
Barring any deeper understanding of what is causing this, looks like the right change to make to me :+1:

I suspect it is actually a race condition, especially given that it's reported to be intermittent.  ModuleTree.Children is a non-thread safe dictionary, and AddChildVisibilitiesExcludingNodeModules is accessing it w/o holding the ModuleTable lock.  _table.RequireModule will acquire the lock, do the lookup, and release it.  Then we access the children in a non-thread safe manner, so we can come back with a corrupted dictionary that has null values (or get other strange behavior).
It probably makes sense to have a ModuleTable.GetChildrenExcludingNodeModules which does the RequireModule + GetChildrenExcludingNodeModules with the lock held the entire time.


[RTM] Minor changes to correct anatomical workflow.

so far: update template being used by robust normilization in anatomical workflow

From local tests the update to the template being used resolved issues seen on some datasets


Error Handling in ViewModel

This is just a quick question:
I have noticed that in the GroceryList, the GCYViewModel class inherits from RVMViewModel and adds the errors signal.
Is there a specific reason why this kind of error handling is not part of RVMViewModel directly?
Maybe its because the assumption, that all errors are handled equally (e.g. just display the error message on the screen) is too specific and its best practice to just subscribe to the error event of each Action of a viewModel individually and handle the error of that Action individually?

The primary reason is that RVMViewModel does not itself generate any errors, so it wouldn't make much sense to have RVMViewModel.errors.
It's easier for subclasses (perhaps even a project-specific base class) to declare themselves.

Ok, thanks for the answer.

perhaps even a project-specific base class

Thats what I've been doing but the error handling was the only thing added to my own base class, so I was wondering why it was not part of RVMViewModel already.


 That won't kill any processes that have been disowned (i.e., removed from the job table).

Doesnt seem relevant to leave open as issue due to its low priority and the fact that the current method is working!


Adding interoperator helm chart package

Based on #700

Added helm chart to helm-charts/sources
Created tgz for latest interoperator release(without multcluster)
Created tgz for interoperator with multi cluster
version 0.3.0 : Without multi cluster support
version 0.4.0 : With multi cluster support
Version 0.4.0 uses image tags 0.4.0 which is not yet created

@vivekzhere I am merging the PR to enable testing.


[0.29.x] Added PyUnicode_AsUTF8AndSize to cpython imports

Co-authored-by: William Ayd<EMAIL_ADDRESS>This is backport of #5163 to 0.29.x branch.

@scoder does it make sense to include to the next release? It was recently merged to master branch. If you think it should not be backported feel free to close the PR.

It's usually ok to backport library extensions. When users start to employ them, they can depend on the given Cython version.


Exclude some files from triggering Travis build

This issue suggests there's a way to do this, though it is not supported directly by Travis.

Instead just add "[skip ci]" to the commit message.


ERR_OUT_OF_RANGE

Hi,
Subflow is running 99% fine with some errors as below:
30 Jul 14:38:08 - [error] [function:check frame] RangeError [ERR_OUT_OF_RANGE]: The value of "offset" is out of range. It must be >= 0 and <= 70. Received 83
30 Jul 14:54:04 - [error] [function:check frame] RangeError [ERR_OUT_OF_RANGE]: The value of "offset" is out of range. It must be >= 0 and <= 77. Received 83
30 Jul 14:57:05 - [error] [function:check frame] RangeError [ERR_OUT_OF_RANGE]: The value of "offset" is out of range. It must be >= 0 and <= 18. Received 83
30 Jul 14:58:04 - [error] [function:check frame] RangeError [ERR_OUT_OF_RANGE]: The value of "offset" is out of range. It must be >= 0 and <= 56. Received 83
30 Jul 14:58:15 - [error] [function:check frame] RangeError [ERR_OUT_OF_RANGE]: The value of "offset" is out of range. It must be >= 0 and <= 26. Received 83
30 Jul 15:06:45 - [error] [function:check frame] RangeError [ERR_OUT_OF_RANGE]: The value of "offset" is out of range. It must be >= 0 and <= 74. Received 83
30 Jul 15:07:15 - [error] [function:check frame] RangeError [ERR_OUT_OF_RANGE]: The value of "offset" is out of range. It must be >= 0 and <= 28. Received 83
30 Jul 15:08:01 - [error] [function:check frame] RangeError [ERR_OUT_OF_RANGE]: The value of "offset" is out of range. It must be >= 0 and <= 51. Received 83
30 Jul 15:09:31 - [error] [function:check frame] RangeError [ERR_OUT_OF_RANGE]: The value of "offset" is out of range. It must be >= 0 and <= 42. Received 83
Anything can be done with that ?

check this subflow from dev-issue_7 branch.
I added too short data handling and some node red warnings logs.

subflow imported. Will let You know how does it work tomorrow - thx :)

try it now.

Tomorrow. We have got night in Poland right now 🤪

31 Jul 08:47:45 - [warn] [function:check frame] data length OK(112)
31 Jul 08:48:15 - [warn] [function:check frame] data length OK(112)
31 Jul 08:48:45 - [warn] [function:check frame] data length OK(112)
31 Jul 08:49:29 - [warn] [function:check frame] data longer than expected
31 Jul 08:49:45 - [warn] [function:check frame] data length OK(112)
31 Jul 08:50:15 - [warn] [function:check frame] data length OK(112)
31 Jul 08:50:50 - [warn] [function:check frame] data length OK(112)
31 Jul 08:51:16 - [warn] [function:check frame] data length OK(112)
31 Jul 08:51:45 - [warn] [function:check frame] data length OK(112)
31 Jul 08:52:15 - [warn] [function:check frame] data length OK(112)
31 Jul 08:52:45 - [warn] [function:check frame] data length OK(112)
31 Jul 08:53:25 - [warn] [function:check frame] data length OK(112)
31 Jul 08:53:55 - [warn] [function:check frame] data length OK(112)
31 Jul 08:54:39 - [warn] [function:check frame] data length OK(112)
31 Jul 08:55:05 - [warn] [function:check frame] data longer than expected
31 Jul 08:55:35 - [warn] [function:check frame] data longer than expected
31 Jul 08:56:06 - [warn] [function:check frame] data length OK(112)
31 Jul 08:56:35 - [warn] [function:check frame] data longer than expected
31 Jul 08:57:17 - [warn] [function:check frame] data length OK(112)
31 Jul 08:57:35 - [warn] [function:check frame] data longer than expected
31 Jul 08:58:05 - [warn] [function:check frame] data length OK(32)
31 Jul 08:58:05 - [warn] [function:check frame] modbus frame too short
31 Jul 08:58:35 - [warn] [function:check frame] data longer than expected
31 Jul 08:59:05 - [warn] [function:check frame] data length OK(112)
31 Jul 08:59:49 - [warn] [function:check frame] data length OK(112)
31 Jul 09:00:06 - [warn] [function:check frame] data longer than expected
31 Jul 09:00:35 - [warn] [function:check frame] data longer than expected
31 Jul 09:01:06 - [warn] [function:check frame] data length OK(112)
31 Jul 09:01:35 - [warn] [function:check frame] data length OK(112)
31 Jul 09:02:21 - [warn] [function:check frame] data length OK(112)
31 Jul 09:02:56 - [warn] [function:check frame] data length OK(112)
31 Jul 09:03:37 - [warn] [function:check frame] data longer than expected
31 Jul 09:03:55 - [warn] [function:check frame] data length OK(112)
31 Jul 09:04:25 - [warn] [function:check frame] data length OK(112)
31 Jul 09:04:57 - [warn] [function:check frame] data length OK(112)
31 Jul 09:05:25 - [warn] [function:check frame] data length OK(112)
31 Jul 09:06:16 - [warn] [function:check frame] data length OK(112)
31 Jul 09:06:25 - [warn] [function:check frame] data length OK(50)
31 Jul 09:06:25 - [warn] [function:check frame] wrong modbus frame length
31 Jul 09:06:55 - [warn] [function:check frame] data length OK(112)
31 Jul 09:07:32 - [warn] [function:check frame] data length OK(112)
31 Jul 09:07:55 - [warn] [function:check frame] data length OK(112)
31 Jul 09:08:25 - [warn] [function:check frame] data length OK(112)
31 Jul 09:08:56 - [warn] [function:check frame] data length OK(112)
31 Jul 09:09:25 - [warn] [function:check frame] data length OK(112)
31 Jul 09:10:10 - [warn] [function:check frame] data length OK(112)
31 Jul 09:10:25 - [warn] [function:check frame] data length OK(112)
31 Jul 09:10:56 - [warn] [function:check frame] data length OK(112)
31 Jul 09:11:26 - [warn] [function:check frame] data length OK(112)
31 Jul 09:11:55 - [warn] [function:check frame] data length OK(112)
31 Jul 09:12:42 - [warn] [function:check frame] data length OK(94)
31 Jul 09:12:42 - [warn] [function:check frame] wrong modbus frame length
31 Jul 09:12:55 - [warn] [function:check frame] data length OK(112)
31 Jul 09:13:25 - [warn] [function:check frame] data length OK(112)
at least no error message.

You can go inside subflow  to node check frame and comment out or remove line 27
node.warn(`data length OK(${frame.length})`);

to reduce warnings to only errors.
112 bytes is normal response from inverter, longer frames are probably 2 frames send together, but i just ignore second.
I will probably make some additional edits and update subflow.

You can go inside subflow to node check frame and comment out or remove line 27
node.warn(`data length OK(${frame.length})`);

to reduce warnings to only errors. 112 bytes is normal response from inverter, longer frames are probably 2 frames send together, but i just ignore second. I will probably make some additional edits and update subflow.

I'll be happy to test it.


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.telegram

[x] [E506] "admin/jsonCustom.json" not found, but custom support is declared

I have also found warnings that may be fixed if possible.

[x] [W801] .npmignore not found

Thanks,
your automatic adapter checker.

Thanks, that all bugs have been fixed.


Commit when CG closed

It would be nice if zookeeperOffsetManager.commitOffsets() got called from ConsumerGroup.Close().
Right now stuff that is 'markedForCommit' after the last commit and before CG close is dropped.
Maybe also provide a Commit() method in the OffsetManager interface to be able to force commits from CG/clients.

During a normal shutdown, there should be no difference between marked for commit, and committed, because the offsets are committed for every partition consumer as they wind down.

Closing this - as this should already work. Also #72 improves how we keep track of offsets.


[DOCS] Add guide/recipe for target transforms

Summary

We would like to have a guide on how to use target transforms and also how to define custom target transforms (for example BoxCox).
Motivation

Currently, documentation for target transformations is only available in the end-to-end walkthrough tutorial.
References

Here's an example.

@narencastellon


Put the path to node-pre-gyp in quotes in install.js

Hi,
When I was installing nodegit, I found that there was compilation issues (#950). On further inspection, I found out that the building was taking place due to it failing to install a S3 binary. This was due to my user folder having a space in.
To fix this, I have placed the command in quotes.
Current Issues:

install.js now assumes that the install is a local install, even though that part of code was untouched.

The linting script appears to be failing (appveyor build)

Do "\"" instead of '"'

Ok

Test status:
It appears to be one test is failing. As I did not touch the tests, I believe this is because of how the tests have been written, or an incomplete/buggy test. Please correct me if I am wrong.

@Gum-Joe you are correct, my server is down which is causing the failure.

@Gum-Joe can you rebase this on top of master please?

:@johnhaley81 Rebasing now

Just rebased. Now pull requests will not merge

Yeah, it pulled in too many commits. You just need https://github.com/nodegit/nodegit/pull/951/commits/612540e02965503f3e91aced8c567374a12fcd3a and  https://github.com/nodegit/nodegit/pull/951/commits/f451b689fe9f3a74e072178a2187d675424ea1df

How can I fix this?

Oh... Now GitHub says it can merge.

It might be better to just merge locally @johnhaley

Merged manually

Thanks @Gum-Joe!

Your welcome @johnhaley81


I'm not sure what this is simplifying as it just seems to remove more granular component of the bzl_srcs pattern. If we have just a top level bzl_srcs, then we can only perform rules on all *.bzl files.
For something like stardoc, this works fine as it operates on the whole tree, but if we wish to work more granular, we would need to rollback these changes selectively.

Yeah, I can't think of why we'd want that granularity though. Did you have a use-case in mind?
Having so many targets and packages is confusing - that's what I'm simplifying :)

I don't have an exact use-case at the moment, as I've just started poking around the idea of having //tools that allows us to perform modifications / generations of the source. So I'll just perform a dump of some things I have been thinking in that area:

Exposing our C# Starlark AST library as a rule a la @d2l_rules_csharp//libs:CSharp.Skylark
Running buildifier as a bazel target
Running clang-format as a bazel target
Generating the framework build files using our internal tool (bazel run //tools/frameworks:generator
Generating the provider.bzl from a set of yml files declaring each of the frameworks
Bazel targets within the rules (non-public) that bundle up parts of the rules (zip/etc)
Windows specific test runs (e.g. net472 runs)
Exposing internal rules of rules_csharp to related packages (e.g. //tools, ``//lib, //docs`)
Exposing the wrappers / etc allowing for testing externally (e.g. run bat tests on a bash wrapper file)

The current model is a bit of 'future-planning' for 'what-if' futures, but since the overhead of adding new packages involves changing a bunch of file references, I'd prefer to keep it with this pattern
note: the use case for bazel_tools is a bit different from us, as we have a public API, and they have an open API.

Ok, then let's keep it simple for now.
I'm not sure how all these extra packages and targets would help with any of those ideas.

I'm merging this because I want to avoid a merge conflict.


added comet logger

Before submitting

Was this discussed/approved via a Github issue? NO (but asked for in the docs)
Did you read the contributor guideline? YES
Did you make sure to update the docs?   YES
Did you write any new necessary tests?  NO

What does this PR do?
Adds a new logger for interacting with comet
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

fixed tests. can you rebase master?

fixed tests. can you rebase master?

No problem, will do inmediately


Lint of line

Description
Investigate why isinstanceof can't be used here:
https://github.com/Ingenjorsarbete-For-Klimatet/ifk-smhi/blob/f988757507c4860d58b8cbc4d2ac8da0a75d35e5/tests/unit/test_unit_metobs.py#L397

My bad didn't add not.


request help: failed to receive RPC_HTTP_REQ_CALL: closed

Issue description
There will be exceptions from time to time :  failed to receive RPC_HTTP_REQ_CALL: closed .

What is the reason for this? How to solve?
Environment

Docker deployment;  An apisix-doker image built by myself based on centos7 + apisix ；
Upstream uses nacos;
Run a Java plugin;

open the java runner log for more information, here: https://github.com/apache/apisix-java-plugin-runner/blob/cbd3b3a6ad536df5d1ea71336c4a319a28c0622d/runner-core/src/main/resources/application.properties#L19
BTW, use text instead of images.

logging.level.root=DEBUG   is OPEN 。
errorinfo :
2022/02/16 09:59:42 [error] 47#47: *116549586 [lua] init.lua:689: phase_func(): failed to receive RPC_HTTP_REQ_CALL: closed, client: <IP_ADDRESS>, server: _, request: "POST /byd-intergral/in
tergralRule/getIntergralRulePageList HTTP/1.1", host: "<IP_ADDRESS>:9080"2022/02/16 09:59:42 [warn] 47#47: *116549586 [lua] plugin.lua:722: run_plugin(): ext-plugin-pre-req exits with http status code 503, client: <IP_ADDRESS>, server: _, request: "POST /byd-inte
rgral/intergralRule/getIntergralRulePageList HTTP/1.1", host: "<IP_ADDRESS>:9080"

我的运行环境是：
1、自己基于centos 和 apisix 2.11.0 构建的Docker镜像；
2、启动的时候 把java插件jar包 ，挂载到容器里面。
3、上游服务是基于nacos的微服务；
4、路由关联的服务，服务里面配置的插件是：ext-plugin-pre-req 。插件使用Java语言编写。
问题出现的时间点：
当我的上游服务重启后就会偶发的出现这个问题，然后我重启apisix 又能够正常访问；

You can keep an eye on whether the container has restarted or whether the java runner process has exited abnormally.
From the information you provided, it's hard for me to determine the problem.


test

just a PR to confirm it works in CI

@nha Shall we close this pull request? Or are there any further steps here?

Sorry, closing.

Sorry, closing.


Bug: READONLY keyword

Describe the bug
surrealdb-migrations apply
Error:
   0: Parse error: Failed to parse query at line 19 column 2 expected one of FLEX(IBLE), TYPE, VALUE, ASSERT, DEFAULT, or COMMENT
   0:    |
   0: 19 | READONLY;
   0:    | ^
   0:
   1: Parse error: Failed to parse query at line 19 column 2 expected one of FLEX(IBLE), TYPE, VALUE, ASSERT, DEFAULT, or COMMENT
   1:    |
   1: 19 | READONLY;
   1:    | ^
   1:

To Reproduce
Using Surreal's new READONLY feature
Information

SurrealDB version: v1.2.0-beta.1
surrealdb-migration version: 1.0.1

I just published surrealdb-migration v1.2.0. It should work now if you update to this version.


Accessing the location.

Hey how can we access the location which is clicked on the search bar? Does have any variable for accessing it or any method which returns location or LatLng?

I will update the readme today. Sorry, should have done it a while back.

Thanks for the reply please tell how can i access it!

@Teja-developer @jayanthsaikiran what functionality of the app are you using exactly... the above code snippet is for when you are using the autocomplete searchbar ONLY, if you want to use the place picker... you have to override onActivityResult in your fragment and then check for the result ... the following is an example, also please look through the example project.
@Override
    protected void onActivityResult(int requestCode, int resultCode, Intent data) {
        super.onActivityResult(requestCode, resultCode, data);

        if (resultCode == Activity.RESULT_OK && requestCode == 300) {
            NiboSelectedPlace selectedPlace = data.getParcelableExtra(NiboConstants.SELECTED_PLACE_RESULT);
            Toast.makeText(this, selectedPlace.getPlaceAddress(), Toast.LENGTH_LONG).show();
        } else if (resultCode == Activity.RESULT_OK && requestCode == 200) {
            NiboSelectedOriginDestination selectedOriginDestination = data.getParcelableExtra(NiboConstants.SELECTED_ORIGIN_DESTINATION_RESULT);
            Toast.makeText(this, selectedOriginDestination.getOriginFullName() + " - " + selectedOriginDestination.getDestinationFullName(), Toast.LENGTH_LONG).show();
        } else {
            Toast.makeText(this, "Error getting results", Toast.LENGTH_LONG).show();
        }

    }

No further activity


Setting multiple (small) fields, big size overhead

Hi,
you can set multiple fields, but internally it's always stored as a float64..which take up 8 bytes,
I have a lot of boolean values/ small integer values < 255 that could fit in one byte,
would be nice if there was an option to define the size during set..
maybe some prefix b:  f.e. SET FIELD b:somefield 5
Regards
Jayme

Good idea. We could probably detect the minimum size of the value when SET FIELD is called. So the server would recognize that SET FIELD somefield 5 can fit in one byte and choose to store it efficiently. This way one syntax could handle float64, byte, or other sizes in the future. What do you think?

Sure, or change low level storage in collection.go from
type itemT struct {
	ID     string
	Object geojson.Object
	Fields []float64
}

to :
type itemT struct {
	ID     string
	Object geojson.Object
	Fields [][]byte
}

storing a byte array?
 []byte(strconv.Itoa(<someinteger>))

You can convert back to float64 when fetching, would require minimal code changes imho

Something like what you suggest should work. Though the overhead of a byte array in Go can be quite large. Perhaps we can pack up to 8 individual bytes in a float64. Then the collection structure can know which field indexes are used for packed bytes or float64. Then we can use the math.Float64bits function for translating.
For example, the collection could dedicate the second float64 as 8-8bit values, and we could retrieve the 2nd byte like such.
b := byte((math.Float64bits(item.Fields[1])>>1)&0xFF)


Release 0.12.2 is broken due to #4480 and #4513

🐛 Bug
https://github.com/facebookresearch/fairseq/pull/4480 was added Jun 16th, 2022, but then reverted (by https://github.com/facebookresearch/fairseq/commit/956fcf495b2d5d696ba114520363f82148a8a649) on Jun 22nd.
However it seems the 0.12.2 release was done between this (despite the release date saying Jun 27th).
To Reproduce
E.g. pip install fairseq, then go to where the files have been installed and look under fairseq/modules/transformer_layer.py and the __init__() function of TransformerEncoderLayerBase contains lots of references to BT.
Also models/transformer/transformer_layer.py has load under forward_scriptable().
(Aside: shouldn't this kind of version detection code be done globally, and added to cfg, rather than in the forward() function that is going to get called a lot?)
Additional context
The revert commit describes how this was causing a test failure.  (So I think you need to check your release procedures are running tests on the release candidate branch!)
I think it is also the cause of #4518?
And also the cause of #4740?
But beyond that, the code in transformer_layer.py was also buggy: when loading a big transformer model (created with a version of fairseq prior to this) that buggy code added a number of variables to every encoder layer, which added 75 million float32 weights to the saved model. It has taken me a day and a half of troubleshooting to narrow it back to this "BT" code.
(BTW, it was also not mentioned anywhere that "BT" means the BetterTransformer code added to PyTorch 1.12.)
I would suggest 0.12.3 be released based on the intended Jun 27th version of main?

Aha! It was re-landed on Jun 24th (see https://github.com/facebookresearch/fairseq/pull/4513)
and then backed out a second time on Jul 14th.
So that explains how it ended up in the 0.12.2 release.

Hi @DarrenCook
You are right and loading 0.12.2 checkpoints with 0.12.1 is also buggy/risky.
A 0.12.3 release seems to be indeed the right action.
Kind

Using pip install fairseq==0.12.1 has been working for me so far.
I had to throw away the models made with 0.12.2: the extra weights added to the encoder seem to be part of the model that was training, i.e. it has created a different architecture.  Which also means supporting models built with 0.12.2 in 0.12.3 and later is not going to work.

@frank-wei can you investigate please ? @dianaml0 @cbalioglu  do we need to cut a new release to prevent people working on the broken 0.12.2 ?

Yeah, we should have a 0.12.3 release and mark 0.12.2 as deprecated. Let me sync up internally.

Yes, the code has some problems(bring more variables and takes more memory) and we decided to back it out later https://github.com/facebookresearch/fairseq/pull/4568

@cbalioglu  any news on 0.12.3 ?

Things like mms have been added to the repo after the latest release, would it be possible to have a new release that includes the mms things ?
I'm asking as I would like to package this for nixos. Using an officially released version rather than latest master would be great.
Thank you for the software!

Hey all, any news on a new release? We are currently unable to use this repo because we have a dependency on numpy > 1.24, which requires a fix for the deprecated np.float alias, which seems to have been taken care of here: https://github.com/ncoish/fairseq/commit/a24fdf2d1b36e699d8f4e3efd33b7b78d6a02e7e


[mgmt] compute release

https://github.com/Azure/sdk-release-request/issues/5069

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/arm-compute


Update RustCui.cs

More Add/Destroy operations
Should be backwards compatible
Note: have not tested
Slight fix

As far as I know, there isn't any point in these since the CUI is only initialized when the player actually exists, so there's no point in having methods for connection.

Sadface.  The point was to refactor the helper to allow people such as myself shortcut methods to AddUI/RemoveUI without having to convert everything to CUIelements or use Facepunch ObjectLists in our own code.
The connection argument isnt important, the new signatures allow us to primarily use json/name strings (more similar to how it works by default) rather than force people to use the CUIElement system.

Hmmm, I'll pass it by @Nogrod and @bawNg again and see what they think.

@Nogrod made a change in PR #569, though we feel that the connection overloads aren't needed. Let us know if that's enough for you.


NoMethodError: undefined method `visit_Arel_Nodes_Casted' for #Arel::Visitors::DepthFirst:

I get this error when I try to run:
Game.where.not(score: [0,nil]).size
This is the stacktrace:
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:63 in binary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:62 in binary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:17 in unary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:17 in unary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:17 in unary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:57 in block in nary
org/jruby/RubyArray.java:1593 in each
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:57 in nary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in block in visit_Array
org/jruby/RubyArray.java:1593 in each
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in visit_Array
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:146 in visit_Arel_Nodes_SelectCore
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in block in visit_Array
org/jruby/RubyArray.java:1593 in each
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in visit_Array
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:153 in visit_Arel_Nodes_SelectStatement
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:9 in accept
/gems/arel-6.0.3/lib/arel/nodes/node.rb:56 in each
org/jruby/RubyEnumerable.java:568 in grep
org/jruby/RubyEnumerable.java:542 in grep
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/calculations.rb:216 in perform_calculation
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/calculations.rb:133 in calculate
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/active_record_helper.rb:70 in block in calculate
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/transaction.rb:849 in with_database_metric_name
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent.rb:569 in with_database_metric_name
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/active_record_helper.rb:69 in calculate
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/calculations.rb:48 in count
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation.rb:258 in size
/app/app/decorators/game_page.rb:925 in encode_game_page
/app/app/decorators/game_page.rb:629 in fetch
/app/app/workers/invalidate_game_job.rb:10 in perform
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:152 in execute_job
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:134 in block in process
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:128 in block in invoke
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/sidekiq.rb:33 in block in call
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/controller_instrumentation.rb:363 in perform_action_with_newrelic_trace
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/sidekiq.rb:29 in call
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:130 in block in invoke
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/server/active_record.rb:6 in call
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:130 in block in invoke
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/server/retry_jobs.rb:74 in call
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:130 in block in invoke
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/server/logging.rb:11 in block in call
/gems/sidekiq-4.1.4/lib/sidekiq/logging.rb:32 in with_context
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/server/logging.rb:7 in call
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:130 in block in invoke
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:133 in invoke
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:129 in block in process
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:168 in stats
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:128 in process
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:81 in process_one
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:68 in run
/gems/sidekiq-4.1.4/lib/sidekiq/util.rb:17 in watchdog
/gems/sidekiq-4.1.4/lib/sidekiq/util.rb:25 in block in safe_thread
Caused by NoMethodError: undefined method `visit_Arel_Nodes_Casted' for #<Arel::Visitors::DepthFirst:0x39f75eb5>
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:63 in binary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:62 in binary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:17 in unary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:17 in unary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:17 in unary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:57 in block in nary
org/jruby/RubyArray.java:1593 in each
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:57 in nary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in block in visit_Array
org/jruby/RubyArray.java:1593 in each
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in visit_Array
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:146 in visit_Arel_Nodes_SelectCore
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in block in visit_Array
org/jruby/RubyArray.java:1593 in each
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in visit_Array
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:153 in visit_Arel_Nodes_SelectStatement
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:9 in accept
/gems/arel-6.0.3/lib/arel/nodes/node.rb:56 in each
org/jruby/RubyEnumerable.java:568 in grep
org/jruby/RubyEnumerable.java:542 in grep
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/calculations.rb:216 in perform_calculation
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/calculations.rb:133 in calculate
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/active_record_helper.rb:70 in block in calculate
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/transaction.rb:849 in with_database_metric_name
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent.rb:569 in with_database_metric_name
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/active_record_helper.rb:69 in calculate
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/calculations.rb:48 in count
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation.rb:258 in size
/app/app/decorators/game_page.rb:925 in encode_game_page
/app/app/decorators/game_page.rb:629 in fetch
/app/app/workers/invalidate_game_job.rb:10 in perform
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:152 in execute_job
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:134 in block in process
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:128 in block in invoke
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/sidekiq.rb:33 in block in call
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/controller_instrumentation.rb:363 in perform_action_with_newrelic_trace
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/sidekiq.rb:29 in call
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:130 in block in invoke
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/server/active_record.rb:6 in call
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:130 in block in invoke
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/server/retry_jobs.rb:74 in call
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:130 in block in invoke
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/server/logging.rb:11 in block in call
/gems/sidekiq-4.1.4/lib/sidekiq/logging.rb:32 in with_context
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/server/logging.rb:7 in call
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:130 in block in invoke
/gems/sidekiq-4.1.4/lib/sidekiq/middleware/chain.rb:133 in invoke
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:129 in block in process
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:168 in stats
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:128 in process
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:81 in process_one
/gems/sidekiq-4.1.4/lib/sidekiq/processor.rb:68 in run
/gems/sidekiq-4.1.4/lib/sidekiq/util.rb:17 in watchdog
/gems/sidekiq-4.1.4/lib/sidekiq/util.rb:25 in block in safe_thread
Caused by NoMethodError: undefined method `visit_Arel_Nodes_Casted' for #<Arel::Visitors::DepthFirst:0x39f75eb5>
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:63 in binary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:62 in binary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:17 in unary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:17 in unary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:17 in unary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:57 in block in nary
org/jruby/RubyArray.java:1593 in each
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:57 in nary
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in block in visit_Array
org/jruby/RubyArray.java:1593 in each
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in visit_Array
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:146 in visit_Arel_Nodes_SelectCore
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in block in visit_Array
org/jruby/RubyArray.java:1593 in each
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:169 in visit_Array
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:153 in visit_Arel_Nodes_SelectStatement
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:29 in visit
/gems/arel-6.0.3/lib/arel/visitors/depth_first.rb:12 in visit
/gems/arel-6.0.3/lib/arel/visitors/visitor.rb:9 in accept
/gems/arel-6.0.3/lib/arel/nodes/node.rb:56 in each
org/jruby/RubyEnumerable.java:568 in grep
org/jruby/RubyEnumerable.java:542 in grep
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/calculations.rb:216 in perform_calculation
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/calculations.rb:133 in calculate
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/active_record_helper.rb:70 in block in calculate
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/transaction.rb:849 in with_database_metric_name
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent.rb:569 in with_database_metric_name
/gems/newrelic_rpm-<IP_ADDRESS>8/lib/new_relic/agent/instrumentation/active_record_helper.rb:69 in calculate
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation/calculations.rb:48 in count
/gems/activerecord-<IP_ADDRESS>/lib/active_record/relation.rb:258 in size

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/36974265-nomethoderror-undefined-method-visit_arel_nodes_casted-for-arel-visitors-depthfirst?utm_campaign=plugin&utm_content=tracker%2F136963&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F136963&utm_medium=issues&utm_source=github).

We had similar problems and believe it is related to the lack of thread safety in the rescue clause during a visit of an arel node; although not related to activerecord-jdbc it seems this issue is made more prevalent because of jruby threading
https://github.com/rails/rails/issues/26571
The rails issue outlines the problem as seemingly only related to count but we decided to add a patch to our servers that should account for any concurrent visit type ... tries to minimize locking, but should only hit the lock up to the level of concurrency that exists on a server
we haven't seen it happen since applying the below patch ... but we continue to monitor for it and watch for an update to arel to be able to remove our patch
require "thread"
require "arel/visitors/visitor"

# NOTE: Fixing thread safety problems for Arel, hopefully can be removed in future
# https://github.com/rails/rails/issues/26571

module Arel
  module Visitors
    class Visitor
      NO_METHOD_MUTEX = ::Mutex.new

      def visit object
        send dispatch[object.class], object
      rescue NoMethodError => e
        superklass = object.class.ancestors.find { |klass|
          respond_to?(dispatch[klass], true)
        }

        raise(TypeError, "Cannot visit #{object.class}") unless superklass

        NO_METHOD_MUTEX.synchronize do
          dispatch[object.class] = dispatch[superklass] unless respond_to?(dispatch[object.class], true)
        end

        if respond_to?(dispatch[object.class], true)
          retry
        else
          raise e
        end
      end
    end
  end
end

I'm getting the same issue when running from a Sidekiq job.
Was the above patch submitted as a PR to the Rails project? @abrandoned
Here is the error
NoMethodError: undefined method `visit_Arel_Nodes_Casted' for #<Arel::Visitors::DepthFirst:0x3a5cae68>

@prama10 the above linked PR into arel was merged in on June 4; if you have a more recent version it may be there but we actually have both patches applied on our systems so I'll have to verify if the PR merged fixed it for both cases (in our testing we thought it would)

@abrandoned provided great HINTs on want the issue really is about, let's close it as invalid at AR-JDBC's


Add word status

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

0 of 20 (0.0%)  changed or added relevant lines in 6 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.04%) to 5.992%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/index.tsx
0
1
0.0%

src/utils/getWordStatus/index.ts
0
1
0.0%

src/components/Modals/TranslationsModal/TranslationsModal.tsx
0
2
0.0%

src/services/dictionary.ts
0
2
0.0%

src/components/ResultsCard/ResultsCard.tsx
0
5
0.0%

src/utils/getWordStatus/getWordStatus.ts
0
9
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/index.tsx
1
0.0%

src/services/dictionary.ts
1
0.0%

src/components/Modals/TranslationsModal/TranslationsModal.tsx
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>0:
-0.04%

Covered Lines:
162

Relevant Lines:
2991

💛  - Coveralls


Because flutter_david depends on package_info >=0.4.0+13 which requires Flutter SDK version >=1.10.0 <2.0.0, version solving failed.

how ?

Run flutter upgrade

Hi @DavidMGT
could you please run
flutter channel stable
flutter upgrade --force
flutter clean
flutter pub cache repair
If the issue persist please provide your updated
flutter doctor -v and flutter run --verbose
together with a a minimal code sample to reproduce the problem
thank you

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


 Sometimes the scrollbars behave strange, changing of color, and I can't distinguish the bars from the total space assigned it.
Here is an image that explain better the problem

Like you see the vertical and horizontal bars (of the text editor) doesn't appear, and the total space, where could be located, have been replaced with a grey bar.
Anyone knows the reason of this problem?

I resolve the problem setting "overlay_scroll_bars": "enabled"  in the user settings.

Seems to be a recurring issue on Windows and Linux devices. I'll be sure to add this to the setup guide.


Raspberry Pi 400 - sshnpd v3.4.2 not working // v3.3.0 works

Describe the bug
When attempting to connect to Raspberry Pi 400 from client machine (MacOS {also tried Windows 10}), version 3.4.2 cannot establish a connection with the Pi 400.
When down grading to version 3.3.0, a connection was successfully established.
Steps to reproduce

First I Install v3.4.2 sshnpd on Raspberry Pi 400
Then run v3.4.2 sshnp on MacOS
And then error occurs: Remote sshnpd error: Failed to establish connection - exit code 255 (use --local-port to specify unused port)
Terminate sshnpd tmux session on Pi 400
Run installer, specified version with --version v3.3.0 on Pi 400
Then run v3.4.2 sshnp on MacOS
Connection established

Expected behavior
v3.4.2 sshnpd establishes connection
Screenshots

Above, connection was attempted on v3.4.2 on Pi 400. Connection that succeeded was v3.3.0
Smartphones
No response
Were you using an atApplication when the bug was found?
No response
Additional context
No response

@TylerTrott I'm not able to reproduce this issue, can you try reinstalling v3.4.2 to the pi, if that doesn't work I can debug with you further.
FYI Windows is not a supported client until (at least) v4.

I will try reinstalling to the pi today. What image of Pi do you have on the 400? I imaged mine with x64 Raspberry Pi OS Desktop. I will ping you if it does not work still.
With the above issue and screenshot, I was using MacOS as the client.

I will try reinstalling to the pi today. What image of Pi do you have on the 400? I imaged mine with x64 Raspberry Pi OS Desktop. I will ping you if it does not work still.
With the above issue and screenshot, I was using MacOS as the client.

I don't have a Pi 400

@TylerTrott is this still an issue?

@TylerTrott is this still an issue?

Just tried it on version 3.4.2 and it now works with no problem for both client and Raspberry Pi.

I bore witness :) +1

Just tried it on version 3.4.2 and it now works with no problem for both client and Raspberry Pi.

Yay! Whatever happened that other time must of been some weirdness related to you running the beta version of the installer.

Just tried it on version 3.4.2 and it now works with no problem for both client and Raspberry Pi.

Yay! Whatever happened that other time must of been some weirdness related to you running the beta version of the installer.

Maybe, it could have potentially just been user error.


Parse DateTime objects using Ruby

This ensures that timezones are preserved when converting a DateTime object to a unix epoch

Resolves issue #1


rename to hexo-renderer-jade

Is it possible to rename this repo to hexo-renderer-pug? Or at least publish it on https://www.npmjs.com/package/hexo-renderer-pug?
This may be related:
https://github.com/iwarner/hexo-renderer-pug/issues/1

@ASVBPREAUBV
Thank you for your report :)
We replaced from hexo-rendere-jade to hexo-rendere-pug

repository
npm


HLG: get_all_external_keys()

This PR introduces HighLevelGraph.get_all_external_keys(), which makes it possible to cull high level graphs that contains ShuffleLayer without materializing them.

[x] Tests added / passed
[x] Passes black dask / flake8 dask

cc. @rjzamora

@jrbourbeau I think this is ready to be merged


[Pipeline] Support duplicate component output as pipeline output

Description

Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml

Can someone help review this? Our migration to SDK v2 is currently blocked by this issue.


JarCreator is non-reproducible

JarCreator has non-reproducible behavior in it's jar index creation.
 I copied the upstream version of the JarCreator and it fixed the non-reproducible behavior: https://github.com/gkk-stripe/rules_scala/commit/3687cb0a8d5e9b119f1059dbbed87cf1734b75b4
Tasks:

can we depend on bazel's jarcreator without copying code
if not, copy over the code again

cc @jmillikin-stripe @johnynek @ianoc

Note: the important part of that snippet is Zip file size: 1344 bytes vs 1308 bytes despite identical content and zip indexes.

😱
On Wed, 21 Mar 2018 at 21:59 jmillikin-stripe<EMAIL_ADDRESS>wrote:

Note: the important part of that snippet is Zip file size: 1344 bytes vs 1308
bytes despite identical content and zip indexes.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/447#issuecomment-375076319,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIFzX4WriOwtMztkaEKo8POxHeBOSdks5tgrE6gaJpZM4S1lAv
.

I'd rather copy the code for short term, but then I'd like to see if we can get jarcreator into @bazel_tools so we can access it without adding an external repo (or any git fetching hijinks).
Good work tracking this down.
Can we find a test that we can write that would have caught this?

note, we actually made a few minor changes.... With regards to making the default to normalize timestamps I think.

@gkossakowski are you working on this, or should some one else take it?

i'll take it, i'll send a pr after the weekend


Convert string to template literal

Convert a string, or string concatenation to a template literal
Key Behaviors:

Accessible by the same means as other automated tools

Context menu (conversion submenu)
Action palette
No keybinding

Allow user to place cursor on string or concatenation "expression" (tree of binary concatenation expressions)
Validate selection is a valid, convertable expression
Build new string template with appropriate variable interpolation notation on all non-string values
Does not descend into parenthetical expressions (for now, leave the expression as is)

Error Conditions:

Selection contains no string
Selection contains an operation other than concatenation (+)

Skipping parenthetical descent appears to be a topic of research for another time -- parenthetical statements have no representation in estree (which is fine and largely unsurprising).
First iteration complete in latest release:
https://github.com/cmstead/js-codeformer/releases/tag/v0.25.0


Request Missing HighQC Block For Timeout Message

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

#1061  👈
#1058
#1057
feature/pos-networking-and-syncing

This stack of pull requests is managed by Graphite.


Transaciton not working if exception was thrown after 1 sec.

In what version(s) of Spring for Apache Kafka are you seeing this issue?
2.8.0
Describe the bug
In a slow transactional method, messages are send even this method fails after 1s
To Reproduce
@GetMapping("/test")
    @Transactional
    public void test() {
        kafkaTemplate.send("top", "hello").addCallback(result -> {
            System.out.println("send succ");
        }, e -> {
            System.out.println("send failure");
        });
        Thread.sleep(1000);
        throw new IllegalStateException("test");
    }

Expected behavior
Consumers unable to consume such message.

I debugged and I am sure that  org.apache.kafka.clients.producer.KafkaProducer#beginTransaction and org.apache.kafka.clients.producer.internals.TransactionManager#maybeAddPartitionToTransaction was called.

Records are written to the log even if the transaction is rolled back - Kafka adds a marker record after the published records to indicate commit or rollback; consumers must be configured with isolation.level=read_committed to prevent them receiving rolled-back records.

@garyrussell Thank you very much, setting isolationLevel resolved my problem. I thought the default level was read_committed so I never think in the right direction.

@garyrussell I am trying to understand how transaction works in kafka. Do you know why didn't kafka broker just filter out all aborted partition data and send it to consumer? I saw current implentation is consumers did all the filtering in org.apache.kafka.clients.consumer.internals.Fetcher.CompletedFetch#nextFetchedRecord.

I have no idea. We don't have extensive knowledge of Kafka internals here; you'd be better off asking on one of their mailing lists. https://kafka.apache.org/contact


Tenant setup space corrections

Fixed inconsistent spacing - noticed when viewing on mobile that the spacing around the portal link was barely visible. Looks like the space character used was incorrect.

@mrose-sorted : Thanks for your contribution! The author(s) have been notified to review your proposed change.

LGTM - Thanks @mrose-sorted


Debug views: add line-height to center items in trees/lists

At least the breakpoints view seems to be missing a line-height: 22px so that the items are perfectly centered. I did not check the other views.
E.g. compare with the SCM view where each item has a line-height associated.

@bpasero the breakpoints view was the only missing this since I transformed it to use a list a forgot to apply this css rule. Thanks for finding this.

@isidorn great


AoT Compiler introduces bugs at opt level > 0

Commit Hash: f69ff7058783bc5dcaaa81c9d6c88c0d534881fc
Issue Description
When i compile certain WASM modules using an opt-level > 0, its execution fails with a division by zero error.
opt-level 0 and the fast interpreter do not raise any errors.
Environment Description
Input file:
wikisort.tar.gz
Compilation settings:
wamrc --target=thumbv7 --cpu=cortex-m4 --cpu-features=vfp4d16sp --opt-level=3 --size-level=3
Target hardware:
STM32L4R9-DISCO
WAMR Build Configuration:
set (WAMR_BUILD_TARGET "THUMBV7_VFP")
set (WAMR_BUILD_INTERP 1)
set (WAMR_BUILD_FAST_INTERP 1)
set (WAMR_BUILD_LIBC_BUILTIN 1)
set (WAMR_BUILD_LIBC_WASI 0)
set (WAMR_BUILD_LIB_PTHREAD 0)
set (WAMR_DISABLE_HW_BOUND_CHECK 0)
set (WAMR_BUILD_AOT 1)
set (WAMR_DISABLE_STACK_HW_BOUND_CHECK 0)

WAMR Runtime Settings:
RuntimeInitArgs runtime_args = {
    .mem_alloc_type = Alloc_With_System_Allocator,
    .running_mode = Mode_Interp,
};

Instructions for Recreation
Run the _initialize function followed by _run of the AoT compiled module, both with no arguments. the WASM module is compiled as a reactor.

I have an STM32F4291-DISC1 board on my hand, I will try your program and see what I can do.

Hi, I am getting a little rusty with how to run the program on an embedding device, could you please tell me how you run the program on that device? It would be helpful if you tell me which framework(RTOS) you are using and how you run your program.
Were you using Zephyr/simple with test_wasm.h replaced with binary content of AOT file?

Hi!
My setup takes a bit of configuration to reproduce:
I use libopencm3 to setup the C runtime and execute WAMR from plain C.
libopencm3 is gnu make based but you can include it in a CMake project as follows:
(note: this is for the l4, not f4)
add_custom_command(OUTPUT ${OPENCMDIR}/lib/libopencm3_stm32l4.a
                    COMMAND make TARGETS='stm32/l4 CC=$ARMGCC/arm-none-eabi-gcc'
                    WORKING_DIRECTORY ${OPENCMDIR})

target_sources(wamr PRIVATE ${OPENCMDIR}/lib/libopencm3_stm32l4.a)
target_link_libraries(wamr PUBLIC ${OPENCMDIR}/lib/libopencm3_stm32l4.a)
target_link_options(wamr PUBLIC -T ${CMAKE_CURRENT_LIST_DIR}/device.ld)
target_compile_options(wamr PUBLIC -DSTM32L4=1)

You can debug the program using JLink/ST-link.
If you need printf output you have to manually set it up using arm hardware registers and implement newlib stub _write.
In my case this is done as follows but the f4 may have different clocks/registers:
static void clock_setup(void)
{
	rcc_clock_setup_pll(&rcc_hsi16_configs[RCC_CLOCK_VRANGE1_80MHZ]);

	rcc_periph_reset_pulse(RCC_AHB1RSTR);
	rcc_periph_reset_pulse(RCC_AHB2RSTR);
	rcc_periph_clock_enable(RCC_AHB1ENR);
	rcc_periph_clock_enable(RCC_AHB2ENR);

	/* Enable GPIOA clock for LED & USARTs. */
	rcc_periph_clock_enable(RCC_GPIOA);

	/* Enable clocks for USART2. */
	rcc_periph_clock_enable(RCC_USART2);

	systick_set_reload(UINT32_MAX);
	// systick_set_frequency(1000, 80000000);
	systick_set_frequency(1000, 80000000);

	systick_interrupt_enable();
	systick_counter_enable();

	return;
}

static void usart_setup(void)
{
	/* Setup USART2 parameters. */
	usart_set_baudrate(USART2, 115200);
	usart_set_databits(USART2, 8);
	usart_set_stopbits(USART2, USART_STOPBITS_1);
	usart_set_mode(USART2, USART_MODE_TX);
	usart_set_parity(USART2, USART_PARITY_NONE);
	usart_set_flow_control(USART2, USART_FLOWCONTROL_NONE);

	/* Finally enable the USART. */
	usart_enable(USART2);
}

static void gpio_setup(void)
{
	/* Setup GPIO pins for USART2 transmit. */
	gpio_mode_setup(GPIOA, GPIO_MODE_AF, GPIO_PUPD_PULLUP, GPIO1 | GPIO2);

	/* Setup USART2 TX pin as alternate function. */
	gpio_set_af(GPIOA, GPIO_AF7, GPIO1 | GPIO2);
}

int init(void)
{
	int i, j = 0, c = 0;

	clock_setup();
	gpio_setup();
	usart_setup();

	return 0;
}

You can find examples on the libopencm3 github repository, I pieced together my startup code from there 😅.

Thanks! I will try to set up the config(wish me luck for it's been a while since the embedding course from college) and reproduce the bug. I will keep you updated!

You might be able to reproduce the issue with any framework.
WAMR reported the division by zero error/exception, and my board did not panic/trap.


Xmx dev

When there are multiple css in the page that need to be inlined，a part of css will not be inlined.

fix https://github.com/Runjuu/html-inline-css-webpack-plugin/issues/25


Product Detail Page: Image check fails

Even if no image is set for the product, the $sArticle.image variable
will be initialized with a empty array. The check always returns true
and the image zoom will try to zoom a empty image.
Better check if the $sArticle.image.id variable is set.

Hi @jhit, thanks for your contribution.
Unfortunately I am not able to reproduce this error:

If no image is set for the product, $sArticle.image is null instead of an empty array
Even with an hardcoded empty array in $sArticle.image the condition will cast the empty array to false and the zoom is disabled.

Can you please verify the contents of $sArticle.image in your template using {$sArticle.image|var_dump}. Maybe a broken plugin is putting some garbage values in there.

Hi Benjamin. I did this var_dump with this result:
 array(2) {
    ["description"]=> NULL
    ["relations"]=> NULL
}

Plugins-Installed:

No worries. I am glad you found the cause.


Add resource support for what if

PR Summary

fix #70
example for resource participation, where whatif is implemented like set but with an additional switch arg:

{
    "$schema": "https://aka.ms/dsc/schemas/resource_manifest",
    // truncated
    "set": {
        "executable": "whatifexample",
        "args": [],
        "return": "state"
    },
    "whatif": {
        "executable": "whatifexample",
        "args": [
            "--what-if"
        ],
        "return": "state"
    },
    // truncated
}

PR Context

add resource support for config set --what-if

Should there be a separate schema for what resources return for what-if that aligns with how the engine synthesizes it?  Perhaps with addition of reason field that is open text that the resource can fill in

@tgauth please open an issue to add 'whatif' support to PS Adapter.

Based on the review meeting:

Add new whatIf section as a top-level operation in the resource manifest which only applies to set, all other changes in the code will flow from this

@tgauth can you provide example of what-if output where a resource participates?


luci-base: fix syntax error in hostapd features

Signed-off-by: Martin Böh<EMAIL_ADDRESS>

Sorry for the DCO spam but it didnt like that i tried to use github directly for this minor edit and then it didnt like the mismatch between my realname (which is required for signing off) and my github username.

Merged, thanks!

Error is back.
https://github.com/openwrt/luci/issues/4555#issuecomment-927104087

I have the same error message, but I can't find what's the problem?

This is our "luci" file
luci.txt

@clonin216 can you try hard reset of browser cache?

@lynxthecat Yes, I did, but the problem persists

@jow-
I am using the latest version of luci

I use the same source code, one computer compiles without this error, but another computer compiles with the error.
When we add these codes, there will be errors, please help me to confirm if there is a problem.

I use the same source code to compile on two different computers, one succeeds and the other fails, what's the difference before?


[new-product] opnsense

add opnsense product
close #4065

I think its ready to merge, feedback is welcome

Seems good to me

The only officially supported and actively maintained version of OPNsense is the latest one.

I didn't find any documentation about supported or maintened version
I base this on the versions available and their release dates, also from here on the Staying ahead

Thanks for the review @marcwrobel
Dates should be ok I took them from git. I applied your suggestions
LGTM for merge


Update ms-confluence-jira-plugin-adminguide.md

Updated Confluence Support Version.
Thanks & Regards,
Sagar Patil

@dhivyagana

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Hard to listen to all-test-results event

The code of testem client is
window.Testem = {
  emit: function(evt){
    var args = Array.prototype.slice.apply(arguments)
    socket.emit.apply(socket, circularSafe(args))
    if (this.evtHandlers && this.evtHandlers[evt]){
      var handlers = this.evtHandlers[evt]
      var args = Array.prototype.slice.call(arguments, 1)
      for (var i = 0; i < handlers.length; i++){
        var handler = handlers[i]
        handler.apply(this, args)
      }
    }
  },

Which means, first the socket then the callbacks. In case of all-test-results, the socket message tells testem ci to kill the browser, so depending on how quick this happen, the handler callback might not be called.
It would be good to move the socket.emit at the end of the method

I was able to work around this by setting a timeout in lib/ci/browser_test_runner.js in the onAllTestResults handler. My changes look like this:
  onAllTestResults: function(results){
    var self = this
    setTimeout(function() {
      self.launcher.kill(null, function(){
        self.finish()
      })
    }, 50);
  },

As you can see, you don't need much of a timeout (maybe depending on how performant your coverage server is). I tried it down to 10, but at 10, it was sometimes inconsistent. At 50, it seemed to generate the report reliably for me (and that's still pretty fast - you don't notice the delay at all). @airportyh - Is this a viable fix that could be merged? It seems a bit hackish, and you might prefer something more legit, but this does solve the problem. You could always merge it as a temporary fix to be replaced later with something better.

As a matter of facts, grunt-testem is also using a 50ms timeout.

Well . . . this still doesn't work in PhantomJS. Works with Chrome. I mean . . . the timeout happens (logging stuff tells me so), but it's hard to know if my script that sends coverage to the server is working. Adding console.log does nothing. Should I be seeing logs in the terminal when running under phantom?

Which testem and node version are you using?

Node v0.10.33
Testem v0.6.24 from here

Regardless of the question how effective is adding a timeout (from the answer book: not very, because it relays on an inherent race condition existing in the system) as a Computer Engineer I would not approve a timeout based solution--clearly anyone could add one.
Couldn't the author explain how to add an event that is capture the coverage results???

I believe I'm encountering the same thing, specifically with the scenario of trying to deliver istanbul coverage results after testem finishes running phantomjs (chrome works fine). Any movement on this in terms of a solution or reliable workaround?

+1. Seeing this in an ember-cli app

I'm seeing this too now.

I have been thinking about this and wondering if it makes sense to implement a "sendCoverage" or similar functionality via a config. This seems like a very common use-case, it might sense to wrap the kill command that gets fired on "onAllTestResults" with a check for this flag and run kill on the updated readyState (done) from the ajax request. The URI and port can come from the config. Thoughts? This is a big issue in one of our apps that has a fairly large coverage file.

I would prefer to have a more generic way to register async afterTests / beforeTests hooks, but would gladly accept a PR.

A generic way would be lovely. @jreading These lines shutdown the browser process: https://github.com/testem/testem/blob/master/lib/runners/browser_test_runner.js#L145-L149
As a demonstration you can remove this.finish() and your coverage sending should finish with the test server process never finishing :)
A simple approach would be to call this.finish after a callback as supplied by the configuration. If I get time I can try PRing something to this effect.

I'm working through an "afterTests" queue that handles async callbacks. PR later today and of course open for comments.