text:
Layers

Is the concept of the three layers (legal code, human user, machine readable) still valid?

We still present the licenses in this way and recommend that people use them in this way.
Machine-readable licenses haven't caught on as much as I think we originally hoped but projects like Google's recent machine learning CC-BY image set show the value of making it easy for software to process licensing information.
Out of interest, if you were to propose different (or no) layers, how would you recommend we structure them now?

I'm closing this since there don't seem to be any open questions here. Feel free to reopen if needed.

authors: farfallosa kgodey robmyers

repo: creativecommons/labs


text:
Always add an image to an existing list if that list name exists

It's possible to create duplicate lists with the same name using the List interface if you don't pick from the autocomplete. On the server side, check whether a list with the same name exists before adding a new one.

Fixed

source: gharchive/issue

repo: creativecommons/open-ledger

license: mit

license_type: permissive

license_source: bigquery


text:
[TODO] Change lsp config to object structure

Change to an object structure, since each LSP server is unique. If no config is required then use a boolean:
{
    "projectlocal": {
        "lsp": {
            "tsserver": {
                "init_options": {},
                "root_dir": ["tsconfig.json"]
            },
            "pyright": true
        }
    }
}

In addition to the structure change, let's change the base key from lsp to nvim-lsp to differentiate that this is from the built-in nvim-lsp client.

repo: creativenull/projectlocal-vim

license: MIT

license_source: github-api


text:
Do keys still make sense at the current granular level?

We will add Network queries to Weaviate.

How will we handle keys in there?
Are keys still aligned with the current vision of what Weaviate should be?

We aligned on this IRL this morning; we don't know yet for the Network; for GraphQL, we'll for now require that the user has a Root Key (and therefore is allowed to access all data).

repo: creativesoftwarefdn/weaviate


text:
TypeScript type declaration file

If you want I can write a TypeScript type declaration file for this package so that TS developers can utilise the type-checking feature of the language as they use this package. Let me know if you are interested.

Hi, sure it would be great if this package could be imported in both cases(JS & TS). You could submit a PR.

Awesome, I'll start on it tomorrow then.

fixed in 0.0.1

authors: boly38 creharmony geopic

repo: creharmony/node-etsy-client


text:
Debugging/logging

@AdilZouitine had a play around with creme and found it slightly difficult to debug a pipeline. He said it would be nice to be able to display the state of the features at each step in the pipeline. I think this is definitely worth thinking about. I'm going to try something in the upcoming weeks, but I'm gladly open to proposals.

I got this.

Done, see here.

repo: creme-ml/creme


text:
how to change env

you use in view
@if (app()->isLocal())     <script src="{{ mix('js/app.js') }}"></script>   @else     <script src="{{ mix('js/manifest.js') }}"></script>     <script src="{{ mix('js/vendor.js') }}"></script>     <script src="{{ mix('js/app.js') }}"></script>   @endif
how you change the env local to production, i update in .env file but it did not work.
also i run "npm run prod"

You probably have the config cached.
Run php artisan config:clear

repo: cretueusebiu/laravel-vue-spa


text:
User icons have stopped working

In Chrome, the icons that represent the users are only showing the "broken image" icon.  The only way to tell who is a reviewer is to hover the mouse over the icon to show the name.  I've tried in an Incognito window also.  It seems to be working in Edge
When I inspect the html element the image src url is correct and when copied into a new tab will load the image correctly.

Desktop (please complete the following information):

OS: Windows 11
chrome
Version 127.0.6533.89 (Official Build) (64-bit)

I have the same issue with Firefox 131.0.3

@MrDark - I did resolve this in the end.  I forgot to update this ticket.  It turned out to be an issue with permissions.  Sadly I cannot remember exactly which.  I worked out by looking at the dev tools and I could see the error messages when it was trying to load the icons.
Hope this helps you mate.

Do you remember if it was a DevOps or browser permission?
I'm the org and project admin, so one would think I have permission to load someone's profile picture.
Looking at the network panel, it seems the profile picture images are being redirected to the sign-in portal. When I open the image URL in a new tab, I see the redirect, but after that, the user icon shows fine.
https://spsprodweu1.vssps.visualstudio.com/_signin?realm=dev.azure.com&reply_to=https://dev.azure.com/xxxxxxxxxxx/_apis/GraphProfile/MemberAvatars/aad.YTI0M2MzZDItMTg4Zi03OTIxLTk3OTktNGFmXXXXXXXXX&redirect=1&hid=ab3dbe84-00a4-4cfb-b2ee-1183f105e045&context=eyJodCI6MiwiaGlkIjoiYmXXXXXXXXXXXWUzMi00YzQ1LTllYTQtZWQ0N2ViZjQ0OGQxIiwicXMiOnt9LCJyciI6IiIsInZoIjoiIiwiY3YiOiIiLCJjcyI6IiJ90

Found the solution! Firefox "Enhanced Tracking Protection" breaks it. Turning this feature off for Azure DevOps made the user icons show.

Thanks for sharing! In Edge this seems related to allowing "Third-party cookies".

Same on chrome. It happened again to me last week. Chrome only allows third
party cookies for 90 days, it seems, before you have to authorise it again.
On Tue, 26 Nov 2024 at 14:19, Alexander @.***> wrote:

Thanks for sharing! In Edge this seems related to allowing "Third-party
cookies".
image.png (view on web)
https://github.com/user-attachments/assets/4fd460ff-7f80-4aee-9d80-4d053e580dad
—
Reply to this email directly, view it on GitHub
https://github.com/cribeiro84/azure-devops-pull-request-hub/issues/281#issuecomment-2500926954,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKEIQ46QLYG7UG3T3MYOEM32CR7QZAVCNFSM6AAAAABL4DVWCSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBQHEZDMOJVGQ
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

authors: MrDark StuartMc77 doormalena


text:
Update scala-library to 2.13.11

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

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala-library" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-lang", artifactId = "scala-library" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #159.

source: gharchive/pull-request


text:
Update publications from Giorgio Mariani

In this commits, I update and improve my publications, by adding a featured image (size: 640x480) and the GitHub repository link. I also made a minor change to my bio.

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
3a1654b7740405b0adf37a67366fdbc778a7a95a

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/63f4aedc598bcb00081e0da7

authors: crisostomi giorgio-mariani

repo: crisostomi/gladia-website


text:
Remove a single line of code with a single line comment.

Remove a single line of code with a single line comment.
The format of the comment is commentStart + removeSingleIf(conditions) + commentEnd
Example
Remove code from JavaScript files
let value = JSON.stringify({key: 'value'}); //removeSingleIf(production)

//removeIf(production)
value = JSON.stringify({key: 'value', production: true}, null, 2);
//endRemoveIf(production)

//removeIf(!development)
value = JSON.stringify({key: 'value', development: false}, null, 2);
//endRemoveIf(!development)

const removeCode = require('gulp-remove-code');

gulp.src('./src/file.js')
  .pipe(removeCode({ production: true }))
  .pipe(gulp.dest('./dist'))

Then, the code will change to:
//removeIf(!development)
value = JSON.stringify({key: 'value', development: false}, null, 2);
//endRemoveIf(!development)

Remove code from HTML files

Running in sandbox environment

    Removing single line code is easy.
    Removing single line code is easy.

Then, the code will change to:

    Removing single line code is easy.

Hi, @jiangyanfu.
You forgot to update README.md with this changes.

authors: brunoziie jiangyanfu

repo: crissdev/gulp-remove-code


text:
Update Readme

Please update Readme file with proper steps how to integrate into Gerrit

Please update Readme file with proper steps how to integrate into Gerrit

repo: criteo/automerge-plugin


text:
17.17-beta 2 Confirmation Dialog Roll #2030 fails to recognize maneuver and character portrait

It's not recognizing that Maneuver: Attack is selected, and it does not show the character portrait.

Can you please give the GCA/GCS exported file for this character?

Can you please give the GCA/GCS exported file for this character?

Bruja.zip
I also clarified this one a bit in follow-up #2082. Looks like the art problem is with .webm token art. Works fine with .png.

Thank you - can you also attach here the .wbem token art, please?

authors: chrismaille devakm

repo: crnormand/gurps


text:
Allowing HTTP traffic

Fixes #1 and allows the user to close dropdown menus in filtering by clicking outside of them.

ok, HTTP works locally.  Just the first commit is unrelated to this change, perhaps better create a separate PR (next time :)
thx.

authors: matszabo mbroz

repo: crocs-muni/SED-Storage-Visualiser


text:
Fix misleading error message for invalid annotation format

Description of your changes
Adds validation to ensure all annotations follow a valid format before proceeding with further processing
Fixes #46
I have:

[x] Read and followed Crossplane's contribution process.
[ ] Added or updated unit tests for my change.

Ah, we forgot to update the README.md with quoted gotemplating.fn.crossplane.io/ready: "True" ?, which folks seem to follow/copy and run into this issue..

Ah, we forgot to update the README.md with quoted gotemplating.fn.crossplane.io/ready: "True" ?, which folks seem to follow/copy and run into this issue...

Good catch! Fixed now! 🙏

authors: jtucci phisco ravilr


text:
Segmentation fault on provider startup when external secret stores flag is enabled

Problem
A segmentation fault occurs on startup of this provider when the external secret store flag is enabled.
{"level":"info","ts":"2024-02-08T03:40:02Z","logger":"provider-keycloak","msg":"Alpha feature enabled","flag":"EnableAlphaExternalSecretStores"}
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x19b8b85]

goroutine 1 [running]:
github.com/crossplane-contrib/provider-keycloak/internal/controller/client/protocolmapper.Setup({0x21c4af8, 0xc000580b60}, {{{0x21a9c90, 0xc00033c408}, {0x21a9c60, 0xc0006189d8}, 0x8bb2c97000, 0xa, 0x0, 0x0}, ...})
        github.com/crossplane-contrib/provider-keycloak/internal/controller/client/protocolmapper/zz_controller.go:35 +0x2e5
github.com/crossplane-contrib/provider-keycloak/internal/controller.Setup({0x21c4af8, 0xc000580b60}, {{{0x21a9c90, 0xc00033c408}, {0x21a9c60, 0xc0006189d8}, 0x8bb2c97000, 0xa, 0x0, 0x0}, ...})
        github.com/crossplane-contrib/provider-keycloak/internal/controller/zz_setup.go:49 +0x19d
main.main()
        github.com/crossplane-contrib/provider-keycloak/cmd/provider/main.go:112 +0x1cf2

