[semantic] When to install the brand?

The initial spec written by @XGHeaven choose to install the brand at the end of constructor, but my original idea is installing it after fields initialization.
Generally speaking there are four timing:

At the start of constructor, but it's impossible as JS semantic (only possible in static type languages like Java)
After this is available, before fields initialization.
After fields initialization.
At the end of constructor.

I believe we should not choose 2, but not sure about 3 or 4. The difference of them is the behavior of class.hasInstance(this) in the constructor. 3 would always give true, 4 would always give false.  4 is stricter, ensure that if class.hasInstance(o) give true, o is completely initialized by the constructor. But it may be too strict for the cases call the methods have brand checks  from the constructor.

For example, below code will throw if we choose 4:
class User {
  name
  friends = new Set()
  constructor(name, friends) {
    this.name = name
    for (let f of friends) this.addFriend(f)
  }
  addFriend(user) {
    if (class.hasInstance(user)) {
      this.friends.add(user)
      user.friends.add(this)
    } else if (typeof user === 'string') {
      new User(user, [this])
    } else {
      throw new TypeError()
    }
  }
}

new User('alice', ['bob'])

Would there be a difference to this:
class Class {
  f1;
  f2:
  // ... more fields;
  fn;
  #brand; // keep this as the last field

  method(v) {
    // class.hasInstance(v)
    #brand in v;
  }
}

"Explicitly choosing the last private field as the brand" would be equivalent to this proposal choosing 3; "explicitly choosing the first private field" would be choosing 2; "explicitly putting code in the start or end of the constructor" would be choosing 1 or 4, respectively.

"explicitly choosing the first private field" would be choosing 2

Strictly speaking, first private field is also possible fail to initialize and install, and we should not forget public fields. A "better" choice for 2 is "explicitly choosing private method".

After discussion with @XGHeaven and @YuriTu , we agree to keep the behavior of current draft text (also the behavior of the experimental babel implementation), aka. "4. At the end of constructor". A more precise word should be: After the return completion of the constructor so if constructor throw, the brand will not be installed. (We also find the current draft has a bug in ordering of some steps, @XGHeaven will fix it soon.)

Close for now. Could reopen if we want to revisit the decision later.

To confirm the equivalent code today would be something like the following?
class Class extends K {
  #status;

  constructor() {
    try {
       super(/* super args here */);
       ... // more constructor logic here
    } catch(err) {
       this.#status = "failed";
       throw err;
    } finally {
      this.#status ?? = "success";
    }
  }

  method() {
     // class.hasInstance(this)
     Class.#hasInstance(this);
  }

  static #hasInstance(v) {
    return typeof v === 'object' && #status in v && v,#status === "success";
  }
}

@acutmore Exactly. 😃


add support for AuthFileCredential

Added AuthFileCredential and its tests to azure-identity.
https://github.com/Azure/azure-sdk-for-net/pull/9312

@g2vinay - Can you help us get this approved for April release?

This is on hold for further discussion.


Make declarations compatible with cviebrock/eloquent-sluggable

It will prevent this kind of warnings:
Symfony \ Component \ Debug \ Exception \ FatalErrorException (E_UNKNOWN)
Declaration of Backpack\CRUD\ModelTraits\SpatieTranslatable\Sluggable::replicate(?array $except = NULL) must be compatible with Backpack\PageManager\app\Models\Page::replicate(?array $except = NULL): Illuminate\Database\Eloquent\Model

It happened to me also.  This fixes issues. Thanks for pr!

@tabacitu It would appear that upstream (i.e. cviebrook/eloquent-sluggable) or something is requiring native types (i.e. not classes) on function declaration - I'll hand this one to you to reivew :)

Is there any problem with approval? These seem to be minor adjustments, but without them whoops keeps complaining like this:
Declaration of Backpack\CRUD\ModelTraits\SpatieTranslatable\SluggableObserver::generateSlug(Illuminate\Database\Eloquent\Model $model, $event) should be compatible with Cviebrock\EloquentSluggable\SluggableObserver::generateSlug(Illuminate\Database\Eloquent\Model $model, string $event)

Thank you for explanation. I guess it may not be possible to support PHP 5.6 and hence Laravel 5.4 and lower with the same code base as Laravel 5.5 which requires PHP >=7.0.

Could this be merged?

Thank you @pamoedo for the PR, and @pxpm , @lloy0076 and @kmlyn for reviewing it. I've checked and double-checked, this seems to be backwards-compatible since Laravel 5.5 requires PHP 7+ and that's the oldest version of Laravel we support right now.
Thanks again for the PR @pamoedo , I just merged it into master and it'll be available with a composer update in 1-2 days, after I bundle it with a few more non-breaking changes.
Cheers!


customer with no pages on facebook

We should be displaying something better than:

what are your thoughts?
The layout seems fine to me, maybe a different text like:
"Selecione abaixo a página onde deseja ativar o módulo de rastreamento de pedidos."

sorry for the bad bug description :stuck_out_tongue:
what the screenshot above is displaying is the empty drop-down presented to an user that does not have Facebook pages to install a bot.
For example: the Fernanda fake user is just an user that was invited to help with unmuting chats, she dont have facebook pages and still the dropdown is displayed.
I don't think we should be displaying empty dropdowns, it would be better to have a statement in the lines of:
"You don't have facebook pages to setup as a channel yet, please create one and try again"

Oooh and about the big blank, maybe a default message like: Você precisa criar ou administrar uma página para selecionar aqui.

my suggestion for empty pageList:
"Para configurar o contato de Facebook Messenger com o qual as pessoas farão consultas de rastreio de pedidos via chat, é preciso que você tenha uma página no facebook."
Just the text, without the dropdown and without the buttons.

Looks good to me.
Lets do like this and if we decide to change the text later it will be easy, although I appreciate the text as well.

feel free to assign to yourself if you want to write that patch, the priority of this is quite low, totally non-blocker for the soft launch


Migrated most methods to prototypes

Joins and ordering do not work yet
Takes care of #23

Now migrated all Table methods to prototypes.
Need to update documentation


Support for .jar Libraries

Would it be possible to implement support for .jar Libraries for (remote) java execution?

Local Java is now available in the latest TestFlight build. You can use java and javac to run Java programs including .jar libraries.


Replace hardcoded sample project items count in SelectedProject.js

Motivation
In /src/Utilities/SelectedProject.js there is a hardcoded SAMPLE_PROJECT_ITEM_COUNT constant which is used in the wizard.
This wizard checks if the Delivery API has already accessible all items in the user's Dancing Goat project and if the project is fully accessible. If the count of all items in the project is more or equals to items from constant then we consider the project to be ready.
However, if the user's sample Dancing Goat project changes (we add/remove new content) this logic might not work.  Under the hood, we clone the Dancing Goat project from 'one-source-of-truth' project.
Proposed solution
The goal of this issue is to implement logic which will get a count of items in from this source (-of-truth) project (projectId: 975bf280-fd91-488c-994c-2f04416e5ee3) and store it to an existing (non)constant which will be checked the same way as it is right now.
Note: It might be a good idea to adjust if condition in Configuration.js, too.
Something like ... if (response.items.length > SAMPLE_PROJECT_ITEM_COUNT) in waitUntilProjectAccessible function.

Hello,
I would work on this issue.

Hi @sheidaee, thank you for your interest! I've sent you an invitation to the organization. Once you accept it, I can assign you this issue. Welcome to  Kentico Open Source Community!

@sheidaee - just sending a related issue https://github.com/Kentico/cloud-sample-app-vue/issues/26

there is a hint how it was solved in VueJs by @liamgold.


klipper-estimator: 3.5.0 -> 3.6.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/klipper-estimator/versions.
meta.description for klipper-estimator is: Tool for determining the time a print will take using the Klipper firmware
meta.homepage for klipper-estimator is: https://github.com/Annex-Engineering/klipper_estimator
meta.changelog for klipper-estimator is: https://github.com/Annex-Engineering/klipper_estimator/releases/tag/v3.6.0
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.6.0 with grep in /nix/store/qm7ax7p4wrynw9daw3d7s68h1izscf37-klipper-estimator-3.6.0
found 3.6.0 in filename of file in /nix/store/qm7ax7p4wrynw9daw3d7s68h1izscf37-klipper-estimator-3.6.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
klipper-estimator

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qm7ax7p4wrynw9daw3d7s68h1izscf37-klipper-estimator-3.6.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A klipper-estimator https://github.com/r-ryantm/nixpkgs/archive/db51cbbd3673e364c8108a6230e493ac44bf1589.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/db51cbbd3673e364c8108a6230e493ac44bf1589#klipper-estimator

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qm7ax7p4wrynw9daw3d7s68h1izscf37-klipper-estimator-3.6.0
ls -la /nix/store/qm7ax7p4wrynw9daw3d7s68h1izscf37-klipper-estimator-3.6.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

klipper-estimator

Maintainer pings
cc @hesiod for testing.

@ofborg build klipper-estimator klipper-estimator.passthru.tests


Adding data to empty chart does not render correctly

I have a Chart.js bar chart that gets instantiated on page load. I then wish to add data to the chart as it becomes available (no data is available at the time of its creation). However, passing in new data always seems to have the chart fail to render the first set of data passed in.
More details here: http://stackoverflow.com/questions/25455613/adding-new-data-to-empty-chart-js-chart-does-not-render-new-data-correctly
jsfiddle: http://jsfiddle.net/9sff8o5d/3/
Update: Looking at the DOM for the created bar_chart object, you can see that the first created bar has an 'x' and 'width' value = NaN. Perhaps these computed values are dependent on other elements being present, in order to calculate a size. Will provide another update as I continue to look into this.

