SR-3175: Include the terminating signal number in the driver output

Add the signal number of the terminated task to the output of the driver on platforms for which the signal number is available. The new key in the parseable driver output is "signal".
Add a test to verify that the signal number is emitted.
Add documentation for the new "signal" key emitted in the parseable driver output.

https://bugs.swift.org/browse/SR-3175

@swift-ci please smoke test.

Whoa, an NPE took down the build!   // @shahmishal
@swift please smoke test OS X platform.

@swift-ci please smoke test OS X platform.

@CodaFi,
I merged the branch hughbe:taskqueue-fixes from #6812

Can you knock off the merge commit either by folding it into the previous one or by cherry-picking @hughbe's branch please?

@swift-ci please smoke test.

@swift-ci please smoke test.

Is this PR ready to go? Merging will unblock the Windows (clang and MSVC) build

@hughbe Yes it's ready.
@CodaFi What do you think?

⛵


Metric: % of schools

https://www.data.gov.uk/dataset/9a6f9d86-9698-4a5d-a2c8-89f3b212c52c/scottish-school-roll-and-locations
100m buffer, but need to decide exactly what it means for the network to cover that. See notes for discussion about low-stress MixedTraffic roads and severances within that 100m

Mostly working


Any alternative working version to Opencv-python

While trying to install from requirements, but terminal prompts: No matching distribution found for opencv-python==<IP_ADDRESS>
Any alternative working version

I think I found it: opencv-python==<IP_ADDRESS>


[Feature Request] Streamed response

Hello,
thank you for this great product.
I have one feature request, sometimes API respond with streamed messages, series of JSON line or other that comes over time.
I think at the moment, Bruno wait for complete response before rendering to the UI.
Would be nice to see the messages as they comes to follow along.
Thank you very much.

This feature is import for some LLM streaming APIs, thanks.
We found postman has supported this feature.  See blow:

Would also love to see this feature. I work heavily with IoT devices and have to document/test a lot of APIs that return text/event-stream content.

Looking at the code, this project uses axios for the requests under the hood. The problem with that is that axios does not support http streams due to being built on top of XHR which is a very outdated and essentially abandoned standard.
Axios would need to be replaced with modern fetch calls. Node supports fetch via a flag from v16, without a flag from v18, and it's fully stable as of v21. This project uses electron v21.1.1 which is on node v16 so to support fetch natively it would need to include the flag. The other options would be to upgrade electron to be able to use a newer version of node, or use a polyfill library like node-fetch which supports streaming.
So first the maintainer would need to approve switching from axios to fetch, a fetch implementation would need to be added so it can be used, and then axios would need to be replaced with fetch, and then a streaming implementation would need to be added.

This project uses electron v21.1.1 which is on node v16

We plan to move to electron v31 which uses node v18 asap.

to support fetch natively it would need to include the flag.

this makes sense

the maintainer would need to approve switching from axios to fetch

this would be very hard to execute at this stage of the project. The approach I am thinking of is to allow the toggling the provider at the collection level. (default to axios, but allow to use fetch / curl ). This would need UI changes, also a placeholder in bruno.json to store this data.

the maintainer would need to approve switching from axios to fetch

this would be very hard to execute at this stage of the project. The approach I am thinking of is to allow the toggling the provider at the collection level. (default to axios, but allow to use fetch / curl ).

According to axios/axios#5806 it should now be possible to configure Axios to use the fetch adapter and support streaming that way.

Would be awesome to use streamed responses - I am using my terminal to see them but if I could do the same in bruno it would be dope.
+1 to what the guy above said, it is very useful for LLM / Text-to-Speech applications, where we use data chunks and stream them for more optimal response times.
thank you!


feat: support to windows phone app from ionic

