SMS reminder tweaks

Add blank line between bullets
Replace first "at" with colon

Will re-open as needed


Improve UI for dark theme

Darktheme UI does not look good.
Please check.

Thanks for reporting.
You are right, will check it.


options.mute for set

This extremely simple code change allows the previous functionality of silent, without removing the new functionality of silent.

https://gist.github.com/1780350
Link to a Gist of an alternate method of replicating this option.


Error in calling bubblechart code on dashboard

Issue Type

[ x] Bug
[ ] Enhancement

Description
By chance I detected the errors in the nginx errorfile and than realized that the weekly activity chart on the dashboard is empty.
Dashboard tries to call the following files:
/static/JUMFlot/JUMFlot.min.js
/static/JUMFlot/jquery.flot.mouse.js
/static/JUMFlot/jquery.flot.bubbles.js
These files have a different location on my system:
/static/JUMFlot/javascripts/JUMFlot.min.js
/static/JUMFlot/javascripts/jquery.flot.mouse.js
/static/JUMFlot/javascripts/jquery.flot.bubbles.js
I changed the respective lines in dojo/templates/dojo/dashboard.html and the error disappears and the chart is filled with bubbles ( oh, how I love bubbles...)
If the issue is a bug, please also include the following:
Operating System
Ubuntu 18.04.1
Install

[x] setup.bash
[ ] docker

DefectDojo Version
1.5.3
Steps to Reproduce
Access the Dashbaord
Sample Scan File (if applicable)
Screenshots (optional)
Console Logs (optional)
2018/10/19 10:53:37 [error] 11255#11255: *24 open() "/opt/dojo/django-DefectDojo/static/JUMFlot/JUMFlot.min.js" failed (2: No such file or directory), client: <IP_ADDRESS>, server: defectdojo, request: "GET /static/JUMFlot/JUMFlot.min.js HTTP/1.1", host: "<IP_ADDRESS>", referrer: "http://<IP_ADDRESS>/dashboard"
2018/10/19 10:53:37 [error] 11255#11255: *24 open() "/opt/dojo/django-DefectDojo/static/JUMFlot/jquery.flot.mouse.js" failed (2: No such file or directory), client: <IP_ADDRESS>, server: defectdojo, request: "GET /static/JUMFlot/jquery.flot.mouse.js HTTP/1.1", host: "<IP_ADDRESS>", referrer: "http://<IP_ADDRESS>/dashboard"
2018/10/19 10:53:37 [error] 11255#11255: *24 open() "/opt/dojo/django-DefectDojo/static/JUMFlot/jquery.flot.bubbles.js" failed (2: No such file or directory), client: <IP_ADDRESS>, server: defectdojo, request: "GET /static/JUMFlot/jquery.flot.bubbles.js HTTP/1.1", host: "<IP_ADDRESS>", referrer: "http://<IP_ADDRESS>/dashboard"

Thanks, will fix it. Your comment made me laugh, "oh, how I love bubbles."

You're welcome. We all need a laugh from time to time, don't we?


Caching AJAX requests

Hi,
Great thanks for the nice react module, I found it the best among other inline svg loading technics, especially for isomorphic apps.
But I noticed one issue though, when using multiple SVG images on a page, there's a lot of identical requests for same images:

What do you think about adding memory cache for previously loaded SVG's? Since SVG's aren't heavy, it should not cause any problems with memory leaks.

+1

+1

Hey Guys.
We had the same issue so added some caching support to a fork of the library: https://github.com/trailsuite/react-inlinesvg/
There's a significant different between our fork and this package so we haven't raised a PR. Our original approach was to make a small change to @matthewwithanm's excellent module but some missing dependencies and a lack of expertise in coffee script caused us to go the long way around. If @matthewwithanm is interested we'll happily raise a PR with just the caching changes but might need some assistance in building the existing library and in stubbing the http.get requests.

This is something I would really like to see in this package. Luckily they're all cached, so shouldn't affect the end user too much, but the fact that it makes a request each and every time is not great. For instance, I have a slider that rotates through a group of SVGs, one every second, and every single one makes a new request even if it's been displayed before.

@lostpebble - They're often uncached. If you have the same icon repeated on a page, as we do, then the browser initiates multiple xhrs'. The first request to fire is only available in the cache once several other requests have started with a cache miss. You're welcome to use https://github.com/trailsuite/react-inlinesvg/ if you like. Our tests are green on Travis - biggest TODO is fixing the linting.

Thanks. I've just tried your fork, but it doesn't seem to solve the problem. I'll create an issue in that repository.

@lostpebble Did you see the config? Caching isn't on by default so you need to enable it (see the Readme.md). I'll enable issues on the repo too.

Ah okay, I see. Sorry, just assumed the Readme.md was exactly the same from the fork. It is working now.

hey @ifuller1
I've refactored the source code to ES2015 and if you like to contribute the cache option back in, it will be great.

Thanks @gilbarbara - totally missed this email. Will come back to it ASAP.

Hi @gilbarbara - PR raised https://github.com/matthewwithanm/react-inlinesvg/pull/32
Comments welcome.

Is this now considered closed? what was the reasoning to make cacheGetRequests default to false?

It would be good for somebody to independently validate this but yes, I think it is.
Defaulting to false as not to cause unexpected behaviour for existing users.

@ifuller1 I'll take a look at the code and get back to you :)

Thanks @export-mike, it's more about validating it works in production (we're using it successfully already - but would welcome validation from some of the authors)


create-app: fix accidental runtime package dependencies

Fixes @backstage/create-app, which currently throws the following:
Cannot find module<EMAIL_ADDRESS>
Fixes #4201

Still haven't figured out how it happened yet, but the last release also bumped create-app to 1.0.0. I've unpublished that version and rolling back the version here as well

Still haven't figured out how it happened yet, but the last release also bumped create-app to 1.0.0. I've unpublished that version and rolling back the version here as well

Still haven't figured out how it happened yet, but the last release also bumped create-app to 1.0.0. I've unpublished that version and rolling back the version here as well

Yeah, bug with changeset, it seems.

Still haven't figured out how it happened yet, but the last release also bumped create-app to 1.0.0. I've unpublished that version and rolling back the version here as well

Yeah, bug with changeset, it seems.

I am getting the next error:
Argument of type 'PluginEnvironment' is not assignable to parameter of type 'CatalogEnvironment'.   The types returned by 'reader.readTree(...)' are incompatible between these types.
Node: 14.15.1
npm: 6.14.8

I am getting the next error:
Argument of type 'PluginEnvironment' is not assignable to parameter of type 'CatalogEnvironment'.   The types returned by 'reader.readTree(...)' are incompatible between these types.
Node: 14.15.1
npm: 6.14.8

@IvanMelzi This fix not quite out yet, but will be soon. In addition to having the above bug the version of create-app that was published was also incorrectly version as 1.0.0, so we unpublished it from NPM.

@IvanMelzi This fix not quite out yet, but will be soon. In addition to having the above bug the version of create-app that was published was also incorrectly version as 1.0.0, so we unpublished it from NPM.


Training time detectron2 vs base implementation

Thank you for this repo!
I am training the ResNet50-based network on the Cityscapes dataset on 4 V100 GPUs with the following settings:
base implementation
config : configs/panoptic_deeplab_R50_os32_cityscapes.yaml
with NUM_WORKERS: 4
detectron2 implementation
config: configs/Cityscapes-PanopticSegmentation/panoptic_deeplab_R_52_os16_mg124_poly_90k_bs32_crop_512_1024.yaml
but made the following changes so that I am using the same setup as configs/panoptic_deeplab_R50_os32_cityscapes.yaml from the base implementation
MIN_SIZE_TRAIN: (1024, )
MAX_SIZE_TRAIN: 2048
IMS_PER_BATCH: 8
SOLVER:
  MAX_ITER: 90000
CROP:
    ENABLED: False
NUM_WORKERS: 4 # this is the fastest in my case

Training time for base implementation: ~25 hours
Training time for detectron2 implementation with modified config: ~52 hours
Training time for detectron2 implementation with configs/Cityscapes-PanopticSegmentation/panoptic_deeplab_R_52_os16_mg124_poly_90k_bs32_crop_512_1024.yaml: ~55 hours
I do not observe any speedup by changing OS16 to OS32 in detectron2 implementation:
RESNETS:
     RES5_DILATION: 1

I would like to check that this is the normal behaviour and that training in detectron2 takes much long, even twice as long as the base implementation. If this is the case, why is training in detectron2 so slow compared to the base implementation?

Unfortunately I do not have breakdown in Detectron2 training time so I don't know why it is slower. But I also notice it is about >1.5 times slower than my original implementation (8 GPUs).
Detectron2 has some hooks to better monitoring the training, I'm not sure if that would cause the training to be slower. I checked my experiment, the data processing time is almost the same for two implementations.

@AndraPetrovai
I found the problem, because depthwise separable conv is not supported in D2 yet, so I simply use the regular conv (in ASPP and decoder). I calculated the FLOPS and found using depthwise separable should give you a large speed-up.
My original implementation uses depthwise separable conv instead.

DepthwiseSeparableConv2d is now supported in the Detectron2 version of Panoptic-DeepLab. This should reduce the training time.


Draft: feat: externalize isDev function && warn if no messages locale matche…

externalized process.env.NODE_ENV check to util function isDev
added console warning in dev mode when no locale was specified and no locale in given messages object matches the default locale of 'en-US'

draft because needs yet to be tested

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (341b6b0) 96.05% compared to head (50328bc) 95.75%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   96.05%   95.75%   -0.31%
==========================================
  Files           5        5
  Lines         203      212       +9
  Branches       39       42       +3
==========================================
+ Hits          195      203       +8
  Misses          6        6
- Partials        2        3       +1

Files
Coverage Δ

src/utils.ts
100.00% <100.00%> (ø)

