Improper JSON serialization of Time values

I was debugging some mysterious "400 Bad Request" errors from msgraph-sdk-go and found that this chunk of code:
	credential := msgraph_models.NewPasswordCredential()
	credential.SetDisplayName(&displayName)
	credential.SetEndDateTime(&endDateTime)

	requestBody := msgraph_apps_addpw.NewPasswordCredentialRequestBody()
	requestBody.SetPasswordCredential(credential)
	options := &msgraph_apps_addpw.AddPasswordRequestBuilderPostOptions{
		Body: requestBody,
	}
	result, err := graph.ApplicationsById(id).AddPassword().Post(options)

was resulting in this request body:
{"passwordCredential":{"displayName":"test","endDateTime":2023-04-13 14:35:55.527758542 -0400 EDT}}

and this OData error:
{
    "error": {
        "code": "BadRequest",
        "message":"Unable to read JSON request payload. Please ensure Content-Type header is set and payload is of valid JSON format."
        ...
    }
}

The format of endDateTime is wrong.
I believe DateTimeOffset values must be in ISO 8601-1:2019 format, at least according to .NET docs.
{"passwordCredential":{"displayName":"test","endDateTime":2023-04-13T14:35:55-04:00}}

The current code just converts to a string.

Hi @mbarnes ,
Thanks for reaching out and reporting this.
You are right, it should follow the Iso 8601 format according to the docs https://docs.microsoft.com/en-us/graph/api/application-addpassword?view=graph-rest-1.0&tabs=http.
Also, the fact that double quotes are missing is another defect. All the write date/time/duration methods should call write string and not write raw.
Would you be willing to submit a pull request?

Sure (and I know I still owe you another one).

Thanks a lot! If you could prioritize this one over the other one please? And don't hesitate to let us know if you need help or to hand them back!


SQLite EF Core Database Provider Limitations - examples

Hi,
Why are there no examples for UInt64, DateTimeOffset and TimeSpan only for Decimal
Is it possible to share an example for all data types?
modelBuilder.Entity<MyEntity>()
    .Property(e => e.DecimalProperty)
    .HasConversion<double>();

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

ID: dbc4d900-28a8-9c04-2443-7407f15a684f
Version Independent ID: 545bb0b5-f15b-584e-cef7-7283a0e0729f
Content: SQLite Database Provider - Limitations - EF Core
Content Source: entity-framework/core/providers/sqlite/limitations.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @bricelam
Microsoft Alias: avickers

@suatsuphi Examples showing what, exactly?

@suatsuphi Examples showing what, exactly?

What do I need to do for UInt64, DateTimeOffset and TimeSpan ?
there is an example for DecimalProperty
modelBuilder.Entity<MyEntity>()
    .Property(e => e.DecimalProperty)
    .HasConversion<double>();

@suatsuphi Right, but how are you planning on using those data types? What properties are important in the resulting database? For example, you could easily convert a uint64 to an int64, but it would have different semantics in the database. You could instead convert to a string, but that would again have other implications. There isn't any one right answer here, or even any common patterns, so it's really going to depend on what your goals are.

@suatsuphi For unint64, there is no conversion that will work in all situations. SQLite simply doesn't support it.
For DateTimeOffset, the doc already says:

Instead of DateTimeOffset, we recommend using DateTime values. When handling multiple time zones, we recommend converting the values to UTC before saving and then converting back to the appropriate time zone.

