[FormBuilder] Take a look for react performance

I think the library "BasicFields" should by default offer optimized react experience.

I think BasicFields should offer a "withDefaults" experience. Most apps might only ever need BasicFields.

What do you mean by "withDefaults" ?

Ideally we would have the most common types of field and they would come with sensible defaults including validation as standard.  I was wondering about using validator.js for this:
https://github.com/chriso/validator.js
https://www.npmjs.com/package/@types/validator
Like you know how in html a Form at it's most basic is:
  <form>
    <input name="name" placeholder="Name" />
    <input name="email" placeholder="Email" type="email" />
    <input type="submit">
  </form>

I wonder if we could make BasicFields (maybe in Fulma?) as close as possible to this sort of simple experience, but of course with all the power of F# and this library behind the simple interface
Apologies this has little to with React, more to do with "optimized experience"

I moved this issue to another repository, and further work will take place there.


Navigation Bar has gone

First, Thanks for the awesome work!
I have this problem when display web page in UIWebView.
This is how it looks when its normal:

But when keyboard shows, the entire webpage is pushed up, and the navigation bar has gone out of the screen. see image below

Is that a bug or just me?

Closing it for now


socket.io was blocked

Recently (porbably) during the last update socket.io was added to the hosts list.. and from what I know socket.io is not a bad site ? 🤔
It's just hosting dev resources about socketio.. ?

@PlatinShadow  it was already reported to the curator of the lists which blocked it and it will be removed in their next update.

Thanks Lars @PlatinShadow and D' @dnmTX.

Alright. Thank you all for responding so quickly and for providing this awesome list :)

@PlatinShadow  just a heads up.Next update might bring that domain back into the list.
I already reported it to the curator.

Oh, good catch Dan @dnmTX.


Look like some symbols are broken for some reason

I just installed ScriptKit on my MacBook Pro 16 M1 Pro and I see broken symbols for some reason. Nothing has been changed so far, it's fresh install of an app.
Here how it looks

Any ideas how to fix this?

@DavidArutiunian Hrmm, that uses a bundled "JetBrains Mono" font. Any chance you have JetBrains Mono installed on your system and there's some sort of conflict?

@johnlindquist yeah, I have JetBrains Mono font installed on my system, but I use it in VSCode 🤔 I could try removing it from system to see if that fixes this bug but I'll still need this font on a machine

@johnlindquist yeah, that fixed a problem but I think need to be handles in some way so that people with JetBrains Mono won't struggle with that again 🙂

@DavidArutiunian Could you tell me the version of your "JetBrains Mono" system font? I don't know if there's a possible version conflict. This seems really strange to me. The CSS just says "font-family: "JetBrains Mono"

I'm not sure I can find a version, cause I installed it via system font book. I'll try to upgrade JetBrains Mono to a latest version

Sorry, I don't know what "Nerd Font version" is

It's a font with icons for a powerline prompt. It enables icons in terminals

I'll see if I can do something like "font-family: "JetBrains bundled" or some other trick to force it to use the bundled version

@johnlindquist yeah, seems that was an issue with outdates JetBrains Mono installed. Issue is gone after I upgraded both JetBrains Mono and JetBrains Mono Nerd Font versions, thanks! 🙏

@DavidArutiunian Cool, thx for letting me know. I'm changing the font loading regardless to avoid something like this happening again.


SCT-1251 Add time and call id to error structure

Will allow users to provide time & callid to sysops so they can look up
errors in logs. call id will allow for determining which log lines
belong to which request.

Pull Request Test Coverage Report for Build 214

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

Totals

Change from base Build 213:
0.02%

Covered Lines:
1043

Relevant Lines:
1097

💛  - Coveralls

I added a StreamHandler with a custom formatter. For now that's hard coded but it wouldn't be hard to switch to a different handler like a HTTP handler for sending the logs to ELK or whatever.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 66 ms

Resolved: ClamAV performance has improved in 8fd68c0.


Missing docs on how to effectively use scaffolded resources from operator-sdk create webhook

What is the URL of the document?
https://master.sdk.operatorframework.io/docs/building-operators/golang/
Which section(s) is the issue in?
N/A
What needs fixing?
It seems like this section is missing information on what's scaffolded when making a call to operator-sdk create webhook.
The command help text exists in the documentation at this location - https://master.sdk.operatorframework.io/docs/cli/operator-sdk_create_webhook/ - but it doesn't quite clarify what a user's next steps are (e.g. in the case of conversion webhooks, defining their hub and spokes).
Additional context
I don't think it's beneficial to reimplement the docs as they exist here in the Kubebuilder book - https://book.kubebuilder.io/multiversion-tutorial/conversion-concepts.html - but I do think some docs on what gets scaffolded and why would be beneficial. Next steps pointing to the Kubebuilder docs, considering this is probably a kubebuilder-based task in its entirety seems sufficient.
In the past, this document existed and goes more in depth - https://v0-19-x.sdk.operatorframework.io/docs/golang/webhooks/. I'm not sure if this was intentionally left behind because how things worked changed or otherwise, but figured I would mention it.

@komish yes, as you had pointed out the reason for removing documentation on the next steps was because it is already available in the Kubebuilder docs so I think replicating it in SDK docs would not be needed.
Keeping this open to discuss in triage meeting.

@varshaprasad96 Understood! I'll attend and see if there's any questions I can help with.

Ok. Going to close this, please reopen if you have any further issues.

Issue to handle referencing of upstream docs: #4831


BUG Use civis-jupyter-notebook which doesn't clobber dependencies

The versions of civis-jupyter-notebook <0.2.4 have strict requirements which, for example, will force a downgrade to civis==1.6. Update to a bugfix version with more permissive requirements.

