Advisories didn't pushed

Hello,
I saw 4 vulnerabilities for SF 3.4 today, but none of them pushed up in the https://security.symfony.com/ , I have 2 questions  :

Do I need to add all components all present in symfony/symfony in my composer.json ? (for example the Cache component is include in the repository symfony/symfony in sf 3.4)
OR I need to create a pull request in this repository to adding the CVEs in the advisories symfony/symfony ?

Thanks

Thank you @nutama but I think you forget some versions for cve-2019-188888888 : it concerns all versions sf 2.8 / 3.4 / 4.3


Support more chains

I stumbled on your work while trying to test tokens for honeypot scams and i just want to know if there are plans to include more chains ?

And I don't know what his plans are.
By myself, I am not the developer of the honeypot.is website and its api, I am the dev of the library that interacts with the site.
But the dev has a telegram: https://t.me/ishoneypot and channel: https://t.me/honeypotis

Thanks for the response, i will try reaching out to the developer on TG


SCons : Add "scu_limit" argument

"scu_limit" allows specifying the maximum number of includes in a single SCU file (translation unit). A lower limit (e.g. 8) uses less RAM during compilation, but may be slower to compile.
Explanation
The max number of includes in a SCU file (which is a single translation unit used by a single compiler instance) determines very roughly the maximum RAM used by the compiler instances. RAM is important to not let get out of hand, particularly in release builds (which use a lot more RAM), so the defaults are for 1024 maximum includes in dev builds (to maximize compilation speed) and 8 in release (to get some acceleration, but limit RAM use to approx 1Gb per compilation instance).
The total RAM used by SCons approximates to:
number_of_cores * RAM_per_core
So a more powerful machine with more RAM and cores may not always be able to safely increase scu_limit if it has a corresponding large number of cores (e.g. 64Gb machine with 32 cores = 2Gb per core, same as 8Gb with 4 cores).
Although these defaults are fine in most cases, there are a couple of situations where it is nice to be able to manually fine tune the RAM usage:

On computers with large amounts of RAM, for faster compilation in release
On CI runs, where cores and total RAM may be limited

Particularly the CI runs RAM may be more limited, so it may be sensible to set scu_limit for DEV builds to a value closer to the release default (8 or 16 or so, rather than the default 1024). This may require some testing to get the best value for scu_limit to use on a particular system.
Notes

As always, happy to change the name of the parameter etc if we can think of something better.

Thanks!


DS-3533 Projections continued

This is a continuation of the projections work that is expected to improve performance significantly.
Status as of 2020/02/06

All changes described below are implemented
All previous ITs are passing
TBD: Adding additional tests to improve projections coverage

Status as of 2020/01/30

Functionality works as described below
Tests need to be adjusted and improved

Details
This PR:

Makes all subresource GETs use LinkRepositories where possible. This makes it easier to control, on a per-request basis, whether they are embedded or not.
Inverts the default embed behavior, as discussed in the working group meeting in January. Now, subresources are generally NOT embedded by default. They are only embedded now if a projection allows them to be. The "list" projection has been removed and replaced with a "full"  projection. When requested, whether on a single resource or collection resource, projection=full causes all allowed embeds to be included in the response.
Disables embedding of communities within communities, and groups within groups, even in the "all" projection; sub-resources must therefore always be requested explicitly. This effectively avoids recursively embedding the entire tree, when a single top-level community or group is requested.
Updates tests accordingly

Note:

This PR has the effect of standardizing the name of the array property "mappedCollections" to "collections" (using the plural type name as the array property name, as is conventional for all other paged subresources), and has the following minor angular PR that should be merged at the same time: https://github.com/DSpace/dspace-angular/pull/548
This PR also fixes https://jira.lyrasis.org/browse/DS-4248

Thanks @cwilper for this PR.
One thing I noticed from your description without having completed a full review:

You mention renaming "mappedCollections" to "collections". This may become confusing since this doesn't contain the owning collection

Am I correct to summarize this PR as:

When requesting a single item, the embedded data (relationships, owningCollection, bundles) is excluded when using projection=list
For discovery searches, the embedded data of the items is always excluded
When requesting item lists, the embedded data of the items is always excluded
When requesting a single community, the embedded data (collections) is excluded when using projection=list
When requesting community lists, the embedded data of the communities is always excluded

Thanks for taking a look @benbosman

One thing I noticed from your description without having completed a full review:

You mention renaming "mappedCollections" to "collections". This may become confusing since this doesn't contain the owning collection

The "rel" name of the subresource is still mappedCollections -- this just changes the name of the embedded array property inside the "page" of mappedCollections. For this PR, the fact that this inner-array name changed was actually a side-effect of making this info come from a link repository instead of its own controller and related scaffolding. As long as we're consistent, I figured it would be a low friction way to go, as that his how other link repository-provided subresource embeds currently work (using the plural form of the type). But I can also see the argument for forcing it to match the rel name in all cases. I don't mind either style -- I mainly just care about consistency.

Am I correct to summarize this PR as:

When requesting a single item, the embedded data (relationships, owningCollection, bundles) is excluded when using projection=list

true.

For discovery searches, the embedded data of the items is always excluded

by default yes, because the default projection in that context is "list". But that can be overridden by forcing it via projection=default.

When requesting item lists, the embedded data of the items is always excluded

by default, yes, with same exception described above.

When requesting a single community, the embedded data (collections) is excluded when using projection=list

yes.

When requesting community lists, the embedded data of the communities is always excluded

yes.

@benbosman I just wanted to quickly follow up on the paged embedded sub-resource rel naming issue and note that I'm looking at consistency issues in detail.
I found an inconsistency with this branch currently: If I temporarily allow certain paged subresources to be embedded in a resource (e.g. mappedCollections embedded in a collection resource), I see that the array name matches the rel name in that context. However, if I hit the URL for the subresource directly, the embedded array name matches the plural type name of the embedded resources. This inconsistency should be addressed. I'm not sure how yet.
I am looking at the possibility of just always making the rel name match the array name regardless of what context the embed occurs in (inside a resource vs. as a subresource via a link repository).
There are other possibilities but I want to explore this one fully before falling back to another approach.
Marking this PR as in-progress again.

@cwilper what would this imply for the sub-resources (e.g. item's mappedCollections or browse's entries also named browseEntries)
Currently, an item's mappedCollections is not directly embedded, but a HAL link to /server/api/core/items/uuid/mappedCollections is included. Opening /server/api/core/items/uuid/mappedCollections contains:
"_embedded": {
    "mappedCollections": [
...
]
  }

A browse's entries is embedded, using a HAL link to e.g. /server/api/discover/browses/subject/entries and the embed of /server/api/discover/browses/subject is:
"_embedded": {
    "entries": {
      "_embedded": {
        "entries": []
      },
      "page": {
        "number": 0,
        "size": 20,
        "totalPages": 0,
        "totalElements": 0
      },
      "_links": {
        "self": {
          "href": "https://dspace7-demo.atmire.com/server/api/discover/browses/subject/entries"
        }
      }
    }
  }

Opening /server/api/discover/browses/author/entries?size=1 contains:
{
  "_embedded": {
    "browseEntries": [
      {
        "authority": null,
        "value": "Abdominal and Paediatric Surgery",
        "valueLang": null,
        "count": 9,
        "type": "browseEntry",
        "_links": {
          "items": {
            "href": "https://dspace7-demo.atmire.com/server/api/discover/browses/author/items?filterValue=Abdominal and Paediatric Surgery"
          }
        }
      }
    ]
  },
  "_links": {
    "first": {
      "href": "https://dspace7-demo.atmire.com/server/api/discover/browses/author/entries?page=0&size=1"
    },
    "self": {
      "href": "https://dspace7-demo.atmire.com/server/api/discover/browses/author/entries?size=1"
    },
    "next": {
      "href": "https://dspace7-demo.atmire.com/server/api/discover/browses/author/entries?page=1&size=1"
    },
    "last": {
      "href": "https://dspace7-demo.atmire.com/server/api/discover/browses/author/entries?page=468&size=1"
    }
  },
  "page": {
    "size": 1,
    "totalElements": 469,
    "totalPages": 469,
    "number": 0
  }
}

I would expect the end-result to be:

The HAL link still contains the name explaining the actual functionality (e.g. mappedCollections)
The embed in the parent record still contains that same name explaining the actual functionality
The nested embed in the parent record's embed contains the type of object, e.g.

"_embedded": {
   "mappedCollections": {
      "_embedded": {
        "collections": []
      }
   }
}

Opening the HAL link contains the type of object, similar to the inner part of the embed

Does that match your expectations?

Note: The Travis failure above is a timeout, very early in the build, while downloading maven dependencies. Probably a service blip on their end. We have an upcoming commit that will trigger it again soon, but as of yesterday, existing tests were passing.

Merging, as this is at +2.  Thanks for the (continued) hard work on Projections @cwilper !


Add more kinds of underline colour

I'm using Monaco Editor to make my own IDE. I use provideHover to underline certain kinds of code. It seems that different DiagnosticCategory or MarkerSeverity have impact to the colour of the underline. At the moment, it seems that there are only 4 kinds of underline colours.
enum DiagnosticCategory {
    Warning = 0,
    Error = 1,
    Suggestion = 2,
    Message = 3
}

Does anyone know how to add more kinds of underline colour?

I'm sorry it doesn't seem to be working, but with so many requests that we get here and in the VS Code issue tracker, we just cannot provide the support we would like to offer.
You can try asking your question on stackoverflow.
That said, if you can track down your issue to a specific problem of the Monaco editor, we are more than welcoming a PR by the community! Please use the bug-report issue template to report the issue.
Thanks for your understanding!


