Tag management is merging some words

I think this problem was already there with older MMT 3.
When I'm translating this sentence (MS Word XML with tags renamed by my process):
MMT befxml : <LbxTag001>Par<LbxTag002>nature ,<LbxTag003>l' atteinte<LbxTag004>de<LbxTag005>ces<LbxTag006>objectifs est<LbxTag007>soumise<LbxTag008>à<LbxTag009>de<LbxTag010>nombreux<LbxTag011>risques<LbxTag012>et<LbxTag013>incertitudes<LbxTag014>susceptibles.<LbxTag015>
The translation shown in the log file is correct:
   translation = By nature , the achievement of these objectives is subject to many risks and uncertainties that may arise .
But, the final translation with tags is merging 'theseobjectives':
{"status":200,"data":{"translation":"<LbxTag001>By<LbxTag002>nature<LbxTag003>, the<LbxTag004><LbxTag005><LbxTag006> achievement of theseobjectives<LbxTag007><LbxTag008> issubject to<LbxTag009><LbxTag010> many<LbxTag011>risks<LbxTag012>and<LbxTag013>uncertainties<LbxTag014>that may arise.<LbxTag015>","sourceWordCount":21,"targetWordCount":19,"contextVector":{"entries":[{"memory":{"id":1,"name":"USERS.fr-en"},"score":0.09509246}]},"priority":"normal","totalTime":316,"memoryLookupTime":2,"decodingTime":308,"queueTime":0,"queueLength":0}}
PS: updated to show exactly what was sent to MMT

Forget this: I see that the merging was already in the original FR sentence...

After fixing my own code, the problem is still there.
Sent to MMT4:
<LbxTag001>Par<LbxTag002> nature ,<LbxTag003> l' atteinte<LbxTag004> de<LbxTag005> ces<LbxTag006> objectifs est<LbxTag007> soumise<LbxTag008> à<LbxTag009> de<LbxTag010> nombreux<LbxTag011> risques<LbxTag012> et<LbxTag013> incertitudes<LbxTag014> susceptibles.<LbxTag015>
MMT4 LOG (correct):
   translation = By nature , the achievement of these objectives is subject to many risks and uncertainties that may arise .
Final MMT4 reply(word mergin on 'theseobjectives'):
{"status":200,"data":{"translation":"<LbxTag001>By<LbxTag002> nature<LbxTag003> , the<LbxTag004><LbxTag005><LbxTag006> achievement of theseobjectives<LbxTag007><LbxTag008> issubject to<LbxTag009><LbxTag010> many<LbxTag011> risks<LbxTag012> and<LbxTag013> uncertainties<LbxTag014> that may arise.<LbxTag015>","sourceWordCount":21,"targetWordCount":19,"contextVector":{"entries":[{"memory":{"id":1,"name":"USERS.fr-en"},"score":0.09509246}]},"priority":"normal","totalTime":546,"memoryLookupTime":22,"decodingTime":336,"queueTime":0,"queueLength":0}}

