Remote API /build method should accept Dockerfile content as part of the payload

Currently the /build endpoint of the API accept an additional query parameter ?dockerfile= that specify the path to the Dockerfile relative to the context.
This introduce the following limitations:
a. when using -f, the client will fail a build if the Dockerfile is not  contained in the context directory
b. this prevent reusing the same Dockerfile between different context.
What about passing the Dockerfile content in the POST /build payload instead? We could optionally accept a multipart body with both the Dockerfile and the context.tar content.
Related #8418 #12072.

@proppy This is by design if I'm not mistaken.

@cpuguy83 yes, I'd like to discuss the limitation of the current design and possible evolution in this issue :)

"./some-script.sh | docker build -f - some/context" would be pretty cool
too (and possible if it were a separate payload), IMO :+1:

This would also make it possible to use -f to point at a Dockerfile that is technically in the context, but lives in .dockerignore (which is currently not possible).

Gonna have to rescind my most recent comment -- -f on a file in .dockerignore does in fact work today. :+1:

I'm closing this issue since it looks like there have not been any discussion about it in a year. Please, don't start it now. :wink:
Thanks a lot for your comments.


show help text when "dbt" is entered without args

https://github.com/analyst-collective/dbt/issues/40


⚠ Bofrost.at has degraded performance

In b04bfca, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2188 ms

Resolved: Bofrost.at performance has improved in 595a2d5 after 7 minutes.


Server: Touch version file in /src/main/kotlin/resources/client/public/version.json

This will tell the client, if there is a new version.

Done, but env variable does not work :(


Adjust tools version for folks running on 2022

Adjust tools version for folks running on 2022
PR Checklist

[x] Closes annoyance that @lhecker and I have selfhosting VS2022
[x] I work here
[x] Solution built
[x] @dhowett said something like "lol sure" in Teams.

@msftbot merge this in 2 minutes


Missing i18n files for some languages

Opening this issue to do PR with some translations

Please open a PR for just one issue as mentioned in #22.


implements try catch

fixes #424

The Try and TryNoValue opcodes will need added to the ProcDecoder class, which is used by both the debugger and the decompiler.

The Try and TryNoValue opcodes will need added to the ProcDecoder class, which is used by both the debugger and the decompiler.

Im unsure where i need to add these opcodes in the file, as well as how to test it. Can you ping me on discord with instructions?


chore(main): release wombat 4.4.0

:robot: I have created a release beep boop
4.4.0 (2024-11-17)
Features

add days support to timerange (#165) (2ae5510)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

wombat-v4.4.0
:sunflower:


stac-fastapi-core does not support configuring a STAC application with common extensions disabled

Describe the bug
When attempting to define a stac-fastapi-core-based application with the Query Extension disabled, I found that stac-fastapi-core would emit an attribute error and crashfail.
Specifically, the logic for handling a POST Search request here:
https://github.com/stac-utils/stac-fastapi-elasticsearch-opensearch/blob/49fecf95a986ea5cca60af60d6b16a194cd23439/stac_fastapi/core/stac_fastapi/core/core.py#L566
I found that this also happens with the sortby and token pagination extensions. Removing these makes the application inoperable.
To Reproduce

Create a new application, e.g., based off of the mongodb example
Remove the QueryExtension from the application's extensions
Start the app (it will start)
Send any valid POST Search
:boom:

Expected behavior
With Query disabled, searches should function.
The same holds for Sort and Token Pagination, but the case is clearest with Query (which is deprecated).
Additional context
I'm experimenting with building an application for my team, so I don't have a lot of knowledge around STAC. We're still learning the spec(s) and trying things out.
For now, we're trying to get a very early stage prototype up and running, so I've implemented this very clumsy hack:
class CustomizedCoreClient(CoreClient):
    async def post_search(self, search_request, request):
        object.__setattr__(search_request, "query", None)
        object.__setattr__(search_request, "sortby", None)
        object.__setattr__(search_request, "token", None)
        return await super().post_search(search_request, request)

I think the best pattern for the stac-fastapi-core code would be to check with hasattr(...) or three-argument getattr(..., ..., None).
I'm happy to contribute a PR with this sort of fix if it would be welcome.

Hi thanks for bringing this to our attention. Feel free to make a pr. This should definitely be fixed.

I have been working on an adjacent issue and have added a condition for when an extension is enabled/disabled here #267. Maybe it is useful. Let me know if you have any suggestions for this approach.


ng: add flag to start in $$-expression mode

Note that if you symlink ngsh to ng it will start in shell mode. But I agree with you, it should also be a flag on ng.


Inheritance and external load/save functions

Hi,
I am trying to resolve #459. I use 3part library and want to add some serialize functions. So it's obviously I want place then in my code as external.
Let's say I use struct X that is from external library. So I do:
struct X {
    int x;
};

template<class Archive>
void load(Archive & archive, X& x)
{
    archive(
        x.x
    );
}

template<class Archive>
void save(Archive & archive, const X & x)
{
    archive(
        x.x
    );
}

Now I want to create my own struct Y that will inherit from X:
Code is not important, just example. It's just meter it use cereal::base_class<T>
struct Y: X {

    Y()
    {
        force = std::make_optional<bool>(true);
    }

    std::optional<bool> force;

    template<class Archive>
    void save(Archive & archive) const
    {
        archive(
            cereal::base_class<X>(this)
        );

        if(force)
            archive(
                cereal::make_nvp("force", *force)
            );
    }

    template<class Archive>
    void load(Archive & archive)
    {
        archive(cereal::base_class<X>(this));

        bool tmp_val;
        try {
            archive(cereal::make_nvp("force", tmp_val));
            force = std::make_optional(tmp_val);
        }
        catch(cereal::Exception &e)
        {
            archive.setNextName(nullptr);
        }
    }
};

And the problem is the Y class can also by serialized by X load / save funtions... So cereal give me static asscertion

../cereal/include/cereal/cereal.hpp:472:9: error: static assertion failed: cereal found more than one compatible output serialization function for the provided type and archive combination.

How can I go around this problem?

We have a way to deal with this in cereal, take a look at the inheritance section of the documentation, or this doxygen page: http://uscilab.github.io/cereal/assets/doxygen/structcereal_1_1specialize.html

Thx that works. I thought this was just for serialized load/save.


Fix small issues that confuse new devs

What this does
Started a new project recently with devs who hadn't seen this tech stack before. Ran into several points of confusion. Adding those improvements here.
Checklist

[x] PR template
[x] Command and docs to seed Docker with test data
[x] Improve root README with more steps
[x] Better detail descriptions for .env.local.example

Deploy Notes
Add any notes for changes needed when deployed. I.e. API keys, environment variables, database changes, etc.
How to test
Add user steps to achieve desired functionality for this feature.

Something weird is going on in Heroku. This PR doesn't have any crazy changes.
The builds are failing with a reference to react-native. I suspect the cache is shared between our PRs on the Heroku side. Normally that wouldn't be a problem but right now there is another PR open that adds react-native.


Altitude error

Strange bug. I select objects for copying. I will insert the blueprint in a new location. But objects for some reason are built slightly lower than the original. What is the reason for this error?
I recorded a video so that it would be clear what it was about.
https://youtu.be/72Z4PNCB2t4

The precision of float type in c++ is not good enough, it can be solved. But, it need to check tons of codes, I will do it after the main functions.


Management interface does not always use Digest authentication

Right now wildfly::util::resource items are created using direct interaction with the http management API using the wildfly_cli.rb code. This code makes an invalid assumption that authentication to the management API always uses Digest.
However, I observed that when configuring your own security realm with LDAP, and attaching it to the http management interface, the authentication method can differ. E.g. with the authentication=ldap setting, authentication on the http management interface uses Basic auth (!).
It was my invalid assumption that Wildfly would handle http authentication, and only delegate the validation of the credentials to the methods defined by the security realm. This is not true, as the entire http request seems to be forwarded. So the http authentication used, depends on the configuration of the security realm.
I will provide a pull request that makes wildfly_cli.rb detect which authentication method to use, and switch between Digest or Basic accordingly.

Awesome.
I wasn't aware of this... Actually, still trying to understand why it uses BASIC instead of DIGEST for LDAP Authentication


Remove pprof

The pprof version here is old and not maintained. As far as I know there's no reason to use it over the go version, but it's easy to install it and think that it is the one that you should be using, because it has the same name, a similar interface, and it never tells you not to use it.
The only place it tells you is in the plain text readme in the overview, which nobody is going to read.
It should just be removed. Or at the very least it should print a huge warning every time you run it.

I was fairly certain I did the warning, but apparently I didn't. So I've added configure flag to stop installing pprof. Next step is getting distros actually use this flag and ship golang pprof instead.

Probably time to just remove it now surely?

Time, sure. There are several "but"-s.
a) we still use pprof for symbolization (that is, we have backtrace as array of instruction pointers and we want to get to array of function:line strings). pprof there works as a "smart wrapper" around addr2line and similar tools. We need that symbolization replacement. There are several possibilities I am aware of:
**) Someone could re-do the wrapping part in c++ of doing symbolization via addr2line and similar tool.
**) Abseil actually ships the code that does native symbolization for elf+dwarf combination, but we cannot easily take it since it uses plenty of abseil types and utilities. But maybe we could adapt it, somehow
**) Also gcc has been shipping backtracing facility for a number of years how, which also includes and provides backtrace -> functions mapping as well. But integrating it is another piece of work. Plus clang and libc++ popularity implies that it cannot be our only option (well, maybe, if we're able to vendor this code perhaps).
**) llvm ships symbolization tool (but this is perhaps yet another variant for first option above of piping into addr2line or similar tool)
**) c++-20 now has backtracing module, but it seems to be very far from our needs (neither symbolization nor, it seems, backtracing)
**) another option might be to convince go pprof folk to accept or implement public symbolization feature. I've already asked and already got nack, but it was like 8 years ago or so.
b) our various profiling tests rely on pprof and possibly (but worth checking BTW) in a way that is not compatible with go pprof. Tests could be adjusted yes, but it is one piece of work.
c) last year I filed tickets to Debian and Fedora for them to stop shipping pprof and they still did nothing. They really should be shipping go pprof as far more feature-ful version, but they sadly haven't. If we suddenly stop shipping perl pprof I am not sure what is going to happen. We may argue that it'll act as nice forcing function. Or maybe not.
If you have time to spend on this, you're very welcome to contribute (but if so, please ping me first to coordinate)

https://bugzilla.redhat.com/show_bug.cgi?id=2232044
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1049383


Recording size

Dear @joisarjignesh , Can we calculate the size of each recording record id?
The API get me some information but there is not any further information in documentation.
Do we need to use https://github.com/elsonwu/bbb-record-size ?

@cod3r0k it's read only but you can try (i don't know how to install and used this package)

@cod3r0k it's read only but you can try (i don't know how to install and used this package)

