ElasticSearch search queries returning empty list.

ISSUE:
root@cassy1:/home/lakshmi# curl -X GET http://localhost:20000/v2.0/tenant-id/metrics/search?query="*"
[]root@cassy1:/home/lakshmi#

Hit elastic search directly:
root@cassy1:/home/lakshmi# curl -X GET "http://cassy1:9200/metric_metadata/_search?pretty=true&q=tenantId:st2analytics-tester&size=121"
{
  "took" : 19,
  "timed_out" : false,
  "_shards" : {
    "total" : 5,
    "successful" : 5,
    "failed" : 0
  },
  "hits" : {
    "total" : 135,
</snap>

DEBUG INFO:
We are using Blueflood master (jar with all dependencies).

OS - Ubuntu 14.04

root@cassy1:/home/lakshmi# cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=14.04
DISTRIB_CODENAME=trusty
DISTRIB_DESCRIPTION="Ubuntu 14.04.1 LTS"
root@cassy1:/home/lakshmi#

Java 8 runtime (jar probably compiled with java 7)

root@cassy1:/home/lakshmi# java -version
java version "1.8.0_45"
Java(TM) SE Runtime Environment (build 1.8.0_45-b14)
Java HotSpot(TM) 64-Bit Server VM (build 25.45-b02, mixed mode)
root@cassy1:/home/lakshmi#

Relevant elastic search config:
root@cassy1:/home/lakshmi# grep "ELASTICSEARCH" /opt/blueflood/blueflood.conf
ELASTICSEARCH_HOSTS=localhost:9300
ELASTICSEARCH_CLUSTERNAME=elasticsearch
root@cassy1:/home/lakshmi#

OTHER INFO
I browsed the code and it looks like the search is performed on this EVENT_INDEX https://github.com/rackerlabs/blueflood/blob/master/blueflood-elasticsearch/src/main/java/com/rackspacecloud/blueflood/io/EventElasticSearchIO.java#L82
but when metrics are indexed, the index used is metrics_metadata
https://github.com/rackerlabs/blueflood/blob/master/blueflood-elasticsearch/src/main/java/com/rackspacecloud/blueflood/service/ElasticIOConfig.java#L22
Looking at elastic search cluster, I just have the metrics_metadata index.
root@cassy1:/home/lakshmi# curl 'localhost:9200/_cat/indices?v'health status index           pri rep docs.count docs.deleted store.size pri.store.size
yellow open   metric_metadata   5   1        135            0     45.3kb         45.3kb
root@cassy1:/home/lakshmi#

This looks like a bug in configuration settings. It looks like the search should use the index in configuration. Am I missing something or is the fix what I just said? I can make a PR. Just need pointers. Thanks for looking!

@lakshmi-kannan the events index is introduced newly for storing annotations. If you use the same ElasticIO module for Discovery Modules as follows, things should work as before.
DISCOVERY_MODULES=com.rackspacecloud.blueflood.io.ElasticIO

@tilogaat @lakshmi-kannan I'm noticing the same thing as described. I don't have anything custom as I was just running through the 10minuteguide and came across this. Upon inspecting /demo/local/config/blueflood.conf the DISCOVERY_MODULES line is what is listed in the above comment. I can still re-create the issue.
I have done my testing through Vagrant/VirtualBox. You will find the linked gist with the Vagrantfile and related information needed to recreate the issue if needed. I apologize as I don't have the bandwidth to dig into the issue myself and am just hoping this will help with issue recreation and resolution.
https://gist.github.com/mattalline/40d9223daa757cdbbb44

Hey there @mattalline, @lakshmi-kannan.  Still having troubles?
If so, can you try the most recent release rather than running from head on master?  I'm running the same configuration as you, @lakshmi-kannan, except that I'm grabbing the latest release with:
curl -s -L https://github.com/rackerlabs/blueflood/releases/latest | egrep -o 'rackerlabs/blueflood/releases/download/rax-release-.*/blueflood-all-.*-jar-with-dependencies.jar' |  xargs -I % curl -C - -L https://github.com/% --create-dirs -o blueflood-all/target/blueflood-all-2.0.0-SNAPSHOT-jar-with-dependencies.jar

It seems to work fine for me.  I'll try running the same thing from head when I get back later today and see if I run into the same issues you are.
@mattalline, thanks for that Vagrant gist.  I was running through the same exercise myself and it's nice to compare with yours.  I hope to share some Packer builds before the weekend is done.


Is there a way to include a datepicker in a question type alert?

https://github.com/limonte/sweetalert2/issues?q=is%3Aissue+datepicker+is%3Aclosed


Alex Mobs compatibility

Describe your idea, include images if necessary
Flying and water creatures should work.

flying creatures implemented as of 905bf16da878678701a1250f829a49aa8d20ca29. The water creatures already work partially, I haven't found a fresh solution for dolphins and similar mobs.


Entry edit exception from v1.5.8

Hi, I'm using your package to scan QR-codes, it works fine, but I met one issue.
When I'm changing float value bound to Entry I'm getting following exception: Java.Lang.IllegalArgumentException: 'end should be < than charSequence length'
I'met this Issue on version 1.5.8 and 1.5.9.
Version 1.5.7 does not contain this issue.
I guess this issue related to some Xamarin dependencies.

I tried build and run project with dependencies from 1.5.7. It works without described issue.

I've open an issue in MAUI repo

It fixes by putting following code in protected override MauiApp CreateMauiApp() in Platforms\Android\MainApplication.cs
EntryHandler.Mapper.AppendToMapping("DisableEmojiCompat", (handler, view) =>
{
  handler.PlatformView.EmojiCompatEnabled = false;
});


Modifying an animation's current frame or progress

Hey, I'll drop this as a question here because I'm having a limitation with this crate. I'd find it very useful for it to be possible to change an animation's current frame, and to be able to easily know the start and end frames, or the length of the animation so that the start frame is always zero.
This would be great for e.g. air animations (frame depends on velocity) and bow animations (frame depends on load%). I understand that I could do this some other way with maybe slices but it would be great to be able to still use the same component because my player uses the AsepriteAnimationBundle for all of its other animations, and using something else for the air animation would just be a pain in the butt.
I already forked this crate, thought it'd be an easy fix to just expose the AnimationState struct, but it wasn't that simple. How I'd implement this is with an Event, which one can send to change an animation's current frame (usize) or an animation's progress (from 0.0 to 1.0, the safer method)
Also if there is some way to do this please correct me, maybe I can't just find the right solution.
Thanks!

The most straight forward way would be to make the values of the AnimationState component public. On the Animation Component set playing to false and manipulate the current_frame of the state yourself.
You can access the start and endframe of animation from the aseprite asset. It holds a hashmap of all the meta data of a tag. including start and endframe. Take a look at animation.rs line 228-240

Good news, I did not forget.
in the bevy 0.15 release there is now a ManualTick Componet. Adding it will prevent the animation system to take over. The Animation State is now public. Animation state logic and render are now seperated.
You can access the asset, read out the start and endframe of any animation tag and change the state according to your logic.
On frame change you must trigger the FrameChangedEvent on your sprite (observer) by hand. Else the renderer will not update the current frame.

I am closing this for now, if your requirements are not met, let me know.


Document still uses the term 'ACS-Engine'

In this document, there is a paragraph about what ACS-Engine is. The document should be updated to use the term AKS-Engine, which is the new name.

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

ID: 52f172d9-2ca9-da01-1fcb-9f6bbdf383ef
Version Independent ID: 64485d09-0058-a20f-e2be-387e1fa10121
Content: Azure Kubernetes network policies
Content Source: articles/virtual-network/kubernetes-network-policies.md
Service: virtual-network
GitHub Login: @aanandr
Microsoft Alias: aanandr

@david-jarman , Thanks for your feedback. We are looking into this query and will update you as soon as possible.

@david-jarman , Appreciate you patience. The doc has been edited accordingly and it would take some time for the changes to get reflected over the doc.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Model with Generics

I am working on a crate called async-graphql-relay and have recently redone the public API to make it less cumbersome to use (you can checkout the example folder in the repo). When doing this I introduced a type RelayNodeID which is generic on the Model so it is declared as RelayNodeID<Self>. This is done so that among other things all ID's returned from GraphQL are globally unique and contain information about the specific type they are. This type is a wrapper around a standard Uuidv4 and from a database perspective should be treated this way.
After chatting with @billy1624 he pointed me towards this resource which shows how to implement sea-orm support for custom types. The issue I found with this is that my type is generic and this does not currently seem to be supported by sea-orm. When I run my code the DeriveEntityModel macro panics (refer to the code linked below).
I put together some code here which should help show the sort of syntax I am going for without all the complexity and macros of the async-graphql-relay codebase.
Thanks, @billy1624 for creating PR #400 which is going to be the fix for this.

❄🎄

With #400 you could define the following:
#[derive(Clone, Debug, PartialEq, DeriveEntityModel)]
#[sea_orm(table_name = "model")]
pub struct Model {
    #[sea_orm(primary_key)]
    pub id: AccountId<String>,
    pub name: String,
}

But not...
#[derive(Clone, Debug, PartialEq, DeriveEntityModel)]
#[sea_orm(table_name = "model")]
pub struct Model<T> {
    #[sea_orm(primary_key)]
    pub id: AccountId<T>,
    pub name: String,
}

So, I'm think should we also support the use of generics over Model? @tyt2y3

PR #400 works perfectly for my usecase. Thanks, @billy1624.
Something to note was that although my original idea was RelayNodeID<Self> this causes a lot of issues because the ID field is copied to some SeaORM internal stuff (Eg. ActiveModel, etc) and those types do not satisfy the expected trait for my code. I fixed this easily by changing it to RelayNodeID<Model>. This could be worth noting in documentation although this is probably something a Rust developer would be able to work out.


Modify Summary API for fileTypeSummary

Given the set of below it is difficult to specify the type in the json for the file type summary.
{
  "fileCount": 14054,
  "specimenCount": 183,
  "projectCount": 4,
  "totalFileSize":<PHONE_NUMBER>650,
  "organCount": 6,
  "fileTypeSummary": {
    "fastq.gz": {
      "count": 14054,
      "totalSize":<PHONE_NUMBER>650
    }
  }
}

Is it possible to modify the fileTypeSummary to be more like:
{
  "fileCount": 14054,
  "specimenCount": 183,
  "projectCount": 4,
  "totalFileSize":<PHONE_NUMBER>650,
  "organCount": 6,
  "fileTypeSummaries": [
        {
        "fileType": "fastq.gz"
        "count": 14054,
        "totalSize":<PHONE_NUMBER>650
       },
       {
       "fileType": "bam"
       "count": 14054,
       "totalSize": 1720374
       }
   ]
}

@NoopDog requested the current format - is this no longer good? Didn't we already implement file types and sizes?

We made some hacks we can delete if the proposed structure is adopted.
I think it’s easy in the server side to change?
Thx !

Dave Rogers
Partner, Clever Canary
646-286-5371
http://clevercanary.com
@CleverCanary
@_DaveRogers
https://www.linkedin.com/pub/dave-rogers/0/90/92
On Jun 30, 2018, at 3:37 AM, Trevor Heathorn<EMAIL_ADDRESS>wrote:
@NoopDog https://github.com/NoopDog requested the current format - is
this no longer good? Didn't we already implement file types and sizes?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/HumanCellAtlas/data-browser/issues/150#issuecomment-401423531,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADFvEV8XXMBx_Iv3GiMA3MNE8oZACW5lks5uBmW8gaJpZM4U6u3Y
.


[joss review] feedback

Hello w2w team, I am one of the reviewers for your JOSS submission tracked here. I will leave my feedback in this issue. First of all, nice work! I had no problem installing the package or running the sample.
Software

[x] I wanted to test the other sample geo_em.d0X.nc files, but i was lazy and tried using the whole of Europe LCZ geotiff as input to w2w. This churned for awhile before failing and reporting a memory leak (see below). Not saying you should support this workflow, just that you might want to more explicitly warn about using a reasonably targeted LCZ raster.

> Processing MH_URB2D ...
[1]    59619 killed     w2w ./sample_data EU_LCZ_map.tif geo_em.d03.nc
/Users/user/.pyenv/versions/3.9.5/lib/python3.9/multiprocessing/resource_tracker.py:216: UserWarning: resource_tracker: There appear to be 1 leaked semaphore objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '

[x] Just as an FYI, I tried to reproduce the error from #63, but this worked correctly on my system
[x] There is a note that is output at the end of running w2w (see below). Is the user supposed to act on this? If not, may be best to suppress this or log it at a debug level instead.

 ----------- !! NOTE !! ---------
 Set nbui_max to 9 during compilation, in order to optimize memory storage.

Paper

[x] It took me a minute to realize some of the parenthetical content represented arguments to w2w, i.e. "(-f)", "(-l = 1)" and "(-b)" -- i like that this information is in the paper, but it could be nice to more explicitly call out the meaning and/or use the long form expression i.e. "(corresponding to parameter --lcz-band = 1)"
[x] In the Workflow section, the indentation of each step is a bit awkward -- i recommend to use a small header tag or bold text instead

I'll give the paper another more in-depth read next week, and may have some more feedback at that point!

Hi @thurber,
We really appreciated your first round of feedback on our work.
I just created a new branch, and pushed the recommended changes in there.
I'll leave this open for now, in case you have more suggestions after your second read.
Thanks again!

I'll leave the compiled reviewed .pdf here: j1iodiq3tayelfus2tlc.pdf

Thanks @matthiasdemuzere! A few more comments and then I'm ready to approve:
Software

[ ] Can you add a brief note to the README about how to run the automated tests? This can be as simple as "To Launch the automated tests, install pytest with pip install pytest and then run pytest.
[ ] I'm getting a couple deprecation warnings when running the tests; it'd be a good idea to address those now to avoid future conflict

Paper

[ ] I think opening the paper with a statement about WUDAPT gives a false impression that the new contribution is WUDAPT. Working in a sentence or two about w2w prior to the current opening would alleviate this.
[ ] "more simple" --> "simpler"
[ ] "netcdf" --> "NetCDF"

@theendlessriver13 that's fair, i didn't catch that these were generated from the dependencies.

Thanks @thurber  for the second round of comments.
I addressed the three paper items, and adjusted them according to your suggestions.

thanks, you've got my green light!


Are we going to have libm?

Do we need to consider libm?
Julia Community has a open source separated libm, we can also use that for no-std version.

Sure.

Sure.

Yeah!


Fix image build

Description

fix image build issue
fix test scripts path

Issues
List the issue or RFC link this PR is working on. If there is no such link, please mark it as n/a.
Type of change
List the type of change like below. Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)
[ ] Others (enhancement, documentation, validation, etc.)

Dependencies
List the newly introduced 3rd party dependency if exists.
Tests
Describe the tests that you ran to verify your changes.

Vectorstore/pathway: CI test failed https://github.com/opea-project/GenAIComps/issues/741 known issue.

Hey @chensuyue , please see my reply here, this is due to the volume mount path. You can either fix the path or mount the test file instead (if you feel like it has less chance to be moved in the future).

https://github.com/opea-project/GenAIComps/actions/runs/11065727650/job/30745644663?pr=737 seems caused by the invalid link in readme, please fix in another PR. @berkecanrizai


Modal tab

Geo pop up highlight for the CTA is not consistent
Resolves: MWPW-130730
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/suhjain/toc
After: https://modal-tab--milo--suhjainadobe.hlx.page/drafts/suhjain/toc

Thanks for fixing this @suhjainadobe . I remember this css implementation being discussed to be implemented across modals in the jira ticket. Do we plan to implement it for all modals?
cc @vhargrave @rgclayton

I'm confused by the Jira ticket or what I'm testing.
The title says "Modal Tab CSS issue (highlight color on tab) on Windows" but not mention of Windows or what browser in the description.
In the title it says "Modal Tab". In the description it references the CTA. Is it both?
Is this only a problem on Windows using Chrome?
On the example provided in the Jira ticket it says the button is yellow on Mobile. I can not reproduce that.
This is what I see:


specified bridge(s) does not match with the manifest

Greetings!
So, I haven't used albatross and unikernels for a long time - from reading the various mirage tutorials, I have made a bridge in ubuntu, pulled down static_website_tls from mirage-skeleton and built it (the whole flow of building cross-compiled unikernels has become a dream - qudos to all), built the debian version of albatross.deb and installed it. Swimmingly good so far.
When I tried a sudo albatross-client-local create --net=br0 --arg='--ipv4=<IP_ADDRESS>/24' my-hello-unikernel https.spt I got albatross-client-local: specified bridge(s) does not match with the manifest.
I can see that I could add something to the manifest.json in the unikernel/mirage dir, but I couldn't find (and didn't know where to look) for any pointers on how to tweek this.
There might be an issue with my netplan in ubuntu 22.04 - my host interface is enp0s1 with an ip of <IP_ADDRESS> and mac of cb:5c:8f:86:b0:d2.
  version: 2
  renderer: networkd
  ethernets:
    enp0s1:
      dhcp4: no
      dhcp6: no
      addresses:
        - <IP_ADDRESS>/24
      routes:
        - to : default
          via: <IP_ADDRESS>
      nameservers:
        addresses:
          - <IP_ADDRESS>
          - <IP_ADDRESS>
  bridges:
    br0:
      interfaces: [enp0s1]
      addresses: [<IP_ADDRESS>/24]
      routes:
        - to : default
          via: <IP_ADDRESS>
      nameservers:
        addresses: [ <IP_ADDRESS>, <IP_ADDRESS> ]
      macaddress: cb:5c:8f:86:b0:d2
      parameters:
        stp: true
        forward-delay: 4
      dhcp4: no
      dhcp6: no