@EtienneAb3d
we will investigate soon.
in the node log (if you start the engine with the parameter "-v 2",
you should find some additional info related to the actual translation computed by ModernMT
similar to
sentence = Payment
   translation = Pagamento
   alignment = 0-0
   suggestions = [
]

For the "problematic" sentence, could you please report such info.
It is correct that tags are not reported there.

Here it is (I added a code at the beginning of the source text, see #452).
The bad sentence found in the memory is due to the history of translation management, using a previous bad translation made by MMT. But, as you can see, this bad memory entry doesn't produce the bad translation result. The translation result before tag processing is correct.
   sentence = # A # Par nature , l' atteinte de ces objectifs est soumise à de nombreux risques et incertitudes susceptibles .
   translation = By nature , the achievement of these objectives is subject to many risks and uncertainties that may arise .
   alignment = 0-10 0-14 0-18 1-10 1-14 1-18 2-10 2-14 2-18 3-0 4-1 6-4 7-2 7-3 7-4 9-6 10-4 10-7 11-8 12-8 12-9 13-8 15-11 16-12 17-13 18-14 19-15 20-17
   suggestions = [
      ScoreEntry{memory=1, language=fr \u2192 en, sentence=[#, G, #, Par, nature, ,, l', atteinte, de, ces, objectifs, est, soumise, à, de, nombreux, risques, et, incertitudes, susceptibles], translation=[By, nature, ,, the, achievement, of, theseobjectives, issubject, to, many, risks, and, uncertainties, that, may, arise], score=0.48773745}
   ]

As said, I think the problem was already in MMT 3.
Here is a comparison: left column was translated with MMT 3, while right column was translated with MMT 4.

@EtienneAb3d
we will check asap

@EtienneAb3d
are you sure that you reported the log snipper corresponding to the problematic sentence?
you reported
sentence = # A # Par nature , l' atteinte de ces objectifs est soumise à de nombreux risques et incertitudes susceptibles .

but actually in your input
MMT befxml : <LbxTag001>Par<LbxTag002>nature ,<LbxTag003>l' atteinte<LbxTag004>de<LbxTag005>ces<LbxTag006>objectifs est<LbxTag007>soumise<LbxTag008>à<LbxTag009>de<LbxTag010>nombreux<LbxTag011>risques<LbxTag012>et<LbxTag013>incertitudes<LbxTag014>susceptibles.<LbxTag015>

the text # A # is missing.
Could you please double check?
If you translate the same sentence without Lbxtags, which translation and alignment did you get?

The "# A #" is added just before sending to MMT. The source sentence I provided without it is just before this add.

@EtienneAb3d
I just finished to improve the tag projection.
According to my debugging, all problems you experienced are now solved.
But before pushing the change in the master, I would kindly ask that you double-check as well.
So, could you please pull the branch "features/improving_tag_projection", using this branch test on your data, and report to me.
I am sure you know how to setup and compile a branch different from the master.
Otherwise please contact me privately.

@EtienneAb3d
I have not changed the regression unit test yet.
Please, exclude the testing suite from the mvn compilation using the parameter -Dmaven.test.skip=true.
Basically, you can run
mvn -Dmaven.test.skip=true clean install

OK ! On a test sentence, this seems to solve the problem.
:)

@EtienneAb3d
If you would like to verify that the new version is better on a larger test set, you are welcome.

Bad news, I got 2 problems:

many strange ">" added. Example:

Sent to MMT4:
http://localhost:10001/translate?q=%23E%23+%3CLbxTag001%3ERecherche%3CLbxTag002%3E+et%3CLbxTag003%3E+innovation+%3CLbxTag004%3E33.%3CLbxTag005%3E&source=fr&target=en&context=assembl%C3%83%C2%A9e+AG+personnel+gestion+comptabilit%C3%83%C2%A9
MMT4 LOG:
2019-10-21 15:19:22,140 DEBUG eu.modernmt.decoder.neural.NeuralDecoder [pool-2-thread-4] Translation received from neural decoder:
   sentence = # E # Recherche et innovation 33 .
   translation = Research and innovation 33 .
   alignment = 0-4 1-4 2-4 3-0 4-1 5-2 6-3 7-4
   suggestions = [
   ]

Result (see "innovation>"):
{"status":200,"data":{"translation":"<LbxTag001>Research<LbxTag002> and<LbxTag003> innovation><LbxTag004>33.<LbxTag005>","sourceWordCount":8,"targetWordCount":5,"contextVector":{"entries":[]},"priority":"normal","totalTime":74,"memoryLookupTime":0,"decodingTime":66,"queueTime":0,"queueLength":0}}

2) still one merged word

Sent to MMT4:
`http://localhost:10001/translate?q=%23C%23+%3CLbxTag001%3Ed%C3%A9mocratique+%3CLbxTag002%3Edu+Congo.%3CLbxTag003%3E&source=fr&target=en&context=assembl%C3%83%C2%A9e+AG+personnel+gestion+comptabilit%C3%83%C2%A9`

MMT4 LOG:

sentence = # C # démocratique du Congo .
translation = Democratic Republic of the Congo .
alignment = 0-1 0-2 0-4 1-1 1-2 1-4 2-1 2-2 2-4 3-0 5-3 6-5
suggestions = [
]
Result (see "Republicof"):
{"status":200,"data":{"translation":"Democratic Republicof the Congo.","sourceWordCount":7,"targetWordCount":6,"contextVector":{"entries":[]},"priority":"normal","totalTime":93,"memoryLookupTime":0,"decodingTime":85,"queueTime":0,"queueLength":0}}

@EtienneAb3d
I will debug soon.
Could you please confirm that the input strings are
#E# <LbxTag001>Recherche<LbxTag002> et<LbxTag003> innovation <LbxTag004>33.<LbxTag005>
#C# <LbxTag001>démocratique <LbxTag002>du Congo.<LbxTag003>

Yes.
But, I find something more strange with the second example : words are merge in both case, with or without entry tags. I mean, the LOG provides with the good (not merged) translation, with or without entry tags, but words are merged even when no tag is provided in the sentence to be translated. This is observed with both the official MMT4 and the new patch.

@EtienneAb3d
the way ModernMT handles and re-insert spaces and tags is strictly related to the alignment between the tokens.
And in  your second case the alignment is not very good.
By the way, having these two more examples, I should be able to debug and solve some special cases.
One more point.
I noticed that in all your examples your input start with a string like #E# but they are not produced in the translations. This suggests that in your training data you have such string in the source part, but not in the target.
ModernMT works fine even in this case, but it is not advisable to have such differences in the training data.
So if you add on your own that string, I would suggest to add it on both source and target sides.

You are right: the training data are all with source marked and target not marked.

The reason is that, I would like the model to learn this mark as a global influencer of the result. My intuition, is that, marking both sides could bring me with a different learning, where a small part of the network will quickly learn to copy this mark, while the whole network won't take it into account any more since no more included in the error adaptation.

@EtienneAb3d
Regarding the first problem (innovation>).
I just committed a fix.
Regarding the second problem (republicof)
As said, our policy to re-insert spaces and tags is strongly based on the word-alignment between source and target words. In this case the WA is quite bad due to the presence of the "additional" item #C# which actually corresponds to three tokens #, C, and #.
If you still want to use a marker I would suggest to use a different one which is not tokenized, like for instance MARK-C.
At the moment we cannot modify this behaviour, because it would require a full re-structuring of our post-processing policy.
But surely we will take into account this issue in the future.

For the second problem, what I do not understand is why you are removing spaces. The translation is good before tag processing. Inserting tag should not change the spaces.

@EtienneAb3d
the problem is not related to the tags, but to the alignment.
In fact the same issue arise without tags as you observed.
Practically, the two target tokens republic and of are aligned (wrongly) with the two source tokens C and #, respectively.
As we trust on the original spacing between C and #, which  is not present, we do not put any space between republic and of.
This policy works almost perfectly for handling the spacing before and after punctuation and tags, and for all other tokens when alignment is good.

Except for very few word mergings, the new update seems good now. It's a little bit better than what I got with MMT3. But, this is possibly due to my new tuning code addition : there are less lost words, and this is exactly the reason why it is done (see #452).

@EtienneAb3d
good to know
If you think it is fine, I would kindly ask you to close this issue.
Please feel free to open a new one if needed in the future.

You need to commit the new change in the official delivery. Don't forget to tune the test step in the install script.

@nicolabertoldi  I am sorry for posting here again. I am wondering if this is already merged in the official repo?


services.sh e2e test is busted with Vagrant

It looks like the network is broken in some way that is non-obvious.  I spent most of a day digging in and without deeper OVS knowledge, I'm not sure what is going on.
Perhaps we want to switch to flannel here for networking?  It'd be simpler to set up.
I'm going to disable the test for vagrant as part of #3530.

I ran into this same issue with the spanning-tree propagation delay in OVS.
On Tue, Jan 20, 2015 at 4:42 PM, Joe Beda<EMAIL_ADDRESS>wrote:

It looks like the network is broken in some way that is non-obvious. I
spent most of a day digging in and without deeper OVS knowledge, I'm not
sure what is going on.
Perhaps we want to switch to flannel here for networking? It'd be simpler
to set up.
I'm going to disable the test for vagrant as part of #3530
https://github.com/GoogleCloudPlatform/kubernetes/issues/3530.
--
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/3655.

@smarterclayton - is Mrunal the right person to help here?
Sent from my iPhone

On Jan 20, 2015, at 7:50 PM, Tim Hockin<EMAIL_ADDRESS>wrote:
I ran into this same issue with the spanning-tree propagation delay in OVS.
On Tue, Jan 20, 2015 at 4:42 PM, Joe Beda<EMAIL_ADDRESS>wrote:

It looks like the network is broken in some way that is non-obvious. I
spent most of a day digging in and without deeper OVS knowledge, I'm not
sure what is going on.
Perhaps we want to switch to flannel here for networking? It'd be simpler
to set up.
I'm going to disable the test for vagrant as part of #3530
https://github.com/GoogleCloudPlatform/kubernetes/issues/3530.
--
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/3655.

—
Reply to this email directly or view it on GitHub.

Yes, or @rajatchopra

On Jan 20, 2015, at 11:34 PM, Derek Carr<EMAIL_ADDRESS>wrote:
@smarterclayton - is Mrunal the right person to help here?
Sent from my iPhone

On Jan 20, 2015, at 7:50 PM, Tim Hockin<EMAIL_ADDRESS>wrote:
I ran into this same issue with the spanning-tree propagation delay in OVS.
On Tue, Jan 20, 2015 at 4:42 PM, Joe Beda<EMAIL_ADDRESS>wrote:

It looks like the network is broken in some way that is non-obvious. I
spent most of a day digging in and without deeper OVS knowledge, I'm not
sure what is going on.
Perhaps we want to switch to flannel here for networking? It'd be simpler
to set up.
I'm going to disable the test for vagrant as part of #3530
https://github.com/GoogleCloudPlatform/kubernetes/issues/3530.
--
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/3655.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub.

This issue should be closed.  It was resolved.

Thanks!


Snippet for page part doesnt fill applicationarea property

Describe the bug
I think that title is clear enough. While you are creating pages, parts are not filled with the ApplicationArea property, and the subpages are not shown. I think this should be solved in order to avoid errors.
Versions:

AL Language: 3.0.145991

Thanks, @frivadulla ! We'll fix this in the next major release of the extension.

I see no warning or error, as is done when ApplicationArea is missing on a page field.

The rule is already enforced. Is either the AppSource Cop or PTE Cop really active?

We will add the ApplicationArea to the snippet for the next release because the property is there for the page field snippet, but on the part snippet. However, when enabling AppSourceCop or PTECop, you should be receive a warning that the ApplicationArea must be filled in.

Note that from version 11.0, we are also using the ApplicationArea defined on the page level as a default for the application area of controls/actions which do not explicitly specify one. Since most items are addressed already, I am closing this issue already.


[WIP] Active anomaly detection

Please add some docs to aadforest.py

Why it is still WIP?

It is definitely not WIP. It was already done in march.


Latest APIM LZ updates with GenAI capabilities

Updates to reference architectures on APIM secure baseline and added GenAI Gateway capabilities scenario.

@aarthiem,
This PR was opened in the public repo. To submit updates to Cloud Adoption Framework content, you need to open a PR in the private repo, cloud-adoption-framework-pr.
New content proposals must be submitted via this form: http://aka.ms/Contribution. This process ensures your content gets the proper attention and quality checks.
If you don't have access to the private repo yet, follow the instructions under Join GitHub organizations in the Get started contributing with Microsoft Learn Editor article. Then you can use the Learn Editor to edit the article. Double-check that the repo name ends with -pr.
You can open a new PR and close this one, or you can move the commits from this PR into the private repo so you don't have to repeat work.
If you need help moving these commits to the private repo or if you have questions, email the publicprs alias.
I'll close this PR now.

These commits have been moved to the private repo: https://github.com/MicrosoftDocs/cloud-adoption-framework-pr/pull/5963


feat: Merging maps and places demo internship project into maps-sdk-for-ios-samples repository

Maps and Places Demo Application
This demo application showcases some of the features that are found in both Google Maps and Google Places APIs as well as some new features that incorporate both APIs simultaneously. For example, in the Google Places demo application, there is a feature to get an image for a location. In this new demo application, you can search for a location and a picture of that location will show up on the map.  This app uses Material Design Components to improve the user experience. There are MDC Floating Action buttons to zoom in/out, go to the user's current location, and bring up an information view (which is a MDC Card). There is also a MDC Action Menu where the use can select which features and toggles they want to showcase.  A lot of the features found in this application built upon the features found in the existing demo applications. Almost all the features here use both APIs in some capacity. There are also a couple of new features that I thought could potentially be interesting for future developers. Furthermore, the user-oriented interface may help inspire developers as to what their application could look like with Material Design elements.  This demo application was written primarily in Swift.

Thanks so much!!!


CD broken

@Evylon You have to change the workflow permission to "read and write" here: https://github.com/Evylon/HamsterList-Native/settings/actions
Otherwise GitHub actions can not create new releases.

Done


feat: convert project to typescript

Closes #41

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Removed dependency on arrow functions to restore Node<4 compatibility

closes #1113

thank you thank you thank you thank you thank you thank you!


Bump jq to v1.6

Install jq from the OS repository, so that the integrity of the binary can be ensured with apt. This also bumps the version of jq to v1.6, it was previously pulling in v1.4.

@lboynton thanks for the PR - I can get this merged if curl is removed.

Updated


EmbeddedChannel instance should not share the same ID

In current implementation, all EmbeddedChannel instances have the same id, which prevents them to be added to the ChannelGroup. This makes it impossible to test the "broadcast" scenario implemented by ChannelGroup.

@SummerWis You can inject your own ChannelId for this case:
https://github.com/netty/netty/blob/4.1/transport/src/main/java/io/netty/channel/embedded/EmbeddedChannel.java#L82

Thank you! But I found that this method does not exist in netty 5.0.0.Alpha2. Any workarounds?

@SummerWish nope... you should switch to latest 4.1.0.CRx release anyway as the 5.0.x release series was deprecated and we will not do any more releases there in the near future.

Got it, thanks :-)


Queue testTake causes segmentation fault after test passes

Fix for issue https://github.com/hazelcast/hazelcast-cpp-client/issues/85 . Waits for the thread to finish when the test is successfull.

Test FAILed.

verify

Test PASSed.


Use separate empty folder for each S3 test run

FYI i did this because i encountered flakiness. Concurrently run tests interfere, because hadoop -put creates (depending on version) a temporary ..._COPYING_ file, which we try to generate splits over.


task.ansible.FailureAnalyzer: Sort failure items

To reduce unecessary duplication in e.g. Sentry.

Examples:
https://sentry.ceph.com/organizations/ceph/issues/42280/
{'/dev/vg_nvme: already exists in filesystem', "Creating volume group 'vg_nvme' failed", "Run `vgcreate --help' for more information."}
https://sentry.ceph.com/organizations/ceph/issues/42523/
{'/dev/vg_nvme: already exists in filesystem', "Run `vgcreate --help' for more information.", "Creating volume group 'vg_nvme' failed"}
https://sentry.ceph.com/organizations/ceph/issues/42524/
{"Run `vgcreate --help' for more information.", "Creating volume group 'vg_nvme' failed", '/dev/vg_nvme: already exists in filesystem'}


Add minimum version policy to readme

Closes #252
Just calls out the explicit version of rustc that we test against in CI, along with a policy for how we'll change it. I copied it from lazy-static.
cc @sfackler are you on board with this? Or would you prefer something different?

goddonut, wrong repo :) I do this every time


feat(cluster): integrate 42 urduliz campus

Describe the pull request
Added 42 Urduliz cluster map integration
Checklist

[*] I have made the modifications or added tests related to my PR
[*] I have run the tests and linters locally and they pass
[*] I have added/updated the documentation for my RP

Additional context

Hi @samuelcordero
Firstly, thank you for taking the time to contribute to S42! 🎉
✅ I've reviewed your pull request and I'm pleased to say that it looks great and has been approved.
🕐 I need to check the interface and I dont have my laptop in hand currently. I'll keep the pull request open and will merge it as soon as I reviewed.
Once again, thanks for your contribution, and please feel free to reach me and others contributors on the Discord if you have any questions or if there's anything else you'd like to discuss 😄

Hi @samuelcordero
Firstly, thank you for taking the time to contribute to S42! 🎉
⚠ The CI failed in linting can you fix it please ? (yarn make-pretty will do the job)
🕐 I need to check the interface and I dont have my laptop in hand currently. I'll keep the pull request open and will merge it as soon as I reviewed.
Once again, thanks for your contribution, and please feel free to reach me and others contributors on the Discord if you have any questions or if there's anything else you'd like to discuss 😄

Hi @42atomys,
Thank you for everything! I hope it is okay now, linting reported 0 problems, 0 warnings. I'll be joining the discord, I'd love to be in touch with everyone involved. Have a nice day!


Fix/truffle dependencies

Fix truffle dependencies
Release element 0.1.5

Codecov Report

Merging #93 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #93   +/-   ##
=======================================
  Coverage   33.54%   33.54%
=======================================
  Files         242      242
  Lines        3169     3169
  Branches      228      228
=======================================
  Hits         1063     1063
  Misses       1990     1990
  Partials      116      116

Impacted Files
Coverage Δ

...ment-lib/src/adapters/blockchain/ethereum/index.js
88.15% <100%> (ø)
:arrow_up:

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 86e79a7...e3ea5ec. Read the comment docs.


[Ahuna] 'Try' button in Assistant Builder

Description
Fixes issue #3273
The button allows a user to test an assistant version they are currently creating in the builder. It is only available if the assitant is valid (i.e. enough properties have been set for it to make sense).
Pending issue
Scrolling: when talking in the modal to the assistant, the assistant builder screen is scrolled all the way down
Technical notes

Creates a new AgentStatus called draft
extracts & reuse submitForm from the Assistant builder
when clicking on "Try", creates a new 'draft' agent configuration based on the current builder state