Update 2: Got it working - though it seems like more of a hack. I am manually adding the initial values to the charts by accessing the bar_chart_options, then destroying and re-creating the chart.  Seems like there has to be a better way (.update seems to provide this, but isn't working).
jsfiddle: http://jsfiddle.net/9sff8o5d/6/

Running into the same thing. @m4tthumphrey edit seems to do the trick for me!

I believe this has been fixed in the v2.0-dev branch. See the line sample and play with the "Add Data" and "Remove Data" buttons


[Documentation Update]: Add a solution Leetcode problem 606(Medium question)

Is there an existing issue for this?

[X] I have searched the existing issues

Issue Description
I want to add solution for leetcode question number 606
Suggested Change
I want to add solution for leetcode question number 606
Rationale
No response
Urgency
High
Acknowledgements

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository
[X] I have read and followed the Contribution Guidelines
[X] I have followed the code style guidelines of this project
[X] I have checked for any existing open issues that my pull request may address
[X] I have ensured that my changes do not break any existing functionality
[X] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently
[X] I have read the resources for guidance listed below
[X] I have not used AI-generated content (e.g., ChatGPT, other AI tools)
[X] I have not used content from existing sites (e.g., text, images)
[X] I have followed all security rules and only shared trusted resources

This question is already updated by someone it's updated in the deployement also but you assigned me the same question.


codegen pallet bindings on ChainRune subtype

We currently codegen pallet bindings, which are accessible from the root mod. Ie.
import { Balances } from "@capi/my-chain"

const call = Balances.transfer({ /* ... */ })

There's an issue with this however: we may want to specify the discovery value / how to connect at runtime.

We can do this if we utilize chain.
import { chain as chain_ } from "@capi/my-chain"
import { SmoldotConnection } from "capi"

const chain = _chain.with(SmoldotConnection.bind({ relayChainSpec }))

const call = chain.extrinsic(Rune.object({
  type: "Balances",
  value: Rune.object({
    type: "transfer",
    // ...
  })
}))

This experience is different from that of utilizing the generated bindings. It's not the "happy path."

Alternatively, we could expose a means of mutating the internally-held connection state.
import { setConnection } from "@capi/my-chain"
import { SmoldotConnection } from "capi"

setConnection(SmoldotConnection.bind({ relayChainSpec }))

However, this too is not ideal.

Per an offline Capi team convo yesterday, we're thinking of changing the look of the codegen, such that the experience would be as follows.
import { chain } from "@capi/my-chain"

const call = chain.Balances.transfer({ /* ... */ })

There's an issue with this: what if a pallet name conflicts with that of a property of chain? Perhaps...
import { chain } from "@capi/my-chain"

const call = chain.pallets.Balances.transfer({ /* ... */ })

I'd personally prefer not to place pallet bindings under yet another prop. That being said, I'm unsure of whether there are better options.

One more thing to note: this approach would be consumable within pattern libraries, which is not true of the current pallet bindings.

@kratico @ryanleecode @tjjfvi –– thoughts?

There's an issue with this: what if a pallet name conflicts with that of a property of chain?

I think this is fine; the codegen needn't be as robust as the regular API, so no need for a pallets prefix, IMO.


[Snyk] Upgrade @giscus/react from 2.0.5 to 2.0.6

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @giscus/react from 2.0.5 to 2.0.6.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2022-06-19.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report

Merging #294 (691abcd) into main (60b8fec) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #294   +/-   ##
=======================================
  Coverage   81.98%   81.98%
=======================================
  Files          28       28
  Lines         372      372
  Branches       55       55
=======================================
  Hits          305      305
  Misses         61       61
  Partials        6        6

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 60b8fec...691abcd. Read the comment docs.


Documentation

Add user and developer docs including api usage docs

https://github.com/opensearch-project/documentation-website/pull/586


Allow to specify build deps

Some project rely on various utils to generate tar-ball or SRPM. E.g., tito, autotools..It would be nice if I can specify this dependencies and you install them before you try to generate tar-ball or SRPM.

With the current architecture of packit service, this is close to impossible to be done (securely).
We are running all the user-defined commands in a sandbox - an openshift pod, where:

you can't modify the FS
the process has random uid (and certainly is not root)

Please let us know the list of deps you'd like to see and we can include them in the container image for sandbox: so far I can see autotools and tito.

I added the deps you mentioned to our image, plus even some more in the PR mentioned above.
Hence I am going to close this since this is something which we are not planning on implementing any time soon.


Update installation.md

Added information to get this enabled for uefi secured devices.

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area videos

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: cccsss01
Once this PR has been reviewed and has the lgtm label, please assign vjjmiras for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

content/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

The preview is incorrect:

Thanks

@cccsss01, thanks a lot for these great contributions. Apologies again for having taking so long to review them.
Please, have a look to my suggestions and let me know what you think.

I think this looks great, thank you.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: cccsss01
Once this PR has been reviewed and has the lgtm label, please ask for approval from vjjmiras. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

content/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cccsss01 It seems it needs a rebase. Can you do it so I can approve it?

LGTM label has been added.  Git tree hash: 1ea48a98d94c337b93f90915c5015af2af1afe8b

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cccsss01, vjjmiras
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [vjjmiras]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Wow! Amazing work! Thanks a lot!

Hi!
I also wanted to thank you for this wonderful library! The API is super clean and love it so far.
One question I have is regarding the accuracy score like the accuracy_score() function in scikit learn that can be seen here: https://www.kaggle.com/mathvv/prediction-of-red-wine-quality-93-215
Like this:
print('Random Forest:', accuracy_score(y_test, rf_pred)*100,'%')
Which outputs this:
Random Forest: 91.875 %
Many thanks and congrats again!
Flo

Hi @flo8,
I am glad you like the library! In SharpLearning, the accuracy_score from scikit learn can be computed using the TotalErrorClassificationMetric. In general, SharpLearning returns the "error" rather than the "score" (accuracy in this case). So to get the accuracy, you have to subtract the error from 1.0 and multiply by 100 to get percent, like this:
var metric = new TotalErrorClassificationMetric();
var accuracy = (1.0 - metric.Error(y_test, rf_pred) * 100.0;

In the future, I might add an extension method to the classification metrics so you can get the "score" returned directly. Something like:
var accuracy = metric.Score(y_test, rf_pred);

Best regards
Mads

Hi @mdabros
Thanks for the quick reply, makes sense! Yes the metric.Score() would be nice even if it is indeed very simple to compute with current methods :)
Many thanks
Flo


[Feature request] Missing non-Linux/MacOS UNIX-entries

No entries for common *BSD/illumos commands even though description claims "UNIX" coverage. The query "pkg" f.x. should return multiple hits across the BSDs. "pkgsrc" is even available on Linux/MacOS systems too. How to add such coverage?

@NitroNils Thanks for submitting a issue about this, but I do see a couple of 'pkg..' suggestions in the app. Did not see pkgsrc though. FYI the data for the app comes from tldr-pages so if its not there it won't show up in the app. Maybe create a issue there?
cc: @bl-ue

Hi!  What's the problem exactly here?  I'm happy to help, but I'm not quite sure what @NitroNils is having  trouble with.  We don't have any pkgsrc command documented.  I don't have pkgsrc on my Mac either, or at least it's not in my $PATH.  A few more pointers?

@Techno-Disaster Ah, thsnk you for the pointer to tldr-pages - this is indeed where the issue lies.
My version is 1.1.0 and find no results with "pkg"- query.

@bl-ue The upstream project doesn't seem to include entries from other common open source Unix like operating systems (OpenSolaris- and BSD-based ones).

Looks like this issue has been discussed in tldr pages, and should have been opened there. So I will close this for now.


Fix handling of roles with paths

Fix #20

Pull Request Test Coverage Report for Build 100

17 of 17 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 68.351%

Totals

Change from base Build 97:
0.4%

Covered Lines:
514

Relevant Lines:
752

💛  - Coveralls


Pin 9 in display mode V2

Using pin9.write_digital(1) results in ValueError: Pin 9 in display mode which we would expect on V1 as the display is coupled to the Pin.
Should it be the case as per https://tech.microbit.org/hardware/edgeconnector/#edge-connector-pins that Pin9 is no-longer coupled to the display for V2 and if so can we enable use of pin9 without having to disable the display?
MicroPython v1.13 on 2020-12-21; micro:bit v2.0.0-beta.3 with nRF52833
Type "help()" for more information.
>>> from microbit import *
>>> pin9.write_digital(1)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: Pin 9 in display mode

See https://github.com/microbit-foundation/micropython-microbit-v2/pull/64 - will test once there's a CI build

Fixed by a94e53de7e7d6856a97b71ad5667501b05e7bf03


[Evaluation] Improve patch apply in SWE-Bench

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
.rstrip() removes the last newline of the git patch that affects the applied "rate of the generated patch," which hurts the pass rate a little bit.
Give a brief summary of what the PR does, explaining any non-trivial design decisions
This adds a new line at the end of the git patch (if it is not already empty) and improves "CodeAct v1.6 no-hint" from 56 (18.6%) to 58 (19.3%): https://huggingface.co/spaces/OpenDevin/evaluation/commit/a4e8ae805efa8b76a9f7353465baf57bdec415bf
I'm in the process of re-running all the existing runs.

the reasoning would be to have a newline at end of file in any case, but not because of strip

This is more like guarantee to have ONE new line at the end of file, but not multiple newlines, as long as the patch is not empty.


Upgrade gradle

Running recent intellij with java 11 runtime
Cannot determine classpath for resource 'java/sql/SQLException.class' from location 'jrt:/java.sql/java/sql/SQLException.class'.

The project uses Gradle 2.10 which is incompatible with IDEA running on Java 10 or newer.
See details at https://github.com/gradle/gradle/issues/8431
Possible solution:
 - Upgrade Gradle wrapper to 4.8.1 version and re-import the project

ping :-)


Use on own dataset

Hi,
How would I go about using this model on my own dataset with different classes? Any pointers?
Thanks!

Hi @AndreaPisoni ,
Besides prepare your own data, you need to modify two layers' num_output: "rfcn_cls", "psroipooled_cls_rois", change the num_output of "rfcn_cls" to [your class_num] * 7 * 7, and the num_output of "psroipooled_cls_rois" to [your class_num]. And you needn't modify num_classes of "input-data" layer since it is useless when using RPN.


add automated tests

We should add automated tests like those run for the webservices dispatch action here: https://github.com/conda-forge/webservices-dispatch-action/blob/main/tests/run_live_rerender_test.py

closed by #36


ImportError: No module named redis

Hi folks,
we were trying datadog on our servers. The setup was just fine.
After that we tried to integrate haproxy (successfully) and than we tried to integrate redis. And it doesn't work.
We are on a freebsd server (10.3).
As soon as we set up the yaml file we go this error:

2017-04-28 15:26:01,850 | ERROR | dd.collector | collector(agent.py:604) | Uncaught error running the Agent
Traceback (most recent call last):
File "agent/agent.py", line 600, in
sys.exit(main())
File "agent/agent.py", line 526, in main
return Agent.info(verbose=options.verbose)
File "agent/agent.py", line 218, in info
return CollectorStatus.print_latest_status(verbose=verbose)
File "/root/.datadog-agent/agent/checks/check_status.py", line 281, in print_latest_status
message = module_status.render()
File "/root/.datadog-agent/agent/checks/check_status.py", line 179, in render
] + ["", ""]
File "/root/.datadog-agent/agent/checks/check_status.py", line 545, in body_lines
'  ' + '-' * (len(cs.name) + 3 + len(cs.check_version))
TypeError: object of type 'NoneType' has no len()
Traceback (most recent call last):
File "agent/agent.py", line 600, in
sys.exit(main())
File "agent/agent.py", line 526, in main
return Agent.info(verbose=options.verbose)
File "agent/agent.py", line 218, in info
return CollectorStatus.print_latest_status(verbose=verbose)
File "/root/.datadog-agent/agent/checks/check_status.py", line 281, in print_latest_status
message = module_status.render()
File "/root/.datadog-agent/agent/checks/check_status.py", line 179, in render
] + ["", ""]
File "/root/.datadog-agent/agent/checks/check_status.py", line 545, in body_lines
'  ' + '-' * (len(cs.name) + 3 + len(cs.check_version))
TypeError: object of type 'NoneType' has no len()

Investigating a bit more we found that this command
./bin/agent check redis
returns ImportError: No module named redis
so we installed through pip the redis-client with this command:
pip install redis
but nothing changes.
Are we missing something?

Hi @gianpaolodn, thanks for using Datadog.
This issue queue is primarily intended for tracking features, bugs and work items associated with the dd-agent open source project. For your request, please contact our support and send them your logs.
In the meanwhile I can point out that the agent use it's own python env (with pip and all).
The redis pip package is ship with the agent if you use our system package or container see more here.
Here is an example with the latest version of the agent (5.13.0):
$> /opt/datadog-agent/embedded/bin/pip freeze | grep redis
redis==2.10.5

Thanks.

Hi @gianpaolodn, thanks for using Datadog.
This issue queue is primarily intended for tracking features, bugs and work items associated with the dd-agent open source project. For your request, please contact our support and send them your logs.
In the meanwhile I can point out that the agent use it's own python env (with pip and all).
The redis pip package is ship with the agent if you use our system package or container see more here.
Here is an example with the latest version of the agent (5.13.0):
$> /opt/datadog-agent/embedded/bin/pip freeze | grep redis
redis==2.10.5

Thanks.

Oh I'm sorry.
I was writing this bug in the agent repo but this quote stopped me:

A note: All Agent Checks are now frozen in preparation for the release of the Integration SDK. All issues related to checks from here on out should go to the integrations-core repo.

So I went here to write the bug.
By the way, I installed it from source and there's no pip integrated as you pointed out. It's in this place:
[~]# .datadog-agent/venv/bin/pip freeze | grep redis
You are using pip version 6.1.1, however version 9.0.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
redis==2.10.5

and i finally solved my issue :)


Branch

Is there any explanation on why you branched from https://eonasdan.github.io/bootstrap-datetimepicker/ and what the possible differences or Pro/contras are?
Greetings

Hi @DavidDecraene,
We didn't branch off from the bootstrap-datetimepicker. If I remember correctly it was the other way around (but a long time ago).
Off course the main difference is that we don't have time support and they have. For other feature differences you should look at the documentation of each library to find specific differences.


sourced

fixes #327

[ ] dynamically sourced
[ ] sourced — for keepFresh
[ ] new normalization
[ ] allow skipping sourced call in simple cases
[ ] docs
[ ] migration guide

I've though about this change one more time, and it's bad 🫠 it's really harmful because it requires significant effort for end users to migrate on. Instead, in this release, I introduce a softer way to fix #327. Maybe, I'll return to this idea later (it still could simplify types of the library).


[BACKLOG-10661] Fixing bug in tab/spoon logging

@bmorrise

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f ui/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f ui/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  ui/src/org/pentaho/di/ui/spoon/Spoon.java
  ui/src/org/pentaho/di/ui/spoon/delegates/SpoonTabsDelegate.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.ui.spoon.delegates.SpoonTabsDelegate

Complexity Change:  + .06%
Instruction Change: -.28%:small_red_triangle_down:
Line Change: -.71%:small_red_triangle_down:
Method Change:  + .20%

org.pentaho.di.ui.spoon.Spoon

Branch Change: -.01%:small_red_triangle_down:
Complexity Change: -.01%:small_red_triangle_down:
Instruction Change: -.01%:small_red_triangle_down:
Line Change:  + .02%


GitHub Enterprise support

This is partially, but not quite, working with GitHub Enterprise. Would love the ability to specify a GitHub Enterprise URL in the CloudFormation setup.

lambci/sources/github.js will need a bit of tweaking for this, since Enterprise uses a single hostname for both the web frontend and the API. It follows the form https://{hostname}/api/v3/

@jhurliman interesting – is there any documentation on this? Or is the hostname literally the only difference?
It would certainly be straightforward to add this as a config param – which could be administered like the others, via the command line, etc. I'm wary of adding it to the cloudformation template just because I don't want the list of params to start blowing out – although perhaps I could just create a separate lambci-enterprise.template that includes it.

https://developer.github.com/v3/enterprise/
Whatever works! I don't have a specific preference, just looking forward to using this.

+1 for this. We have some great use cases for this but need to use our Enterprise  accounts for obvious reasosns.

+1 for adding this as well. I went through the code to see where to add this and it seems it would be much easier for you to do it here as the CF template pulls the source from your s3 bucket.
Likely two useful additions would be the ability to provide the optional param here, additionally the ability to provide the S3 bucket to pull lambci code from rather than the version. Had this been there it would have been much easier for me to make the above changes. As it stands, even if I made the changes, I'll need to alter other deployment details.
Not be critical by any means, this is amazing work and super excited to use, and help out.

I hate to even ask, but with 3 years since the last bump, has the outlook on GitHub Enterprise support changed?

PRs welcome here. I don't have access to GitHub Enterprise, so it's pretty hard for me to develop for it.

Alright, thanks. I'll see if I can try it out then.


Heya, just some cleaning using prettier and a new theme variant

Reformatted and cleaned using Prettier
Used the Tailwind CSS color palette to achieve my desired result.
Added 2 new rules to the Jena variant:.columns-area--mobile {
    box-shadow: 0 0 25px #27272a;
}

.tabs-bar__wrapper {
  padding-top: 0 !important;
}

Hi,
Thank you for this PR! However, I am not willing to maintain another color variant as of yet besides purple.
You're welcome to keep working on yours as a fork though :)


Shield Floating Message

In ["/dynamic/helpers/boxes/shield_box.lua"] line 37 we have -- local new_message = floating_message.new(x, y, "Shield +1", 1.5, 0xffff00ff, 3) so maybe change it to local new_message = floating_message.new(x, y, "Shield +1", 1.5fx, 0xffff00ff, 3) before hand so as to get a floating message by default rather then changing it each update as it is a bit forgetful for me and also an arrow by default would be nice :sweat_smile:

You could change the code yourself, and make a pull request. That way everyone when they update the helpers will have the new version.

#9 This feature is implemented.


refactor cross-module control-flow

Currently the control flow first goes to emulation first, then to syscall_mocking which then drives the SyscallMock from emulation. That back and forth is a bit confusing. Maybe we should have an entry module to move the entry functions from emulation into. Or just move them into syscall_mocking.
Related:


Runnning google.com/earth live

Runnning google.com/earth live
Originally posted by @Rudy1025 in https://github.com/Rudy1025/Rudy1025/issues/3

Open
Www.google.com/earth live

Originally posted by @Rudy1025 in https://github.com/Rudy1025/Rudy1025/issues/3


Updates to the SLR table fields from the v10 LKML submission

Updates to the SLR table fields from the v10 LKML submission

This repository is in the process of archivization. Could you move this PR to https://github.com/TrenchBoot/TrenchBoot.github.io/?


Enhance SAS authentication

Is your feature request related to a problem? Please describe.
At the moment, if a user closes & opens a vscode window, the user has to log back in. This leads to more redirection than is necessary, and could be frustrating for the user.
Describe the solution you'd like
My hope is that we can use AuthenticationProvider and cache the results so that a new window can use the cached access / refresh token to establish a session

May be worth looking into this SAS Authentication Library for the Browser and/or discussing with @brmorr to see if anything we could learn from or re-use. It looks like that is the approach they are recommending for consumers of the VA SDK.

Currently sas-auth-browser supports cookie based auth and will soon also support oAuth with PKCE.  But, as indicated by the package name, is really meant specifically for browser based auth.  But it has been discussed that it may also be desired to have a similar sas-auth-node package.  I am not familiar with how the auth currently works for the SAS extension, or more generally how other VS Code extensions handle auth.  But it seems like this would probably be a node thing, not a browser thing.


Added Netbeans logo

Updates / Changes

Added Netbeans SVG file in library folder
Added Netbeans IDE SVG information inside JSON file

Thanks 🚀🚀


Add a validator ensuring that each branch has a well-formed range

It does not make sense to have multiple upper bounds for a range, as they would need to match all, meaning that the lowest version of them would be the useful one. Same is true for lower bounds (except that the highest version is the meaningful one).
Defining multiple affected ranges is what branches are representing in the advisory format.
Build will be green once #425 is merged

Travis build before merging #425, showing that it detects that case: https://travis-ci.org/FriendsOfPHP/security-advisories/builds/614028742


How to specify iceberg catalog

I am now trying to use frostbow to read the iceberg table. How can I specify the catalog of iceberg?
AWS_ACCESS_KEY_ID=... AWS_SECRET_ACCESS_KEY=... target/debug/frostbow -u postgres://leolong:postgres@<IP_ADDRESS>:5432
> CREATE TABLE read_people (
    id BIGINT,
    name STRING,
    age INT
)
STORED AS ICEBERG
LOCATION '';
Error during planning: Catalog datafusion is not an Iceberg catalog.