Thank you @joisarjignesh . As you mentioned in https://github.com/joisarjignesh/bigbluebutton#get-recordings , we can get recording items with get recording function. It has one parameter which related to the size. But there isn't any further information that what is this size? is it Byte? is it MB? or ...
Because I check it's related directiory in /var/bigbluebutton/published/presentation/###id### and there are many conflict with that amount size. Do you know it or not?

@cod3r0k i don't know


Support multi-file models

The LSP currently only supports a single model file.
Support should be added for using multiple files within a workspace.

Support for this was added in https://github.com/awslabs/smithy-language-server/pull/32


The uninstall function doesn't actually uninstall the tools.

Hi,
I used your package to install the vue-devtools into my electron app. I ran into an error that is coming from vue-devtools and as part of the troubleshooting process I tried to remove vue-devtools from the electron app using your provided uninstall method.
Regardless of how I attempt the removal it does not actually remove the vue-devtools. Is there a proper way of doing this? Please let me know if there's any other information that I could provide that would be helpful in tracking down this issue.
The project I'm working on is an opensource app if it helps: https://github.com/chris-schmitz/app-launcher/tree/vue-devtools-issue. I created a new branch and pushed a WIP commit so that you could see the files as they are now. The electron entry point is here.

Thanks!

Already fixed


[TACHYON-782] Disable flakey delete during eviction test.

https://tachyon.atlassian.net/browse/TACHYON-782
This is a temporary bandaid for ticket 782. The long term solution should be to repurpose the test to the new worker delete and eviction logic.

+1 LGTM! This ticket spoiled my day... I copied your modification above to my PR so that it can pass my local mvn test.

+1 LGTM. I will dig into this test


MedSAM 2 checkpoint release?

When will checkpoints for SAM 2 (video, image) be released, if they aren't available already @JunMa11 ?

Hi @25benjaminli ,
We are using the office SAM2 checkpoint for benchmarking.
Fine-tuning the video segmentation model is not trivial, we are still working on the coding. Stay tuned.

@JunMa11 thanks for the response. For the original MedSAM, you released the medsam_vit_b.pth checkpoint - will there be a similar checkpoint released for MedSAM 2?

Hi @25benjaminli ,
Yes, once we figure out the training, a fine-tuned model will be released.


feat: Add task-type-embedding notebook

Adding a new notebook for task-type-embedding.

Hi Holt, thanks a lot for the quick and detailed review!! Please check my fixes. Also, I've added a new section "Try the task types" so please review it too.


switch to babel-transpiler and sprockets 4

why not switching to the babel-transpiler gem with sprockets version 4?

cc @MattMSumner

I followed the instructions from babel itself https://babeljs.io/docs/setup/#rails. I can checkout babel-transpiler on Friday to see if it makes sense to switch.

@iDuuck why do you think we should switch to babel-transpiler? Thank you!

@MattMSumner @iDuuck do we need to work on anything for this? Id there anything else we should discuss?

The official rails repositories update guide to the newer sprockets said that babel-transpiler is the way to go with Rails. Don't know if this is better than sprockets-es6.
As far as I can read from this README, the only thing that needs to be done for this, is to add the babel-transpiler gem to the Gemfile

Sprockets 4 ships with a babel processor

It looks like this is currently in beta2. As soon as a stable release come out I'll open a PR to change this.
Thanks @iDuuck for pointing this out.

@MattMSumner do you think it's ok to apply this fix while sprockets is in beta? I think it's better than the myriad deprecation messages that appear if you add ES6 files to a new suspended project. Thanks!

I just found out, that this is possible (and used by this company for the clients):
*The .babelrc needs to be there and all needed packages in package.json (babel-core and plugins)
Gemfile:
gem 'sprockets', '4.0.0.beta2'
gem 'babel-transpiler'
gem 'sprockets-commoner'

app/assets/config/manifest.js
//= link_tree ../images
//= link_directory ../javascripts .js
//= link_directory ../stylesheets .css

Read more...

@tute you might be right. Alternatively do you think I should remove babel entirely and have suspenders setup without a default until sprockets ships?
@iDuuck thanks for the link. I'm not sure I want to remove the asset pipeline entirely in favor of webpack by default.
I'll have some time on Friday to open a couple of PRs and spark more conversation.

Suspenders is now on Rails 6, which uses webpacker for JS, so I think this issue is no longer relevant.


support of AWS ELB in kubeapi-load-balancer

on AWS it makes more sense to use ELB instead of dedicated VM for kubeapi-load-balancer
using there m3.medium instance balancer comes at the high price and with no HA :)

Juju currently doesn't have any awareness of the AWS ELB proxy resource. There has been talk off/on about this previously. I think tapping someone like @marcoceppi on this one would be good, and he can address accordingly.

This seems like it can be resolved with the new cloud-controller-manager daemon thats shipping as beta in 1.7.  We've spoken to this briefly internally and will open a lot of cloud integration points.
Documentation relating to this new daemon is here https://kubernetes.io/docs/tasks/administer-cluster/running-cloud-controller/


quantile2 throws an error when probs=1 and type="caaqs"

R version 4.3.1, rcaaqs version <IP_ADDRESS>0
When using the quantile2 function to calculate percentiles and error is thrown when probs=1 and type="caaqs:
library(rcaaqs)

pm25_sample_data %>%
  dplyr::group_by(year) %>%
  dplyr::summarise(p100=rcaaqs:::quantile2(value,
                                           probs=1,
                                           na.rm=FALSE,
                                           type="caaqs"))

This is the error:

When a different probs value is chosen there isn't an issue:
pm25_sample_data %>%
  dplyr::group_by(year) %>%
  dplyr::summarise(p99=rcaaqs:::quantile2(value,
                                           probs=0.99,
                                           na.rm=FALSE,
                                           type="caaqs"))

Here's the output:

Interesting.
It works when you use one of the stats::quantile type argument options:
rcaaqs::pm25_sample_data |>
  dplyr::group_by(year) |>
  dplyr::summarise(p100=rcaaqs:::quantile2(value,
                                           probs = 1,
                                           na.rm = FALSE,
                                           type = 7))
#> # A tibble: 4 × 2
#>    year  p100
#>   <int> <dbl>
#> 1  2011   103
#> 2  2012   116
#> 3  2013   122
#> 4  2014    20

Which is the same as using stats::quantile() directly (the rcaaqs:::quantile2 internal function is a wrapper around stats::quantile):
rcaaqs::pm25_sample_data |>
  dplyr::group_by(year) |>
  dplyr::summarise(p100=stats::quantile(value,
                                        probs = 1,
                                        na.rm = FALSE,
                                        type = 7))
#> # A tibble: 4 × 2
#>    year  p100
#>   <int> <dbl>
#> 1  2011   103
#> 2  2012   116
#> 3  2013   122
#> 4  2014    20

The warning is telling us the issue is withdplyr::summarise(), and that some deprecated code in dplyr has changed some behaviour for the "caaqs" method:
rcaaqs::pm25_sample_data |>
  dplyr::group_by(year) |>
  dplyr::summarise(p99=rcaaqs:::quantile2(value,
                                          probs = 1,
                                          na.rm = FALSE,
                                          type = "caaqs"))
#> Warning: Returning more (or less) than 1 row per `summarise()` group was deprecated in
#> dplyr 1.1.0.
#> ℹ Please use `reframe()` instead.
#> ℹ When switching from `summarise()` to `reframe()`, remember that `reframe()`
#>   always returns an ungrouped data frame and adjust accordingly.
#> Call `lifecycle::last_lifecycle_warnings()` to see where this warning was
#> generated.
#> `summarise()` has grouped output by 'year'. You can override using the
#> `.groups` argument.
#> # A tibble: 0 × 2
#> # Groups:   year [0]
#> # ℹ 2 variables: year <int>, p99 <dbl>

> lifecycle::last_lifecycle_warnings()
[[1]]
<warning/lifecycle_warning_deprecated>
Warning:
Returning more (or less) than 1 row per `summarise()` group was deprecated in dplyr
1.1.0.
ℹ Please use `reframe()` instead.
ℹ When switching from `summarise()` to `reframe()`, remember that `reframe()` always
  returns an ungrouped data frame and adjust accordingly.
---
Backtrace:
    ▆
 1. ├─dplyr::summarise(...)
 2. └─dplyr:::summarise.grouped_df(...)
 3.   └─dplyr:::summarise_deprecate_variable_size()

Looking at the type = "caaqs" code, I can see that when prob = 1 nothing is returned for each year, and dplyr::summarise is complaining.
x <- rcaaqs::pm25_sample_data |> dplyr::filter(year == 2014) |> dplyr::pull(value)

probs <- .99

x <- sort(x, decreasing = TRUE)
n <- length(x)
i.d <- probs * n
i <- floor(i.d)
ret <- x[n - i]
ret
#> [1] 20

probs <- 1

x <- sort(x, decreasing = TRUE)
n <- length(x)
i.d <- probs * n
i <- floor(i.d)
ret <- x[n - i]
ret
#> numeric(0)

I'd love to know if this code ever worked with prob = 1, or how dplyr pre-v1.1.0 behaved? But moving forward, given all the other quantile methods return the first integer in the vector of values when prob = 1, I am guessing the "caaqs" method should do the same? If so, maybe tweaking the code to return x[1] when i == n?
x <- rcaaqs::pm25_sample_data |> dplyr::filter(year == 2014) |> dplyr::arrange(desc(value)) |> dplyr::pull(value)

probs <- 1

x <- sort(x, decreasing = TRUE)
n <- length(x)
i.d <- probs * n
i <- floor(i.d)

if (n == i) {
  ret <- x[1]
} else
ret <- x[n - i]
ret
#> [1] 20

Thanks @stephhazlitt! Yes that is my understanding - that x[1] should be returned when type="caaqs" and probs=1 i.e. the maximum value in x should be returned. I think your proposed update will do the trick for this case.

Thanks @stephhazlitt! Yes that is my understanding - that x[1] should be returned when type="caaqs" and probs=1 i.e. the maximum value in x should be returned. I think your proposed update will do the trick for this case.

OK, thanks for confirming @DonnaHaga. I'll try and put up a pull request today which this behaviour change.

Issue closed with https://github.com/bcgov/rcaaqs/pull/64, thanks @stephhazlitt


Uses package main even with -o

Contrary to what documentation claims the package directive is still main if I ask pkger to write the file in another directory.
$ pkger -o config
$ head -n 1 config/pkged.go
package main

👍

Ran into this issue recently when upgrading, and had to revert to v0.12.8.
Did a git bisect, and it indicated:
https://github.com/markbates/pkger/commit/b334bd73acb5839bc08969a0c85960ef64dca383
Lines 146 to 148 may be reverting name back to main.

Using v0.12.7 appears to avoid this problem (thanks @fpaniagua for doing a bisect).

