Updated arriba 2.3.0 and arriba drawing 2.3.0 hints

Changes

Dockerpull moved to public aws biocontainers
Use an appropriate instance size (4 CPU and 14 GB)

TODO

[x] #230
[x] #231
[x] #241

That's fine, it will auto-sync with dev and make a new workflow version in development on PR approval

@alexiswl - okay then are you happy for this to be updated to "ready for review"?


[cli] Mode to print response to stdout

I have checked the following:

[X] I've searched existing issues and found nothing related to my issue.

Describe the feature you want to add
Hi there and thanks for the good work !
I was testing Bruno to see if it could replace the use of httpyac in a project where we use it as a wrapper for curl requests, supercharged with hook to handle authentication and the like.
I had expected the bru command to actually output the request result to stdout but I just realized that the cli was mostly thought of a test runner.
Would you consider just® printing the response to a request and optionally its headers to the terminal, without anything else ?
Mockups or Images of the feature

Thanks for your quick answer !
I find this solution extremely complicated though and not really usable in the long :+1:

The overall structure of the output may change overtime
it is really test oriented and gives me a lot of information I do not need

I think I'm rather looking for a new bru sub command that would handle one request only !

What about a send subcommand that would be the cli equivalent of running the request in the GUI and that would print to stdout the content of the response body and headers ?

Should this be part of a discussion ? I would be willing to work on such a subcommand, if this would be accepted as a feature for the community edition !

Well, I'd rather put up some effort if it's bound to be merged at some point and this is where I would like to be sure that it would be accepted as a community feature.
Besides, I may create a certain level of code dupplication given what such a subcommand would have in command with run, regarding arguments for example.


Update GitHub CI to use actions/devx

Description
Why input-output-hk/actions/devx-action? The devx action provides all the system dependencies with nix, but haskell dependencies from cabal. This makes builds easily reproducible from both CI and local development.
This means that if anything goes wrong in CI, you can enter the devx shell, then run the same commands CI runs, and the result should be the same.
Checklist

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] Any changes are noted in the changelog
[ ] Code is formatted with fourmolu on version <IP_ADDRESS> (which can be run with scripts/fourmolize.sh)
[x] Self-reviewed the diff

Migrations

[ ] The pr causes a breaking change of type a,b or c
[ ] If there is a breaking change, the pr includes a database migration and/or a fix process for old values, so that upgrade is possible
[ ] Resyncing and running the migrations provided will result in the same database semantically

If there is a breaking change, especially a big one, please add a justification here. Please elaborate
more what the migration achieves, what it cannot achieve or why a migration is not possible.

LGTM 🎉 one little question will I still be ok to run ghc 9.6.2 locally?

Good point, I need to add this back to the matrix

Just a few ideas. Should also shellcheck scripts/postgresql-test.sh in ci?

There are a lot of violations in all of our other shell scripts, so I'll address that in another PR. In the meantime, I've fixed all the violations in postgres-test.sh

Should also shellcheck scripts/postgresql-test.sh in ci?

https://github.com/input-output-hk/cardano-db-sync/pull/1508


Update nf-d3d12-id3d12graphicscommandlist-setdescriptorheaps.md

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

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


Look up server IP address on every connect

cc @valkum

I wonder if it makes sense to include a variant where you can set your own resolver at this point? Similar to https://docs.rs/reqwest/latest/reqwest/dns/trait.Resolve.html.

One reason would be to allow the usage of trust_dns_resolver to enforce DNSSEC when resolving in application code.
If I understand the Tokio code correctly, it uses the std resolve function, which relies on libc.
I guess it might be better to discuss my remarks in a separate PR to find a good API for that. I just wanted to reduce the amount of breaking changes by mentioning that here.

Note that by bumping the version here I'm not implying that a new version should be released immediately after merging this -- merely that the main branch will now be semver-incompatible with the 0.4.x releases.
DNSSEC is not used very much and there are some problems with it so it doesn't seem like a good reason to make this more complex.


fix empty sql string

Fixes #442
Problem
SELECT 0 WHERE FALSE

is not valid in mysql 5.6
Solution
change it to
SELECT 0 FROM (SELECT 0) AS QUILL_EMPTY_SET WHERE FALSE

Checklist

[x] Unit test all changes
[x] Update README.md if applicable
[x] Add [WIP] to the pull request title if it's work in progress
[x] Squash commits that aren't meaningful changes
[x] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

It's the second time we try to fix this bug. Could you also add a test to each of the submodules verifying that it works with all databases?

@fwbrasil actually there is a test for every sub module, just search for "Example 8 - contains empty", to catch this problem we should test to previous versions of MySQL.

👍

:+1:


[App]: chart data is always behind what is logged

Current Behavior
I have a training run that logs data every 2000 steps, starting with 0.  At 2000 steps shown on my local machine console log, the metrics for step=0 show up in the wandb charts for my run.  At 4000 steps, the metrics for step=2000 show up.  The chart data seen for the run on the wandb website seems to always be 1 data point behind what has been logged to wandb.
Expected Behavior
I expect the metrics logged from my machine to show up on the wandb website charts for my run after a short delay (1-30 secs).  Instead they seem to be delayed until the next logging call (about 40 minutes in my case).
Steps To Reproduce
No response
Screenshots
No response
Environment
OS: Windows 11
Browsers: Chrome
Version:
Additional Context
No response

Note that the system metrics/charts are up-to-date.  This just applies to my explicitly logged metrics.

Hi @rfernand2  , appreciate you patience here as we had an issue with github notifications syncing with our tracking system. To debug this furthur, could you provide:

Link to the workspace(s) or run(s) you are experiencing this behavior with
Debug.log and Debug-internal.log files in your wandb working directory for the affected runs
What current wandb version are you using?
If you are not using out latest sdk version, could you update to our latest sdk version and execute a test run, does this issue persist?
Are you using an integrations in your experiments (e.g PyTorch) and how are you using them in your experiment
Aside from the metrics being logged, are you also logging and additional files? If yes what is the frequency of logging of these files
If possible, could you provide a toy example we can work with to reproduce this behavior.
Thanks

Hi @MBakirWB : I am super busy this week.  I suggest we close this for now, and I'll open another ticket if I run into this again, so I can provide you with necessary info at that time.


experimental welder fuel regeneration

About the PR

adds solution regeneration component/system
fixes #15430 by regenerating 1u of fuel every second for the experimental welding tool

stats are arbitrary but seem fine from 1 minute of testing
Media
https://user-images.githubusercontent.com/39013340/232413104-e99ddef5-51ac-4bf9-868d-ac2ae9327ce5.mp4

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl:

fix: NanoTrasen lost a lawsuit regarding the false claims of fuel regeneration, experimental welding tools now regenerate 1u of fuel every second.

This just opened up so many opportunities of copy->paste.
I wish I was more confident with git to:

make sinks replenishable (0.5u should be fine)
make water fountains with drink component
make atmospheric condenser shitcode (it won't actually use physics - just consume gases to auto produce water)


Add support for cycles

These commits add support for cyclic graphs.
 Briefly:

Cycles are removed for GSSW alignment.
The VG class now indexes edges by the node sides they touch, instead of the node IDs.
The topological sort has been extended to do something smart with cyclic graphs; the general problem of "topologically sorting" a cyclic graph is NP-hard, so this could be improved with better heuristics.
GCSA2 indexing now operates on a graph where everything has been doubled out to describe the forward and reverse strands. This is necessary for proper handling of reversing edges, since GCSA2 doesn't have native support for them. In this paradigm, a reversing edge connects the forward strand of node A to the reverse strand of node B, and visa versa, while a non-reversing edge connects forward strand to forward strand and reverse strand to reverse strand.
Kmers that cross start-to-start edges are now produced correctly from kmer iteration.
Kmer iteration has been revised to flip kmers with negative offsets onto the negative strand when necessary, and to be able to produce all kmers starting on both strands of a node. The deduplication logic has been revised to support this.
Nodes along paths are now pointed to in a way that is unambiguous if the path loops through the same node twice.

Wonderful work. Thanks @adamnovak!


More giveaways

The twitter eth giveaway onslaught continues, these guys are hitting dozens of ICOs, here are the latest domains in use.

Thanks @s0dium for the report!

free-5000eth.com
https://urlscan.io/result/3928cb92-180f-482d-b63e-53c3bfffd523#summary
address: 0x41Cd1343C6e497C075b7ed1E9f003d65141833E2
giveawayether.com
https://urlscan.io/result/9d1bcab7-f7e2-4318-a52f-6fd3cd8d4680#summary
address: 0x7C14369F68d7812C105a597Beb13FF8dA64520c5
giveaway-ethereum.com
https://urlscan.io/result/fc5b790a-bb33-4cf3-a6b0-828afa2a0901#summary
address: 0xf8178B379Ec4Fd758230D28c55c89c064708DFE5
ethpromo.gift
https://urlscan.io/result/666c6e28-31f7-4ab6-8617-f198b9190f52#summary
address: 0x6a13e6d0251fba7b474cf4468aa157ee0c6a736c
ethergiftwallet.com
https://urlscan.io/result/1073c8cb-e653-44c3-8d78-93446e5922b8


Epic 10

Issues Support

Update Tree Details Page to show issues

Builds tab
Boots tab
Tests tab

Update Build Details Page to show issues
Update Boot Details Page to show issues
Update Test Details Page to show issues
Add issues as a Filter

Hardware Monitor

Charts & Graphs components
Search feature
Filters feature
Builds tab
Boots tab
Tests tab
Executions list
Execution Details
Execution Details Logs


Typo  fixup

Should be easy to merge this.

It's still spelled wrong, it is "transceiver". And it should probably capitalized, since it is part of "Dimensional Transceiver".

Better, but you should probably squash those into one commit.


Feature Request: add the ability to pass a custom path to Package.resolved file

Hello. Thanks for your great tool.
I have a suggestion to add some improvements to your tool.
For now, we can define the path to xcodeproj/xcworkspace because we need the Package.resolved file.
But for now, in iOS community, teams are using Tuist to configure their project. With using Tuist you don't have Package.resolved file like other standard projects. Instead, you have a .package.resolved file that creates by Tuist when you setup the project.
What do you think about it?

I found an easy solution to handle it.
After Tuist generated .package.resolved file, I can just move it to xcshareddata/swiftpm/Package.resolved before running swift-package-list command.
But if you see that it's the valid flow that can be handled by your tool it would be nice to add it.

Hey @myak0v1ev, as you can see I have added Tuist support which can be now tested on the release/4.0 branch. It works for me with the latest version of Tuist (3.37.0) on a simple project but it would be good if you can give it a shot via pulling the branch and install it from source.
For Tuist projects the entry point is the Project.swift file; there are some more changes in release/4.0 so make sure you specify the desired options. A good example could be swift-package-list Project.swift --output-type json --output-path ~/Desktop which will generate ~/Desktop/package-list.json.

@FelixHerrmann Hey. Thanks. I'll try and give you feedback on it.


Multiple authentication keys for each exchange

Hi Mike and guys
This is an important feature for me as I am using Bitstamps "sub accounts" feature which gives you access to isolated sets of balances and api keys for more control over trading. Before Bitstamp I have needed multiple accounts at the same exchange to achieve this fine control. In any case, being able to add multiple keys for an exchange in Gekko, I think, would be useful. I forked and tried to do this myself. but as I am new to vue and nodejs, I struggled. I have been trying to make changes to the UI but cannot seem to see my changes.
Wondering if this is straight forward for someone to do, or instead perhaps someone can guide me. For example can anyone tell me why the display doesn't change when I modify gekko/web/vue/src/componentsa/config/config.vue in this area:
  ul
    li(v-for='exchange in apiKeySets') {{ exchange }}
      ul
        li(v-for='name in exchange') {{ name }} (
          a(href='#', v-on:click.prevent='removeApiKey(exchange, name)') remove
          | )
        | )
      | )