Currently, a new 'draft' agent is created each time the modal is used. The goal of not doing that would be to save DB calls & DB space, but other immediately available options would not impact that:

deleting => doesn't change anything since currently we don't delete, we mark archived
keeping the same agent => we update versions, by recreating everything so that would also not change anything

Screenshots

Risk
Inteferes with assitant creation => will be tested on front edge
Deploy Plan

No migration needed for the new status.
deployed first on front edge.

Addressed, merging


MILU0Preconditioner constructor

A parameterless constructor should be added, so instead of having only
MILU0Preconditioner(bool modified = true)

there should be
MILU0Preconditioner() : this(true)
MILU0Preconditioner(bool modified)

Without a parameterless constructor, the following code (used for automated testing) will throw an exception.:
var M = Activator.CreateInstance(typeof(MILU0Preconditioner));

EDIT: Maybe the second constuctor should be dropped. One can use object initializers instead:
MILU0Preconditioner() { UseModified = false }

Try this:
http://stackoverflow.com/questions/11002523/activator-createinstance-with-optional-parameters
-----Original Message-----
From: "wo80"<EMAIL_ADDRESS>Sent: 18/02/2016 7:16 AM
To: "mathnet/mathnet-numerics"<EMAIL_ADDRESS>Subject: [mathnet-numerics] MILU0Preconditioner constructor (#377)
A parameterless constructor should be added, so instead of having only
MILU0Preconditioner(bool modified = true)there should be
MILU0Preconditioner() : this(true)
MILU0Preconditioner(bool modified)Without a parameterless constructor, the following code (used for automated testing) will throw an exception.:
var preconditioner = Activator.CreateInstance(typeof(MILU0));—
Reply to this email directly or view it on GitHub.

@kjbartel: Thanks for the tip. Works great.
I'll leave this issue open for now.


robustness improvement

Parallel and Concurrent Programming in Haskell - Simon Marlow
Page 150: It is always better to use throwIO rather than throw in the IO
monad because throwIO guarantees strict ordering with respect to other
IO operations, whereas throw does not.

Known bug happened unfortunately again....

@keremispirli please rerun semaphoreci again: we should get all green and accept this pull request.


support longer unit test names

resolves #

Problem

Solution

Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
[ ] This PR includes type annotations for new and modified functions

closing in favor of: https://github.com/dbt-labs/dbt-core/pull/9396


[CONFIGURATION-753] Use only first value for interpolation

When a property has multiple values, interpolation of that property in
the middle of another property should use the same value as that
returned by the getString method and not include all values.

May you please update changes.xml?

I've updated the changes.xml.
RE. the failing tests, they don't seem related to my changes. Are they expected errors or have I introduced a regression elsewhere?

May you please rebase on master?

This is now rebased onto the current master

Replaced by #181


chore(ts-migration): hits per page rename files

This pull request renames the files that migrate hits per page to TypeScript.

@francoischalifour Nice catch, let me catch.


feat(migrations) add 'kong migrations migrate-apis' command

Summary
Add kong migrations migrate-apis command for automated APIs to Routes and Services migration.
The command kong migrations migrate-apis will by default migrate apis with bundled plugins to routes and services, but it will skip migrating apis with custom plugin(s). To migrate apis with custom plugins you can add -f or --force option.  Verbose (debug) --vv will with Postgres also output each SQL transaction SQL statements when executing migration.
The requirement is a database of 0.14 for migration to work.
The migrations transaction (postgres) / batch (cassandra) structure is (here is postgres example, but Cassandra follows the same):
BEGIN;
  INSERT INTO services (id, created_at, updated_at, name, retries, protocol, host, port, path, connect_timeout, write_timeout, read_timeout)
       VALUES ('be288ba6-b90e-47a7-a93b-d6289c66c947', TO_TIMESTAMP(1557939665) AT TIME ZONE 'UTC', TO_TIMESTAMP(1557939665) AT TIME ZONE 'UTC', 'test3', 5, 'http', 'httpbin.org', 80, '/anything', 60000, 60000, 60000);

  INSERT INTO routes (id, created_at, updated_at, service_id, protocols, methods, hosts, paths, regex_priority, strip_path, preserve_host)
       VALUES ('28a19a6e-7f87-4171-8bb6-e602fd8b2168', TO_TIMESTAMP(1557939665) AT TIME ZONE 'UTC', TO_TIMESTAMP(1557939665) AT TIME ZONE 'UTC', 'be288ba6-b90e-47a7-a93b-d6289c66c947', ARRAY['http','https'], NULL, NULL, ARRAY['/test3'], 0, TRUE, FALSE);

       UPDATE plugins
          SET route_id = '28a19a6e-7f87-4171-8bb6-e602fd8b2168', api_id = NULL
        WHERE id   = '0dfdf824-f9b6-4c31-8106-5e84054c2789'
          AND name = 'key-auth';

       UPDATE plugins
          SET route_id = '28a19a6e-7f87-4171-8bb6-e602fd8b2168', api_id = NULL
        WHERE id   = 'f072332c-8c12-41ae-b80e-4b3a78b036a8'
          AND name = 'basic-auth';

  DELETE FROM apis
        WHERE id = '3ac9d9b8-3b68-4738-bfe7-3b1a740e6f79';
COMMIT;

Running the command looks like this:
$ kong migrations migrate-apis
Upgrading from 0.14, bootstrapping database...
migrating api 'test1' ...
migrating api 'test1' done
migrating api 'test2' ...
migrating api 'test2' done
migrating api 'test3' ...
migrating api 'test3' done
3/3 migrations succeeded

Still all the review comments welcomed.


Ansible upgrade to 2.6.0 is failing 3.6.5 rule

amazon-ebs: TASK [RHEL7-CIS : SCORED | 3.6.5 | PATCH | Ensure firewall rules exist for all open ports] ***
amazon-ebs: failed: [default] (item=ssh) => {"changed": false, "item": "ssh", "msg": "firewall is not currently running, unable to perform immediate actions without a running firewall daemon"}
amazon-ebs: failed: [default] (item=dhcpv6-client) => {"changed": false, "item": "dhcpv6-client", "msg": "firewall is not currently running, unable to perform immediate actions without a running firewall daemon
Ansible thinks firewalld is not running?

what is the systemctl status firewalld output

Hello,
I wanted to reach out and let you know that this issue is being closed. We have re-worked the role and want to start with a fresh issues list with this latest version. There was a post in the Ansible-Lockdown google group (https://groups.google.com/g/ansible-lockdown) with the details of the changes that are coming. Please checkout the thread titled RHEL 7 CIS and STIG Changes for all of the details, I also have the message pasted at below.
Please as you use the latest version and open issue tickets as you find them, it is the best way for us to improve the role for everyone. Thank you for being part of the community and providing awareness of problems or advice on improvement. Reporting is a huge part of improving this project.

Hello,
Thank you to everyone in the Ansible-Lockdown community who has contributed to RHEL7 STIG/CIS. Our team at MindPoint Group has been working with the entirety of the Ansible-Lockdown project, and we have some significant updates for both RHEL 7 STIG and CIS. With these updates, some larger changes have been made. I have these changes/updates outlined below.
Testing:

CI/CD - We have implemented some automated testing pipelines to test pull requests into the devel and main branches. With the current workflow, the community will PR into the devel branch (never the main branch) for review by the administrators. When your PR is created, the first check will remain the DCO check. The second check is a functional testing pipeline that will automatically perform a functional test of the branch the PR is initiated from. Once both tests pass, someone from the Administrator Team will review the changes and merge them into the devel branch. From there, an additional review is completed before the devel branch is merged into the main branch. Only the Admin Team will perform PRs/merges into the main branch. There is also an automated pipeline for PRs from devel to main. Please do not edit the .github/workflows files since those are used as part of the pipeline.
Compliance Checking – MindPoint Group has been working to create our own compliance audit scan tool. The tool uses a goss framework executable to run custom checks that we have created. The goal is to provide a more thorough check for control compliance and decrease the number of false positives/negatives. For example, it will check the configuration file related to the control and as well as checking if that configuration is active. With a smarter scan, we can hopefully identify attempts to trick scanners as well (for example stacking a parameter in a config file where the first instance is enabled and second disabled. Most audit tools search for the first instance but the application might look for the last instance of the parameter, thus making the scanning tool think it's enabled). In testing, we have found that our audit scan runs significantly faster than other audit tools, reducing audit times. Our audit tool and profiles will have their own repositories in the Ansible-Lockdown org, but within the remediation role there will be an integrated way to incorporate the audit. Keep an eye out for the audit tools as they are released. We plan on developing a goss audit profile for each current remediation role. Going forward, we plan to release a remediation role and goss audit tool profile simultaneously.
Role Updates:
RHEL 7 STIG/CIS – We have re-written much of the RHEL 7 STIG and CIS roles to increase clarity and readability and address some functionality items. We performed these updates while creating our goss testing framework for each of these roles. We plan on pushing our update to the devel and main branches. We will move the current devel and master branches to a devel_stable_ and master_stable_ branch in the respective repositories. Accordingly, community members who rely on the current version can still use that version going forward; this process will not remove what is currently there. The latest versions of the roles have also been updated to comply with the latest benchmarks.
Role Architecture – All roles will change with regard to the structure in the tasks folder. Taking CIS as an example, there will be a folder per section and yaml files for each sub-section. For example control 1.2.1 in CIS will be located in RHEL7-CIS/tasks/section_1/cis_1.2.x.yml. The cis_1.2.x.yml file will contain all controls related to section 1.2.x. This will hopefully make updates to roles a bit easier with less risk. This matches the architecture of our audit tool, creating consistency across remediation and audit platforms. The end goal is to repeat this architecture (the best we can) on STIG roles, but we are starting with CIS.
Existing PRs and Issues – With all of these changes comes the task of cleaning up existing PR’s and issues. Our plan is to close all of the existing PR’s and issues because of the re-work. Our team is growing and should be able to stay on top of the new issues and PRs as they come in.
Again, I would like to thank the community for your involvement in this project. The input and work from the community has contributed significantly to the success of this project. Please keep an eye out for these changes, which will be rolling out in the coming weeks.


Template column implementation

Hello, for my project needs I added DataGridTemplateColumn implementation in my fork.
Also updated sample (replaced one column type from Text to Template) + updated dependencies.

I've been trying this and it is working well with templates with StackPanels, TextBlocks and Borders but controls like Button and CheckBox don't seem to render.  I've been using with dependencies updated to v0.6.2-build6147-beta

can approve what @ahopper said
but yet haven't time to find out why it doesn't work well
may be someone has ideas

Sorry for the slow response on this. I due to some implementation details necessary to make the DataGrid work with Avalonias validation system, the DataGridTemplateColumn should inherit from DataGridColumn and not DataGridBoundColumn

I've tried to change class inheritance for DataGridTemplateColumn from DataGridBoundColumn to DataGridColumn, but it didnt solved the problem :(

@PurpleGray @sdoroff could be an issue in Avalonia itself? do those controls make it to visual tree?

Is it worth merging this (maybe with the inheritance sorted) just to make it easier to play with and test and encourage some more input?

@ahopper id be happy with that

Sorry I've been away from this for a while. I just added an implementation of the DataGridTemplateColumn. It seems to now render all controls correctly. I'm closing this pull request.


Mark extra_footer_scripts as safe

dictsort returns tuples (like .items()), not values.
Followup #670

My local testing setup was borked, but it has been unborked now. I actually verified this one this time. Apologies for the previous two PRs!

Checks are happy now....so I'm gonna take your word that this is right incantation :-)


"Podișoru" is listed in the drop-down menu

Description:
"Podișoru" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Podișoru" in the search bar.

Expected results
The location is listed in the drop-down menu as "Podișoru, Argeș (Bârla)".


Modular JSON serialization for contrib and vendor packages

https://github.com/quantumlib/Cirq/pull/3539 adds modular json serialization for cirq and cirq.google.
This task is to add JSON serialization testing for all the future optional packages, namely:

[ ] cirq.contrib
[ ] cirq.ionq
[ ] cirq.pasqal
[ ] cirq.aqt

Also,

[ ] we should modify the json_test_data folder to live under cirq instead of cirq.protocol to be consistent with the new design.

Resolved with two exceptions:

cirq.contrib never moved out of cirq-core, so no issue there
json_test_data remains under cirq.protocols, but moving it is nice-to-have


Infer strict return type of function

🔎 Search Terms
"Generic constraints", "Infer strict return type"
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about Strict function types

⏯ Playground Link
No response
💻 Code
First example:
type Primitives = number | string | boolean | bigint | symbol | undefined | null | void;

declare function call<
  R extends Primitives,
  O extends { a: (...args: any[]) => R },
  Args extends { a: Parameters<O['a']> }
>(o: O, args: Args): ReturnType<O['a']>;

const check = call(
  {
    a(a: string) {
      return `${a}a`;
    }
  },
  { a: ['a'] }
)

Second example:
type Primitives = number | string | boolean | bigint | symbol | undefined | null | void;

declare function call<
  R extends Primitives,
  O extends { a: (...args: any[]) => R },
>(o: O, args: { a: Parameters<O['a']> }): ReturnType<O['a']>;

const check = call(
  {
    a(a: string) {
      return `${a}a`;
    }
  },
  { a: ['a'] }
)

🙁 Actual behavior
I'm trying to write a function that retrieves the function's strict return type (which is a property of an object) and takes a second argument as an object with the same properties, but with arguments for these functions.
There is a problem with how TypeScript infer the return types.
In the first example check will have string type. But in the second example check will have ${string}a type as expected.
I have noticed that problem occur when I try to write generic for args argument of call function.
This looks like a bug, since generic for args argument has no effect on other types.
🙂 Expected behavior
In both examples the return type must be ${string}a.
Additional information about the issue
No response

If you don't require a relatively complicated type, you might be able to do this here:

The caller should proactively declare that the type is const. If the type here is narrowed, it will affect other users who do not want to narrow it.


regular updates and fixes (early September)

Fix #448, and brings other ad hoc changes.

run -> regenerate. Not sure this is a good idea after all. It sounds like stage files are being regenerated in this context w/o specifying explicitly the that outputs are regenerated. So, wither we should come up with something else or keep the previous one. @jorgeorpinel what was the motivation for this change, btw?

@shcheklein

run -> regenerate. Not sure this is a good idea after all... what was the motivation for this change, btw?

It's one of the checkboxes I added in #448. The reason is that I never liked the previous phrase "run again", which was a replacement for "rerun".
I'm using this term in the context of dvc repro, in addition to the "reproduce", of course, but you don't always want to explain or define repro with the same word. It should mostly refer to the outputs or results of pipeline stages though, for stage commands you're right that it's not correct so I'm using the word "execute" instead. I made some wording improvements in 492cfc6. What do you think?

Sorry for the delay here @shcheklein.  So many file changes... Sry 😅

p.s. I'm done adding commits here. Maybe one more involving the term "project metrics" but that can always go into a future PR.

huge stuff :) congrats 🎉


npm install fails

Here is what I got
├── UNMET PEER DEPENDENCY angular2@*
└──<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of angular2@* but none was installed.

Closing issue as time related... will hopefully be working on a new version.


Only textToSpeech in response from Dialogflow

I having an issue where only textToSpeech responses seem to work properly. I'm trying to test that the correct suggestions appear but only the textToSpeech array does. When I test it with the google actions console, I can see that the suggestion chips are appearing. I have tried it with a card as well and had the same results.
test.js:
'use strict';
const chai = require('chai');
const winston = require('winston');
var expect = chai.expect;

winston.loggers.add('DEFAULT_LOGGER', {
    console:{
        level: 'error',
        colorize: true,
        label: 'Default logger',
        json: true,
        timestamp: true
    }
});

const { ActionsOnGoogleAva } = require('actions-on-google-testing');
const action = new ActionsOnGoogleAva(require('./test-credentials.json'));

action.startTest('NAME OF THE TEST', action => {
    return action.start('my test app')
        .then(res => {
            return action.send('lmao');
        })
        .then(res => {
            console.log("\n----------\n");
            console.log(res);
            console.log("\n----------\n");
        });
});

test.js output:
(node:10552) DeprecationWarning: grpc.load: Use the @grpc/proto-loader module with grpc.loadPackageDefinition instead
(Use `node --trace-deprecation ...` to show where the warning was created)
| ** Starting test NAME OF THE TEST **
/ > Talk to my test app my test app
/ {
  micOpen: false,
  textToSpeech: [
    'Getting the test version of my test app.\n' +
      'Good day! What can I do for you today?'
  ],
  displayText: [],
  ssml: [],
  suggestions: []
}
- > lmao
| {
  micOpen: false,
  textToSpeech: [ 'These are suggestion chips' ],
  displayText: [],
  ssml: [],
  suggestions: []
}
/
----------

{
  micOpen: false,
  textToSpeech: [ 'These are suggestion chips' ],
  displayText: [],
  ssml: [],
  suggestions: []
}

----------

test passes
> cancel
\ {
  micOpen: false,
  textToSpeech: [],
  displayText: [],
  ssml: [],
  suggestions: []
}
test ends
/ NAME OF THE TEST

  1 test passed

raw response from actions console test:
{
  "conversationToken": "[]",
  "expectUserResponse": true,
  "expectedInputs": [
    {
      "inputPrompt": {
        "richInitialPrompt": {
          "items": [
            {
              "simpleResponse": {
                "textToSpeech": "These are suggestion chips"
              }
            }
          ],
          "suggestions": [
            {
              "title": "suggest 1"
            },
            {
              "title": "suggest 2"
            },
            {
              "title": "suggest 3"
            }
          ]
        }
      },
      "possibleIntents": [
        {
          "intent": "assistant.intent.action.TEXT"
        }
      ]
    }
  ],
  "responseMetadata": {
    "status": {
      "message": "Success (200)"
    },
    "queryMatchInfo": {
      "queryMatched": true,
      "intent": "5e3602df-efba-4ac8-b6c2-1962e678c745"
    }
  }
}

Can you disable and re-enable testing in the console?

Can you disable and re-enable testing in the console?

I just did and am seeing the same results.

I have the same problem. When going deeper into this and doing some debugging I found out that the debug_info field in the response received from the AssistRequest is always null - although this actually should contain the suggestion chips, cards etc.
In the Logs Viewer I can see the request, and Dialogflow returns everything I expected - it just isn't part of the AssistResponse.
@Fleker any idea whats wrong here ?
Received response from agent with body: HTTP/1.1 200 OK Server: nginx/1.13.6 Date: Thu, 20 Aug 2020 12:14:52 GMT Content-Type: application/json; charset=utf-8 X-Content-Type-Options: nosniff google-actions-api-version: 2 Cache-Control: no-cache, no-store, max-age=0, must-revalidate Pragma: no-cache Expires: Mon, 01 Jan 1990 00:00:00 GMT P3P: CP="This is not a P3P policy! See g.co/p3phelp for more info." Content-Security-Policy: script-src 'report-sample' 'nonce-xfl/xTl7c2AWJqq0DeyH4w' 'unsafe-inline';object-src 'none';base-uri 'self';report-uri /_/DialogflowRuntimeAogHttp/cspreport;worker-src 'self' Content-Encoding: gzip X-XSS-Protection: 0 X-Frame-Options: SAMEORIGIN Set-Cookie: NID=204=aaaa expires=Fri, 19-Feb-2021 12:14:52 GMT; path=/; domain=.google.com; HttpOnly Via: 1.1 google Alt-Svc: clear Transfer-Encoding: chunked { "conversationToken": "[]", "expectUserResponse": true, "expectedInputs": [{ "inputPrompt": { "richInitialPrompt": { "items": [{ "simpleResponse": { "textToSpeech": "Default response of send buttons" } }, { "simpleResponse": { "textToSpeech": "suggestions:" } }], "suggestions": [{ "title": "Item1" }, { "title": "Item2" }, { "title": "Item3" }] } }, "possibleIntents": [{ "intent": "assistant.intent.action.TEXT" }] }], "responseMetadata": { "status": { "message": "Success (200)" }, "queryMatchInfo": { "queryMatched": true, "intent": "89404df9-1732-4016-8530-e1789008bbe6" } } }

Any fix on this problem? @Fleker

Is your action built using Dialogflow or Actions Builder? If the latter, you should use the Assistant Conversation testing lib.

I have many projects on Dialogflow and actions builder and I'm trying to use these libraries depends what I need.


The height of Tab can not be set.

There is a SizedBox in Tab class, the height of SizedBox cannot be modified.
I need a less height of Tabbar tab.
same as #13322
Thank you.

Duplicate of #13322

Closing as duplicate of #13322


Approximate 1- and 2-level cuts when zipping BDDs in if_then_else

Closes #272

Codecov Report

Merging #346 (195b1bd) into main (aa52bbc) will increase coverage by 0.007%.
The diff coverage is 100.000%.

@@              Coverage Diff              @@
##              main      #346       +/-   ##
=============================================
+ Coverage   96.469%   96.476%   +0.007%
=============================================
  Files           58        58
  Lines         3568      3575        +7
=============================================
+ Hits          3442      3449        +7
  Misses         126       126

Impacted Files
Coverage Δ

src/adiar/bdd/if_then_else.cpp
95.638% <100.000%> (+0.105%)
:arrow_up:

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 aa52bbc...195b1bd. Read the comment docs.


Payment for E/E #247614 Add created as parameter to MarkAsUnread and ReadNewestAction

Add created as parameter to MarkAsUnread and ReadNewestAction #247614

PR - https://github.com/Expensify/App/pull/13404
C+ @mollfpr
Amount $1000
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ccddd33d55b56333
Upwork Job ID:<PHONE_NUMBER>525574144
Last Price Increase: 2022-12-30

@mollfpr can you please accept the job and reply here once you have?
https://www.upwork.com/jobs/~01ccddd33d55b56333

@mallenexpensify Accepted, thank you!

@mollfpr paid $1000, thanks for the help


Failed to disable several packages + ads coming back

Describe the bug:
On my Fire HD 10, latest OS version (<IP_ADDRESS>), I encounter 2 issues:

When trying to debloat, I get a "success", but I also get a list of packages that the tool fails to disable:

Failed to Disable: com.amazon.alexa.multimodal.gemini
Failed to Disable: com.amazon.tcomm.client
Failed to Disable: com.amazon.kindle.otter.oobe
Failed to Disable: com.fireos.usagestats.proxy
Failed to Disable: com.amazon.sync.service
Failed to Disable: com.amazon.client.metrics
Failed to Disable: com.amazon.tv.launcher
Failed to Disable: com.amazon.minerva.client.api
Failed to Disable: com.amazon.tcomm
Failed to Disable: com.fireos.arcus.proxy
Failed to Disable: com.amazon.device.sync
Failed to Disable: com.amazon.sync.service
Failed to Disable: com.amazon.csapp
Failed to Disable: com.amazon.diode
Failed to Disable: com.amazon.device.sync.sdk.internal
Failed to Disable: com.amazon.identity.auth.device.authorization
Failed to Disable: com.amazon.legalsettings
Failed to Disable: com.amazon.platform
Failed to Disable: com.amazon.device.messaging
Failed to Disable: com.amazon.device.metrics
Failed to Disable: com.amazon.securitysyncclient
Failed to Disable: com.amazon.pm
Failed to Disable: com.amazon.kindle.kso
Failed to Disable: com.amazon.whisperlink.core.android

Despite the failures, it seems the device is pretty much debloated; however, after a few hours, ads announcements are coming back on the lock screen of the device.

To Reproduce:
Steps to reproduce the behavior:

Connect the tablet to my Mac and run the tool
Check the tool logs (see them copy-pasted in the previous section) and noticed how several packages fail to be disabled
Also, wait for some hours and notice how ads are back

Expected behavior:
Device should be successfully debloated with no error logs and ads should not come back.
Platform:

OS: MacOS 13.0.1
Latest Version of Script? Yes, 22.10.1

Device:

Model Fire HD 10 (11th gen)
Version: Fire OS <IP_ADDRESS>

Additional context:
Add any other context about the problem here.

Fire OS <IP_ADDRESS> added a device administrator application that protects certain apps from being disabled, although most get disabled. As for the ads coming back, did you log in to your Amazon account on the tablet?

Yes, in fact that’s quite possible.
So I guess ads are getting pushed back from Amazon account?

Il giorno 4 dic 2022, alle ore 22:40, Hayden Plumley @.***> ha scritto:

Fire OS <IP_ADDRESS> added a device administrator application that protects certain apps from being disabled, although most get disabled. As for the ads coming back, did you log in to your Amazon account on the tablet?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Yes, Amazon checks if you purchased the ad-free version of the tablet using your account. You can deregister (sign out) your device by going to Settings > My Account > Deregister. After de-registering I recommend re-running the tool then rebooting the tablet.

Ok, thanks for the info!


Support role_name and entity_alias in VaultTokenRequest

Closes gh-690

Thank you for your contribution. That's merged and polished now.


bump supabase-js

Hi, not sure where to post this so opening an issue
Is there a reason we are are stuck on ~2.0.4 of supabase-js? 2.7.0 has been released which includes gotrue-js 2.10.0 which fixes https://github.com/supabase/gotrue-js/issues/564, which I think this would of great benefit in the auth-helpers 🙂

Version bumped, but as David has stated here this shouldn't matter unless new Typescript changes were introduced.


ROX-9120: Improve unauthenticated message in roxctl

Description
This PR addresses the ambiguous message we currently print when a user is not setting any authentication information via either the token file, the ROX_API_TOKEN environment variable, or the --password/-p flag.
The original request in the ticket ROX-9120 to return an error earlier before sending the request has been already achieved through the environment changes. When env.GRPCConnection() will be called,
the authentication information will be verified and an error returned, if no sufficient information is set.
Now, the error message is more clear in the different types of authentication information, and how they could be set.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed

manually testing against a central instance without providing any authentication information, e.g.:

roxctl central whoami

Current dependencies on/for this PR:

master

PR #4880   👈

This comment was auto-generated by Graphite.


Make it optional to save unfeaturized slots in Interactive Learning

What problem are you trying to solve?
Building stories with Interactive Learning without having to manually edit them afterwards.
In any bot with a large number of unfeaturized slots, stories built with IL become unreadable very quickly; this is especially so with forms, which usually set several unfeaturized slots.
However, there is a valid use case for saving unfeaturized slots in test conversations, when e.g. you want to be able to test that changes to the featurization of a slot have not changed existing behaviour.
What's your suggested solution?
Slots that are unfeaturized should be excluded by default from the story persisted by interactive learning.
There can be a flag to enable saving unfeaturized slots.
Examples (if relevant)
No response
Is anything blocking this from being implemented? (if relevant)
None known
Definition of Done

[ ] Unfeaturized slots are not persisted during interactive learning by default
[ ] A flag enables saving unfeaturized slots
[ ] Documentation updated

Removing them is tedious and error prone.


travis build failures since december?

The last passing pull request was  Dec 23, 2016. I re-ran the last passing set of tests on master (Commit 0a03cf2) and got all failures (rerun of Commit 0a03cf2).
Something has changed in the travis environment. Maybe due to the release of rvm 2.4?. I will be pursuing a fix so please comment if you have ideas. Best lead currently is before_install:  - gem install bundler (ref).

@7yl4r: This can be closed now, right?


[Bug] [Flink-connector-v2-e2e] Flink docker image not contains hadoop dependency

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Because the flink official docker image didn't have hadoop dependency, so when we try to use flink run some connectors that need hadoop dependency such as connector-file-hadoop connector-hive and so on it will throw some exceptions like NoClassFound.
In order to solve this problem, I think we should build our own flink images. I have builded an example flink image (tyrantlucifer/flink:1.13.6-scala_2.11_hadoop27) and push it to docker hub, the docker file is very simple as the below shown:
FROM flink:1.13.6-scala_2.11
ENV HADOOP_UBER_URL=https://repo.maven.apache.org/maven2/org/apache/flink/flink-shaded-hadoop-2-uber/2.7.5-10.0/flink-shaded-hadoop-2-uber-2.7.5-10.0.jar
RUN wget -nv -O /opt/flink/lib/flink-shaded-hadoop-2-uber.jar "${HADOOP_UBER_URL}";

If the internet speed is not good, we also can download the jar and use this dockerfile:
FROM flink:1.13.6-scala_2.11
COPY ./flink-shaded-hadoop-2-uber-2.7.5-10.0.jar /opt/flink/lib/

What do you think about it? @CalvinKirs @ruanwenjun
Is to write my mirror or seatunnel official production of a special mirror?
SeaTunnel Version
dev
SeaTunnel Config
none

Running Command
none

Error Exception
NoClassFound Exception

Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

How about add it in our kubernetes guide? Add command to download jar in dockerfile.

How about add it in our kubernetes guide? Add command to download jar in dockerfile.

It's a good idea, we can add the step in our own Dockfile and I can do this work. For e2e test case we can use the docker image that I had builded, what do you think about it?

How about add it in our kubernetes guide? Add command to download jar in dockerfile.

It's a good idea, we can add the step in our own Dockfile and I can do this work. For e2e test case we can use the docker image that I had builded, what do you think about it?

Ok for me.

pr #2297


DO NOT MERGE! Fast Forward stable to b52935b7c2dfc5235a7133c22e445dee85849ed2.

This PR will be merged automatically once checks are successful.

This change is

Jenkins job mixer/postsubmit passed

Jenkins job mixer/e2e-suite passed


Update docs api

Core: 1.3.0
Compiler: 11.0_release branch (aec2b600d)

Docstrings fixed https://github.com/rescript-association/rescript-core/pull/213


v2.7.8 - error messages - cluster.x-k8s.io

Rancher Server Setup
single docker installation, v.2.5.x updated many times, last update :   v2.7.7 to 2.7.8
Information about the Cluster
rancher-provisioned cluster 1.26.8-rancher1 on Hetzner VMs

User Information
admin local login in rancher, since OpenLDAP stopped working in 2.7.1
Describe the bug
docker logs of the rancher container:
W1006 05:55:39.749726      48 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineDeployment is deprecated; use cluster.x-k8s.io/v1beta1 MachineDeployment
W1006 05:55:55.768430      48 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineSet is deprecated; use cluster.x-k8s.io/v1beta1 MachineSet
W1006 05:57:11.373174      48 warnings.go:80] cluster.x-k8s.io/v1alpha3 Machine is deprecated; use cluster.x-k8s.io/v1beta1 Machine
W1006 05:57:35.235644      48 warnings.go:80] cluster.x-k8s.io/v1alpha3 Cluster is deprecated; use cluster.x-k8s.io/v1beta1 Cluster
W1006 05:59:33.698876      48 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineHealthCheck is deprecated; use cluster.x-k8s.io/v1beta1 MachineHealthCheck
To Reproduce
install Rancher single container, update 2.7.7 to 2.7.8
Result
Expected Result
no error messages

might be related to https://github.com/rancher/rancher/issues/42999

a part of https://github.com/rancher/rancher/issues/43075

I'm observing the same error message in the Rancher pod logs on a fresh 2.7.9 install on RKE1 1.24:
W1123 12:26:42.459049      32 warnings.go:80] cluster.x-k8s.io/v1alpha3 Cluster is deprecated; use cluster.x-k8s.io/v1beta1 Cluster
W1123 12:27:08.292747      32 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineHealthCheck is deprecated; use cluster.x-k8s.io/v1beta1 MachineHealthCheck
W1123 12:27:11.440043      32 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineDeployment is deprecated; use cluster.x-k8s.io/v1beta1 MachineDeployment
W1123 12:27:38.271223      32 warnings.go:80] cluster.x-k8s.io/v1alpha3 Machine is deprecated; use cluster.x-k8s.io/v1beta1 Machine
W1123 12:29:14.659582      32 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineSet is deprecated; use cluster.x-k8s.io/v1beta1 MachineSet