PRs are welcome

@markbates It would help if it was more obvious why b334bd7 does what it does. I'm quite happy to work up a fix if I know what I'm fixing :-)

Without digging in my gut is that it is meant to check if the output directory is main and set the name to that regardless.
--
Mark Bates

On Feb 14, 2020, at 6:19 PM, Alastair Houghton<EMAIL_ADDRESS>wrote:

@markbates It would help if it was more obvious why b334bd7 does what it does. I'm quite happy to work up a fix if I know what I'm fixing :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Facing a similar issue here - if the root directory is a module, i.e. is not of package main and has any *.go files under root, pkger -include xxx will fail (on package name conflict).
@markbates I don't think the assumption that "check if the output directory is main" is correct for the PR https://github.com/markbates/pkger/pull/53 It should be inspecting the actual output dir's current package name and set it to that, even if at top level, it's not guaranteed to be main.
To show how painful it is for me to generate pkger.go correctly:

Rename all the *.go's package names to main.
Call pkger -include xxx.
Change all *.go package names back to <original>.
Rename pkger.go's package name to <original>.
Build.

---end of rant---
I am providing a fix for both my and OP's problem here: https://github.com/markbates/pkger/pull/80
Thanks for consideration.

Thank you everyone for tracking this down and for fixing it. Release in the making.

Is this fixed? I am having this issue


Change OomKillDisable to be pointer

It's like MemorySwappiness, the default value has specific
meaning (default false means enable oom kill).
We need to change it to pointer so we can update it after
container is created.
Signed-off-by: Qiang Huang<EMAIL_ADDRESS>

LGTM after rebase

Wait for https://github.com/docker/engine-api/pull/21

Can you add a test that will fail on master?
Seems like it's specifically an issue with docker update?

looks that your need a rebase

@wenchma Thanks for reminding, I'm waiting for a new tag for engine-api.
@cpuguy83 Yeah it's an issue with docker update, but docker update hasn't support oomKillDisable yet, so the test case needs to be added when I add oomKillDisable support for docker update, and that won't be a "fail on master" case either, because this PR fixes the fail case.

I'm carrying this PR with the api vendoring in #19243 to make everything work. Thanks!


Server crash during SQL injection

Hi,
I just crashed the app. I'm trying not to spoil myself and haven't read most of it - I union'ed sqlite_master. I added the traceback I found to the npm.log.
npm-debug.txt

