Google Ads not showing

Hi !
i'm using:
Windows 10, Godot 3.0.6
Compiling Godot with and without admob works really fine, test ads are showing, good !
But the problem is when i try to show real ads with the IDs i get from my Google ads account, the app works find, the ads are loaded successfuly, but no ads shows up...

Android or iOS?
Are your ad ids new?

Android !
and i created my ad ids yesterday.

Please check the logs with adb (as stated on the README) and post the output here.

I am getting this :
02-09 20:01:19.964 27911-28178/org.godotengine.collide W/Ads: Not retrying to fetch app settings
02-09 20:01:22.834 27911-27911/org.godotengine.collide W/godot: AdMob: onAdFailedToLoad(int errorCode) - error code: 3
02-09 20:01:22.834 27911-27911/org.godotengine.collide W/godot: AdMob: _on_interstitial_not_loaded
02-09 20:01:22.834 27911-27911/org.godotengine.collide I/Ads: Ad failed to load : 3
02-09 20:01:22.884 27911-27911/org.godotengine.collide W/godot: AdMob: onAdFailedToLoad -> ERROR_CODE_NO_FILL
02-09 20:01:22.884 27911-27911/org.godotengine.collide I/Ads: Ad failed to load : 3
The strange this is, a banner ad appeared in one of my tests, but i cannot make it reapear now

As stated in the Known Issues (README file) it's a google Admob issue, most of the times it works after some time.

Oh ok.. So i will test it tomorow and hope it will go away, thanks !

Hello,
thank you for helping us with our issues.
How can I remove the tiny label "Test Ad"? The real ad is already showing and it works. But there is still the "Test Ad" label at the top of my banner and i don't know how to deleted it. Thank you

It's not possible to remove test ads on debugging and test with test unit id mode, this will be automatic removed when your game is published on Google Play or App Store with real unit id ads

So don't worry about the "Test Ad" label, this wont appear on apps released on Google Play or App Store

Alright, thank you for your help
Am Sa., 23. Mai 2020 um 17:32 Uhr schrieb Gustavo Maciel <
<EMAIL_ADDRESS>
So don't worry about the "Test Ad" label, this wont appear on apps
released on Google Play or App Store
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kloder-games/godot-admob/issues/67#issuecomment-633077134,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALKQNWZX5XZRVOEM6BHX2ILRS7UCBANCNFSM4GWHBSWQ
.

Hi, it's me again, my app is now online but the "Test Ad" label is still
showing, Any idea how to fix this problem? Thanks
Am Sa., 23. Mai 2020 um 18:51 Uhr schrieb B El<EMAIL_ADDRESS>
Alright, thank you for your help
Am Sa., 23. Mai 2020 um 17:32 Uhr schrieb Gustavo Maciel <
<EMAIL_ADDRESS>
So don't worry about the "Test Ad" label, this wont appear on apps
released on Google Play or App Store
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kloder-games/godot-admob/issues/67#issuecomment-633077134,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALKQNWZX5XZRVOEM6BHX2ILRS7UCBANCNFSM4GWHBSWQ
.

Did you put a real ads and disabled the variable isReal?

the real ads are active. I set the isReal boolean on true and deleted the
Strings (the test ad ids) and added my own ids . Am I supposed to disable
the Variable isReal ?
Gustavo Maciel<EMAIL_ADDRESS>schrieb am Di. 2. Juni 2020 um
00:18:

Did you put a real ads and disabled the variable isReal?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kloder-games/godot-admob/issues/67#issuecomment-637155217,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALKQNW2UJXF72BSTVTW3MHLRUQSLXANCNFSM4GWHBSWQ
.

Im using a new version of it, it seems to work, Thank you
B El<EMAIL_ADDRESS>schrieb am Di. 2. Juni 2020 um 10:27:

the real ads are active. I set the isReal boolean on true and deleted the
Strings (the test ad ids) and added my own ids . Am I supposed to disable
the Variable isReal ?
Gustavo Maciel<EMAIL_ADDRESS>schrieb am Di. 2. Juni 2020 um
00:18:

Did you put a real ads and disabled the variable isReal?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kloder-games/godot-admob/issues/67#issuecomment-637155217,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALKQNW2UJXF72BSTVTW3MHLRUQSLXANCNFSM4GWHBSWQ
.


tentative integration test for nodejs backend environment

Will create follow up PR to ensure all documentation reflects changes

Merging this for now with two stipulations:

documentation will be updated pending conversation with Sean
another iteration will be completed enabling the node application test to use Jest and the web build test to use cypress


No flask inregration?

Would be nice to use with existing flask apps that need additional services.

Last time I looked at this, getting flask run inside of sparts seemed to be impossible due to implementation specifics/limitations of the main flask engine.
There's also so many different ways of running flask (debug server, uwsgi, etc), that I'm not sure which would be best to support.
Another option might be to try to run sparts inside of flask instead of vice versa, but I'm not sure what that might look like.
Do you have any thoughts on what an ideal API would look like for integrating sparts with flask for you?


Question about delay

Im having a problem with what seems to be de-sync? I start the timer on time but 2-3 waves in the timing is always a few hits early. It seems to be a problem only on my PC atm.. I tried using the plugin on my laptop and it works great. Anyone know a possible solution to what it may have been caused by and how to fix it?

It is usually caused by server lag. I have added buttons to adjust the timer in a new version of the plugin. This should be available in-game in a couple of days.


Prevent interactions and referrals without a company linking as an activity

Description of change
When relating interactions to an activity, there are some interactions on staging which have no company and this causes a crash. This PR prevents interactions or referrals without a company to be related to a company activity.
Checklist

[ ] Has this branch been rebased on top of the current main branch?
[ ] Is the CircleCI build passing?

General points

Other things to check

Make sure fixtures/test_data.yaml is maintained when updating models
Consider the admin site when making changes to models
Use select-/prefetch-related field lists in views and search apps, and update them when fields are added
Make sure the README is updated e.g. when adding new environment variables

See docs/CONTRIBUTING.md for more guidelines.

Should there be a failing test for this?

Should there be a failing test for this?

Yeah I should add one, there shouldn't be any interactions or referrals without a company but on staging there are 2 interactions without a company from 2021.


feat: add alwaysOnTop menu

This menu toggle enable/disable alwaysOnTop.

fix #39

I renamed the branch from master to main, which unfortunately, automatically closed this PR.
If Quick Camera is placed in borderless mode, it will automatically switch to "always on top". Do you have a use case for needing always on top when not in borderless mode? Thanks!


[IOTDB-2638][Metric] add features into metric module.

See more details: https://issues.apache.org/jira/browse/IOTDB-2638

Coverage decreased (-0.03%) to 67.71% when pulling 9d2bbfcbe586114a0816d6982d60387294fd4737 on stormbroken:feature/metric_interface_update into 9d33535d23fbb98b6c41e7eaa52a7bc2e81b435e on apache:master.


test: add test in case of add test task on org without training

/e2e

End to end tests: :hourglass_flowing_sand: STARTING


Add authorization services support to Gatekeeper

What:
As a Gatekeeper developer, I want to enable a policy enforcer to my service or application so that I can enforce access to protected resources using fine-grained permissions.
This issue is all about enabling policy-enforcer configuration, as we have in other adapters, to Gatekeeper.
Reference:

KEYCLOAK-7502 - Add authorization services support to Gatekeeper

i checked how it is done for application adapters and i think it is quite static, ideally there would be some context providers which would automatically generate body of request for permission ticket from parameters of incoming HTTP request e.g. keycloak_http_method_context_provider() would generate permission ticket request with this body:
'[
  {
    "resource_id": "{resource_id}",
    "resource_scopes": [
      "GET"
    ]
  }
]'

as request_id is passed in permission ticket it should be gathered from keycloak server e.g. by this request:
To query resources given an uri, send an HTTP GET request as follows:

http://${host}:${port}/auth/realms/${realm_name}/authz/protection/resource_set?uri=/api/alice

and if these providers would be stackable/additive e.g. something like:
authorized_context.add(keycloak_http_method_context_provider)
authorized_context.add(keycloak_http_headers_context_provider)
keycloak.create_permission_ticket(authorized_context)
or something like that
etc...

There's one thing adding support for UMA 2, but not sure that fully covers Keycloak authorization services.
We also probably want the authz layer to be extensible I can see at least the following options in the future:

Basic DSL built into Louketo
WebHook - some clean/simple webhook API to plugin something external
UMA 2
Custom Go
Keycloak Authorization Services - not sure if this is needed or we can do what we need to through UMA 2, or even supporting the Louketo defined WebHook API in Keycloak may be better

https://github.com/grepplabs/kafka-proxy, maybe this could be inspiration

@stianst I would not consider UMA 2 for now. It solves a very specific set of use cases that are beyond API/HTTP security.
Regarding Authorization Services, it is all about implementing a PEP that talks with Keycloak (PDP) to obtain a decision. Not different than what we have in our Java Adapters or NodeJS. In fact, the WebHook can also be used to integrate with Authorization Services, as long as we provide the endpoint OOTB leveraging our policy model to process the payload.
The DSL idea is interesting and focusing on API/HTTP security should allow us to provide a more specific DSL for HTTP-based resources. However, that would also mean implementing some components to understand this DSL and process the conditions/policies (access control mechanisms) accordingly.
We could even have some form of switch that decides whether or not a decision should be made locally or externally by a PDP/Webhook. The use case would be that for certain HTTP resources you may want some basic role, or group, or user access enforcement. But for others, you may want more rich and contextual access enforcement.
There is also the possibility to have Louketo fetch policies using the DSL from some external service. If in Kubernetes, that would help Louketo obtain these policies from CRDs. Or just allow fetching these DSLs from some external service. That would make things more dynamic too and leverage centralized authorization to some degree as these policies could be managed from a central location.