After upgrading rancher version from v2.6.3 to v2.7.9, getting error in the rancher pod
W1124 11:42:15.684397      33 warnings.go:80] policy/v1beta1 PodSecurityPolicy is deprecated in v1.21+, unavailable in v1.25+
W1124 11:45:21.274961      33 warnings.go:80] cluster.x-k8s.io/v1alpha3 Machine is deprecated; use cluster.x-k8s.io/v1beta1 Machine
W1124 11:46:05.968093      33 warnings.go:80] cluster.x-k8s.io/v1alpha3 Cluster is deprecated; use cluster.x-k8s.io/v1beta1 Cluster
W1124 11:47:00.971701      33 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineSet is deprecated; use cluster.x-k8s.io/v1beta1 MachineSet
W1124 11:50:23.400226      33 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineDeployment is deprecated; use cluster.x-k8s.io/v1beta1 MachineDeployment

One message left with v2.8.0
W0110 22:34:36.409921      33 warnings.go:80] cluster.x-k8s.io/v1alpha3 Cluster is deprecated; use cluster.x-k8s.io/v1beta1 Cluster

I believe this is happening as we use a "dynamic" client that is registering controllers, but as it's dynamic it's trying to grab any usable version i.e. even deprecated ones.
More investigation into this is necessary to ensure we're not actually trying to use v1alpha3 or v1alpha4 anywhere ...