However, sudo brctl show gives
bridge name	bridge id		STP enabled	interfaces
br0		8000.9c5c8e86b0da	yes		enp0s1

which suggests that there isn't an issue.
So, I'm a bit stuck on what to do next - any suggestions?

Hi @nickbetteridge, thanks for your report. So there's an easy solution, but it requires some more documentation (sorry for the lack thereof).
At some point (when introducing support for multiple network interfaces), solo5 introduced names for network devices -- i.e. --net:service=tapX and --net:management=tapY.
Now, albatross is keen on setting up tap devices, and indeed supports a mapping between host system bridge names and solo5 network device names. The --net parameter is either a single name (when bridge name and solo5 interface name are the same) or of the form name:bridge to map the solo5 name name to the bridge bridge. In your example, passing --net=service:br0 should do the trick.

Hi @hannesm - thanks for quick reply, and that did the trick! Just spent the last two days grinding through linux networking and getting up to speed - great fun (?!) - everything working. I'll make a PR for albatross, netplan and ubuntu 22.04. Thanks again.


Doc/Attribute range mapping is broken with inner file docs + outer impl docs

Pasting the following snippet in a file causes the highlighting to break
//! This is a module to test doc injection.
//! ```
//! fn test() {}
//! ```

/// This is an impl with a code block.
///
/// ```
/// fn foo() {
///
/// }
/// ```
impl () {}

This only seems to happen for when the annotated item is an impl, another requirement is that number of lines of doc comments on the impl have to be at least 3 more than the inner doc comments. This also breaks hovering on intra doc links so this seems to be a problem with the source mapping?
This impl stuff seems so oddly specific ...

Looks like we dont attach comments to impls?


Additional Features & Options

read license identifier from licenses string
validate license identifier to be a valid spdx expression
add options to replenish default license texts from spdx github repository
additional settings/options validation
prefix errors and warnings with the name of the plugin

Codecov Report

Merging #35 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #35   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          15     16    +1
  Lines         236    277   +41
  Branches       41     53   +12
=====================================
+ Hits          236    277   +41

Impacted Files
Coverage Δ

src/defaultOptions.ts
100% <ø> (ø)
:arrow_up:

src/WebpackLicensePlugin.ts
100% <100%> (ø)
:arrow_up:

src/ModuleDirectoryLocator.ts
100% <100%> (ø)
:arrow_up:

src/LicenseIdentifier.ts
100% <100%> (ø)
:arrow_up:

src/WebpackAlertAggregator.ts
100% <100%> (ø)
:arrow_up:

src/LicenseTextReader.ts
100% <100%> (ø)
:arrow_up:

src/LicenseFileWriter.ts
100% <100%> (ø)
:arrow_up:

src/LicenseMetaAggregator.ts
100% <100%> (ø)
:arrow_up:

src/DefaultLicenseTextProvider.ts
100% <100%> (ø)

src/OptionsProvider.ts
100% <100%> (ø)
:arrow_up:

... and 1 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 c9f7802...02254a7. Read the comment docs.


Drop peers when outbound message queue is full

Motivation
Closely related to #2435
Partially addresses #1868, #2386 (and maybe #2405)
Changes
When the outbound message queue for a given connection is full, rather than blocking on Send, begin dropping messages and printing errors (rather than allowing the blocking to propagate up the gossip stack)
It would be good to go one step further and drop peers that are hanging, but this is a quick, simple fix for the acute issue we're seeing on the testnets. See #2385.
Test Plan
Includes a new unit test for both conn and msgconn types
TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[ ] Update documentation as needed

DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