Your log doesn't really spoil any challenges, so I'll re-post it here:
0 info it worked if it ends with ok
1 verbose cli [ '/opt/node-v4.3.0-linux-x64/bin/node',
1 verbose cli   '/opt/node-v4.3.0-linux-x64/bin/npm',
1 verbose cli   'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info prestart<EMAIL_ADDRESS>6 info start<EMAIL_ADDRESS>7 verbose unsafe-perm in lifecycle true
8 info<EMAIL_ADDRESS>Failed to exec start script
9 verbose stack Error<EMAIL_ADDRESS>start: `node app`
9 verbose stack Exit status 1
9 verbose stack     at EventEmitter.<anonymous> (/opt/node-v4.3.0-linux-x64/lib/node_modules/npm/lib/utils/lifecycle.js:214:16)
9 verbose stack     at emitTwo (events.js:87:13)
9 verbose stack     at EventEmitter.emit (events.js:172:7)
9 verbose stack     at ChildProcess.<anonymous> (/opt/node-v4.3.0-linux-x64/lib/node_modules/npm/lib/utils/spawn.js:24:14)
9 verbose stack     at emitTwo (events.js:87:13)
9 verbose stack     at ChildProcess.emit (events.js:172:7)
9 verbose stack     at maybeClose (internal/child_process.js:821:16)
9 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
10 verbose pkgid<EMAIL_ADDRESS>11 verbose cwd /home/saftladen/juice-shop
12 error Linux 3.16.0-4-amd64
13 error argv "/opt/node-v4.3.0-linux-x64/bin/node" "/opt/node-v4.3.0-linux-x64/bin/npm" "start"
14 error node v4.3.0
15 error npm  v2.14.12
16 error code ELIFECYCLE
17 error<EMAIL_ADDRESS>start: `node app`
17 error Exit status 1
18 error Failed at the<EMAIL_ADDRESS>start script 'node app'.
18 error This is most likely a problem with the juice-shop package,
18 error not with npm itself.
18 error Tell the author that this fails on your system:
18 error     node app
18 error You can get their info via:
18 error     npm owner ls juice-shop
18 error There is likely additional logging output above.
19 verbose exit [ 1, true ]

To be honest, this error does tell anything about what happened during the crash. Did the app crash during subsequent npm startcalls after you did this SQLI UNION?
If you can reproduce this (or a similar) crash with the latest version, please post the SQLI string you used along with the crash log.

Maybe try  https://github.com/rook2pawn/node-ddos to prevent death from request overload...?


Fluentd security impact with root user

Docker provides a privileged mode, which lets a container run as ‘root’. Running a container in privileged mode provides the capabilities of that host—including:
Root access to all devices
Ability to tamper with Linux security modules like AppArmor and SELinux
Ability to install a new instance of the Docker platform, using the host's kernel capabilities, and run Docker within Docker.
Privileged containers create a major security risk—enabling attackers to easily escalate privileges if the container is compromised. Therefore, it is not recommended to use privileged containers in a production environment. Best of all, never use them in any environment.
This is extremely important to mitigate vulnerabilities in daemons and container runtimes, which can grant root access of entire nodes and clusters to an attacker.
Earlier there was a thread but no outcome came in that
https://github.com/fluent/fluentd-kubernetes-daemonset/issues/420
Please provide some solution to resolve this so that  fluentd with non-root user can work in a similar way without keeping the security on risk.
Reference link for Docker security guidelines
https://cheatsheetseries.owasp.org/cheatsheets/Docker_Security_Cheat_Sheet.html

Is there any update? I have similar requirement for our client who doesn't want to use ROOT user to process log files. Tried options to add UID:GID in Dockerfile as well as ENV FLUENTD_UID. None worked.

Any update on how to run Fluentd as a non-root user in a Kubernetes environment as a daemonset?


⚠ Bofrost.de has degraded performance

In 6839868, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1102 ms

Resolved: Bofrost.de performance has improved in 7773e7c after 25 minutes.


Add define command to define common words and terms

Add a simple command to define a word. I was thinking this should be an API somewhere we can hit to get a reasonable answer but realistically it will have to be a curated list.
The terms to be defined would probably just be Go terms, but there could be a fallback to a dictionary API for ESL people. It's probably over-complicating though.
Previous discussion here: https://github.com/gobridge/gopher/issues/33

@ScottMansfield are you aware of anything that exists like this today, even in a text form?

This is basically finished, but am thinking it makes sense to get more definitions in before closing this:
https://github.com/gobridge/gopherbot/blob/master/glossary/terms.go#L5
Mind contributing some?


Reliability - Critical - Configurable timeout for TransferProcess

TransferProcesses might get stuck in a state for a long time, or even indefinitely when errors occur. The TransferProcesManager should monitor this situation and react to it after a configurable threshold by moving the process to an ERROR state, effectively taking the TransferProcess out of the state machine processing loop.
The threshold should be configurable on a TransferProcess basis, as appropriate thresholds may diverge vastly depending on the nature of the transfer itself (moving few KB VS several GB). A sensible default value should be used in case no configuration is available for a given TransferProcess.

For reference, here we implemented a watchdog process that cancels long running TransferProcesses. Probably it is easier to solve this issue directly in the main state machine loop though.

@juliapampus this can be the issue that could make an use of the stateCount field.
We should check, after an error on state transition, how many times that happened and, over a certain threshold (5? 10? configurable?) the TransferProcess should be cancelled.
This behavior should be valid also for the other state machines (provider/consumer contract negotiation)

Seems to be closed by #710.

@juliapampus probably not, but this issue will become obsolete since the 2-state transitions will be applied on all the state machines (#831 #870 ), because there will be no "staled states" anymore as everyone will have it's "processor" on state machine (apart from "final" states).
So I'm ok to close this anyway.


Build job update

Add koji build to supported jobs and update triggers for build jobs.
I've marked koji builds as in preview to make it sure it's not fully supported.

[x] I'll regenerate html the content after the review.

Or I can push the re-generate commit here to avoid all the (different hugo version caused) whitespaces added/removed noise.

@jpopelka That would be nice, thanks!

force pushed :-)

Thanks!


Add autocomplete to the searchboxes with table names or map names

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


experiment_id - please add new FAFMIP experiments

Please could you add the following three entries to the experiment_id table. This change to the experimental design was agreed by the steering committee and project participants at a meeting in Reading in April 2019.
"faf-heat-NA50pct":{
"activity_id":[
"FAFMIP"
],
"additional_allowed_model_components":[
"AER",
"CHEM",
"BGC"
],
"description":"1xCO2 experiment, parallel to piControl, forced over the ocean by the same method and with the same surface net heat flux perturbation field as in faf-heat, except that within part of the North Atlantic ocean the perturbation is multiplied by 0.5",
"end_year":"",
"experiment":"control plus perturbative surface flux of heat into ocean",
"experiment_id":"faf-heat-NA50pct",
"min_number_yrs_per_sim":"70",
"parent_activity_id":[
"CMIP"
],
"parent_experiment_id":[
"piControl"
],
"required_model_components":[
"AOGCM"
],
"start_year":"",
"sub_experiment_id":[
"none"
],
"tier":"1"
},
"faf-heat-NA0pct":{
"activity_id":[
"FAFMIP"
],
"additional_allowed_model_components":[
"AER",
"CHEM",
"BGC"
],
"description":"1xCO2 experiment, parallel to piControl, forced over the ocean by the same method and with the same surface net heat flux perturbation field as in faf-heat, except that within part of the North Atlantic ocean the perturbation is zero",
"end_year":"",
"experiment":"control plus perturbative surface flux of heat into ocean",
"experiment_id":"faf-heat-NA0pct",
"min_number_yrs_per_sim":"70",
"parent_activity_id":[
"CMIP"
],
"parent_experiment_id":[
"piControl"
],
"required_model_components":[
"AOGCM"
],
"start_year":"",
"sub_experiment_id":[
"none"
],
"tier":"2"
},
"faf-antwater-stress experiment":{
"activity_id":[
"FAFMIP"
],
"additional_allowed_model_components":[
"AER",
"CHEM",
"BGC"
],
"description":"1xCO2 experiment, parallel to piControl, forced over the ocean with the momentum flux perturbation field of faf-stress and a surface freshwater flux of 0.1 Sv in total to be applied uniformly around the coast of Antarctica in whatever way is most suitable in the model",
"end_year":"",
"experiment":"control plus perturbative surface fluxes of momentum and freshwater into ocean, the latter around the coast of Antarctica only",
"experiment_id":"faf-antwater-stress",
"min_number_yrs_per_sim":"70",
"parent_activity_id":[
"CMIP"
],
"parent_experiment_id":[
"piControl"
],
"required_model_components":[
"AOGCM"
],
"start_year":"",
"sub_experiment_id":[
"none"
],
"tier":"2"
},

@JonathanGregory thanks for this. @taylor13 @veyring any comments? I plan to complete these registrations along with a number of model registrations later today

@veyring, @JonathanGregory (See last paragraph for bottom line.)    I have no strong feelings about this, but I think the formal rule is that experiments can be added, but only if they are lower than tier 1.  ( faf-heat-NA50pct above is specified as tier 1.)  As I recall, this rule was put in place because modeling groups interested in a MIP had to commit to perform all the tier 1 experiments defined by that MIP.  It seems to me to be unfair to the modeling groups at this point to insist they perform additional experiments to participate in a MIP.
On the other hand if the experiment is really essential for understanding the suite of already-defined FAFMIP experiments, then I definitely think the rule should be bent.
Reading the experiment description, I can see this is an important but perhaps not essential additional experiment.  For the sake of following the rules, would it be o.k. to change the designation from tier 1 to tier 2 above, and by other means communicate to FAFMIP participants what priority you assign to each of the FAFMIP experiments?
All that being said, we are clearly not monitoring which modeling groups are living up to their commitments, so these commitments are somewhat meaningless.  Given that reality, perhaps we should let tier 1 experiments be added to accurately reflect their priority in the eyes of the MIPs.  I would favor that.

@JonathanGregory those 3 new experiments are now registered, please take a peek at CMIP6_experiment_id.html and let us know of any issues.
As an aside, your 3 new experiments have pushed the total CMIP6 experiment count to a total of 299,  it's almost a 300-strong experiment behemoth CMIP phase..

Dear Paul @durack1
Thanks for adding them - that looks just right. Would you like us to think of a further experiment to bring it to a round number? If the ocean is the bulk of the monstrous effort of CMIP6, I'd prefer to call it a leviathan.
Best wishes
Jonathan

@JonathanGregory thanks for the offer, however there is no need, there will be string of additional experiments registered before the end of the year I am sure, 300 will be blown past in no time.
Regarding naming, I would assume that the 3D ocean output is going to account for a fairly large proportion of the project storage footprint, Leviathan should be raised at the next CMIP panel meeting as a candidate

@charliepascoe @eguil @bnlawrence @martinjuckes just pinging you all that there are now 3 new FAFMIP experiments which should be updated on ES-DOCs. Also note #805 which will likely add another 6 experiments

Sorry I didn't answer Karl's comment, which I hadn't noticed. Yes, I understand the rule about not adding to Tier 1. I'd like to reassure you that this change was made with the agreement of FAFMIP participants. We did discuss at some length whether it was reasonable to add a requirement, and everyone agreed that it was.

Closing as noted in email earlier this week


Rename IndexOf overloads to IndexOfAny

To reflect what they do more correctly
Resolves https://github.com/dotnet/corefxlab/issues/1352
/cc @ahsonkhan

Looks good.
I will update my PR that is porting these APIs over to System.Memory.dll in corefx.
https://github.com/dotnet/corefx/pull/17357

Now you've asked the question its obvious the methods were wrongly named.
Feel free to close with your update

Yeah. Good rename. Thanks!


results is coming up but i'm unable to move to next page

Challenge Where art thou has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function where(collection, source) {

  var arr = [];
  var keys = Object.keys(source);

  return collection.filter(function(item) {
    return item[keys] === source[keys];
  });

}

where([{ "a": 1 }, { "a": 1 }, { "a": 1, "b": 2 }], { "a": 1 });

@devmurali6 we've been changing function names lately so change where to whereAreYou and you should see some results. Happy coding!


The MD5 in the request is overwritten

Describe the bug
The MD5 in the request is overwritten.
Expected Behavior
The MD5 in the request can be used by aws.
Current Behavior
The MD5 in the request is overwritten.
Reproduction Steps
RUN:
void put_object_buf(Aws::String& bucketName, Aws::String& objectName, S3::S3Client& s3_client)
{
    Aws::S3::Model::PutObjectRequest putObjectRequest;
    putObjectRequest.SetBucket(bucketName);

    std::shared_ptr<Aws::IOStream> bigStream = Create5MbStreamForUploadPart("La");

    putObjectRequest.SetBody(bigStream);
    putObjectRequest.SetContentLength(static_cast<long>(putObjectRequest.GetBody()->tellp()));
    putObjectRequest.SetContentMD5(Aws::Utils::HashingUtils::Base64Encode(Aws::Utils::HashingUtils::CalculateMD5(*putObjectRequest.GetBody())));
    putObjectRequest.SetContentMD5("123");  // still put success
    putObjectRequest.SetKey(objectName);

    Aws::S3::Model::PutObjectOutcome outcome = s3_client.PutObject(putObjectRequest);

    if (!outcome.IsSuccess()) {
        std::cerr << "Error: PutObjectBuffer: " <<
                  outcome.GetError().GetMessage() << std::endl;
    }
    else {
        std::cout << "Success: Object '" << objectName << "' uploaded to bucket '" << bucketName << "'."  And it overwrites MD5 all the time.
Additional Information/Context
No response
AWS CPP SDK version used
1.11.0
Compiler and Version used
8.3.1
Operating System and version
Centos7

@sdavtaker

@jmklix check this pls

@jmklix any progress?

This should be fixed with recent checksum fixes: https://github.com/aws/aws-sdk-cpp/pull/2826
Please let us know if you still see the MD5 being overwritten


Import not working with mongoose on fastify

As per the title, export works just fine as far as I can tell, import however reports that it has been successful, but nothing really happens.
package.json:
  "type": "module",
  "name": "tmp",
  "description": "This project was bootstrapped with Fastify-CLI.",
  "version": "1.0.0",
  "main": "app.js",
  "directories": {
    "test": "test"
  },
  "scripts": {
    "test": "tap \"test/**/*.test.js\"",
    "start": "fastify start -l info app.js",
    "dev": "fastify start -w -l info -P app.js",
    "lint": "standard",
    "lint:fix": "standard --fix"
  },
  "eslintConfig": {
    "extends": "fastify"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "@adminjs/fastify": "^4.0.1",
    "@adminjs/import-export": "^3.0.0",
    "@adminjs/mongoose": "^4.0.0",
    "@aws-sdk/client-s3": "^3.373.0",
    "@fastify/auth": "^4.3.0",
    "@fastify/autoload": "^5.0.0",
    "@fastify/bearer-auth": "^9.0.0",
    "@fastify/cors": "^8.3.0",
    "@fastify/mongodb": "^7.0.0",
    "@fastify/sensible": "^5.0.0",
    "adminjs": "^7.0.9",
    "bcryptjs": "^2.4.3",
    "crypto-random-string": "^5.0.0",
    "fastify": "^4.18.0",
    "fastify-cli": "^5.7.1",
    "fastify-plugin": "^4.5.0",
    "mongoose": "^7.3.1"
  },
  "devDependencies": {
    "@istanbuljs/esm-loader-hook": "0.2.0",
    "husky": "^8.0.3",
    "standard": "^17.1.0",
    "tap": "^16.1.0"
  },
  "tap": {
    "node-arg": [
      "--no-warnings",
      "--experimental-loader",
      "@istanbuljs/esm-loader-hook"
    ]
  },
  "husky": {
    "hooks": {
      "pre-commit": "npm run lint:fix"
    }
  }
}

+1
Looking into the networks tab, it appears that the import call returns an error saying that no file was uploaded.

{
    "record": {
        "params": {},
        "populated": {},
        "baseError": null,
        "errors": {
            "file": {
                "message": "No file uploaded"
            }
        }
    },
    "records": [],
    "notice": {
        "message": "thereWereValidationErrors",
        "type": "error"
    }
}

However in the payload I can see that the file was sent in it -

Any help here is appreciated!

Any news on this? @JonaszJestem @dziraf @AdamFrydrychRST

The silence really is quite disappointing...

Any news on this? @JonaszJestem @dziraf @AdamFrydrychRST


"Bedrock currently does not offer any reranking models." -> Cohere 3.5 Rerank

Validations

[X] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that requests the same enhancement

Problem
The documentation states

Bedrock currently does not offer any reranking models.

Which is not accurate, there is Cohere Rerank 3.5 available in multiple regions.
The documentation therefore isn't clear on how to configure it.
Solution
Update the documentation to specify that at least one model does exist, and specify how to configure it with Bedrock.

When attempting to use the same syntax as for the chat model or the embedding provider, Continue.dev throws Cannot read properties of undefined (reading 'apiBase').

Also: amazon.rerank-v1:0

Hey! Thanks for sharing this, I wasn't aware of this model before. We're going to need to add support for a Bedrock reranker class before we can officially add it to the docs. Is this something you might be interested in contributing?

Would very much like to, but I'm clueless in typescript.
Although we kinda need it for later this month, so I might end up forking and attempting something, but it wouldn't deserve a PR.


Cannot get room topic after upgraded to 1.12.0

await room.topic() get "Unknown Room Topic"

Yes, this is a known bug, we will fix it in the next version as soon as possible.

This problem has been solved in version v1.12.2, please run npm install<EMAIL_ADDRESS>update to the corresponding version


Tensor product

TO CONTRIBUTORS:
Make sure you have:

[x] reviewed and applied the coding style: coding, naming
[ ] for tactics:

[ ] added or adapted documentation in tactics.md
[ ] write an example of use of the new feature in tactics.lean

[x] make sure definitions and lemmas are put in the right files
[x] make sure definitions and lemmas are not redundant

For reviewers: code review check list

Okay I will work on this and also rebase the tensor-product branch.

@kbuzzard  add_mul has explicitly two constants and it gives (at least for me) a nice way to remember at which position it rewrites. right_distrib is a little bit harder. The problem with add_pair for me is that it is not clear what pair should be, f is not an infix operator.

the other hom classes use map_add and such for the position of f. What about calling these add_map and map_add, or map_add_left/right if you prefer?

Just a note, but protected def assoc : (M ⊗ N) ⊗ P ≃ₗ M ⊗ (N ⊗ P) := is a
great example of something that follows from general nonsense. Once limits
are in mathlib (hopefully the next PR after isomorphisms lands?) we can
just show that tensor product of modules gives us has_products on the
category mod R (Kenny has already done the required work in the UMP
section), and then associativity, commutativity, etc can just be
specialisations of
https://github.com/semorrison/lean-category-theory/blob/master/src/category_theory/universal/products.lean
On Mon, Sep 3, 2018 at 8:27 AM Mario Carneiro<EMAIL_ADDRESS>wrote:

the other hom classes use map_add and such for the position of f. What
about calling these add_map and map_add, or map_add_left/right if you
prefer?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/leanprover/mathlib/pull/303#issuecomment-417964168,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdLBI4KpmphYkTZsuv3erLi7Va0pam9ks5uXFtAgaJpZM4WWlyv
.


C# Project running in the same folder as "UnityEngine.dll" but not in a unity based runtime causes a runtime failure

What version of gRPC and what language are you using?
C# Grpc 2.30.0
What operating system (Linux, Windows,...) and version?
Windows 10 Pro OS build 19041.330
What runtime / compiler are you using (e.g. python version or version of gcc)
dotnet framework 4.6.2
What did you do?
Running a program that contains GRPC in a folder that contains UnityEngine.dll.
What did you expect to see?
GRPC to run as normal.
What did you see instead?
The type constructor for Grpc.Core.Internal.PlatformApis crashed with .
System.TypeInitializationException
  HResult=0x80131534
  Message=The type initializer for 'Grpc.Core.Internal.PlatformApis' threw an exception.
  Source=Grpc.Core
  StackTrace:
   at Grpc.Core.Internal.PlatformApis.get_IsXamarinIOS()
   at Grpc.Core.GrpcEnvironment.get_IsNativeShutdownAllowed()
   at Grpc.Core.GrpcEnvironment.GrpcNativeInit()
   at Grpc.Core.GrpcEnvironment..ctor()
   at Grpc.Core.GrpcEnvironment.AddRef()
   at Grpc.Core.Server..ctor(IEnumerable`1 options)
   at NetStandardLib.Program.Main() in E:\Dev\TestLib\Program.cs:line 19

Inner Exception 1:
TargetInvocationException: Exception has been thrown by the target of an invocation.
  StackTrace:
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)
   at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at System.Reflection.RuntimePropertyInfo.GetValue(Object obj, Object[] index)
   at Grpc.Core.Internal.PlatformApis..cctor()

Inner Exception 2:
SecurityException: ECall methods must be packaged into a system module.
  StackTrace:
   at UnityEngine.Application.get_platform()

Anything else we should know about your project / environment?
This can be recreated by copying the UnityEngine.dll file from Unity next to a C# dotnet framework application, at which point running the application causes the failure. I did not test if adding the dll to any point in the assembly resolution paths would also cause the failure.
Here is a project, that when run demonstrates this failure:
GrpcUnityFailure.zip
It looks like the issue is that Type.GetType(UnityEngineApplicationClassName); will load the assembly if it's not already loaded, and then then invokes a unity-environment specific call with:
var platformProperty = unityApplicationClass.GetTypeInfo().GetProperty("platform");
var unityRuntimePlatform = platformProperty?.GetValue(null)?.ToString();

This call (UnityEngine.Application.platform) explodes because the program is not running in a unity context.
I can try to make a fix for this, but I figured y'all may have a preferred way it should be fixed, as it relates to security things and signals related to runtimes that I have not had experience with so far. If it's open season on this kind of bug though, I'll pick it up.
Context for where the bug was discovered

I think this is basically a duplicate of https://github.com/grpc/grpc/issues/18801? We need a better check for detecting if we are running on unity or not.


footer styling

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
@mansiruhil13
assign links tothefooter
Use Case
footer
Benefits
wante do bring footer in use
Priority
High
Record

[x] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[x] I want to work on this issue
[x] I'm willing to provide further clarification or assistance if needed.

could you plz assign me this and put label of gssoc


no applicable method for 'augment' applied to an object of class "fitdists"

library(ssdtools)
fits <- ssd_fit_dists(ssdtools::boron_data)
augment(fits)
#> Error in UseMethod("augment"): no applicable method for 'augment' applied to an object of class "fitdists"

Created on 2021-07-29 by the reprex package (v2.0.0)

Just hadn't exported (face palm)


LR finder for regression problems

Is this code useable on regression problems, or only classification problems? I've been trying to get it to work for sometime with no success.

Should work. What exactly were you not able to get working?

Here's the error:
Traceback (most recent call last):
File "/Users/danielstoops/Desktop/Work/ML/projects/aerofoils/run_CNN.py", line 149, in
lr_finder.range_test(train_loader, end_lr=100, num_iter=100)
File "/usr/local/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch_lr_finder/lr_finder.py", line 189, in range_test
loss = self._train_batch(iter_wrapper, accumulation_steps)
File "/usr/local/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch_lr_finder/lr_finder.py", line 242, in _train_batch
loss = self.criterion(outputs, labels)
File "/usr/local/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/usr/local/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/modules/loss.py", line 782, in forward
return F.smooth_l1_loss(input, target, reduction=self.reduction)
File "/usr/local/anaconda3/envs/pytorch/lib/python3.7/site-packages/torch/nn/functional.py", line 2153, in smooth_l1_loss
if not (target.size() == input.size()):
AttributeError: 'tuple' object has no attribute 'size'
And my Code
`criterion_ClCd = nn.SmoothL1Loss()
criterion_angle = nn.SmoothL1Loss()
optimiser = torch.optim.SGD(model.parameters(), lr=learning_rate)
scheduler = torch.optim.lr_scheduler.ReduceLROnPlateau(optimiser, 'min', patience=5)
if find_LR:
lr_finder = LRFinder(model, optimiser, criterion_ClCd, device=device)
lr_finder.range_test(train_loader, end_lr=100, num_iter=100)
lr_finder.plot()  # to inspect the loss-learning rate graph
lr_finder.reset()  # to reset the model and optimizer to their initial state`
I assume I'm getting this error because my the output from my dataset class is a tuple:
class AerofoilDataset(Dataset):
...
def __getitem__(self, item):
...
return self.x[item].view(self.num_channels, self.input_size), self.y[item], self.aerofoil[item]

Hi, I've been working on this for a few hours now and I figured out that I my forward method was outputting a tuple and not a single tensor. Now your learning rate finder works very well.
Thank you for your very impressive work and for your speedy response to my query. I'm just starting out in deep learning, hopefully I'll be able to get to your level someday


Support for TypeScript npm modules without declaractions

Enhancement proposal:
I have simple npm module which contains "only" index.js (in commonjs) and index.ts - its source.
But its import cannot be found and typechecked by current atom-typescript because it requires index.d.ts and also definition of that in package.json. First it could be more clever and just change extension from main index.js. but if it cannot find index.d.ts just live with index.ts...
Another option would be to make moduleResolution customizable - like ts.CompilerHost can do.
It would probably need crazy escaping of javascript in tsconfig.json, but ultimately it would allow everything.
till now I used nasty import * as xxx from "node_modules/xxx/index"; but now it should be possible much nicer import * as xxx from "xxx";

But its import cannot be found and typechecked by current atom-typescript because it requires index.d.ts and also definition of that in package.json

Just add declaration:true to the tsconfig to get a index.d.ts and it should just work.

till now I used nasty import * as xxx from "node_modules/xxx/index"; but now it should be possible much nicer import * as xxx from "xxx";

Should just work. Also you can use a typings field to point to an index field. Checkout a package that I just pushed : https://www.npmjs.com/package/onresize for an example

Closing as it already work ... unless there is a bug ... in which case feel free to comment :rose:

Yes, it is true that it works with declaration:true, but it means to have much more files which have basically duplicate information and when you have a lot of modules its add up, so it would be nice to support this also without d.ts files. Think of "Pure" TypeScript modules (just *.ts, no *.js, no *.d.ts).
Second part with customizable module resolver would allow as support both ways in mean time until people will republish all modules with new way.

Yes, it is true that it works with declaration:true,

Thaks for the verification. This is new stuff so always good to get more :eyes: :rose:

Think of "Pure" TypeScript modules (just *.ts, no *.js, no *.d.ts)

The issue with that is that a .ts file becomes a part of a compile context. and needs to be compiled ... which slow. The choice to only use .d.ts was made by the TypeScript team btw.
Reference : https://github.com/Microsoft/TypeScript/blob/971c7770314467964c536242becad2555ef9c758/src/compiler/program.ts#L70-L73

Second part with customizable module resolver would allow as support both ways in mean time until people will republish all modules with new way.

Definitely. But better if done upstream (microsoft/typescript)

You are absolutely right that it should be solved upstream. I just tried you as very responsive person, thanks for all your work.

Yeah, the main issue is a compiler refactoring to support internal vs. external .ts files. For all purposes .d.ts are already considered external so it was easy to jank into the current compiler code to support node_module .d.ts lookup :wink: :rose:

@Bobris @basarat
this is related https://github.com/TypeStrong/atom-typescript/issues/562#issuecomment-147620723


Azure Master and Azure Slave devices have different frame numbers.

Describe the bug
I connected 2 Azure kinects according to the documentation.  After several recording I have different number of frames in master and slave devices.
How to allign frames with close-to-perfect synchronisation?
For example master recording has 1501frames and Slave recording has 1497 frames.
If you rerun the experiment, these numbers are different.
Mostly the difference between master and slave is 1 or 2 frames.
This issue was tested with 2 different kinects.
This issue was tested with --rate 5, 15, 30
To Reproduce
Connect master sync out to slave sync in.
Start slave recording,
Start master recording.
Compare frame numbers.
Expected behavior
Best: Frame count in master and slave are equal and kinects are perfectly synchronized.
Acceptable: Frame count difference is stable. So we can allign master frame X to slave frame Y for every recording.
Unkown : I don't know how system will behave if we have more than 2 kinects.
Questions
Is there any way to synchronise kinect azure clock so we know when the frame was captured exactly. Is there any code sample how to allign kinect frames according to the system time, or how to get this system time in C/C++ interface
Important
This bug is related to the frame difference that happens in the beginning. As it was mentioned. Kinects don't have a stable behaviour in the first few seconds.
Desktop:

OS with Version: Windows 10.
Firmware version : AzureKinectDK_Fw_1.6.108079014
SDK 1.4.0

The device timestamp is synchronized acrossed all devices. Don't use frame counts.
Look at get_synchronized_captures in our green screen example for how to do it with a live stream.
For MKV's the device timestamps always start at zero and start_timestamp_offset_usec from k4a_record_configuration_t needs to be applied to get the recorded device timestamp. start_timestamp_offset_usec is effective an offset removed from every sample to they streams can start at zero. From there you can compare to the device timestamp from the other stream to create a synchronized pair.

Ok. I am working on it right now.

Here is my setup description when one kinect is a master :

External camera starts recordning when it recieves a signal.
Here are the screenshots from anoher recording with external camera.

You can see the difference between RGB and IR image.
As we can see from screenshots: delay for both cameras is around 2 timestamp units

After I connected both kinect devices to external signal generator I get the following result

For one kinect it is still 22 timestamp units,
for another one it is 11 timestamp units.

Here are extra screenshots from Azure Kinect Viewer. (When running in slave configuration)
33 timestamp units difference

45 timestamp units

These numbers are not stable and have shift between 2 kinects.

Our device timestamps are in microseconds. We don't expect them to be exactly the same as they are from two different clocks on two different devices. We do expect them to be within 100us of each other, which from what I can tell is what you do see.

But why doesn't the difference between RGB and IR frame stays stable?

Can you be more specific about what you consider unstable? I think you are talking about 33us or 45us of difference between color and depth images.  This is real hardware with real clocks, and a calculation to find center of exposure.  You can't expect a perfect timestamp match.


refactor: use Deno.Command instead of Deno.run #278

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


Roles are not getting listed for users from Active Directory User Store

Description:
When an AD secondary user store is configured, users cannot list their roles from the carbon management console.
If we check the roles and list their assigned users are getting listed.
Steps to reproduce:

Create an Active Directory user store with users assigned with some roles.
Configure the above as a secondary user store.
Login to the carbon management console and try to list the roles for a specific user.

Affected Product Version:
IS-KM-5.9.0
Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

This will be addressed in the https://github.com/wso2/product-is/issues/11691


very big changes

About

Description of changes

Pre-Merge checklist

[ ] The project was correctly built with npm run build.
[ ] If there was any conflict, it was solved correctly.
[ ] All changes were documented in CHANGELOG.md.
[ ] All tests on CircleCI have passed.
[ ] All Percy visual changes have been approved.
[ ] Two people have :dancer:'d the pull request. You can be one of these people if you are a Dash core contributor.

Reference Issues

Closes #[issue number]
Other comments

sorry this is just a test please ignore. i'm trying to figure out how
circleci works
On Tue, Apr 6, 2021 at 5:48 PM Sebastien Jourdain @.***>
wrote:

@.**** approved this pull request.
? is it just a formatting issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/plotly/dash-vtk/pull/39#pullrequestreview-629436926,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFBTAWN5ZEU55GVYCIYG43TTHN6TLANCNFSM42PREX4A
.


useNativeDriver whenever possible

Configure Animted to useNativeDriver whenever possible to:

Improve animation performance
Remove useNativeDriver not set warnings

I think this causes conflicts with the ability to stack because we are using useNative everywhere. @eyalyoli thoughts?

I think this causes conflicts with the ability to stack because we are using useNative everywhere. @eyalyoli thoughts?

I'll check the stack option. I moved this to a newer version of the project check - https://github.com/swaplet/react-native-swipe-cards-deck

I'll check the stack option. I moved this to a newer version of the project check - https://github.com/swaplet/react-native-swipe-cards-deck

@RookTKO yes it is a problem, the fix is simply to replace top with translateY and left with translateX in lines 387, 391 when defining style at renderStack(). I'll do so in react-native-swipe-cards-deck.
Although it seems to me that stack mode had issues in the original meteor-factory/react-native-tinder-swipe-cards lib. Maybe you can help to fix these issues...

@eyalyoli yea I'll working on trying to resolve them. Seems that they work on ios but not android atm.


Cannot download BarcodeScannerView higher than 1.3.0

Hi,
I added your library with the usual jitpack in root build.gradle
maven { url "https://jitpack.io" }
and this line in app build.gradle:
implementation 'com.github.markusfisch:BarcodeScannerView:$version'
However I always failed if I put any version of 1.4.0, 1.4.1, and 1.5.0 as listed in jitpack.io website. I only succeeded after I set 1.3.0 version. Thanks.

Ah, sorry, this was fixed with the latest commit but I forgot to set a tag 🙈I'm away for a couple of days and will only be able to fix next week, unfortunately. In the meantime you could clone the repo and build the aar manually (just run "make aar").Sorry, I will fix it as soon as possible.

Version 1.5.1 is now available in JitPack.


API key fail to generate

When trying to generate a new api key, I get the following error in alertad :
[Tue Apr 21 19:02:19 2015] [error] 2015-04-21 19:02:19,883 - alerta.app.cors[24193]: INFO - Request from Origin:http://alerta.emv2.com, setting Access-Control-Allow-Origin:http://alerta.emv2.com
[Tue Apr 21 19:02:19 2015] [error] 2015-04-21 19:02:19,890 - alerta.app[24193]: ERROR - Exception on /key [POST]
[Tue Apr 21 19:02:19 2015] [error] Traceback (most recent call last):
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/flask/app.py", line 1817, in wsgi_app
[Tue Apr 21 19:02:19 2015] [error]     response = self.full_dispatch_request()
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/flask/app.py", line 1477, in full_dispatch_request
[Tue Apr 21 19:02:19 2015] [error]     rv = self.handle_user_exception(e)
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/flask_cors/extension.py", line 110, in wrapped_function
[Tue Apr 21 19:02:19 2015] [error]     return cors_after_request(app.make_response(f(*args, **kwargs)))
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/flask/app.py", line 1381, in handle_user_exception
[Tue Apr 21 19:02:19 2015] [error]     reraise(exc_type, exc_value, tb)
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/flask/app.py", line 1475, in full_dispatch_request
[Tue Apr 21 19:02:19 2015] [error]     rv = self.dispatch_request()
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/flask/app.py", line 1461, in dispatch_request
[Tue Apr 21 19:02:19 2015] [error]     return self.view_functions[rule.endpoint](**req.view_args)
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/flask_cors/decorator.py", line 120, in wrapped_function
[Tue Apr 21 19:02:19 2015] [error]     resp = make_response(f(*args, **kwargs))
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/alerta/app/auth.py", line 94, in decorated
[Tue Apr 21 19:02:19 2015] [error]     payload = parse_token(token)
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/alerta/app/auth.py", line 53, in parse_token
[Tue Apr 21 19:02:19 2015] [error]     return jwt.decode(token, key=app.config['SECRET_KEY'], audience=app.config['OAUTH2_CLIENT_ID'])
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/jwt/api.py", line 137, in decode
[Tue Apr 21 19:02:19 2015] [error]     self._validate_claims(payload, options=merged_options, **kwargs)
[Tue Apr 21 19:02:19 2015] [error]   File "/opt/alerta/lib/python2.6/site-packages/jwt/api.py", line 216, in _validate_claims
[Tue Apr 21 19:02:19 2015] [error]     raise InvalidIssuedAtError('Issued At claim (iat) cannot be in the future.')
[Tue Apr 21 19:02:19 2015] [error] InvalidIssuedAtError: Issued At claim (iat) cannot be in the future.
[Tue Apr 21 19:02:19 2015] [error] 2015-04-21 19:02:19,891 - alerta.app.cors[24193]: INFO - Request from Origin:http://alerta.emv2.com, setting Access-Control-Allow-Origin:http://alerta.emv2.com

I had to remove the 2 following lines to get it work in /opt/alerta/lib/python2.6/site-packages/jwt/api.py
if iat > (now + leeway):
raise InvalidIssuedAtError('Issued At claim (iat) cannot be in the future.')
But that's just a trick

Is the server you are running alerta on time synced? Definitely check that ntpd is running and that date doesn't return an time in the future.

Both were running on the same server....
can it be an issue if my server time is not as my client computer time ?

The server where alertad is running and the client where you have your web browser need to be time synced. They don't need to be the same time zone, though. We have our Alerta servers configured to use UTC time and people log in from their laptops set to local time (in the UK and Australia) and it all works fine.


Add "timezones" config entry

For some reason, ETS doesn't use true or false for this one 🙄 (0 and 1 instead)

For some reason, ETS doesn't use true or false for this one 🙄 (0 and 1 instead)

Yeah they really gotta figure out their config options. Some names are hella confusing 😕


Checkbox on Array name

PHP need checkbox name in array when submit multiple ckecker

when I give name not in array checkbox have correct behav..
but.. when I give array on name, checkbox not give a correct result

note: on safari and chrome is fine, only on firefox

Can you include the outputted html in a jsfiddle so i can debug?


Branch name escape problem

Description
Related: #25830

https://github.com/go-gitea/gitea/pull/25812#discussion_r1262140690
This is the right link:
https://try.gitea.io/yp05327/testrepo/src/branch/test%2311

https://github.com/go-gitea/gitea/issues/25830#issuecomment-1634050628

Gitea Version
latest
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
build
Database
None

Thanks for summarizing it in short.
You are correct about Github.

I don't get it. What is wrong on https://try.gitea.io/yp05327/testrepo/src/branch/test%2311?

Thats the correct url for test#11 branch. Nothing wrong with it.

I don't get it. What is wrong on https://try.gitea.io/yp05327/testrepo/src/branch/test%2311?

After access this link, you can select this branch again in the selector.
Then the url will become https://try.gitea.io/yp05327/testrepo/src/branch/test#11


Failed to register data using the new system - casting to Comparable?

I get this stacktrace when trying to register data,
https://pastebin.com/rVT03NX7
Code being used:
        DataRegistration<LastPowerData, ImmutableLastPowerData> lastPowerData =
                DataRegistration.<LastPowerData, ImmutableLastPowerData>builder()
                        .dataClass(LastPowerData.class)
                        .immutableClass(ImmutableLastPowerData.class)
                        .builder(new LastPowerDataManipulatorBuilder())
                        .manipulatorId("last_power")
                        .dataName("LastPower")
                        .buildAndRegister(CraftBookPlugin.spongeInst().container);

        Sponge.getDataManager().registerLegacyManipulatorIds("com.sk89q.craftbook.sponge.util.data.mutable.LastPowerData", lastPowerData);

Those are the changes I made to make my custom data work:
https://github.com/SpongePowered/SpongeCommon/tree/fix/newcustomdata

Hmm, okay - so the fix is waiting on that to merge?

well I dont know what @gabizou intended to do
also ItemStack#toContainer is still broken when you have a Sponge DataManipulator like DisplayName


iframe portlet sandbox

Describe your use-case which is not covered by existing documentation.
I want to disable the sandboxing or at least allow same-origin javascript in the iframe portlet. I noticed the code had variables DO_NOT_USE_SANDBOX and SANDBOX_VALUE which are later referenced in the jelly file
I tried to set them on the jenkins script console but it still did not remove or modify the sandbox
System.setProperty("hudson.plugins.view.dashboard.core.IframePortlet.doNotUseSandbox", "true"))
System.setProperty("hudson.plugins.view.dashboard.core.IframePortlet.sandboxAttributeValue","allow-scripts"))
how could I use this functionality to allow javascript to run from the same host?
Reference any relevant documentation, other materials or issues/pull requests that can be used for inspiration.
No response

Hmm, that's strange. Those properties were introduced when fixing SECURITY-2559, but I admit I never tested if they work as intended...
Let my verify this...

Okay, the properties work as intended, BUT since they are stored in static variables, they need to be set early enough (before the first IframePortlet is loaded) to take effect.
I couldn't apply the property from the Script Console at all, so you probably have to apply it from the command line (see https://www.jenkins.io/doc/book/managing/system-properties/#usage)


fix(content-docs): render category with no subitems as a normal link

Motivation
Fix #6301. I think this is the most sensible fix?
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Added a dogfooding example

LGTM thanks 👍


Drop outdated comment in expand<Box1, Box2>

It is templated to support different types of boxes, not to avoid adding a volatile overload like in its original form.

LTGM. Could have used [ci skip].

Sorry I forgot

Ignoring the tests.


Display of instructions based on the mode

Current

Future

We will just put a question mark icon to link to About page


Numpy like slicing on Tensors

I was wondering if it is possible to implement Numpy like slicing annd updating a[1:10,2:20....]  in Tensorflow. It would make life much easier. Right now it code just gets bigger and uglier and bug prone.

I would like to take up this task, if at all it's possible. Need your comments @mihaimaruseac @fchollet

I'm not familiar with what that would entail, but it looks like a good project.

@phquanta Could you provide more details on the feature? May be take a simple example code and demonstrate what was missing with the current TF and what would be achieved with this new feature. Thanks!

Sure , basically instead of writing down :

aa=tf.slice(hIn,[p,j,0],[1,1,self.n_h])[0][0], One would write aa=hIn[p,j,:]
tf.scatter_update(self.vOut,i,[[1. for i in range(self.n_v)] for j in kk])], One would write self.vOut[i,:,:]=[[1. for i in range(self.n_v)] for j in kk])]