To plugin Authorization Services with Gatekeeper we should define the simpler Authorization Webhook API that is then implemented by both Keycloak and Gatekeeper. This API will be simple enough that anyone can implement it in a different PDP, which makes it usable outside of just Gatekeeper.
For the DSL (although thinking about it I would rather call it a Policy Config/Definition, as we don't want to make a full blown DSL) approach it really is about Gatekeeper supporting what it does now, but defined in the standard Policy Config file that Keycloak is also able to read. So not much more than mapping the config file to what it already implements.

Is there anybody working on that feature right now ?

Is this feature available in keycloak gatekeeper?

@pawneeranger nobody is working on this feature, please see https://github.com/louketo/louketo-proxy/issues/683
@sachinjohar2 no, this feature is not available on Gatekeeper and we don't have any plans to implement it.


On the horizontalpartitioning.png picture we have PowerBI word underlined by spellchecker

On the horizontalpartitioning.png picture we PowerBI underlined by spellchecker - does not look nice, should be removed

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

ID: 36c7839a-d1c6-d5a8-fffd-0c3d81c2bc67
Version Independent ID: c26ce8c9-1b8f-fbcf-70d8-b64c20ff9ca8
Content: Reporting across scaled-out cloud databases
Content Source: articles/sql-database/sql-database-elastic-query-horizontal-partitioning.md
Service: sql-database
GitHub Login: @MladjoA
Microsoft Alias: mlandzic

@mikerodionov Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

Document updated, will be published later today. Thanks @mikerodionov for the feedback.

Fixed and merged

#please-close

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


JSX: TS1127 for <>#</>

TypeScript Version:  Version 2.8.0-dev.20180217

Search Terms: TS1127, jsx invalid character
Code
const elm = (<>#</>);

Expected behavior:
Compile with no error.
Actual behavior:
test.tsx(1,16): error TS1127: Invalid character.
Related Issues: #4207
Thanks.

I reproduced this with <>#</>, and also with emojis like <>🙂</>.

also note that this seems to be specific to fragment syntax, since <span>🙂</span> works fine.


Surelog should error out on undefined hierarchical path

#3110 puts the final nail on this long project. Surelog now errors out on hierarchical references that cannot be bound to objects.
We checked 1000s of designs with this feature. If you see a bogus error it means we don't model a construct correctly in UHDM, file an issue.


Move travis build to public

The build is currently in the organization and even I don't have permissions to see it.

same for me. @konsti can you transfer the repo to travis-ci.org?

I don't have access for the travis repo as well but now for this github repo. I will just delete the travis integration and set up a new public one @fspoettel


Don't consider tag refs as valid for branch name

name-rev does not discriminate so we need to explicitly ensure that tags won't be taken into account

ping LG-TM


[Dashboard]Fixed dashboard start failed

Master Issue: https://github.com/apache/pulsar/issues/5847
Motivation
Installation error of dashboard database postgres 11 resulted in startup failure.
Modifications

Update apachepulsar/pulsar-dashboard:2.5.1 image https://hub.docker.com/layers/apachepulsar/pulsar-dashboard/2.5.1/images/sha256-61b47a7302639aba1357d09ca69a842c4a67bff38b230753d6bd638df0461c6b?context=explore
Update Docker file for fix postgresql version 11.

Verifying this change
Local test pass

/pulsarbot run-failure-checks


flink-packt image no longer available

@je-ik , I am getting an error when trying to install flink
flink-jobmanager-7fcdd685cf-97rzp    0/1     ErrImageNeverPull   0          9m4s
flink-taskmanager-85db6c9b98-xpj8g   0/2     ErrImageNeverPull   0          9m4s

I cannot seem to find the image flink-packt anywhere. I had a similar problem with kafka-packt also but was able to find an alternative for that

These images are built using the scripts provided in env/ directory.
You can build everything using ./env/start-all.sh, or by the individual scripts ./env/install-flink.sh and ./env/install-kafka.sh. IIRC this should be described on the beginning of each chapter. Let me know if it works for you.


feature: add a way to fetch template from the remote server

ansistrano already allow to use templates with ansistrano_parameters_templates during the parameters step.
Those template are locally from the host who run the playbook.
This commit add a new step before parameters to be able to fetch template on the remote host fetch-remote-templates.
The usecase here is : Our customer provide the template of the configuration file in the git of his application in jinja format.
With this feature we are able to use the configuration template provided in the application code and render it.
Ex of usage :
      ansistrano_fetch_remote_templates:
        - dest: /tmp/play-application.conf.j2
          src: "{{ansistrano_release_path.stdout}}/conf/application.conf.j2"

      # Then use this fetched template to configure the server
      ansistrano_parameters_templates:
        - src: /tmp/play-application.conf.j2
          dest: "{{ansistrano_release_path.stdout}}/conf/application.conf"

done


Add recommended config to the ESLint plugin

Fix #691
Changes

Add recommended config to the ESLint plugin
Update the ESLint plugin document

Codecov Report

Merging #698 (e7f24c6) into main (82b3142) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              main      #698    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           66        56    -10
  Lines         2861      1970   -891
  Branches       475       322   -153
==========================================
- Hits          2861      1970   -891

Impacted Files
Coverage Δ

...ges/eslint-plugin-redos/src/rules/no-vulnerable.ts

packages/recheck/src/browser.ts

packages/recheck/src/lib/env.ts

packages/recheck/src/lib/worker-pool.ts

packages/recheck/src/lib/exe.ts

packages/recheck/src/lib/pure.ts

packages/recheck/src/main.ts

packages/recheck/src/lib/java.ts

packages/recheck/src/lib/native.ts

packages/recheck/src/lib/agent.ts

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


OWASP depdency check fails npm audit

There is a vulnerability in the "got" library that allows a redirect to a unix socket. This is failing NPM audit and other security scanning tools. The dependency comes from the "download" library, which hasn't been updated in two years.
It is possible to do npm audit --omit=dev, but this is slightly skirting the issue! Explicitly specifying the version of download or got doesn't seem to fix this issue, since NPM still looks at the dependency versions in the parent package.
Is it possible to get a fix that removes the dependency on the "download" library? (Or alternatively, get an update into the "download" library)?
# npm audit report

got  <11.8.5
Severity: moderate
Got allows a redirect to a UNIX socket - https://github.com/advisories/GHSA-pfrx-2q88-qq97
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/download/node_modules/got
  download  >=4.0.0
  Depends on vulnerable versions of got
  node_modules/download
    owasp-dependency-check  >=0.0.4
    Depends on vulnerable versions of download
    node_modules/owasp-dependency-check

I replaced the library with a different one. The new version 0.0.19 shoudl now be with 0 vulnerabilities.


CUDA out of memory

Hi~ I'm using this repo to test my custom video, but get CUDA out of memory error. The resolution of my video is 4k, video length is 60s at 25FPS. Which settings could I change to make it possible to run this excellent work? Many thanks~~

XMem scales well to long video but unfortunately not to high-resolution. The easiest way is to infer at a lower resolution and upsample the masks after.

Hi, has anyone detected if this loses any data / output performance during the downsample and upsampling process. We're working with 1920x1080 and want to scale to use 4K data.  I'd suspect this wouldn't scale too well...


The real-time inference can't convert the first beginning voice well and lose a lot of linguistic information

This is an observation.

A tricky is to insert a zero buffer array into the recored array.

However, this trick may not work well as I expected.


🚀 [Feature] JWT-Token only last is valid

🚀 Feature

Make sure that only the last  JWT Token generated by the Server for a given Session/User is valid. All others are no longer accepted.
Suggestion:

Store the last Session for a User in Memory https://dev.to/franciscomendes10866/simple-in-memory-cache-in-node-js-gl4
Generate the secret for the JWT on startup to invalidate all JWTs generated by the last instance of the server.

Implications:

If the Server restarts/crashes all previous active Sessions become invalid
Since we generate the secret we rely on the random generator we use - its security becomes our security.

The suggested solution might impact our scalability! Please investigate


Code Availability Inquiry for Experiments: MCM with CoOp/CoCoOp and LSN's Positive Prompt with LoCoOp

Hello, I'm currently reading through papers and accompanying code. First of all, thank you for helping me conduct some great experiments!
However, after reviewing the citation you provided, I would like to inquire about whether there are separate codes available for conducting experiments with 'MCM+CoOp' or 'MCM+CoCoOp' as mentioned in 1. Additionally, I came across an openreview stating that using LSN's positive prompt with LoCoOp yielded good results ('Learn to say no + LoCoOP' as in 2). Is there code available for the experiments conducted in these cases?
Thank you.

Subsequent work for Parameter-Efficient OOD detection methods
Parameter-efficient OOD detection is a promising research direction, and LoCoOp can be a baseline approach for this field.
To catch up with this field, we summarized the subsequent work for CLIP-based efficient OOD detection methods. (Last update: 2024.04.14)
, code
This is the journal extension version of MCM. This paper examines the effectiveness of MCM with vanilla prompt learning methods (e.g., CoOp). To implement this, you can utilize our LoCoOp's code following https://github.com/AtsuMiyai/LoCoOp/issues/2#issuecomment-1923390359.
LSN learns negative prompts for OOD detection, which is an orthogonal approach to LoCoOp and can be combined with LoCoOp.
, code
IDPrompt leverages ID-like outliers in the ID image to further leverage the capabilities of CLIP for OOD detection, which is a similar concept to LoCoOp.
, code
LSA first tackled full-spectrum OOD detection in the context of CLIP-based parameter-efficient OOD detection.
, code
NegPrompt learns a set of negative prompts with only ID data. Also, this paper tackled a novel promising problem setting called Open-vocabulary OOD detection.
If I missed some work, feel free to contact me by opening an issue!

@JB0527 Hi! Thanks for your interest!
My answers are as follows:

For MCM+CoOp,  you can comment out  like #  + self.lambda_value * loss_en in https://github.com/AtsuMiyai/LoCoOp/blob/master/trainers/locoop.py#L299 (or in https://github.com/AtsuMiyai/LoCoOp/blob/master/trainers/locoop.py#L317)
For  'MCM+CoCoOp', we do not provide the code, but I think you can easily write the code for MCM+CoCoOp because MCM is really simple.

We do not provide the code with LoCoOp + LSN. If you want it, could you please contact the author of LSN?  Also, since LoCoOp+LSN is a rebuttal experiment, I think you do not need to include it as a comparison method in your paper.

Did you solve this problem?
I will close this issue, but if you have another question, please feel free to open this!


[feature request] support sync mode

Hi, thanks for your helpful module! This is a feature request. Can we have sync mode so I can correctly block my other code executions while using? For example:
const treeKill = require('tree-kill').sync;

console.log('try kill', pid);
treeKill(pid);
console.log(pid, 'killed')

I switched to this:
https://github.com/nktnet1/kill-sync


Constructor parameters in wrong order after setting default value with transform

I'm trying to use a transform to set a default value to a property in the Form Recognizer swagger (link here). Here's the property:
"rowSpan": {
  "description": "Number of rows spanned by this cell.",
  "type": "integer",
  "format": "int32",
  "minimum": 1,
  "default": 1
}

And here's the transform. Note that I'm redefining the required property to include rowSpan:
directive:
  from: swagger-document
  where: $.definitions.DocumentTableCell
  transform: >
    $.required = ["rowIndex", "columnIndex", "content", "spans", "rowSpan"];
    $.properties.rowSpan["x-ms-client-default"] = 1;

The generated code sets the default as specified, but it doesn't place the optional rowSpan parameter at the end of the constructor signature, making the build fail:
public partial class DocumentTableCell
{
    // Places rowSpan (optional) before required parameters, so build fails.
    internal DocumentTableCell(int rowIndex, int columnIndex, int rowSpan = 1, string content, IEnumerable<DocumentSpan> spans)
    {
        ...
    }
}

cc @ShivangiReja


Code Example gives TS Errors

When using the example
const app = new Elysia()
    .use(bearer())
    .get('/', () => 'Bearer')
    .get('/sign', ({ bearer }) => bearer, {
        beforeHandle({ bearer, set }) {
            if (!bearer) {
                set.status = 400
                set.headers[
                    'WWW-Authenticate'
                ] = `Bearer realm='sign', error="invalid_request"`

                return 'Unauthorized'
            }
        }
    })
    .listen(8080)

You get the following TS error on line 4
Property 'bearer' does not exist on type 'Context<{ body: unknown; params: Record<never, string>; query: undefined; headers: undefined; response: unknown; }, {}> & {}'.ts(2339)

The code itself works great, there is the bearer object being made available to the context passed to the closure, so you can do validation with it. However the api decorating feature doesn't seem to be working with the typechecker quite as it should.

! useless comment !
I had the same problem but with html plugin

same same

upgrading fix this for me

Confirmed, fixed!


[WW-5070] Adds more sophisticated logic to search for the Root

Resolves WW-5070

Coverage increased (+0.003%) to 49.184% when pulling 19c71ff1537711ce0b802545e8327d564ee8f769 on WW-5070-root-action-model into 3ccbcc6f9c9f84f6f1e5644e1e266754d88a8017 on master.


rtupgrade dont work

For more information go to:
https://github.com/arakasi72/rtinst/blob/master/README.md
Select from the following options:
1.) rtsetup - update the rtinst scripts
2.) rtadduser - add new user
3.) rtremove - delete a user
4.) rtdload enable - enable https downloads
5.) rtupdate - up/down-grade rtorent/libtorrent
6.) rutupgrade - upgrade to latest version of RuTorrent
7.) rtwebmin - install webmin
8.) rtletsencrypt - install Lets Encypt ssl certificate
9.) rtsslcert - generate new self signed ssl certificate
Enter option or q to quit
6
Installed version: 3.10
Latest version: 4.0
Choose which build to install, (if you are not sure select 1)

