[BUG] Some queries failing with code 500: all shards failed | Failed to compile inline script

Describe the bug
I started TDVT test suite using JDBC driver 5 times and got 5 different results:
test-1 315 passed
test-2 312 passed
test-3 311
test-4 321
test-5 251

To Reproduce
Steps to reproduce the behavior:

Install TDVT, see links: one, two, three.
Unpack connector to C:\Users\%username%\Documents\My Tableau Repository\Connectors
Launch an instance of OpenSearch
Load test data into it using files in test-tables.zip
Create tds files as it described in links above or use attached ones: tds.zip. Note, they works only with OS running on http://localhost:9200 without authentication
Run tests: python -m tdvt.tdvt run OpenSearch few times

Expected result
Tests would produce the same result all times
Actual result
214 tests failed at the 5th run with the following error:
Shard[0]: GeneralScriptException[Failed to compile inline script [rO0ABXNyADRvcmcub3B ... <omitted>] using lang [opensearch_query_expression]]; nested: CircuitBreakingException[[script] Too many dynamic script compilations within, max: [75/5m]; please use indexed, or scripts with parameters instead; this limit can be changed by the [script.context.aggs.max_compilations_rate] setting];\n\nFor more details, please send request for Json format to see the raw response from OpenSearch engine.

Additional context
The connector forces driver to produce log at %temp%\TableauTemp\jdbc-driver-log.txt
See logs attached:
test-5.zip
test-1.zip
test-2.zip
test-3.zip
test-4.zip
tests-3-4-5-result-compare.csv
test-5-jdbc-driver-log.txt
test-5-part-of-server-log.txt

Many of the expressions are translated to DSL before executing, this results in a lot of compiling work, so when many requests hit the engine in parallel, the engine might run out of the resources and give out compiling error. We should optimize the expression calculation to fix this issue, but short termly it is recommended to scale your instance up or out. Optionally, we also recommend you change the queries to our preferred semantic, for example, use implicit/explicit cast from string literal to timestamp (e.g. cast('2021-11-09 00:00:00' as timestamp) or directly '2021-11-09 00:00:00') instead of using functions like timestamp('2021-11-09 00:00:00') to do the data converting.

I have conducted some research into the performance of the functions you mentioned and data type inputs for those functions to see how quickly they run to see if that is an issue that could cause the test failures. The results are in operations per second meaning that the higher the number is the faster it runs, below I have laid out my findings:

Operation
Data Type
2.4
upstream/main

CAST AS TIMESTAMP
STRING
834.418
382.739

CAST AS TIMESTAMP
TIMESTAMP
3538.903
2129.168

CAST AS TIMESTAMP
DATE
N/A
1165.644

CAST AS TIMESTAMP
TIME
N/A
1117.844

TIMESTAMP()
STRING
648.069
282.688

TIMESTAMP()
TIMESTAMP
2872.664
5274.195

TIMESTAMP()
DATE
2531.714
719.601

TIMESTAMP()
TIME
N/A
693.222

The overall efficiency of almost all functions have gone down in the newer version with the exception of giving TIMESTAMP() a TIMESTAMP but that doesn't matter as much compared to the difference in performance from TIMESTAMP() to CAST AS TIMESTAMP. Overall it appears that the average TIMESTAMP() function is roughly 60-70% of the speed of casting it to timestamp with the outlier of casting a timestamp to a timestamp being exceptionally quick.
So although it is faster most of the time to case as timestamp it doesn't appear to be a big enough of a performance hit to be causing test to fail

Here is the graph for memory usage on the current opensearchsql version:

Each function is ran 1000 times to get consistent results and the result is in Bytes per operation, because of this you can divide the result by 1000 to get the amount of data per single operation.
Here is the results of the same function compared to the 2.4 engine:

From a precursory glance at both graphs we can see that almost every operation has just about doubled in memory allocation except for inputting a timestamp datatype in the timestamp() function. Looking at the difference in memory usage from using timestamp() compared to casting we can see the casting uses ~30% less memory on average, which I don't believe would be enough to cause tests to fail.


sparse-checkout repo error

Summary 💡
I got an error when using onefetch in git sparese-checkout repository.

Motivation 🔦
No response

