Is android-asynct-http coupled with UI thread?

I want to use this awesome library for my server request in my application to do some work on map. My need is I want to make server request for every 2 minutes without block user interaction on the screen while server requesting process happening and in the onSuccess call back I'll do some work on the map.
How can I use this library for my need?

http request should not in UI thread ,you can see simple like this:
final Activity ctx = this;
FutureTask future = new FutureTask<>(new Callable() {
public RequestHandle call() {
Log.d(LOG_TAG, "Executing GET request on background thread");
return client.get(ctx, URL, headers, null, responseHandler);
}
});
    executor.execute(future);

this request is not in UI Thread;
as you need to request per 2 min ,you just use java.util.concurrent.ScheduledExecutorService to do this

OT discussion, see the https://github.com/loopj/android-async-http/blob/master/CONTRIBUTING.md for proper way of using Github issues

any one can more explain and sample for me?


bug about keep-alive and @click.native caused repeated attach event handler?

Version
2.6.10
Reproduction link
https://codepen.io/action-hong/pen/eaKmOy
Steps to reproduce

click "show/hide button" to hide button "add 1"
click "show/hide button" to show button "add 1"
click "add 1"

What is expected?
Repeat step 1 and step 2, Click the "add 1" to add 1 to the number
What is actually happening?
Repeat step 1 and step 2 , Click the "add 1" to add n(repeat times) to the number

5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3729.169 Safari/537.36

Seems like a bug indeed. As a workaround, you can pass a regular @click and handle it manually with $listeners.click
<button @click="$listeners.click">{{ text }}</button>

<my-count text="add 1" @click.native="add" v-if="show"></my-count>
Repeated execution of 'v-if' means repeating the 'add event' for , using 'v-show' will not have this problem, or put 'add event' inside the subcomponent(my-count)

What?


Interactive kubectl (Slack)

A user needs interactive access to the most common kubectl commands directly from Slack in order to work efficiently. Currently users must be familiar with kubectl in order to effectively use BotKube. Many developer users are not k8s experts and thus additional guidance is helpful for them.
AC

A user can summon a list of common commands
The commands are interactive and can be run by clicking a button
Relevant output is returned to the user
Where relevant, the output can easily be used as input to another command (e.g. get pods -> select pod from list -> get pod logs)