Use the latest stable release
Use the most recent master build
Select 1 or 2: 1

Installing version 4.0 of rutorrent
Your existing rutorrent folder will be moved to /root/rutorrent_old
If you are not happy with the upgrade you can restore the old version, by rerunning this script
Installing Rutorrent (v4.0-stable)
Setting permissions, Starting services
Installation of RuTorrent complete
plex@webservices:~$ rt
rtorrent is running
Result after upgrade:
[19.01.2023 16:01:16] WebUI started.
[19.01.2023 16:01:16] Bad response from server: (200 [parsererror,getplugins]) TypeError: theWebUI.createSelectableLabelElement is not a function
[19.01.2023 16:01:16] Error downloading files. Make sure autodl-irssi is started and configured properly (eg. password, port number): Could not get files listing: AJAX error, status: error

autodl-irssi doesnt work with rutorrent v4 or above delete the plugin or stay on v3.10


Change to add additional properties to hbase-site.xml to enable customizations

Change to enable customizing HBase bucketcache bucket sizes -> Property added hbase.bucketcache.bucket.sizes.
hbase.fs.tmp.dir property added to handle situations descripbed in HBASE-15129

Hi Biju, thanks for this PR. Is there any value to force users to learn a "chef-bach" vocabulary of HBase tunings? Ideally, I would prefer us to use the same names as hbase-site.xml to avoid confusion renaming settings already agreed upon by the HBase community.

Updated PR to set hbase.fs.tmp.dir property directly in site_xml hash.
site_xml['hbase.fs.tmp.dir'] = "/user/${user.name}/hbase-staging"

Hi @cbaenziger , @leochen4891  can you have a look and let me know if further changes are required.

Thanks @leochen4891 . Updated the PR.

One nit but it looks like rubocop/foodcritic best practices weren't followed for the new lines? (Particularly quotes look like they'd flag.)


Ability to filter response to API /footprint by subscription id (s)

When making a call /footprint in the API I would like to be able to send one or more Azure subscription ids as a parameter.
The response from /footprint should only contain results for the subscriptions specified in the input parameter
Feature Suggestion
I work in a company that has a lot of Azure resources. I have an app registration that has access to 268 subscriptions.
My goal is to create a widget for Azure Devops that will display a graph of cloud carbon footprint.
The widget would call the Cloud Carbon Footprint API to get the data.
The problem with the current API is that it is really hard to start it with 268 subscriptions (there is a limit in Azure on how many requests you can make to get data) and after the timeout the request limit is hit again. The other problem is that it would take a long time to return the data.
Describe the solution you'd like
I would like a parameter to the /footprint-endpoint that contains the subscription ids that we want data from.
I could then specify the subscriptions when calling the API from the widget. After that the widget could display the cloud carbon footprint from the subscriptions that are relevant for a team.
As a result a team that wants to know the cloud carbon footprint for their subscriptions could add the widget to a dashboard in their Azure Devops project (and specifying the subscription ids that are relevant for that team).
Describe alternatives you've considered
It would be possible to create an app registration that only has access to a limited number of subscriptions and then create an API instance that uses that app registration and then call that API instance from the widget.
That solution would not be optiomal for me since we have many teams and if all teams have to create Azure resources to be able to see the Cloud Carbon Footprint for their subscriptions that would take a lot of effort but also increase the Cloud Carbon Footprint that we would like to reduce.
Additional context
Possible Implementation

Add subscriptionIds to FootprintEstimatesRawRequest (could be comma separated)
Add subscriptionIds to EstimationRequest
Modify App.ts so that subscriptionIds are sent to AzureAccount.getDataFromConsumptionManagement
Modify getDataFromConsumptionManagement so that we only call getDataForSubscription if the subscription is in subscriptionIds

I realize that this suggestion will only work for Azure resources.

HI @nilshedstrom.
Thanks for the suggestions - we think this option to specify subscription (or account, project) is a good one. We would ideally have this option for each of the cloud providers supported by Cloud Carbon Footprint. We will create an issue for this and comment here with it. You're more than welcome to work on this if you have capacity.
Also worth noting - we are currently prioritizing improvements on the performance of Cloud Carbon Footprint by adding an option to use mongoDB instead of a cache file. Perhaps this could also work to solve your root issue of accessing the estimates.

Hi!
I will start working on this (Azure resources to start with). Is it ok if I name the parameter to the API subscriptionIds or do you have another name that could work for multiple providers in the future?


feat: Probot v11

ah tests are misleading as they are mocked :sweat:

ah tests are misleading as they are mocked 😓

yeah that is a problem I've seen in many apps. Only mock http requests, never mock the methods.
Anyway, I'd like to update the adapter based on the latest Probot version, so we can use it in the example:
https://github.com/probot/example-github-action/blob/73ba8af469c1ebb4ce7574ed2c14bdaad9b781e4/index.js
I think the example code should look something like this
const { runProbot, createProbot } = require("@probot/actions-adapter")
const app = require('./app')

runProbot(app, { probot: createProbot })

createProbot should preset the token as it's currently done in the example, and runProbot should do the checks for the environment variables.
Would you like to give that a try?
Once we update the adapter, we could also create an official probot/action that you can use directly in your workflows, with the path to your app.js Probot app function.

something like this: 4ade1d3

Noted in testing that createProbot does not log properly to github actions 😓

something like this: 4ade1d3

looks good 👍🏼

Right the default log level in createProbot is warn.

Maybe it would make sense to change the default logging behavior for GitHub Actions, utilizing @actions/core. Probot is using pino for logging, I think the best approach would be to create a transport for github actions, unless one already exists? The benefit is that the logging will be better integrated with GitHub Action's built-in feature.
What do you think? Would you like to give that a go?

superseded by https://github.com/probot/adapter-github-actions/pull/19

Thanks for taking over 😅


Fresh install - /var/log/ permission denied

Apr 07 12:21:08 cs_devel sphinxsearch[2786]: Starting sphinxsearch: [Tue Apr  7 12:21:08.449 2020] [2795] using config file '/etc/sphinxsearch/sphinx.conf'...
Apr 07 12:21:08 cs_devel sphinxsearch[2786]: [Tue Apr  7 12:21:08.461 2020] [2795] FATAL: failed to open log file '/var/log/sphinxsearch/searchd.log': Permission denied

uname -a
Linux 4.15.0-96-generic #97-Ubuntu SMP Wed Apr 1 03:25:46 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Could you please provide me additional info about your installation:

apt-cache show sphinxsearch
lsb_release -a
ls -la /var/log/sphinxsearch/

Thank you in advance

I  think this is my fault. I ran searchd as user once and it created root
owned files.
wt., 7 kwi 2020, 15:41 użytkownik Semyon Deviatkin<EMAIL_ADDRESS>napisał:

Could you please provide me additional info about your installation:

apt-cache show sphinxsearch
lsb_release -a
ls -la /var/log/sphinxsearch/

Thank you in advance
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/devjatkin/sphinx/issues/18#issuecomment-610393552,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABK7F6LV5GO23XBPKIWQQDDRLMUQNANCNFSM4MDCFYAA
.


imjp19

[x] I have read the Code Of Conduct.

[X] I have followed all the steps of submission properly

Hello, @imjp19 ,
Your pull request has some merge conflict, kindly pull the latest version of the repository using git pull and update your pull request.


Differentiation of Automate and Automate HA in CLI Page 3rd draft

:nut_and_bolt: Description: What code changed, and why?

:chains: Related Resources
:+1: Definition of Done
:athletic_shoe: How to Build and Test the Change
:white_check_mark: Checklist
All PRs must tick these:

[ ] I have read the CONTRIBUTING document.
[ ] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[ ] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[ ] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[ ] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[ ] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[ ] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

Development PR: https://github.com/chef/automate/pull/7669


Autofocus in Sasmung Galaxy S4 mini (GT-I9195) not working.

Hi,
I have successfully run the autofocus of the CameraDemo in all my devices and caught the callback, but in the Samsung Galaxy S4 mini (GT-I9195) running Android version 4.2.2 is not working, (this device has autofocus, so it should work).
After the call to autoFocus() in CameraView all my other devices receive the onAutoFocus() callback. However, in the S4 mini it never happens.
I have tried to figure out why this happens, but all my efforts were unsuccessful.
Other users with this model are having the same problem so it is not specific to my device.
If you could tell me why is this happening and how can I fix it, I would really appreciate it.
I would be glad to help if you require further information.
Thank you in advance.

Try calling _camera.cancelAutoFocus() before calling _camera.autoFocus()


Instance: Copy snapshot expiry date

Ensure that snapshot expiry date is maintained when copying instance locally or remotely.
Fixes #8283

Testsuite passed

Testsuite passed


feat: ask user for email on first run of Kurtosis

https://github.com/kurtosis-tech/kurtosis/assets/6670312/d371e60f-17b7-4e11-a29f-3012bfc3b7a0


Example for connecting from Cloud Run to Cloud Sql

It would be nice if there were an example of establishing a connection from a Cloud Run instance to a Cloud sql instance, where it clearly and explicitly describes how the connection url should be formed.

Hi @safield, we currently have documentation here for how to form the a connection URL when connecting with Cloud Run: . We recommend using the Cloud SQL JDBC Socket Factory along with the recommended JDBC driver for your database engine. You also have the option of connecting using a Private IP address (currently in beta), and the steps to configure your application to do that are here. Examples for how to form a connection string given an IP address can be found in the documentation for your driver (for example, the connection info for Postgres is here).
If there's any specific information that's missing in the links above that you're looking for, please reply and we can work on updating our samples.

I'm going to close this then, feel free to reopen.


Add value_type() cast operator to xfunction when all arguments are of type xscalar

Fixes #412

Even when universal functions are only called with scalar arguments, they continue returning a xfunction object.
This preserves the laziness.
When universal functions are called with scalar arguments only, a cast operator to the underlying scalar return type is made available, so that you can assign the value to a scalar.

In this example, close is an xfunction, just like when arguments are arrays

auto close = xt::isclose(1.0, 1.0);

However, it is convertible to bool,  triggering the evaluation, so that you can do

bool close = xt::isclose(1.0, 1.0);

@JohanMabille this should do it.

cc @ukoethe


New page: Something went wrong (confirmation emails)

Need messaging for a new "Something went wrong" page in the event that there's a system error and the email notification wasn't sent to staff and/or the user.
Should be along the lines of "something went wrong, please submit your request again" ...
--
[@zakkain to submit copy for @pcraig3 @timarney]

Here's the possible scenarios that I see:
1 - staff email failed
2 - user response email failed
3 - both staff and user email failed
We'll get back a fairly detailed response from Amazon (I have been able to track done a list showing possible errors as of yet)
There's also the possibility that an email failed but Amazon will retry sending X amount of times (I don't think we can handle this case as there's no time limit on this ... user has left the site).

@timarney so, this page would load after the user hits "Send request" on the final page, and would essentially replace the /confirmation page that loads when everything works properly. Correct?

Elaborating on that: depending on which case Tim outlined above, we could present this as very different levels of severity. If the staff email failed, absolutely we should ask the user to repeat the process and try again.
But if the user confirmation failed while the staff email succeeded, then I would argue we shouldn't ask the user to repeat the process. We shouldn't fire a "Something went wrong" page—we should fire the confirmation page, but with an added piece explaining they won't get a confirmation email (and direct them/include the information to review, so they can write it down).
@emanelfy thoughts?

@emanelfy What do you think of this layout for Case 1 - only user email fails?

obviously a rough mock, I made cutting and pasting screenshots, and not final copy. There's a much better way to handle this layout in design, have any thoughts?

The error feels too buried on this page, could be missed.
Needs to go above "What happens next," reversing the order? @zakkain
Please make a copy of your request -- also feels vague. Perhaps "Please make note" is a more action-oriented way of phrasing that, as in, you're definitely writing this down. (or taking a screenshot or whatever).
I would also put this on a second line, so it doesn't appear to be such a long sentence and potentially could be missed when skimming.
I think we also need to make it super clear somehow that staff have a copy, it's just that users need to note it on their end. That everything else is okay and going according to plan.
Like: "not to worry, IRCC staff have received your request, but..." along these lines?

Update: After discussing w/ Tim, we have a solution that cuts the number of separate error states and reduces user confusion:
Case 1 (user email failed) - point users to a new "confirmation" page. Design and copy on it's way
Case 2 (staff email failed) - In this case, we will change the logic so that if the staff email fails the user email will not send. We'll direct users to the standard "something went wrong" page. This avoids the confusion of users receiving an email, bu being told to ignore it. This will result in more calls.
Case 3 (both fail) - points to the standard "something went wrong" page.
So in effect we only have 1 new error page, not 3., with a bit of smart error handling logic on the back end.
cc @emanelfy @pcraig3 @timarney

Case 1 (user email failed) confirmation page variant:

Changes:

removal of the text below the page title reading "We sent you a confirmation email"
Addition of a "warning" component below the title with the copy: Sorry, something went wrong.   Please make note of your request information:
Addition of the dates table from the Review page below the warning , aligned left to the warning text
2px #e1e1e1 rule indenting the dates section

cc @timarney @emanelfy

Closing this as it's been merged.


chore(conda-build): Fix docker run for conda-build

PR Checklist
Ensure that:

[x] it includes tests.
[x] the tests for your implementation are executed on CI.
[ ] it includes documentation
[x] the PR's title is according to the semantic-release pattern.

chore: is used for CI, test, support tasks or any other task that is not user facing task.
feat: is used for a new feature or when a existent one is improved.
fix: is used when a bug is fixed.
breaking change: is used when there is a compability break.

[x] pre-commit hooks were executed locally

RuntimeError: Solver could not find solution.Mamba failed to solve:
 - __glibc >=2.17,<3.0.a0
 - llvmdev 13.0.1.*
 - llvm 13.0.1.*
 - libstdcxx-ng >=10.3.0
 - arrow-cpp 7.*
 - libclang-cpp 13.0.1.*
 - clangdev 13.0.1.*
 - libgcc-ng >=10.3.0
 - lld 13.0.1.*

with channels:
 - local
 - conda-forge
 - conda-forge

The reported errors are:
- Encountered problems while solving:
-   - nothing provides requested __glibc >=2.17,<3.0.a0

there is still a conflict with glibc, but  at least the docker run step is working properly now.


Down/Upload Progress

It's more a question than an issue. Do you know how I would go about getting a idea of how far along a download/upload is?
I'm thinking comparing remote and local file sizes somehow, but any suggestions of how to go about it?
thanks!
Antoine

Hey, @aarlaud. Only speaking as a contributor here, so the owner may have some other things to say.
This is tricker than one might think, and is a consistent problem with http/s libraries as well as request. When directly invoking those libraries, there are ways to accomplish this (sort of messily). However, the SDK obfuscates away the http/s libraries for actually making the request, and opts for use of fs-based file inputs and callbacks rather than streams/emitters (in terms of the actual upload method).
By their nature, callbacks are only called on completion or failure (since they are only called once). To accomplish allowing the SDK consumer to calculate progress, we'd likely have to move to an emitter pattern so that the req/res variables and/or events are exposed. That would definitely be a non-trivial change, and certainly a breaking one (in terms of symver).

Thanks for the response. I think I may have an idea that I need to experiment with, I'll provide update if I can make something useful ;). Please close the issue in the meantime.

Dear @aarlaud,
I will consider it for the next update. Thanks for your suggestion !


[DO NOT REVIEW] Deploy versions 2.0 to 3.9 [latest] + dev

Purpose
Merge this PR when https://github.com/openfun/ralph/pull/483 is accepted but before it is merged.
This will deploy documentation for versions:

2.0
2.1
3.0
3.1
3.2
3.4
3.5
3.6
3.7
3.8
3.9 [latest]
dev (doc that is on master branch)

Proposal

🔥(docs) remove documentation before versioning with mike
Deployed from verions 2.0 to 3.8 with MkDocs 1.5.3 and mike 1.1.2
Deployed 3c6fdb7 to 3.9 [latest] with MkDocs 1.5.3 and mike 1.1.2
Set default version to latest with mike 1.1.2
Deployed ae2cec6 to dev with MkDocs 1.5.3 and mike 1.1.2

Closed as it is deployed


A generic Must function

The MustParse and MustNewUUID functions have been removed since they can be replaced simply using the new function.

    uuid.Must(uuid.Parse(s))
    uuid.Must(uuid.NewUUID())

Create NewRandom func consistent with other version constructors.The New function is kept for convenience and relies on the NewRandom function.

This also fixes a spurious bug in the UnmarshalJSON method that prevented compiling the json.go file.

Would remove json.go from your pull request?  Other than that, it looks pretty good.  (sorry for pulling json.go out from under you, it was not supposed to still be there).

Oops. Of course. I have rebased the changes.

Thanks!


Allow Symfony 6 components

Thanks for your contribution :).

That was quick :-) Would you mind tagging a new release?

I will create the new release later :).

Thank you!

The new released version is created now :).
https://github.com/peter279k/fork-CalendR/releases/tag/3.1.2

Thanks a lot!


fix(notice-templates)!: Re-align with generalization of notice reporter

As of 1 ORT's PlainTextTemplateReporter, formerly NoticeTemplateReporter, no more adds the NOTICE_ prefix to the output filename. Instead, the respective filename of the template, without the .ftl suffix, is used as output filename. So, add the NOTICE_ prefix to the template filenames in order to keep that prefix in output filenames.
While at it turn the whole filename to uppercase for consistency. So, NOTICE_SUMMARY is now output instead of NOTICE_summary.

Not strictly required by this change, but I think it makes sense to also update the ORT version used in this repo to a version that includes the mentioned changes.

Addressed by #99.


Adds mispelled service:bicycle tags

Since version 2.23 iD adds the tag
service:bicycle:bicycle_sales_and_service
instead of the common spelling
service:bicycle:Bicycle_Sales_and_Service
Might be related to https://github.com/openstreetmap/id-tagging-schema/issues/654

The root cause for this is the existence of the (invalid) key service:bicycle:Bicycle_Sales_and_Service which is "promoted" by taginfo because it is a relatively frequently found.
Before iD v2.23, there was a bug which allowed iD to use the upper case letters for the different options of this field. Now, iD converts any upper case letters to lower case spelling, resulting in the addition of the "new" version of this misspelled tag.
I think the best way to approach a fix for this issue is for iD to filter out any taginfo suggestions which don't match a field's spelling requirements.


Update eslint-plugin-jsdoc

Description

Update eslint-plugin-jsdoc to 7.x
fix object types

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Chore, documentation, cleanup

Related Issue

n/a
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation (if required).
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Pull Request Test Coverage Report for Build 332

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

Totals

Change from base Build 317:
0.0%

Covered Lines:
60

Relevant Lines:
60

💛  - Coveralls


Potential regression on global args

Rust Version
rustc 1.22.0-nightly (417c73891 2017-10-05)
Affected Version of clap
clap 2.26.2
Expected Behavior Summary
Using a YAML definition with an arg named quiet, defined as global I'm expecting to be able to use it in any subcommands.
Actual Behavior Summary
The quiet flag are only usable on default command, but disapear in subcommands.
Steps to Reproduce the issue
Here's the yaml
name: Test cli
version: "1.0"
args:
    - quiet:
        short: q
        help: Display only needed output
        global: true
subcommands:
    - auth:
        about: Authenticate command

main.rs
fn main() {
    let yaml = load_yaml!("cli.yml");
    let app = App::from_yaml(yaml);

    let matches = app.get_matches();

    println!("quiet: {:?}", matches.is_present("quiet"));
    println!("quiet_o: {:?}", matches.occurrences_of("quiet"));
}

Debug output
$ ./target/debug/cli
quiet: false
quiet_o: 0

$ ./target/debug/cli -q
quiet: true
quiet_o: 1

$ ./target/debug/cli -q auth
quiet: true
quiet_o: 1

$ ./target/debug/cli auth -q
quiet: false
quiet_o: 0

Thanks for filing this! I'll add it to the list as it might be related to #978  and the associated PR #1060

@Freyskeyd could you test this with the current master branch and see if it's fixed? #1070 just merged which may have helped. If not I can look into it.

@kbknapp I will try it as soon as I can :)

@kbknapp Unfortunately, it's doesn't work with #569ced1f


Prevent duplicate pings

Fixes #1

Add test infrastructure
Refactor to make more testable
Convert to using async/await instead of explicit Promises
Add lots of tests

and of course:

Change to trigger only on pull_request.synchronize
Only ping owners that haven't already been pinged

I'm going to manually test this tomorrow to ensure that it works as expected and I didn't blow anything up. If that goes well, I'll merge this :+1:

Looks like we need to also respond to pull_request.opened. I'm getting on an :airplane: right now though, so I'll have to fix this when I land :grinning:

:zap: nice work!


TabWidget

Hi,
for an application that i'm making i needed a tab widget to manage the user interface, so i have realized it. I think that now it is in a good shape from a functionality point of view, visually it is ugly but at least work correctly. Maybe with some css magic could be better.
Since i saw that there is a tab widget request on the project page, i thought: "why not share?".
Following an example code that i have used during testing:
use orbtk::prelude::*;

fn main() {
      Application::new()
        .window(|ctx| {
            Window::new()
                .title("OrbTk - minimal example")
                .position((100.0, 100.0))
                .size(600.0, 500.0)
                .resizeable(true)
                .child(
                    TabWidget::new()
                    .tab("Tab header 1",TextBlock::new().text("Tab content 1").build(ctx))
                    .tab("Tab header 2",TextBlock::new().text("Tab content 2").build(ctx))
                    .tab("Tab header 3",TextBlock::new().text("Tab content 3").build(ctx))
                    .build(ctx)
                )
                .build(ctx)
        })
        .run();
}

Currently the TabWidget does not share values about tabs because setting them from the outside could lead into inconsistencies because i cannot get some change callbacks that hint the widget to change state. So for now the widget state is managed internally by the TabWidgetState structure.
The widget will not draw everything during the update call, but only what it is changed, so this make it very lightweight.
There are 2 values shared by TabWidget:

close_button(bool): should set the visibility of the close button on the right of the tabs, currently not implemented
spacing(f64): set the spacing between tabs

I'm writing some other widgets that are less fundamental and more specialized. If you want i can also share those (maybe putting in a separated folder inside widget folder called "extra" or something similar?)
Some of those widget already completed:

DiscreteBar: Similar to ProgressBar, but instead of working on 0%-100% work on discrete value that can be setted as "min" and "max", then the current value can be setted with "val". The widget will display the value using Container widgets that look like bars.
Fraction: It is a simple TextBlock that have "val" and "max" property and they are used to display a fraction like "3/10". Since they are managed using properties, setting a shared property will make the widget automatically adjust it's values (i have used it in combination with the DiscreteBar)

Hope it could be useful,
Have a good day

Hi,
it looks awesome so far. I can review it deeply tomorrow. If it is ok for you I can fix the styling after the PR is merged 😉.
You are very welcome to create PRs for the other widgets. A sub module extra would be fine, similar o the behaviors sub module.
Thank very much
Have a nice day too

If it is ok for you I can fix the styling after the PR is merged

Sure, i'm still learning how to use the css theming system properly. Also, looking to how you style the widget can guide me to what is the looking design that Orbtk want to achieve, so in future i could also help with design.
I tried to be more clear as possible on comments, but documenting is definitely not my strong point, so if there is something unclear i can improve it. Also, if there are some functionality that you believe could be added or improved, obviously you can do whatever change you want.

You are very welcome to create PRs for the other widgets. A sub module extra would be fine, similar o the behaviors sub module.

Ok, i will improve the documentation of the widgets and i will send tomorrow.

Sure, i'm still learning how to use the css theming system properly. Also, looking to how you style the widget can guide me to what is the looking design that Orbtk want to achieve, so in future i could also help with design.

In the meanwhile we switched from css to a custom RON based solution. You find a short example on the Readme. I have an initial design created by two of my colleagues. Unfortunately it is not complete and created  with a propitious tool. When I develop a new widget I use the other widgets as orientation for colors and fonts and use the material design guidelines for sizes.

All the problems that you have reported should have been fixed. I Also took the opportunity to enable the possibility to toggle the visibility of the close button. I have prepared a fast example that allow to check this feature:
use orbtk::prelude::*;
use orbtk::behaviors::MouseBehavior;
fn main() {
      Application::new()
        .window(|ctx| {
            let tab_widget = TabWidget::new()
            .close_button(true)
            .tab("Tab header 1",TextBlock::new().text("Tab content 1").build(ctx))
            .tab("Tab header 2",TextBlock::new().text("Tab content 2").build(ctx))
            .tab("Tab header 3",TextBlock::new().text("Tab content 3").build(ctx))
            .build(ctx);

            Window::new()
                .title("OrbTk - minimal example")
                .position((100.0, 100.0))
                .size(600.0, 500.0)
                .resizeable(true)
                .child(
                    MouseBehavior::new()
                    .on_scroll(move|states,_|
                    {
                        let current = states.get::<TabWidgetState>(tab_widget).get_close_button_visibility();
                        states.get_mut::<TabWidgetState>(tab_widget).set_close_button_visibility(!current);
                        true
                    })
                    .child(tab_widget)
                    .build(ctx)
                )
                .build(ctx)
        })
        .run();
}

If you scroll the mouse wheel, the close button visibility will be inverted.
Lastly, to pass the close button visibility property to the TabHeader from the TabWidget, i have needed to add &self to create_tab_header function, so i moved it inside the TabWidget functions (so i have not added the // --- Helpers -- comment).
If there are something else to be fixed (or maybe something that i have missed), please tell me, so i will quickly fix it.
Have a nice day

Done.
After running cargo fmt i have noticed that other files have been changed on the commit. It is normal?

Yes it’s normal if other files where not formatted with cargo fmt. Thank you I will review your changes tonight 🙂.

Now the TabHeaders are not displayed:

I also got this problem but i have fixed it before pushing the commit. Let me check...

For me it's working, i have pushed all the changes, so i should have the same code you are using. It is surely caused by the close button visibility that, for some reason, apply also to the tab header. If you want, i can temporary disable this feature while looking for the root of the problem.

  TabHeader::new()
            .visibility(if self.close_button_visibility {
                Visibility::Visible
            } else {
                Visibility::Collapsed
            })

This could it be?

Yes it is. If I remove this code part the tab headers are displayed.

Yes it is. If I remove this code part the tab headers are displayed.

Yes, that is the problem, i set the wrong property, it should be:
  TabHeader::new()
            .close_button(if self.close_button_visibility {
                Visibility::Visible
            } else {
                Visibility::Collapsed
            })

So this set the visibility on button only, not the whole header.
I'm testing the fix and i will commit when finished.

Done, sorry for the inconvenience.
On my test code was working because there was 1 more line that set the visibility property from the outside of the widget.
I have noticed it because mine window header name was different from your. Sorry again

Hi no problem. Thank you for your work.

Merged ;-). WIll make it a little bit more styleable and adjust the style a little bit ok?

Merged ;-). WIll make it a little bit more styleable and adjust the style a little bit ok?

Of course, it really needs it ^^

@uniformbuffer styling is done.

@uniformbuffer styling is done.

I'm seeing it right now. Wow, that does not even look like the same widget. It's a very good styling job :smile:

Thank you :-). Good styling very good base ;-)


Support for Spark version >= 3.1 and AssertionError query

Hi!
I wanted to confirm if XGBoost supports Spark version 3.1.2. I have been trying to run XGBoost on the latest version of Apache Spark on a dataset > 3TB on a 28 node cluster.
Also, I have been getting the following error and haven't been able to figure out what might be causing this.
21/10/10 10:50:17 INFO XGBoostSpark: starting training with timeout set as 1800000 ms for waiting for resources
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger: Exception in thread Thread-1:
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger: Traceback (most recent call last):
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:   File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:     self.run()
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:   File "/usr/lib/python2.7/threading.py", line 754, in run
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:     self.__target(*self.__args, **self.__kwargs)
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:   File "/tmp/tracker2488390078882395804.py", line 324, in run
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:     self.accept_slaves(nslave)
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:   File "/tmp/tracker2488390078882395804.py", line 268, in accept_slaves
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:     s = SlaveEntry(fd, s_addr)
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:   File "/tmp/tracker2488390078882395804.py", line 64, in __init__
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:     assert magic == kMagic, 'invalid magic number=%d from %s' % (magic, self.host)
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger: AssertionError: invalid magic number=542393671 from <IP_ADDRESS>
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger:
21/10/10 13:44:27 INFO RabitTracker$TrackerProcessLogger: Tracker Process ends with exit code 0

I look forward to hearing back from you. Thank you in advance for your help and time.

For the error, what kind of format would you prefer? Because this setup is done on a CloudLab cluster under our assigned projects and the files are very large, total amounting to more than 3TB of data stored in HDFS.

Could you please let me know what version of Python is the tracker compatible with?


getting messages with huge delay on same region

Hi,
I'm running multiple servers on same datacenter (same region) but getting messages from same channel with different delays.
Channel has 3xxx members.
Channel message sent time: Mon Jan 16 20:25:01 UTC 2023
This is message is received on one of the server: Mon Jan 16 20:25:07 UTC 2023
on same region but different server: Mon Jan 16 20:25:12 UTC 2023
I'm using same tdlib version with same config of server.
What can be the reason of this delay?

This is expected. See https://github.com/tdlib/td/issues/1668#issuecomment-912960761.

Actually it's not expected neither normal situation.
I've no idea why Telegram is providing this kind of bad service to channel subscribers. a new miliseconds may be acceptable but there's 11 seconds which is anormous. In some cases it comes in 30 seconds of delay too.
If Telegram will continue to not to make better this service, I'm sure they will loose crypto currency users..

Everything there works as expected in the best possible way.

I'm saying there's latency up to 30 seconds and you're saying as it's normal as expected?
Waov, congratz..


add Parser helper

add new helper to return instance of CodeIgniter\View\Parser

A few tests would also be superb. :)

@mostafakhudair Are you going to be able to make these changes?


Observers being called before the element is ready or attached.

It seems like the properties are parsed in order and if you have an observer setup on a property which references a property declared after, that property will be undefined.
properties:{
"myObject":{
    "type": Object
    "value": { "someProp": null }
    "observer": "myObjectChanged"
}

"myBoolean":{
    "type": Boolean,
    "value": false

}

}
myObjectChanged: function(_arg1, _arg2){
// This logs as undefined when the observer runs before this element is ready or attached.
console.log( this.myBoolean );

}

Yep. If you depend on multiple properties, you should be using a multi-property observer:
observers: [
   'myObjectChanged(myObject, myBoolean)'
]

That makes sense, however the only reason i went with the other way is because I wanted to see the old and new values on the observer.  So I still feel like this is a bug, unless this was intended behavior.

I think this is the consequence of observers being synchronous. I managed to work around this issue by adding debouncers to most of the observer functions. However, this proved to be a bad practice leading to more unexpected behaviors. IMHO, the observers should stay synchronous, EXCEPT in the case of initialization. In other words, if the element is not ready, the observers should hold off from firing right away.

Any update on how to work around this?

/sub

/sub

/sub

/sub

This has been fixed in Polymer 2: http://jsbin.com/qolotufudu/edit?html,console,output


uncaughtException: TypeError: res.onData is not a function

Im migrating this server configuration to a Socket.io setup in which you can see use uWebSockets.js for the http server under the hood. The commented lines are the previous state (which works perfectly):

`// import express, { Express, Request, Response } from "express";
// import * as http from "http";
import * as uWebSockets from "uWebSockets.js";
import expressify from "uwebsockets-express";
import next, { NextApiHandler } from "next";
import * as socketio from "socket.io";
import SocketManager from "./SocketManager";

import allowCors from "../pages/api/cors";
import dotenv from "dotenv";

dotenv.config();
const port: number = parseInt(process.env.SERVER_PORT || "5000", 10);
const dev: boolean = process.env.NODE_ENV !== "production";
export const nextApp = next({ dev });
const nextHandler: NextApiHandler = nextApp.getRequestHandler();

nextApp.prepare().then(async () => {
  // const app: AppOptions = express();
  const app: uWebSockets.TemplatedApp = uWebSockets.App();
  const myapp = expressify(app);
  // const server: http.Server = http.createServer(app);
  const io: socketio.Server = new socketio.Server({
    cors: {
      origin: "*",
      methods: ["GET", "POST"]
    }
  });

// io.attach(server);
  io.attachApp(app);

  myapp.get("/", async (_, res) => {
    res.send("Running");
  });

  myapp.use("../pages/api/cors", allowCors);
  io.on("connection", SocketManager);

  myapp.all("*", (req: any, res: any) => nextHandler(req, res));

  myapp.listen(port, () => {
    console.log(
      `> Server listening at http://localhost:${port} as ${
        dev ? "development" : process.env.NODE_ENV
      }`
    );
  });
});`

Here is a SocketManager.ts file that contains all the processes undertaken by the socket:
`import * as socketio from "socket.io";

const io: socketio.Server = new socketio.Server({
  cors: {
    origin: "*",
    methods: ["GET", "POST"]
  }
});

export default function SockerManager(socket: socketio.Socket) {
  try {
    socket.emit("me", socket.id);

    socket.on("disconnect", () => {
      socket.broadcast.emit("callEnded");
    });

    socket.on("callUser", ({ userToCall, signalData, from, name }) => {
      io.to(userToCall).emit("callUser", { signal: signalData, from, name });
    });

    socket.on("answerCall", (data) => {
      io.to(data.to).emit("callAccepted", data.signal);
    });
  } catch (ex) {
    console.log(ex);
  }
}`

And this is the CORS middleware:
import Cors from "cors";
import initMiddleware from "../../lib/init-middleware";
import { Request, Response } from "express";

// Initializing the cors middleware
const cors = initMiddleware(
  Cors({
    origin: "*",
    methods: ["GET", "POST", "OPTIONS"],
  })
);

export default async function allowCors(req: Request, res: Response) {
  // Run cors
  await cors(req, res);

  // Rest of the API logic
  res.json({ message: "Cors responding here!" });
}

I can't think of anything that's missing. I've been trying to replace express parameters on the CORS documents for the uWebSockets methods without success. If you have any ideas they will be much appreciated.
Cheers!

Well that's just horrible and you've completely missed the whole point of this project.

Thank you for your reply @alexhultman. Can you point me to where I can find in this project's documentation a way to implement uwebsockets with NEXTJS. Now that you have seen my code, do you think there is something in your docs I can use to translate it?

Just implement the uwebsocketjs as it own server. Maybe don’t use nextjs
api. Just use nextjs as static web. Then deploy the server with docker. I
haven’t done it. Just think that might be what Alex mean.
On Tue, 18 Jan 2022 at 23:35, Nicolás Guasca @.***>
wrote:

Thank you for your reply @alexhultman https://github.com/alexhultman.
Can you point me to where I can find in this project's documentation a way
to implement uwebsockets with NEXTJS. Now that you have seen my code, do
you think there is something in your docs I can use to translate it?
—
Reply to this email directly, view it on GitHub
https://github.com/uNetworking/uWebSockets.js/issues/687#issuecomment-1015418490,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIWOHAZ27ITOCHDLRK33BTTUWVUDPANCNFSM5MHC6QVQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
James Nguyen
Master of Information Technology – UQ
M:<PHONE_NUMBER>
@.***> http://www.linkedin.com/in/jamesnguyen868

Express and Socket IO are the worst two projects ever created by man. So shoehorning uws into both of those is completely pointless and circumvents the entire point of this project

@nicolasguasca1 You are taking 3 things that are incompatible with uWebSockets.js (Express,js, Socket.io, Next.js) and trying to combine them all together. If you want best performance you should get rid of those and code server with uWebsockets.js, it's not hard. But for this issue you are trying to use that uwebsockets-express library so really you should be opening issue over there not here. But you can see there is no activity over there because there is no point to that library because they are not compatible and you lose performance

Thank you so much @e3dio @alexhultman for your comments. I spent my day checking at the uWebSockets.js docs and I want to start with the following:
import * as uWebSockets from "uWebSockets.js";

dotenv.config();
const port: number = parseInt(process.env.SERVER_PORT || "5000", 10);
const dev: boolean = process.env.NODE_ENV !== "production";

const app = uWebSockets
  .App()
  .ws("http://localhost", {
    idleTimeout: 30,
    maxBackpressure: 1024,
    maxPayloadLength: 512,
    compression: uWebSockets.DEDICATED_COMPRESSOR_3KB,
    open: (ws) => {
      let ip = ws.getRemoteAddressAsText();
      console.log("Connection opened and started from" + ip);
    },
    close: (ws, code, reason) => {
      console.log("Connection forcefully closed: " + code + " " + reason);
    },
    message: (ws, message, isBinary) => {
      console.log("Message: " + message);
      let ok = ws.send(message, isBinary, true);
    }
  })
  .get("/", (res, req) => {
    res
      .writeStatus("200 OK")
      .writeHeader("Access-Control-Allow-Origin", "*")
      .end("Running awesome!");
  })
  .any("/*", (res, req) => {
    res.end("Nothing to see here!");
  })
  .listen(port, (listenSocket) => {
    if (listenSocket) {
      console.log(
        `> Server listening at http://localhost:${port} as ${
          dev ? "development" : process.env.NODE_ENV
        }`
      );
    }
  });

However, when using nodemon to restart the server it stands by without response:
[nodemon] starting `ts-node --project tsconfig.server.json server/index.ts index.js` [nodemon] spawning [nodemon] child pid: 58621 [nodemon] clean exit - waiting for changes before restart
Is there anything else I might be missing to have a working socket? The next step would be to deploy this socket on docker or something like that which I can use to feed my front-end, am I right? I'm trying to build a video conferencing meet/zoom like prototype so all the information you can provide will be much appreciated.

Looks like it did not listen to the port, it did not log the "Server listening" so make sure it is listening to a valid open port, log the port before listening and check if anything else using port. This is not related to the original issue so you should close this issue and open Discussion if you have Q&A questions

I've checked an implementation to modify my code. Thank you all for the guidance.  Although it was challenging to get into the docs and try to change the chip to uWebSockets, it was nice to see the interest to help and bring solutions to the table. I personally admire projects for the people behind them rather than the mere technology.
Keep up with the great work!


Change filesystem paths to monospace in Minikube setup guide

Signed-off-by: Jubayer Abdullah Joy<EMAIL_ADDRESS>PR discription

[x] Changed  filesystem paths to monospace in /docs/setup/learning-environment/minikube.md mounted host folder section

This PR fixes #18710
Note for Reviewer:

Used monospace style for filesystem path for https://kubernetes.io/docs/setup/learning-environment/minikube/#mounted-host-folders. Changed two rightmost columns in the table -

HostFolder
VM

/assign @sftim

/lgtm
/approve
Thank you @JubayerJoy!


Make Makefile BSD-Make compatible

Issue was that BSD-Make doesn't support the $(shell ..) syntax as this
is a GNU extension. Fix is to define the LIBS variable in a way that it
expands to an inline call to pkg-config instead of expansion in the
variable itself.
Tested on FreeBSD 12.1-RELEASE amd64

@herrhotzenplotz looks good to me! Thank you very much for the contribution! :+1:


Brave browser won't start in Win 8.1 x64

Brave browser version:  brave 1.32.106-78
OS: Windows 8.1 x64
The error:

Does brave-beta work?

There's probably a wrong exe path in 'brave' shortcut created in C:\Users\USERNAME\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Scoop Apps. It was pointed to brave.exe instead of brave-portable.exe

The manifest has the correct location for shortcut https://github.com/ScoopInstaller/Extras/blob/297854268ca528a08bb516244f4b1c6376b71a29/bucket/brave.json#L23
Run scoop reset brave to recreate shim and shortcut

Did it already before posting the issue, yet something changed the target in the shortcut


[TikTok] Iterate through app versions

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Explanation of your pull request in arbitrary form goes here. Please make sure the description explains the purpose and effect of your pull request and is worded well enough to be understood. Provide as much context and examples as possible.
aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
Should handle cases where random versions randomly get blocked

they blocked all versions listed here

As per https://github.com/yt-dlp/yt-dlp/issues/2476 :
Where are the app and app manifest version numbers coming from?
If I knew where they were coming from, then I would be tempted to automatically update the version numbers (as needed) with whatever the current, working versions are (as per the current TikTok download request).
That approach has to be better than manually hard-coding all known version numbers and then randomly selecting one and/or iterating through them until one works.
I tried finding the version numbers with the Facebook crawler UA, but no joy. Maybe a mobile UA would reveal these numbers?

Call me crazy, but is this not the app version here?:

Also, it seems the app manifest version numbers are a direct extension of the app numbers themselves, e.g.:
_APP_VERSIONS = [('20.9.3', '293'), ('20.4.3', '243'), ('20.2.1', '221'), ('20.1.0', '210'), ('20.0.0', '200')]
So, I wonder if 22.9.5 and 2295 would work? When I have some more time, I will try it.
In general, if the TikTok app version is published on any viable, reliable/credible site, then we could potentially scrape the version number to dynamically generate the correct version numbers as needed.