Those type of operations.

Most numpy slicing is already supported in tensorflow; tensor[p, q, :] for example is perfectly supported.
What isn't currently supported is slice assignment. TensorFlow tensors are immutable, so we'd need an operation which takes a tensor and returns another one. It should be relatively straightforward to repurpose the existing kernel for StridedSlice to do this, similar to how TensorScatterAdd etc repurposed the ScatterNdAdd kernel.
I'm happy to provide pointers to the C++ code if anyone is interested in contributing this.

If i'm not mistaken, that type is supported but not tensor[p, :, q] or tensor[:, :, q] or tensor[:, p,q] or other more fancier slices.
Also, i was wondering if it is possible to make tensors mutable at some point ?
Thanks.

We'll not make tensors mutable (as this will make compilation substantially
harder) but we can do a better job at hiding the immutability of tensors
after tf 2.0 goes out.
If you can enumerate specific slicing formats which currently do not work
we can try to add them to the strided slice operation.
On Tue, Mar 19, 2019 at 6:52 PM Andrei Buin<EMAIL_ADDRESS>wrote:

If i'm not mistaken, that type is supported but not tensor[p, :, q] or
tensor[:, :, q] or tensor[:, p,q] or other more fancier slices.
Also, i was wondering if it is possible to make tensors mutable at some
point ?
Thanks.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/26839#issuecomment-474651731,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAATxTG66CgsMvM3tXqbhfq0JVBSYyORks5vYZRfgaJpZM4b6Nv9
.

