Sigma intrinsic and new whitening method summary

Coverage decreased (-2.4%) to 13.604% when pulling 3d529c2d25b9c8c079aac977c850c0ad4ebc2c12 on chto:add_sigma_intrinsic_and_new_whitening into f7f37a1a97cbd16e31363fc8baf076525c9bc280 on DarkEnergyScienceCollaboration:master.

Hi, @drphilmarshall
In this branch, I add two notebooks which are Sigma_Intrinsic_Analysis.ipynb and SLTimer_TDC2_NewWhitening.ipynb.
In Sigma_Intrinsic_Analysis.ipynb: I do an analysis on how sigma_intrisic affect the likelihood.
In  SLTimer_TDC2_NewWhitening.ipynb: I do the same analysis as in Sigma_Intrinsic_Analysis.ipynb, but change the whitening method.
The likelihood for new whitening method is strange and I am still finding the reason. But I think it is not likely to be caused by some bugs in the code, after checking the code again and again.
Therefore, I think it is a good time to let you know.
If you have any suggestion or comments, please let me know.  Thanks.

@chto Great work! It's hard to comment inline on notebooks, so I'll do it here instead.
Sigma_Intrinsic_Analysis.ipynb

Your plots show very nicely how the inclusion of the $\sigma_{intrinsic}$ parameter makes a huge difference to the log likelihood of the parameters given the data.
Look how sharp the peak in likelihood is with respect to $\sigma_{intrinsic}$ - you had to take logs twice! This is what I meant by "there's always more information in the data about the noise than there is about the signal". The sharpness of this peak is a potential problem, if its not sharp for every value of $\Delta t$ - but I suspect it is. Because the likelihood for $\sigma_{intrinsic}$ is so sharply peaked, a good strategy will be to first optimize  $\sigma_{intrinsic}$, and then sample close to this value and across the prior range for $\Delta t$. Can you try adding a few more curves to your plot of the log(log(likelihood)) for  $\sigma_{intrinsic}$ at fixed $\Delta t$, please? If you consistently find the same optimal $\sigma_{intrinsic}$ no matter what $\Delta t$ you are at, you can just pick one $\Delta t$ (nominally zero) and first estimate $\sigma_{intrinsic}$ that way.
The plot of log L vs $\Delta t$ at optimal $\sigma_{intrinsic}$ is interesting. Can you please try exponentiating all of log L, log prior and log posterior, and overlaying them on the same graph? (You'll need to manually renormalize L to unit integral for a good comparison). I want to see whether we have multiple peaks close to the true value (as well as far away from it).
The conclusion about log L not returning the true time delay when $\sigma_{intrinsic}$ is far from optimal is a good lesson; it's related to the way your plots show that even when $\sigma{intrinsic}$ is optimal_ there may still be times when maximizing the log L returns the wrong time delay. This is basically because while our new model is better, it's still wrong! Welcome to observational cosmology: all our models are wrong, but we care deeply about the cosmological (hyper-)parameters behind them. Every cosmological probe suffers from this issue: our task (in general) is to mitigate against this problem however we can. Ideas include: make better models; test against simulations to see how bad our models are; design robust data summaries that can be shown (with simulations) to reduce the impact or avoid entirely having a bad model; and so on.

SLTimer_TDC2_NewWhitening.ipynb

This whitening looks to be doing much better! The flux error bars are unchanged when seasonal=True, so I wonder: can we compare the absolute value of the likelihood in this analysis with that in your other Sigma_Intrinsic_Analysis.ipynb notebook, where seasonal=False? I think we might be able to. With new whitening, the log L values are around +1000, whereas in the other notebook they are more like -300 - so it does seem like the new whitening is a better model. You should think about whether or not this is a fair comparison though.
I think you should make the same plot showing prior, likelihood (renormalized) and posterior for $\Delta t$ at optimal $\sigma_{intrinsic}$ that I suggested above. I bet the posterior is a bit different to the prior, but it's just hard to see in the logarithmic scale plots. I agree that the log likelihood is strange, but it is mostly flat (on the scale of the prior) rather than weirdly peaked. So if there is an issue it might be that the whitening is somehow removing the signal we are looking for. Maybe seasonal is also the wrong approach?

OK, all these commits were also in your issue/42/initialization branch, which I just merged into issue/40/tdc2gatewaytest. Can you please pull this branch from the base repo and check that everything looks right to be your current version? Any problems, please issue them. Thanks!


Use network instead of host mode

What is the motivation for adding / enhancing this feature?
As mentioned in #3299 there is an issue with the current docker-compose configuration on OSX. I suggest to use external networks instead of the network_mode: host.
What are the acceptance criteria

[ ] network_mode: host is removed fro the docker-compose.yml
[ ] the api project network is added as external network to the docker-compose.yml

Can you complete this feature request by yourself?

[x] YES
[ ] NO

Which Release Cycle state this refers to? Info for developer.
Pick one option.

[ ] This is a normal feature request. This should be available on https://test.storefrontcloud.io and then after tests this can be added to next Vue Storefront version. In this case Developer should create branch from develop branch and create Pull Request 2. Feature / Improvement back to develop.
[ ] (Pick this option only if you're sure) This is an important improvement request for current Release Candidate version on https://next.storefrontcloud.io and should be placed in next RC version. In this case Developer should create branch from release branch and create Pull Request 3. Stabilisation fix back to release.
[x] (Pick this option only if you're sure) This is a critical improvement request for current Stable version on https://demo.storefrontcloud.io and should be placed in next stable version. In this case Developer should create branch from hotfix or master branch and create Pull Request 4. Hotfix back to hotfix.

Additional information
My suggested modifications:
version: '2.0'
services:
  app:
    # image: divante/vue-storefront:latest
    build:
      context: .
      dockerfile: docker/vue-storefront/Dockerfile
    env_file: docker/vue-storefront/default.env
    environment:
      VS_ENV: dev
# REMOVE this line
#    network_mode: host
    volumes:
      - './babel.config.js:/var/www/babel.config.js'
      - './config:/var/www/config'
      - './core:/var/www/core'
      - './ecosystem.json:/var/www/ecosystem.json'
      - './.eslintignore:/var/www/.eslintignore'
      - './.eslintrc.js:/var/www/.eslintrc.js'
      - './lerna.json:/var/www/lerna.json'
      - './tsconfig.json:/var/www/tsconfig.json'
      - './tsconfig-build.json:/var/www/tsconfig-build.json'
      - './shims.d.ts:/var/www/shims.d.ts'
      - './package.json:/var/www/package.json'
      - './src:/var/www/src'
      - './var:/var/www/var'
      - './packages:/var/www/packages'
    tmpfs:
      - /var/www/dist
# ADD this line
    networks:
      - vue-storefront-api_default
    ports:
      - '3000:3000'

# ADD this lines
networks:
  vue-storefront-api_default:
    external: true

in the configuration file localhost:8080 has to be replaced with app:8080


Added pre-onSubmit callback to attach 'called' state

Fixes #42

Just noticed that there's also #41  for this, so this fixes that as well.

This still lets the person click the button a bunch, and it probably won't work if the submit button fails (network issue, error, anything other than "successful") - the button will be disabled forever.
I would recommend using React Hook Form to do this, as it's already using it.
react-hook-form/react-hook-form#834 (comment)

While it does let the user click the button a bunch, I don't think that the fix fails if the other cases (e.g. network error) happen instead of a successful submission, as the 'called' state is set as soon as form is submitted. Do note that I did test this only for valid event creation.
I also actually did find that same thread that you've linked, but I couldn't get that implementation to work and tried this one out. I'll try the other implementation again, but I'm still not totally sure as to what the benefit is in comparison to this one.
On typing this out I now realize, and this might be what you're already saying, but there is still the issue of if a user submits a 'invalid' form, the 'called' state will be set as true, and it won't allow the user to correct the form for re-submission. This isn't too bad to fix with the current implementation.


Unable to use postConverter example for transforming data without TypeScript

I'm attempting to use the "postConverter" example provided in the the "Transforming data" section of the docs. I'm using Firebase v9 without TypeScript and I'm not sure how to implement the example. The only reason I want to use a the postConverter example is to easily get the id of a doc while still using the "useCollectionData" hook.
Sorry for making an issue, I just need some assistance on this. Thanks!

It doesn't seem to work. cause a dead loop or something, run out of quote quickly.

@squidee100 @railty I was able to use the converter but not exactly the way it is in docs.
You need to return a new class object from fromFirestore as follow:
class YourClass {
  customField
  ref
  id

  constructor(customField, ref, id) {
    this.customField = customField
    this.ref = ref
    this.id = id
  }
}

const converter = {
   toFirestore: (entity) => ({ customField: entity.customField }),
  fromFirestore: (snapshot) => {
    const data = snapshot.data()
    return new YourClass(data.customField, snapshot.ref, snapshot.id)
  },
}

I hope it works for you guys.

Be careful about referential equality when using converters @railty.
I ran into a similar issue with an endless loop of queries. It's because this package is checking if query is referentially equal when doing a query. The idea is that if you change query, then it should do another query with your new input.
When creating a converter inside a React component, you will get a new object created on each render, thus when you pass that into the firestore query, the query will constantly change on every render, thus triggering this package to run another query, and you end up in an endless render loop.
The solution is to always define converters outside of the React component (so it only gets defined once), or put it inside a memo, so it's always referentially equal.

Closing as this appears to have been answered. Please re-open if it is still an issue


Mount method is impacted from previous test function

Version
1.0.0-beta.26
Steps to reproduce
I write some test for my child component. But below mount function can not render child component, if above function use shallowMount. But when I change first test function to mount, below function work properly.
What is expected?
Function mount work properly
What is actually happening?
Function mount doesn't work properly

Here is my code:
test('test UI with create User', () => {
    const wrapper = shallowMount(AddUser, {
      localVue,
      store,
      mocks: {
        $router,
      },
    });
    console.log(wrapper.html());
    expect(wrapper.find('alertpanel-stub').exists()).toBe(true);
    expect(wrapper.find('userform-stub').exists()).toBe(true);
    expect(wrapper.find('button[id="addBtn"]').exists()).toBe(true);
  });
  test('add user with invalid input', async () => {
    const wrapper = mount(AddUser, {
      localVue,
      store,
      mocks: {
        $router,
      },
    });
    console.log(wrapper.html());
    wrapper.find('#addBtn').trigger('click');
    await flushPromises();
    expect(wrapper.vm.errors.count()).toBeGreaterThan(0);
  });

Thanks for the bug report.
Can you please post a full code reproduction (that I can run) in a GitHub repository.

Good day,
I think I have this problem too, since yesterday, after upgrade.
@vue/test-utils version 1.0.0-beta.26
I have a component with child component. Top level component, "Buddies", has nested component "BuddyCard". When top level component "Buddies" is mounted it looks more like shallowMount result.
Here is how I mount:
const cmp = mount(Buddies, {store, localVue, stubs: ['v-tooltip', 'v-menu']})

The result html (from cmp.html()) has component tag "":

    <buddycard buddy="[object Object]"></buddycard>

While I expect to have html code of "buddycard" component.
Thanks!

We have the same issue since @vue/test-utils version 1.0.0-beta.26

Can somebody with this issue post a full reproduction that I can run, ideally in a GitHub repo.

Same here in combination with Vuex. Tests that have nothing to do with Vuex now start failing. On inspection I find that in a test where I use vue router with localvue the router-link now automatically gets stubbed. When I disable Vuex it passes. This is on beta 25 even.


the chart should provide an extra volumes field.

Is your feature request related to a problem? Please describe.
there are several issues with the chart including broken css from a broken linking between the different containers/pods. (made an issue in erpnext main repo because its not only on kubernetes other users reported the issue on docker as well)
however today i went to install an extra app, i did try it as a custom job, then i did install it in a running pod.
Some pods started to crash because the lack of ~/frappe-bench/apps/$APP in it's filesystem (it's present on the pod i did install but not the others, like the scheduller)
So now i need to manually patch every single deployment to add a new pvc with the installed app
Describe the solution you'd like
I'd like to request an extra volumes field so we can add an extra pvc (or more, or even re-use the rwx sites volume)
and place installed apps in this pvc.
maybe i can even place the frappe and erpnext apps on it directly and solve the css mismatch issue...
Additional context
Add any other context or screenshots about the feature request here.

update:
it seems that there is a underlying issue that prevents the solution that i suggested, when mapping the pvc using the subpath
example:
  - mountPath: /home/frappe/frappe-bench/apps/persona
    name: extra-apps
    subPath: persona

when trying to install the app, bench tries to issue a rmdir which fails...

Additional volume is fine
Additional volume for apps will make the container mutable. Then you need frappe-bench/env common as well. It is not a good idea.
Any other volumes for any other locations is great. Will help as secrets for this #197


update symbol-sdk version to 1.0.3

fixes #1756

I tested quick smoke testes. Looks good. Full regression will be done in dev.


Only allow Openable content in Storage Access Framework

Add the CATEGORY_OPENABLE to the Intent to launch the Storage Access Framework (SAF) when sending a file. Setting this category ensures that the OpenableColumns data contract is met by the opened data, which we rely upon during data upload.
In practice, some content, such as Contacts, will no longer display in the SAF UI.
Fixes #935
@RocketChat/android

Could you resolve the conflict @MariusVolkhart ?

@MariusVolkhart Could you change the base branch from RocketChat:develop-2.x to RocketChat:develop?

No problem. I'll merge that branch with develop.
Thanks for this PR @MariusVolkhart!


The selection of input in form of number,text,boolean, label is inaccurately reflected on the centre block in the window

On the flyout area, under My blocks click on "make a block"
Add multiple inputs(text,boolean,label) on the block display on centre of the window.

As per Scratch,

The inputs are accurately displayed on the centre block in "Make a block" window.


chore(test): remove global test scheduler

Description:
This PR removes the global rxTestScheduler from the test infrastructure. To get this working it was necessary to also remove the tests for the global test scheduler.
In addition the custom mocha UI was also removed because it is no loger necessary.

This is great, @tmair ... it looks like there's a conflict though.

Thank you so much, @tmair !


Restrict access to the key, add helper methods to interact with Context.

I've just realized how badly OC got hit by the mistake to expose the Key and gRPC using directly the key, will be a bit mess to change that.

/cc @songy23 we need the same thing in oc-java and change gRPC to use this before we can migrate to openconsensus.

Filed https://github.com/census-instrumentation/opencensus-java/issues/1863 under OpenCensus-Java.


Correct ContextFactory link to the Twisted docs.

Coverage decreased (-0.2%) to 96.24% when pulling 97969287edd8de75e904fecb39140802d681eb33 on gruns:master into 1fb04622941b1579eb743380242cd82b340b75a5 on habnabit:master.

thanks for your contribution!


Poipole wc7 IVs read wrong

Hello,
I dropped the shiny Poipole .wc7 to RNG it today. The tool mistakenly read it as locked to 31s in Attack, Defense, Special Attack, Special Defense, and Speed. When I tried RNGing it ingame, it rolled a completely different spread with the usual 3 31s. I'm also using the latest commit.

I'll submit the final results later if needed.

Source code note: https://pastebin.com/bbpS6svw


The requested URL /files/4.0.1/dbeaver-ce-4.0.1_amd64.deb was not found on this server.

Just in case you aren't yet aware of this, but at least the 64-bit Debian package of 4.0.1 seems to be missing (haven't checked other versions).

Thanks for report,
fixed.


Bug: Predefined Input can not be assigned to InputSource

I just installed magic-regexp via yarn add magic-regexp and tried the first example from https://regexp.dev/getting-started/examples
Typescript does not like it though: Argument of type 'Input<"\\d", never>' is not assignable to parameter of type 'InputSource<string, never>'.
The code can also be simplified even more:
const regex = createRegExp(oneOrMore(digit));
I have created a codesandbox for showcasing this: https://codesandbox.io/s/gracious-swartz-3sdfcs

Addition: oneOrMore('x') fails with the same when used inside and,
i have updated the sandbox: https://codesandbox.io/s/angry-wave-fr0m5s?file=/src/index.ts
Code that fails:
createRegExp(   oneOrMore('a')   .and('.')   .and(oneOrMore('b')) )

You can see it working fine in stackblitz (here). It also works fine for me locally.

Are you getting this locally, or just in codesandbox (which seems to be using TS 4.4)? If so, what is your local TypeScript version?

I get this locally with 4.6.3 as well.

Yes - confirmed. You need features from TS 4.7+; versions below that show this error.

Excuse the question, but am I supposed to to something with the ticket?

@tim3trick No. For now you can use a newer version of TypeScript. I'll keep this open until I've had a chance to see whether it's possible to make the features of this library work with earlier versions of TS or whether I should instead mention it in the README.


Release Google.Cloud.Metastore.V1Beta version 2.0.0-beta05

Changes in this release:
New features

Added ScalingConfig (v1) (commit ca0d304)
Added Auxiliary Versions Config (v1) (commit ca0d304)
Added Dataplex and BQ metastore types for federation (v1alpa, v1beta) (commit ca0d304)

Created release for Google.Cloud.Metastore.V1Beta-2.0.0-beta05


add CommitOperationCopy

close https://github.com/huggingface/huggingface_hub/issues/1083

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

feel free to merge once it's good for you ;)


[Database] execute Base Model callbacks as hooks using the Hooks component

Right now the database base model executes pre and post callbacks, and the user has to define a pre/post callback callable in model properties. This makes it harder for the user to create general pre/post callbacks that would affect all models without introducing another layer of inheritance.
Exchange the callback system with the hook system, and call a pre and post callback. Include the object of the model($this) in the parameters for the hook call.

The format of the hook names should be: model.classname.before/after.modelmethod where classname is the base name of the class(without the namespace) converted to lowercase. Concrete example:

Model: \App\Model\User
Method: update
Before Hook: model.user.before.update
After Hook: model.user.after.update

Please also provide an option that the user can provide a specific hook name with a protected property. If that specific hook name is defined it must replace the classname part of the hook name.

Requires documentation update.


Copy Packages needs to improve the logging

What happened:

I would like the copy packages to specify how the destination registry should be provided.
I would like the copy packages to log the different phases that it goes through. i.e. Fetching images, Fetching charts, and so on.

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

EKS Anywhere Release:
EKS Distro Release:

https://github.com/aws/eks-anywhere/pull/5506


Question: rawCardinalSDKVerificationData is missing from verifyPayload

General information

SDK version: 3.85.3 (@types/braintree-web: 3.75.19)
Environment: Sandbox

Issue description
Hi!
Using the latest braintree-web and type versions in my Angular app, the ThreeDSecureVerifyPayload (coming as a response of .verifyCard()) does not include the rawCardinalSDKVerificationData object hence I can see it in the documentation (https://braintree.github.io/braintree-web/current/ThreeDSecure.html#~verifyPayload)
What am I missing?

I think the main issue was that @types does not necesserily follows braintree. I only missed the type, not the actual field.
I made a workaround and I do not have any questions in this topic. Thank you for your reply!


ANSI color codes in candidates

Is there any way to use ANSI color codes in candidates?
candidates=('HELLO' $(echo -e $'\e[36mWORLD'))

will output

Don't know how this happened. Duplicate of #101

Don't know how this happened. Duplicate of #101


Add remote tracking toggle to Source Control "Push To"

I frequently use the Source Control feature on VSCode and it's a bit painful not having an option to auto add a tracking reference for my local branch when I publish it to origin for the first time.
Could there be an option in the triple dot menu for push to inclue a "-u" flag on the push fired when I use source control?

When it's toggle all pushes should add a "-u" flag to tell git to create a tracking reference.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Update browse.blade.php

Show the display name instead of field name on browse.

Thanks for catching that man 👍


🔄 Synced file(s) with BetaHuhn/github-files

Synced local file(s) with BetaHuhn/github-files.

Changed files

Synced local  .github/workflows/release.yml with remote  workflows/versioning/release.ymlSynced local  .github/workflows/lint.yml with remote  workflows/node/lint.ymlSynced local  LICENSE with remote  LICENSE

This PR was created automatically by the action-github-file-sync workflow run #474147881

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


[Feature Request]: Allow calculating and then formatting values

Problem to solve
We have a case with two float properties, like price and taxes, and then we would like to sum the values, using the add function, and after that, format the result as currency.
Proposed solution
We tried to achieve that by using something like
{d.price:add(d.taxes):formatC()}
But the result is a NaN value.
Describe alternatives you've considered
We also tried the following
{(d.price:add(d.taxes)):formatC()}
hopping it would use the sentence between () before performing the formatting, but that seems to be invalid.

Thank you for taking the time to answer.
Yes, that's it... it worked like a charm.
About the documentation, yes, I think it could be improved in some way.
In this case, as I was looking at the formats section, I would never think about reading the dynamic section.
Maybe a simple mention with something like "for chaining formatters, check the dynamic-parameters docs"
I am going to close the issue, as you answer solved it.
Thank you so much again.


Move and rename /src/generated folder to /.yoga

@maticzav had the great idea to move and rename the /src/generated folder to /.yoga.
Additionally, we should encourage to add .yoga to your .gitignore file since the generation of the generated files should be deterministic based on the yoga NPM version.

Fixed, closing


Add perfect-numbers

I left this unmerged for your consideration: do you have the power to merge in this repo?

No, this rep is maintained so I don’t have write access as a cross-track reviewer.


Compile functions as async functions

ES2017 introduces async functions. It may be useful for elixirscript to produce async functions instead of normal functions. Is this something worth while? That is the question. It may or may not unlock the ability to implement receive at some point, but more research would be needed to know.

After experimenting a bit, this would be a bigger task than originally thought, but still may be a way to go

Reviving

Opening again. I've started working on this again in the async_switch branch

The async compilation branch seems to be at parity with master. Only problem I've seen so far is that with the Todo example app, there are issues with react's event pooling.

The good news is that compiling to async functions did work, but using async/await with a scheduler to achieve processes has not. I'm thinking about compiling to generator functions instead. Those are at least known to work as processes. My concern there is what to do when a javascript function expects a callback. I'm guessing some function that wraps around the generator will be needed in those cases

I think re implementing an actor model should be out of scope for elixirscript, maybe elixirscript_actor, it just seems in the same path as elixirscript_vdom,  the question is do you want a vdom and actor model baked in? it seems not good for maintainability and code separation to me.
Having everything an async function thought is different.  It would allow the code to function like its written (death to promises), perhaps with an extra await() on the elixir side.  If everything is an async function its a huge win, we can now write
r = async(API.Query())
r = async(r.stream())

VS

API.Query().then(fn(r)->
    r.stream()
end).then(fn(r)->
    r
end)

If async functions are working without schedulers it would be amazing.

I admit using it with the branch was nice, but the end goal is to support receive. Supporting receive is within the scope of ElixirScript and need a way to do that. If there is a way to do it with standard async/await than that would be optimal.
I am thinking about compiling to generators instead. Processes and receive all work in that case. The only thing would be to create a function that takes a generator function and return a function for using as callbacks with JavaScript things that require callbacks.
These are all long term plans. This probably won't make the next version and any changes should not effect the FFI at all.

I guess we have differing views on receive. I am thinking in terms of multiple backends to run the processes: generators, web workers, wasm future constructs for multithreading, etc.  Personally I would like to be able to plug and swap the "engine" vs being forced to have it all in 1 project.
Receive is more an engine construct related to the actor model vs a feature of the syntactical language.    It just seems to me the actor model implementation of elixirscript (receive) will be subject to a lot of tuning and performance tweaks, plus it looks like a project that would be useful in javascript land as a standalone.


Feature: Forgot username web form

It would be nice to have a web form to request a reminder of your username in addition to the existing Forgot Password page.

This issue has been moved to: https://github.com/chef/chef-server/issues/195
Thanks!


Recycle the agent when following a redirect

This fixes https://github.com/request/request/issues/1304. Passes all tests.

Any drawbacks to doing this?  Any thoughts on why it was done this way in the first place?
Can you think of an easy way to add a test for this behavior?

I can't think of any drawbacks.
No idea why it was done in the first place. The message from the commit that added that line offers no explanation.
You mean an easy way to test whether the agent is reused?

You mean an easy way to test whether the agent is reused?

yes.

Closing pending discussion on #1304.


[Event Hubs] Enable running live tests in the central repo

For client we need the following env vars pointing to existing resources
EVENTHUB_CONNECTION_STRING=
EVENTHUB_NAME=

For processor in addition to the above, we also need
IOTHUB_CONNECTION_STRING=
STORAGE_CONNECTION_STRING=

I have the resources created in the developer playground. For now we could use those.  We also should create the test resources as part of the test setup, similar to what Service Bus tests are doing.

Relevant Service Bus PR
https://github.com/Azure/azure-sdk-for-js/pull/1164

azure-sdk-for-js - live - eventhubs-client
azure-sdk-for-js - live - eventhubs-processor


Misleading error when certifi package is missing

🐛 Bug
I am using Arch Linux OS, and I installed napari[pyqt5] using pip (user local installation). When I started the basic program which creates a viewer and calls napari.run(), it shown me error that QApplication is not created. I am experienced with PyQt5, and I understand that its a very trivial issue to exist as a bug. After sometime searching in the code I realized that I was missing the "certifi" package, which is not even listed as a dependency.
After I installed certifi, it worked flawlessly.
To Reproduce
Steps to reproduce the behavior:

Install napari[pyqt5].
Uninstall certifi package.
Launch any napari app (any basic tutorial app will do).

import napari
v = napari.Viewer()
napari.run()

Expected behavior
The napari Viewer should open.
Environment
napari: 0.4.11
Platform: Linux-5.14.3-arch1-1-x86_64-with-glibc2.33
System: Arch Linux
Python: 3.9.7 (default, Aug 31 2021, 13:28:12) [GCC 11.1.0]
Qt: 5.15.2
PyQt5: 5.15.4
NumPy: 1.20.3
SciPy: 1.6.2
Dask: 2021.09.0
VisPy: 0.8.1
OpenGL:

GL version: 4.6.0 NVIDIA 470.63.01
MAX_TEXTURE_SIZE: 16384

Screens:

screen 1: resolution 1920x1080, scale 1.0

Plugins:

console: 0.0.4
scikit-image: 0.4.11
svg: 0.1.5

Additional context
Adding dependency on package "certifi" should fix the problem.

good catch!  it's kind of amazing that you ended up with an environment without certifi 😂 ... but I did just double check and none of our direct dependencies depend on it.  It comes in with requests, which tons of our dev and docs packages use, but we missed this.  thanks!


Power BI Report Server does not support custom visuals created in R

The docs should be updated to say that only custom visuals created in TypeScript/JavaScript are supported with Power BI Report Server.  Custom visuals that are written in R are not supportrf with Power BI Report Server.
For example, the Clustering with Outliers visual will not work with Power BI Report Server.
https://appsource.microsoft.com/en-us/product/power-bi-visuals/WA104380889?tab=Overview

Thanks for your comment, @SteveWinward ]. We’ve forwarded your feedback to the writer, who should respond to this thread.

@markingmyname - please route based on where an update/mention should go for this.

@SteveWinward, thanks for bringing this to our attention.
I will investigate where is the best place for this information.
Thank you.

@SteveWinward We do have this noted in the article - https://docs.microsoft.com/en-us/power-bi/report-server/compare-report-server-service.
Hope this helps.
Thank you.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


nord_uniform_status_lines does not work with airline

The setting nord_uniform_status_lines does not work with airline. The statusline does change its colors.

Hi @astier :wave:, thanks for your contribution :+1:
@jmurinello submitted PR #169 that implements the missing support for the theme configuration. I've reviewed and merged it already so it'll be deployed with the next release version 0.13.0.


Update versions

Updated to latest versions

Closing it, as @mattdangerw already created this in https://github.com/keras-team/keras-io/pull/1655


Upsert with Alternative KEY

Instead of having only GUID to match, we should be able to match with Alternative KEY set in CDS. Pretty please? ;)

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