Type: feat
Platform: mobile webview
@Ionicframework eagerly waiting windows phone app build in ionic when can we expect (at least release (#alpha-version) year or month )

Greetings @iiitmahesh!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

support to windows phone app from ionic. we wanna(ionic developers team) develop  windows phone app , When can we expect that feature (at month year releasing  alpha version ) .Since phone gap providing

We probably won't be supporting WP8. Especially considering Windows 10 will be coming out.

@mhartington thank you.I have been waiting for that.


v0.3.1 not available for python 3.8 on pypi.org

Would it be possible to build the 0.3.1 version for python 3.8 as well and make it available on pypi.org?
➜  ~ docker run -ti python:3.8 /bin/bash
root@9c5eb8a054cd:/# pip install sktime==0.3.1
ERROR: Could not find a version that satisfies the requirement sktime==0.3.1 (from versions: 0.1.dev0, 0.1.0, 0.2.0, 0.3.0)
ERROR: No matching distribution found for sktime==0.3.1

With release v0.4.0 we now have Python 3.8 wheels ... finally 🎉


Feat/tooltip error ts conversion

tooltip conversion && errors conversion

@danethurber I have made the changes I wanted on this.  Would you mind adding your review when you're ready, please?


UI automation tests setup

based on https://github.com/trustbloc/wallet/issues/857#issue-947914035

This has been implemented https://github.com/trustbloc/wallet/pull/885


Node 14 - EOL

Ref: https://github.com/devcontainers/images/issues/445

stop
Get Outlook for iOShttps://aka.ms/o0ukef

From: Josh Spicer @.>
Sent: Friday, May 5, 2023 8:40:42 PM
To: devcontainers/images @.>
Cc: m7md226 @.>; Comment @.>
Subject: Re: [devcontainers/images] Node 14 - EOL (PR #553)
@joshspicer approved this pull request.
—
Reply to this email directly, view it on GitHubhttps://github.com/devcontainers/images/pull/553#pullrequestreview-1415113396, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A6XS67XZGKNIXUV4HQQDKQDXEUUQVANCNFSM6AAAAAAXWM2ZNM.
You are receiving this because you commented.Message ID: @.***>


Possible to set _ttl option in mappings?

I'm new to elasticsearch + tire. I'd like to use _ttl setting.
My index creation code:
Tire.index 'site' do
   delete

   create mappings: {
     document: {
       properties: {
         Collections: { type: 'string', index: 'not_analyzed' },
         Categories:  { type: 'string', index: 'not_analyzed' },
         Sizes:       { type: 'string', index: 'not_analyzed' },
         Segments:    { type: 'string', index: 'not_analyzed' }
       }
     }
   }
 end

Do I just need to set the option in the hash, and tire will read it just fine? Or does tire need to be tweaked to support it?

Yes, setting the option properly in the Hash is fine:
require 'tire'

Tire.configure { logger STDERR, :level => "debug" }

Tire.index 'index-with-ttl' do
  delete
  create mappings: {
    document: {
      # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      _ttl:       { enabled: true , default: "1d" },
      # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      properties: {
        title: { type: "string", analyzer: "snowball" },
        # ... rest of properties mapping ...
      }
    }
  }
  refresh
end

Ping, any news?

Yes - worked like a charm, after I jumped off the homebrew version of elasticsearch.
Homebrew presently installs 0.16.2, from May 31.
Thanks for quick response on this.

Great. I don't think Homebrew installs 0.16.2: https://github.com/mxcl/homebrew/blob/master/Library/Formula/elasticsearch.rb#L4

Duh - my homebrew was out of date. brew update is my friend (after I cleared myself of this issue.)

Yeah, the ES community updates the Homebrew formula pretty frequently...


Use crypto_one_time_aead on OTP 23 and onwards

updated rebar3_hex_user to use crypto:crypto_one_time_aead/6 and
crypto:crypto_one_time_aead/7 if OTP is equal to or greater than 23

added private function cipher/1 to determine the correct cipher to
use with crypto:crypto_one_time_aead/6 and crypto:crypto_one_time_aead/7

standardize on upper case platform_define(s) in rebar.config

Great stuff. I approve these changes. Would it be possible to request for a tag post-merge? (projects with warnings_as_errors thank you)

@paulo-ferraz-oliveira Yes, will definitely do a release after this is merged to get this and a few other patch level changes out.


Make currency column numeric function

This PR resolves #(put issue number here).
PR Checklist
Please ensure that you have done the following:

[X] PR in from a fork off your branch. Do not PR from <your_username>:master, but rather from <your_username>:<branch_name>.
[x] If you're not on the contributors list, add yourself to AUTHORS.rst.

Quick Check
To do a very quick check that everything is correct, follow these steps below:

[x] Run the command make check from pyjanitor's top-level directory. This will automatically run:

black formatting
pycodestyle checkin
running the test suite
docs build

Code Changes
If you are adding code changes, please ensure the following:

[x] Ensure that you have added tests.
[x] Run all tests ($ pytest .) locally on your machine.

[x] Check to ensure that test coverage covers the lines of code that you have added.
[x] Ensure that all tests pass.

Documentation Changes
If you are adding documentation changes, please ensure the following:

[X] Build the docs locally.
[X] View the docs to check that it renders correctly.

PR Description
Please describe the changes proposed in the pull request:
I added a new function called make_currency_column_numeric. It basically takes a column that should be a float, but was stored as str because of dollar signs, etc.
It also includes:

Removing 'accounting' style fields (i.e. the numbers with parentheses, like (1.00) )
Casting words to numeric values that may be found in the column
Removing rows where the value is a word or other string

Relevant Reviewers
Please tag maintainers to review.

@ericmjl

I was considering putting this in the finance submodule, but thought it was too common of a use case just to have there.

I was considering putting this in the finance submodule, but thought it was too common of a use case just to have there.

If so, then should the function name be perhaps changed to a more generic one, rather than a currency column?

Maybe the name should be more general, but I was having problems coming up with something that wouldn’t be misleading.
For example, to_numeric has a specific definition in most APIs and something like “ 0.04” would fail because of the white space.
Similarly, I feel that calling it something like “string_to_numeric” is also misleading since that might imply that the function converts words to numeric values (like “Gold” —> 3, “Silver” —> 2, etc).

@szuckerman understood!
I'm definitely getting a good feeling about the utility of the function; I just want it to have a good name (because the name corresponds to the API, and it's hard to change the API once people become used to it).
You know, I have an alternate idea, let's see what you think about this:
There are some pretty good ideas baked into the function. Let's put this in the finance submodule for now. What we can do is refactor out the nice generalizable parts later on. Doing this will give us a domain-specific name that maps nicely to a very specific finance-related task, while buying us time to think of the generalizable name later. Maybe what we need is some non-finance example use cases, really, to clarify what the generalized name will be.
What are your thoughts?

(Sorry for being MIA for a few days)
I think you're right; we can leave it in the finance submodule for now until it's more generalized.
However, my problem now is naming the function. string_column_to_numeric? That almost sounds like I have a column of words that I'm going to map to numeric values.
If we generalize the function I think that having a type="currency" argument similar to the type="accounting" argument would be useful.
I think that this is a good example of what might be the threshold for including in the main package or the submodules. The main package should ensure that a DataFrame is in a state to run pandas functions without any extra work. If a number column is encoded as str, there's extra work involved.
Whereas for the submodules it's more specific to the use-case. For example, and I'm not saying we should implement this but just a thought, for the finance submodule there could be a moving_average function for a column. I know there's some gray area, but the overall idea seems to make sense, thoughts?

The travis build failed on a line with whitespace :(

(Sorry for being MIA for a few days)

No worries, @szuckerman! Good software is developed thoughfully!

However, my problem now is naming the function. string_column_to_numeric? That almost sounds like I have a column of words that I'm going to map to numeric values.

In finance, is there a term that accountants/financial people use? A verb that would match their mental model? I think that’ll give us a good verb (let’s try to keep it to two verbs generally) that can describe succinctly the action being taken.

If we generalize the function I think that having a type="currency" argument similar to the type="accounting" argument would be useful.

:thumbsup: Let’s see whether we or other users encounter a general use case for the function. That way it’ll solve someone’s actual need. Amazon-style (listen to our customers, which in our case, includes ourselves).

@szuckerman damn, I think i've left this PR hanging for too long. Is it ready to merge? Would you be kind enough to update the relevant branch from master? Let's make this the last thing we PR into master.


AXI4 coverage exclusion for tx_online and rx_online

Current coverage runs does not toggle tx_online and rx_online this can be done by asserting axi reset during operation. This should be applicable for all AXI modes (MM, Stream and Lite)

Fixed as of 0.9.3

this is now included as part of directed_reset_test for AXIMM
run_tests="directed_test directed_4kboundary_test random_wr_rd_test reorder_wr_rd_test directed_reset_test"
Can this also be included for AXIST? Right now it using these tests only:
run_tests="zero_delay_random_test random_test random_reset_test"

random_reset_test is the AXI-ST version.
On Wed, Jan 12, 2022 at 4:00 PM nij-intel @.***> wrote:

this is now included as part of directed_reset_test for AXIMM
run_tests="directed_test directed_4kboundary_test random_wr_rd_test
reorder_wr_rd_test directed_reset_test"
Can this also be included for AXIST? Right now it using these tests only:
run_tests="zero_delay_random_test random_test random_reset_test"
—
Reply to this email directly, view it on GitHub
https://github.com/chipsalliance/aib-protocols/issues/57#issuecomment-1011489980,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUYPV2JEH6OVITN232TDNFDUVX2XBANCNFSM5KETXH7A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were assigned.Message ID:
@.***>

--
The information contained in this electronic message and any attachments to
this message are intended for the exclusive use of the addressee(s) and may
contain proprietary, confidential or privileged information. If you are not
the intended recipient, you should not disseminate, distribute or copy this
e-mail. Please notify the sender immediately and destroy all copies of this
message and any attachments. WARNING: Computer viruses can be transmitted
via email. The recipient should check this email and any attachments for
the presence of viruses. The company accepts no liability for any damage
caused by any virus transmitted by this email. www.wipro.com
http://www.wipro.com/
--
The information contained in this electronic message and any attachments to
this message are intended for the exclusive use of the addressee(s) and may
contain proprietary, confidential or privileged information. If you are not
the intended recipient, you should not disseminate, distribute or copy this
e-mail. Please notify the sender immediately and destroy all copies of this
message and any attachments. WARNING: Computer viruses can be transmitted
via email. The recipient should check this email and any attachments for
the presence of viruses. The company accepts no liability for any damage
caused by any virus transmitted by this email. www.wipro.com
http://www.wipro.com/


re-enable system_debug_ios

Description
Re-enable system_debug_ios because #41133 is resolved by https://github.com/flutter/flutter/pull/43915.
Related Issues
Fixes #43029.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read [Handling breaking changes]). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

@jmagman I opened this so we won't forget. Let's wait till Monday to land it though :)

This will still fail.  I'm working on fixing the test driver now, which doesn't go through the same resident_runner code path.

Oops, I knew that...

Is this PR still relevant? Or should we close it?

Let's close it for now.


[TASK]: UX Design: Stratify Model refinements

There are a couple of refinements we have made to the Stratify Model design.

Diagram (simplified) matrix nodes have been re-designed. On hover, a tooltip will provide additional at-a-glance strata colour associations and and labels.

Two new items have been added to the diagram legend: Transition and No Transition (in relation to the colours in the matrix nodes)

When the user clicks a matrix node, we display another tear sheet with the detailed matrix table (non-editable).

Below are some specs for the new matrix nodes and tooltip detailed matrix diagrams:

Please let me know if you have any questions, thanks! -Neil

@Jami159 @YohannParis Heads up, we've made some design revisions to the Stratify Model designs, specifically in relation to the matrices. Let me know if you have any questions - thanks!


Transport endpoint is not connected

Tried running it in a repl project/container, got this error.

Are you using the latest snapshot of discord?
For the script to work, you need a discord client.


switch accounts

hi
on Ramme 3.2.5 I have multiple accounts but do not save login information but earlier i see switch account  but now do not see and save information
please fix it

Pretty sure @facebook can do that on Instagram Web.

Pretty sure @facebook can do that on Instagram Web.

can you explain in the simplest way and step by step, please??

i wondering, do you don't mind use  Western-Union ? i mean i have to cash
in a person, then send you via recipient, you to get in person near you by
Western union  to get a cash? let me know
On Tue, Aug 27, 2019 at 3:03 PM Marcos Abreu<EMAIL_ADDRESS>wrote:

I would like use switch accounts feature. I didn't find in the current
version.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/terkelg/ramme/issues/308?email_source=notifications&email_token=AGTWEGE6DH35FGIY2KIVU53QGWQDVA5CNFSM4FU4KQEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5JIE4I#issuecomment-525501041,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGTWEGBYVY5VSGI3AWJNKQTQGWQDVANCNFSM4FU4KQEA
.


Can't bind [options] and no directive for angular-mydatepicker

I added AngularMyDatePickerModule.forRoot() to my imports but I'm getting the following errors below in my html template. I'm using Angular 7.1.2

Check that you are imported needed modules in your app module. Here is working example: https://github.com/kekeh/angular-mydatepicker/blob/master/example/app/app.module.ts

I double checked and I appear to be importing the necessary things into my app.module.ts. I'm using the FormsModule.
I am switching from the old ngx-mydatepicker to this new library. Not sure if that is causing some kind of issue? I did npm uninstall ngx-mydatepicker before installing this new library though.

I created a new project and reproduced the same issue:
app.module.ts
import { BrowserModule } from '@angular/platform-browser';
import { NgModule } from '@angular/core';
import { FormsModule } from '@angular/forms';

import { AppRoutingModule } from './app-routing.module';
import { AppComponent } from './app.component';
import { AngularMyDatePickerModule } from 'angular-mydatepicker';

@NgModule({
    declarations: [
        AppComponent
    ],
    imports: [
        BrowserModule,
        AppRoutingModule,
        FormsModule,
        AngularMyDatePickerModule.forRoot()
    ],
    providers: [],
    bootstrap: [AppComponent]
})
export class AppModule { }

app.component.html

am I not importing something correctly? Do I need to import something into app.component.ts?

I have used angular 7.2.0 in this new component. You have older version.

I created my new project and used Angular 7.2.0 and still had the same issue. Sounds like it is just me though. I'll just continue to use ngx-mydatepicker. Closing.


Bug: When "Open Workspace from File" eslint complains that it cannot find imports, when "Open from Folder" the same code, no issues

Bit of an odd error, seems to be related to if the .eslintrc has a parserOptions.project, as when I change from

.eslintrc with relative paths
folders renamed but same structure
"parserOptions": {
    "ecmaVersion": 2020,
    "sourceType": "module",
    "ecmaFeatures": {
      "jsx": true,
      "generators": true,
      "experimentalObjectRestSpread": true
    },
    // 'project' required for the no-floating-promises rule,
    "project": [
      "./a/tsconfig.json",
      "./b/tsconfig.json",
      "./c/tsconfig.json"
    ]
  },

to

.eslintrc with glob path
folders renamed but same structure
"parserOptions": {
    "ecmaVersion": 2020,
    "sourceType": "module",
    "ecmaFeatures": {
      "jsx": true,
      "generators": true,
      "experimentalObjectRestSpread": true
    },
    // 'project' required for the no-floating-promises rule,
    "project": [
      "**/tsconfig.json"
    ]
  },

then I no longer see the error, but I'm not sure if that's a bit of a hacky workaround?

repo-name.code-workspace file
folders renamed but same structure
{
  "folders": [
    {
      "name": "root folder",
      "path": ".."
    },
    {
      "name": "A",
      "path": "../a"
    },
    {
      "name": "B",
      "path": "../b"
    },
    {
      "name": "C",
      "path": "../c"
    },
    {
      "name": "cypress",
      "path": "../cypress"
    }
  ],
  "settings": {
    "jest.disabledWorkspaceFolders": [
      "root folder",
      "cypress"
    ],
  },
}

To my knowledge typescript eslint doesn't support glob patterns for projects. If you think they should be supported please file an issue against the typescript eslint repository.

To my knowledge typescript eslint doesn't support glob patterns for projects. If you think they should be supported please file an issue against the typescript eslint repository.

that's the thing, it only works with the globs, but that feels like a hack.

the specific error it gives is that if I use "Open Workspace from File" with the relative paths config linked in the issue, then entering any of the JS or TS files in the monorepo gives this error:

Parsing error: Cannot read file 'FILEPATH\a`FOLDER`\tsconfig.json'

my only thought, maybe it's that I include the root folder in there, which includes all the other workspaces, I'm going to see locally if that's true then if it's still an issue I'll get you a minimal setup

This issue has been closed because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


fix(server): fix reshaping of bloom past_key_values in concatenate()

Introduced in #214
Fixes #249

It works now!


Not require validation when field is blank

Hey @WellingtonCosta ,
You could enhance you validations making them to validate fields, but not require the validation when field is blank, for example validate an e-mail but let users save the field if it is blank.

Ok. I'll create a parameter called required with default value will be true in EmailValidator, LengthValidator, OnlyNumberValidator, PatternValidator, CpfValidator and in the new validations suggested in #32 and #31.

Implemented and it will be available in release 2.2.0.


Toast not working

By default, the toast is not appearing after making
[self.view makeToast@"text"];
unless the position and duration is specified.

Toast works without issue on iOS 13 and the demo project works as expected. There must be something else wrong with your project.


add matcher definitions in matchers.rb

Description
This allows the resources to be used with notification tests like so:
let(:download) { chef_run.aws_s3_file('my_file') }

it 'notifies something' do
  expect(download).to notify('execute[something]').immediately
end

I ran into a problem when I was trying to do this in a cookbook that is wrapping aws.
I ran into some trouble running the tests but just thought I'd see whether this is something the maintainers are interested in adding before continuing. Not sure if this is the kind of thing that needs to be tested or added to the README. Could also move the code somewhere else if preferred!
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[X] The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

There's a lint failure here, but thanks for adding this (and using the DCO)


sort.h line 833: warning: variable 'BD' set but not used [-Wunused-but-set-variable]

It seems in the function TIM_SORT_COLLAPSE(), in line 833 of sort.h, there is a set but not used variable BD. The Timsort algorithm works properly, so I'm wondering if we can just erase this BD variable.

Thanks for reporting this! Fixing it now.


Users wanted the code emailed to them at the end of the journey

As Verify does not share the users email address, this should be an optional feature linked to from the confirmation page.


mcMMO minecraft plugin and Occultism conflict.

Describe your problem
Hi! I'm running a hybrid 1.20.1 server and i have a conflict between Occultism and a plugin named(mcMMO)mcMMO
,when i run it with the mod, mcMMO stops working, but the server loads up just fine, the error happenes
during mcMMO's recipe conversion(Shapeless recipes cannot have more than 9 ingredients).
Here is the part of the log that mentions the problem:
[19dec.2023 21:59:00.709] [Server thread/ERROR] [mcMMO/]: There was an error while enabling mcMMO!
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: java.lang.RuntimeException: Error converting recipe occultism:ritual/possess_elder_guardian
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/arclight@1.20.1-1.0.1-c3fc1d3/org.bukkit.craftbukkit.v1_20_R1.inventory.RecipeIterator.next(RecipeIteratorMixin.java:554)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/arclight@1.20.1-1.0.1-c3fc1d3/org.bukkit.craftbukkit.v1_20_R1.inventory.RecipeIterator.next(RecipeIteratorMixin.java:1)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at com.gmail.nossr50.util.skills.SkillUtils.getRepairAndSalvageQuantities(SkillUtils.java:329)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at com.gmail.nossr50.config.skills.salvage.SalvageConfig.loadKeys(SalvageConfig.java:146)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at com.gmail.nossr50.config.skills.salvage.SalvageConfig.(SalvageConfig.java:28)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at com.gmail.nossr50.config.skills.salvage.SalvageConfigManager.(SalvageConfigManager.java:20)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at com.gmail.nossr50.mcMMO.loadConfigFiles(mcMMO.java:593)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at com.gmail.nossr50.mcMMO.onEnable(mcMMO.java:208)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/arclight@1.20.1-1.0.1-c3fc1d3/org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPluginMixin.java:266)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/arclight@1.20.1-1.0.1-c3fc1d3/org.bukkit.plugin.java.JavaPluginLoader.enablePlugin(JavaPluginLoaderMixin.java:342)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/arclight@1.20.1-1.0.1-c3fc1d3/org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:480)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/arclight@1.20.1-1.0.1-c3fc1d3/org.bukkit.craftbukkit.v1_20_R1.CraftServer.enablePlugin(CraftServerMixin.java:546)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/arclight@1.20.1-1.0.1-c3fc1d3/org.bukkit.craftbukkit.v1_20_R1.CraftServer.enablePlugins(CraftServerMixin.java:460)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.handler$cno000$arclight$enablePlugins(MinecraftServerMixin.java:2137)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_129815_(MinecraftServerMixin.java:385)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_130006_(MinecraftServerMixin.java:308)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.dedicated.DedicatedServer.m_7038_(DedicatedServerMixin.java:162)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_130011_(MinecraftServerMixin.java:2028)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_206580_(MinecraftServerMixin.java:251)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at java.base/java.lang.Thread.run(Thread.java:833)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: Caused by: java.lang.IllegalArgumentException: Shapeless recipes cannot have more than 9 ingredients
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at MC-BOOTSTRAP/com.google.common@31.1-jre/com.google.common.base.Preconditions.checkArgument(Preconditions.java:145)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/arclight@1.20.1-1.0.1-c3fc1d3/org.bukkit.inventory.ShapelessRecipe.addIngredient(ShapelessRecipe.java:130)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.world.item.crafting.ShapelessRecipe.bridge$toBukkitRecipe(ShapelessRecipeMixin.java:538)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	at TRANSFORMER/arclight@1.20.1-1.0.1-c3fc1d3/org.bukkit.craftbukkit.v1_20_R1.inventory.RecipeIterator.next(RecipeIteratorMixin.java:552)
[19dec.2023 21:59:00.709] [Server thread/ERROR] [STDERR/]: 	... 19 more
[19dec.2023 21:59:00.709] [Server thread/INFO] [mcMMO/]: Disabling mcMMO v2.1.224
[19dec.2023 21:59:00.709] [Server thread/INFO] [mcMMO/]: Server shutdown has been executed, saving and cleaning up data...
[19dec.2023 21:59:00.712] [Server thread/INFO] [mcMMO/]: [D] Completing 0 unfinished Alchemy brews.
[19dec.2023 21:59:00.717] [Server thread/INFO] [mcMMO/]: [D] [Party Data] Saving...
System (please complete the following information):

