KML: only blue fill and stroke

Polygons/lines have only standard blue colors. Individually styling every polygon is hard and i'm 100% sure there's an easier way to this.

Change the colour of the poly lines (not the icons):
        const myCustomColour = '#ff4f00'

        const customLayer = L.geoJson(null, {
          // http://leafletjs.com/reference.html#geojson-style
          style: function(feature) {
            return { color: myCustomColour };
          }
        });
        // this can be any kind of omnivore layer
        const runLayer = omnivore.kml(kml, null, customLayer)

        runLayer.addTo(w.map)


Provide easier way to create Uni

Wouldn't it be more general-purpose to provide methods to produce a Uni of null, which can be any type?
e.g.
<T> Uni<T> nullUni() {
    return (Uni<T>) NULL_UNI;
}

@dmlloyd 's suggestion make sense to me. @kenfinnigan could you try?

Updated with another method, nullItem(), for use with Uni<T>

Add nullItem to what test? I added a few tests to some obvious places where voidItem was tested.
Will add the convert to Multi test

Sorry I missed the nullItem tests. So yes, just the toMulti test.

Thanks @kenfinnigan and @dmlloyd !

Maybe it's too late to comment but this was merged while I was still in bed so it is what it is. :grin:
There should only be one constant UNI_OF_NULL, and it should be returned with an unsafe cast for all callers (because null is all types, it's actually a safe cast).  Then the uni-of-void method can be removed.  Otherwise you're running off redundant, identical objects every time the method is called.

One of the reasons we proceeded with the current approach is because voidItem doesn't require <TYPE> be added to the usage.
For example, we can do Uni.createFrom().voidItem() instead of Uni.createFrom().<Void>voidItem().

One of the reasons we proceeded with the current approach is because voidItem doesn't require <TYPE> be added to the usage.
For example, we can do Uni.createFrom().voidItem() instead of Uni.createFrom().<Void>voidItem().

It already shouldn't be necessary, you should be able to say: Uni<Void> uni = Uni.createFrom().nullItem() or Uni<Banana> uni = .. or Uni<PairOf<Skis>> uni = ....  The type should be trivially inferrable.

It wasn't, compilation failed without <TYPE>

It wasn't, compilation failed without <TYPE>

Can you give an example?  Something feels wrong... maybe there is a conflicting method overload?

If I have Uni.createFrom().nullItem().subscribe().withSubscriber(subscriber); in a test, it fails with:
[ERROR] /Users/kenfinnigan/projects/smallrye/mutiny/implementation/src/test/java/io/smallrye/mutiny/operators/UniCreateFromItemTest.java:[92,48] method withSubscriber in class io.smallrye.mutiny.groups.UniSubscribe<T> cannot be applied to given types;
  required: S
  found: io.smallrye.mutiny.operators.UniAssertSubscriber<java.lang.String>
  reason: inferred type does not conform to upper bound(s)
    inferred: io.smallrye.mutiny.operators.UniAssertSubscriber<java.lang.String>
    upper bound(s): io.smallrye.mutiny.subscription.UniSubscriber<? super java.lang.Object>

Ah I see, so it's when there's a long chain.  Well, that's fine then to have a separate overload I guess - but they should still be all sharing the same constant object rather than constructing a new one every time.
Maybe having a type token version reads better in some cases?  Uni.createFrom.nullItem(String.class).subscribe().withSubscriber(subscriber)?

Showing my lack of generics knowledge here.
How can Uni.createFrom().item((T) null) be static without T? Do I store it as Uni<Object>?

Yeah, or even a raw Uni.  It'll complain about the cast but you can suppress it for this one.

Ok


Suggestion - streaming?

Instead of starting a recording, maybe starting a stream to youtube or something with all the metadata (private stream).

Interesting idea. Starting/stopping a stream should be fairly easy but the metadata integration might be out of scope for this project if it's platform-specific. Can you elaborate on the integration you're looking for?

u could use tracker.gg as a way to at least leave information in the stream description about the match. Tracker.gg makes a link of every match so.

But tbh what I wanted was just to not keep all my games in my hard drive, instead use something like youtube where they can archive the match for you.

I'm considering automated streaming out of scope for this project


WIP - DEVEXP-423: Mounting node's pull secrets on build pod.

Mounting node pull secrets on build pod, according to:
openshift/enhancements/enhancements/node-pull-credentials

/retest

/retest

/retest

/retest

/test

/retest

/retest

/retest

/retest

/retest

/assign @gabemontero @adambkaplan

/retest

/retest


Hack to compile firmware in our distro

The way the firmware gets a version when it compile requires us to do a hack in our distro. Under the firmware folder run
git tag 2014.01.0-11-gead7f60b5ce1
mkdir .git

This should be properly fixed and we should figure out the right convention for version naming.

Not relevant anymore with new version. It should just work.


Templating System

I tried to make the templating system as easy as possible for the user, so instead of "requesting" or "marking" the room as a template it can be used as one by default; it generates a template ID once you create a room. This might use a bit more space in the database, however storing a string is practically nothing in the grand scheme of things.
In summary:

Template model for creating / getting template
Added template ID to the share modal
Cannot clone rooms without using the template ID
Added templates table to RethinkDB

Looks good. I made some small changes:

Reformatted share modal so its a little bit clearer
Disabled "Clone" button in settings modal when input field is empty. Also modified clone error message to be more ambiguous.
Added extra comments and tweaked some variable/function names so things are a bit clearer

Although we should definitely wait to merge this branch until after the docker branch has been merged since there could be some minor conflicts. Mainly what I'm thinking of is that code to create the "templates" table (if it doesn't exist) on server startup should be added with the other table-creating code I added in that branch.


node-client setup, eslint configuration and URI file conversion

index.js will essentially perform the same task as of init.py in the python code base. It will simply import other functions and classes of other files and export them.


[Request] add Reset button

This app is very user-friendly as it can be operated intuitively:)
It would be better if I could return to the default position simply by clicking the 'Reset' button.

Thank you for using SectionViewer.
As you suggested, I added ‘Reset pos’ button below the main frame.
Please check ver.1.3.2.
2023年12月6日(水) 4:22 Izuki Watanabe @.***>:

This app is very user-friendly as it can be operated intuitively:)
It would be better if I could return to the default position simply by
clicking the 'Reset' button.
—
Reply to this email directly, view it on GitHub
https://github.com/Fukumasu/SectionViewer/issues/4, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHWLSVDWC5Z4S2OPBUV2XRLYIBBMZAVCNFSM6AAAAABAJEEAO6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGAZDQMRRGUZDINI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I have confirmed ‘Reset pos’ button. Thank you for your prompt assistance!


New Docker client from config

Description
This PR

adds helper function for creating a new Docker client from existing configuration and respective unit tests.
adds another unit test to test whether Up, Stop & Remove commands work as expected since this chronology is not guaranteed when running respective unit tests individually.
updates Ps function from mimicking docker ps to docker compose ps

Notes for Reviewers
Signed commits

[X] Yes, I signed my commits.

@acald-creator, I wonder if you've a perspective to offer. // @Aisuko

@Revolyssup, I wonder what you think of @Azanul's responses. He is trying to land this PR today and get a new MeshKit release out the door. With that said, we need to "get it right" before a merge/release happens, so let's ensure that each of your concerns are satisfactorily addressed.

@Azanul do you last commits address @Revolyssup's feedback?

@Azanul do you last commits address @Revolyssup's feedback?

Do you want to incorporate those changes regarless of my response?

@acald-creator what is this force-push about?


border-color of .list-group inside .card doesn't match .card's border-color

