Test failures using PyTorch master

While most test failures on pytorch-0.5.0 are resolved, the following tests are still failing with the latest master branch. Some of these are recent failures.
CPU Tests:

[x] #1417 tests/ops/test_contract.py::test_ubersum_sizes (#1414)
[x] #1590 test_svi_irregular_batch_size[JitTraceEnum_ELBO]
[x] many HMC/NUTS tests with step size adaptation (#1487) wontfix - this is an upstream pytorch issue.
[ ] tests/contrib/oed/test_ewma.py::test_ewma - Assertion failures (#1373)
[ ] test_eig_lm[model5-design5-y-w-donsker_varadhan_eig-args5-True-0.3] (#1373)
[ ] test_eig_lm[model10-design10-y-w1-donsker_varadhan_eig-args10-True-0.3] (#1373)
[x] tests/infer/test_enum.py::test_mixture_of_diag_normals (Assertion failures) (#1425)

Closed because these tests have been solved in #1617.


Allow a global TODO.md file

It would be nice if you could specify a path to a single, generic TODO.md file in settings.json.
This could let coddx function as a more generic task management system.
Sample settings.json file:
{
   "coddx.use_generic_todo_md": true,
   "coddx.todo_md_path": "path to the TODO.md file"
}

+1 to this - I'd like a global one as well. Often I'm in multiple repos and code bases to accomplish one task.

This will be released shortly:

@sbruce @katiawheeler

v0.2.15 is available now. Let me know if it works. Thanks.

This is great! Thanks @ngduc!

Thanks! @ngduc


Investigate test failures.

When running ctest locally I get failures in:

[x] spirv-tools-test_spirv_unit_tests (Failed)
[ ] glslang-testsuite (Failed)
[ ] vk_loader_validation_tests (Failed)
[ ] vk_layer_validation_tests (SEGFAULT)

The glslang-testsuite is added by shaderc and does things to copy and rewrite the runtest script. There are CLs out (https://github.com/KhronosGroup/glslang/pull/1933) and (https://github.com/google/shaderc/pull/838) to move the glslang-testsuite into glslang and use the cmake variables to retrieve the binary paths.

The loader test looks like the suite which is added with add_test in the ctest file will always fail when run. There are layers which need to be exported in order to execute the tests. We should change the add_test to call the run test script and figure out how to make it work on Windows.


[Incompat] Tierify does not work

Mod Name
Tierify
Mod Homepage
https://legacy.curseforge.com/minecraft/mc-mods/tierify
Minecraft version
1.20.1
Describe the bug
Crash on start-up
Steps to reproduce

Install Tierify
Install Necronomicon API (dependency)
Launch

Logs
https://gist.github.com/Cixon/9b95e62bec2238a5d0e2c64f5681c5bc
Additional context
Using Connector-1.0.0-beta.41+1.20.1
Neoforge - 47.1.99

Extra context:
dependency works just fine alone // Tierify seems to be the issue
https://legacy.curseforge.com/minecraft/mc-mods/necronomicon

Moved to #1044, which better describes the issue


ENH: sparse: speedup LIL matrix slicing

The current implementation for slicing LIL matrices is very pessimal. Replace it with a more optimal implementation.
Gives a factor of ~1/density speedup:
In []: A = rand(3000, 3000, density=1e-3, format="lil")

# Before (9ccc68475)
In [4]: %timeit A[::2,::-2]
10 loops, best of 3: 87.5 ms per loop

# After (d47e27b0)
In [4]: %timeit A[::2,::-2]
1000 loops, best of 3: 603 µs per loop

Gives speedups also for smaller slices:
# Before
In [5]: %timeit A[3,:]
1000 loops, best of 3: 213 µs per loop
In [6]: %timeit A[:,3]
1000 loops, best of 3: 901 µs per loop
In [7]: %timeit A[1:5,1:5]
10000 loops, best of 3: 83.3 µs per loop

# After
In [5]: %timeit A[3,:]
10000 loops, best of 3: 38.8 µs per loop
In [6]: %timeit A[:,3]
1000 loops, best of 3: 702 µs per loop
In [7]: %timeit A[1:5,1:5]
10000 loops, best of 3: 40.3 µs per loop

(Column slices in LIL are still problematic, but that's due to the matrix format.)
EDIT updated benchmarks

This needs only a rebase and a very minor tweak it looks like. Time to get it in?

This needs only a rebase and a very minor tweak it looks like. Time to get it in?

Rebased in #5789, closing.

Reopening so that the status of this will be "merged" when I merge #5789.


MINOR: convert RackAwareAutoTopicCreationTest.scala to KRaft

hmm, looks like this is a duplicate of #15318 . I will review that one.

Closing as duplicate.


Add manual locale override capability

Jira issue:
https://glia.atlassian.net/browse/MOB-2798
What was solved?
The configuration accepts a manual locale override, which is nil by default. If the Core SDK returns an incorrect manual locale override, then the Widgets SDK also returns an error on configuration to let know the integrator about it. If the integrator provides a wrong site API key, this commit also makes sure to inform the integrator instead of hanging.
Also, the settings screen now includes the possibility to add a manual locale override visually. If it's empty, then it sends nil.
Release notes:

[ ] Feature
[ ] Ignore
[x] Release notes (Is it clear from the description here?)
[x] Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

There's a new completion that is mandatory and that gives a Result, so that the integrator knows if there has been an issue with configuration instead of figuring it out when starting the engagement. Also, when creating the configuration, the integrator can add a manualLocaleOverride.
Additional info:

[ ] Tests fixed, added? Unit, acceptance, snapshots?
[ ] Logging necessary for future troubleshooting of customer issues added?

Screenshots:

!squash


Unable to download a particular video

I was unable to retrieve https://www.youtube.com/watch?v=QPRc0NIeQrQ
I checked other youtube videos and had no issue.  The tool is outputting the error in the attached text file.  Has anyone encountered
yt-error.txt
this in the past?

You are using an outdated version, you have to update youtube-dl.


Revert "Repo sync"

Reverts github/docs#27250

I feel like I need to give up on GitHub because of Microsoft. Nothing connects
Sent from Yahoo Mail on Android
On Sun, Aug 13, 2023 at 4:51 AM, @.***> wrote:
Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

El dom, 13 de ago de 2023, 4:41 a. m., Dale T @.***>
escribió:

I feel like I need to give up on GitHub because of Microsoft. Nothing
connects
Sent from Yahoo Mail on Android
On Sun, Aug 13, 2023 at 4:51 AM, @.***> wrote:
Thanks for opening this pull request! A GitHub docs team member should be
by to give feedback soon. In the meantime, please check out the
contributing guidelines.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/27423#issuecomment-1676330229, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A72X273OSOVTOSKB2ITXZLLXVC4P5ANCNFSM6AAAAAA3OSBMFY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

El dom, 13 de ago de 2023, 5:47 a. m., Jose Ramos <
@.***> escribió:

El dom, 13 de ago de 2023, 4:41 a. m., Dale T @.***>
escribió:

I feel like I need to give up on GitHub because of Microsoft. Nothing
connects
Sent from Yahoo Mail on Android
On Sun, Aug 13, 2023 at 4:51 AM, @.***> wrote:
Thanks for opening this pull request! A GitHub docs team member should be
by to give feedback soon. In the meantime, please check out the
contributing guidelines.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/27423#issuecomment-1676330229, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A72X273OSOVTOSKB2ITXZLLXVC4P5ANCNFSM6AAAAAA3OSBMFY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Embeddable object cannot be used as query parameter

We have an embeddable object which represents geographic coordinate. When using this object as query parameter with custom DBAL type Doctrine tries to load identifier for this object, which does not exist, because embeddables are simple DTO objects (as per doc).
Error with stack trace:
Error: Call to a member function getValue() on null

vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php:732
vendor/doctrine/orm/lib/Doctrine/ORM/UnitOfWork.php:2920
vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php:429
vendor/doctrine/orm/lib/Doctrine/ORM/Query.php:371
vendor/doctrine/orm/lib/Doctrine/ORM/Query.php:321
vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php:974
vendor/doctrine/orm/lib/Doctrine/ORM/AbstractQuery.php:928

Can you turn this into a failing test case?

yes, I'm working on it.

I used embeddables found in the tests models, not sure if its acceptable.

@mishal can you open a PR with the fix? Seems legit, but may need to do some adjustments. Once the PR is open, we can review it :-)

PR created #5937

Thanks! Closing this issue as duplicate :-)


fieldGroup not populating on add() when repeat type is loaded first in parent fieldGroup

Question
Do you have any idea why fieldGroup wouldnt populate the correct object when my repeat field type if first in the parent fieldGroup array?
Description
I have a fairly complex form that I've created with formly as the logic engine (see below all the custom types).
  imports: [
    FormlyModule.forRoot({
      types: [
        { name: 'repeat', component: RepeatTypeComponent },
        {
          name: 'intake-repeat',
          component: IntakeRepeatTypeComponent,
          defaultOptions: { defaultValue: [] }
        },
        {
          name: 'customer-repeat',
          component: CustomerRepeatTypeComponent,
          defaultOptions: { defaultValue: [] }
        },
        {
          name: 'customer-section',
          component: CustomerSectionTypeComponent,
          defaultOptions: { defaultValue: {} }
        },
        {
          name: 'intake-section',
          component: IntakeSectionTypeComponent,
          defaultOptions: { defaultValue: {} }
        },
        {
          name: 'customer-form',
          component: CustomerFormTypeComponent,
          defaultOptions: { defaultValue: {} }
        },
        {
          name: 'intake-form',
          component: FormIntakeTypeComponent,
          defaultOptions: { defaultValue: {} }
        },
        {
          name: 'question-group',
          component: QuestionGroupTypeComponent,
          defaultOptions: { defaultValue: {} }
        },
        { name: 'address', component: AddressComponent },
        { name: 'input', component: CustomInputComponent },
        { name: 'firstName', component: CustomFirstNameComponent },
        { name: 'lastName', component: CustomLastNameComponent },
        { name: 'select', component: CustomSelectComponent },
        { name: 'radio', component: CustomRadioComponent },
        { name: 'checkbox', component: CustomCheckboxComponent },
        { name: 'phonenumber', component: PhoneNumberComponent },
        { name: 'textarea', component: CustomTextareaComponent },
        { name: 'table', component: CustomTableComponent },
      ],
      wrappers: [
        { name: 'customer-section-panel', component: FormViewerCustomerSectionPanelWrapperComponent},
        { name: 'question-panel', component: FormViewerQuestionPanelWrapperComponent },
        {
          name: 'section-panel',
          component: FormViewerSectionPanelWrapperComponent,
        },
      ],
    })
....

In this form I have a repeat type similar to your stepper example that shows or hides sections based on the active index. In order to get the repeat type to work, I have to run the add() method on init so that the data is populated correctly (see below):
.ts file
  ngOnInit() {
    if (this.field.fieldGroup && this.field.fieldGroup.length === 0) {
      this.add();
    }
  }

.html file
<div
    *ngFor="let fg of field.fieldGroup; let index = index;">
    <formly-field *ngIf="to.subIndex === index" [field]="fg"></formly-field>
</div>

Ideally I'd create an example on stackblitz but I can't seem to reproduce it and because its a complex implementation its hard to copy everything over.
My question to you is do you have any idea why fieldGroup wouldnt populate after add() is ran on a repeat type similar to the stackblitz below? Usually in less complex examples where Im using the repeat type, both fieldArray and fieldGroup have values once add() is ran but when I have the repeat type first in the parent fieldGroup array the fieldGroup value is empty after add() is ran. If you look at the two images below, one is the actual app where fieldGroup doesnt populate after add() and the other is a less complex stackblitz (https://formly-cf-test.stackblitz.io/) where it does. Both have similar setups.
Empty fieldGroup
https://ibb.co/vx7SkN7
Not Empty
https://ibb.co/FYYbMgR
Any guidance would be much appreciated!

fieldGroup is populated based on the model value so:

Ensure add function is called
Check the field.model and the root model values they should both include the new item after the add call, else somehow the field model is detached from the root model.
use setTimeout before calling add

The setTimeout worked, thank you!

Great, setTimeout is just a workaround so don't hesitate to reopen in case you've got a reproduction.


Initial value with getNormalizedX() or Y is 0

Like the title says, when I haven't touched the joystick and use getNormalizedX() or getNormalizedY() the value is 0 instead of 50. This is my xml:
<io.github.controlwear.virtual.joystick.android.JoystickView
                xmlns:custom="http://schemas.android.com/apk/res-auto"
                android:id="@+id/joystick"
                android:layout_width="180dp"
                android:layout_height="180dp"
                custom:JV_autoReCenterButton="true"
                custom:JV_backgroundColor="#20000000"
                custom:JV_backgroundSizeRatio="70%"
                custom:JV_buttonColor="#000000"
                custom:JV_buttonSizeRatio="30%" />

I expected the same issue! @makowildcat do you know how to fix this?

Hello @GijsGoudzwaard and @henri98
I just made a quick fix (work around), just update to 1.10.1 and it should be fine for now.
Thank you for your feedback!
I will close the issue in a couple of days.
Kindly,
Damien


Custom layouts keep getting deleted from fancy zones editor

Microsoft PowerToys version
0.61.1
Running as admin

[X] Yes

Area(s) with issue?
FancyZones Editor
Steps to reproduce
At the moment, I'm not sure what the steps are or what's causing it I will just suddenly go to snap windows and my layouts will be gone.
PowerToysReport_2022-08-15-09-26-05.zip
✔ Expected Behavior
Custom layouts to persist between powertoys updates and windows updates ect.
❌ Actual Behavior
Custom layouts disappearing from the layout editor.
Other Software
No response

/dup #20239


They don't load cmproject files in blockbench

When opening a .cmproject file it does not load, this happens to me with all the files of this type, but when opening it in minecraft everything loads correctly.

modelmonkey.zip

Open your model in-game, save it, so it is updated, then try Blockbench, some older models don't load in Blockbench.

Duplicate of #186


[tests] testing fixes

to @austinbyers
cc @airbnb/streamalert-maintainers
size: small
changes

Updating the boto3 lambda invocation to utilize a qualifier if one is provided in the output configuration.
Updating lambda output/alert tests to go with above change.
Patching a function to load the unit test outputs configuration during tests to avoid breaking tests with production configuration changes.

Coverage decreased (-0.1%) to 79.191% when pulling 9b989ac616053e3ded7d34a4ebb491268ec38a7b on ryandeivert-testing-fixes into 3d5e9fd4f96e39b5f9a1ffd0fac1e4ed76863216 on master.

Coverage decreased (-0.1%) to 79.191% when pulling 9b989ac616053e3ded7d34a4ebb491268ec38a7b on ryandeivert-testing-fixes into 3d5e9fd4f96e39b5f9a1ffd0fac1e4ed76863216 on master.

Coverage decreased (-0.1%) to 79.191% when pulling d90b8ff7868d14ae12f54be1923848c96bd9f23b on ryandeivert-testing-fixes into 3d5e9fd4f96e39b5f9a1ffd0fac1e4ed76863216 on master.


Dev playback func

test PR validation

Can one of the admins verify this patch?

If PowerMock is being used elsewhere, I'm fine with adding it as a test dependency. Relatedly, did you look into Spock mocking at all (Spock is the actual test framework and has its own mocking functionality)

Is it not possible for the recording framework to use the name of the file/type when creating the json file instead of prepending the type name to all the method names?

If PowerMock is being used elsewhere, I'm fine with adding it as a test dependency. Relatedly, did you look into Spock mocking at all (Spock is the actual test framework and has its own mocking functionality)

I did not find it in our dependency days before when I develop. But I realized eventhub recently imported as an dependency. I will chat with Mayuri to the decision.
To Spock and java test, they are not able to mock static method. Link
GroovyTest is only able to do that if src code is in Groovy as well, which is a limitation to us.

Is it not possible for the recording framework to use the name of the file/type when creating the json file instead of prepending the type name to all the method names?

I raised the design. However, @jianghaolu suggested we do a quick change by changing the test name instead of a system change. So we decided to move forward simply change the test method name.

Live test results:
https://dev.azure.com/azure-sdk/playground/_build/results?buildId=45590

@sima-zhu has spoken with the storage team and we don't think we need this going forward.


Question regarding AWS Lambda and javaslang

Hi there! I am sorry if this is not the right place but I don't know where else to ask.
I am building a small application that will run on AWS Lambda and I use the javaslang library for it.
The problem I have is that when I invoke the function, I get the following error:
javaslang/λ: java.lang.NoClassDefFoundError java.lang.NoClassDefFoundError: javaslang/λ
The weird part is that the class is there in the packaged jar. When I use a shell to manually run the jar it works.
Has anyone had similar problems? I maybe think it is a problem with AWS Lambda and not recognizing properly the λ character(ironic).
I know this is not the right place probably but I have been hitting my head for 2 days now.

Hi @geolich, thx for using Javaslang. I haven't heard about that specific problem yet. It might be a known AWS Lambda problem that is not related to class names but to packaging of dependencies.
Please take a look at this: http://stackoverflow.com/questions/32782980/aws-lambda-noclassdeffounderror
There are links provided how to use the maven-shade plugin to circumvent that problem.
Greets
- Daniel

Hi @danieldietrich Thanks for your quick reply. I found that it is a problem with AWS Lambda. I created a test λ.class and tried to run it and got the same error. It has a problem to load classes with special characters. A bit ironic that AWS Lambda cannot detect the lambda character.
Thank you for your help.

Thx!


[tf.keras] Enable tf.keras.callbacks.CallbackList API for tf.keras

In keras I'm used to use keras.callbacks.CallbackList I'd like to be able to use it also in the tf.keras this PR should enable the API tf.keras.callbacks.CallbackList.

@karmel @harshini-gadige any progress reviewing this PR? It'd be nice to have this in one of the next tensorflow releases. I assume that this might be beneficial for people that are used to standard keras in order to have the same API also for the tf.keras.

Thanks for the PR. Can you clarify why you want to use CallbackList as a public API?

Assigning to @fchollet for thoughts on exposing CallbackList to the public API.

@fchollet @karmel any progress reviewing this PR.

@fchollet @karmel @harshini-gadige any progress reviewing and possibly merging this one-line PR?

@fchollet @karmel @harshini-gadige any progress reviewing and possibly merging this one-line PR?

@karmel  @fchollet  Any update please ?

@omalleyt12 do you see any downsides here? I am okay with this in theory.

@omalleyt12 do you see any downsides here? I am okay with this in theory.

Hm, I'm torn because It's mostly an implementation detail, but I see the benefit for the use @ziky90 is mentioning. I don't see CallbackList documented in external Keras docs https://keras.io/callbacks/
2 potential downsides are:

Users might do: model.fit(x, y, callbacks=keras.callbacks.CallbackList([cb1, cb2])) which will actually short-circuit our Callback configuration logic as it's currently written

If anyone subclasses this class and overrides methods for use in Keras's own fit/eval/predict, there will be some unintuitive behavior. (Methods like on_test_batch_begin are never called directly in these loops, instead _call_batch_hook is called). But I don't see many users wanting to do that.

@ziky90 you ought to be able to do:
from tensorflow.python.keras.callbacks import CallbackList

I'm not sure where to submit this, but this line of code wasn't in the 2.0 version I built from source. I added it in manually to my local version, but it still did not fully fix the problem.
Basically, I can use
from tensorflow.python.keras.callbacks import CallbackList
but not
from tensorflow.keras.callbacks import CallbackList
after adding the line of code in the original pull request above.
To fix this, I had to go into the machine-generated code (not ideal I know) in "tensorflow/_api/v1/keras/callbacks/init.py" and add
from tensorflow.python.keras.callbacks import CallbackList
Perhaps someone could reopen this pull request, add that to the build code (that creates the machine-generated code) and close it? That's kind of outside of my knowledge. Thanks!


test_imds_tracer: check bcc-tools working before deploying aws-imds-packet-analyzer

Thanks, @libhe I will merge it.


Github Actions - allow updating major versions only

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
GitHub Actions follow docker-like tag pattern, where a new version updates 3 tags simultaneously:

full version: v1.2.3
minor version: v1.2
major version: v1

A common practice then is to use a major version tag (action@v1) to allow automatic minor version drift, and update major versions manually (as that often implies breaking changes)
At the moment, Dependabot is not aware of this convention, and unconditionally suggests the "max" version. This leads to situation when an action updates an action v2 to v3.0.0 example, and that leads to a lot of extraneous PRs after (updating 3.0.0=>3.0.1, 3.0.1=>3.0.2 etc.)
Suggestion: add versioning-strategy value for GitHub actions (e.g. major) which will instruct Dependabot to exclusively use tags where only major version is specified

Note: I am aware of ignore: update-types: mechanic, but this is slightly different as ignore will still result in dependency being updated to full version spec, i.e. v1.2.3

Mmmm, this feels like a deja vu... I thought we had already discussed and fixed this. This is what everyone expects, Dependabot should not be proposing an upgrade that uses a different "versioning style", and instead respect what's already there.
I'll have a look at what's going on, thank for letting us now and providing a public repo as a example 🥇.

Hei @DarkWanderer!
So I tried this using:

Our dry-run.rb script.
Dependabot's CLI.
A fork of your repo.

All three times Dependabot bumped the dependency from v2 to v3 as expected.
There's still something fishy going on here since Dependabot did open an incorrect PR on your repository, but I have no idea how that happened. This should've been working as expected 17 days ago too, since no related changes have been released since that I know of.
Anyways, I'm going to close this, if you ever get another PR by Dependabot showing this behavior, please reopen this issue as quickly as possible so we can figure out the root cause and fix it!
Thanks for your helpful report, by the way!

No worries. This is an excellent tool and I'll be happy if I can contribute back.
One thing I noticed is that setup-dotnet repo uses manual release scheme, and a separate action to mark 'major tag, e.g.:

3.0.0 version was released manually at 29 Sep 17:56 CEST
Additional workflow was triggered which ran at 30 Sep 8:28 CEST (I assume also manually).
The pull request in question was raised at 30 Sep 3:55 CEST - i.e. after creation of v3.0.0 tag, but before v3.

Speculatively, this time gap might explain both the bug and failure to reproduce it - as now there is a valid v3 tag available.
I assume that in the absence of 'correct' v3 tag, Dependabot just defaults to the highest available
P.S. And I guess my search skills could use a brush-up, there is actually a prior on this: #2304

Wow, fantastic investigation! That makes total sense and I'm sure that's the cause of the issue.
Dependabot should always respect the precision chosen by the user, and it already does this in general, except for this normally very small time frame where an action has a release, but no "major tag/branch" pointing to it.
I will fix this.

Ok I created https://github.com/dependabot/dependabot-core/issues/5891 to fix this. I learnt though that there's a reason for the current behavior, so it will need some team discussion to figure out whether it should be changed or not.


Problem setting this up with Electron

I'm trying to add rich presence to an electron app.
I've tried the default code:
After accepting discord auth, it redirects to my redirect Url, but not access token, I got this instead : http://localhost:4242/oauth/redirect#error=invalid_scope
So I tried removing scopes. I just kept rpc.api scope.
This time, I get the token.
But when I try to
client.login(config.clientId, {
        accessToken: token,
        scopes     : config.scopes
});

all I get is

Could you help on that ?
Thanks

you can't use rpc.api without being whitelisted. if you want to use rich presence you should use IPC not WebSockets, see the source code in this repo for a pretty good implementation.

you can't use WebSocket for rich presence, you have to use IPC.

If I use the Rich Presence example on the readme I get:
Cannot read property 'accessToken' of undefined

Ok, updated from beta1 to beta2 and now it show Ready, setting rich presence but no rich presence
It still show the old format

https://discordapp.com/developers/docs/rich-presence/faq#q-i-don't-see-any-playing-status-for-my-game

I've already done that.
As i've told you It still show the old format
I can also see that the event SET_ACTIVITY  is correctly sent to discord with - from what I can understand - correct datas

Is there an error event I can listen to ?

I got it working now.
It's working with nodejs and electron
For anyone in the same case as me, you must put the code inside the main.js file and not the renderer.js file.
I still don't know why, I will try to find more infos.
You can still communicate between the main and the renderer process using IPC or any other methods

it has to go in main because its a separate process from renderer. Discord will think that your process has no window and therefore won't think its a game if its in the renderer

Ok, thanks 👍


Added aci discovery tags into head meta tags

This is the first step to getting ipfs.io to serve aci files.
Next is to update the go-ipfs make file to generate the aci file.
Then sign and upload to https://dist.ipfs.io/ in the correct place with a gpg key available.
Signed Off: Simon Kirkby<EMAIL_ADDRESS>

Thanks zignig! :+1: Wanna have a look at the previous discussion about signed builds too? ipfs/distributions#51 too?

rkt run ipfs.io/ipfs --debug
image: searching for app image ipfs.io/ipfs
image: meta tag 'ac-discovery' not found on ipfs.io/ipfs
└─expected a 200 OK got 400
image: remote fetching from URL "https://dist.ipfs.io/images/ipfs.io/ipfs-latest-linux-amd64.aci"
image: fetching image from https://dist.ipfs.io/images/ipfs.io/ipfs-latest-linux-amd64.aci
pubkey: meta tag 'ac-discovery-pubkeys' not found on ipfs.io/ipfs
└─expected a 200 OK got 400
image: error adding keys
└─error accessing the key https://dist.ipfs.io/ipfs.gpg
└─error getting key
└─Get https://dist.ipfs.io/ipfs.gpg: dial tcp [2604:a880:1:20::203:d001]:443: connect: network is unreachable
image: downloading signature from https://dist.ipfs.io/images/ipfs.io/ipfs-latest-linux-amd64.aci.asc
fetch: bad HTTP status code: 404
it works !
image : https://dist.ipfs.io/images/ipfs.io/ipfs-latest-linux-amd64.aci
key :  https://dist.ipfs.io/ipfs.gpg
ascii sig : https://dist.ipfs.io/images/ipfs.io/ipfs-latest-linux-amd64.aci.asc


merge package from arlac77/npm-package-template

package.json

chore(devDependencies): update rollup:^0.59.4 from template
chore(devDependencies): add rollup-plugin-commonjs:^9.1.3 from template
chore(devDependencies): add rollup-plugin-node-resolve:^3.3.0 from template
chore(devDependencies): add rollup-plugin-babel:^3.0.4 from template
chore(devDependencies): add rollup-plugin-json:^3.0.0 from template
chore(devDependencies): add rollup-plugin-executable:^1.2.0 from template
chore(devDependencies): add babel-preset-env:^1.6.1 from template

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


not implemented step function

I need step function for Perceptron Neuron.
I make  PR https://github.com/scalanlp/breeze/pull/753

thanks, sorry about being slow


[New Hub] [Aug 9] binder.nasa-veda.2i2c.cloud

A part of https://github.com/2i2c-org/meta/issues/1368
A part of https://github.com/NASA-IMPACT/veda-jupyterhub/issues/47
Phase 1: Account
Done
Phase 2: Cluster
Use existing nasa-veda cluster
Phase 3: Hub Setup
Hub 1: binder
Phase 3.1: Initial setup (READY)

Question
Answer
Notes

Name of the hub
binder

Dask gateway?
no

URL
binder.nasa-veda.2i2c.cloud
(may change)

BinderHub UI?
yes

Authentication Mechanism
None

Admin Users
NA
Because BinderHub UI will be used

Persistent Home Directory
no

Phase 3.2: Object storage access (READY)

Question
Answer
Notes

Scratch bucket enabled?
yes
retention set to 1 days, instead of 7

Persistent bucket enabled?
no

Requestor pays requests to external buckets allowed?
no

Phase 3.3: Profile List (READY)
N/A, as BinderHub UI will be used
Phase 3.31: Resource configuration (READY)
No profile list, properties set under singleuser

Question
Answer

memory limit
2G

memory request
1G

CPU limit
1

Phase 3.4: Authentication tuning (READY)

Question
Answer

Authentication Mechanism
None

Phase 3.5: Profile List finetuning (NA)
NA, as BinderHub UI will be used
Phase 3.6: BinderHub config (READY)

Question
Answer

Allowed Repositories?
all

Banner Message
default

BinderHub About Message
default

Phase 4: Customizations

[x] Use binder.openveda.cloud as the community manged domain. There's already a CNAME pointing from binder.openveda.cloud to binder.nasa-veda.2i2c.cloud so all the community managed steps are taken. (Deadline: Aug 9)
[x] Apply same extra_iam_policy as the staging hub  (Deadline: Aug9)
[x] Create a new nodepool dedicated to this hub, with max size of 4 nodes and xlarge size. This is pathway to limiting cost. This should use the tags in https://github.com/2i2c-org/infrastructure/issues/4482 as well. (Deadline: Aug 14)
[x] Set max lifetime of users to 12hr. This is set as a property on the idle culler. (Deadline: Aug 14)

I will sign off for the rest of the week and this issue is almost done with the following pieces missing:

[ ] Figure out why #4597 still not works :(
[ ] because my IAM user on this cluster has expired I was not able to run and apply, the changes in https://github.com/2i2c-org/infrastructure/pull/4595
[ ] need to confirm in the user culling configuration is correct as we didn't do it before
[ ] need to decide if it's ok to enable logging of launch events to the 2i2c GCP account (I wanted to be more conservative with this one to make sure it's ok)

cc @sgibson91

@sgibson91, the issue was that I had forgotten to add the robot account to the owners team on quay 🤦🏼♀ It should work now. Sorry about it!

Thanks @GeorgianaElena!

I may have broken things further https://github.com/2i2c-org/infrastructure/pull/4595#issuecomment-2276156179

New builds are currently not working https://github.com/2i2c-org/infrastructure/pull/4595#issuecomment-2277596346

I ended up using a nodegroup-per-hub (as per https://github.com/2i2c-org/infrastructure/issues/4482) and removing the taint/toleration on the binder nodegroup. I couldn't get the build pods to schedule properly into the binder nodegroup and then they couldn't find the docker socket. This way at least works.

Outstanding Actions

need to confirm in the user culling configuration is correct as we didn't do it before
need to decide if it's ok to enable logging of launch events to the 2i2c GCP account (I wanted to be more conservative with this one to make sure it's ok)

For (1) I've spun up some binders, so we can see if they still exist in 12 hours
@yuvipanda can you provide a yes/no answer for (2)?

@sgibson91 the answer for (2) is yes! It's ok to do that for now.

I also see that the idle culler is set to cull after 4h of inactivity. Let's leave it unset so it uses the default (1h I think) instead.


Integration Test - Release - 2019/11/19 01:42:32

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#reassign PRMergerTest2

PRMergerTest2 is not a valid GitHub ID, or is not a collaborator on this repo.

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#reassign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#please-close


Incorrect/incomplete information

The section of 'Manual agent provisioning' contains incorrect steps that doesn't exist.
1st Issue: Turn off the Log Analytics agent in Environment Settings > Monitoring coverage > Settings.

There is no option of 'Monitoring Coverage' within 'Environment Settings' in MDC anymore.

2nd Issue: Set the workspace on which you're installing the agent. Make sure the workspace is in the same subscription you use in
Defender for Cloud and that you have read/write permissions for the workspace.
Select Microsoft Defender for Cloud on and Save.

There is no plan called 'Microsoft Defender for Cloud' on the workspace level. The plans are 'Foundational CSPM', 'Servers' and 'SQL'.

[Enter feedback here]

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

ID: 90bcf3a5-063b-3581-173b-50f1d5be0df9
Version Independent ID: e73871d6-a39d-6595-76d3-041542333b31
Content: Collect data from your workloads with the Log Analytics agent
Content Source: articles/defender-for-cloud/working-with-log-analytics-agent.md
Service: defender-for-cloud
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

@the-sentinental-guy
Thanks for your feedback! We will investigate and update as appropriate.

@dcurwin
Can you please check and add your comments on this doc update request as applicable.

#reassign:alizabernstein

#backlog-item-created
https://dev.azure.com/msft-skilling/Content/_workitems/edit/118205

#label:"backlog-item-created"

@dcurwin Any update on this?

@the-sentinental-guy , this seems to be fixed. Can you confirm?

Resolved per merged PR https://github.com/MicrosoftDocs/azure-docs-pr/pull/246798
#please-close


Chat templates

Moved state to the PromptState and made further changes to align with the expected templating of huggingface.

import lmql
from transformers import (
    AutoTokenizer,
)

tokenizer_string = "HuggingFaceH4/zephyr-7b-beta"

lmql_model = lmql.model(
    f"local:gpt2",
    tokenizer=tokenizer_string,cuda=True
)

tokenizer = AutoTokenizer.from_pretrained(tokenizer_string)

dialogue = [
    {"role": "system", "content": "You are a bot"},
    {"role": "user", "content": "Hey bot"},
    {"role": "assistant", "content": "hey there pal"},
    {"role": "assistant", "content": "more assistant continuation"},
    {"role": "user", "content": "Nice"}
]

@lmql.query(model=lmql_model, name="lmql_chat",chat_template=tokenizer.chat_template)
def lmql_chat():
    '''argmax
        "{:system} You are a bot"
        "{:user} Hey bot"
        "{:assistant} [ANSWER]" where ANSWER in set(["hey there"])
        "pal"
        "{:user} Nice"
    '''

out  = lmql_chat()
prompt_from_lmql = out.prompt
prompt_from_huggingface  = tokenizer.apply_chat_template(dialogue,tokenize=False)
assert prompt_from_huggingface == prompt_from_huggingface, "Prompt from lmql does not match prompt from huggingface"

Tested with this example

State handling looks good now thanks. I just wanted to test with "meta-llama/Llama-2-7b-chat-hf", however, it seems the start/end tag extraction does not work very well there.
First of all, some templates seem to expect a raise_exception callback for error. E.g. the Llama-2 template, raises an exception if we don't alternate user/system/user/system.
I ignored exceptions using the following implementation of get_start_end, but this leads to invalid rendered templates
        eos_token = self.model.adapter._tokenizer.tokenizer_impl.tokenizer.eos_token
        bos_token = self.model.adapter._tokenizer.tokenizer_impl.tokenizer.bos_token
        split_template_text = "split_template_text" # Dummy text to split the template - just needs to not be present in the template

        def raise_exception(message):
            warnings.warn(message)
        role_start, role_end = Template(chat_template, trim_blocks=True, lstrip_blocks=True)\
            .render(messages=[
                {'role':role,'content':split_template_text}
            ],
            bos_token=bos_token,
            eos_token=eos_token,
            raise_exception=raise_exception
        ).split(split_template_text)

        return role_start, role_end

I am testing with the following query:
argmax
    "{:user} Say 'this is a test'"
    "{:assistant} [RESPONSE]"
from
    "meta-llama/Llama-2-7b-chat-hf"
where
    len(TOKENS(RESPONSE)) < 10

I see, we could track the previous role as well and pass it in with another dumy text but parsing the start+end from the resulting string would be very tricky. Also for the same meta-llama/Llama-2-7b-chat-hf template returns an empty string when passing a system role without any messages after so that would also break it. Not sure I see any reasonable way to deal with these kind of templates. Would only supporting "simple" templates make sense? Or maybe make this feature more general where the user can pass a function that takes in a role, message and spits out start, end so people can deal with more complex requirements on a case by case basis?

The problem with the Llama template seems to be that it does not allow all tag interleavings as expressible with our current {:role} syntax. If users specify e.g. two system messages, it will raise an error.
I think it would be fine to limit support to simpler templates for now. For Llama, we should probably still add a custom template so that it works out of the box, as it is a common prompt template. But this can be a separate PR.
For this, it should be fine to merge if we can make sure that template errors are reported properly and that users know how to fix errors by providing a simpler template. With the current system, what is the concrete requirement a template has to satisfy to be considered "simple"? Maybe that should be specified in the error message.

import lmql
from transformers import (
    AutoTokenizer,
)

tokenizer_string = "HuggingFaceH4/zephyr-7b-beta"

lmql_model = lmql.model(
    f"local:gpt2",
    tokenizer=tokenizer_string,cuda=True
)

tokenizer = AutoTokenizer.from_pretrained(tokenizer_string)

dialogue = [
    {"role": "system", "content": "You are a bot"},
    {"role": "user", "content": "Hey bot"},
    {"role": "assistant", "content": "hey there pal"},
    {"role": "assistant", "content": "more assistant continuation"},
    {"role": "user", "content": "Nice"}
]

@lmql.query(model=lmql_model, name="lmql_chat",chat_template=tokenizer.chat_template)
def lmql_chat():
    '''argmax
        "{:system} You are a bot"
        "{:user} Hey bot"
        "{:assistant} [ANSWER]" where ANSWER in set(["hey there"])
        "pal"
        "{:user} Nice"
    '''

out  = lmql_chat()
prompt_from_lmql = out.prompt
prompt_from_huggingface  = tokenizer.apply_chat_template(dialogue,tokenize=False)
assert prompt_from_huggingface == prompt_from_huggingface, "Prompt from lmql does not match prompt from huggingface"

Tested with this example @lbeurerkellner

Don't know if this is (still) relevant, but in the last line of code the hf prompt is compared to itself instead of the lmql one. Just wanted to let you know in case you didn't catch that yet :)


feat: bootstrap_docker skips build it can pull from ecr.

As title.

@ludamad This escalated a bit. Might want another review.
When working locally, you want to be able to build images without committing. This attempts (I think correctly, at least in my tests), to compute the correct hash from the working tree.
We could take a dumb approach of just clobbering the content hash in some way for any project that has files within its rebuild path modified, but this does have the nice property that your image has the right hash after committing, otherwise you end up needing to rebuild the image right after committing it.


Base64 Blob

How can I use this with a base64 blob???
It wants to open files... But, I want to pass it the blob..

Hello @atebit!
This gem works with any io-like object, as long as it responds to methods like read, seak.
To prepare base64 encoded mp3 file for consumtion by ID3Tag you need to decode it and pass to StringIO class.
Then you can pass this object to ID3Tag.read as any other file.


ngsF directory missing

the NGS_F.sh script points to a ngsF directory that doesn't exist

Yep, need to add in the source code for that.

Added in latest commit.


"ProviderFailed" status

I getting "provider failed" status when I try to deploy windows nanoserver-iis usiing given nano-iis.yaml.I tried multiple time by creating new AKS and getting same issue.please help.
PS Azure:> kubectl get pod
NAME                                                              READY   STATUS           RESTARTS   AGE
aci-helloworld-7b9ffbf946-gtxxj                                   0/1     ProviderFailed   0          4m11s
nanoserver-iis-5d999b87d7-5c4md                                   0/1     ProviderFailed   0          10s
virtual-kubelet-linux-eastus-virtual-kubelet-for-aks-89795vkzw9   1/1     Running          0          6m43s
virtual-kubelet-windows-eastus-virtual-kubelet-for-aks-5f7zw4fk   1/1     Running          0          6m36s
nano-iis.yaml.txt

@Sujittest1987  Can you provide more info on the above query and also share us document which you are  referring too

@Sujittest1987 Since we have not heard back from 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 reopen the issue and continue the discussion.


Implement JS code to test dead blocks and uninitialized uses!

label:enhancement
Load and store cases in Different JIT levels ie such as
"var i = []; //declare an Array varible"
"do something..."
"i[0];i[1];i[0];i[1] //repeatedly for maybe 20 more times to take up space on the stack and use up registers with get_by_val op.
//which will probably force JIT compilation to spill certain registers to stack....
"do something crash accessing some unitialized register"

After finding a reproducible POC with some modifications to fuzzilli and had 17 crash cases with just 19% coverage... I took notice of the corpus and the crash POC that fuzzilli yielded i took note that it if anything mostly what the code was generating would if anything trigger use after free, type confusions, and or oob accesses but not specifically unitialized use directly.


fix, sort poddecorations

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

fix https://github.com/KusionStack/operating/issues/187

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

already fixed in #168


CSA Week 26 Review Ticket

Lily Wu
Generics

2.7

Hacks are well organized and complete
no extra things


All browsermob-proxy-intercepted requests have empty content (body), but the request's body is not actually empty, why is this happening?

I am using browser mob 2.1.5 with Selenium in Java and trying to capture the body of requests in FireFox on a remote Selenium Server. The requests are sent to an HTTPS webpage.
I initiate a request filter in my code and try to capture the content (body) of the requests, but they all appear empty and I do not understand why? When the compiler reaches the contents variable I get a nullPointException, because it = null.  This happens also for all responses. Am I missing some code or configuration?
For comparison, however, I manage to intercept the HttpRequest request and the HttpMessageInfo messageInfo. I am using BrowserMobProxy and BrowserMobProxyServer and NOT the legacy!!!

I tried to install the browsermob certificate in the FireFox browser in trusted authorities section.

I also set the following (nevertheless, the result is the same with and without the below-set capabilities):
         // SSL related
         opts.setCapability(CapabilityType.ACCEPT_SSL_CERTS, true);
         opts.setCapability(CapabilityType.ACCEPT_INSECURE_CERTS, true);
         opts.addArguments("--ignore-certificate-errors");

and this is what I am trying to do within the test before I execute an action in the UI:
    String expectedContents = "subFolder2";

    RequestFilter filter = new RequestFilter() {
        @Override
        public HttpResponse filterRequest(HttpRequest request, HttpMessageContents contents,
                HttpMessageInfo messageInfo) {

            if (request.getMethod().toString().equals("POST")) {

                if (contents.getTextContents().contains(expectedContents)) {
                    contents.setTextContents("{\"newFilePath\":\"/subFolder2/FileWithChangedName\"}");
                }
            }
            // in the request filter, you can return an HttpResponse object to "short-circuit" the request
            return null;
        }
    };

    proxy.addRequestFilter(filter);

    userUiPage.moveActionButtonInMoveWindowClick();

I expect to capture at least one post request with the following payload / body
(of type JSON) so that I can change it:
{"newFilePath":"/subFolder2/TestFile"}

i've faced a similar problem, and by debuging cause i found the target website uses Brotli compress algorithm, but browsermob-proxy only supports GZip decompression and other 'encoding-content' type content will be automaticlly decompressd as base64 binary data, therefore the content text is either empty or unusable

i've faced a similar problem, and by debuging cause i found the target website uses Brotli compress algorithm, but browsermob-proxy only supports GZip decompression and other 'encoding-content' type content will be automaticlly decompressd as base64 binary data, therefore the content text is either empty or unusable

how to solve this problem? I'm facing this promble too, and its Response headers has one field that is “content-encoding: br”。So Is it use Brotli compress algorithm?

i've faced a similar problem, and by debuging cause i found the target website uses Brotli compress algorithm, but browsermob-proxy only supports GZip decompression and other 'encoding-content' type content will be automaticlly decompressd as base64 binary data, therefore the content text is either empty or unusable

how to solve this problem? I'm facing this promble too, and its Response headers has one field that is “content-encoding: br”。So Dose it use Brotli compress algorithm?

I have also encountered this problem. Can I seek help or how can I change the response to remove the br?

i've faced a similar problem, and by debuging cause i found the target website uses Brotli compress algorithm, but browsermob-proxy only supports GZip decompression and other 'encoding-content' type content will be automaticlly decompressd as base64 binary data, therefore the content text is either empty or unusable

how to solve this problem? I'm facing this promble too, and its Response headers has one field that is “content-encoding: br”。So Dose it use Brotli compress algorithm?

I have also encountered this problem. Can I seek help or how can I change the response to remove the br?

Yes,  'Br' stands for Brotli compress algorithm. If I remember it correctly there isn't an ideal workaround for this problem, the only thing you can do is to fork the source code and change that part of the code by yourself.

i've faced a similar problem, and by debuging cause i found the target website uses Brotli compress algorithm, but browsermob-proxy only supports GZip decompression and other 'encoding-content' type content will be automaticlly decompressd as base64 binary data, therefore the content text is either empty or unusable

how to solve this problem? I'm facing this promble too, and its Response headers has one field that is “content-encoding: br”。So Dose it use Brotli compress algorithm?

I have also encountered this problem. Can I seek help or how can I change the response to remove the br?

Yes, 'Br' stands for Brotli compress algorithm. If I remember it correctly there isn't an ideal workaround for this problem, the only thing you can do is to fork the source code and change that part of the code by yourself.

and by the way, this is an old library, the last update was 6 years ago, please consider other libraries such as https://github.com/mitmproxy/mitmproxy


Set the default language when iframe is loaded

What feature or change would you like to see made?
I've started OHIF viewer in an iframe, but my application language is already set by the user.
Is there a way to define the language used by OHIF as an argument in the frame?
Or maybe is there another way to force the language to change if needed
Why should we prioritize this feature?
LOW priority

Sorry for the too-fast question asked
The answer was very simple: Add the query param &lng=es


Dark theme [$100]

I have a proposal to add a dark theme.
Especially work at night - the default theme burns the eyes;)

---
There is a **[$100 open bounty](https://www.bountysource.com/issues/41234340-dark-theme?utm_campaign=plugin&utm_content=tracker%2F47456670&utm_medium=issues&utm_source=github)** on this issue. Add to the bounty at [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F47456670&utm_medium=issues&utm_source=github).

This is related to #56

As most devs work at night when the kids have gone to bed (own up... you do), this would be a welcome feature. Coffee alone is not enough :)

I would propose to do this in 1.3.0
It perfectly fits in with the new logo.

@qdbdbp any update?

I present the second version.
It is proposal by the well-known theme ARC.
Added green color - related to the logo and the entire visual concept of the Gitea.
p.s. Warning. This is the concept. Not ready products!!
Will prepare the final versions after the arrangements.

During the design I came up with one more idea of what style to choose.
I will soon propose to you.
Now I'm inviting to discuss

Second is better, but IMHO

backgrounds should be more to neutral "gray" rather than "blue"
Main text color should be brighter, to keep contrast reading, but avoid #FFFFFF (and near as possible =)
May be links be more green, like logo

Sublime's default background and text colors pretty good

Thank you for your work @qdbdbp :) What is the current status on this?

The status looks like this:

I was waiting for the version that will pick the community.
I have found a few minor stylistic errors - it will adapt to the version 1.2 Gitea

I definitely like the second one better :D

I prefer the first one.

Hi all,
this is basically the first thought I had when I was testing Gitea after first installation. ("Is there a dark theme available? This one burns my eyes. How do I work at my main-hours -- at night")
Since the proposals are from end of August and the last comments are from about a month ago, I wanted to kindly ask you @qdbdbp if there is anything new regarding this issue? Any status updates you can provide?

I'm currently running using the second one, and I love it!
Rather than using the user styles I've just injected the styles into my CSS file for my gitea instance. Only the first option's userstyle was posted above, so here is a link to the second version: https://userstyles.org/styles/147413/gitea-io-dark-arc-green

the second options looks great 😄 🎈

@techknowlogick @qdbdbp It would be nice to have a "WIP" pr so that the users / reviewers could submit change requests and feedback. This would also make it more easier to test the design.

@techknowlogick thank you for providing this temporary solution. That's very generous of you! @qdbdbp, thank you for your work!

will there be a chance to switch between dark and light themes?

@davydov-vyacheslav currently there is no such option, that should probably be made as new feature request

@davydov-vyacheslav @lafriks There is already an open issue for that: https://github.com/go-gitea/gitea/issues/56

Than I think this can be closed

Probably @techknowlogick we could move that dark theme repository to go-gitea organization

@lafriks sounds good. let me give you admin permissions over it to move the repo.

Ok, so now it is under https://github.com/go-gitea/gitea-darkmode

Thanks @qdbdbp. I think you could apply the bounty from  https://www.bountysource.com


feat: better error message when the Any type is used

Signed-off-by: Oliver Mannion<EMAIL_ADDRESS>TL;DR
Better error messages when Any is used as a type, eg: in a task signature.
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[ ] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
The following:
from typing import Any
from flytekit import task

@task
def could_be_anything() -> Any:
    return "surprise!"

Will now error with
ValueError: Type typing.Any not supported currently in Flytekit. Please register a new transformer

rather than the less informative message
TypeError: issubclass() arg 1 must be a class

Tracking Issue
None
Follow-up issue
NA

hi @tekumara thanks for the PR! to address the lint failures, mind running
pip install -r dev-requirements.txt && make fmt ?

thanks for the tip @katrogan i've pushed a formatted version


Add sbt.argsfile option

There are a number of scenarios, including long classpaths, that
can cause linux and other systems to fail to fork a jvm. We can work
around this on JDK9 and above by using an "arguments file" [1].
This patch adds an "sbt.argsfile" property and SBT_ARGSFILE environment
variable. This option defaults to true.  When active on
Java 9+, long command lines are translated to an arguments file by
adding all command line arguments to the arguments file format at [1]
and java is invoked using the arguments file instead of passing the
arguments on the command line.

https://docs.oracle.com/javase/9/tools/java.htm#

Fixes: https://github.com/sbt/sbt/issues/2974

Thanks for the merge!


2.0.0-pre3 - Clustering Support

I just released 2.0.0-pre2. Give it a try!
Release notes:

Added clustering support
Added padding parameter to FitMapRegionToPositions and FitToMapRegions
Removed dependency on Xamarin.Forms.Maps
Renamed CustomPins to Pins
Removed property name constants
MapCenter is now read-only

Probably more but I forgot ...

I am getting below exception when calling await Navigation.PopModalAsync();
Any ideas?
Xamarin caused by: android.runtime.JavaProxyThrowable: System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'Android.Gms.Maps.MapView'.
  at Java.Interop.JniPeerMembers.AssertSelf (Java.Interop.IJavaPeerable self) [0x00029] in <7cfbebb561c54efc9010b018c0846c7e>:0
  at Java.Interop.JniPeerMembers+JniInstanceMethods.InvokeVirtualVoidMethod (System.String encodedMember, Java.Interop.IJavaPeerable self, Java.Interop.JniArgumentValue* parameters) [0x00000] in <7cfbebb561c54efc9010b018c0846c7e>:0
  at Android.Views.View.set_OnFocusChangeListener (Android.Views.View+IOnFocusChangeListener value) [0x00036] in <2cbc8bc37f564147a10de1abb0c4e399>:0
  at Xamarin.Forms.Platform.Android.ViewRenderer`2[TView,TNativeView].Dispose (System.Boolean disposing) [0x0002c] in <b5b1173f4d49442b898c45e26ae39e48>:0
  at TK.CustomMap.Droid.TKCustomMapRenderer.Dispose (System.Boolean disposing) [0x0013e] in <a8020b0f1856498891c040e965344fc8>:0
  at Java.Lang.Object.Dispose () [0x00000] in <2cbc8bc37f564147a10de1abb0c4e399>:0
  at Xamarin.Forms.Platform.Android.VisualElementRenderer`1[TElement].Dispose (System.Boolean disposing) [0x0007b] in <b5b1173f4d49442b898c45e26ae39e48>:0
  at Java.Lang.Object.Dispose () [0x00000] in <2cbc8bc37f564147a10de1abb0c4e399>:0
  at Xamarin.Forms.Platform.Android.VisualElementRenderer`1[TElement].Dispose (System.Boolean disposing) [0x0007b] in <b5b1173f4d49442b898c45e26ae39e48>:0
  at Java.Lang.Object.Dispose () [0x00000] in <2cbc8bc37f564147a10de1abb0c4e399>:0
  at Xamarin.Forms.Platform.Android.VisualElementRenderer`1[TElement].Dispose (System.Boolean disposing) [0x0007b] in <b5b1173f4d49442b898c45e26ae39e48>:0
  at Xamarin.Forms.Platform.Android.PageRenderer.Dispose (System.Boolean disposing) [0x00011] in <b5b1173f4d49442b898c45e26ae39e48>:0
  at Java.Lang.Object.Dispose () [0x00000] in <2cbc8bc37f564147a10de1abb0c4e399>:0
  at Xamarin.Forms.Platform.Android.Platform+<>c__DisplayClass47_0.<Xamarin.Forms.INavigation.PopModalAsync>b__0 (Android.Animation.Animator a) [0x00010] in <b5b1173f4d49442b898c45e26ae39e48>:0
  at Xamarin.Forms.Platform.Android.GenericAnimatorListener.OnAnimationEnd (Android.Animation.Animator animation) [0x0000e] in <b5b1173f4d49442b898c45e26ae39e48>:0
  at Android.Animation.AnimatorListenerAdapter.n_OnAnimationEnd_Landroid_animation_Animator_ (System.IntPtr jnienv, System.IntPtr native__this, System.IntPtr native_animation) [0x0000f] in <2cbc8bc37f564147a10de1abb0c4e399>:0
  at (wrapper dynamic-method) System.Object:22a53cf7-3f60-4f79-8ff0-2df17d27aa07 (intptr,intptr,intptr)
	at md5b60ffeb829f638581ab2bb9b1a7f4f3f.GenericAnimatorListener.n_onAnimationEnd(Native Method)
	at md5b60ffeb829f638581ab2bb9b1a7f4f3f.GenericAnimatorListener.onAnimationEnd(GenericAnimatorListener.java:40)
	at android.view.ViewPropertyAnimator$AnimatorEventListener.onAnimationEnd(ViewPropertyAnimator.java:1121)
	at android.animation.ValueAnimator.endAnimation(ValueAnimator.java:1149)
	at android.animation.ValueAnimator.doAnimationFrame(ValueAnimator.java:1309)
	at android.animation.AnimationHandler.doAnimationFrame(AnimationHandler.java:146)
	at android.animation.AnimationHandler.-wrap2(AnimationHandler.java)
	at android.animation.AnimationHandler$1.doFrame(AnimationHandler.java:54)
	at android.view.Choreographer$CallbackRecord.run(Choreographer.java:925)
	at android.view.Choreographer.doCallbacks(Choreographer.java:702)
	at android.view.Choreographer.doFrame(Choreographer.java:635)
	at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:913)
	at android.os.Handler.handleCallback(Handler.java:751)
	at android.os.Handler.dispatchMessage(Handler.java:95)
	at android.os.Looper.loop(Looper.java:154)
	at android.app.ActivityThread.main(ActivityThread.java:6692)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1468)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1358)