--

Alex

@alextp, I'm interested in contributing this, and hope you could give me some reference. Thanks.

@a6802739 happy to help!
The current logic for numpy-style slicing in tensorflow is spread out over two functions: boolean_mask and strided_slice.
tf.boolean_mask is implemented in python in terms of gather here: https://github.com/tensorflow/tensorflow/blob/49414867df30277c7ea7839837cc8e3397c712cb/tensorflow/python/ops/array_ops.py#L1275
when slicing we first build a spec here for the strided_slice op https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/array_ops.py#L526 and then we call the strided_slice op (or boolean_mask), which is implemented in C in https://github.com/tensorflow/tensorflow/blob/master/tensorflow/core/kernels/slice_op.h and the corresponding .cc files in the same directory.
Depending on what extra types of slicing you want to add support for we can do this in python by reducing to the existing ops or by adding a new op for that type of slicing and plumbing it through tf.
I can help you more if you tell me which other features of numpy slicing are important to you.

@alextp , Thanks for your reference.
I have tried the slice operation, I found it could support operation like tensor[p, q, :], tensor[p, :, q], tensor[:, p, q], etc
But it could not support fetch multi elements one time, just like in numpy, we could done it like this:
import numpy as np

arr = np.array([
    [1, 2, 3, 4],
    [2, 4, 6, 8],
    [3, 6, 9, 12],
    [4, 8, 12, 16]
])