Please add a database SQL script file to the project

Can the project offer users an SQL script for the database? Otherwise, executing 'php artisan migrate' in the project will always result in errors.

Hi @JustinSunli What are the errors encountered with the artisan command?  I will look for the script, but it could make the application obsolete if the migrations are modified

@JustinSunli please, find this sql file :  erp.zip

thank you!


fixes #35 Upgraded bamboo version

Plugin was installed on our bamboo version and tested.

fixes #35


Make Until consistent with PJ paper, change american option builder. Fixes 69

Fixes https://github.com/digital-asset/contingent-claims/issues/69
This is the PR where the dynamics of Until as well as the american option builder are changed. The problem and proposed solution are described in the related issue.
What I am not yet happy with is the fact that acquire' is now a very complex functions, as it

evaluates When andAnytimepredicates
fixes acquisition time of non-deterministic predicates
evaluates Cond predicates
consumes Cond nodes once they are reached
expires Until nodes when the predicate is True
expires worthless sub-trees (When and Anytime node where the predicate will never be met)

This can probably be split in different functions to make it a bit simpler. lifecycle would then be a composition of these simpler functions.

The latest commits decompose the acquire' function in three components:

non-effectful acquire' : its job is to acquire the sub-trees and calculate their acquisition time
expire' : expires sub-trees that will never be acquired
fixAcquisitionTime' : fixes the acquisition time for When nodes with a non-deterministic predicate

I think this makes the code easier to read.
I also got rid of expire as it is no longer needed

As discussed in person this seems fine to me. The only thing I would think about is having clear combinators for distinguishing time =< >= and decimal. at and upTo sound fine though.

Thank you. Will open an issue to discuss the names within the team.


Implementing CSS recommendations per Design Team

See https://saucedev.atlassian.net/browse/TCD-1698?focusedCommentId=314934

build failed; opening new PR


DO NOT MERGE purge memory targets, now built into ORFS

@lpawelcz Can you fix this PR so that it purges the memory targets? memory targets are now built into ORFS, so it is no longer a bazel-orfs cocern.

Sure, we will handle that. I imagine that you would like to be able to call memory make targets from ORFS with scripts generated in bazel-orfs, is that right?
Looking at https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/pull/1727 we notice that the memory report is now also a part of the synthesis so it should be sufficient to just test memory make targets from ORFS with scripts generated in bazel-orfs

Sure, we will handle that. I imagine that you would like to be able to call memory make targets from ORFS with scripts generated in bazel-orfs, is that right? Looking at The-OpenROAD-Project/OpenROAD-flow-scripts#1727 we notice that the memory report is now also a part of the synthesis so it should be sufficient to just test memory make targets from ORFS with scripts generated in bazel-orfs

Yes. Though you don't need to add a CI test for this. This is now just one of many, many ORFS features that just works out of the box and that we don't need CI bazel-orfs tests for.

this was just an example, easier to do from scratch when other changes are in.


Additional pilot questions, love to help colab

Hey Walker,
I've been using your alexa app but the quesitons run out pretty quick. I have access to both the Gleim and sporty's course and could use typing them into your database as a good way to both help others using your alexa skill, and myself memorize them.

Hey Brian, great to hear from you! I agree -- I didn't add too many questions, so of course you're more than welcome to type up as many as you'd like and I'll update the app. If you're familiar with JS and Git, feel free to just submit a pull request and I'll review it and merge it (you'll see how they're formatted here). If you're not familiar with all that, just let me know and we'll go from there.
Sorry the code isn't too documented or streamlined -- I never expected it to live long!

Never used JS or Git other than once or twice. That being said I can take a look at the link and see the first answer is the correct one, is there a minimum limit on answers? all the test prep books, including the actual FAA exam are 3 answers, two wrong and a right.
I think changing Alexa to only read off three would make it better for test prep as she could go through questions faster and make it more relatable to the test prep material. Thoughts?

hey @walkman1915 did you see my above comment?


Update gridsearch params and golden files.

What broke the tests this morning was:

The change to the grid search params
The change to the default of using a holdout (and the test invoking that)

This change reapplies your change to the gridsearch params, and it also changes the test to use
defaults for the holdout. Then I just updated the goldens, given that nothing looked crazy. I'll let
you decide on merging this.

Yes / no?

"broke" may not be the right word, but I'll role with it. The grid search parameters are something that we need to be able to change. They should not be static. The approach is brute force, but it is important to generically tuning the model and to understand the explanatory power of the findings. Why don't we pass the grid search parameters in a tuple instead?

When I said "broke", I'm not saying the behavior of your code was wrong, I
mean the test was broken by your change. The test is saying "I expect that
if I run the pipeline with these parameters, then I'll get this output". If
you're intending to change the output with a change, then the testdata
needs to be updated, otherwise the test will break. If you're not
intending to change the output, then it provides assurance that you haven't
done so accidentally. It's generally helpful (when working with a test like
this) to break changes up into refactoring changes (potentially major code
changes, but no testdata change) and behavior changes (focused code
changes, testdata change to match new behavior).
In terms of passing the grid search params, I'd say that we probably need
to factor some of these parameters into config classes so that it's not a
huge list. Like, maybe a DatabaseConfig and some other config? shrug I'll
think about it some more.
A few questions:

Does the pre-commit hook work for you? (and other scripts)
Some parts of the code suggest that this is supposed to work with
databases other than the one provided. What exactly is required of a
database for it to work with your software (ideally)? All the same columns?
A superset of some required columns?
What should I prioritize next?

On Wed, Mar 9, 2016 at 2:26 AM, PandaStabber<EMAIL_ADDRESS>wrote:

"broke" may not be the right word, but I'll role with it. The grid search
parameters are something that we need to be able to change. They should not
be static. The approach is brute force, but it is important to generically
tuning the model and to understand the explanatory power of the findings.
Why don't we pass the grid search parameters in a tuple instead?
—
Reply to this email directly or view it on GitHub
https://github.com/PandaStabber/Goldberg_et_al_2016/pull/10#issuecomment-194227108
.

A few questions:

Does the pre-commit hook work for you? (and other scripts)
**Yes, but I'm not sure how to do this. I'll follow your lead here. **

Some parts of the code suggest that this is supposed to work with
databases other than the one provided. What exactly is required of a
database for it to work with your software (ideally)? All the same columns?
The column names, as well as which columns are categorical variables, should be flagged. The categorical variables then need to be encoded one-hot style. In this doc, I've also got some lines where i drop columns that aren't necessary. This is typically required, as most databases have columns with too few data, which need to be removed, or aren't related to the actual data (e.g., publication title, authors, database notes, etc).

A superset of some required columns?

What should I prioritize next?
Good question. The physicochemical conversion should be done before it's put into the main(). We should pull this out and fix up a script to run to convert features.

Answers:

Pre-commit hook: When you're in the main directory, in a shell:
$ scripts/install_presubmit.sh
This will make it so when you run git commit, it will first run the presubmit. Also, if you want to just run the presubmit or the tests or lint or whatever, you just run the appropriate script directly:
$ script/run_tests.sh  # runs the tests

For dropping columns, I can imagine two scenarios. Which do you want?:

We only support some predetermined columns. Anything else in your database gets dropped.
We will support any column unless you explicitly drop it. That column will be untransformed.

And the last one:

Yes, I agree that pulling out the conversions into separate code is a good idea and makes sense to approach soon.


feat(NFTs): Added NFT view

:tada: This PR is included in version 1.4.0 :tada:
Your semantic-release bot :package::rocket:


Implement fixes requested by ESMValTool team

Required steps to manually add fixes:

[ ] check that our data exhibits the error to be fixed
[ ] write the xarray fixing code in daops, with associated unit tests
[ ] add the fix class to dachar.fixes...., with associated unit tests if required
[ ] decide/agree on content of fix metadata:

URL to issue, or code, or both in ESMValTool repo
description of the fix
"source" property to identify that this came from ESMValTool, maybe with release version or github commit number.

[ ] use inventory to identify all data sets that will be affected
[ ] generate (by hand?) fix proposals:

create fix proposal per datasets, OR
create fix proposal template and list of datasets file

[ ] do we need some kind of QC when the fixes are being proposed?
[ ] publish fix proposals:

Start with a one-off proposal:  dachar propose-fixes -p cmip6 <json_file>
Adapt to: dachar propose-fixes -p cmip6 --file-list=datasets_files.txt <fix_template.json>

[ ] process fixes as usual - to accept them

Relevant to:
https://github.com/ESMValGroup/ESMValCore/issues/755
https://github.com/ESMValGroup/ESMValCore/issues/787

@ellesmith88 , I have assigned this to the two of us.

Suggested content for source:
"source": {
    "name": "esmvaltool...",
    "version": "3.4.5",
    "comment": "any useful info",
    "url": "https:.....isssue/34"
}

Write up of what I have done so far:

So far I have been working with the o2 fix: https://github.com/ESMValGroup/ESMValCore/blob/master/esmvalcore/cmor/_fixes/cmip5/hadgem2_cc.py#L34-L55

Identifying the error in our data:
I have found that in our cmip5 data the long name and standard name of the o2 variable in the hadgem2-cc model doesn’t match the ‘corrected’ names in the fix - so this shows that the error exists in our data.

Write xarray fixing code in daops:
I have written a generic fix that can be used to update any of the attributes of the main variable in a dataset:
https://github.com/roocs/daops/blob/esmval_fixes/daops/data_utils/var_utils.py#L4