Thanks for reporting this @lizhuoqi.
It seems like libgit2 (onefetch still uses libgit2 for displaying pending changes) currently still does not fully support sparse checkout (https://github.com/libgit2/libgit2/issues/6044). It lacks support for Git extensions.worktreeconfig. There is an ongoing PR for that: https://github.com/libgit2/libgit2/pull/6202.
We can keep this issue open until upstream supports is added.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

We can keep this issue open until upstream supports is added

Pinning the issue

Should be fixed with the latest release 2.20.0
Please re-open if the issue persists.


Added rook ceph operator config map reconcile func

@ezio-auditore Please add a description to the PR


Injecting env vars in MAPDL process

Using subprocess.Popen(..., env), we should be able to start the MAPDL process with specifics env variables.
Close #1218

I'm going to ignore the Code Static analysis because we control the shell part which is flagged.


Testing with react-testing-library

Create an initial test structure using jest and @testing-library/react with initial tests for the components:

Anchor
AppBar
Avatar
Bar
Button
Cutout

Obs: This is related to #27

:tada: This PR is included in version 3.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Two Way Barrier for Authenticating ClientID and Secret for OAuth - Not Possible with Twitch/Streamlabs

Hi there,
So I have been trying to set up the stream notifications extension in LNBits, and I have discovered that perhaps since this was implemented as an extension in LNBits, the Stream Notifications will not function properly because of the OAuth2 process and how both Twitch and StreamLabs have built their registration system.
To Reproduce:
Install LNBits and Stream Notifications Extension
Follow The Guide in the Documentation for LNBits in legacy
Upon Creating "Add Service" enter twitch username and Use Streamlabs for Notification Service.
Asks for Client ID and Client Secret
Go to Twitch or StreamLabs and Follow their Documentation for Registering an App for their API
The registration process requires the Redirect URI (in my case it would be: [https://sats.satorinet.love] https://sats.satorinet.love/streamalerts/api/v1/authenticate/* ( requires an API string where * is )
This does not exist and is not created because the extension wants first the client ID and secret before generating a redirect URI
The Client ID and Secret do not exist because Twitch and StreamLabs want a valid redirect URI during the registration process before offering the Authentication token.
One cannot Authenticate with OAuth2 because both requirements require one another and therefore cannot be done.
Is it possible to generate a redirect URI for the service in question first in LNBits without requiring the Client ID and Client Secret to generate one, so that we can add it when Registering with Twitch or StreamLabs.
And then input the Client ID and Secret as given by Twitch/Streamlabs to LNBits and continue with OAuth2 Authentication in LNBits.
My guess is this was designed before Twitch and StreamLabs created a developer App registration process, or made adjustment to it, and beforehand it would have been possible to obtain the ID and Secret with them before having a redirect URI. Now they have barricaded the registration process and this Stream Notifications extension will be impossible to create a service with, it is not up to date with their process. All that is needed to to generate the redirect URI before entering the ID and Secret and not after.
Thanks

Hey there, thanks for opening the issue! I personally no longer maintain the extension, and I don't want to work with Streamlabs anymore in general. I think you guess is correct. If Streamlabs implemented a new app registration process, that is likely the problem. I've used StreamElement's current API for personal project before, and should I come back to this, I would probably replace Streamlabs with StreamElements.
But since I currently have no time to work on the extension, it might be best to ask if anyone in the lnbits repo or Telegram group would be willing to investigate this. I'm sorry for the inconvenience.
I will leave this issue here open, and you can link back to it!


Create an Azure Red Hat OpenShift 4 private cluster

I was walking through this: https://docs.microsoft.com/en-us/azure/openshift/howto-create-private-cluster-4x from a brand new account.
 Correlation ID: 1228a457-7ca4-4ade-b122-9b638a8db330. Resource quota of cores exceeded. Maximum allowed: 10, Current in use: 0, Additional requested: 36.

It seems I have to bump my quota, but I have no idea what version of the CPU I need to bump. Going through the support request I had a list of over 50+ CPUs and completely overwhelmed.
Maybe add a note on what CPU this command attempts to spin up and mention the quota change if needed?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0b9ab474-2ead-2841-31ed-df59e1f83a60
Version Independent ID: fbc0a299-c4e7-b8fb-1437-5bdfd56978c5
Content: Create an Azure Red Hat OpenShift 4 private cluster
Content Source: articles/openshift/howto-create-private-cluster-4x.md
Service: azure-redhat-openshift
GitHub Login: @sakthi-vetrivel
Microsoft Alias: suvetriv

Thanks for taking time to share your valuable feedback! We are taking a look into this and will get back to you soon.

@sakthi-vetrivel Can you please check on this issue and add your comment accordingly.
Thanks

It looks like @paulczar pointed me to this: https://mobb.ninja/docs/quickstart-aro.html#adding-quota-to-aro-account which looks like how to fix the issue. It would be nice if this was in the official docs so future people don't get caught and have to search for an answer.

#reassign: @johnmarco

@jjasghar sorry for the long delay in replying to this issue for you. I was recently assigned to it and have taken care of it by adding a note to the procedure with the link you provided. The change should be live soon. I will now close this issue. Thanks.

#please-close


Distribute sea-orm-cli by Binstall

cargo-binstall is a project letting prebuilt binaries to be downloaded instead of compiling the code locally like cargo install does. This would save a lot of time.

Hey @erayerdin, thanks for the suggestion! Yeah, this would be a huge speedup :P

I'd like to take on this task, although it doesn't seem like you have a release strategy as of now (other than manually doing so). I think that some established workflow may be necessary to include sea-orm-cli in releases, since it's not there already. Would you like to establish some form of CD to include sea-orm-cli and other binaries with releases, or rather do it manually @billy1624? Maybe mirroring a release branch alongside the standard master branch?

Hey @tristan-zander, thank you for the interest!
As you have said, I think we can have a GitHub Actions (CD) to to include sea-orm-cli and other binaries when new release is published.
Just like what mdbook did

Releases: https://github.com/rust-lang/mdBook/releases/tag/v0.4.15.
Workflows: https://github.com/rust-lang/mdBook/blob/master/.github/workflows/deploy.yml

Oh gotcha, I didn't realize they had a hook for when you create a release. That works perfectly then. I'll try to get a PR going this weekend.

Hahaa! Thanks!! Looking forward to the PR


Add Replicate demo and API

Hey @rentainhe! 👋
Great work and fun project!
This pull request makes it possible to run RAM-Grounded-SAM & SAM-HQ Replicate (https://replicate.com/cjwbw/ram-grounded-sam) and via API:
import replicate
model = replicate.models.get('cjwbw/ram-grounded-sam')
model.predict(image=...)

We would also like to invite you to transfer the ownership to IDEA-Research so you could easily modify the model and push changes :)

Thank you so much for providing such a nice platform for us to show the demo! We're very likely to highlight this nice demo playground

We would also like to invite you to transfer the ownership to IDEA-Research so you could easily modify the model and push changes :)

I was wondering what should I do to update the demo~, maybe we can inform you if there're any updates of our demo~

Thank you for the feedback! I have now added GroundingDINO  the citation and updated the page under IDEA-Research.
To modify the page, simply sign-in to Replicate with your Github account here, and let me know - I will add you to the IDEA-Research organisation. When signed-in you can easily modify the README, example pages etc on the browser.
For further updating the model, here is the documentation of Cog that powers Replicate on how to push and modify models, but if it is complicated we can always help you to update and maintain the page!

Thank you for the feedback! I have now added GroundingDINO the citation and updated the page under IDEA-Research.
To modify the page, simply sign-in to Replicate with your Github account here, and let me know - I will add you to the IDEA-Research organisation. When signed-in you can easily modify the README, example pages etc on the browser.
For further updating the model, here is the documentation of Cog that powers Replicate on how to push and modify models, but if it is complicated we can always help you to update and maintain the page!

Got it! Thank you so much! I will send you en email after I registered Replicate~


Volca Sample with 200 samples?

The Korg Volca Sample is wonderful but 100 samples memory is a bit short...
The problem is that when you use the Volca Sample for the first time, you immediately have to delete some factory samples, if you want to upload user samples, because the memory is already full when you first use it!
Do you think it could be possible with an update to have :
S000 to S099 = factory samples
S100 to S199 = user samples
(With a firmware mod, or something else)

Unfortunately, the limitation is with the 4MB memory size, not the sample number so there's no way of avoiding the initial delete stage. BTW neat tool!

In terms of User Experience / product philosophy, asking the user to destroy/delete things, as the FIRST thing to do when unpacking a product, is not very nice  (in a philosophical point of view).
Why not have 8MB memory in the product and let 4MB free?
So the user is not "annoyed" by having to loose things at the start of the user experience (loosing things involves making choices => which sample to delete?  => this is hard  ;) )

Even limited to 4 MB it would be nice to not be arbitrarily limited to 100 samples. I replaced a lot of the samples and still am using far less than the 4 MB, yet I am hitting the 100 sample limit.

Even limited to 4 MB it would be nice to not be arbitrarily limited to 100 samples. I replaced a lot of the samples and still am using far less than the 4 MB, yet I am hitting the 100 sample limit.


choosing settings from the system menu no longer works

Navigating to app settings from the main menu (instead of the side tabs) is broken.
Needs to update by loading content rather than page load

working as expected


Changed to dark mode by default.

The main downside is the initial page stays light mode but it transitions well, I will probably make a different version that chooses based on system preferences. I would add a button to toggle between the too but docify seems to do alot of html generation and I'm afraid it would mess with it. If I have the time I might try to implement it.

I was compelled to do this after being blinded while studying for the psoft final at night and going in to the source to edit the settings.


How to create transaction. Please help

[root@bitcoin src]# ./omnicore-cli omni_send "mx7r52d2oaNTeiMDr2zrcw2VGm1BQurC9K" "n4YZv8eV8VTXW2ae2uZhnb64KdK3jS4GCx" 2 0.2
error code: -32601
error message:
Method not found
This method is not work.
https://omniexplorer.info/address/1ExXugzdTNKSHfEF4RBx8by915h5VPEE4S
i have 20 TetherUS
how can transfer is to other address?
Please halp me

@hakerkiler  please double check your bitcoin.conf to make sure the omnicore-cli command is actually connecting to an omnicored client.
Omnicore-cli is just like bitcoin-cli it does not do anything itself and instead sends the commands to the backend daemon (as specified in the bitcoin.conf file) .
If it is using the normal bitcoin.conf for a bitcoind daemon running on that machine it will not have access to the omni protocol. It must connect to an running omnicored instance to enable the Omni protocol commands.


Version bump to latest d.py

Requires rewrite of cog load and bot start, since they're async now

might convert to slash commands ion know


Can I use a proxy? Thank you

Can I use a proxy? Thank you
tdesk = await client.ToTDesktop(flag=UseCurrentSession)

// like
client = TelegramClient('sessions.session',111,'c1111111119',proxy=('socks5','<IP_ADDRESS>',8080))

you cant convert to Tdesktop

you cant convert to Tdesktop

ths..very much

what im joking?

what im joking?

i mean
desk.ToTelethon how to use  proxy ?

what im joking?

i mean
desk.ToTelethon how to use  proxy ?

yes document say yes

from opentele.tl import TelegramClient

client = TelegramClient(
            "name.session"
            proxy={
                "proxy_type": "socks5",
                "addr": "ip,
                "port": "port",
                "username": "username",
                "password": "password",
                "rdns": True
            }
        )


Why the default animation of the ImageView is BrightnessSaturationFade?

Why the default animation of the ImageView is BrightnessSaturationFade?

Because it's recommended by Google. You can overwrite the style if you wish

tks


🚀 Feature: Ability to re-abroad the administrator

Feature description
Ability to reset administrator passkey without wiping all users / clients.
Pitch
Synopsis:
Password managers (Bitwarden) are not capable of moving passkeys to another URL
Situation:
I bought new domain and would like to migrate my Pocket ID instance.
Issue:
I can't login from a new URL, because passkeys are hard-tied to previous URL
Same situation will hapen if I somehow lose a passkey (or reverse situation - if I delete all of them from Pocket ID UI)
Easy solution to this will be ability to reset administrator passkey without wiping all users / clients. Is there any way I can do this right now?

I have a Yubikey 5 which also handles passkeys.  From there, I have it tied to Pocket-ID as well as Bitwarden.  One is Yubikey as the backup and the other is the primary.   So, with one, I could delete a Bitwarden passkey easily.  Then re-tie to the new domain.
You could also create a new user only known to your's and then promote them to admin, then use the one time link to enroll your passkey, edit your original account to remove the enrollment and then when done, delete that account you created.
You may also be able to edit the database where you stored it and look under the 'webuathn_credentials' and remove the passkey entry.  I use "sqlitebrowse" which allows you to view it like a spreadsheet of sorts and modify as needed.   It would look similar to this.  (Blurred out just for my peace of mind)

I have NOT tested this method, but maybe @stonith404  can chime in on this.
The userID Not shown the screenshot will match your username under the 'users' table.
I know you could also do this if you are handy with sqlite table modifications via command line by logging into the container using /bin/ash as your shell to do so.   (I'm more familiar with MYSQL so not as well versed with sqlite).
I hope this helps with your immediate need at least, while the feature request is evaluated. :)

Thank you for your response! In my case, I figured out I could create one-time link for myself on old domain, then switch it, login with that link it on another domain - without secondary user.
I think the feature, on another hand, is still need to be done - we need some kind of recovery, especially because migrating passkeys between managers is not a thing (yet).
By the way, it's a good oportunity to say how much value your work weight. It's so good and simple - the thing was a blast for me, and it inevitable will be for the others.

elias@my-server:~/docker/pocket-id$ docker compose exec pocket-id sh ./scripts/create-one-time-access-token.sh

Usage: ./scripts/create-one-time-access-token.sh [-d <database_path>] <username or email>
  -d   Specify the database path (optional, defaults to ./backend/data/pocket-id.db)

elias@my-server:~/docker/pocket-id$ docker compose exec pocket-id sh ./scripts/create-one-time-access-token.sh elias

A one-time access token valid for 1 hour has been created for "elias".
Use the following URL to sign in once: https://<your-pocket-id-domain>/login/QXZYlNQYpRq8MTUW

What do you think about this solution? I've added a script that allows you to create a one time access token over the CLI for a specific user.

It seems good! It's not complicated at all, and I like really like it

Great thanks for the feedback. This script has been added inv0.10.0. I'll updated the docs ASAP.


feat: remove diff api

Remove diff api
Description
The diff-api calls are being removed from the lsp, this PR removes those and some un-used code related to the diffs, openFileSlice and warningSlice.
After this role="diff" messages become read only
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (no functional changes, only code improvements)
[ ] Documentation update

How to Test

Find a chat with a diff message.
Open it, there shouldn't be any apply buttons

Screenshots (if applicable)
Checklist

[ ] My code follows the code style of this project.
[ ] I have performed a self-review of my code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.
[ ] I have updated the documentation where necessary.

Linked Issues
Additional Notes
Publish after this PR is merged https://github.com/smallcloudai/refact-vscode/pull/147

Cannot launch a build in vscode of this branch, try to merge alpha inside, because I'm getting an issue with DiffAppliedStateResponse type not exported from refact-chat-js/dist/events


Small UI issues

I updated to the latest version, 1.9.0, and encountered a number of small, yet annoying, UI issues.

Breaking numbers across rows looks bad

Displaying a single digit from a number looks even worse (should have a minimum displayed chars after which font is reduced or text is removed)

You have same size rectangles for which some display text and others do not (even with same text)

Is the 'special' group rectangle which held 'the rest' of the smaller items still available/configurable?
The current situation makes the tree-map unusable with larger number of rectangles (they get too small to read the text, or even properly click them)

Is there a way to configure the "Drawing visualization" text to not be displayed for certain actions?
I have a custom element that toggles the tree-map text. It looks much cooler before when it  just flicked and the new text appeared than having 10-15% of the map covered with the message for 1-2 seconds.

Would you mind splitting this list out into separate issues?
The special group can be turned on using the "threshold" variable of .size( ): https://github.com/alexandersimoes/d3plus/wiki/Visualizations#size
The "Drawing visualization" text can be modified using the .messages( ) method: https://github.com/alexandersimoes/d3plus/wiki/Visualizations#messages

Done.


DOC add parameters, return, and example to docstring

"""Bernoulli numbers B0..Bn (inclusive).

    Parameters
    ----------
    n : int
        Indicated the number of terms in the Bernoulli series to generate

    Returns
    -------
    ndarray
        The Bernoulli numbers [B(0), B(1), ..., B(n)].

    References
    ----------
    .. [1] Zhang, Shanjie and Jin, Jianming. "Computation of Special
           Functions", John Wiley and Sons, 1996.
           https://people.sc.fsu.edu/~jburkardt/f_src/special_functions/special_functions.html

    Examples
    --------
    >>> from scipy.special import bernoulli, zeta
    >>> import numpy as np
    >>> bernoulli(4)
    array([ 1.        , -0.5       ,  0.16666667,  0.        , -0.03333333])

    The wikipedia article points out the relationship between the
      Bernoulli numbers and the zeta function, B_n^+ = -n * zeta(1 - n)
      for n > 0:

    >>> n = np.arange(1, 5)
    >>> -n * zeta(1 - n)
    array([ 0.5       ,  0.16666667, -0.        , -0.03333333])

    Note that, in the notation used in the wikipedia article,
      `bernoulli` computes B_n^- (i.e. it used the convention that
      B_1 is -1/2).  The relation given above is for B_n^+, so the
      sign of 0.5 does not match the output of `bernoulli(4)`.

    """

Reference issue
 Contributes to #7168
What does this implement/fix?
 Added description for n parameter, description for return parameter, example
Additional information

Thank you @rgommers!


Adding the new tests to observable bundle.

Adding the tests from kubernetes-core.

Nice 👍  LGTM.  Porting the dis-wait is nice to have here and will make our tests much more reliable


Don't publish program migrations if the app doesn't use programs

Probably need a "setup" script to set the configs and publish the correct migrations

Maybe split add_columns_to_users migration in half to more easily publish the correct columns.

fixed in #11


Create the menu window

Currently, there is no menu window.  Make the menu window and add the corresponding paths.

Hey @jazellemaira , Could you please assign me this issue.
I want to work on this.

Hi @ritik2358, this repo is not open for contributions right now. Thanks anyway!

okay,
tell me if u want any contribution in this.
I'll contribute


Add support for Objective-C Class Properties

related to https://github.com/NativeScript/NativeScript/issues/2456

Closed in https://github.com/NativeScript/ios-runtime/pull/635


Consider dropping support for template_files and doc_html

Description
A Kedro install is around 15M in site-packages, but only about 1M of that is even runnable code.  Much of it is the docs and template.  With the docs now deployed to readthedocs and kedro starters being released,  these two components are only adding bloat and should be considered for deprecation in favor of better solutions that are already in place.
Context
While 14M is not a lot of space, it only slows down installs and docker builds for unnecessary components.
Possible Implementation
Remove  template_files + doc_html_files from setup.py

This is actually a good point @WaylonWalker! I'll note it down for discussion. We did do a check a while ago of who was using kedro docs and a few legacy users still relied on it. I might just check in to see if it would be okay if we deprecated it.

Thanks for considering @yetudada.

Have you found anyone utilizing the docs command?  It would be great to see kedro a bit lighter.

Hi @WaylonWalker, we suspect we still have users that are using the kedro docs command. We're implementing product usage analytics which will show how many users still need this. Stay tuned until we know whether we can drop the support for template_files and doc_html


edit-readme button is misplaced, when readme contains no headings

Screenshot:

I think the edit button with the pencil icon should be in the top right.
Example pages where the bug manifests:

https://github.com/theam/aws-lambda-haskell-runtime
https://github.com/ThomasTerp/LunkyBox

Example where everything is fine, despite the document not starting with a heading: https://github.com/scottrogowski/mongita  (Presumably OK because the document contains headings later on.)
Reproduces in FF and Chrome. Extension version 21.4.13.

Probably related to https://github.com/sindresorhus/refined-github/issues/4157.

In the no TOC variant page .Box-title is now wrapped in a flexbox, which breaks edit-readme.


Use Factory Pattern instead of new  [moved]

This is Issue 65 moved from a Google Code project.
Added by 2010-06-12T14:46:26.000Z by<EMAIL_ADDRESS>Please review that bug for more context and additional comments, but update this bug.
Original labels: Type-Enhancement, Priority-Low, Milestone-Release1.1
Original description
It would be nice if instances of mapped classes would be constructed using a factory.

This would allow the use of dependency injection frameworks like Spring or Guice with morphia.

Maybe MapperOptions could hold that factory, which by default just uses ReflectionUtils.createInstance() but can be overridden if necessary.

The factory could also take care of constructor parameters (Issue 26)

This feature already exits:  org.mongodb.morphia.mapping.MapperOptions#getObjectFactory
The guice support makes use of this feature.


Pin to use specific version of official AWS tf modules

The fact that the version of the official modules are not fixed by default is now an issue.
For instance, for the /modules/boundary, the terraform-aws-modules/autoscaling/aws module version isn't fixed and so the following error is displayed :
Error: Unsupported argument

    on modules/aws-boundary/modules/boundary/main.tf line 47, in module "autoscaling":
    47:   iam_instance_profile        = var.iam_instance_profile

An argument named "iam_instance_profile" is not expected here.

Error: Unsupported argument

    on modules/aws-boundary/modules/boundary/main.tf line 54, in module "autoscaling":
    54:   recreate_asg_when_lc_changes = true

An argument named "recreate_asg_when_lc_changes" is not expected here.

Same issue with the module "alb", module "postgresql" and module "vpc".
Do you have in mind the versions that has been initially used so that we can pin them.
Next step update the code to call the latest version of these official modules.

@ptavares if you want to follow this one :)