print arr[[0, 2], [3, 1]]

So I will first add this operation, how about that?

That sounds good. Fetching multiple elements like that sounds like
something which we can support with tf.gather_nd. So I'd try implementing
this by adding code to _slice_helper to detect that the slice expression
looks like this and then forward it to gather_nd in that case.
On Wed, Mar 27, 2019 at 3:30 AM songhao<EMAIL_ADDRESS>wrote:

@alextp https://github.com/alextp , Thanks for your reference.
I have tried the slice operation, I found it could support operation like tensor[p,
q, :], tensor[p, :, q], tensor[:, p, q], etc
But it could not support fetch multi elements one time, just like in numpy,
we could done it like this:
import numpy as np
arr = np.array([
[1, 2, 3, 4],
[2, 4, 6, 8],
[3, 6, 9, 12],
[4, 8, 12, 16]
])
print arr[[0, 2], [3, 1]]
So I will first add this operation, how about that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/26839#issuecomment-477083309,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAATxWfM_Xc00CboIRIhBDrgEk42imBKks5va0hZgaJpZM4b6Nv9
.

--

Alex

@alextp, Thanks. I have come up with the PR #27238, please take a loot at that.
And would us have any plan to support this kind of operation tensor[p, q, j] = xxx?

We cannot support mutating assignments because tf tensors are immutable. Not at least until a future release where we can hide the mutation somewhat better.

Thanks! @alextp

In order to be compatible with numpy array operation, I list the items that we could consider:
for example for bumpy array arr = np.array([[1,2,3], [4,5,6], [7,8,9]]).

[ ] arr[[0, 1], [1, 0]]

[ ]  arr[[0], 1]

[ ] arr[[0, 1], 0]

[ ] arr[np,array(1), 0]

@a6802739 can you find an exhaustive list of the operations supported by numpy slicing somewhere?

@alextp, I'll have a try.

@alextp  From
https://github.com/tensorflow/tensorflow/blob/b43029085bfa48caaef718c9d44abdc116efc600/tensorflow/python/ops/array_ops.py#L653
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/array_ops.py#L653

yes we should fix this inconsistent behavior

@alextp, Okay, I could have have a try for the inconsistent behavior.

Hello,
I want to do specific slicing on the tensor, when searching to conduct the slicing I came across with this issue and I was wondering if it is doable at all?
This is my detail explanations https://stackoverflow.com/questions/56532577/filter-tensorflow-array-with-specific-condition-over-numpy-array
Thank you!

Any idea guys how can I accomplish this kind of slicing? @a6802739 @alextp

Hi guys,
after reviewing tf.gather, tf.gather_nd and advanced indexing using tf.stack and then tf.gather_nd, I got to know that the advanced indexing like this numpy nparray[ arr_rows, arr_cols] is not supported in tensorflow.
any plan to add this feature?
the reproducible code you can find here:
https://stackoverflow.com/questions/56636994/non-continuous-index-slicing-on-tensor-object-in-tensorflow-advanced-indexing-l
a simple version is being supported, but for example in case the number of columns is different from the source tensor is not been supported.
We can do it easily in numpy but tensorflow no!

Hi,  @a6802739  It has been a while, any progress on it ?
I think I am able to take over it if you don't mind.

Hi. Urgent help needed in doing the below operation. Mask for input is given, its shape is [batch size, no of timesteps]. From this, I need to collect X number of embeddings of shape [batch size, timestep index, embedding size] such that they are grouped just before each grouping of False.
say mask is T,T,T,F,F,F,F,|T,T,F,F,F,F,F and X=2  (by '|', I assumed a break which indicates a row split length=7), then should get list of concatenated embeddings given by indices (1,2) (8, 9).
Any help is appreciated.

@phquanta,
Could you refer Introduction to tensor slicing guide, may help you. Thanks!

Are you satisfied with the resolution of your issue?
Yes
No


docs(v2): showcase BoxyHQ

Motivation
Adding BoxyHQ website built w/docusaurus v2 into the showcase page.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
N/A
Related PRs
N/A

thanks 👍

Thanks @slorber


Merge requests do not show up correctly in commit message