Again, there is no conversion that will "just-work".
(Also, note that, as the doc says, the conversion of decimal to double also doesn't work in all cases.)

thank you


WIP : Upgrade vendor versions

Questions
Answers

Description?
Upgrade vendor version via composer.json

Type?
improvement

BC breaks?
yes

Deprecations?
no

Fixed ticket?
no

How to test?
Check every commands

Possible impacts?
Some command can fail (it seems not)

@Amoifr is this still a WIP?

Thank you @Amoifr


Please make sure that the argument PdfToken(default) at index [5] is available in the AppModule context...

Hi there! im trying to implement your library by following your instructions but i'm getting this error:
[Nest] 21182   - 06/19/2020, 11:24:57 PM   [ExceptionHandler] Nest can't resolve dependencies of the AppService (ProfilesService, UsersService, MyLogger, RulesService, ConfigService, ?). Please make sure that the argument PdfToken(default) at index [5] is available in the AppModule context.

Potential solutions:
- If PdfToken(default) is a provider, is it part of the current AppModule?
- If PdfToken(default) is exported from a separate @Module, is that module imported within AppModule?
  @Module({
    imports: [ /* the Module containing PdfToken(default) */ ]
  })

Is there anything missing on the exports/imports side?
thank you

Update:
this only happens when using:
    PDFModule.registerAsync({
      useFactory: (): PDFModuleOptions => ({
        view: {
          root: './templates',
          engine: 'ejs',
        },
      }),
    }),

It works fine when using:
    PDFModule.register({
      view: {
        root: './templates',
        engine: 'ejs',
      },
    }),

Hi @ecorona, apologies for the late response.
This bug has been fixed by #5 . Please update to version 1.0.4 of the package.

Hi, I am using the latest version "^1.0.4", and  I'm getting the same error describe above:
[```
NestWinston] Error     2020-11-12 12:28:34 [ExceptionHandler] Nest can't resolve dependencies of the PdfService (?). Please make sure that the argument PdfToken(a) at index [0] is available in the MyAppModule context.
Potential solutions:

If PdfToken(a) is a provider, is it part of the current MyAppModule?
If PdfToken(a) is exported from a separate @Module, is that module imported within MyAppModule?
@Module({
imports: [ /* the Module containing PdfToken(a) */ ]
})

AppModule
...
    PDFModule.register({
      view: {
        root: '/commons/template',
        engine: 'pug',
      },
    }),
...
export class AppModule {

  static port: number | string;

  constructor(private readonly _configService: AppConfigService) {
    AppModule.port = _configService.get(Configuration.PORT);
  }
}

LiquidacionesModule
@Module({
    imports: [
        MongooseModule.forFeature([
            { name: Liquidacion.name, schema: LiquidacionSchema },
            { name: Hes.name, schema: HesSchema },
        ])
    ],
    controllers: [LiquidacionesController],
    providers: [LiquidacionesService, HesService, ReportService, PdfService, Logger],
    exports: [LiquidacionesService]
})
export class LiquidacionesModule {}

I move config from AppModule to LiquidacionesModule and it works

I think you just have to make your config module global to get it to work that way.


BUG - Sections like Contribute + Contact don't have a HTML title

@StefanVinica Contribute and Contact don't have a title that is recognized by search engines - can you please look into this?

FWIW I believe this applies to other pages as well.
Without knowing how the backend is configured, I suspect it might be the template we use for provisioning pages is the root cause.

@andrejhristovski Please be sure to look into this because if it is a template or setting - it could be a very minor change.  Please feel free to comment on this issue when you have a fix implemented so that we can QA it.

@andrejhristovski Contribute page still does not have a title.

done QA


Add ability to specify which import map to patch

Hello,
we are happy users of import-map-deployer. We have it configured to update an import map hosted in S3 bucket. Url is like s3://<something>/prod/import-map.json.
We are going to have a brand new product with its own import map. However duplicating all our AWS infrastructure for the new product seems to be an overkill. Instead, we'd like to make import-map-deployer to be able to update s3://<something>/prod/{productName}/import-map.json where productName we will send as in a request body for PATCH/import-map.json endpoint.
Is there any reason import-map-deployer does not currently allow to specify import map to update during PATCH operation?
I am ready to open a pull request implementing that feature, but first wanted to make sure it conforms to the vision of this project.
Thanks!
CC @joeldenning @TheMcMurder

Knowing what I know about the Adobe Workfront infrastructure why doesn't using the env flag work for you?
Did y'all change the implementation in a way that makes it so you can't use that flag to change a specific import map?

Env flag will work, but it will require making changes in import-map-deployer config for each new product.
To be honest I discovered that flag after opening this issue. Making change for each new product is an acceptable tradeoff.
However, ability to specify import map during PATCH request will allow to make everything dynamic and that way we will not hardcode names of import maps. I wonder why import-map-deployer was not built that way and requires to list all import maps you work with in its config.

Thank makes sense. Thanks!


response.context is not a thing (?)

https://github.com/JohnGWebDev/django-htmx-refresh/blob/master/django_htmx_refresh/middleware.py#L18-L21
I believe this should use the attributes context_data instead? That is what is done in the original blog post as well.

Thank you @benjamin-kirkbride for the tip. I ran into the same problem.


until we can get secrets to work in openshift 0.4.x lets use an env var to define the namespace to watch for builds

until we can get secrets to work in openshift 0.4.x lets use an env var to define the namespace to watch for builds

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/332/
Test PASSed.


Zhaoyu/port conflict after refactor

change the hdfs name node and data node ports to avoid conflict with k8s.

Coverage decreased (-16.2%) to 59.918% when pulling 199013eb04aa37d185f75f207f2db9536f5d4f8e on zhaoyu/port-conflict-after-refactor into f048ddcec68c43410b5c740cdd144b46bf74abe6 on folder-refactor.


It's not working

Do I need to install extra dependencies or other Atom packages?

Must be drunk... Apparently this is a formatter not a linter. Sorry!


Support bareNumber option for integer and number types

Overview
See http://specs.frictionlessdata.io/table-schema/#number. Also we should update current behavior on currencies etc.

Just to be entirely clear: what exactly do you mean by "update current behavior on currencies etc"?

All done. Thanks for catching this missing feature! Leaving the issue opened until we clarify the description.

@danielfireman
There was a change in specs that replaces number.currency by number.bareNumber. E.g. for JavaScript  it leads to - https://github.com/frictionlessdata/tableschema-js/commit/8c399e25de98cf3463c54879e1afd79e78a7d684

Cool. That is exactly how I implemented it (followed the specs). Thanks for the heads up!
Marking this issue as finished.

:+1:


Some error occurred 'request cancelled while waiting for connection'

Describe the bug
Error message pop up upon opening the app
Screenshots

This problem took more than 3 days and I don't know the reason, even though my internet speed is high

My account has 5$ balance


Add the CSV hash table in Hash layer and fix a bug.

delete the Lambda sublayer in LocalActivationUnit Layer class

add vocabulary_path in the SparseFeat to support the csv HashTable functionality

update docs and add examples in doc

Remove trailing whitespace

this PR will be merged into release branch, and so you need merge into master branch when this PR is merged. @shenweichen

I added some pytest code of hash vocabulary_path parameter

@shenweichen I used your layer_test function in pytest, and when that run the line model.predict and throws the exception  Failed precondition: Table not initialized., I think the pytest environment is the tfv1.

It's strange, I have tested successfully in my local linux, but it always failed when using CI action with exception: Failed precondition: Table not initialized..

@shenweichen I used your layer_test function in pytest, and when that run the line model.predict and throws the exception  Failed precondition: Table not initialized., I think the pytest environment is the tfv1.

i think you need to  test it in your local machine in both tf1 and tf2

I have modified some code to fix a bug An op outside of the function building code is being passed a "Graph" tensor in PostionalEncoding Layer, which substituted the tf.concat with np.zeros when zero_pad=True, and remove the tf.compat.v1.disable_eager_execution() in sequence_test.py, so all the unit tests are successfully.

committers are not allowed to merge their own branches to release, we will review your pull request soon.

OK


Grammarly: losing line breaks when returning from Grammarly

Grammarly injects a link in one's documents written in web form. Clicking will take you to a Grammarly page where it will suggest grammar and spelling corrections. This works fine with most tools and sites.
However, with Classeur, when returning from Grammarly, I find that all the carriage returns are missing and am left with one big line of text.
I don't know if the issue is with Grammaly or Classeur, but I've never seen the problem before.

Dear @johnecobo,
Could you guide me through the process in order to reproduce the problem?
Thanks!
~Robert

Steps to re-create the problem:

sign up for Grammarly (I have a paid account)
obtain the Grammarly app, "Grammarly for Chrome"
use Classeur in Chrome writing some Markdown, make grammar and spelling mistakes
a small green Grammarly button will appear in the Classeur editor, click it
your document will be loaded into a pop-up Grammarly editor with Grammarly grammar and spelling suggestions.
exit Grammarly with the button on the top-left
I find (usually) that I have now lost all carriage returns and the markdown document is a mess, all on one long line.
---------------------------------------------------------this document:

this is a test

of the American
broadcast
system

why
oh, why

I could writee forever and ever without failing I suppose but it might be a long long long time.
--------------------------------------------------------became this:
this is a test 1. of the American 1. broadcast 1. system * why * oh, why I could writee forever and ever without failing I suppose but it might be a long long long time.

Thanks @johnecobo!
I can confirm the issue in this screencast:
https://cloudup.com/cwabCCJB8-s
@benweet do you have any idea what is causing the issue?
Warmly,
~Robert

Reproduced!
Investigating...

I'm afraid the Grammarly app is too intrusive. There are multiple problems with it. The biggest one: it breaks the whole layout with some weird scrollbars.
Sorry we can support that.
By the way, @orschiro your link doesn't work :)

That is too bad. Being able to dip into Grammarly is something I find extremely useful as it saves the first cut of editing and proofreading.

Hi both,
I will get in touch with Grammarly to make them aware of the problem.
Maybe they have an idea!
~Robert


Add support for constants for input values in Recipe Types

In order to improve reusability of Job Types, Recipes should support specifying constants for values of Job Type inputs.
Example: Job Type "Send E-mail" that has inputs "sender" and "recipient". Scale currently requires creating separate Job Types and Docker images for each unique configuration. The Recipe Type should support user defined values for these inputs.

Chris:  Good thinking!
I was talking to Derick this morning that having a way to attach a "parameter" object to a recipe (or job) like we assign a workspace would be very powerful.  For example: for strike ingests may use one "parameter" set whereas a experimental, background, add-hoc or reprocess could use a completely different "parameter" set.  In actuality might be as simple as a JSON with tuples.  Things we could put in might be database names and passwords for ingesters, or file locations for publish-type jobs, or maybe control options for processing type jobs... We should be careful to make sure that one job can't see another jobs parameters though. I.e. JOB A, shouldn't be able to see or know JOB B's database password for example... thats bad juju.  But there might be room for recipe-level globals that any job might be able to see.  We could build a "parameter" editor that goes with each recipe that let's you build up name, value pairs, then assign them as job-level, or recipe level... maybe even a switch to tell scale to make these environment variables inside the docker container, or supply on command line, or add the JSON as a file in the docker container...  assuming we can figure out how to do that in docker.

Closing all issues > 2 years old.


Feat/delete package

Adding the delete package to delete run / experiment
TODO:

[x] Connect the GCS Package to the delete package, so the package handle the delete artifact
[x] Error Handling
[x] Unit Test

Feature Workflow:

Delete Experiment
~Soft Delete Experiment →~ Soft Delete All Run Related to the experiment id → Delete Artifact (All Run) from GCS

Delete Run
Soft Delete Run → Delete Artifact from GCS

Thanks @Zenovore for this PR and for addressing all the comments! Everything looks good and ready to go! 🚀 I've just added one tiny comment but it'll take a few seconds to fix that. Also, you'll need to rebase this branch on the main branch of your fork (after you've synced it with this repo's main branch) because I've just added new commits to it. Once you're done, feel free to merge this PR! 🚀🚀🚀🚀

Everything looks good! Thanks for addressing the comments again! :D 🚀🚀🚀


Chat Search

Restore chat search box

use ?query={encodeQuery(queryString)} with shallow link as in search,
if it fails to refresh then do the same workaround for now, both should follow exact implementation patter
use supabase server client

depends on #166

@AndlerRL Is this up?  can I get more info? this is the search in the chat page right ?

@AndlerRL Is this up?  can I get more info? this is the search in the chat page right ?

Yes, it is up @sheriffjimoh . This is for search functionalities on alpha.masterbots.ai for this we want to enhance the search to be SSR instead of client side (which is the current version of MasterBots.ai). With this task we want to restore, it seems the queries are not optimized to give a good result. The search consists of keywords on messages and it is failing because it is not finding the role of each message on thread. @gaboesquivel may have more info about it, he has a initial version for search, but the issue right now is that the DB doesn't have data on alpha version.
Check the main issue related to this, there is a little more on context.

Restore chat search box

use ?query={encodeQuery(queryString)} with shallow link as in search,
if it fails to refresh then do the same workaround for now, both should follow exact implementation patter
use supabase server client

depends on #166

New approach should use SSR with supabase? Correct?

@sheriffjimoh — Yes, we have supabase backend set up. main and develop are hearing from genql (like usual) but the rest of the branches use supabase more directly.

@AndlerRL another one from a different branch than the current one. But this one is co-related to the search improvement we can do in our current path.
We should close this one. We only need to create one issue for Searching improvements in our current branch and path

Yes, we have to update the search requirements indication. Now the search will go through the ICL vectoring instead of other strategies. I will be closing as not planned, the ICL by product design will handle not only the search bar but also search of examples for new chats inputs.

 Brandon Development Support - October 2024


Remove the need for PlaceholderBackgroundColor and making sure no

"superfluous" placeholder is added at the end
Hi,
I was struggling with the need to define the PlaceholderBackgroundColor and making sure the height of the MarkDownView had the correct height - when I hit me....
I removed the need for defining a PlaceholderBackgroundColor (Breaking Change) as defining a BoxView - with an active BackgroundColor is not needed.
Also made sure that no extra "Placeholder" is outputted in the end after everything else is rendered.
Please note to make the "fix" more apparent I've changed the background color of the sample plage to "beige".
Attached are some images

Best Regards,
Rene´

Hi Rene´
Thank you for participating and taking care about this spacer placeholder.
We merged you pull request.
Best regards.


add: Multiple Linear Rgeression

Describe the algorithm/code
Multiple linear regression attempts to model the relationship between two or more features and a response by fitting a linear equation to the observed data. Please assign me this under hacktoberfest tag
Do you want to work on this feature?

[X] I am willing to implement this feature.

Hi @adi271001! 👋🏻
You can start working on this issue! Do let me know if you need any help. ✨

created pull request and added codes


Extending the tour service leads to unregistered anchor ids

Hi,
in my Angular Application I wrote a new Service which extends the TourService so that I can add some additional methods I need.
Extending the service does not work as expected tough. Using the inherited methods like this.tourService.initialize() or this.tourService.startAt() works but in the end the service always fails with: Can't attach to unregistered anchor with id app-home in the showStep() method because this.anchors seems to be empty.
During debugging I realized that somehow the information about registered anchor IDs is missing.
I set a a minimal example using your lazy-loaded-tour example which you can find in this repo. It is also setup for debugging with VSC if this helps.
Is this a bug or am I doing something wrong regarding the inheritance?

The TourService class is not supposed to be extended. You have to use dependency injection like demonstrated here.


[GEP-22] Change control plane migration flow to fetch relevant shoot state ad-hoc and push it to the ShootState resource

How to categorize this PR?

/area scalability
/kind enhancement
What this PR does / why we need it:
This PR changes the Shoot control plane migration flow such that the relevant shoot state is collected ad-hoc (after extension resources have been migrated) and pushed to the respective ShootState resource in the garden cluster.
Note that both gardenlet's shootstate-extensions and shootstate-secret controllers are still active and running as well. In the future, they will be dropped entirely from the codebase when periodic ShootState backups have been implemented for Shoots running on unmanaged Seeds.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/8073
Special notes for your reviewer:
/cc @timebertt
Release note:

NONE

/assign

/assign


feat: update bazel cache path

update bazel cache path

/hold
wait replay test passed  https://do.pingcap.net/jenkins/blue/organizations/jenkins/pingcap%2Ftidb%2Frelease-7.5%2Fghpr_build/detail/ghpr_build/36/pipeline


fix: Delete src/actions/mparticle directory

Removing mParticle action as this integration is no longer supported by mParticle.
Please also confirm if/what additional action must be taken in order to remove this action from Looker's UIs.

@jakeaustin In order contribute to Google open source projects you need to submit a Google Contributor License Agreement. You can do that here: https://cla.developers.google.com/. Thanks.

@jakeaustin In order contribute to Google open source projects you need to submit a Google Contributor License Agreement. You can do that here: https://cla.developers.google.com/. Thanks.

@drstrangelooker I will see if I can sign the agreement on behalf of my employer.
This action was originally developed by a third party and has been deprecated by mParticle - we are seeking to remove it from Looker's Action hub to avoid any client confusion. Is there a team within Looker that I can coordinate with on this?

This action was originally developed by a third party and has been deprecated by mParticle - we are seeking to remove it from Looker's Action hub to avoid any client confusion. Is there a team within Looker that I can coordinate with on this?

@phillipperalez would you handle this?

I've checked usages and this is fine to delete. I can this to own and remove


Update cli doctor command response

We're in the process of moving from Azure AD to Microsoft Entra ID.
One thing we should do is update the response of cli doctor
cliAadAppId --> cliEntraAppId
cliAadAppTenant --> cliEntraAppTenant

Since we have v8 branch we may now start working on it!
Let's get this 🚢
and Let's put CLI to the 🚀🌙

Can I work on it?


add some missing command options

Description of the PR
Add some missing command options. the descriptions are copied from the docs in the command class here: https://github.com/dirigeants/klasa/blob/master/src/lib/structures/Command.js#L15
Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)

Semver Classification

[X] This PR only includes documentation or non-code changes.
[ ] This PR fixes a bug and does not change the (intended) framework interface.
[ ] This PR adds methods or properties to the framework interface.
[ ] This PR removes or renames methods or properties in the framework interface.

Command options are also shown in these 3 places, and all 3 have differences, should we update all 3 to be consistent?
https://klasa.js.org/#/docs/main/master/typedef/CommandOptions
https://github.com/dirigeants/klasa/blob/master/src/lib/structures/Command.js#L15
https://klasa.js.org/#/docs/main/master/Piece Basics/CreatingCommands

Command structure and CreatingCommands should be updated, CommandOptions is generated thru JSDoc


Correction for raw affiliation LTSI – INSERM UMR 1099, University of Rennes 1, F35000, Rennes, France

Correction needed for raw affiliation LTSI – INSERM UMR 1099, University of Rennes 1, F35000, Rennes, France
raw_affiliation_name: LTSI – INSERM UMR 1099, University of Rennes 1, F35000, Rennes, France
new_rors: 015m7wh34;02vjkv261;01f1amm71
previous_rors: 015m7wh34;02vjkv261
works_examples: W4382701231
contact: 2cd5de67ee5f367aac016a03cf94f2cd:b04796914325e2b3e49d95fe3bcae0fe @ univ-rennes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Wrong project version in 2.4.0 release

I noticed that in the release 2.4.0 the cmake project version is 2.3.1
https://github.com/marzer/tomlplusplus/blob/1baad21734d6d28a4efeb9ee3288c63487519bdb/CMakeLists.txt#L3
This is actually a problem when cmake find_package method is used in a consumer library.
Indeed
find_package(tomlplusplus 2.4.0 REQUIRED)

will fail with an error similar to this one
  Could not find a configuration file for package "tomlplusplus" that is
  compatible with requested version "2.4.0".

  The following configuration files were considered but not accepted:

    /usr/share/miniconda/envs/test/lib/cmake/tomlplusplus/tomlplusplusConfig.cmake, version: 2.3.1

On the latest master https://github.com/marzer/tomlplusplus/commit/4f21332bdd7555bbf9add1aafe82909a2f8aa52b the version is correct

Ah, thanks for reporting this @GiulioRomualdi. There have been a few other minor useful improvements since 2.4.0 too so perhaps it's time for me to publish a new release.

Released in v2.5.0 :)

Thanks @marzer !


Can I use this library?

Hi, Can I use this library in my application that I want to publish on
google play? Is there an obstacle to this? (I ask you in terms of licence)

Yes, sure. I built it for that purpose ;). It is Apache 2 licensed so you can use freely (check license restrictions here https://tldrlegal.com/license/apache-license-2.0-(apache-2.0))

thank you very much👍 I wanna ask one small question. When I use your sample codes, I see hours instead of dates like on youtube video. How can I change hours to dates?

(I mean, 01:00, 02:00.... after 02 SEP) (It should be 03 SEP, 04 SEP)

Then you should return a date with hour, minute, second and millisecond equals to 0. It your dataset contains intraday times, the library try to adjust the tick labels to display the proper time.

If you have timestamps in a database you should aggregate your data into a day-truncated timestamp. You can do that with a query like:
select cast(strftime('%s', date(my_timestamp_field_name/1000,'unixepoch','localtime'), 'utc') AS INT)*1000 my_timestamp_field_name, sum(my_event_count_field_name) my_event_count_field_name
from mytable
where ....
) group by (cast(strftime('%s', date(my_timestamp_field_name/1000,'unixepoch','localtime'), 'utc') AS INT)*1000)

where my_timestamp_field_name is the field containing the timestamps events, and my_event_count_field_name the field to aggregate.

I just use your sample codes (https://github.com/jruesga/timeline-chart-view/tree/master/sample) I haven't done extra things for now. Can you share a link or codes because I am a beginner and I couldn't solve with these codes. (sorry)

If you want only hours in the sample app, just change the HOUR_OF_DAY by DAY_OF_YEAR (here is the sample activity modified https://paste.ruesga.com/view/2e3c9005). This changes are unrelated to the library, but justto Java Calendar. As said before, if you want to show only dates (and not hour), you should truncated dates to the library.

Thanks for your effort, it works as I want. (I said that it is small question and I am beginner. I should work harder with java calendar)


Possible bug on the |= operator inside FORCONT loop.

In a vendor script, the vendor receives items and "locks" them (so they can be displayed in his bag but not moved around by anyone). This part goes like this:
[CHARDEF vendor]
(...)
ON=@ReceiveItem
(...)
ATTR |= ATTR_MOVE_NEVER                       <-- THIS PROPERLY SETS THE ATTR
IF (<ARGO.TYPE> == t_container)
FORCONT  10
ATTR |= ATTR_MOVE_NEVER       <-- THIS DOES NOT PROPERLY SET THE ATTR! (maybe a bug?? dunno...)
ENDFOR
ENDIF
Jhobean and burpee told me that this could be a bug and so I changed the unfunctional line to:
ATTR = ATTR_MOVE_NEVER
ATTR = |ATTR_MOVE_NEVER
Both solutions worked fine.
Comment 1: if I try to change other parameters of the container's content, they work fine. For example:
ON=@ReceiveItem
(...)
ATTR |= ATTR_MOVE_NEVER
IF (<ARGO.TYPE> == t_container)
FORCONT  10
COLOR = 0455                            <-- works fine
ENDFOR
ENDIF
Comment 2: this is the "reverse" part of the script, in which the player gets the items from the vendor:
ON=2                                                                       <-- this is a button on a gump
REF1 = <UID.<TAG.CURRENT_ITEM>>
REF1.ATTR &=~ ATTR_MOVE_NEVER              <-- works just fine
IF (<REF1.TYPE> == t_container)
FORCONT  10
ATTR &=~ ATTR_MOVE_NEVER       <-- works just fine
ENDFOR
ENDIF
Our conclusions: it might be a bug on the "|=" operator. Thus, I created this issue. Thanks again to Jhobean and burpee for helping my scripting.

The issue with your script is that you are giving only 1 argument to FORCONT, which expects 2 instead. Check that: https://wiki.spherecommunity.net/index.php?title=Chapter_7#FORCONT
Of course, Sphere should give a warning instead. I'll send a fix for that. In any case, thanks for the issue, it highlighted a missing warning!


Multi-node integration test refactor

Refactoring of the multi node integration tests, into a marketplacetestsuite. Allows us to run many node integration tests on our local machine, which saves quite a lot of time as compared to the distributed tests.
Notable features:

Allows arbitrary number of nodes for clients, providers, and validators.
Allows log files for each started node to be captured (including hardhat), saved in tests/integration/logs/<start_datetime> <suite_name>/<test_name>/<node_role>_<node_idx>.log.
Allows log topics to be specified
Allows simulation of proof failures to be specified
Allows log output to stdout

Two notable changes:

OnChainClock.now was changed so that it always get the time from the last block time. Without this change, the clock was drifting from the actual last block time, which was causing validators to mark proofs as missing when not required, or not mark them missing when required, and providers were sending proofs when the period had not yet ended. Once the change was in, these errors disappeared.
Cancellation of the SaleProving prove loop was removed as it was hanging during integration tests. This is not needed as the state machine will cancel SaleProving.run, of which the proveLoop is a child and should also be cancelled.

Leaving this in draft for now as I'd like to try moving the node config setup into each test, as opposed to being defined at the suite-level.

Closes https://github.com/codex-storage/nim-codex/issues/594

🎉

Closing in favour of https://github.com/codex-storage/nim-codex/pull/662.


Build instructions

I am using folktale In NetSuite SuiteScript 2.0 which is using the Rhino 1.7 JS engine and a modified requireJS module. I obtained a copy of folktale 2.0 from a CDN. 1. Can I be pointed to where I can make my own build so I can use the latest version?  Any help with this would be great thanks
Steps to reproduce
I got the build I am using at https://cdnjs.com/libraries/folktale
Expected behavior
Observed behaviour
(Describe what happened instead)# Environment
(Describe the environment where the problem happens. This usually includes:

OS
JavaScript VM
Folktale version
)

Additional information
(Anything else you feel relevant to the issue)

thanks, very much, does this also minify
Gerald Klein DBA
<EMAIL_ADDRESS><EMAIL_ADDRESS>708-599-0352
Arch, Gentoo I3, Ranger & Vim the coding triple threat.
Linux registered user #548580
Brought to you by the Amish Mafia
On Sat, Oct 13, 2018 at 8:24 AM Quil<EMAIL_ADDRESS>wrote:

The easiest way is probably by using Browserify.
$ npm install folktale@latest browserify
$ ./node_modules/.bin/browserify -r folktale -s folktale -o folktale.js
This will generate a packaged folktale.js file that you can use with AMD
modules, CommonJS modules, or systems that don't support either type of
modules.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/origamitower/folktale/issues/212#issuecomment-429541612,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABnsxFn3CWeSfGAdWvC7j82t71G_Zjt6ks5ukel4gaJpZM4XaaEE
.

Sorry I'm a dope I will work it out thanks again

I created the file and it ran with no errors but when I use it I get the same errors. I am attempting to use "resultToMaybe" and it throws an error "Cannot find function matchWith in object Error: 1. (folktale.min.js$1314923$debugger.user#2639)"  as the locations differ I have included the code block it is pointing to, It is pointing to the return aResult.matchWith statement in
	var resultToMaybe = function resultToMaybe(aResult) {
		return aResult.matchWith({
			Error: function Error(_ref) {
				var _ = _ref.value;
				return Nothing();
			},
			Ok: function Ok(_ref2) {
				var value = _ref2.value;
				return Just(value);
			}
		});
	};

I have loaded the es5 and es6 shims along with babel/polyfill, I appreicate your help with this. This library is helping me solve some annoying gaps in my data flow. thanks
module.exports = resultToMaybe;
},{"../maybe/maybe":74}],31:[function(require,module,exports){
'use strict';

Ok, Apparently it has something to do with how you retrieve "resultToMaybe", I was able to reproduce the error in Node by using object dot syntax so in other words getting to resultToMaybe via folktale.conversions.resultToMaybe. So it isn't the code it's the approach that I have to use for access to different portions of the code. OK thanks, I will work on it and figure it out.


how set programmatically rsb_max and rsb_min?

Environment

Library Version: [2.0.2]

how set rsb_max and rsb_min programmatically? setMinProgress and setMaxProgress does not work

try to use setRange(min, max)


Memory spike when programmatically scroll calendar without animation

I'm using the latest version, 7.1.6.
I created a demo project that calls scrollToSegment(.next) in the didSelectDate delegate method and the memory would keep climbing by about 1MB for every new segment and would only drop after I scroll the calendar on the screen. I have also tried this with scrollToDate and it causes the same problem. There doesn't seem to be any issues when I enable scroll animation.
func calendar(_ calendar: JTAppleCalendarView, willDisplay cell: JTAppleCell, forItemAt date: Date, cellState: CellState, indexPath: IndexPath) {
        let cell = cell as! CustomCell
        cell.label.text = cellState.text
    }

    func calendar(_ calendar: JTAppleCalendarView, cellForItemAt date: Date, cellState: CellState, indexPath: IndexPath) -> JTAppleCell {
        let cell = calendar.dequeueReusableJTAppleCell(withReuseIdentifier: "customCell", for: indexPath) as! CustomCell
        cell.label.text = cellState.text

        return cell
    }

    func configureCalendar(_ calendar: JTAppleCalendarView) -> ConfigurationParameters {
        dateFormatter.dateFormat = "dd mm yyyy"
        let startDate = Date()
        let endDate = dateFormatter.date(from: "25 01 2025")
        let parameters = ConfigurationParameters(startDate: startDate, endDate: endDate!)

        return parameters
    }

    func calendar(_ calendar: JTAppleCalendarView, didSelectDate date: Date, cell: JTAppleCell?, cellState: CellState) {
        //print(date.toFormat("MMMM dd yyyy"))
        //calendar.scrollToDate(date.dateAtStartOf(.month).dateAt(.nextMonth), animateScroll: false)
        calendar.scrollToSegment(.next, animateScroll: false)
    }

I don't know how helpful this screenshot will be, sorry if it's not lol

i guess some more investigation is needed here.
Under the hood for this library, i use UICollectionView code :/
I do not do anything fancy under the hood, so I am not sure as the cause.
Is this issue for device? or on simulator.

closing this issue.
I did not notice this on latest build.
You can chip in your thoughts on this.


orbit/jsonapi support for Queryable interface using "Fancy Filters"

Hi!
First, let me say that I've been following this repo for quite a while and am tremendously impressed and excited by it.
I just saw your reply in #478 about supporting Queryable interface and decided it was a great place to (finally) jump right in.
I'm a maintainer of the Drupal JSON API module and the primary SME for our query/filtering strategy. We're quite proud of it and think it's pretty flexible and robust :)
We tried to formalize the filtering syntax into a spec extension here: https://gist.github.com/e0ipso/efcc4e96ca2aed58e32948e4f70c2460
And it's also pretty well-documented here: https://www.drupal.org/docs/8/modules/json-api/collections-filtering-sorting-and-paginating#filtering
Both those links might be a little outdated, but I would love to work with someone outside the Drupal community to implement a client with support for this extension and gain some traction on it.
I personally think orbit+Drupal is a match made in heaven :)

@gabesullice as you and I both know, JSON:API v1.1 is almost complete, and will bring with it new support for profiles and extensions 🎉
Even though we talk every week these days, it's been ages since we've discussed the potential for using Orbit and Drupal together. Let's talk more if this might still be on the horizon.
I'm going to close this issue, since it's grown a bit stale. But let's definitely discuss more soon and reopen an issue once there's a JSON:API profile for advanced filtering.


Isolation segments <-> Organizations

Created a "shared" Isolation Segment that gets seeded in the database.
Organizations have a list of "allowed Isolation Segments"

Only Admins can manage the allowed list
Space's Isolation Segments must belong to an Organization's allowed list
Org managers can manage Space's Isolation Segment

Organizations can set a 'default' from the allowed list.
 I rebased and pushed to the isolation_segments branch

I fixed most of the issues, but I think the following remain:

Moving the IS model hooks into v3 action(s)
Adding a v2 presenter for ISes (does not block PR, but needs to happen before GA)
Moving one negative spec from IS request spec into controller spec
Need to come to some agreement on response codes - for the most part, I think the 405s are correct.  The one that still sticks is the 501 case, which may need to become something else (405 or 409, possibly?).  Need a verdict on that.  Does this block the PR?

Regarding all the discussion around HTTP status codes. One of our objectives for v3 was to avoid the status code pedantry that I've been gleefully engaging in.
Zach reminded me that in our v3 style guide we have a small list of response codes: https://github.com/cloudfoundry/cc-api-v3-style-guide#response-codes. The idea is to use the response codes somewhat generically, but have descriptive error messages, since those are what most people care about anyway.
Given that, it seems like 422 is appropriate for the 501/405 cases. Thoughts?

@Gerg - I don't think so - I actually think that a 405 is correct for the two cases where I'm defending it (there is an entity identifiable by the URI path for which the verb used will not be accepted ever).  It's not because there is some semantic error (I realize that the scope of what could qualify for a 422 is pretty broad, which is one of the reasons to be careful about using it).  In the case where we had a 501, but you argued for a 405, I think you are probably correct - I think that should be a 405.  Since I'm X-team pairing with CAPI today, we can probably work this out in real time.

@gerg @SocalNick any news on when we might get this merged?

Our pipeline went green Friday EOD, so we should be able to push today.
On Monday, October 24, 2016, Sandy Cash<EMAIL_ADDRESS>wrote:

@gerg https://github.com/gerg @SocalNick https://github.com/SocalNick
any news on when we might get this merged?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cloudfoundry/cloud_controller_ng/pull/685#issuecomment-255735363,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1ddM-ZyIbV5Mgu7h-oEaabP1nH6GWSks5q3K3XgaJpZM4KEhoS
.

@gerg @SocalNick @zrob - it appears that all the commits from this PR are now in master.  Can we close this?


setting width and height

Hey, guys, another request to add to the documentation. This isn't directly related to the code; I was hoping to add it as a useful suggestion for implementation.
When using piece it together with text (e.g. labelling components is a really common one), it can be helpful to set a min-width and min-height (in percent) on the div containing your text, to create a larger object. This makes it easier for the user to "hit" the drop target.  As an example, here are the rules I like to add to each of my little text tags.
.light{
display:flex;
align-items:center;
justify-content: center;
min-width:19%;
min-height:5%;
}

Rather than use Flex and min sizes, can you just use display: inline-block and some padding?

Yes, you can get nearly the same effect, except that flex handles the
spacing nicely. With padding, I need to be sure to pick a larger number for
padding than half of the min height to get the text to centre vertically. I
thought flex was less complicated, and I've had good results testing
cross-browser. Is there a reason to avoid using it?
On Thu, Apr 2, 2015 at 11:54 AM, Ivan Reese<EMAIL_ADDRESS>wrote:

Rather than use Flex and min sizes, can you just use display: inline-block
and some padding?
—
Reply to this email directly or view it on GitHub
https://github.com/cdig/piece-it-together/issues/8#issuecomment-88991017
.

--
Robyn Hoffman
Designer/Animator
CD Industrial Group Inc.
···········································
<EMAIL_ADDRESS><EMAIL_ADDRESS>Ph: 1·866·628·3224
www.cdiginc.com http://www.carldyke.com/

Sorry, I see you had asked me to avoid min sizes as well. No, I don't get the same result doing that; I'm trying to create a div that fits nicely into a box onscreen, and shrinks and grows as the image shrinks or grows on resize.

See the drag and drop from the Centrifugal pumps module (almost at the end) for an example.

I won't have wifi for another month (rural Spain), so I can't check anything in the Dropbox. My thought was just looking for something simpler. Flex and min are both one notch higher into complexity than I'm usually comfortable with, but that's just personal heeby-geebies.

Now that we are using wikis for documentation that goes beyond "how to get started", this is very much the sort of info that belongs in a wiki page. Robyn, would you be okay if I started a wiki page with your pasted style snippet, and closed this issue?

wow - I completely can't even remember typing this, or what was going on.
But yes, feel free to shuffle it to a wiki!
On Wed, Jun 24, 2015 at 5:58 AM, Ivan Reese<EMAIL_ADDRESS>wrote:

Now that we are using wikis for documentation that goes beyond "how to get
started", this is very much the sort of info that belongs in a wiki page.
Robyn, would you be okay if I started a wiki page with your pasted style
snippet, and closed this issue?
—
Reply to this email directly or view it on GitHub
https://github.com/cdig/piece-it-together/issues/8#issuecomment-114843919
.

--
Robyn Hoffman
Designer/Animator
CD Industrial Group Inc.
···········································
<EMAIL_ADDRESS><EMAIL_ADDRESS>Ph: 1·866·628·3224
www.cdiginc.com http://www.carldyke.com/


Tooltip for the hex text box in the color selector window prevents typing

Godot version
4.1.1.stable.mono.official [bd6af8e0e]
System information
Windows 11
Issue description
In the color selection window (such as 'BG Color' in the 'StyleBoxFlat' inspector, if you click into the 'Hex' input field and start typing, but leave your mouse over the box, you can start typing but once the tooltip shows up, all typing is blocked until the tooltip is dismissed.
Steps to reproduce
See issue description.
Minimal reproduction project
N/A

Thank you for reporting! This is a duplicate of:

#79500

Consolidating there


Bug with defaultAccount and accounts when using MetaMask

Current Behavior
The defaultAccount and accounts props look like this in the console:
accounts ["0x496B1e7F4A0abbb93443A35e70E1c3F9613791dC"]
defaultAccount 0x496B1e7F4A0abbb93443A35e70E1c3F9613791dC

Whereas my MetaMask account is 0x4bcb303609F19e71Ab82A3A3393c46BfEa1e44Fc.
Expected Behavior
The accounts and defaultAccount should be the accounts of MetaMask.

Update
It happens only when I first open a plugin page. If I land on a plugin after seeing the home page, it works fine.

Update 2
I suspect that it has to do with updating props downstream. The account in the top-right corner shows up correctly as the account from MetaMask, but the plugin's props (web3, accounts and defaultAccount) all return the local burner account.

Update 3
Thanks to @dmihal and his awesome tech support, I solved this issue by moving the code in componentDidUpdate instead of componentDidMount. I don't think this is a bug per se, but it is something worth mentioning in the README/ dev guide imo.


Local RDP Checker "Socket Connection Failed"

I have a Windows 10 Home and the RDPWrap does not work for me. Plus, every time I use the Local RDP Checker, it just says "Checking..." on the screen, and less than 10 seconds later, it puts "Disconnected." and "Socket Connection Failed." up on the screen.
I've already used everything possible with both the Remote Desktop and RDPWrap settings, and nothing works.
Here's some information:

OS Name	Microsoft Windows 10 Home
Version	10.0.16299 Build 16299
Other OS Description 	Not Available
OS Manufacturer	Microsoft Corporation
System Name	LAPTOP-3BFQDJQ9
System Manufacturer	HP
System Model	HP Pavilion Notebook
System Type	x64-based PC
System SKU	V8L99UAR#ABA
Processor	AMD A4-6210 APU with AMD Radeon R3 Graphics, 1800 Mhz, 4 Core(s), 4 Logical Processor(s)

Do you see Not listening notification? Read README.

Same Issue.
Anyone, please help. Thanks!

Same issue. Please share experience on solving this

Same issue. Please share experience to solve this. Windows 11 Pro 22H2 22621.1413

RDP Wrapper Library v1.6.2
Installer v2.5
Copyright (C) Stas'M Corp. 2017
[*] Notice to user:

By using all or any portion of this software, you are agreeing
to be bound by all the terms and conditions of the license agreement.
To read the license agreement, run the installer with -l parameter.
If you do not agree to any terms of the license agreement,
do not use the software.
[] Installing...
[] Terminal Services version: 10.0.22621.2361
[-] This version of Terminal Services is not supported.
Try running "update.bat" or "RDPWInst -w" to download latest INI file.
If it doesn't help, send your termsrv.dll to project developer for support.
[+] TermService found (pid 2356).
[] No shared services found.
[] Extracting files...
[+] Extracted config -> C:\Program Files\RDP Wrapper\rdpwrap.ini
[+] Extracted rdpw64 -> C:\Program Files\RDP Wrapper\rdpwrap.dll
[] Configuring service library...
[] Checking dependencies...
[] Checking CertPropSvc...
[] Checking SessionEnv...
[*] Terminating service...
[


Fixing Analytics usage

What's this PR do?
Made a silly mistake & referenced the wrong Analytics
How should this be reviewed?
...
Any background context you want to provide?
...
Relevant tickets
Fixes #
Checklist

[ ] Documentation added for new features/changed endpoints.
[ ] Tested on staging.
[ ] Pinged a PM if this is a larger PR that would benefit from some additional testing love
[ ] Post a message in #phoenix if this includes something that causes a rebuild!

Thanks!


Subfleet Fares and Flight Fares Mismatch (Double Fare Issue)

vmsAcars gets the applicable fares from subfleet only. (Not checking the flight fare)
Version
7.0.0-dev+201102.a2de53
To Reproduce

Create a subfleet (ex. A321N-PGT)
Insert some aircrafts to the database and assign them to that subfleet
Create some fares like Economy - Domestic (YD) and Economy - International (YI) , assign prices and costs to them
Return back to subfleet settings , assign both Economy - Domestic (YD) and Economy - International (YI) fares to the subfleet with defining max seat capacity (239 economy seats)
Create two short haul domestic flights with %90 load factor, assign only Economy - Domestic (YD) fare to this flights.

Setup complete, continue on test flight with bid/vmsacars

Bid one of the flights you created
Generate OFP and check load route fuel etc. Export files as required
Start sim, load the aircraft as per flight plan/bid
Run vmsAcars, search for bid, select and load bid/flightplan, select aircraft

Expected behavior
At this point you should be able to see one matching fare and maximum capacity only. So vmsAcars should show Economy - Domestic only (with a max capacity of 239 and the autogenerated pax figure should not exceed this value)
Screenshots

Still open?

I need to re-create the same scenario to see if it is still open ...


Support window/showMessage(Request)

As seen in the ocaml language server (#57), there is great value in language servers telling the user how to fix configuration etc. The language server protocol exposes a notification and a request from the server:
showMessage: show in status bar + allow user to specify getting a message box
showMessageRequest (with actions to respond with?): quick panel?

@tomv564 I am working on a fix to have the actions in a quick panel.
Needed for this LSP server for example: https://scalameta.org/metals/docs/editors/new-editor.html#import-build
Can I issue an WIP PR ?


Update stardoc dependency to newest version.

Thanks Tony!

https://github.com/bazelbuild/bazel-federation/releases/tag/0.0.2
On Tue, Jul 28, 2020 at 1:38 PM Julie<EMAIL_ADDRESS>wrote:

Thanks Tony!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/bazel-federation/pull/119#issuecomment-665177727,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXHHHCAWOF5MHX7HYJLQBTR54EJHANCNFSM4PKYALLA
.


Allow for async processing

This builds on #5 and adds options.async to requests, which internally uses null for the request id. This just means that the JSON-RPC server should not block the client while it waits for an answer, it will simply complete the request asynchronously.
This could be useful where we don't necessarily care about waiting for the response, such as during sign up. Subsequent gets will just silently fail until the background work is done. This is just an idea, we'll have to see if it works in practice.
/cc @stephenmathieson @stevenmiller888

Good idea!

actually can we do this on the per-request basis? we generally use the same rpc client to make requests we do care about AND requests we don't

@stephenmathieson: yeah, this is per-request (you add { async: true } after your params)

@stephenmathieson: yeah, this is per-request (you add { async: true } after your params, this is not part of the constructor)

haha gotcha whoops

test added!

lgtm


A cancellation doesn't work in some cases and app hangs.

I've played a little bit with an issue raised here it seems that without setting NamedPipeClientStream to PipeOptions.Asynchronous after disposing stream it endlessly waits in PipeStream.Read for local Windows connection.
Steps to repro:
create console app (NET71, .NETCORE) on Windows machine with empty Docker.
    public static async Task Main(string[] args)
    {
        var cts = new CancellationTokenSource();

        var _client = new DockerClientConfiguration().CreateClient();
        cts.CancelAfter(3000);
        cts.Token.Register(() => { Console.WriteLine("Cancel happened"); });
        var task = _client.System.MonitorEventsAsync(new ContainerEventsParameters(), new Progress(), cts.Token);
        await task.ConfigureAwait(false);
        Console.WriteLine("finished");
        Console.ReadKey();
    }

    class Progress : IProgress<JSONMessage>
    {
        internal Action<JSONMessage> _onCalled;

        void IProgress<JSONMessage>.Report(JSONMessage value)
        {
            _onCalled(value);
        }
    }

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

I'm running into the same issue.
It looks like microsoft kind of ignores this repo, in the meanwhile does anyone have a good workaround? (Other than just keeping all my tasks alive, which does not sound great.)

Hello @gurustron
I'm going through all unmerged PR, my intention is to create a PR with code compatible with latest Docker API.
There are outdated models in current definition and this is causing automatic tests to fail within the CI/CD pipeline, causing PR to not be even reviewed.
Would you check if MonitorEventsAsync is working? I haven't managed to

Hello @gurustron
I'm going through all unmerged PR, my intention is to create a PR with code compatible with latest Docker API.
There are outdated models in current definition and this is causing automatic tests to fail within the CI/CD pipeline, causing PR to not be even reviewed.
Would you check if MonitorEventsAsync is working? I haven't managed to

@dgvives - Unrelated to this PR but if you are going to do that I would prefer that we just remove tools/specgen anymore and just manually update the models. Docker changed how all of this worked long ago when the broke the branches into EE/CE and made specgen less than helpful. I'm all for updating the models I just think that specgen is not worth the effort anymore. I no longer have the time to really contribute here but I am happy to merge PR's if you want to do this work. Thanks!

@dgvives - Unrelated to this PR but if you are going to do that I would prefer that we just remove tools/specgen anymore and just manually update the models. Docker changed how all of this worked long ago when the broke the branches into EE/CE and made specgen less than helpful. I'm all for updating the models I just think that specgen is not worth the effort anymore. I no longer have the time to really contribute here but I am happy to merge PR's if you want to do this work. Thanks!

Yeah, that is what I’m working into. A client that doesn’t depends on Go anymore.
Sorry if it is taking too long. Life is being hard recently...
My idea is to generate things based on the yaml from the API with Roslyn.
Still working on it...
Get Outlook for iOShttps://aka.ms/o0ukef

From: Justin<EMAIL_ADDRESS>Sent: Monday, January 4, 2021 2:22:30 PM
To: dotnet/Docker.DotNet<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [dotnet/Docker.DotNet] A cancellation doesn't work in some cases and app hangs. (#351)
@dgviveshttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdgvives&data=04|01||7c614e6a5d4c45aacd4708d8b0d55193|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453777528915147|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=9XE%2BVdOwqDkfdkt3Uy9j8QRIt6G6kZBp%2F%2BcfDiue0mo%3D&reserved=0 - Unrelated to this PR but if you are going to do that I would prefer that we just remove tools/specgen anymore and just manually update the models. Docker changed how all of this worked long ago when the broke the branches into EE/CE and made specgen less than helpful. I'm all for updating the models I just think that specgen is not worth the effort anymore. I no longer have the time to really contribute here but I am happy to merge PR's if you want to do this work. Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2FDocker.DotNet%2Fpull%2F351%23issuecomment-754104913&data=04|01||7c614e6a5d4c45aacd4708d8b0d55193|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453777528925141|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=zVYArtXkkERQ%2F4FsU%2BO9T%2BFmips0WI3pYo91UDdxXLY%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABD64OE65QJUOXSCE4L6J7LSYH2NNANCNFSM4F3UFFHA&data=04|01||7c614e6a5d4c45aacd4708d8b0d55193|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453777528925141|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=CAkIgGoTi5aTFgXC81MkVzJrQw5CRJGJzRHbuHfdygA%3D&reserved=0.

Yeah, that is what I’m working into. A client that doesn’t depends on Go anymore.
Sorry if it is taking too long. Life is being hard recently...
My idea is to generate things based on the yaml from the API with Roslyn.
Still working on it...
Get Outlook for iOShttps://aka.ms/o0ukef

From: Justin<EMAIL_ADDRESS>Sent: Monday, January 4, 2021 2:22:30 PM
To: dotnet/Docker.DotNet<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [dotnet/Docker.DotNet] A cancellation doesn't work in some cases and app hangs. (#351)
@dgviveshttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdgvives&data=04|01||7c614e6a5d4c45aacd4708d8b0d55193|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453777528915147|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=9XE%2BVdOwqDkfdkt3Uy9j8QRIt6G6kZBp%2F%2BcfDiue0mo%3D&reserved=0 - Unrelated to this PR but if you are going to do that I would prefer that we just remove tools/specgen anymore and just manually update the models. Docker changed how all of this worked long ago when the broke the branches into EE/CE and made specgen less than helpful. I'm all for updating the models I just think that specgen is not worth the effort anymore. I no longer have the time to really contribute here but I am happy to merge PR's if you want to do this work. Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2FDocker.DotNet%2Fpull%2F351%23issuecomment-754104913&data=04|01||7c614e6a5d4c45aacd4708d8b0d55193|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453777528925141|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=zVYArtXkkERQ%2F4FsU%2BO9T%2BFmips0WI3pYo91UDdxXLY%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABD64OE65QJUOXSCE4L6J7LSYH2NNANCNFSM4F3UFFHA&data=04|01||7c614e6a5d4c45aacd4708d8b0d55193|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453777528925141|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=CAkIgGoTi5aTFgXC81MkVzJrQw5CRJGJzRHbuHfdygA%3D&reserved=0.

@jterry75 @galvesribeiro  Some of the models overridden on modeldefs.go are missing the required rest tag for correct model generation. I've updated some I found broken, but not all of them.
Code generation using specgen is what I've used. All the generated models are based on specgen details, as I think they should be at least for now.
Continue on #480

@jterry75 @galvesribeiro  Some of the models overridden on modeldefs.go are missing the required rest tag for correct model generation. I've updated some I found broken, but not all of them.
Code generation using specgen is what I've used. All the generated models are based on specgen details, as I think they should be at least for now.
Continue on #480

@dgvives  I will take a look into it this week.

@dgvives  I will take a look into it this week.

@gurustron I confirm these changes have been incorporated in the last merge, MonitorEventsAsync can be cancelled
Current solution is based on #343 using an extension method and throwing an exception whenever token gets cancellated.
However, this behaviour is not the most appropriate as it produces slightly different results when running tests on different environments. I expect to submit a PR with a different approach to this, and include references to yours and others' work  as contributor

@gurustron I confirm these changes have been incorporated in the last merge, MonitorEventsAsync can be cancelled
Current solution is based on #343 using an extension method and throwing an exception whenever token gets cancellated.
However, this behaviour is not the most appropriate as it produces slightly different results when running tests on different environments. I expect to submit a PR with a different approach to this, and include references to yours and others' work  as contributor


Az Login failed temporarily in the Pipeline

Describe the bug
We use az login in one of our Azure DevOps release pipelines to deploy stuff to azure (probably not relevant in this issue).
Today the pipeline failed temporarily (worked after retry) with an error I couldn't find any google results for.
Therefore I decided to report it here.
Command Name
az login
Output & Errors:

Python (Windows) 3.8.9 (tags/v3.8.9:a743f81, Apr  6 2021, 13:22:56) [MSC v.1928 32 bit (Intel)]

Legal docs and information: aka.ms/AzureCliLegal

Setting AZURE_CONFIG_DIR env variable to: D:\Agent1\_work\_temp\.azclitask
Setting active cloud to: AzureCloud
C:\WINDOWS\system32\cmd.exe /D /S /C ""C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\wbin\az.cmd" cloud set -n AzureCloud"
C:\WINDOWS\system32\cmd.exe /D /S /C ""C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\wbin\az.cmd" login --service-principal -u *** "--password=***" --tenant *** --allow-no-subscriptions"
ERROR: The command failed with an unexpected error. Here is the traceback:
ERROR: cannot import name 'register_algorithm' from 'jwt.api_jwt' (C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\jwt\api_jwt.pyc)
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 657, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 691, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 328, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/profile/custom.py", line 118, in login
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\adal/__init__.py", line 34, in <module>
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\adal/authentication_context.py", line 34, in <module>
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\adal/token_request.py", line 34, in <module>
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\adal/self_signed_jwt.py", line 35, in <module>
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\jwt/__init__.py", line 19, in <module>
ImportError: cannot import name 'register_algorithm' from 'jwt.api_jwt' (C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\jwt\api_jwt.pyc)
To open an issue, please run: 'az feedback'
##[error]Error Code: [1]
##[error]Error: Azure login failed
##[error]Script failed with error: ERROR: The command failed with an unexpected error. Here is the traceback:
ERROR: cannot import name 'register_algorithm' from 'jwt.api_jwt' (C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\jwt\api_jwt.pyc)
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 657, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 691, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 328, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/profile/custom.py", line 118, in login
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\adal/__init__.py", line 34, in <module>
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\adal/authentication_context.py", line 34, in <module>
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\adal/token_request.py", line 34, in <module>
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\adal/self_signed_jwt.py", line 35, in <module>
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\jwt/__init__.py", line 19, in <module>
ImportError: cannot import name 'register_algorithm' from 'jwt.api_jwt' (C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\site-packages\jwt\api_jwt.pyc)
To open an issue, please run: 'az feedback'

To Reproduce:
See output above
Expected Behavior
Works
Environment Summary

Name
Value

Agent.Name
TEAMBUILD17_B1

AzureCLI
2.3.0

DotNetCore
3.1.401

Agent.Version
2.190.0

Agent.ComputerName
TEAMBUILD17

Agent.HomeDirectory
D:\Agent1

Agent.OS
Windows_NT

Agent.OSArchitecture
X64

Agent.OSVersion
10.0.19043

Path
C:\Python27;C:\Python27\Scripts;C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\wbin;C:\WINDOWS\system32;C:\WINDOWS;C:\WI...

AzurePS
5.7.0

PowerShell
5.1.19041.1151

Additional Context
So it seems like we use python 2.7 there. That is still supported, correct?

You are using Azure CLI 2.3.0 which is a very old version. Could you use the latest Azure CLI and see if the problem still exists?


fix: working codecov on CI

test pr, for actions check

@yegor256 looks like pipeline is fixed now, take a look, please

@l3r8yJ thanks!


Introduce callstack splitting to avoid stackoverflows with large AQL queries

Scope & Purpose
Fix potential stack overflow when executing large queries. This is achieved by splitting the callstack and moving part of the execution to a separate thread. The number of execution nodes after which such a callstack split should be performed can be configured via the query option maxNodesPerCallstack and the command line option --query.max-nodes-per-callstack; the default is 250.

[x] :hankey: Bugfix (requires CHANGELOG entry)
[x] :book: CHANGELOG entry made

Backports:

[ ] No backports required
[ ] Backports required for: (Please specify versions and link PRs)

Related Information

[x] Jira ticket number: https://arangodb.atlassian.net/browse/APM-77

Testing & Verification

[x] The behavior in this PR was manually tested
[x] This PR adds tests that were used to verify all changes:

[x] Added new integration tests in shell_server_aql

Documentation

All new features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the CHANGELOG so that
support, end users, and other developers have a concise overview.

[ ] Added entry to Release Notes
[ ] Added a new section in the Manual
[ ] Added a new section in the HTTP API
[ ] Added Swagger examples for the HTTP API
[ ] Updated license information in LICENSES-OTHER-COMPONENTS.md for 3rd party libraries

@goedderz Yes, the additional threads are an intentional tradeoff for this simple and somewhat hacky fix until we have a proper fix. A documentation PR with more details is on my todo list. The default value of 250 was sufficiently low to work correctly on my windows machine which uses a default stack size of 1MB. AFAIK Linux uses a larger stack size, so theoretically we could use OS dependent defaults. WDYT?
Thanks for pointing out other places with recursive algorithms on the plan. I was aware of ExecutionNode::walk which was next on my todo list, but not ExecutionNode::estimateCost().
Regarding increased stack size - I would rather not touch that, but if a user wants to increase the stack size they are free to increase maxNodesPerCallstack. Sooner or later we need a proper fix for this anyway.

@goedderz Yes, the additional threads are an intentional tradeoff for this simple and somewhat hacky fix until we have a proper fix. A documentation PR with more details is on my todo list. The default value of 250 was sufficiently low to work correctly on my windows machine which uses a default stack size of 1MB. AFAIK Linux uses a larger stack size, so theoretically we could use OS dependent defaults. WDYT?

250 sounds like a fine default to me. While Linux has defaults, these can always be changed per system, so if we want to adapt the default to the stack size, we might want to look at the actual stack size. However, that might be overly complicated for a small gain, So I'm fine with letting the default be 250.

Thanks for pointing out other places with recursive algorithms on the plan. I was aware of ExecutionNode::walk which was next on my todo list, but not ExecutionNode::estimateCost().

Regarding increased stack size - I would rather not touch that, but if a user wants to increase the stack size they are free to increase maxNodesPerCallstack. Sooner or later we need a proper fix for this anyway.

Sure, sounds sensible.


Missing search results

The search:
repo:^github\.com/torvalds/uemacs$ define
https://www.sourcebot.dev/search/search?query=repo%3A^github\.com%2Ftorvalds%2Fuemacs%24 define
shows a bunch of results in uemacs for define, but
repo:^github\.com/torvalds.* define
https://www.sourcebot.dev/search/search?query=repo%3A^github\.com%2Ftorvalds.* define
does not include "define" results from uemacs.
The repo must be matching though, because:
repo:^github\.com/torvalds.* ncol
https://www.sourcebot.dev/search/search?query=repo%3A^github\.com%2Ftorvalds.* ncol
does contain uemacs results.
I think this is a bug? Would be good to dig deeper.

Looks like that search query returns so many results that the maxMatchDisplayCount limit (which defaults to 10000) is hit. If you click "(load more)", we multiply the display count by 2. For example, when the display count is 10000000, we get all of the results (including those that are in uemacs):
https://www.sourcebot.dev/search/search?query=repo%3A^github\.com%2Ftorvalds.* define&maxMatchDisplayCount=10000000

Ah, whoops. Misunderstood the behaviour.


remediate CVE-2019-5736

Changelog

remediates CVE-2019-5736

Questions (open the PR then click the check boxes)
Did you update the documentation related to your changes?

[ ] Yes
[x] My changes were not already documented

Did you run make before committing code and opening this PR?

[x] Yes
[ ] I didn't change any code

Did you run porter create-stack and porter sync-stack to verify provisioning
works?

[x] Yes
[ ] N/A

Failed CI is unrelated. I need to pin the testing library versions

👍


Laboratory Instruments seem to have wrong icon while in an inserter

Mod source
PyAE Beta
Which mod are you having an issue with?

[ ] pyalienlife
[ ] pyalternativeenergy
[X] pycoalprocessing
[ ] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[ ] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[X] Graphical
[ ] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
the laboratory instruments look like a mortar and pestle when in an inserter.  they look correct on the ground.  is it just set to use the wrong icon in one case?
Steps to reproduce
put a laboratory instrument in an inserter and look at it
Additional context

Log file
factorio-current.log

This can be a bit misleading as many py items have variations for when they're "in the world". You can see the same thing when putting glassware on a belt. I've put the variations below, they're all "lab instruments" but the recipe icon only shows one of them.


YARN-11191. Fix potentional deadlock in GlobalScheduler refreshQueues…

… (#6732)
(cherry picked from commit ecf665c6facf89d3b87b6e3cc684274b8155ca60)

Description of PR
backport.
Jira: YARN-11191
Original PR: #6732
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 21s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ branch-3.4 Compile Tests _

+1 :green_heart:
mvninstall
33m  3s

branch-3.4 passed

+1 :green_heart:
compile
0m 34s

branch-3.4 passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
compile
0m 33s

branch-3.4 passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
checkstyle
0m 34s

branch-3.4 passed

+1 :green_heart:
mvnsite
0m 38s

branch-3.4 passed

+1 :green_heart:
javadoc
0m 37s

branch-3.4 passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javadoc
0m 30s

branch-3.4 passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
spotbugs
1m 10s

branch-3.4 passed

+1 :green_heart:
shadedclient
20m 29s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

-1 :x:
mvninstall
0m 23s
/patch-mvninstall-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch failed.

-1 :x:
compile
0m 27s
/patch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdkUbuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1.txt
hadoop-yarn-server-resourcemanager in the patch failed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1.

-1 :x:
javac
0m 27s
/patch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdkUbuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1.txt
hadoop-yarn-server-resourcemanager in the patch failed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1.

-1 :x:
compile
0m 22s
/patch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdkPrivateBuild-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06.txt
hadoop-yarn-server-resourcemanager in the patch failed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06.

-1 :x:
javac
0m 22s
/patch-compile-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdkPrivateBuild-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06.txt
hadoop-yarn-server-resourcemanager in the patch failed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06.

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 24s

the patch passed

-1 :x:
mvnsite
0m 24s
/patch-mvnsite-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch failed.

+1 :green_heart:
javadoc
0m 27s

the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javadoc
0m 28s

the patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

-1 :x:
spotbugs
0m 26s
/patch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch failed.

-1 :x:
shadedclient
7m 43s

patch has errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
0m 25s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch failed.

+1 :green_heart:
asflicense
0m 23s

The patch does not generate ASF License warnings.

69m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6768/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6768

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux<PHONE_NUMBER>e5 5.15.0-94-generic #104-Ubuntu SMP Tue Jan 9 15:25:40 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.4 / a840de1027304e5f8e28134f462aa26056770705

Default Java
Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6768/1/testReport/

Max. process+thread count
556 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6768/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 21s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ branch-3.4 Compile Tests _

+1 :green_heart:
mvninstall
32m 50s

branch-3.4 passed

+1 :green_heart:
compile
0m 32s

branch-3.4 passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
compile
0m 31s

branch-3.4 passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
checkstyle
0m 31s

branch-3.4 passed

+1 :green_heart:
mvnsite
0m 36s

branch-3.4 passed

+1 :green_heart:
javadoc
0m 37s

branch-3.4 passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javadoc
0m 29s

branch-3.4 passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
spotbugs
1m  9s

branch-3.4 passed

+1 :green_heart:
shadedclient
20m 36s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 26s

the patch passed

+1 :green_heart:
compile
0m 30s

the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javac
0m 30s

the patch passed

+1 :green_heart:
compile
0m 25s

the patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
javac
0m 25s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 24s

the patch passed

+1 :green_heart:
mvnsite
0m 27s

the patch passed

+1 :green_heart:
javadoc
0m 27s

the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javadoc
0m 25s

the patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
spotbugs
1m 12s

the patch passed

+1 :green_heart:
shadedclient
20m 31s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
87m  5s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 26s

The patch does not generate ASF License warnings.

172m 52s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6768/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6768

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 111d4e1feebc 5.15.0-94-generic #104-Ubuntu SMP Tue Jan 9 15:25:40 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.4 / 985da47eda7acc4d92a0a5c4095c164f33520725

Default Java
Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6768/2/testReport/

Max. process+thread count
955 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6768/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@tomicooler Can we retrigger compilation?

@slfan1989 Yes, sure. Re-triggered.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
6m 57s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ branch-3.4 Compile Tests _

+1 :green_heart:
mvninstall
32m 44s

branch-3.4 passed

+1 :green_heart:
compile
0m 36s

branch-3.4 passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
compile
0m 30s

branch-3.4 passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
checkstyle
0m 31s

branch-3.4 passed

+1 :green_heart:
mvnsite
0m 34s

branch-3.4 passed

+1 :green_heart:
javadoc
0m 37s

branch-3.4 passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javadoc
0m 29s

branch-3.4 passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
spotbugs
1m 16s

branch-3.4 passed

+1 :green_heart:
shadedclient
20m 32s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 27s

the patch passed

+1 :green_heart:
compile
0m 28s

the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javac
0m 28s

the patch passed

+1 :green_heart:
compile
0m 28s

the patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
javac
0m 28s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 24s

the patch passed

+1 :green_heart:
mvnsite
0m 28s

the patch passed

+1 :green_heart:
javadoc
0m 25s

the patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1

+1 :green_heart:
javadoc
0m 28s

the patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

+1 :green_heart:
spotbugs
1m  9s

the patch passed

+1 :green_heart:
shadedclient
20m 24s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
87m 33s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 26s

The patch does not generate ASF License warnings.

179m  4s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6768/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6768

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 10200f517789 5.15.0-94-generic #104-Ubuntu SMP Tue Jan 9 15:25:40 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.4 / 31f1424fcd7011fdd78d507e9482cffee9bcd09b

Default Java
Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu220.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_402-8u402-ga-2ubuntu1~20.04-b06

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6768/3/testReport/

Max. process+thread count
950 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6768/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@tomicooler Thanks for the contribution!

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

_ Prechecks _

+1 :green_heart:
dupname
0m 01s

No case conflicting files found.

+0 :ok:
spotbugs
0m 01s

spotbugs executables are not available.

+0 :ok:
codespell
0m 01s

codespell was not available.

+0 :ok:
detsecrets
0m 01s

detect-secrets was not available.

+1 :green_heart:
@author
0m 00s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m 00s

The patch appears to include 1 new or modified test files.

_ branch-3.4 Compile Tests _

+1 :green_heart:
mvninstall
88m 19s

branch-3.4 passed

+1 :green_heart:
compile
5m 43s

branch-3.4 passed

+1 :green_heart:
checkstyle
4m 35s

branch-3.4 passed

+1 :green_heart:
mvnsite
5m 36s

branch-3.4 passed

+1 :green_heart:
javadoc
5m 31s

branch-3.4 passed

+1 :green_heart:
shadedclient
148m 13s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 26s

the patch passed

+1 :green_heart:
compile
3m 06s

the patch passed

+1 :green_heart:
javac
3m 06s

the patch passed

+1 :green_heart:
blanks
0m 00s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
2m 11s

the patch passed

+1 :green_heart:
mvnsite
3m 18s

the patch passed

+1 :green_heart:
javadoc
2m 55s

the patch passed

+1 :green_heart:
shadedclient
155m 05s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
323m 16s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch passed.

-1 :x:
asflicense
5m 28s
/results-asflicense.txt
The patch generated 2 ASF License warnings.

747m 18s

Reason
Tests

Failed junit tests
hadoop.yarn.server.resourcemanager.metrics.TestCombinedSystemMetricsPublisher

hadoop.yarn.server.resourcemanager.recovery.TestLeveldbRMStateStore

hadoop.yarn.server.resourcemanager.resource.TestResourceProfiles

hadoop.yarn.server.resourcemanager.scheduler.capacity.conf.TestFSSchedulerConfigurationStore

hadoop.yarn.server.resourcemanager.scheduler.capacity.conf.TestLeveldbConfigurationStore

hadoop.yarn.server.resourcemanager.scheduler.capacity.TestCSAllocateCustomResource

hadoop.yarn.server.resourcemanager.scheduler.capacity.TestQueueConfigurationAutoRefreshPolicy

hadoop.yarn.server.resourcemanager.scheduler.fair.converter.TestFSConfigToCSConfigArgumentHandler

hadoop.yarn.server.resourcemanager.scheduler.fair.converter.TestFSConfigToCSConfigConverter

hadoop.yarn.server.resourcemanager.scheduler.fair.converter.TestFSConfigToCSConfigConverterMain

hadoop.yarn.server.resourcemanager.scheduler.fair.converter.TestFSQueueConverter

hadoop.yarn.server.resourcemanager.scheduler.fair.TestAllocationFileLoaderService

hadoop.yarn.server.resourcemanager.TestClientRMService

hadoop.yarn.server.resourcemanager.TestSignalContainer

hadoop.yarn.server.resourcemanager.TestWorkPreservingRMRestart

hadoop.yarn.server.resourcemanager.volume.csi.TestVolumeProcessor

hadoop.yarn.server.resourcemanager.webapp.TestRMWebappAuthentication

hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesAppCustomResourceTypes

hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesAppsCustomResourceTypes

hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesContainers

hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesDelegationTokenAuthentication

hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesHttpStaticUserPermissions

hadoop.yarn.server.resourcemanager.webapp.TestRMWebServicesSchedulerActivitiesWithMultiNodesEnabled

hadoop.yarn.webapp.TestRMWithXFSFilter

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/6768

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
MINGW64_NT-10.0-17763 99549ba5ee15 3.4.10-87d57229.x86_64 2024-02-14 20:17 UTC x86_64 Msys

Build tool
maven

Personality
/c/hadoop/dev-support/bin/hadoop.sh

git revision
branch-3.4 / 31f1424fcd7011fdd78d507e9482cffee9bcd09b

Default Java
Azul Systems, Inc.-1.8.0_332-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch-windows-10/job/PR-6768/3/testReport/

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch-windows-10/job/PR-6768/3/console

versions
git=2.44.0.windows.1

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


Do you have another way to implement DTO?

I think using comments for logic code is not a good idea.
So, Do you have another way to implement a validated data request body?

No way in @nestia/core which is using typia.
If you need additional type checking, just make a new function doing that.

Got it, thank you.

Only way supporting custom function is just using parameter, but I can't sure it may effective
@TypedBody((input: YourDto) => YourValidationLogic...)


Hello, can you pass out _frame_scores as a list that can be obtained, so that I can do post-processing on the split results

Hello, can you pass out _frame_scores as a list that can be obtained, so that I can do post-processing on the split results

min_length_met = (frame_num - self._last_scene_cut) >= self._min_scene_len
if self._frame_score >= self._threshold and min_length_met:
self._last_scene_cut = frame_num
return [frame_num]

as above
Only split frames are passed out, if you are not satisfied with the result, you cannot add post-processing edits

Hi @babyta;
You can obtain the scores for each frame using a StatsManager when doing scene detection.  Is this what you were looking for?
Thanks!

‘’scene_manager.stats_manager._frame_metrics ‘’    works, thank you very much


feat: Add a workaround for a Snowflake IN list limitation

Add a workaround for a Snowflake IN list limitation including support for splitting IN lists into a series of smaller IN list expressions.

/gcbrun

/gcbrun


Merge Editor user study Part II

Planned for end of October.
Same scenario as https://github.com/microsoft/vscode/issues/159561
To validate all the great improvements we have done to the merge editor

The user studies have been done. I will do a short write up of our findings tomorrow.
Thus closing this as completed.


pod: drop invalid check for rootless

rootless networks support port binding if the underlying slirp4netns
is new enough.
Signed-off-by: Giuseppe Scrivano<EMAIL_ADDRESS>

LGTM
@mheon @vrothberg @TomSweeneyRedHat @umohnani8 @baude PTAL

we have a comment in f182946bef84d2766c4ff9faef171cf69afacb97.
0.3 will be released soon, so we can point the rpm packages to it


activeClassName not supported in 3.0.0 with esModuleInterop: true

activeClassName gets passed through to the a element as a regular attribute and is not used in the matching-style that is documented in the README.
It seems to be because the Link component defined in match.js is not actually exported. Instead the regular a passthrough component is exported.
https://github.com/preactjs/preact-router/blob/3eb5b31fe75c34672eeb13f2d310dee6d51b1f97/src/index.js#L251-L253

For additional info, it seems like this might be an artifact of too smart commonjs compat which uses the Link defined on Router instead of the export {..., Link}.
I'm using parcel + typescript with esModuleInterop: true.

@patrickhulce hmm - are you importing Link from preact-router/match? I just merged a PR to correct the docs, which were erroneously stating that { Link } from 'preact-router' supports activeClassName (it does not).
import { Link } from 'preact-router';
<Link activeClassName="foo" />  // does not work

import { Link } from 'preact-router/match';
<Link activeClassName="foo" />  // works

I was indeed importing from root according to the previous docs.
Why is import { Link } from 'preact-router' not the link with matching behavior?

It's a way to reduce the size of the core library. I think it may be worth evaluating what the size impact would be to merge <Match> and <Link activeClassName> into preact-router core. Only a percentage of folks use them, but they aren't very large.