Minecraft Version 1.20.1
Modpack Link and Version, or list of mods Forge 47.1.1
Specific mods that you experience compatiblity issues with: A plugin called mcMMO

Closed in 058807df2a97855c9d6817769c5be84b39a3db4b but the above message still stands, mcMMO likely needs to adjust their code


etcdctlv3: version command proposal

@xiang90
etcdctl --version in etcdv2 just show client version. It would be much helpful if it can demonstrate server side version(for example all members' version).
I'd like etcdctl version command in etcdv3  print as follows:
Client Version: etcdctl version 3.0.0
Server Version: infra1 version 3.0.0
                infra2 version 3.0.0
                infra3 version 3.0.0

Does this sounds good?

etcdctl version is not a good place for this sort of thing. To get the server version, ctl needs to communicate with the server over the network. etcdctl version, on the other side, should be as simple as possible.
Another issue is that server version is not even exposed by v3 rpc (https://github.com/coreos/etcd/blob/master/etcdserver/etcdserverpb/rpc.proto). probably the first step is to define a memberStatus rpc in cluster service https://github.com/coreos/etcd/blob/master/etcdserver/etcdserverpb/rpc.proto#L65 to expose some server status.

Agree.


Roman/feature/edit item

Closes #70 #69

Travis automatic deployment: https://71-pr-ching.surge.sh

Travis automatic deployment: https://71-pr-ching.surge.sh

Travis automatic deployment: https://71-pr-ching.surge.sh

👍 It's coming in the next PR

Travis automatic deployment: https://71-pr-ching.surge.sh


CSS issue with dreditor 1.2.9 and Firefox 30.0

Please see attached. Patch filenames seem to be trimmed.

Here's how I temporarily fixed it:

FYI I have the browser zoom set to 150%

Dreditor is being decommissioned due to the lack of availability of current maintainers and the fact that Drupal.org is becoming more feature rich at a much faster rate than Dreditor itself now.
This decision was also partially made due to the amount of work (complete rewrite) it would take to create an "official" Firefox extension (#256).
If you already have the extension installed, it should continue working for a while until, inevitably,  the markup on Drupal.org changes. It will no longer be updated.
We understand that many of you love this extension, as do we. This isn't the end, in fact, it's a much brighter beginning. All the things we love about Dreditor can, and should, be moved into Drupal.org natively.
For more information, please follow: https://www.drupal.org/node/2779729

This issue was moved to dreditor/dreditor#15


Updated DefaultSqlSessionFactoryProvider.java

https://jira.cvent.com/browse/CPCTD-850
Need a new method to get sql session factory by env name, if not found then return default
More details -> https://wiki.cvent.com/display/~VibhorJ/Connection+issue+on+postgres

Should I add the method declaration in interface as well?

This seems like it puts the work on the caller to decide which approach they want to take. Which means anyone in the future could run in to the situation that the microsite service is in right now, because people won't know they're supposed to call this other method for Postgres session factories in staging.
Could you modify the current method to find if there is a session factory to the same database server, and if so, return that instead of creating a new one?

Also, this won't directly help you on microsite service, since microsite service uses dropwizard 0.7/dropwizard-mybatis 3.1.x, and master of this library is dropwizard 1.x/dropwizard-mybatis 4.0.x

The team is looking at alternative ways to solve this problem instead.


Remove date specific pull request warnings

After January 2021 CodeQL will no longer display pull request results if on.pull_request is not set.
Now we only need to warn if on.pull_request is configured without a corresponding on.push.

I guess it should be very rare people are depending on the old behavior now, so I think just removing the warning is also fine.

:+1: Cool. We can always put it back in if people run into it a lot.

I guess it should be very rare people are depending on the old behavior now, so I think just removing the warning is also fine.

:+1: Cool. We can always put it back in if people run into it a lot.


Zimg 3.1.0 log 不能rotate

请问，如何配置 zimg 的log rotation？

自带的log模块比较简单，没有这么高级的功能，建议用supervisor来启动，supervisor里自带了log切分。

谢谢回复，我配置了linux的 logrotate.d 解决了这个问题

Share the configuration here:
Suppose your Zimg located in /usr/local/zimg
vim /etc/logrotate.d/zimg   #zimg is application name, you can specify by yourself
Add following inputs:
/usr/local/zimg/zimg.log   #absolute path of your zimg log file
{
daily   #rotate log daily, you can use weekly or monthly. no others
dateext #append '-YYYY-MM-DD' in the rotated log file name
copytruncate #truncate the old file after rotate
nocompress  #don't compress the old log
rotate 3          #keep max 3 rotated log files
}
then:
service rsyslog restart  #maybe it's 'syslog', not 'rsyslog', depends on your OS version.

感谢分享！


remove obsolete record.go

In #636 this file became obsolete and can now be removed.

Seems like this will be fixed in #664


Question about semantic segmentation

Thanks for publishing such an excellent work ! It will be perfect if you could provide the semantic segmentation version. Really appreciate for your help !

For segmentation
Requirement:
Pytorch >= 1.7
MMSegmentation == v0.12.0 (https://github.com/open-mmlab/mmsegmentation/tree/v0.12.0)


Automated cherry pick of #11563: feat(host): aware of kubelet eviction config

Cherry pick of #11563 on release/3.7.
#11563: feat(host): aware of kubelet eviction config

/lgtm

/approve


Node.js  Loaders Team Meeting 2022-04-12

Time
UTC Tue 12-Apr-2022 18:00 (06:00 PM):

Timezone
Date/Time

US / Pacific
Tue 12-Apr-2022 11:00 (11:00 AM)

US / Mountain
Tue 12-Apr-2022 12:00 (12:00 PM)

US / Central
Tue 12-Apr-2022 13:00 (01:00 PM)

US / Eastern
Tue 12-Apr-2022 14:00 (02:00 PM)

EU / Western
Tue 12-Apr-2022 19:00 (07:00 PM)

EU / Central
Tue 12-Apr-2022 20:00 (08:00 PM)

EU / Eastern
Tue 12-Apr-2022 21:00 (09:00 PM)

Moscow
Tue 12-Apr-2022 21:00 (09:00 PM)

Chennai
Tue 12-Apr-2022 23:30 (11:30 PM)

Hangzhou
Wed 13-Apr-2022 02:00 (02:00 AM)

Tokyo
Wed 13-Apr-2022 03:00 (03:00 AM)

Sydney
Wed 13-Apr-2022 04:00 (04:00 AM)

Or in your local time:

https://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+Loaders Team+Meeting+2022-04-12&iso=20220412T1800
or https://www.wolframalpha.com/input/?i=06PM+UTC%2C+Apr+12%2C+2022+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1AfLy-RrwW7ZwkNKBcO7A9OWaEXFOeOdDaICUGtTn2Ao/edit

Agenda
Extracted from loaders-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
Invited

Loaders team: @nodejs/loaders

Observers/Guests
Notes
The agenda comes from issues labelled with loaders-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting

link for participants: https://zoom.us/j/97506450082

For those who just want to watch: https://www.youtube.com/c/nodejs+foundation/live

No agenda, do we have anything to meet about?
I’m excited by the progress in https://github.com/nodejs/node/pull/42623, but I don’t know of anything related to it that we need to meet to discuss.

I think I'll have the chaining PR ready by then. I don't foresee anything to discuss about it though. Touch wood, it's going way easier than expected.

Canceling per above.


Style Note App

I have added CSS to the Notes App.
Below is the screenshot of the same.
As well as added many of the effects like hover,letter spacing, font and bordering.
And the home button which was deactivated that is also activated.
Kindly check this.
If any changes please inform me.
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] New and existing unit tests pass locally with my changes.

Screenshots

Will you please look into this?

@SanskrutiRP  Please link the corresponding issue related to this PR.


Variables give an error

The variables {{date}} {{time}} in the front matter give an error. Looks like that {{date}} {{time}} does not need to be mentioned.

Did you activate the Obsidian core plugin "Templates"?

The {{date}} {{time}} will automatically be inserted when using "Insert template" from the menu. More about: https://help.obsidian.md/Plugins/Templates


fixed new user and new application api calls

@Klaven I am not expert in go but when you do r.Form.get  it's getting the fields from  the formData but ware sending the response in the body in json format. I might be wrong but i remember having testing it in the marketing api. Also you can validata using the validator.
like this.

	err = validate.Struct(marketingSubscribeDTO)

	if err != nil {
		errors := err.(validator.ValidationErrors)
		http.Error(w, fmt.Sprintf("Validation error: %s", errors), http.StatusBadRequest)
		return
	}

This should validate any condition  you put like this here it validate if it's in valid email format and it exisit
type MarketingSubscribeDTO struct {
	Email string `json:"email" validate:"required,email"`
}


Add toggles for requireEmailUpfront and showAgentAvailability on getting started page on dashboard

Is your feature request related to a problem? Please describe.
When a non technical user wants to customize the widget for requiring email upfront and showing agent availability they can't do it through the GUI.
Describe the solution you'd like
There should be a toggle for both requireEmailUpFront and showAgentAvailability in the https://github.com/papercups-io/papercups/blob/master/assets/src/components/account/GettingStartedOverview.tsx#L242 section where it generates and the flags for the widget
Additional context
This is a customer request from inStock

Should this also incorporate other widget options (as indicated here: https://github.com/papercups-io/chat-widget#options)? And just so I'm certain, this only affects the generated HTML/React installation scripts, correct?

Something like this:

hey @markholmes! awesome, looks pretty good so far :)
only feedback would be:

true and false need to be booleans in the code snippets rather than strings
you'll also need to apply these updates to the widget on the page so the user can see what it looks like when e.g. showAgentAvailability is true

let me know if you have any questions! going to assign this to you for now 👍

hey @markholmes! any updates on the widget side of this? (i remember you were working on it, sorry if i missed any latest updates!)

Since I'm already looking at this page, I can look at this too.

Oops I believe this is done now! We can close this.
On Fri, Mar 5, 2021 at 9:21 AM Andy Tran<EMAIL_ADDRESS>wrote:

Since I'm already looking at this page, I can look at this too.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/papercups-io/papercups/issues/297#issuecomment-791449088,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIFHF3CYX3OYNROJ5ZVRG3TCDSIHANCNFSM4SJEVCWA
.


Develop

Coverage remained the same at 86.535% when pulling be43b0ba6b7b77e8a15516de47e73b454ff27be9 on develop into 44e15345a3feaa8acae1bca99c9c19705eb7c376 on master.


Cascading filters in publishing mode

This is probably an improvement and not a bug.
Currently cascading filters only work in edit mode.  Cascading filters should also work in publish mode if possible. To be discussed depending on the investment needed.
see exemple here :
https://test.visualize.admin.ch/fr/v/xAVhqoA3PtsH

If they are cascading, interactive filters must be sortable.


[Enhance]For trace CompletableFuture thenApply or thenAccept

https://github.com/apache/skywalking/issues/7991
I'm not sure if this is a common feature.
Follow the example of SupplierWrapper，enhance Function , Consumer.
Thanks.

@will2020-power Please follow my comments, and make sure the tests passed locally.

I can see your tests passed. Please follow the review comments to update the document.
This PR should be generally good. Thanks.

https://github.com/apache/skywalking-java/blob/main/CHANGES.md, this missed. I think this is the last piece.


Potential Bug in Image Resizing for interactive_infer_image

Hi,
Two potentials bugs at /scratch/ssd004/datasets/med-img-data/mohammed/BiomedParse/inference_utils/inference.py.
1:
https://github.com/microsoft/BiomedParse/blob/6f731d0a6ebf3d3f3ad1c2db9cf26d405f595984/inference_utils/inference.py#L22
The line above only resizes one of the dimensions. If you start with an image with shape (1080, 1240, 3) and apply the above transformation, you will end up with (1024, 1175, 3). Instead, it should be changed to  t.append(transforms.Resize((1024, 1024), interpolation=Image.BICUBIC))
2:
https://github.com/microsoft/BiomedParse/blob/6f731d0a6ebf3d3f3ad1c2db9cf26d405f595984/inference_utils/inference.py#L74-L78
in the above, the goal is to resize to the original size. But data['height'] and data['width'] does not save the original size.
https://github.com/microsoft/BiomedParse/blob/6f731d0a6ebf3d3f3ad1c2db9cf26d405f595984/inference_utils/inference.py#L40-L47
They are storing the image size after resizing.

Thanks for identifying the issue! We have fixed the image resizing problem. The issue was hidden because all the image file loading utils functions was doing resizing beforehand, so the corner cases were all overwritten by the desired size.


 It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 70fbb1c8-1435-e762-8824-78cf96faed84
Version Independent ID: 062879b5-72cf-2353-63f2-d862fe1b40ba
Content: Microsoft Docs contributor guide overview - Contributor Guide
Content Source: Contribute/index.md
Product: non-product-specific
GitHub Login: @Jim-Parker
Microsoft Alias: jimpark

The Microsoft Open Source Code of Conduct, which outlines the expectations for community interactions in and around docs.microsoft.com, is designed to help "provide a welcoming and inspiring community for all." The content of this issue appears to be out of sync with the code of conduct, so we have closed it.


Question regarding train.ipynb - num_tokens

see: https://github.com/yl4579/PL-BERT/blob/main/train.ipynb
bert = AlbertModel(albert_base_configuration)
bert = MultiTaskModel(bert,
                      num_vocab=max([m['token'] for m in token_maps.values()]),
                      num_tokens=config['model_params']['vocab_size'],
                      hidden_size=config['model_params']['hidden_size'])

num_tokens=config['model_params']['vocab_size'] = 178. However, if you view the len(dicts) = 177
_pad = "$"
_punctuation = ';:,.!?¡¿—…"«»“” '
_letters = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz'
_letters_ipa = "ɑɐɒæɓʙβɔɕçɗɖðʤəɘɚɛɜɝɞɟʄɡɠɢʛɦɧħɥʜɨɪʝɭɬɫɮʟɱɯɰŋɳɲɴøɵɸθœɶʘɹɺɾɻʀʁɽʂʃʈʧʉʊʋⱱʌɣɤʍχʎʏʑʐʒʔʡʕʢǀǁǂǃˈˌːˑʼʴʰʱʲʷˠˤ˞↓↑→↗↘'̩'ᵻ"

# Export all symbols:
symbols = [_pad] + list(_punctuation) + list(_letters) + list(_letters_ipa)

letters = list(_letters) + list(_letters_ipa)

dicts = {}
for i in range(len((symbols))):
    dicts[symbols[i]] = i

So, we have only 177 unique phoneme character, based in the length of the dict. We would have a length of 178, if we would count the duplicate sign in _letters_ipa twice.  Maybe there is a special phoneme that is not stored inside the dicts, that I am missing here? So the question is, why do we have num_token = 178 and not 177?
Thank you for effort

@jav-ed
I see that the symble "'" is duplicatied in _letters_ipa. As long as the dicts  is used for phoneme2id mapping everywhere, it does not make problem.

@tekinek
Sorry, I do not understand this. From what I understand, the number of tokens (num_tokens) is a parameter that decides how many phonemes the model will try to learn. The higher the number (num_tokens) the more phoneme characters, the model will try to learn. We can put it also in another way, the higher the number of tokens, the more flexibility the model will have to learn something. If something is only linear, but we gave it the freedom of a let's say cubic (x^3) function then, within the range of the training data, we have a very good interpolation. However, outside the seen training data, the extrapolation can be assumed to be terrible.
So, allowing the model too much freedom could have negative consequences.

By the way, @tekinek yes are right with the found duplicate symbol

Yes, setting 178 is to prevent out of index issue (even though there are only 177 symbols and one duplicate token). We can repupose some tokens (for example, the last 177th token) for some other use later if we need it.


Bug 1800628: Switch to ghodss yaml package

The gopkg.in YAML package does not respect json tags in types, which breaks our policy config when it is marshalled down. ghodss' yaml works by first converting to JSON, which applies the correct tags.

/lgtm

/bugzilla refresh


TRIQ failure on images of particular size range

Hello,
Thank you for the great implementation of TRIQ.
I am able to run TRIQ successfully on most images, however it seems a particular range of resolutions causes failure.
First, I load in TRIQ model,
args = {}

args['n_quality_levels'] = 5
args['backbone'] = 'resnet50'
args['weights'] = 'path/TRIQ.h5'

model = create_triq_model(n_quality_levels=args['n_quality_levels'],
                          backbone=args['backbone'])

model.load_weights(args['weights'])

An example image link is below,
_https://hpmlawatl.com/wp-content/uploads/2013/07/640x4802.gif_
test_image = "/path/640x4802.gif"
image = Image.open(test_image).convert('RGB')
image = np.asarray(image, dtype=np.float32)
image = image[:,:,:3]
image /= 127.5
image -= 1.
prediction = model.predict(np.expand_dims(image, axis=0))

This shows an error,
InvalidArgumentError:  required broadcastable shapes
	 [[node model/tri_q_image_quality_transformer/add_1
 (defined at /home/ubuntu/production/triq/src/models/transformer_iqa.py:197)
]] [Op:__inference_predict_function_10094]

However, I can then resize the same image, to sizes both LARGER OR SMALLER, and the image will run successfully. As an example, this image can be set to either 512 X 384 OR 1024 X 768 and TRIQ will run fine.
test_image = "/path/640x4802.gif"
image = Image.open(test_image).convert('RGB')
img_sizes = image.size
print("Original Image size is, " + str(img_sizes[0])+  " " + str(img_sizes[1]))

size_cutoff = 1024 # This sets to 1024 X 768
size_cutoff = 512 # This sets to 512 X 384

if img_sizes[0] != size_cutoff and img_sizes[1] != size_cutoff:
    max_size = max(img_sizes)
    scale_factor = size_cutoff / max_size
    x_dim = round(img_sizes[0]*scale_factor)
    y_dim = round(img_sizes[1]*scale_factor)
    image = image.resize((x_dim,y_dim),Image.ANTIALIAS)

image = np.asarray(image, dtype=np.float32)
image = image[:,:,:3]
image /= 127.5
image -= 1.
prediction = model.predict(np.expand_dims(image, axis=0))

In order to pin this down I did a bit of empirical testing, and:
Values of  size_cutoff = 513 will fail, while size_cutoff = 512 is okay.
Similarly, size_cutoff = 1057 will fail while size_cutoff = 1056 is okay.
I understand if an image is too small or large the TRIQ will fail. What I am not understanding is why images of a particular size (640X480) will fail, while the same image resized to be smaller (512, 384) or larger (1024, 768) will run successfully.
Any insight you have would be helpful.

Hi, thanks for finding this bug. There is a potential bug in line 180 in transformer_iqa.py. I don't have time to do a full test now, so I won't make any changes. But if you rotate your image to 480x640, the model will work. Or you can change line 180 to:
if tf.shape(x)[1] >= 16 or tf.shape(x)[2] >= 18:
It will also work. So this is not because of this particular size, it's because the spatial pooling is not performed which will cause a problem.


Can't determine working URL for this file

https://github.com/AnnArborDetroit-CPP/aadcpp-meetup-content/blob/working_branch/05-2018/PITCHME.md
It's on a feature branch, and in a subfolder, is there a limitation where this scenario doesn't work?
I'm trying this URL and getting "Presentation 404" error:
https://gitpitch.com/AnnArborDetroit-CPP/aadcpp-meetup-content/working_branch?05-2018
If I replace working_branch with master, it works, but of course it's cached so I cannot get updates in a reasonable timeframe.

It's on a feature branch, and in a subfolder, is there a limitation where this scenario doesn't work?

No limitation. If you want to render a PITCHME.md within a subfolder you must use the p query param on your presentation URL. This is true, regardless of what branch you are using:
For example:
https://gitpitch.com/AnnArborDetroit-CPP/aadcpp-meetup-content/working_branch?p=05-2018

Click here to see your presentation live.
More details on the p query parameter here.

That was it, thank you!


constraint build slow

    obj_driver= solver.NumVar(-M,M,'obj_driver')
    solver.Add( obj_driver == solver.Sum(driver_weight[i]*dis_mat[i, j] * x[i, j] for i in range(m) for j in range(n)) )
    print('obj_driver built completed..' )

i: hundreds
j: hundreds
took a long long time to build this constraint
any suggestions to improve the performance?

Use the non algebraic API
use the routing library :-)
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00

Le jeu. 24 oct. 2019 à 13:39, xuanc<EMAIL_ADDRESS>a écrit :

obj_driver= solver.NumVar(-M,M,'obj_driver')
solver.Add( obj_driver == solver.Sum(driver_weight[i]*dis_mat[i, j] * x[i, j] for i in range(m) for j in range(n)) )
print('obj_driver built completed..' )

i: hundreds
j: hundreds
took a long long time to build this constraint
any suggestions to improve the performance?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1663?email_source=notifications&email_token=ACUPL3KUXA2IBLYRLICBIK3QQGCOTA5CNFSM4JESYVR2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HUCZOPQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3OQY6NTEKPX6HAAOI3QQGCOTANCNFSM4JESYVRQ
.

Thx Perron
Can you be more specific?
Any specific examples?
A little bit overwhelmed~

Googled or-tools & non algebraic API, got nothing valuable.
Actually I'm solving a location model (c.f. p-median location model). Took 20+ minutes to build the following constraint, yet solved to OPTIMAL after pre-processing ^_^
solver.Add( obj_driver == solver.Sum(driver_weight[i]*dis_mat[i, j] * x[i, j] for i in range(m) for j in range(n)) )

Would the routing library be helpful to a location problem on CONSTRAINT BUILDING instead of SOLVING to OPTIMAL?

https://github.com/google/or-tools/blob/stable/ortools/linear_solver/samples/linear_programming_example.py
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le jeu. 24 oct. 2019 à 13:57, xuanc<EMAIL_ADDRESS>a écrit :

Would the routing library be helpful to a location problem on CONSTRAINT
BUILDING instead of SOLVING to OPTIMAL?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1663?email_source=notifications&email_token=ACUPL3O6CELHVO6BSARN2KLQQGERDA5CNFSM4JESYVR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECEYLXI#issuecomment-545883613,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3PLVELRRHHGMD2LOTDQQGERDANCNFSM4JESYVRQ
.

Thx
You mean building the constraint in the following way would accelerate the contraint-building process?
Thx. I shall try that.
constraint0 = solver.Constraint(-solver.infinity(), 14)
constraint0.SetCoefficient(x, 1)
constraint0.SetCoefficient(y, 2)

Worth trying.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le jeu. 24 oct. 2019 à 14:16, xuanc<EMAIL_ADDRESS>a écrit :

Thx
You mean building the constraint in the following way would accelerate the
contraint-building process?
Thx. I shall try that.
constraint0 = solver.Constraint(-solver.infinity(), 14)
constraint0.SetCoefficient(x, 1)
constraint0.SetCoefficient(y, 2)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1663?email_source=notifications&email_token=ACUPL3II6RK7APJYRE3DMQLQQGGZHA5CNFSM4JESYVR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECEZ6II#issuecomment-545890081,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3MQLTYMVUY4HO5WWS3QQGGZHANCNFSM4JESYVRQ
.

    obj_driver= solver.NumVar(-M,M,'obj_driver')

solver.Add( obj_driver == solver.Sum(driver_weight[i]*dis_mat[i, j] * x[i, j] for i in range(m) for j in range(n)) )
    obj_driver_constraint = solver.Constraint(0, 0)
    obj_driver_constraint.SetCoefficient(obj_driver, -1)
    for i in range(m):
        if 1==i%100:
            print('driver: i ', i)
        for j in range(n):
            obj_driver_constraint.SetCoefficient(x[i,j], driver_weight[i]*dis_mat[i, j] )
    print('obj_driver built completed..' )

That should work faster?
Is there an elegant way to vectorize the two hundreds times hundreds for cycles of the python code?

Not using python  :-)

Python API is important for community growth ..
For instance, Tensorflow / Pytorch/ Keras ...

Vectorizing the two-fold for cycles would accelerate the Python program several orders of magnitude

import time
import numpy as np
a = np.random.rand(1000000)
b = np.random.rand(1000000)
tic = time.time()
c = np.dot(a,b)
toc = time.time()
print(c)
print("Vectorized version: " , str(1000*(toc-tic)) + "ms")
c = 0
tic1 = time.time()
for i in range(1000000):
c += a[i]b[i]
toc1 = time.time()
print(c)
print("For loop version: " , str(1000(toc1-tic1)) + "ms")
249801.55978809926
Vectorized version:  1.2989044189453125ms
249801.55978811116
For loop version:  1104.9799919128418ms

I welcome any contribution to support numpy in the linear solver, or on the
flow algorithms.
I do not have the time to do it.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le jeu. 24 oct. 2019 à 14:41, xuanc<EMAIL_ADDRESS>a écrit :

import time
import numpy as np
a = np.random.rand(1000000)
b = np.random.rand(1000000)
tic = time.time()
c = np.dot(a,b)
toc = time.time()
print(c)
print("Vectorized version: " , str(1000*(toc-tic)) + "ms")
c = 0
tic1 = time.time()
for i in range(1000000):
c += a[i]
b[i] toc1 = time.time() print(c) print("For loop version: " , str(1000(toc1-tic1))

"ms")

249801.55978809926
Vectorized version: 1.2989044189453125ms
249801.55978811116
For loop version: 1104.9799919128418ms
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1663?email_source=notifications&email_token=ACUPL3M5CKY7M2FUYJUUZRLQQGJXTA5CNFSM4JESYVR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECE4EUI#issuecomment-545899089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3KQ3SLA2H6RYBKW2H3QQGJXTANCNFSM4JESYVRQ
.

Got.
My current issue  is on model BUILDING, instead of SOLVER / ALGORITHM.

Exactly, solving is in C++.
Building the model is in python, and is slow.


[Large scale test]Create 1000 applications and bind them to pvc, then delete the application and pvc, but the lv is not deleted automatically.

Create 1000 applications and bind them to pvc, then delete the application and pvc, but the lv is not deleted automatically.
[root@k8s-master ~]# kubectl get lv |wc -l
313
[root@k8s-master ~]# kubectl get pvc
No resources found in default namespace.
[root@k8s-master ~]# kubectl get pv
No resources found

can you provide csi-provisoner container log and member container log of local-storage?


DNS resolution state bug

Hello,
There is a bug in how resolve.status is set.
When nginx actually does the lookup, ngx_http_upstream_jdomain_resolve_handler is called from polled events which works fine.
But when TTL is set to automatic (no valid argument for resolver directive) and if it is very high (like 30+ seconds, maybe even less), ngx_http_upstream_jdomain_resolve_handler is called from cached values in ngx_resolve_name_locked at line 669.

As you can see the function stack in bottom left, ngx_http_upstream_init_jdomain_peer directly calls ngx_http_upstream_jdomain_resolve_handler.
The bug is caused when above happens. As ngx_http_upstream_jdomain_resolve_handler sets NGX_JDOMAIN_STATUS_DONE for instance resolve status, it returns to  ngx_http_upstream_init_jdomain_peer which sets NGX_JDOMAIN_STATUS_WAIT as the last step in the loop.
This bug is irreversible as once this happens and instance gets set to NGX_JDOMAIN_STATUS_WAIT, it never does another lookup again and stays stuck with old peer address.
I think fixing this bug will resolve #60 and #61

@atomsnc Did you get nginx to run in a debugger with build process integration?
Well done! Care to share your setup steps / environment?

Hello @splitice, I use CLion as it now has Makefile support built-in. You just load Makefile as a project.
https://www.jetbrains.com/help/clion/makefiles-support.html#manual-reload
Since nginx has a pre-make step, you add ./configure to pre-build command.
Once you build it, you have two options (both require daemon mode to be off),

If you disable master process (single nginx process, no worker), you can start a debug build from the start menu.
If you want to run with worker process, you will have to start it manually (script works well), and then attach to the process to debug.

Then you can just set breakpoints in IDE window and it works.
You can set access and error logs to /dev/stdout so you have the full experience in the IDE itself.

Hey @atomsnc , given that #75 is merged, do you suppose we can close this one?

@nicholaschiasson Yes this can be closed. Please go ahead.


Update CCHMapClusterController.h

Added updateAnnotations method to header file. In some cases you have to manually update the cluster annotations. I don't know if this is the best solution but in my app works perfectly.

Can you tell me a bit more about when you need to call updateAnnotationsWithCompletionHandler:? I would prefer not to expose this method, because the library should automatically update the clusters when needed.


Ability to know the slave nodes

In case a master dies, users of the storagecluster need to know the slavenodes so they can continue to operate until ays fixes it.

SV
grid faeae03
jumpscale 92c7a2d


Execution takes far more time than it should

x = [[i for i in range(10000000)]]

Timings in ms:

List Size
Python Time
Total Time (py & JS)

10^4
0
23

10^5
2.98
225

10^6
48.1
2363

10^7
489
24337

I'd be okay with a doubling or even tripling of the execution time but this is clearly rediculous.  Only ~2% of the execution time is actually spent executing the user's code.    Sipping martinis on the beach? 🏖 🍸

i moved issue over to AREPL-backend


[rcore] Get keys that are pressed again, like chars

Feature request
I'm looking for a good way to get the keys that have been pressed again (repeated, like chars) on desktop.
The current way of achieving this for me is to loop through all the possible (or needed) keys and check them with IsKeyPressedRepeat(). It's a bit ugly.
I'm thinking something like GetKeyPressedRepeat could be added to rcore. Personally don't need to differentiate between keys that have been "just pressed" or "pressed again", but I could use it together with GetKeyPressed.
Environment
PLATFORM_DESKTOP, since IsKeyPressedRepeat is desktop only.

@constantitus It has been already discussed in the past and there is no intention to add those functions for now. You should manage it at user level as you are doing, it's not ugly, it would be exactly the same done internally.


v10 chatBubbles API change impact

Describe the bug
I believe the API for the chatBubbles has changed in Foundry V10 so that the current method Polygot provides for passing in a language for the chat bubble is no longer working.
API for .say() and .broadcast(): https://foundryvtt.com/api/v10/classes/client.ChatBubbles.html#broadcast
in v9 the language was passed in on an emote key for the options object ( {emote: {language: 'elvish'}}) but with the v10 change I don't believe the emote will clear through on the hook options. The new cssClasses key may possibly be able to be leveraged here in place of emote, though I don't know how janky that is. adding an object to this key passes through to the chatBubble hook that I believe Polyglot is reading as show below
{cssClasses: {language: 'elvish'}}
To Reproduce
See https://github.com/League-of-Foundry-Developers/fvtt-module-polyglot/wiki/Macros#canvashudbubblessay
Expected behavior
I expect if on my session I invoke a chat bubble with the appropriate key set for the options data to pass in a language that on the player's session, if they don't speak the language, they will see the chat bubble in the polyglot markup
v10 285

Game System and version dnd5e 2.0.2
Polyglot Version 1.8.3
Are you using Babele? no

That cssClasses override was a good hint, but it would be super janky. It is sent to the chat bubble render template before it gets sent to the chatBubble hook, so the data would go to the bubble class before I had any chance to filter it out before.
And chatBubbles hook wasn't getting sent a html object anymore, so it needed to be changed too.
My solution was to override ChatBubbles.prototype.say to have an options.languages parameter and adapted the old logic to make use of cssClasses properly.

brilliant! thank you!


How Device registration with a Notification Hub is accomplished using a Installation in react native app

In this link : https://docs.microsoft.com/en-us/azure/notification-hubs/notification-hubs-push-notification-registration-management
Device registration with a Notification Hub is accomplished using a Installation is explained but how I can use it in React native app.

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

ID: 431c3a81-db2a-b34d-0f2c-50d71dc10c49
Version Independent ID: c44dd034-b17a-663f-2f33-99c1f04edfbe
Content: Push notifications to iOS apps using Azure Notification Hubs
Content Source: articles/notification-hubs/notification-hubs-ios-apple-push-notification-apns-get-started.md
Service: notification-hubs
GitHub Login: @jwargo
Microsoft Alias: jowargo

We are trying to implement installation process for registering device using REST API in React Native code, but how can we create "pushChannel" in React Native code?

@MayurBar a Push Channel is just a string value, the name for the channel.

This was addressed in a different conversation, #please-close.


Add module-info.java to library

Please add module-info.java to this library. At first glance, the following content should do:
module io.github.javadiffutils
{
  exports com.github.difflib;
  exports com.github.difflib.patch;
}

At the moment this project is build using Java 8. So I would not be able to build it including this module stuff.

@wumpz You can release a single JAR file that works for both Java 8 and newer versions of Java simultaneously.
See the Add execution for Java 8 and Add execution for Java 11 sections of<EMAIL_ADDRESS>as an example, or more generally look up how multiReleaseOutput works.

This issue still exists.

Sure but as mentioned above at the moment no module-info.java will be added. However using mavens jar plugin the automatic module name is set:
 <Automatic-Module-Name>io.github.javadiffutils</Automatic-Module-Name>

@wumpz Did you read the part where I explained you could add module-info.java while maintaining JDK 8 support?

@cowwoc I read it, but I am no fan of this multi version jars. If I upgrade to a higher Java version, this will definitly included, but not now.


readme patch task close

closes #119

@rultor merge

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 1min)