Added a fix into dachar fixes:
https://github.com/roocs/dachar/blob/esmval_fixes/dachar/fixes/var_fixes.py#L8
The attributes to be updated and what they should be updated to can be passed as a list of strings - each string should contain the attribute to be changed and what it should be changed to separated by a comma e.g.
["long_name,Dissolved Oxygen Concentration", "standard_name,mole_concentration_of_dissolved_molecular_oxygen_in_sea_water"]

As this data doesn’t exist in the c3s-cmip5 data (and so isn't in the inventory) I found the affected datasets using glob.glob("/badc/cmip5/data/cmip5/output1/MOHC/HadGEM2-CC/*/*/*/*/*/*/o2")

I have provided 2 different ways of proposing a fix via the cli:

By providing a json file e.g. https://github.com/roocs/dachar/blob/esmval_fixes/tests/test_fixes/esmval_test_fixes/o2.json
This option could be used when you have a fix that you only need to propose for one dataset, or just a few datasets.
The cli usage for this is dachar propose-fixes -f <json_file> but more than one json file can be provided: dachar propose-fixes -f <json_file>,<json_file2>,<json_file3>
The fix/dataset doesn't need to be the same in these files if multiple are provided.

By providing a template (json format) and a list of datasets that the fix should be proposed for (in a .txt file)
For example:
Template: https://github.com/roocs/dachar/blob/esmval_fixes/tests/test_fixes/esmval_test_fixes/o2_template.json
Dataset list: https://github.com/roocs/dachar/blob/esmval_fixes/tests/test_fixes/esmval_test_fixes/o2_fix_ds_list.txt
The cli usage for this is dachar propose-fixes -t <json_template> -d <dataset_list>
The fixes are added to the fix proposal store from the cli via these functions:
https://github.com/roocs/dachar/blob/esmval_fixes/dachar/fixes/generate_proposals.py#L38-L73
More than one fix can be proposed in the complete json file or the json template.
The command-line options are checked in cli.py so that incompatible options cannot be used together:
https://github.com/roocs/dachar/blob/esmval_fixes/dachar/cli.py#L255

The fix proposals are checked before being added to the fix proposal store. (QC)(https://github.com/roocs/dachar/blob/esmval_fixes/dachar/fixes/generate_proposals.py#L19).
This checks that the required fields are there and that there are no unexpected fields.

The source property provided in the fix proposal has been updated to include extra fields:

"source": {
    "name": "esmvaltool...",
    "version": "3.4.5",
    "comment": "any useful info",
    "url": "https:.....issue/34"
}

This records the origin of the fix.
For fixes proposed by dachar - the source is currently initialised by the check that identifies that a fix is needed e.g.
https://github.com/roocs/dachar/blob/esmval_fixes/dachar/analyse/checks/coord_checks.py#L16
There may be a better place for this if it needs to be more specific.

The proposed fixes are recorded in the fix proposal store and can be processed and published as normal.

@ellesmith88 Thanks for the detailed write-up. This is looking really good.
One thought about the format of the fixes: at the moment the attribute name and value are being embodied together in a string: key,value. I can think of cases where we might need to preserve the data type of the value and therefore I reckon it is safest to represent them as dictionaries of key/value pairs rather than a comma-separated string.
By way of an example, this current fix:
    {"fixes": [
        "long_name,Dissolved Oxygen Concentration",
        "standard_name,mole_concentration_of_dissolved_molecular_oxygen_in_sea_water"
        ]
    },

would change to:
    {"fixes": [
        {"long_name": "Dissolved Oxygen Concentration"},
        {"standard_name": "mole_concentration_of_dissolved_molecular_oxygen_in_sea_water"}
        ]
    },

If we needed to use floats or ints we could then represent them in their native form, e.g.: {"quality_flag": 3"}

@agstephens Good point, I'll update that 🙂


more beta testing and breaking

In Options tab, should "Page Title" be changed to "Map Title", and "Subtitle" to "Map Subtitle"?

Yes, please go ahead and change these

Yes, please go ahead and change these


Remote Containers: Cannot save notebook locally when disconnected

Type: Bug
I got back to VSCode after a leaving it overnight connected to a Dev Container. It is now in a disconnected state, which is fine, but one of my files (a jupyter notebook) is unsaved and changes will be lost. I cannot reconnect or rebuild the container.
Repro steps

Set up a simple Jupyter Data Science Notebook project
Create a new notebook and execute some python commands
Save the notebook, edit it some more
Leave it overnight maybe?
~~~ time passes ~~~
VSCode is "Disconnected from Dev Container: Jupyter Data Science Notebook"
VSCode has one notebook open with unsaved changes.

Expectations

I should be able to save the unsaved changes somewhere locally.

Reality
I cannot Save the file since the container is not connected:
Failed to save '05.Export.All.Chat.Messages.For.Org.ipynb': No file system provider found for resource 'vscode-remote://dev-container%2B2f55736572732f636d2f6465762f30302f7477696c696f/workspaces/00/twilio/05.Export.All.Chat.Messages.For.Org.ipynb'
I cannot Save As... because
File system provider for vscode-remote://dev-container%2B2f55736572732f636d2f6465762f30302f7477696c696f/workspaces/00/twilio/05.Export.All.Chat.Messages.For.Org.ipynb is not available.
I cannot copy & paste the file because I cannot see the notebook source.
I cannot Reopen folder locally because it launches a new VSCode window without the unsaved changes.
I cannot Rebuild container because it fails:
[934 ms] Error: An error occurred setting up the container.
[935 ms]     at MS (/Users/cm/.vscode/extensions/ms-vscode-remote.remote-containers-0.245.0/dist/spec-node/devContainersSpecCLI.js:254:2186)
[935 ms]     at Jw (/Users/cm/.vscode/extensions/ms-vscode-remote.remote-containers-0.245.0/dist/spec-node/devContainersSpecCLI.js:254:2124)
[935 ms]     at processTicksAndRejections (node:internal/process/task_queues:96:5)
[935 ms]     at async KS (/Users/cm/.vscode/extensions/ms-vscode-remote.remote-containers-0.245.0/dist/spec-node/devContainersSpecCLI.js:260:2006)
[935 ms]     at async co (/Users/cm/.vscode/extensions/ms-vscode-remote.remote-containers-0.245.0/dist/spec-node/devContainersSpecCLI.js:260:3110)
[935 ms]     at async LP (/Users/cm/.vscode/extensions/ms-vscode-remote.remote-containers-0.245.0/dist/spec-node/devContainersSpecCLI.js:359:9352)
[935 ms]     at async DP (/Users/cm/.vscode/extensions/ms-vscode-remote.remote-containers-0.245.0/dist/spec-node/devContainersSpecCLI.js:359:9108)
[936 ms] Exit code 1

I cannot Attach to running container... because it opens a new window without the unsaved changes.
VS Code version: Code 1.70.0 (da76f93349a72022ca4670c1b84860304616aaa2, 2022-08-04T04:39:20.716Z)
OS version: Darwin arm64 21.5.0
Modes:
Connection to 'Dev Container: Jupyter Data Science Notebooks (Community)' could not be established  Reconnect attempt without old result.

System Info

Item
Value

CPUs
Apple M1 Max (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
32.00GB (0.23GB free)

Process Argv
--crash-reporter-id 0dad8ce6-c1b0-4199-a162-02412a694806

Screen Reader
yes

VM
0%

Connection to 'Dev Container: Jupyter Data Science Notebooks (Community)' could not be established  Reconnect attempt without old result.
Extensions: none
A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonvspyt551:30345470
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
wslgetstarted:30449410
cppdebug:30492333
pylanb8912:30545647
vsclangdf:30486550
c4g48928:30535728
hb751961:30543456
dsvsc012cf:30540253
azure-dev_surveyone:30548225

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.74.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

This seems to be an issue in notebook land, here is what happens:

eventually fileWorkingCopyManager.saveAs method will be called
we end up in the doSaveAs method and need the snapshot of the working copy to get at the contents (here)
we end up in the notebookEditorModel to get the notebook data (here)
this call never returns for me in my testing, I wonder if this needs somehow access to the remote?

Here are my steps in macOS:

fresh user data dir, insiders
install docker, remote container and jupyter notebooks
start docker (I had Alpine Linux)
open VSCode on the container
create a new foo.ipynb file and save it
make it dirty
quit docker
dismiss the notification that the instance is disconnected
select File | Save As...
for me this call never steps over the this._notebookSerializer.notebookToData(data) call

Btw this can be reproduced running out of sources via our test resolver and running this command:

this call never returns for me in my testing, I wonder if this needs somehow access to the remote?

This makes sense, that's calling to the notebook serializer extension in the remote EH. I think we can't really support this scenario. However just reloading the window to reconnect to the remote should load with the unsaved changes, doesn't that work?

I cannot Rebuild container because it fails:

I'm not sure what that error is. It seems unrelated. Just reconnecting should do it.

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

I have a similar issue.
I use remote SSH connection in VSCode to work on my virtual machine filesystem (Host: Windows 11, Hyper-V Guest: Ubuntu server 18).
After restarting the host the virtual machine is down and VSCode could not connect to it over SSH, while there was an unsaved file on the remove filesystem. The file is just a shell script text file with SH extension.
I cannot see the content of the file, I see "The editor could not be opened due to unexpected error: Unable to resolve resource vscode-remote...".
I started the virtual machine and can Remote SSH: Connect to Host (in new window).
But if I try to Remote SSH: Connect Current Window to Host, it asks me if I want to save the file first. And if I select Save or Cancel, the connection won't be established, with the remove filesystem provider error.
If I try to Save or Save As, then I get the message Filesystem provider is not available.
Restarting VSCode helps: it connects to the virtual machine upon opening and shows the unsaved file contents. Seems to be not user friendly though.


Change Character Event Animation Bug

Describe your bug here.
When using the change character event, the character that is swapped in by the event has all of their animations play at an incorrect framerate (or incorrect speed? I can't tell).
At first I thought it might have been an issue with the HTML build or a custom character's files, but this issue persists in the Windows build and when using the default characters.
https://user-images.githubusercontent.com/91307107/165630787-ebeb35c0-073f-41ef-b969-efaabfd510e5.mp4
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
HTML5
Did you edit anything in this build? If so, mention or summarize your changes.
No

Messed with the source code for a little bit and it seems like the issue is rooted in the character groups feature.

it is not a bug, it's just offset issue when using non-playable characters as playable


Update scalafmt-core to 3.0.5

Updates org.scalameta:scalafmt-core from 3.0.4 to 3.0.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, semver-patch

Codecov Report

Merging #50 (ecbba51) into main (4438bc4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #50   +/-   ##
=======================================
  Coverage   76.39%   76.39%
=======================================
  Files          17       17
  Lines         572      572
  Branches       20       20
=======================================
  Hits          437      437
  Misses        135      135

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4438bc4...ecbba51. Read the comment docs.


chore(ci): run CI only on main

Runs Github actions only when a pull request is open or when commits are pushed to main.

So far, so good. Do you think it also makes sense to only run CI when the PR is onto main? I don't know if we'll ever be in a situation where we will be PR-ing between branches on the same repo but it could be nice to have.
Also, while we're on the subject of CI, we could create a workflow to automatically make a release whenever we push to a certain branch. We'd have to figure out a way to specify the version number though.

So far, so good. Do you think it also makes sense to only run CI when the PR is onto main? I don't know if we'll ever be in a situation where we will be PR-ing between branches on the same repo but it could be nice to have.

It makes sense 👍   I just made a change.

Also, while we're on the subject of CI, we could create a workflow to automatically make a release whenever we push to a certain branch. We'd have to figure out a way to specify the version number though.

It would be great! (but personally for now don't have time to do this)
The version number could be generated if we are strict about commit (for instance a fix commit bumps patch version whereas a feat commit bumps minor version) and maybe we could also generate the release changelog.

It would be great! (but personally for now don't have time to do this)

Would you mind if I wrote a draft of the functionality for you to look at?

The version number could be generated if we are strict about commit (for instance a fix commit bumps patch version whereas a feat commit bumps minor version)

I'll have to get better about my commit message discipline, but that's a smart way to do it!

and maybe we could also generate the release changelog.

Mhm, it could just be a list of the commits at bare minimum.

It would be great! (but personally for now don't have time to do this)

Would you mind if I wrote a draft of the functionality for you to look at?

Good idea, although for me it's not a real problem to bump the tag manually, I will be happy to see an automation for that.

Hi, I just had an idea for automating releases. What if we made it so releasing was done by making a commit onto main with the title vX.Y.Z?
The changelog would be the commit message unless the commit said "@generate_changelog" and in that case the action could autogenerate the changelog.

Could be nice. Currently it's the addition of a new tag on a commit on main that makes a release on deno.land/x but maybe adding a specific commit to automate it would be better.
Honestly I don't know which is the best practice for that 🤔


An error "Element with id... is already registered" occurs when expanding the attached PostgreSQL server

OS: All
Build Version: 20200908.6
Repro steps:

Attach PostgreSQL server with a database (add the database name next to the connection string).
Attach the above PostgreSQL server without database (remove the database name from the connection string).
Expand the attached PostgreSQL server without database.

Expect:
It shows all databases in the attached PostgreSQL server without error.
Actual:
An error "Element with id... is already registered" occurs.

More Info:

This issue does not reproduce if only attach the PostgreSQL server without database.
This error also reproduces if refresh the attached PostgreSQL server or create a new database for it.

https://github.com/microsoft/vscode-cosmosdb/pull/1640#discussion_r470867053
Options for fix: append /database to id of PostgresDatabaseTreeItem, or handle the "Element with id... is already registered" error to throw a more clear error?
With the current implementation, same server cannot be attached twice even if one is with and other is without database.

Two questions:

Does Mongo have the same behavior?
If I attach both with and without a db, is it easy to detach either one and get into a "good" state?

If so, I think this is low priority and we can fix in a later release

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


Python 3.11 Support

Hi folks,
(when) is Python 3.11 support planned?
Tried to install on Ubuntu 22.04 with conda (not resolving) and pip (pip subprocess to install build dependencies did not run successfully), but both failed.
Thanks!

Hi @do-me, thanks for reaching out. Yes, we do plan to support Python 3.11. We are waiting for all DP's dependencies to be supported by Python 3.11 right now. I will keep you posted.

0.16.0 is released, which adds support for python 3.11


[Feature] Add get-locale and format-date

Describe the solution you'd like
These are already in the bradtaniguchi.github.io repo.

This actually isn't as useful due to how nextjs handles locales being part of the router.
Either the project handles locales directly or it won't know during static build time, only runtime.
Closing this for now.


Support returning the language a translation resolves to

To start with: great little library 👏🏼.
One minor thing I would love to see is a way of knowing which language your returned translation is actually in. This is very useful in case of missing translations.
// i18n.js
import rosetta from 'rosetta'

const i18n = rosetta({ en: {...}, es: {...} })
i18n.locale('en')

export default i18n

// ...elsewhere
import i18n from './i18n.js'

const text = i18n.t('foo.bar', {}, 'it') // <--- No way of knowing, at this point, what language `text` is in

Another way of looking at this is exposing whether or not rosetta managed to find a matching property in the dictionary.
One common use case for this is setting a correct <html language="{{lang}}"> attribute value in an HTML template.

Hey, thanks~!
In your example, text would be in "en" because that was your default language set. If you call .locale() again without any arguments, it will always return your language default: https://github.com/lukeed/rosetta#rosettalocalelang
The reason for a "default" at all is that you're able to change the desired language per-t() call by setting a 3rd argument. Here's the relevant slice from the README example:
// Change default language key
i18n.locale('pt');

// Retrieve translations w/ new defaults
i18n.t('intro.text', data); //=> 'Espero que você ache isso útil.'
i18n.t('intro.text', data, 'en'); //=> 'I hope you find this useful.'

Rosetta does not auto-detect your users' languages for you. This is up to you and your application and should be tracked within your own application state. There are some recipes included in https://github.com/lukeed/rosetta/issues/1#issuecomment-613840436 for detecting/getting a user language.
Separately, rosetta offers a rosetta/debug submodule that you an import from during development which will log an error for any undefined/unknown language+token combination. See Debugging for more info.
Hope that all helps~

Hi @lukeed! Many thanks for your reply.
I'm fully aware of how rosetta works, I've used it extensively. I think you might have misunderstood my question.

In your example, text would be in "en" because that was your default language set.

Yes, that's obvious from reading the modules logic. The real issue here is that there's no built-in way of knowing that programatically. A consumer of the i18n object returned from the rosetta constructor is completely oblivious to the internal contents of the translations dictionary.

The reason for a "default" at all is that you're able to change the desired language per-t() call by setting a 3rd argument.

Exactly. This is demonstrated in the example from my OT. And the reason why I think rosetta is missing a kew feature.

Rosetta does not auto-detect your users' languages for you. This is up to you and your application and should be tracked within your own application state.

My concerns have little to do with auto-detecting a language. In my snippets I'm providing a known, hardcoded language. The problem still persists.

Separately, rosetta offers a rosetta/debug submodule that you an import from during development which will log an error for any undefined/unknown language+token combination.

This brings us closer to fill in the gap, but unfortunately that's not an actual solution as I'm sure you'd agree.
Could you please reconsider re-opening this? I know you didn't ask for it, but as a humble piece of feedback to you, it might be better if you could wait for users responses before deciding whether or not an issue should be closed :)

If I understood correctly, you’re asking for this:
i18n.t(“foo”, data);
//~> { text: string | void, lang: string }

which would also mean this:
i18n.t(“foo”, data, “en”);
//~> { text: “bar”, lang: “en” }

I’m sorry but this is pointless. You pass in a lang target directly, so echoing the value is of no use when the pooint of the library is to store a dictionary of translations so that you can get them. Accessors must always be known.
And the API is programmatic. And because it’s straightforward it’s easily composable. For example, this little utility does exactly as the above without inducing an unnecessary breaking change:
function t(ctx, token, data, lang) {
  lang = lang || ctx.locale();
  let value = ctx.t(token, data, lang);
  return { lang, value };
}

I think you might still be missing the point, given the examples you're providing. Let me elaborate a little bit.

If I understood correctly, you’re asking for this:

Doesn't have to be that specific API, but sure. That's kind of in the ballpark of what I think we are missing, yes.

I’m sorry but this is pointless.

This is a matter of personal preference, clearly - but I disagree. It's not only not pointless, it's kind of essential to be able to provide accurate results in case of missing locales.

You pass in a lang target directly, so echoing the value is of no use

You would not be echoing the value. That's the whole point of my request. You would be answering with the actual language being returned from the translation.

And the API is programmatic.

Not entirely sure what you meant by this - but I'm gonna put my finger on not having implied otherwise. I said that clients have no programmatic way of knowing the locale that the resulting texts are in. Never mentioned the whole API.

For example, this little utility does exactly as the above without inducing an unnecessary breaking change

It doesn't. And this is the reason why I'm inclined to say I'm not getting through with my questions.
function t(ctx, token, data, lang) {
  lang = lang || ctx.locale(); // <--- This will always return the value of `lang` unless falsy (useless)
  let value = ctx.t(token, data, lang); // <--- Still no way of knowing which language `value` is in
  return { lang, value } //<--- If the passed in `lang` was not among the translations, this result will be wrong
}

Does that make things clearer?

Sorry but I don't think you understand how i18n.t works. Here's its source:
t(key, params, lang) {
  var val = dlv(tree[lang || locale], key, '');
  if (typeof val === 'function') return val(params);
  if (typeof val === 'string') return tmpl(val, params);
  return val;
}

If you pass a lang, it is used. That's all that will be used, ignoring the locale fallback (via i18n.locale('<value>'). This means tree[lang || locale] always results in a language object with your translations and then dlv uses that object and your key identifier to find a value, if any. If there's no value, you will see that. If you did not set a lang, then the fallback is used.
In your final snippet, the returned lang will always be "it" because you passed it in. Always. If you did not pass anything in, it will always be the value from i18n.locale(), which is/was your last-defined fallback. If you never set a fallback, then the internal locale remains "" which means you'll never get a translation from i18n.t() unless you explicitly provide a lang value to the t() call.... this be because the internal lookup will evaluate to dlv(tree[undefined || ""], ...) which goes nowhere.
The utility provided does/will do what you've described. If not, then I'm sorry but your expectations are wrong. The lang = lang || ctx.locale() line is exactly the same logic that Rosetta is doing internally on every t() call, so yes, the returned lang value is always going to be correct & align with the value from that response, too. By extension – and by design – that means that you always know (or can always know) exactly what the current language is when asking for a translation.


docs: use templating to auto set version on usage

Description
Utilize Catalog feature for a template on the "usage" string that is displayed at consumption time in the UI.   No need for a new release with this PR.
Check the relevant boxes:

[ ] Bug fix (nonbreaking change that fixes an issue)
[ ] New feature (nonbreaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Examples or tests (addition or updates of examples or tests)
[ x] Documentation update
[ ] CI related update (pipeline, etc.)

Checklist

[ ] If relevant, a test for the change has been added or updated as part of this PR.
[ ] If relevant, documentation for the change has been added or updated as part of this PR.

Merge

Merge using "Squash and merge".
Make sure to use a relevant conventional commit message that is based on the PR contents. The commit message determines whether a new version of the modules needs to be released, and if so, which semver number to use).

More info on which fields within the usage are supported in a template may be found here https://github.ibm.com/ibmcloud/content-catalog/issues/3457 until doc for the Catalog json file is updated.

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

GitHub release
v2.0.0

Your semantic-release bot :package::rocket:


Refine example addon  with 1.5.0 new feature

Description of your changes

How has this code been tested?

Checklist

I have:

[ ] Title of the PR starts with type (e.g. [Addon] , [example] or [Doc]).
[ ] Updated/Added any relevant documentation and examples.
[ ] New addon should be put in experimental.
[ ] Update addon should modify the version in metadata.yaml to generate a new version.

Maybe we should move the example addons to the new dir.


chore(deps): [release-1.3] fix CVE-2024-55565

Description
Fixes CVE-2024-55565 by bumping nanoid to 3.3.8
Which issue(s) does this PR fix

Fixes RHIDP-5204

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

/lgtm
/approve

/retest


Upgrade all dependencies

Because the dependencies are very old, horrifyingly confusing error messages
occur at run time within any application that depends on reductive and uses
a more recent version of ReasonReact. In particular, the newer ReasonReact
in the application cosntructs a component whose shape differs from the
version of ReasonReact in reductive, leading to a bizarre error in the curry
module within BuckleScript.
Also fix the webpack config file formatting.

Also ran into that nasty curry error.


✨  Coffee and meal orders

Watch the idea - https://www.youtube.com/watch?v=qLeeRmg87GY&t=1210s
The pain
Ordering coffees and meals is a bit low-tech, even with Microsoft Forms
Suggestion

Figure: Attendees at Workshops and Superpowers can submit coffee and meal orders via the app
Allow people to submit their coffee and meal orders via the app during events like Superpowers and Workshops.

Hi @matt-goldman
Ordering a coffee is great...
But what if I'm an attendee and I want to ask someone to turn the air-conditioning down down?
Therefore I'd like this feature to be usable for any question
from
an attendee
to
the Event Master
-a

When running an event, presenters are usually under a time crunch and are delivering content which attendees have paid to attend.
My preference would be to keep using Microsoft Forms since the barrier to entry is practically zero. Scan a QR code, which opens a browser on a page, then the attendee fills out a form - nothing to install.
If someone doesn't have the app, then they may opt to miss out on coffee since they'd want to keep up with the workshop.
If this goes ahead, I think it needs to be accessible without a login and usable as an App Clip

If SSW.Rewards can run as a PWA app. I can see this as a good feature, but otherwise, it will make the morning coffee orders painful as people will need to download the app, authenticate, and order coffee.
Also, there will be a few people who will refuse to do that and a couple who might be resentful that they need to login and possibly give out personal data to get coffee.

This will evolve to be more helpful, but for internals I think it will make the lunch process easier.

Thumbs down for this, people who join the event will now need to install an app. Personally, i would not do that unless i consistently join event hosted by the same organization

Not sure we need this...

Agree with most people here, I don't think this feature can give more value


Autocomplete single object selection

Fixes #354
Added a check on cwrp handler to retrieve the query value if the multiple is false. This made the query state to be set as the value instead of the key so the value will be rendered and the key can be stored.
I also fixed the spacing in that file. It was originally indented with a single space and I noticed that the input.jsx file I worked on a few commits ago was indented with two spaces, so I made this match that.

Is there anything changed with the spacing??  Will push fix the conflict

If you want, I close this and open a new issue with only the fix and no space change

Ok!

@KerenChandran Did u forget to add query: this.query(nextProps.value) in componentWillReceiveProps to dev branch or I miss smth ?

@grammka Hey sorry for not replying earlier. 9d2fe8f has the change you requested.


add other builds in build_win.yml

like tools, examples, tests

Done by tools by https://github.com/thorvg/thorvg/pull/1383
Still it needs examples & tests.

https://github.com/thorvg/thorvg/issues/1244

See also:
https://github.com/thorvg/thorvg/issues/1605
https://github.com/thorvg/thorvg/issues/1244


Parallel Responses from Multiple Agents in GroupChat Using AutoGen

I am working on a use case where a moderator interacts with multiple participants in a group setting. I want to send a question from the moderator to all participants at once and have them respond in parallel.
However, when I send a message to the group, only one participant responds at a time. I want all participants to respond simultaneously to the moderator's question. How can I achieve this using AutoGen's GroupChat and GroupChatManager?
Here's my code snippet:
# Create Assistant Agents for Participants
Participant1 = autogen.AssistantAgent(name="Participant1", llm_config=llm_config)
Participant2 = autogen.AssistantAgent(name="Participant2", llm_config=llm_config)
Participant3 = autogen.AssistantAgent(name="Participant3", llm_config=llm_config)

# Set up group chat and group chat manager
groupchat = autogen.GroupChat(agents=[Participant1, Participant2, Participant3], messages=[], max_round=2)
participantPool = autogen.GroupChatManager(groupchat=groupchat, llm_config=llm_config, human_input_mode="ALWAYS", name="participantPool")

# Set up user proxy agent for Moderator
moderator_proxy = autogen.UserProxyAgent(name="Moderator")

# Question for all participants
task = """Question for all participants: Please share your opinions on the topic discussed."""
moderator_proxy.initiate_chat(participantPool, message=task)

Here's the Response:
Moderartor (to participantPool):

Question for all participants: Please share your opinions on the topic discussed

--------------------------------------------------------------------------------
Participant2 (to participantPool):

The discussion was enlightening, and it provided different perspectives on the matter. It's essential to have such conversations to foster a better understanding among us.

--------------------------------------------------------------------------------

For this, you'll need to modify the implementation of GroupChatManager. I can provide guidance for that path.

I am also interested in an implementation of parallel chat. It is similar to the architecture used in gpt-researcher here: https://github.com/assafelovic/gpt-researcher

i'm also interested in this, Anyway to achieve this?

Check #720 please. @joshkyh would like to chime in?

I think there are (at least) two ways to think about the OP's use-case.

Orchestrate it outside of autogen using threads
1a. import autogen
1b. Use threading and spin up one thread per agent.
1c. Have a poller to extract the outputs of each agent. The drawback is that I can't think of an easy way to show output of Agent B to Agent A. i.e., the agents lose context of the group chat.

Orchestrate it within autogen using graphs as a stepping stone #720.
2a. We can extend the notation in #720 so that we indicate tiers of nodes that can be run in parallel.
2b. However, I also can't think of an easy way to show output of Agent B to Agent A.

A couple of other pertinent considerations if we want to orchestrate it within autogen (option 2).

The user might start to need multiple APIs so as to avoid the max. tokens per minute limit when multiple agents start polling the same end point. Otherwise, the bottleneck becomes transferred max. tokens per minute limit after efforts to parallelize the agents.
The GroupChat framework will become a real-time framework rather than turn-based framework, it is a move that requires quite a lot of effort within autogen.

@geetansh Thanks for this idea. Do you have some working code (or PR) around this idea? If not, should we create one?
I have a basic (even naive) use case for this feature, but happy to discuss if you have a good motivating example.

We are closing this issue due to inactivity; please reopen if the problem persists.

this is what I have done to solve it:

create a "moderator agent"
create 4 tasks-specific agents
set my a reply_func to the "moderator agent" as follow:

        async def get_agent_replay(agent: ConversableAgent, msg: str) -> str | None:
        # loop max 4 times
        for i in range(4):
            await agent.a_send(
                message=msg,
                recipient=agent,
                request_reply=True,
                silent=False)
            latest = is_trend_agent.last_message()
            if latest is not None:
                if 'content' in latest:
                    content = latest['content']
                    if content is not None:
                            return content
        return None

    def reply_func(
            recipient: ConversableAgent,
            messages: Optional[List[Dict]] = None,
            sender: Optional[Agent] = None,
            config: Optional[Any] = None,
    ) -> Tuple[bool, Union[str, Dict, None]]:
        last_message = messages[-1] if len(messages) > 0 else None
        if last_message is not None and "content" in last_message:
            # taking only the last message
            content = last_message["content"]

            loop = asyncio.new_event_loop()
            t1 = loop.create_task(get_agent_replay(agent1,content))
            t2 = loop.create_task(get_agent_replay(agent2,content))
            t3 = loop.create_task(get_agent_replay(agent3,content))
             t4 = loop.create_task(get_agent_replay(agent4,content))
            results = loop.run_until_complete(asyncio.gather(t1, t2, t3, t4))
            return True, f"{results[0]}\n{results[1]}\n{results[2]\nresults[3]}"

        return False, None


Correct virtual offset with more that one buckets

This solve some errors when trying to inspect a BKTR section with more than one buckets.
PD: I'm sagumamugas (author of BKTR section stream) I lost access to the other account. I malinterpreted some offset thinking they are accumulative, but they aren't.
This PR should solve jakcron/nstool#98, jakcron/nstool#99 and jakcron/nstool#104

Welcome back! I'll admit I don't have much time for my hobby projects anymore. But I will make time to test and make a new release.
Thank you for finding this issue and resolving it 😁


Is tensorflow Lite support for detection, like SSD？

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug or a feature request.
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

Not currently, we will be adding support for more models in the future.
Currently supported models are Mobilenets, Inception V3 and Smart Reply.
Please look at the list of supported models here.
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/lite/g3doc/models.md


Press section: ensure all releases have the same height

To avoid this:

Images also have too big margins

Fixed


feat(pipelines/benchmarking): create a benchmarking pipeline

compare
gpt-3.5-turbo
gpt-4-turbo
(if I have the time to look into it) open-source model, llama, mistral
GPT + RAG with some "general materials/chemistry papers"
human baseline??
against each other. perhaps in a bar chart where we can show: "GlossaGen is super good and saving us time AND giving high quality output, this is great". (So the presentation could be: Short intro, short methods, showing some charts on benchmarking, Live Demo". Possibly, this is a moonshot and will take longer than 2 minutes)
For this, we would need some sort of "metric" of how good the output of the model is. We could also take "time to create" into account. Basically a number to state the same as what you wrote about the Dentistry Zeolithes output: "It's a convincing start and only takes 3 seconds to create, but it's flat-out wrong". How could we put this into a measurable metric?


Error while executing command adb devices

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[ ] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
OS: Windows 10
Node: 8.11.4
Yarn: Not Found
npm: 6.1.0
Watchman: Not Found
Xcode: N/A
Android Studio: Version  <IP_ADDRESS> AI-173.4907809
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: 0.55.3 => 0.55.3
Description
Trying to run a project which has been working fine yesterday, today when I started debugging the app for Android, the Packager is showing me this:
Metro Bundler ready.
[Info] Packager started.
[Info] Prewarming bundle cache. This may take a while ...
[Info] About to get: http://localhost:8081/index.bundle?platform=android
Loading dependency graph, done.
[Warning] The Bundle Cache was prewarmed.
[Info] Building and running application.
BUNDLE  [android, dev] ./index.js ▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓ 100.0% (1310/1310), done.
[Error] Error: Error while executing command 'F\:\Programs\Android\sdk\platform-tools\adb devices': Command failed: F\:\Programs\Android\sdk\platform-tools\adb devices
The filename, directory name, or volume label syntax is incorrect.
I see that there is an extra '\' in the directory for adb devices but I do not know how to change that. Also run -android shows me this
Could not find device with the id: "simulator".
Choose one of the following:
[ 'emulator-5554' ]
Could anyone please help me with this ?

The problem was with the Visual Studio Code React Native Tools extension. Check out this issue https://github.com/Microsoft/vscode-react-native/issues/781#issue-354774325 if you have encountered the problem.


Added Discord and other new logos. Updated many things.

Fixed Telegram alt.
Changed Cryptocointalk to Cryptocurrencytalk.
Changed IRC link to help channel. (don't need to be registered)
Changed image links to new images.
Added Discord.
Changed Steem alt to Steemit.
Decapitalized all alts.

Great work, hadn't spotted those errors!


Cell contentView size issue with iPhone X in landscape

When swiping a cell on an iPhone X in landscape, the cell content size width is changing causing the text in its label to be re-layouted.
It seems to be related to the safe area because the size of the content view changes by the safeAreaMargin value.
I created a gif with the demo project to illustrate this:

You can clearly see the you being moved to the second line. You can also notice that the border animation does not start from the edge of the screen like in the real Mail app.
Do you think this can be fixed easily?

The content view issue doesn’t happen in latest version and it’s only an example app issue from what I can see. Regarding boarder transition, I’ll need to check but is not a high priority on my list. Feel free to submit a PR.

I still have the contentView issue in my project and in the example app with v2.4.1. From what I understand, it seems to be only with multiline labels.
I looked at the code but didn't find anything obvious to would fix that as the contentView is never changed for the SwipeTableViewCell.

Weird. I will take a look.

If you make subject and body labels cell width equal to that of containing UIStackView, the movement issue goes away.  Also this occurs on simulator only from what I can see.

Thank you for the support. I'll try to change the constraint then.


generate caveats proto

WIP

@vroldanbet Made some progress. Got the updated generated code and the tests passing. I think there's subtle mismatch of dep versions between my workspace and CI which is causing the diff step to fail. Can you pull the latest on your machine and try running buf gen?

@samkim fixed the diff by setting the buf CLI version in authzed/actions/buf-generate. Also fixed grpc vuln reported by trivy.
Something I cannot seem to get to work is the following:

delete everything under authzed/api
regenerate with buf
everything except for the init files should be regenerated.  Could you try that out?

The files not being generated were from deprecated/removed apis. I deleted them and updated to the newer references.


[BUG] save_changes doesn't save dict data correctly when dictionary key has "." in it's value

Describe the bug
save_changes doesn't save data correctly when data is a dictionary and the key has "." in it's value
To Reproduce
from beanie import Document, init_beanie
from pydantic import HttpUrl
from motor.motor_asyncio import AsyncIOMotorClient
import os

class Data(Document):
    data : dict

    class Settings:
        use_state_management = True
        state_management_save_previous = True

MONGO_URI = os.environ.get("MONGO_URI", "mongodb://localhost:27017/")

async def config_db():
    # Create Motor client
    client = AsyncIOMotorClient(MONGO_URI)

    # Init beanie with the Product document class
    await init_beanie(
        database=client["test"],
        document_models=[
            Data,
        ],
    )

await config_db()

data = await Data(data={"https://www.google.com/foo": "bar"}).save()
data.data = {"https://www.google.com/hello": "world"}
await data.save_changes()

Expected behavior
A clear and concise description of what you expected to happen.
Data should be stored as
Data(id=ObjectId('...'), revision_id=None, data={'https://www.google.com/foo': 'bar', 'https://www.google.com/hello': 'world'})

However, beanie split the https://www.google.com/hello into nested level object and stored the data as
Data(id=ObjectId('...'), revision_id=None, data={'https://www.google.com/foo': 'bar', 'https://www/': {'google': {'com/hello': 'world'}}})

Additional context
Add any other context about the problem here.

Hi @yutongp ,
Sorry for the late reply. I'll fix it during the next bug-fixing session. Thank you for the catch

This is VERY weird bug. Thank you for the catch.
The problem is with the $set operator of MongoDB. It looks like it doesn't like dots.
You can reproduce it with:
await data.update({"$set": {"https://www.google.com/foo": "worlds"}})

I'd suggest applying a function to not include dots to keys. Like this:
    key = "https://www.google.com/foo".replace(".", "%2E")
    data = await Data(data={key: "bar"}).save()
    data.data = {key: "world"}
    await data.save_changes()


Reducing payloads with resources

What's this PR do?
This pull request takes advantage of Laravel API resources to help reduce the payload size of the JSON that is provided to the frontend by Inertia. Resources will allow us to define what model data we want to send over so we can only send what the UI actually needs.
How should this be reviewed?
👀
Any background context you want to provide?
I wanted to come back to address reducing the Inertia JSON payload size, because I imagine for some other pages, the payload could end up being pretty hefty. So I wanted to try out Laravel API resource to help address this early on, so it would be something we utilize moving forward!
Relevant tickets
References Pivotal #179016026.
Checklist

[ ] This PR has been added to the relevant Pivotal card.
[ ] Documentation added for new features/changed endpoints.
[ ] Added appropriate feature/unit tests.

This looks very similar to what we accomplish for other routes using Fractal transformers? Do you think this would be a new approach that would replace those eventually? What benefits (and potential downsides) are there from switching?

This looks very similar to what we accomplish for other routes using Fractal transformers. Do you think this would be a new approach that would replace those eventually? What benefits (and potential downsides) are there from switching?

I do think it could be a new approach that would eventually replace those, and figured trying it out on data that isn't available via the NS API would be alright.
It feels like a nice benefit that API resources are natively supported out of the box by Laravel, and generally less setup/pieces to need to manually define like we do in Fractal with items and collections there. I think the downside would be that API Resources may not be as customizable as Fractal Transformers, but some might debate that's a good thing, haha!
Given that API Resources are natively supported, there's no additional package to install and if we decide to switch this to using Fractal, we totally could and I think the lift would be minimal. Just curious to see how well API Resources could work for our needs and simplify our setup!

This PR is also evaluating the use of Eloquent API Resources for how we transform our model data into JSON responses. We want to determine and discuss in an upcoming RFC, whether we want to switch from our existing approach using Fractal, to using Laravel's native approach that was introduced in Laravel v5.5.


Update go version to 1.22.2

What this PR does / why we need it:
Update go version to 1.22.2
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

/kind enhancement

/cherry-pick rel-1.2.0

/lgtm

/test pull-dependency-watchdog-check-vulnerabilities

make check-vulnerabilities passes locally. I think we can merge this PR and once the pipeline is updated and uses go 1.22 the issue should go away.

/approve

/lgtm


[Suggestion] Create-Fresh-App like shop for Merchandise using Uniquery

We have efforts to provide custom store fronts for your NFTs and show up your on-chain things outside of KodaDot UI using Uniquery.

https://deno.land/
https://github.com/denoland/fresh
https://github.com/denoland/merch

Don't forget to list it here

https://github.com/denolib/awesome-deno

Feel free to do PR here

https://gituhub.com/vikiival/shop

Hello! might be

vuestorefront/vue-storefront

been considering vuestorefront as well, yet might be way too complex for starter
checking their repo, maybe storefront UI is more fitting? https://github.com/vuestorefront/storefront-ui


Amend to backward compatibility with compiler 1.3.3

Hi.
I have changed small piece of code to make available compile lime under 1.3.3 version of compiler, which is distributed with Ubuntu 15.04(64bit). I've tested it also in 1.4 version and seem okay.
This is my first pull request so please be understanding if I did something wrong.
Regards.

Coverage decreased (-0.16%) to 62.21% when pulling d94774cc90d306a59da13f4fd6f94cb7df48cb37 on aplause:master into 7eb4511afb7a55bcff27360aa045dc42d7f2cf77 on limetext:master.

Please reopen this PR in https://github.com/limetext/lime-termbox.


add analytics pings to newly integrated JB code

This would include:

local run / debug (probably include meta data to distinguish between these)
new project wizard

I can't think of anything else.

Maybe adding the facet? Can we track that?
On Wed, Aug 24, 2016 at 4:19 PM, Etan Shaul<EMAIL_ADDRESS>wrote:

This would include:

local run / debug (probably include meta data to distinguish between
these)
new project wizard

I can't think of anything else.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-intellij/issues/842#issuecomment-242194623,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HbmJ9-DsYfnrK42UOzYhodFbewIrks5qjKdPgaJpZM4JsZ1J
.

Yes, but it might make more sense to move it up one level to adding framework support (which adds the facet itself). But it is possible to add the facet without framework support, so maybe it should track this too (with proper deduplication).


Close InputStream after reading it

@dmlloyd Can you review it?

@dmlloyd PR updated, please review. I wasn't sure about the code formatter used, so I used the Java Conventions (Built-in) in Eclipse


SSR: window not defined

First issue on this library and it's very cool!
Sever Side Rendering fails when serving a module that contains an ngrx-form.  The NgrxDefaultViewAdapter is making a special request for device testing where it uses window.navigator.
Error:
window is not defined
    at new NgrxDefaultViewAdapter

Reference
https://github.com/MrWolfZ/ngrx-forms/blob/master/src/view-adapter/default.ts#L65
Steps to Reproduce
Create an Angular app with NGRX and ngrx-froms.  Render a page with SSR that has a ngrx-form.

A good solution might be if we could do a isBrowser check before this or something.
import { PLATFORM_ID, Inject } from '@angular/core';
import { isPlatformBrowser} from '@angular/common';
// ...
export class NgrxDefaultViewAdapter ... {
/// ...
    testBrowser: boolean;
    constructor(
        // ...
        @Inject(PLATFORM_ID) platformId: string) {
            this.testBrowser = isPlatformBrowser(platformId)
            if (this.testBrowser) {
                // this is only executed on the browser
            }
    }
...


Possible to graph out number of requests over time on a per second/minute resolution?

I'm looking for a way to graph out the numbers of requests over time
i.e x-axis: time, y-axis: request count per second
Bonus if its possible for each route to be an individual series
Is this possible?

Essentially a time-series report?

shameless plug if there's any interest in embedding an interactive multi-timeseries in the html report: https://github.com/leeoniya/uPlot
166,650 point demo: https://leeoniya.github.io/uPlot/bench/uPlot.html

@allinurl yes, exactly! Currently I can see the number of request over days. I personally use this tool to view requests in real time and would love to see a time series of the routes by the second/minute


Missing vision models

@henry090 I just realized that some models are missing from fastai :
https://docs.fast.ai/vision.models.xresnet.html
Namely all the models such as xse_resnext50.
I could add them too if you don't mind.

That would be great!


add base concept of entities to signals

Right now, the only type of Entity implemented will recursively search a signal for a entity that matches a regular expression (as opposed to specify, say the field/key name, where that entity should be). I will end up implementing both as an option. There is also the concept of a "global" entity, where we will search all signals, recursively, for that entity.
We should prioritize "time-to-first-render" and not let entity correlation be a bottleneck, although I expect this will still be fairly performant, all things considered. Only after a signal/case has been fully processed, will we correlate entities with other cases and signals. We can use UI tricks to try to make this invisible to the user and our UI is already set up well for this, by putting information with tabs on the CaseEditSheet.
Case View Mockup

Global entity types are not implemented. I will remove from front-end, but keep it in the model.

lgtm, some small suggestions

thanks for looking at it


Class 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider' not found

Hi. I've tried to find my mistake, but I can't. I've been following the "Automatic phpDoc generation for Laravel Facades" guide, but i fails.
Running MAMP PRO with php 5.4.10.
After adding 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider', to the app/config/app.php this error is appearing:
Symfony \ Component \ Debug \ Exception \ FatalErrorException
Class 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider' not found in
/Users/x/Sites/laravel/todo/vendor/laravel/framework/src/Illuminate/Foundation/ProviderRepository.php line 158
If I run the ``php artisan ide-helper:generate``` it generates this error
{"error":{"type":"Symfony\\Component\\Debug\\Exception\\FatalErrorException","message":"Class 'Barryvdh\\LaravelIdeHelper\\IdeHelperServiceProvider' not found","file":"\/Users\/michaelsoerensen\/Sites\/laravel\/todo\/vendor\/laravel\/framework\/src\/Illuminate\/Foundation\/ProviderRepository.php","line":158}}{"error":{"type":"Symfony\\Component\\Debug\\Excep [something more I can't see....)````

The php error log have this:
```PHP Fatal error:  Class 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider' not found in /Users/michaelsoerensen/Sites/laravel/todo/vendor/laravel/framework/src/Illuminate/Foundation/ProviderRepository.php on line 158```

Any help is much much appreciated

Thanks :-)

Hi, I found this thread by googling "Class 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider::class' not found". It was my real problem which I was unable to solve. I checked my composer.json file twice, it was ok, but error still occured. Right before uninstalling this package I decided to try to remove "::class" from provider definition in app.php. And it works!
Documentation says:

After updating composer, add the service provider to the providers array in config/app.php
Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider::class,

And it didn't work for me. When I replaced it to

Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider

it works fine. And IDE recognizes all classes correctly now.
I hope it helps to somebody with similar problem.

Did you put quotes around Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider::class? That would break it.

Yes, of course )

Hmm, I didn't even thought that it may work without quotes, so I wrapped it without hesitation. Thank you for hint.

::class is a Scope Resolution Operator. http://php.net/manual/en/language.oop5.paamayim-nekudotayim.php It returns the class which returns the same string as the quoted part.

Yes, I know about ::, but combination with class keyword confuses me.

Well done, thanks guys

I have this problem I update my composer.json and add Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider::class, in my app.php in providers array but when run php artisan ide-helper:generate command I got this error " Class 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider' not found " is any thing that I did not check?

zahraMHD
Class 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider' is used to enable PHPStorm to find classes. When you have “use \Request;” in your php file, PHPStorm cannot find the source file this refers to. Google/search Laravel Ide Helper in github and you will see how 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider' can be used.
Before you try to deploy your code, comment out 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider' in composer.json.
Hope this helps
Paul
From: zahraMHD<EMAIL_ADDRESS>Sent: June-08-16 6:28 PM
To: barryvdh/laravel-ide-helper
Cc: Paul Neal; Comment
Subject: Re: [barryvdh/laravel-ide-helper] Class 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider' not found (#55)
I have this problem I update my composer.json and add Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider::class, in my app.php in providers array but when run php artisan ide-helper:generate command I got this error " Class 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider' not found " is any thing that I did not check?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/barryvdh/laravel-ide-helper/issues/55#issuecomment-224749436 , or mute the thread https://github.com/notifications/unsubscribe/AJApOE4dEwz0uRTmd8YS1IUn2cuwitV7ks5qJ0HVgaJpZM4Be-Y_ .  https://github.com/notifications/beacon/AJApOLHak1Jl_LBwyg7gx25HdwsS7Frhks5qJ0HVgaJpZM4Be-Y_.gif

pneal-vital Saving my job 363 Days after his post

Hi, I am new in Laravel. I am using Netbeans 10 on Windows 10 and xampp. I dont have a folder vendor/barryvdh/laravel-ide-helper, from where could I download it or ...

my case had APP_ENV=prod instead of APP_ENV=production

This might also happen if you install composer require dependencies only and omit require-dev dependencies, as might be configured for production environments:
composer install --no-dev

In that case, the package isn't installed during runtime, and the class reference Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider::class can't be resolved to a class, hence the error is thrown.
If you would like to keep it this way, or can't influence it, you should follow the advice above and use the string variant of the class name ('Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider ') instead, even if it feels dirty :)

In my case was because the package was in require-dev and in my AppServiceProvider i had
if ($this->app->isLocal()) {             $this->app->register(\Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider::class);         }
And in composer.json i had
"post-update-cmd": [       "Illuminate\\Foundation\\ComposerScripts::postUpdate",       "@php artisan ide-helper:generate",       "@php artisan ide-helper:meta"     ]
So when i run any deploy, the post-update-cmd was triggering, and raising a error


PR for RocksDB being ignored :-(

Can someone please please take a look at https://github.com/brianfrankcooper/YCSB/pull/1052

https://github.com/brianfrankcooper/YCSB/pull/1051#issuecomment-376290805

That said I added a quick "braindump" review 👍  I encourage you to do the same for other PRs to ease the burden on manolama and busbey

as soon as I get a staging branch cut for 0.14.0 I'll start working in merges for new stuff.  right now I see three PRs for RocksDB: #876, #1052, and #1075. if the three authors could come to a consensus position on what to include it'll make things go much faster.

@busbey PR #1052 should replace #876. I had a quick look at #1075 and it seems to be for several things - MongoDB, LevelDB and RocksDB. The RocksDB stuff in #1075 seems to be okay but doesn't account for column families or have support for ensuring a single instance.

fixed by #1052


Remove cmd helper functions

What type of PR is this?
/kind code-refactoring
What does this PR do / why we need it:
This PR will remove old helper functions mentioned below

CmdShouldRunAndTerminate
CmdShouldFail
CmdShouldPass
CmdShouldFailWithRetry
CmdShouldRunWithTimeout

Which issue(s) this PR fixes:
Fixes #4517
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[ ] Update changelog

[x] I have read the test guidelines

How to test changes / Special notes to the reviewer:

@anandrkskd can you PTAL if the test failures are related to this PR? I'm inclined to approve & lgtm this but don't want to create an infinite loop.

/approve

@anandrkskd can you PTAL if the test failures are related to this PR? I'm inclined to approve & lgtm this but don't want to create an infinite loop.

Yeah I am looked into the previous one there was a typo already updated it. looking into k8s failure.

/test v4.7-integration-e2e
/test psi-kubernetes-integration-e2e

prow failure on v4.7-integration-e2e
/test v4.7-integration-e2e

/lgtm


nc on busybox still causes problems

We want to set NCFLAGS="-w 5 -q 1 -i 1" on busybox, but OpenWRT has:
root@OpenWrt:/overlay/RSTunnel# nc --help
BusyBox v1.24.2 () multi-call binary.

Usage: nc [IPADDR PORT]

Open a pipe to IP:PORT

Fixed 095ef97.


Figure out how to handle the rename of MTRAttributeCacheContainer to MTRClusterStateCacheContainer compatibly

We renamed MTRAttributeCacheContainer to MTRClusterStateCacheContainer; need to figure out how to make code compiled against the old headers work on tip.

Done in #23639


Implicit operators would help with option types.

Create implicit operator overloads.
    public static implicit operator Option<T>(T? t) =>
        new(t);

This will allow returning a type directly and it will be converted to an Option automatically.


[FEATURE] Support detailed logging of render function when live editing Hugo site

One of CloudCannon's distinct features it its support for live editing. However, configuring bookshop components that are compatible with live editing can sometimes be quite tricky. Although CloudCannon does give some visual feedback when the site configuration is incorrect (e.g. a partial is malfunctioning or an extra file cannot be found), not all required debugging information is available. In several cases, the live editing just results in a blank page (usually after a timeout) without providing any feedback.
To improve the development experience, it would be great to have access to more detailed debugging information. The current build pipeline generates a file bookshop-live.js when building a Hugo site. This JavaScript includes a function render() that invokes the Hugo wasm. It appears the array window.hugo_wasm_logging contains detailed information about the live page build. However, currently this information is discarded entirely.
It would be great to somehow expose the build information contained in window.hugo_wasm_logging. As a workaround, I usually set a breakpoint using Chrome dev tools on the line that clears window.hugo_wasm_logging (see the code snippet below). However, this is quite tedious and error prone.
async render(target, name, props, globals, logger) {

  […]

  while (buildError && render_attempts < 5) {
    if (this.componentQuack(buildError, window.hugo_wasm_logging) === null) {
    break;
  }
  window.hugo_wasm_logging = []; // set a breakpoint on this line to view detailed build logs
  buildError = window.buildHugo();
  render_attempts += 1;
  }

  […]
}

Shipped in v3.11.0 🙂
If there's any error Bookshop will re-log that content to the browser console.


ALL JOBS WONT DISPLAY AND CANT EDIT / CREATE GRADES AND CAN'T ALLOW CERTAIN THINGS LIKE CUFFING OR BILLING

ALL JOBS WONT DISPLAY AND CANT EDIT / CREATE GRADES AND CAN'T ALLOW CERTAIN THINGS LIKE CUFFING OR BILLING

Nowhere is it absolutely stated that it should create new grads and jobs.
If you want, you can find how to create a grade directly with a job on doc.polisek.io documentation.
Nowhere in the video or in the description of the script is it stated that it should create a job in the database or possibly at the QB in the config.
For billing use your own script via export or the same as bossmenu.

I use esx

just one job


Make it possible to disable ingress/route

If you deploy a grafana with an Ingress set in the spec and then remove Ingress from the spec, Ingress object remains in the cluster.
A dangling ingress is not necessarily a big issue, because it's very unlikely for someone to specify ingress and then delete it. Though, if it's something that doesn't require much effort to fix, it'd be nice to do so.

How do we want to handle the creation of ingress/route? One solution would be to just say that if ingress/route is empty don't create a ingress or route, which seems like the easiest way.
But the bad thing here is that the OCP users that is currently used to have a route auto created for them won't get that fuctionallity.
Personally I think it's better that we have a consistent way of working and I think route and ingress should work the same way. So my suggestion is to have ingress/route disabled by default.
Another option is of course to keep the current way of working but we add some kind of ingress_enabled resource but it's kind of ugly and you could still have ingress config defined and have  ingress_enabled: false and it would still be disabled which is a rather strange thing from a user point of view.
This would be good to fix before rc0.


Update certificate-authorities.md

sharing the solution for the LE rate limiting

هنگام بازدید از www.okx.com/ با خطا مواجه شدم.
کد خطا: 1020
شناسه اشعه: 83c60f6a8e3f5d5b
کشور: IR
مرکز داده: fra14
IP: 2a01:5ec0:9021:889c:8c64:100b:663d:368f
مهر زمانی: 2023-12-28 01:34:19 UTC

Thanks for your suggestion! Since this is a reference page, adding that solution would be out of context - not every customer will necessarily find a problem with this limitation. As discussed before, PCX-9202 will be adding further information on this to a troubleshooting page once the related error messages are also surfaced to users.


remove deprecated behavior of touchableHandleActivePressIn/Out called for focus/blur

Follow up to #18470, which has not yet landed in a release so we may want to hold off merging this until it has so that people receive the deprecation warning.

This PR removes the previously deprecated behavior of touchableHandleActivePressIn/Out (as well as onPressIn/Out being called on TV platforms for focus/blur. Instead, users should utilize the new onFocus and onBlur for these events.
This was because on TV platforms onPressIn/Out was overloaded to trigger for not only presses but these focus events as well. The normal behavior of true presses will still defer to touchableHandleActivePressIn/Out (which defers to onPressIn/Out).
Related PRs
#18470
Release Notes

[TV] [BREAKING] [Touchable] - On TV platforms onPressIn and onPressOut on Touchables will now only be triggered for press events, not for focus/blur. Instead, you can use onFocus and onBlur.

Cc/ @matthargett

Warnings

:warning:

:clipboard: Test Plan - This PR appears to be missing a Test Plan.

  Generated by :no_entry_sign: dangerJS


WIP: Add group level fNIRS waveform analysis

Codecov Report

Merging #267 (e722ad3) into master (b40fa41) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #267   +/-   ##
=======================================
  Coverage   97.55%   97.55%
=======================================
  Files          41       41
  Lines        1104     1104
  Branches      157      157
=======================================
  Hits         1077     1077
  Misses         11       11
  Partials       16       16

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b40fa41...e722ad3. Read the comment docs.


feat: added timestamp conversion feature

What I did
Added TimestampConversions class to _conversions and added to init.py under plugins

fixes: #
How I did it
Added the feature, wanted some feedback, have to think of other instances that this may fail
How to verify it
Testing and help from friends
Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

Tests failed, I wasn't sure I should add this the the init.py file. And there are still some other cases I wanted to work through.

What do we do about this linting issue?

don't forget to run pre-commit before your commits

I forgot pre-commit every time. I'll add that to the notes, sorry about that

I forgot pre-commit every time. I'll add that to the notes, sorry about that

You can make it happen automatically via pre-commit install in the repo