And I found that tag v0.0.1 can support the normal form of creating tables, but the latest version only supports create table ... store as .... I don't know if this is a bug.

When I try to run the test case, I get some unexpected errors. I think I have not configured something.
./target/debug/frostbow -u arn:aws:s3tables:us-east-2:050451389186:bucket/my-prefix-
> CREATE SCHEMA warehouse.tpch;
External error: service error
> CREATE TABLE iceberg.public.orders (
      id BIGINT NOT NULL,
      order_date DATE NOT NULL,
      customer_id INTEGER NOT NULL,
      total_price DOUBLE NOT NULL
)
STORED AS ICEBERG
LOCATION ''
PARTITIONED BY ( "month(order_date)" );
External error: service error

Looking forward to your reply！

Generally you can use different catalogs with Iceberg, the currently supported ones are: Filesystem, S3Tables, Glue, SQL. The Glue and SQL catalog are not bundled with the default package but have their own package.
If you want to use postgres as your catalog you should use frostbow-sql.
The simplest catalog to use is the Filesystem catalog as it uses the filesystem as the catalog and doesn't require any external service. Filesystem catalog.

I started with the Filesystem catalog：
AWS_ACCESS_KEY_ID=... AWS_SECRET_ACCESS_KEY=.... ./target/debug/frostbow -u s3://leolongbucket
thread 'main' panicked at /home/leolong/.cargo/git/checkouts/iceberg-rust-a475049abca24a95/b6120eb/catalogs/iceberg-file-catalog/src/lib.rs:621:14:
called `Result::unwrap()` on an `Err` value: ObjectStore(Generic { store: "S3", source: ListRequest { source: BareRedirect } })
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

But it redirects incorrectly and I don't know why.

Could you try to pass the region with AWS_REGION=us-east-2?

I configured all the options I thought I needed to configure, but it really doesn't work：
leolong@dc:frostbow-jankaul$ aws configure
AWS Access Key ID [****************3H2N]:
AWS Secret Access Key [****************rqfp]:
Default region name [us-east-2]:
Default output format [json]:
leolong@dc:frostbow-jankaul$ aws s3 ls
2024-12-09 18:04:53 iceberg-buckett
2024-11-27 13:44:09 leolongbucket
leolong@dc:frostbow-jankaul$ AWS_ACCESS_KEY_ID=... AWS_SECRET_ACCESS_KEY=... AWS_DEFAULT_REGION=us-east-2 AWS_ALLOW_HTTP=true ./target/debug/frostbow -u s3://leolongbucket
thread 'main' panicked at /home/leolong/.cargo/git/checkouts/iceberg-rust-a475049abca24a95/b6120eb/catalogs/iceberg-file-catalog/src/lib.rs:621:14:
called `Result::unwrap()` on an `Err` value: ObjectStore(Generic { store: "S3", source: ListRequest { source: BareRedirect } })
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Hmm, I'm sorry about that. I just tried it and it works for me.
Could you try to use AWS_REGION instead of AWS_DEFAULT_REGION? Some sdk might only use the former: https://stackoverflow.com/questions/59961939/what-is-the-difference-between-aws-default-region-and-aws-region-system-variable

Are you running it from an EC2 instance? Are you using an instance profile? If yes, you need to make sure that the profile has the required permission to read and write to your bucket.

There was a bug loading the environment variables. I think I fixed it now. Could you try it with the new version.
Sorry for the inconvenience and thanks for reporting the issue.

Hello, I updated your newly submitted code and tested it and found the following problems:

After inserting table data, it shows that 0 rows have been inserted, but the data has been inserted：

> CREATE TABLE iceberg.public.people (
    id BIGINT PRIMARY KEY,
    name VARCHAR(255),
    age INT
)
STORED AS ICEBERG
LOCATION '';
> INSERT INTO iceberg.public.people (id, name, age) VALUES (1, 'Job', 25);
+-------+
| count |
+-------+
| 0     |
+-------+
> select * from iceberg.public.people;
+----+------+-----+
| id | name | age |
+----+------+-----+
| 1  | Job  | 25  |
+----+------+-----+

When using OSS object storage, the table created already exists but cannot be read:

> CREATE TABLE iceberg.public.people (
    id BIGINT PRIMARY KEY,
    name VARCHAR(255),
    age INT
)
STORED AS ICEBERG
LOCATION '';
External error: Table already exists. Path doesn't have the right format
> select * from iceberg.public.people;
Error during planning: table 'iceberg.public.people' not found

I don’t know if this is a problem caused by the previous changes.
Thank you again for your reply and answer!

Thank you for your feedback. The count is a bug that I will look into.
By OSS object-storage do you mean Minio? I'll try to find out what happens.

Minio is supported, but OSS will report an error.It should be some protocol incompatibility, not the problem of the project

After repeated attempts, I found that OSS object storage would report an error about inconsistent signatures SignatureDoesNotMatch, but when I used the SDK to write rust code with the same function, it could be executed normally：
leolong@dc:frostbow-jankaul$ AWS_ACCESS_KEY_ID=*** AWS_SECRET_ACCESS_KEY=*** AWS_DEFAULT_REGION=oss-cn-beijing AWS_ENDPOINT_URL=https://leolongossbucket.oss-cn-beijing.aliyuncs.com AWS_ALLOW_HTTP=true target/debug/frostbow -u s3://
> CREATE TABLE iceberg.public.people (
    id BIGINT PRIMARY KEY,
    name VARCHAR(255),
    age INT
)
STORED AS ICEBERG
LOCATION '';
External error: The operation lacked the necessary privileges to complete for path iceberg/public/people/metadata/v0.metadata.json: Client error with status 403 Forbidden: <?xml version="1.0" encoding="UTF-8"?>
<Error>
  SignatureDoesNotMatch
  <Message>The request signature we calculated does not match the signature you provided. Check your key and signing method.</Message>
  <RequestId>6769315299F00D323592A2F7</RequestId>
  <HostId>leolongossbucket.oss-cn-beijing.aliyuncs.com</HostId>
  <AWSAccessKeyId>LTAI5tMCtEZumMEb1CwTEtnt</AWSAccessKeyId>
  <SignatureProvided>5c514dedac769963f329c87f4cac093148cfbc2db1e757b570be9a0f0195728c</SignatureProvided>
  <StringToSign>AWS4-HMAC-SHA256
20241223T094554Z
20241223/oss-cn-beijing/s3/aws4_request
9281ed4495565fb17d18a399fc7ab696cd19e080a9e958a4ae49dd811ef41957</StringToSign>
  <StringToSignBytes>41 57 53 34 2D 48 4D 41 43 2D 53 48 41 32 35 36 0A 32 30 32 34 31 32 32 33 54 30 39 34 35 35 34 5A 0A 32 30 32 34 31 32 32 33 2F 6F 73 73 2D 63 6E 2D 62 65 69 6A 69 6E 67 2F 73 33 2F 61 77 73 34 5F 72 65 71 75 65 73 74 0A 39 32 38 31 65 64 34 34 39 35 35 36 35 66 62 31 37 64 31 38 61 33 39 39 66 63 37 61 62 36 39 36 63 64 31 39 65 30 38 30 61 39 65 39 35 38 61 34 61 65 34 39 64 64 38 31 31 65 66 34 31 39 35 37 </StringToSignBytes>
  <EC>0002-00000429</EC>
  <RecommendDoc>https://api.aliyun.com/troubleshoot?q=0002-00000429</RecommendDoc>
</Error>

I want to know what is the difference between SDK and CLI execution, which causes the error?

After repeated attempts, I found that OSS object storage would report an error about inconsistent signatures SignatureDoesNotMatch, but when I used the SDK to write rust code with the same function, it could be executed normally：
leolong@dc:frostbow-jankaul$ AWS_ACCESS_KEY_ID=*** AWS_SECRET_ACCESS_KEY=*** AWS_DEFAULT_REGION=oss-cn-beijing AWS_ENDPOINT_URL=https://leolongossbucket.oss-cn-beijing.aliyuncs.com AWS_ALLOW_HTTP=true target/debug/frostbow -u s3://
> CREATE TABLE iceberg.public.people (
    id BIGINT PRIMARY KEY,
    name VARCHAR(255),
    age INT
)
STORED AS ICEBERG
LOCATION '';
External error: The operation lacked the necessary privileges to complete for path iceberg/public/people/metadata/v0.metadata.json: Client error with status 403 Forbidden: <?xml version="1.0" encoding="UTF-8"?>
<Error>
  SignatureDoesNotMatch
  <Message>The request signature we calculated does not match the signature you provided. Check your key and signing method.</Message>
  <RequestId>6769315299F00D323592A2F7</RequestId>
  <HostId>leolongossbucket.oss-cn-beijing.aliyuncs.com</HostId>
  <AWSAccessKeyId>***</AWSAccessKeyId>
  <SignatureProvided>5c514dedac769963f329c87f4cac093148cfbc2db1e757b570be9a0f0195728c</SignatureProvided>
  <StringToSign>AWS4-HMAC-SHA256
20241223T094554Z
20241223/oss-cn-beijing/s3/aws4_request
9281ed4495565fb17d18a399fc7ab696cd19e080a9e958a4ae49dd811ef41957</StringToSign>
  <StringToSignBytes>41 57 53 34 2D 48 4D 41 43 2D 53 48 41 32 35 36 0A 32 30 32 34 31 32 32 33 54 30 39 34 35 35 34 5A 0A 32 30 32 34 31 32 32 33 2F 6F 73 73 2D 63 6E 2D 62 65 69 6A 69 6E 67 2F 73 33 2F 61 77 73 34 5F 72 65 71 75 65 73 74 0A 39 32 38 31 65 64 34 34 39 35 35 36 35 66 62 31 37 64 31 38 61 33 39 39 66 63 37 61 62 36 39 36 63 64 31 39 65 30 38 30 61 39 65 39 35 38 61 34 61 65 34 39 64 64 38 31 31 65 66 34 31 39 35 37 </StringToSignBytes>
  <EC>0002-00000429</EC>
  <RecommendDoc>https://api.aliyun.com/troubleshoot?q=0002-00000429</RecommendDoc>
</Error>

I want to know what is the difference between SDK and CLI execution, which causes the error?