ID: 800c3bde-6083-7a7c-3469-48137843198f
Version Independent ID: d1466842-8d49-67ad-1ca1-326511b2560e
Content: Create an automated flow with Common Data Service - Microsoft Flow
Content Source: flow-docs/connection-cds.md
Service: flow
GitHub Login: @brandonsimons
Microsoft Alias: bsimons

@thomassandsor - Thanks for your feedback. We are closing this issue because we are either unable to find relevant information or reproduce this. If you're still experiencing the issue described here, please reopen or create a new issue. Or, for product-related feedback, visit https://ideas.flow.microsoft.com.


Remove duplicated subscriber for one device

After installation multiple times the same application, pushd has registered multiple token to the same device. In the result is multiple the same messages from GCM for one unique device.
To prevent this in the future removes all subscriber whose token has been changed.

Can this be checked and merged of good please :-)


Lua 5.1 minimum compatibility

See https://github.com/Omikhleia/resilient.sile/pull/70

All good.
Updated the vendored qrencode from speeddata - There are still some linting errors, so I left the lua-libraries unchecked, but nothing relates to the Lua min profile.
No change in behavior on my tests.
-> No need for a patch release.


Ensure that mutable borrows are always specified when needed

Many methods in the various public stucts exported by git2-rs (such as Repository#remote) end up mutating one ore more underlying libgit2 objects without taking a &mut self reference. This is normally fine, since libgit2 methods usually return copies of the internal repository state. For example, Repository#find_remote can still be used after a call to Repository#remote_delete.
However, it's possible that not all libgit2 structures after the source data (which they were copied from) has been modified or deleted from the root Repository. It might be useful to have (public-facing?) documentation for such methods explaining why a mutable self reference is not required.

Yeah lots of the structs in C I believe are akin to something like Arc<RefCell> in the sense that they allow aliasing via reference counting and don't require mutable access. Note that most of the structures (I don't think any?) are Sync, so there's no worry about concurrent access.
In general It's somewhat difficult to extract the right mutability and whatnot while also  having the rust version be usable (as C isn't as strict as Rust is). Did you find a specific problem, though? If something is segfaulting that's very bad!