src/create-i18n.ts
92.30% <90.90%> (-1.25%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add separate logic for  tags in HTMLtoLines

I went with having separate logic for preformat tags in handle_data and
get_lines functions of HTMLtoLines class. This is for two reasons:

Not cleaning whitespace characters with re.sub() in handle_data:
Whereas with other tags like <blockquote>, which aren't completely
whitespace-dependent, <pre> text requires newlines and
indentation/tabs be preserved for... well, formatting.
Different logic for <pre> text in get_lines: textwrap defaults to
replace_whitespace=true for wrap(), and it's suggested that
str.splitlines() be used for newlines rather than setting
replace_whitespace=false to prevent inconsitencies with formatting.
I think this may cause issues with other indent tags, s rather
than trying to balance parsing non-preformat tags against
preformatted text, it seemed more reasonable to have the preformat
text be parsed separately.

Aside from that, I tried to keep the variable names and logic consistent
with the current code.

Whoaa thanks so much for this, this looks good....


Design email templates

Design good looking email templates for all the emails sent to participants and organizers

shifted to discussion #1334


Remove tables from settings

PR Overview
This PR removes tables from the settings models/yaml files, as discussed in #2272.
As @bendnorman mentioned in the PR linked above, it looks like we could use dagster's multi-asset subsetting to allow us to select subsets of tables, but I'm not sure if it's worth the time/energy. Open to other's thoughts on this though. Perhaps it's something we could leave to the future if we have time.
PR Checklist

[x] Merge the most recent version of the branch you are merging into (probably dev)
[x] Include unit tests for new functions and classes.
[x] Do your own explanatory review of the PR to help the reviewer understand what's going on and identify issues preemptively.

Same with eia861, I think


429 API throttling errors should be retried

Code currently treats 5XX as retriable, and 4XX as fatal. Need to take a closer look at these, because a 429 API throttle causes the bot to quit with error.
It's amazing that I've made it this long without hitting an API throttle.
2023-05-10T23:30:38.030Z - INFO - discord-bot-ol-bootsie:0.7.0 ready!
2023-05-11T22:22:05.602Z - ERROR - An HTTP 429 (Too Many Requests) was returned. This indicates a bad request. Not retrying.
node:events:491
      throw er; // Unhandled 'error' event
      ^

Error: {
  error: {
    message: 'That model is currently overloaded with other requests. You can retry your request, or contact us through our help center at help.openai.com if the error persists. (Please include the request ID eefa453c93f519cb311d7c7c6a17b0e9 in your message.)',
    type: 'server_error',
    param: null,
    code: null
  }
}
    at module.exports.requestChatCompletion (/usr/src/app/lib/lib-openai.js:112:17)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async probablyReactToMessage (/usr/src/app/app.js:180:25)
    at async Client.<anonymous> (/usr/src/app/app.js:111:7)
Emitted 'error' event on Client instance at:
    at emitUnhandledRejectionOrErr (node:events:394:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:84:21)

Node.js v18.16.0

Re-opening due to a new bug introduced by gracefully handling a retriable error:
node:events:491
      throw er; // Unhandled 'error' event
      ^

TypeError: Cannot read properties of undefined (reading 'trim')
    at module.exports.requestChatCompletion (/usr/src/app/lib/lib-openai.js:122:22)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Client.<anonymous> (/usr/src/app/app.js:63:32)
Emitted 'error' event on Client instance at:
    at emitUnhandledRejectionOrErr (node:events:394:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:84:21)

This is caused by a fall-through:
Try createChatCompletion, catch (error) (when this happens) delay retry, fall-through to:
if (responseText.trim() == '' && response.status == 200) {  // responseText is undefined.
lib-openai.js:120-134 should be moved to within the try block, as it should only run when the OpenAI call succeeds.


Add two resources to resources page

Disclaimer: Brisk Forms is a service that I created. However I believe it fits on this list because it provides the same functionality as other services with no cost.
Firebase Hosting is a resource I have used as a way to get static sites online with a custom domain name at no cost. While the free tier has bandwidth limits imposed, I presume it sufficient for majority of users with the added benefit that there are no restrictions on advertising.

:+1: Thanks a lot for this! I'd be happy to add them - I'm actually out of town right now, so I'll get this merged later this week or next.

Thanks - this has been merged and will go live on the next publish soon. I've got a new version coming out soon, so I'll probably do a publish then. If not, I'll get a interim version out next week.

Sounds good. Welcome back. Not sure about you, but for me, it always feels good to get back into code after a break.

:smile: As is typical for a long break, I'm having a little trouble remembering how to computer at the moment...


Bad download of preview files

Hello.
For a very long time I have been facing the problem of bad loading of preview apk files with this theme. I thought mixplorer was the problem, but after using a dozen of other themes, I found out that it was the dracula theme.
On my own, I could not figure out what is the cause, I hope you can do it!

screen

After I logged into the folder, I have to refresh it forcibly to make the previews appear...
Android: 13
Device: Google Pixel 5a

Added WhatsApp, Telegram, Twitter, Instagram icon. because developer added new bookmarks.

What version do you use?

What version do you use?

Mix – 6.30.3 silver
Theme – 1.2.0 (last version)

What version do you use?

Mix – 6.30.3 silver Theme – 1.2.0 (last version)

have you tried resetting mixplorer to default?

What version do you use?

Mix – 6.30.3 silver Theme – 1.2.0 (last version)

have you tried resetting mixplorer to default?

Yes, of course. Also tried different versions: silver and xda.

What version do you use?

Mix – 6.30.3 silver Theme – 1.2.0 (last version)

have you tried resetting mixplorer to default?

Yes, of course. Also tried different versions: silver and xda.

did you find any .nothumbnail file in the folder?

No, there is no such file. If it were so, then all the other themes would also have chosen bad previews, but this only happens to me with the dracula theme ... Now I'm using another theme, everything is fine.

No, there is no such file. If it were so, then all the other themes would also have chosen bad previews, but this only happens to me with the dracula theme ... Now I'm using another theme, everything is fine.

No, there is no such file. If it were so, then all the other themes would also have chosen bad previews, but this only happens to me with the dracula theme ... Now I'm using another theme, everything is fine.

Ok, I've tried on several devices all seem fine. but calm down I will try to find the cause and if necessary I will release it to the latest version so please wait.

++ Added WhatsApp, Telegram, Twitter, Instagram icon. because developer added new bookmarks.

Hey how are you, please share your email address to try a theme that I have not committed and published publicly and give me your review or criticism in the email.

++ Added WhatsApp, Telegram, Twitter, Instagram icon. because developer added new bookmarks.

Hey how are you, please share your email address to try a theme that I have not committed and published publicly and give me your review or criticism in the email.

Send me your email, I'll write to you. I just do not want to distribute my mail in the public domain :)

++ Added WhatsApp, Telegram, Twitter, Instagram icon. because developer added new bookmarks.

Hey how are you, please share your email address to try a theme that I have not committed and published publicly and give me your review or criticism in the email.

Send me your email, I'll write to you. I just do not want to distribute my mail in the public domain :)

++ Added WhatsApp, Telegram, Twitter, Instagram icon. because developer added new bookmarks.

Hey how are you, please share your email address to try a theme that I have not committed and published publicly and give me your review or criticism in the email.

Send me your email, I'll write to you. I just do not want to distribute my mail in the public domain :)

<EMAIL_ADDRESS>

Hei sir

https://github.com/dracula/mixplorer/releases/tag/v1.3.0


journal: delete old journal entries and blocks

Do you think this needs tests?
Issue: KBFS-1410

Yeah, mind adding basic tests, also?
Also, should we clean up the MDs as we remove them also, or leave that for another PR? The only tricky thing is the convertBranch case.

Added tests, MD garbage collection (forgot about this!) and simplified/fixed the block deletion code.  Ready for another look!

@akalin-keybase: add those MD flush tests, and it indeed revealed some issues which I fixed as well.  Good call!

(Ready for another look!)

lgtm after comment


[DOCS] Improving up the contributing.md

Description
Will be creating a cool and interactive contributing.md
1.will also be improving up the installation guide style .
2.and will also add up the images for best and good results .
Pls do assign me up first ,for the best suggested changes .
Will be making it awesome and cool.
Screenshots
No response
Additional information
No response

Is this issue still open? I can see contributing.md is already pretty good.

@abhay-raj19 As mentioned in the discord, we'll keep the documentation for the end


Add support for Enlighter code blocks

To add support for Enlighter code blocks, I have added a turndown rule.
An Enlighter code block typically gets exported by Wordpress to the following format:

// foo equals "bar"
const foo "bar";

The selector therefore is a combination of the node name 'PRE' and the class 'EnlighterJSRaw'. The language can be found in the data-enlighter-language attribute. Lastly, instead of using the node.innerHTML property, I use the content property. If I use node.innerHTML, the < are escaped to < and so on...
Output then becomes:
```typescript
// foo equals "bar"
const foo = "bar";
```

@lonekorean Would you have time to look at this perhaps? 😅

This was super helpful to discover, @drikusroor. Thanks for sharing it.
I ran into a problem that my underscores _ were being escaped into \_. The default behavior of Turndown is to escape characters that may be confused with the Markdown syntax unless they are in a code block.
I was able to tell markdown we were in a code block by adjusting your code to use this line instead of the default content function parameter. Then we return the code instead of the content.
Now my underscores are no longer escaped!
// preserve enlighter code blocks
turndownService.addRule("enlighter", {
  filter: (node, options) => {
    return (
      options.codeBlockStyle === "fenced" &&
      node.nodeName === "PRE" &&
      node.firstChild &&
      node.classList.contains("EnlighterJSRAW")
    );
  },
  replacement: (content, node) => {
    const language = node.getAttribute("data-enlighter-language") ?? "";
    const code = node.firstChild.textContent;
    return "\n" + "```" + language + "\n" + code + "\n" + "```" + "\n";
  },
});

@drikusroor thank you for the helpful example and PR! Sorry this took... a while.
Fixed in v2.2.6.
I ended up with a broader solution to also cover other cases with
. I didn't merge this PR directly, but I did cherry-picked your commits to make sure you got contributor credit.


Add new max_last_will_delay value and change default

The current default of 0s was confusing to the users and as long as
it is possible from the broker side to limit this then it makes sense
for the default to be what the client provides. This can now be done
by specifying client instead of a time value.

@larshesel can this be merged or does it need to wait for a specific release?

I've postponed merging minor features as this makes it much easier to create bugfix releases. This will likely be merged either this or next week.

@wlisac merged! :)


Feature: save responses for the proxied requests

@idodav can you assign to me?

Add description to get what is expected in this

Add description to get what is expected in this

of course! will do

done


Crash when showsEmptyAlbums is false and no pictures are available

When the image picker is set to showsEmptyAlbums = false and the are no non-empty albums to show, the components and app crashs on a optional unwrap which is nil.
The error exactly is fatal error: unexpectedly found nil while unwrapping an Optional value and it happens on DKAssetGroupDetailVC.swift func updateTitleView() line 136, where there is a force un-wrap on self.selectedGroupId! which is nil in this case.
One quick-solution I found could be to add an if !groupIds.isEmpty { around calls to strongSelf.updatePartial(...) in fetchGroups of DKGroupDataManager.swift => then the image picker shows up empty. Not perfect but good enough as a quick-fix. What do you think?

It's my fault!  Could you send me a PR?


valuelink cannot be used server side

Trying to require valuelink server-side results in an error complaining that window is not defined:
$ node
> require('valuelink')
ReferenceError: window is not defined
    at /Users/jwalton/benbria/loop/node_modules/valuelink/dist/index.js:1:982
    at n (/Users/jwalton/benbria/loop/node_modules/valuelink/dist/index.js:1:131)

It looks like this is some kind of artifact of using rollup to bundle things, because window is only referenced here, which would never get executed server side in the first place.

My co-worker @dule notes that removing this line seems to fix this.

Applied your fix, thanks.


Use initial expressions to select the initial value from a dropdown list populated using answer expressions

Is your feature request related to a problem? Please describe.
When I populate a dropdown using the answer expression extension as per this example, I cannot set the initial value using the initial expression extension. This is due to the item being of a reference type and the type I'm trying to populate is a string type.
Describe the solution you'd like
I should be able to use initial expressions to set the initial value
Describe alternatives you've considered
Instead of using answer expressions, I can populate the dropdown with answer options instead but this is challenging where there are a large number of options.
Additional context
Here is the questionnaire I tried and this is the logcat. If there is another way to do this, please point me in the right direction.

@KhumboLihonga  I can replicate your issue using the questionnaire I defined below, which works with  resources  fetched  from the public  https://hapi.fhir.org/baseR4/ site.
Will continue to debug
{
  "resourceType": "Questionnaire",
  "id": "test-questionnaire",
  "language": "en",
  "title": "Test Questionnaire",
  "status": "active",
  "publisher": "mcore",
  "contact": [
    {
      "name": "http://www.d-tree.org"
    }
  ],
  "item": [
    {
      "extension": [
        {
          "url": "http://hl7.org/fhir/StructureDefinition/questionnaire-itemControl",
          "valueCodeableConcept": {
            "coding": [
              {
                "system": "http://hl7.org/fhir/questionnaire-item-control",
                "code": "drop-down",
                "display": "Drop down"
              }
            ],
            "text": "Drop down"
          }
        },
        {
          "url": "http://hl7.org/fhir/uv/sdc/StructureDefinition/sdc-questionnaire-initialExpression",
          "valueExpression": {
            "language": "text/fhirpath",
            "expression": "Patient.id"
          }
        },
        {
          "url": "http://hl7.org/fhir/uv/sdc/StructureDefinition/sdc-questionnaire-answerExpression",
          "valueExpression": {
            "expression": "Patient?address-city=NAIROBI",
            "language": "application/x-fhir-query"
          }
        },
        {
          "url": "http://hl7.org/fhir/uv/sdc/StructureDefinition/sdc-questionnaire-choiceColumn",
          "extension": [
            {
              "url": "path",
              "valueString": "name"
            },
            {
              "url": "label",
              "valueString": "name"
            },
            {
              "url": "forDisplay",
              "valueBoolean": true
            }
          ]
        }
      ],
      "linkId": "district",
      "text": "District",
      "type": "reference",
      "required": true
    }
  ]
}```

@KhumboLihonga I created PR #1831  to propose fixing. Please check out and lmk if you have any thoughts

Hey @omarismail94, my colleague tested your PR and was unable to verify that the issue was fixed. This is the comment  he left. Is there any update on this?

Hey @KhumboLihonga and @sevenreup , please see the attached video of the fix working:
Where is the app crashing for you, and what are the logs that you are getting?
I also had to implement the DataCaptureConfig.Provider interface in the FhirApplication class to provide an xFhirQueryResolver:
https://github.com/google/android-fhir/pull/1831/files#diff-dc8902f9b4a0cb8dab12c5cdb57cfefbcf29b4482c5d2e28e60c6e51ac932389
https://user-images.githubusercontent.com/44980219/219350894-cb813ee7-06df-4044-b8b5-bfa101996320.mp4

@omarismail94 the initial expression is working when I use the example questionnaire you gave here, it only shows the resource reference as a label.
But when I try to change the label to something else for example the Patients name, it does not work.
The questionnaire I am using is the one below.
{
  "resourceType": "Questionnaire",
  "id": "test-questionnaire",
  "language": "en",
  "title": "Test Questionnaire",
  "status": "active",
  "publisher": "mcore",
  "contact": [
    {
      "name": "http://www.d-tree.org"
    }
  ],
  "item": [
    {
      "extension": [
        {
          "url": "http://hl7.org/fhir/StructureDefinition/questionnaire-itemControl",
          "valueCodeableConcept": {
            "coding": [
              {
                "system": "http://hl7.org/fhir/questionnaire-item-control",
                "code": "drop-down",
                "display": "Drop down"
              }
            ],
            "text": "Drop down"
          }
        },
        {
          "url": "http://hl7.org/fhir/uv/sdc/StructureDefinition/sdc-questionnaire-initialExpression",
          "valueExpression": {
            "language": "text/fhirpath",
            "expression": "Patient.id"
          }
        },
        {
          "url": "http://hl7.org/fhir/uv/sdc/StructureDefinition/sdc-questionnaire-answerExpression",
          "valueExpression": {
            "expression": "Patient?address-city=NAIROBI",
            "language": "application/x-fhir-query"
          }
        },
        {
          "url": "http://hl7.org/fhir/uv/sdc/StructureDefinition/sdc-questionnaire-choiceColumn",
          "extension": [
            {
              "url": "path",
              "valueString": "name.given"
            },
            {
              "url": "label",
              "valueString": "text"
            },
            {
              "url": "forDisplay",
              "valueBoolean": true
            }
          ]
        }
      ],
      "linkId": "district",
      "text": "District",
      "type": "reference",
      "required": true
    }
  ]
}

@sevenreup hmm, is name.given a valid fhir path? can you try name.first().given? That should give you a list of all given names in the choices

It is a valid path. Even when I use first() I am still getting the same problem

When I do name.first().given, this is what I get:

Is what you want that the Patient Id appears initially (not the given name), and then when the dropdown is selected, the given names are shown?

We want what we have defined as the label to appear initially and in the selection ( in this case, the patient's name). Because what is happening now is the initial selection is not working when you change the path.

@omarismail94 I feel there are two issues here:

If the path attribute is changed to name.first().given, the dropdown options are indeed updated to display as intended BUT the initial expression does not set the initial value (regardless of how it is meant to be presented)
We should be able to define how the selected item should be presented using the label attribute

I believe point 1 is still a part of this PR while point 2 would be a separate PR. What do you think?

@omarismail94 Just following up on this issue. What do you think about my comment above?

@omarismail94 following up on this once again. Let me know your thoughts on the comment above so we can decide how to best proceed.


🔒 | auth: SAWO integration

Description
SAWO Integration in the Web Application

You can view the web app without authorization

This PR fixes #NA
Notes for Reviewers:

Signed commits

[x] Yes, I am a Member of DEVSTRONS'

Discord ID (e.g.: DEVSTRONS'#0238)

aminoxix  🇮🇳#2567

@tusqasi: We made authentication via SAWO Labs, you need to check out our configurations, & separated them in terms of util.py and main.py using the Quart server.
Also, we need to integrate SAWO authentication with respect to the SIGNUP/LOGIN button on the top right corner.
@ Contributors
Future Scope:
To create an upvote button for every news and integrate it with authentication, if users want to upvote any news, they must need to LOGIN :shield:
Reference: https://news.ycombinator.com
If you try to upvote with the micro button on the left side of the news, you must cross auth: https://news.ycombinator.com/vote?id=31248833&how=up&goto=news


Dont work in Firefox Browser

When i try the Demo-project and highlighted text in Firefox and press the hotkey, the hotkeylistener get the URL not the highlighted text.

In Chrome also. It is possible to select the URL but text is not copied to the form.
OS: Windows 10 64bit

Yes. As reported before, browser text selections esp. from the address bar, fails in some instances due to browser behaviour when handling texts.
Will be working on customizing text capture in browsers.

You may now test this issue with the latest release, v1.8.0.


Correct default value of strictParsing in README

Description
This commit updates the README to reflect the current behaviour of the strictParsing prop, which is set to true by default.
See: https://github.com/YouCanBookMe/react-datetime/blob/d0b63bcd5330053a64331014be9e118c66205e80/DateTime.js#L49
Motivation and Context
The documentation for this prop is currently incorrect :smile:
Checklist
[ ] I have added tests covering my changes
[x] All new and existing tests pass
[x] My changes required the documentation to be updated
  [x] I have updated the documentation accordingly
  [ ] I have updated the TypeScript 1.8 type definitions accordingly (not required)
  [ ] I have updated the TypeScript 2.0+ type definitions accordingly (not required)

Oops. Didn't mean to do that.
@bradchristensen Nice catch, thanks for this!


Use an older visual studio for 1.14

Cargo built for Rust 1.13.0 was linked with Visual Studio 2012. Cargo was also
compiled against msvcrt.dll, with C code compiling with /MD. In Rust 1.14.0
C code is compiled with the same flags and Cargo's also linked against
msvcrt.dll. In 1.14.0, however, Cargo is linked with Visual Studio 2015,
presumably the default on AppVeyor. This introduced a dependency on the VS
redistributables which was not intended.
This dependency will be fixed in Rust 1.15.0 where cargo links to libcmt and
compiles C code with /MT. For the 1.14.0 branch this is taking the strategy of
reverting the build environment to where it previously was, using Visual Studio
2012 instead of Visual Studio 2015. This will hopefully drop the dependency on
the redistributables and restore the same list of dependencies as before.

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

 Warning

Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.14.0. Please double check that you specified the right target!

@bors r+

:pushpin: Commit 258f4f3 has been approved by brson

:hourglass: Testing commit 258f4f3 with merge 7c37935...

:broken_heart: Test failed - status-appveyor

@bors: r=brson

:pushpin: Commit ec68b6e has been approved by brson

:hourglass: Testing commit ec68b6e with merge d4dbe92...

@bors: r=brson

:pushpin: Commit e7cf58f has been approved by brson

:hourglass: Testing commit e7cf58f with merge 298a012...

:sunny: Test successful - status-appveyor, status-travis
Approved by: brson
Pushing 298a0127f703d4c2500bb06d309488b92ef84ae1 to rust-1.14.0...


Error in firebase_functions/private/util.py processing event time attribute

After updating firebase-tools, using the functions emulator, any time an @firestore_fn-decorated function is executed, I get this error:
...
>    File "/.../venv/lib/python3.11/site-packages/firebase_functions/firestore_fn.py", line 115, in _firestore_endpoint_handler
>      is_nanoseconds = _util.is_precision_timestamp(time)
>                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>    File "/.../venv/lib/python3.11/site-packages/firebase_functions/private/util.py", line 333, in is_precision_timestamp
>      _, s_fraction = time.split(".")
>      ^^^^^^^^^^^^^
>  ValueError: not enough values to unpack (expected 2, got 1)

The relevant code includes this block from firebase_fn.py and this block from private/util.py.
After upgrading to the latest firebase tools npm install -g firebase-tools to v12.5.4, the CloudEvent event's time attribute has this iso format: 2023-09-16T13:32:06Z extracted by adding a print() call to the firebase_functions' site-package source.
So, that unprotected time.split('.') call isn't accounting for the ostensible change to the time attribute's format.

I'm using Python 3.11.4 on macOS 12.6 with firebase-functions v0.1.1 (latest).

It's worth noting that downgrading firebase-tools from the latest v12.5.4 to the previous v12.5.3 fixes the issue.
tl;dr fix this bug by running: npm install -g<EMAIL_ADDRESS>Firebase staff - please consider adding an integration test that covers @firebase_fn-triggered functions to the firebase-tools CI.

Thanks for sharing.
npm install -g<EMAIL_ADDRESS>Also worked for me.

I have the same issue, but running npm install -g<EMAIL_ADDRESS>didn't fix it for me. What command is invoking firebase-tools? I am assuming its used by firebase emulators:start?

Same issue for me using the latest firebase.tools. Using 12.5.3 via npm for now until the issue is resolved.
Also, I'd recommend some better docs on "installing" a specific version binary from the firebase.tools website.

Snce this issue has been fixed, I'll close this issue. Please comment if you disagree with my decision

Snce this issue has been fixed, I'll close this issue. Please comment if you disagree with my decision

This is still an issue in the latest version, this issue shouldn't be closed.

Yeah, this is the 2nd recent issue @exaby73 has closed that hasn't actually been fixed.

@jackall3n @joaqo Sorry about this. I realise that the workaround was downgrading the firebase_tools version. Correct me if I am wrong. I'll reopen this issue and investigate further

+1 this issue still happens and if not for this thread it would have been very difficult for me to resolve. I spent ~1 hour fruitlessly playing around with version of various components before finding this. npm install -g<EMAIL_ADDRESS>works.

+1 this issue still exists

Could you please check with the latest version 0.2.0 and confirm if this issue is fixed on your side? There was a PR that was merged but not released earlier which may have fixed this issue

I just tried it with firebase-tools v13.3.0 and it still fails.

Hey @antopolskiy if you can still reproduce it, could you provide me with your stack trace?


divergence expression varies dramatically with gradient algorithm

Divergence expression is an alias for a gradient expression. The gradient expression accepts and option algorithm selection integer Possible values are 0,1,2,3 for sampled, logical, nzqh and fast. What is the difference between these algorithms? Does nzqh take mesh size into account whereas logical does not? Is default chosen appropriately? For a Miranda user at LLNL, the default appears to be 0, sampled, and that produced results with block boundaries. Examine the attached images to see dramatic variation in results based on algorithm.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 2718
Status: Rejected
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: divergence expression varies dramatically with gradient algorithm
Assigned to: -
Category: -
Target version: -
Author: Mark Miller
Start: 11/22/2016
Due date:
% Done: 0%
Estimated time:
Created: 11/22/2016 02:19 pm
Updated: 11/30/2016 07:04 pm
Likelihood: 3 - Occasional
Severity: 2 - Minor Irritation
Found in version: 2.10.0
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
Divergence expression is an alias for a gradient expression. The gradient expression accepts and option algorithm selection integer Possible values are 0,1,2,3 for sampled, logical, nzqh and fast. What is the difference between these algorithms? Does nzqh take mesh size into account whereas logical does not? Is default chosen appropriately? For a Miranda user at LLNL, the default appears to be 0, sampled, and that produced results with block boundaries. Examine the attached images to see dramatic variation in results based on algorithm.
Comments:

2718_div_expr_issues_alg_0.tiff.zip
2718_div_expr_issues_alg_1.tiff.zip
2718_div_expr_issues_alg_2.tiff.zip
2718_div_expr_issues_alg_3.tiff.zip


Add security policy

As this project is now deprecated, CommerceHub will not be contributing further on this ticket.  Please feel free to work on it further yourself under the terms of the project license if desired.


[BUG] curve-squeeze-leverage and curve-healthy bots are missing swap_route params

the backend service needs to get swap route to exchange the token to settlement token on Curve and put it as swap_route params. Because we are missing the parameter, the backend script returns an error.
Done when

[x] curve-squeeze-leverage bot has been updated with the missing parameter
[x] curve-healthy bot has been updated with the missing parameter

Some logs from service
Jun 10 18:36:56 bots-server start.sh[3654061]: CURVE-JS IS CONNECTED TO NETWORK: { name: 'MAINNET', chainId: 1 }
Jun 10 18:36:56 bots-server start.sh[3654061]: bot_info [
Jun 10 18:36:56 bots-server start.sh[3654061]:   {
Jun 10 18:36:56 bots-server start.sh[3654061]:     bot: '0x1BcCb8863E7aC10f9E048cCDAF7b5fca56D7076A',
Jun 10 18:36:56 bots-server start.sh[3654061]:     callbacker: '0xb911D7e59BA82FDF477a2Ab22Ff25125072C9282',
Jun 10 18:36:56 bots-server start.sh[3654061]:     callback_args: [ '0', '4924', '344782640' ]
Jun 10 18:36:56 bots-server start.sh[3654061]:   }
Jun 10 18:36:56 bots-server start.sh[3654061]: ]
Jun 10 18:36:56 bots-server start.sh[3654061]: createAndSignTx ERROR Error: Request failed with status code 500
Jun 10 18:36:56 bots-server start.sh[3654061]:     at createError (/home/rootedbox/app/curve-squeeze-leverage-bot/node_modules/@palomachain/paloma.js/node_modules/axios/lib/core/createError.js:16:15)
Jun 10 18:36:56 bots-server start.sh[3654061]:     at settle (/home/rootedbox/app/curve-squeeze-leverage-bot/node_modules/@palomachain/paloma.js/node_modules/axios/lib/core/settle.js:17:12)
Jun 10 18:36:56 bots-server start.sh[3654061]:     at IncomingMessage.handleStreamEnd (/home/rootedbox/app/curve-squeeze-leverage-bot/node_modules/@palomachain/paloma.js/node_modules/axios/lib/adapters/http.js:322:11)
Jun 10 18:36:56 bots-server start.sh[3654061]:     at IncomingMessage.emit (node:events:525:35)
Jun 10 18:36:56 bots-server start.sh[3654061]:     at IncomingMessage.emit (node:domain:489:12)
Jun 10 18:36:56 bots-server start.sh[3654061]:     at endReadableNT (node:internal/streams/readable:1359:12)

Jun 10 18:36:56 bots-server start.sh[3654061]:     data: {
Jun 10 18:36:56 bots-server start.sh[3654061]:       code: 2,
Jun 10 18:36:56 bots-server start.sh[3654061]:       message: "failed to execute message; message index: 0: Error parsing into type curve_healthy_juice_bot_cw::msg::ExecuteMsg: missing field `swap_infos`: execute wasm contract failed [!cosm!wasm/wasmd@v0.50.0/x/wasm/keeper/keeper.go:404] with gas used: '97028'",
Jun 10 18:36:56 bots-server start.sh[3654061]:       details: []
Jun 10 18:36:56 bots-server start.sh[3654061]:     }
Jun 10 18:36:56 bots-server start.sh[3654061]:   },

wc117 — Today at 2:56 PM
I found backend service needs to get swap route to exchange the token to settlement token on Curve and put it as swap_route params. Because we are missing the parameter, the backend script returns an error.

@wc117 can you confirm this issue is fixed?

@ranshaojiqing @wc117  (cc @fieldtheory123 )
did the following to confirm curve-healthy-bot

pull the latest code on the bot server in the ~/app/curve-healthy-bot directory
restart service sudo service curve-healthy-bot restart
observe the logs journalctl -u curve-healthy-bot -f

I'm still seeing the same error - reopening

For curve-squeeze-leverage the error message changed - new ticket opened here https://github.com/VolumeFi/paloma/issues/1682

@wc117 I don't see your comments on this ticket. We need your response here, ser. Also, please see: https://discord.com/channels/987730379182051408/1164263785956134992/1250201351145914530

@taariq no immediate need for @wc117 's comments -- backend needs to supply the correct arg. in order for me to do that, i need to know what's really being asked here, because as far as i can tell i'm missing data i need.
agreed on https://discord.com/channels/987730379182051408/1164263785956134992/1250201351145914530 -- let's all discuss so i can get my bearings on what the ask is here

marking as blocked until the smart contract bugs are fixed


Team 1 - Refactor DictionaryMatcher

Refactor DictionaryMatcher so that it returns the spanList along with the Tuple
Make the classes Schema and ITuple immutable
Use boundaries while matching dictionary terms

Created the pull request #80 for this issue.

The issue is resolved and the PR has been merged.


use getter in case of overwritten from child class

@amarkevich
@reta

Makes sense, thanks @arthurchan35 !


I can't log in anymore

I can no longer log in after the last update, I don't know if it was a TV update or an application update.
TV model: UN32T4300
Every time I try to log in it quickly gives me a loading screen and then suddenly goes back to the login screen.

Hi @Clauberto, It was already reported and resolved in https://github.com/jhassan8/crunchyroll-tizen/issues/77
If you continue with the error, let me know, regards


Implement "Edit project" functionality

Extracted from https://github.com/Learn-by-doing/bitstarter/issues/7

would you mind if I claimed this?
which approach would you like to take @chill117?
edit the properties on a standalone page (hence loading the data of the project with the load of the page)
OR
edit the properties in the page with the project listing, so all is done on the background as AJAX?

I've taking this :)

I've came to a set of questions how to do some features to make multiply token. I'll have to add and remove edit fields for tokens dynamically to the form and I missing the knowledge - which approaches/libraries/techniques should be used for this. Also, seems to me it's not a very good idea to write to database every time once address added or removed - it's better to do it in once step. But for this I'll have to store all the changes somewhere. And I have no idea where :)

I think this is not a concern for this particular issue. Here, you only have to add the functionality for editing a project that has a single token. We will only have to deal with multiple token edit functionality once we introduce the ability to create projects with multiple token.


Fix #273, Create empty man directory to fix missing manpage directory issue

Can one of the admins verify this patch?


Implement EntityBlockCollisionEvent

This PR implements a method to detect and cancel collisions, this allows for logic such as proper liquid handling and collision events such as fireball explosions.

Closing to push to new block api


Usage of PKCE

In the RFC001, specifically in section 3.5, authorization request, it is defined that we are using PKCE. The example, including the description, defines it in a way that makes it seem that PKCE is mandatory. Nevertheless, the use of PKCE is optional according to the OIDC standard.
On the other hand, EBSI WCT requires the usage of PKCE, but our EWC ITB issuer implementation does not use it.
Now, what would we like to define in the RFC, namely making the usage of PKCE optional or mandatory?
@lalc @endimion @ntsbs

PKCE is optional is OIDC Core 1.0 but required in OIDC iGov and OIDC FAPI and in OAuth 2.0 Security Best Current Practice.
we can say that PKCE is not present in OIDC Core 1.0 because Core was released before the born of PKCE and it keeps it's original requirement asset for interop retrocompatibility.
At the same time, for the purpose of the issuance, using the auth code flow, PKCE is mandatory according to OpenID4VCI.
https://github.com/openid/OpenID4VCI/blob/main/openid-4-verifiable-credential-issuance-1_0.md#authorization-endpoint-authorization-endpoint

The RFC001 is based on the draft 12 of OIDC4CI (https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0-12.html) and it only RECOMMENDS to use PKCE. The same is true for the latest version of OIDC4CI (https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0.html) which is the first approved implementers draft.
Since I believe its a good security feature, I was raising the question if we should make it mandatory or optional within the EWC scope.

oh yes @andreasabr, I made confusion between openid4vci and the italian profile that made PKCE mandatory to prevent replay attacks
https://github.com/openid/OpenID4VCI/blob/main/openid-4-verifiable-credential-issuance-1_0.md#replay-prevention
I agree with you

In my mind we should stick to the spec unless there is any strong need from the LSP perspective.
We could consider the security aspects in future scope.

My two cents.  Maybe we should keep it optional as draft 12 of [OIDC4CI] (https://openid.net/specs/openid-4-verifiable-credential-issuance-1_0-12.html)  only RECOMMENDS to use PKCE? I agree, of course, that it is a pretty important security feature but as @lalc  suggests lets stick to the specs for now (just in case any wallet hasn't implemented it yet) and security is not really our focus at the moment....  I will add support for it in the EWC RFC Issuer service @andreasabr within the next few days hopefully and let everyone know :)

Hey all PKCE is supported by the auth code flow and the tests are added in the ITB.


RuntimeError: CUDA out of memory. just on documented ape training of Linemod

As new person to the pytorch/6d-pose estimation world, I finally got PVD3D set up after some issues.  When I try to just run the normal training python3 -m train.train_linemod_pvn3d --cls ape it very quickly uses up all my GPU memory.  I've tried setting the batch size to 1 in the linemod/dataset_config/models_info.yml and in common.py, but I always get the exact same behavior.  It's as if the actual batch size is computed elsewhere, and I don't have control over it.  Any help you might provide is greatly appreciated.

First use nvidia-smi to view the memory usage of each GPU.  You can then use CUDA_VISIBLE_DEVICES=1,2 to specify which GPU(s) to use. For example, running with CUDA_VISIBLE_DEVICES=1,2 python3 -m train.train_linemod_pvn3d --cls ape will use GPU 1, 2 for training.

Yeah, I only have 1 GPU.  So ultimately, it looks like PVDNet is set up in a way that uses up my whole 4GB of GPU just on loading the data, no matter what match size I set.  I realize that 4GB isn't ideal for training, but at the same time, I feel like ther should be a tradeoff in time versus batch size/memory usage that should be tune-able somewhere.

As new person to the pytorch/6d-pose estimation world, I finally got PVD3D set up after some issues. When I try to just run the normal training python3 -m train.train_linemod_pvn3d --cls ape it very quickly uses up all my GPU memory. I've tried setting the batch size to 1 in the linemod/dataset_config/models_info.yml and in common.py, but I always get the exact same behavior. It's as if the actual batch size is computed elsewhere, and I don't have control over it. Any help you might provide is greatly appreciated.

Have you solved the same problem

请问作者这个问题怎么解决

请问作者这个问题怎么解决

加钱上大内存GPU (#^.^#)

I think this can close.  My guess is that this is likely due to having only a 4GB GPU.


Automatically reset metrics on every epoch

🚀 Feature
PyTorch Lightning should reset metrics on every epoch.
Motivation
In the current docs, there's a big warning:

From v1.2 onward compute() will no longer automatically call reset(), and it is up to the user to reset metrics between epochs, except in the case where the metric is directly passed to LightningModules self.log.

Obviously, this is a bit of a pain to have to do manually, especially because there doesn't seem to be any hook for epoch_start that a LightningModule can define here.
Pitch
PyTorch Lightning should automatically reset all metrics whenever it starts a new epoch. Implementationally, I think this is as simple as:
def reset(c):
    if isinstance(c, pl.metrics.Metric):
        c.reset()
    return c
module.apply(reset)

This would be limited to metrics that are correctly identified as children, but since that's already required to get the metric on the right device, I think that's fine:

when properly defined inside a LightningModule , Lightning will automatically move the metrics to the same device as the data. Being properly defined means that the metric is correctly identified as a child module of the model (check .children() attribute of the model). Therefore, metrics cannot be placed in native python list and dict, as they will not be correctly identified as child modules. Instead of list use ModuleList and instead of dict use ModuleDict.

Alternatives
We could implement some kind of epoch_start hook for LightningModule where the user could nicely implement this. Otherwise, I cannot figure out where to reset each epoch -- I have tried doing it at the end of the epoch (via validation_epoch-end), but that seems to give incorrect results on the first epoch:

 This script prints out 400 for the first epoch rather than 100
import pytorch_lightning as pl
import torch
from torch import nn

class Counter(pl.metrics.Metric):
    def __init__(self):
        super().__init__()
        self.add_state("test", default=torch.tensor(0.0), dist_reduce_fx="sum")

    def update(self, x: int):
        self.test += x

    def compute(self):
        return self.test

class Module(pl.LightningModule):
    def __init__(self):
        super().__init__()
        self.linear = nn.Linear(5, 1)
        self.metric = Counter()
        self.metric.reset()

    def configure_optimizers(self):
        return torch.optim.Adam(self.linear.parameters())

    def training_step(self, batch, batch_idx):
        self.metric.update(torch.tensor(float(len(batch))))
        return self.linear(batch).sum()

    def validation_step(self, batch, batch_idx):
        return batch_idx

    def validation_epoch_end(self, outputs):
        print("VALIDATING", self.metric.compute())
        self.metric.reset()

if __name__ == "__main__":
    m = Module()

    datasets = [torch.rand([5]) for __ in range(100)]
    train_loader = torch.utils.data.DataLoader(datasets, batch_size=8)
    val_loader = torch.utils.data.DataLoader(datasets, batch_size=1)

    trainer = pl.Trainer(
        num_sanity_val_steps=0,
        max_epochs=5,
        accelerator="ddp_cpu",
        num_processes=4,
    )
    trainer.fit(m, train_loader, val_loader)

prints out:
  warnings.warn(*args, **kwargs)
Epoch 0:  14%|████████████████▉                                                                                | 4/29 [00:00<00:00, 64.38it/s, loss=-3.59, v_num=39VALIDATING tensor(400.)                                                                                | 0/25 [00:00<?, ?it/s]
VALIDATING tensor(400.)
VALIDATING tensor(400.)
VALIDATING tensor(400.)
Epoch 1:  14%|████████████████▊                                                                                | 4/29 [00:00<00:00, 242.12it/s, loss=-3.79, v_num=39VALIDATING tensor(100.)                                                                                | 0/25 [00:00<?, ?it/s]
VALIDATING tensor(100.)
VALIDATING tensor(100.)
VALIDATING tensor(100.)

Even if this works, I still think it's subpar, b/c this seems like something that could be handled automatically fairly easily.

I'm calling reset() on validation_epoch_end, does that work for your case?

That does not work for me, since something seems to funky with the first epoch (expand the <details> tag in the "Alternatives" section for the full details).

It was automatic before, this is why they removed it: https://github.com/PyTorchLightning/pytorch-lightning/pull/5409/#issue-551272376

IIUC, that's totally separate -- that's about whether we call Metric.reset when we call Metric.compute() (which I totally agree is unexpected and confusing).
But I would argue that users would expect that Trainer calls Metric.reset for you at the beginning of each epoch (or if it does, it would be seen as "neat integration" not "confusing behavior). After all, Trainer already calls reset for you if you use LightningModule.log with the metric, but that seems to be an artificial limitation (given that you can have a 5-line implementation to reset every metric child).

@alanhdu it seems to be that you are mixing training and validation calls. update is happening in training method and compute+reset is happening in validation. Could you move the compute+reset to train_epoch_end and see if that fixes your problem.

@SkafteNicki  -- I have the same issue if I rename validation_epoch_end to training_epoch_end -- for some reason the first epoch (epoch 0) has a count of 400 instead of 100.

Dear @alanhdu,
Would you mind sharing a reproducible script ?
Best,
T.C

Ah -- sorry. I guess embedding it in the <details> tag made it hard to find.
import pytorch_lightning as pl
import torch
from torch import nn

class Counter(pl.metrics.Metric):
    def __init__(self):
        super().__init__()
        self.add_state("test", default=torch.tensor(0.0), dist_reduce_fx="sum")

    def update(self, x: int):
        self.test += x

    def compute(self):
        return self.test

class Module(pl.LightningModule):
    def __init__(self):
        super().__init__()
        self.linear = nn.Linear(5, 1)
        self.metric = Counter()
        self.metric.reset()

    def configure_optimizers(self):
        return torch.optim.Adam(self.linear.parameters())

    def training_step(self, batch, batch_idx):
        self.metric.update(torch.tensor(float(len(batch))))
        return self.linear(batch).sum()

    def validation_step(self, batch, batch_idx):
        return batch_idx

    def training_epoch_end(self, outputs):
        print("LENGTH", self.metric.compute())
        self.metric.reset()

if __name__ == "__main__":
    m = Module()

    datasets = [torch.rand([5]) for __ in range(100)]
    train_loader = torch.utils.data.DataLoader(datasets, batch_size=8)
    val_loader = torch.utils.data.DataLoader(datasets, batch_size=1)

    trainer = pl.Trainer(
        num_sanity_val_steps=0,
        max_epochs=5,
        accelerator="ddp_cpu",
        num_processes=4,
    )
    trainer.fit(m, train_loader, val_loader)

I also feel like we're getting a little side-tracked by the script -- even aside from the potential bug in the metrics + DDP, I think that calling self.counter.reset() "manually" is unnecessary. Would people object to a PR that adds the:
def reset(c):
    if isinstance(c, pl.metrics.Metric):
        c.reset()
    return c
module.apply(reset)

to the appropriate place in Trainer?

@alanhdu that will not work in all cases.
Lets say that i set val_check_interval=0.5 then I will be evaluating my validation metrics twice as often as my training metrics. This means that I need a way to differentiate which metrics needs be reset when, and I cannot do a global reset.
I therefore think the best solution is the user explicitly tells when they want to reset which metrics.

@alanhdu just tested your script in a multi-gpu setting and it works fine (prints 100 for all epochs). The problem therefore just seems to be with "ddp_cpu" (which I for some reason cannot get working locally), that is only intended to be used for debugging.

Lets say that i set val_check_interval=0.5 then I will be evaluating my validation metrics twice as often as my training metrics. This means that I need a way to differentiate which metrics needs be reset when, and I cannot do a global reset.

Ah -- I did not know about that feature. Yes, that makes sense to me.


Click n times or all to the end

For testing, it can be good to just click n times on + Vis flere and then check if content looks okay. For crawling all content, just wait until the end.

Not doing it this way


RELAX_COORD_BOUND broken?

No doubt this is my fault but relaxed coord bounds are broken in master.
@WardF you will certainly want to fix this before release. I will have a PR up in a moment.

I'm not observing any test failures in master with relaxed coord bounds turned on or off; can you expand on the failure you're seeing?

I am not sure if this is broken, when I look at it from the netcdf-c side, it seems to be working fine. But when PIO tries to use this, I get an EEDGE error where I would expect relaxed coord bounds to let me get through, and it is not. Quite possibly this is a PIO bug. I just didn't want you to release 4.6.3 the day after I found a bug, but didn't tell you. ;-)

Thanks! Also no worries; I released 4.6.3 on Friday, so barn door etc etc. can always shoot 4.6.4 out immediately if there’s a reason to

OK, I am starting to believe this is a PIO problem anyway. I will close this ticket and reopen if I find I was wrong.


Incompatbility with Obscure API (Armor Piercing does not work)

The Issue
When Obscure API is installed, Attributelib's  armor piercing  effect does not function. I have tested this in multiple formats:

Potion effect form (personal commision)
Gems
/attribute command
Adding it to equipment via /give command

With Obscure API, my Netherite Sword does not get any piercing.

When uninstalled, my Netherite Sword deals the full 8 damage (as I set my armor piercing attribute to 31)

Obscure API is a closed-source all rights reserved mod.  No support is provided for these mods.  You can report this as a bug to them and ask them resolve it on their end.


Can't enter indexes

There is a known issue where if you have a wide enough time range with enough scans you will be unable to see the textbox needed to enter indexes. A fix is being investigated and should be released this afternoon.

Fixed in release v1.01


Remove: PR

I created a pull request in the upstream of my forked repo in error.
I closed it but can we remove or hide that in any way from public view?

Hi,
Sorry I don't think we can delete pull requests, they can only be closed.
If there's any sensitive information in the PR, then you need to contact Github.


Update MCE sub list and label handling

Issue: https://github.com/stolostron/backlog/issues/26116
Signed-off-by: Jakob Gray<EMAIL_ADDRESS>

/override ci/prow/sonar-pre-submit


unrecognized

hello,
I need help : I am on virualbox, and when I start the fake AP, it write me that :
[I] Restarting tail in 1 sec... (press 'CTRL + C' again to stop)
tail: unrecognized file system type 0x794c7630 for ‘/root/Desktop/mitmAP/logs/mitmap-sslstrip.log’. please report this to<EMAIL_ADDRESS>reverting to polling
what can I do ? thank you already ;)

I had the same issue but after i created the log file it seemed to be ok, not sure if that is the best fix but worked for me.
nano /root/Desktop/mitmAP/logs/mitmap-sslstrip.log Then add a few spaces and save ?

Ok thank you !
I'll try it...

but what do you mean about few spaces ?

Press space bar so the file isn't empty ?

ok but after, what will I do to save ?

crtl + x  then y will save the file.

I know but what about the File Name to Write ?

This is the file name: mitmap-sslstrip.log


⚠ ClamAV has degraded performance

In 65bc192, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 87 ms

Resolved: ClamAV performance has improved in ceaa219.


White flicker when clicking on form fields

Current Behavior
Currently it flickers white when you click on an form field for example a input (see the videos below)
https://user-images.githubusercontent.com/49494752/210417672-588304de-579d-4869-b875-418026400222.mp4
https://user-images.githubusercontent.com/49494752/210417887-ba7cb60a-d92e-46bb-a6cc-ef093c8ffd65.mp4
Steps To Reproduce
Go to the form page in the documents (https://www.skeleton.dev/elements/forms/preview)
Anything else?

The issue occurs in the latest versions of Chrome, Firefox

Just noting this doesn't occur on the Tailwind plugin preview page, so it's not an inherited issue from that.
https://tailwindcss-forms.vercel.app/

@jerebtw I've pushed a small fix for this to our dev branch, if you wish to preview:
https://dev.skeleton.dev/elements/forms/preview
The issue was related to use the use of Tailwind's transition-all class. When I replaced that with transition the issue stoped. I'm not 100% of the cause, but the fix works so I won't complain!
Furthermore, I realized we were double up on both ring and border style for inputs, making the input edges look much thicker than intended. So I've resolved that while I was at it. You may note that each theme has slightly thinner borders, but these should now match the theme settings and user expectations.
Given this is merged into our dev branch, it means it'll be a part of the next release, scheduled for a week from today. Thanks for reporting this!

It looks really great thanks 😄


Fix incorrect minimum input size for applications

This PR fixes incorrect minimum input size for applications.

Line 274: VGG16
Line 329: VGG19
Line 441: InceptionV3
Line 496: InceptionResNetV2
Lines 623-: DenseNet

While I agree with those changes, I think it would make sense if docs/templates/applications.md was in keras-team/keras-applications. I'm pretty sure we can fetch it automatically in the autogen.py. What do you think @taehoonlee @fchollet ?


Clicking on "Glamorous Toolkit Book" produces "KeyNotFound: key 'feenk.com/gtoolkit/lepiter' not found in LeLogicalDatabase"

Version
OS

0.8.690
Manjaro

Here is the error screenshot:

Thanks! We will solve this shortly.

Should work now. Please reopen the issue if the error appears again.


DataTable - Cannot display html inside cells

It's currently impossible to display html inside a cell.

Not a bug.. Was adding the html as a string instead of an element


NPM package

Hey!
I discovered this repo via the Community Showcase of the March Svelte newsletter.
Your radial menu is awesome, you should publish it on NPM, it really had a "Wow! effect" on me! Amazing work! 🤩

Hi @Nilmanduil, thank you for the kind words! I definitely want to do this 😄
I'll leave this issue open so I can remind myself to work on it 👨💻


 For the data storage use either of localStorage or a server side json file.

Also decide whether to use any plugin, Here is a list to look at

redux-persist-store
redux-localstorage
redux-storage

#11

We are going use falcor for most of the data fetching, so most of the storage modules will not be needed.

Closing this as this should be handled by #6


[WIP] Common library no longer depends on finatra

What is this PR trying to achieve?
Remove finatra dependencies
Who is this change for?
🏷

Closing as too long in the toothy.


Revert "Revert "[TypeChecker] PreCheck: Don't strip away tuple/paren from subscripts …""

Reverts apple/swift#37221

@swift-ci please smoke test


[Release-1.28] - K3s etcd snapshot reconcile consumes excessive memory when a large number of snapshots are present

Backport fix for

#10450

Validated on release-1.28 branch with commit 2701d8fca45cf675b481e927827dd1dceb51b01c
Environment Details

Infrastructure

[x] Cloud
[ ] Hosted

Node(s) CPU architecture, OS, and Version:

$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.2 LTS"

$ uname -m
x86_64

Setup Size: 4Gb Memory, 2vCPU, 30G disc size.
Cluster Configuration:

HA: 3 server/ 1 agent

Config.yaml:

token: xxxx
cluster-init: true
write-kubeconfig-mode: "0644"
node-external-ip: <IP_ADDRESS>
node-label:
- k3s-upgrade=server

etcd-snapshot-retention: 255
etcd-snapshot-schedule-cron: "* * * * *"
etcd-s3: true
etcd-s3-access-key: <access_key>
etcd-s3-secret-key: <secret_key>
etcd-s3-bucket: <bucket>
etcd-s3-folder: <folder>
etcd-s3-region: <region>

Testing Steps

Copy config.yaml

$ sudo mkdir -p /etc/rancher/k3s && sudo cp config.yaml /etc/rancher/k3s

Install k3s

curl -sfL https://get.k3s.io | sudo INSTALL_K3S_COMMIT='2701d8fca45cf675b481e927827dd1dceb51b01c' sh -s - server

Verify Cluster Status:

kubectl get nodes -o wide
kubectl get pods -A

Apply the k3s extra metadata here:

kubectl apply -f https://gist.githubusercontent.com/aganesh-suse/52c3d6c3d7fe70141fa3a49431ac0032/raw/20039a159ab0f5fce1930f5ec12f6afc2b034784/k3s-etcd-snapshot-extra-metadata.yaml

Monitor the memory usage of k3s.service, while taking snapshots every 1 minute for upto 255 snapshots.

for (( I=0; I < "${ON_DEMAND_SNAPSHOT_COUNT}"; I++ ))
do
    sudo k3s etcd-snapshot save
    sleep 5
done

write_mem_usage_k3s_to_file () {
    while true; do
        echo "$(top -b -n 1 | grep k3s)"  | tee -a top-output.log
        sleep 1
    done
}

ttyplot_k3s_memory () {
    K3S_PID=$(ps aux | grep 'k3s' | head -n 1 | awk '{print $2}')
    while :; do grep -oP '^VmRSS:\s+\K\d+' /proc/$K3S_PID/status \
    | numfmt --from-unit Ki --to-unit Mi; sleep 1; done | ttyplot -u Mi
}

P.S: We run out of disc space before running out of memory space by ~280 snapshots, so capping the snapshot count to 255 for testing purposes.
Replication Results:

k3s version used for replication:

$ k3s -v
k3s version v1.28.12+k3s1 (4717e2a5)
go version go1.22.5

Validation Results:

k3s version used for validation:

$ k3s -v
k3s version v1.28.12+k3s-2701d8fc (2701d8fc)
go version go1.22.5

Memory Usage Comparison Results
Plot to compare memory usage between released version and latest commit:
Compare % Memory, Max Memory used in Mb, Avg Memory in Mb for both released version and latest commit for various snapshot counts - 120, 150, 170, 200, 230, 250, 255.
          v1.28.12       |  release-1.28 commit: 2701d8fc
Snaps   %M  Max     Avg     %M   Max    Avg
100     47  2019    1870    41  1863    1730
120     53  2175    1994    50  1965    1843
130     56  2245    2157    53  2077    1969
150     61  2477    2347    55  2190    2113
170     68  2766    2573    59  2413    2302
200     75  3046    2841    66  2628    2564
k3s service restarted by itself around 220 snapshots for the old version. so the following comparison numbers would look skewed/lesser for the old version. so providing only latest commit version results. The results range should be similar to other branch results.
230                         67  2638    2384
250                         70  2879    2538
255                         73  2879    2679

Observations
Memory wise, till about 120 snapshots, the new commit is 2% lesser than the older vesion memory usage.
On an average, the difference starts increasing to ~5% till 200 snapshots, then to ~10+% lesser memory usage for more than 200 snapshots.


[rom_cntrl, dv] Test to verify successful rom check

Following files have been added/ modified in this commit:

rom_ctrl_chk_successful_vseq: Sequence that writes the top 8 address
spaces to ensure we have the expected digest matching with KMAC digest.
kmac_app_device_seq: It assigns the rsp_digest_share0/1 fields from the values
present in kmac_app_agent_cfg. Randomization of the rsp_digest_share0/1
fields has been removed.
rom_ctrl_env: Sets the value of rsp_digest_share0/1 in
kmac_app_agent_cfg.
kmac_app_agent_cfg: rsp_digest_share0/1 fields have been added.

This PR contains a merge commit. Could you clean things up? (I think you probably just need to do a hard reset to the commit before the merge, then a rebase: if the output of git log is confusing, it's probably clearer with gitk)

@weicaiyang: This PR touches the KMAC app agent because the sequence on the rom_ctrl side needs to be able to predict the digest that we're going to get as a response. Would you mind taking a look at this? I want to make sure we're not messing up your code! :-)


Using Grafana platform (SaaS) -  PoC

User Story
As a Modernisation Platform Engineer or a Modernisation Platform user
I need to use our own Grafana offerings
So that I can verify the health/cost/security/usage/capacity of the Modernisation Platform or the Application (or Application Infrastructure)
Value / Purpose
This is to understand what it means if we were to continue using the existing AP solution of the Observability platform and so that we can compare it to other Grafana based solutions and understand advantages and disadvantages of them all.
The platform should be assessed and compared to other solutions based on the following criteria:

ease of use (this could be, but is not limited to: documentation, user friendliness/experience from both end user and engineering user point of view, configuration)
management overhead
price
scalability
expandability
limitations

Context / Background
The Modernisation Platform has a need to observe the platform and application teams have a need to observe their applications or application infrastructure.
Currently Modernisation Platform uses a number of monitoring and alerting tools/platforms: Observability Platform (by AP), slack alerting, AWS Cloud Watch -> SNS -> PagerDuty -> slack notifications, Security Hub data.
Application teams may or may not be using observability tools to meet their observability needs.
There is a need to have a centralised place where Modernisation Platform and Modernisation Platform users can observe their workloads. This PoC is one of a number of solutions we are going to explore. The solution should also take it into account that this may be the future Observability Platform for the whole MoJ.
Useful Contacts
@ASTRobinson
Additional Information
Useful documents:

State of observability in Modernisation Platform
Observability Tools research
AP Observability Platform code
AP Observability Platform tenant module

Definition of Done

[ ] https://github.com/ministryofjustice/modernisation-platform/issues/8778
[ ] #8692
[ ] #8679
[ ] #8680
[ ] #8681
[ ] #8682
[ ] #8683

https://epmp.grafana.net/


Climate Mill - Internal Server Error

Home Assistant release with the issue:

0.89.0, 0.88.2
Last working Home Assistant release (if known):
0.87.x
Operating environment (Hass.io/Docker/Windows/etc.):

Hassio on Docker, host Ubuntu 18.04.2
Component/platform:

https://www.home-assistant.io/components/climate.mill/
Description of problem:
Alot of log events showing
Failed to send request, {"errorCode":3000,"error":"internal error"}
23:06 components/climate/mill.py (ERROR) - message first occured at 20:36 and shows up 150 times
Im only using HA to read values from Mill entity. Do not now how this error affects other setup's.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

climate:
  - platform: mill
    username: !secret myemail
    password: !secret default_password

Traceback (if applicable):
Failed to send request, {"errorCode":3000,"error":"internal error"}

Additional information:

I see the same problem in the log.

I have the same issue as well. Started with .88 or .89 but worked well in .87.

Same error here, even though it seems to be working fine... I can set temperatures and read data from my Mill oven without any issues (except this annoying error) :)


Using DeepVecFont for icon generation

Hi,
Thank you for the great work and for releasing your code for public use. The current preprocessor converts fonts (ttf files) to sfd format and then maps that to a restricted svg format for training the model. If we already have the svg files instead of the ttf files (e.g. for icons) and want to convert that to the same simplified svg format, in order to use the model to train on icons, could you please provide some guidance on the preprocessing needed for this direct conversion from svg to the simplified svg needed for the model?
thanks

Hi athena,
Thank you for your advice!
Extending my work to common svg generation is very interesting.
I will try to add some scripts to handle *.svg inputs, stay tuned!

Hi,
Thank you for the great work and for releasing your code for public use. The current preprocessor converts fonts (ttf files) to sfd format and then maps that to a restricted svg format for training the model. If we already have the svg files instead of the ttf files (e.g. for icons) and want to convert that to the same simplified svg format, in order to use the model to train on icons, could you please provide some guidance on the preprocessing needed for this direct conversion from svg to the simplified svg needed for the model?
thanks

Hi athena,
Thank you for your advice!
Extending my work to common icon generation is very interesting.
I will try to add some scripts to handle *.svg inputs, stay tuned!


Table 252 General Posting Setup

It would be nice if there could be a Handled Publisher in all functions for the determination of the respective G/L accounts. So it would be possible with appropriate adjustments in the financial accounting to let determine own accounts.
e.g. GetSalesPrepmtAccount

Sorry, please simply create you own setup if you need this.


Alpn

There seem to be a problem with ClientHelloAlpnAdderStream and ServerHelloAlpnAdderStream. AuthenticateAsClientAsync/AuthenticateAsServerAsync fails when calling with this stream.
Even though we can see that the servers respond with positive alpn h2 in its initial handshake with our modified client hello, something is happening inside that makes the Authentication fail in subsequent internal steps in AuthenticateAsClientAsync/AuthenticateAsServerAsync.
@honfika I know you mentioned this before. I initially thought it could be because we didn't yet support HTTP/2 preface in the proxy. But I think that still the call shouldn't fail, since HTTP/2 traffic will start only after AuthenticateAsClientAsync/AuthenticateAsServerAsync is complete.

Apparently the issue is message hash mismatch as observed in comment from MS employee
https://github.com/dotnet/corefx/issues/4721#issuecomment-313239925

Yep, check here: https://tools.ietf.org/html/rfc5246#section-7.4.9
Both ends keep hashes of every message exchanged during the handshake. If you intercept the ServerHello message from SslStream and change it, the client will compute a hash for that message that's different than what the server computed before sending it.

Then it should be changed, too:)

@honfika I noted that the call fails at below line for AuthenticateAsServer
https://github.com/dotnet/corefx/blob/master/src/System.Net.Security/src/System/Net/Security/SslState.cs#L810
At that point its about to finish the negotiation. The SslStream is a wrapper around Windows SChannel. So I think the hash itself is computed and verified by SChannel with native calls to it.
We are stuck with two options either rewrite an updated SslStream (in System.Net.Security.sln) with ALPN support as done with the PR in corefx (also for other platforms & Mono) or recompute the hash like you said which seems tedious or may be even impossible.


How to save the svg?

Hi, the documentation mentions:
"Then you can visit http://flowchart.js.org/ and translate the generated textual representation into SVG flow chart diagrams"
But how do I download the SVG generated at http://flowchart.js.org/?

I suggest you can open the developer tools (see https://developer.chrome.com/docs/devtools/open/ for help), go Elements tab and copy the svg source code
<svg>...</svg>

from
#demo1 > table > tbody > tr > td.diagram.diagram1 > svg

and paste it into a .svg file.


Add missing insert APIs for predicates

What problem does this solve or what need does it fill?
For consistency, we are missing the following methods:

insert_if_new_and  (insert_if_new + insert_if)
try_insert_if_new_and (try_insert +insert_if_new + insert_if)

Additional context
https://github.com/bevyengine/bevy/pull/14897 added the predicate methods, and we decided to leave these out for some naming bikeshed. This bikeshed happened on Discord. This is a rather uncontroversial result that does not require any big changes.

Technically there are 4 modifiers for insert: try_, _by_id, _if_new, and _if. Adding all the missing methods would mean having 16 different insert methods. I assume this issue is not proposing to add all of those methods, but some subset of them. What would that subset be?

I didn't know about by_id, huh.
This issue was primarily for the _if variants that were missing


Proposal: Do not payout apps that solve the same problem of a higher ranked app

This issue is about enforcing diversity of apps and about encouraging collaboration between app developers.

Do you have an example in mind of two apps that are competing head-to-head?
I think we want to see both a diversity of apps and healthy competition between apps that are gunning to solve the same underlying problem for customers. So, I doubt we'd want to programmatically penalize apps that directly compete given their comparative ranking.
However, if we want to promote diversity without penalizing competition, perhaps there's a way in which we can give apps a positive boost for solving a problem that hasn't yet been attempted yet. It could be a "new problem boost", for example, give during the first few months of its debut in App Mining.

I started a spreadsheet with competitors to Google Drive, Google Keep and
Google Blogger/Medium at:
https://docs.google.com/spreadsheets/d/1ABMMy9bR_vbdOGtzryNGVzlNmwe32TjywNAM2LIuNaM/edit#gid=0
On Wed, 20 Feb 2019, 12:09 Mark Hendrickson<EMAIL_ADDRESS>wrote:

Do you have an example in mind of two apps that are competing head-to-head?
I think we want to see both a diversity of apps and healthy competition
between apps that are gunning to solve the same underlying problem for
customers. So, I doubt we'd want to programmatically penalize apps that
directly compete given their comparative ranking.
However, if we want to promote diversity without penalizing competition,
perhaps there's a way in which we can give apps a positive boost for
solving a problem that hasn't yet been attempted yet. It could be a "new
problem boost", for example, give during the first few months of its debut
in App Mining.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/blockstack/app-mining/issues/37#issuecomment-465530263,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYcWX-EB3IQnX4qEfu12KXH5aF28o4kks5vPSzUgaJpZM4bE9Xn
.

Sounds good. That spreadsheet looks like it may need to be set to public though?

The apps could compete between each other for a share of the amount that the highest ranked app would get. Lets say app A ranks 5th, app B ranks 6th, app C ranks 100th then app A, B and C share $8,192.00  (5th rank's amount): A the most, B a little less than A, C much less than B.

Wouldn't this also dissuade folks from trying to make a massive improvement to the status quo due to revenue share proposed here? Also, aren't folks already incentivized to be unique in order to stand out? The new app booster bonus Mark described seems like a simple way to approach this actually becomes a real problem. My feeling is that if major changes are to be made to App Mining, I'd assume this would be low on the current list as it adds complexity and is not a clear hair on fire problem with how App Mining works today.

You propose that app mining should and can give incentives to be unique and
to make massive improvements to the status quo?
For the current reviewers, I don't see any that gives higher score for
uniqueness. And the current ranking shows that in the top 20 there are a
few duplicates.
From a user perspective and for the developer ecosystem, I favor diversity
over competition.
You can make massive improvements by contributing to the best app.
On Wed, 20 Feb 2019, 17:20 Patrick Stanley<EMAIL_ADDRESS>wrote:

Wouldn't this also dissuade folks from trying to make a massive
improvement to the status quo due to revenue share proposed here? Also,
aren't folks already incentivized to be unique in order to stand out? The
new app booster bonus Mark described seems like a simple way to approach
this actually becomes a real problem. My feeling is that if major changes
are to be made to App Mining, I'd assume this would be low on the current
list as it adds complexity and is not a clear hair on fire problem with how
App Mining works today.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/blockstack/app-mining/issues/37#issuecomment-465648261,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYcWU7asNZU3yoblcdEHUG6MuA5i-Ruks5vPXXRgaJpZM4bE9Xn
.

That would be to say that Larry & Sergey could have made the biggest impact by contributing to Altavista, Lycos, Excite, or whichever search engine you would have picked as best. Or Zuck should have gone to work for MySpace or Friendster.
SpringRole doesn't view Note Riot as a competitor. In fact, it may be a partner. And we do more than just that one feature. So which do you pick to be the one to use or do you divide it up among many?
For the proposal to give #100 an equal share to #5, that means that there would be a perverse incentive to create a lot of garbage apps as parasites.
It's rarely a case where there are too many competitors in a field. More often, there aren't enough. The reason that most disappear isn't that they shouldn't have been started (although that is the case for some), they disappear because the best (in tech and/or marketing) win. If you cut down competition, you won't get as many different takes on a problem.

That would be to say that Larry & Sergey could have made the biggest impact by contributing to Altavista, Lycos, Excite, or whichever search engine you would have picked as best. Or Zuck should have gone to work for MySpace or Friendster.

This. Free markets for the worth, let the best prevale.
Also, it's super subjective to try to categorize apps, specially at this early stage, even the ones you mention in you spreadsheet have substantial differences and could end up pivoting into very different markets.

@thedavidlewis @benoror We are not in a free market here, it is a very regulated competition for money. Apps in the app mining program get build to be optimized for the algorithms of the distribution of this money.
My proposal could be more compared to saying that Jeff Bezos should invest in Amazon and Washington Post to achieve his goals. Your proposal would be to say that Jeff Bezos should invest in Amazon, Bol, Alibab and in Washington Post, NY Times,..

SpringRole doesn't view Note Riot as a competitor. In fact, it may be a partner. And we do more than just that one feature. So which do you pick to be the one to use or do you divide it up among many?

From a data/blockstack perspective, both deal with "Taking notes". The algorithms should encourage that SpringRole and Note Riot cooperate.

For the proposal to give #100 an equal share to #5, that means that there would be a perverse incentive to create a lot of garbage apps as parasites.

No, not equal shares.

Also, it's super subjective to try to categorize apps, specially at this early stage, even the ones you mention in you spreadsheet have substantial differences and could end up pivoting into very different markets.

Washington Post and NY Times have also substantial differences. The algorithms should pick one.

That would be to say that Larry & Sergey could have made the biggest impact by contributing to Altavista, Lycos, Excite, or whichever search engine you would have picked as best. Or Zuck should have gone to work for MySpace or Friendster.

I hope we do not create another Zuck and Larry & Sergey that sucks the data out of us.

I tend to agree that this change will create more downsides than upsides. This assumes that we could accurately sort and compare apps, which I don't believe we could.
Is there a good solution here and/or anyone still feel worth pursuing?


Fix contributed Notebook Action bar actions

Fixes https://github.com/microsoft/azuredatastudio/issues/11563
So first off - I'm not really pleased with this solution. I feel like there has to be a better way and I'm going to do some more digging. One thing I wanted to look at is overriding the action runner instead. But this does fix the issue and is scoped down so if we need to for the release this is probably an acceptable short term fix.
edit : Nevermind the below. Chris pointed out this was working until a recent VS Code merge which rewrote a lot of this actionbar stuff. So while the end result is the same (we're trying to send over the entire NotebookComponent which fails) the cause is because of this rewrite.
The root of the problem was introduced in
https://github.com/microsoft/azuredatastudio/commit/47687ff6b2000fc28768113e9cef800226c7fec5. New actions were added for doing things on the Notebook. These need the Notebook context in order to run their actions and so the context for the Taskbar was set to the NotebookComponent here : https://github.com/microsoft/azuredatastudio/blob/main/src/sql/workbench/contrib/notebook/browser/notebook.component.ts#L498
This works fine for all the core actions - but we get in trouble with the contributed actions since those are sent across the RPC boundary to the extension host. And so we fail at trying to serialize the NotebookComponent.

Coverage decreased (-0.002%) to 37.621% when pulling 5a4500f31c0012dbb1c78a11d519f5c01b430794 on chgagnon/fixNotebookMenuActions into 6751dffbc39adff702feb2768fb2df7de26d4e45 on main.

Closing this as I found a better solution.
@halerankin - No, before we were setting the context to the NotebookComponent itself. But that doesn't pass over the RPC boundary and so for the contributed actions like the Manage Packages it was throwing when it tried to convert the object into raw JSON to pass over.


Android 10 - ERR_NO_SUPPORTED_PROXIES

Hello,
I'm just working with ur library and gets this error using android 10 and not reproduced in android 8
DailymotionFragment$onViewCreated: WebView [166493760] received an error with code: -1, description: net::ERR_NO_SUPPORTED_PROXIES from URL:  https://www.dailymotion.com/embed/video/[videoID]
Can u help me ?

Hi,
This looks like an error of a proxy configuration.
Did you check if there is a proxy setting for you connection ? Is it correctly set up?
Best regards,


ContractFactory provider type

The type of providerOrAccount  in the ContractFactory constructor is currently a Provider | Account. Should it be a ProviderInterface | ContractInterface instead?

yes it should, thanks


setup prettier

Introduces Prettier.
It's integrated as an ESLint plugin. So yarn lint:js will lint that all files are formatted with Prettier. yarn lint:js --fix will prettify the code.
It uses the same prettier configuration as ember source. I think just following that one helps to avoid discussions about best configuration. Also the source code will look similar to the one of other projects in ember ecosystem.
It adds a .gitattribute. This enforce a specific git configuration for this repository. It addresses a common issue if using Prettier on Windows.
It adds a pre-commit hook as listed in the requirements in #1033. But to be honest I don't like it. Have explained reasons in #1033.

Addressed outstanding review comments and fixed merge conflicts by a rebase. Ready to be merged.

@jelhan after merging the other PR, this now has some merge conflicts! Sorry! :)

@jelhan after merging the other PR, this now has some merge conflicts! Sorry! :)

Rebased again. Luckily not much work involved. :smile:

Thanks! :tada:


Calling whereNotNull on a Collection does not remove the nullable type

Larastan Version: 2.1.2
--level used: 8

Description
Calling ->whereNotNull on a collection does not remove the nullable type.
Laravel code where the issue was found
/**
 * @return Collection<int, string>
 */
public function getStrings() {
  return collect(['foo', null, 'bar'])
      ->whereNotNull();
}

The code above will throw an error that the TValue should be string|null.

Hi,
Yes, there is no support for that now. Until it gets implemented you can try to use filter instead if possible.

For other people have this issue, I have changed the code fragment from:
/**
 * @return Collection<int, string>
 */
public function getStrings() {
  return collect(['foo', null, 'bar'])
      ->whereNotNull();
}

to
/**
 * @return Collection<int, string>
 */
public function getStrings() {
  return collect(['foo', null, 'bar'])
      ->filter(fn(?string $value) => ! is_null($value));
}

Hope it helps someone.


Add viem to read/write the blockchain in EVM Modal PnP

Update some packages and add Viem
Add the possibility to choose Viem to read/write the blockchain

@rtomas is this ready for review?


Typo "Azure function"→"Azure Function"

https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/how-to-custom-commands-update-command-from-web-endpoint

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

A nomenclature overhaul for Azure Functions references in this repo is underway internally. We won’t merge pull requests that make updates in this area at this time. Given that, we'll close this pull request. @hyoshioka0128 Thank you for your contribution to the documentation.

A nomenclature overhaul for Azure Functions references in this repo is underway internally. We won’t merge pull requests that make updates in this area at this time. Given that, we'll close this pull request. @hyoshioka0128 Thank you for your contribution to the documentation.


Extract common toSerializedPage test function

Refactors toSerializedPage function to add a common helper function for serializing
RowVector to PrestoPage format, in file SerializedPageUtil.cpp.

Hi @czentgr, @aditi-pandit, could you please help review this PR?

@kgpai @kagamiori : Please can you help with review and merge.

Hi @kgpai, @kagamiori, could you please help review/merge this change?

Hi @pramodsatya, could you rebase the PR onto the latest main? It's needed for merging. Thanks!

Hi @pramodsatya, FYI, there are some internal build and test failures. I'm investigating. Will get back to you. Thanks!

Hi @pramodsatya, FYI, there are some internal build and test failures. I'm investigating. Will get back to you. Thanks!

Thanks @kagamiori! Please let me know if I need to rebase again or any other changes are needed.


Running DOOM

How do you run original doom? If this indeed gets popular people will try to run doom on "embedded devices" though there is no tutorial online

Nah man I was just kidding, sorry

No worries, I had a good laugh when I read this one, but we should keep the issues light. :D


Make a file-based way to update pool incentives

Currently, EditPoolIncentives proposals have to be done via the CLI using command line arguments.  There should be a way to easily put the records in a file and just pass in a file to the CLI.  (similar to gamm's create-pool)

Does https://github.com/osmosis-labs/osmosis/issues/368 close ^ or no?

I think at least currently, pool-incentives related proposals can be done via. a yml file (at least with beaker)

I briefly looked into this:
https://github.com/osmosis-labs/osmosis/blob/fe98f6e4453cd32035d277c21ef2f3669b677bb2/osmoutils/parse.go#L61
I don't see a way to parse a yml file, only json.
I think the goal of this issue is to make it possible to utilize yml format via CLI like beaker does.
I suspect that beaker has custom logic for parsing yml files that's why it works. cc: @iboss-ptk is that the case?

@p0mvn yes, it's this struct which, when the file has yaml or yml extension, it gets deserialized to the said struct.

Closing due to obsolete


fix: TimeTable average with nulls calculations

SUMMARY
Fix for issue #12962
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

ADDITIONAL INFORMATION

[X] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thanks for the fix, tested locally with same dataset, and LGTM! ✅


Vault

Is your feature request related to a problem? Please describe.
It allows pass vault data to values.yaml/helmwave.yml
Describe the solution you'd like
$ vault kv get -format=json secret/webapp/prod
{
    "pass": "12345"
}

values.yaml
replica: 1
env:
  password: {{ vault 'secret/webapp/prod', 'pass'  }}

$ helmwave plan
replica: 1
env:
  password: "12345"

Describe alternatives you've considered
gomplate #103 ?
https://docs.gomplate.ca/datasources/#using-vault-datasources
Additional context
Btw, I prefer use vault-operator. Because it is safer.

@r3nic1e Could you show example how to use it. Here or Docs.

for helmwave 0.14.0
Step 1. Create next secret:
$ aws secretsmanager create-secret --name hello/foobar --secret-string '{"foo":"bar"}'

helmwave.yml
repositories:
  - name: bitnami
    url: https://charts.bitnami.com/bitnami

template:
  gomplate:
    enabled: true
    data:
      sources:
        secret:
          url:
            scheme: aws+sm
            path: 'hello/foobar'

releases:
  - name: app
    chart:
      name: bitnami/nginx
    namespace: test
    values:
      - vaules-secret.yaml

vaules-secret.yaml
secret: {{ ds "secret" }}

helmwave build
vaules-secret.yaml
secret: {"foo":"bar"}

And also for vault it wiil
helmwave.yml
template:
  gomplate:
    enabled: true
    data:
      sources:
        secret:
          url:
            scheme: vault
            path: 'hello/foobar'

Wouldn't it be difficult for @r-mironov  to reflect this in the documentation?