Optional scripts for machines

@dannyroberts here was my idea for automating the log pulls for amelia. also think it'd be nice to use this playbook to deploy things optionally, like the touchforms restarter script that we have on only the prod machine
cc: @TylerSheffels

(note i haven't actually run this code yet, will play around with it once i deploy the log format)

neat! Looks good

cool g2g


Typescript issue when navigating without passing the to

Which project does this relate to?
Router
Describe the bug
Considering a route like:
/users/$userId/details
/users/$userId/posts
/dummy/$dummyId
If we have a component that is used in the users routes that knows how to navigate to a different user, keeping the current location (details or posts),
then we can use the navigate returned from useNavigate to do so. It was suggested not to provide a to to have this working properly, and it works but Typescript complaints:

with navigate({ params: { userId: value }}); it complains that Object literal may only specify known properties, and userId does not exist in type
with navigate({ params: (prev) => ({...prev, userId: value })}); it complains that the { userId: string } is not assignable to type never

Using the from or the to, parameters solves the typescript issue, but we need to use a specific route otherwise we lose the current location,
One possibility is to use, but it might be overcomplicating the solution.
const matches = useMatches();
const to = matches[matches.length - 1].fullPath;

navigate({to, params: { userId: value });

This issue is a followup of the discussion on Discord
Your Example Website or App
https://stackblitz.com/edit/tanstack-router-xlb7ip?file=app%2Fcomponents%2FNextUserNavigator.tsx
Steps to Reproduce the Bug or Issue
Use navigate to change only one parameter keeping any sub-route.
Expected behavior
A cleaner way to change only the parameter without TS complaints.
Screenshots or Videos
No response
Platform

OS: Linux
Browser: Chrome
Version: 130

Additional context
No response

Just so I'm clear on the issue at hand here.
In this configuration, the type error goes away when setting the from value to /users/$userId/details during the navigate call, but it exits the sub-route as a whole.

Whereas, when I used this modified configuration of having the from value be added onto useNavigate hook, it fixes the navigation whilst giving you the correct types.

TLDR; So what we are looking to fix here is only the runtime behaviour right?

Also, for clarity on the PR, this also works.

Just to clarify that the problem with the solution of not passing a to or a from, it is not a runtime issue, but a TypeScript one. On your example we can see:

The TS error is:

Object literal may only specify known properties, and 'userId' does not exist in type 'ParamsReducerFn<Router<RootRoute<undefined, {}, AnyContext, AnyContext, {}, undefined, RootRouteChildren, FileRouteTypes>, TrailingSlashOption, boolean, Record<...>, Record<...>>, "PATH", string, string | undefined>'.(2353)
link.d.ts(65, 5): The expected type comes from property 'params' which is declared here on type 'NavigateOptions<Router<RootRoute<undefined, {}, AnyContext, AnyContext, {}, undefined, RootRouteChildren, FileRouteTypes>, TrailingSlashOption, boolean, Record<...>, Record<...>>, string, string | undefined, string, "">'
(property) userId: string


minor typo/grammar fixes for comments

@shime could you just rebase before I can merge, please?

@CodeTheory sorry, rebased.


Chapter 3 Markdown with Images (#46)

Signed-off-by: atc-vince<EMAIL_ADDRESS>Added Chapter 3 images and fixed Supply Chain II TOC link
Signed-off-by: atc-vince<EMAIL_ADDRESS>Chapter 3 Markdown and commits squashed

@tkuhrt Hi Tracy,
Requested changes implemented.  Links in TOC also tested.
-Vince


Describe static site behaviour better in documentation

Is your feature request related to a problem? Please describe.
I am using nuxt in static mode to generate my web site, and recently added a footer to provide cookie usage information. I am using the browser's localStorage to save the user's acceptance and privacy settings, and a v-if to hide the footer if they have accepted the info.
I've added the code into the footer itself, reading and setting the localStorage during initialisation, and this works fine in dev mode and in mixed mode, but fails in full static mode. Right now I can't quite work it out but it looks as though both my asyncData and fetch calls get rendered away during static rendering, but create or mount are still there - however, the v-if statement is triggered during render and it messes up the display of the footer.
I'm still stuck trying to get a simple footer to display or not based on localStorage in the browser
Describe the solution you'd like
The nuxt lifecycle is beautifully documented for the situation where you're running in SSR or universal rendered mode, but this doesn't seem to be the same lifecycle that ends up in a statically rendered site. It would be incredibly helpful if there was more documentation around how a statically rendered site works.
Describe alternatives you've considered
working it out myself
Additional context
none

I agree. Would love to know more about how static works and best practices for using it.

Just been bitten by the static generation again - apparently the fetch() function never makes it into the static site :-(
Where are you supposed to put code that accesses dynamic content within static pages?

Just been bitten by the static generation again - apparently the fetch() function never makes it into the static site :-(
Where are you supposed to put code that accesses dynamic content within static pages?
@davydnorris

You're able to set fetchOnServer: false in your components to disable it running when being statically generated.


Improve benchmark code

This is a fairly low priority PR for tidying up the benchmark code. I've already made some quick changes that, in my opinion, improve readability.
This includes:

Ensuring spaces either side of an =
Enforcing a 100-column line length
Spacing out if statements
Ensuring arguments to functions are separated by spaces

Other things that could be improved:

Commenting
Cleaning up the timing and error calculation to avoid repetition.

I've changed this to a PR for general improvements of the benchmark code, since the latest commit on verbosity levels is not exactly about tidying up.

I definitely think that the gstats output should be with "-v" as well.
"-vv" is more for debugging.
What would be the intent of running the benchmark without "-v" (i.e. no output)?
The only reason I could think of is to run it with a profiler, or see if it crashes?

Yeah I haven't thought that through 😅
It's a benchmark, so people will be running it to see how fast it goes primarily. Then, some basic profiling should be provided at verbosity level 0 (i.e. without -v or -vv).
As you say,  we could have only verbosity levels 0 (equal to current -v level 1), and 1 (equal to current -vv level 2).

I was just running
ectrans-benchmark-dp --grid O320 -t 319 -n 10 --nfld 20

And I think we should aim for output like this:
======= Start of spectral transforms  =======

time step      1 took  4.5541
time step      2 took  5.2916
time step      3 took  5.0156
time step      4 took  4.8555
time step      5 took  4.8176
time step      6 took  4.3537
time step      7 took  4.5978
time step      8 took  4.3869
time step      9 took  4.0378
time step     10 took  4.5145

======= End of spectral transforms  =======

======= Start of time step stats =======

Inverse transforms
------------------
avg  (s):   2.4532
min  (s):   2.0991
max  (s):   2.8292
med  (s):   2.3802

Direct transforms
-----------------
avg  (s):   2.1893
min  (s):   1.9387
max  (s):   2.5616
med  (s):   2.1302

Inverse-direct transforms
-------------------------
avg  (s):   4.6425
min  (s):   4.0378
max  (s):   5.2916
med  (s):   4.5541
loop (s):  46.4677

======= End of time step stats =======

===-=== START OF TIMING STATISTICS ===-===

STATS FOR ALL TASKS
 NUM ROUTINE                                     CALLS      MEAN(ms)       MAX(ms)   FRAC(%)  UNBAL(%)
   0 transform_test - Main transform loop            1     46469.821     46469.821    100.00      0.00
 102 LTINV_CTL      - Inv. Legendre transform       10       269.444       269.444      5.80      0.00
 103 LTDIR_CTL      - Dir. Legendre transform       10       302.693       302.693      6.51      0.00
 106 FTDIR_CTL      - Dir. Fourier transform        10       521.794       521.794     11.23      0.00
 107 FTINV_CTL      - Inv. Fourier transform        10       578.920       578.920     12.46      0.00
 140 SULEG          - Comp. of Leg. poly.            1       132.831       132.831      0.29      0.00
 152 LTINV_CTL      - M to L transposition          10        45.117        45.117      0.97      0.00
 153 LTDIR_CTL      - L to M transposition          10        82.178        82.178      1.77      0.00
 157 FTINV_CTL      - L to G transposition          10      1546.665      1546.665     33.28      0.00
 158 FTDIR_CTL      - G to L transposition          10      1261.537      1261.537     27.15      0.00
 400 GSTATS         - GSTATS itself               3515         0.002         0.002      0.01      0.00
TOTAL MEASURED IMBALANCE =       0.0 SECONDS,  0.0 PERCENT
TOTAL WALLCLOCK TIME      46.470 CPU TIME    126.775 VECTOR TIME     126.775

===-=== END   OF TIMING STATISTICS ===-===

Now, would it make sense to add some more gstats labels so that the timings of the time step listed in the first sections are also in gstats?

time steps
Inverse transforms
Direct transforms

Further I was thinking that perhaps norm computations are something we want to output and test in the tests, but not necessarily with the full verbose output.
Could we add a "--norms" option to enable norms computations? not by default enabled with (current) "-vv".
It should be documented in the "--help" that this will skew the timings. That could also be made clear by doing the norms computations within a gstats region.

Interesting, yes. I mean, if we leave profiling entirely to GSTATS then we wouldn't need to do any of the current manual profiling anyway. I will have a play around later.

And additionally, even without verbosity, I would be interested in the time spent in the trans_setup.
We could also introspect memory.

record memory before setup.
record memory after setup.
record memory before/after each time step. The memory should not grow much between time steps. Between first and second there will be a difference because of additional memory allocations on first use.

How is that done in practice? I can do this manually with, e.g.
ectrans-benchmark-sp <options> 2>&1 > /dev/null & watch "grep VmRSS /proc/$!/status"

Do we need to use a C standard library call to get such a number from within the benchmark? The only solutions I've seen from within Fortran involve literally reading /proc/<pid>/status, e.g..

I was just running
ectrans-benchmark-dp --grid O320 -t 319 -n 10 --nfld 20

And I think we should aim for output like this:
======= Start of spectral transforms  =======

time step      1 took  4.5541
time step      2 took  5.2916
time step      3 took  5.0156
time step      4 took  4.8555
time step      5 took  4.8176
time step      6 took  4.3537
time step      7 took  4.5978
time step      8 took  4.3869
time step      9 took  4.0378
time step     10 took  4.5145

======= End of spectral transforms  =======

======= Start of time step stats =======

Inverse transforms
------------------
avg  (s):   2.4532
min  (s):   2.0991
max  (s):   2.8292
med  (s):   2.3802

Direct transforms
-----------------
avg  (s):   2.1893
min  (s):   1.9387
max  (s):   2.5616
med  (s):   2.1302

Inverse-direct transforms
-------------------------
avg  (s):   4.6425
min  (s):   4.0378
max  (s):   5.2916
med  (s):   4.5541
loop (s):  46.4677

======= End of time step stats =======

===-=== START OF TIMING STATISTICS ===-===

STATS FOR ALL TASKS
 NUM ROUTINE                                     CALLS      MEAN(ms)       MAX(ms)   FRAC(%)  UNBAL(%)
   0 transform_test - Main transform loop            1     46469.821     46469.821    100.00      0.00
 102 LTINV_CTL      - Inv. Legendre transform       10       269.444       269.444      5.80      0.00
 103 LTDIR_CTL      - Dir. Legendre transform       10       302.693       302.693      6.51      0.00
 106 FTDIR_CTL      - Dir. Fourier transform        10       521.794       521.794     11.23      0.00
 107 FTINV_CTL      - Inv. Fourier transform        10       578.920       578.920     12.46      0.00
 140 SULEG          - Comp. of Leg. poly.            1       132.831       132.831      0.29      0.00
 152 LTINV_CTL      - M to L transposition          10        45.117        45.117      0.97      0.00
 153 LTDIR_CTL      - L to M transposition          10        82.178        82.178      1.77      0.00
 157 FTINV_CTL      - L to G transposition          10      1546.665      1546.665     33.28      0.00
 158 FTDIR_CTL      - G to L transposition          10      1261.537      1261.537     27.15      0.00
 400 GSTATS         - GSTATS itself               3515         0.002         0.002      0.01      0.00
TOTAL MEASURED IMBALANCE =       0.0 SECONDS,  0.0 PERCENT
TOTAL WALLCLOCK TIME      46.470 CPU TIME    126.775 VECTOR TIME     126.775

===-=== END   OF TIMING STATISTICS ===-===

Now, would it make sense to add some more gstats labels so that the timings of the time step listed in the first sections are also in gstats?

time steps
Inverse transforms
Direct transforms

Further I was thinking that perhaps norm computations are something we want to output and test in the tests, but not necessarily with the full verbose output. Could we add a "--norms" option to enable norms computations? not by default enabled with (current) "-vv". It should be documented in the "--help" that this will skew the timings. That could also be made clear by doing the norms computations within a gstats region.

I definitely agree with the idea of --norms :+1:

Question about dimensions setup from sumpini (l 285 in your revised version) do we want to expose something like nprtrv as a run-time argument ?

Question about dimensions setup from sumpini (l 285 in your revised version) do we want to expose something like nprtrv as a run-time argument ?

Good point. This is quite important to control. Are there more?

There are I'm sure several variables previously read from a namelist that could be parametrised as options. nprtrv but also nprgpns are good candidates. There'd have to be some validation to ensure both can divide into the total number of requested MPI ranks.
We could also reintroduce the namelist approach, but I'm personally reluctant since it adds another couple of steps to running the benchmark.

@wdeconinck - @marsdeno pointed out that nflevg is set to nfld currently: https://github.com/ecmwf-ifs/ectrans/blame/main/src/programs/ectrans-benchmark.F90#L233. Should it actually be set to nlev?

Indeed that should be nlev. It must be a remnant of when nlev was nfld and nfld was 1, or something of that sort.

Another thing to consider is meminfo.txt. Currently output is printed to meminfo.txt when calling MPL_END (we don't provide any LDMEMINFO argument to MPL_END in the benchmark, but it's .TRUE. by default). Subsequent invocations of the benchmark in the same directory append to this same file.
If we wanted to support a --meminfo option for the benchmark which outputs to stdout, what would be the easiest way to do that?
If we didn't have to worry about breaking all of Fiat's other dependencies (namely the IFS) then it would be nifty to remove the LDMEMINFO argument and replace it with an integer, with -1 sending the output to meminfo.txt, and every other value interpreted as a unit number, so 6 would send it to stdout, for example.
Alternatively maybe there's a way in the benchmark to intercept the 501 unit number used by EC_MEMINFO so it prints to stdout instead of meminfo.txt. But that's a bit hacky.

The latter option isn't possible.
@wdeconinck - what do you think about adding an optional argument to MPL_END (and EC_MPI_FINALIZE) which contains the unit number for EC_MEMINFO? We could leave the LDMEMINFO option untouched so we don't break existing code. 6  would be given in the benchmark for this option.
Or is this all too much effort for something trivial?

I would call the ec_meminfo function manually with all the arguments you need.
#include "ec_meminfo.intfb.h"
call ec_meminfo( ... )

and then not ask mpl_end to print:
mpl_end(ldmeminfo=.false.)


xla compile op cache many tensors to out of memory .

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Mac
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: -
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): master
Python version: 3.8
Bazel version (if compiling from source): 3.7.2
GCC/Compiler version (if compiling from source): Apple LLVM version 10.0.1 (clang-1<IP_ADDRESS>)
CUDA/cuDNN version: -
GPU model and memory: -

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
zsh: command not found: v1.12.1-54214-gb51e7cff1aa

Describe the current behavior
xla OOM
Describe the expected behavior
xla not OOM
Contributing

Do you want to contribute a PR? (yes/no): yes
Briefly describe your candidate solution(if contributing):
flat_hash_map -> LRUCache(with large size) .

OOM
XlaCompilationCache in XlaCompileOp maybe cache many tensors in absl::flat_hash_map<Signature, std::unique_ptr<Entry>, Signature::Hash> cache_.
My XlaCompileOp has an input named TruncatedNormalV2, and TruncatedNormalV2 will output a dynamic shape, so my XlaCompileOp will compile many exe file and cache lots of input tensors.

LOG
I spend 3 days to find tensor OOM, so I think  If there is too much content in cache_, at least print a warning to remind it.

I want to confirm whether there is such a problem in TF ? or I have a mistake usage ?
If you think it's a bug, I am happy to fix this problem.

There's a known limitation of XLA that for dyamic size, it may need to do multiple recompilations and so that may be the reason for the cache to get filled and cause OOM. It can also add unexpected latency for the dynamic shapes. You can take a look at this section of the video which explains about the same.

When the cache reaches a certain amount, should we tell the user that XLA caches a lot of data?

There's a known limitation of XLA that for dyamic size, it may need to do multiple recompilations and so that may be the reason for the cache to get filled and cause OOM. It can also add unexpected latency for the dynamic shapes. You can take a look at this section of the video which explains about the same.

Should we replace flat_map with LRUCache ?
If user misuse leads to OOM, is there a log to remind them of this in time ?

Displaying warning message on memory usage will not be a good option since it will be specific to system hardware specific level.
Instead you can mange your GPU memory using options like set_memory_growth, more details here.
Additionally, you can investigate further on memory consumption using HLO graph, you can find the reference and similar discussion here. Thanks!


Pointer layout support

Most of the effort here is around trying to work around issues with Ptr types producing infinite recursive loops.

In ir lowering, the fix is to add agg types to the map before they are completed
In IR Layout IRPointerLayout is added.

Ideally this type would contain a pointer to the value type, but making this work with the other mechanisms (hoisting/Builder) is significantly more difficult. For now this is just avoided, which seems ok because for layout the value type isn't needed to be known.

For reflection tool, on pointer types it only outputs the name of the thing being pointed to

This isn't great. If the pointer is to a generic the name returned is just the generic name (not it's parameterization)
The layout for the thing being pointed to isn't necessarily emitted

Has a fix for legalization

Assumes a pointer must be to something that resolves to itself
Can detect if that isn't the case, and asserts.

The pointer mechanism used here assumes that a pointer is to some representation in a single address space. For some targets this may not actually be possible, for example when resources are in effect in their own address space.
Future work:

Making IRPointerTypeLayout able to hold the value type layout
Layout adds GetPointerLayout, but for CPU targets it is assumed that the pointer is 64 bits

That isn't right for 32 bit targets, or with CHERI for 128 bit pointers
We probably want a command line mechanism to specify the pointer width for the target

Reflection tool should probably use JSON system

We should reflect the layout for pointed to types

Issue in emit can cause infinite recursion needs a fix will do in subsequent PR.

I guess it must be correct given the test passes, but I didn't find the place where the struct type is being registered before it is fully lowered.

I found a fix for the emit issue, so now can run the previously disabled test.
There is a section in visitAggTypeDecl just after the "empty" type is created that registers is such that there isn't infinite recursion.
// We add the decl now such that if there are Ptr or other references
        // to this type they can still complete
        context->setValue(decl, LoweredValInfo::simple(finishedVal));

This PR doesn't seem to include the change to visitAggTypeDecl?


Azure OpenAI Service Assistants: additional_instructions isn't a supported parameter when creating a run.

additional_instructions isn't a supported parameter when creating a run.
I intended to utilize it, but it appears that it is not currently supported. Nevertheless, it is listed as a valid parameter during the creation of a run.
Additionally, see comment corroborating this.

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

ID: 203de00d-2364-ba7a-00e8-b72eac5e4966
Version Independent ID: 0366ab22-d578-9142-34f6-fb25f13af15c
Content: How to create Assistants with Azure OpenAI Service - Azure OpenAI
Content Source: articles/ai-services/openai/how-to/assistant.md
Service: azure-ai-openai
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@CarterBland
Thanks for your feedback! We will investigate and update as appropriate.

@AjayBathini-MSFT Thanks for the fast response, is there a roadmap that will add support for this feature?

@CarterBland thanks for calling this out. I have updated the reference docs internally to pull the mention of that parameter. The update will go live tomorrow morning. I am going to close out the issue on the docs side, but I will follow up with the engineering team to see if we can get that parameter added into a future version of the Azure OpenAI inference spec that includes the Assistants API.
From glancing at things on the OpenAI side that particular parameter was added a little bit later after the initial assistants launch which may have impacted its inclusion in the first Azure OpenAI Assistants spec:

https://github.com/openai/openai-python/pull/995/files

#please-close

@mrbullwinkle thanks for responding and making the update!

@mrbullwinkle any update on what's happening on the API side? This gap prevents us from trying out azure assistants, which, given the OpenAI assistants service instability, is a huge bummer.

Is the 'additional_instructions' feature available in 1106 preview?
When using this option, it responds with an HTTP 400 error.


lfmerge fails to install on Ubuntu 16.10

Hi Palaso people,
I used to be in Sentani, and attended an IT conference about 4 years ago. We had to leave to be close to my parents, but maybe I can write some tests/refactor/put in bugfixes now and then.
I always run the latest Ubuntu. I'm sure you target LTS versions (as you should), but I'm happy to provide fixes I find (I already put in a pull request for the ops-devbox repo to bring it up-to-date).
I'll trace through the dependencies and fix it myself if I can, but since it is LF-specific I thought I'd ask here first since info might be scarce via usual DuckDuckGo search.
Here's the error I got:
failed: [localhost] (item=[u'lfmerge']) => {"failed": true, "item": ["lfmerge"], "msg": "No package matching 'lfmerge' is available"}

Umm... yeah, is there a ppa I'm missing?

Hi there,
Sorry for the late response.  Next week, you can contact our software manager @megahirt for ways to contribute to Language Forge.
I'm closing this issue for two reasons:

Language Forge and its issues go in the web-languageforge repo.  I think this web-cat repo is intended for separate computer-assisted translation development.
lfmerge is the server-side application that allows Language Forge to Send/Receive with Fieldworks projects.  Our resource constraints limit our development & packaging to LTS Trusty and Xenial.

@jecompton, you are not missing a PPA. The ansible script adds the correct apt repos, there just aren't any lfmerge packages for 16.10 in the repos. As @darcywong00 mentioned, we don't have the resources to support them right now.

@jecompton, just wanted to say thanks for the open-source contributions, of
which we receive very little.  As you know we can't support anything else
than the LTS Ubuntu but feel free to check out language forge and lfmerge
on the LTS - your input on installation and running it could be valuable.
On Mon, Jan 9, 2017, 11:29 AM Damien Daspit<EMAIL_ADDRESS>wrote:

@jecompton https://github.com/jecompton, you are not missing a PPA. The
ansible script adds the correct apt repos, there just aren't any lfmerge
packages for 16.10 in the repos. As @darcywong00
https://github.com/darcywong00 mentioned, we don't have the resources
to support them right now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sillsdev/web-cat/issues/1#issuecomment-271209710, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADSPKehztcY3sGVGoNDBJUYEVC_-wZ8Aks5rQbeegaJpZM4LVz3x
.


Convert youtube video to mp4?

Is this possible with this gem or ffmpeg in general?

Check https://github.com/ytdl-org/youtube-dl
You may not need ffmpeg, though you can use ffmpeg to convert it after.


Using closestKodein in fragments

I'm using ActivityRetainedScope to create dependencies:
Kodein {
  bind<Foo>() with scoped(ActivityRetainedScope.MultiItem).singleton { Foo() }
}

class MyActivity : AppCompatActivity(), KodeinAware {

    override val kodein by closestKodein()
    override val kodeinContext: KodeinContext<Activity> get() = kcontext(this)

    val myFoo: Foo by instance()
}

This works great. The same instance survives activity rotations, and ScopeClosable.close() gets called when the activity finishes for good.
The problem comes when I want to inject this same instance into fragments defined in the XML of the activity:

This makes a new, un-scoped instance.

class MyFragment : android.support.v4.app.Fragment(), KodeinAware {
   override val kodein: Kodein by closestKodein()

   val myFoo: Foo by instance()
}

This causes a crash, as the fragment hasn't been attached yet (since the kodein reference is still in the constructor).

class MyFragment : android.support.v4.app.Fragment(), KodeinAware {
   override val kodein: Kodein by closestKodein()
   override val kodeinContext: KodeinContext<Activity> get() = kcontext(requireActivity())

   val myFoo: Foo by instance()
}

How can a fragment get a dependency with the activity's context? It doesn't need to access it before onAttach but the nature of by instance() means we can't even set up the lazy loading without the activity being attached. Seeing the same thing on 5.3.0 and 6.0.1.

Turns out the main problem was that I didn't include  import(androidModule(this@MyApplication)) which I didn't realize was required, but it is since it sets up the proper context translators from Fragment -> Activity.
In the process of investigating I found another bug, I'm going to open a separate issue for that.


refactor: the azure encorder file name

Update file name in Azure Encoder from "zure" to "azure" for improved code readability.

@Yogendra0Sharma can you please change the title of the commit for this PR so that it can pass linting?
Should be as follows:
git reset HEAD~

# Add back your changes
git add semantic_router/encoders/

git commit -m "<commit title under 60 char>"
...

Thank you!

@Yogendra0Sharma can you please change the title of the commit for this PR so that it can pass linting?
Should be as follows:
git reset HEAD~

# Add back your changes
git add semantic_router/encoders/

git commit -m "<commit title under 60 char>"
...

Thank you!

Done ! Please review.

@bruvduroiu Please review this PR.