[WEB-817] [turborepo] Package development with Precontruct and Next.js often crashes

What version of Turborepo are you using?
1.8.7
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Windows
Describe the Bug
When I launch my packages and my examples with precontruct and Next.js in development mode and at the same time, preconstruct is too slow to build the packages which makes Next.js apps crash because it does not find the "modules" which are not yet built, so in the end I find myself with only Precontruct launched
Expected Behavior
What I would have liked is that it waits until precontruct is ready (so build) before launching my Next.js apps.
To Reproduce
Nothing could be easier:

Create a monorepo with precontruct and a simple package
Add an app in Next.js that uses this package
Run it in dev mode

If this works for you, you probably have a very light package, but when you make a UI Kit or something really advanced, the initial build time of precontruct is really slow.
Reproduction Repo
https://github.com/tonightpass/kitchen
WEB-817

Hi @AntoineKM, thanks for opening an issue!  If that is the case, you can add your preconstruct task as an entry in your Next.js task's dependsOn array. For more information, I'd check out the Dependencies between tasks section of our documentation.

Hey @NicholasLYang,
That's exactly it but the problem is that dependOn doesn't support "watch", so for example if I run precontruct with "watch" my Next.js apps will never start.

Ah gotcha. Does the Next.js app not refresh after the modules are built?

Sometimes it does and sometimes it doesn't because my package contains a HOC to be used in the next.config.js and if the package is not built before launching the Next.js app it crashes with the error message `error - Failed to load next.config.js, see more info here https://nextjs.org/docs/messages/next-config-error, Cannot find module, etc...."

Yes, I had already tried that, here's what I tried:
Current scenario

Step 1: preconstruct watch with next dev in parallel

Sometimes it works, but it's only when the watch is faster than Next.js apps to launch, and when it doesn't work, it's because preconstruct takes too long to build its watch, so Next.js crashes with Failed to load next.config.js.
Pre-build scenario

Step 1: precontruct build has dependOn
Step 2: preconstruct watch with next dev in parallel

Sometimes it works, but it's only when the watch is faster than Next.js apps to launch, and when it doesn't work, it's because preconstruct build, then preconstruct clean the build file at the watch launch, so Next.js crashes with Failed to load next.config.js.
Also not ideal because precontruct build take some time, so it degrades our developer experience.
Current workaround

Step 1: preconstruct watch without turbo
Step 2: ...waiting for precontruct to be ready...
Step 3: next dev with turbo

It's clearly not as convenient as preconstruct watch with next dev in parallel with turbo but at least it works !
Expected Behavior
If there is a way to dependOn on a message in the console it could be a good way to launch an app after a watch starts so it could dependOn a watch (and here I'm not talking only for those who use preconstruct)

So I found a workaround that works, but I'm not entirely satisfied, I would have liked something like that to be integrated directly into turbo, if you want to see how I managed to solve this problem go check this pull request: https://github.com/tonightpass/kitchen/pull/219

Closing as there is a workaround. But, @AntoineKM, you're right that Turborepo needs to provide better tools for a "watch" mode. Thanks for your example, I like the idea of being able to wait for output.


Move flash_end to the right place

Doesn't fix anything, but does prevent metadata overwriting ITCM if you attempt to use it. (Making that actually work for games is more work...)

Now I'm curious as to what you've been working on to run into this :laughing:
Thank you!

Now I'm curious as to what you've been working on to run into this :laughing:
Thank you!


Units radio button group should have same rounded corner treatment as canvas buttons

A team member will update this issue with more details soon.

This is your friendly Microsoft Issue Bot. I created this issue automatically as requested by a team member.


Adding multiple drawers on same page

Thanks for very nice plugin.
I have a question though if we can have multiple drawers on same page. For example a left sidebar toggle drawer as well as a right sidebar fixed drawer. And another horizontal drawer ?

For the exact use case, was trying to have 2 level collapsible left sidebar, and fixed right sidebar for notification. For third and fourth level pages, I was thinking to use horizontal drawer from within main section in place of tabs.

👍

try this:
https://jsfiddle.net/6sfp16kn/


Change image size match assertion to warning

Useful for when the retina icons are odd sized, leading to 1px off normal sizes

Can you elaborate on why retina icons would be odd sized? From all the definitions I have seen, they should always be 2x for exact scaling.

Sometimes designers are not the developers, and they provide retina icons that are odd-sized because they don't know of such a requirement. I've been able to get away with using odd-sized retina assets by just adding padding when generating the sprite sheets.
Of course, the ideal situation would be to have even-sized retina icons, or even just adding 1px padding to the source files. But it just seems odd to have the sprite generation abort from a 1px difference when the worse result would be an incorrectly-offset sprite sheet.
But I definitely think this should be an advanced option. Perhaps an option to toggle the assertion on and off would be better?

Designers might not be developers but I am sure they would care about the sprites being consistent across machines/devices. Please update your images to properly be the 2x equivalent.
The warning is there for a reason and it tells you exactly which images are broken instead of you finding it by hand.

Totally agree with your reasoning. spritesmith is great at doing its task well.
In the spirit of open source software, for others who are stuck in a situation with no influence on designers and would still like to automate the sprite generation process, here's a gulp task that will add padding to retina icons to make them even-sized. This expands the canvas size on the bottom right, and of course will not be the correct tool in every situation (say, when you need the sprites to line up without gaps). You will need to install GraphicsMagick (or ImageMagick) and gulp-gm.
var gm = require('gulp-gm');
gulp.task('generateEvenRetina', function () {
  var evener = gm(function (gmfile, done) {
                  gmfile.size(function(error, size) {
                    done(null,
                      gmfile.background('transparent')
                        .extent(size.width+(size.width%2), size.height+(size.height%2))
                    );
                  });
                });

  return gulp.src(INPUT_PATH+'/*.png')
      .pipe(evener)
      .pipe(gulp.dest(OUTPUT_PATH+'/2x'));
});

Awesome, thanks =)


I can't login to evernote account.

I am trying to login with Evernote account for scrapping some things. But I can't go through it. I used "username" and "password" titles for giving the credentials. Is this the right way ?.
Thank you in advance.
 package main

import (
"log"
"github.com/gocolly/colly"
 )

 func main() {
     // create a new collector
     c := colly.NewCollector()

  // authenticate
  err := c.Post("https://www.evernote.com/Login.action", map[string]string{"username":
<EMAIL_ADDRESS>"password": "*********"})
  if err != nil {
	log.Fatal("Error : ",err)
  }

    	// attach callbacks after login
       c.OnResponse(func(r *colly.Response) {
        	log.Println("response received", r.StatusCode)
       })

    	// start scraping
       c.Visit("https://www.evernote.com/")
   }

@davisjohndavis3 Post data names and values can be determined from the target site's source code.
In this case they have 4 items:
<input type="email" placeholder="Email" id="email" name="email" data-validate="email_required, email, email_in_use" >
<input type="password" placeholder="Password" id="password" name="password" data-validate="password_required" >
<input type="submit" id="contact-form-submit" class="button-primary" name="register">
<input type="hidden" name="_sourcePage" value="TRewqUBoP0TiMUD9T65RG_YvRLZ-1eYO3fqfqRu0fynRL_1nukNa4gH1t86pc1SP">

So you need to send email, password and _sourcePage as post variables.

So it should look more like this?
// authenticate
  err := c.Post("https://www.evernote.com/Login.action", map[string]string{"email":
<EMAIL_ADDRESS>"password": "*********", "submit": "*********"}, "hidden": "TRewqUBoP0TiMUD9T65RG_YvRLZ-1eYO3fqfqRu0fynRL_1nukNa4gH1t86pc1SP")


What is is with the ToString() Warning?

Hey there,
There's a warning in the supporting documentation that says the following:

What does that mean and why is the warning in place? Looking at the implementation, it seems like it generates a string with all of the parameters filled in with set Bindings... which is the desired behavior. (A query that could just be copy-pasted directly into SSMS and be ran, for example)

hello @psylithic
it is because of culture in ToString method.
ToString method creates the sql by converting variables (sql arguments) depend on culture on your local machine, what can be incompatible with sql server parse settings depend on server culture. It is about DateTime, double and float data types.
You have to use only compile result with raw sql and bindings, and ADO should combine it in right way.
ToString just for debugging.

in addition to what @for7raid said, you will be open for SQL injection and lose the query plan caching mechanism when using ToString() to execute your queries within the app, using it for debugging purposes is totally fine.

Thanks.


Update the runner to exclude run_with_error_handling

With the updates from #115 using the run_with_error_handling method in the runner is now redundant (not that it previously worked).  The runner needs cleaning up to remove this.

Closed with #164

Closed with #164


Please Add RenderAction and RenderAction Partial Again

Hi,
I think removing RenderAction and RenderActionParital is not a good idea, these ViewConponents require whole new coding files and very irritating, request you please keep RenderAction again.

Hello @iamjassiesandhu ... This is product unit feedback, and I think they decided that they aren't going to do that. See ...

https://github.com/aspnet/AspNetCore/issues/329
https://github.com/aspnet/AspNetCore/issues/1919


Reload cached resources in runtime on file reimport

Closes #93629.
Sponsored By: 🐺 Lone Wolf Technology / 🍀 W4 Games.

I wonder if we should put the reload_cached_files behind the live_debug flag (i.e. synchronize scene changes), and thus also rename the message to live_reload_cached_files and move handling to the live editor.

I thought about that, but decided against it because that's not really a "scene change", and more of an asset data modification.

Thanks!


Discussion about the Building Infrastructure class Diagrams

Open discussion about the Building Infrastructure class Diagrams

From @ycardinale:
Regarding Section 13.1.2: Geographic Information (Hemispheres, Climate types, Orientation), there some concepts explicitly considered in the ontology proposed by Richard (Onto_H2G).
The ifc:SpatialStructuralElement concept hasOrientation, and ifc:Building isA ifc:SpatialStructuralElement. We can integrate Hemispheres and Climate types information related to
ifc:SpatialStructuralElement.
Also, in Onto_H2G, it is proposed a classification of concepts related to energy equipments (Power System) into five categories named IGOEE: Identification, Geographical, Operational, Economical, and Ecological.
We can complete the Geographical concepts with (Hemispheres, Climate types, Orientation. Currently only concepts related to Mobility (Location and time) are considered.
In Minerva Ontology there is a GeographicModel, described as "Geographic representation of the entities: This package is focused on linking the SSN module with geographic contextualisation of the information. This ontology will support in the future the geospatial reasoning over the elements of the domain-specific ontologies."
I think that we can extend this Minerva:GeographicModel with this information.
However, in the APIs, the Geographic information is attached to the building description, while in Onto_H2G is related partially to ifc:SpatialStructuralElement and to the energy equipments. In Minerva Model ssn:System and ssn:FeatureofInterest inherit from geo_Feature that in turns hasGeometry (GML, WKT).
Thus, we should see together this subject to decide if it is necessary to create other concepts and relations in Onto_H2G or Minerva Geographic Model. I prefer the first option: Adjust ifc:SpatialStructuralElement.
Section 13.1.3 Energy categories. This information was taken from ONTO-MG. Then, it is well considered.
Section 13.1.4 Systems. This information was taken from IFC and Haystack. However, there are concepts (Monitoring, Wearable) and relations described in the document D.2.7 that are not described in the document of the Onto_H2G.
We have to take into account these concepts and relations when we work on Building and sensor modeling.
Section 13.1.5 Measures This information is from MONDAS?
Section 13.1.6 Occupancy. Some of this information is considered on the USER-Simulation Module. However there is other information (such as Distant to a point, Level of Knowledge) that we should discuss how they will be modeled.
I think that a meeting to discuss all these aspects is necessary.

I'm reviewing the link about this aspect and I'will let you know my suggestions.
When can we have a meeting to discuss these aspects?


Deleting blog posts

Hi, I really love the template, thank you so much. One thing: with the newest version (from May 4,2021, 0.3.0, commit 69a4193) it still breaks when deleting posts. To reproduce: I just deleted the blog files. I got

I found a similar closed issue posted by mrhut10 here, but noticed that this happens in the newest version as well..

Sorry it seems I made a mistake -- system files seem to have interfered.

In case someone new encounters the same problem:  It was the .DS_Store file in the blog/ - folder that messed things up and which I didn't see..


Complicated aggregate with group on array of Ids

I'm not sure if this is documented anywhere, just point me in that direction if it is.
I have got 2 collections, Fleet and Aircraft. There is a list of Aircraft Ids in Fleet:
data class Aircraft(var _id:Id<Aircraft> = newId(), ...)
data class Fleet(var _id:Id<Fleet> = newId(),
                 var aircraft_id_list:MutableList<Id<Aircraft>> = mutableListOf(),
                 ....)

I would like aggregate this list of ids into actual aircraft objects and then project only a couple of fields in Aircraft.
How would I go about doing this with the Aggregation pipeline?
This is what I've tried so far:
val fleetList = fleetCollection
        .aggregate(
                unwind("aircraft_id_list"),
                lookup("aircraft",
                        "aircraft_id_list",
                        "_id",
                        "aircraft"
                ),
                unwind("aircraft"),
                group(
                        "_id",
                        pushEach(Fleet::aircraft_id_list)
                ),
                project(Aircraft::_id, Aircraft::registration)
        )
        .toMutableList()

But I am getting an error on group telling me it's looking for List<BsonField>
Let me know if there's an easy way to do this, otherwise I can also just do it in two find queries. Cheers!

Please use gitter or the dedicated google group for this kind of question.
You have to specify the type of the result:
data class Result(val _id:Id<Aircraft>, val registration:String)

val fleetList = fleetCollection.aggregate<Result>(
or
val fleetList : MutableList<Result> = fleetCollection.aggregate(

Here is the doc: http://litote.org/kmongo/extensions-overview/#aggregate
But the real issue is that you use the Update push operator : https://docs.mongodb.com/manual/reference/operator/update/push/ instead of the aggregate one https://docs.mongodb.com/manual/reference/operator/aggregation/push
Two options:

Write the bson directly:

...
unwind("aircraft"),
"{\$group:{_id:\"\$_id\",registration:{\$push:"\$aircraft_id_list\"}}}".bson,
...

Use typed query (better!):

...
unwind("aircraft"),
group(Aircraft::_id, Aircraft::registration.push(Fleet::aircraft_id_list)),
...

HTH


run_task_task not finding recently added tasks (until 5 seconds later)

I was trying the code recommended from issue 39 like this:
from django.test import TestCase

from background_task import background
from background_task.tasks import tasks

@background(schedule=5)
def my_background_task(message):
    send an email ...

class MyBackgroundTaskTestCase(TestCase):
    def test_run_task(self):
        my_background_task(...)
                    # time.sleep(5)  # See below
        tasks.run_next_task()
        print len(mail.outbox)

The problem is that tasks.run_next_task won't find the task until 5 seconds later.  I was under the impression run_next_task would be a way to run pending tasks immediately even if they're scheduled for the future.
Is there a different method that does that?  Would it make sense to add a method like that for unit testing?

I changed this issue a lot from what I originally wrote, so make sure to reread if you saw it on email first.

Yes you are right, @gregpinero. run_next_task does not run delayed tasks immediately. I too think such a method would be a useful feature.

This is how BGT works "by design" at the moment. Closing this.


Mags-text-updates-july18

Updates to the confirmation page to include a survey link and to AR filing text for references to the BCA and tool tips.

Great, just so you know, the locales directory only accepts plain text. HTML can only be used inside the markdown (content/) files. I will need to make a small change to support the compliance warning link.
If you could please add the French translations, then we will merge this into the Dev environment. Then I will fix the compliance warning link with a follow up PR.

@deetz99 thanks! I updated them - please note that line 262 of the French file also needs a link for the BCA Section 182. Since there is no French version of the B.C. Business Corporations Act, I've kept that title in English.
@trishreimer, @janisrogers
I've ran the English text through Google Translate and AI chatbots (w/ the prompt to keep it Canadian French, plain language, grade 6 reading level, etc). Louise is retired and Gen is away until the 22 - who can validate these translations?
Line 238
To update your office addresses and directors, visit Corporate Online. Allow up to one business day for changes to take effect before returning here to file your annual report.
Pour mettre à jour les adresses de vos bureaux et vos administrateurs, visitez Corporate Online. Attendez jusqu'à un jour ouvrable pour que les changements prennent effet avant de revenir ici pour déposer votre rapport annuel.
Line 243
AGM information is used to calculate dates for AGM extensions and location changes. It also helps ensure your business stays compliant and in good standing.
Les informations sur les AGA sont utilisées pour calculer les dates des prolongations et les changements de lieu des AGA. Cela aide aussi à s'assurer que votre entreprise reste conforme et en règle.
Line 262
Generally, companies hold AGMs every year. Review the Business Corporations Act (Section 182) to make sure you meet the legislative requirements.
En général, les entreprises tiennent des AGA chaque année. Consultez la Business Corporations Act (Section 182) pour vous assurer de respecter les exigences législatives.

/gcbrun


Two way sync update

Adds the ability to update the Mapillary view by dragging and dropping the location marker.
Adds the ability to enable / disable map synchronization.
Adds the ability to recenter Web on the current Mapillary location.
Current tilt and fov of the Mapillary camera are reflected in the Web location marker.

Web 5.9 will be required to build this

@jonbystedt, @ianschmitz, My feeling is that this has probably now surpassed the scope of what these SDK samples were meant to be, i.e. small, focused samples that each illustrate a single concept. We had discussed creating a separate repo for more advanced samples that are ready to use with little or no customization; I think this would be a good first version of that. Later we would add the ability to configure the Mapillary key. Thoughts?

@ianschmitz @eric-parsons The above makes a lot of sense to me and I think this repo could become very useful as a staging area for components that are not quite ready for the product but have a lot of use for sales/marketing and services nonetheless.

@jonbystedt @eric-parsons I have a story for this new repo high up on my backlog (43050), but might not get to it for a week or two.
Perhaps a smaller change here to bring it up to speed (tilt, and some of the other minor improvements) are still worthwhile, and the rest of the changes we can bring over to the new repo?

Will reopen in the new repo


add funding suggestions

Need to change the hash of the service when v0.2.7 of optimus will be released


Revert "(PUP-4489) Add data_provider to module metadata"

Reverting this change. having binding arguments declared in metadata.json is too complex. A symbolic name will suffice.

CLA signed by all contributors.


Type parameter of GetLogger/GetCurrentClassLogger isn't clear

We can create a Logger of a specified type in LogManager, LogFactory and LogFactory<TLogger>
With these methods:
        /// <summary>
        /// Gets the specified named logger.
        /// </summary>
        /// <param name="name">Name of the logger.</param>
        /// <param name="loggerType">The logger class. The class must inherit from <see cref="Logger" />.</param>
        /// <returns>The logger reference. Multiple calls to <c>GetLogger</c> with the same argument aren't guaranteed to return the same logger reference.</returns>
        /// <remarks>The generic way for this method is <see cref="LogFactory{loggerType}.GetLogger(string)"/></remarks>
        [CLSCompliant(false)]
        public static Logger GetLogger(string name, Type loggerType)

        /// <summary>
        /// Gets the logger named after the currently-being-initialized class.
        /// </summary>
        /// <param name="loggerType">The logger class. The class must inherit from <see cref="Logger" />.</param>
        /// <returns>The logger.</returns>
        /// <remarks>This is a slow-running method.
        /// Make sure you're not doing this in a loop.</remarks>
        [CLSCompliant(false)]
        [MethodImpl(MethodImplOptions.NoInlining)]
        public static Logger GetCurrentClassLogger(Type loggerType)
        {
            return factory.GetLogger(GetClassFullName(), loggerType);
        }

It's unclear to people what loggerType does, as seen in #960 and https://github.com/NLog/NLog/issues/263.
What it does:
1. It expects it inherits from `Logger`
2. Instead of creating an instance of `Logger`, it will create an instance of `loggerType`

It can be useful when custom Loggers are created.
What we need to improve:

[x] The error message
[x] the xml doc
[x] The documentation in code
[ ] The documentation in the wiki, e.g https://github.com/NLog/NLog/wiki/Tutorial#expose-logger-to-sub-classes

working at this in code.

Resolved by #3585 with NLog v4.6.8.
Notice NLog v5.2 marks GetLogger(string name, Type loggerType) as obsolete and redirects to GetLogger<T>(string name) where T : Logger


Delete duplicated files

There is a Keyboard-Teacher project inside another Keyboard-Teacher. I suppose this is an error. Please fix it, deleting the unused files.
If for any reason project are both used, give them a different name to clarify who is who.

Thanks! I didn't really notice that... now the problem is solved!


Enforce the deprecation of conda methods on Image

Changelog

The Image.conda, Image.conda_install, and Image.conda_update_from_environment methods are now fully deprecated. We recommend using micromamba (via Image.micromamba and Image.micromamba_install) instead, or manually installing and using conda with Image.run_commands when strictly necessary.

@prbot approve (got thumb up on slack)


Refactor and add unit tests for Bundler::RubyVersion

Add extensive unit test coverage for lib/bundler/ruby_version.rb
Refactors Bundler::RubyVersionRequirement as a subclass of Bundler::RubyVersion (eliminates the odd, misdirecting code involving input_engine) and redundant methods.

Feedback on this would be appreciated as the formatting, level of mocking/stubbing, use of shared examples, general unit test design on this is how I'd aim to proceed with backfilling further unit tests.
Moreso than PR #4202 and #4208, as I realize now that I eschewed line break formatting for easier readability, which is present in this PR.

@RochesterinNYC none of those files point to people actually using SystemRubyVersion

@segiddins Ah okay, I understand what I'm seeing now. It's cached/in-repo copies of old Bundler versions that have the Bundler code with SystemRubyVersion. Gotcha.

Refactored with the removal of SystemRubyVersion and RubyVersionRequirement.

:+1:
@indirect r?

Updated the PR with the moving of the system functionality to Bundler::RubyVersion.

Absolutely excellent work, @RochesterinNYC !
@homu r+

:pushpin: Commit aa0dd2d has been approved by segiddins

:hourglass: Testing commit aa0dd2d with merge 15a9891...

:sunny: Test successful - status

Thanks for all the review/critique! Always happy to contribute to Bundler.


Add diff utility

Related to #18
After #27

same here, will have a look soon. Sorry for absence

I think that covers it. Now we can quickly find out what Lars changed in the abaplint.json schema :-)

merging, thanks ! sorry for delays ...


Persist method in Chroma no longer exists in Chroma 0.4.x

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain_chroma import Chroma
db = Chroma()
db.persist()

Error Message and Stack Trace (if applicable)
AttributeError: 'Chroma' object has no attribute 'persist'

Description
I saw in a langchain-related issue on the Chroma GitHub page that they mention that in Chroma 0.4.x, the persist method no longer exists: https://github.com/chroma-core/chroma/issues/2012#issuecomment-2053917062
Therefore, I believe it should also be removed here:
https://github.com/langchain-ai/langchain/blob/87d31a3ec0d4aeb7fe3af90f00511677c38f3a3b/libs/community/langchain_community/vectorstores/chroma.py#L613
System Info
System Information

OS:  Darwin
OS Version:  Darwin Kernel Version 23.4.0: Fri Mar 15 00:10:42 PDT 2024; root:xnu-10063.101.17~1/RELEASE_ARM64_T6000
Python Version:  3.9.11 (v3.9.11:2de452f8bf, Mar 16 2022, 10:44:40)
[Clang 13.0.0 (clang-13<IP_ADDRESS>)]

Package Information

langchain_core: 0.1.45
langchain: 0.1.16
langchain_community: 0.0.34
langsmith: 0.1.49
langchain_chroma: 0.1.0
langchain_text_splitters: 0.0.1
langchainhub: 0.1.15

Packages not installed (Not Necessarily a Problem)
The following packages were not found:

langgraph
langserve

Fixed in #20855


how to expect reducer?

mocha + expect

What do you want to test exactly? Most of the time, tests involve testing the new state after the action has been dispatched.


Trait where clause isn't type-checked until used

Aim
This program:
trait Foo<U> where U: Uknown<U> {}

fn main( ) {}

doesn't give an error.
Expected Behavior
The compiler should say that it can't find Unknown.
Bug
The compiler doesn't error.
To Reproduce

Workaround
None
Workaround Description
No response
Additional Context
No response
Project Impact
None
Blocker Context
No response
Nargo Version
No response
NoirJS Version
No response
Proving Backend Tooling & Version
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response

@noir-lang/core Actually, it seems a trait's where clause is completely ignored. "Find all references" of NoirTrait.where_clause gives no results (except when it's created from parsing).

I didn't know we allowed where clauses in this position at all. Checking these is somewhat similar to trait inheritance (which we also don't support) where we'd need to check/assume it everywhere the trait is used.


Ed25519 signature decorator using SHA-256 instead of SHA-512

The signature decorator RFC references the ed25519Sha512_single scheme; however, the linked document is called the ed25519sha256_single specification.
Given that Ed25519 signing does in fact use SHA-512 I'd like to see the spec updated to ed25519sha512_single.

@glcohen feel free to push a PR making the naming change. However, the community has largely agreed to move on to using signed attachments.

I should have noticed the status of the document. Thanks @llorllale I will ignore and move to using signed attachments.

FYI - I pushed a PR to move this RFC into "retired" state - #554 .


crash log not showing a file location

Describe your problem here.
while i was trying to rechart the official game
i kept getting this crash
PsychEngine_2022-10-03_20'47'17.txt
this thing is annoys me so hard and im trying to fix it but i don't have time to do that cuz im just 14
i dont know if it was a bug so i setted this as a help
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

Broken week file

mine was actually missing


[WIP] - adds skeleton, some basic methods for BigQuery

This is how I plan to set up the new BigQuery samples and tests. I'll be finishing the stubbed ones tomorrow.

I changed the format for browse_table, query, and query_as_job. Will this work?

Before we add this new sample, let's update it to use the google-cloud-bigquery gem (as part of gem renaming)

FYI I'm working on some updates to this that I'll send for review later today

Reformatted this code a bit to fit into the template we've been using for samples more recently.  See #53 for follow-up to this.


Is it possible to query all endpoints of an Eureka client?

As titled, if I have instances of multiple REST services with one invoking another, is it possible for them to get the list of all endpoints the other services have?

I am not sure I understand the question.  Are you looking for a way to get all instances of a particular service from Eureka?

No, I'm looking to find all end points of an Eureka client (e.g, Eureka client A consumes service of Eureka client B, can A get all endpoints of B?).

No, eureka, currently only registers the host and port of an instance.

Sounds like you might want to look at something like Spring Hateoas