@alexis-renard see #7 for the proposed changes


Dark 01

[1240] - Q2NKxXPh

[1240] - djcQ8UVR

[1240] - JlFQQ1mZ

[1240] - Allure report on Netlify: https://5ede7aab23c7447ab2299715--jdi-dark.netlify.app

[1249] - FfoQr5ee

[1249] - LO70wMIH

[1249] - OaXP9xeC

[1249] - OaXP9xeC

[1249] - Allure report on Netlify: https://5edf6a4a585083021c186000--jdi-dark.netlify.app

[1250] - KC2N6V96

[1250] - veZJYfLQ

[1250] - nAEjw4iv

[1250] - Allure report on Netlify: https://5edf74ba19e09384c17abdd8--jdi-dark.netlify.app


ModuleNotFoundError: No module named 'torch.ao'

Hi,
I installed the conda env according to the readme file. But I got this error msg "ModuleNotFoundError: No module named 'torch.ao'" when I try to run training code on toy_desk_2 demo.
Does anyone have the same problem?
Thanks

Hi hengfei-wang,
I'm sorry for the running issue.
Can you share your conda environment file?
Here is my conda environment: https://github.com/zju3dv/object_nerf/issues/11#issuecomment-1193954792.

Hi,
I rebuilt env according to you env file. And I removed readline and pillow's version request because conda reported there is a conflict in these modules' version. But problem remains and error msg is same.
This is my env file: my_env.yaml.txt

Well, I guess I have solved the problem. It's because the mismatch between pytorch and pytorch-lightning. I installed newest version of pytorch using conda. After that, I installed pytorch-lightning using conda, but it installed very old version by default. So I upgraded pytorch-lightning to 1.5.4 by myself even though it takes several mins for conda to parse the environment.


Remove None reviewers from validation

Fixes #4020
This was caused by an issue where the "---" (None) reviewers were causing the validation to fail, meaning that you had to fill out n-1 reviewers in the submission page in order for the save to go through.
The solution was to only check for duplicates with reviewers that were selected.

This has been updated with a second commit relating to the batch reviewer change code.  Let me know if you want me to squash them!

works great! thanks @frankduncan!


Example code at pub.dev doesn't compile

In the example code at pub.dev, the following line does not compile:
supportedLocales: FormBuilderLocalizations.delegate.supportedLocales,

I get The getter 'supportedLocales' isn't defined for the type 'LocalizationsDelegate<FormBuilderLocalizationsImpl>'.

Solved in 7.8.0


Add deliberately minimal readme.md

Obviously this requires more detail,
but a quick readme is prettier than none.

Shouldn't the filename be shouty? i.e. README.md?

Good point about the filename. Done.


TileSet zooming/positioning of one spritesheet changes zooming/positioning in others in all TileMaps

Tested versions
v4.2.1.stable.official [b09f793f5]
System information
Linux Kubuntu 23.04 KDE Plasma Version: 5.27.4 KDE Frameworks Version: 5.104.0 Qt Version: 5.15.8
Issue description
When editing TileSets, zooming/positioning a spritesheet changes the other ones in all TileMaps to the same level. It is an annoyance and impediment especially dealig with different sized sheets to have to go back reposition and rezoom. Each tilesheet in a tilemap should keep its zoom level and position. see issue recording below
With one TileMap
https://github.com/godotengine/godot/assets/930478/e4e0dee1-8f95-4cc2-ba0e-610a67f43942
2 TileMaps
https://github.com/godotengine/godot/assets/930478/2cfee809-14ed-4fa4-81b3-9a771391b8ba
Steps to reproduce
see screen recording
Minimal reproduction project (MRP)
see screen recording

I agree each TileSet should have its own zoom level, although this may prove to be an annoyance if we don't default to a suitable zoom level when opening new TileSets for the first time;

Well currently it opens up at 100%, center top. I don't see why that needs to change, but I guess it could be saved as part of the tileset.