I found myself stuck on the opposite problem while trying to put both a Repository and a Remote in the same struct using rental.
Why does Remote require a &mut self reference for methods that mutate the underlying Repository? Repository itself doesn't require it when updating head for example. At first I thought methods used mutable references or not depending on whether the underlying C function used a const pointer or not, but that's not the case.
Is mut being used to ensure thread safety? The mention above about Sync seems to imply that's not the case.
I'm trying to understand this to find out whether certain mutable references can be changed to immutable references.

The usage of mut may have been overly conservative. I agree that if types aren't Sync it's likely fine to remove the mut requirement!

I'm going to close this since IIRC most APIs should be appropriately conservative today, but there are problems then an issue can certainly be opened!


Deploy widget_driver_test to pub.dev

See other ticket

Done with:
https://github.com/bmw-tech/widget_driver/pull/62
https://github.com/bmw-tech/widget_driver/pull/64
https://github.com/bmw-tech/widget_driver/pull/65


Can't delete Additional Hosts

Describe the bug
When trying to delete all the additional hosts, I get:

Failed to save add-on configuration, Missing option 'additional_hosts' in root in Cloudflared (90xxxxfa_cloudflared). Got {'external_hostname': 'axxxxxxxe.ga', 'tunnel_name': 'homeassistant', 'tunnel_token': ''}

I can only edit to make the additional host pointing nowhere.
To Reproduce
Steps to reproduce the behavior:

Go to Cloudflared Configuration tab
Delete all Additional Hosts
Save
See error

Expected behavior
Configuration is saved without errors. Additional Hosts are removed
Screenshots

Add-On / Home Assistant Information (please complete the following information):

Add-On Version (e.g. 0.7)
Supervisor Version (e.g. supervisor-2021.10.8)
Core Version (e.g. core-2021.11.5)
Operating System (e.g. Home Assistant OS 6.6)
Devices (e.g. Raspberry Pi 4 Model B)

Additional context
Add any other context about the problem here.

Try to use the YAML editor to change the add-on config and make sure to add:
additional_hosts: []

