Clean way to subscribe to a filtered set of events

Say I wanted to implement a real-time chat app with multiple chat rooms that can be created and deleted dynamically. The client who is in chat room "football buddies" wants to subscribe to the messages service but is only interested in messages to that room. If I understand feathers correctly, feathers will send all events on the message service, which might be a lot if there are a lot of small chat rooms.
Perhaps the socket.io concept of namespaces and rooms is useful here, but I'm not sure of the best way to set it up within the feathers ecosystem.
I think feathers-reactive would effectively accomplish this from the point of view of my app, but the client side is still dealing with all the useless websocket traffic, which is undesirable.
I also see that event filtering might accomplish this, but it seems like I'd need to add some server-side functionality for tracking which users are currently connected to a given chat room, which seems less clean than simply being able to subscribe to a subset of the events on a service.
Any help is appreciated.
Thanks.

This is exactly what event filters are for. Primus and other (potential) real-time providers don't have a concept of rooms and everything that rooms can do - and more complex functionality (which you almost always need) - can also be done through event filters. There are two options:

Add the rooms a user is in to the user object and compare the messages room to the rooms the user is subscribed to. This is usually the best way since you can e.g. show offline users that joined a room (which isn't easily possible with Socket.io rooms)
Add a custom Socket.io join event and add the connected room directly to the connection and then filter by the rooms on that connection:

app.use(socketio(function(io) {
  io.on('connection', function(socket) {
    socket.on('join', function(roomId) {
      socket.feathers.rooms.push(roomId);
    });
  });
}));

app.service('messages').filter(function(data, connection) {
  return connection.rooms.indexOf(data.room) !== -1;
});

Yup. That will do it! I'm going to close but this has come up a few times I'm wondering if we should put it in as an example in the docs under "rooms" or "channels". I dunno... maybe people just aren't seeing the that part of the docs too....

I added it to the FAQ at http://docs.feathersjs.com/help/faq.html#how-do-i-create-channels-or-rooms


Enable support for topics property

Include the topics property in the repository results using the application/vnd.github.mercy-preview+json media type.

Closed as needs some work on the tests. Will reopen once sorted

@jekyllbot: merge +minor


Wrong positioning in Safari

Steps to reproduce the problem

In Safari the vertical position is wrongly calculated on rubber scrolling past top. See https://codepen.io/anon/pen/zPrZZK

What is the expected behavior?
Popper and reference should stay together.
What went wrong?
Popper stays together with the reference element only when scrolling a few pixels top. Afterwards the popper scrolls with the background and should stay fixed (the reference element is fixed and the popper is inside the reference element)
Any other comments?
In the codepen, the reference element corresponds to a fixed navigation bar and the popper to a popup menu attached to an option. It is very annoying to see the popup displaced from the navigation bar.

What version of Safari?

11

May you share a video recording of the problem? I don't seem to reproduce it.
You may use QuickTime or LiceCAP

There it is: https://www.dropbox.com/s/cdpejt4zxdiqabz/popper.mov?dl=0

I see, I don't think I can do anything for it. It's something handled at OS level and there aren't JS APIs for it. Sorry.

Well getBoundingClientRect() for the reference element is returning correct values always at top-left corner. How is the popper position then calculated in this case to give incorrect values?


renamed examples to example to match all other modules

Motivation
Need the directory structure to match on all modules so scripts can be run
Description
rename wfm/examples to wfm/example
Progress

[x] renamed wfm/examples to wfm/example

Additional Notes

@austincunningham - When doing rename please do move and add to make sure that changes appear as move. This change appears as new and it's removing previous history.


Allow Openlane version to be overridden

Use the OPENLANE_IMAGE_NAME environment variable to override the default
version of Openlane.

I wasn't sure if we wanted to go with an environment variable or a dffram.py command line option

Merged into #158.


BOUNTY FOR UPDATING THIS PROJECT?

@prc5 Would you be interested in reviewing the PR's and addressing the memory leak issue against whatever support we can raise?
Anyone who wants to match my 100 USD please comment here and show your support.

@kerns Thanks for the support, I hope the memory leak is fixed 👀


Add js-docs shortcode to ensure consistency between doc and js code

Description

add js-docs shortcode
update documentation to point on snippets.js and sanitizer.js where relevant:

alerts.md => pointing on snippets.js
modal.md => pointing on snippets.js
toats.md => pointing on snippets.js
javascript.md => pointing on sanitizer.js

Motivation & Context

Js-docs shortcode will help avoid copy/paste errors and documentation update oversights
Type of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactoring (non-breaking change)
[ ] Breaking change (fix or feature that would change existing functionality)

Checklist

[x] I have read the contributing guidelines
[x] My code follows the code style of the project (using npm run lint)
[x] My change introduces changes to the documentation
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Live previews

https://deploy-preview-38316--twbs-bootstrap.netlify.app/

Direct links:

https://deploy-preview-38316--twbs-bootstrap.netlify.app/docs/5.3/components/alerts/#live-example
https://deploy-preview-38316--twbs-bootstrap.netlify.app/docs/5.3/components/modal/#varying-modal-content
https://deploy-preview-38316--twbs-bootstrap.netlify.app/docs/5.3/components/toasts/#live-example
https://deploy-preview-38316--twbs-bootstrap.netlify.app/docs/5.3/getting-started/javascript/#sanitizer

Related issues

Nice, looking good so far ❤


add a symfony benchmark with custom normalizers

I've half added this as I was intrigued to how it would perform, after I saw SerializardClosureBenchmark.
I'm not even entirely sure that I agree with this being included (as well as SerializardClosureBenchmark), as it makes the benchmarks non-comparable. You can't compare this new benchmark with the JMS benchmarks for example, as it's just comparing apples with pears - they are not doing the same thing. Unless ofcourse you added a JMS benchmark with custom handlers, like this one.
Edit: faster than JsonSerializableBenchmark - funny.

Does it make sense to add some kind of note on the benchmark results for some serializers?

Does this benchmark goes recursively trough the object graph? (I'm not a  symfony serializer expert...)

probably, or group the benchmarks into ones that use reflection, ones with custom normalizers etc...
could you add one for jms?

Not sure about this. IMO it does not matter how a serializer gets the data, the important is the final result.
Does it make sense to verify somehow the result of the serialization process? Like checking the resulting JSON?

Actually jms uses internally both reflection and closure, switching automatically between them to get the best performance result.

Well, it's a perfectly valid (and recommended) usage of the Symfony Serializer: it's one of its strengths, it comes with a very handy (but slow, even if it's improving dramatically thanks to @fbourigault and @bendavies) normalizer, and you can easily create your custom, optimized ones.
It's very similar to what Serializard allows. Anyway, I'm not sure that it should be in this benchamark. Or we should add a note somewhere in the README explaining that the feature set between JMS, ObjectNormalizer etc, and the one of JsonSerializable, SerializardClosure, and Symfony custom normalizers is definitely not comparable.

Does it make sense to verify somehow the result of the serialization process? Like checking the resulting JSON?

:+1:
It's hard to compare how a benchmark is fast without looking at which features it has. What about defining a common list of features (such as custom-normalizer, groups, etc...) and for each benchmark return the list of the enabled features.
Adding a note when displaying the benchmark results could be valuable too.

I agree with @fbourigault regarding the serializer's capabilities. We need to define a top 3 (or more) features used by the community and compare only those libraries which implements these features. In addition, the benchmark runtime should force the libraries to use these features. For example, each benchmark should implement one custom normalizer, define one virtual property, define one serialization group and so on. Only with these features in act we'll make a fair comparison among the serializers.

Does it make sense to verify somehow the result of the serialization process? Like checking the resulting JSON?

Yes. It totally make sense! I've created my own benchmark application a few time ago and I can say that we need to validate the output. I was trying to understand why a specific library could be so fast and some minutes after I realized that library doesn't work properly with collections of objects and doesn't serialize the entire object graph.
Challenge: how can we validate the output without side effect on measuring, I mean, the validation shouldn't influence on the mean time.

I understand the overall sentiment of not comparing solutions with different feature sets, but serializer definition is simple: any tool which traverses the object graph to produce its representation in a different format (and back). I think that we should define the list of capabilities a serializer library can have and then prepare a feature matrix, while still comparing all of them in the same list. These capabilities could be defined with specific test cases where test runner provides data and verifies the result, with the list of serializers as data provider array.
Examples: serializes objects, serializes arrays, serializes scalars, detects object graph cycle, allows custom normalizers, supports deserialization, allows custom hydrators. Feel free to add yours.
BTW @tsantos84 as your benchmarking project contains the most libraries, could you maybe port the missing ones here? Let's use the momentum generated here and prepare the PHP serializer benchmark. :)

BTW, I have in my repository a Symfony Serializer with custom normalizer. Can I bring it to here?

BTW, I have in my repository a Symfony Serializer with custom normalizer. Can I bring it to here?

Isn't this PSR exactly about a custom symfony Serializer normalizer?

I start to think that @thunderer has a valid point. Should be up to the user compare features while the benchmark should just do the benchmaking.
Any criteria we decide as "top 3 features" for a serializer could be not part of some candidates even if they are perfectly valid serializers for some use cases.

BTW, I have in my repository a Symfony Serializer with custom normalizer. Can I bring it to here?

Isn't this PSR exactly about a custom symfony Serializer normalizer?

Yes. Reply this thread thinking I'm on issue instead of the PR. Ignore it.


Introduce variable TokenCalculator based on selected LLM Provider

Different LLM providers use different tokenizers, which can lead to varying token counts for the same text. To address this, we'll need to modify our implementation to use the appropriate tokenizer based on the selected Cloud LLM


users list only goes up to 10, doesn't paginate

There are 24 users in the DB when I just query it, but the users table shows 10 and says '1 to 10 of 10 entries'.

I found this bug earlier today actually. It is a fairly minor issue, a workaround before I get the fix in is to add ?limit=100 in the query string and then refreshing the page.

Fixed in recent PR.


AI integration: Refactor code how container and database name is flowing to opentelemetry module

Description
Problem with current design
Right now, containerName and databaseName is only available when there is a successful response available. In case of any exception, this information is lost and hence not available in open telemetry module.
Solution
In this new design, containerName, databaseName, operationType, operationName is getting flown to open telemetry module during initialization of open telemetry only to make sure this information is always available.
Type of change

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

This PR fixed this issue #3536


Delay validation of configuration file arguments

Currently, Protostar will throw an error if an argument is invalid for any command (not only for the command that is being executed).

Can you give an example of such behaviour to reproduce the issue?


Version stays at 1

Version is always 1, no matter what you put in. It also causes the upgrade function to be called every time.
kvsEnvStorage({
  name: 'data',
  version: 2,
  upgrade: (_, old, new) => console.log('I am always called', old, new), // I am always called 1 2
})

Can you provide reproduce example?
browser:
(async () => {
  const { kvsEnvStorage } = await import("https://cdn.skypack.dev/@kvs/env@2.1.2");
  const storage = await kvsEnvStorage({
    name: "database-name",
    version: 2,
    async upgrade({ kvs, oldVersion, newVersion }) {
      console.log({
        oldVersion, // 0
        newVersion // 2
      });
    }
  });
})();

node:
https://replit.com/@azu2/kvs-env-new-version?v=1#index.js

It is reproducible it in your node replit example. Check ~/kvs-env-new-version/.cache/kvs-node-localstorage/database-name.__.__kvs_version__. Version is always 1, regardless of the version in the app.

Another example:
<EMAIL_ADDRESS>

https://github.com/azu/kvs/blob/7b4953ee016bae296d6f8ad5847803de06322823/packages/storage/src/storage.ts#L120-L131
Maybe, we need to write version to storage.

Fixed in https://github.com/azu/kvs/releases/tag/v2.1.3


Add runtime exception for misleading PrimaryKey config

When the PrimaryKey attribute is used to set a composite key, the AutoIncrement option is forced to false.

Thanks will make into the next major v4 alpha


Crawl / manually curate / map Publications resources into individual metadata records based off of schema

https://docs.google.com/spreadsheets/d/1FFyRhI5TeUb-B4t50HRZYF_XngRt_mfAIz82UQ3geFk/edit#gid=0
Need to check how often preprints from individual journals (Lancet, Nature, etc.) get incorporated into Pubmed index.  Don't want to be redundant, but also want to be as up-to-date as possible.

To start, I think harvesting from LitCovid  would be the easiest starting point.  My guess is that the PubMed/NCBI team is working hard on getting COVID-19-related literature into PubMed as quickly as possible, so I think we should old off on doing any direct scraping of journal sites.  Preprint servers like Biorxiv and Medrxiv would be more useful I think.
So to summarize, I think the priority list should be 1) LitCovid, 2) Biorxiv / Medrxiv, and 3) journal sites.
(not seeing a biorxiv/medrxiv API, but maybe can use their rss feed and/or https://www.rxivist.org/docs)...

Makes sense.
BioRxiv has a third-party API: https://api.rxivist.org/v1/papers?q=nCoV&timeframe=alltime. Little bit wonky; &metric=downloads gives different results. Unclear how frequently is updated. Could also be gotten via scraping https://www.biorxiv.org/search/ncov or Twitter feeds.  Medrxiv searches seem to include Biorxiv but bio doesn't include med.  There's also Chemrxiv, but lower priority.

plan:

Pull data from LitCOVID/Pubmed: https://ftp.ncbi.nlm.nih.gov/pub/lu/LitCovid/. Use litcovid2BioCJSON.gz to get the IDs, and then use the PubMed API to get the actual records
Scrape Biorxiv
Scripe Medrxiv
Align metadata with schema (#1 )

The mapping from PubMed/Medline to the schema can be found on the worksheet in columns A and J

Additional information to grab from efetch:
example record (not COVID-related): https://eutils.ncbi.nlm.nih.gov/entrez/eutils/efetch.fcgi?db=pubmed&id=28841416&rettype=abstract
MeSH/OT terms --> keywords
abstract --> abstract
author affiliations --> author.affiliation
grants --> funding
referenceList --> citedBy
@gtsueng has additional mapping of PubMed/Medline columns to our schema in columns A/J of the worksheet

Note that there are two methods for pulling the data from pubmed.  One method is to use the API which yields an xml file:
https://eutils.ncbi.nlm.nih.gov/entrez/eutils/efetch.fcgi?db=pubmed&id=28841416&rettype=abstract
The benefit of this method is as follows:
-Author given name and family name
-References (Citations)
-Specific grant information
The other method uses the biopython library to fetch and parse a medline record which treats it like a json file. Sample gist here.
The benefit of this method is as follows:
-json
-easy to map (the dictionary keys already mapped to schema fields in the worksheet: Publications, column J)
-PublicationType available as PT via this method
-Other terms aside also available (OT) aside from just MeSH (MH)

Abbreviated Journal titles may work better than the actual journal titles when it comes down to the visualization part. Since there are different ways to abbreviate journal titles, it may be better to use a dictionary for consistency. The NLM list of journal can be found here and parsed into a dictionary. Once parsed, a manual review/edit could help to ensure that the most common abbreviations are rather than just the NLM abbreviations (which are by no means the de facto standard).

Abbreviated Journal titles may work better than the actual journal titles when it comes down to the visualization part. Since there are different ways to abbreviate journal titles, it may be better to use a dictionary for consistency. The NLM list of journal can be found at ftp://ftp.ncbi.nih.gov/pubmed/J_Medline.txt and parsed into a dictionary. Once parsed, a manual review/edit could help to ensure that the most common abbreviations are rather than just the NLM abbreviations (which are by no means the de facto standard).

would propose having journalName be the full journal name (for instance The New England journal of medicine) and something like journalNameAbbrevation be N. Engl. J. Med.

Added into schema as suggested above. Still suggest populating from a single dictionary and caution against loading it from metadata from different sources since they may abbreviate journal titles differently.

referenceList --> citedBy (note: will need to pick an identifier to map to the other publications that cite this publication. ideally, this should refer to the _id we use to index these publications)

@flaneuse In my mind the reference list should map to isBasedOn since those would be explicitly called out/referenced in the text and the citedBy would refer to other publications that cite this publication.

referenceList --> citedBy (note: will need to pick an identifier to map to the other publications that cite this publication. ideally, this should refer to the _id we use to index these publications)

@flaneuse In my mind the reference list should map to isBasedOn since those would be explicitly called out/referenced in the text and the citedBy would refer to other publications that cite this publication.

Oh, sorry, you're right; I'm backwards.  I thought these were the citedBy listed in the PubMed entry -- I care more about those than the references they cite.  Wonder where these are hidden in the PubMed API...

I just assumed that they could query for those as long as all their other publications have references. In terms of related articles where the relationship is unclear, Medline will sometimes provide them as [AID] (associated ID). These are often related articles. Eg- if Publication A is a letter to the editor with regards to Publication B. An identifier for Publication B would be included as AID in the record for Publication A.  Unfortunately, AID is generic enough that they'll sometimes include the DOI for Publication A in the record for Publication A.

@marcodarko can you also track the LitCovid / BioRxiv / MedRxiv original url?  And maybe the source type (LitCovid, BioRxiv, or MedRxiv)?
per @andrewsu on the front-end: "assuming we get that from litcovid, would be good to add a note and/or link.  Yes, probably somewhat redundant to do it on every page, but would rather err on the side of overattribution..."
I think we want to reserve url for the actual link to the paper (I don't want to get routed through PubMed when I just want to read the paper).  I'm not sure I see a natural schema.org way to capture this information... thoughts @gtsueng?  Proposal for your reaction (as an aside: we also need to decide on what we're using for canonical IDs; this will be used as the key to link between records, and also for the url to route on outbreak.info):
{ _id: "pmid32292904",
pmid: "32292904",
doi: "10.1016/j.xkme.2020.03.001",
url: "https://www.kidneymedicinejournal.org/article/S2590-0595(20)30049-2/pdf",
sameAs: {
name: "LitCovid",
url: "https://www.ncbi.nlm.nih.gov/research/coronavirus/publication/32292904"
}
}

@marcodarko can you also track the LitCovid / BioRxiv / MedRxiv original url? And maybe the source type (LitCovid, BioRxiv, or MedRxiv)?
per @andrewsu on the front-end: "assuming we get that from litcovid, would be good to add a note and/or link. Yes, probably somewhat redundant to do it on every page, but would rather err on the side of overattribution..."
I think we want to reserve url for the actual link to the paper (I don't want to get routed through PubMed when I just want to read the paper). I'm not sure I see a natural schema.org way to capture this information... thoughts @gtsueng? Proposal for your reaction (as an aside: we also need to decide on what we're using for canonical IDs _id; this will be used as the key to link between records, and also for the url to route on outbreak.info):
{ _id: "pmid32292904",
pmid: "32292904",
doi: "10.1016/j.xkme.2020.03.001",
url: "https://www.kidneymedicinejournal.org/article/S2590-0595(20)30049-2/pdf",
sameAs: {
name: "LitCovid",
url: "https://www.ncbi.nlm.nih.gov/research/coronavirus/publication/32292904"
}
}

update: creating new base class property curatedBy to store this info

Noting source of BioRxiv data

https://github.com/marcodarko/litcovid
https://github.com/marcodarko/biorxiv


I have implement  dropdown view and run the app found an error ?

Terminating app due to uncaught exception 'NSUnknownKeyException', reason: '[<UIView 0x1489de1d0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key optionLabel.'
terminating with uncaught exception of type NSException
CoreSimulator 802.6 - Device: iPhone 13 (8DCE265B-916B-4CE6-A59C-9F89B0553862) - Runtime: iOS 15.4 (19E240) - DeviceType: iPhone 13

update your pod in podfile to 2.3.12 and you will be good to go... like pod 'DropDown', '2.3.12'

you solved issue??
I have met same problem..

i have used the source file only.
On Mon, Aug 1, 2022 at 3:25 PM airakivela @.***> wrote:

you solved issue??
I have met same problem..
—
Reply to this email directly, view it on GitHub
https://github.com/AssistoLab/DropDown/issues/304#issuecomment-1200979167,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI5BY5M6X6FO3JAFU4INN2DVW6NHTANCNFSM5UOLTJ7A
.
You are receiving this because you authored the thread.Message ID:
@.***>


Consider registering plugins in FlutterApplication

From @collinjackson:  consider autoregistering in the FlutterApplication and adding an activity lifecycle callback so plugins get added in onCreate automatically without having extra code in the client activity.

@mravn-google can you look into this a bit?

Tagging M2 as this impacts API stability

@goderbauer

It would also be nice if plugins can be informed about the "onNewIntent" event. That would for example make the setup for the firebase_messaging plugin a lot simpler, as we could then drop step 2ii from our instructions: https://github.com/flutter/firebase_messaging

I think generally FlutterActivity should allow plugins to register for onNewIntent and a lot of the other Activity methods, like onUserLeaveHint(). It might be better to fix #3671 first, since it could have impact on the API design.

https://github.com/flutter/flutter/pull/9715 and friends contains a basic approach that we can extend until it handles all the Activity methods we want. It does not tie plugins to a FlutterActivity, though still to an Activity. Comments/suggestions on the PRs are most welcome.
Android:

https://github.com/flutter/flutter/pull/9715
https://github.com/flutter/engine/pull/3641

iOS:

https://github.com/flutter/flutter/pull/9818
https://github.com/flutter/engine/pull/3645

BTW: I don't immediately see a simple way to avoid plugin registration code in the client's Activity as long as we have the auto-generated PluginRegistry.java. Calling the auto-generated code from flutter/engine framework code seems to require reflection which (in my view) isn't simple, even if it's just a few lines of code.

onUserLeaveHint and onNewIntent are now both included:
public final class MyPlugin implements
  MethodCallHandler, NewIntentListener, UserLeaveHintListener {

  public static void registerWith(Registrar registrar) {
    final MethodChannel channel = new MethodChannel(registrar.messenger(), "my");
    final MyPlugin instance = new MyPlugin(registrar.activity(), channel);
    registrar
      .addNewIntentListener(instance)
      .addUserLeaveHintListener(instance);
    channel.addMethodCallHandler(instance);
  }

  private MyPlugin(Activity activity, MethodChannel channel) { ... };

  @Override
  public void handleMethodCall(MethodCall call, Result result) { ... }
  @Override
  public void onUserLeaveHint() { ... }
  @Override
  public void onNewIntent(Intent intent) { ... }
}

Thanks! I don't have any particular attachment to onUserLeaveHint, it's just an example, but hopefully we'll get all the useful ones over time.

Nice!

Fixed by https://github.com/flutter/flutter/pull/9715 and friends.


Fix for potential issue retrieving policies from rego

Proposed changes
Sometimes, due to automatic performance optimizations done by OPA, the http.send function in rego isn't carried out, causing policies not to be retrieved and resulting in 403 responses.
Moving the retrieval of the policies to a lua filter in the envoy and passing it to opa ensured this doesn't happen anymore.
Types of changes
What types of changes does your code introduce to the project: Put
an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after
creating the PR. If you're unsure about any of them, don't hesitate to
ask. We're here to help! This is simply a reminder of what we are going
to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the Contributor License Agreement
[x] I have updated the RELEASE NOTES
[x] I have added tests that prove my fix is effective or that my
feature works
[x] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in
downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion
by explaining why you chose the solution you did and what alternatives
you considered, etc...

this is rather weird, also because by default caching is off:
https://www.openpolicyagent.org/docs/latest/policy-reference/#http
Could it be that it takes some time to flush data in the database? How did you verify the issue mentioned?

this is rather weird, also because by default caching is off: https://www.openpolicyagent.org/docs/latest/policy-reference/#http
Could it be that it takes some time to flush data in the database? How did you verify the issue mentioned?

It's mentioned here https://www.openpolicyagent.org/docs/latest/policy-reference/ , under the http.send function description.
It's not a caching mechanism, it's some sort of automatic optimisation for rego that ends up not firing off those requests. I'm not sure how exactly it decides, but after making this change I have't been able to get the scripts to fail again like we saw happen before, so either way this is probably better in the long run.

this is rather weird, also because by default caching is off: https://www.openpolicyagent.org/docs/latest/policy-reference/#http
Could it be that it takes some time to flush data in the database? How did you verify the issue mentioned?

It's mentioned here https://www.openpolicyagent.org/docs/latest/policy-reference/ , under the http.send function description.
It's not a caching mechanism, it's some sort of automatic optimisation for rego that ends up not firing off those requests. I'm not sure how exactly it decides, but after making this change I have't been able to get the scripts to fail again like we saw happen before, so either way this is probably better in the long run.

what is weird though, is that this does not happen with sql-lite right? why this makes a difference to OPA???
regardless, as mentioned, if that function has issues, better having a new built-in, rather than using lua.

This built-in function must not be used for effecting changes in external systems as OPA does not guarantee that the statement will be executed due to automatic performance optimizations that are applied during policy evaluation.

if that's the case, i.e. the call is not executed, this should be easy to spot both in OPA and in the api, right?

what is weird though, is that this does not happen with sql-lite right? why this makes a difference to OPA???

I believe I did see it happen occasionally with sqlite, but rarely. It's possible that the reason it happens more with postgres is that postgres takes up more resources, and thus causes these rego optimisation to happen more frequently.

regardless, as mentioned, if that function has issues, better having a new built-in, rather than using lua.

That's the only one built-in. Rego isn't really meant to be used to make HTTP calls, so it doesn't have much support there.

what is weird though, is that this does not happen with sql-lite right? why this makes a difference to OPA???

I believe I did see it happen occasionally with sqlite, but rarely. It's possible that the reason it happens more with postgres is that postgres takes up more resources, and thus causes these rego optimisation to happen more frequently.

regardless, as mentioned, if that function has issues, better having a new built-in, rather than using lua.

That's the only one built-in. Rego isn't really meant to be used to make HTTP calls, so it doesn't have much support there.

in fact, as by first reply, the idea would be to build a custom one:

Also, even in the case the issue you spotted is correct, it would better to create a new runtime function in go rather than inventing weird things with lua on request time: https://www.openpolicyagent.org/docs/latest/extensions/#custom-built-in-functions-in-go

what is weird though, is that this does not happen with sql-lite right? why this makes a difference to OPA???

I believe I did see it happen occasionally with sqlite, but rarely. It's possible that the reason it happens more with postgres is that postgres takes up more resources, and thus causes these rego optimisation to happen more frequently.

regardless, as mentioned, if that function has issues, better having a new built-in, rather than using lua.

That's the only one built-in. Rego isn't really meant to be used to make HTTP calls, so it doesn't have much support there.

in fact, as by first reply, the idea would be to build a custom one:

Also, even in the case the issue you spotted is correct, it would better to create a new runtime function in go rather than inventing weird things with lua on request time: https://www.openpolicyagent.org/docs/latest/extensions/#custom-built-in-functions-in-go

That warning about automatic optimisation is there on the documentation about custom functions as well regarding HTTP operations, so building a custom function won't solve that problem, since these appear to be I/O optimisations OPA is doing. I couldn't find any configuration for OPA to change this, and error/exception handling in rego isn't really a thing. Hence why I chose to use the filters to retrieved the policy data from the API prior to the OPA evaluation.

at the current stage, I don't think we should merge this PR

at the current stage, I don't think we should merge this PR

also, considering the ultimate goal is to push data from the api into opa, it's non sense to use this work around, unless the issue pops up again

@chicco785 Alright, so let's close this pull request for now, but keeping the branch in case we need to look into this issue again.


ResourceGroupName should be required

It is not clear from<EMAIL_ADDRESS>that resourceGroupName, or resourceGroupNameRef, or resourceGroupNameSelector are mandatory (one of the three).

Hi @vfarcic,
Thank for reporting this.
We had a discussion on referencer field markers with @muvaf and our conclusion back then was something similar to the following considering some of the discussion in https://github.com/crossplane/crossplane/blob/master/design/one-pager-managed-resource-api-design.md:
// +immutable
ResourceGroupName string `json:"resourceGroupName,omitempty"`
// +optional
// +immutable
ResourceGroupNameRef *xpv1.Reference `json:"resourceGroupNameRef,omitempty"`
// +optional
ResourceGroupNameSelector *xpv1.Selector `json:"resourceGroupNameSelector,omitempty"`

Right now, what we are generating with Terrajet is:
// +kubebuilder:validation:Optional
ResourceGroupName *string `json:"resourceGroupName,omitempty" tf:"resource_group_name,omitempty"`

// +kubebuilder:validation:Optional
ResourceGroupNameRef *v1.Reference `json:"resourceGroupNameRef,omitempty" tf:"-"`

// +kubebuilder:validation:Optional
ResourceGroupNameSelector *v1.Selector `json:"resourceGroupNameSelector,omitempty" tf:"-"`

Btw, we can generalize this issue for all cross-resource referencer fields, and probably discuss this in API design doc.

Regardless of the markers, because we have no marker defining a set of fields, we had also better to link this to:
https://github.com/crossplane-contrib/terrajet/issues/140
One alternative could be just generating the referencer fields (specified with the +crossplane:generate:reference markers) with appropriate comments.

@turkenh do you think if it'd be possible to add a one-liner before the referencer marker in the description of the field to say at least one of the three needs to be given if the field is required originally?


[btcd] Failed to parse request

There is an error with btcd json: cannot unmarshal array into Go value of type btcjson.Request:
$ counterparty-server start
[INFO] Running v1.1.0 of counterparty-server.
[INFO] Running v9.51.1 of counterparty-lib.
[INFO] Acquiring lock.
[DEBUG] Lock acquired.
[INFO] Connecting to database.
[DEBUG] Creating connection to `.config/counterparty-server/counterparty.db`.
[DEBUG] Checking database integrity.
[INFO] Connecting to backend.
[DEBUG] Starting API Status Poller.
[DEBUG] Creating connection to `.config/counterparty-server/counterparty.db`.
[DEBUG] Checking backend state.
[INFO] Starting API Server.
[DEBUG] Checking version.
[DEBUG] Creating connection to `.config/counterparty-server/counterparty.db`.
[DEBUG] Backend state check passed.
[DEBUG] Checking database state.
[DEBUG] API Status Poller: Counterparty database is behind backend.
[DEBUG] Version check passed.
[DEBUG] Checking database version.
[INFO] Resuming parsing.
[DEBUG] Initialize mempool cache
[ERROR] Unhandled Exception
Traceback (most recent call last):
  File "/bin/counterparty-server", line 9, in <module>
    load_entry_point('counterparty-cli==1.1.0', 'console_scripts', 'counterparty-server')()
  File "/usr/lib/python3.4/site-packages/counterpartycli/__init__.py", line 16, in server_main
    server.main()
  File "/usr/lib/python3.4/site-packages/counterpartycli/server.py", line 135, in main
    server.start_all(db)
  File "/usr/lib/python3.4/site-packages/counterpartylib/server.py", line 372, in start_all
    blocks.follow(db)
  File "/usr/lib/python3.4/site-packages/counterpartylib/lib/blocks.py", line 912, in follow
    backend.init_mempool_cache()
  File "/usr/lib/python3.4/site-packages/counterpartylib/lib/backend/__init__.py", line 237, in init_mempool_cache
    mempool_tx = BACKEND().getrawtransaction_batch(txhash_list, verbose=True)
  File "/usr/lib/python3.4/site-packages/counterpartylib/lib/backend/btcd.py", line 125, in getrawtransaction_batch
    batch_responses = rpc_batch(payload)
  File "/usr/lib/python3.4/site-packages/counterpartylib/lib/backend/btcd.py", line 74, in rpc_batch
    responses += rpc_call(chunk)
  File "/usr/lib/python3.4/site-packages/counterpartylib/lib/backend/btcd.py", line 54, in rpc_call
    raise BackendRPCError('{}'.format(response_json['error']))
counterpartylib.lib.backend.btcd.BackendRPCError: {'message': 'Failed to parse request: json: cannot unmarshal array into Go value of type btcjson.Request', 'code': -32700}

This was encountered with the development branch of counterparty-lib and counterparty-cli.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11559740-btcd-failed-to-parse-request?utm_campaign=plugin&utm_content=tracker%2F494307&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F494307&utm_medium=issues&utm_source=github).

should be fixed with: https://github.com/CounterpartyXCP/counterpartyd/commit/3f874edddfe1da7fcc08fe1143cc35738d31118d


Don't save extraneous state when saving connection profiles

A saved connection profile shouldn't have state related to the active taskspace or connection.  Ignore taskspace inst var when writing out ston profile. One method addition.

I believe this is superseded by #88


Unable to batch DELETE objects by id

Hi there, I am trying to batch DELETE objects by id, but it fails on 1.5.0
`
await this.client.batch
    .objectsBatchDeleter()
    .withClassName('whatever')
    .withWhere({
      path: ['id'],
      operator: 'ContainsAny',
      valueTextArray:[x,y,z],
    })
    .do()

`
The error I am getting is: usage error (422): {"code":606,"message":"match.match.where.operator in body should be one of [And Or Equal Like Not NotEqual GreaterThan GreaterThanEqual LessThan LessThanEqual WithinGeoRange IsNull]"}

This looks like a server error (due to the 422 report)!
Can you confirm whether you're using Weaviate 1.21.x? ContainsAny/All were introduced in that release 😁

Hi @tsmith023, yes, that was the issue. Updated to 1.21.3 and works like a charm!


Refactor codebase to async/await - m365 planner task

Closes #5004

and another one 👍. Nice 🤩. We will review it ASAP


Interactive training save to domain fail when domain file has forms section on 0.12.0

Rasa Core version:0.12.0
Python version: 3.6
Operating system (windows, osx, ...):osx
Issue:
When save domain file after interactive training, if the old domain file has forms section, it fails
  File "/usr/local/lib/python3.6/site-packages/rasa_core/domain.py", line 251, in __init__
    action_names) + form_names
TypeError: can only concatenate list (not "dict") to list

As in training/interactive.py. there is no hander of forms section and it will be a empty dict.  But in domain.py Domain init forms should be list
training/interactive.py
domain_dict = dict.fromkeys(domain.keys(), {})  # type: Dict[Text, Any]
domain_dict["intents"] = _intents_from_messages(messages)
domain_dict["entities"] = _entities_from_messages(messages)
domain_dict["actions"] = list({e["name"] for e in actions})

new_domain = Domain.from_dict(domain_dict)

domain.py
self.action_names = action.combine_user_with_default_actions(
action_names) + form_names

Content of domain file (if used & relevant):

Could you post the domain file you're talking about and the command you used and when this error occurred? It's hard to understand what you mean

Having the same issue, happens if you add some form to your domain.yml, e.g.:
forms:
  - login_form

Same here. @akelad Issue is not with some specific domain file. As @cookie314 pointed out, in the file interactive.py, empty dict is created for each domain present in the domain: https://github.com/RasaHQ/rasa_core/blob/master/rasa_core/training/interactive.py#L805 But domain 'forms' (as well as 'intents', 'entities' and 'actions') needs to be list, in further processing.
See attached pull request.

yeah i know, i just didn't quite get the problem initially. But now I understand, we'll make sure to fix this


Dashed files routing in public is broken in production

Environment

Operating System: Linux
Node Version:     v22.1.0
Nuxt Version:     3.13.1
CLI Version:      3.13.1
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      compatibilityDate, devtools
Runtime Modules:  -
Build Modules:    -

Reproduction
https://github.com/daniluk4000/nuxt-dashed-routing
yarn && yarn build && yarn preview
Then visit both /file.txt and /-file.txt
Describe the bug
When file started with - is present in public directory, it will not be displayed in production only.
We really need this for static storybook build (:
Additional context
I have really hoped this issue will be on our side.
We didn't have this issue in <3.13.0 (probably).
Logs
No response

This is because files beginning with - are by default ignored by Nuxt (see https://nuxt.com/docs/api/nuxt-config#ignore). If you do not wish this to be the case you can modify this option:
export default defineNuxtConfig({
  modules: [
    function (options, nuxt) {
      nuxt.options.ignore = []
    }
  ]
})

I don't believe this is a new change.

And I was searching for an issue in globby... maybe we just were lucky before.
That worked, thanks so much!


KCP does not update api-server's list of etcd node ips

KCP does not update api-server's list of external etcd's node ips when the etcd node's ip change from DHCP issues. This happens especially when just the etcd node's ip changes without the node going down.

I see similar issue , what was the root cause and how did you fix it ??


Opponent position in low_level_features_set

I am going to train my agent for dribbling the opponent using DQN.
I am using the low_level_features_set in order to have access to velocities and all players orientation.
The problem is that according to the HFO manual, I only have access to 8 feature of the opponent which is :
0- pos_valid
1-vel_valid
2-3-vel_ang
4-vel_mag
5-6-ang
7- Frozen
(if I am not mistaken, because the manual do not explain exactly and I believe that the features should be the first 8 features which are available for self)
These features to not indicate the exact position of the opponent or even it's relative position to self which I need for the feature set and reward.
How can I have these data ?
Did I understand the features correctly cause it seems a bit weird that through these data we can not get any useful information from the opponent.
Thanks

Yes - it looks like the manual is not very specific on this point (if you want to add some better information, happy to accept PR). Here are the features for each opponent: https://github.com/LARG/HFO/blob/master/src/feature_extractor.cpp#L75
1-3: Distance and angle to the opponent (e.g. relative position)
4-5: Opponent's own angle (e.g. where are they facing)
6: Opponents velocity magnitude
7-8: Opponent's velocity angle
Hope this helps!

Thanks for your answer.
Also there is another point that I can not understand about the opponents.
When the full_state flag is set to false, there are lots of cycles that the agent do not see the opponent, as I checked whenever it can not see the opponent the velocity is set to zero but the position still gets updated.
How is the position calculated during these cycles?
Thanks

I'm not sure how/why the position is getting updated as this happens at a lower level of the code than the HFO interface.
For what it's worth, here is some speculation: When full-state is off the agent should only receive updates from the things inside of its view cone. In particular if other agents aren't inside of the view cone, they will likely not be updated. However there may be a couple of reasons why you would see such an update:

If player's agent moves, perhaps all the other objects on the field (opponents included) have their relative positions updated.
If there are teammates who view the opponent, they can potentially communicate their observations about the opponent's updated position.


Update to UriHelper.OnLocationChanged required

This UriHelper.OnLocationChanged event has new parameters that were causing an error with this example

Thank you!


Feature to directly upload resume from a job page.

There should be an option for an applicant to upload resume directly from the job page.

Solved. Added functionality to upload resume directly.


Broken Link in /blog/2023/05/24/oci-security-profiles/

This is a Bug Report
Problem:
Link for Bundle Bar  in https://kubernetes.io/blog/2023/05/24/oci-security-profiles/#managing-oci-security-profiles is broken.
Proposed Solution:
Should be updated with correct link
Page to Update:
https://github.com/kubernetes/website/blob/main/content/en/blog/_posts/2023-05-24-oci-security-profiles.md

/language en

/area blog

There seems to be an issue with the Bundle Bar registry, as it is currently unavailable. I think there is no link that can be updated.

There seems to be an issue with the Bundle Bar registry, as it is currently unavailable. I think there is no link that can be updated.

@AmarNathChary  yes it seems same to me also so i think we can either remove the bundle bar  from the list or  should wait for more comment  from community for the new link.

/help

/triage accepted
@niranjandarshann it's best not to mark issues as help wanted when they are not also triaged accepted

As the Bundle Bar registry is not working. I think a proper fix could be removing the link for the Bundle Bar

@niranjandarshann you can assign this issue if what to fix it

As the Bundle Bar registry is not working. I think a proper fix could be removing the link for the Bundle Bar

Agree with you @AmarNathChary . @sftim what is your opinion. can we remove Bundle Bar

/assign

As the issue is solved now we can close the issue

Thanks @AmarNathChary  for resolving the issue. Marking the issue resolved as per PR #41533 is merged.
/close


fix: update preload.php for 4.5

Description
Fixes #8801
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Thank you @kenjis


Admin after NameChange in Default group

Hey,
I have an Issue. When I change my Name in Minecraft, my Admin is not there and I'm in the Default group.
I use Spigot 1.8.8 and BungeeCord. The Server is running on online-mode= false .
How can I fix this?

Update to a version of PEX that uses UUIDs, and convert your user database -- check the wiki to see how.


Trying to scale a nonexistent process type results in 400, not a 404

POST /v2/apps/<id>/scale/ Body: {"foo": 2}
will return 400: {"detail":"Container type foo does not exist in application"}

I think this is intentional. /v2/apps/appid/scale is a a valid endpoint, but the data received is incorrect.

Over most of the rest of the API, we use get_object_or_404 when loading child objects, so in this situation they return a 404.


Add integration_test fix tests to analyze_flutter_flutter.sh

As part of flutter #124346, we're adding support for dart fix to the integration_test library. Currently, analyze_flutter_flutter.sh only runs the fix tests for the main flutter package, but it may be worth adding the integration_test fix tests to the script.

@srawlins @devoncarew
tools/bots/flutter/analyze_flutter_flutter.sh
This test is running on the https://ci.chromium.org/ui/p/dart/builders/ci.sandbox/flutter-analyze builder


Change the Bg color game

I want to contribute here under the hacktoberfest.
@vibgyor786 please assign me the task

Assigned it to you
Make sure you add the game in React and go through readme.md and contributing.md

@cleveranu please assign me under GSSOC@#

I'm here to contribute to the GSSOC'23.
Please assign me to do that.

@cleveranu which tags are required fro GSSOC23 mam like please assign and add the tags of difficulty also

i would like to contribute to this under GSSOC'23

i would like to contribute to this under GSSOC'23


bigint generate to string,how to generate to number

  @PrimaryGeneratedColumn({ type: "bigint", name: "id", comment: "id" })
  id: string;

Generating number would not be a correct behavior here.
Sql Bigint type support larger numbers then JS Number type.  If you use JS Number type you can loose your data without noticing it(no overflow errors).


Disable radio button

Details

We disable radio button when this option is disabled
Fixed Issues

$ https://github.com/Expensify/App/issues/28331
PROPOSAL: https://github.com/Expensify/App/issues/28331#issuecomment-1737787088
Tests

Create a group chat
On the group chat click on the plus icon > Split bill
Enter a valid amount and continue
Hover on the radio button of payer
Verify that the cursor is disabled

[x] Verify that no errors appear in the JS console

Offline tests

Same above
QA Steps

Create a group chat
On the group chat click on the plus icon > Split bill
Enter a valid amount and continue
Hover on the radio button of payer
Verify that the cursor is disabled

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
Screencast from 05-10-2023 10:36:10.webm

Mobile Web - Chrome
https://github.com/Expensify/App/assets/129500732/7d2fa84c-ae77-487a-bf3a-8a713409738f

Mobile Web - Safari
https://github.com/Expensify/App/assets/129500732/43b5d796-1b3e-4293-be78-982bb3aaec5c

Desktop
https://github.com/Expensify/App/assets/129500732/d77c6dda-74fe-44e5-a655-da34d340d1ce

iOS
https://github.com/Expensify/App/assets/129500732/143d3f97-a54d-489a-865e-746aa2bca93c

Android
28331.webm

Sorry bout that was unexpectedly ill this past week. @sobitneupane feel free to ping me next time.
@dukenv0307 you got conflicts. code LGTM.

@dukenv0307 As the issue is already fixed by another PR, you can close this PR.


Feature Request: UDRs to custom object

Feature Request: UDRs to custom object.
Use case: Opportunities representing restricted donations to funds managed by a foundation.  Funds have to be a custom object rather than Accounts because a donation can be received from an Account (Opp.Account), with that donation "residing in" a restricted Donor Advised Fund.  The Donor Advised Fund (DAF) is a standard construct of foundations and can have contact info associated with it so it needs to be an object.  If UDRs could rollup to custom objects, then the total of donations into the fund, and dispersed grants out of the fund, could be UDR fields on the Fund record.
Current solution is to use 3rd party vendor RollupHelper.

GAUs should solve for this


Improve game starter args

Besides refactoring, at least these changes are included:

--human protoss argument removed and merged into --player1 human.protoss
Fix -release to -r
Fix -map to --map
Add -m along with --map
State defaults of each parameter in help text
Sort available maps and bots in helpo text

When this PR is merged, I would like to add different usages scenarios into Running Games wiki page:
Bot vs. bot
Zerg 12 pool against protoss proxy zealot rush on random map.
py run_custom.py -p1 12pool -p2 zealot

Bot vs. in-game AI
Zerg 12 pool against in-game AI.
py run_custom.py -p1 12pool -p2 ai.terran.hard

Human vs. bot
Human terran against cannon rush on on AcolyteLE map.
py run_custom.py -p1 human.terran -p2 cannonrush --map AcolyteLE


fix a few compiler warnings

This commit fixes a few compiler warnings. It also adds some Windows #ifdef logic. The resulting Windows code does not work, but it didn't work before this either (the goal is to get it compiling).
Fixes: https://github.com/cjihrig/uvwasi/issues/26 (I hope)
Green CI: https://github.com/cjihrig/uvwasi/commit/5df92f87e3259a7c941a83fd0c4449e230127780/checks?check_suite_id=262439257 (apologies if I'm doing something wrong with the GitHub CI, this is my first exposure to it)
cc: @gengjiawen

I can test it tormorrow :)

Looks like something wrong still with cmake windows config, but the previous error goes away.
Looks like error building libuv
C:\projects\appveyor-playground\uvwasi\cmake-build-debug\_deps\libuv-src\src\win\udp.c: In function 'uv__udp_set_source_membership6':
C:\projects\appveyor-playground\uvwasi\cmake-build-debug\_deps\libuv-src\src\win\udp.c:793:15: error: 'MCAST_JOIN_SOURCE_GROUP' undeclared (first use in this function); did you mean 'ERROR_NO_SUCH_GROUP'?
     optname = MCAST_JOIN_SOURCE_GROUP;
               ^~~~~~~~~~~~~~~~~~~~~~~
               ERROR_NO_SUCH_GROUP
C:\projects\appveyor-playground\uvwasi\cmake-build-debug\_deps\libuv-src\src\win\udp.c:793:15: note: each undeclared identifier is reported only once for each function it appears in
C:\projects\appveyor-playground\uvwasi\cmake-build-debug\_deps\libuv-src\src\win\udp.c:795:15: error: 'MCAST_LEAVE_SOURCE_GROUP' undeclared (first use in this function); did you mean 'UV_LEAVE_GROUP'?
     optname = MCAST_LEAVE_SOURCE_GROUP;
               ^~~~~~~~~~~~~~~~~~~~~~~~
               UV_LEAVE_GROUP
mingw32-make.exe[3]: *** [_deps\libuv-build\CMakeFiles\uv_a.dir\build.make:483: _deps/libuv-build/CMakeFiles/uv_a.dir/src/win/udp.c.obj] Error 1
mingw32-make.exe[2]: *** [CMakeFiles\Makefile2:369: _deps/libuv-build/CMakeFiles/uv_a.dir/all] Error 2
mingw32-make.exe[1]: *** [CMakeFiles\Makefile2:84: CMakeFiles/uvwasi.dir/rule] Error 2
mingw32-make.exe: *** [Makefile:117: uvwasi] Error 2

@gengjiawen you might be running into https://github.com/libuv/libuv/commit/2140e2c08cb1137d43da5b1be8999c5509f2b609, which is scheduled to be released in libuv 1.33.0, but is broken in 1.32.0, which is what we're using here. You can try applying that commit to your local libuv and see if that makes a difference.
I also noticed that in the CI, Windows runs seem to be coming back green, but it doesn't look like anything is actually compiled. For example, compare the "build it" step in https://github.com/cjihrig/uvwasi/commit/e8146cf7c05a4721a63c625c2ef4b3ec11d8a53a/checks?check_suite_id=262775582 on Windows and the other platforms.

I also noticed that in the CI, Windows runs seem to be coming back green

Yeap. Need to separate envinfo to make it run on windows.
      - name: Envinfo
        run: npx envinfo
      - name: build it
        run: |
          cmake .
          cmake --build . --target uvwasi
        env:
          CI: true

compiler error
C:\projects\appveyor-playground\uvwasi\src\uvwasi.c(1156): error C2036: 'void *': unknown size
C:\projects\appveyor-playground\uvwasi\src\uvwasi.c(1161): error C2036: 'void *': unknown size

I updated https://github.com/cjihrig/uvwasi/pull/29 with your suggestion from https://github.com/cjihrig/uvwasi/pull/27#issuecomment-541379454. The legitimate Windows build failures are showing up now. Thanks!

New compiler error
C:\projects\appveyor-playground\uvwasi\src\uvwasi.c(1156): error C2036: 'void *': unknown size
C:\projects\appveyor-playground\uvwasi\src\uvwasi.c(1161): error C2036: 'void *': unknown size

After resolve this, I get
fd_table.c.obj : error LNK2019: unresolved external symbol _S_ISREG referenced in function _uvwasi__get_type_and_rights
fd_table.c.obj : error LNK2019: unresolved external symbol _S_ISDIR referenced in function _uvwasi__get_type_and_rights
fd_table.c.obj : error LNK2019: unresolved external symbol _S_ISSOCK referenced in function _uvwasi__get_type_and_rights
fd_table.c.obj : error LNK2019: unresolved external symbol _S_ISFIFO referenced in function _uvwasi__get_type_and_rights
fd_table.c.obj : error LNK2019: unresolved external symbol _S_ISBLK referenced in function _uvwasi__get_type_and_rights
fd_table.c.obj : error LNK2019: unresolved external symbol _S_ISCHR referenced in function _uvwasi__get_type_and_rights

@gengjiawen you might be running into libuv/libuv@2140e2c, which is scheduled to be released in libuv 1.33.0, but is broken in 1.32.0, which is what we're using here. You can try applying that commit to your local libuv and see if that makes a difference.

Also, this commit works.


Mini Peaks

Before jumping into Pr first comment for assign.
Problem statement : Write a function that returns all the elements in an array that are strictly greater than their adjacent left and right neighbors.
Probelm Link :  https://edabit.com/challenge/5KAsvzdc9DLfEx65t
Choose one Cateorgy - Medium
question category : Arrays, Loops

Before submitting a PR please Check some details.

[ ] fileName formet should be in camelcase
example- returnTheNextNumber.py
[ ]  Choose label for Problem , example- "good first issue", "easy"
[ ]  In file there should be problem statement and link to problem

/* Problem-Task : This program will add two numbers
 * Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
*/
int main() {
    int bla;
    return 0;
}

This repository is vaild for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck,

@aayushi-droid Please assign this to me

I would like to work on this

I would love to work over this.

i would like to contribute to this, please assign?

please assign it to me. thank you

Please assign this to me! Thank you!

please assign me

Please assign me

Please assign me

Please assign this issue to me. I believe I would be able to do justice to this issue.

Please assign this to me

Please assign this to me.


Fix temperature sensor's unit of measurement for HomeAssistsant

This fixes the unit of measurement for temperature sensors when being used in HomeAssistant's auto discovery feature. My sensors were throwing errors when being used inside of HomeAssistant.
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/const.py#L343
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/util/temperature.py#L28

@adrum Thanks for your pr but I can't accept it because after long discussion with openzwave author he has highly recommended to not use string compare inside code, even for units, because starting from ozw 1.6 there is multi language support and those will fail. I suggest you to fix the configurations files of your devices in ozw database

There is not a config file variable here. There is a ValueID exposed for each sensor  (>255) that allows you to select which “unit” the device should send (C/F). You could use that to find the available scales and which one is selected.
That being said - there are devices out there that ignore it and have a configuration option you need to change to change the scale.

Ok no problem. Thank you for your consideration. I was originally going to go down that path, but thought this approach might worthwhile.
Thank you providing this great software.


APIException REPR broken when using multiple entries

Describe the bug

When raising an APIException with multiple RedditErrorItem objects, the REPR breaks and doesn't stringify the items correctly.
Code/Logs
In [9]: raise RedditAPIException([RedditErrorItem("arg1", "arg2", "arg3"), RedditErrorItem("arg1", "arg2", "arg3")])
---------------------------------------------------------------------------
RedditAPIException                        Traceback (most recent call last)
<ipython-input-9-b19562183eb5> in <module>
----> 1 raise RedditAPIException([RedditErrorItem("arg1", "arg2", "arg3"), RedditErrorItem("arg1", "arg2", "arg3")])

RedditAPIException: (RedditErrorItem(error_type='arg1', message='arg2', field='arg3'), RedditErrorItem(error_type='arg1', message='arg2', field='arg3'))

System Info

OS: MacOS 12.14.6
Python: 3.7.7
PRAW Version: 7.0.0, 7.0.1dev0

It would be nice if you clarified what is "incorrect" about this repr. Perhaps alongside the actual output you could show desired or "correct" output.

It would be nice if you clarified what is "incorrect" about this repr. Perhaps alongside the actual output you could show desired or "correct" output.

Updated!


Feature: Create operational script to cleanup orphaned role assignments

Parameter to:

delete: all, only MIs, only MIs and SPN

This feature will not be part of EPAC


No watch file when extraFileExtensions on WatchCompilerHost

TypeScript Version:  3.9 up

Search Terms:
extraFileExtensions, WatchCompilerHost, vue
Code
https://github.com/yoyo930021/TypeScript/blob/970d435c8d5848a369b5516a5e597f63f50afae6/src/testRunner/unittests/tscWatch/watchApi.ts#L127
The code is a test about this.
When filename ends with ts, It is pass,
but filename ends with vue, it isn't pass.
Expected behavior:
The test is pass.
Actual behavior:
The test is not pass.
Playground Link:
No
Related Issues:
No
The typescript-eslint tool is broken when vue with typescript 3.9.
The @sheetalkamat add a PR to fix it in https://github.com/microsoft/TypeScript/pull/37726.
But this PR is only work in startup.
When we add a new file, The WatchCompilerHost isn't watch this.
ref: https://github.com/typescript-eslint/typescript-eslint/issues/2127

I dont understand this issue. We have a test https://github.com/yoyo930021/TypeScript/blob/970d435c8d5848a369b5516a5e597f63f50afae6/src/testRunner/unittests/tscWatch/watchApi.ts#L156 as you pointed out that passes where new .vue file is detected and added to the program

Oh Sorry about that. i thought the test you pointed is already in there so thats where confusion came from.

Thanks.
I tested well with typescript-eslint and Vue files.
When will this fix be released?

@DanielRosenwasser can we pick this for 4.1 ?

If you already merged it into master, it already has been 😅


🐛 [BUG]: Can not use :not() selector in dragHandle

Is there an existing issue for this?

[X] I have searched the existing issues and this is a new bug.

Current Behavior
Node's props:
{ id: id, dragHandle: 'div:not(.input-area)' }
It still can drag by div with class 'input-area'
Expected Behavior
Can not drag by div with class 'input-area'
Steps To Reproduce
No response
Relevant log output
No response
Anything else?
No response

The string for drag handle has to be a valid selector for element.querySelector.
Not sure what this issue is supposed to report anyway so im closing it until there’s an actual bug to report here.


Logstash Version 5x & 6x support

Hi any chance for supporting Logstash Version 5x and 6x?
When installing on Logstash 5x and 6x you get the following error:
logstash-input-s3 (>= 0) java depends on
logstash-mixin-aws (>= 0) java depends on
logstash-core-plugin-api (<= 2.99, >= 1.60) java

This plugin hasn't been updated to use the current version of the logstash plugin api, which is why it's not supported on modern Logstash.
We would welcome a Pull-Request that adds support, but do not currently have plans to do so.


How to Import a GPX file as track

Hey Community
We did a Track with the App "Outdooractive" and exported the recorded track to a gpx file.
I enabled file drop on the worldmap and set a "read file" on the gpx file, wired it to "worldmap tracks" and further wired to the worldmap but I do not understand how to get the gpx track into the worldmap.
Debug log shows the content from the gpx file but after this nothing more happens
Can anyone explain how to go further with this?

Example GPX file is linked here
https://nc.hagels.net/index.php/s/rWGNBmNdHAFcF3E

I am able to read the gpx file, but I don't understand in which order, I have to push the trkpt to the worldmap- or track-node. Can anyone explain how the workflow should look like?

I switched to the gpx overlay command, but still unable to get anything printed to my map. Someone can help?
[{"id":"847a8e7aebfb5838","type":"tab","label":"Dev","disabled":false,"info":"","env":[]},{"id":"b4b39e021dbc0882","type":"file in","z":"847a8e7aebfb5838","name":"","filename":"/data/map_data/sup-giethoorn.gpx","filenameType":"str","format":"utf8","chunk":false,"sendError":false,"encoding":"none","allProps":true,"x":240,"y":100,"wires":[["304dd74f3c2b2d50"]]},{"id":"e97d74407de83614","type":"inject","z":"847a8e7aebfb5838","name":"","props":[{"p":"payload"},{"p":"topic","vt":"str"}],"repeat":"600","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"","payloadType":"date","x":130,"y":40,"wires":[["b4b39e021dbc0882"]]},{"id":"7ebdf43a8416a443","type":"link out","z":"847a8e7aebfb5838","name":"link out 1","mode":"link","links":["599be14b22c9aa81"],"x":735,"y":60,"wires":[]},{"id":"dedc10b82b2d52a1","type":"debug","z":"847a8e7aebfb5838","name":"debug 1","active":true,"tosidebar":true,"console":true,"tostatus":false,"complete":"payload","targetType":"msg","statusVal":"","statusType":"auto","x":710,"y":180,"wires":[]},{"id":"fddabac3720fd9e0","type":"function","z":"847a8e7aebfb5838","name":"function 2","func":"msg.payload = {\n    \"command\": {\n        \"map\": {\n            \"overlay\": \"Touren\",\n            \"gpx\": msg.payload.track\n        }\n    }\n};\nreturn msg;","outputs":1,"noerr":0,"initialize":"","finalize":"","libs":[],"x":520,"y":120,"wires":[["7ebdf43a8416a443","dedc10b82b2d52a1"]]},{"id":"304dd74f3c2b2d50","type":"change","z":"847a8e7aebfb5838","name":"","rules":[{"t":"move","p":"payload","pt":"msg","to":"payload.track","tot":"msg"}],"action":"","property":"","from":"","to":"","reg":false,"x":320,"y":180,"wires":[["fddabac3720fd9e0"]]}]

msg.payload = {
    "command": {
        "map": {
            "overlay": "Touren",
            "gpx": msg.payload.track
        }
    }
};
return msg;

I don`t know what happens, but I think it was a timing problem. Route is now visible on the map


Keyboard is closed on orientation change on Google Pixel/Android 14

Steps to reproduce

Run the app on Google Pixel with Android 14 (we checked on Google Pixel 5, 6, and 7).
Tap on the text field to open the keyboard.
Rotate device to change orientation.

Expected results
Keyboard is open.
Actual results
Keyboard is closed. (UPD: the field still has focus and text caret is blinking.)
It works fine on our test Redmi with Android 10 and several other devices. Also it works fine on Pixel C with Android 14.
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'Keyboard bug',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: TextFormField(),
      ),
    );
  }
}

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.22.3, on Microsoft Windows [Version 10.0.22631.3880], locale uk-UA)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.10.4)
[√] Android Studio (version 2021.1)
[√] VS Code (version 1.91.1)
[√] Connected device (4 available)
[√] Network resources

• No issues found!

same issue with Google Pixel

This may or may not be a bug in flutter. I tested some other android apps (one compose sample) and some google apps, for some, the keyboard disappeared after changing orientation, in others, it did not.
Whatsapp, Docs, Drive -> Keyboard retained after switching orientation
Telegram, Gmail, Compose Sample, Flutter Sample -> Keyboard dismissed after switching orientation
The apps that retain the keyboard after switching orientation seem to handle it differently. Whatsapp enters a fullscreen with the keyboard and the text field (and you don't end up seeing the content). Docs and Drive, otoh, seem to relaunch the keyboard on switching the orientation.
Labeling for further insight from the team.

flutter
compose
gmail
docs

compose sample
package com.example.textfield
import android.os.Bundle
import androidx.activity.ComponentActivity
import androidx.activity.compose.setContent
import androidx.compose.foundation.layout.Arrangement
import androidx.compose.foundation.layout.Column
import androidx.compose.foundation.layout.fillMaxSize
import androidx.compose.material3.MaterialTheme
import androidx.compose.material3.Surface
import androidx.compose.material3.Text
import androidx.compose.material3.TextField
import androidx.compose.runtime.Composable
import androidx.compose.runtime.mutableStateOf
import androidx.compose.runtime.remember
import androidx.compose.ui.Alignment
import androidx.compose.ui.Modifier
import androidx.compose.ui.tooling.preview.Preview
import com.example.textfield.ui.theme.TextfieldTheme
class MainActivity : ComponentActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContent {
            TextfieldTheme {
                // A surface container using the 'background' color from the theme
                Surface(modifier = Modifier.fillMaxSize(), color = MaterialTheme.colorScheme.background) {
                    Greeting("Android")
                }
            }
        }
    }
}

@Composable
fun Greeting(name: String, modifier: Modifier = Modifier) {
    val text = remember {
        mutableStateOf("")
    }
    Column(verticalArrangement = Arrangement.Center, modifier = Modifier.fillMaxSize(), horizontalAlignment = Alignment.CenterHorizontally) {
        TextField(value = text.value, onValueChange = {
            text.value = it
        })
    }
}

flutter doctor -v
[✓] Flutter (Channel stable, 3.22.3, on macOS 14.4.1 23E224 darwin-arm64, locale en-GB)
    • Flutter version 3.22.3 on channel stable at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (9 days ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[!] Flutter (Channel master, 3.24.0-1.0.pre.291, on macOS 14.4.1 23E224 darwin-arm64, locale en-GB)
    • Flutter version 3.24.0-1.0.pre.291 on channel master at /Users/nexus/dev/sdks/flutters
    ! Warning: `flutter` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 98c5e683fd (10 hours ago), 2024-07-25 21:20:11 -0400
    • Engine revision 8714b54a87
    • Dart version 3.6.0 (build 3.6.0-83.0.dev)
    • DevTools version 2.37.1

Yes, apps behave differently, I did test some of them either to understand what is "by design". But I think the Flutter app's behavior should be consistent and predictable. Now we have different behavior on different Android devices.
In my real app I have a place where keyboard is open after changing orientation, but sometimes it doesn't. I think it can be related to the app itself, that's why I created a very basic testing app.

Thanks for all of that research @danagbemava-nc.
I agree that in an ideal world a Flutter app developer should be able to decide if the keyboard should be hidden on orientation change or not. I did some research and I think the call to close the keyboard is happening at the engine level somewhere, because I see no call being made to TextInput.hide from the framework.
https://github.com/flutter/flutter/blob/715e476545e5504a57cd3972a1966e0d1f20623f/packages/flutter/lib/src/services/text_input.dart#L2323

import 'package:flutter/material.dart';
void main() {
runApp(const MyApp());
}
class MyApp extends StatelessWidget {
const MyApp({super.key});
@override
Widget build(BuildContext context) {
return const MaterialApp(
title: 'Keyboard bug',
home: MyHomePage(),
);
}
}
class MyHomePage extends StatefulWidget {
const MyHomePage({super.key});
@override
State createState() => _MyHomePageState();
}
class _MyHomePageState extends State {
@override
Widget build(BuildContext context) {
return Scaffold(
body: Center(
child: TextFormField(),
),
);
}
}


MP-CNN PyTorch Re-implementation

Re-implementation of MP-CNN in PyTorch.
Currently this re-implementation can get the following numbers, using parameters similar to the ones in the README.

This is before a hyperparameter grid search, so we can expect to get better results if we do one.
@hohoCode please review?
Next step is to explore symmetry, external features, attention, and the bells and whistles..

Thanks @Impavidity, I'm would be happy to refactor the code to use torchtext in the near future or after it comes out of the WIP stage.

They look great Micheal. I do not have the write access though but someone with the access should merge this whole thing!

Lots of good efforts are in and this is great.


Getting error trying to use this extension with bitbucket

Uncaught Error: AMD module bitbucket-plugin/pullrequest-list-modifier must not depend on internal module(s) bitbucket/internal/feature/pull-request/pull-request-table.
The server extension drop list isn't even showing on the PR page.  I'm just getting a console error.
Doug

@ra100 @zzzep  Folks,
if you need just to replace the list of reviewers - I've written a JS script for BitBucket 6.7 and can publish it.
It is run with the help of ViolentMonkey on chosen domain and creates a link "REVIEWERS" that replaces the list.

The gist script looks similar to this:
https://github.com/dragouf/Stash-Reviewers-Chrome-Extension/blob/master/extension/chrome/src/js/stash_page.js#L357-L414
What is the main difference?
I guess the issue, metioned above, is problem with PR list. Maybe just disabling the list would solve the issue.

@ra100 yes, code works.
Did not expect that bitbucket has jquery.
I've reverse-engineered the API requests as I made a wrong conclusion based on GitHub issues that API has changed ;)

I guess the issue, metioned above, is problem with PR list. Maybe just disabling the list would solve the issue.
My BB throws an url containing the issue description:
Uncaught Error: AMD module bitbucket-plugin/pullrequest-list-modifier must not depend on internal module(s) bitbucket/internal/feature/pull-request/pull-request-table. See the API changelog for details: https://developer.atlassian.com/server/bitbucket/reference/api-changelog/#expected-future-changes     at g (require-lite.js?....)
Seems they've blocked some internal API and it won't worked without major refactoing.


Data science refactor 1

Thanks!

Solid work. Left a few notes. Make some corrections and move on, no resubmission needed.


scattered reloc r_address too large for architecture i386

How frequently does the bug occur?
All the time
Description
After major update from 10.7.7 to the latest version we faced some weird issue when trying to compile the app with Apple Watch extension.
Realm is used through SPM.
AppleWatch extension uses Realm as well.
Compiling for devices works fine, only simulator is a problem.

Stacktrace & log output
Ld /Users/alex/Library/Developer/Xcode/DerivedData/Project-evohrlyjldalplfwzjtxawskwehz/Build/Intermediates.noindex/RealmDatabase.build/Debug-watchsimulator/RealmCore.build/Objects-normal/i386/Binary/RealmCore.o normal i386 (in target 'RealmCore' from project 'RealmDatabase')
    cd /Users/alex/Library/Developer/Xcode/DerivedData/Project-evohrlyjldalplfwzjtxawskwehz/SourcePackages/checkouts/realm-core
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang++ -target i386-apple-watchos2.0-simulator -r -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/WatchSimulator.platform/Developer/SDKs/WatchSimulator8.5.sdk -L/Users/alex/Library/Developer/Xcode/DerivedData/Project-evohrlyjldalplfwzjtxawskwehz/Build/Products/Debug-watchsimulator -L/Applications/Xcode.app/Contents/Developer/Platforms/WatchSimulator.platform/Developer/usr/lib -F/Users/alex/Library/Developer/Xcode/DerivedData/Project-evohrlyjldalplfwzjtxawskwehz/Build/Products/Debug-watchsimulator/PackageFrameworks -F/Users/alex/Library/Developer/Xcode/DerivedData/Project-evohrlyjldalplfwzjtxawskwehz/Build/Products/Debug-watchsimulator -iframework /Applications/Xcode.app/Contents/Developer/Platforms/WatchSimulator.platform/Developer/Library/Frameworks -iframework /Applications/Xcode.app/Contents/Developer/Platforms/WatchSimulator.platform/Developer/SDKs/WatchSimulator8.5.sdk/Developer/Library/Frameworks -filelist /Users/alex/Library/Developer/Xcode/DerivedData/Project-evohrlyjldalplfwzjtxawskwehz/Build/Intermediates.noindex/RealmDatabase.build/Debug-watchsimulator/RealmCore.build/Objects-normal/i386/RealmCore.LinkFileList -nostdlib -Xlinker -object_path_lto -Xlinker /Users/alex/Library/Developer/Xcode/DerivedData/Project-evohrlyjldalplfwzjtxawskwehz/Build/Intermediates.noindex/RealmDatabase.build/Debug-watchsimulator/RealmCore.build/Objects-normal/i386/RealmCore_lto.o -Xlinker -export_dynamic -Xlinker -no_deduplicate -Xlinker -objc_abi_version -Xlinker 2 -Xlinker -debug_variant -fobjc-arc -fobjc-link-runtime -Xlinker -dependency_info -Xlinker /Users/alex/Library/Developer/Xcode/DerivedData/Project-evohrlyjldalplfwzjtxawskwehz/Build/Intermediates.noindex/RealmDatabase.build/Debug-watchsimulator/RealmCore.build/Objects-normal/i386/RealmCore_dependency_info.dat -o /Users/alex/Library/Developer/Xcode/DerivedData/Project-evohrlyjldalplfwzjtxawskwehz/Build/Intermediates.noindex/RealmDatabase.build/Debug-watchsimulator/RealmCore.build/Objects-normal/i386/Binary/RealmCore.o

ld: scattered reloc r_address too large for architecture i386
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Can you reproduce the bug?
Yes, always
Reproduction Steps
No response
Version
10.28.4
What SDK flavour are you using?
Local Database only
Are you using encryption?
No, not using encryption
Platform OS and version(s)
Any iOS simulator
Build environment
Xcode version: 13.4.1
Dependency manager and version: SPM

Have you done a clean build / deleted derived data? Does this happen on a fresh project? I can't reproduce this, but I am also on a M1 Mac.

@leemaguire Yes, I have tried a clean build and deleting derived data. I'm trying on a fresh project right now but seems like I can't reproduce it either. It's weird that it worked with old Realm version though. I will spend more time on playing with a fresh project to see if it's something with configuration between my current and fresh projects.

Just to add some info:
Tried to add all dependencies (SPM + Cocoapods) to the fresh project, also checked all configs -> still the fresh project doesn't have this issue.
Tried to remove the whole codebase from problematic project except empty AppDelegate (and ExtensionDelegate), also removed all dependencies -> still RealmCore linking error.
Compared linking commands between the problematic project and the fresh one:
-Xlinker -debug_variant

Only this parameter is different, everything else is exactly the same. According to this page this parameter shouldn't make a lot of influence on a linking phase. But since these commands have a difference, it means something with configs, right?
The question now: where it could be set/configured? It possible that some config influences on RealmCore compiling/linking only on latest versions for example.

@AlexIzh would you be able to attach a sample project that replicates this issue? Also are you using an Intel Mac?

Closing as issue cannot be reproduced and no more information given.


Example of TCP echo server from README.md fails

Example of TCP echo server from README.md fails.
$ git clone https://github.com/luvit/luv.git --recursive
$ cd luv/
$ make test

$ cat echo.lua
local uv = require('luv')

local function create_server(host, port, on_connection)

  local server = uv.new_tcp()
  server:bind(host, port)

  server:listen(128, function(self)
    -- self is the same as server
    assert(self == server)

    -- Accept the client
    local client = uv.new_tcp()
    server:accept(client)

    on_connection(client)
  end)

  return server
end

local server = create_server("<IP_ADDRESS>", 0, function (client)

  client:read_start(function (err, chunk)

    -- Crash on errors
    assert(not err, err)

    if chunk then
      -- Echo anything heard
      client:write(chunk)
    else
      -- When the stream ends, close the socket
      client:close()
    end
  end)
end)

print("TCP Echo serverr listening on port " .. server:getsockname().port)

uv.run()

$ luajit echo.lua
TCP Echo serverr listening on port 38555

# In another teminal:
$ nc 127.1 38555

Uncaught Error: echo.lua:10: assertion failed!
stack traceback:
        [C]: in function 'assert'
        echo.lua:10: in function <echo.lua:8>
        [C]: in function 'run'
        echo.lua:41: in main chunk
        [C]: at 0x00404d60

Versions of my software:
$ uname -a
Linux ubuntu 3.13.0-74-generic #118-Ubuntu SMP Thu Dec 17 22:52:10 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 14.04.3 LTS
Release:        14.04
Codename:       trusty
$ luajit -v
LuaJIT 2.0.2 -- Copyright (C) 2005-2013 Mike Pall. http://luajit.org/
$ git log --pretty=tformat:'%h %s' -1 | cat
dba2bb6 Update bundled lua to 5.3.2 for testing

Can you try the version in luarocks (assuming it's setup to build for your luajit)?  The default make target builds a custom luajit and then compiles against that.  You can also try using the build luajit instead of your global one.

Nevermind, I found the issue. Sorry for the trouble and thanks for reporting!


Use customer_address id as address reference

Closes #433
This PR will use the customer_address id when autoselecting the single customer address to automatically link it to the order.

What I did

How to test

Checklist

[x] Make sure your changes are tested (stories and/or unit, integration, or end-to-end tests).
[x] Make sure to add/update documentation regarding your changes.
[x] You are NOT deprecating/removing a feature.

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


.off works only on events explicitly defined with .on

Hello, I have been faced with the following issue :
I have created a handler on the swiping event like this :
.swiping(handler);
when calling off('swiping', handler), it doesn't unhandle it. But, when I create the handler with on('swiping, handler), it works well. I haven't taken time to understand why from the source code, so I am reporting it as a bug.
Thanks,

I'm sorry but now QuoJS only supports unsuscribers (off) when you use (on).

@soyjavi Ok, thanks. I can leave with that.


No example of what a harness environment yaml should look like, or ability to display current environment values

Hi @rob-casper, thanks for opening this issue!
You're right that we don't include YAML examples in the CLI docs or built-in help. The CLI is basically a wrapper for the API, which is separate from the YAML schema used by Harness entities.
We have some examples in this repo. Creating the resource in the Harness UI and seeing the YAML it generates also helps get a sense for what's needed. For an environment, the YAML needs at minimum a name, ID, and to designate the type as either Production or PreProduction.
environment:
  name: myenv
  identifier: myenv
  description: ""
  tags: {}
  type: PreProduction
  variables: []

Regarding the second part of your question, we don't have a command right now to display environment details, though we do have an API endpoint. I'll submit to the team as a feature request.
Thank you again for reaching out with this. Please let us know if you have any other questions.


[BugFix] Add column comment info for jdbc catalog table

Why I'm doing:
When execute "show create table  xxx " in a jdbc catalog table, the result data don't contain column comment.

What I'm doing:
Add comment info at JDBCSchemaResolver.convertToSRTable()
The result data after fixed:

Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[x] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.eivenchen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Task/show local stream

Be able to watch yourself while on a CallRoulette session 💅

LGTM. There are some conflicts, other than that feel free to land it.


props-table-cli

Summary
Adding script for props-table to the generator.
https://github.com/cerner/terra-core/pull/473
https://github.com/cerner/terra-clinical/pull/107
Thanks for contributing to Terra.
@cerner/terra

Released in https://github.com/cerner/generator-terra-module/releases/tag/v1.0.4


HandlingData inconsistency

Please note: ScriptHookVDotNet is NOT ScriptHookV!
If you get a FATAL error (e.g. FATAL: Unknown game version after a game update), than this project is not responsible and you instead need to wait for ScriptHookV to update.

Version
Which version of SHVDN was the problem encountered with?
3.4.0
Description
Add a clear and concise description of what the bug is and how to reproduce it.
Hello
I wrote a simple handling.meta parser, but discovered some problem when comparing the values in handling.meta and the one from Vehicle.HandlingData
Below is a simple demonstration:

As you can see:
SuspensionCompressionDamping, SuspensionReboundDamping are 1/10 of their original value from handling.meta
and AntiRollBarBiasFront, SuspensionBiasFront, TractionBiasFront  was multiplied by two when reading from SHVDN,
alsoTractionLossMultiplier, SteeringLock don't match.
I'm aware that these values are read/written directly from/to the memory, but the inconsistency between it and handling.meta is a little confusing, maybe you can sanitize them in the getter/setter or at least add some note/comment to the mismatched properties?

Since SHVDN does not alter any handling values, I don't think this issue is not relevant to SHVDN bug if memory offsets are correct. In GTA SA, the game loads handling values from handling.cfg, then multiplies or divides some of them before the game put handling values to some memory region. We should have clarify some values can be different from handling.meta files, though.

I tested if both HandlingData.TractionLossMultiplier and HandlingData.SteeringLock point to correct members. Turns out the former is incorrect, which I didn't intend to. Actually the former pointed to the member of fLowSpeedTractionLossMult. As for HandlingData.SteeringLock, the value (which is in degree) is in radian in a CHandlingData instance.


Update from upstream repo

Description 🎙

GIF 🎁

Context or background 🏞

Data inputs and responses ↔

Task 📋

Screenshots and recordings 📸

How should this be tested 🧪

Checklist ✅

[ ] 🎨 My code follows the style guidelines of this project
[ ] 🔎 I have performed a self-review of my own code
[ ] ✍ I have commented my code, particularly in hard-to-understand areas
[ ] 📜 I have made corresponding changes to the documentation
[ ] ⚠ My changes generate no new warnings
[ ] 🧪 I have added tests that prove my fix is effective or that my feature works
[ ] 🛂 New and existing unit tests pass locally with my changes
[ ] 👭 Any dependent changes have been merged and published in downstream modules
[ ] 📃 I have updated documentation in either the README.md or Notion

Closing open PRs


signature_pad under bootstrap always returns height 0 not working

when ever i try to put the signature_pad under a bootstrap container, row, or even a column the signature pad collapses into a height of 0.  I´m not using, any hide, or modals, or tabs, I have posted the example on https://jsfiddle.net/qjw18qsb/ hope some one can help
I think its still something to do with bootstrap.css not letting the signature_pad have any height.
using signatrue_pad 2.3.2 i have tried it on safari and firefox mainly

It works fine if I set height and width in px instead of 100% for .signature-pad.
BTW. It's a better idea to use https://www.jsdelivr.com/ (e.g<EMAIL_ADDRESS>than to link directly to github, because Chrome and Firefox won't load such scripts.

thank you so much for this development, and its true it works fine with px instead of %, but then the size of the signature pad is lock to that size only and I loose the responsiveness that I get with bootstrap. If I change the window size to be smaller then the signature pad with and height set in px the signature pad gets "cut" out of frame of the window instead of resizing to a smaller size.
and yes, I actually use signature_pad_min.js from local source in my actual project, i used the github for the jsfiddle test.

@apaulcb I guess you could leave the height and width at 100%, but then its parent element would need to have non-zero size, either using px or flexbox or grid or sth else.

@apaulcb The best option is to use @medis and set up the different sizes you need in your css, see here


Meaning of Printed KITTI Evaluation

In your documentation, the example of KITTI metrics is as the following:
Car<EMAIL_ADDRESS>0.70, 0.70:
bbox AP:97.9252, 89.6183, 88.1564
bev  AP:90.4196, 87.9491, 85.1700
3d   AP:88.3891, 77.1624, 74.4654
aos  AP:97.70, 89.11, 87.38
Car<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:97.9252, 89.6183, 88.1564
bev  AP:98.3509, 90.2042, 89.6102
3d   AP:98.2800, 90.1480, 89.4736
aos  AP:97.70, 89.11, 87.38
What does this output exactly mean?
From my understanding, columns are easy, moderate, and hard using IOU thresholds 0.7, 0.7, 0.7 in the first row. But then in the second group of the output, it used 0.7 for easy again but got different results, i.e. 88.3891 vs 98.2800 for 3d.
Could someone kindly explain how to read the metrics? Thank you!

Please refer to #185.