[fix-#13999] add some dependent version

Purpose of the pull request
add some dependent version
this close #13999
Brief change log
add some dependent version
Verify this pull request

@zhuangchong

Hello @fuchanghai , thanks for your contribution, but version 3.0.x is prepare to end of life, and I check missing version in bom module will not throw error because they may have version in these module itself, such as https://github.com/apache/dolphinscheduler/blob/dc3c9651342529fcfde47dd0949a0ced18ce9e4a/dolphinscheduler-data-quality/pom.xml#L32, So maybe this pr will not accept for version 3.0.x

hi @zhongjiajie
I think the bom module should have the function of version management, maybe we should introduce the version of bom in other modules

hi @zhongjiajie If there is a next version of 3.0.x.
I think the bom module should have the function of version management, maybe we should introduce the version of bom in other modules. if 3.0.x ha

do you mean all module versions should be dependent on dolphinscheduler-bom? yeah, we should do it, but should not in bugfix version, should be correct the dependent in dev version

do you mean all module versions should be dependent on dolphinscheduler-bom? yeah, we should do it, but should not in bugfix version, should be correct the dependent in dev version

Agreed with @zhongjiajie. Closing this PR.


Missing scalar JSON. Defaulting to String

hello everyone,
I after spending 2 days on this, and trying every solution I found out there, couldn't resolve this issue, I would appreciate if you can help me out with this.
using dep:
ferry_generator: ^0.8.0+1
json_serializable: ^6.5.4
freezed: ^2.2.0
graphql_codegen: ^0.12.0
my query:
scalar JSON

type Filters {
    categoryId: Int
    id: Int
    name: String
    type: String
    values: [JSON!]!
}

query getFilters {
    getFilters {
        Filters {
            type
            id
            categoryId
            name
            values # is a simple Map<String, dynamic>
        }
    }
}

build.yaml
targets:
  $default:
    builders:
      gql_build|schema_builder:
        enabled: true
        options:
          type_overrides:
            JSON:
              name: JsonObject
              import: "package:built_value/json_object.dart"
      gql_build|ast_builder:
        enabled: true
      gql_build|data_builder:
        enabled: true
        options:
          schema: startup_namer|lib/graphql-schema/schema/schema.graphql
          type_overrides:
            JSON:
              name: JsonObject
              import: "package:built_value/json_object.dart"
      gql_build|var_builder:
        enabled: true
        options:
          schema: startup_namer|lib/graphql-schema/schema/schema.graphql
          type_overrides:
            JSON:
              name: JsonObject
              import: "package:built_value/json_object.dart"
      gql_build|serializer_builder:
        enabled: true
        options:
          schema: startup_namer|lib/graphql-schema/schema/schema.graphql
          custom_serializers:
            - import: "package:built_value/src/json_object_serializer.dart"
              name: JsonObjectSerializer

      ferry_generator|req_builder:
        enabled: true
        options:
          schema: startup_namer|lib/graphql-schema/schema/schema.graphql
          type_overrides:
            JSON:
              name: JsonObject
              import: "package:built_value/json_object.dart"

error occurs:
Missing scalar JSON. Defaulting to String
I also tried it with this:
build.yaml
targets:
  $default:
    builders:
      ferry_generator|serializer_builder:
        enabled: true
        options:
          schema: startup_namer|lib/graphql-schema/schema/schema.graphql
          custom_serializers:
            - import: 'package:startup_namer/json_serializer.dart'
              name: JsonTypeSerializer

      ferry_generator|graphql_builder:
        enabled: true
        options:
          schema: startup_namer|lib/graphql-schema/schema/schema.graphql
          type_overrides:
            JSON:
              name: Map<String, dynamic>

and json_serializer.dart
import "package:built_value/serializer.dart";
import "package:gql/language.dart";
import "package:gql_exec/gql_exec.dart";
import 'package:built_collection/built_collection.dart';
import "package:gql_code_builder/src/serializers/json_serializer.dart";

class JsonTypeSerializer implements PrimitiveSerializer<BuiltMap<String, dynamic>> {
  @override
  BuiltMap<String, dynamic> deserialize(
      Serializers serializers,
      Object serialized, {
        FullType specifiedType = FullType.unspecified,
      }) {
    assert(serialized is String,
    "JsonTypeSerializer expected 'String' but got ${serialized.runtimeType}");
    return BuiltMap<String, dynamic>.from(serialized as Map);
  }

  @override
  Object serialize(
      Serializers serializers,
      BuiltMap<String, dynamic> abdel, {
        FullType specifiedType = FullType.unspecified,
      }) =>
      abdel.toString();

  @override
  Iterable<Type> get types => [Map];

  @override
  String get wireName => "Abdel";
}

// class BuiltMapJsonSerializer extends JsonSerializer<BuiltMap<String, dynamic>> {
//   @override
//   BuiltMap<String, dynamic> fromJson(Map<String, dynamic> json) {
//     print('MyJsonSerializer fromJson: $json');
//     return BuiltMap.of(json);
//   }
//
//   @override
//   Map<String, dynamic> toJson(BuiltMap<String, dynamic> operation) {
//     print('MyJsonSerializer toJson: ${operation.toString()}');
//     return operation.asMap();
//   }
// }

class OperationSerializer extends JsonSerializer<Operation> {
  @override
  Operation fromJson(Map<String, dynamic> json) => Operation(
    document: parseString(json["document"] as String),
    operationName: json["Abdel"] as String,
  );

  @override
  Map<String, dynamic> toJson(Operation operation) => <String, dynamic>{
    "document": printNode(operation.document),
    if (operation.operationName != null)
      "operationName": operation.operationName,
  };
}

and got the same issue

your first example uses the old build.yaml syntax.
the latter one is correct, but is missing the type_overrides in the serializer_builder.
another user managed to get this working here: https://github.com/gql-dart/ferry/issues/493#issuecomment-1443555107

update, I also tried that solution
here is my current build.yaml
targets:
  $default:
    builders:
      ferry_generator|graphql_builder:
        enabled: true
        options:
          schema: startup_namer|lib/graphql-schema/schema/schema.graphql
          when_extensions:
            when: true
            maybeWhen: true
          type_overrides:
            JSON:
              name: Map<String, dynamic>
              import: 'package:metadata_serializer.dart'
      ferry_generator|serializer_builder:
        enabled: true
        options:
          schema: startup_namer|lib/graphql-schema/schema/schema.graphql
          when_extensions:
            when: true
            maybeWhen: true
          type_overrides:
            JSON:
              name: Map<String, dynamic>
              import: "package:metadata_serializer.dart"
          custom_serializers:
            - import: 'package:metadata_serializer.dart'
              name: GenericScalarSerializer

and metadata_serializer.dart
import 'package:built_value/serializer.dart';
import "package:gql_code_builder/src/serializers/json_serializer.dart";

class GenericScalarSerializer extends JsonSerializer<Map<String, dynamic>> {
  @override
  Iterable<Type> get types => [Map<String, dynamic>];

  @override
  String get wireName => 'JSON';

}

this didn't work

you don't need an import for Map<String,dynamic>

targets:
  $default:
    builders:
      ferry_generator|graphql_builder:
        enabled: true
        options:
          schema: startup_namer|lib/graphql-schema/schema/schema.graphql
          when_extensions:
            when: true
            maybeWhen: true
          type_overrides:
            JSON:
              name: Map<String, dynamic>
      ferry_generator|serializer_builder:
        enabled: true
        options:
          schema: startup_namer|lib/graphql-schema/schema/schema.graphql
          when_extensions:
            when: true
            maybeWhen: true
          custom_serializers:
            - import: 'package:metadata_serializer.dart'
              name: GenericScalarSerializer
          type_overrides:
            JSON:
              name: Map<String, dynamic>

:( same error

I don't understand what I'm doing wrong, I'm loosing it

@knaeckeKami any Idea how to overcome this issue?

Can you share a Minimal reproducible example?

@knaeckeKami here is a minimal working example that you can reproduce the error
https://github.com/LatVAlY/graphql-serrializer

The warning you see is from graphql_codegen, not from ferry.
graphql_codegen is a code generation library for the graphql and not compatible with ferry. you should not use graphql_codegen and ferry together.

omg!! I very much appreciate it!!!!


singleton realtime instance obtains multiple connections and confuses them

Which version of the Ably SDK are you using?
ably-cocoa 1.2.25
On which platform does the issue happen?
iOS 16.x, 17.x (on iPhone and iPad) and macOS 14.2.1 (using MacCatalyst)
Are you using Carthage?
No
Are you using Cocoapods?
No
Which version of Xcode are you using?
Xcode 15.2
Build version 15C500b
What did you do?
I obtain a single realtime client object and use it to open two channels, sending one packet on the first and two packets on the second.  This works some of the time.  However, on app startup, there is an intermittent failure in which the realtime client opens two connections to the Ably server rather than one, sends the packets on the first connection, and gets a mismatched connection id error response from the server that references the second connection.
NOTE: As a workaround, I have written my code so that it always sends a throwaway packet first thing after attaching a channel, and it looks for the mismatched connection ID error to come back from that first packet.  If it gets that error, it totally tears down and closes the client, and then goes through the complete client creation/attach sequence again.
I’ve attached a debug log of a typical failure case.  In this case the failure happened not once but twice, causing the client to tear down the connection a second time and then build it back up.  On the third time, there is no second connection opened to the Ably server and so there is no connection mismatch error thrown and things proceed as expected. I can show you logs where there is never a second connection opened at startup, logs where a second connection is only made once on the first startup and not thereafter, and logs like this one where the second connection is made on two successive startups but not thereafter. Once I get a clean single-connection startup, I can tear down and rebuild connections as often as I want and the error never recurs.  It only happens on the first (or first two) connection attempts that come after application startup.
My code is open source on GitHub.  Most of the relevant code can be found in this class.  The three most relevant code snippets are:

The function that does the channel open/attach, and which notices the error on first packet, is tryOpenChannel.
For context, the function that calls tryOpenChannel is openChannels and it calls it twice: once to open the "content" channel, which is named by a UUID, and once to open the "control" channel (named with "control").  Once the control channel is opened a non-throwaway packet is sent to announce this client's presence to others.
Authentication is done in the TcpAuthenticator class, which obtains a signed AuthTokenRequest from my server after authenticating with a JWT signed by a private secret issued by my server via APNS.

What did you expect to happen?
The expected outcome is that, once authentication is complete, only one channel to the server would be opened, and all packets would be sent on that.
What happened instead?
What happens in the failure case is that two different connections are made by the realtime client to the server, messages are sent based on the first connection opened, then the second connection is opened, then the initial set of messages are resent, and at that point the server rejects them because it’s expecting them to be sent using the second connection ID.  In the attached log, for example, the id of the first connection is _29fvQ8ykQ (line 192 of the log) and the id of the second connection is CcI3ilmBdc (line 261 of the log).
Additional
I was asked to open this bug by Mike Clark of the support team.  Developer evangelist Cameron Michie is familiar with my application and has access to a running beta.
┆Issue is synchronized with this Jira Task by Unito

Thanks @clickonetwo  We're investigating this, in case of urgency please use v1.2.24 by updating your Podfile:
pod 'Ably', '1.2.24'

Hi @maratal thanks for the suggestion. In fact I first observed this problem on 1.2.24 and only recently updated it hopes that 1.2.25 would have a fix.  So I know this problem exists in 1.2.24 as well.

@clickonetwo thanks! This changes a lot (still investigating though).

@clickonetwo couldn't you try the branch for the fix? Thanks!

@maratal Thanks for the quick work - I've switched my dependency to the fix/1882-fix-reachability-activation branch and I will start testing right away!

Quick update after initial testing: seems to have fixed the problem!  I've gone through about 15 app launches and initial connections and not seen the problem once.  That would never have happened before - it was about 1 out of every 5 launches.
I am about to release a build with this library to my beta testers.  That will get us close to 1000 launches in the next week or two.  I'll report back on what I hear.

Nice work! @clickonetwo

Hi @maratal just wanted to report that I've now had hundreds of sessions started against your branch library and there has been no trace of this problem.  Thanks for much for your quick work on this fix! Do you have any estimate for when your PR will be accepted into the main line and released?

Thanks @clickonetwo I will make a release today.

This is now released @clickonetwo

Hooray!  Thanks so much for the fast fix!
On Fri, Mar 8, 2024 at 8:38 AM Marat Al @.***> wrote:

This is now released
https://github.com/ably/ably-cocoa/releases/tag/1.2.26 @clickonetwo
https://github.com/clickonetwo
—
Reply to this email directly, view it on GitHub
https://github.com/ably/ably-cocoa/issues/1882#issuecomment-1986025493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AX7F4IKVQ3KPTBDQUCD6ZLTYXHSRHAVCNFSM6AAAAABEALXZGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBWGAZDKNBZGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Automate estimation of backlash

Estimation of backlash should be straightforward using optical tracking with a bright star. The motion of the star can be compared with the reported change in position of the mount. The result should be a pair of values in degrees for the azimuth and altitude directions.

There are a couple of approaches I can think of:

Slew the mount back and forth along one axis (either azimuth or elevation) by some number of degrees per slew. Decrease the total angle of the slew on each iteration until the position of the object in the camera frame does not move. Then increase the total angle by very small increments on each slew until the object starts to move again. This could be implemented as a sort of binary search. At the beginning of the search the slew rate could be fast but near the end it should be slow to minimize error introduced by the lag between sending slew commands and reading back the mount position.
Slew the mount and intentionally move the target within the camera's field of view by a particular amount. Compare the motion of the object to the mount's reported change in position. The problem I see with this approach is that the accuracy will depend on the accuracy of conversion between motion of the object in the field of view (pixels vertical and horizontal) to degrees in azimuth and elevation. For the elevation axis this is straightforward, but at higher elevations the azimuth axis is not parallel to the horizontal axis of the frame. Getting this coordinate transformation right is not impossible but it would be more robust to eliminate this as a source of error.

Elaborating on the first approach, do the following for each axis. This assumes that a bright object (a star or planet) has been placed within the camera's FOV.

Center the object in the FOV (with software).
Slew on the selected axis in the + direction. Do not slew on the other axis.
Stop the slew when movement of the object is detected. Record the position of the object in the frame and the Az/Alt position of the mount.
Slew in the - direction until movement is detected. Record the positions.
Take the difference in the Az/Alt positions reported by the mount at the end of the two initial slews. This is the upper bound on the backlash. The backlash must be less than this because the object moved in the FOV on the second slew.
(more steps need to be filled in here)

On detecting position accurately: Near the start of the algorithm, it may be prudent to estimate the RMS level of noise in the object's detected position over a significant number of frames. This information could be used to determine how far the object needs to move for detection of movement to be reliable, and/or it could be used to determine how many frames should be averaged for each position estimate.

The first round of testing revealed the following:

The binary search was basically useless. The search would eventually find a value nearly equal to the value found on the initial search. It seems more useful to perform the initial search procedure (originally thought of as the upper bound search) several times and take the average.
Default optical tracking loop parameters resulted in undamped oscillations, probably (and somewhat ironically) due to backlash. I had better results with a bandwidth of 0.1 Hz and a damping factor of 2.0.
The default loop period of 0.25 s was too short. I had better luck with 0.5 s.
Should add estimation of object velocity in FOV to do the following:

Subtract expected motion when detecting optical movement
Slew on the other axis to keep object in FOV

May need to do something to keep object in the FOV during a single iteration. In the case I was testing (azimuth axis only) the star was moving mostly in the elevation axis due to sidereal motion. The star moved nearly off the top of the frame by the time the iteration was complete.

The code now supports both azimuth and altitude axes. It now also continues to slew on the axis not under test at the same rate the tracking loop converged at in an effort to keep the object from drifting out of frame. I ran the script a few times this evening on a star but got wildly inconsistent results with the default parameters. I think the following refinements are necessary:

As noted in the previous comment, estimate object's apparent sidereal motion and compare observed positions to predicted positions accounting for this motion.
When detecting movement after the initial slew, this should be a comparison between the error before the slew is initiated rather than compared to zero error. Comparing with zero error assumes that the initial centering of the object was perfect. The default parameters set a wider bound on the initial centering than on the movement threshold, which sometimes causes the algorithm to "detect movement" even though the optical tube has not actually moved.
Perhaps run each set of iterations twice in each axis, once starting in the positive slew direction and again starting in the negative direction. The two sets should be consistent if the object's apparent motion is compensated properly.

Output of first complete run after algorithm overhaul. Getting more consistent results now.
backlash_estimation_output_2017_08_09.txt

While performing alignment testing I am seeing evidence that the first run significantly overestimated the backlash. Need to take a closer look at the results and figure out what went wrong.

Some specific problems:

Estimation of tracking slew rates is inconsistent between runs--perhaps dwell for a fixed time interval rather than relying on the TrackUntilConverged stopping condition
Camera-to-mount rate ratio is not close to 1 when object sidereal motion is small
Estimation of sidereal motion can take forever when rate is very slow, consider using a fixed time-based observation interval instead

The star-tracking approach to backlash estimation has proven to be complex and disappointing. I have discovered a new approach that appears to be much simpler and more reliable. Rather than tracking a star, which introduces complexities due to the sidereal motion of the sky, I can track a terrestrial object that has no relative motion. I had avoided this previously because the current computer vision algorithm in the OpticalErrorSource class can't handle anything but a bright small object against a dark background. Fortunately this can be worked around by replacing computer vision with human vision. To aid in the detection of motion in the frame I found a bit of sample code that uses OpenCV to track the optical flow of features. Along with this visual aid I wrote a simple Python script that simply prints out the mount's change in position continually in both axes. By slewing the mount with the hand controller and watching the paths of the features in the OpenCV window it is reasonably straightforward to get a reliable estimate of the backlash deadbands. This new approach is not fully automated but since this is an activity that should not be required on a recurring basis I think the manual component is justified. In fact, seeing the motion in the camera frame provides much more confidence in the results. I suspect it would be too much trouble to achieve the same level of reliability with a fully automated solution.

For the 130 SLT my estimates of backlash deadbands are 450 arcseconds in altitude and 1000 arcseconds in azimuth. The error in these measurements is at least +/- 50 arcseconds.


stages/selinux: directly call setfilecon

Instead of using chcon, directly call selinux.setfilecon. On systems without SELinux support, i.e. coreutils was built without <selinux.h> present, chcon will return ENOTSUP for all selinux related calls like setfilecon even if the selinux libraries are later installed. Therefore we directly call the library function which should ensure that we don't error out as long as the library is present.  The only other thing chcon is doing besides a cal to the setfilecon method is to convert the context string to a contex_t and back to validate it. This should not be needed since the kernel will do this for us. On system without SELinux support context_new will also not validate the context.

I tried re-running the LGTM CI, but Github is really confused so I think you have to re-force push :confused: @gicmo

Apparently we are missing python3-libselinux on the CI image.

I checked our 8.x images and we started labelling cp in 8.5 and I think in that images we have python3-libselinux in the build root everywhere; but since this technically a (breaking) behaviour change, I will change this to try to use selinux.setfilecon and if that doesn't work fall back to chcon.

Also locally verified that this is still doing the correct thing:
gicmo@snow ~/C/o/osbuild → ls -laZ /var/osbuild/store/refs/9590527d97c7cffe87595ff5a239b605258610e2cad0df8a7979e4f9c83d7e5b/bin/cp
-rwxr-xr-x. 1 root root system_u:object_r:install_exec_t:s0 149776 Mar 26  2021 /var/osbuild/store/refs/9590527d97c7cffe87595ff5a239b605258610e2cad0df8a7979e4f9c83d7e5b/bin/cp


Last line in quicktour notebook points to cygrid

Following the online quicktour notebook, there is a Note at the bottom, wich links to a cygrid notebook.

Good catch. I'll fix this.


Error with Enum, TypeError: Cannot read properties of undefined

Describe the bug
Hi, 👋
tsc transforms some declared enum into its value while swc looks like it does not.
I might have missed something in the configuration to make it work?
Input code
export declare const enum EnumA {
  A = "A",
  B = "B",
  C = "C",
}

export const f = (a: EnumA) => {
  console.log(a);
};

console.log("Hello World!!!");
f(EnumA.A);

Config
{
  "jsc": {
    "parser": {
      "syntax": "typescript",
      "tsx": false,
      "decorators": true,
      "dynamicImport": true
    },
    "target": "es2020",
    "baseUrl": "."
  },
  "module": {
    "type": "commonjs"
  }
}

Playground link
https://play.swc.rs/?version=1.3.41&code=H4sIAAAAAAAAA02MQQrCMBBF93OKn6xSkB7AUiEtgjdwHdqpm7GR2IIgvbuTZONmeMz%2F%2F%2FHnFdOGmScJiTHF9b2B1%2F2Jqx6PLwEePay3J8Uh41BwzDgqHkRcLXW9aODCuQoa9JciyVkUbiU%2BXGg6Ojqi%2F5%2B9sUjEPSaZjTFWK4sritYr%2FwCajC53qAAAAA%3D%3D&config=H4sIAAAAAAAAA1VPSQ7DIAy88wrkcw8Vx%2F6hj0DUiYjYhB2pKMrfCymkzc2exTPehJSwkIGH3OpYl6QzYT73ilAJrN8VAS4JyWSbGG6DZWrUpB3hAe1fBljnGbm5kNRdqe4AFyPhcHTM22Cn8p9pok8Zia7CJtVhdnhNFD0VfHytB9l%2FaX1bg3rNx7AQ%2FJQj8bwOlp7DznlFsX8AqE9BlhwBAAA%3D
Expected behavior
tsc output (with replaced constant and generated comment):
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
exports.f = void 0;
const f = (a) => {
    console.log(a);
};
exports.f = f;
console.log("Hello World!!!");
(0, exports.f)("A" /* EnumA.A */);

Actual behavior
"use strict";
Object.defineProperty(exports, "__esModule", {
    value: true
});
Object.defineProperty(exports, "f", {
    enumerable: true,
    get: ()=>f
});
const f = (a)=>{
    console.log(a);
};
console.log("Hello World!!!");
f(EnumA.A); // <- 💥 TypeError: Cannot read properties of undefined

Version
1.3.41
Additional context
I do not have control over how the enum is defined (external index.d.ts lib), I found a way to make swc work if I could change it.

Thank you @kwonoj 🙏
Working ts playground (set to commonjs).
The problem seems to be related to the declare, without it, it works. But it my case I cannot change it as it is exposed by some lib I am using.
So this problem seems not related to the const enum mentioned in the other issue.

The playground link you shared shows this

which should raise same error.

@kwonoj Executing it with node it works, I executed it with node:


[Performance] When markProblemsDirty, should only fetchAllProblems when get back to the problem list page.

[Performance]
Currently: when call markProblemsDirty, fetchAllProblems immediately invoked.
Expected: When call markProblemsDirty, should only fetchAllProblems when getting back to the problem list page.
So that we can reduce the number of API calls.

Since statemanagement mechanism is changed in #209
this issue is obsolete

Since statemanagement mechanism is changed in #228
this issue is obsolete


Need new diagnostic/prognostic models that work with VRS 2.0

@ahwagner we need new diagnostic/prognostic models that will work with VRS 2.0.

The following are for CIViC Evidence objects.
Therapeutic Response
Modify the VariantTherapeuticResponseStudy:

rename tumorType to disease

revise definition of disease to "The disease ~tumor type~ context in which..."

Diagnostic
Start with the modified VariantTherapeuticResponseStudy and make the following changes:

Class name is VariantDiagnosticStudy
Update description to: "A study summarization describing whether a variant is associated with a disease (positive) or lack of a disease (negative)."
Drop therapeutic property
Drop the disease property
Add a new disease property

extends object (the VA notion of object)
initial definition as "the disease that is evaluated for presence"

revise predicates to positive and negative

Prognostic
Start with the above VariantDiagnosticStudy and make the following changes:

Class name is VariantPrognosticStudy
Update description to: "A study summarization describing whether a variant is associated with a disease (positive) or lack of a disease (negative)."
Revise the disease property definition as "the disease that is evaluated for outcome"
revise predicates to Improved Outcome and Worse Outcome

NOTE: This is not consistent with the CIViC predicates, which are Better Outcome / Poor Outcome/ "N/A"

@korikuzma see requirements above. Let me know if you have any questions. Thank you for taking this on!

@ahwagner created a draft PR and had one question from your requirements: https://github.com/ga4gh/va-spec/pull/145/files#r1628473160

@ahwagner I also made other comments on #145


Can't debug my mod if loaded by Wetstone

First of all great job! Thank you!
If I load my mod without Wetstone (directly by BepInEx) with the Attach Unity Debugger option I'm able to debug my mod; while if I load my mod through Wetstone when I try to set a breakpoint I get an error (like code unreacheable...)
I also tried to manually specify the .pdb file inside the WetsonePlugins directory... but didn't helped.
Do you have any suggestion?
Thank you

Thank you for your kind reply @molenzwiebel .


build failed

I failed building wheel in ubuntu with the following env variables:
export PYTHON_VERSION=3.6
export TF_VERSION_GIT_TAG=v1.11.0
export USE_GPU=0

after about 3 hour building I get this error:
...
In file included from tensorflow/contrib/lite/kernels/slice.cc:21:0:
./tensorflow/contrib/lite/kernels/internal/optimized/optimized_ops.h: In function 'void tflite::optimized_ops::LogSoftmax(const uint8*, const tflite::RuntimeShape&, int32, int32, int32, int32, int, uint8*, const tflite::RuntimeShape&)':
./tensorflow/contrib/lite/kernels/internal/optimized/optimized_ops.h:4498:53: warning: typedef 'using FixedPoint0 = class gemmlowp::FixedPoint<int, 0>' locally defined but not used [-Wunused-local-typedefs]
   using FixedPoint0 = gemmlowp::FixedPoint<int32, 0>;
                                                     ^
In file included from ./tensorflow/contrib/lite/builtin_op_data.h:20:0,
                 from tensorflow/contrib/lite/kernels/slice.cc:19:
tensorflow/contrib/lite/kernels/slice.cc: In function 'TfLiteStatus tflite::ops::builtin::slice::Prepare(TfLiteContext*, TfLiteNode*)':
tensorflow/contrib/lite/kernels/slice.cc:115:48: warning: suggest parentheses around comparison in operand of '==' [-Wparentheses]
   TF_LITE_ENSURE(context, NumDimensions(begin) == NumDimensions(size) == 1);
                                                ^
./tensorflow/contrib/lite/context.h:119:11: note: in definition of macro 'TF_LITE_ENSURE'
     if (!(a)) {                                                             \
           ^
ERROR: /tensorflow/tensorflow/BUILD:592:1: Executing genrule //tensorflow:tensorflow_python_api_gen failed (Exit 1)
Traceback (most recent call last):
  File "/root/.cache/bazel/_bazel_root/68a62076e91007a7908bc42a32e4cff9/execroot/org_tensorflow/bazel-out/host/bin/tensorflow/create_tensorflow.python_api.runfiles/org_tensorflow/tensorflow/python/tools/api/generator/create_python_api.py", line 27, in <module>
    from tensorflow.python.tools.api.generator import doc_srcs
  File "/root/.cache/bazel/_bazel_root/68a62076e91007a7908bc42a32e4cff9/execroot/org_tensorflow/bazel-out/host/bin/tensorflow/create_tensorflow.python_api.runfiles/org_tensorflow/tensorflow/python/__init__.py", line 81, in <module>
    from tensorflow.python import keras
  File "/root/.cache/bazel/_bazel_root/68a62076e91007a7908bc42a32e4cff9/execroot/org_tensorflow/bazel-out/host/bin/tensorflow/create_tensorflow.python_api.runfiles/org_tensorflow/tensorflow/python/keras/__init__.py", line 25, in <module>
    from tensorflow.python.keras import applications
  File "/root/.cache/bazel/_bazel_root/68a62076e91007a7908bc42a32e4cff9/execroot/org_tensorflow/bazel-out/host/bin/tensorflow/create_tensorflow.python_api.runfiles/org_tensorflow/tensorflow/python/keras/applications/__init__.py", line 22, in <module>
    import keras_applications
ModuleNotFoundError: No module named 'keras_applications'
Target //tensorflow/tools/pip_package:build_pip_package failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 9175.670s, Critical Path: 94.40s
INFO: 5604 processes: 5604 local.
FAILED: Build did NOT complete successfully

At https://github.com/hadim/docker-tensorflow-builder/blob/master/tensorflow/ubuntu-16.04/build.sh#L15, you can try to add keras_applications or add a command to pip-install it of it's not in conda-forge.
If it works, a PR is welcome.

thanks, adding conda install -c conda-forge keras-applications to build.sh worked


Template items choose the same songs over and over

When using a QuickPlayer, the same songs are chosen over and over, rather than new songs. There are over 100 songs matching the template item, yet the same song is chosen multiple times, rather than another one.

Possibly related: #44


Withdraw/deposit Event API payload has member email

Needed for applogic consumers which doesn't have User model now

AppLogic should use Barong API to get user profile. Who requested this feature?

AppLogic should use Barong API to get user profile. Who requested this feature?
You are right, will try suggested approach


Use ES for Index and torrent pages

Index use ES and torrent pages should keep PG for more recent changes. I think we can close this @ewhal


Helm chart: allow unprivileged deployment of Beyla

Closes #1075
This PR allows securityContext.privileged to be set to false which adds the required configurations to the DaemonSet (based on the unprivileged example).
If required or more logical from a flow perspective, this can also be changed to a top level privileged property rather than using the existing property in securityContext.
Chart version is increased to 1.4.0 and app version to 1.8.0 - there could possibly be more changes required to the chart.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 52.33%. Comparing base (ecc9dc1) to head (dd4bd7f).

:exclamation:  There is a different number of reports uploaded between BASE (ecc9dc1) and HEAD (dd4bd7f). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (ecc9dc1)
HEAD (dd4bd7f)

oats-test
1
0

k8s-integration-test
1
0

integration-test
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1128       +/-   ##
===========================================
- Coverage   73.06%   52.33%   -20.73%
===========================================
  Files         139      140        +1
  Lines       11553    11562        +9
===========================================
- Hits         8441     6051     -2390
- Misses       2454     5106     +2652
+ Partials      658      405      -253

Flag
Coverage Δ

integration-test
?

k8s-integration-test
?

oats-test
?

unittests
52.33% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Create diagram of framework

Description
The docs website visitor wants to get an overview of how the open-sourced packages relate to each other.
It was suggested to use mermaid to create the diagrams.

[x] Create a basic framework diagram with mermaid
[ ] Assess/test integration of mermaid into docs repo (consult with @rouille and @jon-hagg )

[x] Add mermaid to sphinx with the sphinxcontrib.mermaid extension
[ ] Add dependencies to dockerfile so that the github action can build docs automatically
[x] Document requirements for local use

[ ] Create PR for mermaid integration into repo
[ ] Iterate on package relation diagram(s) and create PR

Acceptance criteria

Mermaid integrated into docs repo
Diagram published on the docs website

Owner
@ahurli
Support
@rouille
@jon-hagg

mermaid
https://mermaid-js.github.io/mermaid/#/
Installation
The  cerf readme has instructions for how to install mermaid:

markdown linting
I'm using:
"flake8-rst-docstrings"
My sphinx conf uses "sphinx.ext.napoleon" to convert google style docstrings to rst. What's interesting is that I can still use RST if I want to, and I do this sort of thing for inserting latex for rendering formulae:

I'm having an issue with the docker build where I'm getting following error when tox is run:
docs_1  | Could not import extension sphinxcontrib.mermaid (exception: cannot import name 'ENOENT' from 'sphinx.util.osutil' (/docs/.tox/prereise/lib/python3.7/site-packages/sphinx/util/osutil.py))
As far as I can tell, everything loads/installs correctly, but there may be some versioning issues.

Not sure to understand what is the issue. I have just checked out anna/mermaid-diagrams and inserted the following:
.. mermaid::

   sequenceDiagram
      participant Alice
      participant Bob
      Alice->John: Hello John, how are you?
      loop Healthcheck
          John->John: Fight against hypochondria
      end
      Note right of John: Rational thoughts <br/>prevail...
      John-->Alice: Great!
      John->Bob: How about you?
      Bob-->John: Jolly good!

which is the example provided here, into the index.rst file. Then I run tox and get the following without ant error/warning:

@rouille we needed to use docker initially since there were build errors when running tox directly on github's runners. Not sure if that's still the case, but it's nice to have the build self contained so anyone can run it quickly. It is faster to use tox locally (outside docker), so I'd like to keep both options if possible.

@ahurli I think this is the issue we're seeing https://github.com/mgaitan/sphinxcontrib-mermaid/issues/72. Not sure why it only happens in docker, but I got it to work by adding the code snippet suggested there in conf.py

@ahurli I think this is the issue we're seeing mgaitan/sphinxcontrib-mermaid#72. Not sure why it only happens in docker, but I got it to work by adding the code snippet suggested there in conf.py

I confirmed that adding:
import errno
import sphinx.util.osutil
sphinx.util.osutil.ENOENT = errno.ENOENT

to the conf.py does the job.

Awesome! Thank you @jon-hagg


docs: update docs around email reports

SUMMARY
I noticed while debugging with this flag that you have to run superset init in order for the menu/pages to appear. Some people have apparently gotten bit by something similar in the past, and it takes a while to figure out if you don't know what you're looking for, so I added it to the docs.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11783 (b91a668) into master (cdd01f4) will decrease coverage by 8.62%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11783      +/-   ##
==========================================
- Coverage   63.14%   54.52%   -8.63%
==========================================
  Files         897      411     -486
  Lines       43460    14521   -28939
  Branches     4188     3725     -463
==========================================
- Hits        27443     7917   -19526
+ Misses      15837     6604    -9233
+ Partials      180        0     -180

Flag
Coverage Δ

cypress
54.52% <ø> (?)

javascript
?

python
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

...src/dashboard/components/gridComponents/Header.jsx
10.52% <0.00%> (-86.85%)
:arrow_down:

...rc/dashboard/components/gridComponents/Divider.jsx
13.33% <0.00%> (-86.67%)
:arrow_down:

...ontend/src/dashboard/util/getDashboardFilterKey.ts
14.28% <0.00%> (-85.72%)
:arrow_down:

...set-frontend/src/views/CRUD/welcome/EmptyState.tsx
5.71% <0.00%> (-82.10%)
:arrow_down:

...et-frontend/src/components/ListView/ActionsBar.tsx
11.11% <0.00%> (-81.20%)
:arrow_down:

...et-frontend/src/components/Menu/LanguagePicker.tsx
0.00% <0.00%> (-80.00%)
:arrow_down:

... and 808 more

Continue to review full report at Codecov.

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


docs: the ' is noneed

it will be error 'configCmd'.AddCommand(createCmd)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Implement automatic creation of issues in case plugin crashes

Implemented in that commit.
Currently, the user is able to send a report with staketrace from IDE in case plugin crashes. The reports will collected in a private JB Exception Analyzer service.


Allow local archetypes in interface types of local declarations

This cleans up some logic for variables inside pack iteration for loops in preparation for variables whose types contain element archetypes becoming a more general thing.

@swift-ci Please smoke test

@swift-ci Please test source compatibility


CSS Refactoring and removing !important rules

Trello card
None specifically, but this will fix https://trello.com/c/zJnK0DV7/443-fix-usage-of-important-in-css
Releated content changes
DFE-Digital/get-into-teaching-content/pull/138
Context
The !important directive referenced in the above Trello ticket couldn't easily be removed and replaced by more-specific selectors. Some additional work is needed to split out the repeated functionality.
Changes proposed in this pull request
The bulk of this PR is the replacement of copied CSS rules with composable mixins, and adjusting the HTML structure to match. Additionally, the addition of the 'external link' CSS has been moved out to the markdown.scss file where over time more of the content-specific rules can live. All Markdown-rendering layouts now apply this class to the markdown's container.
Guidance to review
It's quite a big one, probably best to run it and look through the affected pages and make sure nothing looks awful!

Looks good, just spotted a few differences when comparing to the current version on dev:

@ethax-ross - I added some additional CSS rules to ensure those links are $black. I'll tweak the steps styles in #563


Update node version for developing

The required version has been updated in the package.json, but not in the README.md

Coverage remained the same at 83.046% when pulling e2debb22f5308b9872b01c822d1864a20de799f1 on Otto-AA:patch-1 into 716db5e4631f4aab30ee62d21dc0443185242f7a on solid:master.


Maybe the shader code memory isn't being freed properly?

I've found a couple of memory leaks reported by the debugger and it seems that this line may be related:
vulkantools.cpp: shader_code = malloc(size);
at vkTools::readBinaryFile(const char *filename, size_t *psize)

A quick fix would be to add the following line:
free(const_cast<char*>(shaderCode));
Before the return statement of
VkShaderModule loadShader(const char *fileName, VkDevice device, VkShaderStageFlagBits stage)
Not very nice if you ask me, as the allocation is done somewhere else, but it certainly reduces the number of leaks.


gitlab: 14.7.2 -> 14.7.3

Motivation for this change
https://about.gitlab.com/releases/2022/02/14/gitlab-14-7-3-released/
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)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

The tests pass and it works in manual testing. Thanks!


Request comments on iOS versioning in README

Current coverage is 89.14%

Merging #591 into master will not affect coverage as of 0e8aa01

@@            master    #591   diff @@
======================================
  Files            2       2
  Stmts         1087    1087
  Branches         0       0
  Methods
======================================
  Hit            969     969
  Partial          0       0
  Missed         118     118

Review entire Coverage Diff as of 0e8aa01

Powered by Codecov. Updated on successful CI builds.


DateTime.tryParse parses invalid dates, substituting a valid date

DateTime.tryParse("2022-02-34")
DateTime (2022-03-06 00:00:00.000)

The input string was invalid.  The correct value to return is null

Dart core libraries ("dart:async", "dart:io", etc.)

This the correct place?
% dart --version
Dart SDK version: 2.18.1 (stable) (Tue Sep 13 11:42:55 2022 +0200) on "macos_arm64"

parse method - DateTime class - dart:core library - Dart API says :

This method accepts out-of-range component values and interprets them as overflows into the next larger component. For example, "2020-01-42" will be parsed as 2020-02-11, because the last valid date in that month is 2020-01-31, so 42 days is interpreted as 31 days of that month plus 11 days into the next month.

See https://github.com/dart-lang/sdk/issues/11189
We decided at some point to not try to be strict here.

What a terrible idea.
There is no valid output for invalid input


Unable to configure the provider for multiple IRSA roles

What happened?
I want to use different IAM Roles (IRSA), but I'm unable to do it.
As per docs, IRSA roles must be configured as an annotation in the controllerConfig CR.
There is no relation between providerConfig and controllerConfig, isn't it?
So I don't understand how can I enable it.
How can we reproduce it?
Try to configure two IAM Roles (IRSA), one for team A, and another for team B.
What environment did it happen in?

Universal Crossplane Version:  1.10.1
Provider Version: 0.2.0
Cloud provider or hardware configuration: AWS
Kubernetes version (use kubectl version): v1.23.9
Kubernetes distribution (e.g. Tectonic, GKE, OpenShift): Vanilla
OS (e.g. from /etc/os-release): Flatcar Container Linux by Kinvolk 3227.2.1 (Oklo)
Kernel (e.g. uname -a): 5.15.58-flatcar

So basically,
instead of using the 1st option: using-iam-roles-for-serviceaccounts,
I should go for the using-assumerolewithwebidentity approach, right?

Right - sorry about that, I had my providers mixed up.  As a workaround you can put the assumeRole section in the terraform provider config and patch in the account/role information that you want to assume:
              provider "aws" {
                region = "%s"
                assume_role {
                   role_arn = "arn:aws:iam::%s:%s"
                }
              }

This is based on the ControllerConfig having the appropriate assumeRole annotation:
eks.amazonaws.com/role-arn: arn:aws:iam::<ACCOUNT>:role/<ROLENAME>

so that the pod gets AWS credentials, and can assume the role specified in the terraform provider configuration.
Hope this helps.


Error pages middleware break other services

Adding middleware to a service or middleware chain beaks whenever the error pages services is down.
To reproduce add the error pages middleware to a service and than stop the error pages service and try to access the other service.
There should be another way to catch this error pages but without breaking other services if error pages service goes down.

Hi @adlion! Looks like the https://github.com/traefik/traefik repository will be more relevant for your issue?

Closed due inactivity


Move to the MavenCentral

Could you move your library in the MavenCentral?

well actually i wanted to publish project on MavenCentral, but i didn't figure out, is there any tutorials to do that?

@ozodrukh check this out:
http://central.sonatype.org/pages/gradle.html
It will be much better if your library is published in the Maven Central! Thank you! :)

Whats wrong with just jitpack? Makes releasing much easier for devs. Also, releasing to bintray is much simpler with this plugin https://github.com/novoda/bintray-release

Some companies like ours have a policy of not using jitpack :-(
There are plugins to make maven central release easier too...


Remove "_count" suffix from doublewrite metrics

Fixes: #36580
Follow-up to https://github.com/cilium/cilium/pull/31920
Remove "_count" suffix from doublewrite metrics

These files should probably also be co-owned by the @cilium/metrics team in CODEOWNERS to help with consistency.

/test


gc 5.4.1 c printf with unsigned long long have issue

windows 10.0.15063
#include <stdio.h>
#include <inttypes.h>
typedef  unsigned long long u64;
typedef  unsigned int u32;
int main(int argc,char *argv[])
{
u64 d[3]={0};
d[0]=1<<63ULL;
printf("for big data is 0x%llx(0x%llu)\n",d[0],d[0]);
}

Is there anything more on this issue ... ;)

This is an issue with your above example.  You need to mark the "1" as a unsigned long long.
d[0]=1ULL<<63ULL;
Gcc 5.4.0 is even printing a warning when compiled about data type overflow:
benhill@BENHILL-STUDIO:~$ vi print.c
benhill@BENHILL-STUDIO:~$ gcc print.c
print.c: In function ‘main’:
print.c:8:7: warning: left shift count >= width of type [-Wshift-count-overflow]
 d[0]=1<<63ULL;
       ^
benhill@BENHILL-STUDIO:~$ ./a.out
for big data is 0x0(0x0)

After changing:
benhill@BENHILL-STUDIO:~$ vi print.c
benhill@BENHILL-STUDIO:~$ gcc print.c
benhill@BENHILL-STUDIO:~$ ./a.out
for big data is 0x8000000000000000(0x9223372036854775808)
benhill@BENHILL-STUDIO:~$

Thanks,it was my bad code.


Pollyfilling IE Array.from

Hey @ZitRos, I think this should be enough for fixing #8
What do you think? I didn't test it though.
Thanks!

Hi!
This way you made the dist/index.js file of size 100kb...
I managed just to replace Array.from in the sources to its analog. One moment I'll publish the fixed package.


Cannot update an existing record in the keychain

This isn't actually a pull request, as (a) there's a lot of code duplication in the tests, and (b) the tests currently fail! But it was the easiest way that occurred to me of bundling up the code I have currently that's failing.
It naïvely appears to be a recurrence of the very first issue in this project, but I have absolutely no experience in debugging Swift so I couldn't say.
For a Secret Work Project, we need an app that logs into our web service, and we need to remember those login details. So GenericPasswordSecureStorable looked like the easiest way of implementing this, albeit with a constant account value as we'd only ever store one set of user details.
As you'll hopefully see when running the tests, the C, R and D part of CRUD work fine; but when I try to update the details, I get an exception. And I get that whether I update the original object or I try to store a new one (which is good - if it had varied between which object was trying to store stuff that would have been the stuff of nightmares).
I'm new to Swift (normally I'm a Perl programmer), so some of the code might be unidiomatic, and hopefully the fix is to say "why on Earth are you doing that‽", update the documentation, and tell me not to be such a fool. But in case it actually is an issue in the guts of something horrible, you have a test method to run now.