Thank you. That works. I had tried like the empty tunnel but didn't work.


Reset / destroy implementation

Hello,
I would like to know how to destroy every object created with a terraform deploy using this repository
For exmaple I run
./ansible-wrapper stacks/SAP-Prerequisites/playbook.yml -e @stacks/SAP-Prerequisites/vars/deploy-vars.yml
./ansible-wrapper stacks/WebDisp/playbook.yml -e @stacks/WebDisp/vars/deploy-vars.yml
How can I remove everything after the deploy?
Thank you

@placed87 .. You can append this additional argument -e state=absent to your stack deploy command to delete any stack and that destroys the infra of the stack. This argument essentially points to running terraform destroy by ansible.
example:
./ansible-wrapper stacks/SAP-Prerequisites/playbook.yml -e @stacks/SAP-Prerequisites/vars/deploy-vars.yml -e state=absent


audio: enable to call UnplayedBufferSize from Read

Now this is impossible due to a mutex.

UnplayedBufferSize itself was gone.


2024-10-24_01-35-04_master-next_aws-c-compression

Automatically created.

You have exceeded a secondary rate limit and have been temporarily blocked from content creation. Please retry your request again later. If you reach out to GitHub Support for help, please include the request ID 8B81:12FA33:E1F622:1B73DB9:671A0947 and timestamp 2024-10-24 08:46:01 UTC.

You have exceeded a secondary rate limit and have been temporarily blocked from content creation. Please retry your request again later. If you reach out to GitHub Support for help, please include the request ID 8B81:12FA33:E20011:1B750EE:671A094A and timestamp 2024-10-24 08:46:04 UTC.

You have exceeded a secondary rate limit and have been temporarily blocked from content creation. Please retry your request again later. If you reach out to GitHub Support for help, please include the request ID 8B81:12FA33:E20898:1B76181:671A094C and timestamp 2024-10-24 08:46:06 UTC.


e2e: install: enable install lane to reuse existing kubeletconfig

Add an option (env var) to let the install suite avoid to deploy kubeletconfig. This enables CIs to test different environments and configs easily.
Switch the default scope to "pod" - as is the only one we support atm.
Also add minor fixes to improve the debuggability.

/cherry-pick release-4.10

/approve
/lgtm

/retest
known flake

/retest
infra issue

/retest


ComponentManager vs ServiceManager

ComponentManager+ServicesManager vs ComponentManager
For now we keep the two interface. We can always later put them in one (SM extends CM)

I think just have one interface ComponentManager.
Most access for looking up services is going to go through


New Slime Science Resources

Feature Suggestion
Brownium, Fiery Glass Shard, Ice Jam, Icecomb, Icy Honey, Cyborg Fossil, Alcohol Brine, Greenium, Golden Sand, Spiral Dust, Omega Jelly, Redstone (Reddium), Blue Jelly Stone, Green Jelly Stone, Yellow Jelly Stone, Mythic Diamond, Dead Wax, Rainbow Echo, Steel Shard, Tooth of Giant Saber Slime, Omega Dimond.
What will this feature do?
Add a lot of different and new Slime Science resources.
How it should integrate into the mod? (Optional)
Possible names for the feature? (Optional)
New Slime Science Resources.
Additional context
Brownium - Looks like Indigonium, but it's brown.
Fiery Glass Shard - A Glass Shard, heated by a Firestorm.
Ice Jam - Like a Pepper Jam, but it can be mined by an Apiary in Frostbite Forest.
Icecomb - Like a Hexacomb, but it can be mined by an Apiary in Frostbite Forest.
Icy Honey - Like a Wild Honey, but it's can be mined by an Apiary in Frostbite Forest.
Cyborg Fossil - Like a Slime Fossil, but it,s fossil of Cyborg Slime.
Alcohol Brine - Like a Deep Brine, but it is made of alcohol.
Greenium - Looks like Indigonium, but it's green.
Golden Sand - Like a Silky Sand, but it's golden.
Spiral Dust - Like a Spiral Steam, but it is dust, not steam.
Omega Jelly - Like a Royal Jelly, but VERY rare. 1/1000 cycles of Apiary.
Redstone (Reddium) - Looks like Indigonium, but it's red.
Blue Jelly Stone - Looks like Jellystone, but it's blue.
Green Jelly Stone - Looks like Jellystone, but it's green.
Yellow Jelly Stone - Looks like Jellystone, but it's yellow.
Mythic Diamond - Like a Strange Diamond, but it's VERY rare. 1/1000 cycles of Drill.
Dead wax - Like a Buzz Wax, but it has lay 24+ game hours outside the Silos.
Rainbow Echo - Like all Echoes, what exist in Slime Rancher, but Rainbow Echo is changing colors, to create echo, you will need to throw Omega Diamond into the Echo Net, and you will get the Rainbow Echo. You will need it in few Slime Science Gadgets.
Steel Shard - Like a Glass Shard, but it is made of steel.
Tooth of Giant Saber Slime - Tooth of a Giant Saber Slime, what was been exist during the Jellassic Period, a time when gigantic slimes roamed the planet.
Omega Diamond - Like Strange Diamond, Mythic Diamond, but it can't be mined, you can buy Omega Diamond for 100000 Newbucks.

maybe you could get the icy resources inside the frostbite forest, which is my suggestion

#11

Ok, i will edit it

Pretty pog


Kentico Xperience 29.0.0 not compatible with azure ai search?

Describe the bug
When upgrading to Kentico Xperience 29.0.0, I'm not able to

publish any pages or i'm getting an azure ai search related error
Visit the azure ai search module in the admin panel. I get an unexpected error

To Reproduce

Make sure you are using kentico 29.0.0
Navigate to azure ai search in the admin panel
We have encountered an unexpected error. Error status code: 500 - Internal server error.
see screenshot below for more info

Make sure you are using kentico 29.0.0
publish any page in kentico
An error occurred while submitting the data. See the event log for more information.
see screenshot below for more info

Expected behavior
I expected the settings of azure ai search to be opened
I expect to be able to publish pages.
Screenshots
error when navigating to the azure ai search settings in the backend

error when publishing a page

Library Version
I tried with 2 versions

Kentico.Xperience.AzureSearch 1.0.0
Kentico.Xperience.AzureSearch 2.0.0

Desktop (please complete the following information):

OS: Windows
Browser Chrome
Version 123.0.6312.123

Additional context

Hello, @Stefan-Truelime there are API breaking changes (mostly for .Internal namespace APIs) in Xperience v29.0.0.
We'll need to update the Xperience dependencies for this library, adjust the namespaces/APIs, and publish a new version which requires a minimum Xperience version of v29.0.0.
We are taking care of this.

We have updated the NuGet package to support Kentico v29.0.0


[EconomistAllBridge] Add Bridge

There's already a bridge for The Economist, but it doesn't seem to work, and at any rate, can't work the way I want it to work.
There are a couple of peculiarities with The Economist's RSS:

There's no main feed, just one feed for each section.
The Economist issues the weekly edition on Thursdays, which sometimes creates ~100 articles at a time. Some articles first appear in the "Latest" feed and only then are added to the weekly edition.
There are audio versions for articles, but they only become available in the weekly edition. So, an article often appears in the "Latest" feed and gets an audio version a few days later.

So, this bridge merges all the available feeds and re-orders entries to match the journal structure more closely. There's an option to only show entries that have an audio version, so such a feed can be used in a podcast app.
It also takes a lot of time to retrieve everything, but it seems impossible to do anything with that (yeah, I see it timed out in the CI). So add that one at your discretion.

I want to repair the bridge we currently have so that existing feeds start working again.

I can backport my changes to fix the existing bridge then, and leave this one in my personal config.
Will do in a few days.


libOpenNI2.so: file format not recognized; treating as linker script

Following the README instructions let to this error when using catkin_make on Noetic:
catkin_ws/src/ros_astra_camera/include/openni2_redist/x64/libOpenNI2.so: file format not recognized; treating as linker script
catkin_ws/src/ros_astra_camera/include/openni2_redist/x64/libOpenNI2.so:1: syntax error
I replaced the libOpenNI2.so file with the one from commit 0d2b91c3decfe6db0401b218f09799e9b8604399 and it worked. Not sure wether I did something wrong while installing though. Maybe someone can look into that and close this issue if there is nothing wrong.

Hello, @tizianrichter , The problem Is libOpenNI2.so store by git  LFS , Could you please try
sudo apt-get install git-lfs
git lfs install

Then  git clone this repo and  try again

This worked, thank you :)


enclave-tls: unify ASN.1 parser

Currently, all tls wrapper instances need to do the duplicated work on parsing x509 and finding quote OID. This work should go etls core.

abstract x509 parsing,  OID encoder and decoder into etls core layer between different ssl


Why is a tracker based on offline tracking so robust?

The paper is similar to siamfc++, except that the network layer is changed. Why is it so robust that it is better than online updates, such as dimp. How does this offline tracking tracker distinguish distractor?

Hi, thanks for your interest to our work :)
I think the reason why LightTrack achieves good performance is that the searched architecture is more suitable to the object tracking task and the learnt features are more powerful(discriminative) than previous trackers.
However, like other offline trackers, it is not easy for LightTrack to distinguish the target and similar distractors.

@MasterBin-IIAU @makalo
HI,professor:
is lighttrack an offline tracking algorithm? that means it can not use in the scene which need tracking an object in a live camera?
please help!

@jcyhcs Hi, here "offline" means "without any update of model parameters during tracking". LightTrack is still a causal tracker, which predicts tracking results of the current frame only based on the information from previous frames. So it can still be applied in tracking scenarios like live cameras.


Implementation of Retention Policy TTL

This issue is about implementing the background service for Retention,
As we would like to have retention policies based on tenant, product and component.
The level of priority for retention policy is the one which is the nearest to Topic.
Level 0 Component Retention Policies;
Level 1 Product Retention Policies;
Level 2 Component Retention Policies;
In this issue the logic should be implemented for all 3 levels for HARD_TTL and SOFT_TTL.
SOFT_TTL, in case when a message was not consumed from a subscription will not be deleted if the TTL has come.
HARD_TTL, it will delete the message even if is not consumed by a subscription, these messages will be skiped from subscription.

Also in this issue will implement an update on credentials, adding user roles and support for multiple users in andy x

Merged with v3/main


Provide Topic parameter

Transparent topic extraction can be problematic. One certificate can serve several topics, VoIP and debug ones. When a message is sent to a specific topic, there has to be a way to specify it.

okay. I'll add Topic back to Headers


TIM-153 feat(newsfeed): add newsfeed_likes table schema

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#109
#108
#107  👈
#106
#105
#104
main

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

Join @xdjeremy and the rest of your teammates on  Graphite


ELU: change greater than equal to strict greater.