As per #2476 :
Where are the app and app manifest version numbers coming from? Are you manually pulling them from the mobile app and/or app URLs?
If I knew where they were coming from, then I would be tempted to automatically update the version numbers (as needed) with whatever the current, working versions are (as per the current TikTok download request).
That approach has to be better than manually hard-coding all known version numbers and then randomly selecting one and/or iterating through them until one works.
I tried finding the version numbers with the Facebook crawler UA (accessing app URLs like https://vm.tiktok.com/ZSe4FqkKd), but no joy. Maybe a mobile UA would reveal these numbers?

They came from... nowhere? @llacb47 initially found 2.0.8/208 version combos by reading through the app data, but iirc the rest are complete guesses

So, I wonder if 22.9.5 and 2295 would work? When I have some more time, I will try it.

So, I tested this, and this theory officially has no basis in reality. :/ I.e., it doesn't work.

It also could be that the version numbers are not the real issue.
I can send any version numbers to the "feed" endpoint, and it always works.

feed doesn't even need to take in any parameters

They came from... nowhere? @llacb47 initially found 2.0.8/208 version combos by reading through the app data, but iirc the rest are complete guesses

Sounds like we need to know more about how "@llacb47 initially found 2.0.8/208 version combos by reading through the app data"...That knowledge could be a game changer.
It sure beats (manually) generating (and hard-coding) random numbers and then iterating through them, only to have them all blocked a few hours or days later.

@RealDeviil he MITM'd the app's network traffic and read the query parameters

https://httptoolkit.tech/blog/inspect-any-android-apps-http/
That's the next step to finding a better solution. When that environment is up and running, then inspect the app's HTTP requests/responses to determine the one that reveals TikTok API version numbers. Then see if it's possible to reproduce that request/response outside of the app. Easy enough to try if you already have the environment installed, configured, and ready to go. I personally do not, but if the "feed" endpoint stops working, then maybe I try it.


When generate a authentication error page with customize login portal changes enabled given error is not aligned in the page

How to reproduce:

Get pack https://wso2.org/jenkins/view/products/job/products/job/product-is/org.wso2.is$wso2is/4200/
Configure pack with customizable login portal feature https://docs.google.com/document/d/1_XUbmrokb1Yy7T1eZ8wpiq5i3eKwxEXxrwbBmX5xC4Q/edit
Login to managemnt console
Enable multi attribute login for email address claim (add only this claim under claim list). https://is.docs.wso2.com/en/latest/learn/multi-attribute-login/#:~:text=Configuring WSO2 IS for Multi Attribute Login,-Start WSO2 Identity&text=Click Main > Identity Providers > Resident,regular expression for allowed claims.
Create a user and set a value for email address claim
Access console
Try to login to console with a incorrect user id value to get a error page generated

user id -<EMAIL_ADDRESS>( correct value will be shanikanishadhi1992<EMAIL_ADDRESS>pw - shanika
Error page generated with authetication end point with customized login page, the error message is not aligned in the page. More white space towards bottom in the page

Environment information (Please complete the following information; remove  any unnecessary fields) :
H2 default

Error Message is aligned to middle

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Add missing fields to PtctlPacket

I think it looks similar to end of walk packets, it's likely speed of the entity. Not always 9, changes for different mates

It looks to me that the speed is not sent from the server. It probably has to be obtained from nostale data files, from IInfoService.GetMonsterInfoAsync

I think it looks similar to end of walk packets, it's likely speed of the entity. Not always 9, changes for different mates

I renamed the fields to their names.

It looks to me that the speed is not sent from the server. It probably has to be obtained from nostale data files, from IInfoService.GetMonsterInfoAsync

The speed property type is inconsistent is the project. In packets sometimes defined as byte other times as short and in ILivingEntity it is an integer. I think all speed properties should use the same type.

You are absolutely right, I think there are more inconsistencies like that. Sadly I did not manage to make it consistent yet even though I wanted.


style: UI code formatting

You can run make format and it'll format the UI code as well. However, there are still errors like this:
/home/kenn/Projects/autonomouslogic/eve-ref/ui/components/dogma/DogmaAttributeValue.vue
  4:7   error  Destructuring the `props` will cause the value to lose reactivity  vue/no-setup-props-destructure
  9:18  error  'useI18n' is not defined                                           no-undef

which can't be automatically fixed. Will have to sort these out before merging this PR.


Update christopher-maneu.yml

Docs Build status updates of commit ac2ce13:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

advocates/christopher-maneu.yml
:x:Error

Details

advocates/toc.yml
:warning:Warning

Details

advocates/index.html.yml
:warning:Warning

Details

advocates/christopher-maneu.yml

Line 13, Column 1: [Error: yaml-syntax-error] While parsing a block mapping, did not find expected key.

advocates/toc.yml

Line 64, Column 10: [Warning: xref-not-found] Cross reference not found: 'advocates.christopher-maneu'.

advocates/index.html.yml

Line 278, Column 8: [Warning: xref-not-found] Cross reference not found: 'advocates.christopher-maneu'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[RDY] Basic implementation of shell.core

Basic as in "enough to have a demo by Friday". Not everything is set in stone yet, but you can see the TODOs in the code for that.

I think we should merge ReplModule and CoreModule.
Does the implicit requirement still exist that, say, EvalCommand should not use AnalyzeCommand.execute? If so, this needs to be documented.
Can the "hardcoded" analysis step be skipped in this PR, or would that be too much work?
Lastly, are you planning on writing more tests before this gets merged?

PR is incomplete (lacking tests and unfinished comments), but is merged anyway such that the sprint tag has a workable REPL.


    update the AD settings for opensearch.

Signed-off-by: Alex<EMAIL_ADDRESS>Description
Rename plugin settings from opendistro to opensearch.
Issues Resolved
https://github.com/opensearch-project/anomaly-detection/issues/40
Check List

[Y] Commits are signed per the DCO using --signoff

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

@spbjss Changed to main branch for OpenSearch in CI (https://github.com/opensearch-project/anomaly-detection/pull/50). Can you merge main branch to check if it can pass the github CI workflow?

From CI workflow log, the compile can pass, but integTest failed, error message
=== Standard output of node `node{::integTest-0}` ===
> Task :integTest FAILED

»    ↓ errors and warnings from /home/runner/work/anomaly-detection/anomaly-detection/build/testclusters/integTest-0/logs/opensearch.stdout.log ↓
» ERROR][o.o.b.OpenSearchUncaughtExceptionHandler] [integTest-0] fatal error in thread [main], exiting
»  java.lang.NoSuchMethodError: 'org.opensearch.common.settings.Setting org.opensearch.common.settings.Setting.positiveTimeSetting(java.lang.String, org.opensearch.common.settings.Setting, org.opensearch.common.settings.Setting$Property[])'
»  	at org.opensearch.ad.settings.AnomalyDetectorSettings.<clinit>(AnomalyDetectorSettings.java:72) ~[?:?]
»  	at org.opensearch.ad.AnomalyDetectorPlugin.getSettings(AnomalyDetectorPlugin.java:639) ~[?:?]
»  	at org.opensearch.plugins.PluginsService.lambda$getPluginSettings$0(PluginsService.java:107) ~[opensearch-1.0.0-beta1.jar:1.0.0-beta1]
»  	at java.util.stream.ReferencePipeline$7$1.accept(ReferencePipeline.java:271) ~[?:?]
»  	at java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1625) ~[?:?]

The integTest will always use opensearch-1.0.0-beta1 to build test cluster for integTest. @dblock  Any suggestion how to unblock the integTest? Should we build beta2 or 1.x to include the code change of OpenSearch PR 665?  This PR removed final TimeValue minValue from input param list of function Setting#positiveTimeSetting. So we can't use beta1 to build test cluster to run latest code.

Several options to unblock the integTest,

Use old setting function with some redundancy code: https://github.com/ylwu-amzn/anomaly-detection-2/commit/36160805c22571061cca5064257e952d4d7ff4c0.
Build new artifact with latest Setting class for integTest.
Ignore the integTest failure, just merge it directly. We may have no way to run integTest until we have a new OpenSearch artifact ready.

@ylwu-amzn @spbjss would you please followup with infra team on (1)? cc: @bbarani

@ylwu-amzn @spbjss would you please followup with infra team on (1)? cc: @bbarani

Also can we open an issue in opensearch-build to track this?

@ylwu-amzn @spbjss would you please followup with infra team on (1)? cc: @bbarani

Also can we open an issue in opensearch-build to track this?

Talked with Peter @peterzhuamazon , we will have a new tarball tomorrow. Do we need to change the tarball link used in integTest ? Who will change it? @dblock  @saratvemulapalli

@ylwu-amzn @spbjss would you please followup with infra team on (1)? cc: @bbarani

Also can we open an issue in opensearch-build to track this?

Talked with Peter @peterzhuamazon , we will have a new tarball tomorrow. Do we need to change the tarball link used in integTest ? Who will change it? @dblock @saratvemulapalli

As a general rule of thumb when we need something we should be opening issues. I got pinged by several people about this, so opened https://github.com/opensearch-project/opensearch-build/issues/39 and just point them to it.

Do we need to change the tarball link used in integTest ? Who will change it? @dblock @saratvemulapalli

@spbjss who opened this PR wants it to get to green so I imagine he would ;)

Do we need to change the tarball link used in integTest ? Who will change it? @dblock @saratvemulapalli

@spbjss who opened this PR wants it to get to green so I imagine he would ;)

Haha I think the link should work, we have agreed upon a format between Infra and Opensearch, Distribution Downloader should be able to handle that.
https://github.com/opensearch-project/OpenSearch/blob/0e9f74e35f1255cb9ec45be3d8960aad195a9f6e/buildSrc/src/main/java/org/opensearch/gradle/DistributionDownloadPlugin.java#L159

Do we need to change the tarball link used in integTest ? Who will change it? @dblock @saratvemulapalli

@spbjss who opened this PR wants it to get to green so I imagine he would ;)

Haha I think the link should work, we have agreed upon a format between Infra and Opensearch, Distribution Downloader should be able to handle that.
https://github.com/opensearch-project/OpenSearch/blob/0e9f74e35f1255cb9ec45be3d8960aad195a9f6e/buildSrc/src/main/java/org/opensearch/gradle/DistributionDownloadPlugin.java#L159

I could change the tar url of the integration test to "https://artifacts.opensearch.org/releases/core/opensearch/1.0.0-rc1/opensearch-1.0.0-rc1-linux-x64.tar.gz", waiting for infrastructure team to build the rc1 artifact.

Codecov Report

Merging #47 (4325929) into main (0f3b118) will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main      #47      +/-   ##
============================================
+ Coverage     79.28%   79.41%   +0.12%
- Complexity     2691     2692       +1
============================================
  Files           242      243       +1
  Lines         11065    11123      +58
  Branches       1012     1012
============================================
+ Hits           8773     8833      +60
+ Misses         1878     1876       -2
  Partials        414      414

Flag
Coverage Δ
Complexity Δ

plugin
79.41% <100.00%> (+0.12%)
2692.00 <2.00> (+1.00)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ
Complexity Δ

...va/org/opensearch/ad/AnomalyDetectorJobRunner.java
75.00% <ø> (ø)
36.00 <0.00> (ø)

.../java/org/opensearch/ad/AnomalyDetectorPlugin.java
95.38% <ø> (ø)
10.00 <0.00> (ø)

src/main/java/org/opensearch/ad/model/ADTask.java
98.12% <ø> (+1.56%)
68.00 <0.00> (+2.00)

.../java/org/opensearch/ad/model/AnomalyDetector.java
95.59% <ø> (ø)
91.00 <0.00> (ø)

...va/org/opensearch/ad/model/AnomalyDetectorJob.java
60.68% <ø> (ø)
26.00 <0.00> (ø)

...in/java/org/opensearch/ad/model/AnomalyResult.java
99.40% <ø> (ø)
58.00 <0.00> (ø)

...est/handler/IndexAnomalyDetectorActionHandler.java
49.39% <ø> (ø)
32.00 <0.00> (ø)

.../handler/IndexAnomalyDetectorJobActionHandler.java
60.99% <ø> (ø)
16.00 <0.00> (ø)

...ain/java/org/opensearch/ad/task/ADTaskManager.java
73.21% <ø> (ø)
82.00 <0.00> (ø)

...d/transport/AnomalyDetectorJobTransportAction.java
88.57% <ø> (ø)
5.00 <0.00> (ø)

... and 15 more


Fix for Go 1.16+ & bump to 1.17, from sylabs 21

This pulls in sylabs PR

sylabs/singularity-userdocs#21
which fixed
sylabs/singularity-userdocs#20

@DrDaveD I am not sure how exactly to do submodule, therefore maybe it is the reason that publish is failing?

No, I think it is failing due to lack of the secret key.  I'm not sure why sometimes it skips the publish check.