❤❤❤ love a bug report with a good test case—thanks!
These are the failures I'm getting here—let me know if you're seeing something different

I might be misinterpreting here, but I think the library is behaving as intended in this case (though maybe how it works should change...). When secondUser.store() is called, in ends up calling createInSecureStore which won't replace the existing item in the keychain, and instead will throw an error. (.Duplicate, which leads to the logging of Couldn't store: Duplicate.)
The keychain just looks uses the values of the account/service pair to determine what's unique or already stored, so changing the username in this instance won't cause a new account to be stored (since, as you say, the account is constant).
It's possible to extend CreateableSecureStorable with an update method (or you can put this on your User type if that's preferred):
extension CreateableSecureStorable where Self : DeleteableSecureStorable {
    func replaceData() throws {
        do { try self.deleteFromSecureStore() } catch {} // Ignore any errors
        try self.createInSecureStore()
    }
}

Let me know if I've misunderstood/misinterpreted and I can take another look :)
Side note (would love to hear your thoughts on this, but feel free to skip if you don't have time to answer):
You are definitely not the first person who has expected that the create method would overwrite any existing data. The decision not to overwrite was made in an earlier incarnation of the library just never changed, though I'm not against changing it if there are considerable numbers of people confused by it. Although maybe this is just a documentation problem. I guess I have a couple of questions:

do you think the create method should overwrite or does the documentation just need to be improved?
how would you improve the documentation?

I'm super interested in the second point because you're a new Swift dev and I want to try to make Swift stuff as accessible to everyone as possible—what have you found confusing about Swift, Locksmith and its documentation in general?
(Also worth mentioning if you weren't aware that Locksmith probably isn't the easiest library to work because it's "protocol-oriented", which is a really new (and cool) idea that I wanted to share. It's certainly not the be all and end all of keychain libraries. Maybe I should add a note on that to the docs too?)
Thanks so much for taking the time to drop in the detailed test case and issue report!

Very quickly: yes, those are the same test failures I'm getting. Will now write a longer answer about the rest.

The protocol-oriented bit didn't bother me, actually - if anything, it makes me happy about using a library that's probably over-engineered for me at the moment because I suspect I'll be able to carry on using it for more complicated things in the future without getting fed up at limitations.
But then again, at $WORK we use Moose Roles a lot, which are basically the same thing (although Perl being a dynamic language means you can do funkier things with OO).

I mentioned CRUD (Create, Read, Update, Delete), which is a standard idiom these days, and having gone back to the documentation one thing that stands out is that it only mentions three things:

Creating, reading, and deleting each have their own protocols: CreateableSecureStorable, ReadableSecureStorable, and DeleteableSecureStorable. And the best part?

You can conform to all three protocols on the same type!

// CreateableSecureStorable lets us create the account in the keychain
try account.createInSecureStore()

// ReadableSecureStorable lets us read the account from the keychain
let result = account.readFromSecureStore()

// DeleteableSecureStorable lets us delete the account from the keychain
try account.deleteFromSecureStore()

I think the documentation should mention how to update an existing record, even if by saying that it's a Keychain-ism that you can't update a record in place, you have to delete it and create it anew instead.
Also, as I think I mentioned in another issue, it's really confusing that the example code talks about creating an account in the keychain, then reading it from the object that we just created. If nothing else, in the sample code above it should use a different variable name from account, e.g.
let accountAgain = TwitterAccount()
let result = accountAgain.readFromSecureStore()

Finally, I think some sort of explicit method would be useful, if nothing else because I suspect that there could be annoying race conditions involving threading if something happens between the delete and the create. This is the sort of really annoying intermittent problem that, if it ever happens, is best dealt with at the library level, rather than everyone having to find the problem for themselves. Also, because people used to the CRUD metaphor expect an explicit update method. I'm happy to roll my own for now, though.

There is a SecItemUpdate which is working fine. I had the same issue in SSKeychain
I fixed an issue with unabling to set a new value with this flow:
//try to set a value
if (![SSKeychain setPassword:pwd forService:svc account:@"test" error:&error] || error)
{
    [SSKeychain updatePassword:@"" forService:svc account:@"test"error:&error];
    [SSKeychain updatePassword:pwd forService:svc  account:@"test"error:&error];
}

I made a patch to SSKeychain.m:
+ (BOOL)updatePassword:(NSString *)password forService:(NSString *)serviceName account:(NSString *)account error:(NSError **)error
{
    NSData *data = [password dataUsingEncoding:NSUTF8StringEncoding];
    return [self updatePasswordData:data forService:serviceName account:account error:error];
}

+ (BOOL)updatePasswordData:(NSData *)password forService:(NSString *)serviceName account:(NSString *)account error:(NSError **)error
{
    OSStatus status = SSKeychainErrorBadArguments;
	if (password && serviceName && account) {
        NSMutableDictionary *query = [self _queryForService:serviceName account:account];
        [query setObject:password forKey:(__bridge id)kSecValueData];
#if __IPHONE_4_0 && TARGET_OS_IPHONE
        NSDictionary *update = @{(__bridge id)kSecValueData:password};
        if (SSKeychainAccessibilityType) {
            [query setObject:(id)[self accessibilityType] forKey:(__bridge id)kSecAttrAccessible];
            update = @{(__bridge id)kSecValueData:password,
                       (__bridge id)kSecAttrAccessible:(id)[self accessibilityType]};
        }
#else
        NSDictionary *update = @{(__bridge id)kSecValueData:password};
#endif

        status = SecItemUpdate((__bridge CFDictionaryRef)query, (__bridge CFDictionaryRef)update);
	}
	if (status != noErr && error != NULL) {
		*error = [NSError errorWithDomain:kSSKeychainErrorDomain code:status userInfo:nil];
	}
	return (status == noErr);
}

You, probably, could use the same technique. Couldn't post a PR to SSKeychain with this fix, but it works for me in my objective-с projects, now I would like to Locksmith to have this behavior too =)
If you want, I could post a PR with swift code.

Closing because I think this is fixed by @Sega-Zero in #106 :D


Missing products when retrieving all products

I am using this PHP library to connect Microsoft Access with WooCommerce. Nevertheless, when retrieving all products some of them are missing. This is the function I am using:
function getAllWCProducts(){
  //Get WooCommerce client
  $woocommerce = getWCClient();
  // 200 products per page
  $array = Array('posts_per_page' => 200);
  // Initialize counter
  $i = 1;
  // Initialize array
  $result = Array();
  // Execute a query while products are returned
  do{
      $page = $i;
      try {
        // Store every page in the array
        $result[$i] = $woocommerce->get('products', ['page'=>$page,'filter'=>$array]);
      } catch (Exception $e) {
        echo $e->getMessage();
      }
      $i++;
  }while(!empty($result[$i-1]));
  return $result;
}

In WooCommerce there are about 600 products. The function retrieves most of the products, but particularly, when post_per_page filter is set to 200 products with ID from 694 to 761 are not retrieved. And when post_per_page is set to 20, missing products ID goes from 694 to 818.
I have been debugging the code throroughly and my conclusions are that $woocommerce->get() is the culprit. Nonetheless, I do not know if the mistake is either on the filter parameters or in the function design.

If something is missing, I don't see how is an issue with this wrapper.
It should be reported on https://wordpress.org/support/plugin/woocommerce

The issue is with this wrapper. WooCommerce works perfectly and does show every product perfectly. The issue is indeed with this client.
If posts_per_page filter is changed to 10000, every product is correctly.

Is shown correctly*

The issue can be on the REST API and not this wrapper, for what you are saying, I don't see how this wrapper change the number of products or hide some product.
Just read the code here, it's not possible.

Understood. Thank you for your help.


Optimize Image Draw Functions.

Previously, for every pixels, ImageDrawPixel() was called which checks for image format and performs calculations on the given pixel coordinate.
https://github.com/raysan5/raylib/blob/07240b96ad7c689119e70714f3fb39852b3657b7/src/rtextures.c#L2419-L2432
The functions optimized here just copies same data over and over, so the calculation doesn't need to be performed more than once.
This optimization is already used somewhere else.
https://github.com/raysan5/raylib/blob/07240b96ad7c689119e70714f3fb39852b3657b7/src/rtextures.c#L2797
I was thinking of a general function sth like this.
void _repeat_pixels_in_memory(unsigned char *src, int count, int bytesPerPixel) {     for(int i = 1; i < count; i++)     {         memcpy(src + i * bytesPerPixel, src, bytesPerPixel);     } }
But not many use cases, so didn't add that.

@AnilBK thanks! this is a nice improvement! :)


Add Poznan Education

Poznan University of Medical SciencePlease add our school

file (2).zip
New Taipei Municipal Taishan Senior High School Please add our school

@qq1119506230 I am sorry, this request has signs of fraud. I have to close it without merging. If you are a student of this school, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).