I restarted gekko and nothing. Are these template files being cached somewhere?
Thanks

Poloniex also do linked accounts!


implementation of SA1139UseLiteralSuffixNotationInsteadOfCasting

First working and tested implementation of #1562

[Current coverage][cc-pull] is 96.75%

Merging [#2148][cc-pull] into [master][cc-base-branch] will decrease coverage by <.01%

@@             master      #2148   diff @@
==========================================
  Files           564        568     +4
  Lines         74822      75156   +334
  Methods           0          0
  Messages          0          0
  Branches       2710       2725    +15
==========================================
+ Hits          72394      72717   +323
- Misses         1618       1625     +7
- Partials        810        814     +4

Powered by Codecov. Last updated by [c219678...7d1a4ca][cc-compare]
[cc-base-branch]: https://codecov.io/gh/DotNetAnalyzers/StyleCopAnalyzers/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/DotNetAnalyzers/StyleCopAnalyzers/compare/c219678a6e39aabcb01b8ae5b972edd80c7a2e17...7d1a4caa23eb1d483ac55a025a7c975169eee760
[cc-pull]: https://codecov.io/gh/DotNetAnalyzers/StyleCopAnalyzers/pull/2148?src=pr

I reviewed this PR and this looks mostly fine. I don't like the use of regular expressions. This would also have to be updated for C#7 with its binary literals and literal seperators. I played around with it a little bit and I came up with a solution that does not rely on them. I sent you a PR (tmaczynski/StyleCopAnalyzers#1) so you can have a look.

Hi,
Thank you for review, I've merged you changes. I've also added one trivial change to this pull request (4434257).
I really like that you used SemanticModel API and deleted my extension methods (which are redundant).
If you prefer imperative loop to regex, I think that all other changes also make sense - especially that you code works better when C# 7 code is analysed.

I've resolved conflicts between tmaczynski:sa1139 topic branch and master.

Could you merge my changes or give me more feedback?

Loogs good to me :+1:


Add HINDI translation

Added HINDI translation

Are you willing to merge this to the master branch? @ParthJadhav

We already got Hindi translation.


[WB-3755] Kaggle internet check

Throws an error if the user has not enabled internet in kaggle

Hey @jsbroks i have been trying to test this for a while now and can't figure out how to get the kaggle kernel to install this particular branch to test. Did you manage to get that working?

@tssweeney Yup here are the steps (I had to ask Jeff to help me with):

Turn the internet on under settings (its off by default).

 !pip uninstall -y typing
 !pip install --upgrade git+git://github.com/wandb/client.git@justin/kaggle-warning

Turn internet off

 import wandb
 wandb.init(project="testing")
 ```

You should see the following up:


[Android] Only merge JARs which we have built ourselves.

Add a new parameter to merge_jars.py, --build-dir, and exclude JAR files
which lie outside this directory from merging.
The immediate effect is that android-support-v13.jar will not be part of
xwalk_core_library.jar anymore. The dependency on
org.apache.http.legacy.jar was already dropped with chromium-crosswalk
commit 5909ede ("[Android] Stop using org.apache.http classes in
AccessibilityInjector").
We should not bundle android-support-v13.jar in xwalk_core_library.jar,
as it can cause build problems for users of the embedding API that also
pull the same package themselves or via another dependency. Gradle, for
example, produces the following error message:
UNEXPECTED TOP-LEVEL EXCEPTION:
com.android.dex.DexException: Multiple dex files define Landroid/support/annotation/AnimRes;
	at com.android.dx.merge.DexMerger.readSortableTypes(DexMerger.java:596)
	at com.android.dx.merge.DexMerger.getSortedTypes(DexMerger.java:554)
	at com.android.dx.merge.DexMerger.mergeClassDefs(DexMerger.java:535)
	at com.android.dx.merge.DexMerger.mergeDexes(DexMerger.java:171)
	at com.android.dx.merge.DexMerger.merge(DexMerger.java:189)
	at com.android.dx.command.dexer.Main.mergeLibraryDexBuffers(Main.java:454)
	at com.android.dx.command.dexer.Main.runMonoDex(Main.java:303)
	at com.android.dx.command.dexer.Main.run(Main.java:246)
	at com.android.dx.command.dexer.Main.main(Main.java:215)
	at com.android.dx.command.Main.main(Main.java:106)

BackgroundSyncLauncherService in content/ is currently the only class
part of Crosswalk's build that depends on android-support-v13.jar
because it uses android.support.v4 API (which is bundled in the
support-v13 JAR).
Users of the embedding API must now depend on at least
android-support-v4.jar themselves. It appears to be done by default for
new applications in both Eclipse and Android Studio. We may need to
update our documentation and make the dependency explicit in Maven as
well as cordova-plugin-crosswalk-webview.
Related to: XWALK-5092

ping @lincsoon @fujunwei @halton @r0b5t4

LGTM.

lgtm and merging.

Should it be in Crosswalk -16?

@heke123 please consider port this commit into lite branch.

Yes, I intend to backport all the necessary commits once we are sure everything works.


encoding/xml: createAttrPrefix should ban "XMLanything" case insensitive, not just "xmlanything"

What version of Go are you using (go version)?

$ go version
go version go1.13.3 linux/amd64

Does this issue reproduce with the latest release?
AFAIK yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/home/gthomas/bin"
GOCACHE="/home/gthomas/.cache/go-build"
GOENV="/home/gthomas/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GONOPROXY="unosoft.hu"
GONOSUMDB="unosoft.hu"
GOOS="linux"
GOPATH="/home/gthomas/go"
GOPRIVATE="unosoft.hu"
GOPROXY="http://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build491111576=/tmp/go-build -gno-record-gcc-switches"

What did you do?
https://play.golang.org/p/u1HWd2zff8x
What did you expect to see?
<element xmlns:_XMLSchema-instance="http://www.w3.org/2001/XMLSchema-instance" _XMLSchema-instance:nil="true"></element>

What did you see instead?
<element xmlns:XMLSchema-instance="http://www.w3.org/2001/XMLSchema-instance" XMLSchema-instance:nil="true"></element>

https://www.w3.org/TR/xml-names/ says

All other prefixes beginning with the three-letter sequence x, m, l, in any case combination, are reserved.

under 3 "Declaring Namespaces".
Also https://www.w3.org/TR/REC-xml/ says

Names beginning with the string "xml", or with any string which would match (('X'|'x') ('M'|'m') ('L'|'l')), are reserved for standardization in this or future versions of this specification.

under "2.3 Common Syntactic Constructs".

Can I help with anything?
The second version of my CL is simpler and more explicit, I hope.

Ping.

Thank you for filing this bug and for mailing the CL! I shall provide some feedback on your change, and awesome to have your contribution to the Go project!

@tgulacsi I sent some feedback to your CL about 20 days ago, please take a look, so that we can get it going before the code freeze. Thank you.

Sorry, haven't seen it.

Reopening and retargeting for Go1.16 to address the raised cause of rollback during Go1.15.

Ping

Ping

@tgulacsi A minor point, but if you want us to review a change, please ping the change, not the associated issue.  Thanks.

But now I'm not sure what is going on here.  Is there a change to review?  What is the ping for?  Who are you pinging?

I'd like some directions - my previous CL was reviewed, accepted, then reverted.
Now I don't know what was the problem with it, what should I change to have it accepted - without information, I can only reissue the same CL.

PTAL https://go-review.googlesource.com/c/go/+/254978 - it reintroduces the same code change, but puts it behind an Encoder option.

@tgulacsi Thanks for the context.  I look at dozens of different issues every day, so I can't always keep them all in my head.
The CL that reverted your CL, https://golang.org/cl/240179, says "Reason for revert: This change changes uses of tags like "XMLSchema-instance" without any recourse" and refers to #39876.  #39876 explains the problem, and you commented there.  That issue has a test case whose behavior changed.  Any change we make should not change the behavior of that test case, or other similar test cases.
The Go project takes backward compatibility very seriously.  We do not want to break existing programs.  There needs to be a very good reason to not do that.  Perhaps we made a mistake in the fix for #5040, but we still don't want to break existing programs.
Worse, as #39876 says, after your earlier change (https://golang.org/cl/203417) there was no way for a program to get the string that it got before.  So not only did the behavior change, there was no way to get the old behavior.
In your new CL (https://golang.org/cl/254978) you are introducing new API: a new method xml.(*Encoder).XMLPrefixCaseInsensitive.  We usually use the proposal process for new API.
But first let's take a step back.
What is the real problem here that we are trying to solve?  What goes wrong if we don't fix this?  Perhaps you understand that, but I don't.  Perhaps that can help guide us to the right solution.
Thanks.

From one of my clients, I got back this:
<bmRating xmlns:XMLSchema-instance="http://www.w3.org/2001/XMLSchema-instance" XMLSchema-instance:nil="true"></bmRating>

error: Prefix can't begin with XML: XMLSchema-instance

Prefixes beginning with "xml" (regardless of casing) are reserved for use by XML.

And https://www.w3.org/TR/REC-xml/ says she's right.
That's what I try to solve.

Sorry if this is a dumb question, but where is the string XMLSchema-instance coming from?

As far as I see, it's https://github.com/golang/go/blob/master/src/encoding/xml/marshal.go#L339 where it starts to pick a short name (tag?) for the namespace. If possible, it uses the namespace url's path's last section.
AFAIK a random name would work, too - that's why I don't understand what does it break.
That short name could be anything - though it cannot start with [xX][mM][lL].

@tgulacsi OK, I've looked into this, and I've come around to the view that you are correct: we should make this change and fix code that breaks.

Thanks for your patience with this.

Thank you @ianlancetaylor  and @odeke-em !
This is not a big issue for me, I have and use a not-so-simple-but-working workaround.
The big thing is that I thought I can contribute to the Go project, and have a tiny improvement on Go,
my beloved language, standard library and ecosystem.
I was in joy and then it's been rolled back.
So thanks again for looking again into it, and rolling forward!


Typescript magic

added a little blog post about my ventures with typescript in palette-mobile.

why is this failing.. some strange error
 Deprecation: posts.each should be changed to posts.docs.each.
                    Called by ["/home/circleci/project/vendor/bundle/ruby/2.6.0/gems/jekyll-3.9.0/lib/jekyll/collection.rb:39:in `method_missing'"].
jekyll 3.9.0 | Error:  Max nesting depth of 256 exceeded

oop failing to build. going to try dropping the CI config changes you made (the blog is...a little finnicky)

oop failing to build. going to try dropping the CI config changes you made (the blog is...a little finnicky)

heyoooo we're back

Thank you so much for fixing this! 💜  That error was definitely way more complex for my link missing the markdown things 😅.
Merge/publish whenever! I'll assign it to you.


Display max drawdown in a trade

Drawdown is a good indicator for trade entries.
Will be great if Ninjabot display the max drawdown reported in backtesting.
Reference https://en.wikipedia.org/wiki/Drawdown_(economics)

Hi, I am interested to take a look at this. Could you please assign the task to me?

Hi, I am interested to take a look. Could you please assign the task to me?

Hi @FreddyJilson, shure!

thanks!

No problem @FreddyJilson


ChildChain.sendTransaction() returns change

Fixes #54

Ideally, I'd like to have a send(from, to, amount) method, where the caller doesn't have to care about any of the details.
But... utxo selection and management is not that simple due to the limitations of 4 inputs, 4 outputs (currently 2in, 2out), so this requires a bit more thinking about...


Features requested

Hi, I dont know if you are taking feature requests but heres a few:

show the day of week on the top of the chart when zoomed to day level (eg, M T W)
vertical bands support - eg for the end date of a project, a red line down the chart would make that date obvious
support handler for single click. double click is supported currently and thats great. single would be good too (eg to show a side panel detail for a clicked task etc)

thanks

Thanks for the list!
These sound good - though I don't fully get #3. You want a different popup for a single click?

I was thinking it would be very nice to have more of a single click handler,
eg this on_click below actually triggers on double-click (unless I am doing it wrong),

    ganttChart = new Gantt('#gantt', tasks, {
        header_height: 50,
        view_modes: ['Quarter Day', 'Half Day', 'Day', 'Week', 'Month'],
        view_mode: 'Day',

        on_date_change: (task, start, end) => {
            debouncedSave(task.id, 'start', start);
            debouncedSave(task.id, 'end', end);
        },
        on_progress_change: function(task, progress) {
            debouncedSave(task.id, 'progress', progress);
        },
        on_click: function(task) {
            if (task.id.startsWith('task-')) {
                const taskId = task.id.replace('task-', ''); // Extract the integer part
                edit_task(taskId);

            } else if (task.id.startsWith('group-')) {
                const groupId = task.id.replace('group-', ''); // Extract the integer part
                edit_group(groupId);
            }
        },
        language: 'en'
    });

@coda1024 for single click event trigger, you can put the code in the custom_popup_html{} function
this.gantt = new Gantt(this.$refs.gantt, this.tasks, {
        header_height: 50,
        column_width: 30,
        step: 10,
        view_modes: ["Quarter Day", "Half Day", "Day", "Week", "Month"],
        bar_height: 30,
        bar_corner_radius: 10,
        arrow_curve: 5,
        padding: 18,
        view_mode: this.view,
        date_format: "YYYY-MM-DD",
        custom_popup_html: (task) => {
          // put single click code here

          `;
        }
      });

Implemented! Single and double clicks are separate now (since May, actually).


(ios)the same uri work in 3.0.0 but not work in 3.1.0

Current behavior
When i upgrade from 3.0.0 to 3.1.0, the video can not load with the same uri, in 3.0.0 the uri is ok,but in  3.1.0 it not work, have something to mind before upgrade?
Platform

iphoneX iOS 11.4

Video sample
<Video
              source={{uri:audio_uri}}
              paused={!should_play}
              onLoad={this.on_load as any}
              onEnd={this.on_audio_end}
              onProgress={this.render_animate}
 />

android also have this problem

I will go ahead and test this to see if I can find any problems. Which uri are you using, I am guessing 3.1.0 has a bug for some files but not all.

hi cobarx, I use local resource, find in 3.0.0 need use absolute path，but in 3.1.0 ,need use relative path

ok,i will make a sample example for you later

In my project the ios directory structure like this
├── rnVideoTest
│   ├── AppDelegate.h
│   ├── AppDelegate.m
│   ├── Base.lproj
│   ├── Images.xcassets
│   ├── Info.plist
│   ├── main.m
│   └── static
│       └── test.mp3

in early version, we need use the MainBundlePath in the uri can make it work
import React, {Component} from 'react';
import {
  StyleSheet,
  View,
} from 'react-native';
import Video from 'react-native-video';
import { MainBundlePath } from 'react-native-fs';

type Props = {};
export default class App extends Component<Props> {
  render() {
    console.warn('mainPath:', MainBundlePath);
    return (
      <View style={styles.container}>
        <Video
          source={{uri: MainBundlePath + '/static/test.mp3'}}
        />
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: '#F5FCFF',
  },
  welcome: {
    fontSize: 20,
    textAlign: 'center',
    margin: 10,
  },
  instructions: {
    textAlign: 'center',
    color: '#333333',
    marginBottom: 5,
  },
});

But recently ,we update our version to 3.1.0, in 3.1.0, we use this relative path can work, if add MainBundlePath , it can't work
 <Video
          source={{uri: 'static/test.mp3'}}
        />

i found uri: 'static/test.mp3' also work in 3.0.0, our team's member tell me they use MainBundlePath  because early version can't work with uri: 'static/test.mp3

Same issue here, works on 3.0.0 but doesn't on 3.1.0 with a local uri

For me it works with 3.0.1 in dev mode but not once released.

I just released version 3.2.0 which contains PR #1162 by @ashnfb for this problem. Let us know if that fixes it or not.

Thanks @cobarx and @ashnfb, this now works perfectly. Awesome work and thanks for a great library!

Quick fix, thanks a lot :)

YOU amazing HEROS!!

3.2.0 is good use,but 3.2.1 & 3.1.0 is not


Attachments with special characters lost when uploaded from MS Teams

Version
Lastest
Describe the bug
When I upload an attachment from Microsoft Teams with certain special characters, such as a '+', the attachment "disappears" when it is received in the bot's backend.
For example, uploading a file with a "normal" name like test.txt works as expected, and I receive the following in the bot's backend:
{
    "attachments": [
        {
            "contentType": "application/vnd.microsoft.teams.file.download.info",
            "content": {
                "downloadUrl": "https://ckm-my.sharepoint.com/personal/xxxxxx/_layouts/15/download.aspx?UniqueId=xxxxx-xxxx-xxxx-xxxx-xxxxx&Translate=false&tempauth=v1.xxxx.xxxx.xxxx&ApiVersion=2.0",
                "uniqueId": "xxxx-xxxx-xxxx-xxxx-xxxxx",
                "fileType": "txt"
            },
            "contentUrl": "https://ckm-my.sharepoint.com/personal/xxxxxx/Documents/Microsoft Teams Chat Files/test.txt",
            "name": "test.txt"
        },
        {
            "contentType": "text/html",
            "content": ""
        }
    ],
    "type": "message",
    "timestamp": "2024-07-11T10:29:11.4680572Z",
    "localTimestamp": "2024-07-11T12:29:11.4680572+02:00",
    "id": "1720693751445",
    "channelId": "msteams",
    // ....
}

However, if I upload a file with a name like test+.txt, I only receive the following:
{
    "attachments": [
        {
            "contentType": "text/html",
            "content": ""
        }
    ],
    "type": "message",
    "timestamp": "2024-07-11T10:36:45.0287287Z",
    "localTimestamp": "2024-07-11T12:36:45.0287287+02:00",
    "id": "1720694205007",
    "channelId": "msteams",
    //...
}

To Reproduce
Steps to reproduce the behavior:

Use the basic example from BotBuilder-Samples.
Upload a file with a normal name (e.g., test.txt) from Microsoft Teams.
Observe the attachment received in the bot's backend.
Upload a file with a special character in the name (e.g., test+.txt) from Microsoft Teams.
Observe the attachment received in the bot's backend.

Expected behavior
Attachments with special characters in their names should be processed correctly and received in the bot's backend just like attachments with normal names.
Screenshots
N/A
Additional context
This issue has also been posted on Stack Overflow.

I also have this very same issue.
We cannot control the name of the files the user's upload from Teams to the bot's backend.
Please help!

Same issue here :(

I can reproduce. As it is happening on the Teams side (before even getting to the bot), I am investigating on where/how to have this handled.

Hello there! Are there any news on this?

Hi @tracyboehrer, this is the same issue reported in #4750. It's on MS Teams' side, the bot is not receiving the attachment when there are special characters in its name.

OP should refer this to the Teams Samples or Teams AI SDK repo.


Investigate integration module singletons deployment

According to Filecoin team the deployment of singletons could be simplify by calling the construct method of singletons.
It is linked to this PR and how we are testing actors in fvm environment. We are currently using our own fork and it is not a long term solution.
see https://github.com/filecoin-project/ref-fvm/pull/1387

I try to follow the instructions and created a poc but it failed. More feedbacks are needed from the Filecoin team.
see https://github.com/rllola/fevm-integration/blob/misc/testing-integration/testing/src/main.rs#L48

So it is possible to instantiate singletons after instantiating the machine. Doing this will remove the need to maintain our own fork.


Upgrade Badger to 2.0

BadgerDB released the 2.0 version, this PR upgrade the dependency and makes the necessary changes to work with the new API

Thank you so much @victorcoder for this contribution. It looks like your commit broke tests, maybe you forgot to push some other change.

@aalda oops! CI passing now

Codecov Report

Merging #4 into master will decrease coverage by 0.4%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   64.08%   63.68%   -0.41%
==========================================
  Files           2        2
  Lines         181      179       -2
==========================================
- Hits          116      114       -2
  Misses         52       52
  Partials       13       13

Impacted Files
Coverage Δ

badger_store.go
59.87% <75%> (-0.49%)
: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 ee05bc4...d336629. Read the comment docs.


WIP - Use PSR4 for src

Use PSR4 for src

Strange... passing the PR in WIP

Yes..The tests also pass when I try it locally. Really strange


"Be made without using a security context"

The explanation attempt does not add any information to the description.
It is unclear how the connection is made.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ac4dbeb2-b244-27af-7cfe-4f441ca26d2e
Version Independent ID: ee282927-e7df-a8d9-da5c-755d0bc41b0f
Content: Create Linked Servers - SQL Server
Content Source: docs/relational-databases/linked-servers/create-linked-servers-sql-server-database-engine.md
Product: sql
Technology: ****
GitHub Login: @stevestein
Microsoft Alias: sstein

@dudu-markovitz -- דודו, thank you for your feedback.  You may find some information here:

DBA Stack Exchange
Stack Overflow

@stevestein -- Steve, please look into this issue 3936.


DatabaseHealthTrigger is only available on 2016 & later

Type of Change

Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Make function work against my SQL2014 servers
Approach

Wrap parameter that only applies to 2016 or greater in server version test
Commands to test

Screenshots

Learning

perfect but I'd move BasicAvailabilityGroups too. Aren't they from 2016 onwards ?

ok, more coffee needed...
13 IS 2016.
Perfect PR!

awesome, thank you! this will be released sometime today.

Least I can do, thanks for all the great code


dscan - record total exposure time for the pe1 detector at XPD beamline

The Perkin-Elmer pe1 detector is setup to sum up multiple exposure frames into a single tiff image that makes up one measurement.  The frame exposure time and number of frames are both configurable.  To my knowledge, there is no record of these values or the corresponding total exposure time in the databroker events as produced by dscan.

This probably should be inserted into BeamlineConfig by the run engine.

beamline config?  do the other scan parameters go in beamline config?  That name is confusing.  Alot of people would expect 'beamline config' to be something infrequently changed, perhaps only by staff, unless there's automation or something.  Like default settings or what detector is installed or whatever changes are involved in changing techniques on a multi technique beamline.
RunConfig would be much better for how this seems to be used.
Do we only change what I think of as beamline config in yaml files or something?  But that info gets recorded along with scan params in BeamlineConfig?

As currently understood by me, BeamlineConfig is where to put any
information which is not about the sample which does not change over the
course of the experiment which describes the configuration of the beam
line.  Things like detector exposure time seem like the fall squarely into
that category, as does things like sample-to-detector distance, the model
of the goniometer used, and all fe the calibration values (even if we are
not capturing any of that right now).  If you are changing the exposure on
a per-event basis then it needs to be something that goes into your event
streams.
On Thu, Jun 4, 2015 at 8:37 AM cowanml<EMAIL_ADDRESS>wrote:

beamline config? do the other scan parameters go in beamline config? That
name is confusing. Alot of people would expect 'beamline config' to be
something infrequently changed, perhaps only by staff, unless there's
automation or something. Like default settings or what detector is
installed or whatever changes are involved in changing techniques on a
multi technique beamline.
RunConfig would be much better for how this seems to be used.
Do we only change what I think of as beamline config in yaml files or
something? But that info gets recorded along with scan params in
BeamlineConfig?
—
Reply to this email directly or view it on GitHub
https://github.com/NSLS-II/ophyd/issues/126#issuecomment-108874979.

I think I understand how it's used, and it functions fine, just find the name throws me off.
I guess I'm really used to dividing things up into:
~ changes often, with every run/scan, customized params  (exposure time)
... data collection parameters, level 0 data,
write once, never change
~ changes rarely, and/or only by staff (gonio, det, and default
params [including exposure time])... beamline configuration, level -1 settings/state,
could be overwritten as long as values are copied into mds when actually used or saved
in the archiver
John and I are trying to find the best place to stick various bits of
beamline state info and defaults, and maybe some persistent per user settings.
BeamlineConfig sounds right, but isn't quite.
-matt

Beamline's configuration should not be managed based on information within this collection. As @tacaswell  also suggested, this is just a simple way for analysis to have some information regarding the configuration that was part of the process which resulted in the given run data. Managing beamline configuration with lots of updates and user interaction requires more logic than a simple collection like BeamlineConfig.
If you're thinking of managing your beamline configuration with lots of writes/updates, there are other tools, such as RocksDB that you might want to look into.
John and I discussed his specific application and he suggested his confusion was resolved. His use-case is different and I believe he has already decided to stick to his existing methods. We named it BeamlineConfig beacuse naming it BeamlineRunConfigForGivenRunHeadersAtTheSpecificRunTimeForAnalysisPurposes was plausible :)

@ericdill has a PR in to allow corrections to be over-laid on top of MDS is a safe way https://github.com/NSLS-II/metadatastore/pull/150
The RunStart documents are flexible so if you want to split your data into two (BeamLineConfig and BaseConfig or BeamlineConfig and RunConfig?) I think that is ok.  We have a fair amount of flexibility on these documents still as ohpyd has not actually been emitting them with much meaningful content and we have no consumers relying on a given spec.
I sort of suspect that the exact layout of these documents is going to end up being beam line specific anyway.  We probably should tack a BLC_schema string into runstart as well.

attn #165
Need to shove configuration into the output of describe.  The machinery is already there in bluesky to handle this.


Adding vs_build dependency to Windows hot_mode_dev_cycle_win_target__benchmark

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
List which issues are fixed by this PR. You must list at least one issue.
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

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

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

I listed at least one issue that this PR fixes in the description above.

In the future, please ensure there's an issue or context attached to this PR.
For those following along at home, this was a bug observed by the engine team as the tool is outputting in the logs unable to find visual studio. This causes the test to fail early on cold bots.


Fix Bug from Error Slice Optimization

In this codepath, if error is GRPC_ERROR_NONE, then slice will never get set and will be uninitialized, which could cause a crash if details ever got turned into a c string.
I missed this in the original PR.

Failures: #9551, #9542, #9011.


Change Aegis Safe API to enable multiple secrets per workload

This part is done; yet there is work to do in the Aegis Sentinel too.


Removal of label does not work

the following test code:
entity = new Entity();
entity.addLabels("TestLabel1", "TestLabel2");
session.save(entity);
session.clear();

entity = session.load(Entity.class, entity.getGraphId());
entity.removeLabel("TestLabel1");
session.save(entity);
session.clear();

labels = session.load(Entity.class, entity.getGraphId()).getLabels();
assertEquals(1, labels.size());

fails with:
java.lang.AssertionError:
Expected :1
Actual   :2

@jasperblues Are you on this?

@jasperblues2 Is this something you can look at?

Hmmm, there's a passing test for this. Looking.

@danielmischler Can I see the source for Entity? Was not able to reproduce the issue with the following test:
@Test
    public void removeLabels()
    {
        Pizza entity = new Pizza();
        List<String> labels = new ArrayList<>();
        labels.add("TestLabel1");
        labels.add("TestLabel2");
        entity.setLabels(labels);
        session.save(entity);
        session.clear();

        entity = session.load(Pizza.class, entity.getId());
        labels = new ArrayList<>();
        labels.add("TestLabel2");
        entity.setLabels(labels);
        session.save(entity);
        session.clear();

        labels = session.load(Pizza.class, entity.getId()).getLabels();
        assertEquals(1, labels.size());
    }

Only difference I can see above is:

Use Pizza instead of Entity
This class does not have convenience methods for adding/removing labels. Used plain (verbose) collections API.

test

Reproduced with the following:
    @Test
    public void removeLabels()
    {
        Pizza entity = new Pizza();
        List<String> labels = new ArrayList<>();
        labels.add("TestLabel1");
        labels.add("TestLabel2");
        entity.setLabels(labels);
        session.save(entity);
        session.clear();

        entity = session.load(Pizza.class, entity.getId());
        entity.getLabels().remove("TestLabel1");
        session.save(entity);
        session.clear();

        labels = session.load(Pizza.class, entity.getId()).getLabels();
        assertEquals(1, labels.size());
    }

It looks like the labels collection is not updated as dirty in this case. Working on a fix.

@mangrish: I was not able to find a test for this in the code
@jasperblues: Sorry for not submitting the issue using the pizza code right away. I was starting from that anyway.
BTW is there a reason that you work with lists? Wouldn't it be more appropriate to use sets since a label can only be present once?

@danielmischler We allow any type of Collection for flexibility. You're right that the test itself would be better with a Set though. Working on a fix for the issue, should be ready soon.

@danielmischler please note that 2.0.5-SNAPSHOT has not yet been published to m2.neo4j.org. We will update you as soon as it is.

@luanne: excellent! looking forward to start using it :-)


[Roadmap][Move] Compiler enhancements

This issue tracks a set of enhancements that we would like to make to the Move compiler before launch.

[ ] Constraints to avoid borrow checker iteration
[x] Constants
[x] Resolve open syntax questions
[x] Support aliasing for functions, structs, and constants
[x] Script blocks
[x] New "signer" primitive type with move_to bytecode

Script blocks were added in #3669

Aliasing support was added in #3920 (except for constants, which we haven't yet implemented) and in #4139 for spec contexts

Signer and move_to support was added in #3919

The two open syntax questions mentioned here have been resolved:
(1) Brackets for conditionals and loops: Should you be allowed to write "if (e) break" or "while (e) i = i + 1"? We will keep the current syntax where brackets around a single expression are optional. We generally prefer brackets except for conditionals on one line, but we can use a (future) auto-formatting tool to make that distinction.
(2) Semicolons after conditionals and loops (with brackets): Should you be required to write "if (e) { s }; x = x + 1"? We will keep the current syntax where semicolons between expressions in a sequence are mandatory. The consistency and clarity here is more important than compatibility with Rust and C-based languages.

Compiler support for constants landed in #4653

Constraints to avoid borrow checker iteration were enabled in #4853 (There is more work to do later to take advantage of this, but for now we will ensure that the necessary constraints are satisfied.)
This task is now complete.


Tokenizer doesn't work with string tensors (AttributeError)

System information

OS Platform: Windows 10
TensorFlow version: 2.3.2
Python version: 3.8.5
CUDA version: 10.1.243

Issue
Tokenizer doesn't work with tensors (at least string tensors).
Standalone code to reproduce
Fails with a string tensor:
a = ['hello world', 'what is your name', 'start with a scene']

t = tf.constant(a, dtype=tf.string)
tokenizer = tf.keras.preprocessing.text.Tokenizer()
tokenizer.fit_on_texts(t)

Throws this error:
AttributeError: 'tensorflow.python.framework.ops.EagerTensor' object has no attribute 'lower'

Works fine with a regular python list:
a = ['hello world', 'what is your name', 'start with a scene']

tokenizer = tf.keras.preprocessing.text.Tokenizer()
tokenizer.fit_on_texts(a)

@princyok
Sorry for the delayed response, can you please refer to this link which has it explained in detail.
Also similar resolved issues: link,link1

@Saduf2019
Thanks. I've gone through the links, although I'm not sure if the first link to keras.io was a mistake.
None of those links seem to address the key issue here: Why the Tokenizer class has no support for tensors or even numpy arrays. It's strictly list of strings (even the documentation clearly spells this out), but why should this be the case? The TextVectorization layer (also keras API) supports tensors, so does the various classes in text_tensorflow.
But I guess it is simply what it is.

i am able to replicate the issue reported on f 2.3 and 2.4, please find the gist here

@princyok Input Argument text sequence can be can be a list of strings, a generator of strings (for memory-efficiency), or a list of list of strings.  So, after changing one line in your code (# t = tf.constant(a, dtype=tf.string)), everything worked as expected.
Here is a gist for reference. Thanks!
Please verify once and close the issue if this was resolved for you. Thanks!

@jvishnuvardhan , thats what @princyok tried to point out in the issue. The code works well with strings as you mentioned but throws in the error when using a tensor. I think this needs to be fixed. Please revert back if you would like a fix for this issue.

@jvishnuvardhan I already addressed your in my other comment above. What you are pointing out is the problem itself.

Was able to replicate the issue in TF 2.6.0-dev20210528,please find the gist here..Thanks !

@princyok tf.keras.preprocessing.text.Tokenizer will be deprecated in future version since it does not operate on Tensors, and is most unlikely to get any update.
We recommend you to use tf.keras.layers.TextVectorization which has similar functionality which operate on Tensors.
For details see here. Thanks!

I'm able to Tokenize string data by using TextVectorization.
Below is the code reference.
import tensorflow as tf
from tensorflow.keras import layers
data = ['hello world', 'what is your name', 'start with a scene']

layer = layers.TextVectorization()
layer.adapt(data)
vectorized_text = layer(data)
print(vectorized_text)

tf.Tensor(
[[10  3  0  0]
 [ 5  9  2  8]
 [ 6  4 11  7]], shape=(3, 4), dtype=int64)


Set introspection request body

Hi,
How can I set the body of the introspection request ?
Like in mod_auth_openidc :
OIDCOAuthIntrospectionEndpointParams "uri=[]&method=[]&token=[]"
I have the following conf for token introspection :
OAuth2TokenVerify introspect https://auth:2626/token_introspection introspect.ssl_verify=false
My auth server require a request body with the following parameters :
method=[]
uri=[]
token[]
Thanks

that's not possible since it is not configurable

So I have to stay with mod_auth_openidc even if it is deprecated ... ? :(

you can probably get your money back


cp: cannot stat '/home/dnanexus/in/None': No such file or directory

Hi @Maciek416
These days I have tried the parliament2 to call SVs. I get the following error:
cp: cannot stat '/home/dnanexus/in/None': No such file or directory

current working directory is named /home/dnanexus/ and looks like this:
.
├── input.bam
├── input.bai
├── ref_genome.fa.gz
└── outputs

Command used as follow:
docker run -v /home/dnanexus/:/home/dnanexus/in -v /home/dnanexus/outputs/:/home/dnanexus/out dnanexus/parliament2:latest --bam input.bam --bai input.bai -r ref_genome.fa.gz --breakdancer --cnvnator --manta --genotype

This was reported before but no solution yet.

I am having this same problem with docker
cp: cannot stat '/home/dnanexus/in/None': No such file or directory
mv: cannot stat '/home/dnanexus/in/None': No such file or directory


CLI: running rack model import --clear puts RACK in a bad state (NullPointerException)

To reproduce:

get a RACK VM started
issue on the command line, from the cli directory:

rack model import --clear ../Turnstile-Ontology/99-Utils/import.yaml

The output makes it look like the command succeeded.
Error message in the SparQLGraph browser:
Service "ontologyinfo" failed
endpoint: getOntologyInfoJson
full URL: http://localhost:12057/ontologyinfo/getOntologyInfoJson
xhr contents:
  message: "operations failed."
  rationale: "ontologyInfoService/getOntologyInfo threw java.lang.NullPointerException null
com.ge.research.semtk.ontologyTools.OntologyInfo.getSuperclassNames(OntologyInfo.java:302)
com.ge.research.semtk.ontologyTools.OntologyInfo.getSuperclassNames(OntologyInfo.java:276)
com.ge.research.semtk.ontologyTools.OntologyInfo.findCommonSuperclass(OntologyInfo.java:257)
..."
  status: "failure"

I'm not sure how to get out of that state, short of respawning a VM.

Ah nevermind, it's simply that I didn't respect the order (should import OwlModels/import.yaml first).
I'll close this for now, though, it'd be nice if such mistakes were caught by the command line tool.

Ah nevermind, it's simply that I didn't respect the order (should import OwlModels/import.yaml first).
I'll close this for now, though, it'd be nice if such mistakes were caught by the command line tool.


Naming convention changes break "Tower" index logic

Right now, when the settings are loaded, the tower is set by determining the index in the options of the GUI window for current box. The options are outdated now, so the tower index will never be set.

One option is to simply add the new boxes to the options in the tower index.
alternative (my preference), we remove all logic and instances of "tower" since its only used to create folders for data saving.
When we update the room again to use one box per computer, we can greatly simplify this logic, to just have the current_box setting in the setting files specify the full box "447-1-A"

Or, we need to rename this to "box", and add the new names to the default options in the GUI

"When we update the room again to use one box per computer, we can greatly simplify this logic, to just have the current_box setting in the setting files specify the full box "447-1-A""
I like this one to just set the full name in the setting files.


Support Rocket

Please consider this a feature request. It would be neat if this supported the Rocket web framework.

I do have this in mind! :smile:


cant connect to server

hi =)
I have a problem since I update my server with the last waterfall and last spongeforge, we can't join it
in logs,
https://paste.md-5.net/yonuniyafe.vbs 4
But all the players have the same errors and have the same mods like me and the server
so that’s strange !!
I have waterfall https://destroystokyo.com/ci/job/Waterfall/178/ 1
and the last spongeforge 1.12.2-2705-7.1.0-BETA-3160
thanks to help me ! :slight_smile:
have a nice day !!

up, Can I have a reply please from a developper of waterfall
yepidoodles of sponge 's forum tried to help me and think too it's a waterfall issue
She answered me at https://forums.spongepowered.org/t/server-dont-works/23970/7 and by private message (thanks to her)
If you can add a look it can be great =)
thanks !!! :)

I've been on the sponge discord and have asked for logs on several occasions and been handed nothing in return.

hi again :)
so with electronicboy we made test
all we said / i show to him :
my bungeecord plugins
https://www.zupimages.net/up/18/24/envt.png
the startup of bungee https://paste.md-5.net/qaxesiyute.coffee
i tried without spongeforge but spongeforge server don't start https://paste.md-5.net/nerelilivu.vhdl
i tried to remove each mod  by one and replace  by one but same error like before https://paste.md-5.net/yonuniyafe.vbs
so i don't know what can i do more
and i tried to put a save of my server when all worked it's the same :/
So if someone else have an idea :) ..
thanks =)

Can you paste your Waterfall's config.yml somewhere?

Thanks for the reply :)
https://paste.md-5.net/wetutujace.tex
Ask me if you need something else :)

I add that i have nothing in logs
But in console i have
https://www.zupimages.net/up/18/25/1rpb.jpg

We would need to see the full error.

The problem is that i dont know where is it ???
In the log file i see nothing
But at the same time in the console via my phone i see that but i cant see more

If you're using something like screen or tmux, they generally have means of scrolling up, all log info should be in the log file unless you've modified the logger config or have a plugin messing around with that

i used juiceSSH android for the screen but can't scroll
with puty on pc I don't find this error and in the logs the error is not here too
so i don't know :(

The trouble is it is very difficult for us to assist you when we don't have stack traces to go off, we don't even know what exception is being fired here. I don't get why you've not been able to obtain for us the logs with the full stack trace - are you saying they are not present in the log file? If so, we need to get that sorted out, or we're not going to be able to help much here.

Look in the log file

https://paste.md-5.net/lafaficosa.avrasm
i find that, it can help you ??

Also it is possible to scroll up in screen application if you google it

First result in google if you search for "screen scroll up"

Something installed on the server itself appears to be mangling the login process

Taken a closer look at this, looks like either a plugin or the client is messing up the authentication process, you've got what looks like a handshake packet coming in with the ID of an encryption response packet which is throwing itself into hell as it's trying to read the packet improperly due to this.


Add a HashReporter for other tools to use

Currently, the Code Climate engine I've been working on has to go
through a dance to capture the JSON output of a report. By extracting
the hash construction into a reporter that can be used by tools outside
of Haml-Lint proper, we can encourage others to write such tools.
The existing structure of the JSON reporter is really nice, so this
extracts that behavior and makes the existing JSON reporter a subclass
that only handles the printing of the report to the log.

Thanks!


Reachability and deadlock check on project with no main component

Trying to auto generate queries for reachability or deadlock checking causes exceptions when the main component for the project is not set

Will be handled by making sure that the project always has a main component


[BUG]miniconda3/lib/python3.9/uuid.py", line 186, in init     raise ValueError('bytes is not a 16-char string')

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
To help us reproducing this bug, please provide information below:

Your Python version
The version of Mars you use
Versions of crucial packages, such as numpy, scipy and pandas
Full stack of the error.
Minimized code to reproduce the error.

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

CPU 无问题，GPU Ubuntu22.04 存在以下报错 ctypes ValueError: [address=<IP_ADDRESS>:33995, pid=381804] bytes is not a 16-char string

Do you have the code to help us to reproduce the error?

import mars
mars.new_session()
import mars.tensor as mt
a = mt.random.uniform(-1, 1, size=(1000, 1000), gpu=True)

Thanks, will try to fix it ASAP.

env：tesla v100  cuda11.3   Ubuntu 22.04


Always load .env file

I think it might be worth considering to always load the .env file. As far I can see it is only loaded when running pipenv shell.
An example use case for this would be to share the configuration of PIPENV_VENV_IN_PROJECT so that the venv is created in the local directory. See also steps to replicate.
Describe your environment

OS Type: Windows 10
Python version: $ python -V: Python 2.7.11
Pipenv version: $ pipenv --version: pipenv, version 8.2.7

Steps to replicate
Run in powershell:
PS C:\tmp\pipenv> echo "PIPENV_VENV_IN_PROJECT 1" > .env
PS C:\tmp\pipenv> cat .env
PIPENV_VENV_IN_PROJECT 1
PS C:\tmp\pipenv> pipenv --two

Expected result
.env file is loaded, resulting in:
Virtualenv location: C:\tmp\pipenv\.venv

Actual result
.env file is not loaded, resulting in:
Virtualenv location: C:\Users\Karim\.virtualenvs\playground-pipenv-RDyG8wOS

To my understanding a .env file is specified to take KEY=VALUE specifiers that are newline separated. Is there something unique to Windows that is allowing you to exclude the = in your specification? Does your result change when you add the =?

or this may possible be a duplicate of #879.

Does your result change when you add the =?

When adding the = in between I get the following error when running pipenv shell where the .env file is actually loaded: TypeError: must be string without null bytes, not str

or this may possible be a duplicate of #879.

I guess not as there is no sign of the .env file being loaded at all when running pipenv --two.
Commands like pipenv shell show a message that the .env file is loaded:
Loading .env environment variables…

@kriim So is this more of a problem or an enhancement in your opinion?  There are other tools out there like direnv that can execute .env files upon entering a directory.

@erinxocon It feels counterproductive to use a second tool to handle env vars when pipenv already has all the info it needs, and is already doing it partially. I feel the .env file (out of version control) would be a good place to store for instance the tokens for private repositories.

I have the the same problem as others, need to install a package from a private repository. Having pipenv install using .env would be very very useful.


Change the type of pnls form the returns of simulator.

In the current pnls returned from simulator and passed to evaluator are the sum of instruments in each day.
In the notation now,  results.pnl[0] means the total pnl of first simulation day.
It would be changed into  liked results.pnl[0]['USD'] means the pnl of USD  in the first simulation day.
Thus, the line 58 in evaluator.py should be changed.
@hcshen916

It may just change line 58 from
pnls = np.array([item.pnl for item in data])
to
pnls = np.array([sum(item.pnl.values()) for item in data])
@hcshen916


Issue in Boostrap where you cannot add a second page

In Bootstrap on develop branch and you can only add a Section at top level and at page level

Yet you can create a page by selecting the default and duplicating... is this a bug?

This is me being silly and not having used Bootstrap for a while... now of course i've realised that you add a Page from the menu...


ALL JAILBREAK

When i am rebooting device and im trying to get into cydia i cant open it. ( when i am trying to open it it pops and crashes ) also tweaks which i installed before reboot. I cant see it and i could not see it before rebooting.

iOS Version: 12.4
iOS Device: iPhone 6s
unc0ver Version: 3.5.2

Place an "x" between the brackets if true:

[ ] this is a bug others will be able to reproduce
[ ] this issue is present with all tweaks uninstalled(except for default packages) or disabled
[x] this issue is present after a rootfs restore
[x] this issue is present on the latest version of unc0ver

Every time you restart your device you need to re-jailbreak with unc0ver jailbreak.

Kalakus... but what do i Have to do coz i cant see my tweaks


Better formatting of command output

When running
kubectl get service web -o jsonpath='{.spec.ports[*].nodePort}

The output of that command can be difficult to read because it's prepended to the start of the next command prompt.
30947mylaptopname:kubernetes-aws-workshop mitch (master) $

By using the following command, the output is more easily read
echo @($ kubectl get service web -o jsonpath='{.spec.ports[*].nodePort})

Here is the output:
mylaptopname:kubernetes-aws-workshop mitch (master) $echo $(kubectl get service web -o jsonpath='{.spec.ports[*].nodePort}')
30947

The command that works is:
echo $(kubectl get service web -o jsonpath='{.spec.ports[*].nodePort})

Fix coming


Reduce registerThemingParticipant

Ref #165169

[x] src/vs/editor/browser/viewParts/lineNumbers/lineNumbers.ts
[x] src/vs/editor/contrib/find/browser/findOptionsWidget.ts

Thanks for this PR! Can you look into the CI failure?


fix noJit builds

fix noJit builds

Hi @rajatd, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Rajat Dua). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@dilijev can you review this?

@dotnet-bot test nojit please

Jenkins is running slowly. I'm confirming locally that the change merged with master compiles and passes tests.

Looks like the test script is looking for the NoJit binaries in the wrong place for the -disableJit config. That may cause the disablejit checks I triggered to fail. I'll commit an updated script. This change builds, so I think this change will be fine as-is, once it can merge with the new test script.

@dotnet-bot test nojit please

LGTM barring checks and the test script issue I mentioned.

Please stop queuing more builds of the same changes - all you are doing adding more builds to the queue, making Jenkins take even longer.
Jenkins is a shared service between many teams, so be mindful of resources.
//cc @mmitche

@davkean @mmitche I understand your frustration. We feel it too. In this case I don't believe we're abusing the resource. This change affects the dailies so we need them to run to verify its correctness. As for the extra requeue of all checks, I apologize. I should have reviewed the change first and waited for the update before adding the dailies.

@rajatd All tests pass on my machine when this change is merged on top of #117.

@davkean @mmitche By the way, I've been wondering if there's an easy way to cancel all queued requests that haven't been issued build machines for a given PR? When a new commit is added to a PR and checks re-queue, do the old ones that haven't run yet get cancelled?

It doesn't automatically cancel, but looking at the source I'm led to believe it may be able to be done.
Sent from my Windows Phone

From: Doug<EMAIL_ADDRESS>Sent: 1/14/2016 5:51 PM
To<EMAIL_ADDRESS>Cc: Matt<EMAIL_ADDRESS>Subject: Re: [ChakraCore] fix noJit builds (#114)
@davkeanhttps://github.com/davkean @mmitchehttps://github.com/mmitche I understand your frustration. We feel it too. In this case I don't believe we're abusing the resource. This change affects the dailies so we need them to run to verify its correctness. As for the extra requeue of all checks, I apologize. I should have reviewed the change first and waited for the update before adding the dailies.
—
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/ChakraCore/pull/114#issuecomment-171843211.

I cancelled the daily checks that were already triggered on this one as I believe they will fail until #117 gets merged in. I'll re-queue the dailies once that's done.

@dilijev I was referring to these:

I thought these were duplicated because of your mention double mention of the trigger phrase for your nojit tests, but I was wrong. Apologies. Looking at them, I can't figure out why both sets were triggered:

@mmitche Maybe this is just Jenkins/Github catching up with the original request, and then queue again up when the PR branch was updated? They are so close to each other in time.

@rajatd You're good to merge this one one tests pass.

@dotnet-bot test dailies please

@digitalinfinity Do we currently support ARM builds for the disablejit configuration? Just want to know whether they were supposed to work.

@rajatd I accidentally triggered all the dailies instead of just the relevant ones. Windows 7 daily_x64_debug is a known failing configuration at the moment. I'll trigger another one and then kill it so at least this PR will be yellow instead of red.

@dotnet-bot test Windows 7 daily_x64_debug please

@dilijev , do we have any historical data pertaining to state of arm builds with no-jit?

@rajatd Not according to Jenkins. @digitalinfinity was working on this before so that's why I tagged him. I guess we could try the commit at the time Hitesh originally merged in the addition of the NoJIT configurations and see what happens there.

I tried from that commit with the current jenkins configs I recently corrected.
jenkins.buildone.cmd arm release "/p:BuildJIT=false"
Produces errors:
Chakra.Runtime.Language.lib(arm_Thunks.obj) : error LNK2001: unresolved external symbol "private: static void * (__cdecl*__cdecl
Js::DynamicProfileInfo::EnsureDynamicProfileInfo(class Js::ScriptFunction *))(class Js::RecyclableObject *,struct Js::CallInfo,..
.)" (?EnsureDynamicProfileInfo@DynamicProfileInfo@Js@@CAP6APAXPAVRecyclableObject@2@UCallInfo@2@ZZPAVScriptFunction@2@@Z) [E:\dev
\Chakra3\core\bin\ChakraCore\ChakraCore.vcxproj]
Chakra.Runtime.Language.lib(arm_Thunks.obj) : error LNK2001: unresolved external symbol "private: static void * (__cdecl*__cdecl
Js::InterpreterStackFrame::EnsureDynamicInterpreterThunk(class Js::ScriptFunction *))(class Js::RecyclableObject *,struct Js::Cal
lInfo,...)" (?EnsureDynamicInterpreterThunk@InterpreterStackFrame@Js@@CAP6APAXPAVRecyclableObject@2@UCallInfo@2@ZZPAVScriptFuncti
on@2@@Z) [E:\dev\Chakra3\core\bin\ChakraCore\ChakraCore.vcxproj]
E:\dev\Chakra3\core\Build\VcBuild\bin\arm_release\ChakraCore.dll : fatal error LNK1120: 2 unresolved externals [E:\dev\Chakra3\co
re\bin\ChakraCore\ChakraCore.vcxproj]

Same error as currently.  I think your change is good to check in as-is and we'll follow up with the ARM issue in a separate PR.

Committed as https://github.com/Microsoft/ChakraCore/commit/a0f58b0f499e488ed69bde93570229e169f488d1

Thanks @dilijev , I was waiting to see the build status turn green

@rajatd It was green before I marked as merged. That's how I knew it was in.


Minor comments for clarity

I did you did your edits on a stale version of the readme, it reverts a lot of my earlier PR - could you sync to master and pull in again?


Feature Request - Cache-Control

Add the ability to automate the Cache-Control header instead of an in-memory-cache.  This is how NextJS, and NextJS13.  This would be great for CDNs on Serverless Functions.
J

Hello, sorry for late reply.
This is doable, but can you explain more the benefits over the current way?
Thanks


[CI] Publish build scans to develocity.scala-lang.org

For the Context, Gradle is supporting the Scala Center with a free instance of Develocity that we can use to analyze the build scans of our open-source projects.
This PR configures publishing of build scans for every GitHub workflow of the scala/scala3 CI. All build scans will be publicly available on https://develocity.scala-lang.org/.
The sbt-develocity plugin can also do test retry and flaky test detection, which I will activate in a subsequent PR, on some test classes.

@adpi2 Here is the Develocity banner that should be added to the README.

The Cmd Tests action in the CI fails on:
/__w/scala3/scala3/project/scripts/../../project/scripts/sbt ';scalac tests/pos/HelloWorld.scala -d /tmp/tmp.gM6j1UN9n9 ;scalac -print-tasty -color:never /tmp/tmp.gM6j1UN9n9/HelloWorld.tasty' > /tmp/tmp.SRKPwqL5Yy
grep -qe '0: ASTs' /tmp/tmp.SRKPwqL5Yy

The TASTy tree is expected to be printed in the temporary file /tmp/tmp.SRKPwqL5Yy but it is only printed in the console.
I believe there is some sort of interaction between sbt-develocity and the build on the loggers and/or the System.out, which change the way log messages are directed to the console or the file.
@dotta @c00ler  @Duhemm can you help me with this.

This is ready for review. The scans should appear automatically on develocity.scala-lang.org after we merge.

It is still not ready to merge because we want to disable local and remote cache. We will re-enable it later once we agree on the rollout plan.


dotmailer provider

Provider for dotmailer

[ ] create prod account & setup oauth through dotmailer support & test it
[x] create conf.json
[x] authorize
[x] access_token
[x] refresh token
[ ] request
[x] create me.js
[x] get oauth keys for dev account
[ ] get oauth keys for prod account
[ ] test, test, & test

making a note:
dotmailer OAuth2.0 does not let you access their API. it's only purpose is to give users single sign on access
been working with their dev/support the past couple of days to see if it's something in their roadmap.


Adding Sonatype publishing information

Mostly to follow this: http://central.sonatype.org/pages/sbt.html
Before the first publish:

Follow 'First - PGP Signatures' to generate a GPG keyring
Follow 'Fourth - Adding credentials' to set your Sonatype credentials

To (always) publish run:
sbt publishSigned
and close the stage in the Sonatype Nexus as per the regular Sonatype workflow to trigger the push to Maven Central

Thanks, fixed this myself.


Remove requestID in grpc call

Can you please add your tests also before merging this PR

tested the results in IKS public cluster.


A11y_SQLAzureDataStudio_SchemaCompare_SchemaCompareOptions_Keyboard: Keyboard focus is not shifted to the latest popped up window on the screen.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
#A11yMAS;#A11yTCS;#SQL Azure Data Studio;#DesktopApp;#Win32;#MAS2.4.3;#Keyboard; #Benchmark;#A11y_SQLAzureDataStudioBenchmark;#A11ySev2;
Environment Details:
Version: 1.16.0-insider
Commit: 506c6a5e5f8423d3249670f55b1466b1980a5f03
Date: 2020-02-17T05:43:18.399Z
VS Code: 1.42.0
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363
Additional Details:
MAS reference: MAS2.4.3
Repro Steps:

Launch Azure Data Studio application.
Connect to server with windows credentials.
Navigate to Extensions from left menu.
Search & install Schema Compare extension.
Navigate to Connections from left menu.
Expand the server connected in step 2.
Right click an on prem database and select Schema compare from Context menu.
Navigate till Select Source or Select Target & hit enter.
Provide all the details for Source & Target.
Navigate till Ok button & hit enter.
Navigate till Options menu from Schema compare Menu bar.
After hitting enter, check if focus order is proper.

Actual:
The focus is not shifted to the new window that pop ups on the screen after hitting enter on the options menu.
Expected:
As soon as any new window pop ups which contains any interactive element inside it the focus should shift inside that new screen.
User Impact:
Keyboard as well as screen reader users will get confused as the focus remains on the previous screen and not shifted to the latest screen that is visible to us.
Attachment link reference:
9301_A11y_SQLAzureDataStudio_SchemaCompare_SchemaCompareOptions_Keyboard.zip

the new window should gain focus.

Dup of #9409

https://user-images.githubusercontent.com/100270074/217802104-84c70821-82c2-4690-86f8-e3b07989e077.mp4
Verified this issue in the below environmnet.
App Name: Azure data studio.
Version: 1.41.1 (user setup)
Commit: 33dce95b152227c4f10f6251ab30690fe9895c0c
Date: 2023-01-28T00:46:39.274Z
VS Code: 1.67.0
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22623
GitHub Tags:#Closed;

GitHubTags:#SQLADS-Win32-Feb2020;


Refactor captureException -> capture_exception (camelCase vs. snake_case)

(I know this  is maybe a low-priority issue compare to all the other 112, at the moment. Sorry in advance!)
I've happily seen in the source code that the camel case captureException is deprecated in the base implementation. (EDIT: Ooops, it captureExceptions! With an s. Anyway, this ticket is about camelCase vs. snake_case.)
However, the Django docs still instruct to use client.captureException() (source). Is there a reason for this?
I'd really like to use pythonic code wherever I can.
Disclaimer
I must admit, I have not invested more that a few minutes to figure out which code does exactly what. I've seen it's a bit complicated, probably for historical reasons (client is in raven.contrib.django.models, which is imported by raven.contrib.django.raven_compat.models).
I would guess this is really something that only needs to be fixed in the docs, but I'm really not sure what the history or technical background of the code snippets in the Django-related docs actually is.
It would be nice though, to have camel case replaced by snake case for pythonic code style. Not sure whether you run flake8 or pylint over the code base every once in a while. I'm not seeing any related configuration in the project root. It's probably not the top priority atm.

This is mostly a doc concern I agree. I think its worthwhile to get things updated, but theres a lot of APIs/general cleanup that we need to get done in the Python SDK.


Save/Run/Console Output/Error Handling

Hi Vsolina,
this is a VERY interesting and promising project!
I'm currently searching for a simple solution in the educational sector. A school class of 13/14 year old teens shall do simply programming on ESP32, but they only have iPads which makes it very hard. (No direct USB connection, no Thonny-IDE etc)
So a completely self hosted solution like your micropython-web-editor would be ideal.
Creating and saving files works great already. I saw you started some effort to run the programs in the development branch. I tried this version but it does not work well.
May I ask where you hit a wall? Do you have ideas how it could be improved?
Finally do you have ideas/hints on how to display console output? That would be especially important to return information about coding mistakes.
Thank you!

Hi Klemens,
Wow, this is such an interesting and novel use case!
It should definitely be possible to resolve any issues you're experiencing currently.
Regarding the problem with Run functionality: It is indeed a feature (kind of stuck) in development, using a tricky workaround mechanism (since doing this is not really supported cleanly by MPY), but last time I checked it everything worked well.
My initial assumption was - newer versions of MicroPython changed some system API which affected the implementation.
But I've tested it just now on a Generic ESP32 board with the latest stable MPY version (v1.22.1) and Run feature appears to be working well. (at least on a PC browser)
I'll test it on an iPad simulator (hopefully later today) to see what might be the issue.
What could help me diagnose the problem would be if you could tell me a bit more about your setup:

What kind of boards do you use?
Which MPY version is installed there?
Which iOS version are you using on your iPads?

This last question is likely the most important one.
Since you mentioned WebREPL terminal is not working or visible ("Finally do you have ideas/hints on how to display console output?")
I suspect your issue with Run might just be that you don't see the result of program execution.
Because often that's the only effect you'll see when you run a simple test python program: sysout print output; [unless you have some additional output hardware attached to your boards - e.g. LEDs, displays, etc.]
If you don't see the terminal output, could you elaborate a bit more on how exactly does the editor interface looks like to you?
Sounds like it could be the WebREPL setup issue, e.g.:
password must be enabled (import webrepl_setup; and follow the questions in the process),
your boot.py must contain:
import webrepl webrepl.start()
otherwise webrepl "service" does not start at all.
But it could also just be that the IDE web layout breaks on a specific version of iPadOS.

Wow, thanks for the quick and extensive reply!
I tried it from scratch and running scripts in the development branch works just fine. Sorry - probably Thonny interfered.
One major problem: On iOS Safari it's not possible to enter anything in webrepl because the virtual keyboard does not popup. Any idea about that?
I tried classic webrepl on port 8266 in Safari, but it doesn't work either, it just displays the HTML source code. I then tried webrepl from micropython.org, but I didn't manage to make a connection to my ESP32.
Same with Chrome.
BTW: I tried to eliminate the webrepl password as it is not necessary in this context. Any idea about that? Setting PASS = '' does not eliminate the login, you still have to press Enter.

No worries, I'm glad it works!
I've tested interaction with WebREPL box in the editor on iOS 9.3.x and iOS 17.1.2 and virtual keyboard pops up normally on both, allows password entry and further input.
My hunch is that you're experiencing this issue due to something unrelated because if it works on v9 and v17 - I suspect it should work on all versions in between. It is still possible that it does not work on a specific iOS version, so please tell me which iOS version is running on your iPad.
Two other ideas why the virtual keyboard does not show:
Having a bluetooth keyboard connected will disable virtual keyboard completely
Hardware issue or a software issue could make an iPad believe it has a keyboard connected
If it works in other text fields (e.g.: search/address bar) that pretty much eliminates these two possibilities.
Also worth mentioning: there's a limitation of MPY webrepl to a single connection (no concurrent/parallel sessions from multiple computers). If you have an active connection from your PC (or another iPad) it will not allow you to connect from another one in parallel.
But I've tested this exact scenario and virtual keyboard still shows up, it's just logging in that does not work.
And you can identify this state easily: immediately on load the WebREPL box will print "Disconnected" instead of asking for a password.

webrepl authentication is a part of micropython's binary. Overriding this behaviour from python seems impossible.
Apparently you'd have to change the implementation of webrepl_make_new and compile the MicroPython yourself:
https://forum.micropython.org/viewtopic.php?t=7289
What should be possible is adding an automated login feature in the editor's webrepl.
It would login automatically on page load instead of you, using some default password. (or could even ask for the pass on the first page load, store it locally (in safari) and then pre-populate it on subsequent loads)
But let's first figure out why the keyboard does not show up, then we can discuss this usability improvement.

The virtual keyboard works fine everywhere except for the webrepl passwort prompt in Safari. It's an iPad 8. It didn't work with iOS 16.7.2, and it does not work with iOS 17.2 (after updating). Very strange.
You're right that it must be a very specific problem. It works with Chrome now, and it works with Safari on an ancient iPad 4 with iOS 10.3.3...

What should be possible is adding an automated login feature in the editor's webrepl.
It would login automatically on page load instead of you, using some default password.
That would be great!

Apropos: "the WebREPL box will print Disconnected": The webrepl box disconnects when stopping a program with an infinite loop. That would be nice if fixed. Maybe an auto-reconnect together with the other change?
Thanks!

Thank you for the additional information.
Good news: I've reproduced the keyboard issue on an iPad simulator with 17.2
Should be relatively easy to resolve.
I'll take a look at the disconnect caused by an infinite loop.
Will get back to you as soon as I have some progress


Search box stopped working (HTML disabled on demo page)

Can be reactivated in the UI

Reactivated.


Update specs2-core, specs2-mock to 4.13.1

Updates

org.specs2:specs2-core
org.specs2:specs2-mock

from 4.6.0 to 4.13.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2" } ]

labels: test-library-update, early-semver-minor, semver-spec-minor

Superseded by #441.


DataTable does not support sorting with pagination

DataTable sorting only applies to the data in the DataTable
You can customized how the data is sorted, but it still only sorts the data in the Table
Example
I have 100 rows of data
I page my data using the carbon pagination compoenent
I want to display 20 rows in the first page
I give those 20 rows to the DataTable to show
When I sort on any column header the DataTable does not know about 80 of the rows that are not in the current page, so it can't sort on them. Sort results are therefore wrong.
Solution
A simple solution to to allow for a prop that is sortExternally
If this is present then the DataTable does not do any default sorting
Instead when you click on the header it gives to the sortExternally method the key of the header being sorted, and the direction.
Who will do the work?
I'm happy to make the necessary code changes, if you guys are happy to accept this change (after review etc)
What am I doing now
Currently I have an observer that watches when the column headers are clicked (based on class names)
That gives me the key, but not the direction of sort
The observer then look's at the column header, and gets the direction of the sort based on the icon
I also have a dummy custom sort method set in the props to turn off the default sorting
I'm assuming the next time the DataTable is changed this code will break
I would like to get the code change into the DataTable so that I can get rid of all my hacking code and just use the standard DataTable when I next upgrade my Carbon

This is already achievable.  Just keep a state on the direction and pass it to the table header.  You can also pass in your own sort function for onClick.
https://github.com/IBM/carbon-components-react/blob/master/src/components/DataTable/README.md#custom-sorting
https://github.com/IBM/carbon-components-react/blob/master/src/components/DataTable/TableHeader.js#L25

@nicoxxie I don't see how this will achieve what I want.
The customer sort only applies to the data that is in the table
For example I have 100 rows in my data
If I have 20 rows in the table the method is called 20 times to compare all the rows
However I need it to compare across all 100 rows
The DataTable does not know about the other 80 rows so it cant sort on it
The customer sort does not tell me the header that was clicked on - so I cant call back to my data becasue I dont know what column was sorted by the click
In relation to storing the direction of the sort we have the same kind of problem
The sortdirection will be different for each column, so if I dont know the column storing the direction wont help me

@paulbarr Yes it's achievable you can do the sorting on your own side and pass in the data yourself.  The table is just a display, you can do you own logic anyway you want.  We've been doing that for a while.

Going to close this issue as there are solutions that exist in user-space (thanks @nicoxxie for the insights here! 🎉)
At a high-level, we're definitely hoping to add in Pagination in a future release, alongside other behavior like inline editing. However, in the meantime this will be something that teams will need to implement, unfortunately.
As always, will also support PRs for adding in this kind of behavior!