Following the discussion in [1] and the original implementation in [2].
In the original implementation > 0 was used not as reported in the
paper.
[1] https://github.com/BVLC/caffe/pull/3388
[2] https://github.com/untom/binet/commit/2c8a6bd61ed59fd65453ae107a6a7c7c9e65f5dc

I think with BN, activation function is less important. But let's merge first.


add a ligne to chech if the first character is a letter

Closes: #1
Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[ ] Appropriate changes to README are included in PR

cla is now created


Consider exposing Base32 encoding code as a package

Per @joelverhagen  suggestion: https://github.com/NuGet/NuGetGallery/pull/5031#pullrequestreview-77887299
We should consider creating a package with our Base32 implementation on nuget.org for the community to use.
The idea was discussed with .NET team and the recommendation is to use Microsoft.Extensions.* namespace:
https://github.com/dotnet/corefx/issues/25393

Please consider making the decoder case insensitive so that it may be used in other case insensitive environments without the need to cast to upper case.
For example "mzxw6ytboi======" or "mZxw6YtboI======" are a valid base32 string just like "ff" or "FF" or "fF" are valid hex (base16) strings.

Please also  consider exposing  Base16  and Base36

Not sure this is the right place , but when you create a new nuget package please also consider exposing Base16 , Base36, and Base64 url safe to that package


Certificate is re-requested when container restarts

When a container restarts the certificate is re-requested and blows through quota for let's encrypt. It would be nice to reuse the cert already mounted when the container restarts, i.e. once per pod kind of thing.
Not sure if this is possible but if someone deploys a bad deployment and leaves it, or an app crashes in the wee hours of the morning for example, i have to wait like 12 hours after i stop the continued crashed container to be able to request a new certificate.

Noticing issues with crashed containers unable to unmount the certificates since the pod no longer exists. Seems to be related, if the certificates were reused after a crash, this would likely resolve the problem.


Performance scripts fix.

Adding TestAssets project to the TestPlatform.sln. So that it's restored and built.
Fixing some bugs in perf.ps1 script. And adding it to the sln file.
Added myget feeds of the adapters to the Nuget.config file.

@dotnet-bot test Windows_NT / Debug Build


Upgrading secondary password requirements does not trigger email

In gitlab by @CalumTowers on Jul 31, 2016, 17:54
When upgrading the requirements of someone's secondary password, it does not trigger an email to get them to create a password.
The workaround is for the end user to click "Forgotten Password" but a cleaner solution would be an email explaining why secondary passwords are important etc.

In gitlab by @A-Lawrence on Aug 1, 2016, 16:27
This was resolved early morning.


Opsexp-1700 Imagemagick-7.1.0-16 build

Imagemagick-7.1.0-16 build

Paths are corrects. This is seems to be because other distributions are having more rpms then deb .

It will be part of different PR.

Ubuntu 20.04 is missing

Will be part of different PR.


broadcast_to raise Disallowed host-to-device transfer when used inside jax.jit

Description
Reproduction:
@jax.jit
def test():
  return jnp.broadcast_to(0.1, (4,))

with jax.transfer_guard('disallow'):
  test()

Fail with:
XlaRuntimeError: INVALID_ARGUMENT: Disallowed host-to-device transfer: aval=ShapedArray(float32[]), dst_sharding=SingleDeviceSharding(device=TpuDevice(id=0, process_index=0, coords=(0,0,0), core_on_chip=0))

The 0.1 is a constant in the XLA graph, so this should not trigger any host-to-device transfer.
It's possible to silence the issue with jnp.broadcast_to(jnp.asarray(0.1), (4,)) but original issue is a bug.
What jax/jaxlib version are you using?
HEAD
Which accelerator(s) are you using?
TPU

Thanks for the report. It looks like the issue comes from the implementation of lax.asarray (used by broadcast_to), which uses device_put for non-jax-array inputs: https://github.com/google/jax/blob/31efc2cc6ae6f7f20b78c742def23afcd6507625/jax/_src/lax/lax.py#L129-L137
I think we need to use a different strategy for converting scalars to arrays if we want to avoid triggering the transfer guard here.

#19511 should fix this


Contains vulnerable copy of Expat - please update to 2.2.5

Hi!
This repository contains an outdated vulnerable copy of Expat 2.1.0. Please update your copy to version 2.2.4 with the latest security fixes. A change log with details is available at https://github.com/libexpat/libexpat/blob/master/expat/Changes. If you happen to run into compile errors, please check post-2.2.2 commits in Git as well. Thank you!
PS: Are you going to update NuGet as well? Else, who should I contact?
Best

Sebastian

Any news or issues with updating?

Well I don't have any time until mid August, so if nobody else does first it I will do it then.  The coapp bits need some adjusting to work with the updated source.

FYI Expat 2.2.4 has just been released. It would be cool to have it updated.
PS: The list of authors could used an update as well (compare with upstream author list).

Ok I think I'm done, I can't upload the NuGet packages as I don't have permission - but they are available at https://www.myget.org/F/raggles/api/v2 if somebody wants to test them.

Commit b96571124af2c4a5aec89b2e7a78a49e0ab25154 seems to be post-2.2.4 but even better: it includes Windows fixes.

I can't upload the NuGet packages as I don't have permission

How do we continue?

@virmitio or @fearthecowboy control the Nuget side of things I think, but as far as I know the coapp project is dead so not sure if you will get much traction there.  You could ask them if they can add you as a package owner or to transfer ownership.

I see. I cannot take it over myself, I haven't touched real Windows in years.

@Raggles @hartwork
I'm afraid that @fearthecowboy is the only one who may be able to hand off package ownership.  I only had an api key for uploading as the group, and that was on a machine that died without recovery a year and change ago.
If there's someone who has the time and genuine interest in updating and maintaining the various packages, I'll happily prod @fearthecowboy to "gift" such ownership.
The best alternative I can offer in the meantime would be to push the packages up to the feed under a revised name with similar tags so others can find the more current release.

I'd be willing to put some time into this (and have off and on over the past years, for the packages I have access to), but I don't want it to be just me.

Any news?

Any news? There is Expat 2.2.6 now.

Closing to keep https://github.com/issues clean for myself…


typo fix

typo fix
None

/assign @seanknox

this mistake has been fixed in another PR
close it.


wrong reference to dev list on mxnet.io

Reference to dev list at http://mxnet.incubator.apache.org/community/contribute.html#mxnet-dev-communications
for dev list correct reference is:
For Contributors to MXNet, join the DEV mailing list:
MXNet Apache DEV mailing list (dev@mxnet.apache.org): To subscribe, send an email to<EMAIL_ADDRESS>@aaronmarkham please fix.

@mxnet-label-bot please add [Doc, Bug].

@eric-haibin-lin The PR has been merged: https://github.com/apache/incubator-mxnet/pull/12271 we can close this off.


Implement 'strip' function

Not entirely sure if this is exactly how we want this to work, but I've implemented a basic 'strip' function. If somebody wants to write tests or whatever that's fine

Hopefully, I did not break anything... seems to be working fine... prob will be duplicate files

Whoops forgot we do not have python 3 support yet...

@TheMonsterFromTheDeep Do you mind if you make a new PR just to clean up the repo (without the travis stuff sense they are in master)... I kind of messed it up adding the Travis code?

Ignore circleci

@CosmicWebServices what do you mean? What's unclean about the repo, currently?

@TheMonsterFromTheDeep I made the travis ci changes in this repo and master... you know what I will just move this

https://github.com/ilovecode1/Pyfancy-2/pull/19


Update README instructions for building the project with nix

The commands for building with nix changed when we switched to haskell.nix in https://github.com/serokell/tezos-btc/pull/78
Description

Related issue(s)

None
:white_check_mark: Checklist for your Merge Request

Related changes (conditional)

Tests (see short guidelines)

[ ] If I added new functionality, I added tests covering it.
[ ] If I fixed a bug, I added a regression test to prevent the bug from
silently reappearing again.

Documentation

I checked whether I should update the docs and did so if necessary:

[ ] README
[ ] Haddock
[ ] docs/

[ ] I updated changelog unless I am sure my changes are
really minor.

Stylistic guide (mandatory)

[ ] My commits comply with the following policy.
[ ] My code complies with the style guide.

LGTM


Add feature for reading a textfile from URLs

After cleaning the branches we have now a clear pull request with 4 files.
It allows us to specify a URL for the Java and Scala File-Source.

PR has been merged.


JavaDoc : Propagator references Open Telemetry classes which is not part of Micrometer

The JavaDoc in the Propagator class references TextMapPropagator from Open Telemetry which is not part of Micrometer.
This is present at line 96 and 133

Closed by #236


USI Mods

This is replacing #3016

A proper review of this will involve a manual test of each netkan to see what it's generating as a .ckan and ensuring it falls into the correct place in the version continuum CKAN and its users currently have in place. At a quick glance the .netkans in BobPalmer/CKAN/ seem like verbatim copies of the previous .netkans that were hosted here, but I'll need a solid block to time to work through all 19 files. If someone else can get to this before me great, otherwise I'll take a look at this after the weekend.

@BobPalmer You are deleting FirespitterCore.netkan which will cause the mod not to be updated anymore in the future. I don't know if you forgot to update the .netkan file or not, but I saw in your BobPalmer/CKAN repo that you made a .netkan file for FirespitterCore but are not linking to it in this PR.
Here is what the new .netkan should look like:
{
    "spec_version": "v1.2",
    "identifier"  : "FirespitterCore",
    "author"      : [ "snjo", "RoverDude" ],
    "$kref"       : "#/ckan/netkan/https://raw.githubusercontent.com/BobPalmer/CKAN/master/FirespitterCore.netkan",
    "x_netkan_license_ok": true
}