Cause
Digging into this issue a bit, it seems that the segmentation fault occurs here:
https://github.com/crossplane-contrib/provider-keycloak/blob/6c075c42c056e09a205aded3874ee548d6b1f8a2/internal/controller/client/protocolmapper/zz_controller.go#L35-L36
The cause seems to be that ESSOptions is nil since it is not set here in the initial setup:
https://github.com/crossplane-contrib/provider-keycloak/blob/6c075c42c056e09a205aded3874ee548d6b1f8a2/cmd/provider/main.go#L93-L110
Proposal
Looking at the source code for the Azure provider, they have a few additional lines of code in this section to set ESSOptions.
	if *enableExternalSecretStores {
		o.SecretStoreConfigGVK = &v1alpha1.StoreConfigGroupVersionKind
		log.Info("Alpha feature enabled", "flag", features.EnableAlphaExternalSecretStores)

+		o.ESSOptions = &tjcontroller.ESSOptions{}
+		if *essTLSCertsPath != "" {
+			log.Info("ESS TLS certificates path is set. Loading mTLS configuration.")
+			tCfg, err := certificates.LoadMTLSConfig(filepath.Join(*essTLSCertsPath, "ca.crt"), filepath.Join(*essTLSCertsPath, "tls.crt"), filepath.Join(*essTLSCertsPath, "tls.key"), false)
+			kingpin.FatalIfError(err, "Cannot load ESS TLS config.")
+
+			o.ESSOptions.TLSConfig = tCfg
+		}

		// Ensure default store config exists.
		kingpin.FatalIfError(resource.Ignore(kerrors.IsAlreadyExists, mgr.GetClient().Create(context.Background(), &v1alpha1.StoreConfig{

I believe adding these few lines of code to the Keycloak Provider will resolve this issue. I would greatly appreciate any help!

Hi,
thx for the bugreport.
v0.7.0 should be available now, please test.  https://marketplace.upbound.io/providers/crossplane-contrib/provider-keycloak/v0.7.0

@Breee I've tested the new release and everything seems to be resolved. Thanks so much for the quick fix! :shipit:

repo: crossplane-contrib/provider-keycloak


text:
Release v0.27.0 on May 04, 2022

Checklist

[x] Create the new release branch with minor version, i.e. release-0.27 for v0.27.0.

You can use the existing branch for patch releases.

[x] Tag release by running Tag action in Github UI against release branch, i.e. release-X branch.

Use v-prefixed version, like v0.27.0, for both description and tag.

[x] Run CI action against release branch.
[x] Tag the next pre-release by running Tag action in Github UI against master branch, if it's not a patch release.

The tag should be v0.28.0-rc.0 if you're releasing v0.27.x.

[ ] Validate that you can install the published version, basic sanity check.
[ ] Run Promote action to promote it in alpha channel if it's pre-1.0.
[ ] Use Github UI to generate release notes.

Make sure to scan all PRs marked with breaking-change and add instructions for users to handle them.

[ ] Create the next release issue with a title that has its date, 4 weeks after the current release day.
[ ] Announce in Slack, Twitter etc.

We have one blocking issue https://github.com/crossplane/provider-aws/issues/1272

During the review of this PR, we identified another bug. It would be great if we can wait for the PR.

since #1298 is fixed we will start with the release 0.27

@haarchri Got back from the holiday break, do you want me to assign the issue to you?

I currently publish the release ;) think will be finished next 30 min

basic test is done
NAME                                                           HEALTHY   REVISION   IMAGE                             STATE    DEP-FOUND   DEP-INSTALLED   AGE
providerrevision.pkg.crossplane.io/provider-aws-866abfbb37fc   True      1          crossplane/provider-aws:v0.27.0   Active                               10m27s

NAME           INSTALLED   HEALTHY   PACKAGE                           AGE
provider-aws   True        True      crossplane/provider-aws:v0.27.0   10m58s

@haarchri Thank you!!

authors: ezgidemirel haarchri muvaf

repo: crossplane/provider-aws


text:
[wrt][xwalk-3648] Remove one manual test from rtcoremanu

Impacted TCs num(approved): New 0, Update 0, Delete 1

The auto test pr link: https://github.com/crosswalk-project/crosswalk-test-suite/pull/1919

LGTM

authors: ibelem wanghongjuan

repo: crosswalk-project/crosswalk-test-suite


text:
[Android] Set the visibility for XWalkViewInternal.

If the visibility of XWalkView is defined in layout, it will be passed
to XWalkViewInternal by AttributeSet, just invoke XWalkView.setvisibility()
will not work since it's child view(XWalkViewInternal)'s visibility was
set implicitly by AttributeSet.
Invoke the child view's setVisibility() explicitly when set the parent
view's visibility.
BUG=XWALK-5753

@axinging @fujunwei @lincsoon @xzhan96 @XiaosongWei PTAL, thanks.

LGTM

LGTM

authors: axinging wuhengzhi xzhan96

repo: crosswalk-project/crosswalk


text:
[Windows] Implement support for native notifications.

Ask permissions to the user (previously it was not granted). Android
will not grant permissions because it works differently.
Hook up the notification support in content to the Windows C++ API. It
comes with some limitation such as not being able to have URL based
icons part of the notification. Workaround is to copy it in a temporary
file before firing the notification.
Windows 7 is not supported as there is no notification center.

Known limitations which I plan to fix in a following patch : persistent
support (aka it says in the notification center), activation even when
Crosswalk is closed.
BUG=XWALK-4990

@pozdnyakov PTAL

lgtm with nits

authors: darktears pozdnyakov


text:
Set environment via RAILS_ENV

Unless there is a RACK_ENV use the RAILS_ENV to set the environment.
This is required if we mount the app via rails.

+1

authors: rsalevsky toabctl

repo: crowbar/crowbar-init


text:
Clicking on request in table view should highlight request on map and vice versa

Summary
Table view of requests has been recently added. However, there is no correlation between the map and table view at the moment. We want to populate the table view with more data e.g. the grocery list in the future. To make the table view more useful, focus on a request on the map should highlight the request table row as well. Conversely, clicking/hovering on the request on the table should open the popover for the request on the map.
Requirements

If the table row is out of view, it should scroll into visible view
Similarly, if the map popover is out of view, it should be panned into view
This may require sharing state across components i.e. ClusterMap and DeliveryTable. May have to introduce some sort of state management like Redux.

(rough sketch, please use a subtle highlight background color)

closed by https://github.com/crownheightsaid/mutual-aid-app/pull/137

repo: crownheightsaid/mutual-aid-app


text:
empty XSLT file on creating encoding profiles

/encoding/profile/create should prefill the text box with an empty but syntactically correct XSLT file.

Could you suggest a default?

<?xml version="1.0" encoding="UTF-8"?>
<xsl:stylesheet
        version="1.0"
        xmlns:xsl="http://www.w3.org/1999/XSL/Transform"
        xmlns:exslt="http://exslt.org/common">

    <xsl:template match="//properties">
        <job>
            <tasks>

                <task type="encoding">
                </task>

                <task type="postencoding">
                </task>

            </tasks>
        </job>
    </xsl:template>
</xsl:stylesheet>

authors: a-tze jjeising

repo: crs-tools/tracker


text:
Two identical columns for displaying fees

The two identical columns on the /explorer/fees page are very confusing.
They only make sense when the Preferred Unit of Account is not BTS but for all new users this is not the case and when they look at those two columns they have no clue what's going on.
So I guess we need some column headers or just remove the second column if the Preferred Unit of Account is set to BTS.

After xeroc's recent changes there were 4 columns, I felt that was too much so I've reduced it to two again, but I only kept the "equivalent value" ones.

repo: cryptonomex/graphene-ui


text:
Rename option "CSV" to "export as CSV file"

Not all users has to know what is CSV. Option should be more descriptive.

During Usability Tests I already seen once, that someone tried to backup a wallet by clicking CSV option. I think this is because right now this option is just bellow all balances. When people see their balances they think that somewhere close should be a button to backup account with that balances.
Knowing that, we could even rename "CSV" to "export recent activity as CSV file" or move this link "export as CSV file"/button to left (closer to "Recent activity" headline)

@svk31 This is yet another reason why things should be reviewed (with Pull Requests) before merging to master branch (like in any projects which cares about quality and users feedback):
This should not looks like that:

First of all, this should not be represent as link. And even if it that would be a good idea (and it is not), then brackets should not be a part of link, like this:

But still... this should not be a link, from Users perspective this action generate something... so this should be a button, not a link, like this:

I don't want to be ungrateful, but I would prefer make things in slower pace but ones and correctly, and not quickly but with multiple fixes to production.

Buttons are too big imo, and I don't want to waste the vertical space by moving it below the title. I'll put it inside the brackets though.


text:
Unable to set custom status message with custom headers in HTTP::Server::Response

HTTP::Server::Response has a field @status_message which defaults to nil (thus falling back to @status.description) and it can only be altered via #respond_with_status() method (unlike @status which is a writable property). However, since #respond_with_status() resets the response stream and then closes it, it's impossible to provide custom headers (e.g. for CORS) along with custom status message.
I would say it's a bug, since if you're allowing to provide a custom status message in the first place, it should be available for modification with the same rules as the status code (that is, I should be able to alter it at any time before the headers are written to the stream). I suggest changing it to either a regular property, or supply it with a regular setter along with a getter that falls back to @status.description.

I think the idea behind respond_with_status resetting the headers was that the headers might already set up for a successful response and then an error response shouldn't use those headers. I'm not sure that's a good idea though, because it's really unexpected when I set a header before respond_with_status and then it doesn't show up.
response.headers["Foo"] = "Bar"
response.respond_with_status(418, "I'm a teapot")

Maybe we should  really change respond_with_status to not reset the response. If you want that behaviour, you could always reset the headers manually.
This method is also completely broken when headers have already been sent, because the reset causes that flag to disapear and respond_with_status just writes a second response. https://carc.in/#/r/a2ha
That aside, it also makes sense to make status_message writable because it should also be possible to set the status message while writing content to the response body. With respond_with_status this is not possible.

I'm not sure that's a good idea though, because it's really unexpected when I set a header before respond_with_status and then it doesn't show up.

That's exactly how I found out about this particular behavioural detail, actually :-D

Maybe we should really change respond_with_status to not reset the response.

It can be made optional, I'd say. it can even default to reset (as that seems to be the intended usage), but the main problem of course is that this method is the only way that status_message can be specified (unlike status itself).

This method is also completely broken when headers have already been sent, because the reset causes that flag to disapear and respond_with_status just writes a second response. https://carc.in/#/r/a2ha

Now this one looks like a separate bug (and should probably be posted as a separate issue)… Although it does add on to the problem of reset behaviour.

@straight-shoota I think you completed the things to do for this issue with the two PRs above. Is there anything else to do or can you close the issue?

authors: LeXofLeviafan potomak straight-shoota

repo: crystal-lang/crystal


text:
Floating-point manipulation operations should not allow arbitrary numbers

The functions Math.ilogb, logb, ldexp, scalbn, scalbln, frexp, and copysign are only meaningful when their arguments are already the matching floating-point types; it is meaningless to talk about the "exponent" of 42_i32, because it is very clearly a fixed-point number. Thus I propose that we deprecate the restriction-less overloads of these methods and remove them in 2.0. What happens afterwards is that any such calls with incompatible number types will be an error:
module Math
  def ldexp(value : Float32, exp : Int32); end
  def ldexp(value : Float64, exp : Int32); end

  def ldexp(value : Float32, exp : Int); ldexp(value, exp.to_i32); end
  def ldexp(value : Float64, exp : Int); ldexp(value, exp.to_i32); end
end

Math.ldexp(1, 5)        # Error: ambiguous call, implicit cast of 1 matches all of Float32, Float64
Math.ldexp("1".to_i, 5) # Error: no overload matches 'Math.ldexp' with types Int32, Int32

Any necessary floating-point conversions, when desired, must then occur at the call site. (In this example, the exp parameter is not a floating-point value being manipulated, so it may still perform type conversions.)

For reference, these methods currently have overloads with unrestricted parameters implicitly casted to float:
https://github.com/crystal-lang/crystal/blob/dd40a2442fa186add8a82b74edb14a90aa1dae05/src/math/math.cr#L602-L604

I think this is a bug because calling the function Math.ldexp with a floating-point exponent will return a wrong result:
# Math.ldexp(value, exp) is equivalent to:
value * 2 ** exp

# With an integer exponent the results match:
pp! Math.ldexp(1.2, 4) # => 19.2
pp! 1.2 * 2 ** 4       # => 19.2

# But not with a floating-point exponent:
pp! Math.ldexp(1.2, 4.5) # => 19.2
pp! 1.2 * 2 ** 4.5       # => 27.152900397563425

Math.besselj(order, value) and Math.bessely(order, value) have the same problem:
pp! Math.besselj(4  , 1.2) # => 0.005022666277311586
pp! Math.besselj(4.5, 1.2) # => 0.005022666277311586
# Correct result from https://www.wolframalpha.com/input?i=besselj(4.5,+1.2)
# 0.00179578...

pp! Math.bessely(4  , 1.2) # => -16.68618734525732
pp! Math.bessely(4.5, 1.2) # => -16.68618734525732
# Correct result from https://www.wolframalpha.com/input?i=bessely(4.5,+1.2)
# -40.9739...


text:
be more specific about error fetching user details

>> ocm-share-create -grantee marie -idp https://revanc2.docker /home/my-folder
error: code=CODE_INTERNAL msg="error fetching remote user details" support_trace="0b9e389c583245ee3046bf7e9c018c5d"

it instead could say "idp not found", available idps are: ...

Also seeing this one now:
022-11-14 12:33:15.581 ERR ../../reva/internal/grpc/services/ocminvitemanager/ocminvitemanager.go:148 > error fetching remote user details error="error: not found: marie" pid=7 pkg=rgrpc traceid=e3ce0992bc9aa56eab6566a023e38a46

ocminvitemanager is doing status.NewInternal(ctx, err, "error fetching remote user details"), which seems right, it seems the error="error: not found: marie" comes from the err there.


text:
fix static registry lookups

cc @kobergj @aduffeck

Hi @butonic @kobergj what's the progress on this one? Could I give you a hand?

@ishank011 we have gone another approach to make it until beta. We are trying to include the benefits of the static registry into the spaces registry.
We introduced storageproviderid prefixing with this PR: https://github.com/cs3org/reva/pull/2790
And we will improve registry performance for ID based requests with this PR: https://github.com/cs3org/reva/pull/2792
Eventually there will be another branch improving performance on path based registry requests.

authors: butonic ishank011 kobergj


text:
"More Faction Interaction (Continued)" No AI function

Describe the bug
IMPORTANT: Please first search existing bugs to ensure you are not creating a duplicate bug report.

bug description here !!!
Upon creating new game pods won't land and there seems to be no functioning AI in animals (no movement).
If pawns are set to spawn standing they will not be able to move or accept forced work.

To Reproduce (VERY IMPORTANT)
Steps to reproduce the behavior:

Install "More Factions Interaction (Continued)"
Start new game
Watch as nothing happens
See error

Error Log
There were no error logs about this.
Mod List
Harmony,
Core, Royalty,
HugsLib, More Faction Interaction (Continued),
RimThreaded
Screenshots
NA

This happens due to a recent update to More Faction Interaction that broke some stuff. Gonna have to revert to an older version for the moment. GitHub version here
https://github.com/emipa606/MoreFactionInteraction/releases/tag/1.0.5

I cannot reproduce this, can you provide a save maybe?

[Tested on RT 1.3.2]
I tested this issue on several fresh games, on different biomes and world map sizes (30 -100%). Both tribal and crashlanded starts were used.
I was not able to reproduce the described problem, despite several attempts. As far as starting the game on various settings goes, everything seems to work correctly.
I used the most up-to-date version of "More Faction Interaction (Continued)"
Conclusion
Generating a new game with RT and "More faction Interaction (Continued)" seems to work fine. It is likely that the mod has been updated since this issue was encountered, which may have fixed the problem.

authors: Eldulac bababooeyy sophiachan213

repo: cseelhoff/RimThreaded


text:
Add ability to define a Call id that will be added as a log property

Hi @csnemes , once again thanks for the great work.
So I need to add as a  log property a call id that would be defined on the TraceEnter and used again on the TraceLeave, this way would be easy to tied both calls together,, since when doing multi threaded  developments  the trance enter and leave calls get all mixed up and is difficult to follow them even when adding it to the database.
so not sure if this is already possible or would imply a change to the Tracer.
if a change was needed maybe the easiest way would be to define an interface that could then be implemented by the class something like IHasTracerCallId {string CallId {get;}} when implemented by a type then it would be call and passed as a parameter to both the TraceEnter and TraceLeave methods, if the class didn't implement that interface then null would be passed.
Thoughts?

Hi, Thanks.
First of all if you use multithreading in a synchronous way the thread Id would be a good candidate to correlate the entries and exits. If you use async it won't work as the exit might run on a different thread than the enter. For this scenario the easiest and the less intrusive way from Tracer's perspective would be to create a new Adapter (based on an existing one) and extend the TraceEnter and TraceLeave methods. There in the Enter you can create a unique id put it on a AsyncLocal (or on CallContext pre 4.6) so that Leave can get it from there. Of course it is possible to extend the rewriter to generate a unique id for each methodcall and pass it to enters and leaves. This would imply that every adapter should be updated to support the two new parameters. I've been thinking of extending the rewriter to add some kind of extensibility in terms of data logged/passed to the adapter, but I'm really at the starting point on this one.

Thanks , so I did what you suggested created a method called NewCallId on the static log and then added the implementation on the TracerAdapter, Using a AsyncLocal field and on the LogImp method of the tracer just added the Value to an event property called callId it works like a  champ and cause I have the NewCallId method I can determine where I want  to start a New Call Id it works very nice.
Once again thanks for the direction.

repo: csnemes/tracer


text:
Escaped chars

Discovered & fixed a failing test case which I believe is related to https://github.com/webpack/css-loader/issues/79

merged/fixed by https://github.com/css-modules/css-selector-tokenizer/commit/5c8584009766eea9d03f8b3d676ece2d93ac06de

authors: joshwnj sokra

repo: css-modules/css-selector-tokenizer


text:
Pass styled component as a prop?

Sorry if this has been addressed in a past issue.
Is it possible to pass a styled component as a prop? I have a Dropdown component which positions a dropdown menu below any arbitrary button. Below is a simplified example
class App {
  render() {
    return (
      <Dropdown
        button={injectSheet(styles)(props => (
          <button className={props.classes.button}>Submit</button>
        ))}
      >
        <Link />
      </Dropdown>
    );
  }
}

In my example, App also has styles injected, and it's sharing the same sheet as Button. I'm seeing the button component rendered, but the class is not being applied.
I'm not sure if this a bug, not possible, an anti-pattern, or something else.
Thanks

🤦♂
I realized that the issue was stemming from me nesting classes in my styles object. I'm still not sure if the component prop approach above is a good practice, but it's all behaving as expected now.

repo: cssinjs/react-jss


text:
Get original image crop bounds

Using getCropBounds() I can get the cropped bounds of the canvas, and using a hacky way of getting the clamp properties I can get the bounds of the crop, but they seem to be wrongly scaled.
What I'm after is a way of getting the crop box of the original image so the actual cropping can be done elsewhere.
My own hacky solution is to copy most of the code from getCroppedImage in imageCropper.ts and use the values passed to drawImageIOSFix.

There is the onCrop event. please check the sample app.
On Wed, 11 Jan 2017 at 21:04, Simon Altschuler notifications@github.com
wrote:

Using getCropBounds() I can get the cropped bounds of the canvas, and
using a hacky way of getting the clamp properties I can get the bounds of
the crop, but they seem to be wrongly scaled.
What I'm after is a way of getting the crop box of the original image so
the actual cropping can be done elsewhere.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cstefanache/angular2-img-cropper/issues/119, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC7c4Sz0LLdCerw_pH86P5M5qCgAjOXwks5rRTXcgaJpZM4LhAkh
.

If the behavior is as expected please close the issue.

authors: altschuler cstefanache


text:
How can i get the cropped file?

in order to use ng2-file-upload to upload the cropped image to server, i need to get hold of the cropped image, and convert it to javascript File object. Any tip on how to do that?

You can take the cropped image from this.data.image. This will give you the base 64, then you can convert it to blob and then to a file.

I used this code from SO to convert
private convertImageUriToFile(uri: string) {
    const arr = uri.split(',');
    const mime = arr[0].match(/:(.*?);/)[1];
    const bstr = atob(arr[1]);
    let n = bstr.length;
    const u8arr = new Uint8Array(n);
    while (n--) {
      u8arr[n] = bstr.charCodeAt(n);
    }
    return new File([u8arr], this.fileName, { type: mime });
  }

Which I call with .image property
this.convertImageUriToFile(this.imageData.image

authors: AmirGilboa Never7Dice webcoderr


text:
Support for AOSP su (without -c argument)

AOSP "su" version never had "-c" flag option, like I see from sources.
I tried not to break compatibility with previous tsu version, while trying to determine AOSP su by --help, or (in case for older android versions) you may explicitly set it like SU_AOSP=1 sudo id

Besides adding support for aosp su binaries - there are plenty fixes I found which I had to fix, sorry for this LARGE pull request... I hope it could be merged to upstream anyway...

./scripts/tadb was outdated and not functional - replaced it with ./scripts/adb-termux script; it will be good if ./scripts/adb-termux-helper will be included in termux "tsu" package.. @Grimler91
setting empty LD_PRELOAD breaks my system with linking errors - so setting LD_PRELOAD only if it was already set before;
tsu added ONLY termux path but no android system path at all, even with -p/-a options - obviously this was error on adding new LD_LIBRARY_PATH logic (PATH was empty if LD_LIBRARY_PATH was set - or this was really intentional?) related to #63
-a option is just unnecessary - it's set by default anyway (if prepend was not requested)
added /vendor/bin to extra syspath
added extra check if /sbin/su exists - not to get error in stderr that binary not exists on trying to check it version
merged #75
hope haven't broke anything besides

TODO/NOTES:

I don't like ~/.suroot home name - maybe it should be named ~/.tsu for consistency, maybe event ~/.termux/tsu. Better if we have less unique names that you should remember...
I don't see why ~/.termux/tsu/.tmp should be prepended with .
calling su -c PATH=x env VARIABLE=y CMD - like I understand - made for cases when we need to find "env" binary? This is 100% fails on my aosp su with error PATH=x executable not found - but I guess PATH=x should be moved after env anyway?

repo: cswl/tsu

license: ISC


text:
Safe to update Tasmota to 6.7 via the built-in updater?

Just wondering if it is safe to update Tasmota to 6.7 via the built-in updater.
I flashed several devices and ran the curl http://10.42.42.42/flash3 command only.

tuya-convert doesn't have a built in updater, I believe this is a Tasmota question. I don't see why it wouldn't work though.

Thanks. I was just wondering if i need to keep something in mind due to available sizes of the space on the device...

authors: blautob kueblc

repo: ct-Open-Source/tuya-convert


text:
ctapipe-quickstart tool

adds a simple tool to generate a working directory containing the latest configuration files.
% ctapipe-quickstart --workdir "ProdX"
% ls ProdX

README.md
stage1_config.json
stage2_config.json
training_config.json

we could use this to do what I do in the interface for protopipe...
https://github.com/HealthyPear/protopipe-grid-interface/blob/master/create_analysis_tree.py

we could use this to do what I do in the interface for protopipe...

Yes that's the idea: in the future it would create a full directory structure and fill in some variables in the config files, which are used as templates.  For now, I wanted to start simply just to be able to run a basic analysis, but we can definitely extend this to be more fancy.
Just noticed I need a test... I'll add one

added one new feature: the contact information is now filled in from user options (prompted if the user forgets it), so the resulting config file has the correct contact name, email, and organization.

Though if you want, we could just copy your code into it now (or leave that for a future PR?)

I'll check and test it

Since I'm away, I think let's just start simple with this one, and not yet add the feature to generate the full directory structure.  That can be a PR we add later, when @HealthyPear  start to merge the features of protopipe (and maybe it's not needed to be as fancy as protopipe here)

authors: HealthyPear kosack

repo: cta-observatory/ctapipe


text:
Refactor stereo reconstruction

This pull request includes some minor refactoring of the code of the stereo parameters reconstruction. Here is a brief summary of the changes:

changed the option name of the function stereo_reconstruction from magic_only to magic_only_analysis (but the argument name --magic-only used when running the script does not change).
cleaned up the code of checking the angular distance of the pointing directions
modified the logger information display

I checked that the outputs do not change before and after the refactoring and so it would be safe to merge now, but again I keep it here a few days so any comments or requests are very welcome.

Also here changes look good, no particular comment from my side

I just pushed a commit about the change of an output file name when using the --magic-only argument to clarify from name that the data are processed by only MAGICs.

@aleberti I also modified the stereo reco script by following suggestions by flake8 (max-line-length = 90), and run isort and black. It would be much appreciated if you could check it and merge to the master if it looks OK to you. Thanks!

authors: YoshikiOhtani aleberti

repo: cta-observatory/magic-cta-pipe


text:
ParamA OR ParamB is required

How would I validate that either ParamA or ParamB is required.  In other words if ParamA is present then it's ok for  ParamB to not be present and vice versa?

Duplicate of #108.

authors: gustavohenke stubar

repo: ctavan/express-validator


text:
Customvalidators doesn't work

I use express-validator customValidators to add some specific validtors:
middlewares.js
module.exports = function (app) {
     console.log('to be sure that Im called');
     return function (request, response, next) {
         app.use(expressValidator({
            customValidators: {
                checkObjectId: function(name) {
                    return /^[0-9a-fA-F]{24}$/.test(name);
                }
            }
         }));
         next();
     }
};

route.js
const middleware = require(__path + '/middlewares');
module.exports = function (app, passport) {
    router.use(baseUrl, middleware(app));
    // some codes
    router.put(baseUrl + '/invoice/:invoiceId', api.invoices.invoices.update);
}

invoices.js
update: (request, response) => {
       // some codes
       request.checkBody('from', 'invalid Supplier Id').checkObjectId();
       // some codes
   },

My problem is checkObjectId is not recognized and I have this error:
TypeError: request.checkBody(...).checkObjectId is not a function

The problem lies in your middlewares.js file.
The returned function is only adding more express-validator middlewares whenever a request comes in. This means the 1st request will always fail, and more and more express-validator middlewares will be made available.
You could do simply the following instead:
module.exports = app => expressValidator(...)

authors: gustavohenke medkhelifi


text:
Fix All General & Admin Dashboard Bugs

Fix All General & Admin Dashboard Bugs
Make sure that everything matches the Figma

/cib all admin general bugs

Branch 190-all-admin-general-bugs created!

authors: claudeisakeeb ctc-devops

repo: ctc-uci/find-your-anchor-frontend


text:
TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/89645


text:
Apollo Portal登录能否加上验证码以防止暴力破解？

现在使用Apollo，发现很多问题，首先就是Apollo的portal登录没有提供图形验证码的功能，以防止暴力破解，不符合公司的安全规范， 后续版本能够加上这个功能？
另外，还需要考虑SQL注入，权限管理，是否存在越权的安全问题，实际生产环境中使用，还是需要考虑很多安全问题，尤其是Portal存在用户操作，要求比较严格。 @nobodyiam

目前默认提供的spring security登录页面确实没有验证码功能，后续可以尝试加一下。不过还是建议接入公司自己的SSO系统，一方面可以提升用户使用体验，一方面也能统一做安全策略。
你提到的其它问题如SQL注入、权限管理等问题应该是不存在的，如果有发现，欢迎提交issue和PR。

authors: nobodyiam tanxiaosheng

repo: ctripcorp/apollo


text:
多环境portal一直启动不起来

直接下载的 .zip的包   不支持自定义多环境吗？必须要修改源码吗

直接下载的 .zip的包 不支持自定义多环境吗？必须要修改源码吗

是的。

通过集群来实现了单个appId, 不同环境不同配置。

直接下载的 .zip的包 不支持自定义多环境吗？必须要修改源码吗

目前支持的环境应该满足你的需求了，具体可以看看这里。1.3、环境

好的  多谢


text:
[Feature Proposal] Apollo with Kubernetes Service Discovery

Is your feature request related to a problem? Please describe.
Apollo作为非常底层的基础设施，对可用性要求非常高，所以从设计之初就极力降低对外部的依赖，其中就包括通过内置Eureka来实现服务发现：

上图是Apollo的整体架构，可以看到Eureka在其中担当了非常重要的角色。
该架构在非K8S环境下比较自然，运行良好。不过在K8S环境下，由于需要配置eureka.service.url，所以采用了statefulset来部署config service，一方面带来了运维的复杂度，另一方面也脱离了k8s自身的服务发现体系，无法享受生态红利，社区中对此也有一些讨论：#2666，#2903。
Describe the solution you'd like
当部署在Kubernetes环境中，可以借助于K8S原生的Service、健康检查等来实现服务发现和通信，从而可以一方面简化部署复杂度，另一方面可以享受生态红利，如Istio。

如上图所示，Eureka已经从整体架构中消失，取而代之的是Kubernetes的Service。为了减少对Client和Portal的影响，依然保留了Meta Service角色负责Config Service和Admin Service的寻址，区别在于在K8S环境下，返回的并不是单节点的地址，而是对应的服务内部域名，客户端在DNS解析后最终通过clusterIp来访问服务。
Describe alternatives you've considered
上述方案最终通过clusterIp来访问服务，对客户端而言，访问的始终是同一个IP，客户端不再有负载均衡逻辑，灵活度有所降低，不过优点是能够无缝对接生态体系，如Istio。
另一种方案是使用spring-cloud-kubernetes，优点是客户端能够获取到Pod列表从而保留负载均衡逻辑，不过缺点是对一些生态工具的引入会有影响。

原来的 Eureka 方式是否兼容还是直接转向 Kubernetes？

非 K8S 环境下仍然使用 Eureka，在 K8S 环境下建议使用原生的方式，不过用户应该也可以继续使用statefulset 来实现 Eureka 作为服务注册中心

简单地理解为Apollo 通过SPI 的方式选择 Eureka 还是 Kubernetes 来切换 Apollo 内部服务发现逻辑？

是的

期待

So glad to hear about this~

ok，我再k8s部署一下

所以k8s部署还需要配置 eureka.service.url吗，根据 apollo-on-kubernetes  这个部署好服务后，去掉eureka.service.url后服务是不能正常工作的

@kayyellow 可以参考基于Kubernetes原生服务发现

@kayyellow 可以参考基于Kubernetes原生服务发现

ok，已成功

”为了减少对Client和Portal的影响，依然保留了Meta Service角色负责Config Service和Admin Service的寻址“， 请问 k8s部署方式，这里的影响主要是哪些呢，兼容？

@kayyellow client 和 portal 始终从 meta service 获取 config service 和 admin service 的地址，所以保留了 meta service 这一角色，如果不保留该角色的话，client 和 portal 就要修改寻址方式了（比如直接配置 Service 地址）

@kayyellow client 和 portal 始终从 meta service 获取 config service 和 admin service 的地址，所以保留了 meta service 这一角色，如果不保留该角色的话，client 和 portal 就要修改寻址方式了（比如直接配置 Service 地址）

明白了，thx

authors: JaredTan95 helloworldtang kayyellow lianfulei nobodyiam qixiaobo


text:
[Feature]: Optimize RPC Audit Log Filtering Rules

Backround:
The audit log is a key basic component of each project and fully records the entire process of each request. However, in high-concurrency HTTP services, too many detailed audit logs will bring some problems, such as reducing service performance, occupying storage capacity, and wasting log indexing services. So a set of rules is needed to filter useful audit logs.
high-quality code, tests, and documentation
Requirement：
audit log is a kind of structured data, maybe the Query DSL design of ES will provide some help,  filter performance require in nanoseconds

Hi, I am very interested in this feature and I will submit my application report before the deadline.

@shouao How is the progress, welcome to come here to discuss together

Hi，How is the progress?  welcome to come here to discuss together

Hi, guys I am very interested in this feature, I have internship experience in the infrastructure team of a tech company. I will apply for this task at OSPP and submit a feature report before the deadline of OSPP.

Hi, How is the progress? welcome to come here to discuss together

done

authors: JasonHu520 sejust shouao

repo: cubefs/cubefs


text:
[Bug]: Decommission  both replicas of the dp results in both replicas failing to decommission.

Contact Details
hechi1014@126.com
Is there an existing issue for this?

[X] I have searched all the existing issues

Priority
low (Default)
Environment
- CubeFS version: 3.4.0

Current Behavior
If two replicas of a data partition are in different zones and are simultaneously undergoing decommission operations, and both replicas fail to decommission.
Expected Behavior
One replica is decommissioned successfully, while the other replica fails to decommission and reports an error indicating that another replica is already undergoing the decommission process.
Steps To Reproduce
1. Decommission the node where replica A of the data partition is located.
2. Decommission the node where replica B of the data partition is located.
3. Check the decommission progress for both nodeA and nodeB

CubeFS Log
No response
Anything else? (Additional Context)
No response

closed by #3559


text:
Is there a way to define an empty set or tuple on axis?

Consider following query, returning a view, similar to a traditional "extract" view:
SELECT
{some tuple} ON ROWS
{} ON COLUMNS
FROM CUBE

Sure. Just write this:
from mdxpy import MdxBuilder, MdxHierarchySet

query = MdxBuilder.from_cube(cube="c1")
query.add_hierarchy_set_to_column_axis(MdxHierarchySet.all_leaves(
    dimension="d1",
    hierarchy="d1"))
query.add_hierarchy_set_to_column_axis(MdxHierarchySet.tm1_subset_to_set(
    dimension="d2",
    hierarchy="d2",
    subset="Default"))

mdx = query.to_mdx()

print(mdx)

It produces this MDX without selection on the rows
SELECT
{TM1FILTERBYLEVEL({TM1SUBSETALL([D1].[D1])},0)} * {TM1SUBSETTOSET([D2].[D2],"Default")} ON 0
FROM [C1]

MDX requires at least one selection on the columns. All other axes can be empty.

Fantastic, thank you for sharing. As a separate question - is there a way to define and use a "extract" (not a stargate) view through the REST API? So the server will not need to ViewConstruct() and will process such view row-by-row (or column-by-column) ?
Thanks
Alex

Not that I know.
To work around this, you can break a large MDX query into multiple smaller ones.
Or you can "create a cell set" (= execute MDX or a view) and retrieve the content through REST/ TM1py in small chunks with the top, skip arguments. However this still "constructs" the full view in memory on the server-side.

MDX requires at least one selection on the columns. All other axes can be empty.

Actually, it is leading back to my question, consider this query (which works perfectly fine):
SELECT
{} ON 0,
{TM1FILTERBYLEVEL({TM1SUBSETALL([D1].[D1])},0)} * {TM1SUBSETTOSET([D2].[D2],"Default")} ON 1
FROM [C1]
So, I think it might be useful to have an ability to define an explicit "empty tuple" for more flexibility.
Thanks,
Alex.

Ah yes. I didn't think of this.
Please feel free to open an MR for this empty MdxHierarchySet

Hi Marius - maybe a dumb question, but how do I open the MR?

Hi @headmann,
you need to fork this project. Then do the changes to the code and do all the git stuff (git add, git commit, git push, etc.) and then create an MR (Merge Request) in GitHub.
I don't have a tutorial at hand, but I'm sure you will find a good blog post about it! If you do get stuck on the way feel free to post questions in this thread.
Looking forward to your contribution :)

Hi @MariusWirtz - I'll sure give it a try.

authors: MariusWirtz headmann

repo: cubewise-code/mdxpy


text:
Add manage_transaction_log decorator

Add transaction_log_is_active function
Fixes #343

@scrumthing
What do you think?

I think that is the best way to do it. It returns True or False and with that the developer can work. :-)

I think that is the best way to do it. It returns True or False and with that the developer can work. :-)

Thanks @scrumthing
actually I would like this decorator to work for the more efficient write_values_through_cellset function as well. Wouldn't that be nice?
I Will leave this MR in wip state for the moment

authors: MariusWirtz scrumthing


text:
Adding the new function : extracting_dataframe_with_consolidated_elements

Hello Marius,
I sent you an email to explain the current progress of my work to integrate the function.
Have a good day
Poulenard Tom

Closed: Subject to Future Re-evaluation

authors: MariusWirtz TomPoulenard


text:
allow np.arrays to be passed as 'elements' in the function 'get_attri…

allow np.arrays to be passed as 'elements' in the function get_attribute_of_elements() and get_elements_dataframe().

Just wondering why would we need NP Array support?
From: Kevin-Dekker @.>
Reply-To: cubewise-code/tm1py @.>
Date: Wednesday, November 1, 2023 at 10:30 AM
To: cubewise-code/tm1py @.>
Cc: Subscribed @.>
Subject: Re: [cubewise-code/tm1py] allow np.arrays to be passed as 'elements' in the function 'get_attri… (PR #987)
@Kevin-Dekker commented on this pull request.
This merge request prevents the get_attribute_of_elements function from failing if elements argument is an np.array.
Two nice to haves would be:

use any instead of list to do the check.
in the type hints it currently says Union[str, List[str]], which could be replaced by Union[str, Iterable[str]]

Using any should be about 2 times faster than using list, validate with below sample:
res_list = timeit.timeit('import numpy as np; '
          'list(np.array([100]*1000000))', number=1000)

res_any = timeit.timeit('import numpy as np; '
          'any(np.array([100]*1000000))', number=1000)

@MariusWirtzhttps://github.com/MariusWirtz any additional comments on numpy compatability?
—
Reply to this email directly, view it on GitHubhttps://github.com/cubewise-code/tm1py/pull/987#pullrequestreview-1708343901, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEK7GZSHP543HCB7WIH3S3TYCJMGRAVCNFSM6AAAAAA6ZEBVVOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMBYGM2DGOJQGE.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Sorry for the late reply.
I think in places like this, where we make the TM1py functions more generic and assume iterables instead of lists, it makes sense to adapt it so that it works with numpy arrays or potentially other structures that implement any.
I will merge and add the additional commit for the performance optimization that @Kevin-Dekker suggested.

Thank you for your first contribution to TM1py @MBonotto

authors: MBonotto MariusWirtz rclapp


text:
Relay: Fix go/done signals and register write enables

As mentioned in #182. A good first goal would be to make sure all the tests compile with the current FuTIL compiler; then we can add functional testing to make sure they produce the right outputs.

I was trying to see if the generated futil files can compile
   import "primitives/std.lib";
   component main() -> () {
     cells {
       x = prim std_reg(32);
       ret = prim std_reg(32);
       const0 = prim std_const(32, 10);
       add1 = prim std_add(32);
     }
     wires {
       group group2 {
         add1.left = x.out
         add1.right = const0.out
         ret.in = add1.out;
         ret.write_en = 1'd1
         group2[done] = ret.done;
       }
     }

     control {
       seq {
         group2
       }
     }
   }

somehow Futil compiler is yelling at me for
Error: FuTIL Parser:   --> 12:7
   |
12 |       add1.right = const0.out␊
   |       ^---
   |
   = expected guard_eq, guard_neq, guard_leq, guard_geq, guard_lt, guard_gt, guard_or, guard_and, or guard_not

and I couldn't figure out why : (

My best guess is that it's the missing semicolon on the previous line?

@ViviYe is this working now? If so, can you close the issue? Otherwise, can you update us?

Trying to compile the programs under frontend/relay-futil/tests/*.expect using the FuTIL compiler. Ran into two errors:
⇒  fud e --from futil --to verilog frontends/relay-futil/tests/ifelse.expect
Stage 'futil' had a non-zero exit code.
--------------- stderr ---------------
Error:
30 |      res9.in = const7.out;
   |           ^^ [Papercut] Required signal not driven inside the group.
When driving the signal `res9.in' the signal `res9.write_en' must also be driven. The primitive type `std_reg' requires this invariant.

and
⇒  fud e --from futil --to verilog frontends/relay-futil/tests/tensors_add.expect
Stage 'futil' had a non-zero exit code.
--------------- stderr ---------------
Error: FuTIL Parser:   --> 61:7
   |
61 |       while le0.out with cond0 {␊
   |       ^---
   |
   = expected stmt

This might be a good place to leave a link to Jared's suggestion from Twitter:
https://twitter.com/roeschinc/status/1303840284304793602
He's got a proper Rust interface to TVM going. So once we figure out #199, maybe a direct translation from in-memory representation to in-memory representation would be possible, without going through the FuTIL text format.

authors: ViviYe rachitnigam sampsyo

repo: cucapra/futil


text:
Automatic par

Ugly code. Welcome to make it better.

@tissue3 Cleaned up the code. Let me know what you think

I see. I've added more documentation for the code and refactored it a bit more. Please the code for has_conflicts and make sure it works.
Also, add more tests to document the behavior for what happens when there is a non-mergable node in the middle of two mergeable nodes.

This pass doesn't correctly deal with the case where subsequent enables have components in common.
For example,
(enable A B)
(enable B C)

might be collapsed if there are no edges between these two subgraphs.

authors: rachitnigam sgpthomas tissue3


text:
Scenario execution priority attributes

Priority attribute tells the executor how early need to add scenario to the execution queue.
In some cases there is no way to tell the executor to run scenarios in specific order.
For example: Scenarios that will be executing for a long time must be added to execution queue before others.
Design proposal:
@priority [integer]
Scenario: my slow scenario

Scenarios with lower priority value will be added to execution queue earlier (or vice versa).

@50U10FCA7 @tyranron

For example: Scenarios that will be executing for a long time must be added to execution queue before others.

To really solve this issue, in addition to the @priority=n tag, we improve default Runner. For now, in case of concurrent steps, it collects batch of the Steps to execute and waits for until all are complete. We should start executing next Step right after any of them is finished.
Also maybe we should leave an option to wait for all Steps in the batch, as it may help with distributing the load on tested application.

@ilslv I'm not sure this is the responsibility of a Runner. It seems that a Parser may decide which Features should be fed first to the Runner (however, Runner still should be able to distinguish that on Scenario level).
This priority issue doesn't have high priority for us at the moment 😊
We can give it some time and think about better designs.

We should start executing next Step right after any of them is finished.

That would be worthful improvement anyway.

Also maybe we should leave an option to wait for all Steps in the batch, as it may help with distributing the load on tested application.

How it should help? I don't really grasp your idea.

@tyranron

How it should help? I don't really grasp your idea.

If in the batch there is Scenario which is very heavy to execute and other concurrent Scenarios may throttle it, our batch approach may be helpful, as batch will shrink over time and lower load on the tested target. But maybe it's better to use @serial in such cases.

@ilslv yup, or even lower/higher @priority.

Note: I'm an outsider that's new to the Rust community. I just stumbled on this project because I wanted to potentially explore Rust-Cucumber as an option.
Wouldn't priority of scenario be handled by the default parsing behavior of Gherkin files? Maybe this project is different, but the default behavior is to sort steps by the feature file name (maybe file path), then the top-to-bottom order of scenarios. If you want to enforce a priority order, preface the features with a numeral that represents your desired order, and you can even use a dense-ranking system for scenarios that don't need to be run before or after a particular group.
Example:
\---features
        01-runs-first.feature
        02-runs-next.feature
        03-indeterminate-test.feature
        03-still-dont-care.feature
        03-technically-fifth.feature
        99-must-run-last.feature

@Solonotix this library works the following way:

The whole execution pipeline consists of Parser -> Runner -> Writers.
The parser of Gherkin files is abstracted in the way to be just a Stream of parsed Features. It could be anything emitting Features, not just glob-walker over fs .feature files.
Runner polls the Parser's Stream and executes Scenarios concurrently (the concurrency level can be configured).
Writer receives events about execution status from Runner in out-of-order manner, but the writer::Normalized normalizes the output in the way that once some Feature is finished, it's fully printed right away.

So, at the moment, we're not preserving any order when executing and printing Scenarios, only steps inside a Scenario are executed sequentially.

@Solonotix I also would argue that Scenarios should be distributed across .feature-files based on some domain-specific needs, not execution priority.

authors: 50U10FCA7 Solonotix ilslv tyranron

repo: cucumber-rs/cucumber


text:
Make the TC count event optional

It is not a good idea to always stop all test cases in a filter before passing them on, because maybe a formatter that is interested in the test case count is used (from the cukenspace workshop), see also https://github.com/cucumber/cucumber-ruby/pull/1082#issuecomment-310316166

Who cares about fixing this?

Closing because the PR is outdated.

repo: cucumber/cucumber-ruby


text:
cucumber reorder scenario execution

cucumber 1.3 executed scenarios in the order specified on the command line. Now it does some kind of sorting and no matter of command line order, it runs them in the same order. Originally I reported the issue in #778 along with other related issues.
$ cucumber --version
2.0.0.beta.5
$ cucumber ./features/3scenarios.feature:9:6
Feature: ASD

  Scenario: gah2     # ./features/3scenarios.feature:6
    Given I love you # features/step_definitions/gah.rb:1
      hey

  Scenario: gah3     # ./features/3scenarios.feature:9
    Given I love you # features/step_definitions/gah.rb:1
      hey

2 scenarios (2 passed)
2 steps (2 passed)
0m0.042s
$ cucumber ./features/3scenarios.feature:6:9
Feature: ASD

  Scenario: gah2     # ./features/3scenarios.feature:6
    Given I love you # features/step_definitions/gah.rb:1
      hey

  Scenario: gah3     # ./features/3scenarios.feature:9
    Given I love you # features/step_definitions/gah.rb:1
      hey

2 scenarios (2 passed)
2 steps (2 passed)
0m0.041s

$ cucumber ./features/3scenarios.feature:6 ./features/3scenarios.feature:9
Feature: ASD

  Scenario: gah2     # ./features/3scenarios.feature:6
    Given I love you # features/step_definitions/gah.rb:1
      hey

  Scenario: gah3     # ./features/3scenarios.feature:9
    Given I love you # features/step_definitions/gah.rb:1
      hey

Feature: ASD

  Scenario: gah2     # ./features/3scenarios.feature:6
    Given I love you # features/step_definitions/gah.rb:1
      hey

  Scenario: gah3     # ./features/3scenarios.feature:9
    Given I love you # features/step_definitions/gah.rb:1
      hey

4 scenarios (4 passed)
4 steps (4 passed)
0m0.045s

This is the bare minimum test project: https://github.com/akostadinov/testcuke
btw I still see the duplicate execution issue as visible in the last cucumber run above as I reported in #778. I'm running cucumber 2.0.0.beta5

btw I still see the duplicate execution issue as visible in the last cucumber run above as I reported in > #778. I'm running cucumber 2.0.0.beta5

@akostadinov, thanks for the detailed update regarding duplicate execution. I think the fix for that issue is not currently included in 2.0.0.beta.5 and may go out in a future release (the core team would be able to confirm).
FYI I'm able to get the following results by pointing to a more recent version of the cucumber gem in a fork of your example.
$ bundle exec cucumber --format progress features/3scenarios.feature:6 features/3scenarios.feature:9

hey
.
hey
.

2 scenarios (2 passed)
2 steps (2 passed)
0m0.018s

@jdks, ok, so duplicates are removed, only reordering still occurring, thanks

@akostadinov @jdks Yes, as History.md says, #786, which fixes the duplication issue, is on the master branch and will be included in the next release.

Hi @akostadinov, I can confirm the behaviour is as you've described. I'll explain why it works like this at the moment.
When we run your features, we send each feature file through a compiler to generate test cases. Each scenario becomes a test case, as does each row in any examples tables you have. The file:line:line argument you pass on the command-line is a filter. Each test case is passed, in turn, through the filters you've specified (you might also specify other filters, like tags). Once the test case makes it through the filters you've specified, we start executing it.
So the ordering you're experiencing simply comes from the order of the scenarios in the feature file.
Can you explain why it's bothering you? Is there a reason you wan to be able to run the scenarios in a particular order?

@mattwynne, the reason I want the original order is so that multiple processes going over the same set of test cases has smaller chance of executing the same test at the same time. We use an external test tracking system to avoid duplicate runs but it's not ACID and reordering really helps. Actually if cucumber has an option to randomize execution, that can also help. Or if I can do it programatically from env.rb that can also work for our purposes.
A less significant reason I have is that sometimes I want to run a number of scenarios where I want to run the more riskier first so I know about issues earlier. It is more convenient to prepare a list of scenarios than running one after the other manually.
I'm curious though, how duplicates happen to appear as in #778 given your explanation of how scenarios are filtered. Because doing .uniq in #786 looks like just a workaround to the real problem.
I don't know how you're doing the filtering. If there's any feasible way all individual filters to have their own associated bucket, then when a filter matches it could insert the scenario into it's respective bucket and finally flatten the buckets in to a single one in the filter order. I'm guessing though things are more complicated than that.

Yeah I can see how this happens now.
The .uniq call here is used to generate the list of files to be parsed. Parsed files are immediately fed into the filter/execution chain. So when that list contained dupes, we were parsing (and therefore executing) the duplicate files twice.

The locations filter we currently use only filters out scenarios, it does not sort them.
What we need to solve this issue is to create another filter that sorts test cases, based on the list of locations. My preference is to have this separate to the existing Core::Test::LocationsFilter just to keep everything modular and single-responsibility.
Anyone want to have a crack at building that sorter?

Where should that sorter be hooked? The location filter appears to just compare a case to the locations and if it is included, pass it down.
To sort things on the other hand, one needs to pile the scenarios and then sort them out, then finally pass them down. I imagine this can be emulated in a filter but how would that filter know when the last scenario has been reached and pass down the scenarios to be executed?
I think that easiest would be to reuse the locations filter as it does locations matching anyway and having a separate filter would require matching the scenarios to locations again.. double work. But still the question remains - how does filter know when last scenario has been reached to pass down the sorted scenarios.
At least this is my ignorant view on the issue.

Should be fixed by b43e5d59958eb623ab5370be676ea414f61e6f06

This will be included in the next release. I'm going to close it but please do open another ticket if it's not working when we release rc4

authors: akostadinov brasmusson jdks mattwynne

repo: cucumber/cucumber


text:
Failing spec for html formatter scenario-level comment

Between 1.3.19 and 2.0.0, if you have a comment in a feature file at the scenario level, the html formatter does not include it in the output.  This failing spec demonstrates that behaviour.

Fixed by 95bcefa and https://github.com/cucumber/cucumber-ruby-core/commit/2c79be9e0a9f8aa481c9cd0a561a1c91bd707554

authors: brasmusson smiller


text:
[question] validation list of elements

Hello!
I trying to validate list like this:
---
- id: FIRST_EL
  queries:
    - sql: "select 1 from database1"
      name: first_query
    - sql: "select 2 from database1"
      name: second_query
      intrval: 4
      tim: 5
- id: SECOND_EL
  queries:
    - sql: "select 3 from database2"
      name: first_query1
    - sql: "select 4 from database2"
      name: second_query1

I was try Schema like:
#query: {
	name: string
	sql: string
	interval?: *1 | int
        timeout?: *5 | int
}

#ids: {
    id: string
	queries: [...#query]
}

[...#ids]

and try validating my yaml:
 cue vet -c -E -s -v config.yaml rules.cue

Validating was successful.
I was try to use only one element, like:
#query: {
	name: string
	sql: string
	interval?: *1 | int
        timeout?: *5 | int
}

#ids: {
    id: string
	queries: [...#query]
}

[#ids]

and only first item in config:
---
- id: FIRST_EL
  queries:
    - sql: "select 1 from database1"
      name: first_query
    - sql: "select 2 from database1"
      name: second_query
      intrval: 4
      tim: 5

after this validating was failed:
0.queries.1: field not allowed: intrval:
    ./config.yaml:3:2
    ./config.yaml:8:8
    ./rules.cue:1:1
    ./rules.cue:1:9
    ./rules.cue:10:15
    ./rules.cue:13:1
    ./rules.cue:13:2

What wrong with using list of items? Why validating of lists is not working?
Maybe I wring with my schema?

I'm not sure we should close this, there is ambiguity as to whether your original example should work. My intuition is that it should

I checked my code with you solution. And its work.

Yes, my solution works.
What I am saying is that I think your original code should probably work too.

I think should too. But this issue just question. And i think for bug needed new issue?
If needed, I can open this issue.

Apologies, I completely missed the point!
As a couple of additional data points, it seems like this is only an issue with open-ended lists, closed lists seem to work fine, as suggested by the example of using single element, but also a closed list of multiple elements:
// rules.cue
#query: {
        name:      string
        sql:       string
        interval?: *1 | int
        timeout?:  *5 | int
}

#ids: {
        id: string
        queries: [...#query]
}

[#ids, #ids]

# config.yaml
---
- id: FIRST_EL
  queries:
    - sql: "select 1 from database1"
      name: first_query
    - sql: "select 2 from database1"
      name: second_query
- id: SECOND_EL
  queries:
    - sql: "select 3 from database2"
      name: first_query1
    - sql: "select 4 from database2"
      name: second_query1
      intrval: 4
      tim: 5

$ cue vet config.yaml rules.cue
1.queries.1: field not allowed: intrval:
    ./config.yaml:2:2
    ./config.yaml:14:8
    ./rules.cue:1:1
    ./rules.cue:1:9
    ./rules.cue:10:15
    ./rules.cue:13:1
    ./rules.cue:13:8

This also causes cue vet to (incorrectly) succeed:
// rules.cue
#query: {
        name:      string
        sql:       string
        interval?: *1 | int
        timeout?:  *5 | int
}

#ids: {
        id: string
        queries: [...#query]
}

[#ids, #ids, ...#ids]

However, removing [...#ids] from the rules.cue and then using the same expression via -d validates correctly:
$ cue vet config.yaml rules.cue -d '[...#ids]'
1.queries.1: field not allowed: intrval:
    --schema:1:5
    ./config.yaml:2:2
    ./config.yaml:14:8
    ./rules.cue:1:9
    ./rules.cue:10:15

@slewiskelly Thank you for suggesting!

authors: juev slewiskelly verdverm

repo: cue-lang/cue


text:
Modularize turf dependencies

Instead of requiring turf whole-cloth, just install the functions that are configured for dropchop. Currently:
"turf-along": "^3.0.12",
"turf-bezier": "^3.0.12",
"turf-buffer": "^3.0.12",
"turf-center": "^3.0.12",
"turf-centroid": "^3.0.12",
"turf-destination": "^3.0.12",
"turf-envelope": "^3.0.12",
"turf-explode": "^3.0.12",
"turf-midpoint": "^3.0.12",
"turf-simplify": "^3.0.12",
"turf-tin": "^3.0.12",
"turf-union": "^3.0.12",
"turf-within": "^3.0.12"

@nickpeihl I'm (understandably) getting turf is not defined here when I run dropchop locally with only the modules installed (instead of the whole package).
The parent object turf seems to be required, but (many apologies) I can't find where it's defined in the codebase to replace it. Adding it back to the vendorJS array points to a nonexistent node module, so that doesn't work. Other things that don't work include:

var result = turf-buffer.apply(null, params);
var result = buffer.apply(null, params);
var result = turfBuffer.apply(null, params);

@wboykinm I believe that, since we are still not fully using browserify to build dropchop, you'll have to do a similar thing that @jczaplew did with topojson that:

includes a source file in lib that requires the necessary modules
and a browserify command in gulp that builds the script for dropchop before ...
loading into the js_vendor command

All set here.

authors: mapsam wboykinm

repo: cugos/dropchop


text:
Shift to Apollo Server 2.0 and use checkNpmVersions

https://github.com/apollographql/apollo-server/issues/1172#issuecomment-397974075

I couldn't figure out how to connect WebApp with 2.0. I tried a few variations and gave up pretty quick since it's beta anyway. I think it has something to do with the built-in http server in 2.0.

@mattblackdev that would be bad, unless meteor would have a way to start without it's http server. But I'm sure they would still want to be able to apply themselves to express() or others.
I think there should be a way for this: https://www.apollographql.com/docs/apollo-server/v2/essentials/server.html#middleware

How would you obtain a reference to the express app?
registerServer({ server, app }); // app is from an existing express app

This is exactly what I tried but I don't think I figured out the right "app" reference.

Haven't tested yet myself, but apparently there is a newer version not yet referenced in the docs:
apollo-server-express@rc

From @evans' comment:

"[...] with apollo-server-express rc, you can import the ApolloServer class without the dependency on apollo-sever"

And to answer my own question about WebApp, use connectHandlers.
I'm guessing it goes something like:
import { ApolloServer } from 'apollo-server-express'
import { WebApp } from 'meteor/webapp'

const server = new ApolloServer()
server.applyMiddleware({
  app: WebApp.connectHandlers,
  server,
  path: '/graphql'
})

That is excellent. I'm waiting for the release but this looks promising, we're gonna remove a bunch of npm dependencies!

Looks like it is out: https://blog.apollographql.com/announcing-apollo-server-2-2b69fb4702ce

server.applyMiddleware({ app: WebApp.connectHandlers,
Were you able to get this working? I'm getting a Error: Can't set headers after they are sent. error:
https://github.com/apollographql/meteor-integration/issues/123#issuecomment-406768233

@lorensr yes, I know the error. Just add this:
  // We are doing this work-around because Playground sets headers and WebApp also sets headers
  // Resulting into a conflict and a server side exception of "Headers already sent"
  WebApp.connectHandlers.use('/graphql', (req, res) => {
    if (req.method === 'GET') {
      res.end();
    }
  });

authors: StorytellerCZ lorensr mattblackdev theodorDiaconu

repo: cult-of-coders/apollo


text:
Drop NumPy<1.15 support

Following NEP 29.
Fixes #2900.

I sent a PR to fix TestRfft by adjusting tolerance (#3056), but I don't think the above TestRfftn error  can be fixed similarly (numerical errors are too large).
Btw, could you retrigger CI to see if the error is persistent?

Jenkins, test this please

Jenkins CI test (for commit 4228e5e05734300dc4fe7390996b3d984039ce6e, target branch master) failed with status FAILURE.

@peterbell10 @grlee77 Do you guys have clues on why rfftn tests are failing here?

I think this may be CUDA bug mentioned in https://github.com/cupy/cupy/issues/2569#issuecomment-571863448
It looks like those failing cases were using a P100 (Pascal architecture) GPU.

Jenkins, test this please

Jenkins CI test (for commit 4228e5e05734300dc4fe7390996b3d984039ce6e, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 4228e5e05734300dc4fe7390996b3d984039ce6e, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 4228e5e05734300dc4fe7390996b3d984039ce6e, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 4228e5e05734300dc4fe7390996b3d984039ce6e, target branch master) succeeded!

authors: chainer-ci emcastillo grlee77 leofang niboshi

repo: cupy/cupy


text:
[backport] Fix random generator output not being raveled

Backport of #5478 by @emcastillo

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 9e7307fee946d83af3aeddeda62b93ae9462611b, target branch v9) succeeded!


text:
[backport] Conditionally change identifiers for ROCm

Backport of #7079 by @hubertlu-tw

/test jenkins

authors: chainer-ci kmaehashi


text:
Is there a way i can test it out on my data on Google Colab?

Could you please provide a setup notebook to run this in Google Colab?

I'm also curious about this

I'm also curious about this

There is an implementation in the Colab mentioned in this blog post on huggingface:
https://huggingface.co/blog/autoformer

repo: cure-lab/LTSF-Linear


text:
OSError: We couldn't connect to 'https://huggingface.co' to load this model, couldn't find it in the cached files and it looks like ../MagicDrive-pretrained/SDv1.5mv-rawbox_2023-09-07_18-39_224x400/controlnet is not the path to a directory containing a config.json file.

I would like to test generating videos using your pre-trained model.However,when I run the code:python ./tools/test.py resume_from_checkpoint=/root/autodl-tmp/MagicDrive-video/pretrained/SDv1.5mv-rawbox-t_2023-12-04_17-51_2.0t_0.4.3E4-S77040 runner.validation_times=4 runner.pipeline_param.init_noise=rand_all ++dataset.data.val.ann_file=/root/autodl-tmp/MagicDrive-video/data/nuscenes_mmdet3d-12Hz/nuscenes_interp_12Hz_infos_val.pkl,an error occurs as shown in the figure.I have set up the directory as  you provided in the file.What I confuses me is that I should use the model in /root/autodl-tmp/MagicDrive-video/pretrained/SDv1.5mv-rawbox-t_2023-12-04_17-51_2.0t_0.4.3E4-S77040/weight-E4-S77040/unet/diffusion_pytorch_model.bin,but it actually load the model in /root/autodl-tmp/MagicDrive-video/MagicDrive-pretrained/SDv1.5mv-rawbox_2023-09-07_18-39_224x400/controlnet/diffusion_pytorch_model.bin,and the error occurs which is OSError: We couldn't connect to 'https://huggingface.co' to load this model, couldn't find it in the cached files and it looks like ../MagicDrive-pretrained/SDv1.5mv-rawbox_2023-09-07_18-39_224x400/controlnet is not the path to a directory containing a config.json file.

Maybe you can change the config "configs/model/SDv1.5mv-rawbox_t.yaml" line 12 "pretrained_magicdrive: ../MagicDrive-pretrained/SDv1.5mv-rawbox_2023-09-07_18-39_224x400" to "pretrained_magicdrive: ./MagicDrive-pretrained/SDv1.5mv-rawbox_2023-09-07_18-39_224x400" (select the first point in the path)

Yes, the default config will try to find models located here (as in the readme):
{ROOT}/../MagicDrive-pretrained/
└── SDv1.5mv-rawbox_2023-09-07_18-39_224x400

rather than {ROOT}/MagicDrive-pretrained/

Maybe you can change the config "configs/model/SDv1.5mv-rawbox_t.yaml" line 12 "pretrained_magicdrive: **../**MagicDrive-pretrained/SDv1.5mv-rawbox_2023-09-07_18-39_224x400" to "pretrained_magicdrive: **./**MagicDrive-pretrained/SDv1.5mv-rawbox_2023-09-07_18-39_224x400" (delete the first point in the path)

I made the modifications according to your suggestion,but there is another mistake.Since I have  prepared the file as shown in the picture,I'm confused why the jpg file cannot be found.

Maybe the pkl file "nuscenes_interp_12Hz_infos_val.pkl" you used is provided by the author which contains full nuscenes, but you don't download full nuscenes?
------------------ 原始邮件 ------------------
发件人:                                                                                "cure-lab/MagicDrive"                                                                                @.>;
发送时间: 2024年11月26日(星期二) 晚上8:30
@.>;
@.@.>;
主题: Re: [cure-lab/MagicDrive] OSError: We couldn't connect to 'https://huggingface.co' to load this model, couldn't find it in the cached files and it looks like ../MagicDrive-pretrained/SDv1.5mv-rawbox_2023-09-07_18-39_224x400/controlnet is not the path to a directory containing a config.json file. (Issue #119)
Maybe you can change the config "configs/model/SDv1.5mv-rawbox_t.yaml" line 12 "pretrained_magicdrive: **../**MagicDrive-pretrained/SDv1.5mv-rawbox_2023-09-07_18-39_224x400" to "pretrained_magicdrive: **./**MagicDrive-pretrained/SDv1.5mv-rawbox_2023-09-07_18-39_224x400" (delete the first point in the path)
I made the modifications according to your suggestion,but there is another mistake.Since I have  prepared the file as shown in the picture,I'm confused why the jpg file cannot be found.
2024-11-26.201845.png (view on web)
2024-11-26.202014.png (view on web)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

是的，默认配置将尝试查找位于此处的模型（如自述文件中所示）：
{ROOT}/../MagicDrive-pretrained/
└── SDv1.5mv-rawbox_2023-09-07_18-39_224x400

而不是{ROOT}/MagicDrive-pretrained/
像这样的错误通常表示找到预训练模型的路径错误。

You are quite right,I have reorganized the structure of the project to fix this error.Thanks for your help.

authors: CSUXA Miayui flymin

repo: cure-lab/MagicDrive


text:
After we complete the assessment the screen is not exiting.

Describe the bug
Once we complete the assessment at the end screen it says "click on the screen to exit" but here even if we click it will not exit from the app.
To Reproduce
Steps to reproduce the behavior:

Complete the assessment and go to the end screen.
Try to click on the screen to exit.

Screenshots

Smartphone (please complete the following information):

Device: MI Note 6 pro
OS: Android 9

This issue is fixed.

repo: curiouslearning/FeedTheMonsterJS


text:
Render updates

Fixes a bunch of issues in rendering, started using the gerbv test files to check rendering, its not an automated test (since we don't support everything they do yet)  but an easy way to catch rendering issues. The biggest change to the rendering side is that primitives, macros, etc. are rendered to a mask and then that mask is painted to the image, which prevents holes from clearing primitives below them, per the spec.
I'm gonna keep working on rendering issues until our output looks close enough to gerbv's

Coverage increased (+1.02%) to 80.581% when pulling c1b29035218467b496fffed76ea85390461150c7 on render_updates into 521fe89150c6aaa0ff0954cc8d32e4b6f8009324 on master.

Looks good, using gerbv was always a goal for me, hope we can automate it in the future.

authors: coveralls hamiltonkibbe phsilva

repo: curtacircuitos/pcb-tools


text:
ValueError: Config entry has already been setup!

I am seeing errors saying certain items are "already setup" at startup on a system with both Nabu Casa and this community integration combined.
I assume that's the cause anyway... having a hard time determining which one is running first and which is throwing the error.
I also assume I could probably disable the items that this integration tries to do and only leave the ones the Nebu Casa one doesn't do, activated, but not sure if that'll break things.   Not sure if it would break nice things like the add-on being able to do TTS while the paid integration doesn't.
I suppose it might be nice if the add on detected an existing integration and skipped those items automatically?
Overall the bug seems harmless. Just reporting for folks who might see similar at startup.

Home-assistant (version): 0.100.3
Hassio (Yes/No): Yes
Version: 2.3.4 from HACS

2019-10-26 14:43:05 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry ***REDACTED*** - amazon.com for media_player
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 190, in async_setup
    hass, self
  File "/usr/src/homeassistant/homeassistant/components/media_player/__init__.py", line 340, in async_setup_entry
    return await hass.data[DOMAIN].async_setup_entry(entry)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 151, in async_setup_entry
    raise ValueError("Config entry has already been setup!")
ValueError: Config entry has already been setup!
2019-10-26 14:43:05 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry ***REDACTED*** - amazon.com for alarm_control_panel
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 190, in async_setup
    hass, self
  File "/usr/src/homeassistant/homeassistant/components/alarm_control_panel/__init__.py", line 74, in async_setup_entry
    return await hass.data[DOMAIN].async_setup_entry(entry)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 151, in async_setup_entry
    raise ValueError("Config entry has already been setup!")
ValueError: Config entry has already been setup!
2019-10-26 14:43:05 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry ***REDACTED*** - amazon.com for switch
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 190, in async_setup
    hass, self
  File "/usr/src/homeassistant/homeassistant/components/switch/__init__.py", line 79, in async_setup_entry
    return await hass.data[DOMAIN].async_setup_entry(entry)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 151, in async_setup_entry
    raise ValueError("Config entry has already been setup!")
ValueError: Config entry has already been setup!
2019-10-26 14:43:05 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry ***REDACTED*** - amazon.com for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 190, in async_setup
    hass, self
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 60, in async_setup_entry
    return await hass.data[DOMAIN].async_setup_entry(entry)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 151, in async_setup_entry
    raise ValueError("Config entry has already been setup!")
ValueError: Config entry has already been setup!**System details**```

There's already logic to try to determine if a component has been loaded. However, it does run into issues sometime. As you noted, this looks like a cosmetic error since HA won't try to load it again.

Roger! As an old sysadmin with white hair, error messages give me PTSD (ha, joking!) but agreed that it looked cosmetic overall -- I just wanted to let devs know in case the code following it wasn't running after the error.  This is totally a "back burner it" sort of bug for when someone is bored... LOL!  Thanks for checking! Can leave open or close as you all wish...

I landed here with 107.x; haven't seen it earlier.
HA: 107.4, unRAID docker
Plug-in: v2.5.7
I'm only seeing these log entries during the initial HA startup after an update.
However, with v2.5.7, I'm now getting this on each subsequent HA restart: (I can log this separately if desired.)
 2020-03-21 06:08:22 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up alexa_media platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 179, in _async_setup_platform
    await asyncio.wait_for(asyncio.shield(task), SLOW_SETUP_MAX_WAIT)
  File "/usr/local/lib/python3.7/asyncio/tasks.py", line 442, in wait_for
    return fut.result()
  File "/config/custom_components/alexa_media/sensor.py", line 151, in async_setup_entry
    hass, config_entry.data, async_add_devices, discovery_info=None
  File "/config/custom_components/alexa_media/helpers.py", line 111, in wrapper
    result = await func(*args, **kwargs)
  File "/config/custom_components/alexa_media/sensor.py", line 112, in async_setup_platform
    account,
  File "/config/custom_components/alexa_media/sensor.py", line 414, in __init__
    client, n_json, "date_time", account, f"next {self._type}", "mdi:alarm"
  File "/config/custom_components/alexa_media/sensor.py", line 192, in __init__
    self._process_raw_notifications()
  File "/config/custom_components/alexa_media/sensor.py", line 200, in _process_raw_notifications
    self._all = list(map(self._update_recurring_alarm, self._all))
  File "/config/custom_components/alexa_media/sensor.py", line 252, in _update_recurring_alarm
    and alarm.isoweekday not in RECURRING_PATTERN_ISO_SET[recurring_pattern]
TypeError: argument of type 'int' is not iterable

Please open a new issue for the TypeError.

Just FYI: I'm now seeing this, suspecting it started recently with one of the newer releases.
Home Assistant 0.107.7,   alexa_media: Installed: v2.5.11   alexapy==1.5.2
Advise if I can posting anything more...
2020-04-05 08:10:51 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry <MY_EMAIL> - amazon.com for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 216, in async_setup
    hass, self
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 59, in async_setup_entry
    return await hass.data[DOMAIN].async_setup_entry(entry)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 156, in async_setup_entry
    raise ValueError("Config entry has already been setup!")
ValueError: Config entry has already been setup!

It's entirely based on timing in HA. If the component has to load dependent devices later, it will end up triggering that error. It appears entirely cosmetic at this point.

Ok, thank you. Ignoring.
But curious: 'dependent devices'?
Is this initiated by something I'd doing, perhaps disabling in the Entity mgr?
Tnx.

The switches (repeat, do not disturb) Guard mode, and timer notifications are all dependent on the media player loading. So if a media player hasn't been fully loaded, all those dependent devices need to wait to load.

authors: alandtse denverpilot genem2


text:
cvat_sdk task.export_dataset fails if dataset export has been called once recently.

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce
Assumes from cvat_sdk import make_client being used and high level SDK

Call export_dataset once: task.export_dataset("CVAT for images 1.1", dest_path_zip, include_images=False)
Wait until its done.
Call export_dataset a second time: task.export_dataset("CVAT for images 1.1", dest_path_zip, include_images=False)

The second call will throw an exception because it receives a 201 status since the request is already created / request is ready for download, but code assumes only good response is 202 (it received the request and will start prepping download).
https://github.com/cvat-ai/cvat/blob/f3dcc3e8352f6d495f22398811f24d2944b6f7cb/cvat-sdk/cvat_sdk/core/downloading.py#L100
Expected Behavior
The second call succesfully downloads dataset to dest_path_zip should work.
Possible Solution
If the downloading helper receives a 201 it should find the corresponding request and download the zip.
I'm not certain joining the request to the right id will be trivial or not. Could the 201 also return the request id?
Context
I intend to work around by writing my own code using the low level API to query requests for completed one if I get a 201 exception.
Environment
Interacting with the hosted cvat.ai app.

Here is my workaround if it helps anyone else:
...
        try:
            task.export_dataset("CVAT for images 1.1", dest_zip, include_images=False)
        except: # TODO: check error is 202 vs 201 issue
            print("Trying low level API approach")
            get_request_result(task_name, dest_zip)
...

def get_request_result(task, dest_path):
    task_id = task.id
    configuration = Configuration(
        host=HOST,
        username=CVAT_NAME,
        password=CVAT_PASSWORD,
    )

    with ApiClient(configuration) as api_client:
        try:
            (data, response) = api_client.requests_api.list(
                status="finished",
                task_id=task_id,
            )
            assert len(data['results']) == 1, "can't assume correct result if more than 1"
            result = data['results'][0]
            result_url = result['result_url']
        except exceptions.ApiException as e:
            print("Exception when calling RequestsApi.list(): %s\n" % e)

    with make_client(host=HOST, credentials=(CVAT_NAME, CVAT_PASSWORD)) as client:
        downloader = Downloader(client)
        downloader.download_file(result_url, output_path=Path(dest_path))

With using CVAT Version = 2.16.2,
I am facing the same issue when exporting the jobs with include_image argument to True. But when I set it to False, I am able to download the annotations.
code:
for jobs in client.jobs.list():
     if jobs.id == job_number:
         jobs.export_dataset(filename=f'{job_number}.zip', format_name='COCO 1.0', include_images=True)

Error
Exception raises
Traceback :Status Code: 201
Reason: Created
HTTP response headers: HTTPHeaderDict({'Allow': 'GET, HEAD, OPTIONS', 'Content-Length': '0', 'Cross-Origin-Opener-Policy': 'same-origin', 'Date': 'Tue, 06 Aug 2024 22:18:29 GMT', 'Referrer-Policy': 'same-origin, strict-origin-when-cross-origin', 'Server': 'nginx', 'Vary': 'Accept, Origin, Cookie', 'X-Content-Type-Options': 'nosniff, nosniff', 'X-Frame-Options': 'DENY, deny', 'X-Request-Id': '9406bba6-e97e-4e6f-8ed7-22305898df33'})

repo: cvat-ai/cvat


text:
Assignee set to None in API response

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce

Create project with a task
Open task and assign job to a user
Use CVAT API to get project id by name

response = client.api_client.projects_api.list(name="test")
project_id = response[0]["results"][0]["id"]

List tasks

  tasks = client.api_client.tasks_api.list(project_id=project_id)

Check assignee field value

Expected Behavior
I expected value to be equal to some ID or username
Possible Solution
No response
Context
No response
Environment
I'm using `app.cvat.ai`

Actually even in UI, when I try to filter by assignee it won't work

Steps are not clear. You assign a job to a user, not a task. But expect task have a user assigned

How can I assign a Task to a user? I didn't find that option anywhere.

My goal was to assign jobs via API to a specific user, but I couldn't figure out where to find the assignee_id necessary for this. So I wanted to check existing jobs that are assigned to figure out the value

My goal was to assign jobs via API to a specific user, but I couldn't figure out where to find the assignee_id necessary for this. So I wanted to check existing jobs that are assigned to figure out the value

Did you find a way to find where can find "assignee_id"? I also need that value to be used when creating a task. Your sharing will be appreciated.

Unfortunately nothing yet, was hoping to get some reply here. I will share if I figure it out

@yili-han-86 do you have any updates on this? @bsekachev if this is possible, would be nice to add it to documentation, couldn't find anything so far


text:
How to use hloc to obtain accurate poses when only using a camera to capture an indoor scene？

Noting the excellent performance of HLOC, I wanted to use it for pose estimation in indoor scenes.
I collected multiple sets of indoor data, but when visualizing poses, I found that in many cases, there were incorrect pose estimates.

I used the following configuration:
     matching_method: Literal["vocab_tree", "exhaustive", "sequential"] = "sequential",
     feature_type: Literal[
         "sift", "superpoint_aachen", "superpoint_max", "superpoint_inloc", "r2d2", "d2net-ss", "sosnet", "disk"
     ] = "superpoint_aachen",
     matcher_type: Literal[
         "superglue", "superglue-fast", "NN-superpoint", "NN-ratio", "NN-mutual", "adalam"
     ] = "superglue",

Please what kind of technique can I use HLOC to get a better pose than colmap in indoor scenes?

I had the same problem, I guess maybe it's the low texture problem.

I want to know how to extract distinguishable features for indoor low texture?
I found that indoors, hloc has a very high failure rate. In my experiments, more than 80% of scene captures could not be used to train a detailed nerf algorithm.
This greatly increases the difficulty of indoor 3D modeling, because relying solely on image information does not seem to be enough to obtain usable poses.
If you have any suggestions I would be very grateful

authors: Bin-ze aboutyy

repo: cvg/Hierarchical-Localization