In raw format, what the original repo commit looks like:
'e: a526bf9 ff65c57Merge pull request #4 from cewbdex/patch-1Enhance Fedora dependency part'
Need to find a way for it to show up as:
[Merge pull request](https://github.com/cryinkfly/SOLIDWORKS-for-Linux/commit/1994b42dce04897695974a41accd7b7a5a3561a6) https://github.com/cryinkfly/SOLIDWORKS-for-Linux/pull/4 [from cewbdex/patch-1](https://github.com/cryinkfly/SOLIDWORKS-for-Linux/commit/1994b42dce04897695974a41accd7b7a5a3561a6)
Probably want to use regex to detect in commit message if it was a merge, then go accordingly.

Found that commit 2d0a224bdbdcc759c968f000ebf68363de380ff1 merge message "Merge branch 'main' of https://github.com/chirmstream/CloneLab" did not get passed correctly.  It is the 24th (I think) commit of this repo.  It was a slightly different merge than the others so that must be why the regex didn't catch it.

Corrected https://github.com/chirmstream/CloneLab/commit/2d0a224bdbdcc759c968f000ebf68363de380ff1
But Commit 5b65f8b60a91d7836f739f48bd2edc296fc55778 is not showing up correctly.  It is roughly commit # 169.

Correct as of commit https://github.com/chirmstream/CloneLab/commit/2e6102fb201e3586595caacb420f288c81d6021e


Inline Completion Provider Sample Needs Update

/cc @hediet ⬆
Originally posted by @jrieken in https://github.com/microsoft/vscode/issues/131165#issuecomment-978939021
There's a bunch of things that seem to need to change in the samples that reference the new API:

Documentation
package.json - how to modify it to add the proposed API
post install script - needs to be updated to download the correct file.

thanks!

From @CodeCasterNL:
documentation like Inline Completions Sample still links to the old file locations (https://github.com/microsoft/vscode/commit/a4d426a1c2ece4222f1343459d31fe948098b2f6).

Perhaps a relevant search: https://github.com/search?q=org%3Amicrosoft+vscode.d.ts&type=code

Hi


Add support to different website

Can we add a feature to set up a different Danbooru site as the target?

It is possible to change current danbooru url in extension settings.
Or do you mean adding extra context menu options (e.g. upload to aibooru in addition to danbooru)?

Or do you mean adding extra context menu options (e.g. upload to aibooru in addition to danbooru)?

Yeah, I mean a feature like this one.


WebServiceTarget support for JSON & Injecting JSON serializer into NLog

Added support for POST of JSON and XML documents and added respective tests. I tried to keep it simple, comprehensible and not to break with existing functionality.
I'd have to change existing tests in WebServiceTargetTests to extend their usability to the new types but I wasn't sure about altering existing tests of other authors, so I added new parts.
Have a look and tell me what you think about the direction it's heading.

This change is

fixes #700

Cool!
Could you undo the moved methods? It would be easier to review and otherwise we have a hard time merging this also in the .NET Core branch. Thanks!

Oh I see, that auto sorting and formatting of VS has created a diff-hell.
I think I'll deal with this tomorrow, just adding the new stuff at the bottom and try to prevent VS from exchanging TAB<=>spaces
Good night.

How do you want it to be 'injected' ?
For XmlSerializer too?

For XmlSerializer too?

Maybe, but that's already in non system.web libraries so less a problem IMO

So JSONSerializer injecting by defining an interface and adding a property to the target?

So JSONSerializer injecting by defining an interface and adding a property to the target?

Yes, and it would be nice if the property is at higher level than this target. I thinkConfigurationItemFactory or a new class.

good night1

So, tried a completed reformatting as good as possible.
Is it feasible this way?
I would then go and deal with JsonFormatter injection.

 Last update 35259db...d7325e2

Two possible ways spontaneously cross my mind:

Just add an public IJsonSerializer' JsonSerializer {get; set;} to ConfigurationItemFactory which in turn sets an property to every WebServiceTarget instance it creates
introduce a IServiceResolver interface and an InjectedAttribute that marks properties of objects created by ConfigurationItemFactory as injectable and so is done, if that resolver 'knows' the type of that property

The first option would be more simple and quickly done, the second option indeed slightly more complex but more extendible and may be used for further scenarios. On the other hand, if no one can think of any additional use cases, that would be a perfect example of overkill.

Interface so far:
public interface IJsonSerializer
{
	void Serialize(TextWriter writer, object value);
	void Serialize(TextWriter writer, object value, Type objectType);
	object Deserialize(TextReader reader, Type objectType);
	T Deserialize<T>(TextReader reader);
}

I prefer the first one. More in line with the current options in NLog. :)

After reflecting about it...
Which 'object' shall this serializer convert to JSON, actually? Because by now WebServiceTarget writes parameters that are named in config, one by one. There is no complex object to serialize.

Well then the simple json serializer will work.
Except that is an issue when there is a quote in the current value AFAIK.

Yup, now as you say it, forgot to escape strings.

I'm on it by tomorrow.

So, since the named parameters in config are just objects, I decided to add a slim serilizer interface, namely ICompactJsonSerializer, containing only a single method to serialize a value. In addition there is a default implementation, doing very rudimentary serialization and string escaping.
Tests for serializer are included.
Extended ConfigurationItemFactory so, that it provides a virtual method InterceptCreation, which is invoked right after a new object has been created via the instantiator in CreateInstance.
The base implementation of InterceptCreation realizes the injection of the JSON serializer into WebServiceTarget, if one was set by ConfigurationItemFactory.SetJsonSerializer()
Let me know, what you think... uhm, and I just see, I have to deal with Travis complaints :-/

Don't understand this: AppVeyor is complaining about things, that aren't contained in that files ?!?

It's he SL build, but I see you already seen it :) 👍

yeah, several fixes later... yeehah, there we go :)

Reviewed 2 of 3 files at r3, 1 of 8 files at r4, 12 of 21 files at r5, 7 of 11 files at r6, 2 of 2 files at r7.
Review status: 24 of 25 files reviewed at latest revision, 5 unresolved discussions.

src/NLog/Targets/DefaultJsonSerializer.cs, line 144 at r7 (raw file):

        }

        private static string escapeString(string str)

please proper camels here ;)
(CamelCase)

src/NLog/Targets/ICompactJsonSerializer.cs, line 42 at r7 (raw file):

    /// Useful for wrappers for existing serializers.
    /// </summary>
    public interface ICompactJsonSerializer

I would prefer IJsonSerializer
Dunno why it should be "compact". Maybe I would write an implementation which won't "compact".
Let me know what you think!

src/NLog/Targets/WebServiceProtocol.cs, line 64 at r7 (raw file):

        /// Do an HTTP POST of a JSON document.
        /// </summary>
        Json,

Maybe JsonPost and JsonXml?

src/NLog/Targets/WebServiceTarget.cs, line 192 at r7 (raw file):

            get
            {
                if (_jsonSerializer == null) _jsonSerializer =  DefaultJsonSerializer.Instance;

I think we should take there the ConfigurationItemFactory.JsonSerializer?

tests/NLog.UnitTests/Targets/DefaultJsonSerializerTests.cs, line 42 at r7 (raw file):

#endif
    using System;
    public class DefaultJsonSerializerTests : NLogTestBase

thanks! Looks great!

Comments from Reviewable

Thanks for the contributions! I have checked the code!
I think it will work, but I won't prefer the "interception". I think we should read the ConfigurationItemFactory.JsonSerializer inside the WebserviceTarget and then we don't need the "interception"?
Also check your camel-casing ;)

Ok, to do: CamelCasing (sorry, lowerCamel for private members is a habit of mine),
rename WebProtocol (JsonPost, XmlPost).
I used that interception way to make it reusable and to decouple WebServiceTarget from ConfigurationItemFactory, because it don't necessarily have to know about it. But I will switch that.
Compact JSON, because the WebServiceTarget produces a compact JSON and I wanted to prevent indented serialized values inside an actual single line document. I choosed compact JSON since it's quite common to use it for web transfers. But we may also switch that, as you want.

So, I took care for everything except the compact JSON format, meaning a single line document without indentation.
What may I do against the conflict of ConfigurationItemFactory?

appveyor is waiting for a while now.

close/open to wake up AppVeyor ;)

Could you explain what a "compact json" is?

Compact JSON just means no tabs, no spaces, and no line breaks.
But I read the RFCs and the ECMA again, maybe it just doesn't matter, even if you mix tabbed and single line JSON, so we could drop the comment and change the name of the interface.

I hope it now somewhat fits your idea of how it should be :-)
Let me know if I can do something more about it ;-)

Reviewed 2 of 5 files at r8, 6 of 7 files at r9, 17 of 17 files at r10.
Review status: all files reviewed at latest revision, 6 unresolved discussions.

src/NLog/Targets/IJsonSerializer.cs, line 50 at r10 (raw file):

        /// <param name="value">The object to serialize to JSON.</param>
        /// <returns>Serialized value.</returns>
        string SerializeValue(object value);

I was thinking of SerializeObject. What do you think?

Comments from Reviewable

Looks very good! Thanks!
One final thing, was doubting between the method name of the interface (SerializeValue). I think SerializeObject is more appropriate. What do you think?

I think originally I choosed SerializeValue since it should just serialize the value part of a JSON key/value-pair. But actually it serializes objects, so... yes, I'll just rename it.

Just curious regarding unicode characters in the string. Can see JsonEncodeLayoutRendererWrapper has special logic for characters not within the ASCII-range (NeedsEscaping).

Just curious regarding unicode characters in the string. Can see JsonEncodeLayoutRendererWrapper has special logic for characters not within the ASCII-range (NeedsEscaping). Or is this handled by URL-Escaping?

Which file do you mean?

@snakefoot Which file do you mean?

I'm looking at the exiting JSON-encoder implementation (JsonEncodeLayoutRendererWrapper), and I can see it has special handling of unicode characters. I cannot see this handling in the new JSON.-encoder. So I was just curious why.

Ah good one!

Didn't find any clue that would be necessary or even mandatory. Since the data is transferred with unicode encoding, I didn't see a cause for escaping characters.

Unicode characters may be placed within the
quotation marks except for the characters that must be escaped:
quotation mark, reverse solidus, and the control characters (U+0000
through U+001F).

JSON text SHALL be encoded in Unicode.  The default encoding is UTF-8.

https://tools.ietf.org/html/rfc4627
So it seems that normal unicode issues can be ignored, but control characters needs to be handled. While it  is optional to escape non-ascii characters like this: "\u005C". Again just curious as I  know nothing about JSON.

May be more clean now, all control chars (<$1f) are now escaped. The special ones (\t, \r, \n etc) as before and the rest as unicode codes \00xx

Now that you have added support for serializing IEnumerable and IDictionary, then you have also added the potential cyclic-graph-pitfall.
Ex. calling SerializeObject on the following object, might take a while:
List<object> hello = new List<object>();
hello.Add(hello);
SerializeObject(hello);

Maybe have a level counter (default starting at 2) of how many levels down the object-graph that it will explore.

So I'm tempted to say, we keep the serializer rudimentary and leave out serialization of collection types, because there are ready, mature serializers out there one can use, which have been the reason for this interface.

Maybe have a level counter (default starting at 2) of how many levels down the object-graph that it will explore.

The easier and better way is to store the reference the a hashset? (and compare if it's already used)

So I'm tempted to say, we keep the serializer rudimentary and leave out serialization of collection types,

Good point, doubting as I think we are almost there.

So:
switch do hashset for numeric types
and
use a hashset to track visited references to prevent circular paths.
Allright?

Yes that would be nice!

I'm on it

Although the question may be: how to handle reference loops? JSON.NET for example per default breaks with an exception. You can alter this behavior to 2 alternatives.
We could end with an exception or skip serialization of objects, when they're child of themselves. In both cases we track every 'path' of object serialization.

Consider extracting the method JsonLayout.IsNumeric() into a helper-class, then you can skip the HashSet-Lookup.

I prefer skip instead of exception

Now it uses Hashset (and Contains() ) for checking current serialization path for loop. In the (unlikely) case of an infinite loop because of special implementations of IEnumerable for example, there is also an maximum recursion depth of 10, just protecting application from a stack overflow or out of memory.

👍
cool. Maybe unlikely, but stackoverflows are one the worst issue you could have in an application. So thanks for fixing it!

This is ready to merge right?

Reviewed 1 of 17 files at r10, 1 of 4 files at r11, 21 of 22 files at r13, 1 of 1 files at r14.
Review status: all files reviewed at latest revision, 5 unresolved discussions.

Comments from Reviewable

@tetrodoxin I have one final comment: please don't create variables with just 1 char! l, c, d etc. Please do use meaningful names. Only in math single chars are allowed ;)


Missing melt owned owner operation in ickb_logic.ts

As proposal mentioned, user should melt their owned owner cell with the Nervos DAO cell of withdrawal phase 1 to finally complete the CKB extraction operation, but I didn't find it in this SDK, is it of work in progress?
reference: https://github.com/ickb/proposal?tab=readme-ov-file#melt-owned-owner

Hey @ashuralyk, thank you for publicly expressing your interest in iCKB as part of the integration of iCKB in NervDAO, I personally appreciate a lot!! 🙏
Melt is implemented by addWithdrawalRequestGroups.
Love & Peace, Phroi

Closing this issue for inactivity, feel free to reopen it later on @ashuralyk :hugs: