Running VNM on a MacBook pro as per the instructions. All went well, but in an open terminal window, module avail returns an error:

Running VNM on a MacBook pro as per the instructions. All went well, but in an open terminal window, module avail returns an error:
Lmod has detected the following error:  avail is not possible,
MODULEPATH is not set or not set with valid paths.
root@36e04b583208:~# export | grep MODULE
declare -x LMOD_DEFAULT_MODULEPATH="/vnm/containers/modules"
declare -x MODULEPATH="/vnm/containers/modules"
declare -x MODULESHOME="/usr/share/lmod/lmod"
root@36e04b583208:~# ls -l /vnm
total 0
Originally posted by @TomEmotion in https://github.com/NeuroDesk/neurodesk/discussions/36

fixed in container version vnmd/vnm:20210104

fixed in container version vnmd/vnm:20210104


[SPARK-17019][Core] Expose on-heap and off-heap memory usage in various places

What changes were proposed in this pull request?
With SPARK-13992, Spark supports persisting data into off-heap memory, but the usage of on-heap and off-heap memory is not exposed currently, it is not so convenient for user to monitor and profile, so here propose to expose off-heap memory as well as on-heap memory usage in various places:

Spark UI's executor page will display both on-heap and off-heap memory usage.
REST request returns both on-heap and off-heap memory.
Also this can be gotten from MetricsSystem.
Last this usage can be obtained programmatically from SparkListener.

Attach the UI changes:

Backward compatibility is also considered for event-log and REST API. Old event log can still be replayed with off-heap usage displayed as 0. For REST API, only adds the new fields, so JSON backward compatibility can still be kept.
How was this patch tested?
Unit test added and manual verification.

Test build #63656 has started for PR 14617 at commit a5e9d46.

Test build #63656 has finished for PR 14617 at commit a5e9d46.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class SparkListenerBlockManagerAdded(
class StorageStatus(

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63656/
Test FAILed.

Test build #63657 has started for PR 14617 at commit 06cb6d9.

Test build #63657 has finished for PR 14617 at commit 06cb6d9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63657/
Test PASSed.

Merged build finished. Test PASSed.

Hi @jerryshao. I think this is a great idea and fills in an important gap in the app's UI. Going by the screenshot you posted, instead of putting both on and off heap memory in a single column, how about putting them in separate columns? For one thing, this would make them independently sortable. Also, how about splitting on- and off-heap memory in the summary table as well?
Another missing piece of the puzzle is the total executor memory that executors report back to the master. For example, in the standalone master UI each app only reports allocated heap memory per executor. This is perhaps an even bigger omission as it misrepresents the actual memory footprint of the executors, and because of that the scheduler does not compute or report memory usage per worker correctly. This breaks memory resource allocation. Is this issue something your patch addresses?

@mallman thanks a lot for your comments, I will change the UI to split into separate columns.
Yes, as you mentioned current executor memory usage tracked in Standalone Master only shows on-heap memory, it is ideal to also count into off-heap memory, and this is already considered in yarn mode, so maybe we should also do it in Standalone mode. But I think it would be better to handle it in a separate PR.

@mallman I changed the UI based on your comment, here is the new one (separate the on heap and off heap memory usage in two columns):

Test build #64406 has started for PR 14617 at commit 838840d.

Test build #64406 has finished for PR 14617 at commit 838840d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64406/
Test FAILed.

@jerryshao The UI changes look great. I have not had a chance to scrutinize the source changes. Hopefully we can get someone else to help review.

Jenkins, retest this please.

Test build #64450 has started for PR 14617 at commit 838840d.

Test build #64450 has finished for PR 14617 at commit 838840d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64450/
Test PASSed.

hi @jerryshao sorry this went unnoticed for so long, if you bring this up to date I'll keep an eye on it.
Before this change, is off-heap storage completely ignored in the UI?  Or does the UI just lump together off-heap and on-heap?  I'm wondering if most users won't care about the distinction, and it is just extra noise in the UI.
It certainly would be good to expose in the API for sure.
cc @CodingCat @ajbozarth @tgravescs as you often have good ideas for the UI.

Thanks @squito , finally get someone's attention :). I will update the code soon.
Before this change off-heap on on-heap memory are added together (https://github.com/apache/spark/pull/13920). I think it would be better to expose this details to users.

ah...somehow I missed this email.....
I like this idea, the comments here would be that if the last screenshot is the latest design, will the user have to sum up the total amount of used memory?
how about exposing the all of total/on-heap/off-heap; and specifically, in UI, the default behavior would be displaying total, and user can check on-heap/off-heap details when the cursor hover around?
just replace data-original-title in the following code snippet with the on-heap/off-heap memory details in some way (I think it is doable)

              Storage Memory

That's a good idea, thanks @CodingCat .

Test build #74884 has started for PR 14617 at commit 7094a5b.

Change the UI according to @CodingCat 's comment.

Test build #74884 has finished for PR 14617 at commit 7094a5b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74884/
Test FAILed.

Looks like the failures are real (you probably just need to regenerate the expectations for the new blacklisting tests)

while I kind of like the hover because it doesn't clutter the page, it does bring up a couple concerns:

user can't sort by them
user might not know to hover (none of the other pages that I know of show useful values like this)

The latter one is perhaps the one that is more concerning.  Unless we can highlight it or somehow draw the users attention to it they will not know its there.   But the first one is also pretty valid. If I have thousands of executors and looking for one that is out of off heap memory, its pretty difficult to hover over all of them.
I don't see any changes to the Storage UI page, do we want to add something there as well?

good points @tgravescs .  What about making them additional metrics, turned on by a checkbox, like the extra task metrics?

Thanks @tgravescs and @squito for your comments. Based on @tgravescs 's point, looks like making them as a table column is more valid.
So I will revert back to use column and combine with checkbox. I personally have no strong preference, what's your opinion @CodingCat ?

I agree with the checkpoint based solution , thanks for asking

Checkbox sounds good to me.

thanks @jerryshao .  Two other points:

We should have the same treatment in the executor summary table
as tom mentioned, there is also the storage page.  you can do that separately if you like, but might be easier to just roll it into this change.

After reading through the previous comments I agree adding checkboxes to this page is a good idea, I would even suggest that we look at making checkboxes for a few of the current columns (default to show, to keep user compatibility)> I'm not sure which would be best but I know on many apps a few columns are never filled (Disk usage, and shuffle read/write first come to mind).

Test build #75021 has started for PR 14617 at commit 87bfa78.

Hi All, thanks a lot for your comments.
Here is the UI after changed:

And storage page:

Please review, thanks a lot.

Test build #75021 has finished for PR 14617 at commit 87bfa78.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75021/
Test FAILed.

Merged build finished. Test FAILed.

Test build #75022 has started for PR 14617 at commit ced7420.

Test build #75023 has started for PR 14617 at commit 9141d9b.

Test build #75022 has finished for PR 14617 at commit ced7420.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75022/
Test FAILed.

Merged build finished. Test FAILed.

Test build #75023 has finished for PR 14617 at commit 9141d9b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75023/
Test PASSed.

Test build #75085 has started for PR 14617 at commit 9b1eeb5.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75085/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, retest this please.

Test build #75088 has started for PR 14617 at commit 9b1eeb5.

Test build #75088 has finished for PR 14617 at commit 9b1eeb5.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class RddStorageInfo(memoryUsage: Long, diskUsage: Long, level: StorageLevel)
  case class NonRddStorageInfo(var onHeapUsage: Long, var offHeapUsage: Long, var diskUsage: Long)

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75088/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75148/
Test FAILed.

Jenkins test is abruptly with signal -9.
Jenkins, retest this please.

Jenkins, retest this please.

Test build #75160 has started for PR 14617 at commit eb16e51.

Test build #75160 has finished for PR 14617 at commit eb16e51.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class RddStorageInfo(memoryUsage: Long, diskUsage: Long, level: StorageLevel)
  case class NonRddStorageInfo(var onHeapUsage: Long, var offHeapUsage: Long, var diskUsage: Long)

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75160/
Test PASSed.

Thanks @squito , I will change some event logs files to test non zero the off heap memory.

Test build #75259 has started for PR 14617 at commit 60383b9.

Test build #75259 has finished for PR 14617 at commit 60383b9.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75259/
Test FAILed.

Jenkins, retest this please.

Test build #75261 has started for PR 14617 at commit 60383b9.

Test build #75261 has finished for PR 14617 at commit 60383b9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75261/
Test PASSed.

Merged build finished. Test PASSed.

Hi @squito , would you please review the code again? Thanks a lot.

@jerryshao sorry to be very particular about this, but can you add a test case where the off heap and onheap memory used is non-zero?   It looks like you only changed the max memory.  (You should be able to do this by just running something locally in the spark-shell w/ even logs turned on that uses both, and then use that event log file).
other than that, this looks good to me.
@jsoltren @CodingCat @ajbozarth I'd appreciate another pair of eyes on this if anybody has a chance as well.

@squito , AFAIK we don't record block update events in history server. So we could not calculate the used memory from event log.

AFAIK we don't record block update events in history server, so we could not calculate the used memory from event log.

good point, sorry I had totally forgotten about.  Seems like this could be fixed w/ the info already in TaskEnd events though?  separate from this change, but would be a nice improvement.
I'd still like to see some test of non-zero values -- how about here:
https://github.com/apache/spark/blob/a8a765b3f302c078cb9519c4a17912cd38b9680c/core/src/test/scala/org/apache/spark/ui/UISeleniumSuite.scala#L111
you could add caching of an off-heap rdd too, and update that to test for non-zero sizes in the right places.

Test build #75444 has started for PR 14617 at commit b30e7d0.

Test build #75444 has finished for PR 14617 at commit b30e7d0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75444/
Test PASSed.

Test build #75539 has started for PR 14617 at commit 643552c.

Thanks @squito , thanks so much for your review, just address the comments you mentioned.

Test build #75539 has finished for PR 14617 at commit 643552c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75539/
Test PASSed.

lgtm, thanks for the updates
any more comments @tgravescs @CodingCat @ajbozarth @jsoltren?

This looks good to me. Thanks.

merged to master.  Thanks @jerryshao

btw, anybody interested in looking at getting the memory to show up in the history server as well?  this issue we were discussing earlier:

AFAIK we don't record block update events in history server, so we could not calculate the used memory from event log.

good point, sorry I had totally forgotten about. Seems like this could be fixed w/ the info already in TaskEnd events though? separate from this change, but would be a nice improvement.

Thanks @squito .
Regarding showing memory usage in history server. My major concern is that putting so many block update event into event log will significantly increase the file size and delay the replay, that's why in the current code we deliberately bypass the block update event. And IIUC in history server it is not necessary to show the change of used memory, only the final memory usage before application finished will be shown on the UI. So instead of recording and replaying all the block update events, just recording the final memory usage of each executor is enough.

yeah, we definitely don't want to start logging more events.  But it seems like this info is already available -- taskEnd.taskMetrics.updatedBlocks already has everything, doesn't it?

I see. The current code leverages SparkListenerBlockUpdated event to calculate memory usage, let me try to investigate the feasibility of using taskEnd.taskMetrics.updatedBlocks, to see if it is possible to use this instead to calculate.

@squito , by revising this code, I found there're some places which are misleading and could be improved:

All the memory usage referred here about on-heap memory and off-heap memory (used, remaining, max) are only about storage memory. But in our REST API and metrics source we don't specify this, this may mislead the user. I think this problem existed after we incorporated execution memory usage, we neglected that part of memory usage and only count in storage memory usage.

Either we should rename the field in REST API and others to specifically saying that is storage memory, or we should also count in execution memory usage.

With UnifiedMemoryManager incorporated, the maximum storage memory (on-heap and off-heap) will be changed in the runtime according to the execution memory, but here in the BlockManager's StorageStatus and other places we assume this is static. It is not accurate to reflect the current Storage Memory size. We should also handle this.

What is your opinion about it, would be grateful to hear your comments.

hi @jerryshao good points.  First, we should probably move this discussion to jira so its more visible -- feel free to open two issues for these if you want, or first discuss on<EMAIL_ADDRESS>(Sorry its my fault for starting a discussion here in the PR comments after the original change was merged ...)
On your first point, I like the idea of exposing more information on memory usage, but is there anything meaningful to report on execution memory in the rest api?  It doesn't really seem like there is.  Maybe we should rename the fields, but keep backwards compatibility in mind.
For the second point about storage memory limits -- its a good question about what it should report with the Unified Memory Manager.  I thought we'd just expose the portion of memory immune to eviction, spark.memory.storageFraction.  But perhaps I misunderstand what is going on now.
Again, probably better to have the design discussion on jiras.


Add Kas Quiz

https://kasquiz.kaskus.co.id/

Hmm… This looks like a well-done PWA, but the fact that registration/login seems to be required (and the form not being in English) makes me think maybe it doesn’t belong on this list. What do you think?


Support Export & Import Flatbuffer Foramt

For more flexibility we try to support Flatbuffer format

[ ] Define Flatbuffer schema
[ ] Export NNTrainer Model

:octocat: cibot: Thank you for posting issue #2106. The person in charge will reply soon.

It's pending issue
Close this issue and i will reopen when project resume


chore: use npm

⚡ Deploying PR Preview 918b4a286feefb6ec7100248270bce22b33b431b to surge.sh ... Build logs

🤖 By surge-preview


Weird short bright strip

A few galaxies (I have checked 400+ galaxies) have a weird bright strip across the galaxy. For example, check the center of https://portal.nersc.gov/project/cosmo/data/sga/2020/data/334/ESO108-023/ESO108-023-largegalaxy-image-grz.jpg. May I ask are they cosmic ray? It there any way to remove them?

If you mean that bright green line, yes, that's a cosmic ray.  In the centers of galaxies where there is a lot of background light, it is harder to detect and mask cosmic rays.

Thanks!

Here's the direct link for ESO108-023
https://www.legacysurvey.org/viewer?ra=334.1342&dec=-64.3893&layer=ls-dr9&zoom=14&sga
Unfortunately, this is more of a CP-reduction issue than an SGA issue, so I'm closing.

We could in principle try to tune the outlier-detection code to catch these cases where cosmics are harder to catch on top of big galaxies.


chore: add images for assets mee, metal and yum

https://axelarnetwork.atlassian.net/browse/AXE-3950

@blurrytweaks wdty?

@blurrytweaks wdty?

Idk why display isn't working, yum seems ugly, but it's okay I'll provide and tell to replace it after merged :)


microsoft/just conflicts with another project casey/just that installs a CLI with the same name just

casey/just has been using the the cli name just since 11/2017. Perhaps consider using just-cli instead to avoid the conflict?

oh interesting - do you mean npx just would cause an issue here?

Given:

casey/just is installed to /usr/local/bin/just
@microsoft/just is install and available at ./node_modules/.bin/just

@microsoft/just will have a higher priority over /usr/local/bin/just for npm run and npx just.
If a project is using a Justfile, then /usr/local/bin/just will not work.
To resolve the conflicts and have both cli coexists, users would have the following options.
"scripts": {
  "use-ms-just-option-1": "npx just task-name",
  "use-ms-just-option-2": "./node_modules/.bin/just task-name",
  "use-casey-just-option-1": "/usr/local/bin/just task-name",
  "use-casey-just-option-2": "just-alias task-name",
  "use-casey-just-option-3": "./Justfile task-name",
},

For @microsoft/just:

use npx just
./node_modules/.bin/just

For casey/just:

use absolute path
create a symlink _just -> /usr/local/bin/just
a shebang in Justfile (i.e. #!/usr/bin/env just --justfile)

@microsoft/just could provide a known issue and/or a warning if another cli has been detected in $PATH.

@microsoft/just could provide a known issue and/or a warning if another cli has been detected in $PATH.

Oh that's an interesting feature... I'm going to rename this issue so to reflect this idea.


CSS Fixes

Just fixing up a couple loose strings we noticed after merge:
Takes the date out of the ribbon of the desktop version of the homepage so the date no longer appears twice:

Adjusts the layout of the sponsors so they are not just one long list:

thanks @imnikkilee.


[main] [snippet update] add functions to set configs of snippet

Will it make things more clear if we remove const webSnippetCs = "##replaceConnStringSnippet##"? Currently, the only difference between webSnippet and webSnippetCs is:
webSnippet  ends with    connectionString: "InstrumentationKey=INSTRUMENTATION_KEY
webSnippetcs ends with  connectionString: "YOUR_CONNECTION_STRING"\r\n
btw, maybe webSnippet  should end with instrumentationKey: "YOUR_INSTRUMENTATION_KEY, this maybe a mistake when I wrote the rush json months before


dist/vue3-treeview.esm.js is not available in the dist folder

Tried this library in a vue3 project with vite and I got the following error.
error: [plugin: vite:dep-scan] Failed to resolve entry for package "vue3-treeview". The package may have incorrect main/module/exports specified in its package.json: Failed to resolve entry for package "vue3-treeview". The package may have incorrect main/module/exports specified in its package.json.

Upon further investigation, it seems that vue3-treeview.esm.js is missing from the dist folder.

Hey, do you have specific reproduction scenario ?

Sure, made a little example here https://codesandbox.io/s/hello-vite-forked-olm9r?file=/src/App.vue

I changed vue-cli build lib (that does not create esm file) by "vite" (with rollup under the hood). And published a 0.3.4 version. Can you try again ?
I didn't found a way to rename vue extracted style with vite so if you want to use it you can
import "vue3-treeview/dist/style.css";

Yes, is working! Thank you very much!
After a couple of hours of searching for a decent tree view library in vue3, I finally found yours. Simple and stable, is exactly what I wanted!

Thank you ! I updated the documentation to put some codesandbox instead of direct components because vitepress is still not completely finished. Don't hesitate to suggest improvements and put a star if you like it :)


Fixed crash happening due to schema does not exist during GRANT operation

Previously, If end user tries to grant select on table using schema qualified name and if schema does not exist then server will crash due to segmentation fault issue. This commit aims to fix that issue by appropriately checking existence of the schema.
Task: BABEL-5078
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.009%) to 73.036%

Files with Coverage Reduction
New Missed Lines
%

contrib/babelfishpg_tds/src/backend/tds/tdscomm.c
5
73.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.009%

Covered Lines:
42277

Relevant Lines:
57885

💛  - Coveralls


Docs: networking: Remove reference to missing example

Seems it was intended to put an example after this comment. Not there yet so removing and referencing the followup ticket: https://github.com/appc/cni/issues/56.

LGTM, guess this got lost in the differing proposals. Thanks.


mdt-table respecting PaletteColors for dark

It doesn't look like mdt-table respects text color when using a .dark() palette option.  The Entire table background color does update with the primary background palette color, however if one uses dark() it appears the text stays black.
Is there a way to change colors of both background rows, and text explicitly? (either Column Headers or Body Content Text)
I would be happy to attach a code pen if example needed.
Cheers

hi @bengitdev ,
You right, this is a forgotten enhancement I wanted to do.
What you can do is to apply some css rules to the texts to change it. Just remember that provide a stronger css rule than the table has and it will change it.
I'll fix it but too much things to do at the moment so I can't guarantee any timeframe for the fix.


correctly fail on errors from rebuild thread

This test didn't fail when the rebuild call in the thread raised an exception; this PR fixes that.
Clarity question: I've written a function that generically takes a function here. Since it's only used the once, would it be clearer to write:
def rebuild_and_register_exc_info_on_exception():
    """
    Closes over self to catch any exceptions raised by func and
    register them at self.thread_exc_info
    Based on http://stackoverflow.com/a/1854263
    """
    try:
        rebuild()  # closed over
    except Exception:
        import sys
        self.thread_exc_info = sys.exc_info()

cmd1 = Thread(target=rebuild_and_register_exc_info_on_exception)

+1 if it works.

Ditto what Russ said

Cool, thanks, folks. Merging.


fix: use fdkId as key to make sure all events of the same resource ar…

Fixes https://github.com/Informasjonsforvaltning/fdk-mqa-dcat-validator/issues/46

trenger workflow-fiks, og image-navn i deploy/base/kustomization.yaml


Update licensed cache

I'm not sure whether it makes sense for us to keep using licensed, given that the updater is fully open source.
If yes, this PR would still need to add licensed to a CI step, probably the recently added dependency-review.yml would be a good place?
If no, this PR just needs to be amended to delete all the things.

There's some internal discussion related to this, I'll cross link it to you privately so you have context.

I understand that removing the license cache needs some discussion, but fixing the existing license check should be a no-brainer, so setting this as ready!


[date] Fix uninitialized values

Describe the pull request

What does your PR fix?
Fixes #23637

Add patch to apply this change https://github.com/HowardHinnant/date/commit/a74eb1a540263eb883967e4f2dd8ed82f2e863d4
fix the error:
SUMMARY: MemorySanitizer: use-of-uninitialized-value (/home/travis/build/acgetchell/CDT-plusplus/build/src/cdt+0x642ce1) in date::get_tzdb_list()

@acgetchell At present, the upstream has accepted this change. We prefer to submit an issue upstream to release a new version to update this change. We will not merge this PR, but it can be used temporarily to avoid similar problems.

The modified content does not destroy the code logic, only initialized some functions, there is no need to wait for a new version to be released, this pull request can be accepted.

Thanks!


fix: update chart on release

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


Does autoload work?

So I loaded some date in a collection and now I am trying to simply sort the data
//initialize to null if there is no existing file
let mycollection = null;

//Create database
let db = new loki('mycollection.json',
    {
        autoload: true,
        autoloadCallback: loadHandler,
        autosave: true,
        autosaveInterval: 10000
    });

//load Database from file
function loadHandler() {
    // if database did not exist it will be empty so I will intitialize here
    mycollection = db.getCollection('mycollection');
    if (mycollection === null) {
        mycollection = db.addCollection('mycollection', {
            unique: ['id']
        });
    }
}

function sortByDate(){
	let latestFive = mycollection.chain()
        .simplesort('timestamp', true)
        .limit(5)
        .data();

    console.log(latestFive);
}

sortByDate();

I thought that the autoload was guaranteed to load the database, but if I call this function it has no data loaded the value is still null.
TypeError: Cannot read property 'chain' of null
Do I manually have to load the database whenever I want to use it? What is the point of autoload... am I missing something here?
The only way to use that I can see is to load the database using
loadDatabase(options, sortByDate)

Well the loadHandler callback is invoked asynchronously so you need to wait until that gets invoked before querying.  Your sortByDate will be called before loadHandler since you have not yielded thread and possibly also because async io has not yet completed.
Maybe for logical example you could rename sortByDate function to 'init' and invoke that at end of loadHandler()

Thank you for your reply. I see, I am used to having the database being available as soon as i try to do something with it, i have changed the code to run after the database is loaded only :)
Looks like a great improvement for 2.0.0 is there a release date for it? what promises library are you going to use?

No release date for 2.0.0 but soon that branch will become our active branch and existing 1.4 branch will be maintenance.  Probably will start shifting around January not sure when 'release' will be.  By default 2.0.0 will rely on your environment supporting es6 promises as loki is a 'dependency free runtime'... not sure how easily a babelified path can be provided in toolchain... maybe others have ideas on this as this would be only a 'dev dependency'?

Is there a way to store the data in a sorted way by default? Or do I have to sort and persist the sorted array?
Thanks!

sorry i missed this, usually binary indices or dynamic view alleviates need to store data sorted


A remote journal implementation

We can wrap a gRPC Journal similar to the gRPC Storage. I think it is not very hard, so maybe someone can give it a try :)

That sounds interesting. I'd like to give it a try this week. @huachaohuang could you assign it to me?

The first problem I met is that we use a generic Timestap type parameter. But I don't have an idea how to pass it through protobuf. I guess protobuf isn't aware of generic and I possibly need to do some unchecked casting.

The first problem I met is that we use a generic Timestap type parameter. But I don't have an idea how to pass it through protobuf. I guess protobuf isn't aware of generic and I possibly need to do some serde or unchecked casting.

I think we need to add some required methods in the trait sooner or later. For example, we can add methods to encode/decode Timestamp so that we can pass it as bytes.

I may start with a u64 only implementation first.

This also makes sense to me as a start.


Exclude scenarios using the NOT operator '~' for tags

Hi! I needed the ability to be able to commit incomplete scenarios but mark them so that the CI-system would not run them. I simply added the not-operator '~' that can specify tags that must not be present on scenarios if they are to run. The '~@TAG'-style is older gherkin, while the newer style uses "not @TAG" instead, but I did not want to include a space lest it interferes with the CLI.
My first pull request btw :)

Thank you for your contribution, @JakeTheSnake!
I added you as a contributor to the package.json and published v1.4.0. :)


manual phase correction is wrong

The manual phase correction updates datum1D.data. It means that if you want to reapply the filter (it's normal) you need to apply it with the difference between current value and previous one to get the correct behavior. the new inputRange to update ph0 and ph1 values is nicer but the filter is applying with the current value
example
first, update ph0 to
ph0=90 , ph1=0  // the data rotate 90 degrees
second update ph0
ph0=91 , ph1=0 // the result is data rotated 181 degrees, we need to apply a rotation of 1 degree (the difference) to get data rotated 91 degrees

And what about ph1 ?

The same, I'm working to have a prototype working that i can show you


Upgrade to Java 8

Upgrade the version Java versions of the agents to Java 8.
Integration tests succeeded: https://beta-jenkins.mcc.schubergphilis.com/job/cosmic/job/0002-tracking-repo-branch-build/154/

Ping @miguelaferreira @bheuvel @remibergsma
Integration tests succeeded: https://beta-jenkins.mcc.schubergphilis.com/job/cosmic/job/0002-tracking-repo-branch-build/157/
PR can be merged.


New operator: BatchWhile

I was really surprised that MoreLINQ still has no method with such functionality: break the sequence into subsequences by predicate, so I've made it myself :)
Example
Let's recall some commonplace problem from school: find the maximum length of a subsequence of repeating numbers in an array. Using BatchWhile this can be written like this:
int[] array = new[] { 1, 2, 3, 3, 3, 3, 4, 5, 5, 5 };

int maxLength = array.BatchWhile((currentElement, bucket) =>
{
    // Every bucket has at least 1 element
    int previousElement = bucket[bucket.Count - 1];

    // If predicate will return `false`, a new subsequence begins
    return currentElement == previousElement;
},
// Transform each bucket into `int` representing its length
bucket => bucket.Count)
// No comments :)
.Max();

// maxLength == 4

Or easier:
int maxLength = array.BatchWhile((x, bucket) => x == bucket[0], bucket => bucket.Count).Max();

Overloads
IEnumerable<TResult> BatchWhile<TSource, TElement, TResult>(this IEnumerable<TSource> source, Func<TSource, TElement> elementSelector, Func<TElement, IBucket<TElement>, bool> predicate, Func<IBucket<TElement>, TResult> resultSelector);

IEnumerable<TResult> BatchWhile<TSource, TResult>(this IEnumerable<TSource> source, Func<TSource, IBucket<TSource>, bool> predicate, Func<IBucket<TSource>, TResult> resultSelector);

IEnumerable<IBucket<TElement>> BatchWhile<TSource, TElement>(this IEnumerable<TSource> source, Func<TSource, TElement> elementSelector, Func<TElement, IBucket<TElement>, bool> predicate);

IEnumerable<IBucket<TSource>> BatchWhile<TSource>(this IEnumerable<TSource> source, Func<TSource, IBucket<TSource>, bool> predicate);

codecov doesn’t like code duplication for IReadOnlyList<T> case, but it was forced, as you know that IList<T> and IReadOnlyList<T> implement indexing in their own way, so... ¯\(ツ)/¯

break the sequence into subsequences by predicate

Segment operator already does it @Kir-Antipov .

@Kir-Antipov Thanks for your contribution, but I wish you had opened an issue to check before submitting a PR with a full implementation. As @leandromoh point out already, we have this except it's called Segment. I just hope you didn't end up wasting too much time.
If you want to submit a working example for Segment then that would be much welcome & appreciated.

@atifaziz are you sure that BatchWhile is totally duplicate what Segment can do? From what I can see BatchWhile is wider.

@atifaziz are you sure that BatchWhile is totally duplicate what Segment can do? From what I can see BatchWhile is wider.

I think so based on the example in the opening comment of this PR, though I'll admit I didn't look at the code, and the following description (bearing in mind it's missing details that are open for interpretation):

break the sequence into subsequences by predicate, so I've made it myself :)

The problem the example intends to solve can be rewritten with Segment:
int[] array = new[] { 1, 2, 3, 3, 3, 3, 4, 5, 5, 5 };
int maxLength = array.Segment((curr, prev, _) => curr != prev)
                     .Select(g => g.Count())
                     .Max();

as well as, and more simply, with GroupAdjacent:
int[] array = new[] { 1, 2, 3, 3, 3, 3, 4, 5, 5, 5 };
int maxLength = array.GroupAdjacent(x => x)
                     .Select(g => g.Count())
                     .Max();

Segment operator already does it @Kir-Antipov .

@leandromoh, @atifaziz, nah, not really. As @FoggyFinder noted, BatchWhile operator can do much more than Segment.
How can you express the desire "break a sequence into subsequences so that the sum of the elements of each of them does not exceed 42" with Segment? Don't even try, 'cause it's impossible :)
In the same time:
array.BatchWhile((current, bucket) => bucket.Sum() + current <= 42)

Btw, I want to note that the name Segment is extremely unobvious. A couple of times my needs could be covered even by the Segment, but after the Batch I no longer expected to see such operator. Why? Take/TakeWhile - first takes a number as an input, second - a predicate. Batch/BatchWhile - the analogy is preserved, Batch/Segment - nope.
If we continue the topic of LINQ analogies, Segment has one more drawback: LINQ methods that return IEnumerable<IEnumerable<T>> (e.g., GroupBy) make it possible to project inner IEnumerable<T> into new form so that you don't have to call an enumeration in the second round with Select. BatchWhile obeys this rule, Segment does not.
Imho, BatchWhile should replace Segment at all, since one is a superset of the other. But I clearly understand that something like this will never happen here, so let's just keep two operators together.
P.S. - I apologize for answering so long. A little slack because of the weird times we live in :)

How can you express the desire "break a sequence into subsequences so that the sum of the elements of each of them does not exceed 42" with Segment? Don't even try, 'cause it's impossible :)

No one is saying you have to do everything with Segment. That was just to show that your initial example could be solved with it or even more simply with GroupAdjacent. Now your new problem might be impossible for Segment but who says you can't combine existing operations cleverly? In fact, as we've seen, GroupAdjacent is simpler and I'd even say more powerful for your cases so far. Here's a solution using Scan together with GroupAdjacent:
// break a sequence into subsequences so that the sum
// of the elements of each of them does not exceed 42

var result =
    MoreEnumerable
        .Random(10)
        .Take(100)
        .Select(x => new { N = x, Sum = x, Batch = 0 })          // seed all in same batch
        .Scan((s, e) => s.Sum + e.N is {} sum && sum <= 42       // running sum <= 42?
                      ? new { e.N, Sum = sum, s.Batch }          // yes => same batch
                      : new { e.N, e.Sum, Batch = s.Batch + 1 }) // no  => new batch (+1)
        .GroupAdjacent(e => e.Batch)                             // group by batch
        .Select(g => $"({g.Select(m => m.N).ToDelimitedString(", ")}) = {g.Last().Sum}")
        .ToDelimitedString(Environment.NewLine);

Console.WriteLine(result);

Here is the output from one run:
41 = 9 + 0 + 8 + 3 + 9 + 5 + 7
42 = 3 + 1 + 4 + 6 + 6 + 3 + 2 + 4 + 7 + 6
36 = 4 + 8 + 4 + 9 + 7 + 3 + 1 + 0
39 = 9 + 9 + 4 + 3 + 6 + 0 + 0 + 3 + 5
39 = 5 + 5 + 6 + 2 + 6 + 3 + 1 + 8 + 3
40 = 6 + 5 + 7 + 8 + 3 + 1 + 3 + 6 + 1
41 = 3 + 8 + 2 + 1 + 6 + 7 + 3 + 0 + 5 + 2 + 0 + 4
37 = 3 + 6 + 2 + 8 + 7 + 8 + 0 + 3
41 = 8 + 4 + 2 + 1 + 4 + 6 + 6 + 2 + 0 + 5 + 3
42 = 9 + 0 + 9 + 6 + 5 + 2 + 3 + 8
40 = 9 + 9 + 6 + 8 + 8
26 = 7 + 9 + 2 + 8

and here is another:
38 = 4 + 9 + 6 + 2 + 3 + 2 + 3 + 9
42 = 6 + 1 + 7 + 0 + 1 + 0 + 9 + 9 + 9 + 0
38 = 3 + 4 + 9 + 7 + 6 + 3 + 0 + 1 + 4 + 1
35 = 6 + 3 + 3 + 4 + 3 + 7 + 8 + 1
36 = 8 + 1 + 6 + 8 + 6 + 7
38 = 9 + 0 + 6 + 8 + 7 + 8
36 = 6 + 8 + 9 + 7 + 6
42 = 8 + 9 + 7 + 2 + 3 + 1 + 2 + 7 + 1 + 2
40 = 2 + 8 + 2 + 6 + 4 + 2 + 8 + 0 + 8
40 = 4 + 5 + 7 + 2 + 8 + 6 + 7 + 1
40 = 3 + 6 + 5 + 8 + 4 + 6 + 8
39 = 5 + 8 + 0 + 8 + 5 + 9 + 3 + 1
21 = 5 + 4 + 7 + 2 + 3

So Scan is used to keep a running total/sum and as soon as it is about to exceed 42, the accumulator is reset with a new batch number. Next GroupAdjacent is used to simply group elements by their batch number. A nice thing about this is that both, Scan and GroupAdjacent, stream their results so you can work with very large data efficiently and observe the results as soon as they become available. In the case of GroupAdjacent, the groups (as opposed to the elements) are streamed; that is, a group is yielded as soon as it ends and before the next one is starts. At any one time, only a single group is committed to memory and so need to only account for the largest group in terms of space complexity.
Note that batches don't need to be numbered uniquely since GroupAdjacent will switch groups whenever the key changes. You can rewrite the above with a Boolean key that you simply toggle (!) whenever the sum of elements is about to exceed the threshold (below, thinking in terms of two bins):
var result =
    MoreEnumerable
        .Random(10)
        .Take(100)
        .Select(x => new { N = x, Sum = x, Bin = false })   // seed all in same bin
        .Scan((s, e) => s.Sum + e.N is {} sum && sum <= 42  // running sum <= 42?
                      ? new { e.N, Sum = sum, s.Bin }       // yes => same bin
                      : new { e.N, e.Sum, Bin = !s.Bin })   // no  => "switch" bin
        .GroupAdjacent(e => e.Bin)                          // group by bin
        .Select(g => $"{g.Last().Sum} = {g.Select(m => m.N).ToDelimitedString(" + ")}")
        .ToDelimitedString(Environment.NewLine);

Btw, I want to note that the name Segment is extremely unobvious. A couple of times my needs could be covered even by the Segment, but after the Batch I no longer expected to see such operator. Why? Take/TakeWhile - first takes a number as an input, second - a predicate. Batch/BatchWhile - the analogy is preserved, Batch/Segment - nope.

Yeah, naming is hard 😉  and we've had naming debates in the past that go as early as issue #10 (some 11 years ago) in the project's long history. I am not quite sure how to solve the problem of allowing someone to quickly navigate the 100+ extensions and get familiar with them. Sometimes, people come with a specific idea and try to find a matching name that's already in their head, but often a solution can be found by combining two or more operators cleverly.

But I clearly understand that something like this will never happen here

I hope you are not discouraged from contributing in the future. Part of the review process is to challenge things that have often lead to more generalisations or dead ends (which has happened to me several times). For what it's worth, I checked out the code you submitted and it's really high on the quality bar and I wish every contribution would begin like yours. Right now, it seems redundant with existing operators, which is something important to consider for maintenance.

P.S. - I apologize for answering so long. A little slack because of the weird times we live in :)

It's understandable and there is no need to apologize now or any another time since this is open source. It's generally volunteered work and time. Remember, I took also almost 2 weeks to post my first reply. 😉


BelongsToManyMultiSelect displays already attached options in dropdown on initial load

Package
filament/filament
Package Version
v2.12.2
Laravel Version
v9.11.0
Livewire Version
No response
PHP Version
PHP 8.1.4
Bug description
When using the BelongsToManyMultiSelect component, already attached records are still displayed in the dropdown when the form is loaded initially.
With no records attached:

Attaching some records (selected options is removed from dropdown):

After having saved the new selection and then editing the list again:

I would expect the dropdown to exclude the already attached records to shown in the dropdown.
This is how I configure the field:
// File: RolesRelationsManager.php
public static function form(Form $form): Form
{
    return $form
        ->schema([
            TextInput::make('label')->required(),
            TextInput::make('name')->required(),
            BelongsToManyMultiSelect::make('role_permissions')
                ->placeholder(__('Select permission(s) to be assigned to role'))
                ->preload()
                ->relationship(
                    'permissions',
                    'label',
                    fn (Builder $query, $livewire) => $query->whereBelongsTo($livewire->ownerRecord)
                )->columnSpan(2),
        ]);
}

This might be related to #1966.
Steps to reproduce
Use a BelongsToManyMultiSelect to edit a belongsToMany relationship.
Relevant log output
No response

Not related to #1966, btw.

Please test #2377, the documentation for this feature is in the PR.

Any update ???

Is this still an issue? If so, could you set up a repository that allows us to reproduce the problem easily, @henninghorn?

Yeah this is a confirmed issue

I have a similar problem, only using a normal select (not multiselect).
In my case it happens when I use options with searchable (and I don't use either getSearchResultsUsing or getOptionLabelUsing).
It basically duplicates the results:

Forms\Components\Select::make('technician_id')->options(User::role('technician')->pluck('name', 'id'))->searchable()->label('Técnico responsable'),
When I can, I will share a repository with the example, as zepfietje says.

Should be fixed by #4195, please verify in the next release

Lately something seems to not be working right with MultiSelect and Many To Many relationship.

When i write something, all options already chosen with the same name disappear from the MultiSelect field and are transferred in the list below the field. However, if I save, all the original options present are still saved.

Can you share some code please @RX-77IT ?

It's just a simple many to many relationship between Product and Color models. The only thing that is somewhat different is the intermediate table name that is not in alphabetical order.
class ProductResource extends Resource
{
    ...
    public static function form(Form $form): Form
    {
        return $form
            ->schema([
                ...
                Forms\Components\MultiSelect::make('colors')
                    ->label('Colori associati al prodotto')
                    ->relationship('colors', 'name'),
                ...
            ]);
    }
    ...
}

class Product extends Model
{
    ...
    public function colors()
    {
        return $this->belongsToMany(Color::class, 'product_color');
    }
    ...
}

class Color extends Model
{
    ...
    public function products()
    {
        return $this->belongsToMany(Product::class, 'product_color');
    }
    ...
}

Feel free to ask anything else if you need.

Reverted

following

I've encountered this too, following.

Fixed in v3


Smarter new scene dialog

Godot version:
11d7738
Issue description:
The new scene dialog is a good addition, but in my opinion it can be improved in the following ways:

Add also the "Node" as one of the defaults
Add a button for instanced scenes (I think it is common for people to click on "new scene" when they actually want an "inherited scene"
Add a list of the last used nodes

The list structure should be like this in my opinion:

Custom Node ...
Inherited scene ...

2D
3D
User Interface
General Node

[list of the last 3-4 used nodes]

I believe this layout would improve even more the scene creation, as I noticed for myself that those are the nodes that I am most likely to search for when creating a new scene.
Separators would be a nice to have inside of the list

Personally I'd like to see the new scene dialog also moved into the middle of the stage where the 2D/3D views are located, instead of under the Scene nodes list. Along with a list of recent scenes and thumbnails for quickly reopening something you just had open.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Create a workflow to automatically update pre-commit hooks

Create a workflow that runs daily at midnight, or can be triggered manually, and which does the following:

Checks out the repository.
Sets up Python.
Installs pre-commit with Pip.
Updates all pre-commit hooks with pre-commit autoupdate.
Repeatedly runs the pre-commit hooks on all files until there are no changes.
Commits any changes to a branch and creates a pull request.
Creates a Towncrier .misc news fragment for the pull request.

This time, I think it works...  See #34 & #35.

This looks good - the one caveat I think is that if e.g. a flake8 check fails, then I think pre-commit will never succeed in running? This is probably okay here but will cause issues with larger repos like dials. I'm not sure what a good solution to this is.
Maybe it doesn't need to be run repeatedly? Just run once and commit any changes? Would need to somehow verify that black and isort didn't crash halfway through....


Need to disable syntax module

The syntax module of Quill in canvas is enabled by default, which may lead to a problem.
Code block would be stored in database as format:

function...

In that case, at frontend, I can only use highlight.js, other syntax packages won't work, like prism.js.
FIY, the code format of Quill is
...
, which don't include sub  child and language class name, could we support it?

It looks like this answer here could provide some help if adding this to the blot itself.
As far as building support for other syntax highlighters like Prism, I'd need to dig into it and see if there's a viable solution.
Feel free to submit a PR with any proposed changes! If not, I'll see what I can do for future releases.

Sounds greet, I'll look into it.

Closing this for now. If you submit a PR I'd be more than happy to take a look.


Use cachetools to cache already evaluated models during lookahead

https://stackoverflow.com/questions/30730983/make-lru-cache-ignore-some-of-the-function-arguments
when extracting the actual model evaluation from the _optimize function we need do pass the training data and so on to this cached function. however dont want to check for equality in the trainingdata.

A better way would be to use this library to ignore the the irelevant parameters of the cached function
and always delete the cache at the beginning of the fit method.

maybe its even more elegant to define that function as a closure function within the fit function, therefore we can be sure that the cache is clean. the problem here is that this might interfere with joblib

            @lru_cache
            def _loss_at_k(f, s, t):
                clf = _ClassifierAtK(
                    list(f), list(s), list(t), **self.stage_clf_params_ex
                ).fit(X, y)
                return (
                    self.stage_loss(
                        clf,
                        X,
                        y,
                    ),
                    clf.thresholds[-1],
                )

that function is actually simmilar to the already existing, but not cached function fit and store

That interferes with threshold optimization, because even if we evaluate models with the same score-feature-pairs, those pairs where added in different order, resulting in another prefix of thresholds already partially fitted. this could only be done if the threshold calculation would be changed significantly


Add additional reviewers in EMEA and US

Will probably tweak in the future

/approve

/retest

/override ci/prow/e2e-aws

/retest

/override ci/prow/e2e-aws


Hard-coded openread causing errors

I am using Debian 8 Jessie (stable) and Python 3.4 installed from the stable repos.
Here is what the installation looked like:
tom@tomspc ~ $ sudo pip3 install teafiles
[sudo] password for tom:
Downloading/unpacking teafiles
  Downloading teafiles-0.71%20dev.zip (44kB): 44kB downloaded
  Running setup.py (path:/tmp/pip-build-x9ggs07g/teafiles/setup.py) egg_info for package teafiles

Installing collected packages: teafiles
  Running setup.py install for teafiles
      File "/usr/local/lib/python3.4/dist-packages/teafiles/teafile.py", line 1310
        TeaFile.openread('C:\Users\hase\AppData\Local\DiscreteLogics\TeaShell\Dow\AA.day.tea')
                            ^
    SyntaxError: (unicode error) 'unicodeescape' codec can't decode bytes in position 2-3: truncated \UXXXXXXXX escape

Successfully installed teafiles
Cleaning up...

When running this simple test program right out of the docs:
from teafiles import *

tf = TeaFile.create("acme.tea", "Time Price Volume", "qdq", "Acme at NYSE", {"decimals": 2, "url": "acme.com"})
tf.write(DateTime(2011, 3, 4, 9, 0), 45.11, 4500)
tf.write(DateTime(2011, 3, 4, 10, 0), 46.33, 1100)
tf.close()

I get this result:
/usr/bin/python3.4 /home/tom/Desktop/teafiles/test.py
Traceback (most recent call last):
  File "/home/tom/Desktop/teafiles/test.py", line 31, in <module>
    from teafiles import TeaFile
  File "/usr/local/lib/python3.4/dist-packages/teafiles/__init__.py", line 16, in <module>
    from teafiles.teafile import *
  File "/usr/local/lib/python3.4/dist-packages/teafiles/teafile.py", line 1310
    TeaFile.openread('C:\Users\hase\AppData\Local\DiscreteLogics\TeaShell\Dow\AA.day.tea')
                        ^
SyntaxError: (unicode error) 'unicodeescape' codec can't decode bytes in position 2-3: truncated \UXXXXXXXX escape

A Windows user named hase seems to have hard-coded some data opening operation on line 1310 of teafile.py.

please simply use the sources from here to avoid any install issues.

Will do.
Thanks for the really neat project! :)

thank you!


refactor: remove extra attrs from combobox wrapper

refactor: remove extra attrs from combobox wrapper

:hammer_and_wrench: Type Of Change

[ ] Fix
[ ] Feature
[x] Refactoring
[ ] Documentation

:book: Description
The vue2 version of this component was refactored a while ago and these attrs were moved onto the input. For the vue3 version these were not removed from the wrapper, so it was having duplicated attributes. This change unifies the vue2 and vue3 versions and after this change the code is exactly the same for both versions.

Here's how VO currently reads this. Notice the menu box is duplicated

And here's how it's after the fix. I assume it doesn't "highlight" the entire combobox anymore since there's no role on it

Yeah that is definitely duplicating what's in the input. Not what it should be doing.


Prepare release 0.1.1-RC1

Summary
Preparing release 0.1.1-RC1
Background, Problem or Goal of the patch
N/A.
Design of the fix, or a new feature
N/A.
Related Issue, Pull Request or Code
N/A.
Wanted reviewer
N/A.

BUILD SUCCESS.


Fix: Linked footer navbar links to matching sections

Description
Navbar links in the footer section do not link to their matching section.
I have fixed it by editing href attributes of them and linking them to their matching section.
Fixes:  #21

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] I have made this from my own
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Thanks for contributing here.


Undefined is not an object(evaluating react3.default.proptypes.string), does not work with latest<EMAIL_ADDRESS>

The library is not updated to suit with latest<EMAIL_ADDRESS>

@yogendrakarekar if you install from 2a4a95784426e55cc16e7acfd38c683b7d87f233, it'll work.
@tlenclos do you plan on releasing a version bump to publish the fix in npm?

Hi,
Well, are you asking me whether I wish make corrections and release a new
version of this plugin to suit up with react<EMAIL_ADDRESS>well I don't
have any such plans.
But would love to help you out, take up a task, that you suggest, and do it.
Thanks & Regards
Yogendra U. Karekar
Mobile:-9769856284
<EMAIL_ADDRESS>On Fri, Oct 27, 2017 at 1:50 PM, Nick DiRienzo<EMAIL_ADDRESS>wrote:

@yogendrakarekar https://github.com/yogendrakarekar if you install from
2a4a957
https://github.com/tlenclos/react-native-audio-streaming/commit/2a4a95784426e55cc16e7acfd38c683b7d87f233,
it'll work.
@tlenclos https://github.com/tlenclos do you plan on releasing a
version bump to publish the fix in npm?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tlenclos/react-native-audio-streaming/issues/118#issuecomment-339905908,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMhrFZvAC7G7qwayq77aXGrcuMEd46VEks5swZJIgaJpZM4PwJZk
.

@yogendrakarekar no I wasn't suggesting that. There's a fix already in master, so you can have your react-native-audio-streaming dependency point to the SHA (or #master). Then it should work for you.
I was wondering if the maintainer had plans to publish a new version on npm which includes that fix since it's been in the repository for a while now.

Please bump the npm version

@nickdirienzo what's the right way to point it master. I am currently just doing npm i -S https://github.com/tlenclos/react-native-audio-streaming#master. Is there another way to do it?

@TheeOMM in your package.json, try "react-native-autogrow-textinput": "git+https://github.com/wix/react-native-autogrow-textinput.git"

@TheeOMM yes, what @BuffMcBigHuge said but replace react-native-autogrow-textinput with react-native-audio-streaming. Or you can do npm install tlenclos/react-native-audio-streaming. Hope this helps!

@nickdirienzo Ah! Thanks for pointing that out! I was the autogrow package had the same issue!

Please ... am confuse how to install from #master. Can someone do step by step instruction. Thanks for your time.

Finally i was able to install but app crashes before launch. I think there is an issue using it with Wix React Native Navigation. Can someone help. Thanks

import PropTypes from 'prop-types';
Player.propTypes = {
url: PropTypes.string.isRequired
};


Admin Lite UI: Program Status column and associated filter don't reflect new Active/Inactive binary

Affects: Programs page in the Admin Lite UI.
Related issue: #1450
Issue
We have changed the Program setup form so that admins no longer choose a GAIT status upon setup (e.g. Funded, Completed), and new programs are instead designated Active until an admin chooses to switch that to Inactive. But when I view the programs list in the Admin Lite UI, I see that the Status column contains entries from the old Django manual admin, like "funded", "Completed", "old", etc. The filter option to the left also lists "Active"/"Closed" as the options.

Acceptance criteria

[ ] 1. The program Status column only lists "Active" or "Inactive" as the status for each program. The migration rule should be that any variation on "Funded" (regardless of capitalization) should be Active, and all other statuses converted to "Inactive".

[ ] 2. The Status filter options should be Active/Inactive, and should appropriately filter the Programs list.

Housekeeping

[ ]  Unit test/s written wherever applicable
[ ]  i18n tags included wherever applicable
[ ]  Functionality is accessible to people using screen readers

QA - Verified

Multiple pages of programs were filtered, and status was reflected by Active / Inactive:

Filtering through inactive programs effectively:


[Question] ThumbnailCard button does not work as expected in Microsoft Teams but works in Bot Emulator

System Information (Required)

SDK Language:  .NET/C#
SDK Version: 3.8.1
Development Environment: AZURE_APP

Issue Description
I am currently using a ThumbnailCard which has a button that leads to a website url for authentication purposes. The ThumbnailCard displays as expected in Microsoft Teams chat but clicking the button does nothing. The button does however work in Bot Emulator as expected.
Code Example
ThumbnailCard plCard = new ThumbnailCard()
 {
       Title = "Please login in order to complete creation of support request:",
       Subtitle = " ",
       Images = new List<CardImage>(),
       Buttons = GetCardActions(url, "signin")
 };
plAttachment = plCard.ToAttachment();

IMessageActivity response = context.MakeMessage();
response.Type = "message";

response.Attachments = new List<Attachment>();
response.Attachments.Add(plAttachment);

context.PostAsync(response);

Issue has been fixed by using "openUrl":
Updated Code
Buttons = new List<CardAction> { new CardAction(ActionTypes.OpenUrl, "Get Started", value: url) }


feat: add output engine_version to force provider order

Description
I have added the engine_version output
Motivation and Context
context; https://github.com/terraform-providers/terraform-provider-postgresql/issues/2#issuecomment-567887609

This is a helper to the postgres provider, to use its mechanics so there is a way you can instantiate your Aurora postgres instance and use the postgres provider in one run/plan.

Breaking Changes

No Breaking changes.
How Has This Been Tested?

I have tested it in my code base, plans now succeed even though the RDS has not been created yet.

v2.28.0 has been just released.


Get errors trying to install

nim version is 1.2.0
± |master ✓| → nimble install
Error: Could not read package info file in /home/darren/NWN/nim/nasher/nasher.nimble;
...   Reading as ini file failed with:
...     Invalid section: .
...   Evaluating as NimScript file failed with:
...     [Package]
... version: """0.11.0"""
... author: """Michael A. Sinclair"""
... description: """A build tool for Neverwinter Nights projects"""
... license: """MIT"""
... srcdir: """src"""
... backend: """c"""
... bin: "nasher"
... [Deps]
... requires: "nim >= 1.0.2, neverwinter >= 1.2.8, glob >= 0.9.0"
... /home/darren/NWN/nim/nasher/nasher_11612.nims(5, 22) Warning: imported and not used: 'strutils' [UnusedImport]
... printPkgInfo() failed.

I can't reproduce this error on my machine. Can you tell me what version of nimble you are using?

± |master ✓| → nimble install
Error: Could not read package info file in /home/darren/NWN/nim/nasher/nasher.nimble;
...   Reading as ini file failed with:
...     Invalid section: .
...   Evaluating as NimScript file failed with:
...     [Package]
... version: """0.11.0"""
... author: """Michael A. Sinclair"""
... description: """A build tool for Neverwinter Nights projects"""
... license: """MIT"""
... srcdir: """src"""
... backend: """c"""
... bin: "nasher"
... [Deps]
... requires: "nim >= 1.0.2, neverwinter >= 1.2.8, glob >= 0.9.0"
... /home/darren/NWN/nim/nasher/nasher_28979.nims(5, 22) Warning: imported and not used: 'strutils' [UnusedImport]
... printPkgInfo() failed.
2020-04-15 12:45:54 ⌚  odyssey in ~/NWN/nim/nasher
± |master ✓| → nimble --version
nimble v0.11.0 compiled at 2020-04-03 12:49:03
git hash: couldn't determine git hash

Sorry, could you also try installing with --verbose to get the extra error information?

± |master ✓| → nimble install --verbose
Error: Could not read package info file in /home/darren/NWN/nim/nasher/nasher.nimble;
...   Reading as ini file failed with:
...     Invalid section: .
...   Evaluating as NimScript file failed with:
...     [Package]
... version: """0.11.0"""
... author: """Michael A. Sinclair"""
... description: """A build tool for Neverwinter Nights projects"""
... license: """MIT"""
... srcdir: """src"""
... backend: """c"""
... bin: "nasher"
... [Deps]
... requires: "nim >= 1.0.2, neverwinter >= 1.2.8, glob >= 0.9.0"
... /home/darren/NWN/nim/nasher/nasher_1719.nims(5, 22) Warning: imported and not used: 'strutils' [UnusedImport]
... printPkgInfo() failed.

Tried it the other way too, doesn't work.
○ → nimble install nasher
Downloading https://github.com/squattingmonk/nasher.nim using git
Tip: 2 messages have been suppressed, use --verbose to show them.
Error: Could not read package info file in /tmp/nimble_3533/githubcom_squattingmonknashernim/nasher.nimble;
...   Reading as ini file failed with:
...     Invalid section: .
...   Evaluating as NimScript file failed with:
...     [Package]
... version: """0.11.0"""
... author: """Michael A. Sinclair"""
... description: """A build tool for Neverwinter Nights projects"""
... license: """MIT"""
... srcdir: """src"""
... backend: """c"""
... bin: "nasher"
... [Deps]
... requires: "nim >= 1.0.2, neverwinter >= 1.2.8, glob >= 0.9.0, regex >= 0.11.1"
... /tmp/nimble_3533/githubcom_squattingmonknashernim/nasher_3533.nims(5, 22) Warning: imported and not used: 'strutils' [UnusedImport]
... printPkgInfo() failed.

Some more info
○ → nim --version
Nim Compiler Version 1.2.0 [Linux: amd64]
Compiled at 2020-04-03
Copyright (c) 2006-2020 by Andreas Rumpf
active boot switches: -d:release
2020-04-17 18:20:12 ⌚  odyssey in ~/NWN/nim
○ → nimble --version
nimble v0.11.0 compiled at 2020-04-03 12:49:03
git hash: couldn't determine git hash

Doesn't work for neverwinter either. Maybe nim 1.2.0 doesn't work with those packages?
± |master ✓| → nimble install neverwinter
Downloading https://github.com/niv/neverwinter.nim using git
Tip: 2 messages have been suppressed, use --verbose to show them.
Error: Could not read package info file in /tmp/nimble_6049/githubcom_nivneverwinternim/neverwinter.nimble;
...   Reading as ini file failed with:
...     Invalid section: .
...   Evaluating as NimScript file failed with:
...     /tmp/nimble_6049/githubcom_nivneverwinternim/neverwinter_6049.nims(6, 18) Warning: import os.nim instead; ospaths is deprecated [Deprecated]
...  [Deps]
... requires: "nim >= 1.0.2"
... printPkgInfo() failed.

Looks like it was a problem with /tmp/nimblecache. Solution is this:
rm -rf /tmp/nimblecache/


Question - Defender 2.2 Clarify confidence level determination for DLP policies

💡 Summary
Add a clarifying statement to the Defender 2.2 policy introduction to make it clear to readers that the confidence level associated with supported information types (SITs) such as SSN, credit card, etc. is up to the organization to determine and is not part of the baseline policy recommendation.
Motivation and context
The current Defender 2.2 policy states that DLP policies should include checks for PII and sensitive information, however, neither the policy item nor its implementation steps mention confidence levels associated with those SITs.  While Microsoft includes a recommended confidence level the policy leaves it up to interpretation whether or not the recommendation is meant to be the default, if the organization should tune the level as desired, or if another specific setting for each level is recommended.
Clarifying policy on confidence levels associated with the SITs would benefit the reader and provide additional guidance for tenant owners configuring DLP policies.
Implementation notes
Clarifying language should be inserted into the policy item description regarding SIT confidence levels and what the recommended approach may be.  The primary options are:

Confidence levels are organizationally defined, when in doubt use vendor defaults
Always use vendor provided defaults
Use baseline recommended defaults (these would need to be created for at least 3 SITs and potentially up to 40+)

Based on the chosen recommendation, update the policy language and implementation steps accordingly.
Acceptance criteria

[ ] Baseline recommendation for setting confidence levels for SITs used in DLP policies is determined
[ ] Defender 2.2 policy language is updated to reflect the chosen recommendation

Makes sense.  Please add clarification.


Tests failing

I tried running make test and got the following error because there is no nproc on mac. I switched it to use sysctl -n hw.logicalcpu and this part was no longer a problem.
 Thank you for the report.


Fix plutono dashboard ConfigMap update

How to categorize this PR?

/area monitoring
/kind regression
What this PR does / why we need it:
The plutono dashboard ConfigMap was not updated properly after https://github.com/gardener/gardener/pull/9624. This resulted in some dashboards provided from extensions to not appear in the UI.
Which issue(s) this PR fixes:
Follow-up of https://github.com/gardener/gardener/pull/9624
Special notes for your reviewer:
Release note:

A bug has been fixed which prevented Plutono dashboards contributed from extensions to appear in the UI.

/cherry-pick release-v1.94

/cc @acumino
as release responsible


Feedback

Please provide list of screens that your app will have

List of screens -

Login / Sign up
Home Page
Description Page (Consisting of description for a particular page)
Payment Screen
Review Screen
User Profile page

Please provide list of screens that your app will have

Updated Sir. Thank You

Closed


Adding multiple rewards to giveaway

As a giveaway owner I would like to add multiple rewards for a single command call. However, bot can't process multiple items in the single message and any user have to do multiple calls for each reward that must be a part of the giveaway.
The required changes:

Add a new method in the GiveawayManager implementation that will consume the incoming message called by the owner and appends multiple rewards in the same way how it works with the single reward per message
Declare a new command in the handlers module that takes the incoming message, do a users checks, parsing via the GiveawayManager instance. For the reference see the implementation of the add_reward command
Updated README.md in the root directory: append information about the next command
Add unit tests for new functionality

Done in PR #4


none-ls: updating + refactoring

Overview issue to track specific issues, PRs, and ideas related to updating and refactoring none-ls.

[ ] Switch to mkNeovimPlugin (move options to settings)

[ ] Done locally. Needs some cleanup, then I'll open a PR.

[ ] Move sources.*.withArgs to sources.*.settings

[ ] Use either strLua submodule to allow a rename-alias
[ ] Include "filetypes" from builtins.json in option types/defaults/descriptions?

[ ] Consider how enableLspFormat should work with rfc42 settings?

[ ] When enabled, it sets on_attach default to require('lsp-format').on_attach.
[ ] Maybe we should just suggest users include this in their on_attach manually?
[ ] Current description is misleading: #1690

[ ] Support "custom" sources submodules

[ ] Custom sources need to be registered first
[ ] Submodule shape: { method, generator, filetypes[], name?, condition? }.

[ ] General refactoring

[ ] Move "builtin sources packages" list into a dedicated file
[ ] Simplify pkg states: can be derivation or bool, don't need null.
[ ] Move builtinPackages out of sync with builtins.json errors out of module and into a CI check

[ ] IFD

[ ] Check if the IFD is contributing to #1154

Removing tests/test-sources/plugins/none-ls.nix and none-ls from plugins/default.nix is not enough to fix nix flake check, however that doesn't rule it out as contributing to the issue.

[ ] Weigh pros/cons of IFD vs a separate "update" script

General notes and discussion can be had on this issue, but I'd encourage opening discussions or issues for specific issues. These can be linked in the relevant checklist above.
Tagging @traxys because he's done most of the work on the none-ls module so far.

Consider how enableLspFormat should work with rfc42 settings?
I'd say we leave it as a top level option, top level options are options that are defined by nixvim, not the wrapped plugin

Consider how enableLspFormat should work with rfc42 settings?

I'd say we leave it as a top level option, top level options are options that are defined by nixvim, not the wrapped plugin

Are we still happy for it to default to true (when lsp-format is enabled)? I think that's a sane default, but it is configuring things for the user without them explicitly enabling anything...

I have mostly taken the approach of avoiding to introduce new magic in nixvim in the last year, so if it were a new plugin i'd suggest not defaulting to false.
For this specific case I don't know

If I enable the lsp-format plugin and set enableLspFormat to true in none-LS, the files are automatically formatted on save. Is there a way to disable this behavior?


Adding AWS SNS as a transport option

I'm currently building out a test logstash environment and to allow me to;

replicate the log entries being processed in prod
enable/disable sending logs to test as and when required

I decided to use SNS as the transport layer which Logstash SQS queues can then subscribe to.

@josegonzalez is there an issue with the tests? They timeout locally and on Travis without providing any error message.

Can you rebase onto master?

Thanks @josegonzalez. Rebased and tests passed.

Merged and released.


Remove ErrDockerfileEmpty from builder/evaluator.go

Problem
Remove ErrDockerfileEmpty from builder/evaluator.go.
Solution
We return non-hardcoded errors all over the package, so it makes sense instead of ErrDockerfileEmpty return error with file name/path like in other places.

#dibs

@icecrime can be closed

That was fast :p


Staking issues

2 staking issues to report:

listsinceblock/listtransactions report staking rewards as category 'receive' with no indication of it being staking rewards so we had to turn it off since we had no way to tell if it was a real deposit or just staking reward.

Even though staking has been off (confirmed by getstakinginfo and debug.log) we got a reward today in https://qtum.info/tx/c636a4b974bcaad5de06c32f47794ec273575f407689f6f60b9dfa7b58dbc841
The input address isn't one of ours so not sure if it's possible to earn PoS even when we aren't staking ourselves?

The way staking works is it spends a utxo and sends the same amount+reward to the same address that the utxo was spent from, the way to determine if it's staked coins or a received coins using listtransactions is to look at blockindex, if it equals 1 then it's staked coins, if it is strictly greater than 1 then it's received coins, the reason is that staking rewards will always be put as the second transaction in a block (first is coinbase, which is empty for PoS blocks, second is coinstake, then other transactions are put in the block).

Qtum uses Mutualized Proof Of Stake, the rewards are distributed over 10 blocks, the first 1/10 is paid in the found block, the rest is paid after 500 blocks over 9 blocks, 1/10 is paid 9 times.

Our staking has been off over a week though, not sure why we'd still be getting rewards?


how can we set hostname using google compute instance template using MIG?

While creating a vm instance as stand alone it shows an option of hostname which works and it can be shown after the vm instance is up and running. But when we try to do the same using the template hostname value shows null.

I don't think you can set hostname on a template. Closing this, feel free to reopen if you find documented proof it can be done.

I don't think you can set hostname on a template. Closing this, feel free to reopen if you find documented proof it can be done.

But there is an option of using hostname while creating a template in the UI of GCP is it futile or working?

I don't see hostname in the GCE API Template object...

See the above two images 1) entering hostname in vm instance template(which does not work)

entering hostname in vm instance(which works)

what can be the reason? since there is an input value to enter hostname in the vm instance template

I believe you, but the console is immaterial for Terraform.
If the field is not exposed in the API and the resource, there's nothing that can be implemented
https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/compute_instance_template

Can you please expose it in the API? xD

Ahah, I wish...

Ok so off topic!! can you please give your inputs for the below issue as well?
https://github.com/GoogleCloudPlatform/cloud-foundation-fabric/issues/1657
Depends_on or time_sleep not working #1657
I am actually trying to register the dns of my VMs which are getting provisioned through MIG(google_instance_template and resource manager) for I am using fabric module. But when I use terraform apply it throws me the below error
Error: Invalid for_each argument
│
│   on .terraform\modules\private-dns\modules\dns\main.tf line 165, in resource "google_dns_record_set" "dns_record_set":
│  165:   for_each     = local.recordsets
│     ├────────────────
│     │ local.recordsets will be known only after apply
│
│ The "for_each" map includes keys derived from resource attributes that cannot be determined until apply, and so
│ Terraform cannot determine the full set of keys that will identify the instances of this resource.
│
│ When working with unknown values in for_each, it's better to define the map keys statically in your configuration and
│ place apply-time results only in the map values.
│
│ Alternatively, you could use the -target planning option to first apply only the resources that the for_each value
│ depends on, and then apply a second time to fully converge.
But to bypass it, I am using depends on dns module which first provisions the vms and then on the basis of ip addresses it registers the DNS. Please help me why is depends_on or time_sleep not working?
resource "time_sleep" "wait_for_workflow_mig" {
depends_on = [module.workflow-mig-resource-manager]
create_duration = "5m"  # Adjust the duration as needed (e.g., 5 minutes)
}
depends_on = [time_sleep.wait_for_workflow_mig]

Please don't cross post unrelated issues. I already replied to that one in #1657

The Cloud Console ("the UI") is misleading you, @rahulguptaft9. An instance template does not have a hostname property. You can see it for yourself in the Compute Engine API docs: instanceTemplate.
You can also verify this by creating an instance template with the hostname of your choice in the Cloud Console. Fill out the fields that you need, including the Hostname but before you click the Create button, check out the Equivalent Command Line or the Equivalent REST option. You will see that there is no mention of the hostname in the actual request that Cloud Console sends to the API. So while there is a Hostname field on the form, it is not being used. I'd say it's a bug in the UI. They probably reuse the same form that is being used to create a new VM instance.
If you proceed and create the instance template and then try to create a VM instance with that template using Cloud Console, you will see that the Hostname field is not populated but you can provide a value at that point. Before you click Create on that form, though, check out the EQUIVALENT CODE section where you can see the REST request that the form is going to send. You will notice that there is a hostname field in the request. The Compute Engine API resource Instance docs confirm that such field is supported.
I think this makes sense – all hostnames are unique, so each VM is going to have its own. An instance template could not have had a hostname field because then it would be limited to creating a single instance. Instead, the instance template allows you to define the properties shared by all the VMs created using that template and it leaves it to you to define the unique properties when you create instances using the template. You can also selectively override the values defined in the instance template when you create a new VM instance with that template.
To implement this in Terraform:

Create your instance template using the google_compute_instance_template resource.
Create VM instances from that template using the google_compute_instance_from_template resource.

The key thing here is that in Step 2 you can use the hostname argument to provide a hostname for this VM. You can also override other parameters set in the template, if you wish.
resource "google_compute_instance_template" "mytemplate" {
  # ...
}

resource "google_compute_instance_from_template" "myvm" {
  # ...
  source_instance_template = google_compute_instance_template.mytemplate.self_link_unique

  # Override fields from instance template
  hostname = "vm-1.example.com"
  # ...
}

You can see the full list of arguments that you can override in the google_compute_instance resource docs.
I hope this helps!

Also, please note that custom hostnames are not resolved by internal DNS so you will need to create a corresponding DNS record in the appropriate zone yourself. For more information, see create a VM instance with a custom hostname.


Fallback samples_dir to current directory if the configuration is not set

Thanks to suggestions from @ruppde , I realized that it would be a nice default to fallback and search for the samples in the folder with the currently opened file.
I am currently not sure how to implement this, since I believe that we can't just use ., because it might not work in every scenario.
But we might be able to extract this information from currently opened Document.

Good idea to use the folder of the .yar because of the vscode structure, you might have different . directories.
The perfect solution would be, to use something like https://github.com/mruffalo/hash-db and regularly create a database of hashes and filepaths, which could then be used. Would allow to keep the rules separated from the samples.


[Resolve Errors] [Review Warnings] Tables - full borders will only show when overflowing (horizontal scrolling active)

closes #547
Note: Requires https://github.com/cfpb/design-system-react/pull/358 to be merged in first
Changes

enhancement-style(tables): full borders will only show for overflowing
chore(tables): removed the need for showTableBorders

How to test this PR

yarn install to get the latest DSR update with the table ref change
use all-errors-warnings-no-syntax-errors.csv or any register with multifield errors that causes overflowing/horizontal scrolling
Verify that tables without overflowing don't have full borders, tables that do have overflowing/horizontal scrolling have full borders

Screenshot

@shindigira
In the screenshot the bottom border is missing. It should be there if the other borders are present.


Adding faucet dockerfile

Usage:
docker build -f packages/faucet/Dockerfile .

Ah, could you please remove the file docs/faucet.Dockerfile which is superseded by this approach?

I added the build and publish instructions to https://github.com/cosmos/cosmjs/wiki/Maintenance#how-to-publish-to-npm-yarn-2-cosmjs-026. This can be automated better later on.


ci: twister: fix workflow

Fixed twister workflow with PRs
tested here:
https://github.com/zephyrproject-rtos/zephyr-testing/actions/runs/11925947380?pr=294
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

@nashif compliance check is sad


Fix async

Adds a reference count to async enabling to pass it properly as thread/async argument.
Protects async arg using a mutex and defer clear to GC.
Copy async arguments rather than using Lua ref.
Fix #505
Fix passing an async as async send argument then use it outside the callback function
Fix suspect pop, here
Fix thread join after exited
This PR attempts to keep the async/thread behavior, except for setting the userdata metadata in the async callback and passing empty userdata as light userdata.
It may not be the best solution as it results in rather conplex and messy code.

Using a reference count is not enough, the async handle from another thread should have a specific behavior such as dedicated GC and close methods.
I put this fix on hold looking for a simpler solution.


Instructions

I’ve got a model attribute whose type is ‘json’ that I’d like to edit within RailsAdmin’s interface.  Based on the name of your gem it seems like a good fit - but I don’t know how to use it ?  I did try installing it hoping that through the ‘magic of rails’ (perhaps via introspection) it would just work - but it did not.
How does one use this gem ?
Thanks
Dave

I'm sorry, I didn't got the time to write a proper readme and release. I'm also not sure if this package runs properly on current releases of ruby, rails and rails-admin. Please advice! I do use it in 2 production apps without any problems though :)
Some notes to get you on the way:

You can give the column where you're storing json a type of text instead of json. I don't have any experience with the json-type, but text will work fine.
You'd have to do the parsing (when you're rendering a page in the frontend) yourself, this is only a interface for RailsAdmin to structure a json-blob.
In initializers/rails_admin.rb, add the following config for your column (components_json in my case)