After continuous attempts, I found that I can use frostbow-sql to read and write OSS ICEBERG table data. There is little description of frostbow-sql in the document. I look forward to the subsequent supplement in this regard. Thank you for your contribution and answer, I benefited a lot.


java.lang.ClassNotFoundException: org.acme.mybatis.config.PageConfigurationFactory

quarkus version : 2.9.2.Final
quarkus-mybatis version : 1.0.3
When I use quarkus. Mybatis. Configuration-factory to configure a custom ConfigurationFactory, then start the application throws java.lang.ClassNotFoundException

I think the reason for the problem of the product is that Class#forName( is used when loading the class. If Resources#classForName is used to load the class, this problem can be solved

@hjh7516 can you provide a re-producer or test? I can take a deep look at it. Thanks!

re-producer step：

Create a new class with an arbitrary name. For my example, the contents of this class are shown below

2.Use this class to configure in the configuration file(application.properties), as shown below:

3.Then start app
Expected behavior
The application can start normally
Actual behavior
Exception is thrown.
Caused by: java.lang.RuntimeException: java.lang.ClassNotFoundException: org.acme.mybatis.config.PageConfigurationFactory 	at io.quarkiverse.mybatis.runtime.MyBatisRecorder.lambda$setupConfiguration$3(MyBatisRecorder.java:344) 	at java.base/java.util.Optional.ifPresent(Optional.java:183) 	at io.quarkiverse.mybatis.runtime.MyBatisRecorder.setupConfiguration(MyBatisRecorder.java:340) 	at io.quarkiverse.mybatis.runtime.MyBatisRecorder.createSqlSessionFactory(MyBatisRecorder.java:83) 	at io.quarkus.deployment.steps.MyBatisProcessor$generateSqlSessionFactory1685105135.deploy_0(Unknown Source) 	at io.quarkus.deployment.steps.MyBatisProcessor$generateSqlSessionFactory1685105135.deploy(Unknown Source) 	... 16 more

@hjh7516 can you provide a re-producer or test? I can take a deep look at it. Thanks!


Support #![feature(unchecked_math)]

The following code:
#![feature(unchecked_math)]

pub fn nop(num: i32) -> i32 {
    unsafe { (num.unchecked_mul(2)) / 2 }
}

gives:
example::nop:
        lea     eax, [rdi+rdi]
        sar     eax
        ret

while cg_llvm gives:
example::nop:
        mov     eax, edi
        ret

because it seems cg_gcc always enable checked maths.

hello @antoyo assign this task to me

Done. Feel free to ask questions if you need help.

hello @antoyo, what part of the code is this issue located?

This issue is actually much harder than I initially though, so I removed the good first issue tag.
It would be in this part of the code and would require changes to libgccjit.
You might want to pick another good first issue if you prefer.

honestly, i would opt for another right now

thanks


Update creator-ee version to v0.14.1

Closes #1439

Merging this to get it out of the way w/o other approvals.


[BUG]

Built the project, Ubuntu 24 LTS on RPI4
The sample video project works fine but the video+audio one does not and errors out with:
[DEBUG] [11-06-2024 18:33:25:750.377 GMT] Stream is ready: ExampleStream
[DEBUG] [11-06-2024 18:33:26:243.545 GMT] appsink configured
[INFO ] [11-06-2024 18:33:26:369.404 GMT] Using video device /dev/video1
[INFO ] [11-06-2024 18:33:26:372.366 GMT] Using audio device hw:1,0

**(kvs_gstreamer_audio_video_sample:51265): GLib-GObject-CRITICAL **: 14:33:26.372: g_object_set: assertion 'G_IS_OBJECT (object)' failed
Not all elements could be created:**
[INFO ] [11-06-2024 18:33:26:376.668 GMT] stopKinesisVideoStreamSync(): Synchronously stopping Kinesis Video Stream 0000aaaad3977ce0.
[DEBUG] [11-06-2024 18:33:26:376.803 GMT] streamClosedHandler invoked for upload handle:<PHONE_NUMBER>3709551615
[INFO ] [11-06-2024 18:33:26:376.879 GMT] Freeing Kinesis Video Stream for ExampleStream

Note that it doesn't list the elements that could not be created.
export LD_LIBRARY_PATH= and export GST_PLUGIN_PATH= have been done.
The error happens regardless of using video0 or video1 (I have both the standard RPI camera and a USB camera attached).

@wimdecorte ,
If the sample video is working and not audio, perhaps there is an issue with an audio specific gstreamer element (maybe some RPi specific package that is not installed?).
The error comes from here:
https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/blob/master/samples/kvs_gstreamer_audio_video_sample.cpp#L775-L780


Description fixes

https://mangadex.org/title/33537

https://mangadex.org/manga/36075/

https://mangadex.org/manga/36075/

https://mangadex.org/title/47406

https://mangadex.org/title/47406

https://mangadex.org/title/60349

https://mangadex.org/title/60349

https://mangadex.org/title/43725

https://mangadex.org/title/43725

https://mangadex.org/manga/44992/


⚠ Bofrost.fr has degraded performance

In 7c36268, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 861 ms

Resolved: Bofrost.fr performance has improved in e8f9a95.


glslang: fix build on riscv

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[x] rsicv64-linux (cross)
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

Confirmed that it actually works on gcc13.


Allow to use write targets allocation via followers in proxying chunk service

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Resolved: https://github.com/ytsaurus/ytsaurus/commit/c8141a87f252754efefe2cb6910d38a5177082ce


index 2 is out of bounds for axis 0 with size 2