I also wanted to make a PR on your repo to fix the FirespitterCore.netkan (but for some reason I couldn't) because when there is more than one author you need to use [ ].
So change "author" : "snjo, RoverDude", to "author" : [ "snjo", "RoverDude" ],

Closing for #3031


highlight-words libray is  giving Math.random vunerbility scan as high

material-react-table version
in all vesions
react & react-dom versions
in all vesions
Describe the bug and the steps to reproduce it
highlight-words libray is  giving Math.random vunerbility scan as high
Minimal, Reproducible Example - (Optional, but Recommended)
highlight-words libray is  giving Math.random vunerbility scan as high
Screenshots or Videos (Optional)
highlight-words libray is  giving Math.random vunerbility scan as high
Do you intend to try to help solve this bug with your own PR?
No, because I do not know how
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

I just rechecked the source code for highlight-words and the only place where it uses Math.random is in a snippet like this
while (num--) {
    str += HEX[(Math.random() * 36) | 0];
  }

It uses this code simply to create a small uuid key for identification of chunks to tell them apart.
Feel free to share more info about this vuln, as if it was a real issue I'd take it seriously, but I don't see the severity at all right now.
MRT itself using Math.random to create random width skeleton loaders.
These use cases of Math.random() in the front-end are extremely inconsequential. Neither of these use cases are relying on it for storing ids in databases or anything like that.

Thank you so much for the response, and I understood what you mentioned. We are using Material-React-Table, and this is the only high warning we got because highlight-words is used by Material-React-Table. No other libraries used by Material-React-Table gave any warning. That probably means you (or a plugin you are using) is using the JS native Math.random() which has been deemed insecure. You can replace those functions with the latest JS crypto stuff here as a recommendation. What you commented makes sense, but if you can replace Math.random(), it will make our scan clean. We mentioned it only because this is the only warning we got when using Material-React-Table and sometimes companies refuse to use it just because of the warning even though its not anything significant. See the attached screenshot.

.

I would support them fixing this too, and understand the paperwork problem for some orgs. I will update as soon as possible if they fix it. Either of us could PR.
However, for anyone reading this and wondering if you should be concerned about this, the answer is no. These random values are being used just in the front end for some rendering, and barely even need to be unique. Math.random is fine for such use cases. However as said above, to help anyone out with paperwork, we can still change this.

Thank you so much for understanding.  I am also new to react . This is not an item to be fixed ,I agree . But just for paper work if it will  will be looked will be great . If it get fixed in both react 17 and 18 versions will be great. I am new to rest and thank you so much for a big developer from material-react-table to spend time for me. Great product and great support . You are all good human beings,be blessed

Sir , i did a pull request to fix it https://github.com/tricinel/highlight-words/pull/35, can you apply it if are you contributor to hilight-words, i am new to this kind of process


Document which button types are pressable

KeyCode and ScanCode aren't buttons. "Input types" perhaps?
I suggest to do the same exact changes to Axis<T> struct (which has GamepadButton and GamepadAxis, and that's not documented anywhere either).
It is a bit unfortunate that Input is not self-documenting, ideally we'd have Input<T: InputType> instead of Input<T: Copy + Eq + And + Stuff>, and users could've just looked up which types implement InputType trait. But that's probably gonna break something, so better not.

ideally we'd have Input<T: InputType>

I wanted to do it, it resulted in strange compilation errors, I suspect because of this dependency cycle:
https://github.com/bevyengine/bevy/blob/24c6a7df05f0aadf9cd9ddc7183fa8de93e4bdd5/crates/bevy_input/Cargo.toml#L29-L30


React-bootstrap Nav cause issues with testRef

Hello,
I'm encountering a issue when using react-bootstrap's Nav.
I have the following code:
import React from 'react';
import { Nav, NavItem } from 'react-bootstrap';
import { LinkContainer } from 'react-router-bootstrap';

const ENROLL_URL = '/ui/device/enroll';

class EnrollDeviceNavigation extends React.Component {
  render() {
    const activePage = parseInt(this.props.activePage, 10);
    return (
      <Nav justified bsStyle="pills">
        <LinkContainer to={ENROLL_URL} query={{...this.props, activePage: 0 }} active={activePage === 0} testRef="selectType">
          <NavItem> 1. Select Device type </NavItem>
        </LinkContainer>
        <LinkContainer to={ENROLL_URL} query={{...this.props, activePage: 1 }} disabled={this.props.device !== 'orbitx'} testRef="wifi">
          <NavItem> 2. WiFi (Orbit X) </NavItem>
        </LinkContainer>
        <LinkContainer to={ENROLL_URL} query={{...this.props, activePage: 2 }} testRef="enroll">
          <NavItem> 3. Enroll & Manage </NavItem>
        </LinkContainer>
      </Nav>
    );
  }
}

And when accessing any of the testRef's I get "undefined".
However, if I replace the Nav node with a simple div everything works.

I think I also see a similar issue with OverlayTrigger

Bleurgh, bloody react-bootstrap!
I have a suspicion it is to do with this line https://github.com/react-bootstrap/react-bootstrap/blob/master/src/Nav.js#L81 which clones the children, which means the original LinkContainer is never actually rendered (a clone is instead). Could you please confirm this by putting a testRef on the NavItems you have inside and seeing if they work?

Tested, and your suspicion is correct.
I guess the reason this worked before, was that you used the "ref" property, and that probably gets cloned per default...but testRef doesn't.


Not possible to play with variations on drops that are minted out

Its not possible to play with generate button once the collection is minted out? It's generating the same picture. I think it should still work so people can play with the code. Right?
https://github.com/kodadot/nft-gallery/assets/104380776/89925e5c-a3ec-4ecf-bcd0-ad405798df2b

can you try again now @JustLuuuu ? it was probably fxart error

doesn't look like its working 😔
https://github.com/kodadot/nft-gallery/assets/104380776/703b9629-98d7-42ff-9967-7b6557bd7787

can you try again now @JustLuuuu ? it was probably fxart error

doesn't look like its working 😔
Screen.Recording.2024-04-11.at.18.16.50.mov

can you show us your console to see what's the error?


Gemfile.lock scan is broken

Hi I just tried using the "Check a lock file" feature and got a 500 when I pasted the .lock content and hit Checknow. let me know how can I help.

@Alchiman Thanks for submitting this issue! I believe it was related to #108 and it was solved by #109
I just tested that it is working fine with this Gemfile.lock: https://gist.github.com/etagwerker/73c8d38f8e6712701886338b20a78413 (Gitlab's Gemfile.lock)
Please try again and let me know if you still run into the 500 error. Thank you! 🙏🏼


Adding conditional styling to label based on value.

Problem
I would like to apply zettelkasten to my vault and key thing is to have notes of maximum size 150 words each.
Idea
I would like to add style rules, for example if number of words in note is higher than 150, it will color the size label to the red. It can be done with CSS classes (add CSS class when it meets conditions) and style it via snippets.
Inspiration
This feature is inspired by kanban plugin when date and time is colored based on value - useful for deadlines warning.

If you're looking for a visual indicator when a note has reached/surpassed a certain word count, the Writing Goals plugin may be a good fit.
I don't think it makes sense for this plugin.

Thank you for your suggestion, I will take a look at that. I fully understand why it does not make sense to do, when it's in another plugin already implemented.
Thank you for your response, love your plugin btw <3


HBASE-26371 Prioritize meta region move over other region moves in region_mover

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 25s
Docker mode activated.


Fix missing newline, breaking markdown parsing of lists

See https://jmfayard.github.io/refreshVersions/migrate/

Thank you!


Geo-replication is not available at this time when using extra replicas.

Geo-replication is not available at this time when using extra replicas.
I have a Premium Redis Cache with a single replica and data persistence disabled, there I got an option to add cache replication link after going to Geo-replication section. But I have another Premium Redis Cache with two replicas and data persistence enabled(only one availability zone), I could see this when I tried to avail Geo-replication:-> "Geo-replication can be enabled for this cache if you scale it to 'Premium' pricing tier and disable data persistence. This feature is not available at this time when using extra replicas"

@animeshdas11 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

Any updates on this pull request?

Hi @flang-msft - Could you take a look? Thanks.

HI @animeshdas11,
Thanks for the suggestion.
We added a not in the pre-req section but left the note as is. We will push this from a private repo.
'''diff

Neither cache can have more than one replica.

#please-close


Order expires on renew

Describe the bug:
cert-manager is running on aws/eks and uses a private ACME. Requesting initial certificate works fine. When it comes to an update the order expires:
  Normal   Reused   59m (x19 over 19h)  cert-manager  Reusing private key stored in existing Secret resource "our-stack-certificate"
  Normal   Issuing  59m (x18 over 18h)  cert-manager  Renewing certificate as renewal was scheduled at 2021-01-27 14:59:51 +0000 UTC
  Warning  Failed   59m (x18 over 17h)  cert-manager  The certificate request has failed to complete and will be retried: Failed to wait for order resource "our-stack-certificate-rgmn6-1231785326" to become ready: order is in "errored" state: Failed to finalize Order: 400 urn:keyon:params:acme:error:expired: Order expired

Even when the order fails, the challange is valid
kubectl -n our-stack-int get certificaterequest,order,challenges,certificate
NAME                                                                    READY   AGE
certificaterequest.cert-manager.io/our-stack-certificate-l29jv   True    19h
certificaterequest.cert-manager.io/our-stack-certificate-rgmn6   False   17h

NAME                                                                       STATE     AGE
order.acme.cert-manager.io/our-stack-certificate-l29jv-1231785326   valid     19h
order.acme.cert-manager.io/our-stack-certificate-rgmn6-1231785326   errored   17h

NAME                                                                                STATE   DOMAIN                                                 AGE
challenge.acme.cert-manager.io/our-stack-certificate-rgmn6-1231785326-1823297271   valid   test.our-stack-nonprod.internal.net   17h

NAME                                                       READY   SECRET                         AGE
certificate.cert-manager.io/our-stack-certificate   True    our-stack-certificate   22h

(please don't be confused by the times/ages, we have added the renewBefore: 332h in the certificate so we only need to wait 2 hours and not 2 weeks until the certificates wants a renewal - this produced the same error we actually had on the normal one after 2 weeks)
The expired order appears seconds after the renew was triggered.
Anything else we need to know?:
Environment details::

Kubernetes version: 1.16
Cloud-provider/provisioner: aws/eks
cert-manager version: 1.1
Install method: helm

/kind bug
Update:
As we use the ClusterIssuer with HTTP01 Challenge Provider, adding the serviceType enhanced the success rate but still errors occurs on some certs  (testing ongoing)
solvers:
    - http01:
        ingress:
          class: nginx
          serviceType: ClusterIP

I was thinking about trying to reproduce the issue this morning but I then realized that the ACME implementation is closed-source. That would be Keyon's Microsoft PKI RA ACME+, is that correct?
Not sure how to proceed with this bug 🤔 How could the cert-manager team help with debugging the issue?
/triage needs-information

We have just updated to 1.2.0 and now we are waiting to see if this helps. We have requested the informations what server they are using, but they are not really "responsive"


Fragment instance

Some field component have multiple top-level elements. This will create fragment instance that should be avoided.
Concerned fields:

fieldChecklist
fieldColor
fieldRange

Thanks!


Updates tar distribution to conform to changes in install demo configuration script in security plugin

Description
Following up on the changes to demo configuration script in security plugin, https://github.com/opensearch-project/security/pull/3669, the OS supported distributions need to adapt to changes in the behavior of demo configuration script.
What has changed?
The ./opensearch-tar-install.sh will now have these execution paths:

initialAdminPassword value is set to a strong password: Demo configuration script execution suceeds
initialAdminPassword.txt contains a strong admin password: Demo configuration script execution suceeds
If both of the above are provided, the initialAdminPassword value is picked always.
If weak passwords are provided in either of the cases, tar-install.sh execution will fail.
If no password is provided, the script will generate one and print it to the logs.

For detailed testing output, refer to the comments on the issue mentioned below.
Issues Resolved

Resolves https://github.com/opensearch-project/security/issues/3634

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Just to be clean, the user is still admin and only the password have to be provided, right?

Adding @derek-ho, looking this is having a problem to deal multi node cluster? Derek can you please add more details. Thank you

Is the variable really initialAdminPassword?! All OpenSearch variables are OPENSEARCH_SOMETHING_SOMETHING, where did we deviate from this convention?

@prudhvigodithi @rishabh6788 Would you please review this?

I am okay with approving this change but we cannot merge this until security plugin changes have been back-ported to 2.x as it will start breaking 2.12.0 tests and benchmark runs.

I am okay with approving this change but we cannot merge this until security plugin changes have been back-ported to 2.x as it will start breaking 2.12.0 tests and benchmark runs.

The backport PR is currently in draft mode, waiting couple of PRs to be merged into main: https://github.com/opensearch-project/security/pull/3845

https://github.com/opensearch-project/security/pull/3845 is merged and this PR is unblocked.

@peterzhuamazon @prudhvigodithi @rishabh6788 Could I please get reviews on this?

Need to change these as well if you plan to change docker-compose:
https://opensearch.org/samples/docker-compose.yml
https://opensearch.org/docs/latest/install-and-configure/install-opensearch/docker/#sample-docker-composeyml

Also need @prudhvigodithi to confirm on @DarshitChanpura comment above regarding keeping the exit 1 for docker and tar, but remove for deb and rpm.

This PR only address TAR ball, no docker/windows/rpm/deb changes are present

@peterzhuamazon Could I get some re-reviews on this PR?

I'm ok with exit 1, even though having set -e is better as the execution codes are directly dictated by install_demo_configuration.sh but needs proper testing on how the script would behave (using set -e) with docker etc.
@peterzhuamazon add your final thought.

@DarshitChanpura can you please use as echo -e "This is a line with a new line.\nThis is the second line." to print statements in a new line. We dont need 3 echo lines.

echo -e maybe slightly unstable. I've used printf instead which is more stable for combining 3 lines with \n

@peterzhuamazon @prudhvigodithi @rishabh6788 Could I get some more reviews?

@peterzhuamazon is there anything blocking this PR still?


Fix code scanning alert - Uncontrolled data used in path expression

Tracking issue for:

[ ] https://github.com/lnovitz/jobseeker-analytics/security/code-scanning/2

commit b0b5e41bae51d0f4abc0ddf1d9a2dd83e6799c42 (HEAD -> main, origin/main, origin/alert-autofix-2, origin/HEAD)
Author: Lianna<EMAIL_ADDRESS>Date:   Sun Dec 29 21:40:43 2024 -0800

    Fix code scanning alert no. 2: Uncontrolled data used in path expression

    Co-authored-by: Copilot Autofix powered by AI<EMAIL_ADDRESS>


fix(model/audit-log)!: mark some old fields optional

In role create audit log events, and possibly channel create as well, the old_value may be a missing key for these AuditLogChanges. This PR marks them as optional.

Codecov Report

Merging #1284 (8b9670d) into main (efb6045) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1284   +/-   ##
=======================================
  Coverage   31.03%   31.03%
=======================================
  Files         360      360
  Lines       14241    14241
=======================================
  Hits         4420     4420
  Misses       9821     9821

Impacted Files
Coverage Δ

model/src/guild/audit_log/change.rs
6.55% <ø> (ø)

http/src/routing/route.rs
5.95% <0.00%> (-0.63%)
:arrow_down:

http/src/routing/route_display.rs
10.36% <0.00%> (+0.38%)
:arrow_up:

Continue to review full report at Codecov.

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


Multi chain protein trajectory

Hi,
Is there any way to use a multi chain protein MD trajectory?
I noticed that the script throws the following error when my ca pdb file contains chain identifiers:
Generate Train
Start: 1th PDB
Traceback (most recent call last):
File "convert_dataset.py", line 345, in
feature_size=feature_size, num_residues=num_residues, interval=train_interval, pdb_start=pdb_start, pdb_end=pdb_end, aa_start=1, aa_end=num_residues)
File "convert_dataset.py", line 305, in convert_dataset_md_single
i)+".pdb", timestep_size, feature_size, num_residues, interval, j, aa_start, aa_end))
File "convert_dataset.py", line 127, in read_feature_MD_file_slidingwindow
elif(line.startswith("ATOM") and words[2] == "CA" and int(words[4]) >= aa_start and int(words[4]) <= aa_end and flag):
ValueError: invalid literal for int() with base 10: 'A'

Got it. Thank you for your reply


3.0 - Final Phase, stage continued #14

Move to default fetching method PDO::FETCH_ASSOC as optimizing the database server bandwidth and introduce auto-configuration of fetching method in Services.

I prefer objects over arrays so when using this to use an object I would use FETCH_OBJ in my config class corrected? otherwise the default assoc would be used.

@daveismyname It is configured from Database configuration.
You need only to put there PDO::FETCH_OBJ, instead of default PDO::FETCH_ASSOC
And the Services auto-switch the Engine in this mode.

great.


HTTP Exception handling

I am trying to catch HTTP errors. I used following options: 1) googleapiclient.errors.HttpError 2) google.api_core.exceptions.NotFound
But neither of these options work for me. Any ideas/tips why? Thank you!
Python version 3.6.9
from pprint import pprint

from googleapiclient import discovery, errors
from oauth2client.client import GoogleCredentials
#from google.api_core.exceptions import NotFound

credentials = GoogleCredentials.get_application_default()

service = discovery.build('compute', 'v1', credentials=credentials)

# Project ID for this request.
project = 'aweqwfnwrueifubwerfbiu'  # TODO: Update placeholder value.

try:
    request = service.firewalls().list(project=project)
except errors.HttpError:
    pprint('Error!')

while request is not None:
    response = request.execute()

    for firewall in response['items']:
        # TODO: Change code below to process each `firewall` resource:
        pprint(firewall)

    request = service.firewalls().list_next(previous_request=request, previous_response=response)

Traceback (most recent call last):
  File "test.py", line 20, in <module>
    response = request.execute()
  File "/lib/python3.6/site-packages/googleapiclient/_helpers.py", line 130, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/lib/python3.6/site-packages/googleapiclient/http.py", line 856, in execute
    raise HttpError(resp, content, uri=self.uri)
googleapiclient.errors.HttpError: <HttpError 404 when requesting https://compute.googleapis.com/compute/v1/projects/aweqwfnwrueifubwerfbiu/global/firewalls?alt=json returned "The resource 'projects/aweqwfnwrueifubwerfbiu' was not found">

Solved


Add an attribute that indicates you only want to serialize the Id and Type property

The previous serializer we used has the ability to specify that you only want to serialize the Id and Type property of a specific resource. For example:
[JsonApiResource("other-model")]
public class OtherModel
{
    public String Id { get; }

    public string OtherValue { get; }
}

[JsonApiResource("model-with-attribute")]
public class ModelWithAttribute
{
    public string Id { get; set; }

    public string Value { get; set; }

    public int IntValue { get; set; }

    [SerializeAs(Type = RelationshipSerializationTypeEnum.IdOnly)]
    public OtherModel OtherModel { get; set; }
}

This indicates that the serialization of OtherModel only gets serialized with the Id and Type property even if the OtherValue property is filled.
Is this something you're open to adding to the repo? If so, I can try to create a PR for this. I just wanted to make sure you were on board with this concept before creating the PR.

Yeah makes sense, basically you're wanting to describe whether a relationship is output as a compound document via included:[] or whether it's just a type/id relationship. If I'm reading this right, this would have no bearing on deserialization, only on serialization.
Feel free to go ahead with a PR. I'll have some opinions about naming 😉 , but I think the API you've described is sound. There's also going to be some edge-cases with array relationships.


Russian translate

Hi Dear, I made a Russian translate for this.  I pushed it to my fork, but I don't gamble to push it here. My translation isn`t still ideal, but it is worked. https://github.com/vhot2076/gitblit/commit...

If you feel your translation is worth adding to Gitblit, please feel free to open a pull request.

Ok, did this.


Protected and Public APIs

Hi! I'm checking out this awesome boilerplate repo. It's all find and working until I added my own endpoints which are meant to be public. I somehow can't find the config to exclude it from authentication. I always get the 'Not authenticated' response. Can anyone point me to the right direction?  I think it's just there but am already splitting my hairs but still couldn't figure it out.

okay. as expected, I made some stupid mistake in the router config when doing include_router.
Sorry for the trouble.

Thanks for reporting back and closing the issue! :+1:


Understanding Column Labels (01, 02, 03, etc.) in FMA Dataset

Hello, I'm currently working on my thesis about music genre classification using the FMA dataset. However, I'm having trouble understanding the meaning of these columns labeled 01, 02, 03, and so on. Could you please assist me with this? I would greatly appreciate it.

In case an answer to this would still be helpful: MFCC produces a sequence of coefficients, numbered 01, 02, ..., for each small time window of a track. To generate that table, the programmers calculated the first 20 of these coefficients, and then took their mean, variance, etc. over the whole track. So the column ("mfcc", "median", 02) gives the median of the 2nd MFC coefficient over the whole track.
The situation's the same for the other features, and many of those are easier to understand than MFCC. For example, there are 12 chroma_cqt features for each time window, representing the strength of the 12 chromatic pitch classes in that window. The summary statistics for those features tell you about the strength of the 12 pitch classes over the whole track.


Train the NYU dataset, but the result is not good.

I attempted to test the NYU dataset using the provided weight file and obtained a result that was consistent with 85.17 in the paper.
But when I tried to train the NYU dataset, it was difficult to achieve the same accuracy and the gap was significant.
I have not made any changes to the training configuration of train, using a 3090TI.

I am not sure if there are any configuration conditions that have not been set as default options in the code besides those mentioned in the paper. If you still remember some details, I would greatly appreciate it.

There are few articles on estimating table plane normal vectors from a single image.
Your article is still a benchmark in this direction even today. The success of reproducing the accuracy of the paper greatly affects other students who wish to study in this field.

exp_dir: ./experiments/exp00_test/
exp_name: exp00_test
visible_gpus: 01
architecture: GN
use_baseline: False
sampling_ratio: 0.4
importance_ratio: 0.7
loss_fn: UG_NLL_ours
n_epochs: 5
batch_size: 4
validate_every: 5000
visualize_every: 1000
distributed: True
workers: 12
weight_decay: 0.01
lr: 0.000357
same_lr: False
grad_clip: 0.1
div_factor: 25.0
final_div_factor: 10000.0
dataset_name: nyu
input_height: 480
input_width: 640
data_augmentation_color: True
data_augmentation_hflip: True
data_augmentation_random_crop: False
num_threads: 12
mode: train
exp_model_dir: ./experiments/exp00_test//models/
exp_vis_dir: ./experiments/exp00_test//vis/
exp_log_dir: ./experiments/exp00_test//log/

total_iter: 5000
mean median rmse 5 7.5 11.25 22.5 30
38.050 35.639 44.162 3.527 6.796 12.270 30.207 41.788
total_iter: 10000
mean median rmse 5 7.5 11.25 22.5 30
37.014 34.177 43.714 4.668 9.045 15.647 33.464 44.225
total_iter: 15000
mean median rmse 5 7.5 11.25 22.5 30
35.905 31.601 43.314 4.855 9.957 17.958 36.793 47.755
total_iter: 20000
mean median rmse 5 7.5 11.25 22.5 30
33.596 29.393 40.789 7.073 12.327 19.922 39.492 50.873
total_iter: 25000
mean median rmse 5 7.5 11.25 22.5 30
30.600 24.442 38.644 9.411 16.359 25.794 46.973 57.809
total_iter: 30000
mean median rmse 5 7.5 11.25 22.5 30
27.787 20.209 36.379 11.813 20.074 30.835 53.938 64.413
total_iter: 35000
mean median rmse 5 7.5 11.25 22.5 30
25.923 17.471 34.673 12.125 21.840 34.493 59.016 68.453
total_iter: 40000
mean median rmse 5 7.5 11.25 22.5 30
24.833 15.984 34.082 14.782 24.670 37.487 62.002 70.908
total_iter: 45000
mean median rmse 5 7.5 11.25 22.5 30
23.196 14.173 32.517 17.344 28.517 41.961 65.105 73.120
total_iter: 50000
mean median rmse 5 7.5 11.25 22.5 30
22.580 13.074 32.308 19.077 31.062 44.797 66.849 74.086
total_iter: 55000
mean median rmse 5 7.5 11.25 22.5 30
22.276 13.061 31.848 18.954 30.767 44.799 67.221 74.752
total_iter: 60000
mean median rmse 5 7.5 11.25 22.5 30
22.371 13.480 31.842 18.548 30.219 43.702 67.058 74.760
total_iter: 65000
mean median rmse 5 7.5 11.25 22.5 30
21.183 12.596 30.262 19.146 31.754 46.001 68.717 76.282
total_iter: 70000
mean median rmse 5 7.5 11.25 22.5 30
20.808 12.078 29.944 21.470 33.815 47.596 68.965 76.379
total_iter: 75000
mean median rmse 5 7.5 11.25 22.5 30
20.328 11.331 29.803 22.613 35.818 49.755 70.425 77.408
total_iter: 80000
mean median rmse 5 7.5 11.25 22.5 30
20.401 11.620 29.554 20.897 34.330 48.873 70.140 77.281
total_iter: 85000
mean median rmse 5 7.5 11.25 22.5 30
20.059 11.298 29.339 23.348 36.078 49.856 70.490 77.533
total_iter: 90000
mean median rmse 5 7.5 11.25 22.5 30
19.746 10.673 29.418 25.366 38.233 51.694 71.325 78.140
total_iter: 95000
mean median rmse 5 7.5 11.25 22.5 30
19.476 10.660 28.776 24.273 37.805 51.772 71.517 78.454
total_iter: 100000
mean median rmse 5 7.5 11.25 22.5 30
19.137 10.321 28.540 26.420 39.461 52.761 72.026 78.773
total_iter: 105000
mean median rmse 5 7.5 11.25 22.5 30
19.244 10.469 28.610 24.954 38.532 52.352 72.156 78.914
total_iter: 110000
mean median rmse 5 7.5 11.25 22.5 30
19.027 10.308 28.347 25.833 39.239 52.799 72.235 79.080
total_iter: 115000
mean median rmse 5 7.5 11.25 22.5 30
18.652 9.862 28.041 27.609 40.954 54.154 72.876 79.456
total_iter: 120000
mean median rmse 5 7.5 11.25 22.5 30
18.546 9.805 27.941 28.195 41.376 54.201 72.972 79.622
total_iter: 125000
mean median rmse 5 7.5 11.25 22.5 30
18.567 9.785 27.950 28.098 41.399 54.253 72.857 79.541
total_iter: 130000
mean median rmse 5 7.5 11.25 22.5 30
18.445 9.632 27.951 29.051 42.102 54.705 73.074 79.707
total_iter: 135000
mean median rmse 5 7.5 11.25 22.5 30
18.365 9.522 27.847 29.200 42.398 55.026 73.235 79.766
total_iter: 140000
mean median rmse 5 7.5 11.25 22.5 30
18.451 9.656 27.882 28.579 41.888 54.664 73.122 79.694
total_iter: 145000
mean median rmse 5 7.5 11.25 22.5 30
18.339 9.501 27.833 29.475 42.558 55.075 73.275 79.780
total_iter: 150000
mean median rmse 5 7.5 11.25 22.5 30
18.345 9.553 27.780 29.217 42.347 54.920 73.202 79.767
total_iter: 154560
mean median rmse 5 7.5 11.25 22.5 30
18.319 9.507 27.768 29.469 42.577 55.031 73.219 79.774

Hi, have you tried training on the full NYUv2 dataset which consists of 30K images?

I have also tried to expand the number of epochs.

But it has never been reproduced successfully.
Do you need me to provide anything? For example, code?
I'm worried that I won't be able to reproduce it due to my own changes, so I haven't changed anything except the address.

I meet the same problem in Scannet dataset. The loss seems not converging well.
But when I train the model in my own dataset, the result is pretty good.
After visualizing the predicted result in Scannet, I found the model learn the data bias in early training phase. The bias is not previous in my own dataset.

@zhangshaos
Can you train the accuracy of the article using the ScanNet dataset?
I saw at the time that the Scannet dataset was too large and my computing resources were not very large, so I have been trying to reproduce the accuracy of NYU.
But I'm curious about the accuracy of the ScanNet dataset.

@zhangshaos
I don't quite understand what you're saying about data bias？

@baegwangbin
I have been thinking about why the accuracy of 85.17 cannot be reproduced.
I guess the problem may be with the data. Would it be convenient for you to publicly disclose the processed dataset? Baidu Netdisk or other Netdisk can be used.
The dataset I am currently using is inferred from your processing steps. The data provided by GeoNet is a floating-point precision label, but your label data is an uint8 type image. I processed the annotation to the image in reverse based on the data in your paper.
Or did you include test data in the training txt in the final training?

@BayMaxBHL Hi, sorry for the delayed reply. I took GeoNet's data and saved them into uint8 png type by doing ((n+1) * 0.5) * 255.

@baegwangbin
Then I really don't know why...
The accuracy of 85.17 in the paper is only 30914 images of the training part, or is it supplemented by the test set?

Do you also save two log files?

I choose a random subset from the total ScanNet dataset. I split it into 23228 pictures for training and 5639 pictures for testing.
I make other hyper parameters default except visualization params.
In early phase, the NN learns nothing and just output a direct result:

In middle phase, it output over smooth result:

In last phase:

But for my own dataset, the NN converge in the early phase. And output pretty good result in last phase.
I guess the slow converging in early phase ruins the final output.

@baegwangbin
Do you still have the two log files for training NYU? I really want to get them.

@BayMaxBHL

For obvious reasons, test images were not included in the training set
Yes I also save two log files
When evaluating on NYUv2, which image/GT are you using? Are the images in .png format or .jpg? Is the GT defined for pixels with invalid depth? For some versions, the holes in the depth map are filled and the GT normal is computed from the filled depth map. This shouldnt be the case and the GT normal should only be defined for the pixels with valid depth.

@baegwangbin
Firstly, I should apologize to you. I should not question the test data during training. I was also too anxious.
Regarding data, both testing and training data are saved in png format. Excluded invalid information from deep data.
I am sure there is no problem with the test data because I used the weight file you provided for evaluation, and the accuracy can reach 85.17 (consistent with the paper).
The main problem is that I am unable to train weights with the same accuracy through training code and default configuration, and even the difference is far (the accuracy can only reach 79).
So I really hope to obtain two log files from your training at that time. So examine what issues I have with my reproduction.
In the end, I would like to apologize to you again for the harm caused by my impulsive words before.


Dirichlet distribution

If your code uses Dirichlet distribution, the src/server/base.py file will get an error: self.client_id_indices, self.client_num_in_total = get_client_id_indices(self.args.dataset)

Thank you for your answer.
In the case of --type sample, can your code ensure that the data is not I.I.D?

Close this issue due to the inactiveness.


Deduplicate information in configuration documentation

Summary
This changes deduplicates information in the configuration section of the documentation, corrects a numbering error, and makes it (hopefully) clearer to read.
Test Plan
Manually checked the change by running MkDocs locally. All unit tests and pre-commit checks pass.

Thanks, I agree that this is an improvement.


clean up some tests

Fix some code that resulted in deprecation warnings, and other code that didn't follow unit testing conventions ...

Looks good!


chore: automate update of installation docs on release

What this PR does / why we need it:
Updates installation link (https://raw.githubusercontent.com/Azure/eraser/master/deploy/eraser.yaml) to new version (https://raw.githubusercontent.com/Azure/eraser/v1.0.0/deploy/eraser.yaml) on release pr.
Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #368
Special notes for your reviewer:

Codecov Report

Merging #595 (f457a8e) into main (c522a5d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #595   +/-   ##
=======================================
  Coverage   15.10%   15.10%
=======================================
  Files          13       13
  Lines        1523     1523
=======================================
  Hits          230      230
  Misses       1267     1267
  Partials       26       26

Flag
Coverage Δ

unittests
15.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Rate / attitude controller tuning

The fixed wing attitude controller tuning depends on getting the rate P, I and outer loop correct. As we have no rate control mode (yet), its however challenging to tune the rate controller correctly independently of the P / feedforward gain.
Paul found a way to link the two gains statically, which makes tuning much less error prone:
https://github.com/PX4/Firmware/blob/fw_att/src/lib/ecl/attitude_fw/ecl_pitch_controller.cpp#L72
While we will want to carefully understand the approach, its in many ways elegant - because at the end of the day the performance of a controller is the combined theoretical performance + the ability to tune it, and this fixed coupling prevents users from getting bad results because they try to change the pitch response with the wrong gain.

We still want this eventually but it's a feature request and nobody picked it up. Closing.


Close or minimize to tray [feature request]

I'd love to be able to natively close or minimize to Windows tray and have a preferencs toggle to set up tray behaviour.
Not sure if there is any similar functionality in other oses, I am purely Windows guy.

I have the same request, but on Linux


A better way: fix compiler warnings by adding 'else' return value

A better way to fix it
original pull request:
https://github.com/Sygmei/ObEngine/pull/181

Thanks :) You could also have commited in the original PR !