field :components_json, :json_editor do
  label 'Components'
  help ''

  schema do
    model 'image' do |c|
      c.label 'Image'
      c.field :image, :picker do |f|
        f.picker model: Image, preview_image_field: :cdn_url
      end
      c.field :caption, :string
      c.field :size, :enum do |f|
        f.enum ['large', 'small']
      end
      c.field :modal_enabled, :boolean do |f|
        f.label 'Fullscreen'
      end
    end

    model 'quote' do |c|
      c.field :body, :text
      c.field :cite, :string
    end

    model 'image-text-list' do |c|
      c.label 'List of images and text'
      c.field :items, :list do |f|
        f.list ['image-text-list-item']
      end
    end

    model 'inset' do |c|
      c.label 'Inset'
      c.field :body, :text
      c.field :components, :list do |f|
        f.list ['image-text-list'], max_length:1
      end
    end

    model 'video' do |c|
      c.label 'Video'
      c.field :vimeo_url, :string
    end

    # and so on...
  end
end

I think those are the options and types you have. I'll be happy to assist further on (and hopefully make a proper release of this any time soon..)

You could also take a look at lib/rails_admin_json_editor.rb and app/views/rails_admin_json_editor/main/component_fields for further options.


(🎁) Support relative base-dir

So basically I was trying to invoke mypy-primer with --base-dir ./workdir but it failed, saying that it couldn't find mypy. I think it was looking for workdir/new_mypy/venv/bin/mypy, but the cwd is ./workdir/projects/mypy.
👉 mypy_primer --repo ../mypy --old HEAD -k mypy --base-dir ./workdir

mypy
https://github.com/python/mypy
----------

old mypy
> workdir/old_mypy/venv/bin/mypy --config-file mypy_self_check.ini -p mypy -p mypyc --no-incremental --cache-dir=/dev/null --show-traceback --soft-error-limit ' -1'
UNEXPECTED FAILURE
        /bin/sh: 1: workdir/old_mypy/venv/bin/mypy: not found
----------

new mypy
> workdir/new_mypy/venv/bin/mypy --config-file mypy_self_check.ini -p mypy -p mypyc --no-incremental --cache-dir=/dev/null --show-traceback --soft-error-limit ' -1'
UNEXPECTED FAILURE
        /bin/sh: 1: workdir/new_mypy/venv/bin/mypy: not found
----------

diff
        - /bin/sh: 1: workdir/old_mypy/venv/bin/mypy: not found
        + /bin/sh: 1: workdir/new_mypy/venv/bin/mypy: not found
==========

The solution was to invoke it with --base-dir (pwd)/workdir(🐟), but this is not straightforward from the output displayed, I think it would be good if this usage was supported.
The change is simply just absolute-ifying the base-dir, I'm okay with this, but perhaps there's thing's I'm not accounting for. I'm happy to rework this if there are any alternative suggestions.

Thanks!


Update version: Telegram.TelegramDesktop.Beta version 4.6.4

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

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

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/115246&drop=dogfoodAlpha

@wingetbot run


The file path warning with //

During the submission of the project the Snakemake submission such warning is occuring:
File path histograms/histograms_wjets__nominal__/cms/upload/agc/1.0.0//WJetsToLNu_TuneCUETP8M1_13TeV-amcatnloFXFX-pythia8/cmsopendata2015_wjets_20548_PU25nsData2015v1_76X_mcRun2_asymptotic_v12_ext4-v1_80000_0024.root contains double '/'. This is likely unintended. It can also lead to inconsistent results of the file-matching approach used by Snakemake.
[CRITICAL] File path histograms/histograms_wjets__nominal__/cms/upload/agc/1.0.0//WJetsToLNu_TuneCUETP8M1_13TeV-amcatnloFXFX-pythia8/cmsopendata2015_wjets_20548_PU25nsData2015v1_76X_mcRun2_asymptotic_v12_ext4-v1_80000_0024.root contains double '/'. This is likely unintended. It can also lead to inconsistent results of the file-matching approach used by Snakemake.

This occurs, due to the dependence of the path to histograms on the filename:
root://eospublic.cern.ch//eos/opendata/cms/upload/agc/1.0.0//ST_t-channel_top_4f_inclusiveDecays_13TeV-powhegV2-madspin-pythia8_TuneCUETP8M1/cmsopendata2015_single_top_t_chan_19408_PU25nsData2015v1_76X_mcRun2_asymptotic_v12-v1_00000_0000.root

This issue was addressed in #1.


Increasing evaluate_total_loss value

Is it normal for training processes on AISHELL or HKUST that the total loss value on evaluation set increases after certain steps? Specifically, for HKUST corpus, the evaluate_total_loss is about 12 at 30k step and will increase to over 15 at 100k step; for AISHELL corpus, the loss grows less, but still increases from 6 to 9. These experiments are almost conducted with default configs, only with multiple gpus (8 in total). Is this what should be expected? or is there anything wrong with hyper parameters? Maybe learning rate should be adjust accordingly when using multiple gpu? Thanks

evaluate_total_loss on aishell using athena

as a comparison, evaluate loss on aishell using espnet is like this

@cookingbear please take a look at this

Sorry for the late reply. We have observed similar trends for evaluate_total_loss and we haven't figured out why our loss curves would be different from espnet. Despite it, it wouldn't cause any degradation in performance because the attention and ctc accuracy for each epoch is almost always increasing, and that's the metrics we used when selecting checkpoints for decoding.


Add support for Build contextDir in Buildpacks

Idea:
Currently specifying a subpath where the source code is located does not work for Buildpacks strategies. This is done via the contextDir key. For Buildpacks, it would be interesting to have sample files for different runtimes. For example this repository host many different runtime applications, so using a single repo source for future buildpacks samples would be ideal. But without the contextDir support, this is not possible.
Acceptance_Criteria:
Add the missing functionality in buildpacks strategies, PR this and ensure all tests are happy.

Got it, so the buildpacks strategy doesn't acknowledge contextDir at the moment - and that's gotta be fixed.


Fix items overflowing out of the ComposeRecyclerView in Column/Row

Before:

After:

Note: Apps using a Compose version below 1.4.3 / BOM 2023.05.00 (like the sample app currently) will not show items under/on the right of the ComposeRecyclerView due to a bug fixed in this version.
https://developer.android.com/jetpack/androidx/releases/compose-ui#1.4.3

Thanks :)


Custom Modernizr JS icon conflict

When using custom versions of Modernizr, the Timber icon font no longer works. This is with the current version of Modernizr, with the @fontface component included.

This seems to happen when adding "CSS Animations" to the Modernizr build

Change instances of supports-fontface to what ever your version of modernizer is adding instead. (source). I believe there are just three.

Thanks @cshold. We did actually use the same version with the "supports" prefix and everything. I'll play around with it though, and see if I can find anything else. But I am starting to suspect this is a Modernizr issue, not a Timber issue.

Alright. I'll close this for now, but I'd love to hear what's causing the issue.

Sounds good. I'll be sure to update you when I have more info.


TypeScript Version Mismatch

Describe the bug
Unable to generate browser application bundles (using the standard Angular Tutorial):
(node:6) UnhandledPromiseRejectionWarning: Error: The Angular Compiler requires TypeScript >=4.4.2 and <4.5.0 but 4.5.4 was found instead.
at checkVersion (file:///home/projects/angular-8marnu/node_modules/@angular/compiler-cli/bundles/index.js:17524:11)
at verifySupportedTypeScriptVersion (file:///home/projects/angular-8marnu/node_modules/@angular/compiler-cli/bundles/index.js:17528:3)
at new NgtscProgram (file:///home/projects/angular-8marnu/node_modules/@angular/compiler-cli/bundles/index.js:17678:7)
at AngularWebpackPlugin.updateAotProgram (/home/projects/angular-8marnu/node_modules/@ngtools/webpack/src/ivy/plugin.js:321:32)
at eval (/home/projects/angular-8marnu/node_modules/@ngtools/webpack/src/ivy/plugin.js:198:24)
at Hook.eval [as call] (eval at _0x493c66 (https://angular-8marnu.w.staticblitz.com/blitz.a3339ebb87d0fe73c7995fe67420660e0dc95de2.js:11:221380), :20:1)
at Hook.CALL_DELEGATE [as _call] (/home/projects/angular-8marnu/node_modules/tapable/lib/Hook.js:14:14)
at Compiler.newCompilation (/home/projects/angular-8marnu/node_modules/webpack/lib/Compiler.js:1054:30)
at eval (/home/projects/angular-8marnu/node_modules/webpack/lib/Compiler.js:1099:29)
at eval (eval at _0x493c66 (https://angular-8marnu.w.staticblitz.com/blitz.a3339ebb87d0fe73c7995fe67420660e0dc95de2.js:11:221380), :31:1)
(Use node --trace-warnings ... to show where the warning was created)
(node:6) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:6) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Link to the blitz that caused the error
https://stackblitz.com/run?file=src/app/app.component.ts
Steps to reproduce

Go to https://angular.io/generated/live-examples/getting-started-v0/stackblitz.html
Wait.
That's it, in the third step, you should encounter this error.

Expected behavior
The tutorial Angular webapp opens up.
Screenshots
No response
Platform
Browser name  = Chrome
Full version  = 96.0.4664.93
Major version = 96
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.93 Safari/537.36 Edg/96.0.1054.53
performance.memory = {
  "totalJSHeapSize": 86913043,
  "usedJSHeapSize": 80629635,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = a3339ebb87d0fe73c7995fe67420660e0dc95de2

Additional context
No response

@atutcuog Thanks for the report, I suspect this is an upstream issue, see https://github.com/angular/angular/issues/44480 and https://github.com/stackblitz/webcontainer-core/issues/473. Please, do comment if that does not apply to your problem.


Unread message not displayed properly on small screens

Pity that there are no very small screens, as the styling for the wider screens can also fit on smaller screens


Dev inspector

Summary by Sourcery
Refactor the error handling mechanism by extracting the global error handler initialization into a new common module. Update the Webpack configuration to reflect the new entry point for the renderer. Rename and adjust the inspector renderer class for better clarity and functionality.
Enhancements:

Refactor error handling by moving the global error handler initialization to a new common module, improving code reuse and maintainability.

Build:

Update Webpack configuration to use the new renderer.desktop.ts entry point instead of renderer.ts.

@sourcery-ai review


Respect all git versions

Since 1.8.5 git doesn't use #  in output. See http://blogs.atlassian.com/2013/12/whats-new-git-1-8-5/

If you add the -b option to --porcelain you'll get the branch information (ahead/behind/diverged). That's where lib/git.zsh gets it from for the git-enabled theme prompts.

@nicorevin what about what @apjanke just suggested?

I would better get rid of it at all as it is way too expensive for a shell prompt