We do pass GVK to dynamic client that we key off resource fields like so https://github.com/rancher/rancher/blame/v2.7.8/pkg/controllers/capr/machinenodelookup/controller.go#L111.
We use discovery client to set the servers GVK's. These are most of what is returned when requesting schemas. The UI uses schemas and may be watching against these old versions which would throw the warning.
@smirnov-mi can you share the output of <rancher-url>/v1/schemas?

clean installation v2.8.0    -> 2.8.1 -> 2.8.2      - the issue is still present. https://github.com/rancher/rancher/issues/44755

the output of <rancher-url>/v1/schemas?

it only contains "betav1", no "alpha"s. :-(
57k lines of code, can be found here: https://github.com/smirnov-mi/k8s/blob/main/Rancher/various/2.8.0-2.8.2-x-k8s.io.schemas

In my case (fresh install 2.8.3) i have no single "betav1", in the https://rancher.orange-ftgroup.ru/v1/schemas but  827 "v1alpha"s. :-(

On a fresh installation of Rancher 2.8.3 the issue is still present:
W0428 11:10:18.481280      32 warnings.go:80] cluster.x-k8s.io/v1alpha3 Machine is deprecated; use cluster.x-k8s.io/v1beta1 Machine
W0428 11:12:15.895399      32 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineSet is deprecated; use cluster.x-k8s.io/v1beta1 MachineSet
W0428 11:13:04.434826      32 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineDeployment is deprecated; use cluster.x-k8s.io/v1beta1 MachineDeployment
W0428 11:13:21.684418      32 warnings.go:80] cluster.x-k8s.io/v1alpha3 Cluster is deprecated; use cluster.x-k8s.io/v1beta1 Cluster
W0428 11:14:54.082643      32 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineHealthCheck is deprecated; use cluster.x-k8s.io/v1beta1 MachineHealthCheck