This queue is not only for broadcast, but also for direct requests (msg server and fetcher), is it the case? If so we can't simply drop messages, we must disconnect peer and return with error.

This queue is not only for broadcast, but also for direct requests (msg server and fetcher), is it the case? If so we can't simply drop messages, we must disconnect peer and return with error.

This is correct. I updated this to drop peers if the outbound queue is full. Could you please take a look and see if this is what you had in mind?

bors try

Thanks. Not sure why the apptests failed, I can't recreate the TestShutdown failure locally. It looks like it's this goroutine but I don't see anything wrong with the shutdown logic:
        goroutine 313099 [select]:
        github.com/spacemeshos/go-spacemesh/activation.(*Builder).waitOrStop(0xc01a486000, 0x132b6a0, 0xc02159a300, 0xc01a47cb40, 0xc01a47bee0, 0xc02159bb80)
        	/home/runner/work/go-spacemesh/go-spacemesh/activation/activation.go:283 +0xbd
        github.com/spacemeshos/go-spacemesh/activation.(*Builder).loop(0xc01a486000, 0x132b6a0, 0xc02159a300)
        	/home/runner/work/go-spacemesh/go-spacemesh/activation/activation.go:309 +0x1ff
        created by github.com/spacemeshos/go-spacemesh/activation.(*Builder).StartSmeshing.func1
        	/home/runner/work/go-spacemesh/go-spacemesh/activation/activation.go:246 +0x15f

bors try

i am wondering how it can fail. are we ever sending bursts of data over short period of time,? if we do it will naturally take some time and queue may fill up with smaller messages.

Yes, definitely. This pattern is pretty common due to gossip: every node does something, such as sending a hare message or creating blocks, then gossips them at the same time, so there's a massive explosion of data across the network in a brief period. If the node has a stable connection to its peers then the queue should not fill up since it should be able to send messages pretty quickly, but we do need to parameterize this so that the queue channel is large enough to handle these bursts.

bors merge

bors merge

bors merge

bors merge-

bors merge

bors merge

this change clearly makes p2p systest fail more often

It's probably the debug logs, will try to comment those out, I've seen this issue before

bors merge

bors merge
On Sun, Aug 29, 2021, 11:50 bors[bot] @.***> wrote:

Build failed:

ci-stage2
https://github.com/spacemeshos/go-spacemesh/runs/3455403702

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/spacemeshos/go-spacemesh/pull/2450#issuecomment-907815055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZJWNFKYEZCXXA6537NYDTT7JJLVANCNFSM46O5KTTA
.

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

This is done and ready to be merged, but stuck on flaky tests. Assigning to @antonlerner to make sure it gets merged. Let me know if I've missed anything or if there are any questions. Thanks!

bors try

bors merge


CORE-12000 - Consolidate commands from mgm and network plugin

This task is about consolidating commands available in both into a single plugin. Proposal is to retain the name network, which will be a better fit for hosting capabilities related to application networks (not necessarily only about MGM).

This PR shows only added files - should the old files be deleted also?

I also don't think this does what we need in principle. The new plugin seems to contain both mgm and network commands. The intention was to have everything moved under network. We would only need to maintain mgm groupPolicy that's publicly documented to preserve backwards compatibility for existing clients, which could practically be just an alias to network static group-policy.

I added a couple of comments.
Could you also include some results from your testing in the PR description? Perhaps some screenshots if you tested manually


Updated the readme

Besides some minor text cleanup, update the notes with respect to the change in base image for the Dockerfiles made in c0125f982703dfd307513d5ef8b638c440226622.

Thanks!


Fix Mono versions not being fetched due to strange suffix

Fixes #48
Godot Mono releases for Linux since v4.0 use a different suffix which was not on the suffixes list. This PR adds this suffix to the list.

Additionally, the suffix for the extracted executable is also different in v4.x, so I've added another check to account for it, allowing the version manager to launch these executables.

Merged. Thanks a lot!


Okta Backend: enhancement: Associated users to Groups

Right now, this plugin has consumed all users and groups for my organization; however, the membership to those groups is not being ingested at this point, based on the default configuration of the README for the plugin.  It would be good to be able to associate this so as to leverage the built in backstage permissions model and RBAC that some others have made available.

so we do this in a maybe backwards way. in our okta, users have fields in their profile like department: "platform", team: "developer-experience" which we use to populate UserEntity.spec.memberOf in our userTransformer. in our groupTransformer we just set spec.children: [].
i don't know if these fields are the default for an okta organization or if it's something custom to our org.
anyway, curious if you're asking for some new support in the provider to do this or just an example in the readme?

@zhammer  I think an example in the README would be super helpful.  In our org, members are directly added to groups with team or department not coming into the mix.
I'm almost thinking we'd need to use either something like:
user.listGroups()  to propagate UserEntity.spec.memberOf
or...
group.listUsers()to propagate spec.children: []
to propagate this correctly, but I don't believe that functionality is implemented now?

It's all good.  Nature of the OSS community.  The user does come from Okta; however, it is not coming with any groups in the profile response.


Fix outer wheel greater than inner

Proposed changes

Fixed outer wheel greater than inner when reserved greater than used.
Change styles according to https://github.com/rancher/rancher/issues/18553#issuecomment-469404081.
I will change the circle to the triangle in the future.  @vincent99

When reserved greater than used:

Normal situation:

Types of changes

Bugfix (non-breaking change which fixes an issue)

Linked Issues
https://github.com/rancher/rancher/issues/18553

Further comments

This isn't the fix I was taking about, you're just capping it at 100%.  Both wheels should be based on the same scale, from 0 to max(2,4) = 4 for the CPU example.
Then the inner wheel gray should end at the top of the wheel (2 is half of the max of 4) because the rest is not possible (yet), and the green fill 54% of that gray area (which is the same as 1.1/4).


Added Vagrant support and Misc fixes

Added Vagrant support
Removed reliance on supervisor
Set Docker to use Ubuntu 14.04.4 instead of latest
Updated requirements.txt
Rebranded from Swagger to StackFocus
Updated boostrap and jquery
Fixed mobile view
Changed HOST_NAME variable to ip.stackfocus.org
Removed unnecessary -URI from PowerShell command
New screenshots

@thatarchguy, let me fix the build.

@thatarchguy I'm done now with my commits

@thatarchguy, ready for review.

Excellent job. Thank you for understanding my concerns about the Docker stuff

:+1:


in #325 oetiker removed the trailing / in loader-browser.tmpl.js to g…

…et rid of //. No it's necesarry to append / to resource.

Maybe instead of removing the slash, it would be better to add a .replace(/\/\//,"/") where necessary.

merging, thank you @hkollmann !

note that I did NOT mess with the resources path:
https://github.com/qooxdoo/qooxdoo-compiler/pull/325/files?utf8=✓&diff=unified&w=1#diff-48adb84e3faed9722c679affa66153fdL59
I think there are probably more problems ahead ... I did some experiments, but found that I ended up with double slashes in resources paths!

@hkollmann do you have an example of why you needed to add the trailing '/' in?  I've just tried this and it looks that resourceUri does not need to have a trailing "/" because qx.util.ResourceManager.toUri always adds one

Yes, i have. Just an example with
"externalResources": {
"script": [
"js/zip.js"
]
}
test.zip
Running gives:  Cannot load script test/../resourcejs/zip.js

See also my example in the chat.


bump cardano-addresses to include fix for script hash

Issue Number

N/A
Overview

[ ] I have bumped cardano-addresses to include fix for script hash. Note that this doesn't use master because the module re-oganisation on master broke the nix setup :/

Comments

see also: https://github.com/input-output-hk/cardano-addresses/pull/50

bors r+

bors retry


Add g3 internal run job for ios c-core test

Follow up to https://github.com/grpc/grpc/pull/28029 for adding internal kokoro run job

no-op, closing


Code lens shows broken signature

Real signature is

let inline notNull name arg =
  match arg with
  | null -> raise (ArgumentNullException name)
  | _ -> arg

Fixed in 2.6.4


Client IDs are vulnerable to spoofing

When a client connects to the server, it uses a client-generated ID to tell the server and other players its position/name/etc data. Since the ID is sent to all other users upon the joining handshake, it's publicly available to everyone else on the server.
It would not be hard for someone to modify their page JS to spoof someone else's ID and cause problems. This will also automatically kick the person currently using the ID, since collisions aren't tolerated by the server.

Tthe easiest solution I can think of is keeping the client-generated ID private with the server, and letting the server generate a public ID for the client, which is used to transmit information to other clients.
Ideally, the public ID would be random or derived from the client's private ID, and the pair stored on the server for the session.
Unless anyone has a better solution that doesn't require giving the server more work?


fix: replace deprecated url.parse() with new URL()

Checklist

[x] I have ensured my pull request is not behind the main or master branch of the original repository.
[x] I have rebased all commits where necessary so that reviewing this pull request can be done without having to merge it first.
[x] I have written a commit message that passes commitlint linting.
[x] I have ensured that my code changes pass linting tests.
[x] I have ensured that my code changes pass unit tests.
[x] I have described my pull request and the reasons for code changes along with context if necessary.

Replaces the other occurrences of url.parse() with new URL(). Continuation of PR #1802.

This is possibly a breaking change. Previously, a value like "<IP_ADDRESS>" was accepted, while it now errors with "<IP_ADDRESS> is not a valid URL"

@alumni I think this change broke IPv6 queries. See https://github.com/ladjs/superagent/pull/1805 . The change makes node resolve [x]:port as a hostname instead of IPv6 address.

There's a pr open to fix it ;)

@alumni ah, I was hoping you would review that PR  :-D (also, there was an edit in that PR saying it's hard to test)


HTTPRequest without body: Content-Length shall not be send

This pr ensures that we don't send the HTTPHeader Content-Length if we don't have a body.
The RFC states:

The presence of a message-body in a request is signaled by the inclusion of a Content-Length or Transfer-Encoding header field in the request's message-headers.
https://tools.ietf.org/html/rfc2616#section-4.3

@swift-server-bot add to whitelist

close in favor of #191


Plot work values

[ ] Per-step work values with confidence bands
[ ] Work distributions?

We have code to do this in fah_analysis now


Write test for the files added in this PR

https://github.com/Real-Dev-Squad/website-status/pull/248

I would like to pick this issue if it is not assigned to anyone. @vvaibhavdesai  could you please give me a walk through of the PR mentioned as the description is missing from the PR #248

The PR closed this issue https://github.com/Real-Dev-Squad/website-status/issues/234
It has details but please let me know if the information there isn't subsequent enough or if you have any specific questions


Add Validation Checks for Extension Dependencies

Many extensions require that other extensions must be enabled.  Add validation checks to enforce these requirements.

This is in-work.


Fix bug pointed out by the OQS team

@patricklonga, is this ok to merge? I'll integrate into OQS afterwards.


support for typescript

Hi, I'm using your great library.
I develop using typescript, so would be great to have a typescript definition for such library
I've created a proposal here and I'd to know if it is possible pushing it to npm following the procedure described here.
Thanks in Advance

Hi, can you open a pull request for the modification requested in this topic?

Should be addressed by #7


Updated broadband data page and information

Pull request description

I'll commit this after you make the strong changes...mmmmkay? Thanks!

Okay @steveoh. I believe all bolds are now strong.


[REST] Remove unused parameters in SessionOpenRequest

Why are the changes needed?

We should put the authorization info in request header, user,password,ip in SessionOpenRequest are unnecessary.
How was this patch tested?

[X] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Are you touching a released publish API? cc @turboFei

Are you touching a released public API?

I am ok with this, we can release it in 1.7.0.

Codecov Report

Merging #3739 (b320069) into master (f21ad83) will decrease coverage by 0.01%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master    #3739      +/-   ##
============================================
- Coverage     52.57%   52.55%   -0.02%
  Complexity       13       13
============================================
  Files           492      492
  Lines         27705    27690      -15
  Branches       3830     3827       -3
============================================
- Hits          14565    14552      -13
+ Misses        11753    11752       -1
+ Partials       1387     1386       -1

Impacted Files
Coverage Δ

...e/kyuubi/client/api/v1/dto/SessionOpenRequest.java
55.00% <50.00%> (-10.72%)
:arrow_down:

...apache/kyuubi/server/api/v1/SessionsResource.scala
71.42% <100.00%> (ø)

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
80.74% <0.00%> (-0.63%)
:arrow_down:

...he/kyuubi/ha/client/etcd/EtcdDiscoveryClient.scala
67.64% <0.00%> (-0.59%)
:arrow_down:

...g/apache/kyuubi/operation/BatchJobSubmission.scala
74.64% <0.00%> (ø)

.../org/apache/kyuubi/session/KyuubiSessionImpl.scala
76.10% <0.00%> (+0.88%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

If it's an unreleased API, the change makes sense to me.

@pan3793 @turboFei This API has released in 1.5.0-Incubating, but I think it worth revising. It is mislead users to pass authentication information through the HTTP POST body, which leads to authentication failure.

could you help rebase the code @hddong ?
@pan3793 I think the change is fine, it was a mistake and now we are fixing it.

Should we mention it in the migration guide?

It is better to mention it in the migration guide.

Should we mention it in the migration guide?

It is better to mention it in the migration guide.

it's fine then.

@turboFei  Have rebased, and added on migration guide.
cc @pan3793 @yaooqinn


neo4j:migrate task is not creating a session (and not initializing the app)

If I run:
$ bin/rake neo4j:migrate

I get this error:
rake aborted!
No session, please create a session first with Neo4j::Session.open(:server_db) or :embedded_db

Tasks: TOP => neo4j:migrate:all
(See full trace by running task with --trace)

Instead if I run this:
$ bin/rake db:migrate neo4j:migrate

The migration runs correctly. I think that the rake task is not initializing the environment.
Runtime information:
Neo4j database version:
neo4j gem version: 7.0.8
neo4j-core gem version: 6.0.4

My guess is that db:migrate is loading the Rails environment and thus the railtie.rb.  The legacy_migrate seems to load the environment like this:
https://github.com/neo4jrb/neo4j/blob/master/lib/neo4j/tasks/migration.rake#L5
But would that work outside of Rails?  Could we use a generic way to open the session?  Maybe the code from railtie.rb should be extracted and simply called from railtie.rb and then we could also call it from things like these rake tasks or give people the option of using it manually so that they could just get things like config/neo4j.yml etc... for free

Good idea! Maybe I can take a look ;)

By all means! ;)

Closing here ;)