I have a Problem when I start spring-boot apps on server.port=0 to be able to start multiple instances from a service. In that case the port returned for all instances is 0 (and not the coorect port on which the instance   has been started.
Is there a bug in the eureka implementation ?
The rest api http://localhost:40001/eureka/apps and the java api
List<String> apps = discoveryClient.getServices();                for (String app : apps ){                      List<ServiceInstance> instances = this.discoveryClient.getInstances(app);                         for (ServiceInstance instance : instances){                             String url = "http://"+ instance.getHost() + ":"+ instance.getPort();
Both returns the incorrect value.

@larbigj please don't comment on a possibly unrelated question. We monitor stack overflow as well.


‘T’ was not declared in this scope

In file included from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz/utility/math/decl/tensor.h:6:0,
from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz/utility/math/math.h:17,
from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz/utility/math.h:3,
from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz/utility/image/load_image.h:5,
from /home/wwl/data/ImageQuilting/lib/agz-utils/src/image/load_image.cpp:6:
/home/wwl/data/ImageQuilting/lib/agz-utils/include/agz/utility/math/decl/tensor_view.h:42:32: error: ‘T’ was not declared in this scope
std::is_same_v<T, U> && CONST_TENSOR && !CONST2>>
Hello, I encountered this problem when I "make". May I ask which step I went wrong?

Fixed. Thanks for reporting this!

Scanning dependencies of target AGZUtils
[  1%] Building CXX object lib/agz-utils/CMakeFiles/AGZUtils.dir/src/graphics_api/gl/immediate2d.cpp.o
[  2%] Building CXX object lib/agz-utils/CMakeFiles/AGZUtils.dir/src/mesh/load_mesh.cpp.o
In file included from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz-utils/file.h:3:0,
from /home/wwl/data/ImageQuilting/lib/agz-utils/src/mesh/load_mesh.cpp:1:
/home/wwl/data/ImageQuilting/lib/agz-utils/include/agz-utils/file/dir.h:3:10: fatal error: filesystem: No such file or directory
#include
^~~~~~~~~~~~
compilation terminated.
Hello i have a new problem，filesystem: No such file or directory. I tried to use these to solve this problem.
#include <experimental/filesystem>
std::experimental::filesystem;
But there will be new problems：
Scanning dependencies of target ImageQuilting
[ 97%] Building CXX object CMakeFiles/ImageQuilting.dir/src/main.cpp.o
[ 98%] Linking CXX executable ImageQuilting
CMakeFiles/ImageQuilting.dir/src/main.cpp.o: In function run(int, char**)': main.cpp:(.text+0x3215): undefined reference to std::experimental::filesystem::v1::__cxx11::path::_M_split_cmpts()'
CMakeFiles/ImageQuilting.dir/src/main.cpp.o: In function agz::file::create_directory_for_file(std::experimental::filesystem::v1::__cxx11::path const&)': main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE]+0x38): undefined reference to std::experimental::filesystem::v1::current_pathabi:cxx11'
main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE]+0x46): undefined reference to std::experimental::filesystem::v1::absolute(std::experimental::filesystem::v1::__cxx11::path const&, std::experimental::filesystem::v1::__cxx11::path const&)' main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE]+0x56): undefined reference to std::experimental::filesystem::v1::__cxx11::path::has_parent_path() const'
main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE]+0x65): undefined reference to std::experimental::filesystem::v1::__cxx11::path::parent_path() const' main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE]+0x6d): undefined reference to std::experimental::filesystem::v1::status(std::experimental::filesystem::v1::__cxx11::path const&)'
main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE]+0x7c): undefined reference to std::experimental::filesystem::v1::create_directories(std::experimental::filesystem::v1::__cxx11::path const&)' main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt12experimental10filesystem2v17__cxx114pathE]+0xd9): undefined reference to std::experimental::filesystem::v1::status(std::experimental::filesystem::v1::__cxx11::path const&)'
collect2: error: ld returned 1 exit status
CMakeFiles/ImageQuilting.dir/build.make:99: recipe for target 'ImageQuilting' failed
make[2]: *** [ImageQuilting] Error 1
CMakeFiles/Makefile2:77: recipe for target 'CMakeFiles/ImageQuilting.dir/all' failed
make[1]: *** [CMakeFiles/ImageQuilting.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2
I am a novice, can you help me see what the problem is? Thinks

2022年1月5日 19:04，Z Guan @.*** @.***>> 写道：
Fixed. Thanks for reporting this!
—
Reply to this email directly, view it on GitHub https://github.com/AirGuanZ/ImageQuilting/issues/2#issuecomment-1005588991, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHXBF6PNFOUTKE4QEGF4OCTUUQQUNANCNFSM5LJRZLRQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.

That looks like a compiler environment problem. std::filesystem requires a C++17-compatible compiler while early versions of g++/clang++ require specific link flags to work with filesystem. Could you please tell me your compiler version?

 After I just updated gcc, the previous problem has been solved. This is the previous version of the compiler.
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/8/lto-wrapper
OFFLOAD_TARGET_NAMES=nvptx-none
OFFLOAD_TARGET_DEFAULT=1
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 8.4.0-1ubuntu1~18.04' --with-bugurl=file:///usr/share/doc/gcc-8/README.Bugs --enable-languages=c,ada,c++,go,brig,d,fortran,objc,obj-c++ --prefix=/usr --with-gcc-major-version-only --program-suffix=-8 --program-prefix=x86_64-linux-gnu- --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-vtable-verify --enable-libmpx --enable-plugin --enable-default-pie --with-system-zlib --with-target-system-zlib=auto --enable-objc-gc=auto --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-offload-targets=nvptx-none --without-cuda-driver --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 8.4.0 (Ubuntu 8.4.0-1ubuntu1~18.04)
But I encountered a new problem
[ 97%] Building CXX object CMakeFiles/ImageQuilting.dir/src/imageQuilting.cpp.o
/home/wwl/data/ImageQuilting/src/imageQuilting.cpp: In member function ‘ImageViewagz::math::_simd_float3_t ImageQuilting::pickSourceBlock(Imageagz::math::_simd_float3_t&, Imageagz::math::_simd_float3_t&, int, int, std::default_random_engine&) const’:
/home/wwl/data/ImageQuilting/src/imageQuilting.cpp:377:34: error: no match for call to ‘(const std::uniform_int_distribution) (std::default_random_engine&)’
const int srcX = disX(rng);
^
In file included from /usr/include/c++/8/bits/stl_algo.h:66,
from /usr/include/c++/8/functional:65,
from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz-utils/misc/hash.h:3,
from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz-utils/misc.h:10,
from /home/wwl/data/ImageQuilting/src/imageQuilting.cpp:4:
/usr/include/c++/8/bits/uniform_int_dist.h:165:2: note: candidate: ‘std::uniform_int_distribution<_IntType>::result_type std::uniform_int_distribution<_IntType>::operator()(_UniformRandomNumberGenerator&) [with _UniformRandomNumberGenerator = std::linear_congruential_engine<long unsigned int, 16807, 0,<PHONE_NUMBER>>; _IntType = int; std::uniform_int_distribution<_IntType>::result_type = int]’
operator()(_UniformRandomNumberGenerator& __urng)
^~~~~~~~
/usr/include/c++/8/bits/uniform_int_dist.h:165:2: note:   passing ‘const std::uniform_int_distribution’ as ‘this’ argument discards qualifiers
/usr/include/c++/8/bits/uniform_int_dist.h:170:2: note: candidate: ‘template std::uniform_int_distribution<_IntType>::result_type std::uniform_int_distribution<_IntType>::operator()(_UniformRandomNumberGenerator&, const std::uniform_int_distribution<_IntType>::param_type&) [with _UniformRandomNumberGenerator = _UniformRandomNumberGenerator; _IntType = int]’
operator()(_UniformRandomNumberGenerator& __urng,
^~~~~~~~
/usr/include/c++/8/bits/uniform_int_dist.h:170:2: note:   template argument deduction/substitution failed:
/home/wwl/data/ImageQuilting/src/imageQuilting.cpp:377:34: note:   candidate expects 2 arguments, 1 provided
const int srcX = disX(rng);
^
/home/wwl/data/ImageQuilting/src/imageQuilting.cpp:378:34: error: no match for call to ‘(const std::uniform_int_distribution) (std::default_random_engine&)’
const int srcY = disY(rng);
^
In file included from /usr/include/c++/8/bits/stl_algo.h:66,
from /usr/include/c++/8/functional:65,
from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz-utils/misc/hash.h:3,
from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz-utils/misc.h:10,
from /home/wwl/data/ImageQuilting/src/imageQuilting.cpp:4:
/usr/include/c++/8/bits/uniform_int_dist.h:165:2: note: candidate: ‘std::uniform_int_distribution<_IntType>::result_type std::uniform_int_distribution<_IntType>::operator()(_UniformRandomNumberGenerator&) [with _UniformRandomNumberGenerator = std::linear_congruential_engine<long unsigned int, 16807, 0,<PHONE_NUMBER>>; _IntType = int; std::uniform_int_distribution<_IntType>::result_type = int]’
operator()(_UniformRandomNumberGenerator& __urng)
^~~~~~~~
/usr/include/c++/8/bits/uniform_int_dist.h:165:2: note:   passing ‘const std::uniform_int_distribution’ as ‘this’ argument discards qualifiers
/usr/include/c++/8/bits/uniform_int_dist.h:170:2: note: candidate: ‘template std::uniform_int_distribution<_IntType>::result_type std::uniform_int_distribution<_IntType>::operator()(_UniformRandomNumberGenerator&, const std::uniform_int_distribution<_IntType>::param_type&) [with _UniformRandomNumberGenerator = _UniformRandomNumberGenerator; _IntType = int]’
operator()(_UniformRandomNumberGenerator& __urng,
^~~~~~~~
/usr/include/c++/8/bits/uniform_int_dist.h:170:2: note:   template argument deduction/substitution failed:
/home/wwl/data/ImageQuilting/src/imageQuilting.cpp:378:34: note:   candidate expects 2 arguments, 1 provided
const int srcY = disY(rng);
^
/home/wwl/data/ImageQuilting/src/imageQuilting.cpp:411:39: error: no match for call to ‘(const std::uniform_int_distribution) (std::default_random_engine&)’
const auto xy = allowedXYs[dis(rng)];
^
In file included from /usr/include/c++/8/bits/stl_algo.h:66,
from /usr/include/c++/8/functional:65,
from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz-utils/misc/hash.h:3,
from /home/wwl/data/ImageQuilting/lib/agz-utils/include/agz-utils/misc.h:10,
from /home/wwl/data/ImageQuilting/src/imageQuilting.cpp:4:
/usr/include/c++/8/bits/uniform_int_dist.h:165:2: note: candidate: ‘std::uniform_int_distribution<_IntType>::result_type std::uniform_int_distribution<_IntType>::operator()(_UniformRandomNumberGenerator&) [with _UniformRandomNumberGenerator = std::linear_congruential_engine<long unsigned int, 16807, 0,<PHONE_NUMBER>>; _IntType = int; std::uniform_int_distribution<_IntType>::result_type = int]’
operator()(_UniformRandomNumberGenerator& __urng)
^~~~~~~~
/usr/include/c++/8/bits/uniform_int_dist.h:165:2: note:   passing ‘const std::uniform_int_distribution*’ as ‘this’ argument discards qualifiers
/usr/include/c++/8/bits/uniform_int_dist.h:170:2: note: candidate: ‘template std::uniform_int_distribution<_IntType>::result_type std::uniform_int_distribution<_IntType>::operator()(_UniformRandomNumberGenerator&, const std::uniform_int_distribution<_IntType>::param_type&) [with _UniformRandomNumberGenerator = _UniformRandomNumberGenerator; _IntType = int]’
operator()(_UniformRandomNumberGenerator& __urng,
^~~~~~~~
/usr/include/c++/8/bits/uniform_int_dist.h:170:2: note:   template argument deduction/substitution failed:
/home/wwl/data/ImageQuilting/src/imageQuilting.cpp:411:39: note:   candidate expects 2 arguments, 1 provided
const auto xy = allowedXYs[dis(rng)];
^
CMakeFiles/ImageQuilting.dir/build.make:62: recipe for target 'CMakeFiles/ImageQuilting.dir/src/imageQuilting.cpp.o' failed
make[2]: *** [CMakeFiles/ImageQuilting.dir/src/imageQuilting.cpp.o] Error 1
CMakeFiles/Makefile2:77: recipe for target 'CMakeFiles/ImageQuilting.dir/all' failed
make[1]: *** [CMakeFiles/ImageQuilting.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2

2022年1月5日 20:10，Z Guan @.***> 写道：
That looks like a compiler environment problem. std::filesystem requires a C++17-compatible compiler while early versions of g++/clang++ require specific link flags to work with filesystem. Could you please tell me your compiler version?
—
Reply to this email directly, view it on GitHub https://github.com/AirGuanZ/ImageQuilting/issues/2#issuecomment-1005631599, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHXBF6MZAJJUJSH3N5O5BPDUUQYLBANCNFSM5LJRZLRQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.

Fixed. Please try the new version.

I have updated the code, but there are still problems.
CMakeFiles/ImageQuilting.dir/src/main.cpp.o: In function run(int, char**)': main.cpp:(.text+0x3160): undefined reference to std::filesystem::__cxx11::path::_M_split_cmpts()'
CMakeFiles/ImageQuilting.dir/src/main.cpp.o: In function agz::file::create_directory_for_file(std::filesystem::__cxx11::path const&)': main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE]+0x30): undefined reference to std::filesystem::absolute(std::filesystem::__cxx11::path const&)'
main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE]+0x38): undefined reference to std::filesystem::__cxx11::path::has_parent_path() const' main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE]+0x53): undefined reference to std::filesystem::__cxx11::path::parent_path() const'
main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE]+0x5b): undefined reference to std::filesystem::status(std::filesystem::__cxx11::path const&)' main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE]+0x6e): undefined reference to std::filesystem::create_directories(std::filesystem::__cxx11::path const&)'
main.cpp:(.text._ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE[_ZN3agz4file25create_directory_for_fileERKNSt10filesystem7__cxx114pathE]+0x151): undefined reference to `std::filesystem::status(std::filesystem::__cxx11::path const&)'
collect2: error: ld returned 1 exit status
CMakeFiles/ImageQuilting.dir/build.make:99: recipe for target 'ImageQuilting' failed
make[2]: *** [ImageQuilting] Error 1
CMakeFiles/Makefile2:77: recipe for target 'CMakeFiles/ImageQuilting.dir/all' failed
make[1]: *** [CMakeFiles/ImageQuilting.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2

2022年1月5日 20:44，Z Guan @.***> 写道：
Fixed. Please try the new version.
—
Reply to this email directly, view it on GitHub https://github.com/AirGuanZ/ImageQuilting/issues/2#issuecomment-1005655276, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHXBF6PNXFSQ3BV2FGIMBRDUUQ4MNANCNFSM5LJRZLRQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.

This is still an environmental problem. Try adding link flag -lstdc++fs.


FLAME GPU makefile problem with Iceberg

FLAME GPU make files do not work on Iceberg (The Sheffiled Uni HPC facility). The issue resolves around the call to nvcc.
The make file includes the CUDA_PATH so that calls to nvcc end up looking like /usr/local/cuda-7.5/bin/nvcc rather than just nvcc. I think that the former changes the location of where nvcc looks for files to the bin directory rather than the makefile location so that the cuda input files are not found. E.g.
/usr/local/cuda-7.5/bin/nvcc -ccbin g++ -I../../common/inc  -m64     -gencode arch=compute_20,code=sm_20 -gencode arch=compute_30,code=sm_30 -gencode arch=compute_35,code=sm_35 -gencode arch=compute_37,code=sm_37 -gencode arch=compute_50,code=sm_50 -gencode arch=compute_52,code=sm_52 -gencode arch=compute_52,code=compute_52 -I ../../include/ -I src/model/ -I src/dynamic/ -I src/visualisation/ -I ../../include/GL/ -o io.o -c src/dynamic/io.cu bash: /usr/local/cuda-7.5/bin/nvcc: No such file or directory
From the makefile fails where as modifying the call to
nvcc -ccbin g++ -I../../common/inc  -m64     -gencode arch=compute_20,code=sm_20 -gencode arch=compute_30,code=sm_30 -gencode arch=compute_35,code=sm_35 -gencode arch=compute_37,code=sm_37 -gencode arch=compute_50,code=sm_50 -gencode arch=compute_52,code=sm_52 -gencode arch=compute_52,code=compute_52 -I ../../include/ -I src/model/ -I src/dynamic/ -I src/visualisation/ -I ../../include/GL/ -o io.o -c src/dynamic/io.cu
works correctly.

The CUDA_PATH env variable in the makefile is also used to link against the cuda lib files.
For each make file it should be replace (source)
NVCC          := $(CUDA_PATH)/bin/nvcc -ccbin $(HOST_COMPILER)

with
NVCC          := nvcc -ccbin $(HOST_COMPILER)

The only other occurrence of CUDA_PATH is related to: (source)
# build flags
ifeq ($(TARGET_OS),darwin)
    LDFLAGS += -rpath $(CUDA_PATH)/lib
    CCFLAGS += -arch $(HOST_ARCH)
else ifeq ($(HOST_ARCH)-$(TARGET_ARCH)-$(TARGET_OS),x86_64-armv7l-linux)
    LDFLAGS += --dynamic-linker=/lib/ld-linux-armhf.so.3
    CCFLAGS += -mfloat-abi=hard
else ifeq ($(TARGET_OS),android)
    LDFLAGS += -pie
    CCFLAGS += -fpie -fpic -fexceptions
endif

Honestly I'm not sure we need this as we only support x86_64 hosts at this time.
If CUDA_PATH is removed the readme should be updated to reflect this.

In the READ ME (makefiles), there is a note on how to set your CUDA_PATH, have you tried that ?

If you are using a different version of CUDA compiler, use below command first to change the CUDA_PATH environment variable:
export CUDA_PATH=/path/to/CUDA

It is not an issue with setting the CUDA_PATH as this is correct and nvcc is found. The issue is that having /path/to/cuda/nvcc rather than just nvcc means that nvcc is looking for the input files (e.g. simulation.cu) in the /path/to/cuda/ location rather than the current working directory.

Using an absolute path to an executable doesn't change the working directory, so it shouldn't be looking for the files in the /path/to/cuda directory.
The error message included in the original issue bash: /usr/local/cuda-7.5/bin/nvcc: No such file or directory is stating that the file nvcc does not exist at that location.
I assume that on iceberg cuda is not located in /usr/local/, or cuda 7.5 is not available (the latest docs I can find online only suggest 3.2, 4.0 and 6.5 (source)
On Iceberg with the relevant cuda version loaded what is the output of which nvcc?

Ok. Totaly correct @ptheywood I thought it was an issue with nvcc not with nvcc not being found. This is because on iceberg nvcc is not in the default location so setting CUDA_PATH fixes this. Not sure why we need to specify CUDA_PATH though. It should be the case that nvcc should be in the users PATH already.


rusttype successor

orbtk will probably merge it in, if there is a benefit. It seems it has no high priority yet, since rusttype is working.
What is the name of the successor project?

ab_glyph: https://gitlab.redox-os.org/redox-os/rusttype/issues/148


Azure AD advance provisioning (Provision employees having a contract that starts in the future)

Section Provisioning users in the Onboarding module say that :

Inbound user provisioning from SAP SuccessFactors to on-premises Active Directory and Azure AD now supports advance provisioning of pre-hires present in the SAP SuccessFactors Onboarding 2.0 module.

There is no information if advance provisioning (provisioning of employees with a contract that starts in the future) is supported for Organisations only using Employee Central without SAP SuccessFactors Onboarding 2.0
Please clearify
thanks
Pirmin

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

ID: 56401d85-b771-94ed-5505-c0ba929ba27b
Version Independent ID: d7a38614-2897-1c94-5d24-120431cb018a
Content: Azure Active Directory and SAP SuccessFactors integration reference - Microsoft Entra
Content Source: articles/active-directory/app-provisioning/sap-successfactors-integration-reference.md
Service: active-directory
Sub-service: app-provisioning
GitHub Login: @kenwith
Microsoft Alias: kenwith

@pirminf
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Moved to work item in backlog. #please-close


New package: Duplicati.Duplicati.Canary version <IP_ADDRESS>

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] There's a linked pull request. Resolve #146361, resolve #146404

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

The name is apropos, insofar as browser tabs.


kv Index.Walk function matches the foreign key by prefix only

What
When the kv store searches the index, it utilises a forward cursor matching by prefix:
https://github.com/influxdata/influxdb/blob/15b95312736f1ea651d2e14ba8343e483d5cd92a/kv/index.go#L192-L193
The cursor will return all keys that have a matching prefix, in the following loop, including those keys which are longer:
https://github.com/influxdata/influxdb/blob/15b95312736f1ea651d2e14ba8343e483d5cd92a/kv/index.go#L206-L208
For example, if the foreignKey is hello and the index contains the keys hello, hello-world and hello-there, the loop will gather them all.
In most cases, the index is typically the result of combining 16-byte IDs, so this is not a problem. However, for dbrp, it creates an additional orgID + database foreign key:
https://github.com/influxdata/influxdb/blob/b274e15eacb2cf6c8915a9f75caf63d69fb2aac1/dbrp/service.go#L70-L76
Creation of DBRPs fail:

Create bucket1

Create a dbrp mapping for bucket1 with db jenkins-aws, rp never:
curl -H 'Content-type: application/json' --header "Authorization: Token $INFLUX_TOKEN" -XPOST "$INFLUX_HOST/api/v2/dbrps" --data-binary '{"bucket_id": "<BUCKET1_ID>","database": "jenkins-aws","default": true, "organization_id": "<org-id>", "retention_policy": "never"}'

Create bucket2

Attempt to create a dbrp mapping for bucket2 with db jenkins, rp never:
curl -H 'Content-type: application/json' --header "Authorization: Token $INFLUX_TOKEN" -XPOST "$INFLUX_HOST/api/v2/dbrps" --data-binary '{"bucket_id": "<BUCKET2_ID>","database": "jenkins","default": true, "organization_id": "<org-id>", "retention_policy": "never"}'

Fails with the following error:
another DBRP mapping with same orgID, db, and rp exists

Hey @stuartcarnie 👋
Sorry, definitely a gap in the documentation here. This index is currently a non-unique many to one secondary index.
resource >- association
dashboard >- org

And so the index is structured like so:
indexprefix/foreign_key/primary_key -> primary_key

bucketsbyorgv1/org_foo/bucket_one -> bucket_one
bucketsbyorgv1/org_foo/bucket_two -> bucket_two
bucketsbyorgv1/org_foo/bucket_three -> bucket_three
bucketsbyorgv1/org_foo/bucket_four -> bucket_four

Such that we can have multiple entries per foreign key. Meaning we could retrieve the many resources for a particular foreign key in a single range / cursor with a prefix lookup.
As this was originally designed around creating a URM by User index.
The Walk function visits each key / value pair in the source bucket. Not the index bucket. You are effectively walking the source, based on entries in the index:
https://github.com/influxdata/influxdb/blob/15b95312736f1ea651d2e14ba8343e483d5cd92a/kv/index.go#L218-L229
I have thought about adding support for a secondary index with a uniqueness constraint. In that scenario walk should find one item. Or perhaps a better name might be Find or similar. Infact it could just be another type which also consumes the IndexMapping type. The mapping type describe a source and index bucket name and a function for retrieving the associated index foreign key, for sure when backfilling and unpopulated index.

@GeorgeMac thanks for the reply! I do understand the many to one feature. There is a bug or unexpected behaviour, as described in the issue, that because it matches the foresight key using a prefix match, it will find any keys with that prefix.
So, back to the dbrp example, if a user adds a mapping to a database named jenkins-aws, the indexes foreign key would be Jenkins-aws. The next trying to add a mapping to a database named jenkins fails as the lookup (a prefix match) for jenkins finds jenkins-aws.
It seems it is expected behaviour as a number of other tests fail when I fix that issue. It seems we might need a WalkExact API to match the foreign key exactly.

ooooooh I seee 😂 now I am with you. Took me long enough. I added a comment to your draft PR.


'Follow Collectives' Feedback

The 'Follow Collective' project #6866 is live in production. However, there were a few UI issues/UX jerks that we wanted to address  as feedback.
We acknowledge that the 'Follow Collective' button works well and there is no comments on that.
However, the 'unfollow collective' button has a few issues.

It is too jarring on the user's activity page. It catches your attention too quickly compared to the rest of the information. We'd like to avoid that.

The unfollow button is inconsistent on the activity page and Collective's profile page

I will provide a few solutions shortly and add them to the thread.

Also: https://github.com/opencollective/opencollective/issues/7276

Some feedback:

It is unclear why I am seeing the "Follow Collective" button on an update that already appears in my timeline, and it creates confusion as to what the Following feature does:

I think the action to Unfollow (or Follow, if hovering the Collective associated to an expense or contribution) could be available inside an Account hover card instead of being printed out in the timeline, to avoid unnecessary repetition

However, I'd like to avoid actions in hover cards.

@aminakazi I think this makes sense if the action is a primary action or critical to the page you're viewing, as hover cards are not available on mobile. But in this case, it is not a primary action of the page you're viewing and you can click through to the profile page on mobile to unfollow or follow there.
Also some additional context, as part of adding the timeline messages/translations for the Collective Overview, there are now cases where the Account is in the beginning or middle of the message, for example, here used as a fallback since we're currently not storing what user rejected the contribution:

Or here the Account is in the middle of the message:

To enable that, I removed the pill styling, follow button, and avatar, since a timeline item begins with a bigger standalone avatar, and instead added the account hover cards, where I've also put the follow button:

clicking different parts of the pill seem to have different behavior
as part of fixing the timeline for the Collective Overview, some messages place the Account in the beginning or middle of the message, and so having the Following state or button in the middle of that breaks the flow of the message
I've fixed this in https://github.com/opencollective/opencollective-frontend/pull/10088

Sharing a zoomed-in version to make it clear. On click of the down chevron, the user has the option to unfollow.

I like the idea of a dropdown to Unfollow, but tying it to the Chevron only would make it hard to click, especially on mobile, I presume the other part of the pill takes you to the Collective page.

For the collective page, the unfollow button will look like the one on the overview page.

This feels inconsistent (why not show the Following state first here also?), and grabbing too much attention, making "Unfollow" more emphasized than "Follow". At a glance, with the red color and icon, it looks like there is an error on the page.
I like what Twitter and Facebook do by focusing on the positive, as in either showing the action to Follow, or showing the current state if you're following. Unfollow can be less emphasized.
Building on your suggestion of a dropdown to Unfollow, the universal follow button could work like this:
https://github.com/opencollective/opencollective/assets/1552194/fa9a6dbc-58d7-4fd6-a601-e54fe3e542a1
That makes the same button usable on mobile without any modification, which the previous iteration I had didn't do, yay for reusability and keeping things simple 😃
And I think it needs to be consistent on the Collective page:

(On a side note: we should find a different placement of the button on the Collective page, as it currently shifts the layout as the button is added after the logged in user is loaded)


Migrate from CefRefPtr to std::shared_ptr

Get rid of CefRefPtr / replace with std::shared_ptr
Performance tests

I will use std::shared_ptr for new implementations as well.

Is the related code only in hanni/NewVisualScripting?
If yes, than the milestone tagging seems odd to me.
If no, I would suggest to open a pull request and move things forward.


Error with get_coreclr

I am getting the following error.
rt = get_coreclr("TallyConnector.json") #Json we created earlier
TypeError: get_coreclr() takes 0 positional arguments but 1 was given
Can you help me with this issue?

Now there is Stable release in Python. Net and they changed their API to load .Net runtime
Now we dont require any config file simply call load("coreclr")

I also updated this repo with sample code that supports latest version of tally connector and Python .Net, you can use that as starting Point


change paxel parser to grammar

Mostly changes '=' into 'by' and reformat using IDE to 2 spaces.
This will produce error messages like (i removed the 'e' from 'let x = ')
Parse Failed reading lt x = (p + 1) select x - 1
ere p < 1 lt x
          ^
Expecting select at line 0, column 24
[Traceback]
  at topLevel
  at expressionWithQualifier
  at selectExpression

/gcbrun

/gcbrun

/gcbrun

/gcbrun


fix(PodDetail.js): deleted the restart menu item Closes DCOS_OSS-4270

Testing
should check podDetail page actions menu.
Trade-offs

Dependencies

Screenshots

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

I'm currently having some problems with my configurations a bit :smile: cannot amend only change the title :disappointed:

No worries! I can do it, would you please add me as a collaborator to your fork?

CI is green we need one more review to merge it :)

:tada: This PR is included in version 2.27.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


cmd/compile: better append of unmodified slices

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

$ go version
go version go1.14 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/Users/s.rabot/go/bin"
GOCACHE="/Users/s.rabot/Library/Caches/go-build"
GOENV="/Users/s.rabot/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/s.rabot/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/z0/rcywz4nn3jg6g96h67c_4xzdw31lvl/T/go-build757609694=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
package main

import (
	"testing"
)

func BenchmarkNoCap(b *testing.B) {
	for i := 0; i < b.N; i++ {
		var slice []string

		slice = append(slice, []string{"a", "q", "w"}...)
		slice = append(slice, []string{"z", "s", "x"}...)
	}
}

func BenchmarkCap(b *testing.B) {
	for i := 0; i < b.N; i++ {
		slice := make([]string, 0, 6)

		slice = append(slice, []string{"a", "q", "w"}...)
		slice = append(slice, []string{"z", "s", "x"}...)
	}
}

func BenchmarkCapNoMagicNumber(b *testing.B) {
	for i := 0; i < b.N; i++ {
		base := []string{"a", "q", "w"}
		fixed := []string{"z", "s", "x"}

		slice := make([]string, 0, len(base)+len(fixed))

		slice = append(slice, base...)
		slice = append(slice, fixed...)
	}
}

func BenchmarkCapNoMagicNumberFixedSlices(b *testing.B) {
	for i := 0; i < b.N; i++ {
		base := [...]string{"a", "q", "w"}
		fixed := [...]string{"z", "s", "x"}

		slice := make([]string, 0, len(base)+len(fixed))

		slice = append(slice, base[:]...)
		slice = append(slice, fixed[:]...)
	}
}

What did you expect to see?
I would have expected BenchmarkCapNoMagicNumber and BenchmarkCapNoMagicNumberFixedSlices to have the same throughput.
What did you see instead?
$ go test -bench . -count=3
goos: darwin
goarch: amd64
BenchmarkNoCap-12                          	 7771320	       142 ns/op
BenchmarkNoCap-12                          	 7877163	       142 ns/op
BenchmarkNoCap-12                          	 7857156	       143 ns/op

BenchmarkCap-12                            	59972187	        20.2 ns/op
BenchmarkCap-12                            	58082768	        23.4 ns/op
BenchmarkCap-12                            	44653009	        25.6 ns/op

BenchmarkCapNoMagicNumber-12               	15412473	        74.8 ns/op
BenchmarkCapNoMagicNumber-12               	14540749	        75.1 ns/op
BenchmarkCapNoMagicNumber-12               	15240964	        74.7 ns/op

BenchmarkCapNoMagicNumberFixedSlices-12    	64831053	        21.0 ns/op
BenchmarkCapNoMagicNumberFixedSlices-12    	64266398	        21.4 ns/op
BenchmarkCapNoMagicNumberFixedSlices-12    	60569865	        22.7 ns/op

@randall77, I think you've got the title backwards. It looks like the sliced array is outperforming the bare slice for some reason.

Sorry, yes.

Slow routine allocates because len(base)+len(fixed) of slices isn't determined at compile time, whereas with arrays the expression is constant?

Or to be more precise, during the escape analysis phase, len(base)+len(fixed) isn't known to be a constant. (Later on, SSA figures it out.)

Since neither base nor fixed are modified, shouldn't the escape analysis catch that len(base)+len(fixed) is a constant ?

So both are optimized to the extent of what they can be or is there room for improvement in the slice case ?

It’s definitely possible for them to be improved. The question—which I can’t answer offhand—is whether it is possible to make a small, local improvement that would cover this case, or whether it would require an overhaul. (There are good independent reasons for an overhaul, but what is needed—eliminating one of the compiler’s intermediate representations—is a huge undertaking.)

Is this a duplicate of #20533?


Project: Document All Modules

The Problem
There are many modules in the ecosystem that lack any useful documentation. I've actually heard someone refer to this as "Perl 6's culture of no documentation."
The Solution
We're about to have many new comers and having a simple way to contribute would be good for the growth of the community.

Same as with the Most Wanted Modules Wiki, we should have one for Documentation Wanted.
Those who are new to the language could study those modules, figure out how they work, document the behaviour, and submit a doc PR.

In summation, I think this can be a great entry-level task for new comers to contribute to, while at the same time eradicating the "no documentation culture".

Someone needs to make something that does something like Pod::Coverage :)

but :+1:

:+1: on making Pod::Coverage :p

Would the idea be to add another badge which is displayed when documentation is present and passes coverage tests?

That could work

There was discussion on IRC of making a list of the documentation status of modules. What would be a good place to put such a list for people to see/contribute to?

I think a wiki-like thing akin to Most Wanted could be ideal.

Since we're soon gonna switch to a Mojolicious-power app instead of static files, I think we could add something easier to use than a wiki hosted else-where. Something like a flag users could click if module's misssing docs.
And at the top of the page we could have some call to action, like "Looking for an easy way to contribute?" and people click on it and get a list of all the modules without docs that they could write up and then unlag the module.

Hi,
This is result of an one hour experiment https://github.com/teodozjan/pod6-coverage
Sample results:
 perl6 Pod/Coverage.pm6
  what isPerl6::Metamodel::ParametricRoleGroupHOW.new ?
  (Logic) Package
   (LacunaCookbuk::Logic::IntelCritic) class is not documented
   (LacunaCookbuk::Logic::Ambassador) class is not documented
   (LacunaCookbuk::Logic::Transporter) class is not documented
   (LacunaCookbuk::Logic::ShipCritic) class is not documented
   (LacunaCookbuk::Logic::Secretary) class is not documented
   (LacunaCookbuk::Logic::OreCritic) class is not documented
   (LacunaCookbuk::Logic::Commander) class is not documented
   (LacunaCookbuk::Logic::PlanMaker) class is not documented
  (Model) Package
   (Structure) Package
    (LacunaCookbuk::Model::Structure::BuildingView) class is not documented

If you like I can go further or someone can fork it

That's great! Yes, please go further :) As stated above, for start, we really just need something that tries to work, not a comprehensive error-prone solution :smile_cat:
:shipit:

Hi, I've tried my best to provide coverage but I see little issue about noninline documentation.
There several ways people do their documentation.
JSON::RPC is documented inside README
DBIsh uses EOF pod for major classes
File::Find uses EOF pod mentioning method as C<>
also someone used headers to document subs/methods
Are we going to support anything? Or just cover some standard?

I think any pod in the main lib or bin dirs would be a good start.

@awwaiid I added option option to check for pod inside provides files https://github.com/teodozjan/pod6-coverage/commit/84517d8859f2603ff5941b7150394beee0336110 what do you think about that?

Hi,
would it be possible to refactor the way it checks and then outputs the results such that it could be used in test file?  e.g. something eventually returns a list of objects with "what", "name","has-pod" attributes then one could do something like:

ok $obj.has-pod, " { $obj.what }  { $obj.name } has pod";

something eventually returns a list of objects

I'd expect the interface to be a single sub call and just prints TAP results. akin to P5's version. Is there a need to manually write the ok?

Yes and "P5's version"  A) uses Pod::Coverage (a separate module) which returns the data as I described and B) uses Test::Builder to generate the results. You appear to be dis-agreeing with something I didn't say.

I thought users will just want to pass meta and get report with maybe boolean result. Now everything changed. Test builder sound good. I can also flatten different @!results and give them through method.
https://github.com/teodozjan/pod6-coverage/commit/d48ef59ab185810071233f09f56a4901c16c0ae7

I've used the P5 Pod::Coverage in the past to do things other than test (perhaps to generate stub entries for the documentation or something.) Splitting the interface, such that there is one part that actually does the checking and another part that uses that to compile the test results allows people to create alternative interfaces (such as e.g. class based testing or something similar.)

@jonathanstowe I wasn't disagreeing but merely asking. Discussing all the merits prior to implementing them produces a cleaner and well-thought-out interface. I don't know why you consistently seem to be taking my attempts to do so as personal attacks.

I was merely suggesting there should be something like:
sub get-results( ... ) {  # returns list of result objects
....
}

sub test-coverage() {
    for get-results() -> $obj {
       ok $obj.has-pod, " { $obj.what }  { $obj.name } has pod";
    }
}

The get-results can even be optionally exported.
I wasn't taking your words in any way except that they seemed to misunderstand what I had typed, but hey it's the weekend and I'm tired so maybe it's me.

A big step would be to try to remove everything that does something like:
Impletement Perl5 Foo::Bar, see p5 documentation.
Or having a way to import these doc easily?

@Skarsnik That's why I propose to check it's completeness not only the presence
@jonathanstowe I modified class to use instance of pod Coverage per ::($str). Public @.results attribute contains anything wrong with package with also are-missing gives boolean result. It should be easy to write TAP support. Only coverage method that creates instance has little complicated arguments.
One thing I really like to have policy if is visible to user -> must be documented. Anyway I distinguish require string, package to check and file where code lies. Maybe someone can rewrite it to be simpler for example with compunitrepo? I still did not invent how to get EOF pod for class/module while it does not have to be Module::Name::Classname.pm6

Looks good :+1:

Finally Pod::Coverage is travis capable. So I made it work but I cannot say it works right.
It can take metafile and check if things in provide section contain any pod. So anypod name felt fine here for me.
It can traverse  into package documentation without thinking how deep is object inside package(maybe should be configurable) and check for .WHY
It reads pod only relying on headers and its level. I'm thnking about looking for NAME SYNOPSIS and METHODS header in future.
It has hacky implementation of TAP. anypod_ok subtest_anypod_ok and coverage_ok subtest_coverage_ok

:+1: awesome!

I think it's time to try it out :)
Nice one :+1:

$ pod-coverage
Could not find Text::CSV in any of:
  file#/home/merijn/.perl6/2015.10-328-g3177463/lib
  inst#/home/merijn/.perl6/2015.10-328-g3177463
  file#/pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6/lib
  file#/pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6/vendor/lib
  file#/pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6/site/lib
  inst#/pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6
  inst#/pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6/vendor
  inst#/pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6/site
  in method coverage at /pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6/site/lib/Pod/Coverage.pm6:83
  in block  at /pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6/site/lib/Pod/Coverage.pm6:59
  in method use-meta at /pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6/site/lib/Pod/Coverage.pm6:57
  in sub MAIN at /pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6/site/bin/pod-coverage:7
  in block <unit> at /pro/3gl/CPAN/rakudobrew/moar-nom/install/share/perl6/site/bin/pod-coverage:6

Exit 1
$ find * -name CSV.pm | xargs ls -l
lrwxrwxrwx 1 merijn users    15 Apr 17  2015 CSV.pm -> lib/Text/CSV.pm
-rw-rw-r-- 1 merijn users 56685 Nov 11 08:17 lib/Text/CSV.pm
-rw-rw-rw- 1 merijn users  1029 Aug  8  2014 sandbox/CSV.pm

@Tux I'm not sure why it requires module to be loadable but is supports only metafile now. And checks only for provides.
https://github.com/teodozjan/pod6-coverage/issues/14
 $ panda look Text::CSV
 $ panda install ./
 $ pod-coverge --anypod
  Str::Text::CSV has no pod

That is weird again. It is Text::CSV, not Str::Text::CSV
$ find * | grep pod
Text-CSV.pod

Should it be renamed to Text::CSV.pod? Should it reside in lib/ ?

The Str problem fixed here https://github.com/teodozjan/pod6-coverage/issues/15
The second one needs talk. Now Pod::Coverage is not reading pod files at all. If it is wrong I can fix but if doubt I use p6doc as reference.  p6doc is getting confused:
      $ p6doc Text::CSV
      No Pod found in /home/kamil/.rakudobrew/moar-nom/install/share/perl6/site/lib/Text/CSV.pm

I think anything that will be accepted by p6doc should be also fine for Pod::Coverage

$ wc -l Text-CSV.pod lib/Text/CSV.pm
  2319 Text-CSV.pod
  1663 lib/Text/CSV.pm

I don't feel for including the pod in the pm. Just tell me how to rename the pod so it will be installed by panda and found by p6doc. I agree that P::C is just to do the same as p6doc regarding finding files

@Tux Try now with lib/Text/CSV.pod

$ ls -l lib/Text/
total 120
-rw-rw-r-- 1 merijn users 56685 Nov 11 08:17 CSV.pm
-rw-rw-r-- 1 merijn users 63066 Nov 17 17:37 CSV.pod
$ pod-coverge --anypod

Correct > pod-coverage --anypod [y|n|E|a]? yes
Could not find Text::CSV in any of:
:

Why does it not work before a panda install?
So,
$ panda install .
:
$ pod-coverage --anypod
===SORRY!=== Error while compiling /pro/3gl/CPAN/Text-CSV6/lib/Text/CSV.pod
Two terms in a row
at /pro/3gl/CPAN/Text-CSV6/lib/Text/CSV.pod:38
------> comma-separated values. An⏏ instance of the Text::CSV class will co
    expecting any of:
        infix
        infix stopper
        postfix
Text::CSV has no pod

Because I didn't have time to separate anypod test from detailed coverage test and I need to put MetaObject into results. I assume for detailed test is obvious that module has to be importable
Anyway you may try something like
PERL6LIB=lib pod-coverage --anypod

Thanks, everyone. I'm closing this Issue with this plan of action to be implemented:
After cleaning up the build script of the new Mojo-app based modules.perl6.org, I'll add @teodozjan's POD Coverage module to the build process to evaluate each dists's doc situation and roll it into a Kwalitee metric (which will be experimental for now).
Once dist info pages are added, a blurb about needing docs might also be added (or the page might list methods that are potentially present, but aren't documented).
And these features will be the springboard for someone to find projects to contribute to by writing docs, as well as for authors to be more diligent with respect to writing docs, since now we'll be measuring that.


Add response Interceptor to ng-file-upload

Hello there,
Is there a way to add a response interceptor like restangular?
The main purpose is to use tokens, and this way I can get i from the server and apply it in the default headers in your Directive, like this:
Upload.upload({
	url: url,
	file: file,
	data: data,
	headers: {
		Token: interceptedToken
	}
})

Use angular transformResponse function.

The problem I want to get the token from the server and apply it in the headers, the equivalent in  Restangular, something like that:
RestangularProvider.setFullResponse(true); // to have a full reponse
Restangular.setResponseInterceptor(function (data, operation, what, url, response) {
  //restangularize data, operation, what, url, response
  var token = response.headers('Token-From-Server');
  RestangularProvider.setDefaultHeaders({ MyToken: token });
  return restangularizedResponse
});

You can use any library or just angular $http to get the header token, and then set it for Upload.upload() calls.


Question about LocalAddress Port with a DNSCRYPT-V2-PROTOCOL resolver

Running dnscrypt-proxy 1.9.5 on Windows 7.
No problem, only a possible optimization question.
I read on \doc\DNSCRYPT-V2-PROTOCOL.txt,

The default port for this protocol should be 443, both for TCP and UDP.

I read on dnscrypt-resolvers.csv, concerning dnscrypt.org-fr,

Properties / Version = 2

Question 1
Does the resolver's version mentioned in dnscrypt-resolvers.csv correspond to the PROTOCOL?
Question 2
If Answer to Question 1 is Yes,  then should I change my LocalAddress Port to 443?
ResolverName dnscrypt.org-fr
LocalAddress <IP_ADDRESS>:53
Include d:\My Data\BLOCKERS\DNSCrypt\all-config.conf

Change <IP_ADDRESS>:53 TO <IP_ADDRESS>:443 ?
I'm not an expert so thanks for taking that into consideration should my two questions be totally insane!

443 is the default port for communications between the proxy and the resolver.
Between clients and the proxy, you should stick to port 53 especially on Windows, that doesn't have any option to change the DNS port.

Merci Frank!
Since I opened this thread, considering the answer must have been most certainly useful for me only, I'll close the door walking backwards with a wide and grateful thanks! (A la royale, en somme!)

The version rather refers to a compatibility level. The initial protocol was not clearly specified and didn't include any protection against amplification attacks. No one is still using this except Cisco. But it doesn't cause any compatibility issues: recent clients can still talk to these servers.
There was then a specification, and additional mechanisms such as the one that requires questions to be as large as the responses. That's what virtually all the resolvers are implementing. Since clients can talk to both these resolvers and to cisco's, the compatibility "version" is the same even if the protocol is not exactly the same. It's compatible.
Another, incompatible encryption system, was specified, and later implemented. This requires both recent versions of the clients, and the latest code for the server. So, that's a different "compatibility version", even though the current dnscrypt-proxy code can handle both versions.

That is an in-depth explanation which brings a naive question (mine) to a pertinent development (yours), likely to find an echo within techies. My pleasure!
I do notice (a simple remark expecting no answer for myself) that in the dnscrypt-resolvers.csv file, dnscrypt.org-fr is the only resolver to be shown as being version=2. I do at least understand this is not related to my LocalAddress port. Pragmatism being often the credo of illiterates I'll stick on that basic philosophy for the time being (I'm afraid it'll last considering I have more years in back of me than in front!)
Thanks again, Frank.


Minor improvements

EngineConfig:
o Let mutators return this to make it possible to chain calls.
mqtt_support:
o Store client_id so it can be logged when connecting.
createEngine:
o Comment out debug logging of URL parsing.
o Better error message when the URL is not set at all.

Merging, is better than before.


[1.5.n] fix the check_userid_exists util function to handle the force pending state

When the guest is in force pending status, check_userid_exists should return true.
(Error scenario:
guest_shutdown
then detach_volume would check whether the guest exist.
if not handled, the detach_volume would return exception.)
Signed-off-by: dyyang<EMAIL_ADDRESS>


Strange behavior of a right-click mouse when workbench.action.zoomin

VSCode Version:
Version: 1.30.2
Commit: 61122f88f0bf01e2ac16bdb9e1bc4571755f5bd8
Date: 2019-01-07T22:49:48.319Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
OS: Linux x64 4.15.0-45-generic

OS Version: Ubuntu 18.04.1 LTS x64 Gnome

Steps to Reproduce:
1.Increase a size of the VSCode window clicking Ctrl+ '='
2.Right click of the mouse button somewhere on a JS code
PFA video file with the exact VSCode behavior.
mouse_right_click-2019-02-03_23.39.41.zip

Does this issue occur when all extensions are disabled?: Yes

@dbaeumer menus is typically Steven

also @joaomoreno  in this case

Dupe #65787


When replacing Nginx Ingress with Emissary mapping, is there an app-root feature?

Hi I am trying to move from nginx to emissary. Here is my nginx ingress. I defined some annotation to rewrite URLs:
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
annotations:
kubernetes.io/ingress.class: default-my-namespace
nginx.ingress.kubernetes.io/app-root: /index.html
nginx.ingress.kubernetes.io/backend-protocol: https
nginx.ingress.kubernetes.io/rewrite-target: /A-sample-url/$1
nginx.ingress.kubernetes.io/use-regex: "true"
name: my-ingress
namespace: my-namespace
spec:
rules:

host: my-host-1
http:
paths:

backend:
service:
name: my-service
port:
number: 443
path: /(.*)
pathType: ImplementationSpecific

My expected behaviour is: https://ambassador.io is routed to https://my-service/index.html, and any subsequent javascripts such as https://ambassador.io/sample.js is routed to https://my-service/sample.js. To replace it with emissary mapping, I have the following:
apiVersion: getambassador.io/v2
kind: Mapping
metadata:
name: my-mapping
namespace: my-namespace
spec:
ambassador_id:

--apiVersion-v3alpha1-only--default
hosts:
my-host-1
prefix: /.*
prefix_regex: true
regex_rewrite:
pattern: '/(.*)'
substitution: '/A-sample-url/\1'
service: https://my-service:443

Can someone please check if my mapping definition file is correct? How can I do an app-root annotation like Nginx in Emissary? I want https://my-service to open https://my-service/index.html.

Hi @ericlinz , if you're on Emissary version 2.x or later, can you check that your Listener is set up correctly?  If you installed with our QuickStart it opens up ports 443 for HTTPS and 80 for HTTP.  To rewrite paths you can follow the examples here, and regex here.  One correction is that prefix: / doesn't work as a catch-all, but prefix: "/.*" with prefix_regex: true does
Also, if you're using ambassador_id pls make sure all your CRDs (Service, Mapping, Listener) are tagged with that same ambassador_id.  Lastly could you join our Slack at a8r.io/slack and we can help further there.

If I understand app-root correctly, I think that can be accomplished with a redirect.


Add OpenNingia's Launchy 2.9.53

Nice!
We can use the bintray API to get the version and by using dl.bintray.com we can extract the hash. 👍
    "checkver": {
        "url": "https://api.bintray.com/packages/openningia/Launchy/LaunchyInstaller",
        "jp": "$.latest_version"
    },
    "autoupdate": {
        "architecture": {
            "64bit": {
                "url": "https://dl.bintray.com/openningia/Launchy/Launchy_Setup_x64-$version.exe"
            },
            "32bit": {
                "url": "https://dl.bintray.com/openningia/Launchy/Launchy_Setup-$version.exe"
            }
        },
        "hash": {
            "url": "$url.sha256"
        }
    }

@r15ch13, thanks. Also, added persist.


turn this repo into a proper npm module

I dream of the day. . .
npm install choochoo-tweets

Investigate rollupjs? https://twitter.com/hungrytrtl/status/1315807338255745025


AUI-2071 Source formatting

Just started reviewing :) :octocat: Sent from GH.

Thanks, pull request merged.


Except doesn't work in Egress ipBlock

What happened:
I'm trying to block egress access from all pods in a particular namespace to everything except the internet. To achieve that I created this yaml:
apiVersion: v1
kind: Namespace
metadata:
  creationTimestamp: null
  name: test-ns
spec: {}
status: {}
---
kind: NetworkPolicy
apiVersion: networking.k8s.io/v1
metadata:
  namespace: test-ns
  name: deny-from-other-namespaces
spec:
  podSelector:
    matchLabels:
  ingress:
    - from:
        - podSelector: {}
---
kind: NetworkPolicy
apiVersion: networking.k8s.io/v1
metadata:
  namespace: test-ns
  name: allow-internet-only
spec:
  podSelector: {}
  policyTypes:
    - Egress
  egress:
    - to:
        - namespaceSelector:
            matchLabels:
              kubernetes.io/metadata.name: kube-system
          podSelector:
            matchLabels:
              k8s-app: kube-dns
      ports:
        - port: 53
          protocol: UDP
        - port: 53
          protocol: TCP
    - to:
        - ipBlock:
            cidr: <IP_ADDRESS>/0
            except:
              - <IP_ADDRESS>/8
              - <IP_ADDRESS>/12
              - <IP_ADDRESS>/16
---
apiVersion: v1
kind: Pod
metadata:
  creationTimestamp: null
  labels:
    run: test
  name: test
  namespace: test-ns
spec:
  containers:
    - args:
        - sleep
        - infinity
      image: alpine
      name: test
      resources: {}
  dnsPolicy: ClusterFirst
  restartPolicy: Always
status: {}

In theory that should block egress traffic to all internal cluster resources except coredns for DNS name resolution and internet access basically. But here is the result of applying that:
$ k -n test-ns exec -ti test -- ash
/ # curl kube-eagle.monitoring.svc.cluster.local:8080/metrics | tail -n 2
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  828k    0  828k    0     0  4596k      0 --:--:-- --:--:-- --:--:-- 4601k
promhttp_metric_handler_requests_total{code="500"} 0
promhttp_metric_handler_requests_total{code="503"} 0
/ # curl ldap.mydomain.com:636 -vvv
*   Trying <IP_ADDRESS>:636...
* Connected to ldap.mydomain.com (<IP_ADDRESS>) port 636

Environment:

Kubernetes version (use kubectl version): Server Version: v1.27.4-eks-2d98532
CNI Version: v1.15.0-eksbuild.2
OS (e.g: cat /etc/os-release):  bottlerocket $ cat /etc/os-release NAME="Amazon Linux" VERSION="2" ID="amzn" ID_LIKE="centos rhel fedora" VERSION_ID="2" PRETTY_NAME="Amazon Linux 2" ANSI_COLOR="0;33" CPE_NAME="cpe:2.3:o:amazon:amazon_linux:2" HOME_URL="https://amazonlinux.com/"
Kernel (e.g. uname -a):  Linux ip-10-1-193-17.ec2.internal 5.15.128 aws/amazon-vpc-cni-k8s#1 SMP Thu Sep 14 21:42:08 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Known issue when except block is specified under catch all entry. Fixed via #58 and will be part of the next patch release. Workaround is specified in one of the PR comments.

@achevuru good to know, thank you. Do you have ETA for the next patch release? :)

@achevuru good to know, thank you. Do you have ETA for the next patch release? :)

And also what is the best way to know when that release has made its way into a release of https://github.com/aws/amazon-vpc-cni-k8s?

We are targeting Oct mid but will update if something changes. We will update the new image tag  in amazon-vpc-cni-k8s with and release notes will be updated here - https://github.com/aws/aws-network-policy-agent/releases

Closing now that VPC CNI v1.15.1 is released on GitHub: https://github.com/aws/amazon-vpc-cni-k8s/releases/tag/v1.15.1. This release contains the Network Policy agent v1.0.4 image tag


Added PostgreSQLStringModel

Hey @gperdomor, you just merged a pull request, have a coin!
You now have 1 coins.

Thanks!


[thci] enhance THCI for 1.2 Certification

This commit enhances THCI for 1.2 Certification:

OpenThread.py: 1.2 non-BR
OpenThread_BR.py: 1.2 BR and Host (otbr-agent solution)

TODO:

[ ] TestHarness regression

Hi @simonlingoogle - I'm curious about the status of this PR.  Any thoughts on when this will be merged.  Also, do you anticipate any other changes to things like log messages or CLI for integration with the TH?

Hi @simonlingoogle - I'm curious about the status of this PR. Any thoughts on when this will be merged. Also, do you anticipate any other changes to things like log messages or CLI for integration with the TH?

Hi @romacdon , thanks for asking.
This PR will be ready for merging once the LGTM alerts are resolved.
The THCI is actually tested on TH, and works for TH1.2, even though many TCs are still failing (mostly fail not because of THCI).
I have fixed most THCI issues I found when running TH, which makes this PR a good step towards a full workable THCI for TH1.2.

Thanks @simonlingoogle for all the work on these changes!


Added support for compound key

See: Issue #110
Changes key_column: str parameter to key_columns: Tuple[str, ...], to allow diffing using multiple key columns.
Also changes the type of min_key and max_key similarly to a tuple.
This breaks backward compatibility for TableSegment, so I think we should bump the version to 0.3

We decided this feature isn't useful for now.


KIP-853 - DescribeQuorum response now carries controller listener endpoint information

(My initial analysis on was flawed, I've removed the triage label and rewrote the description).
Is your feature request related to a problem? Please describe.
As of KIP-853, released as part of Kafka 3.9, DescribeQuorum response (v2) carries ~node~  controller listener endpoint information.
Kroxylicious doesn't currently proxy controllers (#1647), so this means clients of the proxy will see the controllers of the upstream in the DescribeQuorum response.
Describe the solution you'd like
~Apply the established pattern used by BrokerAddressFilter to the DescribeQuorum RPC response and add supporting test-cases.~
Describe alternatives you've considered
We could decide not to do this work and instead alter the ApiVersions RPC response to remove DescribeQuorum v2.  However, the amount of work is similar to doing the work for real.
Additional context
Add any other context or screenshots about the feature request here.

I'd like us to consider #1647 before reconsidering this issue.  If we decide #1647 isn't a priority, we will want to consider downgrading the DescribeQuorum RPC to avoid giving the clients controller listener endpoints of the upstream cluster.

We are not interested in proxying the controllers for now.
We will downgrade the DescribeQuorum response to avoid sending back the controller node list.


Enable transfer acceleration from amplify s3 configuration

Is your feature request related to a problem? Please describe.
Yes, This feature is related to an issue.
I am uploading a video to an S3 bucket using the amplify-swift SDK. I have configured the project with amplify-CLI.
Video is uploaded to the S3 bucket but the upload speed is very slow.
Even though I have enabled transfer acceleration to the S3 bucket, the speed is still very slow.
Describe the solution you'd like
There should be a setting into amplify-SDK like it's present in AWS-s3 SDK. For example in AWS-s3 SDK.
let transferUtilityConfiguration = AWSS3TransferUtilityConfiguration()        transferUtilityConfiguration.isAccelerateModeEnabled = true
Describe alternatives you've considered
You can enable this default into amplify-swift SDK.
Is the feature request related to any of the existing Amplify categories?
Storage
Additional context
As we are moving from AWS-S3 SDK to the amplify-swift library in our existing live project, we require this feature or the highest possible resolution.

Hey @vish7, we released new versions that address the slow upload issue.
Here are the versions where this fix is included:
Amplify Swift v2: https://github.com/aws-amplify/amplify-swift/releases/tag/2.9.1
Amplify iOS v1: https://github.com/aws-amplify/amplify-swift/releases/tag/1.30.0
*Note: If you're using v1, make sure that the AWS SDK for iOS version used is at least 2.31.1. If it's not, you'll need to take the following steps depending on which package manager you're using:
SPM: Xcode > File > Packages > Update to Latest Package Versions
CocoaPods: Delete you're Podfile.lock (if you currently have one) + pod install --repo-update

Closing this due to no activity. The issue of slow upload was resolved as per the previous comment. Please reach us if you are still facing any issue.