Still happening with Rancher 2.8.4.
W0531 13:55:11.363577      55 warnings.go:80] cluster.x-k8s.io/v1alpha3 Cluster is deprecated; use cluster.x-k8s.io/v1beta1 Cluster
W0531 13:56:08.947709      55 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineHealthCheck is deprecated; use cluster.x-k8s.io/v1beta1 MachineHealthCheck
W0531 13:57:05.965458      55 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineSet is deprecated; use cluster.x-k8s.io/v1beta1 MachineSet
W0531 13:57:35.005853      55 warnings.go:80] cluster.x-k8s.io/v1alpha3 Machine is deprecated; use cluster.x-k8s.io/v1beta1 Machine
W0531 13:57:47.156313      55 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineDeployment is deprecated; use cluster.x-k8s.io/v1beta1 MachineDeployment

still see it on a fresh 2.8.5 installation (single container setup)
W0619 11:15:08.881022      44 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineHealthCheck is deprecated; use cluster.x-k8s.io/v1beta1 MachineHealthCheck
W0619 11:15:38.953843      44 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineSet is deprecated; use cluster.x-k8s.io/v1beta1 MachineSet

Getting the same issues on fresh install of rancher via docker
Rancher: 2.8.0

Any updates on this?

I see same at the end of upgrade from v2.7.9 to v2.8.5 ( I think it was already logged when running v.2.7.9 too ):
W0910 08:56:32.824355      50 warnings.go:80] cluster.x-k8s.io/v1alpha3 MachineSet is deprecated; use cluster.x-k8s.io/v1beta1 MachineSet
W0910 08:56:46.817775      50 warnings.go:80] cluster.x-k8s.io/v1alpha3 Machine is deprecated; use cluster.x-k8s.io/v1beta1 Machine
W0910 08:57:40.010199      50 warnings.go:80] cluster.x-k8s.io/v1alpha3 Cluster is deprecated; use cluster.x-k8s.io/v1beta1 Cluster

Hi all, I  fix this by recreating the pod for the rancher agent in k8s cluster.
kubectl delete pod cattle-cluster-agent-xyc -n cattle-system
I also delete:
 kubectl delete pod rancher-webhook-xyc -n cattle-system , not sure if it can help.
After that i run - docker restart container_id
It took me couple hours to figure it out, but finally working.
Hope this will save you time.


Update ReleaseNotes.md

Added styling to 0.8.0 and 0.9.0 release notes to match previous releases.

@RLittlesII wow, great catch! We hadn't noticed that.  Thanks for fixing this!

I happened to be looking at the release notes.  You're welcome.


RTCDataChannel.send() doesn't support subarrays

This change ensures that if the passed TypedArray is a subview on the underlying ArrayBuffer, that we preserve the byteOffset and byteLength when converting to an Uint8Array for base64 encoding.

thanks! :+1:
fixed #347


Replace master word in designspace document files

This PR is an attempt to fix some master word entries in fonttools/Doc/source/ directory files.
Please review this PR and do suggest if needed any changes.
This PR is created to fix issue #2896

I don't think there are any tools left that need  these glyph-specific designspaces.


[Enhancement] V11 Compatibility

Updating the system results in the following as of FVTT V11.315

I just released v0.6.1 that should hopefully address this issue. Let me know if this is not the case.

With a lack of feedback I'm going to assume this is fixed, so I'm closing it.


Update example filters to work with arrays

This should address #420 by making the examples compatible with the real-world variability of the data formats. A few notes:

Managing this optionality in JSON Schema within filters is pretty brutal, but I do think this reflects what it takes to use the current design to accomplish even these relatively simple objectives like "find creditCard VCs that participate in the worldbankfederation.com trust scheme".

I'm not sure whether/when tests are run in the current build process. I tried npm run test on the main branch before making changes, and this failed with "Cannot find module '../fixtures/ajv'".

It'd be nice to have tests that run the example filters against some "known matching" and "known non-matching" example VCs, but that's probably a consideration for another PR

opted for #423 instead, opened tracking issues to return to the holistic overhaul of examples later


s/master/main/g 🥢

@ckavili ; want me to rename the default branch?

yeah I was planning to do that and now I can't remember why I didn't do it 🤦  since our other repos use main as the default branch, we can rename this one too. We just need to updates source refs on UJ side  - like this one: https://github.com/rht-labs/ubiquitous-journey/blob/main/ubiquitous-journey/values-tooling.yaml#L180


Minimize malloc calls in ag_scandir

Previously, malloc(3) was called for every entry in a dir,
results were accumulated, and freed in a loop after a dir was done.
Implement dedicated stack to minimize malloc calls.
Each dir now only does one malloc, one free, and several realloc's.
This should have better cache density (as *lloc/free have to traverse pointer lists).
At least, it cannot be slower.
Inspired-by: https://speakerdeck.com/alex/why-python-ruby-and-javascript-are-slow

This also fixes a voodoo which mystically never failed:
#if defined(__MINGW32__) || defined(__CYGWIN__)
        d = malloc(sizeof(struct dirent));
#else
        d = malloc(entry->d_reclen);
#endif

How did it manage to hold sizeof(struct dirent) + d_reclen in latter, when first is 280 and latter is ~20 ?

This part somehow wasn't crashing; how?
#if defined(__MINGW32__) || defined(__CYGWIN__)
        d = malloc(sizeof(struct dirent));
#else
        d = malloc(entry->d_reclen);
#endif


我设置宽高之后，发现图表比我预期的宽高小了好大一圈，怎么回事啊

Same for me, the chart is larger than the container, how can i resize the chart?

Your problem can be solved with money!

Same for me, the chart is larger than the container, how can i resize the chart?

seems this project are indefinite suspension.


Removed redundant ecr_repo_name from README

[ ] I have added the tests to cover my changes.
[ ] I have updated the documentation and CHANGELOG accordingly.
[ ] I have read the CONTRIBUTING document.

@AlejandroEsquivel Merge this PR first to fix the pre-commit action: https://github.com/AgnostiqHQ/covalent-ecs-plugin/pull/60


Enhance training dataset format support

Overview
Goal: Be able to store SLEAP package files (example) in PoseTraining objects so we can embed images used for training models together with the ground truth poses.
Data model equivalences

sleap-io
ndx-pose

Labels
PoseTraining

LabeledFrame
TrainingFrame

Skeleton
Skeleton

Instance
SkeletonInstance

Video
ImageSeries

Labels.skeletons
Skeletons

Labels.labeled_frames
TrainingFrames

Labels.videos
SourceVideos

TODO
MVP