Reformat DynamicIdeasMapper and NationalIdeas to process standardized input.

Process format described in #3.
Allow for storage of additional specifiers.

Is ready for standard. Rules supported but not implemented.


On-board SQL Commands

@gregjhogan posted new issue for azure-xplat-cli ( https://github.com/Azure/azure-xplat-cli/issues/2204 ), so I have same request for new azure-cli.

Please add the following commands to ARM mode "az" azure-cli:
Commands to manage your SQL Server database servers
sql server create [options]
sql server show [options]
sql server list [options]
sql server delete [options]

Commands to manage your SQL Server firewall rules
sql firewallrule create [options]
sql firewallrule show [options]
sql firewallrule list [options]
sql firewallrule delete [options]

Commands to manage your SQL Server databases
sql db create [options]
sql db list [options]
sql db show [options]
sql db delete [options]

Of course, I don't mind command line syntax. I want an ability to control SQL Database from command line tool.

Hi @jsakamoto we plan on adding SQL support in the coming months. Please check back periodically. Thanks!

@tjprescott Thank you for replay. I'm looking forward to release day.
In my case, I often open/close IP based firewall from my remote working space, that is dynamic changing global IP address, for temporary maintenance job.
I appreciate if this job can finished with one click my script powered by "az" azure-cli.
Thanks!

Thanks @jsakamoto! This will help us prioritize the on-boarding of commands. SQL team is working hard behind the scenes to enable us to bring this functionality to the CLI. Right now I've put this on the backlog, but you should get updates when it is scheduled for an upcoming milestone.

Adding it to upcoming Sprint 8. We will have some key things lighting up and will go incrementally from there.

Adding it to upcoming Sprint 8.

Wow, it is good news!
I appreciate for your rapid response.
Thanks!

Most of the commands are in. @vishrutshah is helping with validating and adding test cases.

All the requested commands have been implemented and validated. Please refer src/command_modules/azure-cli-sql/azure/cli/command_modules/sql/tests/test_sql_commands.py to see the usage examples. Thanks!

Thanks everyone.
azure-cli-sql 0.1.1b2 works fine for my use case! 😄
https://gist.github.com/jsakamoto/827fb9eefc22a1faeae47507a66ab740


Provide support for mapping manager

Please provide support or document example on how to assign managerId

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

ID: 624be0a5-9ab6-ec59-e7a2-74585cb0571e
Version Independent ID: a00a09c8-f0c5-2bfe-75aa-6183346af405
Content: Tutorial: Configure Salesforce for automatic user provisioning with Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/saas-apps/salesforce-provisioning-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@fernando-almeida-sprinklr
Thanks for your feedback! We will investigate and update as appropriate.

Hi @fernando-almeida-sprinklr
You can find an example on how to assign managerId in the following tutorial. In this tutorial, you will learn how to configure automatic user provisioning from Azure AD to Salesforce. The tutorial includes a section on how to map the manager attribute in Azure AD to the managerId field in Salesforce. You can follow the steps in this section to assign managerId.
https://learn.microsoft.com/en-us/azure/deployment-environments/how-to-configure-managed-identity

@fernando-almeida-sprinklr
I'm going to assign this to the document author so they can take a look at it accordingly
@jeevansd
Can you please check and add your comments on this doc update request as applicable.

@v-missam, can you please take a look at the issue?

thanks for pushing it forward... this not addressed and is part of the default attributes available for mapping.
From my perspective it seems like the expected mapping to be defined for Azure AD manager is not properly doing the lookup in Salesforce for the matching user such that it can assigned the expected identifier.

From: HAMSIKA GAMPALA @.>
Sent: Monday, April 10, 2023 4:59 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Fernando Almeida @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Provide support for mapping manager (Issue #107464)
@v-missamhttps://github.com/v-missam, can you please take a look at the issue?
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/107464#issuecomment-1501362288, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3WMEBNRJ4T55BLOMD54C5DXAOAQJANCNFSM6AAAAAAWNY7UZA.
You are receiving this because you were mentioned.Message ID: @.***>

#reassign:twimmers

@twimmers - Could you please look into the managerId concern.

@fernando-almeida-sprinklr You can map the manager attribute in Azure AD to the managerID attribute in Salesforce:

This should resolve the user in Salesforce and then populate the userID of the manager into the manageID attribute in Salesforce. If you're not seeing this happen, check to ensure the user and the user's manager are both in the sync scope. The manager needs to be in scope as well, otherwise it will fail to resolve the reference.

Hi Tom,
Thanks for getting back to me.
After some additional tests it seems I was misreading a warning/error I was getting from the response which did not seem to initially be related. I've tracked it down to one of the developers having created a validation rule that was being enforced and causing the specific manager for the user I was testing to be rejected and even for users out of scope to not be able to be stripped of their manager.
Best,
Fernando

From: Tom Wimmers @.>
Sent: Tuesday, April 11, 2023 4:13 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Fernando Almeida @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Provide support for mapping manager (Issue #107464)
@fernando-almeida-sprinklrhttps://github.com/fernando-almeida-sprinklr You can map the manager attribute in Azure AD to the managerID attribute in Salesforce:
[image]https://user-images.githubusercontent.com/49566142/231206642-85a6efba-1864-48ff-bd36-7b6165e95db5.png
This should resolve the user in Salesforce and then populate the userID of the manager into the manageID attribute in Salesforce. If you're not seeing this happen, check to ensure the user and the user's manager are both in the sync scope. The manager needs to be in scope as well, otherwise it will fail to resolve the reference.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/107464#issuecomment-1503579810, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3WMEBLPE7SMSNVCWB7JQBDXAVYLDANCNFSM6AAAAAAWNY7UZA.
You are receiving this because you were mentioned.Message ID: @.***>

#please-close


Dialog background mask doesn't cover entire viewport in standards mode.

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=185

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


deploy documentation in qiskit.org/ecosystem

Similar to https://github.com/Qiskit/qiskit-aer/pull/1748, see https://github.com/Qiskit/qiskit.org/issues/3038 for more context
Summary
The Qiskit Ecosystem lives in qiskit.org/ecosystem and new place for qiskit related projects that are not technically Qiskit. This PR adds that destination for documentation deployment.
Details and comments
For now, a copy of the same docs in ecosystem would allow to have both q.o/documentation and q.o/ecosystem for testing that everything is fine. Once that's done, the CDN will redirects to the new location and the deploy to q.o/documentation can be removed.

@1ucian0 will the /documentation to /ecosystem redirect persist forever so that links that are hard to update aren't broken?

@1ucian0 Thanks for this! I updated the link in a few more places and the dev branch docs build script too. If this looks good to you then we can merge.

@1ucian0 I've reverted the link changes, does it look ok now?

I've temporarily removed this from the merge queue because I'm running a manual rclone to update the copy of the docs at ecosystem/ from the current version at documentation/ after that finishes I'll re-enqueue it.


Release 3.0.21

Welcome to Clear Containers release 3.0.21.
For this release the virtcontainers package was updated to bring support for SCSI. Now it is possible to choose the hypervisor storage driver trough Clear Containers configuration file.
Clear Containers Kernel now is updated to latest LTS version. This kernel is required as part of the new SCSI support, the previous version was not working correctly.
Changes

vendor: Update virtcontainers vendoring
Add configuration for hypervisor block storage driver

Shortlog
5e27776 vendor: Update virtcontainers vendoring
dc15173 cc-env: Include hypervisor block storage driver in cc-env output
1768dd5 config: Add configuration option for hypervisor block storage driver
accaa9d vendor: Vendor virtcontainers for SCSI support
Compatibility with Docker
Clear Containers 3.0.21 is compatible with Docker v17.12-ce
OCI Runtime Specification
Clear Containers 3.0.21 support the OCI Runtime Specification v1.0.0-rc5
Clear Linux Containers image
Clear Containers 3.0.21 requires at least Clear Linux containers image 20640 and agent version 6f6e9e.
Clear Linux Containers Kernel
Clear Containers 3.0.21 requires at least Clear Linux Containers  kernel v4.14.22-86.container
Installation

BC-Linux

Centos

Clear Linux

Fedora

SLES

Ubuntu

Developers

Issues & limitations
Docker swarm support
See issue #771 for more
Networking
Adding networks dynamically
Resource management
docker run --cpus=
See issue #341 for more information.
docker run --kernel-memory=
See issue #388 for more information.
shm
cgroup constraints
Capabilities
See issue #51 for more information.
sysctl
tmpfs
Other
checkpoint and restore
docker stats
See issue #200 for more information.
runtime commands
ps command
See issue #95 for more information.
events command
See issue #379 for more information.
update command
See issue #380 for more information.
Networking
Support for joining an existing VM network
docker --net=host
docker run --link
Host resource sharing
docker --device
docker -v /dev/...
docker run --privileged
Other
Annotations
runtime commands
init command
spec command
Host rdmsr warnings
More information Limitations

Can we explain why kernel https://github.com/clearcontainers/linux/releases/tag/v4.14.22-86.container is required?

@jodh-intel added a paragraph in the release notes about the kernel why is need new kernel.

Restating ubuntu 16.04 job
16:16:10 # cd .; git clone https://github.com/kubernetes-incubator/cri-o /home/jenkins/jenkins_slave/workspace/clear-containers-runtime-azure-ubuntu-16-04-PR/go/src/github.com/kubernetes-incubator/cri-o
16:16:10 Cloning into '/home/jenkins/jenkins_slave/workspace/clear-containers-runtime-azure-ubuntu-16-04-PR/go/src/github.com/kubernetes-incubator/cri-o'...
16:16:10 error: index-pack died of signal 15
16:16:10 fatal: index-pack failed
16:16:10 package github.com/kubernetes-incubator/cri-o: exit status 128
16:16:10 .ci/install_crio.sh: line 26: pushd: /home/jenkins/jenkins_slave/workspace/clear-containers-runtime-azure-ubuntu-16-04-PR/go/src/github.com/kubernetes-incubator/cri-o: No such file or directory

@grahamwhaley  @chavafg  seems that manage-ctr_mrg.sh hanged in metrics server
 + /srv/jenkins/workspace/workspace/clear-containers-runtime-16.04-PR/go/src/github.com/clearcontainers/tests/cmd/container-manager/manage_ctr_mgr.sh docker configure -r runc -f

yep, is under discussion on https://github.com/clearcontainers/tests/pull/942 and thereabouts.
Something is upsetting the docker on the server - we need to work out what exactly, probably by reproducing locally...
for now, just ignore the (non-blocking) metrics ci fails I'm afraid.

@jcvenegas small typo
storage driver trough Clear


Updated installation.md with respect to information about CVAT_HOST for local installation

Resolves #4013

Motivation and context

The installation guide should mention about exporting CVAT_HOST variable so that CVAT can be access over the network. The current installation method only supports accessing CVAT using http://localhost:8080. This information about exporting CVAT_HOST variable can be derived from AWS-Deployment Guide
source here
actual here
How has this been tested?

This is document level change, therefore the markdown has been checked for sanity.
Checklist

[X] I submit my changes into the develop branch
[X] I have added description of my changes into CHANGELOG file
[X] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[X] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[X] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@anugrahasinha , thanks for your contribution! Great job. I will merge the PR after all linter checks are passed.

Thanks for merging the PR. 👍


Sending inscription

Is there any API for sending inscription?
Equivalent of this from Unisat wallet: https://docs.unisat.io/dev/unisat-developer-service/unisat-wallet#sendinscription

This feature is not currently available, but should be in the next couple of weeks.

The ETA on this was a bit off 😅 It's currently being actively worked on, so expect it soon.

this was released for sats-connect v2.6.0 and now supported from Xverse extension v0.39.0


How to use Pytorch with Waymo data

Hello,
Thanks for sharing the great work.
Since you evaluated your model with Waymo data, I wonder how you could use Waymo dataloader which is in Tensorflow? Is there a way we can bypass that and still have a Pytorch code?
Thanks!

Instead of using the waymo dataloader, we use the parsing function provided in the waymo tutorial to parse raw data into a dictionary, and then convert the tf tensors in the dictionary into numpy arrays.

That's a nice idea!
So then have you tried using both Waymo and Argo data together? Does it help?

Hi @GentleSmile,
I just have a simple follow up question. You used the tf_example set instead of scenario set for waymo dataset, right?


Remove some old Analyzer codes

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] refactor
[ ] others

[FE PR Coverage check]
:disappointed: fail : 5 / 11 (45.45%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/analysis/TimestampArithmeticExpr.java
1
7
14.29%

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/SortNode.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/PlanNode.java
1
1
100.00%

[FE PR Coverage check]
:disappointed: fail : 5 / 11 (45.45%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/analysis/TimestampArithmeticExpr.java
1
7
14.29%

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/SortNode.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/PlanNode.java
1
1
100.00%

[FE PR Coverage check]
:disappointed: fail : 7 / 19 (36.84%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/planner/FileScanNode.java
0
1
00.00%

:large_blue_circle:
com/starrocks/load/ExportJob.java
0
2
00.00%

:large_blue_circle:
com/starrocks/planner/StreamLoadScanNode.java
0
1
00.00%

:large_blue_circle:
com/starrocks/task/ExportExportingTask.java
0
1
00.00%

:large_blue_circle:
com/starrocks/analysis/TimestampArithmeticExpr.java
1
7
14.29%

:large_blue_circle:
com/starrocks/planner/MysqlScanNode.java
2
3
66.67%

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/SortNode.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/PlanNode.java
1
1
100.00%

This PR has passed all daily tests.

[FE PR Coverage check]
:disappointed: fail : 15 / 27 (55.56%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/load/ExportJob.java
0
2
00.00%

:large_blue_circle:
com/starrocks/task/ExportExportingTask.java
0
1
00.00%

:large_blue_circle:
com/starrocks/analysis/TimestampArithmeticExpr.java
1
7
14.29%

:large_blue_circle:
com/starrocks/planner/MysqlScanNode.java
2
3
66.67%

:large_blue_circle:
com/starrocks/planner/StreamLoadScanNode.java
3
4
75.00%

:large_blue_circle:
com/starrocks/planner/FileScanNode.java
5
6
83.33%

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/SortNode.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/PlanFragment.java
1
1
100.00%

:large_blue_circle:
com/starrocks/planner/PlanNode.java
1
1
100.00%


Inconsistent SOCRATES environment variables

The new method for accessing the SOCRATES directory defaults to the SOCRATES env variable. If this is unset, then it uses the directory provided by platformdirs.
However, using SOCRATES is inconsistent with other codes, which instead use the variable name RAD_DIR. This includes SOCRATES itself, AGNI, and PROTEUS. We should decide which is going to be used going forward.
@stefsmeets what do you think?

I'm all for consistency, so if it makes more sense to have to RAD_DIR, we should change it.


feat(easy-mig): add pants to project

Codecov Report

Merging #113 (ef8cd94) into main (b783b62) will increase coverage by 0.06%.
The diff coverage is 100.00%.

:exclamation: Current head ef8cd94 differs from pull request most recent head faef3ae. Consider uploading reports for the commit faef3ae to get more accurate results

@@            Coverage Diff             @@
##             main     #113      +/-   ##
==========================================
+ Coverage   88.76%   88.82%   +0.06%
==========================================
  Files          14       15       +1
  Lines         543      546       +3
==========================================
+ Hits          482      485       +3
  Misses         61       61

Impacted Files
Coverage Δ

cisco_telescope/exporter_factory.py
93.33% <ø> (ø)

cisco_telescope/__init__.py
100.00% <100.00%> (ø)

...escope/instrumentations/instrumentation_wrapper.py
82.05% <100.00%> (ø)

cisco_telescope/options.py
98.11% <100.00%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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


Please fix issues reported by adapter checker

Please fix the issues (at least those marker with bold) for next release:
❗ [E160] "admin" is not allowed in common.dependencies
❗ [E154] common.dependencies must contain [{"js-controller": ">=2.0.0"}] or later - recommended: [{"js-controller": ">=3.3.22"}]
👀 [W171] "common.title" is deprecated in io-package.json
👀 [W105] "common.titleLang" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
👀 [W109] "common.desc" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
👀 [W156] Adapter should support admin 5 UI (jsonConfig) if you do not use a React based UI
👀 [W172] "common.localLink" in io-package.json is deprecated. Please define object "common.localLinks": { "_default": "..." }
👀 [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
👀 [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
👀 [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
👀 [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
👀 [W513] "gulpfile.js" found in repo! Think about migrating to @iobroker/adapter-dev package
👀 [W505] setTimeout found in "main.js", but no clearTimeout detected

replaced by #320


In beta version it asks to downgrade to production version

In beta version it checks for the version and also show to update to lower production version. please check

Thank you.. Please do it as quick as possible.
From: Vadim Shchenev<EMAIL_ADDRESS>Sent: Friday, May 26, 2017 12:41 AM
To: robohorse/gpversionchecker<EMAIL_ADDRESS>Cc: Mohamed Ibrahim<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [robohorse/gpversionchecker] In beta version it asks to downgrade to production version (#10)
ok thanks, i will see it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/robohorse/gpversionchecker/issues/10#issuecomment-304097273 , or mute the thread https://github.com/notifications/unsubscribe-auth/AS3CJj-yD9g7OiWgeg_V8wO2n8rA7jA9ks5r9dJIgaJpZM4NlX02 .  https://github.com/notifications/beacon/AS3CJqDMAlCPueth5ShHEwgdYYK5Jcodks5r9dJIgaJpZM4NlX02.gif

I think, that i cant fix it because i cant parse your beta version number (that does not shown on the app page). This library works fine for production version of usual user, but have problems with test cases like yours.


HIPAA compliant?

Microsoft PowerToys version
Newest version
Installation method
Microsoft Store
Running as admin
No
Area(s) with issue?
General
Steps to reproduce
I want to confirm if PowerToys is HIPAA compliant or not?
✔ Expected Behavior
Just want to confirm if the software is HIPAA Compliant
❌ Actual Behavior
Just want to confirm if the software is HIPAA Compliant
Other Software
Just want to confirm if the software is HIPAA Compliant

I can't think of any instances in which PowerToys would be handling protected health information (PHI) let alone using it. I would assume that HIPAA is not applicable (aka default compliant).
@crutkas, any comments here?

Please look over https://learn.microsoft.com/en-us/compliance/regulatory/offering-hipaa-hitech?view=o365-worldwide and work with your hipaa compliance team.
Everything inside PowerToys has GPO as well so it can be disabled via Group Policy if any feature is deemed something you don't want enabled for you or your team.


User list still shows the deleted users

This is already an issue regarding this opened on the sync gateway repo.
See https://github.com/couchbase/sync_gateway/issues/1557
Action item:

[x] Verify the issue with Couchbase Server instead of Warus

The issue doesn't happen when connecting to the CB Server instead of Walrus.


Add ability to include completely raw files

I have a list of files I need to include without any header or footer, which currently isn't possible. According to the README one should use nftables::config for that which Manages a raw file in /etc/nftables/puppet/${name}.nft, but that always surrounds the provided source or content with stuff.
Currently, I have to create the files manually, which isn't clean at all:
  file { '/etc/nftables/puppet/custom-include-geoipsets.nft':
    ensure  => 'present',
    mode    => '0640',
    content => @(EOT),
    # THIS FILE IS MANAGED BY PUPPET

    include "/var/local/geoipsets/dbip/nftset/ipv4/*.ipv4"
    include "/var/local/geoipsets/dbip/nftset/ipv6/*.ipv6"
    |EOT
  }

  file { '/etc/nftables/puppet-preflight/custom-include-geoipsets.nft':
    ensure => 'link',
    target => '/etc/nftables/puppet/custom-include-geoipsets.nft',
  }

Am I missing something here?
There should be an ability to A) provide a list of external *.nft files to include or B) a flag for nfttables::config to use the content as-is.

Do you have the nftables::config resource you tried.
I don't use this but I would have thought it was.
nftables::config{'inet-filter':
    content => @(EOT),
    # THIS FILE IS MANAGED BY PUPPET

    include "/var/local/geoipsets/dbip/nftset/ipv4/*.ipv4"
    include "/var/local/geoipsets/dbip/nftset/ipv6/*.ipv6"
    |EOT,
}

presumably those dbip files are generated from something else and the sets in there will be added to inet-filter table.

Wrong:
nftables::config{'geoip-sets':
    content => @(EOT),
    # THIS FILE IS MANAGED BY PUPPET

    include "/var/local/geoipsets/dbip/nftset/ipv4/*.ipv4"
    include "/var/local/geoipsets/dbip/nftset/ipv6/*.ipv6"
    |EOT,
}

Though I confess I am bit confused why the title has to xyz-abc

This won't work, since it will always surround it with a table:
table geoip sets {
# THIS FILE IS MANAGED BY PUPPET

include "/var/local/geoipsets/dbip/nftset/ipv4/*.ipv4"
include "/var/local/geoipsets/dbip/nftset/ipv6/*.ipv6"
}

So we need a new type nftables::file that does the the whole pre-flight thing.


Query with bracket notation and object literals (@ in attribute name)

Looking to use this library for querying a directory of 'Thing Descriptions'
The vocabulary of a TD includes an attribute named "@type". Is it possible to query such attribute?
According to the json-path-comparision, the filter expression with bracket notation and current object literal (for example: "$[?(@['@type']=="sensor")]"  is supported by this library but this returns an empty result.
What did work was the bracket notation without the object literal, eg $[?(@['type']=="sensor)].
Btw, many thanks for this library. It just seems to work without the usual head banging :). Great job!

That is interesting. Sounds like a bug that will need to be fixed.

Simple test example that shows this.
Query1 returns 1 item in the result. Query 2 returns no items in the result. The only difference is change from query of 'type' to  '@type', both using the bracket notation.
Hope this helps.
func TestQuery2(t *testing.T) {
	store := make(map[string]interface{})

	jsondoc := `{
		"item1": {
			"id": "item1",
			"type": "type1",
			"@type": "attype1"
		},
		"item2": {
			"id": "item2",
			"type": "type2",
			"@type": "attype2"
		}
	}`
	query1 := `$[?(@['type']=="type1")]`           // bracket notation without object literal
	query2 := `$[?(@['@type']=="type1")]`          // with object literal

	err := json.Unmarshal([]byte(jsondoc), &store)
	assert.NoError(t, err)

	jpExpr, err := jp.ParseString(query1)
	assert.NoError(t, err)
	result := jpExpr.Get(store)         // this returns one result as expected
	assert.NotEmpty(t, result)

	jpExpr, err = jp.ParseString(query2)
	assert.NoError(t, err)
	result = jpExpr.Get(store)
	assert.NotEmpty(t, result)      // this fails with empty result
}

Thanks, that helps.

I found the problem. query2 is checking for @type being equal to "type1". There is no match for that in the json doc. There is a match for "attype1" though. This worked for me:
func TestFilterAt(t *testing.T) {
	jsondoc := `{
			"item1": {
				"id": "item1",
				"type": "type1",
				"@type": "attype1"
			},
			"item2": {
				"id": "item2",
				"type": "type2",
				"@type": "attype2"
			}
		}`
	store := oj.MustParseString(jsondoc)
	x := jp.MustParseString(`$[?(@['@type']=="attype1")]`)
	result := x.Get(store)
	tt.Equal(t, 1, len(result))
}

That test has been added to the jp/get_test.go file in the project.

Arggh, that was a silly mistake with my test cases.
Sorry for wasting your time and thank you for looking into it.

All good. It lets me know there are some users out there. :-)


Compose File Fixes

Small fixes for the docker-compose file

Suggest closing this PR as the docker-compose fix has already gone in elsewhere, and there doesn't seem to be any further conversation on the stack trace logging.


Cannot insert measurement after delete

Bug report
System info: [0.13.0, OS X 10.11.5]
Steps to reproduce:

INSERT cpu,host=serverA,region=us_west value=0.64
DELETE FROM cpu
repeat step 1
SELECT * FROM cpu

Expected behavior: Step 4 should return one result
Actual behavior: Step 4 not return one result

Sorry, my fault. I missed to specify a value field in select.


Which of sap.ui.require or sap.ui.define should I use when writing an UI5-controller?

As far as I understand, in many cases sap.ui.require and sap.ui.define are interchangeable. However, there is a convention:

Use sap.ui.define for controllers and all other JavaScript modules to define a global namespace. With the namespace, the object can be addressed throughout the application.

Use sap.ui.require for asynchronously loading dependencies but without declaring a namespace, for example code that just needs to be executed, but does not need to be called from other code.

I read this statement several times, but I'm not sure I got it completely.
For instance, when I write an UI5-controller for the specific UI5-view, then what should I use? And when I write a BaseController.js, which the rest of the controllers will be based on, then should I chose sap.ui.define or sap.ui.require?

It's more than a convention, there are significant technical differences between the two APIs.
sap.ui.define defines a module (nomen est omen). It is the outermost frame of a module, its entry point so to say.
The dependencies that are listed in the sap.ui.define call will be resolved before the factory function (the function given as argument to the sap.ui.define call) will be executed. And only after the function has executed, will others see your module and its return value (AKA "export"). Code outside your module has to wait for all the steps to complete before it can continue accessing your module.
For sap.ui.require, this second aspect of synchronisation is missing. It only resolves the listed dependencies and calls the callback function. But there's no export from a sap.ui.require call and no code besides the sap.ui.require call will wait for dependency resolution and execution of the factory function to happen. On the other side, sap.ui.require has a quality that sap.ui.define doesn't have. You can decide when to call it whereas sap.ui.define, as outer frame of a module will only be executed when a module is loaded and executed.
Well, if that explanation doesn't help to understand the difference, maybe a comparison with ES6 modules can help. sap.ui.define is more like the static import and export statements and sap.ui.require is more like the dynamic import() statement.
So when should you use what?
Whenever another piece of code shall be able to express a dependency to your code, you should place your code in a separate file and wrap it with sap.ui.define. That's true for controls, for renderers, for controllers, for JSViews, for components, for helpers, for model implementations etc. Most of the time, sap.ui.define is the only top-level code in a module.
In contrast to that, sap.ui.require usually is placed inside some method, often in an event handler. It will be used to load other code lazily that your module does not statically depend on (like with the sap.ui.define dependencies).

Thanks a lot for the clarification, especially for a clear and brief parallel with the ES6 modules.
To conclude, do I understand it correctly, that basically the view controllers' code should start with sap.ui.define, while sap.ui.require should be used if in the middle of controller's code I want to import some functionality, located in another UI5 component?

Yes, exactly.

As your question seems to be answered, I'll close this issue now.

Thanks! Please, consider elaborating the official documentation, since there it's more represented as a matter of convention and taste, rather than a matter of architecture.

I've added this issue as feedback to the page that you linked initially.


TESTING

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Dataflow Runner Nexmark Tests

Run Dataflow Runner Nexmark Tests

Run Dataflow Runner Nexmark Tests

Run Dataflow Runner Nexmark Tests

Run Dataflow Runner Nexmark Tests

Run Dataflow Runner Nexmark Tests

Run Dataflow Runner Nexmark Tests

Run Dataflow Runner Nexmark Tests

Run Dataflow Runner Nexmark Tests

Run Dataflow Runner Nexmark Tests

Run Dataflow Runner Nexmark Tests

@apilloud this should unblock the next release.

@apilloud I manually verified that all the Nexmark pipelines succeeded, and the previous errors are all gone. I'm not sure why the triggered test is marked as failed. The logs don't really give any hint.

Run Java PreCommit

Run Dataflow Runner Nexmark Tests

Failure from nexmark looks to be #8301.
cc: @amaliujia
10:32:56 Exception in thread "main" java.lang.RuntimeException: java.lang.UnsupportedOperationException: Non equi-join is not supported!
10:32:56 	at org.apache.beam.sdk.nexmark.Main.runAll(Main.java:128)
10:32:56 	at org.apache.beam.sdk.nexmark.Main.main(Main.java:415)
10:32:56 Caused by: java.lang.UnsupportedOperationException: Non equi-join is not supported!
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamJoinRel.extractJoinPairOfRexNodes(BeamJoinRel.java:569)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamJoinRel.extractJoinRexNodes(BeamJoinRel.java:554)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamJoinRel.access$900(BeamJoinRel.java:101)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamJoinRel$ExtractJoinKeys.expand(BeamJoinRel.java:257)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamJoinRel$ExtractJoinKeys.expand(BeamJoinRel.java:240)
10:32:56 	at org.apache.beam.sdk.Pipeline.applyInternal(Pipeline.java:537)
10:32:56 	at org.apache.beam.sdk.Pipeline.applyTransform(Pipeline.java:471)
10:32:56 	at org.apache.beam.sdk.values.PCollectionList.apply(PCollectionList.java:176)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamJoinRel$StandardJoin.expand(BeamJoinRel.java:324)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamJoinRel$StandardJoin.expand(BeamJoinRel.java:317)
10:32:56 	at org.apache.beam.sdk.Pipeline.applyInternal(Pipeline.java:537)
10:32:56 	at org.apache.beam.sdk.Pipeline.applyTransform(Pipeline.java:488)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamSqlRelUtils.toPCollection(BeamSqlRelUtils.java:66)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamSqlRelUtils.lambda$buildPCollectionList$0(BeamSqlRelUtils.java:47)
10:32:56 	at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
10:32:56 	at java.util.Iterator.forEachRemaining(Iterator.java:116)
10:32:56 	at java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)
10:32:56 	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
10:32:56 	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
10:32:56 	at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
10:32:56 	at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
10:32:56 	at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamSqlRelUtils.buildPCollectionList(BeamSqlRelUtils.java:48)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamSqlRelUtils.toPCollection(BeamSqlRelUtils.java:64)
10:32:56 	at org.apache.beam.sdk.extensions.sql.impl.rel.BeamSqlRelUtils.toPCollection(BeamSqlRelUtils.java:36)
10:32:56 	at org.apache.beam.sdk.extensions.sql.SqlTransform.expand(SqlTransform.java:111)
10:32:56 	at org.apache.beam.sdk.extensions.sql.SqlTransform.expand(SqlTransform.java:79)
10:32:56 	at org.apache.beam.sdk.Pipeline.applyInternal(Pipeline.java:537)
10:32:56 	at org.apache.beam.sdk.Pipeline.applyTransform(Pipeline.java:471)
10:32:56 	at org.apache.beam.sdk.values.PCollectionTuple.apply(PCollectionTuple.java:248)
10:32:56 	at org.apache.beam.sdk.nexmark.queries.sql.SqlQuery5.expand(SqlQuery5.java:106)
10:32:56 	at org.apache.beam.sdk.nexmark.queries.sql.SqlQuery5.expand(SqlQuery5.java:55)
10:32:56 	at org.apache.beam.sdk.Pipeline.applyInternal(Pipeline.java:537)
10:32:56 	at org.apache.beam.sdk.Pipeline.applyTransform(Pipeline.java:471)
10:32:56 	at org.apache.beam.sdk.values.PCollection.apply(PCollection.java:357)
10:32:56 	at org.apache.beam.sdk.nexmark.queries.NexmarkQuery.expand(NexmarkQuery.java:89)
10:32:56 	at org.apache.beam.sdk.nexmark.queries.NexmarkQuery.expand(NexmarkQuery.java:33)
10:32:56 	at org.apache.beam.sdk.Pipeline.applyInternal(Pipeline.java:537)
10:32:56 	at org.apache.beam.sdk.Pipeline.applyTransform(Pipeline.java:471)
10:32:56 	at org.apache.beam.sdk.values.PCollection.apply(PCollection.java:357)
10:32:56 	at org.apache.beam.sdk.nexmark.NexmarkLauncher.run(NexmarkLauncher.java:1161)
10:32:56 	at org.apache.beam.sdk.nexmark.Main$Run.call(Main.java:90)
10:32:56 	at org.apache.beam.sdk.nexmark.Main$Run.call(Main.java:79)
10:32:56 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
10:32:56 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
10:32:56 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
10:32:56 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
10:32:56 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
10:32:56 	at java.lang.Thread.run(Thread.java:748)

Is there a way to ignore Nexmark SqlQuery5 test for Dataflow Runner nexmark Tests?
Basically Nexmark SqlQuery5 does non equi-join. Due to a bug in BeamSQL, SqlQuery5 was always executed as equi-join (>= was treated as =). Now non equi-join is rejected.


Adding a ray cast feature to Solar2D

It would be great to be able to get a list of all objects based on screen coordinates.
For example screen.rayCast(x, y) that would return a table of all objects that intersect at that point.  It would make drag and drop much easier too.
This would be separate from the normal touch event propagation so wouldn't affect any existing code.

I have used this for that purpose. Perhaps you are asking for something different.
http://docs.coronalabs.com/api/library/physics/rayCast.html


Buildings being seperate gameobject and not a single mesh causing performance issues

I'm not too familiar with the visual aspects of Unity but are the separate building components retained in memory anywhere (imagine grouping objects in an image editor) or do they permanently become one big mesh?
I ask because some games might want to utilize certain buildings as features. I suppose we can always simply get the world coordinates of a tap, convert it back to TSM coordinates, and check the JSON to see if there's a building there, but it seems like that still limits visual changes we can make to any buildings unless we redraw a new object over the existing one.
It's hardly the most pressing issue, but it's something to consider for the future.

I totally agree with that. My initial plan was (actually still is) to make buildings separate object and then maybe detail them even further. This was player will be able to select, click, choose whatever, that object easily.
But the problem is, gameobjects aren't free. They come with at least 2-3 components and require additional draw calls. You can use shared material and make them static to reduce draw calls and improve performance but still, merging them into one big mesh will be much better for mobile for example (assuming you don't need direct interaction, i.e. pokemon go).
I'll make that as an alternative building factory so you'll be able to switch whichever you want.

I actually just realized that I skipped your question; all buildings are kept in a list inside BuildingFactory. You can query anything from there. Let me know if I missed something again ;)

Ha, no worries; I've actually been working my way through a nice tutorial on making a hexagonal grid that covers mesh-making, edge blending, individual selection, etc.

somewhat completed in 0bf25143a3fe87c92d9a48610cb72fa40e35c9ed


make micro ros epsidf work with IDF 5

This pull request enables building micro ros with ESP IDE 5
To test is, a new Dockerfile referencing to IDF 5 is also included.
Changes:

Adding a few includes
fixing a bad format in int32_publisher (may also be needed with the other examples, but i was only testing with this one)
Pull request "Update CMakeLists.txt # 160" ist also included.

Tested with EPS32S3 with included docker container and also with platform.io ESP IDF 5.1.1 platform

hope that its complete now. let's see what to CI says...

finally the ci is working. this build system is not very handsome for integrating new frameworks...

@sebastianohl feel free to propose any improvement to the build system

please merge this

sorry @CasCard in which esp hardware have you tested this ????

There was an issue with the C standard version of some of micro-ROS packages, which has been solved on our side.
We are including your update with some fixes on this PR: https://github.com/micro-ROS/micro_ros_espidf_component/pull/176
Closing, feel free to provide any feedback if you test our implementation with the v5 version.


How to manage SharePoint list view ?

We can "Set List view" :

Update title
Update / set formatter (json)

But

I would like add existing list column to existing view
Create new list view
We can use "set view" but can't "get view" (for retrieve the Id for example)

Currently, I can't deploy a SharePoint list based on template and apply view formatter directly from my CI without manual action on the list to select which columns have to be displayed.
I missed something on the current capabilities of Office365-CLI ?
Something is in pending on that ?
Thx

@Laul0, you can suggest them as separate commands like 'spo listview get' and 'spo listview add'. Someone from the community can implement them. Even you can contribute if you are interested.
That project is community driven and not all the apis are implemented yet since the cli is quite new project.
If you are not interested in writing code, I can assist you so we get the commands available asap.
You, myself or Waldek can split this question to multiple github issues and I can start work on that so we have complete support for list views.

@VelinGeorgiev, it was exactly what I mean about cmdlet.
I'm not against writing code and contributing to this project.
I first opened this issue because as part of an internal training, I'm trying to implement a completely automated solution and these are the cases I am facing today 😕
But ok to start implementing these new commands 😄
If you want, I can split this question to specific issues...

That would be helpful @Laul0. I appreciate your help 👏

Closing this issue in favor of the issues for the specific commands you created, @Laul0


Migration Plan to Vue 3 and Vuetify for CSHR Project (Version 1.1)

Description
In the upcoming version 1.1, we are migrating the UI of the CSHR project from Svelte to Vue 3 and Vuetify to enhance frontend features. This issue covers all the necessary tasks and features involved in this migration.

Implement Auth System (Login/Register Screens)

Issue: https://github.com/codescalersinternships/CSHR/issues/264

Implement Global Calendar View

Issue: https://github.com/codescalersinternships/CSHR/issues/265
Include filters and required forms (meeting, vacation, event).
Calendar should be selectable, supporting date selection via click, drag, and drop.

Implement Profile Screen

Issue: https://github.com/codescalersinternships/CSHR/issues/266
Keep it simple and essential.

Implement Notifications Screen

Issue: https://github.com/codescalersinternships/CSHR/issues/267
Include a toast for timely notification display.

Implement Vacation/Event/Meeting Details Screen

Issue: https://github.com/codescalersinternships/CSHR/issues/268
Allow users to cancel vacation requests while pending approval.
Users can delete requests for other modal types.

Implement My Team Screen

Issue: https://github.com/codescalersinternships/CSHR/issues/269
Display team leaders and members separately.

Implement Organization Team Screen

Issue: https://github.com/codescalersinternships/CSHR/issues/270
Include filters for users based on their office.
Add filters to navigations as query parameters.

Implement Settings Screen

issue: https://github.com/codescalersinternships/CSHR/issues/271
Include only the change password form.

Implement Admin Dashboard with Forms

issue: https://github.com/codescalersinternships/CSHR/issues/272

Update/Post Vacation Balance Form

Reflects current year's data for the respective office.
Admins can modify existing data for 2024.

Update Set User Vacation Balance Form

Set a new balance for the selected user.
Users loaded based on the admin office.

Vacation Balance Adjustment Form

Admins can increment the vacation balance for the entire team.
Transparently document reasons for adjustments.

Add New Office Form

Create a new office with its weekend days.

Add New User Form (or integrate with Auth Issue)

Register form screen for new users.

Update User Information

Only admins in the same office as the user can update this information.

The front-end project has been upgraded to use Vue3.
Closed as fixed, please if you notice that the issue still exists open it again.


The Wazuh indexer removal does not remove the installation directory after upgrade

Describe the bug
The Wazuh indexer installation directory is not being removed from the system after upgrading from 4.8.2 to 4.9.1
Related https://github.com/wazuh/wazuh-qa/issues/5753
To Reproduce
Steps to reproduce the behavior:

Deploy a 4.8.2 All-In-One deployment using the Wazuh installation assistant
Upgrade the Wazuh indexer to 4.9.0 or 4.9.1
Remove the Wazuh indexer

Expected behavior
The /usr/share/wazuh-indexer directory should not exist after being removed
Plugins
Default ones
Host/Environment (please complete the following information):

OS: CentOS
Version: 7

Debian systems should be checked
Additional context

Removal of a Wazuh indexer 4.8.2 package after being upgraded from 4.7.5

...
  Verifying  : wazuh-indexer-4.8.2-1.x86_64                                                                                1/2
  Verifying  : wazuh-indexer-4.7.5-1.x86_64                                                                                2/2

Updated:
  wazuh-indexer.x86_64 0:4.8.2-1

Complete!

....

Removed:
  wazuh-indexer.x86_64 0:4.8.2-1

Complete!
[root@centos7 vagrant]# ls -l /usr/share/wazuh-indexer
ls: cannot access /usr/share/wazuh-indexer: No such file or directory

Removal of a Wazuh indexer 4.9.1 package after being upgraded from 4.8.2

....

  Verifying  : wazuh-indexer-4.9.1-0.x86_64                                                         1/2
  Verifying  : wazuh-indexer-4.8.2-1.x86_64                                                         2/2

Updated:
  wazuh-indexer.x86_64 0:4.9.1-0

Complete!

....

Removed:
  wazuh-indexer.x86_64 0:4.9.1-0

Complete!
[root@centos7 vagrant]# ls -l /usr/share/wazuh-indexer/
total 0
[root@centos7 vagrant]# ls -ld /usr/share/wazuh-indexer
drwxr-x---. 3 wazuh-indexer wazuh-indexer 20 Sep 19 18:55 /usr/share/wazuh-indexer

Removal of the 4.8.2 package
root@ubuntu2204:/home/vagrant# ls -la /usr/share/wazuh-indexer/
total 304
drwxr-x---   8 wazuh-indexer wazuh-indexer   4096 Sep  6 13:14 .
drwxr-xr-x 116 root          root            4096 Sep  6 13:22 ..
drwxr-x---   3 wazuh-indexer wazuh-indexer   4096 Sep  6 13:14 bin
drwxr-x---   9 wazuh-indexer wazuh-indexer   4096 Sep  6 13:14 jdk
drwxr-x---   3 wazuh-indexer wazuh-indexer   4096 Sep  6 13:14 lib
-rwxr-x---   1 wazuh-indexer wazuh-indexer  11358 Sep 19  2023 LICENSE.txt
drwxr-x---  21 wazuh-indexer wazuh-indexer   4096 Sep  6 13:13 modules
-rwxr-x---   1 wazuh-indexer wazuh-indexer 261032 Sep 19  2023 NOTICE.txt
drwxr-x---   5 wazuh-indexer wazuh-indexer   4096 Sep  6 13:14 performance-analyzer-rca
drwxr-x---  21 wazuh-indexer wazuh-indexer   4096 Sep  6 13:14 plugins
-r--r-----   1 wazuh-indexer wazuh-indexer      6 Aug 19 17:14 VERSION

root@ubuntu2204:/home/vagrant# apt-get remove --purge wazuh-indexer -y
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following packages will be REMOVED:
  wazuh-indexer*
0 upgraded, 0 newly installed, 1 to remove and 157 not upgraded.
After this operation, 1050 MB disk space will be freed.
(Reading database ... 190752 files and directories currently installed.)
Removing wazuh-indexer (4.8.2-1) ...
Stopping wazuh-indexer service... OK
(Reading database ... 189618 files and directories currently installed.)
Purging configuration files for wazuh-indexer (4.8.2-1) ...
Deleting configuration directory... OK
dpkg: warning: while removing wazuh-indexer, directory '/var/lib/wazuh-indexer' not empty so not removed
dpkg: warning: while removing wazuh-indexer, directory '/var/log/wazuh-indexer' not empty so not removed
root@ubuntu2204:/home/vagrant# ls -la /usr/share/wazuh-indexer/
ls: cannot access '/usr/share/wazuh-indexer/': No such file or directory

Removal of the 4.9.1 package
root@ubuntu2204:/home/vagrant# apt-get remove --purge wazuh-indexer -y
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following packages will be REMOVED:
  wazuh-indexer*
0 upgraded, 0 newly installed, 1 to remove and 157 not upgraded.
After this operation, 1077 MB disk space will be freed.
(Reading database ... 190765 files and directories currently installed.)
Removing wazuh-indexer (4.9.1-0) ...
Running Wazuh Indexer Pre-Removal Script
(Reading database ... 189618 files and directories currently installed.)
Purging configuration files for wazuh-indexer (4.9.1-0) ...
dpkg: warning: while removing wazuh-indexer, directory '/var/log/wazuh-indexer' not empty so not removed
dpkg: warning: while removing wazuh-indexer, directory '/var/lib/wazuh-indexer' not empty so not removed
dpkg: warning: while removing wazuh-indexer, directory '/etc/wazuh-indexer' not empty so not removed
root@ubuntu2204:/home/vagrant# ls -la /usr/share/wazuh-indexer
ls: cannot access '/usr/share/wazuh-indexer': No such file or directory
root@ubuntu2204:/home/vagrant# ls -lR /etc/wazuh-indexer
/etc/wazuh-indexer:
total 12
dr-x------ 2 wazuh-indexer wazuh-indexer 4096 Sep  6 13:14 certs
drwxr-x--- 2 wazuh-indexer wazuh-indexer 4096 Sep  6 13:23 internalusers-backup
-rw-rw---- 1 wazuh-indexer wazuh-indexer  196 Sep  6 13:14 opensearch.keystore

/etc/wazuh-indexer/certs:
total 20
-r-------- 1 wazuh-indexer wazuh-indexer 1704 Sep  6 13:06 admin-key.pem
-r-------- 1 wazuh-indexer wazuh-indexer 1119 Sep  6 13:06 admin.pem
-r-------- 1 wazuh-indexer wazuh-indexer 1204 Sep  6 13:06 root-ca.pem
-r-------- 1 wazuh-indexer wazuh-indexer 1704 Sep  6 13:06 wazuh-indexer-key.pem
-r-------- 1 wazuh-indexer wazuh-indexer 1285 Sep  6 13:06 wazuh-indexer.pem

/etc/wazuh-indexer/internalusers-backup:
total 4
-rw-r----- 1 wazuh-indexer wazuh-indexer 1145 Sep  6 13:23 internal_users_20240906_132300.yml.bkp

During the uninstallation of the wazuh-indexer package, the /usr/share/wazuh-indexer folder has been removed, like in previous versions.

We have performed several tests and we have been able to reproduce the problem only in RPM based systems. The /usr/share/wazuh-indexer is kept due to a hidden cache folder created by the JNA. The removal of this folder is harmless. As the problem is only reproducible on Red Hat based systems and as we state in our uninstallation guide to remove every wazuh-indexer folder recursively as a manual post-removal step, we have concluded this won't be fixed.
Our tests have been performed under the following operating systems, using Vagrant boxes:

Vagrant box
Reproducible

generic/ubuntu2204
No

generic/rhel9
Yes

generic/rhel7
Yes

bento/centos-8
No


{AKS} Update help message of option --node-osdisk-type

This checklist is used to make sure that common guidelines for a pull request are followed.
{AKS} Update help message of option --node-osdisk-type. The change is to correct the help information and align it with the information in the official azure-cli (p1, p2).
Related command

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

AKS

/azp run


[Feature][UI] the dolphinscheduler front end page needs to add watermark

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
In order to add the company's watermark logo of the dolphinscheduler  users，maybe  the dolphinscheduler   UI needs to add watermark  function, and the content of watermark is current username and its company name.
Use case
we can add watermark  function with js in vue project
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This is not what an open source project should take care of. If you need please do this in your company.


docs: Added badges to README.md

I added the following badges:

Poetry
Ruff
Supported Python versions
Workflow runs

here are the ones on readthedocs (which still needs to be fixed on the build side (either transferring the testcontainers-python subdomain, or the testcontainers subdomain set up via someone's auth who has permissions to create webhooks on this repo):
https://github.com/testcontainers/testcontainers-python/blob/d5b855323be06f8d1395dd480a347f0efef75703/index.rst?plain=1#L1-L11

maybe it makes sense for the badges to match, maybe not, idk

@alexanderankin Thanks for pointing that out. I just added the missing badges for docs and codespace. I choose to add the code space badge below the others is this one is breaking the layout a bit. That way it doesn't look too awkward.
@totallyzen The badges do match now. I just added some additional badges.

sorry, i thought this was merging into main, my apologies.


Set custom name on chrome address bar

Hi,
I'm using puppeteer lastest version and saw someone can set custom name on chrome address bar like this
How can i set this using puppeteer?
Thanks

Puppeteer doesn’t have any built-in API for this. I think what you're seeing there is a custom TLS certificate?

I dont think so because it show even when i visit http website.
When i click on this, it show the current website certificate and nothing named like this custom text


[backport/1.25] repo: Cleanup changelogs

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

the changelog for 1.25.1 is currently empty - which checking, is correct, so far

/retest

/retest


[markdown mode] Issue with quotes in lists

Go to http://marcelgerber.github.io/github/demo/codemirror/markdown-quote-in-list.html.
Result:
  ## foo and    # foo are marked .cm-comment (indented code block).
Expected:
  ## foo and    # foo are marked .cm-header (atx header).
Note:
This is caused by a negative state.indentation, which shouldn't ever happen. There's something wrong with how lists and blockquotes increase state.indentation.
Workaround:
Make state.indentation = Math.max(state.indentation, 0) so it's always positive. That works, but still it'd be better if we completely understand the issue.

@bfrohs Could you maybe take a look at this?

Additionally, is there any interest in using a popular JS markdown parser to run the MD mode instead of a hand-rolled one? Say, for instance, https://github.com/chjj/marked?

I'm not aware of any of these parsers (I think markdown-it is the most mature one in that family) exposing the primitives needed to build a CodeMirror mode on top of them (incremental parsing and being able to copy the parser state).

@bfrohs Could you maybe take a look at this?

Yeah, I'll try to take a look tonight.

Thanks @bfrohs!