Prototype (includes #742)
https://www.figma.com/proto/M9nojtcWOieek3dw0LE763/kubectl?page-id=129%3A1163&node-id=129%3A1165&viewport=1350%2C399%2C0.23&scaling=contain&starting-point-node-id=129%3A1165

Additional acceptance criteria comments:

In the Resource list step, let's dynamically narrow down the list of resources based on the user configuration so that they only see relevant items.
We can allow the user to customise this page at a later point, not necessary for the time being.
The list of resources in the prototype is not definitive, we can discuss further what other resources to include (or exclude).
For a start we can try to render returned lists of things (e.g. pods) in drop-downs and see where it makes sense to render lists as radio button groups.
We can define all the kubectl commands we want to support initially and go from there to design a more generic solution if that makes sense.

If the effort to re-create the whole @Botkube kcc experience as designed in the prototype is deemed too big for one release, we can first focus on making the current BotKube alerts actionable with interactive buttons and build on that for 0.16 to get to what we have in the prototype.
@brampling let us know if you have any disagreements.

One more thing I added to the prototype as a requirement is to display who in the channel made the bot run a command:

TODO in follow-up PRs

~associate verbs with resources~ will be done in https://github.com/kubeshop/botkube/issues/786

don't display resource types drop-down for verbs not associated with nothing e.g. api-versions, cluster-info, etc.
don't display resource names drop-down for verbs not associated with nothing e.g. api-versions, cluster-info, etc.

split resource types into categories

solve bug when resource name is selected and later the verb/resourceType is changed

add the filter output input string

print the data in the dropdown

print the plaintext

discuss the flags approach and introduce the final design. For example, support only the namespaces property (and output).

take into account that we have a given namespace restrictions associated with kubectl command, see: https://github.com/kubeshop/botkube/blob/a8fbeeaa602f8f03dc4acc83e0b76d362f1792ad/helm/botkube/values.yaml#L244-L254

out of scope: don't support command output parsing


CompreFace - only one 'unknown' face is saved

If more than one unknown (below similarity threshold) face is present, only one is saved.
CompreFace returns a list of subjects ordered by similarity.
https://github.com/exadel-inc/CompreFace/blob/master/docs/Rest-API-description.md#recognize-faces-from-a-given-image
Appears the only the first element is saved?
https://github.com/roflcoopter/viseron/blob/55d8380628aa66f5651bc2a6401264bbc96a8a39/viseron/components/compreface/face_recognition.py#L108

You are correct, will fix

Actually when looking at it again, are you sure about that?
I am looping over all the results here:
https://github.com/roflcoopter/viseron/blob/55d8380628aa66f5651bc2a6401264bbc96a8a39/viseron/components/compreface/face_recognition.py#L107
Which has one entry for each found face.
Then i grab the one with the highest accuracy here:
https://github.com/roflcoopter/viseron/blob/55d8380628aa66f5651bc2a6401264bbc96a8a39/viseron/components/compreface/face_recognition.py#L108
If i would loop over all the subjects it would process number of faces in the image * number of trained subjects which is not correct
I ran a test on an image with tho faces and this is the result:
{
   "result":[
      {
         "age":{
            "probability":0.4827486276626587,
            "high":53,
            "low":48
         },
         "gender":{
            "probability":1,
            "value":"male"
         },
         "box":{
            "probability":0.9997,
            "x_max":178,
            "y_max":139,
            "x_min":83,
            "y_min":34
         },
         "subjects":[
            {
               "subject":"rebecca",
               "similarity":0.07799
            }
         ],
         "landmarks":[
            [
               107,
               80
            ],
            [
               129,
               71
            ],
            [
               115,
               92
            ],
            [
               119,
               112
            ],
            [
               138,
               105
            ]
         ]
      },
      {
         "age":{
            "probability":0.7517662048339844,
            "high":43,
            "low":38
         },
         "gender":{
            "probability":0.974775493144989,
            "value":"male"
         },
         "box":{
            "probability":0.99744,
            "x_max":95,
            "y_max":113,
            "x_min":8,
            "y_min":23
         },
         "subjects":[
            {
               "subject":"jesper",
               "similarity":0.33593
            }
         ],
         "landmarks":[
            [
               36,
               63
            ],
            [
               62,
               56
            ],
            [
               49,
               73
            ],
            [
               43,
               92
            ],
            [
               68,
               86
            ]
         ]
      }
   ]
}

As you can see the results list has two entries, one for each face, which both would be saved as unknown faces since the similarity is low

Thanks for digging into it.
In my case, when the object detector sees 2 (or more) persons, Compreface only saves one.
I will have to simplify my config and do more controlled testing.


The info_loss is wrong

Thank you for sharing the code.
However the info_loss in your methods can not match your paper and I think it is wrong.
info_loss = self.alpha * torch.mean(-pred_ul * logp_hat - (1 - pred_ul) * logp_hat) + self.gamma* torch.mean(pred * logp_hat)
The Info_loss in fact does not use the counterfactual pred_ul since it just equals
self.alpha * torch.mean(-logp_hat) + self.gamma* torch.mean(pred * logp_hat)
Based on your paper, I think the Info_loss should be
info_loss = self.alpha * torch.mean(-pred * logp_hat +pred * pred_ul.log() ) + self.gamma* torch.mean(pred * logp_hat)

Hi there, thank you very much for your notice.
Based on the paper, I think the write formula would be
pred_avg = pred.mean()
pred_ul_avg = pred_ul.mean()
info_loss = self.alpha * (- pred_avg * pred_ul_avg.log() - (1-pred_avg) * (1-pred_ul_avg).log()) + self.gamma* torch.mean(pred * logp_hat)

by which the result of MF_CVIB on COAT would reach around 0.75 AUC.

I do not think the info_loss is what you said:
The main problem here is  the cross entropy term between q(y|z_+) and q(y|z_−). But the  (- pred_avg * pred_ul_avg.log() - (1-pred_avg) * (1-pred_ul_avg).log()) mentioned by you is not the cross entropy. I think the cross entropy should be  torch.mean(-pred * logp_hat +pred * pred_ul.log()). Note that the cross entropy is defined on the prediction y not the (1-y) based on your paper


Script Reloader does not unload old script

Steps to reproduce the problem:

mitmdump -s script.py
Modify script on disk.

System information
Mitmproxy version: 3.0.0 (2.0.0dev0407-0x315daa0)
Python version: 3.5.2
Platform: Linux-4.4.0-43-Microsoft-x86_64-with-Ubuntu-16.04-xenial
SSL version: OpenSSL 1.0.2g  1 Mar 2016
Linux distro: Ubuntu 16.04 xenial

Also: We fail badly if the script is deleted during execution.


increased antisperm antibody level

Please add:
increased antisperm antibody level
DEF: elevated level of antibodies to sperm antigens, usually due to compromised Sertoli cell barrier (blood-testis barrier) function; ASAs may be found in the circulation, either free or as immune complexes, in seminal fluid and/or attached to various antigenic sites on the sperm surface (head, midpiece or tail)
SYN: increased anti-sperm antibody level
SYN: increased ASA level
J:213588
Thanks
Anna

MP:020142


[Improvement] Implement mandatory field validations

Better to implement mandatory field validations in Dev studio.
For example: When creating  VFS inbound endpoint from management console UI it is mandatory to select 'sequence' and 'on error Sequence', but this validation is not available in the UI, hence user might forget to add these when creating inbound endpoint

please refer https://wso2.org/jira/browse/TOOLS-3039

Fixed in https://github.com/wso2/devstudio-tooling-esb/commit/644e07d3a385d85d1a808c3a9635a1f95c33eb86


Add get_date_dict_from_date_string to Time

Added a function to create a date dict from a date string. The same already exists for date_time string to date_time dict. I believe the PARSE_ISO8601 should already work and just fail the "has_time" check. I have not added the bindings.
It is my first time trying to contribute or write c++ so apologies for not doing the bindings or if I've made a mistake.

Perhaps "bindings" is not the right term. I haven't added the class to the "_bind_methods()" func yet as I'm not quite sure how it works.

I think I have added the method bind correctly now

You need to run your compiled godot executable with --doctool to update the generated documentation xml. Then fill in the new method description.


Remove deprechiated pinocchio 3 branch link in Readme

Hi, when checking out Pinocchio after a while I was confused about the statement that Pinocchio3 is only available in the preview branch with a dead link. So I thought it would be nice to quickly adapt this to make it clear for others.

Hi ! This project doesn't usually accept pull requests on the main branch.
If this wasn't intentionnal, you can change the base branch of this PR to devel
(No need to close it for that). Best, a bot.


include/signal.h:Expanding SIGNAL to be consistent with Linux

Summary
Expanding SIGNAL to be consistent with Linux
Impact
Testing

This was not reviewed before merging, why?
Also this breaks if CONFIG_SCHED_HAS_PARENT is not defined because SIGCHLD will collide with the new values.

Careful with CONFIG_SCHED_HAS_PARENT
This code breaks my build because this config is not set.

What is the specific error report? This is the SIGNAL implemented according to https://pubs.opengroup.org/onlinepubs/009695399/basedefs/signal.h.html

duplicate switch case because SIGCHLD ==SIGWORK
when CONFIG_SCHED_HAS_PARENT SIGCHLD is forced to 17

https://github.com/apache/nuttx/blob/47ce89f757bd7767124fbe0a88cfc8e76e68c147/include/signal.h#L222-L226
https://github.com/apache/nuttx/blob/47ce89f757bd7767124fbe0a88cfc8e76e68c147/include/signal.h#L308-L314
In actual definition SIGCHLD is 17 and SIGWORK is 31.
https://github.com/apache/nuttx/blob/47ce89f757bd7767124fbe0a88cfc8e76e68c147/sched/Kconfig#L1557-L1563
https://github.com/apache/nuttx/blob/47ce89f757bd7767124fbe0a88cfc8e76e68c147/sched/Kconfig#L1660-L1667
And the two values are not the same in Kconfig

dont you believe me?
CC:  string/lib_strsignal.c string/lib_strsignal.c: In function 'strsignal':
string/lib_strsignal.c:169:7: error: duplicate case value
  169 |       case SIGWORK:
      |       ^~~~
string/lib_strsignal.c:117:7: note: previously used here
  117 |       case SIGCHLD:
      |       ^~~~

The problem is that SIGWORK was changed recently from 17 to 31 and my config kept that.
I cant be the only person in this situation.

It was changed here from 20 to 31
https://github.com/apache/nuttx/commit/e6227e19433c4999d500437d0a8a5c05f476ceb1#diff-285d196ad580d55fdaef3fbe54702ee3ddeed0d93cba52a5441d1866e0031726L1584
The comment still shows that it was previously changed from 17 to 20
This is very messy.

hello, what is the value of CONFIG_SIG_CHLD and CONFIG_SIG_SIGWORK  of your board?

dont you believe me?
CC:  string/lib_strsignal.c string/lib_strsignal.c: In function 'strsignal':
string/lib_strsignal.c:169:7: error: duplicate case value
  169 |       case SIGWORK:
      |       ^~~~
string/lib_strsignal.c:117:7: note: previously used here
  117 |       case SIGCHLD:
      |       ^~~~

The problem is that SIGWORK was changed recently from 17 to 31 and my config kept that.
I cant be the only person in this situation.

so do you use the expend form of defconfig?

I save the current .config as my defconfig, so it contains all parameters.
This way I am sure that the configuration is stable.
@Donny9 hi, it was stuck at 17 because I keep full configs.
This specific situation is hard to track down.
There is no real way to update a full .config file with updated default values, so this is unavoidable.
TBH this is a minor problem with a complex cause.

I save the current .config as my defconfig, so it contains all parameters. This way I am sure that the configuration is stable.

Ok, that is why the main CI doesn't hit the similar problem since all defconfig in boards save in the normalize form.


Refactor bulk subscriptions.

Looks great, feel free to merge once CI passes.  Seems worth adding a bulk user lookup while we're thinking about this.


[BUG] opensearch-cluster chart : imagepullsecret for general and dashboard

What is the bug?
In Opensearch cluster helm chart imagepullsecret not templated for dashboard and general
How can one reproduce the bug?
/
What is the expected behavior?
to have an imagepullsecret setting in template like inithelper does as it works when you modify the manifest generated with dry-run in an helm upgrade/install
What is your host/environment?
running opensearch in k8s, setup via operator and cluster helm charts
Do you have any screenshots?
/
Do you have any additional context?
Solution is to add imagepullsecret to template for general and dashboard sections, like in initcontainer
https://github.com/opensearch-project/opensearch-k8s-operator/blob/main/charts/opensearch-cluster/templates/opensearch-cluster-cr.yaml#L23-L26

here is a draft PR to show the easy resolution of this bug
https://github.com/opensearch-project/opensearch-k8s-operator/pull/905


[Help] Unsuccessful to integrate TypeORM

I try to integrate TypeORM in it, but unsuccessful to import the entity .ts file when initializing the database.
The project structure is
electron
|--database
|--|--index.ts
|--|--entity
|--|--|--test.entity.ts

Here is the code that I use in electron/database/index.ts
import "reflect-metadata";
import { DataSource } from "typeorm";

     this.appDataSource = new DataSource({
        type: "sqlite",
        database: dbPath,
        entities: ["./entity/*.ts"],
      });

The content of test.entity.ts
import { Entity, PrimaryGeneratedColumn, Column } from "typeorm"

@Entity()
export class User {
    @PrimaryGeneratedColumn()
    id: number

    @Column()
    firstName: string

    @Column()
    lastName: string

    @Column()
    age: number
}

Case 1
With the above config, when I run to debug, the database file can be created but failed to create the table.
Case 2
If I change the property of entities from file path to entity class by using import.
import { User } from "./entity/test.entity.ts"

     this.appDataSource = new DataSource({
        type: "sqlite",
        database: dbPath,
        entities: [User],
      });

The table can be created successfully.
Case 3
I took a deep step, by change the property of 'entities' with app.getAppPath()
     this.appDataSource = new DataSource({
        type: "sqlite",
        database: dbPath,
        entities: [path.join(app.getAppPath(), "./electron/database/entity/*.entity.ts")],
      });

It would occurs and error,
import {
^^^^^^

SyntaxError: Cannot use import statement outside a module

From the error message, we could know by using app.getAppPath() to get the absoluted path is working, the relative path in Case 1 is not working. But I have no idea to solve this error.
Maybe change the module in tsconfig.json from ESNext to commonjs?
In official document of TypeORM, about the entities property,

entities - Entities, or Entity Schemas, to be loaded and used for this data source. Accepts both entity classes, entity schema classes, and directories paths to load from. Directories support glob patterns. Example: entities: [Post, Category, "entity/.js", "modules/**/entity/.js"].

In the example, we can notice the path is js, not ts. Is that means we need to translate ts to js and put these js files into dist-electron, but for vite-plugin-electron, the entry only supports for single file not for a path with glob pattern.
How to do this op?

have you ever try import.meta.glob?


make lint: extend checks

This PR includes a series of commits to extend the linter checks to include:

pkg/tracing
docs/
unit tests (i.e., _test.go files)

Especially checking the unit tests could make CI more robust as rigorous error checks could reveal flakes.

Next round.  @TomSweeneyRedHat @rhatdan @baude @mheon PTAL
The next candidate (if time allows) is to include test/ into the checks as well. After that, we can enable the currently disabled one-by-one.

LGTM
Nice work.

The only thing pending is a Prow test, so I'll go ahead and whack this
/lgtm

git hooks have been dropping for prow btw, the suggestion is to spam retest until it triggers
/retest


Two tap to change a tab on mobile (iOS and iPadOS)

As for the title, I noticed that on my iPad if I use an external keyboard (magic keyboard) I can click on a row on the left sidebar and a click is enough to change immediately the active tab.
However on the same iPad, and on my iPhone, if I tried to tab on a row I have to tap twice to switch the active tab.
The first tap seems to select the right row and the second one to effectively activate it.
Is it an expected behavior?

This bug has been fixed in the latest version. Thank you. If you experience this again, please reopen this issue.


ERROR: Could not resolve webpack

Reporting a bug?
I am using
import VueI18nVitePlugin from "@intlify/unplugin-vue-i18n/vite";
vite: {     resolve: {       alias: {         "vue-i18n": "vue-i18n/dist/vue-i18n.runtime.esm-bundler.js",       },     },     plugins: [       svgLoader(),       VueI18nVitePlugin({         include: [           resolve(dirname(fileURLToPath(import.meta.url)), "./locales/*.json"),         ],       }),     ],   },
and when I open my App I get an error that webpack is not installed but I am using vite.
Expected behavior
No Error
Reproduction
See code above.
Issue Package
unplugin-vue-i18n
System Info
System:
    OS: Windows 10 10.0.22621
    CPU: (12) x64 AMD Ryzen 5 5500
    Memory: 8.87 GB / 15.90 GB
  Binaries:
    Node: 18.12.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 8.19.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.22621.819.0), Chromium (107.0.1418.42)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    vue-i18n: ^9.2.2 => 9.2.2

Screenshot
No response
Additional context
[vite] error while updating dependencies:                                                                                19:31:48
Error: Build failed with 3 errors:
node_modules/@intlify/unplugin-vue-i18n/lib/index.mjs:454:27: ERROR: Could not resolve "webpack"
node_modules/esbuild/lib/main.js:1327:21: ERROR: [plugin: vite:dep-pre-bundle] Failed to resolve entry for package "@intlify/shared". The package may have incorrect main/module/exports specified in its package.json: No known conditions for "." entry in "@intlify/shared" package
node_modules/esbuild/lib/main.js:1327:21: ERROR: [plugin: vite:dep-pre-bundle] Failed to resolve entry for package "@intlify/shared". The package may have incorrect main/module/exports specified in its package.json: No known conditions for "." entry in "@intlify/shared" package
Validations

[X] Read the Contributing Guidelines.
[X] Read the README
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.

Thank you for your reporting!
Could you give us minimum reproduction  code via GitHub or StackBliz plaese? 🙏
Issue reporting guidline said:

Please provide a link to a repo that can reproduce the problem you ran into. A reproduction is required unless you are absolutely sure that the issue is obvious and the provided information is enough to understand the problem. If a report is vague (e.g. just a generic error message) and has no reproduction, it will receive a "Status: Need More Info" label. If no reproduction is provided after 5 days, it will be closed.

Ref blog article: https://antfu.me/posts/why-reproductions-are-required

I reproduced the same problem in a larger code base, but were not able to create a minimum reproduction until now.

Hi, I also got the same issue when trying to setup my new project with Nuxt 3.
I realized the problem appeared when I tried to combine the following tools:

Node: 16x or 18x.
Package management:

Yarn: both old yarn (1.x) and new yarn (3.x without pnp) get the same error
Npm: using npm in Stackblitz gives a different error

Nuxt 3 with Vite
Vue i18n: unplugin-vue-i18n.
TailwindCSS: both official instructions or using @nuxtjs/tailwindcss give me the same error

Official installation docs: https://tailwindcss.com/docs/guides/nuxtjs#3
Nuxt Tailwind: https://github.com/nuxt-modules/tailwindcss

Both unplugin-vue-i18n and @nuxtjs/tailwindcss work fine alone but when installing both in the repo the issue appears.
The issue only appears when calling nuxt dev but not when calling nuxt build.
This is a minimum reproduction on StackBlitz: https://stackblitz.com/edit/nuxt3-issue-unplugin-vue-i18n

Image

Experiencing this issue too. Trying to migrate nuxt 2 project to nuxt 3.
I've also removed @nuxtjs/tailwindcss as above comment suggested but nothing changed.
@kazupon please review this issue as it's a huge blocker which basically makes this unusable


Filter complied tasks

How about lowering the completed task to the very bottom so that it does not interfere with the rest of the outstanding tasks?

Good idea, I've added it!


fixup! Workaround GCR bug

This is meant as a review to https://github.com/camallo/dkregistry-rs/pull/162.

Please squash my changes into your commit if you decide to take them.

Thanks, your changes make code much cleaner!


ASC-422 Update MNAIO Var File

In order to capture accurate metadata about Molecule system tests, the
"RPC_RELEASE" and "RPC_PRODUCT_RELEASE" environment variables need to be
stored in the "MNAIO_VAR_FILE".
Issue: ASC-422

recheck_xenial_no_artifacts_swift_deploy

recheck_xenial_no_artifacts_swift_deploy

Blocked by RI-255.

recheck_xenial_no_artifacts_swift_deploy


While grouping it also consider tests which are skipped & result of that is uneven groups

Since grouping does not factor in that there will be some tests which will get skipped and due to that when grouping happened, one group will have 2 tests run and others will have 10 tests run. So overall test execution time gets impacted and maybe more with more number of groups.

I think the solution should be to add the following, so the collection hook will be executed after the filtering was applied
@pytest.hookimpl(trylast=True)
def pytest_collection_modifyitems(session, config, items):

+1 ,  since this is a real show stopper for using this plugin in our project.


Improved Memory View (Editable Memory)

updated layout of memory view (16 bytes, tighter spacing for smaller displays, labels)
added editable memory

Editable memory supports tabbing between memory cells, and "over typing" into the next cell (just don't type too fast).
Updated Layout

Editable Memory

Thanks for your contribution, it's merged!


easyeffects: 6.0.3 -> 6.1.0

Motivation for this change
Update
https://github.com/wwmm/easyeffects/blob/v6.1.0/CHANGELOG.md#610
https://github.com/wwmm/easyeffects/compare/v6.0.3...v6.1.0
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Looking at the changelog, at least the comments in lv2Plugins should be updated.

@jtojnar not totally sure I follow - do you have a link?

https://github.com/NixOS/nixpkgs/blob/ade64c6db4a221d9c44e49c850e9837a6a42179d/pkgs/applications/audio/easyeffects/default.nix#L82
Changes in OP.


Feat: Allow categories in resource table to be compressed

couple of things: first, there's conflicts with master that need to be resolved. Second, the color of the category headers don't update with the style of the world, so they need to be dynamic with that. Third, the size of the resource table changes frequently when opening and closing tabs, which is quite jarring. Try fixing the size of the table to match that of the longest element, even if that element is in a collapsed category. Finally, the transition from open to closed and closed to open isn't animated. If at all possible, see if there's a way to animate the transition.

couple of things: first, there's conflicts with master that need to be resolved. Second, the color of the category headers don't update with the style of the world, so they need to be dynamic with that. Third, the size of the resource table changes frequently when opening and closing tabs, which is quite jarring. Try fixing the size of the table to match that of the longest element, even if that element is in a collapsed category. Finally, the transition from open to closed and closed to open isn't animated. If at all possible, see if there's a way to animate the transition.

It looks like git auto-updated my pr. I should have fixed everything you said, though the column size of the table is annoying some of the time.

It looks like git auto-updated my pr. I should have fixed everything you said, though the column size of the table is annoying some of the time.


Filter notifications by origin

Closes #2065

Actually, I might need to modify this PR, I forgot about gist.github.com.

I forgot about gist.github.com

That's fine, there are no notifications on that domain

There is a notification icon though, do we not care about seeing unread notifications there?

Good point. You can use the URL constructor to parse the url variable and match the main domain only, e.g. github.com instead of the whole origin.

Actually that's a bad idea because url could be hello.co.uk and currentDomainmight bewow.co.uk, which we have to turn into co.uk` to make my logic work.
This needs to explicitly account for gist.github.com instead

This doesn't break github.com and gist.github.com, so it's a-ok for me

I never have notifications on GHE. I'm not sure if they're not fully baked yet or our instance isn't configured to use them. Either way, this looks fine to me.


_WD_CreateSession fails to return session ID with Microsoft web driver

Originally reported here -- https://www.autoitscript.com/forum/topic/191990-webdriver-udf-w3c-compliant-version-05132018/?do=findComment&comment=1394743

Found https://github.com/w3c/webdriver/issues/1201, which indicates that that correct implementation is for the sessionid to be within the value key

Confirmed working in v10.0.17763.1 of microsoftwebdriver.exe


Fix Brass dust and Silver ore smelting

DT silver ore is hardcoded to smelt into DT silver ingot, but I've fixed the rest of the silver ores.
I also made brass dust smelt into brass ingots instead of alchemical brass

Is the way I'm removing the furnace recipe is okay?

The Erebus ores still smelt into the Erebus ingots.

I'd prefer to fix the silver recipe through the recipe functions I wrote, but this'll do for now. I fixed the brass furnace recipe (and possibly other recipes) through commit https://github.com/belathus/Wanderlust-Renewed/commit/0b9f18a6b565bd6046974ae8d7ea661ddc61aeb9. Not sure why you fixed the crushed dark gem typo in this pull request. :P

I've removed the other files from the pull request.

You probably want to use squash and merge


Avoid overlapping of iotd image over header

overlapping of image and go back on the header must be avoided.

Hey @parthpanchal123 , I would like to work on this issue for Hacktoberfest 2021, if it seems okay.

@KarishmaVanwari Nice catch , Go ahead

@parthpanchal123 have a look at #17


Reading Wave File get exception '<' not supported between 'str' and 'int'

I am getting an error trying to read back a wavefile that I created using the soundfile save routine.  It is likely a simple mistake but I am not seeing or understanding what I did wrong.  Hope you guys can help.
The  created wavefile can be opened and played using windows media player or VLC but reading back into my program to recover the stored numpy arrays using Soundfile,  I get the following error.
Enter filename:abc
Opening file:  ./SoundWave/abc.wav   #program crashed here and gave this exception which I do not understand.
exception: '<' not supported between instances of 'str' and 'int'
cannot unpack non-iterable NoneType object
The open code is very simple.  I have tried both 'r' and 'rb' and both give the same error.  The subrountine is :
def OpenSnd(filename,fs=44100, channels=1):
    try:
        if filename == "":
            return
        else:
            filename = './SoundWave/' + filename + ".wav"
            print("Opening file: ", filename)
            with sf.SoundFile(filename, 'rb') as f:
                data, samplerate = f.read(filename)
                return data, samplerate
    except Exception as e:
        print("exception:", e)

The save file routine is below and gave no errors and the file saved seems to work.  Data is the the created np.array
def SaveSnd(filename, data, fs=44100, channels=1):
    try:
        if filename == "":
            return
        else:
            filename = './SoundWave/' + filename + ".wav"
            with sf.SoundFile(filename, 'w', fs, channels) as f:
                f.write(data)
    except Exception as e:
        print("exception:", e)

I posted the full code and the abc.wav file on github at https://github.com/waszee/SignalStudies_RF
The problem file is called: CWMsg_2_Wave_2_Tensor.py

It is likely a simple mistake

It indeed is.
Your are creating the f variable using filename, but later you are using f.read(filename).
The soundfile.SoundFile.read() method doesn't need a filename, because the file has already been opened at that point.
See https://python-soundfile.readthedocs.io#soundfile.SoundFile.read.
You were probably confused because there is also a soundfile.read() function, which indeed needs a filename: https://python-soundfile.readthedocs.io#soundfile.read.

BTW, instead of catching the exception, you should let it bubble up to the terminal during debugging. This way you see an error traceback which tells you exactly where the problem happened. This would probably help diagnosing such errors.

Reply thanks, That fixed the main issue and lead me to a couple of more issues like not having the return data defined properly.  Well so much for  working after being tired.  We can close this one.

Thanks – tried a bunch of variations and just kept getting mistakes last night.  Should have taken a break.  Closed the note out.
Bill
From: Matthias Geier<EMAIL_ADDRESS>Sent: Friday, October 23, 2020 2:49 AM
To: bastibe/SoundFile<EMAIL_ADDRESS>Cc: waszee<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [bastibe/SoundFile] Reading Wave File get exception '<' not supported between 'str' and 'int' (#283)
It is likely a simple mistake
It indeed is.
Your are creating the f variable using filename, but later you are using f.read(filename).
The soundfile.SoundFile.read() method doesn't need a filename, because the file has already been opened at that point.
See https://python-soundfile.readthedocs.io#soundfile.SoundFile.read.
You were probably confused because there is also a soundfile.read() function, which indeed needs a filename: https://python-soundfile.readthedocs.io#soundfile.read.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/bastibe/SoundFile/issues/283#issuecomment-714995221, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARAKYG4QQUPFGWDSY7NC3HLSMERNTANCNFSM4S37SOQA.


Adding Posts not working.

I want to add a post to my website and I follow all the steps which are written in this URL https://github.com/jarrekk/Jalpc/wiki/How-to-add-posts and I wait for 30-35 mins.
But I got nothing as a result. Can you please suggest me. Because I am stuck in my work so I want to solve this problem quickly as much as possible.
Thanks, I have sent you an email also. Please see and suggest me if so.

@agrawalsmart7 can you show me the filename of each post? It should be yyyy-mm-dd-\*.md

i'm getting a similar issue, when I add a post that is not from current month, it not be show in /blog route


Fix ci-app's Cloud Build definition with gcr.io/kpt-fn/gatekeeper container image tag

Fix gcr.io/kpt-fn/gatekeeper:v0.2 container image in Cloud Build definition of the https://cloud.google.com/anthos-config-management/docs/tutorials/app-policy-validation-ci-pipeline tutorial.
latest doesn't work: https://github.com/GoogleContainerTools/kpt/issues/3134
Following up on this https://github.com/GoogleCloudPlatform/anthos-config-management-samples/pull/45.
CC: @poonam-lamba

I was able to test it locally `` and I got this output:
Step #2 - "Validate against policies": Pulling image: gcr.io/kpt-fn/gatekeeper:v0.2
Step #2 - "Validate against policies": v0.2: Pulling from kpt-fn/gatekeeper
Step #2 - "Validate against policies": Digest: sha256:a1c193183dd838625036cb7082c3bbf6d8a39968d2a1082cb1e3c6afcc61d386
Step #2 - "Validate against policies": Status: Downloaded newer image for gcr.io/kpt-fn/gatekeeper:v0.2
Step #2 - "Validate against policies": gcr.io/kpt-fn/gatekeeper:v0.2
...
Step #2 - "Validate against policies": [error] apps/v1/Deployment/nginx-deployment : Deployment objects should have an 'owner' label indicating who created them.
Step #2 - "Validate against policies": violatedConstraint: deployment-must-have-owner
Finished Step #2 - "Validate against policies"
2022/05/11 18:55:18 Step Step #2 - "Validate against policies" finished
2022/05/11 18:55:19 status changed to "ERROR"
ERROR
ERROR: build step 2 "gcr.io/kpt-fn/gatekeeper:v0.2" failed: exit status 1
2022/05/11 18:55:20 Build finished with ERROR status

All good, like expected.
Ready for your review please @janetkuo


test: speed up unit test

Previously, the unit test took 30min to finish: https://jenkins-etcd.prod.coreos.systems/job/etcd-operator-unit-pr/1090/console
With this, it took 3min to finish: https://jenkins-etcd.prod.coreos.systems/job/etcd-operator-unit-pr/1096/console

lgtm

Notes for safety of builds:

Currently we have separate GOPATH for each jenkins job. The cached intermediate builds are stored "$GOPATH/pkg". This is safe for now.
We will eventually migrate jenkins into docker containers for better isolation though.


Update installing-the-microsoft-odbc-driver-for-sql-server.md

Changing msodbcsql17 to msodbcsql, the msodbcsql17 package is no longer in the tap.

@makopov : Thanks for your contribution to the Azure documentation! The author, @MightyPen, has been notified to review your proposed change.

@MightyPen please review.

@David-Engel Hi DAVID, I am approving this change, for I trust @makopov .
But DAVID can you please take a very brief look at this PR 454 in case you see any issue with its change (drops the 17 from msodbcsql).  Thank you.
MightyPen = GeneMi.

#sign-off

@MightyPen - Looking at the date this PR was created, I can understand the proposal to change msodbcsql17 to just msodbcsql. We had a brief gap when the docs were updated but the Homebrew formula had not been published with the msodbcsql17 package yet. msodbcsql17 is there now and I created a PR to revert this change.


Not sure I installed correctly

Hi,
I am new to programming. Therefore, installing PromiseKit has been difficult for me. Upon investigating the various installation methods, I attempted to install via Carthage (unsuccessfully it turned out because than I had to figure out how to install Carthage, which I was never able to do).
I then tried to install manually since it appeared simpler. While the instructions say to "You can just drop PromiseKit.xcodeproj into your project and then add PromiseKit.framework to your app’s embedded frameworks," even that is unclear to me.
I ended up dropping it here:

This seemed to work because now when type "import" at the top of my ViewController.swift file, "PromiseKit" appears as a viable answer.
However, here is where things go bad. Based on the documentation, I expect to be able to write code like this:
firstly {     login() }.then { creds in     fetch(avatar: creds.user) }.done { image in     self.imageView = image }
So, I went to the code for the button in my app.
@IBAction func AddElectionEvents(_ sender: UIButton) {         deleteEvents()         createEvents()     } }
I want to use PromiseKit to first delete my events and then create new events. When I type "firstly", however, this is what I see:

I expected to be able to write:
firstly {     deleteEvents() }.then {     createEvents() }.done { }
I'm feeling stuck. Can anyone help?

You didn't install it manually.
Please use Carthage or CocoaPods if you are new to programming, doing it manually is definitely for advanced users.

I suggest cocoapods for beginners, use homebrew to install it: https://brew.sh
Then do brew install cocoapods, then follow our cocopods instructions, please refer to CocoaPods documentation if you need to.

Thank you, that is exactly the direction I was looking for. Very helpful to me.

I'll close, but if you need more help, feel free to reopen.


HasConversion not used when executing BatchUpdate

When executing a Update where a property of the entity has an "HasConversion", the conversion is not happening.
Repro code:
using System;
using System.Linq;
using Microsoft.EntityFrameworkCore;
using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
using Microsoft.Extensions.Logging;
using Microsoft.Extensions.Logging.Console;
using Z.EntityFramework.Plus;

namespace BatchUpdateRepro
{
    class Program
    {
        [Obsolete]
        static void Main(string[] args)
        {
            var connectionString = "{your_connection_string}";
            var options = new DbContextOptionsBuilder<ExampleDbContext>()
                .UseLoggerFactory(new LoggerFactory(new[] {
                    new ConsoleLoggerProvider((_, level) => level >= Microsoft.Extensions.Logging.LogLevel.Debug, true)
                }))
                .EnableSensitiveDataLogging()
                .EnableDetailedErrors()
                .UseNpgsql(connectionString, b => b.EnableRetryOnFailure())
                .Options;

            var rnd = new Random();

            // adding new entities: success
            using (var dbContext = new ExampleDbContext(options))
            {
                var examples = new[]
                {
                    new Example { Id = rnd.Next(), Value = GenerateGuid() },
                    new Example { Id = rnd.Next(), Value = GenerateGuid() },
                    new Example { Id = rnd.Next(), Value = GenerateGuid() }
                };

                dbContext.Examples.AddRange(examples);
                dbContext.SaveChanges();
            }

            // loading entity and updating it: success
            using (var dbContext = new ExampleDbContext(options))
            {
                var example = dbContext.Examples.First();
                example.Value = GenerateGuid();

                dbContext.SaveChanges();
            }

            // bulk updating entities: error
            using (var dbContext = new ExampleDbContext(options))
            {
                var value = GenerateGuid();

                // Npgsql.PostgresException: 42804: column "value" is of type uuid but expression is of type text
                dbContext.Examples.Where(e => e.Id > 0).Update(e => new Example { Value = value });
            }
        }

        private static string GenerateGuid()
        {
            return Guid.NewGuid().ToString("N").ToUpperInvariant();
        }
    }

    public class ExampleDbContext : DbContext
    {
        private readonly ValueConverter<string, Guid> _guidConverter = new ValueConverter<string, Guid>(v => new Guid(v), v => v.ToString("N").ToUpperInvariant());
        public DbSet<Example> Examples { get; set; }

        public ExampleDbContext(DbContextOptions<ExampleDbContext> options) : base(options) { }

        protected override void OnModelCreating(ModelBuilder builder)
        {
            builder.HasDefaultSchema("efplus");

            var model = builder.Entity<Example>();
            model.ToTable("example");
            model.HasKey(e => e.Id);
            model.Property(e => e.Id).HasColumnName("id");
            model.Property(e => e.Value).HasColumnName("value").HasConversion(_guidConverter);
        }
    }

    public class Example
    {
        public int Id { get; set; }
        public string Value { get; set; }
    }
}

Database table:
    create schema efplus
    create table efplus.example
    (
      id int not null primary key,
      value uuid null
    )

When executing the code for the last context, the string is not converted to a uuid and the database is rejecting the request with error: "Npgsql.PostgresException: 42804: column "value" is of type uuid but expression is of type text"
Thank you.
EFPlusRepro.zip

An exception is also thrown when a conversion should be done in the where clause:
using (var dbContext = new ExampleDbContext(options))
{
    var value = dbContext.Examples.First().Value;
    var newValue = GenerateGuid();

    // System.InvalidCastException: Can't write CLR type System.String with handler type UuidHandler
    dbContext.Examples.Where(e => e.Value == value).Update(e => new Example { Value = newValue });
}

Hello @gbilodeau ,
You are right. The Batch Update currently doesn't support value conversion.
We will look at the difficulty to add the support to it.
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Hello @gbilodeau ,
A new version of EF Extensions has been released with this fix: https://www.nuget.org/packages/Z.EntityFramework.Extensions.EFCore/2.7.8
The DeleteFromQuery and UpdateFromQuery method in this library is free. As most people already know by now, very soon EF Plus will have a dependency to EF Extensions (See discussion) and EF Plus Delete and Update method will call under the hood those DeleteFromQuery and UpdateFromQuery so that's why we didn't add the code on EF Plus since everything was already coded to make this request on EF Extensions.
Let me know if that's working as expected.
Best Regards,
Jonathan

Merci, that was quick! We will look into this today and let you know how it works out.

Awesome!
Looking forward to hearing from you!
Jon

Hello @gbilodeau,
A simple follow up to see if you are satisfied with the latest version of EF Extensions!
Don't hesitate to contact us for questions, issues or feedback!
Looking forward to hearing from you,
Jon

Hi, sorry we didn't have time to test this as we've run into some production issues. As soon as we're done with this, I'll post back here. Thanks for following up!

Hello @gbilodeau
No worries! We all have unexpected tasks!
Looking forward to hearing from you!
Jon

Hello @gbilodeau,
A quick follow up to see if you are still satisfied with the latest version of EF Extensions.
Looking forward to hearing your feedback.
Best regards,
Jon

Works as expected, thank you!


Is it possible to run PixivUtil2 on MacOS apple silicon?

It has been almost 10 years from the lastest issue that I found about running on MacOS so I'm wondering if there any guide on how to run it on MacOs apple silicon. Issue#45

Yes


Give Hover Effect on the 603 logo

Ok first star this repo i will assign you

Sorry but i can not find index.html can you plz tell me where it is located i looked for it in src folder but didn't find it.

check in public folder

oh sorry,my bad
Thanks

@AadishSanghai  Assigned do star the repo thank you!!

@AadishSanghai  Any attractive hover on logo would work, it should work

@harshitkumargupta  assigned


Detox globals no longer available in utility functions (outside of tests)

Describe the bug
Hey Team
We've recently upgraded to RN 0.59 from a much earlier version. When we first started we wrote a number of utility functions that were stored in a separate utility file. These helped us write more concise specs and provided an enhanced debugging experience:
spec:
import { APPNAME } from '../aliases';

describe('Do a thing', () => {
  describe('Show list of things', () => {
    beforeEach(async () => {
      await device.reloadReactNative();
    });

    it('should be able to do thing 1', async () => {
      await APPNAME.withText('$0').expect.toBeVisible();
    });
  });
});

aliases/index.js
const withText = text => {
  return element(by.text(text));
};

This was working very well until updating RN. Now we see the following error:
    ReferenceError: element is not defined

       8 |
       9 |     const withLabel = text => {
    > 10 |       return element(by.label(text));
         |       ^

It seems that element (and other detox globals) are no longer accessible outside of an 'it' statement. I thought at first it might be a jest issue, but I've downgraded jest to the version we were previously using and we're still experiencing the issue.
Have there been any behind the scenes changes that stop detox globals from being accessible outside of a test? Any insight would be really appreciated.
Thanks 👍
To Reproduce

[x ] I have tested this issue on the latest Detox release and it still reproduces

Expected behavior
Detox globals, such as element are accessible outside tests in external utility functions
Screenshots
N/A
Environment (please complete the following information):

Detox: 12.6.1
React Native: 0.59.5
Node: 10.0.0
Device: N/A
Xcode: 10.2.1
iOS: 12.2
macOS: 10.14.4

Device and Verbose Detox Logs
detox test --configuration ios.sim.debug -f e2e/doThings/thing.spec.js --loglevel trace
detox[75892] WARN:  [deprecation.js] Beware: -f, --file will be removed in the next version of Detox.
detox[75892] WARN:  [deprecation.js] See the migration guide:
https://github.com/wix/Detox/blob/master/docs/Guide.Migration.md#migrating-from-detox-120x-to-121x
detox[75892] INFO:  [test.js] configuration="ios.sim.debug" loglevel="trace" artifactsLocation="artifacts/ios.sim.debug.2019-05-02 05-26-42Z" recordLogs="none" takeScreenshots="manual" recordVideos="none" recordPerformance="none" node_modules/.bin/jest --config=e2e/config.json --maxWorkers=1 '--testNamePattern=^((?!:android:).)*$' "e2e/doThings/thing.spec.js"
 FAIL  e2e/doThings/thing.spec.js
  ● Test suite failed to run

    ReferenceError: element is not defined

       5 | };
       6 | const withId = id => {
    >  7 |   return ElementProxy(element(by.id(id)), 'id', id);
         |                              ^
       8 | };
       9 | const withText = text => {
      10 |   return ElementProxy(element(by.text(text)), 'text', text);

      at withId (e2e/aliases/index.js:7:30)
      at Object.<anonymous> (e2e/aliases/index.js:20:9)
      at Object.<anonymous> (e2e/doThings/thing.spec.js:1:222)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        1.352s
Ran all test suites matching /e2e\/doThings\/thing.spec.js/i with tests matching "^((?!:android:).)*$".
detox[75892] ERROR: [cli.js] Error: Command failed: node_modules/.bin/jest --config=e2e/config.json --maxWorkers=1 '--testNamePattern=^((?!:android:).)*$' "e2e/doThings/thing.spec.js"

Hey @abury,
Nothing has changed in Detox in that regard. Did you also upgrade jest or node? I would investigate that direction as well.
Anyway, you can always disable global exports and require detox API manually.

Hey @rotemmiz, thanks for the reply.
We did upgrade jest, but I've tried downgrading it and unfortunately, it didn't do anything. I'll try requiring the detox API manually 👍

Unfortunately, that's not working either. Whilst expect and element is defined, by is undefined when importing it from detox. I followed the instructions from https://github.com/wix/Detox/blob/a954203828f2a466d77a148050ac133b8765df96/docs/APIRef.DetoxObjectAPI.md
I even tried importing by directly from ios expect:
import { by } from '../../node_modules/detox/src/ios/expect';
And received the following error
TypeError: Cannot read property 'apply' of undefined
which is confusing as we're not calling anything called apply anywhere.
If you've got any insights as to why this might be occurring, it would be amazing.
Thanks,
Aron

You're holding it wrong 😉
https://github.com/wix/Detox/blob/master/docs/APIRef.DetoxObjectAPI.md#explicit-imports-during-initialization

@rotemmiz No, it was being imported properly and I followed the instructions in that link. The import direct from node_modules was just me trying to debug things.
It turned out to be a timing issue. We used to call element before detox.init() was called (thanks to an import and lazy instantiation), and it (somehow) used to work without an issue. Since updating react native, it has (rightly) broken.
It's one of those issues where it should never have worked in the first place 😂. We've re-arranged things and it's all working now.


Pset v2 reissuance

This PR forks @altafan branch and add several small fixes in order to let confidential reissuance test to pass.
@altafan @tiero please review

Ci is failing @louisinger


[WIP] Physical components

This PR aims to change the Events to point to PhysicalComponent rather than to each Physical Infrastructure model

Closing this PR as we decided to not move with this approach right now, instead we will add the resources ID's to the event_stream model and eventually think about some refactor on it.


API / dataloader for logging a video file directly

It is common for users to have video files they want to insert into Rerun.
Eventually, we would like to support video compression directly in the viewer (https://github.com/rerun-io/rerun/issues/5815), but doing so in a cross-platform way (including the browser) presents several challenges.
However, at present, we require users to decode video themselves and then log the individual image frames to Rerun. This is challenging to do performantly, generally requiring dependencies on external libraries, and often times multi-threaded/multiprocessing approaches to help with throughput.
It would be nice to offer a user-space API that allowed users to just directly submit a file-path or video-encoded bytestream and let Rerun take care of converting to a stream of images.
Although our data-loader APIs make it possible to do this as a plugin, it might be worth offering a dedicated API for this. In particular, a dedicated API would allow us to be explicit about how frame-ids map to time-points, which is not something we currently have the ability to do through data-loaders.
This is a situation where it would likely make sense to do the implementation in Rust once and then offer bindings through to the other languages.

Ongoing work in https://github.com/rerun-io/rerun/pull/7261

Needs to produce video frame references


Allow upgrading to writer without knowing if reader/writer

I found this very useful in some complex code that had to occasionally upgrade in a few spots
Sorry the spacing/tabs seems off, it keeps massacring the formatting when I try to submit this--

The odds of this being integrated are close to zero with only "a few spots" in "some complex code".
BTW, don't get me started about tabs, but your clue should be that TBB does not use them!

Actually, since TBB 2019 U5 the read-write mutexes in TBB allow upgrades and downgrades to "succeed" if the mutex is already in the requested state. In other words, it's now safe to call upgrade unconditionally, without first checking if the current thread is a reader or writer. The change was inspired by this PR, which can now be closed.

I guess it must have been more than "a few spots" in "some complex code" then...


[Pods proposal] Visual separation of networks and containers

This is a proposal of a visual improvement for pods configuration tab. As Containers and Networks are large it’d be nice to have a visual separation.

Hey @leemunroe, we've extracted the change into a separate PR. Do you have any suggestions for the container icon? Right now we're using gears and I don't think that's the best visual for a container.

@nLight Yes. @ashenden maintains our icon library. I'll run it by him.

Rebased this one.
Hey @leemunroe @ashenden do you guys have a ticket for the icon we can add to this PR?

@nLight  DCOS-10217
For 1.9 will this look the same for both pod and non-pod services? Could you provide us a screenshot of both to preview?

Hi @leemunroe, sorry it took some time to reply.
We definitely have to align all those views. We should come up with a solution on how to show groups of resources. I.e. networks, containers, port mappings etc.
Containers:

Service:

@nLight Sorry I was looking for a screenshot of how this config view looks with your suggested changes i.e. with the solid grey border on the left and the icons next to the ports and containers.

One screenshot for how this looks for a Pod service
Another screenshot how it looks for a non-Pod service
And if you could include more of the screen that will help give me context

Hey @leemunroe sorry for the late response, I've been busy with a core library upgrade.
We've decided not to proceed with this PR because of the CNVS upgrade.


"too many SQL variables" error when using Preload

When using Preload, e.g.:
db.Preload("Teams").Find(&users)

I'm getting the following error:
too many SQL variables

The main issue is that there's no way (or at least I'm not aware of a way) to split into multiple queries based on the limit of variables per query and sqlite3 doesn't have any way to increase the maximum.
Additionally, when outputting queries, I saw that it was not de-duping the preload items.  For example, using just a couple users/teams w/ preload it'll use this query:
SELECT * FROM "teams"  WHERE ("id" IN ('1','1','1','1','1','1','1','2','2','2'))

You can see it's trying to find the same team multiple times.  I assume this is contributing to the issue.

sqlite3 doesn't have any way to increase the maximum

maybe use MySQL / PostgreSQL ?

maybe use MySQL / PostgreSQL ?

Yeah, that's a potential workaround.  The workaround I'm doing is to just load manually instead of using Preload.
Figured it was worth filing an issue though since it would be easier/convenient to use Preload and it looked like there was some room for optimization :)

We have did some work to remove some duplicated values in the where, hope could avoid this issue.

Hi, @jinzhu Thanks for your great work.
I didn't find any solution if I want to use Preload to distinct the where assumption. It still had duplicated values in the where.
I would ask if there was any way to avoid the problem during using Preload()?
I still met the "too many SQL variables" issue.

Hi, I just encountered this problem myself.
From what I'm reading from previous comments this doesn't seem to happen with MySQL and PostgreSQL, so being this an ORM library I think the issue should be addressed in the library, while manually preloading as @jchavannes suggested should only have been a temporary workaround while a proper fix gets implemented.


Allow setting the "desired speed" of agents independently of the "max speed".

Fixes #81.

Codecov Report
Attention: Patch coverage is 98.30508% with 4 lines in your changes missing coverage. Please review.

Project coverage is 97.0%. Comparing base (c50de33) to head (5c81b79).

Additional details and impacted files

Files
Coverage Δ

crates/bevy_landmass/src/agent.rs
86.6% <100.0%> (+0.1%)
:arrow_up:

crates/bevy_landmass/src/lib_test.rs
99.8% <100.0%> (+<0.1%)
:arrow_up:

crates/landmass/src/agent.rs
97.8% <100.0%> (+<0.1%)
:arrow_up:

crates/landmass/src/agent_test.rs
100.0% <100.0%> (ø)

crates/landmass/src/avoidance.rs
95.8% <100.0%> (ø)

crates/landmass/src/debug_test.rs
100.0% <100.0%> (ø)

crates/landmass/src/lib.rs
93.4% <100.0%> (+1.0%)
:arrow_up:

crates/landmass/src/avoidance_test.rs
98.9% <98.1%> (-0.2%)
:arrow_down:

crates/landmass/src/lib_test.rs
99.7% <97.5%> (-0.3%)
:arrow_down:


ARROW-8514: [Developer][Release] Verify Python 3.5 Windows wheel

A number of features are disabled in the 3.5 wheel.

+1


Completion highlighting incorrect when using filterText

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 220f9387c973de4cc44c545642ad2c90170bd3ef (2022-12-06)
OS Version:

Steps to Reproduce:

provideCompletionItems that returns an item with filterText = "x_fooBar" and label = "fooBar" (filtertext doesn't matter, as long as it matches whatever is typed)
type ob
highlight should be fo^o^Bar, but you get foo^Bar (the o should be highlighted and is not)

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

This bug has been fixed in the latest release of VS Code Insiders!
@mroch, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version 270c3b12906a88e9c67fc5ddef87be6aa3b75551 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!


Add ReactPHP topic

[X] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md

I am:

[ ] Suggesting edits to an existing Topic page
[X] Curating a new Topic page
[X] I've formatted my changes as a new folder directory, named for the topic as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME])
[X] My folder contains a *.png image (if applicable) and index.md
[X] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

With this topic it's easier to find packages using, or build for or, build on @ReactPHP.

Awesome, thank you :tada:


mvc model binding json ajax requests

iv noticed when doing an ajax post request using content type 'application/json' in beta 8, its always throwing a json exception before hitting the controller action, and no data/ variables are passed to the action method. Although this works when the content type is set to ''application/x-www-form-urlencoded" as its coming through in [fromForm] mode not [fromBody].
var postData = { abd: '1', d: '2'};

$.ajax({
type: "POST",
contentType: "application/json",
dataType: "json",

data: postData //or data: JSON.stringify(postData)
})

Exception:
Event Type	Event	Time	Duration	Thread	Detail Description
Exception	Exception thrown: 'Newtonsoft.Json.JsonReaderException' in Newtonsoft.Json.dll ("Error reading string. Unexpected token: StartObject. Path '', line 1, position 1.")	1,980.82s		[9108]

@kichalla can you take a look and see what's going over the wire and where Json.NET is throwing?

I am unable to repro this...the data is bound properly...I tried with the following:
function sendData() {
    var postData = { abd: '1', d: '2'};
    $.ajax({
        url:"api/values",
        type: "POST",
        contentType: "application/json",
        dataType: "json",
        data: JSON.stringify(postData)
    });
}

@david-whitehead can you share how the raw request looks like (if you are using Chrome you can copy the request/response details by doing Copy all as HAR)

Hi, thanks for getting back to me, After doing some reason digging i have found some more information out which might be useful.
when using the ajax like below is perfectly fine with specific situations.
ContentType: "application/json",
data: JSON.stringify(postData)
this does does work but only when we are specifically telling the action method parameters to be bound to an object and not directly. For example, the code below will work perfectly fine:-
public class TestDetails
{
[Required]
public string testing1
}
//in mvc not web api
public JsonResult apiEndpoint([fromBody] TestDetails details)
{
.......
}
but the following will not work and is what causes the newton json exception:
public JsonResult apiEndpoint([fromBody] string testing1)
{
.....
}

In the case of this code:
public JsonResult apiEndpoint([fromBody] string testing1)
{
.....
}

That parameter is saying the JSON itself must be a string (e.g. 'foo' was sent over the wire), but it isn't, it's an object.

But if thats the case what would be required for more than one parameter,  as it would have to be sent in object format due to multiple parameters but still has the same issue

You cannot have multiple parameters that have [FromBody]. You would need to have a single type encapsulating all that information.

But with the other content types you can have multiple fromForm variables should this not be the same for all versions?

Your observation is correct. In case of FromBody, the formatters like json and xml and their respective serializers take full control of how the request body is bound to an type. I will let @rynowak @Eilon @dougbu  to give more detailed explanation

@kichalla  just wanted to say thanks for this discussion,
so generically were saying you should really encapsulate every request in its own object?
does that not cause quite a bit of excess code that is not really needed for smaller requests i.e. 1 or 2 parameters which do very little, It just seems were creating classes for the sake of it rather than pre-defining our requirements and making the decision on whether or not we should require the parameters to be encapsulated for further needs i.e. validation etc.

You can check out the MusicStore sample app to see how an AJAX request can send the anti-forgery tokens: https://github.com/aspnet/MusicStore/blob/221bf3049219d66730d308f538f4a613cf2c9430/src/MusicStore/Views/ShoppingCart/Index.cshtml

@david-whitehead Yes, to bind data using formatters you need to define a type for the body as the formatter abstraction is a stream to object conversion.

Hi, so just to confirm, it is expected behavior that a POST with the body { country = "AT" } can NOT be bound to public IActionResult Index(string country) { ... } or public IActionResult Index([FromBody] string country)?
Instead, we need to create a class like this:
public class CountryDto {
  public string Country { get; set; }
}

... and use public IActionResult Index([FromBody] CountryDto country) { ... } ?

@cwe1ss so confirmed.


Disposal improvements for ScriptHost

Related to https://github.com/Azure/azure-webjobs-sdk/pull/1398.

Also... many (most) of these changes apply to dev as well

Addressed feedback - please have another look. Yes - I'll merge this into dev as well.

Merged. Let me know if there is any additional feedback.


Lane editor

I merged in this progress so far, but kept the lanes preset as the old numeric one for now, while we continue to work on the new field.
merged as:
304b3b3 2016-06-22 Restore numeric lanes preset, but keep code for new WIP lanes field
8b0fb33 2016-06-22 Remove legacy pre-module lanes.js
332200b 2016-06-22 display lane count on sidebar
d130456 2016-06-22 make lane test lint free
aa829dd 2016-06-22 Lanes forward backward tests
07827e5 2016-06-22 Adding test case for lanes().default
4083a0e 2016-06-22 adding a lanes() to return lane data


Fixes #11888/spacing issue in ListBlock nested blocks

Please describe the problem you're fixing here. Include the issue number, if applicable.
Resolves #11888
This PR addresses the spacing problem encountered in ListBlock when dealing with nested blocks. It ensures that the spacing between nested blocks is consistent, thereby improving clarity and visual presentation.
Changes Made:

Removed margin-bottom in CSS styles to maintain consistent spacing

Testing:

Tested the fix locally to ensure proper spacing behavior
Ran existing unit tests to verify no regressions

Please review and merge. Thank you!

Before:

After:

now there is no extra spacing over the button

@lb- PTAL !


Timeline: show only working days

Hi
I would like display only working days on my timeline.
I also would like display time between 9AM and 8PM. Because I dont need time betweenv 8PM and 9AM
Is there any possibility for doing this ?

This is not possible with the Timeline of chap-links-library, but it is supported by its successor: http://visjs.org. See this example where the weekends are skipped: http://visjs.org/examples/timeline/29_hiding_times.html


Should be able to run specs after configuring package_to_scan

Actual behavior
When package_to_scan is specified, the step implementations are not recongnized
Steps to replicate

Create a gauge-maven project
gauge init java_maven

with group id grp1

The default package is grp1
Now add a package grp2 at the same level as grp1
Add a package level2 inside grp2
Create a class inside grp2.level2

package grp2.level2;

import com.thoughtworks.gauge.Step;
import com.thoughtworks.gauge.Table;
import com.thoughtworks.gauge.TableRow;

import java.util.HashSet;

import static org.assertj.core.api.Assertions.assertThat;

public class StepImplementation {

    @Step("Something")
    public void implementation1() {

    }
}

Use step "Something" in example.spec
Run the spec example.spec from the IDE.

Execution is successful

Add package_to_scan=grp1,grp2.level2
Re-load the project
Execute the same spec

During validation the steps are marked as unimplemented

Version
Gauge version: 0.9.9.nightly-2018-06-01
Commit Hash: 812f26e

Plugins
-------
java (0.6.9.nightly-2018-06-06)

Related to https://github.com/getgauge/gauge-java/issues/161

Seems to be working fine.
Check if you have added the correct nightly version of gauge-java in the pom.xml as well.

Working as expected with
Gauge version: 1.0.1.nightly-2018-06-19
Commit Hash: 583ef1d

Plugins
-------
java (0.6.9.nightly-2018-06-12)

Re-load after making the following pom changes.
 <repositories>
        <repository>
            <id>maven-gauge-java</id>
            <url>https://dl.bintray.com/gauge/maven-gauge-java</url>
        </repository>
    </repositories>
    <dependencies>
        <dependency>
            <groupId>com.thoughtworks.gauge</groupId>
            <artifactId>gauge-java</artifactId>
            <version>0.6.9.nightly-2018-06-12</version>
            <scope>test</scope>
        </dependency>
...
    </dependencies>


Context-awareness of whole workspace/repository

To be blunt, chat is useless unless it has my workspace/repo as implicit context. I might as well just paste stuff into chatGPT which has a bigger UI since I have a whole page view for my browser.
Compare this to SourceGraph Cody, in which I queue up an embedding of my repo, and then 💥 I can ask questions of my whole codebase. That's the experience I want.
Edit: I see nobody actually replies to the feedback anyone files 🙃

we all want this future, and yea you are right it makes no sense using it if it doesn't include workspace/repo context.

This is exactly what I expected from Copilot Chat. This would be game changer.

This would be awesome!

Copilot appears to understand some context about the folders. At least file names. cmd+shift+p indicates that there are some options that are coming in the future?

I got a bit surprised asking it to use my workspace as context for my question and it answered:

I'm sorry, but as an AI language model, I don't have access to your workspace or any other external resources. However, the code you provided is a function called ...

I agree this would make a huge difference. Though it probably has to be an opt-in per workspace or folder hierarchy, because people will probably have workspaces they don't want to, or don't have the right to, share with external parties.

Sorry for the delay, was at a conference and we just opened this repo for feedback and are still setting up our usual triage tooling.

I also think that being able to make questions about the workspace is a must. It would help to understand and browse large codebases. Adding the option to enable and disable per workspace also looks like a good suggestion to me.
Some example questions using the linux kernel:

can you expand the preprocessor macro SYSCALL_DEFINE5?
which structure is used by the key subsystem to store the keys?
can you point me to the main function for the write syscall?
which are the possible paths to reach the function keyctl_invalidate_key?
can you point to structures which their size is 48 bytes?

I have included SourceGraph Cody output for reference:
https://imgur.com/a/Zr0WZ4v

Duplicate of #14


Don't rely on debug print format

This crate assumes that {:#?} prints valid rust source code. Rust makes no guarantees about Debug format, so this code could break in the future.

Thanks, you are right. Probably rustversion crate also has the same issue.

Published 0.4.3 which fixes this issue.


[analytics] Add OneDollarStats collector

What is this Contribution About?
Please provide a brief description of the changes or enhancements you are proposing in this pull request.
Issue Link
Please link to the relevant issue that this pull request addresses:

Issue: #ISSUE_NUMBER

Loom Video

Record a quick screencast describing your changes to help the team understand and review your contribution. This will greatly assist in the review process.

Demonstration Link

Provide a link to a branch or environment where this pull request can be tested and seen in action.

Hard coded in https://deco.cx/.
Thinking about this PR. Can we just add on Page.tsx? to all websites

Hard coded in https://deco.cx/.
Thinking about this PR. Can we just add on Page.tsx? to all websites

yeah.. let't avoid the need to mangling with user configuration here...


Problem in executing RAINMAKER SWITCH code on ESP32C3(devKit-M1) through BLUETOOTH (MEGH-2565)

I have executed code of SWITCH (esp-rainmaker--> examples --> switch) from RAINMAKER using ESP32C3. The device was visible on RAINMAKER App and I could switch the LED through WiFi connectivity. But my query is "How to execute same code via ESP32C3's Bluetooth.

@Shravani100 , controlling over Bluetooth isn't supported yet.

@shahpiyushv thank you for the information regarding rainmaker and its connectivity to ESP32C3 via bluetooth.
Now with reference to access ESP32C3 Bluetooth  what I did is, I have downloaded nRF Mesh app to form connectivity between ESP32c3 BLE and mobile. And I could connect device successfully. So is it possible to control (on/off) ESP32C3 via same app using BLE or I need to go for some another way.

@Shravani100 , sorry I missed this. This is not really in RainMaker's scope since RainMaker is primarily for cloud communication (with an option of local control over Wi-Fi), but even then could you proceed with what you were trying with BLE mesh?


HBASE-22551 TestMasterOperationsForRegionReplicas is flakey

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
39
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

+1
mvninstall
264
master passed

+1
compile
53
master passed

+1
checkstyle
75
master passed

+1
shadedjars
279
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
207
master passed

+1
javadoc
33
master passed

_ Patch Compile Tests _

+1
mvninstall
257
the patch passed

+1
compile
55
the patch passed

+1
javac
55
the patch passed

-1
checkstyle
74
hbase-server: The patch generated 1 new + 2 unchanged - 3 fixed = 3 total (was 5)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
279
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
786
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
220
the patch passed

+1
javadoc
32
the patch passed

_ Other Tests _

-1
unit
13583
hbase-server in the patch failed.

+1
asflicense
26
The patch does not generate ASF License warnings.

16600

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-295/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/295

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 5e2b4ed98d33 4.4.0-144-generic #170~14.04.1-Ubuntu SMP Mon Mar 18 15:02:05 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / b32e716bee

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-295/1/artifact/out/diff-checkstyle-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-295/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-295/1/testReport/

Max. process+thread count
4473 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-295/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
38
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

+1
mvninstall
255
master passed

+1
compile
56
master passed

+1
checkstyle
76
master passed

+1
shadedjars
279
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
223
master passed

+1
javadoc
34
master passed

_ Patch Compile Tests _

+1
mvninstall
260
the patch passed

+1
compile
59
the patch passed

+1
javac
59
the patch passed

+1
checkstyle
72
hbase-server: The patch generated 0 new + 2 unchanged - 3 fixed = 2 total (was 5)

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
280
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
791
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1
findbugs
187
the patch passed

+1
javadoc
33
the patch passed

_ Other Tests _

-1
unit
15186
hbase-server in the patch failed.

+1
asflicense
31
The patch does not generate ASF License warnings.

18201

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-295/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/295

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 90581ffa9676 4.4.0-144-generic #170~14.04.1-Ubuntu SMP Mon Mar 18 15:02:05 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / aab70b5966

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-295/2/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-295/2/testReport/

Max. process+thread count
4572 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-295/2/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.


Verify dynamical added Fields via JS

Hi,
I've created a form to edit select attribute element (=select data with dynamical option; 1 to N): so, the user can add options when he clicks on an "added value option" button.
In JS (public), I added  a new row witch contains form fields for the new option (as example :  New Option X).
When I submit the form, the server form doesn't know my new dynamical fields, but only those I've created during declaration the form in server side (before form render).
That's normal, but how could I check a dynamical field added to the form in public side and unknown from server side without Ajax called by field ?
My problem is, for the moment, "I add form data to mongodb if the form verification is correct, I'll added it :
New Option X
New Option By HACKER
But, as you can see, for the second record, an hacker hacks my form but pass through verification because the first verification is passed well.
How can I add forms fields in front side via JS dynamicaly (JQUERY) and ensure a good form verification via caolan/forms ?
Thanks in advance !
Raph

You can't, unless you also add the fields on the serverside - since the server has no way of knowing what you did dynamically.
The primary use case for forms involves creating 100% of the final form on the server, and not altering it on the client - so you can use the same form object on both client and server for validation.

Hi !
And thank you for your answer !
Most of the time, when I must create a form for a client, it must be dynamical as I said in my example. It could be interesting to do a widget to verify some pieces of fields like :
`
Child 1

Child 2

`
With server side form field with wrapper type; not verified if does not exist in submitted data and not mandatory, but required and verify if there are.
Without to call server via Ajax or create a dynamical form from server side; just said, "that's cool, you can create a new form field for your child, but if you create it, you must complete his age."
Raph

If you want to use forms to validate this use case, you need to have a way for the server to know which fields it needs to add prior to .binding and validating. There's no way around the fact that if the browser has knowledge the server doesn't, the server can't validate the request.

Hi !
And thank you for your answer !
Mh, not agreed,  we can told via server side : "this field can have 0 to n values,  if submitted, we must verify if ages and firstnames are not empty; it was my proposition via a wrapper widget for dynamical fields.
As example :
var wrapperChildren = fields.string({name: 'children', widget : widgets.wrapper(), fields :  age          : fields.string({required : true, label : "Age"}),  firstname : fields.string({required : true, label : "Firstname"}) });
The server can receive an array of children information and can verify it
children name is posted, it is a wrapper of 0 to n fields, I know the fields value to verify (age and firstname for each one). If I haven't those fields, failed, go to the next. Next match to verification (wrapper[1][age] = 18 && wrapper[1][firstname] = bob), etc.
So the form is initialized from server side, but client can propose 0 to n submit for a field with a correct server side verification.
what do you think ?
Raph


Contrast Normalization

I'm closing this for now because I don't think we actually need this functionality in current or planned pipelines.


Bump Marathon to 1.7.216

High-level description
Bump Marathon to 1.7.216
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:
MARATHON-8596 Introduced global throttling for Marathon health checks
MARATHON-8590 Do not fail on offers with RAW and BLOCK disk types
MARATHON-8575 Map tcp,udp to udp,tcp during migration
MARATHON-8581 Allow all users to execute /marathon/bin/marathon
MARATHON-8562 Response asynchronously for all endpoints
DCOS-49521 Force expunge and Decommission all instances on service removal
Related tickets (optional)
Other tickets related to this change:
n/a
Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: Marathon 1.7 branch is running SI tests against DC/OS 1.12 nightly
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Test Results: Marathon 1.7 Jenkins build
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE

@mesosphere-mergebot label Ready For Review


fix donate.html dark theme

i also added linkedin icon at the bottom

Related Issue
Closes: #1356
Description
i added linkedin logo at the bottom. i also fixed the dark theme for donate.html + there was a text not visible in light mode. tweaked that!
Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

fetch and merge to resolve conflicts

please fetch and merge to resolve conflicts

#1789
i've submitted another pull request maam. kindly check


git-submodule-rewrite: enhance performance by using git-filter-repo

I am studying using this script, but it just took two hours to process one branch, and it's configured to operate on a temporary path that it deletes afterwards, so as soon as I'll want to merge another branch, I think I'm in for another two hour processing per branch
While googling around, I found something quite promising under the name of git filter-repo (https://github.com/newren/git-filter-repo/) whose use is even advised in the documentation of git filter-branch itself
So the question is, how hard would it be to change the script to use filter-repo instead of filter-branch and would there be any reason not to ? While glancing at it, it seems like it could solve a few problems at once: first, I would not have to reprocess the commits for each branch (granted I hook a non-temporary working path in the arguments too instead of the temp dir), and could process the whole repository at once, and second, the perf is promised to be much, much better
And then the corollary question would be, if I change the script to use filter-repo, would you be interested in a PR ?

That seems reasonable.  My original use case with this repo was pretty narrow, but many people have applied it to much more extensive usage patterns.  I'd be happy for someone to enhance the code.  However, this repo is in desperate need of some unit tests, and making such an significant change would likely require some.  Are you interested in contributing?


Early Sync

Sync/Configs mechanism for DR Rules only. We can expand to all types in the future (only Rules are needed for the Go lc-service).
Added a JSON and YAML compat layer to help normalize the datatypes, taken from go-essentials.
Permission check helper

PTAL


Allow setting up Player from local AVAsset

Updated #132

thanks @efremidze ✌

👏 thanks for doing this @efremidze !


RAC Tooltip doesn't work for custom components

Provide a general summary of the issue here
RAC Tooltip doesn't open when the trigger element is a custom component, such as:
function MyButton(props) {
  return <button type="button" {...props} />;
}

🤔 Expected Behavior?
Tooltip should work for custom components, as long as they're focusable and hoverable.
😯 Current Behavior
Tooltip doesn't work for custom components.
💁 Possible Solution
No response
🔦 Context
In our design system, we need to support adding a tooltip to various custom components such as Button, TextField, Select, ComboBox.
🖥 Steps to Reproduce
https://codesandbox.io/p/sandbox/recursing-liskov-mfyj6m?file=%2Fsrc%2FApp.js%3A20%2C63
Version
1.0.1
What browsers are you seeing the problem on?
Chrome
If other, please specify.
No response
What operating system are you using?
macOS 14.2.1
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

It appears that for using Tooltip, you should use the useButton hook instead of directly passing props. Check this out at the following link for a demo: https://codesandbox.io/p/sandbox/distracted-kowalevski-46fp8s?file=%2Fsrc%2FApp.js
Additionally, for other components like TextField, Select, ComboBox, there are specific hooks available for each. It might be beneficial to use them in conjunction with your custom components!

For context, we're not using RAC Select/ComboBox because RAC ComboBox doesn't support multi-selection (https://github.com/adobe/react-spectrum/issues/2140).
And I assume useComboBox wouldn't work with a react-select-based ComboBox implementation.

As @ryo-manba mentioned, the trigger element should be from a focusable/hoverable element either from RAC, or created by a hook like useButton. Here is a good explanation from @LFDanLu in https://github.com/adobe/react-spectrum/discussions/5119 on how to handle this:

Our RAC Button's call of useButton accesses this context via a useFocusable call so if you were to use your own custom button element you'd have to call useButton as well. For non-button elements you'd call useFocusable and usePress.

Closing since this is expected behavior, but feel free to post any related follow-up questions here.

Just noting here that the useButton workaround does not seem to work when it is used for cloned children.
function MyTooltip(props) {
  const triggerRef = React.useRef();
  const { buttonProps } = useButton(props, triggerRef);
  const trigger = React.cloneElement(
    props.children,
    mergeProps(buttonProps, props.children.props, { ref: triggerRef })
  );

  return (
    <TooltipTrigger delay={0} closeDelay={0}>
      {trigger}
      <Tooltip>{props.content}</Tooltip>
    </TooltipTrigger>
  );
}

Full demo here: https://codesandbox.io/p/sandbox/goofy-sunset-plg3c3

Right, there's a context that is consumed inside useButton, so that needs to be in a component inside the TooltipTrigger, not outside it.

Thanks for the explanation @snowystinger! Is there a better way to build your own custom Tooltip that would utilise RAC Tooltip components underneath? We've been using react-aria hooks so far, but the API provided by RAC is much cleaner and we'd love to use that instead.

Could you use our RAC button as well? Or is the trigger element provided to your Tooltip component not controlled directly by you (aka is it provided by the end user?) What setup do you have for your custom Tooltip? For you sandbox, you can probably just wrap the trigger in a wrapper that calls useButton

Or is the trigger element provided to your Tooltip component not controlled directly by you (aka is it provided by the end user?)
Yes, exactly that, hence the RAC Button is off the table for us.

What setup do you have for your custom Tooltip?
Not sure I get this, but it is built as part of an in-house design system component kit. Ideally we'd not change anything functionally and just apply our styles on top of the Tooltip provided by RAC. We do not restrict the children to be of any specific component, so worst case it can be something not button-like, e.g. just an icon.

For you sandbox, you can probably just wrap the trigger in a wrapper that calls useButton
I am aware of that and I've considered it, but I would ideally not wrap children with any additional components to keep the DOM tree clean and avoid causing layout issues.

Thanks for jumping into this!

I am aware of that and I've considered it, but I would ideally not wrap children with any additional components to keep the DOM tree clean and avoid causing layout issues.

The wrapper could do the same clone operation + useButton call that you've done in your sandbox and return the result of that, no? That way you aren't adding an extra DOM node.

The wrapper could do the same clone operation + useButton call

I thought it would have the same problem as the originally suggested solution, but you are right - if I do the cloning in a wrapper component it works 🎉
Thanks a lot @LFDanLu and others for helping me out!
I will leave the working code snippet here in case if anyone wants to use the same solution:
import React from "react";
import "./styles.css";
import { Tooltip, TooltipTrigger } from "react-aria-components";
import { useButton, mergeProps } from "react-aria";

export default function App() {
  return (

      <MyTooltip content="Hello">
        Show me tooltip
      </MyTooltip>

  );
}

function TriggerWrapperWithoutDiv(props) {
  const triggerRef = React.useRef();
  const { buttonProps } = useButton(props, triggerRef);

  return React.cloneElement(
    props.children,
    mergeProps(buttonProps, props.children.props, { ref: triggerRef })
  );
}

function MyTooltip(props) {
  return (
    <TooltipTrigger delay={0} closeDelay={0}>
      <TriggerWrapperWithoutDiv>{props.children}</TriggerWrapperWithoutDiv>
      <Tooltip>{props.content}</Tooltip>
    </TooltipTrigger>
  );
}


[5.2] Merge recent master improvements into swift-5.2-branch

The goal is to pull in the following improvements:

Fixits support via codeActions
Build setting lookup for headers using the index to find main files
Compilation database performance improvements that unblock using
sourcekit-lsp with larger cmake projects like llvm.
Compilation database support for finding indexing-while-building data

This is being done with a merge rather than cherry-picks, because the above changes represent that vast majority of the delta between master and 5.2, and it is less risky to move to a known state that is already in nightly master toolchains and getting (some) dogfooding from myself. The additional changes are also low risk in themselves, specifically

Avoid throwing C++ exception through no-except frames (indexstore-db)
Fix build system delegate functions that were missing a queue.async to avoid racing.
A handful of changes that have no runtime impact (test-only changes, changes to the library API not currently used in the executable, documentation improvements).
Updates that only affect platforms not currently supported.

rdar://59426221

Scope: Affects only sourcekit-lsp. No changes to compiler or sourcekitd.
Origination: N/A
Risk: Low risk to the overall product - this does not touch anything outside of sourcekit-lsp. Low/Medium risk for sourcekit-lsp itself simply because of the quantity of changes.  Mitigating that, the changes here are already relatively well-tested on master, have been manually exercised by myself.
Testing: All individual changes come with appropriate unit tests. I have been living on this state (on master) successfully for several days and intend to live on the new 5.2 toolchain as soon as possible and do some manual testing.

@swift-ci please test

https://github.com/apple/indexstore-db/pull/70
@swift-ci please test


Set IREE_AVAILABLE_BACKENDS in Bazel CI

This will replace IREE_DEFAULT_BACKENDS as part of https://github.com/google/iree/pull/1375, but we'll set both for now so that it can be submitted without further breakage (for annoying infra reasons, we can't submit workflow changes as part of the same PR).
Also finally added line breaks to this ridiculously long line. There's some yaml magic you can do with a chomp operator or something, but I just did boring old line continuations.
Tested: No. It's basically impossible.

The GitHub action has made it far enough in that I'm pretty sure the new command isn't malformed. It was previously failing, but we can check back later if it's got new errors :-/


[resolves #255] Unable to load properties file from classpath

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/192/
Test PASSed.