[x] Make TrainingFrame.skeleton_instance into TrainingFrame.skeleton_instances so it can hold multiple instances per frame (https://github.com/talmolab/ndx-pose/pull/3)
[ ] Put SkeletonInstance creation inside of TrainingFrame loop in the README example
[x] Store image data in Image as 3-D array (https://github.com/talmolab/ndx-pose/pull/2)
[x] Make TrainingFrame.source_video optional so we don't need to keep the source video together with the NWB file (https://github.com/talmolab/ndx-pose/pull/2)
[ ] Integrate into sleap_io.io.nwb
[ ] Update docs/readme
[x] Finish tests

Nice to have

[ ] Add a plain string field to TrainingFrame to point to source video as provenance but without triggering validation on DANDI
[ ] Add support for PNG string encoding representation for Image
[ ] Convert and upload reference SLEAP training datasets to DANDI
[ ] Add support for directly opening SLEAP Dandisets in sleap-io via dandi API

Hi @talmo and @eberrigan, thank you for all your work on this PR and for looking through the recent changes in the main branch at brainhack! I am happy to take on updating the README example and test, but I think you all would be better suited to do Integrate into sleap_io.io.nwb and Update docs/readme. Let me know if you'd like me to jump in to update the README example and test.

Feel free to update the readme. Thanks so much!
On Fri, Mar 8, 2024, 3:05 PM Ryan Ly @.***> wrote:

Hi @talmo https://github.com/talmo and @eberrigan
https://github.com/eberrigan, thank you for all your work on this PR
and for looking through the recent changes in the main branch at brainhack!
I am happy to take on updating the README example and test, but I think you
all would be better suited to do Integrate into sleap_io.io.nwb and Update
docs/readme. Let me know if you'd like me to jump in to update the README
example and test.
—
Reply to this email directly, view it on GitHub
https://github.com/rly/ndx-pose/pull/24#issuecomment-1986544886, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AZOPO6WZEICIYKAD36M2HXTYXI723AVCNFSM6AAAAABD6CULCGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBWGU2DIOBYGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@talmo Can you give me write access to your fork? It will be easier than me creating a fork of your fork and us having to merge each fork in one by one...

@talmo Can you give me write access to your fork? It will be easier than me creating a fork of your fork and us having to merge each fork in one by one...

Done! https://github.com/talmolab/ndx-pose/invitations

Thanks! I made a few changes - I moved the Skeletons type out of the PoseTraining container type because the PoseEstimation type now requires a Skeleton object and, for people who don't have (or want to include) training data with their pose estimates, it feels weird in my opinion, to require them to make a PoseTraining container just to put Skeletons into it.
I also updated the readme and tests and made some other minor adjustments.
@talmo I would like to merge this PR soon so that it will be easier for me to refactor the repo to use the latest ndx-template. Then you could start a new PR with any necessary adjustments for integration with SLEAP and any other changes you want to make (or you can make those changes here if you have time). Would that be all right?

Yes that works for me @rly!


Add files via upload

https://it-lab.am/courses/1

@itlabbf Regrettably, this request has signs of fraud. I have to close it without merging.
Please notice that even if the request did not have signs of fraud, the training centers with short-term courses are not eligible for this repository.
If you are a student, I recommend to apply with an official document proving that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
Thank you for the understanding!


Test flushes that require multiple metrics documents due to labels

Fixed in 0.3.0 branch

0.3.0 published


Radio Button forms

HI, I'm creating a form which utilizes the radio button in groups of 5. Can you help me implement this. I have multiple questions in my form.

What's problem are you facing? Would you please post the code snippet here?

For example if I have 100 questions. Do I have to create 100 instances of this.radioGroup = new MKRadioButton.Group();`?

There're serval answers for a single question?
So I think there should be a group for each question.

And is it possible to just set the number of radio buttons, instead of calling it every time I need a radio button?

So are you suggesting that I have a hundred variables for 100 groups?

Yes. I think the way you use the Radio buttons should just like what you will do using html tags.
You can write a component yourself to wrap multiple Radio buttons.

Sorry, I didn't mean you should have hundreds of variables. But there's should be hundreds of groups.
You can make it local variables, like using a method to do the job, or write a custom component.

Ah yes but , each new MKRadioButton.Group() needs to be assigned to a prop. How do I go about this? It's easy if I could dynamically change the group by name but group prop accepts an object property.

Sorry for replying it so late. But I wrote an example below, hope it may help.
// say you have a ListView for the questions
<ListView
  dataSource={this.state.questionList}
  renderRow={this.renderQuestion}
/>

// rendering a single question
// we create a group for each question, for it must be single selection, if you're using RadioButtons
renderQuestion(question) {
  const group = new MKRadioButton.Group();
  const answers = [];
  question.answers.forEach((answer) => answers.push(<MKRadioButton group={group}/>));
  return <View>{answers}</View>;
}


Introduce the concept of a context and the ability to log into one using device code flow

Background
The goal is to allow anyone to take any of our building blocks and run it anywhere. This actual environment it is running in, is what we refer to as a context. It will hold the different exposed Dolittle services that gets installed and the tooling typically then having the ability to talk to the endpoints of this - relative to the URL typically.
By default, there is the official environment - which is the Dolittle offering - meaning we can provide an optimized mechanism for just logging into it and it would then automatically create a “Dolittle Official” environment or similar and that would be the one logged in.
Commands
All examples are given in the context of the CLI.
Contexts - namespace
A context is a representation of where to find Dolittle services. These are given a name and has an endpoint that represents where the API surface for these servies are. The user needs to be authenticated in order to work with it and for now we want to keep the access token and the refresh token from an OAuth Device Flow.
Add
Adds a context with a given name and URL
$ dolittle contexts add <name> <url>

Remove
Remove a context - it should also have a «Are you sure?» confirmation from the user.
$ dolittle contexts remove <name>

List
List all environments
$ dolittle contexts list

Result should be in a table
Login
Logs the user into a context. Since this is a top level command, it means it implicitly means you’re logging into a context. This is to provide a simpler syntax as a starting point and not having to require the contexts namespace.
The default starting point for anyone using the official Dolittle context in the cloud (https://dolittle.studio) - the procedure should be a very simple login without any arguments or options:
$ dolittle login

If you have another context you need to log into that has e.g. Access tokens that are no longer valid, you can optionally provide the name of this:
$ dolittle login <name of environment>

┆Issue is synchronized with this Asana task

Most of the functionalities described here is implemented for the 8.1.0 release.


Create LICENSE

From a legal standpoint, we might have issues because this repository was created with starter (tutorial) code not created by any of the five of us. While I do not expect the CSC207 teaching team to retaliate, this is a sketchy move. It would be best to obtain explicit permission (and to keep a record of it!).


Fix clap by updating the facts

clap on master is missing some .facts files.
PR #129 added warnings to ~nag~ remind us to refresh this dataset with up-to-date facts. Here they are.
Maybe adding a smoke test could be interesting, since it's an important dataset. The LocationInsensitive variant runs it in 700ms on my machine (1.5s on The Ole Laptop) so maybe it wouldn't be that bad a wait on cargo test (or if not, in a test subcrate: only cargo test --all would run it. And CI does that as well.)

Wew lad

That's a nice way of keeping my +/- ratio in the negatives — especially if I never add any tests ever again 👍


realm file is broken for some reason

The realm file should be deleted when user logout and create new one when login.
I found this case the realm file is not deleted and the file can't opened by realm browser, alert like below.

So what can cause this problem? maybe crash while committing transaction?
thanks~
realm version: objc 2.8.2

If you're deleting a Realm file and you're ending up with a corrupt file, it's probably because the deletion didn't complete successfully, or that Realm APIs were still accessing the file while it was being deleted. We can't guarantee the consistency of Realm files if you're changing them via external APIs, such as file system operations.

I add this check to see if it works
if ([[RLMRealm defaultRealm] inWriteTransaction]) {
      [[RLMRealm defaultRealm] cancelWriteTransaction];
}

The requirement is stricter than simply not being in a write transaction.
To safely delete a Realm file after the Realm has been opened, the Realm needs to be closed. This only happens when:

There are no more valid references to the Realm, or any managed objects from the Realm.
All autorelease pools that previously contained valid references to the Realm (or managed objects) are drained.

It's safer and easier to delete Realm files upon app launch, or at least before you open any Realms.


Conform NIOIPProtocol to Sendable

Motivation
We were missing a Sendable conformance on NIOIPProtocol

@swift-server-bot test this please


Compilation failure in 11.4.6

Bug description:
react-native-webview==11.4.6 fails during > Task :react-native-webview:compileDebugJavaWithJavac
To Reproduce:
Run react-native run-android in a project with react-native==0.64.1 and react-native-webview==11.4.6.
Expected behavior:
Successful compilation.
Logs:
my-project/node_modules/react-native-webview/android/src/main/java/com/reactnativecommunity/webview/RNCWebViewManager.java:119: error: RNCWebViewManager is not abstract and does not override abstract method createViewInstance(ThemedReactContext) in ViewManager
public class RNCWebViewManager extends SimpleViewManager<WebView> {
       ^
my-project/node_modules/react-native-webview/android/src/main/java/com/reactnativecommunity/webview/RNCWebViewManager.java:172: error: method does not override or implement a method from a supertype
  @Override
  ^
my-project/node_modules/react-native-webview/android/src/main/java/com/reactnativecommunity/webview/RNCWebViewManager.java:590: error: method does not override or implement a method from a supertype
  @Override
  ^
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: my-project/node_modules/react-native-webview/android/src/main/java/com/reactnativecommunity/webview/RNCWebViewManager.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
3 errors

FAILURE: Build failed with an exception.

Environment:

react-native version: 0.64.1
react-native-webview version: 11.4.6

Closing as duplicate of https://github.com/react-native-webview/react-native-webview/issues/1995


Cross-browser modal issue in Safari

Looks like the modal functionality has issue on the Safari. I have a login dialog modal. On firing up, the dialog came up fine. The moment you click any where on the screen (including the dialog), everything becomes unresponsive to the mouse click, including the dialog itself. I will not be able to access the form input fields or anything else for that matter.
Basically the whole page gets locked out including the dialog. To unlock it, I go into the console and disable the z-index for the _dialog_overlay. But this disables the dialog_overlay and hence breaking the modal functionality.
Is there any workaround for this problem? Is this a known modal issue with other browser like Safari?
Thanks

If I remove the 'bottom: 0" from the css for _dialog_overlay ,the dialog becomes available. But it basically disable the modal functionality, making it like the regular dialog without modal.
I have tested your modal demo in Safari and it works fine. I just can't see what I have missed. Unless the fact that I am running under Backbone JS view environment might have complicated the case. But it works without a hitch under Chrome.

I'm running into this too, but I think on my end the issue is related to the following limitation (per README):
They should be a child of  or have parents without layout (aka, no position absolute or relative elements)
I'm using react-mdl, and the dialog elements are being created inside the content section of a page that uses layout to support a responsive header and a navbar.

I have similar issue with Safari that the modal doesn't close.  However hitting the esc key did close the modal.

@Garbee look at @prasadsilva 's comment.
A nice fix is to set them to be position: fixed (why this wasn't part of the spec I have no idea) and fix the related CSS of that. That way they just live in the middle of the screen regardless of viewport and don't care about layout.

Ah, interesting. I'll take a look at this on Thursday or Friday if I have time. That sounds like a great fix.

Closing this as it's related to incorrect use of stacking context. We now warn if you're making a mistake, and also are more permissive, so incorrect usage might still work: #131


Option to delete the vacation after it was approved

we need to be able to delete a vacation even after it was approved  in case anyone want to cancel it

@maayarosama What is the status here?

@maayarosama What is the status here?

need help with the backend side of this

Add a delete button for admin only, delete vacation and the reason to delete this vacation text window where the admin state why they want to delete the vacation


fix: hide unsupported assets in fiat page asset search

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

High Risk PRs Require 2 approvals

What protocols, transaction types or contract interactions might be affected by this PR?

Testing

Engineering

Operations

Screenshots (if applicable)

#6533  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @woodenfurniture and the rest of your teammates on  Graphite


Fix url for google fonts

This isn't a fix; it was fine before. It's a protocol-less URL. Don't view your site from file:// and everything will work.

Add http: to the url and it works.

I think its better to make it https because if the page is loaded in SSL enabled page, the URL (or any URL) will be treated as unsafe/insecure

Or add it directly to the skeleton css with @import

Is there a good reason why https isn't added? I understand that the error does not show up when viewed from a server, but what's the harm in adding it so as to avoid the error when you are not on a server?


Add fixtures

Adds fixtures to encourage code reuse and make it easier to write tests. For now, only adding mpt fixtures.
Models

[x] mpt
[x] hf_mpt

Data

[x] finetune dataset/dataloader

Other Models

[ ]  llama2
[ ] neo

hmm having trouble reproducing the CI code quality error locally, and the types it's complaining about seem to match correctly 🤷♀ will take a closer look


Liferay

https://www.liferay.com/

Can you point me to where the source is hosted and its license? I can't find it.

https://github.com/liferay/liferay-portal

Is that linked somewhere on the site? I couldn't find it.

You're 100% right.... I'm letting them know and they are fixing it right now.  The link will probably go on dev.liferay.com as that is the developer site where as liferay.com is the customer facing site.

I'd be happy to post https://dev.liferay.com/ when that happens. In general, we don't publish commercial site that release as OSS unless there is a link on the page directly to where the source is hosted and that location contains a license in the root of the project.

Hi trek
Thanks for the patience.  http://dev.liferay.com now contains the link to Github.


Create eps-marche.txt

@romu242 I have to close this request as a duplicate to #14912. Thank you for your understanding.


feat: add python implementation

This PR adds http-plugin for the python client.
Sorry for all the type stubs, I didn't know that it didn't get automatically generated and added in the final built package. So, I needed to generate it in the dependent package (i.e. http-plugin). this will be fixed in the future python-client release

TODO: CI/CD


Additional settings for region triggers

Currently, region triggers can be set to trigger upon the player, an enemy, or both, entering the region trigger. I propose a new menu system, and additional options for region triggers.
Options are now presented as checkboxes rather than a dropdown menu. Options could include:

Player
Enemies
Cubes
Energy Balls
Lasers
Light Bridges
Funnels

This would give region triggers some more flexibility and usefulness for map design.

This sounds like the large number of options would take up more space in a level file than necessary, especially if it's expanded to not just include portal stuff. You can already set region triggers to detect specific object types, though not with the menu. That would work for all physical objects, and there are already dedicated laser detectors. Nothing yet for funnels though.

in ACE i just made it an input, also you could put any object followed by a "," to add more, I'll port it over if Alesan is fine with that.

02d6fe931f3430a34bfa409f94cdf76a69b3cd09


IPV6 issue

Description
App store submission rejected  due to ipv6 not supported. what changes i need to support ipv6 my application fetch request.
Additional Information

React Native version: 0.35.0
Platform: iOS & Android
Operating System: MacOS

How is this related to React Native?

@Vinu-MysteriousTechVision were you able to fix this?

@fgaleano were you?

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with Canny! Check out its new home: https://react-native.canny.io/feature-requests/p/ipv6-issue

Hi,
I'm having the same issue when submitting the app in Apple it's rejected because of the said problem.

RN 0.43
OS: MacOS
Platform: iOS & Android
#Thanks

I'm also having the same issue.

most cases likely are because your api server doesn't support ipv6.

@teyou ... the cause of the problem is just because the server does not support ipv6?

I can confirm that the root cause is NOT that the api server does not support ipv6
We have an app that has been rejected for the same cause but the app barely makes api call to Apple Maps' service and Google Analytics, using react-native-maps and google-analytics-bridge.
I can only imaging a possible cause would be that some components that you use don't support ipv6

@maxschmeling I am having the exact same issue, NetInfo also isn't working for me with IPv6..
Do you find some way of solving this? Thanks!

@petealves I just removed my use of NetInfo (which isn't ideal, but I'm just dealing with it for now). After removing the NetInfo check on startup, I was able to get through the App Store review with no problems.


Volvo trucks PDP Resource gallery

Fix #87
Test URLs:

Before: https://develop--vg-volvotrucks-us-rd--netcentric.hlx.page/drafts/marko/pdp#resources
After: https://87-resource-gallery--vg-volvotrucks-us-rd--netcentric.hlx.page/drafts/marko/pdp#resources

On a redesign page addPlayIcon needs to be adjusted in scripts/video-helper.js so that you can use icon-play-video
flex-shrink: 0; should be set for an <svg> in the <button>. On mobile the icon shrinks a bit

New icon added to video


OpenVPN server certificate

Furyagent is currently unable to renew the OpenVPN server certificate when it expires. This means that the user needs to resort to manual actions to renew this certificates.

Yes, this will help a lot the day by day activities and is a very nice good first issue.
I guess that the culprit  is the function CreateUser that should take the parameter overwrite in order to force the recreation/upload of the certificate with the same name.


Normalize rewards by standard deviation of discounted return in MuJoCo

#115

Codecov Report

Merging #149 into master will decrease coverage by 0.17%.
The diff coverage is 20.58%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   22.56%   22.38%   -0.18%
==========================================
  Files         128      128
  Lines        7987     8014      +27
==========================================
- Hits         1802     1794       -8
- Misses       6185     6220      +35

Flag
Coverage Δ

#unittests
22.38% <20.58%> (-0.18%)
:arrow_down:

Impacted Files
Coverage Δ

rlpyt/algos/pg/a2c.py
0.00% <0.00%> (ø)

rlpyt/algos/pg/base.py
0.00% <0.00%> (ø)

rlpyt/algos/pg/ppo.py
0.00% <0.00%> (ø)

rlpyt/experiments/configs/mujoco/pg/mujoco_a2c.py
0.00% <ø> (ø)

rlpyt/experiments/configs/mujoco/pg/mujoco_ppo.py
0.00% <ø> (ø)

rlpyt/samplers/base.py
80.00% <ø> (ø)

rlpyt/samplers/collections.py
96.29% <ø> (ø)

rlpyt/samplers/collectors.py
81.03% <ø> (ø)

rlpyt/samplers/parallel/gpu/collectors.py
0.00% <0.00%> (ø)

rlpyt/samplers/serial/sampler.py
97.72% <ø> (ø)

... and 5 more

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 668290d...a15a93b. Read the comment docs.

OK this is interesting and I think it can be made a lot simpler.  As far as I can tell from the PR, the same could be achieved by changing process_returns() of the policy gradient class, right after the following lines
https://github.com/astooke/rlpyt/blob/85d4e018a919118c6e42fac3e897aa346d84b9c5/rlpyt/algos/pg/base.py#L47-L49
by inserting:
if self.normalize_reward:
  return_ = discount_return(reward, done, 0., self.discount)  # NO boostrapping of value
  self.rets_rms.update(return_.view(-1, 1))  # matching the shape you used, not sure if the extra dim is needed?
  std_dev = torch.sqrt(self.rets_rms.var)
  reward = torch.div(reward, std_dev)

# proceed with computed discounted returns or GAE returns using the scaled reward

I think that accomplishes the same math?  And doesn't need to change any files in the sampler :)

Any more comment?  Anyone else used this?


Remember gain & pan levels per domain

Something worth adding?

I don't need that, I mostly use this for YouTube, where different URLs need completely different adjustments (and most URLs don't).

 How hard would it be for someone else to implement? This might be a fun project for me.

I don't think it's hard.

+1
This makes more sense to have it as a setting to remember the setting by domain.
Also, adjusting it manually for every video doesn't make sense, because, how many times does someone watch a video?
Having gain set globally for all domains would also make sense to have it as a setting.

I like the idea of a global gain with a per-domain override.
How I would probably use it is to set the global gain to something like 1.5 .  That way, most content would play a little louder, so I could actually hear it better.  But cross-domain content wouldn't be a huge contrast in volume.
Then I could set individual gains for specific sites with issues.
Honestly, of the two features, I think I will get more value out of the global gain.  Per-domain is like icing on the cake.

Yeah, in instagram.com when I'm watching the stories, it resets after every video, its annoying. It should remember gain per site.

Yeah, in instagram.com when I'm watching the stories, it resets after every video, its annoying. It should remember gain per site.

This doesn't happen to me. Android volume is global for all the apps.
I would like this feature because youtube music has more default level than youtube videos and I don't find anyway to recalibrate than adjust it with windows volume settings or kde one but they still don't remember the domain.

Yeah, in instagram.com when I'm watching the stories, it resets after every video, its annoying. It should remember gain per site.

This doesn't happen to me. Android volume is global for all the apps.

What does it have to do with android?
I'm on Linux, Firefox. Browsing instagram.com
Then I check the stories, and volume resets after each next video

To the best of my knowledge, I have not founds a similar extension that duplicates this functionality and can save a setting per-site. I have my default volume on youtube set low, but Instagram and Tiktok links will always blast full volume, making them a digital landmine. Being able to lower the volume per-domain would be incredible if possible.

6 years later and we are still wanting this feature :)


inconsistent window dragging

When dragging the window using the title bar the event is ignored and the window doesn't move.
It randomly works after a few tries. (using Jerakin:borderless fork Commit #4bd2122477a32e7348b5714ac8415fc72f07b2f3
Using PySide2 5.13.2 + Windows 10

It looks like when I click and immediately drag the window, the window doesn't move.
Holding down the mouse click for a second or more and then dragging works.

I am can't really reproduce this, could it be because you are sometimes clicking "outside" of the window titlebar? The titlebar is currently only about 60% of a "standard" windows titlebar.

I tested on a completely different machine to be sure and I get the same behavior. Are you guys using PySide2 or PyQt5 for testing ?

PyQt5 is the bindings I use. I tried on Window 10 running in a VirtualBox. Will try it on a proper windows machine this weekend.
Does the master branch behave the same?

Maybe I was clicking outside of the small title bar area. I made some changes to make the the drag-able title bat the same height as the min/max/close buttons and it makes it easier to drag now.
I also made small changes so that the title bar background color can be customized if needed which makes it even easier to locate the area that can be dragged.
This can be closed. Thanks again for this great project.

Sweet! Remember that you can always make contributions if you think it is something that should be in the project by default. :)


Create .editorconfig

Forcing to consistent clean code format.
And .gitattributes file resorted to alphabetically.

@benwilkins Please merge it.


Can't debug Node app in WSL

Testing #96296
When using the new debugger in a WSL repo:

Works fine in a Windows folder.
I've tried using browserLaunchLocation, but that seems available to pwa-chrome only.

This is the same as https://github.com/microsoft/vscode-js-debug/issues/451 -- please let me know if you continue to hit this on the latest Insiders!


API version should be independent of the URI

Hi,
In current implementation of swashbuckle the initialisation requires the following:
GlobalConfiguration.Configuration
.EnableSwagger(c =>
{
c.SingleApiVersion(apiVersion, "Some Title");
})
.EnableSwaggerUi(c => { });
where apiVersion has the limitation of not having full-stop(.) as part of the string. In CI environment where api's are in version  [major.minor.bugfix] format we have to transform the dots into underscores(_) so for example 1.0.1 becomes 1_0_1.  This has the side effect of api version showing up in Swagger-UI as 1_0_1 which isn't ideal.
Would it be possible to use a different source to set the api version that shows in swagger-ui and swagger json file? ie:
GlobalConfiguration.Configuration
.EnableSwagger(c =>
{
//where CleanupStr() would turn full-stops into underscore (_) and used in url
// the second paramter can be used for the swagger-ui and json file actual version in original format
c.SingleApiVersion(CleanupStr(apiVersion),apiVersion, "Some Title");
})
.EnableSwaggerUi(c => { });

I've run into the same problem. I'd like a solution to this.

I am having a similar issue with . in the version string. It would be good to have the swagger ui version match the CI build.

same problem!


Pass DOM node to YouTube Player instead of ID

Our use case involves loading the player as part of a component in an iframe, so using getElementById fails. Changing it to use the node has the side benefit of making the component a little more flexible, since now the id isn't required, and subsequently there's no need for tracking unique ids, and it avoids id collisions on the outside chance someone might not be cautious about their naming.

I didn't know you could do this. Much better! Thanks for the PR.