In the Bootstrap 4 example where a list group is included in a card (https://v4-alpha.getbootstrap.com/components/card/#content-types) the border color of the card is #E5E5E5 but the border color of the list items is #DDDDDD which makes them stand out from the card.
Operating system: OS X 10.11.5
Browser: Chrome 51.0.2704.84 (64-bit)

You must have great eyesight to notice such a subtle difference 😄

Thank you! :smile:


React Player does not display Twitch videos inside an iFrame

I have react player embedded inside an iFrame. All urls I tried worked fine except that of Twitch. I get the following error.
player.twitch.tv refused to connect.

If I use React player outside of my iframe Twitch works fine.
Twitch url used - https://www.twitch.tv/x2twins

Is it possibly the new CSP restrictions kicking in? You may need to add parent to your config.options object.
https://discuss.dev.twitch.tv/t/twitch-embedded-player-updates-in-2020/23956
See also: https://github.com/cookpete/react-player/issues/904

Is it possibly the new CSP restrictions kicking in? You may need to add parent to your config.options object.
https://discuss.dev.twitch.tv/t/twitch-embedded-player-updates-in-2020/23956
See also: https://github.com/cookpete/react-player/issues/904


Disable test timeouts in meson check globally

Issue description
So on my mac with employer-imposed antivirus snakeoil, a build of p11-kit just failed because a test timed out after 30s. Running it a few times with hot caches and with no other builds in the background made it finally pass but that really just shouldn't happen.
We have configurable timeouts for the entire build, so I don't think we need any per-test timeouts. If a test needs a few minutes rather than a few seconds one time, so be it. That shouldn't fail an entire build IMHO.
Question to the community: Should we just disable timeouts in meson checks globally?
meson test --timeout-multiplier 0 should do it according to the documentation but I'm a bit at a loss where meson test is even invoked or else I'd have made a PR. Perhaps meson maintainers know how this works (cc @brandonedens @AndersonTorres).

meson test

https://github.com/NixOS/nixpkgs/pull/213845
AFAIK ninja is currently being used to run the tests.

(e.g. setting custom timeouts for tests — my motivation for this change).

It looks like greater minds have already thought of this.
cc @alyssais

There's also other manually run meson test https://github.com/NixOS/nixpkgs/blob/da617ec0118932380cf035769b7fd9fe55ad945f/pkgs/development/libraries/folks/default.nix#L91-L95

I am running in the same problem, trying to build this https://github.com/NixOS/nixpkgs/pull/247766, failing 10 times for 10 different timeout errors

I think the only thing left to do on my PR was to incorporate @jtojnar's documentation suggestion — I can do that soon.  I got a bit fed up of so many rounds of review so I put it down for a while.

#213845 is now in staging.

Sorry for bumping this, but I just had p11-kit's (0.25.3) checks timeout on my Mac VM with the nixos-24.05 branch. Is this properly implemented yet, or is there something that needs to be configured?

I'm running into the same issue. Is there currently a workaround?

@Artturin is there a way to apply this globally? I'm getting quite a few timeouts in different packages

Doing this globally is what this issue is about.
Without rebuilds is not possible as this changes the inputs to the drv.

Thanks -- I mistakenly thought it was about changing the defaults and that the toggle already existed

Tip for those who struggle with this, it appears that the timeouts happen when you nix build a bigger package/nixosConfiguration because your computer is compiling a lot of things at once. A workaround for this is to build the derivation that triggers the timeout in isolation. For example, if you see
error: builder for '/nix/store/ph1h78hg21av397ksi56jz4cxycklrh7-p11-kit-0.25.0.drv' failed with exit code 2;
       last 10 log lines:
       >
       >
       > Ok:                 56
       > Expected Fail:      0
       > Fail:               0
       > Unexpected Pass:    0
       > Skipped:            0
       > Timeout:            2
       >
       > Full log written to /private/tmp/nix-build-p11-kit-0.25.0.drv-3/source/build/meson-logs/testlog.txt
       For full logs, run 'nix log /nix/store/ph1h78hg21av397ksi56jz4cxycklrh7-p11-kit-0.25.0.drv'.

Then you can fix it by running:
nix-build /nix/store/ph1h78hg21av397ksi56jz4cxycklrh7-p11-kit-0.25.0.drv

(Note the hyphen! nix-build rather than nix build)
Wait for that to complete, and then re-run your original command

Just in case anyone still facing this issue, to me instructions were not clear, but I assumed that the settings described above had to be set in an overlay, so below is my snippet:


Update index.html(change the address of the socket.io client)

Closed by https://github.com/socketio/chat-example/commit/c16c98540d43c4ad5033aadf7cc2df468f1b9652.

Closed by https://github.com/socketio/chat-example/commit/c16c98540d43c4ad5033aadf7cc2df468f1b9652.


special workspace missing some fundamental usecases of scratchpads

so the current behavior of putting any number of  windows in only one special workspace that can float in or out , that hyprland's special workspace provide can be really really useful no doubts in that , but its is not  very usual or natural use-case, if some one would want to have a bunch of different windows they can just use another workspace , the use-case where scratchpads really shines is it gives you ability to have different programs in different scratchpads that someone can access with different key binding. if we can have similar functionality in hyprland it would be very very nice.
for example i put my note taking app , a terminal , a file browser, and ncmpcpp in different scratchpads and i does not want any of them to be coupled in same scratchpad or (painfully) all in one scratchpad.
i hope  i explained my point as well as my broken english allow me to do so.
at the end i want to say thanks for developing this software :heart:

dupe of #612


Mirek/predict nesting

It is not finished yet but quite close to it. What remains:

[x] Merge Lasse's test duplicate_identities
[ ] Merge fixed hmodel test
[ ] Test on these tests, fix hmodel, possibly other bugfixes
[x] Delete ad-hoc debug prints
[x] Fix LoggingCounters
[x] Set paranoic = False by default
[x] Try the PR with updated version of coq-tactician-reinforce

so this is now waiting for the tests being merged into main. (or I could first merge Lasse's branch directly if it will take a while)

I'm seeing quite a bit of outcommented code. Do you want to keep that?

I will remove the outcommented debug prints.

Also, is the sanity_check function expensive? If so, should it be behind a flag?

sanity_check is a functionality that was there also before, and I hope it should not be too expensive, so I will keep it. On the other hand, I plan to set paranoic = False as default for the DynamicDataServer.

(Sorry, I didn't read your comment about deleting debug code.)

The code is already in virtually finished state. The only missing part is a functional hmodel test -- I already updated the hmodel code, and tested it (thanks, Jason) on the duplicate_identities test. Unfortunatelly, this test doesn't contain any global argument prediction so I cannot yet guarantee that it is working.
On the other hand, since the only possible planned future changes are bugfixes, the code is ready to be thoroughly reviewed, @jasonrute , @LasseBlaauwbroek

Works about the same with the updated version of coq-tactician-reinforce, I also tested with Set Tactician Neural Autocache. and looks to me as expected.

Great, no more problems with the bugs that existed before?

I'm still working through all the code.  It is fairly complicated.  The big thing I'm confused by is the following line:
https://github.com/IBM/graph2tac/blob/bbe77e619a1f1043e95abf36c305d3b4e4b6abdf/graph2tac/tfgnn/predict.py#L227
If reserve is 0.5 (which is the default), doesn't this cause the array to shrink?  And that isn't what we want.  I've very confused by how this works and pasts our tests.

@LasseBlaauwbroek : I haven't encountered any issues.
@jasonrute Yes :-). First I had there only multiplication but then it seemed to me that adding this is more fitting the parameter name reserve.

Great.  Looks good to me!  (And thanks for fixing the testing docs.).


js variables & functions declared in karate-config.js with com.intuit.karate.junit5.Karate not recognising in feature file

Calling javascript variables and functions declared in karate-config.js when testcases are running using 'import com.intuit.karate.junit5.Karate;' not recognising in feature file.
Tried sample project 'karate-junit' also calling js variables in feature file but never succeeded.
here is files im using to test calling javascript variables in feature file
karate-config.js
function fn(){
var config = { demoBaseUrl: 'sampledomain.com' };
config.authInfo = { authTime: '9:30', authToken: '1234567890' };
return config;
}
sample.feature
Feature: sample
refer to karate-config.js to see how these were initialized

def time = authInfo.authTime
def token = authInfo.authToken
Scenario: first hello world
print 'authInfo:authTime->', time

Scenario: second scenario

print 'authInfo:authToken->', token

SampleTest.java
package karate;
import com.intuit.karate.junit5.Karate;
class SampleTest {
@Karate.Test
Karate testFullPath() {
return Karate.run("classpath:karate/sample.feature");
}
}

@jay2020-design sorry, you haven't followed the instructions here: https://github.com/intuit/karate/wiki/How-to-Submit-an-Issue


No worky with Notepad++ v.7.7.1

I installed the latest plugin release to my plugins directory per the instructions and I see no difference in Notepad++. No button on toolbar, no submenu in Plugins menu, etc...its as though I never installed it.

Confirmed on 7.6.4: no button showing up, no submenu etc.

Works fine for me with 64-bit NP++. Installed plugin with Plugin Admin.

I installed the latest plugin release to my plugins directory per the instructions and I see no difference in Notepad++. No button on toolbar, no submenu in Plugins menu, etc...its as though I never installed it.

I had exactly the same problem. I deinstalled Notepad++ and reinstalled it again.
Then recognized, that every plugin-dll was stored in it's own sub-directory. There was also a sub-folder called "MarkdownViewerPlusPlus" inside the plugins - folder. So I moved the MarkdownViewerPlusPlus.dll inside this "MarkdownViewerPlusPlus" sub-folder and restarted Notepadd++.
-> And it finally worked!  (tested with Notepad++ V7.7.1 32-Bit).

Also working on Notepad++ v7.8.6 32-bit

Markdown Viewer 0.8.2 continues to work, now with Notepad++ v7.8.5 and 7.8.6 (64-bit).
This issue should be closed.


Prevent public read access for field

Which Category is your question related to?
API (GraphQL), Auth
Amplify CLI Version
4.18.0
What AWS Services are you utilizing?
AppSync, Cognito, IAM
Provide additional details e.g. code snippets
I created an API that has both Cognito User Pools and IAM authentication. I want unauthenticated users to read a majority of the fields on the book model (e.g. title, description, etc.) but I want some fields to be limited to authenticated users (e.g. chapters).
However, since the providers are different (Cognito vs. IAM) the unauthenticated users are able to read every field regardless of group field level auth rules.
type Book
  @model
  @searchable
  @auth(
    rules: [
      {allow: public, provider: iam, operations: [read]},
      {allow: groups, groups: ["Customer"], operations: [read]},
      {allow: groups, groups: ["Admin"], operations: [create, read, update, delete]}
    ]
  )
{
  id: ID!
  title: String!
  description: String!
  chapters: [Chapter!] # <--- Chapters should be restricted to only authenticated users
    @auth(rules: [
      {allow: groups, groups: ["Customer"], operations: [read]},
      {allow: groups, groups: ["Admin"], operations: [create, read, update, delete]}
    ])
}

How can I prevent public read access for certain fields while allowing it for other fields?

@TheBenck Since the auth rules are applied from Model then to field you would have to specify for each field explicitly
type Book
  @model
  @searchable
  @auth(
    rules: [
      {allow: groups, groups: ["Customer"], operations: [read]},
      {allow: groups, groups: ["Admin"], operations: [create, read, update, delete]}
    ]
  )
{
  id: ID! @auth(rules: [{allow: public, provider: iam, operations: [read]} ])
  title: String! @auth(rules: [{allow: public, provider: iam, operations: [read]} ])
  description: String! @auth(rules: [{allow: public, provider: iam, operations: [read]} ])
  chapters: [Chapter!]
}

Thanks @ammarkarachi. Would that mean I can't have subscriptions? I get this error on compilation:
Per-field auth on the required field id is not supported with subscriptions.
Either make the field optional, set auth on the object and not the field, or disable subscriptions for the object (setting level to off or public)

@TheBenck there are alternatives to using subscriptions with as outlined here

Thanks @ammarkarachi , that got me further. I'm coming across another strange problem with this case though.
I have similar rules for a couponCode field. Only dynamic groups and the Admin group should be authorized to view it. However, it's visible to unauthenticated IAM users as well even though there's no specific rule for it.
Why could this be happening? It seems like scalars cannot be protected this way like the Chapter type could? I've seen this done in the docs though.
type Book
  @model(subscriptions: {level: public})
  @searchable
  @auth(
    rules: [
      {allow: groups, groups: ["Customer"], operations: [read]},
      {allow: groups, groups: ["Admin"], operations: [create, read, update, delete]}
    ]
  )
{
  id: ID!
    @auth(rules: [
      {allow: public, provider: iam, operations: [read]},
      {allow: private, provider: userPools, operations: [read]}
    ])

  # ...

  groupsCanAccess: [String]!
    @auth(rules: [
      {allow: public, provider: iam, operations: [read]},
      {allow: private, provider: userPools, operations: [read]}
    ])
  couponCode: String
    @auth(rules: [
      {allow: groups, groupsField: "groupsCanAccess", operations: [read]},
      {allow: groups, groups: ["Admin"], operations: [create, read, update, delete]}
  ])
}

Also, is purely field-level auth incompatible with @searcahble?

@edwardfoyle Any update on these issues? Thanks

@edwardfoyle @ammarkarachi Does anyone know what could be the issue here? This has been a problem for me for two weeks now. Thanks

I am having exactly same issue, for example, let's say I have a model like this:
type User @model @auth(rules: [
    { allow: owner, identityClaim: "sub", operations: [create, update, delete]},
    { allow: private, provider: iam },
  ]) {
  id: ID!
  nickname: String!
  phoneNumber: String!
  profilePicture: S3Object
  backgroundPicture: S3Object
  owner: String!
  updatedAt: AWSDateTime
  createdAt: AWSDateTime
}

running
await API.graphql({
        query: listUsers,
        variables: {},
        authMode: "AWS_IAM",
      });

would return absolutely nothing as expected, however if I just allow one field in the model to be accessible publicly like so:
type User @model @auth(rules: [
    { allow: owner, identityClaim: "sub", operations: [create, update, delete]},
    { allow: private, provider: iam },
  ]) {
  id: ID!
  nickname: String!
  phoneNumber: String!
  profilePicture: S3Object @auth(rules: [{ allow: public, provider: iam, operations: [read] }])
  backgroundPicture: S3Object
  owner: String!
  updatedAt: AWSDateTime
  createdAt: AWSDateTime
}

suddenly all fields are now available publicly, why?

@edwardfoyle @ammarkarachi
this works:
type Thing
  @model
  @auth(rules: [
    { allow: owner, identityClaim: "sub", operations: [create, update, delete]},
    { allow: public, provider: iam, operations: [read] }
  ]) {
    id: ID!
    nickname: String @auth(rules: [{ allow: owner }])
    phoneNumber: String @auth(rules: [{ allow: owner }])
    profilePicture: String!
    backgroundPicture: String!
}

However, if you replace the field level auth
nickname: String @auth(rules: [{ allow: private, provider: iam }])
phoneNumber: String @auth(rules: [{ allow: private, provider: iam }])

then the Thing.nickname.req mapping template doesn't have any auth around it.

@TheBenck @codepreneur
You could add @aws_cognito_user_pools this way the field chapters is only accessible by user pool auth
type Book
  @model
  @searchable
  @auth(
    rules: [
      {allow: groups, groups: ["Customer"], operations: [read]},
      {allow: groups, groups: ["Admin"], operations: [create, read, update, delete]}
    ]
  )
{
  id: ID! @auth(rules: [{allow: public, provider: iam, operations: [read]} ])
  title: String! @auth(rules: [{allow: public, provider: iam, operations: [read]} ])
  description: String! @auth(rules: [{allow: public, provider: iam, operations: [read]} ])
  chapters: [Chapter!] @aws_cognito_user_pools
}

Unfortunately it is still not solved, right? I have a User model and want to allow all fields being publicly available except email. How would I do that? Can't find a solution. As @codepreneur stated, making one filed public will result in all fields being public. Any ideas?
type User @model @auth(
  rules: [
    { allow: groups, groups: ["Admin"], operations: [create, update, delete] }
    { allow: public, provider: iam, operations: [read] }
    { allow: private, operations: [read] }
  ]
) {
  id: ID!
  email: String @auth(rules: [{ allow: public, provider: iam, operations: [] } { allow: private, operations: [] }]) # ???
  name: String
  address: String
}

Just want to let you know that I found a solution in #5556


Fixing the mirror gk_geometry unit test

I just updated some headers in this unit test so it compiles. Very minor changes

addressed in PR #469


Fix DeltaBitPack MiniBlock Bit Width Padding

Which issue does this PR close?
Closes #1416
Closes #1417
Rationale for this change
Fixes https://github.com/apache/arrow-datafusion/issues/1976
What changes are included in this PR?
Makes DeltaBitPackEncoder padding deterministic, and makes DeltaBitPackDecoder resilient to non-zero padding
Are there any user-facing changes?
Output that was previously somewhat non-deterministic, is now deterministic.

I have tested Datafusion against this cherry-picked onto the 10.0.0 release and can confirm it fixes https://github.com/apache/arrow-datafusion/issues/1976.
The branch is here. I do wonder if it might be an interesting exercise for me to help cut a patch release for this so I can get some exposure to this process, thoughts @alamb?

@tustvold  is this a regression introduced in parquet? If so, which version was it introduced in (9.1 maybe?)

Was it introduced in https://github.com/apache/arrow-rs/pull/1284 maybe?

#1417 was a regression introduced by #1284, #1416 was a pre-existing issue that has probably existing for a very long time

I think this PR needs some tests

By making it pad with 0xFF when running in test mode we now get coverage from the existing tests?

I added a test for #1417 and whilst doing so fixed a bug where a truncated page could get the decoder stuck in an infinite loop

I am happy to make 9.1.1 and 10.0.1 branches and prepare the releases.
@sunchao  would you recommend we proactively release 9.1.1 and 10.0.1? If so I'll make a ticket and start the process. It is probably the right thing to do I suppose -- I was just dreading the voting process :)

SGTM @alamb , although I'm not sure if we should wait for #1432 too for 10.0.1. BTW I can help on the release process too.

SGTM @alamb , although I'm not sure if we should wait for https://github.com/apache/arrow-rs/pull/1432 too for 10.0.1. BTW I can help on the release process too.

If you could talk on creating the 9.1.1 and 10.0.1 releases that would be great! I'll file a ticket to start tracking it

https://github.com/apache/arrow-rs/issues/1442 tracking the patch releases

Thanks @tustvold


Indirect branch tracking

OpenBSD now enforces indirect branch tracking (BTI on arm64, IBT on x86_64) which breaks Crystal on OpenBSD going forward.
I'm not sure whether this would all be handled by llvm and Crystal's compiler just needs to turn it on (in clang it's -fcf-protection=branch) or if there is assembly in Crystal that needs to be modified with the new instructions.

It says that indirect branch tracking is already on by default unless there is an opt-out linker flag, so do we actually need to do anything?

Yes, the opt-out is temporary.

OpenBSD enforces it now and Crystal has been marked as broken

I'd like to help get Crystal supported on OpenBSD, what work needs to be done in order to get IBT support? I'm not sure where to start.

@joshturge maybe check out QBE's implementation
Note that it was reverted due to tooling reasons, but I am trying to do a roll-forward since the blocker seems gone.

We need to tell LLVM to generate the correct instructions on the x86_64 and aarch64 targets —in gcc/clang this is the -fcf-protection=branch option.
I understand we also need to fix the fiber context switch asm for x86_64 and aarch64 to have these too, so we start & resume a fiber?

Searching for examples, I found out that Rust adds the cf-protection-branch=1 flag when creating a LLVM module when we pass -Z cf-protection=branch to rustc.
See the source code and Rust book.

The -fcf-protection=branch option enables the control-flow enforcement technology (CET) on x86 only. For AArch64 this is the -mbranch-protection=bti option to enable the branch authentication instructions :weary:
Looking at the disassembly of a compiled C program, the x86 option starts all functions with endbr64 while the aarch64 option starts all functions with bti c. We might just need to add these in the ASM of fiber context switches (as the first resume instruction) when the feature is enabled?
There are more specific values to each option, to enable more features. See the links above for details. I don't know about their impact on the ASM of fiber context switches. They're not needed to support OpenBSD; in fact IBT/BTI isn't needed and we could keep it disabled, but it's interesting to follow on the security and see & measure the impact.

Initial attempt: https://github.com/crystal-lang/crystal/compare/master...ysbaddaden:crystal:feature/indirect-branch-tracking?expand=1
It only works with --single-module otherwise we fail with incompatible LLVM module / function / context / type, which means I fail to set the values on every LLVM module (I guess). I must have missed something.


CodeGPT "Unable to connect to the extension services"

I was able to use CodeGPT vscode extension before.
Today, the extension refuses to load with the message "Unable to connect to the extension services".
vscode version 1.91.1
vscode node version 20.9.0
netstat for port 54112 returns empty
I tried all of the following:

Switched from RemoteSSH to WSL to Windows local
Disconnected from VPN
Reboot
Reinstalled the extension a couple of times.

None of the above solved the issue.
Please advise.

The issue resolved with no action from my side.

me too


react-native-maps incompatible with react-native 0.32 on Android

Upgrading to react native .32 from previous versions breaks react-native-maps on Android. The stack trace points to RegionChangeEvent.java and specifically the constructor of this class:

I opened an issue with react native that can be tracked here https://github.com/facebook/react-native/issues/9863

After looking into the Event class located at react-native/android/com/facebook/react/react-native/0.31.0/react-native-0.31.0-sources.jar!/com/facebook/react/uimanager/events/Event.java this definitely seems to be the issue... Its interesting to me that facebook made these changes without any backwards compatibility and without putting this in the release notes...

upgrading to the most recent version of react-native-maps seems to resolve this issue as the constructor was rewritten in following commit: https://github.com/airbnb/react-native-maps/commit/6d6829253d6c8129aad9d55d048aa57700724232


Python Wrapper with Basic Functionalities Added

Thanks. Let's also start to follow our best practices in this project from the beginning. Add issues and match them with PRs. Cheers.


THANK YOU

I just wanted to say thank you so much for this amazing repo and app. This seemingly simple request is insanely hard to pull off well in OSX.

 I'm also willing to see more people join in this project and more FOSS on macOS.
(As this is not an issue, I'll convert it to discussion.)


Bad documentation with Debugging the Add-on

I'm trying to debug an Add-in for Outlook on Windows.
The below documentation talks about a "bundle.js" which needs clarification.
https://learn.microsoft.com/en-us/office/dev/add-ins/outlook/autolaunch?tabs=xmlmanifest#supported-events
"bundle.js"
Where does this come from?
Who creates it?
Where is it download from and which directory?
"What I'm trying to do"
I'm using the sample code in the documentation to create a OnMessageSend Launch Event handler, but the sample code does not work. I would love to get some information on what is going on.
Because this is not on a task pane sidebar view, there is nothing for Edge browser inspect tool to view.

I have transferred this to the repo for documentation.

Hi @geneLavender,
Thanks for reaching out. The bundle.js file is created when you use the Yeoman generator (yo office) to create your add-in project. Yo office uses webpack to take care of mapping file dependencies and bundling assets in your add-in project. The bundle.js file is created in %LOCALAPPDATA%\Microsoft\Office\16.0\Wef\{[Outlook profile GUID]}\[Outlook mail account encoding]\Javascript\[Add-in ID]_[Add-in Version]_[locale] when you run npm run build or npm start (runs npm run build as part of its script).
For guidance with debugging your event-based add-in, please refer to the following to enable runtime logging and set breakpoints in your code:

Debug your add-in with runtime logging
Debug your event-based Outlook add-in


Tutorials improvements

New clean Pull Request to substitute 704 due to
a bad handling of me.
Main points of revision is this:
https://github.com/opesci/devito/pull/704#discussion_r233517834
I would like to ask sorry for my bad handling.
I accidentally merged an old master and force pushed.
TODO squash multiple commits.

Codecov Report

Merging #735 into master will decrease coverage by 5.3%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #735      +/-   ##
==========================================
- Coverage   88.05%   82.75%   -5.31%
==========================================
  Files         100      100
  Lines        9931     9931
  Branches     2014     2014
==========================================
- Hits         8745     8218     -527
- Misses        941     1439     +498
- Partials      245      274      +29

Impacted Files
Coverage Δ

devito/mpi/routines.py
19.13% <0%> (-80.87%)
:arrow_down:

devito/mpi/utils.py
22.72% <0%> (-77.28%)
:arrow_down:

devito/builtins.py
24.24% <0%> (-48.49%)
:arrow_down:

devito/mpi/distributed.py
52.15% <0%> (-38.28%)
:arrow_down:

devito/core/operator.py
49.35% <0%> (-36.37%)
:arrow_down:

devito/function.py
75% <0%> (-17.21%)
:arrow_down:

devito/types.py
85.79% <0%> (-8.85%)
:arrow_down:

devito/data/data.py
82.98% <0%> (-8.77%)
:arrow_down:

devito/core/autotuning.py
78.91% <0%> (-7.57%)
:arrow_down:

devito/ir/iet/scheduler.py
90.21% <0%> (-6.53%)
:arrow_down:

... and 15 more

Continue to review full report at Codecov.

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

then can we close #704 ?

Merged.


Recurring triggers and compile errors

Captured from Comments on Examples Page - Moved here for better visibility
jmealing3 Thu at 8:29 PM
Great looking package, thanks.
Question: How do I schedule a task to run at uniform intervals? I see how to set the start time and the end time for the process being started, but I want to have a process that re-schedules itself at regular intervals.
More examples and better descriptions would be greatly appreciated (not everyone is as strong a C# and OO programmer as you).
Also, I expected the notepad example to actually run notepad - it doesn't seem to do anything. I think that I'm missing some understanding of how the TaskScheduler is supposed to work
dahall Thu at 11:38 PM
See the TimeTrigger section of the "Complex Example" above for an example of how to setup a recurring trigger on an interval. I'll look to improve the examples and descriptions. For the examples, notepad should actually run if the triggers and the task are all enabled.
jmealing3 Today at 1:24 PM
Thanks for the pointer. Just tried to build the editor and it would not compile. Couldn't find the TaskEditDialog. I have added references to all the dlls in the TaskScheculer and Editor but the compiler (VS 2010) cannot find the dialog. What am I missing? Also, every example ends by removing the task. Why and where should I look to find the registered task(s)? I commented out that line and still cannot find the task register file. The root directory come back as "", b ut I've looked everywhere and still cannot find the file. Thanks for your help one this.
jmealing3 Today at 2:48 PM
Some good news: The notepad task did finally run: way late, but it ran.
Environment: .Net 4, VS 2010 Ultimate, WinXP SP3
More problems: I downloaded the source code and only the COMTask would build. Here are the error messages from the TaskEditor:
Error 727 'AeroWizard.WizardPage' does not contain a definition for 'Suppress' and no extension method 'Suppress' accepting a first argument of type 'AeroWizard.WizardPage' could be found (are you missing a using directive or an assembly reference?) D:\Downloads\TaskScheduler\taskscheduler-70200\TaskService\TaskEditor\TaskSchedulerWizard.cs 868 9 TaskEditor
From TestTaskService 17 errors, first one shown:
Error 605 'Microsoft.Win32.TaskScheduler.TaskSchedulerWizard' does not contain a definition for 'AvailablePages' and no extension method 'AvailablePages' accepting a first argument of type 'Microsoft.Win32.TaskScheduler.TaskSchedulerWizard' could be found (are you missing a using directive or an assembly reference?) D:\Downloads\TaskScheduler\taskscheduler-70200\TaskService\TestTaskService\Program.cs 192 10 TestTaskService
jmealing3 Today at 4:19 PM
Got some more information that might help: I'm running Win XP SP3, but behind on updates (lots of them). When I built the Complex example, I found that I have only V1.1 of TaskScheduler. While this won't fix everything, it might reduce the problems somewhat. Any idea where to get V 2?
I will be installing the updates today, hoping that V2 comes in, but the list of updates does not show V2 explicitly. Is there some place that I can get it? MS downloads don't show it anywhere. Thanks in advance.
jmealing3 Today at 5:10 PM
Still more info: Task scheduler doesn't run on XP. My bad!
ComplexTaskSample doesn't compile: line 185 throws an exception. It shouldn't, but it does - at least on my machine. The line is:
MonthlyDOWTrigger mdTrigger = (MonthlyDOWTrigger)td.Triggers.Add(new MonthlyDOWTrigger());
Originally posted: 2011-09-09T11:38:50

**To compile the solution:**You will need to look in the projects at the references and make sure the corresponding assemblies from the current download are in those locations. You don't really need to compile the solution. You should be able to just pull the assemblies from the download section and then reference them in your code. To make this really easy, I would recommend installing NuGet (nuget.org) and using it to pull in the references you need. The base library is available as "Task Scheduler Managed Wrapper". This will work for all programmatic access to tasks. If you need UI elements in your code to give the users the ability to manipulate tasks then also include "Task Scheduler Managed Wrapper UI library". The nice thing about NuGet is that it adds the references and all dependencies for you and then will alert you whenever there are updates.
V2: The V2 version of the library is only available on Vista/Server 2008 and later. You cannot get it for XP or Server 2003. XP and earlier have a "Tasks" item in Control Panel to view the current tasks.
Originally posted: 2011-09-09T19:51:54

Thanks for all your help. Got Wizard running, my bad. NuGet is smooth, thanks for the pointer.
Looking forward to 1.6.4 so that DOW trigger works.
Originally posted: 2011-09-12T11:03:32

Wrote a simple task scheduler to run a task at 5 minute intervals for 30 minutes using TimeTriggerEndBoundry. Set the TimeTriggerRepetition.Duration to 4 hours. Set StopAtDurationEnd to true.
After 1.5 hours, it is still running. It looks like the Duration overrides the EndBoundry - or am I missing something again?
Thanks for the help.
Originally posted: 2011-09-13T09:56:45

EndBoundary has no effect on the time a task is allowed to run. StopAtDurationEnd will kill the task at the end of the time.
Originally posted: 2011-09-13T14:47:20


chore: version bump and min ios bump

Fixes for Xcode 16

I've also accidently upgraded to xcode v16.
This patched helped me

I think we can merge this


Migration doesn't work

When i enter : php yii migrate/up --migrationPath=@vendor/dektrium/yii2-user/migrations
Cmd send me PHP Fatal Error 'yii\base\errorexecption' 'Class 'dektrium\user\migrations\Migration' not found'
in C:\wamp\www\vendor\dektrium\yii2-user\migrations\m140209_132017_init.php:7

And when I writed this command - it printed error.
Migration failed. Directory specified in migrationPath doesn't exist:
@vendor/dektrium/yii2-user/migrations

@Graphyxx the error you mentioned can only appear if you installed yii2-user manually. Please reinstall it using composer.

I try to install via Composer but still get this error!


Catch malformed releases table before merging

These lines don't handle version substitution in a way that retains the valid structure of the table:
https://github.com/cilium/cilium/blob/3bb016ffd4f574ea04e70a8509295a2f0dd36f27/contrib/release/bump-readme.sh#L37-L39
As a result, we can unintentionally break the table and make it disappear entirely. It's easily fixed, for example:
https://github.com/cilium/cilium/pull/13365
However it would be better to catch before we merge it.
Possible paths:

Fix the script to make it understand the table better, or
Add a simple .RST validator for the README.rst to make sure that if we break the table, the CI will complain.

https://github.com/myint/rstcheck seems to do a good enough job:
$ rstcheck README.rst
README.rst:34: (ERROR/3) Malformed table.
$ echo $?
1

Are there other files besides README.rst we would like to check? I'm guessing we don't need to do that for Documentation/ since we already have a renderer there.
We could run on all other .rst files, just in case:
$ find -path ./Documentation -prune -false -o -name "*.rst"
./MAINTAINERS.rst
./examples/minikube/README.rst
./examples/getting-started/README.rst
./examples/policies/l3/simple/README.rst
./FURTHER_READINGS.rst
./README.rst
$ find -path ./Documentation -prune -false -o -name "*.rst" | xargs rstcheck
./README.rst:34: (ERROR/3) Malformed table.

This new check could be added as a GitHub action.

It should use the --report error flag
❯ rstcheck --report error README.rst

Otherwise we would have kind of false positives
❯ rstcheck README.rst
README.rst:269: (INFO/1) Duplicate implicit target name: "getting started".

This is the cause of the INFO/1 warning
Getting Started
===============
...
* `Getting Started`_

@geyslan Done. Don't hesitate to reach out on Slack (e.g., in #development) if you have any question.

Reopening since we reverted the fix. Proposed new solution is presented here:
https://github.com/cilium/cilium/pull/16786#issuecomment-876491875

Reopening since we reverted the fix. Proposed new solution is presented here:
#16786 (comment)

I think the solution is being addressed by @qmonnet, right? If so, I suppose that's ok to unassign me? Thanks.

I don't think that @qmonnet has picked this up, but if you don't plan to work on this then we can unassign you :thumbsup:

@joestringer, said that, no worries, I can continue this work. I suppose that this proposal could solve the issue related to new branches: https://github.com/cilium/cilium/pull/16387#issuecomment-874184376
As I commented, it seems to work at my forked repo: https://github.com/geyslan/cilium/runs/2991422456?check_suite_focus=true
What do you suggest for testing it without the need to remotely merge this workflow?

Please refer to my comment on the reverting PR. We already have rstcheck running on the RST files, and I don't think the current PR brings anything more other than checking README.rst (which is easy to add to the current rstcheck run we have). If I'm wrong, please convince me. I also believe that the current version of rstcheck is not usable to detect malformed tables, because it breaks on tables containing substitution references.
I would love the maintainer of rstcheck to merge the pending PR for that, but sadly it doesn't look like the project is very active. So that leaves us with a few solutions:

Wait more
Use our own rstcheck fork with the fix (but is it worth the trouble?)
Use yet another tool (is it desirable?)

But please let's not add a second rstcheck run if it brings no additional verifications.

Wondering if this is still an issue today and if so, if it's worth addressing and if we can get this scheduled for 1.15. If not, we should consider closing.

I got my fix merged in rstcheck in https://github.com/rstcheck/rstcheck/pull/82, released in v6.0.0. We use v6.1.1 for our checks, so we should be able to detect malformed tables (we stopped skipping related messages in e90df2f1a103b1e363ab215eba8fa9647918c870).
So I think we're all good, and can close this issue.

@qmonnet did we end up adding README.rst to the rstcheck action?

Ah no, sorry, I didn't go through the whole discussion again and forgot we were talking of the README.rst. But this is an easy fix: https://github.com/cilium/cilium/pull/26454


Compactor: Provide more info when unhealthy index found

Is your proposal related to a problem?

Compactor halted when out-of-order chunks were found.
The error log is:
level=error ts=2021-07-22T06:50:47.485784592Z caller=compact.go:477 msg="critical error detected; halting"
err="compaction: group 0@8630763776085252337: block with not healthy index found /var/thanos/compact/compact/0@8630763776085252337/01FB5VKXMFT7HZ2R1WKTNNKG00; Compaction level 1; Labels: map[cluster:lhr1 prometheus:gitops-platform-metrics/k8s prometheus_replica:prometheus-k8s-0]:
1/1041404 series have an average of 1.000 out-of-order chunks: 0.000 of these are exact duplicates (in terms of data and time range)"

It only mentions 1 series has out-of-order chunks but doesn't say which series causes the problem. So the only way to fix this is to delete the whole block, which is not ideal.
Describe the solution you'd like

Provide the series information in the logs so that I can use the bucket rewrite tool to delete the bad series, instead of deleting the whole block.
Describe alternatives you've considered

The only concern here is what if there are too many bad series? Do we need to display them all? But maybe that's okay.
Additional context

(Write your answer here.)

@metonymic-smokey When this kind of error happens, we can log the labels of that series.

/assign me


showing/hiding for mobiles

This is a request for documentation.
Are there classes to show/hide based on the screen width? (similar to visible-xs-block or col-sm-3 in bootstrap) http://stackoverflow.com/questions/28940562/skeleton-css-hiding-content-from-mobiles
Looks like there isn't https://github.com/dhg/Skeleton/blob/master/css/skeleton.css
Given that's a common use case it would be nice to document skeletons take on this.

There isn't any way to do this out of the box with skeleton, the easiest way to do this is to either create your own classes or to use media queries and change the display property for the element

Agree with @benhinchley I am using .six.columns class but I want to make them show in half starting from 768px and not from 550px like skeleton does. So i create a custom class for those divs and add @media (max-width: 767px){
& .side{
width: 100%;
}
}
And problem solve :D After 768px it will apply Skeleton's regular css for six.column. Hope it helps. I guess they Don't add such a helper 'cause they want to keep sooooo but sooo simple 😸


Implicit Flow in Angular 5 Application:  Successfully Authenticating but getIdentityClaims() returning NULL Values

Describe the bug
There are at least two (2) other issues on GitHub that are similar to this one, but neither of them provide a clear solution to the problem.  In my Angular 5 application, I am using a public API for authentication.  I do not have "system administrator" access to the authentication/resource server, so I am unable to make any changes in this regard.  I do know that its a Microsoft machine and does not have a discovery document (hence, I am using the tryLogin() method).
On the client side, I am able to initialize the Implicit Flow and make the call from within my Angular 5 code.    There is no login button for the user to click:  If the app does not detect an access_token, it automatically redirects the user to the login form, prompts them to authenticate using a Smart Card, and if successful, redirects them back to the app with the access_token and id_token in the URL query string.
At this point, everything works as expected.   But when I attempt to run getIdentityClaims(), it returns null.   In other issues I've read, some have asked are there console errors at this point.   In my application IS a console error (in an earlier post I wrote that there wasn't a console error):  Error validating tokens.  Wrong Issuer.  This is odd because a token is returned in the URL.
The following are code snippets from my application.  I've trimmed it down a lot for simplicity purposes, but will provide more detail if requested.
app.module.ts
imports: [
 OAuthModule.forRoot({
      resourceServer: {
          allowedUrls: ['https://apigw-stg.cdc.gov:8443/openid/connect/v1/userinfo'],
          sendAccessToken: true
      }
  })
]

authConfig.ts
export const authConfig: AuthConfig = {
  oidc: true,
  issuer: 'https://apigw-stg.cdc.gov:8443/auth/oauth/v2',
  loginUrl: "https://apigw-stg.cdc.gov:8443/auth/oauth/v2/authorize",
  redirectUri:  window.location.href,
  clientId: xxx,
  responseType: 'id_token token',
  scope: "openid profile email",
  tokenEndpoint:  'https://apigw.cdc.gov:8443/auth/oauth/v2/token',
  userinfoEndpoint: "https://apigw-stg.cdc.gov:8443/openid/connect/v1/userinfo",
  showDebugInformation: true,
  strictDiscoveryDocumentValidation: false,
}

app.component.ts
constructor(private route: ActivatedRoute, private router: Router, private injector: Injector, private http: HttpClient, private oAuthService: OAuthService) {
     this.oAuthCall();
}

oAuthCall() {
     this.oAuthService.configure(authConfig);
     this.oAuthService.setStorage(sessionStorage);
     this.oAuthService.tokenValidationHandler = new JwksValidationHandler();
     this.oAuthService.tryLogin();
}

runOnAppLoad() {
     let accessToken = SharedUtilities.getURLHashValue('access_token');  // Get token from URL
     if (accessToken) {
        const claims = this.oAuthService.getIdentityClaims();
        console.log(claims);  // VALUE IS NULL!
     else {
        this.oAuthService.initImplicitFlow();
     }
}

Expected behavior
Once authentication is successful and the user is redirected back to the application, calling getIdentityClaims() should return object or value other than null.
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome (Version 76.0.3809.100)

What order do your methods (constructor, oAuthCall, and runAOnAppLoad) run in? I presume runOnAppLoad runs too soon, before the tryLogin call does its magic.
I also think you need to call loadUserProfile() to exchange the token for actual details about the user.
I recommend something more like this:
    this.oAuthService.tryLogin()
        .then(() => {
            if (!this.oAuthService.hasAccessToken()) {
                // This will redirect so the promise chain will stop
                this.oAuthService.initImplicitFlow();
            }
            console.log('logged in!');
        })
        .then(() => this.oAuthService.loadUserProfile())
        .then(() => {
            const claims = this.oAuthService.getIdentityClaims();
            console.lg(claims);
            // do whatever
        })
        .catch(err => console.error('login sequence failed with: ', err));

I've typed this without trying, so you may have to adjust the specifics, but hopefully the idea is worth something.

Hi jeroenheijmans - Thank you for your reply!
So to answer your question about what order do my methods run:  The constructor runs first and calls this.oAuthCall().   this.oAuthCall() configures authConfig class and sets the session storage and token validation handler.   tryLogin(), from my understanding, simply determines the approach the application will use (ie, look for a discovery document or not).   In most online tutorials I've seen, the initImplicitFlow() is what actually triggers the authentication process, which is why this method is usually wrapped inside a login() method.   My app doesn't have an actual login button, so the initImplicitFlow() method is triggered IF an access token isn't found in the URL.
I'm very much new to implementing oAuth in an Angular context (as well as using this particular NPM package) so please correct me if my understanding of this process is wrong.
As far as your code suggestion...
I gave it a try (changing "hasAccessToken" to "hasValidAccessToken" which is the method listed in the documentation/index.   The first call back works really well:  no access token is detected so the initImplicitFlow() is run.  Then, ironically, loadUserProfile() runs, and THEN the redirect to the login form occurs.   I then authenticate using the smart card and am redirected back to the authentication server.
Once back in the Angular app, NOTHING runs.   Obviously the initImplicitFlow() wouldn't run because we have an access token now.   But even the loadUserProfile doesn't run.   Instead, we get the same "Wrong Issuer" error in the console.

About the error: Have you checked what the issuer is according to the token? You can use e.g. https://jwt.io/ to decode it. I presume that the issuer is just the plain origin and not the full url you have in your config (https://apigw-stg.cdc.gov:8443/auth/oauth/v2).
About tryLogin: this method will look at the URL and see if the hash fragment contains tokens (as supplied when you got redirected back after logging in). It will grab those tokens and remove them from the URL, putting them into storage, and then resolving. You can check out the code.
About the library's workings: the code (linked above) is quite readable, so you can always decide to have a look to see what it does. Alternatively, you can check out the annotated login sequence I use and pick the pieces you like for your setup.

Thanks for the jwt.io suggestion!  I decoded it and you were right, the origin was different.  I replaced the incorrect one with the correct one, and am no longer getting the "Wrong Issuer" error.  But getIdentityClaims() is still returning null.   I'm stomped.

Hmm, okay that's good. Let's see:

getIdentityClaims() merely returns something from storage
loadUserProfile() should set the relevant storage item

So I think it'll be something specific to your situation, or something I overlooked glancing at your code. Here's some suggested ways to debug this:

Set a breakpoint in loadUserProfile() and inside its lambda functions. This should be your best bet in learning what exactly is going on.
Check the network tab of the dev tools to see what happens with the call to userinfoEndPoint: what does it return?
Add some additional logging to see what's going on.
Check the relevant tab of your dev tools to see what items are set in sessionStorage

Good luck!

Have you looked through the documentation yet? It might be possible to set stuff for jwks without using discovery, but if i recall correctly there's also some docs for using a NullValidationHandler.
Anyways, good to hear you've tracked your problems down, good luck with the final bits!
Remember, there's also Stack Overflow if you have questions or run into implementation issues - the issues list here is an awkward place for this kind of support as there's only very few folks actively responding to questions.


Zigbee device is not added HA

zigbee2mqtt add-on version (if edge, please report commit hash):
NODE_VERSION | 12.19.0
YARN_VERSION | 1.22.5
Operating environment (HassOS, Virtual Machine, Device/platform running Home Assistant):
Docker 18.09.0-0513 on Synology DS1618+ with DSM 6.2.3-25426 Update 2.
Home Assistant Core 0.119.0.dev20201127
Supervisor 2020.11.0
HACS 1.6.2
Description of problem:
I have installed zigbee2mqtt and MQTT, when i add devices zigbee is detecting em and add em too MQTT
(This is some data from my log inside portainer, for zigbee2mqtt)

Zigbee2MQTT:info  2020-11-28 08:40:54: MQTT publish: topic 'zigbee2mqtt/Temp_stue', payload '{"battery":100,"device":{"applicationVersion":5,"dateCode":"20191205","friendlyName":"Temp_stue","hardwareVersion":30,"ieeeAddr":"0x00158d00030b7984","manufacturerID":4151,"manufacturerName":"LUMI","model":"WSDCGQ11LM","networkAddress":1715,"powerSource":"Battery","softwareBuildID":"3000-0001","stackVersion":2,"type":"EndDevice","zclVersion":1},"humidity":48.52,"linkquality":68,"pressure":1018.6,"temperature":22.15,"voltage":3015}'

Zigbee2MQTT:info  2020-11-28 08:40:54: MQTT publish: topic 'zigbee2mqtt/Temp_stue', payload '{"battery":100,"device":{"applicationVersion":5,"dateCode":"20191205","friendlyName":"Temp_stue","hardwareVersion":30,"ieeeAddr":"0x00158d00030b7984","manufacturerID":4151,"manufacturerName":"LUMI","model":"WSDCGQ11LM","networkAddress":1715,"powerSource":"Battery","softwareBuildID":"3000-0001","stackVersion":2,"type":"EndDevice","zclVersion":1},"humidity":48.52,"linkquality":68,"pressure":1018,"temperature":22.15,"voltage":3015}'

So the data from the devices is ok, but when i then open HA then I can't find em as sensor, can someone help me?
Say if you need some files provided...

No one that can help here ?

@tpbillund This repo is just for the add-on wrapper. The discovery & other mqtt functionality are not handled here. You can try the main zigbee2mqtt repo for issue report. Might be worth checking if any existing similar issues exist.
https://github.com/Koenkk/zigbee2mqtt/issues

thx :-)


Allow dispatch caller to get stats back

Added dispatchWithCounts which now returns a DispatchResult which includes the previously returned CompletableFuture<List<X>>, plus a count of how many entries (keys) were dispatched.  This will be used from graphql-java in a different PR.

Verbal approval, besides this isn't going to prod, and it isn't going upstream (yet!)


Stop timeout advice is confusing

Please clarify the advice in the Dockerfile regarding stop timeout:
# An additional setting that is recommended for all users regardless of this
# value is the runtime "--stop-timeout" (or your orchestrator/runtime's
# equivalent) for controlling how long to wait between sending the defined
# STOPSIGNAL and sending SIGKILL (which is likely to cause data corruption).
#
# The default in most runtimes (such as Docker) is 10 seconds, and the
# documentation at https://www.postgresql.org/docs/12/server-start.html notes
# that even 90 seconds may not be long enough in many instances.

The referenced PG documentation mentions the time to become ready, not shut down.  The PG shut down doc (at least for v12) doesn't mention the 90 second issue.  Also, if this comment is in fact accurate/important, may I suggest putting it in the "how to use this image" doc?

https://www.postgresql.org/docs/12/server-start.html#:~:text=consider carefully the timeout setting

Consider carefully the timeout setting. systemd has a default timeout of 90 seconds as of this writing and will kill a process that does not notify readiness within that time. But a PostgreSQL server that might have to perform crash recovery at startup could take much longer to become ready. The suggested value of 0 disables the timeout logic.

It's not necessarily a "90 second issue" so much as even at 90 seconds the docs say it might not be long enough. I imagine the grace period length is really just dependent on the host's environmental factors such as disk speed and amount of data in transit at the start of the signal being sent
It's talking specifically about Docker here though and how Docker has a default grace period of 10 seconds. Similar discussions https://github.com/docker-library/postgres/issues/544 https://github.com/docker-library/postgres/issues/184

I think our intention with pointing to that specific document was just as an example of a critical database operation that might take up to 90 seconds to complete (so if you tried to stop the server during that operation, it would potentially need at least that long before you kill it if you don't want to risk data loss).
There's not a really great document to link to about "smart" vs "fast" (or about shutting down the server in general), but doing Ctrl-F for "smart" on https://www.postgresql.org/docs/12/app-pg-ctl.html has a little bit.

@tianon That's fair. Thanks for addressing my concern. Please consider adding this information to "how to use" since it seems quite relevant there? Otherwise, feel free to close this issue.

It's difficult to balance our limited space on the Hub description with the sheer amount of things we could document there. :grimacing: :see_no_evil:
I'm glad we got it explained well enough for you to understand!  Hopefully this thread can serve as a good reference for other folks in the future. :+1:


HDFS-17093. In the case of all datanodes sending FBR when the namenode restarts (large clusters), there is an issue with incomplete block reporting

In the case of all datanodes sending FBR when the namenode restarts (large clusters), there is an issue with incomplete block reporting

Please push update to your same original branch (for this case which is Tre2878:HDFS-17093), DO NOT pull repeat request for same issue.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 49s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

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

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
52m 27s

trunk passed

+1 :green_heart:
compile
1m 25s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
1m 15s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
checkstyle
1m 11s

trunk passed

+1 :green_heart:
mvnsite
1m 23s

trunk passed

+1 :green_heart:
javadoc
1m 11s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 38s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
spotbugs
3m 25s

trunk passed

+1 :green_heart:
shadedclient
41m  6s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 14s

the patch passed

+1 :green_heart:
compile
1m 17s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
1m 17s

the patch passed

+1 :green_heart:
compile
1m  8s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
javac
1m  8s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m  3s

the patch passed

+1 :green_heart:
mvnsite
1m 18s

the patch passed

-1 :x:
javadoc
0m 57s
/patch-javadoc-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-hdfs in the patch failed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1.

+1 :green_heart:
javadoc
1m 29s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
spotbugs
3m 26s

the patch passed

+1 :green_heart:
shadedclient
41m  7s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
251m 21s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 50s

The patch does not generate ASF License warnings.

411m 12s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestRollingUpgrade

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5856/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5856

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux dffc1d606d5b 4.15.0-212-generic #223-Ubuntu SMP Tue May 23 13:09:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 757c68f18d3b5ff89cf750a1e02116dd86ff07b2

Default Java
Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5856/1/testReport/

Max. process+thread count
2146 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5856/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@Hexiaoqiao Ok, I've committed in the original branch


Unable to use Synopsis Coverity extension on Hosted agents

Is there a way to use the build task on hosted azure pipeline, If so can someone please provide me with the documentation.

Please take a look at:
https://community.synopsys.com/s/article/Azure-Pipelines-Using-Coverity-with-Microsoft-Hosted-Agents


Added replicas for docker compose in README.md

@sabbour,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Java 18.bot-authentication sample does not prompt to show user token

Describe the bug
The Java version of the 18.bot-authentication sample doesn't properly show the prompt after login to show the user their token in the bot framework emulator. It shows the prompt text but not the yes/no buttons as it should.
To Reproduce
Steps to reproduce the behavior:

Run the sample and connect with the bot framework emulator
Login
After a successful login see that the prompt "Would you like to view your token?" shows the text, but doesn't show the yes/no buttons that it should display.

Expected behavior
Yes/No buttons should be provided for the user or the prompt.

Closed by https://github.com/microsoft/BotBuilder-Samples/pull/3369


Refactored tests

This change is

PR is added to the queue for testing as 1. task. (4f82138)

Test 95507328 (4f82138) build started.

Test (Fail) 95507328 build is completed (4f82138)
See https://github.com/ThaliTester/TestResults/tree/955073284f82138_Refactored_tests_evabishchevich/ for the logs

PR is added to the queue for testing as 1. task. (1f892e2)

Test 95507328 (1f892e2) build started.

Test (Fail) 95507328 build is completed (1f892e2)
See https://github.com/ThaliTester/TestResults/tree/955073281f892e2_Refactored_tests_evabishchevich/ for the logs

PR is added to the queue for testing as 2. task. (6569984)

Test 95507328 (6569984) build started.

Test (Fail) 95507328 build is completed (6569984)
See https://github.com/ThaliTester/TestResults/tree/955073286569984_Refactored_tests_evabishchevich/ for the logs

PR is added to the queue for testing as 2. task. (adf6a6a)

Test 95507328 (adf6a6a) build started.

Test (Success) 95507328 build is completed (adf6a6a)
See https://github.com/ThaliTester/TestResults/tree/95507328adf6a6a_Refactored_tests_evabishchevich/ for the logs

Test 95507328adf6a6a(adf6a6a) has failed
See https://github.com/ThaliTester/TestResults/tree/95507328adf6a6a_Refactored_tests_evabishchevich/ for the fail logs


DBZ-7847 Support in query filter mode with 1000+ tables

https://issues.redhat.com/browse/DBZ-7847

/packit test --labels oracle

These failures are apicurio related.  Running the tests locally with apicurio 2.4.3.Final they pass, anything with 2.4.4.Final or later fail.

@Naros Should we override Apicurio version for Oracle test execution then?

I spoke with the Apicurio team, and I think we have a way forward.  Let's discuss this offline.

So only the Oracle fix needs to be backported to 2.6, the apicurio version in 2.6 is still based on 2.4.3.Final which isn't broken.

/packit test labels --oracle

/packit test --labels oracle

/packit test --labels oracle

@Naros Applied, thanks!


Error after "block removed" with Potgres

After successfully getting augur-node synced, after a little while this error occures. And if I try to restart it, it appears again (but with different block number).
new block: 7170949,<PHONE_NUMBER> (Mon Feb 04 2019 00:50:54 GMT+0000 (Coordinated Universal Time))
block removed: 7170949 (0x465dec49b71a65472c07e57c0747d50dd7757ea8fcdfe1a5fb4eac676e5a896f)
Fatal Error, shutting down servers { error: invalid input syntax for integer: "0x6d6b85"
    at Connection.parseE (/home/ubuntu/augur-node/node_modules/pg/lib/connection.js:546:11)
    at Connection.parseMessage (/home/ubuntu/augur-node/node_modules/pg/lib/connection.js:371:19)
    at Socket.<anonymous> (/home/ubuntu/augur-node/node_modules/pg/lib/connection.js:114:22)
    at Socket.emit (events.js:189:13)
    at Socket.EventEmitter.emit (domain.js:441:20)
    at addChunk (_stream_readable.js:284:12)
    at readableAddChunk (_stream_readable.js:265:11)
    at Socket.Readable.push (_stream_readable.js:220:10)
    at TCP.onStreamRead [as onread] (internal/stream_base_commons.js:94:17)
  name: 'error',
  length: 98,
  severity: 'ERROR',
  code: '22P02',
  detail: undefined,
  hint: undefined,
  position: undefined,
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: undefined,
  table: undefined,
  column: undefined,
  dataType: undefined,
  constraint: undefined,
  file: 'numutils.c',
  line: '106',
  routine: 'pg_atoi' }
update "transactionHashes" set "removed" = $1 where "blockNumber" = $2 - invalid input syntax for integer: "0x6d6b85"
error: invalid input syntax for integer: "0x6d6b85"
    at Connection.parseE (/home/ubuntu/augur-node/node_modules/pg/lib/connection.js:546:11)
    at Connection.parseMessage (/home/ubuntu/augur-node/node_modules/pg/lib/connection.js:371:19)
    at Socket.<anonymous> (/home/ubuntu/augur-node/node_modules/pg/lib/connection.js:114:22)
    at Socket.emit (events.js:189:13)
    at Socket.EventEmitter.emit (domain.js:441:20)
    at addChunk (_stream_readable.js:284:12)
    at readableAddChunk (_stream_readable.js:265:11)
    at Socket.Readable.push (_stream_readable.js:220:10)
    at TCP.onStreamRead [as onread] (internal/stream_base_commons.js:94:17)
Stopping Augur Node Server

blockNumber is typed as integer.
When I converted to decimal the number causing the error 0x6d6b85 I got 7170949 which is the number of the block where it failed.
So I guess there is a missing HEX to DEC conversion when "block removed" event happens.

Heya Filip,
Yep, you're correct. Have a fix coming shortly if you can test a branch to make sure it's working alright for you.

@pgebheim great, thanks! 🙂

Fixed in #812 . Thanks @FilipPyrek !


Fix: RandomState.choice reproducibility

Fixes #738
I had to refactor unit test, so that reproducibility of generators is checked every time generating method is called.

Thank you. Fixed _ usage.

LGTM


python311Packages.malduck: 4.4.0 -> 4.4.1

Diff: https://github.com/CERT-Polska/malduck/compare/refs/tags/v4.4.0...v4.4.1
Changelog: https://github.com/CERT-Polska/malduck/releases/tag/v4.4.1
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 310912 run on x86_64-linux 1

6 packages built:

python311Packages.karton-autoit-ripper
python311Packages.karton-autoit-ripper.dist
python311Packages.karton-config-extractor
python311Packages.karton-config-extractor.dist
python311Packages.malduck
python311Packages.malduck.dist


Android generator

Hi, is it possible for you to provide a new version to generate a png sheet with Android emojis? The one in sprite_gen makes a 7 column sheet, and the order of the emojis is different. I see you have a valid sheet in minEmoji/img/ but that one is missing a lot of emojis (they are replaced with their Apple counterpart I think). I tried to adapt your last version (3) but I failed miserably.
Thanks, and I understand if you are too busy to do it.

Hi, I really want to, but I can't find any source for the Android emoji images, if you know any source I'll be more than glad to add them, BTW I have added the iOS 10 / macOS 10.12 new emojis:
https://github.com/rodrigopolo/minEmoji/tree/master/minEmoji_iOS10

I managed to do it. I got the emojis from https://github.com/googlei18n/noto-emoji (an old version because I don't like the current emojis) and the missing flags from my own phone fonts. Then I renamed the images and used your gen3. The result wasn't perfect cause I couldn't resize the source pngs correctly (they were 128x120) but it worked out for my purposes.
Here is the sheet: https://github.com/jgalat0/emojipasta-gen/blob/master/img/android.png. If you want I can upload the folder with all the renamed emojis.

Nice, I'm glad you find the way to handle it.


Sharing the app sends an invalid link

Build: 1.0.0 (10)
Repro Steps:

Launch Soundscape
Open the menu
Select 'Share'
Share to another phone or copy the message in to your notes app
Open the link

OR

Delete and reinstall Soundscape
Kill and relaunch the app repeatedly
On the 5th launch, select "share" on the modal
Share to another phone or copy the message in to your notes app
Open the link

Expected Results:
Ideal: The link will take you to Soundscape on the App Store
If that's no possible because we don't have an App Store link until we launch: The link will take you to the website
Actual Results:
The App Store opens but there's an error saying the app can't be found

It should be possible to determine the future app store URL before launch. Per https://developer.apple.com/forums/thread/127195 , that URL for us should be:
https://apps.apple.com/app/id6449701760

Ah, it looks like @Oliver2213 already fixed that app store ID in commit a7f82010. So I'd expect the current app store link should work after launch.

Excellent, thanks for confirming!


feat(core/windows): Convert UNC paths to simple paths in JS apis.

partially fixes #9407
Not much in here i guess. That's all i found looking at the plugins / searching for various formats of the #[command] macro.

i can't see any events needing adjustments in this repo :)


Introduce forms validation

Would be nice to have some basic inputs validation for our forms.
I had a nice experience using Yup library with Formik we currently use.
One of such helpful validations would be running
ethers.isAddress(form.values.address)
on the inputs expecting evm addresses to be pasted.
Additional things to have:

validate if proper evm address is pasted
validate if proper Hedera AccountId is pasted
required fields
check for proper numbers used, considering min max and decimals per specific token

This is important for dynamic compliance modules forms when deploying TREX token to avoid the situation when we could pass the selected module address, but empty settings for respective module (where applicable). For example choosing requiresNftModule but not adding encoded settings part due to absent form validation not preventing us from submitting the form.


Face-detection on IPC

The face-detection demo is already packaged as a snap (https://github.com/crossbario/iotcookbook/pull/25).
The next step is do make it work it on the IPC327E we now have, publish a proper snap and document it (from a user point of view)

actually, not sure about the status of https://github.com/crossbario/iotcookbook/tree/master/device/edge/face-detect @om26er ? I mean, could I connect a PSEye camera to a say intel NUC and get the face detect? IMO, would be also a very cool follow up demo in the style of https://www.katacoda.com/codelectron/scenarios/crossbar-demo ;) or some docker-compose/readme sugar to make it more approachable .. I could show/post it on mindsphere dev days already ..

I mean, could I connect a PSEye camera to a say intel NUC and get the face detect?

The current demo we have only processes images and not video streams. I will work with my PSEye camera tomorrow to get video facial detection on live videos.
From there we can extend the demo as we like.

@oberstet Well, I experimented with that just now.
Do we want to send the whole video stream from the front-end to the back-end, which will then detect faces on it and send it back to the front-end to draw ? Doing that there is going to be plenty of lag between the whats displayed on screen vs face rectangles.

For the IPC setup, a setup more like this would be best:

PS3 Eye == USB ==> IPC ==> Docker container A (with /dev/videoXX mounted)
inside container A: OpenCV, Python cv2 to do two things: a) JPEG encode into (small batches of) video frames and WAMP-publish those to topic T1 and b) detect faces on raw image data and WAMP-publish detected rects/faces to topic T2
inside container B: run a backend that listens for T2, and only in case a known face is recognized, it publishes to T3 (the backend actual does the face recognition on top of the face detection by matching against a database contained in the backend)
inside container/snap C: run XBR dashboard, with access to video output (wayland), with a Tile1 that receives the raw video frames (jpeg) from topic T1, and a Tile2 that receives the rects/faces from topic T2, and a Tile3 that receives enriched detection events from T3


With @angular/languageservice, non-Angular HTML file report lots of errors as they're parsed as TypeScript

TypeScript Version:  2.8.3 (@angular/languageservice 7.2.9)
Search Terms: Language Service HTML parsing errors
Context: I'm starting the typescript-language-server (delegating to tsserver) adding the @angular/language-service as plugin to typescript.
When opening an HTML that does not have any Angular-specific content
Expected behavior: the typescript-language-server ignores the file, not specific errors are reported
Actual behavior: a lot of errors are reported for such non Angular HTML, the errors are TypeScript parser errors. It seems like tsserver decides to parse this .html file as a .ts one.

TypeScript can't prevent editors or external components from asking it to parse HTML files as if they were TS. Whoever is doing this needs to stop.

See https://github.com/angular/vscode-ng-language-service/issues/317#issuecomment-493149361 . I don't really understand everything here as I'm more on the plain consumer side and don't know any detail about how Typescript-language-server works; but with the linked comment and yours, it seems to me the issue is fully on @angular/language-service, isn't it?
If it is so, feel free to close this ticket.


HOP-3547: add Beta indicator to languages and switch to EN

Following this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [HOP-XXX] - Fixes bug in SessionManager,
where you replace HOP-XXX with the appropriate JIRA issue. Best practice
is to use the JIRA issue title in the pull request title and in the first line of the commit message.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.

Trivial changes like typos do not require a JIRA issue (javadoc, comments...).
In this case, just format the pull request title like (DOC) - Add javadoc in SessionManager.
If this is your first contribution, you have to read the Contribution Guidelines
If your pull request is about ~20 lines of code you don't need to sign an Individual Contributor License Agreement
if you are unsure please ask on the developers list.
To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[ ] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004
[ ] In any other case, please file an Apache Individual Contributor License Agreement.

Thanks for the review @gvdutra ! I'm still waiting if there are some objections on the dev mailing list. If no one objects over the weekend this will be merged


Ctl+Z doesn't work

Thanks for project. It's just cool! Ctl+Z doesn't work on russian keyboard layout. Аfter the hotkeys, the "Step back" button in the lower left corner also stops working.

Hi. I can reproduce the shortcut not working on a russian keyboard layout, but can't reproduce the button (which are also not tied to the keyboard in away) 🤔

The moment with the button is not always reproduced. If I notice it again,
I'll record it on video. What do you recommend with ctrl+z? And then I'm
already thinking, should I create my own website, embed your script and
customize it for myself)
вс, 5 февр. 2023 г., 01:12 David Luzar @.***>:

Hi. I can reproduce the shortcut not working on a russian keyboard layout,
but can't reproduce the button (which are also not tied to the keyboard in
away) 🤔
—
Reply to this email directly, view it on GitHub
https://github.com/excalidraw/excalidraw/issues/6194#issuecomment-1416861319,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5VSG6UQPCE6K7DZP7UBMKDWV3H57ANCNFSM6AAAAAAURJ7MZM
.
You are receiving this because you authored the thread.Message ID:
@.***>

It doesn't work on arabic layout too, I think this probably has to do with using event.key instead of event.code somewhere in the code.

Yes, It doesn't work on non latin layout. Event.code and event.key do not allow to create a universal implementation. A countless number of layouts divorced in the world ))  It is written on the Internet that the event.key is the preferred method. This is exactly what is implemented in Excalidrow. The solution is localization. I'm not a developer. Excalidraw is an open source program. Maybe someone will come up with something. I don't understand yet how to implement localization in the case of the keyboard. Although the localization of the interface is present, as we can see.


Fixed ListPlanAccountsForAccount

Fix for https://github.com/google/go-github/issues/1270

I will defer to @gmlewis but from from glance this looks like it would be better off being a new method.

Thank you, @gauntface. Looking at the GitHub docs, this situation is very confusing indeed.
The docs show a single object being returned, yet imply that paging is supported with the call, and I don't know why.
Before we review and continue with this PR, @domsu - can you please contact<EMAIL_ADDRESS>and ask them why paging is supported in this endpoint if it returns only a single object?
If they decide to remove paging support, then I think the method rename is appropriate along with the removal of the ListOptions. If, however, we learn something else from them, we can respond appropriately.
Thank you, @domsu!

Hello @domsu, do you have time to follow-up @gmlewis request? What is the status of this PR? Thanks.

This PR appears to have been abandoned. Closing.


Preview Image Markdown

The system throws the following error when loading image in markdown file:
An error occurred while loading the image.
Open file using VS Code's standard text/binary editor?
The link for open the image with VS Code's standard doesn't work.

In VS Code release version it works:

Thanks for your help.

Duplicate of #100879


Fixed #33287 -- Replaced eval() with json.loads() in the geojson serializer.

A fix for https://code.djangoproject.com/ticket/33287.
This is a dupe of https://github.com/django/django/pull/15088/, though I filed the Trac ticket.
Some GIS tests seem to be failing with and without this PR. For more details, see https://code.djangoproject.com/ticket/33292.

@dwyde Thanks for PR :+1:  however Maxim assigned the ticket to themself and was the first to submit PR.


Fix BaseScaleform.Dispose not using pointer value in RPH and RageMP

I don't mess with the game using RPH plugins that much and I had never tried playing RageMP, but this was not that hard to catch.
By the way, NativePointer of RPH allocates 128 bytes in the constructor with no parameters. You can confirm by dumping nethelper.dll or rockstarupdater.dll that are created after having RPH loaded and analyze them with ILSpy.

Works fine in RPH! Couldn't test RageMP sadly.


convert some service classes to static props, run [bugzilla buildkite bundlephobia]

Warnings

:warning:
This PR modified service code for bundlephobia but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for buildkite but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for bugzilla but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @calebcartwright!

  Generated by :no_entry_sign: dangerJS against 94a56404d509fe64570e9ca1b5cf9b30bbb1701c

The Bugzilla test is failing as the target instance is apparently having some DB connectivity issues ATM
https://bugs.eclipse.org

API Response:
{
   "code" : 100500,
   "documentation" : "https://bugzilla.readthedocs.org/en/5.0/api/",
   "error" : true,
   "message" : "\nCan't connect to the database.\nError: Can't connect to local MySQL server through socket '/var/run/mysql/mysql.sock' (2 \"No such file or directory\")\n  Is your database installed and up and running?\n  Do you have the correct username and password selected in localconfig?\n\n"
}


Scrolling to the very top causes infinite fetching and content jumps

Describe the bug
When user scrolls to the very top of infinite list, the request to get new messages is called. However, if scroll position is at the top (container.scrollTop equals to zero), then browser will shift scroll to new top, and so new request will be called. And so on...
To Reproduce
codesandbox example
Steps to reproduce the behavior: click on 'add item' and see how contents jump.
This is default HTML behavior which is not suitable for scrolling infinite lists.
Expected behavior
It is expected to retain contents in the visible area and do not scroll to the new top.
Possible solution
Add onScroll event handler on container and adjust container.scrollTop property: set it to Math.max(1, container.scrollTop)
Additional context
There exists a relatively new feature called scroll anchoring, but I haven't found any effect of CSS property overflow-anchor on scroll behavior in such situation.

Tried alternatives

Using scroll snap only for the first element in such a way that it will always scroll a little to the start of first element. Applying those styles:

.first-child {
    margin-top: 5px;
    scroll-snap-align: start;
}

.container {
    scroll-snap-type: y proximity;
}

This approach doesn't work if user is constantly scrolling to the top in such a way that scroll snap does not apply.

Set container.scrollTop property to Math.max(1, container.scrollTop) as described in the description, but do it before setState call. This will probably be a better solution when implementing in MessageList, because in this way user will have a possibility to scroll anywhere. I haven't included this in issue description because in my sandbox example I am currently using onScroll that is fired many times instead of IntersectionObserver.


Documentation Updates

Added HTTP interceptors document
Updated CHANGELOG

Reviewers
@tomblench
@mikerhodes

Could you add a heading to replication.md:
## Customising HTTP requests made during replication

This can be done with HTTP interceptors. These are documented [here](httpinterceptors.md).

done in 0f511a6

+1 after squashing.

+1 once these last tweaks are done


New Gallery UI reprise

Added a workaround for https://github.com/flutter/flutter/issues/17012

LGTM!


Add config.json to .gitignore

config.json should be part of .gitignore, with an example provided in a separate file which can be copied over for easy setup
if we do this, we should also drop dotenv

Maybe allow config through envvars still regardless though...? nobodys gonna do this but if someone wants to containerize


Accumulo 1480 Added namespace filtering to tables

Can now filter namespaces on the table list.
Back end implementation is done, and I would appreciate the feedback on the Front end.
Front end can be seen on the "Master Server" and "Tables" pages.

Mentioning ACCUMULO-1480 so it links to the JIRA issue.

Pushed a commit to fix the checkstyle error I was getting on the build.

Here are screenshots of the Tables Page and the Master Server Page which shows the tables.
Tables Page

Master Server Page

The changes that you made in response to my review look good.

Thanks!

I'm OK if you merge it in now. I'll work on the improvements on top of the changes I made here.


Add a Gitter chat badge to README.md

cloudpipe/multyvac-fork now has a Chat Room on Gitter
@rgbkrk has just created a chat room. You can visit it here: https://gitter.im/cloudpipe/multyvac-fork.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Not sure how I triggered this twice.


Can't connect to internet

Hi, having difficulty getting this setup even after reading through the FAQ section on this. I have my scraper setup as:

This opens up Tor (stays on home page) and chrome which displays the can't connect to internet page
Alternatively using chromedriver (which is in the same folder as YADC.py / main) gives the following however chrome never opens up and nothing happens (tor stays on home page):

Any help would be appreciated as DVSA decided to cancel my test 3hrs before it due today :)

This opens up Tor
That doesn't sound like you're running tor.  It sounds like you're running the Tor browser.  YADC needs a standalone tor it can control.  You can try using scripts/download_tor.py to download it for you, or download it yourself using the url in that file.

The fact that chrome then loads is good.  The fact that it fails is down to no tor proxy being available.

Alternatively
This setup is just incorrect---the path should be to chrome, not chromedriver (YADC starts chrome itself).  However we know this isn't a chromedriver issue, so this is a blind alley.

In general btw copy-paste code, don't take screenshots.  Github has fenced markdown blocks, so you can get syntax highlighting.

DVSA decided to cancel my test 3hrs
Rotten luck on that.  Also goodness, this testing backlog has been going on a long time...

Hi,
Downloaded tor zip from the link in the script, extracted, and pointed to it as follows, is this correct?
TorBrowser(         buster=Path(r"C:\Users\mikey\Downloads\buster"),         chrome=Path(r"C:\Program Files\Google\Chrome\Application\chrome.exe"),         tor=Path(r"C:\Users\mikey\Downloads\tor\Tor\tor.exe"),  # probably not needed on *nix     )
It seems to work but eventually runs into an issue with imperva blocking either before or after queuing. Also is my buster setup correct?
Cheers :)

Imperva used to catch YADC ever so often.  It will recover from the catches.  If it starts catching it too often to be useable, further work might be needed.  I don't really develop it any more, but one step would be to try using undetectable_chromedriver, although this would require intercepting undetectable_chromedriver between its launch of the chrome process and its taking command (we need to pause here to allow an anti-bot js to run, otherwise we get blocked).
So long as you're getting past the queue sometimes everything is good.
I have no idea if your buster path is correct or what's in it, but other than that; yes---you just pass it the path to buster and it will (usually) do the rest.

Sorry i should of specified, the path i specified for buster is just the folder of where i extracted the buster.zip file

That should be fine, then.  You'll know if buster is installed---you'll see it solving captchas (sometimes---it won't always work).


can't copy - source file is being updated

I'm uploading my data to owncloud and I'm always see a lot of errors like this:
2023-09-06 15:14:49 ERROR : 03/de007f21d06c612db486b6a48978ac51d7a1e3a5fd830b0d8281466cc06e43: Failed to copy: Put "https://myowncloud/my_path/training_data/03/de007f21d06c612db486b6a48978ac51d7a1e3a5fd830b0d8281466cc06e43": can't copy - source file is being updated (mod time changed from 2023-09-06 11:07:55.891044 +0200 CEST to 2023-09-06 15:14:48.188031 +0200 CEST)

Typically at the second retry it works. Anyway, according google the rclone flag --local-no-check-updated  should be the solution for that. But how can I set it in dud?

I agree that adding the ability to configure the rclone command is a good feature, but it doesn't exist in Dud today. I'll add this as a feature request. Thanks!
As for the "can't copy" issue, I've never seen this before. It looks a lot like an issue with OwnCloud, or at least your specific environment. Do you have multiple users/clients editing the same Dud project concurrently?
Without a reproducible example, I can't be of much help. If you get further in your investigation, please comment here and I'm happy to advise. Certainly if you have a reproducible example, I'll reopen this issue.


Moved strings from codechecks to bundle.properties

Moved strings from codechecks to a separate bundle as a part of preparation for a larger refactor of codechecks package.

Looks good to me, I assume you somehow checked that everything works (i.e. you don't have a typo in the key preventing it from loading?)

I tested updateHint and tempVarsHint. ReadOnlyPrimitives is bugged right now so can't test it. And I don't know on what should internalHint trigger.

Okay, we will see in the future then.
Thanks for your Contribution! :)


Do not fail if a module is not present on the forge

Use Case
We have some "low value modules" that we do not think are worth storing on the Puppet Forge.  When uploading a Puppetfile with such a module, the import fail because the module is not found and no modules are imported at all.
Describe the Solution You Would Like
Instead of failing, the import should just produce a warning about the unknown module but proceed with all known ones, so that known modules can be tracked.
Describe Alternatives You've Considered
Removing the offending modules should help, but who has time for that :smile:
Additional Context
Screenshot of the error preventing the import:

Thank you, @smortex , We will review internally and priorities the same.

HI @smortex, wanted to let you know that we have implemented the enhancements, and they have been rolled out today. Please try them out and share your experience.
Thanks again for raising the issue and helping us improve Forge.

Awesome!  Works as expected, thanks!