HI,this is a great work. When I run inference, error exists:
radius molecule: mean 7.5976667404174805, std 0.0, max 7.5976667404174805
distance protein-mol: mean 12.849800109863281, std 0.0, max 12.849800109863281
rmsd matching: mean 0.0, std 0.0, max 0
common t schedule [1.  0.5]
Size of test dataset:  1
0it [00:00, ?it/s]Failed on ['data/7rfw_receptor.pdb____data/7rfw_ligand.mol2'] index 2 is out of bounds for axis 0 with size 2
1it [01:09, 70.00s/it]
Failed for 1 complexes
Skipped 0 complexes
Results are in results/user_predictions_small
[2]-  Killed                  python -m inference --protein_path data/7rfw_receptor.pdb --ligand data/7rfw_ligand.mol2 --out_dir results/user_predictions_small --inference_steps 20 --samples_per_complex 40 --batch_size 10 --actual_steps 18 --no_final_step_noise
`

Hi!
I would recommend raising the exception where it is caught for debugging the problem.

Closing the issue for inactivity, feel free to reopen it once you have a stack trace of the exception!


Automated cherry pick of #1059: 避免未设置hypervisor导致创建磁盘失败

Cherry pick of #1059 on release/2.10.0.
#1059: 避免未设置hypervisor导致创建磁盘失败

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@dec1485). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #1060   +/-   ##
================================================
  Coverage                  ?   7.92%
================================================
  Files                     ?     356
  Lines                     ?   56353
  Branches                  ?       0
================================================
  Hits                      ?    4464
  Misses                    ?   51495
  Partials                  ?     394

Flag
Coverage Δ

#aFlag
7.92% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/hostdrivers.go
18.18% <ø> (ø)

pkg/compute/models/disks.go
0.78% <0%> (ø)

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 dec1485...62baf97. Read the comment docs.

/lgtm
/approve


Reformat term list table to Title (Name)

https://genomicsstandardsconsortium.github.io/mixs/term_list/ should have two columns - Title (Name) & Description.
See here for how the changes from this PR will affect the documentation: https://sujaypatil96.github.io/mixs/term_list/

Make decision on whether to merge (or not) this PR on 4/2/24.

Accepted at call on 4/2/24.


Game View: display number of cards remaining in deck

Not necessary to basic game function, but it would be very nice to add.

Maybe restyled, but logic is in


Remove obsolete uses of typing.*

Oh, interesting that you can use type as a generic type, maybe you should replace the other Type uses?
Originally posted by @leandro-lucarella-frequenz in https://github.com/frequenz-floss/frequenz-sdk-python/pull/422#discussion_r1219819082

I want to eventually add pyupgrade to repo-config, which should do this automatically: https://github.com/asottile/pyupgrade#pep-585-typing-rewrites


Loading layout from Xml get IndexOutOfRange

Sometimes when loading layout from Xml an IndexOutOfRangeException is thrown. It always happens on Mono, only sometimes in Windows.

Pull request coming as soon as #431 is in place (currently "in progress")


What does parse_block_html and parse_inline_html mean?

I don't understand the description:

parse_block_html: parse text only in block level html.
parse_inline_html: parse text only in inline level html.

with parse_block_html=True

**text**

will be:

text

Sorry, I must be really slow, I still don't get it 😕

parse_block_html=True: parse text with markdown gram in block level HTML tags.
parse_inline_html=True: parse text with markdown gram in inline level HTML tags.


code release

Thanks for your great work and congratulations on the acceptance! Do you have any detailed timeline for the code release? I can't wait to run it!

We plan to release the code within the next week. Please hold on for a moment!


ci: Test with dependency pre-releases

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

Project coverage is 100.00%. Comparing base (90e367a) to head (f47c094).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #22   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           86        86
  Branches         8         8
=========================================
  Hits            86        86

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


OneTimeWatch

What is the property oneTimeWatch in angular batarang?
When I use ui bootstrap and open a modal it increase to 400

This is a question for StackOverflow


[Feature]: Refactor database resources to no longer cycle passwords (ENG-551)

Describe the feature
Description
Historically we never stored the credentials for DBs and this meant we had to cycle the credentials for every new deploy.  But since we now stored the credentials, we no longer need to cycle them.  Cycling the RDS credentials also takes 1+ minutes.
Requirements:

refactor the functions for provisioning a database user in provisioner/lib.rs so that if a user already exists the password is no longer cycled
Update deployer to always send down the details of persisted resources

Suggestion or Example of how the feature would be used
@Shariffintech and I will be working on this
Duplicate declaration

[X] I have searched the issues and this feature has not been requested before.

@oddgrd could you also add me as a participant on this ticket?

Hey @Shariffintech and @ankitpatel96, are you still interested in this issue? If not I'll just open it up again :slightly_smiling_face:

Hey! So this issue has been resolved, we will now get the resource information from the resource-recorder if it already exists, meaning the call to provisioner that lead to the cycling of passwords no longer occurs. However, we might need to create a new command that allows users to cycle their database password, in case their password is leaked. We're currently working on resource deletion, but we'll discuss this internally and open a new ticket as soon as we have the capacity to review it.

🫡 Thank you for the update @oddgrd! I'm glad to hear that the problem has been addressed. Looking forward to contributing to the new command for resource deletion whenever it's the right time, happy to assist.


Send Audio Messages

Layout for the "hold to record" microphone already there
Add FooterViewState to RecordAudio

Check permissions to record audio (only start recording if they are granted)
Records audio into a temp file
Audio format to be .amr
When recording is cancelled temp file is deleted
When recording is done temp file is added as an media attachment and uploaded using the existing logic (might need to generalize the image upload logic)
Update the footer to show that recording is in progress.

All layouts to match iOS version

@kngako swiping left on footer view while recording will cancel audio


path converter

fixes #405

@davidism  could we get some eyes on this please?  Pathlib is great and i'd love to use it more.

@davidism could you please explain why a request for review for this PR was deemed off-topic?  I'm very confused.

It's nothing against you or this PR. I have a ton of priorities, and randomly popping in to one out of hundreds of issues and asking why I haven't reviewed this one yet isn't particularly productive, and multiply that by many more people who do that on other issues at the same time. Even if I review it and merge it right now it won't actually be usable until another release is made. I or another maintainer will get to it when we can get to it.

Is there any progress on this?

missing tests when path is a bytes instead of str.

@SoniEx2, when does that happen?

POSIX.

sys.argv is always str in Python 3, which is all master supports now.

but paths aren't always valid unicode. how is that handled by click? I just looked and paths can still be bytes, or at least the documentation still says so.

@SoniEx2 as in my example you ignored in #python...  surrogate escapes afaik.
print(os.fspath(pathlib.Path(b'foo\xff\xff\xff\xfd\xfd\xfd'.decode(errors='surrogateescape'))))

Well, os.fsdecode(), but the surrogate escapes is the interesting part.
https://docs.python.org/3/library/sys.html#sys.argv
https://docs.python.org/3/library/os.html#os.fsdecode

It's Python, not Click, that is treating sys.argv as unicode. This has been documented for quite a while in Click: https://click.palletsprojects.com/en/7.x/python3/#python-2-and-3-differences

sys.argv is always Unicode-based. This also means that the native type for input values to the types in Click is Unicode, and not bytes.
This causes problems if the terminal is incorrectly set and Python does not figure out the encoding. In that case, the Unicode string will contain error bytes encoded as surrogate escapes.
When dealing with files, Click will always use the Unicode file system API calls by using the operating system’s reported or guessed filesystem encoding. Surrogates are supported for filenames, so it should be possible to open files through the File type even if the environment is misconfigured.

Based on this, it sounds like Click doesn't need to do anything extra.

that's for File, not Path. Path still claims "str or bytes".

That's an issue with the docs, not with Click. The docs were very focused on Python 2, which is what had the difference in this case. Fairly sure I updated them in master already.

the PR just grabs the former result and optionally wraps it. since the former result can be bytes (according to the docs in master), this could be a problem.

This is getting really frustrating. I feel like either you're not understanding what we're saying, or you're not effectively communicating the issue you have with this PR in a way that could be addressed by Click.
The fact that pathlib doesn't accept bytes, even though the filesystem may be bytes, is an issue (or design decision) of pathlib. There's nothing Click can do about it, all it sees is a function (not necessarily pathlib.Path, it's generic) to pass a value to. It seems like a non-issue as well because Python only sends sys.argv as str, which is what Pathlib accepts. Again, if the docs say it sometimes returns bytes, that's an issue with the docs still referring to Python 2 behavior.

@SoniEx2 since you already found something you think is a problem, could you link it so we know where you found it?  We like to fix things (well I do, I presume @davidism does as well...) but it's a pain to have to dig up what you claim is wrong when you haven't even said what docs you have a problem with.

... meh. I think the problem is that click.Path and pathlib.Path are both called Path tbh.
but just taking the return value of click.Path and passing it to pathlib.Path, when click.Path is documented (in master) as returning bytes sometimes, sounds like a recipe for disaster.

@SoniEx2 'master' is a big space.  Where is the documentation?  Unlike your system GitHub allows you to link to specific lines of code.  Use that feature.  Let us know where the problem is.  I'm not questioning whether an issue exists but I know it's easy to miss stuff.  Another PR here I worked on involved at least three different ways a thing was implemented/documented...  lots of confusion there as people said 'it says x' and someone else said 'it says y' and another said 'it does z' and all were correct.  So, what exactly are you talking about?  What line of documentation in what file?

https://github.com/pallets/click/blob/master/docs/arguments.rst#file-path-arguments


[Starting view. City field] Duplicate values are displayed in the 'City' drop-down menu after changing the language to English

Environment: Windows 11
Chrome 107.0.5304.63
Reproducible: always
Build found: last commit from https://github.com/ita-social-projects/GreenCityClient/tree/dev
E.g. "commit 147fa44"
Preconditions
User is logged in as Admin.
Tariffs panel is opened:[ https://ita-social-projects.github.io/GreenCityClient/#/ubs-admin/tariffs|https://ita-social-projects.github.io/GreenCityClient/#/ubs-admin/tariffs]
EN locale is set for the page.
Steps to reproduce

Click on the  'City' field
scenario 1
Type Lv data

scenario 2
2. Type Dr data

Actual result
The same names of the cities are displayed in the 'City' drop-down menu after changing the language to English
Attached bug screenshots recorded.
Expected result
An unique values are displayed in the 'City' drop-down menu after changing the language to English
User story and test case links
E.g.: "Starting view for the pricing page ("Тарифи") #3835
Test case"
Labels to be added
"Bug", Priority ("pri: medium"), Severity ("severity: medium"), Type ("Functional", "Localization")

I checked, and most likely, it is a bug in the frontend part. The request to the backend does not duplicate the data. It is sent after the localization change and repeatedly adds the data to the dropdown.


Add CSS media query for dark mode

This adds a CSS media query for dark mode.
It makes the toolbar dark with bright icons.

Please merge!

I don't think we should do this, could lead to very unexpected results on user's websites.

Okay, then I'll close this PR.


Frontend initialisation getting out of hand

Too many arguments to NewFrontend.  Need to find a better way.

The best way to do this is to remove the test-only arguments from NewFrontend, which makes the new signature of minimally required arguments:
func NewFrontend(ctx context.Context, baseLog *logrus.Entry, _env env.Interface, db *database.Database, m metrics.Interface, cipher encryption.Cipher) (Runnable, error)

The test-only arguments can then be provided in the tests in one of two ways:

Create a new wrapper newFrontendForTest that takes the original amount of arguments. This will simply shift the issue from frontend/frontend.go to frontend/frontend_test.go. Not ideal if this issue remains as-is.

Create an enrich function in the tests that takes a frontend instance (from the shortened NewFrontend, above) and the test-only arguments and sets those fields in the struct. This issue would then be better named "don't pass test-only arguments to NewFrontend".

I'm leaning towards the second, even though it will result in more code.
I'll put up a PR for each and see what everyone thinks.

I think we need to investigate what can be initiated inside the function. If this will not save us enough, we can create frontendOpt struct as part of Frontend pkg and pass all the same values as one struct.

I don't really think that either of these options is worth the churn.  We know we have so much boilerplate in the frontend tests and they are hard to understand and debug.  Working to reduce the boilerplate, and hence probably the number of calls to NewFrontend, and improve the legibility, is the first step here.


Build instructions do not work with mono 074873ab99fa14c3407d2563f833573ec796d7d7

The instructions state to build/install mono hash 73bb10fff3d7b828c3508c4ab34126baa8345bd9 or newer. I followed the instructions I think exactly and I get this error when compiling csc:
Emit/EditAndContinue/EncHoistedLocalInfo.cs(16,16): error CS0568: Structs cannot contain explicit parameterless constructors
I changed EncHoistedLocalInfo from a struct to a class to get past this compiler error and csc now successfully builds for me. Perhaps changes to mono between 0748  and 73bb now correctly enforce the limitation that structs may not contain explicit parameterless constructors as per http://msdn.microsoft.com/en-us/library/aa288208(v=vs.71).aspx ?

+1

Hello,

The instructions state to build/install mono hash
73bb10fff3d7b828c3508c4ab34126baa8345bd9 or newer. I followed the
instructions I think exactly and I get this error when compiling csc:

The hash has been updated and it builds fine for me.
Thanks
Marek

I just ran into the same problem above, I'm using a Raspberry pi2 (gentoo) with mono <IP_ADDRESS>
In order to get csc.exe to compile from this repo I had to make the same change for a struct to a class

This fork is no longer maintained. Original Roslyn repo https://github.com/dotnet/roslyn now supports Mono build.


Updated @threshold-network/solidity-contract to use 'development' tag

This way, the most recent version of the package will be installed.
It should also help us with resolving the compilation bug that was fixed
in https://github.com/threshold-network/solidity-contracts/pull/97.
See https://github.com/keep-network/keep-core/pull/2981

Reverted in https://github.com/keep-network/keep-core/pull/2985.


running dig *.mydomain.com returns error

After following the whole process on getting to the spot where i should run dig *.mydomain.com it returning an error, please what do you think could have cause and how do you think i can solve it

r

make sure that your firewall is not blocking port 53 tcp/udp.


OCPBUGS-6014: manage-dockerfile: use the original form of HEALTHCHECK

When reconstructing the Dockerfile, use the original version of a HEALTHCHECK instruction, rather than incorrectly reassembling all of the arguments into a single JSON array.

/retest

/retest

/retest

@coreydaley just to set my expectations here, is the e2e-aws-ovn-proxy test expected to pass?

/retest

/skip

/approve
/lgtm

/hold
@nalind It is not a required test, so I have skipped it. If you are confident that the failures are unrelated to your pull request, please remove the hold on this pull request.

/jira refresh

/hold cancel

/jira refresh

/jira refresh

We were having some api issues with github/jira integration yesterday. I have manually updated the JIRA to the modified state.
Does this need to be backported?

/assign @jitendar-singh
for QE

This one includes a unit test update, so I think the risk is low, and 4.10's going to be in maintenance phase for several more months, so I FWIW lean toward backporting it.

/cherry-pick release-4.12


Fixed nav for get started page and links for sub pages

Signed-off-by: Dominik Wilkowski<EMAIL_ADDRESS>

LGTM.


Pulsar Operator On Kubernetes

Add pulsar operator for kubernetes(https://github.com/sky-big/pulsar-operator) based on deployment/kubernetes/*.yaml

Welcome everyone to contribute

Basically realized the function of pulsar operator，include pulsar dashboard, grafana, prometheus and so onPulsar Operator，weclome to use and contribute.

Next step concern pulsar stateful component(include Bookeeper and Zookeeper).

Great initiative! I know you have to start somewhere and cannot do everything at once but have you thought about CRD's for Sources and Sinks? What about configuration like Channels, Tenants and Namespaces?
I would really like to get out of an imperative approach based on pulsar-admin for managing the cluster. Still trying to figure out how to get the whole configuration for a geo-replicated cluster in Git...

I would really like to get out of an imperative approach based on pulsar-admin for managing the cluster.

Maybe the pulsar-manager is a good idea for you.

@wolfstudy thanks for the tip (I am aware) but how is pulsar-manager not imperative?
A web UI where you do point-and-click is virtually the same thing as issuing commands on the command line. At each instance in time you tell the system what to do instead of telling it the desired target state and have it figure out the changes by itself.
In the end I would like to declare the state of my cluster in an external git repository and then have a CI/CD pipeline provision it for me (gitops).
The CRD proposed by @sky-big  is a really good start and I could work with this towards my end goal, but I need to declare other parts of the configuration as well. Point-and-click will not do the job :-)

@chrsoo Cloud you please give a sepcific example based on your sence? then we can add this function in the pulsar operator.

The difficulty here is where to draw the line between more or less static configuration and what should remain dynamically defined at runtime by the clients/users of the system. This might vary in different contexts.
I think it could be useful to define CRDs for the following Pulsar configuration items:

Tenants with namespaces and their relation to Clusters
Persistent Topics and their relation to tenants/namespaces
Sources and Sinks and their relation to clusters/topics

Starting with Tenants the following CRDs might be useful

PulsarTenant - defines a Pulsar Tenant
PulsarNamespace - defines a Pulsar Namespace for a Tenant
PulsarNamespaceBinding - binds a Pulsar Namespace to a PulsarCluster

It would be good if Tenants and Namespaces are defined at the K8s cluster level as a Tenants and Namespaces then could be reused across multiple PulsarClusters defined in the different K8s namespaces. The PulsarNamespaceBinding should  be defined at the namespace level and in the same namespace as the PulsarCluster to which it is bound.
@sky-big - does all this make sense to you?

@wolfstudy  @sijie  @jiazhai  Can you disscuss it together?

@chrsoo I generally understand what you mean, Starting with Tenants CRDs, Can you make a specific design and let everyone discuss？

@chrsoo @sky-big @wolfstudy I think what @chrsoo request is an operator for the full lifecycle management for all the resources within a Pulsar instance. This includes both ClusterOperator and ResourceOperator.
A ClusterOperator is what @sky-big is doing and most of the operators are doing. It is used for setting up a cluster and managing the upgrade, downgrade and scaling up-and-down a cluster.
A ResourceOperator is more about using CRD for provisioning resources within a Pulsar cluster. A resource such as a role (key and tokens), a tenant, a namespace, a topic, a function and etc.
Once everything is modeled as CRD, you can use kubernetes to provision both cluster and resources. Thus you can apply a GitOps patten in a CI/CD pipeline to manage everything.
@chrsoo : this is kind of aligned with what we (StreamNative) are working on. We shall be able to share what we have done soon.

@sijie yes, exactly. Should we take your comment regarding StreamNative to mean that you are planning to open source an Operator for Pulsar on Kubernetes that deals with the full scope of CRD's for Pulsar management? How soon is "soon"?
@sky-big regarding a specific design I would design it as explained above but I will see if I can produce a more detailed design specification as a PR.

@chrsoo OK, Looking forward to you PR.

Operator add pulsar manager component success.

The next step is to publish to OperatorHub

@sijie yes, exactly. Should we take your comment regarding StreamNative to mean that you are planning to open source an Operator for Pulsar on Kubernetes that deals with the full scope of CRD's for Pulsar management? How soon is "soon"?

@sijie any feedback on this?

@sky-big regarding a specific design I would design it as explained above but I will see if I can produce a more detailed design specification as a PR.

@sky-big  sorry for not getting back to you, the pulsar project took a back-seat for while where I work. It is getting reprioritised again so I will start looking into that PR again. Note that I am not talking about code at this point but at least I should be able to put how I think it should work.

@chrsoo OK

I created slack https://pulsaroperator.slack.com/

Why not just an #operator channel in the Pulsar Slack? We hang out in there so not sure if this is big enough effort to need a whole new Slack.
Having a design spec laid out is a great idea because we are starting to build something similar. If we have an agreed upon approach, members from my team could contribute functionality to flesh out the implementation.
Also we can perhaps keep separate Operators as Strimzi did for their Kafka Operators. This would allow the current big-sky Operator to exist for managing the cluster and my team could create another operator for managing the User,Topic, Tenant, Namespace type objects.
https://strimzi.io/docs/overview/latest/#overview-components_str

@nicknezis Hi,
Sorry, I didn't know there was pulsar slack，now, I created one pulsar operator channel in the Pulsar Slack，We will discuss the following here.
There's another team looking at the same thing, we can discuss it together here. We've been talking about priorities for all function points, and we've been talking about rules.

@nicknezis I‘ve invited you to join the channel

Hi all,
I'm very much interested in the discussion revolving around the Cluster and Resource Operators for Pulsar.
So, kind of hope that the reason this ticket went quiet is that you moved the ongoing convo over to Slack?
Is there any update from that side that would make sense adding here?


Async check version.

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
Use callback.
Describe alternatives you've considered
https://github.com/iceBear67/CraftPP/blob/master/src/main/java/com/github/icebear67/craftpp/CraftPPLoader.java
checkUpdate() should use async callback
Additional context
Nothing.

http请求多起来后在考虑使用callback.因为需要额外开一个线程池


CLM-7667-Helium-Error-Boundaries

As a 3rd party developer who has successfully deployed my Helium app, when I visit the deployed app and it’s not working, I should see error logging on the runtime issues.
This can be don't by adding two things:

React ErrorBoundary on the client side
React Suspense on the server side

Moved "react-error-boundary": "^4.0.4" to be in the correct package files
Added the same changes to the hello world theme

Fix: react-error-boundary is now in the correct order in both package.json files


Added Designer.io Application

Should move it down in the array again since I renamed it to Gravit Designer instead of Designer.
Will fix that later.

Apps need to be added at https://github.com/electron/electron-apps#adding-your-app, not here.

@50Wliu I see, thanks!


bananagiveaway.com not working

On https://www.bananagiveaway.com, it adds the top bar but doesn't add any buttons:
Example: https://www.bananagiveaway.com/giveaway/19

Should be fixed now


Write initial test plan

Duplicated through Issue #29, so I'll close this one and we'll use that one


Update notes for AWS object storage read/write

Closes #177
@sfoks Can you read these docs and let me know if they help you with getting access to S3?  Feedback welcome.

I guess there are different angles to present this information, but I was thinking that perhaps it makes sense to talk about interacting with the S3 API instead of AWS S3 and OSN separately.   For the user the only difference between Amazon S3 storage and OSN (or other providers like Wasabi, Cloudflare, XSEDE, etc) is the endpoint-url for example:

AWS S3 us-west-2: https://s3.us-west-2.amazonaws.com
AWS S3 us-east-1: https://s3.us-east-1.amazonaws.com
OSN RENCI pod: https://renc.osn.xsede.org
OSN MGHPCC pod: https://mghp.osn.xsede.org

All the other commands/parameters are the same!

Thanks for humoring all of my suggestions

As an idea to speed the review process along... typos or small additions don't need to go in comments that I commit.  You can edit and commit directly to the branch to fix those problems. Might save some back-and-forth.

As long as you've reviewed the final version and you're ok with it, it's ready to merge @gzt5142


[Python] Add support for GCS URI (gs://) to pyarrow.parquet.read_table

Describe the enhancement requested
Currently, this works:
from gcsfs import GCSFileSystem
import pyarrow.parquet as pq
gcs = GCSFileSystem()
parquet_file = pq.read_table("/bucket/path/to/file.parquet", filesystem=gcs)

But this does not:
import pyarrow.parquet as pq
parquet_file = pq.read_table("gs://bucket/path/to/file.parquet")

It would be nice if the latter worked directly without needing to specify the filesystem.
Component(s)
Parquet, Python

Does this work?
import pyarrow.dataset as ds
dataset = ds.dataset("gs://bucket/path/to/file.parquet", format="parquet")
parquet_file = dataset.to_table()

Yes.

No.
In both cases the following error is generated:
pyarrow.lib.ArrowNotImplementedError: Got GCS URI but Arrow compiled without GCS support

How did you install your PyArrow? wheel? conda?

Wheel file through poetry.
It is resolved to use pyarrow 12.0.1.

Which wheel are you using?
Wheel list: https://pypi.org/project/pyarrow/#files

pyarrow-12.0.1-cp38-cp38-win_amd64.whl
Listed here for 12.0.1: https://pypi.org/project/pyarrow/12.0.1/#files

OK. pyarrow 13.0.0 will solve your problem.
See also: #35255/#35193

I confirmed that it does. Thank you so much for the help.
I did search for the issue before I reported it, but maybe I was only looking at open tickets.


update readme?

No such file or directory -- scripts/build.rb
this got replaced with ec2_elb_start.rb perhaps?

whoops.. sorry, I was in an older branch. Closing


chore(github): Replace feature request as issues with ideas as discussions

Description

Problem*
Noir's feature requests are currently collected and managed as GitHub Issues.
This is not ideal as:
Suggesting "Issues" is unintuitive
The term "Issue" could carry the meaning of "problems / bugs", which leads to GitHub Issues not necessarily being the first place users would check if they would like to raise a suggestion.
Upvoting is unintuitive
The thumbs-up emoji can be used as a pseudo upvote, but it is hidden behind the "reaction" smiley face and doesn't show up in the Issue list unless explicitly sorting with it.
Mental barrier
Filing a GitHub Issue often comes with a mental barrier that it should be logically-detailed and well-written, which might deter users from creating a suggestion.
Summary*
GitHub Discussions could potentially be a better place for collecting and managing feature requests.
This PR:

Removes the existing feature request GitHub Issue template (in favor of the org-wide ideas GitHub Discussions template)
Adds an "Ideas" contact link in the list of actions on the "New Issue" page (for users that are used to / prefer creating feature requests through the creating GitHub Issues workflow)

Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context
For a glimpse of how this would look like in action, checkout the demo noir-lang org: https://github.com/noir-lang-test/noir/issues/new/choose
For ideas that contributors would like to work on, they can easily convert discussions into GitHub Issues and follow up with the usual GitHub engineering workflow:

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

This is a bit of a pain as I can no longer create feature requests issues directly and get pushed towards spamming the discussions page or make do without the template.

Right, it's a choice of tradeoffs between hosting Feature Requests as:

Issues (previous practice)
+ Can create executable pieces of work directly
- Unintuitive / deterring for the community

Discussions first, then convert to Issues when executing (as per this PR)
+ Eases community engagement
+ Clear boundary between what is welcoming votes & pending for contribution (Discussions) <> what is already in execution (Issues)
- 2 additional clicks to create executable pieces of work

Support both (1) and (2)
+ Can create executable pieces of work directly
- Potentially more confusing than (1):

Multiple paths to request feature through "New Issue" (as an Issue or Discussion)
Vague status of open Issues: are they welcoming votes or already in execution
Mixed voting signals for prioritising works (e.g. are 3 upvotes on Discussion equivalent to 3 thumbs-up on Issue?)

Certainly jumped the gun and assumed (2) gives us the best set of tradeoffs. Would you consider the additional friction justifies a switch back to (1)?
Definitely open to adopting an option (4) that achieves the best of both worlds too, suggestions welcome!


Add support for Supabase VectorStore

https://github.com/supabase/supabase-flutter/issues/489#issuecomment-1628504192
https://cookbook.openai.com/examples/vector_databases/supabase/readme

Hi @davidmigloz , I'd like to take care of this task, is that OK?

Hey @matteodg !
Thanks for your interest in contributing 🙂
@luisredondo was going to start working on it (not sure if he started implementing it yet), but I see you already made quite some progress so go for it!

Actually, @luisredondo is telling me that he already started implementing it as well (my bad that I didn't update the ticket). It'd be good to try to sync/merge your changes and his changes. Are you in LangChain.dart Discord @matteodg? We can discuss it in the #contributing channel.

Hey @matteodg! I have been researching how to implement this appropriately and can offer some input if needed. We can sync via Discord. @davidmigloz provided the link above. You already made a good amount of progress, so I don't mind if you take over it. But here I am if you need a hand!

Hi @luisredondo! I'm basically converting to Dart from the Python LangChain (but actually the JS one might be a better source, easier for me at least as I'm more a Java dev). Let me know any input you have or any work you already have done. Thanks!

I found a few inconsistencies between Python LangChain and the JS one:

Python Langchain assumes the documents table has an id of type UUID, see https://python.langchain.com/docs/templates/self-query-supabase#setup-supabase-database
JS LangChain assume the documents table has an id of type BIGSERIAL and has the match_count in the match_documents function, see https://js.langchain.com/docs/integrations/vectorstores/supabase

I used the JS LangChain definition for the moment.

Hi @davidmigloz and @luisredondo, the PR is ready for you to review!
I slightly changed the match_documents functions: it now contains also the match_threshold (see the blog entry Storing OpenAI embeddings in Postgres with pgvector), so we can filter the results by threshold directly in Supabase instead of filtering them locally.

@matteodg  awesome, thank you very much!
Can you open a PR from your repo to this repo?
I'll review it tomorrow

Here it is https://github.com/davidmigloz/langchain_dart/pull/318

Released 🚀
https://pub.dev/packages/langchain_supabase/changelog#001
Docs: https://langchaindart.com/#/modules/retrieval/vector_stores/integrations/supabase


[C++] Support configure debug level logs simply

Motivation
For C++ client, if we want to see debug logs, there're two ways:

Build C++ library with CMake macro -DUSE_LOG4CXX=ON, and configure log4cxx.conf.
Custom the logger factory for Client.

The first way needs log4cxx dependency that is not easy to install. It's also why the USE_LOG4CXX option is disabled by default. The second way needs a lot of extra code. Sometimes, we just want to set logger's default log level to debug.
Modifications
This PR exposes the internal SimpleLoggerFactory and support configure debug level simply, for example
    ClientConfiguration conf;
    conf.setLogger(new SimpleLoggerFactory(Logger::LEVEL_DEBUG));
    Client client("pulsar://localhost:6650", conf);

Add tests for the Logger created by SimpleLoggerFactory::getLogger.
Verifying this change

[ ] Make sure that the change passes the CI checks.


code 299

Every time I restart the computer and open it again, a warning will pop up. But it works.
K32EnumProcessModules failed with code: 299
Complete only part of a ReadProcessMemory or writeProcessMemory request.

#186

thank you


In the calibration curves, only include questions past their scheduled resolve date

Since many questions can either resolve Yes-early or No-late, it will look like you have a Yes bias until you reach the time the long questions resolve No.
The solution is to only include questions in calibration once they've reached their scheduled resolve date.
There should be a note to that effect below the calibration curve.
The questions can still be included in the other graphs for now.

Linking pull request: https://github.com/Metaculus/metaculus/pull/1569

Finished the pull request. Ready for review @SylvainChevalier

Technically this issue is solved. Though I think we should explore further avenues with this (I'm not sure if the standard is to keep this issue open, or to make a new issue).
Things to explore further:

Make unit tests to triple check calibration curve math
Explore why the 2024 data is causing a calibration dip and data between 2020 and 2023 does not have this (bots are a suspicious cause)

Double check that bots are not included when creating the values in Recency Weighted AggregateForecast table rows
Triple check that the bots are in fact filtered out post-AggregateForecast caculation

Do a data dump of the forecasts in the 20-60% range and see if there are any patterns
See if removing 'possibly spam' forecasts helps - Filter out the forecasts made by people who have not been on the site longer than 1month or who have not made a comment or that were marked spam or that are inactive (these filters might already be applied)
Double check that the math for calculating recency weighted forecast is correct
Make sure that the forecasts used in the calibration curve are the same as the ones used from the global leaderboard

Other useful info:

Results found thus far and other ideas: https://metaculus.slack.com/archives/C52PD5LUS/p1733276259333379?thread_ts=1731959762.059169&cid=C52PD5LUS
Molly said to check this out and see if we can redo this analysis: https://forum.effectivealtruism.org/posts/ozPL3mLGShqvjhiaG/some-research-ideas-in-forecasting

@CodexVeritas thank you!! I'm going to close this. Can you start a discussion actually? with these notes of things to explore? I want to use discussions more.

Linking the discussion


Increment value

Not an issue but more of a question:
Could this be used to increment a value, for example a build number?
Something like
  "from": "__BUILD__ = \"(\d+)\"",
  "to": "__BUILD__ = \"{$1+1}\"",

I'm not sure what the syntax would be...
Thanks!

I believe that should be supported along with many different options available in lodash template. Please report back if you get something like that working!

@prisonerjohn Where you able to achieve what you want here?

Hey, I'm a bit of a newbie with all this and did not really understand what you meant :)
I followed the link to see what lodash is, but I'm not clear on how it fits into this plugin or semantic-release. I wanted to do a bit of research before writing back but didn't get around to it yet.
If you're cleaning up issues, you can close this and I can reopen when I get around to it.


TCOMP-71: add url handler for different platform to deal with the authorize url

Sorry, but I did not understand the purpose of this PR, now Nicolas explained it to me and this is not possible to resolve your problem this way.
You have to think of the Component being on a remote server side so you cannot display message to the user using some System.out (which you should never do anyway). You therefore cannot expect any interface of the Components to be implemented on the Client side (there are some rare exceptions for storage repository, but that where made on purpose).
I'll close this PR.


JDK 17 support

Is it possible to add JDK 17 support?
Maybe add this to Makefile:
build-jdk17:
	docker build --rm --force-rm -t odavid/jenkins-jnlp-slave:jdk17 $(DEFAULT_BUILD_ARGS) --build-arg=FROM_TAG=$(UPSTREAM_VERSION)-jdk17 .

And other changes for cicd and release/publish. Thanks.

Fixed by #31


Ubuntu 18.04 with mysql 5.7 - Fails to install

Followed the guide to install, but Installations fails!
--> Creation of iPSK Manager MySQL User Failed
--> iPSK Manager Installation Failure
The installation failed to finish correctly. Please re-run the installation again.

I solved the problem by uninstalling mysql, reinstalling and then it worked!


Update to Tree-sitter 0.15.5

This PR bumps the tree-sitter to 0.15.5, which contains the very latest version of the core Tree-sitter library.
We were already using almost the latest version, after yesterday's upgrade due to https://github.com/atom/atom/issues/19497, but this brings us up to master.
Small Behavior Change
There was a small fix to the behavior of the ts_node_descendant_for_position method in Tree-sitter, which slightly changes the behavior of some methods in Atom:
In particular, it fixes what I consider to be a bug in the behavior of the editor:select-larger-syntax-node where if you had your cursor between two adjacent tokens it would select the token before the cursor instead of the token immediately afterward.
It also fixes what I think was a bug in the bufferRangeForScopeAtPosition method. I've updated a test to reflect that change.
Prebuilt Binaries
Today I merged https://github.com/tree-sitter/node-tree-sitter/pull/50, which I think may make it so that the prebuilt linux binaries of tree-sitter will work for Atom, so that we don't have to manually delete those binaries to avoid weird hangs on Travis. 🤞

Closing in favor of https://github.com/atom/atom/pull/19530, because I forgot to name this branch in a way that would cause Travis to run.


Create test_pr.yaml

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


add to imaging week

Bruce Macintosh lecture:
https://www.youtube.com/watch?v=MBfV6msahdA

https://kipac.stanford.edu/discover