The new civis-jupyter-notebook installation downgrades our cloudpickle version from v0.5.1 to v0.3.1 due to a too-restrictive requirement in the Python API client (fixed in civisanalytics/civis-python#187). I think that's not worth worrying about, though. All other installed packages are the same as in datascience-python v3.3.0, which is what we want.


Autotests: #3047-13-tests-for-floating-windows

How the feature works? / How did you fix the issue?
(Screenshots, videos, or GIFs, if applicable)
Check list

[ ] unit-tests written
[x] e2e-tests written
[ ] documentation updated
[x] PR name follows the pattern #1234 – issue name
[x] branch name doesn't contain '#'
[x] PR is linked with the issue
[x] base branch (master or release/xx) is correct
[ ] task status changed to "Code review"

And resolve conflicts please

Tests from this pull request have been moved to the pull request https://github.com/epam/ketcher/pull/3343


Doc: Add AutoMM Architecture Diagram

Add back the content that was in the old doc page: https://auto.gluon.ai/0.4.0/tutorials/text_prediction/multimodal_text.html#what-s-happening-inside
This will help explain what is happening in AutoMM.

Resolving, it is added here: https://auto.gluon.ai/stable/tutorials/multimodal/index.html


[vim9class] _ description does not conform to common meaning

Steps to reproduce
This issue comes from #13457, it's the only thing in that discussion that has general agreement.
Note: this issue is only about documentation; no code changes are implied/suggested. Read #13457 for the broader discussion.
The following table is summary, taken from https://en.wikipedia.org/wiki/Access_modifiers,

Keyword
maximal access

private
class

protected
derived classes

shows two widely used access methods and how they are named.
Vim uses _ for what most OOP languages call protected, OK; the problem is that it describes _ as private. This causes confusion and is contrary to accepted nomenclature.
Additionally, if there is ever work to extend/update the access model, for example

hiding variables from derived classes (commonly called private)
accessible from within the file defining the class

then using "private" in the docs, when "protected" is the common meaning, likely complicates future changes.
@yegappan, @chrisbra If there's agreement to change the docs from private to protected, and no one wants to address the issue, then I can take a first cut at it and it can be wordsmithed and cleaned up in PR review.
(AFAIK, no response is not agreement)
Expected behaviour
_ is documented as "protected"
Version of Vim
9.0.2092
Environment
ubuntu
Logs and stack traces
No response

fine with me. @yegappan what do you think?

Sounds good to me.

@yegappan so there actually not really a  'private' class var existed? what's the different between _foo and _Foo? seems for now it would not raise err for that.
--
shane.xb.qian

I mean you maybe can make _foo (lower case) work as a real private class var.
OOP language generally had that, tho in vim script not sure if it had to.
--
shane.xb.qian

Sounds good to me.

Can you hold on making these changes?
I am wondering whether we should change the access from protected to private for these variables?
Bram originally intended only for private variables and not for protected variables.  The changes
introduced in 9.0.1804 (inside_class_hierarchy() function) changed this.

Sounds good to me.

Can you hold on making these changes?
I am wondering whether we should change the access from protected to private for these variables? Bram originally intended only for private variables and not for protected variables. The changes introduced in 9.0.1804 (inside_class_hierarchy() function) changed this.

If we go with the current implementation, then in the future, if we need to support private
variables, then we need to either introduce the "private" keyword or find some other way.
As private variables will be more common than protected variables, we can special case
protected variables.

yea, I think it is too late to change that (or adding that 'private' keyword) which had have a very loooong discussion at #12932 since you already made that decision: no 'private' keyword and using _underscore, and the default is 'readonly'.
// tho I may prefer that's not the default, and had a keyword 'private', but just too late to change it back, or change that decision again.
--
shane.xb.qian

Sounds good to me.

Can you hold on making these changes?
I am wondering whether we should change the access from protected to private for these variables? Bram originally intended only for private variables and not for protected variables.

I don't see that. In Bram's last commit of vim9class.txt
Sat Apr 22 22:40:14 2023 +0100
Update runtime files

it says: note the use of (and [in]sub-classes)
https://github.com/vim/vim/blob/f39d9e9dca443e42920066be3a98fd9780e4ed33/runtime/doc/vim9class.txt#L840-L850

The changes introduced in 9.0.1804 (inside_class_hierarchy() function) changed this.

My recollection is that the access model implementation didn't meet spec in several ways.

Sounds good to me.

Can you hold on making these changes?
I am wondering whether we should change the access from protected to private for these variables? Bram originally intended only for private variables and not for protected variables.

I don't see that. In Bram's last commit of vim9class.txt

You are correct.  I built Vim using the last commit from Bram.  In this image,
the child object methods are able to access the private variables in the parent class.
So this is not caused by 9.0.1804.
Interestingly the "No protected object members" section in vim9class.txt states that
only private members are supported.  But the behavior is that of protected members.
Please go ahead with the PR for updating the help text.

Sat Apr 22 22:40:14 2023 +0100
Update runtime files

it says: note the use of (and [in]sub-classes)
https://github.com/vim/vim/blob/f39d9e9dca443e42920066be3a98fd9780e4ed33/runtime/doc/vim9class.txt#L840-L850

The changes introduced in 9.0.1804 (inside_class_hierarchy() function) changed this.

My recollection is that the access model implementation didn't meet spec in several ways.

Interestingly the "No protected object members" section in vim9class.txt states that
only private members are supported. But the behavior is that of protected members.

so conclusion is what? it's really no private class var, and what previously argued the  term 'private' actually not existed and no plan to implement it? or would make _foo work it as it?
--
shane.xb.qian

Interestingly the "No protected object members" section in vim9class.txt states that
only private members are supported. But the behavior is that of protected members.

so conclusion is what? it's really no private class var, and what previously argued the term 'private' actually not existed and no plan to implement it? or would make _foo work it as it?

In the current implementation only protected object variables are supported and private object variables are not supported.

so how about make _foo (lower case) work as private class var? the prefix _ underscore looked like a mark, since already designed like that and vim's tradition usually it should be upper case, then how about make _foo (lower case) work for real private class var?
// dunder __foo and __Foo maybe need to limit to latter one only too.
// though personally I don't like such underscore.
--
shane.xb.qian

note this just is a question, since the 'protect' actually was I recommend (#11827) to Bram,
but/and the 'private' and _underscore was there a lot argu (#12932), so/but now turn out dramatically this is the end?
ok, if yes, I think we can live with it (for now).
--
shane.xb.qian

It looks like Bram intended _ to mean protected behavior all along, see https://groups.google.com/g/vim_dev/c/yYpFNUHdRho.
But for whatever reason, he decided to call it "private", contrary to the accepted meaning of "private" in a class hierarchy. #13520 changes documentation only.


Updating to version 2.3.4

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

@tiagofrepereira2012 this one is a little more tricky you need to further update the meta.yaml file to update its requirements. Look at here please: https://github.com/conda-forge/bob-feedstock/pull/3/files


Failed to generate correct SQL when using ad-hoc and WITH FILL

WITH FILL is moved to the first line (in the generated SQL) when an ad-hoc filter is defined.
If the ad-hoc filter is removed, the generated SQL is correct and the query is working.

@laurent192 could you share original query and generated SQL as text?

Query:
SELECT
    $timeSeries as t,
    sum(FLOW_TOTAL_BYTES) * 8 / $interval AS B
FROM $table

WHERE
    (DATE_PKT_MS_LAST BETWEEN $from AND $to)
GROUP BY t

ORDER BY t WITH FILL STEP ($interval*1000*5)

Generated SQL without the ad-hoc filter:
SELECT
    (intDiv(toUInt32(DATE_PKT_MS_LAST), 50) * 50) * 1000 as t,
    sum(FLOW_TOTAL_BYTES) * 8 / 50 AS B
FROM flows.metadata

WHERE
    (DATE_PKT_MS_LAST BETWEEN<PHONE_NUMBER> AND<PHONE_NUMBER>)
GROUP BY t

ORDER BY t WITH FILL STEP (50*1000*5)

Generated SQL with the ad-hoc filter:
WITH FILL STEP(50 * 1000 * 5)
SELECT
    (intDiv(toUInt32(DATE_PKT_MS_LAST), 50) * 50) * 1000 as t,
    sum(FLOW_TOTAL_BYTES) * 8 / 50 AS B
FROM flows.metadata

WHERE
    (DATE_PKT_MS_LAST BETWEEN<PHONE_NUMBER> AND<PHONE_NUMBER>)
    AND FLOW_HASH_5TUPLE = 435036576
GROUP BY t

ORDER BY t

thanks a lot for reporting


HIVE-27688 hive MoveTask cannot show the correct exception message

What changes were proposed in this pull request?

I modified how the Exception was thrown in org/apache/hadoop/hive/ql/metadata/Hive.java.
Why are the changes needed?

As I have described in HIVE-27688
when you insert into hive table's partition, you will get the correct exception message when you insert into table, but You will get useless message when you insert overwrite table. This will make a confusion when tracing the problem.
Does this PR introduce any user-facing change?

No
Is the change a dependency upgrade?

no
How was this patch tested?

I rebuild the package and test it, it gives me correct exception message.

There is a logic insider getHiveException()
    if (e.getMessage() != null) {
      hiveErrMsg = String.format("%s%s%s", msg, ": ",
          Splitter.on(System.getProperty("line.separator")).split(e.getMessage()).iterator()
              .next());
    } else {
      hiveErrMsg = msg;
    }

Why this doesn't kick in? Can you extend a test case as well. I guess this case is handled via HIVE-16960

@ayushtkn
These two cases might be a little different, because the ErrorMsg returned from these cases are different.
public static ErrorMsg getErrorMsg(Exception e) {
    if (e instanceof AccessControlException) {
      return ACCESS_DENIED;
    }
    if (e instanceof NSQuotaExceededException) {
      return QUOTA_EXCEEDED;
    }
    if (e instanceof DSQuotaExceededException) {
      return QUOTA_EXCEEDED;
    }
    if (e instanceof UnresolvedPathException) {
      return UNRESOLVED_PATH;
    }
    if (e instanceof FileNotFoundException) {
      return FILE_NOT_FOUND;
    }
    return UNRESOLVED_RT_EXCEPTION;
  }

In case HIVE-16960, the Exception is the instance of AccessControlException so the ErrorMsg returned is ACCESS_DENIED, then hiveErrMsg will be showed to users.
But in my case the Exception is HiveException, which cannot be found in the exception list, and ErrorMsg returned will be UNRESOLVED_RT_EXCEPTION. Then the hiveErrMsg will not be showed.
I will try to reproduce this issue in test cases.

@ayushtkn
Hi, I have added a new unit test on how the dirty message was shown, and how to prevent this.
Please help me check this.

I found another case that cannot give me the correct exception message:
when I use pyhive to load data, I accidentally set a wrong config hive.exec.copyfile.maxnumfiles to string "100L". when I try to load data, it returns that (statusCode=3, infoMessages=[ *org.apache.hive.service.cli.HiveSQLException: Error while processing statement: FAILED: Execution Error, return code 1 from org.apache.hadoop.hive.ql.exec.MoveTask. org.apache. hadoop.hive.ql.metadata.HiveException: Unable to move source hdfs://dmp/xxx to destination hdfs://dmp/xxx
this dirty message really confused me, and I checked deeply into the log in hiveserver2, and find the stacktrace of this exception.
It's the same reason with the issue I explained before, the error message was erased in method getHiveException(e, msg).

Why I cannot trigger Jenkins when I made new commit?


How to config the number of GPUs used in re-id?

I ran your all pipeline on AIC, but the program crashed when fork eight GPU processes in re-id. I realized there're only two GPUs in my system. How should I configure the number of GPUs to be implemented? Thank you very much and regards!

See, there's a global var NUM_PROCESS = 8 in extract_image_feat.py. Have altered this and hopefully it works.

Did you solve the problem?


Guide for building WebUSB devices

What's changed, or what was fixed?

This article explains how to build a device to take full advantage of the WebUSB API.

Target Live Date: TBD

[ ] This has been reviewed and approved by (NAME)
[x] I have run npm test locally and all tests pass.
[ ] I have added the appropriate type-something label.
[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

CC @beaufortfrancois


Allow for NaNs for drainage and infiltration

This is a very opportunistic fix to #1027
I'm not sure the !isnan(value) check currently does anything meaningful to be honest?
It's still a bit tedious to setup: to get it working I need to explicitly set the first date to zero, otherwise it triggers the following error:             error("Missing initial data for the Basin variable drainage")
One way of dealing with this would be to initialize to zeros, and allow "unset" values for optionals. Not sure about it.
Needs a decision, and then we could add some simple tests.

From the imod coupler perspective, it's a bit onerous. Having to set all entries to NaN, except the first feels very specific.
Ideally, I think the following would work as preparation for the coupling step:
df = ribasim_model.basin.df
df.loc[df["node_id"].isin(coupled_node_ids), ["drainage", "infiltration"]] = np.nan

Requires altering this:
    drainage = fill(NaN, length(node_id))
    infiltration = fill(NaN, length(node_id))

using zeros instead and ommitting the check.
Having zero as values if they are not defined feels reasonable?

Judging from the test, this does exactly the right thing. I'll try and run this branch on one of my local models to be sure.


Enable autoloading with puppet 2.7

Hi
I moved all the files in their correct place so that they can be autoloaded without using "import puppet-rvm".

Thanks for this pull request.  Works great.


.btn-<sm/lg> definitions in _buttons.scss doesn't use $input-btn-line-height-<sm/lg> variable in Bootstrap 4 beta 1

The new variables $input-btn-line-height-sm/$input-btn-line-height-lg has been introduced in the latest beta of Bootstrap 4, but this isn't used by the .btn-sm/.btn-lg definition in _buttons.scss, this uses general $line-height-sm/$line-height-lg.
The $input-btn-line-height-sm/$input-btn-line-height-lg should also be defined to default beeing $line-height-sm/$line-height-lg
The .btn definition uses the correct $input-btn-line-height.

True, those are used everywhere else, but not in the _buttons.scss file.

First and last paragraphs have been resolved already, but just added a commit in #24199 to fix the middle paragraph's issue.


Fix the test incorrectly failing

Fixes #49
Details are explained in the issue #49

Codecov Report

Merging #50 into master will decrease coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #50      +/-   ##
=========================================
- Coverage   75.87%   75.7%   -0.17%
=========================================
  Files          20      20
  Lines        1202    1202
=========================================
- Hits          912     910       -2
- Misses        233     235       +2
  Partials       57      57

Impacted Files
Coverage Δ

agent.go
79.33% <0%> (-0.56%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks for fixing this! Now I remembered I looked into this in the past: https://gophers.slack.com/archives/CAK2124AG/p1556800113113800


incorporate @bgooren's commit to enable nesting and dropping on connected list at same index

See https://github.com/farhadi/html5sortable/pull/54

Happy to accept a PR.


Provide a more specific Activation event

currently it is always activated.
https://github.com/jboss-fuse/vscode-datavirt/blob/3c73b7a7410f1d5c6893f75b74d9cd0c613df46f/package.json#L133-L135
I think it can be nice to restrain to:

yaml file in workspace
one of the provided commands called
some other?

Please create a FUSETOOLS2 issue for this.

moved to JIRA https://issues.redhat.com/browse/FUSETOOLS2-387


Enhan affine cython

--Added cython functions for Affine measure to improve the performance
--Modified the test cases for Affine measure
--Added functions to find max int and max float in cython_utils.pyx

Coverage decreased (-0.002%) to 99.903% when pulling 4bf42a88b6d2ee4f2a5551e0b9ec6ef90ae10088 on enhan_affine_cython into bf97560f18e0972423bbde8d4dd0039491310bd5 on master.


VT underline, overline, strikethrough, hyperlink are not high-DPI aware

In Windows Terminal 1.4, the rendering of VT attributes such as underline, double underline, overline, crossed and hyperlinks are always 1px thick and hyperlink dashes are always 3;1 pixels regardless of the screen DPI and UI scaling factor.
This makes it very think, and the dashes of hyperlink almost invisible on high DPI displays such as the Surface Book 2 (which defaults at 200%).
It would be more consistent and more readable if the lines and dashes were also scaled according to the scaling factor or to the current font size.
See the following screenshot on Surface Book 2:
echo -e "\e[4mUnderline\e[24m \e[21mDblUL\e[24m \e[53mOverline\e[55m \e[9mCrossed\e[29m \e]8;;http://www.example.com\aHyperlink\e]8;;\a"

They are definitely DPI aware. It's just that they have a minimum size of a pixel (so they're not invisible at low resolution), and some fonts are styled to use a very narrow width that is only 1 pixel even when the DPI is high. So while that would technically be less than a pixel at a lower DPI, it gets rounded up to 1, and thus there doesn't appear to be any difference between the two.
I know Cascadia is particularly thin - Consolas uses a more reasonable width in my view. But note that if your font size is small, that's also more likely to result in an underline that could still be 1px at high DPI.

This one is operating by design. Thanks for filling in the details @j4james! :smile:

Just for reference, this was an issue with the Cascadia font (https://github.com/microsoft/cascadia-code/issues/395) and got resolved in Feb 2021 releases with the new font version.


nightwatch

I cant get drag and drop to work in nightwatch. It selects the target just fine, but afterwards it wont move to the destination to 'mouseButtonUp()

Thank you for the bug report.
Could you show your nightwatch.json to me?

{
"src_folders": [
"tests"
],
"output_folder": "reports",
"custom_commands_path": "custom-commands",
"custom_assertions_path": "./node_modules/nightwatch-commands/assertions",
"page_objects_path": "pages",
"globals_path": "globals.js",
"selenium": {
"start_process": true,
"server_path": "./bin/selenium-server-standalone-2.52.0.jar",
"log_path": "",
"host": "<IP_ADDRESS>",
"port": 4444,
"cli_args": {
"webdriver.chrome.driver": "./bin/chromedriver"
}
},
"test_settings": {
"default": {
"launch_url": "http://localhost",
"selenium_port": 4444,
"selenium_host": "localhost",
"silent": true,
"screenshots": {
"enabled": true,
"on_failure": true,
"on_error": true,
"path": "test/functional/reports/screenshots"
},
"desiredCapabilities": {
"browserName": "firefox",
"javascriptEnabled": true,
"acceptSslCerts": true
}
},
"chrome": {
"screenshots": {
"enabled": true,
"path": /test/functional/reports/screenshots"
},
"desiredCapabilities": {
"browserName": "chrome",
"javascriptEnabled": true,
"acceptSslCerts": true
}
},
"firefox": {
"screenshots": {
"enabled": true,
"path": "/test/functional/reports/screenshots"
},
"desiredCapabilities": {
"browserName": "firefox"
}
}
}
}
2016-04-11 17:18 GMT+02:00 Kuniwak<EMAIL_ADDRESS>
Thank you for the bug report.
Could you show your nightwatch.json to me?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/Kuniwak/html-dnd/issues/2#issuecomment-208398133

'step 4: draganddrop': function (browser) {
browser
.execute(dragAndDrop, ['#main-container > div > div >
div.fc-region-container.fc-container.ui-sortable > div > div.card-list

div.fc-card-list-container > div > div > div:nth-child(1) >
div.panel-body.js-column-container > div > div >
div.fc-region-container.fc-container.ui-sortable > div:nth-child(2) >
div:nth-child(2)', '#main-container > div > div >
div.fc-region-container.fc-container.ui-sortable > div > div.card-list
div.fc-card-list-container > div > div > div:nth-child(2) >
div.panel-heading'])        // .waitForElementVisible('#main-container
div > div > div.fc-region-container.fc-container.ui-sortable > div >
div.card-list > div.fc-card-list-container > div > div >
div:nth-child(1) > div.panel-body.js-column-container > div > div >
div.fc-region-container.fc-container.ui-sortable > div:nth-child(2) >
div:nth-child(2)', globalVars.timeout)

  // .moveToElement('#main-container > div > div >

div.fc-region-container.fc-container.ui-sortable > div > div.card-list

div.fc-card-list-container > div > div > div:nth-child(1) >
div.panel-body.js-column-container > div > div >
div.fc-region-container.fc-container.ui-sortable > div:nth-child(2) >
div:nth-child(2)', 1, 1)
// .mouseButtonDown()
// .moveToElement('div.panel-cards-list:nth-child(2) >
div:nth-child(2)', 1, 1)
// .moveToElement('div.panel-cards-list:nth-child(2) >
div:nth-child(2)', 5, -5)
// .pause(5000)
// .mouseButtonUp()
.end();
}

2016-04-11 17:37 GMT+02:00 Lars Nijboer<EMAIL_ADDRESS>
{
"src_folders": [
"tests"
],
"output_folder": "reports",
"custom_commands_path": "custom-commands",
"custom_assertions_path": "./node_modules/nightwatch-commands/assertions",
"page_objects_path": "pages",
"globals_path": "globals.js",
"selenium": {
"start_process": true,
"server_path": "./bin/selenium-server-standalone-2.52.0.jar",
"log_path": "",
"host": "<IP_ADDRESS>",
"port": 4444,
"cli_args": {
"webdriver.chrome.driver": "./bin/chromedriver"
}
},
"test_settings": {
"default": {
"launch_url": "http://localhost",
"selenium_port": 4444,
"selenium_host": "localhost",
"silent": true,
"screenshots": {
"enabled": true,
"on_failure": true,
"on_error": true,
"path": "test/functional/reports/screenshots"
},
"desiredCapabilities": {
"browserName": "firefox",
"javascriptEnabled": true,
"acceptSslCerts": true
}
},
"chrome": {
"screenshots": {
"enabled": true,
"path": /test/functional/reports/screenshots"
},
"desiredCapabilities": {
"browserName": "chrome",
"javascriptEnabled": true,
"acceptSslCerts": true
}
},
"firefox": {
"screenshots": {
"enabled": true,
"path": "/test/functional/reports/screenshots"
},
"desiredCapabilities": {
"browserName": "firefox"
}
}
}
}
2016-04-11 17:18 GMT+02:00 Kuniwak<EMAIL_ADDRESS>
Thank you for the bug report.
Could you show your nightwatch.json to me?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/Kuniwak/html-dnd/issues/2#issuecomment-208398133

Thanks you, @ForteDell .
I have a question.
How do you implement drag and drop feature?

Using HTML Drag and Drop API

This element is draggable.

Using a library for drag and drops such as jQuery UI Draggable
$(function() {
  $( "#draggable" ).draggable();
});

This is a important question because html-dnd can simulate only the first approach.

I also have a problem on making the patch work on Nightwatch. It's different, though, so I don't know if you want me to open a separate issue. It gives me this error:
value:
   { message: 'java.lang.String cannot be cast to java.util.List',
     suppressed: [],
     localizedMessage: 'java.lang.String cannot be cast to java.util.List',
     cause: null,
     class: 'java.lang.ClassCastException',
     hCode: 413477975,
     screen: null },
  state: 'unhandled error',
  class: 'org.openqa.selenium.remote.Response',
  hCode:<PHONE_NUMBER> }

Any idea? I am using a minimal test case:
var dragAndDrop = require('html-dnd').codeForSelectors;

module.exports = {
  'drag and drop': function(browser) {
    browser
      .url('http://html5demos.com/drag')
      .execute(dragAndDrop, '#one', '#bin')
      .pause(3000) // Just to see if it really happened
      .end();
  }
};

Sorry for my late reply.
@sunyatasattva
The example for nightwatch in README had been wrong (#5).
Try this:
var dragAndDrop = require('html-dnd').codeForSelectors;

module.exports = {
  'drag and drop': function(browser) {
    browser
      .url('http://html5demos.com/drag')
      .execute(dragAndDrop, ['#one', '#bin'])
      .pause(3000) // Just to see if it really happened
      .end();
  }
};

Hello @Kuniwak thank you for your reply! Indeed this solves the error I mentioned, however, drag and drop still doesn't work. Have you tried the code you posted yourself (I mean, with a real drag and drop demo such as the one linked)? Does that work for you?
It doesn't do anything in my case, unfortunately.
By the way, thank you for your work: this is an extremely important issue for the task I am currently working on, as our application depends mainly on drag and drop. Thanks for all your effort.

@sunyatasattva I added compatibility tests for Nightwatch.js (#7), but it seemed working fine.
But I do not test on http://html5demos.com/drag yet.
I'm gonna check it too soon.

Oh, I misunderstood the spec of Drag data stores.
I'm gonna fix it. Please wait for some days.

@Kuniwak Thanks for your quick reply! Can you quickly explain what is the difference between the html5demos and you own test?

html-dnd should use a shared DataTransfer object between a drag-and-drop operation, but html-dnd does not do it:
  var dragstartEvent = dnd.createEventWithDataTransfer(
    dnd.EventType.DRAG_START);
  dispatchEvent(draggable, dragstartEvent);

  // Using a new DataTransfer object...
  var dropEvent = dnd.createEventWithDataTransfer(dnd.EventType.DROP);
  dispatchEvent(droppable, dropEvent);

  // Using a new DataTransfer object...
  var dragendEvent = dnd.createEventWithDataTransfer(dnd.EventType.DRAG_END);
  dispatchEvent(draggable, dragendEvent);

I confirmed fix it on http://html5demos.com/drag .
@sunyatasattva Please update to<EMAIL_ADDRESS>and retry it.

Thanks @Kuniwak ! You single-handedly saved our project! Do you accept tips or donations?

@ForteDell Hi, did you resolve this problem?

Unfortunately not for me, but I'm more and more inclined to think it is
because of the specific page I'm testing. So let's call it resolved for now.
2016-05-08 15:23 GMT+02:00 Kuniwak<EMAIL_ADDRESS>
@ForteDell https://github.com/ForteDell Hi, did you resolve this
problem?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/Kuniwak/html-dnd/issues/2#issuecomment-217718397

How can we achieve drag and drop if the drag-gable and droppable has no Id?


Remove SQSReader from worker

What is this PR trying to achieve?
Simplify the MessageWorker by pushing the SQSReader inside the MessageReader class.
Who is this change for?
🍔
Have the following been considered/are they needed?

[ ] Deployed new versions

This PR also seems to be splitting MessageConfig into MessageReaderConfig and MessageWriterConfig. Could we get those changes as a separate PR?

Addressed review comments. As for splitting the PR, the commit history is in a state where it would be a bit of a hassle to split. Also, the reason for splitting the config class came from the changes to MessageReader, so I don't think they are totally independent changes


Update classfile.cc

PRIx32 macro needs __STDC_FORMAT_MACRO defined on systems instead of __STDC_LIMIT_MACROS

Sorry we don't have the tooling to accept pull request now. Your change have been merged


localIdeographFontFamily: false to load CJK fonts

close #892
this changes works well for CJK fonts.

What would this change do to latin characters? Will they still be using the glyphs?
Also worth opening a PR for maplibre-gl to fix the type of the variable.

@HarelM Thank you for checking this PR.
This change seems not to make no change on latin characters for me.

Let's use this PR to update maplibre-gl to avoid introducing "as any", it should be a few days at max.

4.1.2 should be available now.
Let me know if it solves the as any part.

@HarelM Thank you for supporting! I've fixed.

Should be published in a few minutes, let me know if you encounter any issue with this.


importorskip considered harmful?

pytest.importorskip is routinely used in our tests to skip tests which can only pass when some external dependency is really installed.
This can lead to a whole set of tests being skipped, when there is a single typo in this line:
pytest.importorskip('external_dependency_name')

This skip does not even show the tests being skipped in yellow in the verbose output, they are skipped entirely. See the screenshot below .. I deliberately added a typo in test_sst1meventsource.py in
pytest.importorskip('protozfits_typo!!!!')

which resulted in all tests from that file being skipped without any notice in the output, there should be something between the two highlighted lines.

I am bringing up this topic, because I wanted to ask if supporting this kind of optional dependencies might do more harm than good. What would happen if we'd simply say: This-and-that library is a dependency. Just like numpy is a dependency.
Wouldn't that make our lives and the lives of our colleagues much much easier? And possibly result in a more stable product?

I agree this is bad. For some reason I remember seeing it explicitly informing when these tests are skipped, but I can't find anything that backs up my memory. It seems strange for them to be omitted entirely....
The pytest -rs option does add a summary of the reasons why certain tests were skipped...but isn't a full solution.

I know @pytest.mark.skipped will still print them, but say they are skipped (as in your screenshot), but I'm not sure the best way to do  tests optionally...
Certainly in the continuous integration test suite, we can just ensure all necessary libraries are installed, so it only really affects users running "pytest" locally.

we could also use custom markers, but I don't really like that either, since if the user doesn't enable the marker, the test is skipped.

This is what gammapy does: https://github.com/astropy/astropy/issues/5543#issuecomment-265434710
May be nicer, since it would say the test is skipped

Just so I get this right.
At the moment we use importorskip so that end-users who do not have some of the dependencies installed can execute the tests.
https://github.com/cta-observatory/ctapipe/search?utf8=✓&q=importorskip&type=
We ... in our roles of ctapipe devs would always like to execute all of the tests, and it is dangerous when tests are skipped without us noticing.
Carefully reading the travis output with a human eye to spot skipped tests, feels like a contradiction to the whole continuous integration idea.
What if we would use custom markers, but by default execute all the tests. An end-user who has not all external dependencies installed will thus see failing tests. Some documentation could tell them how to skip some tests, if the know the do not have the external dependencies installed.
This way:

users are made aware that they do not have all dependencies installed
we can be sure all tests are always executed.

Alternatively we could provide different pytest-ini files .. and tell users to do:
pytest -c no_external_deps.ini

while we do simply: pytest.

Or maybe do it the other way around? Add an option to execute all tests that is used on travis?
like this maybe:
if os.getenv('FORCE_ALL_TESTS'):
    import external
else:
    pytest.importorskip('external_dependency_name')

I personally like the gammapy approach. Its simple, and it ensures that if the test is skipped, then it is clearly shown as skipped in the output summary.

But we don't want skipped tests on travis, do we?

True, we don't. How about something similar to both the gammapy approach and with your suggestion: if FORCE_ALL_TESTS has been defined in the environment then the decorator does not skip the test.
Then at least the its more clear on developers machines that some tests have skipped because of a missing dependency. I don't like that the skipped tests are hidden at all.

But we don't want skipped tests on travis, do we?

I don't mind seeing skipped tests, as long as we know why. I think with pytest you can supply a reason for skipping that can be optionally printed.   On travis we can strive to make sure all dependencies are there, hence no skipping.  Right now a few are skipped since they require some input files that are not yet available (like the ImPACTReconstructor needs a model table), but those can be resolved in time.

We want to go with something like
if os.getenv('FORCE_ALL_TESTS'):
    import external
else:
    external = pytest.importorskip('external_dependency_name')

or
ctapipe.import_or_skip
to force running all tests on travis.


[WFCORE-4147] Delete the gc.log after the -version check to see if the GC settings are valid.

https://issues.jboss.org/browse/WFCORE-4147
This replaces #3542

@marekkopecky Could please review this?


PTVS no longer discovers my Unit Tests

Up until recently, I was able to see my unit tests in the VS Test Explorer. Now no tests appear in the Test Explorer window. When I change my project, the output window shows:
------ Discover test started ------
========== Discover test finished: 0 found (0:00:06.2472349) ==========

However there are dozens and dozens of tests that as recently as last week appeared. I tried updating to the latest PTVS but that didn't fix anything. The wiki notes a "beta caveats" for known issues and workarounds but I don't know where those are.

Could you open Tools, Python Tools, Diagnostic Info and either post the contents as a gist and link it here, or email as an attachment to<EMAIL_ADDRESS>

I sent the requested info to the alias.

Thanks @zooba!  Deleting my PythonTools\CompletionDB folder and rebuilding worked for me!


bug: console.sh < test.osql

The console.sh script should be able to take input from a file without going into some kind of infinite loop, but using 2.1.7-SNAPSHOT, it does just that using redirection (console.sh < test.osql).
(The contents of test.osql can be as simple as: set echo true;)
By the way, there is something wrong with the directions given on the page
http://orientdb.com/docs/2.1/Console-Commands.html
In particular the line:
orientdb> console.bat commands.txt
seems to indicate that "console.bat" (or "console.sh" in a non-Windows environment) is an OrientDB console command, which of course it isn't.
One might think that the line in question is meant to imply that one should enter "console.sh commands.txt" at the system prompt, but that does not work either (not surprisingly).

From documentation, redirection isn't necessary: console.sh  is enough.
I can't find:
orientdb> console.bat commands.txt
Can we close this?

@robfrank wrote:

Can we close this?

My answer is: absolutely not.
It would be easy for console.sh to support redirection, as it should. See the original description of the problem.  (The point is that STDIN need not be a file.)
Regarding the documentation -- yes, I believe that the "bug" in the 2.1 documentation has been fixed. (Thank you to the person who fixed it!)

If you know how to fix it, a PR is welcome and it will be part of next releases.
Thank you for reporting.

Docs updated, closing it


Improve CompositeKind handling

The current behavior for CompositeKind data is
https://github.com/JuliaIO/MAT.jl/blob/355bd1a901730ad5713b3283f6ec66603c72dfd9/src/MAT_HDF5.jl#L529-L536
however, there are issues with the isbits(s) test, reported in #162,#143 or with simple user types such as
struct MyType
    field1::Int64
    field2::Int64
end

While these types are clearly isbitstypes, I do not see a reason to not allow their conversion the equivalent matlab struct.
Given
https://github.com/JuliaIO/MAT.jl/blob/355bd1a901730ad5713b3283f6ec66603c72dfd9/src/MAT_HDF5.jl#L535
The requirement seems more to be that fieldcount(T) > 0, regardless of the actual bits type.
Can someone confirm this ?

Bump ?


Fixes #47

Added a history listener, which closes all pointers during page navigation.
As far as I can tell, it's hard to detect navigation on individual workspace components; hence we just do it globally.  The longer term/future solution is to include the workspace ID as part of the stored pointer state and thread this through to the pointer components, so that expansion can be preserved per workspace.

Hi Corey! A few (minor) comments:

It might make sense to put the HistoryListenerWithRouter component in a separate file.
I would probably use connect & mapDispatchToProps with HistoryListenerWithRouter instead of calling store.dispatch directly.
The following action creator

export const closeAllPointerReferences = () => {
  return (dispatch, getState) => {
    dispatch({
      type: CLOSE_ALL_POINTER_REFERENCES
    });
  };
};

could just be
export const closeAllPointerReferences = () => ({
  type: CLOSE_ALL_POINTER_REFERENCES
});

since this isn’t using redux-thunk functionality.

Using history.listen like this means that pointers close when the hash portion of the url changes (I'm pretty sure), and this is something we might not want, though we don’t really use url fragments in the app right now, so this is more a forward-thinking point.

I've cleaned this up to address some of the issues both of you mentioned:

It's now in a separate file.
It uses the standard connect with mapDispatchToProps pattern.
I found a simpler way to write it without the listen/unlisten pattern, which should be clearer.
It should now only close pointers on pathname change, not fragment change.

I also renamed it to "ClosePointerListener" since it's likely to only ever do that one job.  I couldn't figure out any more elegant way to do this with react-router; let me know if you have any recommendations.


0.0.10

@rultor release, tag is 0.0.10

@rultor release, tag is 0.0.10

@h1alexbel OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.10

@h1alexbel Done! FYI, the full log is here (took me 3min)


Received data is a binary file

We have many requests that return text files.
In previous versions of ARC the contents of the file would be displayed in the response window, but now it prompts to save the file and open in order to view.
Viewing in the window is more convenient when quickly testing several requests.
Is there a setting to switch back so it can be viewed directly in the window?
If no, is it possible to restore the previous functionality as an option?

Hi,
Can you please tell me what's the content type of the response? The binary view is only visible when the file is actually a binary file and therefore there's nothing to see in it. It shouldn't treat text files as a binary files though.  What is the content type of the response?

Upon reviewing the server code I need to revise my statement regarding the return type of this call.
Our client app converts the response into a text file, but the server is actually returning "application / octet-stream". The server is Java based and produces a file containing SQL statements , which is read into a byte array. The byte array is then returned as the response.
Currently, ARC prompts for the save which makes a log file containing the SQL statements.
I believe that in this past the Response view showed the SQL statements without needing to export to the log file.

So there is an update almost ready (currently beta channel has it) where the "raw" view is always displayed regardless the response content type. I guess it will fix this issue.

That is good news! Thank you


Help with image attribution after copy / paste

Dendron makes it really easy to insert images by copying from a browser tab and pasting it in the desired Markdown document. Dendron then ensures the image ends up in the assets folder and generates a filename for it, based on the current datetime.
Great feature! I love it :heart:
But people on the internet in general follow very bad practices when it comes to respecting copyright and attribution. Google Images is regarded as a 'steal-what-you-want' treasure chest. Personally I only use open licenses or public domain images, and take care to attribute properly where the license requires this.
The image copy-paste feature is really productive, but it encourages the 'steal' mentality because I lose the information on the original filename and URL location of the image. Of course one can do the attribution manually at the time of pasting (this is the only time where you can still associate the old and new filename). But Dendron could also provide various degrees of help.
Now this feature can be designed various ways. I will just give some suggestions of what this might include:

Dendron creates a file in assets with a lookup table of source-URL --> new-image-filename, adds entries on each copy-paste.

This is insufficient for attribution, but allows me to do that work later on, and can serve as proof of where I found the img.

Dendron (maybe based on config) auto-includes a text below the image linking to the original source, or hyperlinks the image.
Dendron generates a full attribution text (template set in config?) based on metadata found in the image.

Just saw that the image copy-paste is coming from another plugin project, so created an issue there as well.

thanks for the feature request. I totally agree with you about the copy right suggestions.
dendron actually already has an attribution feature when it comes to content on the page: https://dendron.so/notes/f9c646b1-34ab-4b18-8ec1-cabb8b2c7066.html#sources-source
having attribution on the pictures is a logical next step.
the paste image extension is a fork of another project so we can control the configuration options here as well and contribute it upstream depending on which one of us gets to it first
i like adding additional metadata at copy time. there's two ways that come to mind:

using a separate assets file (your suggestion)
doing it inline as a markdown comment below the image

adding it to a metadata file would make it easier to auto-generate attributions. do you have any suggestions for the auto-generated attribution text?

The content attribution is a nice feature indeed. I guess it gives a third option for images, and that is to auto-append to the frontmatter in the same way (but then it should also do that if you reuse the image elsewhere in your notes, and a separate assets file needs to be consulted for that, probably).
As for a markdown comment below an image, I think different people have different preferences. I don't know the details on what constitutes a minimum valid attribution, but based on available metadata people could decide to provide more info. They might set a template for their preferred attribution format.
I usually attribute below the image between parentheses and in italic text, e.g.

(credits: Jane Doe, source, CC-BY-3.0)

This is a shorthand form. If e.g. Title and Author homepage are available it could be:

(Image: "Use your brain. Use Dendron" by Kevin Lin, licensed under CC BY 3.0)

Note that if there is no Author and/or no License information, then a correct attribution is impossible and one shouldn't use the image. Now Dendron should maybe not restict people that want to still use unattributed images, but it could highlight (with red marker or something) where attribution is missing.

Seems reasonable. I'm going to file this under December milestones just because or November is pretty packed at the moment. Will update this if we get to it before hand

Just noting that this issue was mentioned in a discussion about visual UIs in the Discord here: https://discordapp.com/channels/717965437182410783/736965464261132310/773273556858568723

[...] we currently have an issue to add metadata to assets for image attribution: https://github.com/dendronhq/dendron/issues/324

i can imagine we extend that to add arbitrary metadata for assets (eg. think frontmatter but for assets)

this would allow us to build visual UIs for your assets like milanote.

[...]


Update welcome window to match PR on Mirror.

Updated UI and added function to open to change log if the user downloaded a new version.

Please test if you are able to open up to the change log window after the version has been changed.
You can test by editing the GetVersion() method to return a different string.

@Lymdun Please test if the welcome window does the following:

Open to changelog if there is a new version.
After closing, reopening should bring you to the welcome page and not show Change Log (updated) in the change log page.
First time opening (change firstTimeMirrorKey key) brings you to the welcome page.


Docs change: Correct the URL for objectnav_mp3d_v1.zip

Motivation and Context
The original URL in DATASETS.md was incorrect, resulting in an error when attempting to retrieve objectnav_mp3d_v1.zip. The incorrect URL was:

I investigated the old commit history and identified the correct URL:

This appears to be a minor oversight. I kindly request that Meta AI update the URL at dl.fbaipublicfiles.com from m3d to mp3d to resolve the issue.
How Has This Been Tested
I successfully downloaded it from the corrected URL.

Types of changes

[Docs change] Addition or changes to the documentation
issue #1558

Checklist

[x] My code follows the code style of this project.
[x] I have updated the documentation if required.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[x] I have added tests to cover my changes if required.

Thanks!


Update CONTRIBUTING - using gitflow workflow, development branch inst…

…ead master

Fixes #102
Checklist

[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the master branch.
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation about the functionality in the appropriate .md file
[x] I have added in line documentation to the code I modified

Short description of what this PR does:

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

Hello @pushkyn,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Hello @pushkyn,Thanks again for the PR!                                         You have earned 21 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.                                         Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.                                         If you have any questions you can email us at<EMAIL_ADDRESS>                                        Thank you and Happy Hacktobering!                                         Team SendGrid

Closing all open issues and PRs. This repo is no longer maintained. Please use https://github.com/sendgrid/sendgrid-nodejs/tree/master/packages/client instead.


v2/assets/assetid/balances does not work with round

Subject of the issue
I want to list specific owners of asa at specific round.
https://testnet.algoexplorerapi.io/idx2/v2/assets/44526812/balances?round=17866412
It seems that round parameter is not considered in the query.
Documentation says: Include results for the specified round.
This is not true.
Tested also on mainnet https://algoexplorerapi.io/idx2/v2/assets/384303832/balances?round=17279644
Your environment
Indexer v 2.3.1
Steps to reproduce

Go to https://testnet.algoexplorerapi.io/idx2/v2/assets/44526812/balances?round=17866412

Expected behaviour
I do not want to see lines of accounts who optedin after round 17866412
Please either remove round parameter or make it work.
Actual behaviour
I see more information than it should show.

please see #374. The round parameter is not implemented for this endpoint, we will likely be removing it.

i really need to know the balances of the account at specific round
@algoanne  what is your suggestion how to do it?

@algoanne .. you mean you are going to fix the issue with rewinding? https://forum.algorand.org/t/rewinding-past-txn-type-appl-is-not-currently-supported/5930/3
https://github.com/algorand/indexer/issues/384#issue-826652795

no, currently app txns are not supported by rewind.
if you want to check multiple accounts you would have to do that one at a time, or use "--dev-mode" when starting indexer to allow rewinding with the /v2/accounts search endpoint


ol/webgl is prepending style match statement to icon-src value

Describe the bug
ol/webgl is prepending style match statement to icon-src value.
Rendering fails with the following error:
Helper.js:
WebGL: INVALID_VALUE: texImage2D: bad image data
GET http://localhost:5173/match,get,Enforceable_Commitment,Y,data:image/svg+xml;utf-8,<svg xmlns%3D"http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg" width%3D"48" height%3D"48" viewBox%3D"0 0 48 48" ><circle fill%3D"" stroke%3D"%23000" stroke-width%3D"3" cx%3D"24" cy%3D"24" r%3D"19"%2F><circle fill%3D"%23008450" stroke%3D"%23fff" stroke-width%3D"3" cx%3D"24" cy%3D"24" r%3D"16"%2F><path fill%3D"%23fff" transform%3D"translate(12%2C 12)" d%3D"M20.285 2l-11.285 11.567-5.286-5.011-3.714 3.716 9 8.728 15-15.285z"%2F><%2Fsvg>,N,data:image/svg+xml;utf-8,<svg xmlns%3D"http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg" width%3D"48" height%3D"48" viewBox%3D"0 0 48 48" ><circle fill%3D"" stroke%3D"%23000" stroke-width%3D"3" cx%3D"24" cy%3D"24" r%3D"19"%2F><circle fill%3D"%23CC0202" stroke%3D"%23fff" stroke-width%3D"3" cx%3D"24" cy%3D"24" r%3D"16"%2F><path fill%3D"%23fff" transform%3D"translate(13.2%2C 13.2) scale(0.9 0.9)" d%3D"M24 20.188l-8.315-8.209 8.2-8.282-3.697-3.697-8.212 8.318-8.31-8.203-3.666 3.666 8.321 8.24-8.206 8.313 3.666 3.666 8.237-8.318 8.285 8.203z"%2F><%2Fsvg>,data:image/svg+xml;utf-8,<svg xmlns%3D"http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg" width%3D"48" height%3D"48" viewBox%3D"0 0 48 48" ><%2Fsvg> 404 (Not Found)
To Reproduce
Steps to reproduce the behavior:
Run example code from https://github.com/timkeane/ol-web-gl-issue
Expected behavior
The icon source should load from the provided data URI

@timkeane thanks for your report, unfortunately the style format described here isn't yet supported by the WebGL renderers (including the WebGLPoints) class. This is still a work-in-progress, WebGL renderers will hopefully support the "flat" styles very soon.
In the meantime, please refer to examples such as this one for filtering in webgl.

@mike-000 provided the solution


Streamlined Direct Message Flow: Pop-up Dialog for Messaging

Issue Reference:
Closes #4301
Description
This PR implements a streamlined flow for direct messaging from user profiles. Instead of redirecting to the messages page and requiring the user to refresh to see new messages, a pop-up dialog is now presented directly on the user profile page. This dialog allows users to type and send messages seamlessly.
Key Changes:

Added a dialog box component in the user profile for entering messages.
Integrated logic to check for existing chats and create new ones if necessary.
Enhanced error handling for thread creation and message sending (i.e cannot send empty / blank message).

Backend checklist

[x] Run the backend locally and it works

Web frontend checklist

[x] Formatted my code with yarn format
[x] There are no warnings from yarn lint --fix
[x] There are no console warnings when running the app
[x] Clicked around my changes running locally and it works
[x] Checked Desktop, Mobile and Tablet screen sizes

It was added in this commit: 73e6a211471691184edb2848d774e8428dbd3066.
You'll have to either remove those files and then collapse all commits into one, or you can rebase and fix up that one commit if you want to preserve history.

Also please remove the verbose commenting from places, it makes it look like it's fresh out of ChatGPT :P

@mrahmed2026: will you be continuing this PR and pushing it to completion?

Continued in #5392

This is actually unusable and needs to be completely re-done.


Fix Rope Compatibility with Cos/Sin Position Embedding for Batch Size > 1

Summary
Fix Rope Compatibility with Cos/Sin Position Embedding for Batch Size > 1
This PR addresses an issue with the compatibility of the ROPE  implementation when using cosine/sine position embeddings with a batch size greater than 1.
In the default behavior of transformers, position_ids is set to None during training, which results in the following computation:
cache_position = torch.arange(seq_len)
position_ids = cache_position.unsqueeze(0)

This leads to the shape of the position embeddings being (1, seq_len, head_dim), which is consistent with the implementation in Liger.
However, if position_ids are pre-calculated for any reason(In my experiment, I implement  m-rope in another different way making position_ids pre-calculated), the current implementation fails to handle this scenario correctly. This PR introduces a fix to ensure that the ROPE implementation can accommodate pre-computed position_ids.
In the unit test test_ropy.py, I have added a variable expand_position_ids to simulate this condition. The previous implementation fails under this scenario, while the new patch successfully resolves the issue.
pytest details:

Testing Done

Hardware Type:
[x] run make test to ensure correctness
[x] run make checkstyle to ensure code style
[ ] run make test-convergence to ensure convergence

Thanks for the enhancement!


Question: EDS + active health check endpoint removal behavior

Envoy Version: 1.12.3
I'm looking for clarification on the behavior of Envoy when using EDS combined with active health checks. I'm observing behavior highly similar to what was fixed in #6765, but having trouble reasoning about why this is the intended behavior.
The specific issue I'm observing is an extended period of time after which an endpoint is removed via EDS in which Envoy continues to attempt routing traffic to it while actively health checking.
The following timing sequence is observed:

Time
Action

19:01:47
ClusterLoadAssignment received with new endpoint added and old endpoint removed

19:01:56
Active health check performed with trace output connection/stream timeout health_flags=/pending_dynamic_removal

19:01:57 - 19:10:55
Active health checks continue to fail with same output, requests fail to endpoint because it doesn't exist anymore

19:10:56
Final active health check failure with trace output connection/stream timeout health_flags=/pending_dynamic_removal, cluster membership updated

Cluster configuration:
  {
   "@type": "type.googleapis.com/envoy.admin.v2alpha.ClustersConfigDump",
   "version_info": "971951176",
   "dynamic_active_clusters": [
    {
     "version_info": "971951176",
     "cluster": {
      "name": "egress_target_http_80",
      "type": "EDS",
      "eds_cluster_config": {
       "eds_config": {
        "ads": {}
       }
      },
      "connect_timeout": "1s",
      "health_checks": [
       {
        "timeout": "30s",
        "interval": "30s",
        "unhealthy_threshold": 10,
        "healthy_threshold": 2,
        "http_health_check": {
         "path": "/ping"
        }
       }
      ]
     },
    }

My questions are:

Is it the case that when an endpoint undergoing active health checks is removed via EDS, it must breach the unhealthy threshold before it is finally removed?
Is my assumption correct that the approximate 10 minutes of active health checking is due to the combination of the interval and timeout both being 30s?
Assuming (1) and (2) are both "Yes," is there a best practice for forcing the cluster to remove the endpoint upon EDS update prior to the active health check reaching its unhealthy threshold?
Naive question: Why isn't the endpoint removed from the cluster immediately following the EDS update? I would expect Envoy allows any in-flight connections to drain, but any future connections should be sent to the remaining endpoints for the cluster.

Is it the case that when an endpoint undergoing active health checks is removed via EDS, it must breach the unhealthy threshold before it is finally removed?

By default, yes.

Is my assumption correct that the approximate 10 minutes of active health checking is due to the combination of the interval and timeout both being 30s?

Yes. Those are very extreme health check settings. I don't recommend them.

Assuming (1) and (2) are both "Yes," is there a best practice for forcing the cluster to remove the endpoint upon EDS update prior to the active health check reaching its unhealthy threshold?

You can either configure ignore_health_on_host_removal, or send a double update where you first set the host health state to UNHEALTHY and then remove it.

Naive question: Why isn't the endpoint removed from the cluster immediately following the EDS update? I would expect Envoy allows any in-flight connections to drain, but any future connections should be sent to the remaining endpoints for the cluster.

See https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/service_discovery#on-eventually-consistent-service-discovery. Basically because this is an operational recipe for disaster. If your management server has a temporary glitch you will bring down the entire deployment.

mark


#1400 Include used introspection types in based on documents

@kamilkisiela can you please add a test for a document in typescript-documents and then use the introspection types? it should compile ts without issues.

Mr @dotansimha, done 👍

MERGE MERGE MERGE @dotansimha


select input keeps removed option value

Reproduction
https://formkit.link/e6e9b49c13ce209f6c5f888b581d5105
Describe the bug
I am using a select input (<FormKit type="select" ... />) with a variable list of options. If I select an element from that list which then gets removed from the options and therefore the select list, the display changes to a value which is still an option (this is fine). However, if I submit the form, the removed old value is still sent and not the one displayed in the form.
Environment
• OS: Docker Container with node:19.8.1
• Browser: Edge

@fsoffel Your playground link seems to only` link to a single text input. Do you have a reproduction of what you are trying to do with the select input you can share?

@fsoffel Sorry, I'm still a little confused (probably just me not following). So in your example, you have an initial options array defined in your setup script with two elements that in the next line you are setting to just one options = ['first']. You then are assigning an initial value of "second" to your select input. I see there is also an undefined reference to a "reduce" function, was that meant to be part of your example?
Here is your code example with my questions posed in the form of comments: https://formkit.link/b63b743b2d60810b5e6b44695c09c119

Sorry, for the messed up code. Here is an update: https://formkit.link/bff511dd681024001f4d61d0b22eb357.
I'll answer your questions here as well: Yes, the idea of the button was to call a reduce method which removes the second element from the options array. My expectation would have been that the current value of the select input also changes when I remove the selected option from the options list.

@fsoffel Perfect, thank you for the explanation. So this interesting, I think I would actually expect that the current value of the select input would not change, and in fact, the select input would still display the value "second" (it doesn't which I think is a bug and should be fixed) despite it not having a matching option. Perhaps I'm off base about this though, @GustavoFenilli @justin-schroeder @andrew-boyd do you all have an opinion about this?

Well @justin-schroeder and I talked about this once in the discord and I believe this is not a bug, and it doesn't have much to do with the options there, its mostly because you're setting the value manually.
You can use a plugin to change this behaviour like so https://formkit.link/cc238775abcfbaf0b511a38b721f66e4

@GustavoFenilli Thank you for your answer. In my use case I do not manually set the value of the select input but it is selected through clicking. Then the options are changed and the behaviour is the same as in my example. However your suggestion with the plugin is the solution I was looking for. Thanks again.
However, as a User I would expect for the select input (if set) to only have feasible values from the options or to be undefined.

What I meant by manually is that the value is added to the node ( by user clicking an option, manually calling node.input ) after that the options change, so the value is still set to what was clicked, as the options do not call input.
There is also the possibility of when changing the options, to check if the current value is in the new options, if not to call node.input(undefined).

Not sure there is anything for us to do here, so I'm closing for now. We do not want to constrain the value to only the available list — perhaps in the future we could publish a plugin for this and link it on the docs.


Flex wrap the tags if they overflow. Add margin-bottom to the tags.

This was an easy fix!

Hey @TaneliR ! Thank you for taking out the time to contribute to this project and create this pull request.
However, this issue #109  was assigned to @TheNinza and he has already created a PR for the same #119 . That's why I cannot merge this PR. But, don't be disheartened, you can take a look at other issues which are currently open and unassigned and work on those.
Goodluck and Happy Hacking! 🎊


Configurable sleep time to allow Xcom-LAN datalog uploads

The local Xcom-LAN will do a daily upload of datalogs to the portal. This seems to occur at midnight and can take up to 45 minutes.
It might be better to not send requests to the Xcom-LAN during that operation.
Enhancement is to add a configurable 'sleep' start time (default: 00:00) and end time (default: 01:00). Or alternatively a duration (default: 1 hour) instead of the end-time.

Hi ankohanse,
With the switch back from daylight savings back to standard time, I was reminded that the upload window start time isn't based on an external Studer server time but seems to be based  when midnight occurs on the Studer RCC/Xcom-232.  My system was set to start the pause to allow uploads starting at midnight in my HA, but with the switch back to standard time, the RCC/Xcon-232 was at 11 PM when the HA clock was at midnight which resulted in the upload failing.  The uploads failed for a few days until I reset the RCC/Xcom-232 clock back to standard time.  Now the uploads are working again.  Since then, I have also made a change to the HA automation that controls the communication pause start time so that it will automatically be delayed by 1 hr when in daylight savings time so I don't have to remember to switch the RCC/Xcom-232 clock between standard and daylight time twice a year.
I am also wondering if your integration can read the RCC/Xcom-232 clock which is parameter 5002 (device address 501?) and whose value is an integer of the number of seconds since 1.1.1970 00:00:00.  If the integration can read this value, can it also write to it?  I have noticed that the clock on my system tends to drift ahead by about 5 minutes every 6 months.

Hi Michael,
So far I have still not seen any negative effects of my integation on the uploads. And that is without any special pausing. But maybe I am just lucky that other things that may be of influence, such as internet speed, negate any negative effects.
The RCC datetime param 5002 can indeed be read by my integration (note: the screen capure only shows the first 6 digits on screen, but the edit field does contain all digits of the date). Should also be able to write, but did not yet explicitly test that.

My integration is loosely based on the Zocker code only for the part of encoding and decoding reqeuests.
However, I did not agree with Zocker's method of retries for requests if no answer or the wrong answer comes in. His method resuts in more and more requests being send to the xcom device, resulting in it being flooded/overloaded with requests. Worst case, these additional requests can actually block the answer for a next request. Or maybe could lead to errors in the daily upload to Studer servers.
So I completely replaced that part with my own implementation. Especially in the situation where both the integegration as well as the Studer website or Studer download put in requests, my implementation is more well behaved and has greater change of actually getting a response answer.
To illustrate where things go wrong in the Zocker code, consider the following scenario:

Studer requests
Integration Requests
Xcom responses
comment

req XT : 3000

request from studer servers

req BSP 7000 (1)

request from integration

resp XT 3000
response to studer servers, also seen by integration

req BSP 7000 (2)

retry request from integration, as 'wrong' answer seen

req XT 3001

next request from studer

resp BSP 7000 (1)
response to integration

req BSP 7001 (1)

next request from integration

resp BSP 7000 (2)
response to integration from previous retry

req BSP 7001 (2)

retry request from integration, as 'wrong' answer seen

resp XT 3001
response to studer servers, also seen by integration

req BSP 7001 (3)

retry request from integration, as 'wrong' answer seen

resp BSP 7001 (1)
response to integration

req XT 3002

req BSP 7002 (1)

request from integration

resp BSP 7001 (3)
response to integration from previous retry

req BSP 7002 (2)

retry request from integration as 'wrong' answer seen

... and continued, where answers for retries from previous requests will be seen as 'wrong' answers, leading to more and more retries for nexts requests.
In this same scenario, my code will wait a bit for a next answer if a wrong answer is seen. And if no expected answer has come after a while, only then it will retry the request:

Studer requests
Integration Requests
Xcom responses
comment

req XT : 3000

request from studer servers

req BSP 7000 (1)

request from integration

resp XT 3000
response to studer servers, also seen by integration

wait

integration will wait for next answer, as 'wrong' answer seen

req XT 3001

next request from studer

resp BSP 7000 (1)
response to integration

req BSP 7001 (1)

next request from integration

resp XT 3001
response to studer servers, also seen by integration

wait

integration will wait for next answer, as 'wrong' answer seen

resp BSP 7001 (1)
response to integration

req XT 3002

next request from studer servers

req BSP 7002 (1)

next request from integration

Hi ankohanse,
Thanks for the detailed response.  Seeing that your approach is significantly different, I will have to try your integration.
Currently I use the sensors that were created by my python script through out various templates and automations.  Can you recommend a least painful way to make a transition from the existing 30 sensor names in my HA to the ones your integration will create?  Maybe I will just have to make a list of all instances where each sensor is used and make the appropriate replacements in each yaml file.

Probably the least painfull way is to make new template sensors that simply get their value from your script sensors. Then rework your automations to let them only use the new template sensors.
From that point onward it should be simple to switch the template sensors between your script or my integration. You could even create a 'toggle' helper (input_boolean.use_studer_xcom) and write each new template sensor as an 'if' statement:
sensor:
  - name: "Studer XT1 Battery Voltage"
    unique_id: "studer_xt1_battery_voltage"
    state: >
      {% if states('input_boolean.use_studer_xcom') | bool %}
       {{ states('sensor.studer_4001_xt1_3000') }}
     {% else %}
       {{ states('sensor_studer_3000_from_script') }}
     {% endif %}

I could make the template sensors once I know the format of your integration's sensors.  From your example, "sensor.studer_4001_xt1_3000" is a unique ID for the Xtender infos 3000 "Battery voltage" as listed in the SCOM Technical Specification Appendix.  What does the "4001" in the unique ID refer to?  Also is there a format or list for the friendly names?  And what about friendly names and unique IDs for parameters that are written to the Studer equipment?

The unique is always in format 'studer_'.
Where the listen port usually will be 4001 unless you configured differently on the xcom and integration side. It is included just in case you would want to monitor via multiple XCom devices (i.e. multiple houses).
Device is XT1 to XT9 (in case you happen to have multiple Xtender devices) or BMS, BSP, RCC, VT1 to VT15 (for VarioTracks), VS1 to VS15 (for VarioStrings).
Friendly name is always taken from the 'parameter description' colum of the Technical Specification - XTender serial protocol

Thanks, this helps a lot with the planning. That 4001 referred to the port seems obvious now.


microG Sign-in on JellyBean

Hi, is there any fix yet fo this issue? It's the same image from here #757. I can't sign-in atm.
EDIT: Landscape doesn't work either.

Don't create duplicate issues please, but instead comment on existing ones.


Centering Concerto and badges

Issue #79
Changes
Centering the title and badges.
Related Issues

Issue #79
Pull Request #76

Issue #79
Changes
Centering the title and badges.
Related Issues

Issue #79
Pull Request #76

Hi @DEBSUBHRO , thanks for you PR. Can you please add a DCO sign-off?
If you're making the change via the GitHub UI, simply add the signoff text in your comment. If may be easier for you to close this PR and make a new one if you're not familiar with the git command line tool yet.
Signed-off-by: Random J Developer<EMAIL_ADDRESS>

Hi @DEBSUBHRO , thanks for you PR. Can you please add a DCO sign-off?
If you're making the change via the GitHub UI, simply add the signoff text in your comment. If may be easier for you to close this PR and make a new one if you're not familiar with the git command line tool yet.
Signed-off-by: Random J Developer<EMAIL_ADDRESS>

Yes, Sir. I am creating a new one. I apologize for the delay.

Done in PR #109


Maloja randomly goes idle and cannot be used

Describe the bug
Maloja randomly stops scrobbling.
To Reproduce
Steps to reproduce the behavior:

Run for an unknown amount of time

Expected behavior
Maloja target should start when hitting the "Start" button under "Idle."
Logs
It's just this but repeated:
2024-01-26T23:02:36+00:00 info    : [Heartbeat] [Clients] Attempted to restart 1 clients that were not processing scrobbles.
2024-01-26T23:02:36+00:00 info    : [Scrobblers] [Maloja - myMaloja] Scrobble processing retries (0) less than max processing retries (5), restarting processing after 3 second delay...
2024-01-26T23:02:36+00:00 error   : [Scrobblers] [Maloja - myMaloja] Cannot read properties of undefined (reading 'data')
TypeError:
    at t.buildTrackString (CWD/build/server.js:12:93846)
    at m.existingScrobble (CWD/build/server.js:1:40936)
    at m.alreadyScrobbled (CWD/build/server.js:1:63181)
    at m.doProcessing (CWD/build/server.js:1:45857)
    at m.startScrobbling (CWD/build/server.js:1:42873)
    at m.initScrobbleMonitoring (CWD/build/server.js:1:42386)
2024-01-26T23:02:36+00:00 error   : [Scrobblers] [Maloja - myMaloja] Scrobble processing interrupted
2024-01-26T23:02:36+00:00 info    : [Scrobblers] [Maloja - myMaloja] Scrobble processing started
2024-01-26T23:02:36+00:00 info    : [Scrobblers] [Maloja - myMaloja] [Heartbeat] Should be processing scrobbles! Attempting to restart scrobbling...
2024-01-26T22:44:38+00:00 warn    : [Scrobblers] [Maloja - myMaloja] Scrobble processing retries (5) equal to max processing retries (5), stopping processing!
2024-01-26T22:44:38+00:00 error   : [Scrobblers] [Maloja - myMaloja] Cannot read properties of undefined (reading 'data')

Versions (please complete the following information):
Provide version information for any related sources/clients.

multi-scrobbler: v0.6.3
maloja: 3.2.2

Additional context
Other times, maloja scrobbling doesn't happen at all but there is no error spam.

It seems after a couple backlog scrobbles this also happens.
2024-01-27T00:51:50+00:00 debug   : [Scrobblers] [Maloja - myMaloja] Waiting 363ms to scrobble so time passed since previous scrobble is at least 1000ms
2024-01-27T00:51:51+00:00 info    : [Scrobblers] [Maloja - myMaloja] Scrobbled (Backlog)     => (Spotify) [redacted] @ 2024-01-26T03:25:12+00:00 (C)
2024-01-27T00:51:51+00:00 debug   : [Scrobblers] [Maloja - myMaloja] Raw Payload: [redacted]
2024-01-27T00:51:51+00:00 error   : [Scrobblers] [Maloja - myMaloja] Scrobble processing interrupted
2024-01-27T00:51:51+00:00 error   : [Scrobblers] [Maloja - myMaloja] Cannot read properties of undefined (reading 'data')
TypeError:
    at t.buildTrackString (CWD/build/server.js:12:93846)
    at m.existingScrobble (CWD/build/server.js:1:40936)
    at m.alreadyScrobbled (CWD/build/server.js:1:63181)
    at m.doProcessing (CWD/build/server.js:1:45857)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async m.startScrobbling (CWD/build/server.js:1:42862)
2024-01-27T00:51:51+00:00 info    : [Scrobblers] [Maloja - myMaloja] Scrobble processing retries (0) less than max processing retries (5), restarting processing after 4 second delay...
2024-01-27T00:51:55+00:00 info    : [Scrobblers] [Maloja - myMaloja] Scrobble processing started
2024-01-27T00:51:56+00:00 error   : [Scrobblers] [Maloja - myMaloja] Scrobble processing interrupted
2024-01-27T00:51:56+00:00 error   : [Scrobblers] [Maloja - myMaloja] Cannot read properties of undefined (reading 'data')

@FoxxMD I need some kind of workaround for this in the meantime. Around a quarter of my scrobbles are lost because of this and I have to manually scrobble. Could this be fixed by a downgrade?

I also suggest that this error should not cause all scrobbling to stop, because other scrobbles may get through if the error is dropped after a couple retires.

Maloja randomly stops scrobbling.

Unless the whole app is silently crashing there is absolutely a reason why it stops scrobbling and it is logged as the entire scrobble loop is in a try-catch block. Please post the logs before it stops scrobbling so I can better help you.

this error should not cause all scrobbling to stop

The error is showstopping. It is occurs in a function that should not have any exceptions thrown and is occurring because the play information is entirely missing. Continuing execution would be useless because data is likely no longer structured as intended and would only serve to cause more log noise as every queued bad-data-scrobble would throw the same exception. There is a reason it is the way it is.

some kind of workaround

I would recommend at the very least updating to the latest version, 0.6.4. If you are still using flatpak I would also recommend switching to docker installation because:

I run spotify -> maloja on my own server and have never experienced any of the long-running issues you are creating

There may be problems with flatpak and long-running processes I am not aware of.

I publish fixes ahead of official releases using docker images because its tremendously easier for me to control for the user's environment and deploy (flatpak is a huge PITA which is why it only gets offical releases)

A potential fix for this is published to foxxmd/multi-scrobbler:develop right now

Please post the logs before it stops scrobbling so I can better help you.

The main reason I haven't been able to do this is because my setup is currently kind of a mess and I don't know how to save logs to a file while they're being printed. Is there an option or some way to automatically log both to the console and also a file? I think if we could do that, I could send more useful logs.

If you are still using flatpak I would also recommend switching to docker installation

Yeah, I'm using flatpak. I would imagine Docker takes more memory (I don't have much available besides what I've already allocated) but perhaps I should try it. I have also noticed I've been having some strange issues that generally haven't happened to you or anyone else. I wonder what causes these. I was originally thinking it could be a discrepancy in the system clock, but that would only explain backlog issues, not issues like this one.
I'll have to try Docker then. Is it possible to move my cache and configuration (in ~/.var/app/io.github.foxxmd.multiscrobbler/[cache,config,etc]) to Docker?

This is the memory footprint for MS with an empty config running on my machine in both flatpak and docker:

The top two processes are flatpak -- 86 + 31 = 117mb
The bottom two processes are docker -- 53 + 17 = 70mb
That doesn't include the memory footprint of docker itself but at least on the application level there is little difference and docker is even lighter than flatpak.
To move your configuration copy the entire folder found at ~/.var/app/io.github.foxxmd.multiscrobbler/config somewhere else and use that as the volume mount for /config in the docker container

Impressive. I don't know why I thought docker used more memory than that. I'm not worried anymore. I'll swap to Docker, How does this command sound: docker run -e "PUID=1000" -e "PGID=1000" -p 9078:9078 -v [configpath]:/config foxxmd/multi-scrobbler:develop?

I think this issue is solved. I'll run multi-scrobbler for a bit longer, and also try backlog scrobbling, and close the issue if it's solved. I'm using the develop branch on Docker.

I haven't had to try backlog scrobbling yet because multi-scrobbler hasn't crashed since I started using docker 😂

I added a disclaimer about instability with flatpak installs to the docs..
It's honestly disappointing and a little confusing. Flatpak is supposed to be a "batteries included" environment somewhat (but not the same) as docker where your app should "just work" and be sandboxed with its own dependencies etc...there should not be this much variance in behavior between flatpak and docker. MS is only one process, run by node, with very few system calls or dependencies (just writing log files, if enabled). I don't know what could cause it to crash or have such weird behavior that wouldn't also show up in the docker version.  The environment should have very little influence on how it behaves.
On top of that Flatpak is infamously hard to debug. It does not easily output a stacktrace or decent debug options for the tools it does provide.

@duckfromdiscord i may have a fix for that issue. please pull the latest foxxmd/multi-scrobbler:develop image and restart your MS container

Pulled new image, stopped container, removed it, ran again. Will let you know of results.


fix aws-external warning banner visibility

https://github.com/rancher/dashboard/issues/10290 - the banner warning the user that the cloud provider has been changed should be hidden when the cloud provider change is reverted.
https://github.com/rancher/ui/assets/42977925/23de0233-4bca-46be-b56d-0d3cc36addb4

@mantis-toboggan-md do we need this in 2.7.next1 too?


Incompatible libssh2-sys dependencies

When compiling the latest version of bb8-async-ssh2-lite, I get this error:
error: multiple packages link to native library `ssh2`, but a native library can be linked only once

package `libssh2-sys v0.2.7`
    ... which satisfies dependency `libssh2-sys = "^0.2"` (locked to 0.2.7) of package `async-ssh2-lite v0.4.2`
    ... which satisfies dependency `async-ssh2-lite = "^0.4"` (locked to 0.4.2) of package `bb8-async-ssh2-lite v0.3.0`
    ... which satisfies dependency `bb8-async-ssh2-lite = "^0.3"` (locked to 0.3.0) of package `ssh-worker v0.1.0 (/home/nett/Stuff/sisyphos/ssh-worker)`
links to native library `ssh2`

package `libssh2-sys v0.3.0`
    ... which satisfies dependency `libssh2-sys = "^0.3.0"` (locked to 0.3.0) of package `ssh2 v0.9.4`
    ... which satisfies dependency `ssh2 = "^0.9"` (locked to 0.9.4) of package `async-ssh2-lite v0.4.2`
    ... which satisfies dependency `async-ssh2-lite = "^0.4"` (locked to 0.4.2) of package `bb8-async-ssh2-lite v0.3.0`
    ... which satisfies dependency `bb8-async-ssh2-lite = "^0.3"` (locked to 0.3.0) of package `ssh-worker v0.1.0 (/home/nett/Stuff/sisyphos/ssh-worker)`
also links to native library `ssh2`

It seems like the libssh2-sys dependency in this repo needs to be bumped to 0.3

It seems like the crate compiles just fine after bumping the version, though I am unsure how to run the tests.

https://github.com/bk-rs/ssh-rs/pull/35
And v0.4.3


ProxySchemeUnknown: Proxy Scheme Setting NOT Compatible with OpenAI API

Describe the bug

In WSL Ubuntu 22.04.1 LTS, I use the following proxy setting so that I can get to OpenAI and pips behind the proxy.
If I add any http or https schema like export HTTPS_PROXY="http://<IP_ADDRESS>:11111/", the OpenAI API and pip installs will not work.
Is there anyway to compromise the proxy settings, eg. adding compatibles in the codes?
export http_proxy="<IP_ADDRESS>:11111"
export https_proxy="<IP_ADDRESS>:11111"
export ALL_PROXY="<IP_ADDRESS>:11111"
export no_proxy="localhost,<IP_ADDRESS>,::1,192.168.*.*,10.0.*.*"

When I run this code I get error of ProxySchemeUnknown.

from langchain.callbacks import WandbCallbackHandler, StdOutCallbackHandler
from datetime import datetime

session_group = datetime.now().strftime("%m.%d.%Y_%H.%M.%S")
wandb_callback = WandbCallbackHandler(
    job_type="inference",
    project="langchain_callback_demo",
    group=f"minimal_{session_group}",
    name="llm",
    tags=["test"],
)
callbacks = [StdOutCallbackHandler(), wandb_callback]

File [~/miniconda3/envs/lang/lib/python3.11/site-packages/urllib3/poolmanager.py:481](https://vscode-remote+wsl-002bubuntu-002d22-002e04.vscode-resource.vscode-cdn.net/home/ocean/projects/behavior/~/miniconda3/envs/lang/lib/python3.11/site-packages/urllib3/poolmanager.py:481), in ProxyManager.__init__(self, proxy_url, num_pools, headers, proxy_headers, proxy_ssl_context, use_forwarding_for_https, **connection_pool_kw)
    478 proxy = parse_url(proxy_url)
    480 if proxy.scheme not in ("http", "https"):
--> 481     raise ProxySchemeUnknown(proxy.scheme)
    483 if not proxy.port:
    484     port = port_by_scheme.get(proxy.scheme, 80)
ProxySchemeUnknown: Proxy URL had no scheme, should start with http:// or https://

In cmd line interface, when I type wandb, I also get following error:
Traceback (most recent call last):
  File "/home/ocean/miniconda3/envs/lang/bin/wandb", line 5, in <module>
    from wandb.cli.cli import cli
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/wandb/__init__.py", line 200, in <module>
    _sentry.setup()
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/wandb/analytics/sentry.py", line 40, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/wandb/analytics/sentry.py", line 76, in setup
    client = sentry_sdk.Client(
             ^^^^^^^^^^^^^^^^^^
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/sentry_sdk/client.py", line 143, in __init__
    self._init_impl()
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/sentry_sdk/client.py", line 211, in _init_impl
    self.transport = make_transport(self.options)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/sentry_sdk/transport.py", line 534, in make_transport
    return transport_cls(options)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/sentry_sdk/transport.py", line 154, in __init__
    self._pool = self._make_pool(
                 ^^^^^^^^^^^^^^^^
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/sentry_sdk/transport.py", line 444, in _make_pool
    return urllib3.ProxyManager(proxy, **opts)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/urllib3/poolmanager.py", line 481, in __init__
    raise ProxySchemeUnknown(proxy.scheme)
urllib3.exceptions.ProxySchemeUnknown: Proxy URL had no scheme, should start with http:// or https://
Exception ignored in atexit callback: <bound method Sentry.end_session of <wandb.analytics.sentry.Sentry object at 0x7ff642be4a10>>
Traceback (most recent call last):
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/wandb/analytics/sentry.py", line 40, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ocean/miniconda3/envs/lang/lib/python3.11/site-packages/wandb/analytics/sentry.py", line 163, in end_session
    assert self.hub is not None
AssertionError:

Environment
WandB version: 0.15.3
Langchain version: 0.0.172
OS: WSL Ubuntu 22.04.1 LTS,
Python version: 3.11.0

I temporarily changed the urllib3's poolmanager code to deprecate the error. But not sure if any other nasty things will happen. In my env, /home/username/miniconda3/envs/lang/lib/python3.11/site-packages/urllib3/poolmanager.py,
at the start of the code I added
...
from .util.retry import Retry
from .util.url import parse_url

### Manually Chaged for WandB ###
from .util.url import Url
### Manually Chaged for WandB ###

__all__ = ["PoolManager", "ProxyManager", "proxy_from_url"]
...

Around line 480 I changed code as following:
...
        if isinstance(proxy_url, HTTPConnectionPool):
            proxy_url = "%s://%s:%i" % (
                proxy_url.scheme,
                proxy_url.host,
                proxy_url.port,
            )
        proxy = parse_url(proxy_url)

        if proxy.scheme not in ("http", "https"):
            ### Manually Chaged for WandB ###
            proxy = Url(scheme="https", host=proxy.host, port=proxy.port, path=proxy.path)
            # raise ProxySchemeUnknown(proxy.scheme)
            ### Manually Chaged for WandB ###

        if not proxy.port:
            port = port_by_scheme.get(proxy.scheme, 80)
            proxy = proxy._replace(port=port)

...

Hi @firezym, I see, so we need to append the "https" of "http" (depending on which env variable it comes from) before passing it to urllib3 here correct? That way you could use export http_proxy="<IP_ADDRESS>:11111 and  export https_proxy="<IP_ADDRESS>:11111" which would be compatible with the other libraries.
I'm not sure what the standard is here but it sounds like other libraries are doing this. If this seems like the solution, I can pass this onto the engineering team.

@nate-wandb That is correct. But I am not sure if wandb lib can go that deep into the urllib3 proxy level. I think you can pass this problem to the team to see if they can look into the basic library compatibility issue about proxy setting alignment with OpenAI and Pips.

Hi @firezym, sorry for the delay. I've passed this onto the eng team to investigate. I believe wandb pulls the env variable and passes it to urllib3 so there may be an opportunity to append the schema prior to passing it there but I'll have to have the team look into it. I'll follow up here once we have an update on this.

@nate-wandb Thanks a lot. Looking forward to it :)

Hi @firezym, I haven't been able to reproduce this and it seems like PIP expects the http/https prefix. I haven't tested Open AI yet.

@nate-wandb pip actually is not that important during running. OpenAI and wandb will run in the same proxy env at the same time, which matters. Please try OpenAI :)

Hi @firezym, I was able to reproduce this. I'll pass this onto our engineering team to look into and can follow up once they have a chance to look into the option of being able to handle both variations of the url. This would probably be the more robust way to handle this to ensure compatibility with other libraries similar to the change that you made to urllib3

@firezym can you try export WANDB_ERROR_REPORTING=False? I believe this may only be an issue with the way we handle this with our Sentry integration and the above env variable will disable Sentry.

@nate-wandb Thanks a lot. I will try this when I have chance recently :)

WandB Internal User commented:
firezym commented:
I temporarily changed the urllib3's poolmanager code to deprecate the error. But not sure if any other nasty things will happen. In my env, /home/username/miniconda3/envs/lang/lib/python3.11/site-packages/urllib3/poolmanager.py,
at the start of the code I added
...
from .util.retry import Retry
from .util.url import parse_url

### Manually Chaged for WandB ###
from .util.url import Url
### Manually Chaged for WandB ###

__all__ = ["PoolManager", "ProxyManager", "proxy_from_url"]
...

Around line 480 I changed code as following:
...
        if isinstance(proxy_url, HTTPConnectionPool):
            proxy_url = "%s://%s:%i" % (
                proxy_url.scheme,
                proxy_url.host,
                proxy_url.port,
            )
        proxy = parse_url(proxy_url)

        if proxy.scheme not in ("http", "https"):
            ### Manually Chaged for WandB ###
            proxy = Url(scheme="https", host=proxy.host, port=proxy.port, path=proxy.path)
            # raise ProxySchemeUnknown(proxy.scheme)
            ### Manually Chaged for WandB ###

        if not proxy.port:
            port = port_by_scheme.get(proxy.scheme, 80)
            proxy = proxy._replace(port=port)

...

WandB Internal User commented:
firezym commented:
@nate-wandb Thanks a lot. Looking forward to it :)

WandB Internal User commented:
firezym commented:
@nate-wandb pip actually is not that important during running. OpenAI and wandb will run in the same proxy env at the same time, which matters. Please try OpenAI :)

WandB Internal User commented:
nate-wandb commented:
@firezym can you try export WANDB_ERROR_REPORTING=False? I believe this may only be an issue with the way we handle this with our Sentry integration and the above env variable will disable Sentry.

Hi @firezym , I wanted to follow up on my @nate-wandb 's suggestion above. Please let us know if we can be of further assistance or if your issue has been resolved.