there a reason MapCenter has become readonly - used to use it with binding to set map location. What's the alternative?
Not sure if it's related to above - but also get exception when closing the page that contains the map.

I have a sample project here (TK map on location selection page , XForms Map on main page)

I will look at the errors asap. Thanks for the sample project.
Regarding MapCenter, use MapRegion instead.

@InquisitorJax
The disposing issue should be fixed next pre release.
About 3: this is probably an issue with your app using multiple instances of MapView. I already posted that some time ago here. Android has MapFragment to handle multiple map instances in a single activity, but Xamarin.Forms does not support it afaik.

Dispose now works in pre2 thanks!

I can also confirm that the dispose now works.
wrt 3: I found that if I don't show the map in a modal view, then the map shows up just fine.
I'll see if I can log a bug with xamarin on the issue.

trying to call FitMapRegionToPostions causes nullreference exception.
See TKMapView.cs in linked example.
Could it be that the code is firing too early (map hasn't rendered yet)?

It should not raise a NullReferenceException though.
There is a MapReady event/command.

If there are no more concerns, I will move 2.0.0 to stable at the beginning of next week.

The UWP Files don't seem to be getting downloaded as part of the 2.0.0 nuget package,
Is this correct?

I get this when clicking on a cluster marker.
12-06 21:22:37.318 I/MonoDroid( 7634): UNHANDLED EXCEPTION:
12-06 21:22:37.398 I/MonoDroid( 7634): System.NullReferenceException: Object reference not set to an instance of an object.
12-06 21:22:37.398 I/MonoDroid( 7634):   at TK.CustomMap.Droid.TKCustomMapRenderer+<>c__DisplayClass36_0.b__0 (System.Collections.Generic.KeyValuePair2[TKey,TValue] i) [0x0000c] in <4ca3da68f3024c3fbd2be902081d17d0>:0  12-06 21:22:37.398 I/MonoDroid( 7634):   at System.Linq.Enumerable.SingleOrDefault[TSource] (System.Collections.Generic.IEnumerable1[T] source, System.Func`2[T,TResult] predicate) [0x0002c] in :0
12-06 21:22:37.398 I/MonoDroid( 7634):   at TK.CustomMap.Droid.TKCustomMapRenderer.OnMarkerClick (System.Object sender, Android.Gms.Maps.GoogleMap+MarkerClickEventArgs e) [0x00016] in <4ca3da68f3024c3fbd2be902081d17d0>:0
12-06 21:22:37.398 I/MonoDroid( 7634):   at Android.Gms.Maps.GoogleMap+IOnMarkerClickListenerImplementor.OnMarkerClick (Android.Gms.Maps.Model.Marker marker) [0x00017] in :0
12-06 21:22:37.398 I/MonoDroid( 7634):   at Android.Gms.Maps.GoogleMap+IOnMarkerClickListenerInvoker.n_OnMarkerClick_Lcom_google_android_gms_maps_model_Marker_ (System.IntPtr jnienv, System.IntPtr native__this, System.IntPtr native_marker) [0x00011] in :0
12-06 21:22:37.399 I/MonoDroid( 7634):   at (wrapper dynamic-method) System.Object:4666f2c0-b95f-487d-86dc-c22ef8df1081 (intptr,intptr,intptr)

Wow, did not see that awesome post. I'm gonna give it a try next week, and I'll give you a complete feedBack on that @TorbenK  . Thanks for your support.

Using MapRegion to position the map needs a radius.
How do you move the region of the map keep the current zoom level ( radius )? Everything for this is now hidden or read only, like  getting VisibleRegion or setting MapCenter.

I created a Distance property called ZoomLevel in my viewmodel, and whenever I call MoveToMapRegion, I pass the property.  If the user zooms the map out, I update the property as needed.

Ha didn't check if there were zoom events available to interact with that... so in those you do get the actual zoom factor?

After looking at my code, it seems I only manually update the ZoomLevel when the user clicks the “zoom out to see the whole country” button, or the “zoom into my location” button.  In both cases, I know what zoom level to use, so setting the property is easy.
If the user drags the map around, then I don’t need to bother with zoom level.
From: Glenn Versweyveld<EMAIL_ADDRESS>Sent: Thursday, May 3, 2018 7:18 AM
To: TorbenK/TK.CustomMap<EMAIL_ADDRESS>Cc: Joe Cribbs<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [TorbenK/TK.CustomMap] 2.0.0-pre3 - Clustering Support (#272)
Ha didn't check if there were zoom events available to interact with that... so in those you do get the actual zoom factor?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_TorbenK_TK.CustomMap_issues_272-23issuecomment-2D386276151&d=DwMCaQ&c=rxuyg758I4Zd3CDHNny_Hw&r=xfsGa-z-bIufeBjQPcW1xfhnjturTxdFTiGHK16vYx4&m=RKbtamo3ClPDmeClCQY23If7yKbXgVlMXCiyM1yyFa4&s=Bd6XnFPQk88A9sNnYePN4ToZNyYf18akaVkZTFOqHqw&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AZCiK2uV-2DlSJl2CV3Bhsgt5ianXWdbsiks5tuvWRgaJpZM4Qo87o&d=DwMCaQ&c=rxuyg758I4Zd3CDHNny_Hw&r=xfsGa-z-bIufeBjQPcW1xfhnjturTxdFTiGHK16vYx4&m=RKbtamo3ClPDmeClCQY23If7yKbXgVlMXCiyM1yyFa4&s=91hWNFloRlSV90KdygpRTs7jOQEFsdjNmBwqB4maoVU&e=.


Leaf Decay crash

http://pastebin.ubuntu.com/11780195/

This is probably caused by leaf decay mixins being incorrect. I will verify.

@Zidane seems like it's world related which you have some experience with.

@modwizcode "some" is an interesting statement (/me looks at unified world changes)
Stack trace is pretty clear on this, just need to implement getBlockSnapshot.

I'm on it


Added My Project File

My project is based on Blockchain. Which I submitted in HackWeb3Conf
Please Go through with it.
@vansh-codes

Please Go through this i made the changes
@Snehsk

Can you please tell how to do it
@Snehsk


Connection refused - sqlite

Describe the bug
After the update today of the container, I wasn't able to connect to the web interface.
I checked the log and I have this error :
[2023-08-31 12:46:06] production.ERROR: SQLSTATE[HY000] [2002] Connection refused (Connection: mysql, SQL: select * from information_schema.tables where table_schema = speedtest_tracker and table_name = migrations and table_type = 'BASE TABLE')
{"exception":"[object] (Illuminate\\Database\\QueryException(code: 2002): SQLSTATE[HY000] [2002] Connection refused (Connection: mysql, SQL: select * from information_schema.tables where table_schema = speedtest_tracker and table_name = migrations and table_type = 'BASE TABLE') at /var/www/html/vendor/laravel/framework/src/Illuminate/Database/Connection.php:801)

Then, the container is stopped.
31/08/2023 14:46:06
In Connection.php line 801:
31/08/2023 14:46:06

31/08/2023 14:46:06
  SQLSTATE[HY000] [2002] Connection refused (Connection: mysql, SQL: select *
31/08/2023 14:46:06
   from information_schema.tables where table_schema = speedtest_tracker and
31/08/2023 14:46:06
  table_name = migrations and table_type = 'BASE TABLE')
31/08/2023 14:46:06

31/08/2023 14:46:06
31/08/2023 14:46:06
In Connector.php line 65:
31/08/2023 14:46:06

31/08/2023 14:46:06
  SQLSTATE[HY000] [2002] Connection refused

I'm not sure if it was working before the update, I've not checked the interface for the last few weeks.
Environment (please complete the following information):

OS:  Ubuntu
Architecture: amd64
Version: speedtest-tracker_speedtest-tracker updated from ae7429d0e6 to 6991db2ade -  v0.11.20

I was getting similar messages, trying to set up a fresh 0.11.19 (now 0.11.20) but I realized I was just being too aggressive on the timing. It was taking ~30 seconds for my MariaDB container to start up and become ready, so even though there were errors in the log, instead of hitting ⌃c right away, just allowing the main container to restart a few times eventually everything settled, and is now working on 0.11.20 + MariaDB.  A 2-day saga comes to a happy ending 🚀...

I left a comment in #701 but the TL;DR is make sure to pass DB_CONNECTION=sqlite as an environment variable.


Video thumbnail not generated for specific files

Hi @DarthSim!
For some reason, Imgproxy Pro is unable to generate thumbnails for some MP4 files. Log says "Can't download source image: Source image type not supported". I was unable to locate problem myself, files look OK. Can you please take a look? Here is one example: https://drive.google.com/file/d/1CuUuWAT5lkGp1_5D8cch5DWzf6Y_bj6v/view?usp=share_link

Fixed in the latest build. If imgproxy won't recognize other MP4s, feel free to drop me a message at<EMAIL_ADDRESS>The first 12 bytes of the file would be enough :)


Publish latest version to npm please

Current here: 1.1.4
Latest on npm: 1.1.1

I don’t have it published on npm, so it’s not my package.


fixing import statements in examples

If I am not mistaken, the examples in the README need to be changed from a require to an import statement.

Thank you, but I had fixed those already the minute before I saw the PR!


feature request

Hello, i have 2 argus2 cam at work and would like to see camera stream at home with my linux rpm based distro, mageia.
if am  i understant your application only use local, could you add an option who use web conection (throug amazon p2p stream).
I also would like to save motion detection on an local pc with ftp or samba rather save on reolink cloud.
i'm trying at home but seem to don't get anything or perhap's i didn't do it in the right way  :
python3 ./src/main.py --debug
'backupImage': '../public/images/klingel.jpg'}]}

Serving Flask app "main" (lazy loading)
Environment: production
WARNING: This is a development server. Do not use it in a production deployment.
Use a production WSGI server instead.
Debug mode: on
INFO:werkzeug: * Running on http://<IP_ADDRESS>:8000/ (Press CTRL+C to quit)
INFO:werkzeug: * Restarting with stat
{'cameras': [{'name': 'Argus2b', 'deviceSid': 'xxxxxxxxxx', 'username': 'admin', 'password': 'xxxxxx', 'backupImage': '../public/images/klingel.jpg'}]}
WARNING:werkzeug: * Debugger is active!
INFO:werkzeug: * Debugger PIN: x-x-x
INFO:werkzeug:<IP_ADDRESS> - - [23/Nov/2020 11:51:12] "GET /Argus2b HTTP/1.1" 404 -

the 404 message came when i opened the http page with my browser
thank for your good work

Hi,
your request is wrong.
Try to send a get request to "/api/v1/cameras/Argus2b". That should work.
Regarding the p2p stream and the sambda recording. I don't have enough information on the protocol yet to get something like this to work. So no promises. But i would like to have such a local recording feature too.

Thank for your response, for the two cam, argus2a and argus2b i get an infinite "INFO:baichuan_udp_layer:Sending discovery packet" and one time
INFO:werkzeug:<IP_ADDRESS> - - [23/Nov/2020 17:15:48] "GET /api/v1/cameras/Argus2b HTTP/1.1" 200 -
and
INFO:main:Stopping camera stream Argus2
but only your "klingel" picture :-)
pherap's because i'm not localy, i'm at home and cam are at work.

Yes, the discovery packet is send for the broadcast ip <IP_ADDRESS>. This can only work, if you are in the same network with the camera.

if i want test and in the future ad this as an option, which broadcast ip i should use <IP_ADDRESS> ?
and in which file i need to change that ?

hey! i remember i had setup an vpn conection! :-)

From wikipedia:

A special definition exists for the IP address <IP_ADDRESS>. It is the broadcast address of the zero network or <IP_ADDRESS>, which in Internet Protocol standards stands for this network, i.e. the local network. Transmission to this address is limited by definition, in that it is never forwarded by the routers connecting the local network to other networks.

This program currently only supports the local network access to the camera.

Yes, you confirm Wath i understood, so, is it "easy" to add an option to use local or internet network, with wireshark i saw few ip, one seem the reolink author server and other aws cloud server, so somewhere we could say to the Cam to directly send the stream rather pass through aws server

Hi i'am at work, that's ok, except i need to disable my firewall, i tryied to add 2015/udp and 8000/tcp but doesn't work whick rules i need.
thank

Hi
to be clear, here it is my config, at work an 4g router with one PC, let's say PCW and 2 Argus 2 cam, at home one box (freebox) with openvpn server inside.
At first, i connected the PCW with openvpn routed, considering it is less exposed, but as we know, it isn't possible to brodcast, also, from shh, if i try your program, because of vpn, broadcast don't work too, then i tried openvpn bridge mod, with ssh i got this
NFO:baichuan_udp_layer:Sending discovery packet
INFO:baichuan_udp_layer:Received discovery packet answer from ##
INFOcameraSending legacy login packet
INFOcameraReceiving nonce packet
here the program hang
so to keep routed vpn, it is possible to add ip of the cam in the config file and have an option like -b or --nobroadcast to use uuid or directly ip ?
I noticed too, the video don't have sound and was an MJpeg (motion), so under linux vlc can't read it, gst can, but, is there performance issue, another switch should be nice to select low or hig video quality mod.
Thank to take time to read me.

Is the audio turned on at the cameras ?
Robert Gamble
<EMAIL_ADDRESS>(preferred )
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>On Monday, November 30, 2020, 4:25:05 PM EST, surfzoid<EMAIL_ADDRESS>wrote:

@jdillenkofer I am getting video, but noticed no audio coming through from the Argus 2. Is there anything special that is required to get the audio stream?

Same for me,
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@Lanwalker Yes the audio is turned on and streams via the app but not via the proxy.

Just making sure. I did that when I first started working with the Reolink app. Good luck.
Robert Gamble
<EMAIL_ADDRESS>(preferred )
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>On Monday, November 30, 2020, 7:58:43 PM EST, Golden<EMAIL_ADDRESS>wrote:
@Lanwalker Yes the audio is turned on and streams via the app but not via the proxy.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

You are right @Lanwalker, with the official app, by défaut sound is mute, but if i'm remember corectly, the choice of recording sound is in setup, so the mute option is only in the client, not in the Cam ?

Other part, if we look at the code, this is pyav h264, viewer of stream say Mjpeg (motion) or jpeg vidéo

Yes, i only decode the video stream. There are also audio packets but currently the camera proxy just drops these packets.

@jdillenkofer Is this on purpose or do you have plans to decode the audio?

@surfzoid
The port the camera sends its packets to is choosen randomly by the camera proxy and transferred to the camera via the discovery broadcast.
I don't know any way how a direct communication to the camera could work without the broadcast.
There is also currently no way to set a fixed port, but that can be easily implemented.
Feel free to send me a pullrequest.
@goldenhub77
I didn't need the audio support for my use case and i currently don't have any plans to implement it.
But i am not against it and if someone creates a pull request that works, i can merge it.

@jdillenkofer Will do, thank you for the quick reply. Do you have any additional documentation you can share that would assist as I'm going to try to work on the audio?

@goldenhub77
I only have this link, which explains how the BAICHUAN_MESSAGE_ID_VIDEO can be split into video frames (P and I-Frames) and audio frames (AAC-Frames).
The AAC_FRAME in this file should contain the audio stream. I don't know if you also need some kind of metadata like sample rate to decode this audio stream.

I saw yesterday this part of frame def as Jpeg, this is where to investigate ?
Could you please comment your code ?

@surfzoid
The port the camera sends its packets to is choosen randomly by the camera proxy and transferred to the camera via the discovery broadcast.
I don't know any way how a direct communication to the camera could work without the broadcast.
There is also currently no way to set a fixed port, but that can be easily implemented.
Feel free to send me a pullrequest.

F

@jdillenkofer somewhere, you say you have some freez of image, in the official client we have option to switch beetween hight or medium video quality, in the link you provide here, we can see the definition of the résolution and fps, this can be add as an option, perhaps also use other codec suported by the Cam.
I see also langage def in the frame, is there an logic with fps?
Sound like when, the codec is define to h264, the Cam send an stream in h264, why convert in Mjpeg ? We should try Google Web codec.

i just discover python3-livestreamer - CLI for piping video streams to video players whose requiere rtpm dump prog, this is not a better way than libav ?


minimize usage of IControllerLoader.loaded prop

Part of #11914

Codecov Report

Merging #12455 (3847311) into main (0e55a73) will decrease coverage by 0%.
The diff coverage is 26%.

@@          Coverage Diff           @@
##            main   #12455   +/-   ##
======================================
- Coverage     69%      69%   -1%
======================================
  Files        495      495
  Lines      31115    31132   +17
  Branches    4900     4905    +5
======================================
+ Hits       21760    21762    +2
- Misses      7513     7531   +18
+ Partials    1842     1839    -3

Impacted Files
Coverage Δ

src/notebooks/controllers/liveKernelSwitcher.ts
42% <26%> (-19%)
:arrow_down:

.../notebooks/debugger/debugLocationTrackerFactory.ts
96% <0%> (-4%)
:arrow_down:

...kernels/jupyter/launcher/jupyterConnection.node.ts
79% <0%> (-4%)
:arrow_down:

...tive-window/debugger/jupyter/kernelDebugAdapter.ts
60% <0%> (-2%)
:arrow_down:

src/kernels/helpers.ts
70% <0%> (-2%)
:arrow_down:

...c/platform/interpreter/interpreterPackages.node.ts
73% <0%> (-1%)
:arrow_down:

src/kernels/kernelDependencyService.node.ts
82% <0%> (-1%)
:arrow_down:

src/notebooks/debugger/debuggerVariables.ts
69% <0%> (-1%)
:arrow_down:

src/kernels/raw/launcher/kernelProcess.node.ts
74% <0%> (+<1%)
:arrow_up:

src/notebooks/controllers/kernelRanking/helpers.ts
61% <0%> (+1%)
:arrow_up:

... and 2 more


Pass source & target node's data in link Graph callbacks (original: display tooltip with additional data)

Hello @danielcaldas,
not sure if this is already available as a feature or if its a duplicate request. Appreciate your help.
I need to display a tooltip on mouseover / mouseout of a link with data other than source and target. The current methods only take source and target as arguments.
The tooltip needs to display some extra data from the link on mouseover. So I'd imagine a custom-link (just like custom-node) feature is needed to do this.
If there is a solution already to do this (perhaps in a different way), can you please point me to it ?
If not, I wanted to ask this as a feature request.
Thank you for your response.

I think this can be achieve by using existing callbacks onMouseOverLink & onMounseOutLink. It would be best to place a tooltip (e.g., absolute positioning, a component in your app) near the link using the mouse coordinates to compute the tooltip's best positioning.

Thank you for the response. The first thing I tried were the existing callbacks onMouseOverLink and onMouseOutLink. However, they come with predefined arguments of only source and target. I cannot pass any any additional data that I would like to display within those callbacks.
Is there a way to pass additional data to those callbacks ?

Yes, we can work on that. It would be a change similar to the one in https://github.com/danielcaldas/react-d3-graph/pull/385 by @LonelyPrincess. Basically, we could provide in that callback the full node object for the source and target. What do you think about that?
We should also think about forwarding the actual DOM event object from within our Link component.

Yes thank you. It would be great to have entire node object in node callbacks and entire link object in the link callbacks for such custom displays.
Looking forward to the change. Appreciate it.

With this issue, we aim to pass the node's info on our link callbacks. Please drop here a question so that we can discuss the new APIs if you're interested in pick up the issue. Cheers!

Hi @danielcaldas - Just wanted to clarify - can you also pass link info (not just node) - the reason being, there can be data / attributes in link other than the node. The node will be source or target node and they may have their own data / attributes that are different from the link which connects them.
Hope the request is clear. Thanks

Thanks for the clarification @ghpgmer, let's do that and pass the links along as well.

Thanks for the clarification @ghpgmer, let's do that and pass the links along as well.


feat: new data structure saved in keychain

- What I did
Upgrade data structure saved in Keychain to fix this ticket #468
- How I did it
- How to verify it
- Description for the changelog

I have used this branch and tested it with the new flutter project. But looks like we still need to fix flutter_keychain first. But we can still push this code to trunk but not good for publish.

Please try to use more guard clauses, this will increase readability of the code.

Thanks, @gkc @yahu1031, and @XavierChanth
I've pushed a new commit to solve your comments. please help me check again!

Hi @gkc and @XavierChanth,
Could you please recheck the PR?

@gkc please approve this PR and we will prepare to release it.


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative/operations-wg-leads
/assign knative/operations-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/eventing 7a48f42...2a69eec:
  > 2a69eec Clearing out old state info for new placements (# 6172)
bumping knative.dev/serving b28062c...39df601:
  > 39df601 Update actions (# 12652)
  > 24c178d Update net-contour nightly (# 12648)
  > 88edca7 Update net-istio nightly (# 12651)
  > d7409b7 Update net-certmanager nightly (# 12647)
  > fa7aa27 Update net-kourier nightly (# 12638)
  > c1aaaf6 Update net-gateway-api nightly (# 12609)
bumping knative.dev/pkg d481724...80c511a:
  > 80c511a Wait for reconciler/controllers to return prior to exiting the process (# 2437)
bumping knative.dev/hack a8ef7b6...cf8cbc0:
  > cf8cbc0 Fix the regex used for matching GitHub Actions files (# 156)

/test pull-knative-operator-eventing-upgrade-tests
/lgtm
/approve


Fix fix variant lost null info after cast_column

This could result incorrect output in hirachinal cases
sql """insert into ${table_name} values (-3, '{"a" : 1, "b" : 1.5, "c" : [1, 2, 3]}')"""
sql """insert into  ${table_name} select -2, '{"a": 11245, "b" : [123, {"xx" : 1}], "c" : {"c" : 456, "d" : "null", "e" : 7.111}}'  as json_str
            union  all select -1, '{"a": 1123}' as json_str union all select *, '{"a" : 1234, "xxxx" : "kaana"}' as json_str from numbers("number" = "4096") limit 4096 ;"""

mysql> select v["c"] from var_rs where k = -3 or k = -2;
+----------------------+
| element_at(`v`, 'c') |
+----------------------+
| [1,2,3]              |
| []                   |
+----------------------+
2 rows in set (0.04 sec)

Proposed changes
Issue Number: close #xxx

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

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


Configuration file type definitions

It would be awesome if we were able to get autocomplete working in the configuration file. Especially so with theme callbacks and plugins.
This should be possible to achieve by including typescript or jsdoc type definitions, and then doing something like:
/**
 * @type {import('tailwindcss').Config}
 */
const config = {
 // everything inside of here would get autocomplete
};

module.exports = config;

Would be open to a PR for this, not something I have experience with myself though. One thing to be careful of though is that you can have any keys you want in that file, because plugins can expect custom keys to exist, so I'm not sure how easy it's going to be to capture that flexibility in a type definition.

I was about to open the same issue. Glad there are more people interested in this.
@adamwathan in typescript you can define something like:
interface Dictionary {
  some: string;
  keys: string;
  /**
   * and with this you are saying: "this type can have any field
   * as long as the key is a string and it also return a string"
   */
  [key: string]: string;
}

@Glinkis I don't know how to "wire" the type definition to the raw javascript file, but I can help write the definition file if wanted

I don't know how to "wire" the type definition to the raw javascript file
Not quite sure what you mean by this. It should be enough to just create a types.d.ts-file, and link to it via the types field in package.json.

One can then simply do import('tailwindcss') in the JSDoc comment like I did above.
Would you like to create a branch for this, and link it here? That way we could potentially collaborate on the type file.

@Glinkis I will create it on the weekend, I already have some types from a experiment that I did on tailwind. The only thing I can't get right are the default values.
Just inlining all the default values on a e.g DefaultScreens, DefaultColors types should be enough, but it introduces the overhead of modifying two files if Tailwind devs want to change a default value (or introduce new ones).
If you know something that could help (like a tool that extracts default values from an existing object) that would solve this problem.

@LouizFC I started a bit over here, with a very basic setup.
I've granted you contribution privileges.

FYI, this gets you fairly close:
/** @typedef { import('tailwindcss/defaultConfig') } TailwindConfig */
/** @typedef { import('tailwindcss/defaultTheme') } TailwindTheme */

/** @type { TailwindConfig & { theme: { extend: TailwindTheme } } } */
module.exports = {}

It's not perfect (e.g. "purge" is reported as undefined[] instead of string[], some options are objects and some are functions...) but you do get nice autocomplete for most config and theme options and see all the available options without having to check the docs for their precise name. 👍🏻

@jonaskuske thanks seems great
how do we use this snippet to access the type then ?  😅

@GaspardC The snippet directly applies the type to the exported object:
/** @type { DefaultConfig & { theme: { extend: DefaultTheme } } } */
module.exports = {
 // you have autocomplete here, type "c" and it'll suggest "corePlugins"
}

If you want to use the type on it's own, you can define it as well rather than applying it directly:
/** @typedef { import('tailwindcss/defaultConfig') } DefaultConfig */
/** @typedef { import('tailwindcss/defaultTheme') } DefaultTheme */
/** @typedef { DefaultConfig & { theme: { extend: DefaultTheme } } } TailwindConfig */

// now use the type "TailwindConfig" however you like:

// @type { TailwindConfig }
module.exports = {}

// or even in some TS file:

import { TailwindConfig } from './tailwind.config'
const config: TailwindConfig = { ... }

But note that the type is just infered from the default config and theme and thus not really accurate – fine to get autocomplete, not so much for proper TypeScript use.
For TS use, you'd at least want to use Partial<TailwindConfig> so it treats all properties as optional, otherwise TS won't be satisfied until you specify every single possible config property 😬

@jonaskuske thanks, exactly what I'm looking for
as you said I created a tailwind.config.js file with :
/** @typedef { import('tailwindcss/defaultConfig') } DefaultConfig */
/** @typedef { import('tailwindcss/defaultTheme') } DefaultTheme */
/** @typedef { DefaultConfig & { theme: { extend: DefaultTheme } } } TailwindConfig */

/** @type {TailwindConfig} */
module.exports = {
// corePlugins hints working here
}

then in my TS file:
import { TailwindConfig } from './tailwind.config'
const config: TailwindConfig = { test: 1 }
//don't have any errors or autocompletion hints here

Do you know what is wrong with my code ?

@GaspardC Hmm, good question. I tested it in tailwind.config.js and in an index.ts I created next to it and it worked, but my project wasn't actually set up for TS, there was no tsconfig.json. So my guess would be that as soon as you add a tsconfig to your project, TS/VSCode handles interop between .ts and .js with JSDoc differently?
Does it change anything if you add "checkJS": true to your tsconfig?

With nuxt-ts + @nuxtjs/tailwindcss, it's possible to use tailwind.config.ts as the config file (although there are still some bugs with hot-reload).
I think it would be better if we has the type definitions.

With @nuxt/typescript-runtime + @nuxtjs/tailwindcss, it's possible to use tailwind.config.ts as the config file.
Although there are some bugs with hot-reload, that's not a problem for me.
The problem is we don't have the type definitions. It's hard to write plugins or get autocomplete in the config file.


Fix #17660: Wrong edt order in action bar guide

Hi @yajiex, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Yajie Xue). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Not correct: not all elements will be compared with each other
Not elegant: not extensible.


authz/ics20: review comments for initial PR

Initial PR.

[ ] Rename allowed_addresses to allow_list and use in the protodoc the same comment as SDK in authz.proto for send.
[ ] Make IsAllowedAddress private.
[ ] Rename PortChannelAmount to Allocation.
[ ] Rename transfer_authz.go to transfer_authorization.go and transfer_authz_test.go to transfer_authorization_test.go.
[ ] Rename ErrDuplicateEntry to ErrInvalidAuthorization and change the error text to invalid transfer authorization (see this comment).
[ ] Replace usage of this constant with SDK's.
[ ] Format errors returned as sdkerrors.Wrap(<type>, <message>) instead of sdkerrors.<type>.Wrapf(<message>).
[ ] Move authz.proto to proto/ibc/applications/transfer/v1. @damiannolan
[ ] Convert NewTransferAuthorization into a variadic function:

func NewTransferAuthorization(allocations ...Allocation) *TransferAuthorization {
	return &TransferAuthorization{
		Allocations: allocations,
	}
}

Separate PRs against the feature branch can be opened for each of these items.

Will tag myself beside tasks as I pick them up


Implement delete_user endpoint

I like to work on this issues

In which branch i have to work and can you explain more about this issue

Hi! I'm glad you're choosing to work on this. The CONTRIBUTING.md in the root has more details on which branch to start from. it's the experimental branch you're branching off  from. If anything is still unclear, let me know


Build a Node.js App with CockroachDB and the Node.js pg Driver Doc Update

Re: Build a Node.js App with CockroachDB and the Node.js pg Driver
Issue Description
Sample app basic-sample.js will produce multiple lint warnings for javascript.
Suggested Resolution
running the following commands against basic-sample.js will address most of them
npm install standard --save-dev
RUN npx standard --fix *.js

sample output of errors can be found here

We're promoting Javascript in 21.1 so should fix this

We're promoting Javascript in 21.1 so should fix this

Recently rewrote/linted this app... https://github.com/cockroachlabs/hello-world-node-pg/pull/1

@ericharmeling Based on your last comment, can we close this?

@ericharmeling Based on your last comment, can we close this?

I believe so.


Unable to build when implement the generic class

how to use the generic class?
i got this error when build -> error: non-static type variable T cannot be referenced from a static context

i think the readme quite misleading. Under the section the "Generics Support"
the static method should be :
public static  JsonAdapter<Foo> jsonAdapter(Moshi moshi, Type[] types)
instead of:
public static JsonAdapter<Foo> jsonAdapter(Moshi moshi, Type[] types) {

i think the readme quite misleading under the "Generics Support"
public static <T> JsonAdapter<Foo<T>> jsonAdapter(Moshi moshi, Type[] types)
instead of
public static JsonAdapter<Foo<T>> jsonAdapter(Moshi moshi, Type[] types)

Thanks for catching that.  Care to submit a PR with the update?

Fixed with #141


all the methods are individually exportable to reducce the package size

made the methods individually exportable to reduce size

Pull Request Test Coverage Report for Build 504346108

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 19:
0.0%

Covered Lines:
47

Relevant Lines:
47

💛  - Coveralls

My understanding is that the package-lock.json should not be removed.
If some of the package versions in the package-lock are no longer available, you should run npm i and commit the updated package-lock.json file.

It seems like you've just changed named exports to default ones. I don't know any rationale behind such decision. All the methods exported individually without this changes and you can import them directly just like import { formatToTimeZone } from 'date-fns-timezone/dist/formatToTimeZone';.
Also, ignoring lockfiles is absolutely incorrect. They exist for being committed.

@sHooKDT I do agree with the ignoring lock files, that's never a good thing 😄 But the rationale behind exporting individual functions is to keep package size small, this allows us to use tree-shaking in order to not import functions we're not using.
Here's how date-fns does it import format from 'date-fns/format'; which only imports that function and instead of including the entire library on my final build, it'll only include that function.
This a very much needed feature in the project I'm working on, can we please fix this PR and merge it? I can work on it if needed.

@jasongerbes  I agree with you. package-lock.json
https://docs.npmjs.com/files/package-lock.json
https://stackoverflow.com/questions/44206782/do-i-commit-the-package-lock-json-file-created-by-npm-5

I can create my own PR with a different take on this, to support both ways of exporting.

#11
This is my take on how to do this with the least amount of friction, maintaining previous support plus also giving the users of this API the option to do so in a performant way (only including what they require)

I agree with @jasongerbes, that the package lock should stay, as @devpato referred to the best practices about it.
Like @sHooKDT, I wonder, what is the rationale behind using the default exports instead of named imports. Is there are multiple exported items, name exports allow tree-shaking (unreachable code elimination). Wildcard exports are usually frowned upon, because unused exported items cannot be easily detected by the module bundler. named exports make easy to prune the originating module off all code, which is not used not only in any exported function, but also in any exported function, which is not imported by any other module. I do not know, if they analyse all source code to find out, what functions from the imported object are actually accessed. That is the case of the "bundled-interface" modules src/index.js and src/custom/js.
https://news.ycombinator.com/item?id=15765409
https://www.reddit.com/r/javascript/comments/9amkc6/export_default_considered_harmful/
https://blog.neufund.org/why-we-have-banned-default-exports-and-you-should-do-the-same-d51fdc2cf2ad
https://humanwhocodes.com/blog/2019/01/stop-using-default-exports-javascript-module/
https://basarat.gitbooks.io/typescript/docs/tips/defaultIsBad.html
Most JavaScript modules in the src directory export just a single function to keep the sources better maintainable.  Bundles src/index.js and src/custom/js make the integrations easier, which prefer concatenated bundles, which still support the tree-shaking. For the single-function-exporting modules, I admit, the named export might not show any objective benefit, which would show performance, size, or coding efficiency difference. I made these module use the named exports nevertheless, because it makes all modules consistent and it actually helped me, when I renamed some before the release, to make sure, that I did not forget anything.
I'm reluctant to agree with switching the default exports, because it is a breaking change on the interface, as long as I do not see an objective or subjective advantage of it.
@danazkari, how do the named exports prevent you from importing just a single function? How do they hinder the performance? Thank you!


[BUG] Bug fix for checksum validation for mapping metadata

Description
During checksum validation for remote cluster state, for mapping metadata we need to ignore compressed data check.
Related Issues
Resolves #[Issue number to be closed when this PR is merged]
Check List

[ ] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

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.

LGTM, needs more tests?

Code coverage is failing for the toString method added. Do we really need to add test?


Apathy prevents altars from gaining power, but is required for lichdom.

In order to become a lich, you need the apathy curse.
You also need 999 altar ME to start the ritual to ascend to lichdom.
The apathy curse prevents the player from having ME.
But it does not prevent the player from draining from the Altar.

This is a Bewitchment-native issue, you should file an issue there instead