[lat] Dialect specifications

Add "Classical", "Ecclesiastical", and "Vulgar" specifications for Latin.

Not directly related but the Latin extractor is also picking up pronunciations from other languages, like this Spanish one.

Not directly related but the Latin extractor is also picking up pronunciations from other languages, like this Spanish one.

That's quite an interesting problem. It looks to me like the root of the problem is in how the pron selector is defined, in particular how following-sibling as defined here works.
The first following-sibling
 of:

  Etymology 2
  ...

(which is the element that satisfies: //{heading}[span[@class = "mw-headline" and @id = "{tag}"]]) is the
 containing the Spanish pronunciation.
Here is a snippet of code I use to test the extraction functions, it is crude but allows you to test how an extraction function works on a particular entry (specified by providing the Wiktionary URL for that entry).

Thanks for that. I'm really bad at reasoning about these so perhaps I can
convince somebody else to tackle this.
On Wed, May 6, 2020 at 9:30 PM Lucas Ashby<EMAIL_ADDRESS>wrote:

Not directly related but the Latin extractor is also picking up
pronunciations from other languages, like this Spanish one
https://en.wiktionary.org/wiki/grammatica#Pronunciation_3.
That's quite an interesting problem. It looks to me like the root of the
problem is in how the pron selector is defined, in particular how
following-sibling as defined here
https://github.com/kylebgorman/wikipron/blob/master/wikipron/extract/lat.py#L67
works.
The first following-sibling  of:

Etymology 2
  ...

(which is the element that satisfies: //{heading}[span[@class =
"mw-headline" and @id = "{tag}"]]) is the  containing the Spanish
pronunciation.
Here is a snippet of code
https://gist.github.com/lfashby/e940644e31f8caa8f0fc18d7e7666236 I use
to test the extraction functions, it is crude but allows you to test how an
extraction function works on a particular entry (specified by providing the
Wiktionary URL for that entry).
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/kylebgorman/wikipron/issues/143#issuecomment-624976987,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG4ON4DHP6BSXXR5FRJZDRQIFMNANCNFSM4MLCQT2A
.


Numbers do not appear in Slots.

Challenge http://www.freecodecamp.com/challenges/waypoint-create-a-javascript-slot-machine has an issue. The random numbers that are generated don't appear in the slots where they are supposed to be.

It was intentional, numbers are assigned to the slots in a later challenge.


DiagnoseLifetimeIssues: handle called functions.

This

fixes a bug, where we didn't consider that an object can escape via a function call. The pass issued a false warning in this case.

rdar://76115467

improves the accuracy by doing a simple form of interprocedural analysis. E.g. it now can see if a weak store is done in a called function.

@swift-ci test

Thanks @eeckstein! This is awesome. I didn't get a chance to completely review the algorithm yet. Might not get to it until Friday if you want merge.
I would have used a module-level cache in case there are some functions that are called in 100s of places, but I'll certainly take it as-is.


Binary Search

Description
I will rewrite a clean standard code for binary search on a popular containers such as arrays, vectors, sets.. etc and it's enhancement because the current code is abit confusing.
( ADDITION )
I will ADD another standard code for "binary search the answer" problems with a standard example for it.
I will also explain time and space complexities in each case.
I am a competitive programmer who had used binary search in many problems , assign it to me.
Domain
Algorithm
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

What are the updates, @shaaban500 ?

Soon : )
It will be done today or at most tomorrow

Hello , I am a gssoc contributor. please assign this issue to me

Don't
I will finish it in two days
I just have exams now


refact: Add different probabilities for seps

Comma is the most common, followed by new lines and lastly, tabs and
semicolons

Adds space after comma/semicolon to have more similar output to
previous runs

Closes #26

Foi rápido! Thanks!


Trouble while runnig opensmtpd-install.sh on Ubuntu 22.04 LTS

Way to reproduce:
ON Ubuntu 22.04 LTS
Get to step:
Command to run:
sudo ./opensmtpd-install.sh
Response:
Working... this may take a while.
E: Package 'dkimproxy' has no installation candidate
is there a way around this?

I now understand what the problem was, the setup was for windows wsl.
And it worked like a dream until ~ I understand that AMD card would not fly, but installation went smooth


Various requests return different errors

Using this library the only thing that seems to be working is GET requests.
PUT requests return 200, though no data is modified.
POST requests with no data return 200.
POST requests with data return 401 unauthorized.
DELETE requests return 501 not implemented.
Using Postman, all of these requests work.
Any suggestions/advice would be much appreciated.

Solved:
Do not use:
axiosConfig: {
    headers: {}
}

to get rid of the Refused to set unsafe header "User-Agent" warning. This will cause PUT/POST to not work.
For DELETE requests, if the item is unable to be moved to the trash, the API will respond with 501 not implemented


Make -clientserver build optional

@notalex How hard would it be to make the requirement for building Vim with -clientserver optional?
While I'm glad you put in that feature, it has made installing Vmail rather more complicated than it perhaps should be for the majority who don't necessarily need that feature.

It is easily doable. I will get a PR in soon.

Thank you @notalex

Fixed in a new commit on master. Published the new changes to rubygems.


Feature/284 improve health endpoints

Related Issues:

EQ-284

Main Changes:

Added a schema_id column to the etl_log table. This was done to be able to safely join the etl_schemas and etl_log tables.

The way it was before the ids of both tables should match up as long as there were no issues. If any issues occurred then the those ids would be out of sync. The above change should prevent that issue.

Added more output to the /attains/health/etlDatabase endpoint. This includes when the etl completed, the s3 folder that was ingested, the db schema name and how long the etl ran. If the etl failed the last time it ran, we show this information for both the failed run and the last successful run. I used the zLastSuccess and zFailed variable names to ensure that status cake will see the status value first in the object.

Steps To Test:

Verify your app and etl are pointed to your local db.
Create the schema_id column. To do this you can either drop your current database, so the etl gets a fresh start or you can run the query below to add the column and pre-populate the column. The pre-populate query assumes the id in the etl_log table matches the id in the etl_schemas table.
Run the etl

Note: If you started with a fresh database run the etl a couple of times.

Start the app
Navigate to http://localhost:9090/api/attains/health/etlDatabase
Verify the output looks correct
Manually change the active schema to an older db
Repeat steps 5 and 6. The output should include zLastSuccess and zFailed.

BEGIN;

ALTER TABLE logging.etl_log RENAME CONSTRAINT etl_log_pkey TO etl_log_pkey_old;
ALTER TABLE logging.etl_log RENAME TO etl_log_old;
ALTER SEQUENCE logging.etl_log_id_seq RENAME TO etl_log_id_seq_old;

CREATE TABLE IF NOT EXISTS logging.etl_log
(
    id SERIAL PRIMARY KEY,
	schema_id INTEGER,
    end_time TIMESTAMP,
    load_error VARCHAR,
    start_time TIMESTAMP NOT NULL
);

INSERT INTO logging.etl_log (schema_id, end_time, load_error, start_time)
SELECT id, end_time, load_error, start_time FROM logging.etl_log_old;

DROP TABLE logging.etl_log_old CASCADE;

COMMIT;

Thanks for the heads up. I'm planning on running the update query. I'll be sure to run grant for the necessary users as well.


Racon error correction - segmentation fault

Hello,
I'm trying to run racon to correct raw pacbio reads from a plant genome (~900Mb size) overlapped with minimap. I get a segmentation fault when parsing the overlaps file. My paf file is 749Gb size, while my fastq is 124Gb big. I could try to rename read IDs of both paf and fastq file as you suggested in other threads, but I'd like to know if you think, given the size of the input, that this would be enough to load this into memory. My machine has 1Tb RAM.
I also have another paf file obtained from mhap overlaps performed within canu. This one has 'only' a 377Gb size, but I can't get a fastq for those reads because those were filtered with DEXTRACTOR. I could create a new one with fake quality tags,  but do you plan in a near future to allow fasta input, or fastq is really necessary for the analysis ? Isn't quiver or arrow information more useful for pacbio data?
Thanks in advance,
Pedro

Hello Pedro,
sorry for the delayed response. You should definitely try replacing read identifiers with integers as it will save a lot of memory. Just a sanity check, your 749GB paf file contains mappings between reads and contigs right? It seems oddly big.
Racon was designed to use base quality values, which are sequencer independent, and by leaving them out might yield lower accuracy. We are considering to enable FASTA reads as input but it needs more evaluation.
Best regards,
Robert

Hello Robert,
I may replace the read identifiers, but this paf file refers to self
overlappings between raw reads. I want to perform read error correction.
Thank you,
Pedro
On May 26, 2017 10:52, "Robert Vaser"<EMAIL_ADDRESS>wrote:
Hello Pedro,
sorry for the delayed response. You should definitely try replacing read
identifiers with integers as it will save a lot of memory. Just a sanity
check, your 749GB paf file contains mappings between reads and contigs
right? It seems oddly big.
Racon was designed to use base quality values, which are sequencer
independent, and by leaving them out might yield lower accuracy. We are
considering to enable FASTA reads as input but it needs more evaluation.
Best regards,
Robert
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/isovic/racon/issues/29#issuecomment-304228569, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEEmKLL9t66LpAoYmeKUAbeuqpm51pubks5r9pK4gaJpZM4Njv_z
.

Hello,
After replacing the read identifiers (reduced file to 433Gb size), I tried a new round of error correction with racon. As my server only owns 1Tb RAM, I needed to cancel the process as it was reaching the limit at the parsing PAF file stage. Then, I split the overlaps file into 9 chunks, by setting approximately 590,000,000 overlaps per file.  Racon was finally able to finish the first stage, yet using already 300Gb RAM for the first file.
Then, I believe it tries, for each file, correct all the reads i'm giving (in this case around 9.95million), but as the overlaps file doesn't include all of them (at least in the first column of PAF), many reads won't be corrected per round.  Still, after more than 24h running, there was very few reads corrected at all:

19:15:25 ConsensusFromOverlaps] (thread_id = 0) Processing contig 29563 / 9953843 (0.30%)

Thus, I think racon isn't suitable for error correction, at least with my plant dataset, as it shows some problems regarding scaling. I'll cancel the process.
Pedro

@PedroBarbosa, racon 1.2.0 has a wrapper script to enable easier correction of large files (you can split target sequences do decrease memory consumption). Also, all racon versions beyond 1.0.0 should use less memory than version 0.5.0.


Typo in repo name 😉

The Easter bunny left some Easter eggs to find... 🐰
ecliptic/wepback-blocks-copy => ecliptic/webpack-blocks-copy
The github repo link in the package.json is correct, though.

🤣 🤣 🤣
Thanks, Andy!


Add MIT license

Adds an MIT license and makes edits in the readme and the civic.json.
@stvnrlly for yr review.
Fixes #58

LGTM, with one exception: the deprecated section is a little confusing. Is there any reason to keep it rather than just tossing it?

@stvnrlly not quite sure which section you're referring to, can you comment on the line?
FWIW The text in LICENSE.txt is copypasta straight from the choosealicense page.

Ah, I'm talking about the README: https://github.com/colinxfleming/dcaf_case_management/blob/license/README.md#depreciated-readme-stuff.

Ah, gotcha. You are right, that's probably no longer necessary since this
isn't Jekyll oriented anymore and we can ditch it.
On Tue, Feb 23, 2016, 10:11 PM Steven Reilly<EMAIL_ADDRESS>wrote:

Ah, I'm talking about the README:
https://github.com/colinxfleming/dcaf_case_management/blob/license/README.md#depreciated-readme-stuff
.
—
Reply to this email directly or view it on GitHub
https://github.com/colinxfleming/dcaf_case_management/pull/89#issuecomment-188037635
.


Docs/gitpod

Documentation for Starting, Stopping and restarting gitpod workspace.

Moved to new PR due to a lot of changes in the file paths - outstanding issues have been carried over to review there.


Issues with PgPool type

Running this code I get a type error on the last line. The issue seems to be that axum-login is using sqlx_core's implementation of PgPool.
use sqlx::{Pool, Postgres, PgPool};

pub async fn new_app() -> Option<Router> {
    let pool = PgPoolOptions::new()
        .max_connections(5)
        .connect("postgres://moose:test@localhost/test")
        .await
        .unwrap();

    let pg_pool: PgPool = pool.clone();

    let mut secret = [0u8; 64];
    thread_rng().fill(&mut secret);

    let session_store = UserSessionStore::new(&pool);
    let session_layer = SessionLayer::new(session_store, &secret);

    let user_store = PostgresStore::<User, Role>::new(pg_pool);

mismatched types [E0308] expected `sqlx_core::postgres::PgPool`, found `PgPool`

Yes, currently axum-login is thightly coupled with sqlx, we're on our way to decouple it (see #78). You'll need to use the sqlx PgPool for now unfortunately. See: https://github.com/launchbadge/sqlx/blob/main/examples/postgres/todos/src/main.rs#L20 for a use-example of PgPool in sqlx.

Sweet ty for the quick response I'll try that out and see how it goes. It seems like an odd idea to use slx_core's versions of PgPool. But I'm glad it's being looked into. In terms of making a clean API I think it might be a good idea to let end users implement their own traits for UserStrore. I implemented my own when this wasn't working and it's a pretty painless experience since I could follow the macro. If it was covered in documentation it would have saved me a lot of effort.  I'm not sure if my own store works yet, but I'll give both a try.

Thanks for your suggestion @sixilli I'll add it to the TODO for #78.

@sixilli  how did you fix this?


Variable definitions validation

This PR is a required step towards the resolution of #145. It improves the validation of type declaration in variable definitions. For instance it validates the fact that when a document contains $myVariable: MyVar, then the MyVar type actually exists in the schema and it's an InputType.
Remaining tasks:

[ ] test cases for the new ValidationErrors
[x] fix current broken EndToEnd tests

PS: this is my first "real" attempt to write haskell so please, any insight would be greatly appreciated.
Edit: I added DefinesTypes instances on arguments because failing tests were simply not finding their argument types in the schema (schema in the sense of what is returned by makeSchema)

@jml, Now I think I'm done here, for some reason the tests are marked as failed on CircleCI, but there isn't any error at any stage of the pipeline and the coverage increased everywhere:
#!/bin/bash -eo pipefail
STACK_YAML=stack-8.2.yaml ./scripts/hpc-ratchet

alternatives: BETTER (175 missing => 161 missing)
booleans: OK
expressions: BETTER (1494 missing => 1416 missing)
local_decls: BETTER (15 missing => 14 missing)
top_level_decls: BETTER (685 missing => 669 missing)
Exited with code 2

I'm still open to other changes though if things are not right yet.

@theobat Thanks, I'll take a look now.
To fix the coverage error, you need to update the numbers in the script to match the new, better numbers: https://github.com/haskell-graphql/graphql-api/pull/186#issuecomment-401033808

@jml There we are ! Thanks for your reviews. As a side note, it seems like every guard containing an otherwise statement is counted as "always true" by hpc which means increasing the number of otherwise might force us to decrease coverage..
57% guards (11/19), 6 always True, 2 unevaluated

Next step is a PR with the actual Aeson.Object to Value translation. I won't address it before some time though.

As a side note, it seems like every guard containing an otherwise statement is counted as "always true" by hpc

Yeah, that's something I really would like to fix in hpc (or with some kind of tooling), rather than compromising on coverage. The Haskell testing ecosystem needs to get better.

Thanks!


Missing addons in v0.8.0 platforms releases

As reported in istio/issues#368 all addons but grafana.yaml are missing in all platform releases (mac, win and linux).

Duplication of istio/issues#368


Select Retention Policy for Writes

I didn't yet find a way to select a retention policy when writing points (at least in 0.1 branch, as I'm using PHP 5.3 on RHEL6).
Possible solution: Add rp parameter to HTTP request.
I'll probably create a PR that adds this functionality via an additional (default=null) parameter to Client::write

Since 1.3.1 , retention policy is included in InfluxDB\Database::writePoints. Unfortunately the Readme has not been updated to include examples.
    /**
     * Write points into InfluxDB using the current driver. This is the recommended method for inserting
     * data into InfluxDB.
     *
     * @param  Point[]     $points           Array of Point objects
     * @param  string      $precision        The timestamp precision (defaults to nanoseconds).
     * @param  string|null $retentionPolicy  Specifies an explicit retention policy to use when writing all points. If
     *                                       not set, the default retention period will be used. This is only
     *                                       applicable for the Guzzle driver. The UDP driver utilizes the endpoint
     *                                       configuration defined in the server's influxdb configuration file.
     * @return bool
     * @throws \InfluxDB\Exception
     */
    public function writePoints(array $points, $precision = self::PRECISION_NANOSECONDS, $retentionPolicy = null)

can't get this to work with influxdb 1.4.2 and influxdb-php 1.14.5
name     duration shardGroupDuration replicaN default
----     -------- ------------------ -------- -------
autogen  0s       168h0m0s           1        false
mypolicy 12h0m0s  1h0m0s             1        true
inf      0s       168h0m0s           1        false```

so i do this:
```$retentionPolicy = "inf";
$this->database->writePoints($this->points, InfluxDB\Database::PRECISION_NANOSECONDS, $retentionPolicy);```

it will work, but the data will end up in the default retention policy (mypolicy), not in `inf`

any ideas?


Invert plots in dark mode

Maybe we could consider to invert plots (Matplotlib) in dark mode:
img {
   -webkit-filter: invert(1);
   filter: invert(1);
}

turns

into

Matplotlib also supports a dark-background option (https://matplotlib.org/stable/gallery/style_sheets/dark_background.html) which could be used as an alternative. This would however require informing the Backend about whether it is run in dark mode and requires overriding a matplotlib method in this case. So this is only useful if there are obvious problems with the style method mentioned here. Examples would be users loading images (such as plt.imshow) which might not want this altering.

Maybe we add some options to images: save to file, show full screen, ...


Zagg optimizations

Ivan +1d this in STG PR #2428

Merging on a Friday because Ivan is on call and he +1d this.


Fedora 30 Cannot connect to server ,disconnect almost immediately

Describe the bug
Cannot connect,but the app on android is fine.
System : Fedora 30
To Reproduce
Steps to reproduce the behavior:

When click the bottom connect,just disconnect almost immediately

Screenshots

loading web app from file:///tmp/.mount_OutlinKR11M2/resources/app.asar/www/electron_index.html
Checking for update
Error: Error: ENOENT: no such file or directory, open '/tmp/.mount_OutlinKR11M2/resources/app-update.yml'
at /tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/electron-updater/src/AppUpdater.ts:451:27
at Generator.next ()
From previous event:
at AppImageUpdater.loadUpdateConfig (/tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/electron-updater/src/AppUpdater.ts:447:33)
at Lazy.AppUpdater.configOnDisk (/tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/electron-updater/src/AppUpdater.ts:141:43)
at Lazy.get value [as value] (/tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/lazy-val/src/main.ts:18:25)
at /tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/electron-updater/src/AppUpdater.ts:341:46
From previous event:
at AppImageUpdater.getUpdateInfoAndProvider (/tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/electron-updater/src/AppUpdater.ts:337:43)
at /tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/electron-updater/src/AppUpdater.ts:354:31
at Generator.next ()
From previous event:
at AppImageUpdater.doCheckForUpdates (/tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/electron-updater/src/AppUpdater.ts:353:34)
at /tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/electron-updater/src/AppUpdater.ts:289:25
at Generator.next ()
at runCallback (timers.js:789:20)
at tryOnImmediate (timers.js:751:5)
at processImmediate [as _immediateCallback] (timers.js:722:5)
From previous event:
at AppImageUpdater._checkForUpdates (/tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/electron-updater/src/AppUpdater.ts:284:33)
at AppImageUpdater.checkForUpdates (/tmp/.mount_OutlinKR11M2/resources/app.asar/node_modules/electron-updater/src/AppUpdater.ts:223:35)
at checkForUpdates (/tmp/.mount_OutlinKR11M2/resources/app.asar/build/electron/electron/index.js:366:40)
at App.electron_1.app.on (/tmp/.mount_OutlinKR11M2/resources/app.asar/build/electron/electron/index.js:196:9)
at emitTwo (events.js:131:20)
at App.emit (events.js:214:7)
Outline is starting

connecting to bxxxxxxxxx...
a helper has exited, disconnecting
all helpers have exited
disconnected from bxxxxxxxxx
connecting to bxxxxxxxxx...
a helper has exited, disconnecting
all helpers have exited
disconnected from bxxxxxxxxx
could not connect: InvalidServerCredentials (could not connect to remote test website: Socket Closed)
connecting to bxxxxxxxxx...
a helper has exited, disconnecting
all helpers have exited
disconnected from bxxxxxxxxx
could not connect: InvalidServerCredentials (could not connect to remote test website: Socket Closed)
could not connect: InvalidServerCredentials (could not connect to remote test website: Socket Closed)
Promise "DNS lookup" resolved before 10000 ms.

Desktop (please complete the following information):

OS: Fedora
Version :30

emmm,I try the client on ubuntu 18.04, it connect successfully. Why I can't connect on Fedora .
The URL are the same one.

same on windows 10


v-model not updating widget

I have the following code:
<multiselect  v-model="selectedTypes" :select-options="types"
  :options="{multi: true, groups: false}">
 </multiselect>`

In one of my functions I call this.selectedTypes.splice(0,this.selectedTypes.length); to reset the array (deselect all) - it is part of resetting the whole form. However, the widget remains unchanged (selections are kept).
Please advise.

Hi your issue is related to #23
you have to use this example https://vue-multi-select.tuturu.io/#a_multiselect_array.
if u want to make a deselect you have to say to the component to reload.
you have to use reload :reloadInit and @vueMultiSelectInited
Cordialy

Yes, just found that issue. The reloadInit property did the trick but it's missing in the props list documentation :) .
Also found it a bit counterintuitive to solve it that way cuz v-model should be double data binding, right?
Anyway thanks a blast for responding so quick!

Yeah sry, i will update props list
You are clearly right, when i find some times i will try to work on this.


How to apply class to button only?

I want to apply the standard bootstrap button design (dropdown-toggle bnt btn-info) to the button (class picky__input). How can I do that?

I've added a prop called buttonProps so you can pass className to the button component. It appends the class, doesn't replace it.
Usage
buttonProps={{
  className: 'btn btn-info'
}}


RelationshipType embedding from relationships

Describe the bug
This issue regards "to reproduce" steps  2 and 3 of #7960  (embedding in relationships resource response)
While working at above issue, we found out that the embedding of relationshipType in relationships resource would require more work than expected and a Rest contract review.
This because current Relationship rest response reports link to relationshipType resource itself instead of one following relation i.e. https://api7.dspace.org/server/api/core/relationships/561  response contains
    "relationshipType": {
      "href": "https://api7.dspace.org/server/api/core/relationshiptypes/3"
    },

instead of
    "relationshipType": {
      "href": "https://api7.dspace.org/server/api/core/relationships/561/relationshipType"
    },

This response is documented into relationships rest contract.
We propose a change in rest contract so that relationships REST response contains relationshipType response as per above example, and to implement a LinkRestRepository properly embedding relationshipType in relationships response where required.
An additional angular check is needed, to be sure of having the links followed as per architecture design and not hardcoded when for any reason angular needs to retrieve a relationshipType object starting from the relationship
4Science can take in charge this activity, our estimation for rest contract update, LinkRestRepository with a proper test coverage is 6 hours of work

Thanks @corrad82-4s .  Apologies for the late response, but this all seems reasonable to me.  Will add your estimate. Let me know if you'd like to claim this to fix it.


play-pause svg icon

version of svg with painted circle

Great! Works awesome.
The only problem I'd say is the loading after splash screen. Notice that this is displayed while rest of data is being loaded.

URL: static.gapminderdev.org/vizabi/develop/preview
Can you still keep that loading where timeslider is @sergey-filipenko ?


Logging inconsistencies with Flutter

Steps to reproduce
import 'dart:developer';
import 'package:flutter/material.dart';

void main() {
  final StringBuffer sb = StringBuffer();
  for (var i = 0; i < 100; i++) {
    sb.write("$i\n".padLeft(100, "."));
  }
  final string = sb.toString();
  print(string);
  log(string);
  debugPrint(string);
  runApp(const MyApp());
}

Expected results
I would expect no differences between print() and debugPrint() except that debugPrint() would not print in release mode. I would also expect no difference between print() and log() except that log() would not print in release mode.
I would also expect the embedder to do the throttling for all the various ways of logging messages.
Actual results

print()
debugPrint()
log()

max length of printable text when using Flutter
~1000 on iOS/Android, unlimited on macOS (I don't have access to Windows/Linux, so no comments on that)
unlimited
unlimited

max length of printable text when using Dart (without Flutter)
unlimited
n/a
unlimited

throttled (non-throttled logging can drop log messages)
❌
✅
❌

can end up in wrong order with a huge number of logs
❌
✅ (OS logs can show up in the middle of the logs, due to how it's working)
❌

prints in release mode
✅
✅
❌

prints in debug mode
✅
✅
✅

can be intercepted
✅ (via Zones)
✅ (via Zones through print, or via setting the debugPrint variable)
❌

Other remarks about the various ways of logging
print()

Lints recommend not to use this in release mode (https://dart.dev/tools/linter-rules/avoid_print)

debugPrint()

Typically, anything, which is prefixed with debug* in Flutter is only working in debug mode - No (official) lint that warns against this, even though it pretty much has the same problems as print()
This custom lint (https://lints.solid.software/docs/custom_lints/avoid_debug_print) warns and recommends to use log() instead

log()

For various error monitoring services it's really helpful to be able to intercept log messages in order to add more information to exceptions. This is lost with log()

There are also some previous mentions of these inconsistencies:

https://github.com/flutter/flutter/issues/65244
https://github.com/flutter/flutter/issues/22665
https://stackoverflow.com/questions/49138971/logging-large-strings-from-flutter
https://stackoverflow.com/questions/49940719/how-to-log-data-to-the-flutter-console

Since people start to recommend not using debugPrint() but rather log() (see examples a, b)), log message will most likely end up being lost in some scenarios due to missing throttling. It also results in less information for error monitoring services.
The Flutter debugging guide also mentions stdout.writeln() to log. However, it doesn't really recommend any particular way of logging messages. It does explain some differences, though.
Another problem is that debugPrint is Flutter exclusive, meaning that dart only libraries can't make use of it.
So all in all, I feel like debugPrint only solves some problems within the Flutter framework, but the Flutter framework doesn't actually do that much logging. In apps people actually try to log a lot of stuff, but they're somewhat nudged to not use debugPrint and libraries may not even be able to use it.
Code sample

Screenshots or Video
No response
Logs

Flutter Doctor output
Doctor output
 flutter doctor -v
[✓] Flutter (Channel stable, 3.19.3, on macOS 14.3.1 23D60 darwin-arm64, locale en-DE)
    • Flutter version 3.19.3 on channel stable at /Users/ueman/Developer/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ba39319843 (5 weeks ago), 2024-03-07 15:22:21 -0600
    • Engine revision 2e4ba9c6fb
    • Dart version 3.3.1
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/ueman/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.88.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.86.0

[✓] Connected device (4 available)
    • iPhone-GWTGT1UTPLJQ (mobile) • 00008030-000E4D310CD3402E            • ios            • iOS 17.4.1 21E236
    • iPhone 15 Pro Max (mobile)   • 4E4B3D83-EAA8-490B-B699-90A09831AC06 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-4 (simulator)
    • macOS (desktop)              • macos                                • darwin-arm64   • macOS 14.3.1 23D60 darwin-arm64
    • Chrome (web)                 • chrome                               • web-javascript • Google Chrome 123.0.6312.107

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Tagging @matanlurey since he told me to do so

Appreciate you filing this.
I don't have the authority to make such a change, but here is what I would do if I did:

Override print in the Flutter embedder to work the way debugPrint does today
Deprecate debugPrint (for the time being, have it just forward to print)

That would bring us down to 2 ways to do things - one that always works in every mode and device - and one for debug only. I expect folks would likely write higher-level utilities on top, but at least the core functionality in dart:* would work in a reasonable manner.

@darshankawar I don't think this is team-engine (yet), it's true the change would need to happen in the engine, but the framework team would need to sign off on the changes (and, in my proposal above, agree to deprecate debugPrint).

From the framework's point of view you should be using debugPrint (this is also enforced with the avoid_print lint, which is part of the recommended lint set for flutter). By convention (like anything starting with debug*) it should only be called from a debug context (i.e. within an assert or guarded by if (kIsDebug)), see also its documentation. It solves the throttling problem on some platforms and makes it really easy to intercept logs in tests (without requiring the use of error-prone zones).
I guess, it is not clear to me what problem this issue is describing that needs to be solved?

Using debugPrint or surrounding print with if (kIsDebug) isn't even the same behavior though.
Problems:

debugPrint has nothing to do with other uses of the prefix debug in the framework, which imply debug mode.
The default print could just be fixed in the Flutter embedder (read: not require zones) to make debugPrint unnecessary
It's not clear to users when to use dart:developer#log, which is also part of the API that ships with Flutter

debugPrint. The name suggests that it will only log to the console in debug mode, but nope; it’s nothing to do with whether you’re in debug of release mode. Massive foot gun for people new to Dart. I suspect a lot of Dart apps are leaking all sorts of info because of this…


Add Requirements in readme

This program does not work on a clean Windows 10 System. It requires the Visual C++ runtime to start.

as far as i can tell, this program requires the Visual C++ 2019 Redistributable.
Please add a "Requirements", "Dependencies" or similar section to the readme.

Will do. The initially I had it to be in the same folder as the game launcher, which has all the required dlls. Then I thought it was inconvenient so I changed it, and forgot about the vcredist dependencies.


[ja] Localize Distributed Apps into Japanese

Describe your changes
Localize Distributed Apps into Japanese
Related issue number or link (ex: resolves #issue-number)
Resolve https://github.com/cncf/glossary/issues/2863
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

The target page on the deploy preview:

https://deploy-preview-2988--cncfglossary.netlify.app/ja/distributed-apps/

ありがとうございますー。なるほど、そこにリダイレクトの定義があったんですね
どちらも修正しました！


Display transaction failure error message

Closes #3209

Looks great @justindg. I'll test it later today along with the notification PR.


[Feature] Forcing landscape screen at game launch

I'm using:

Device name: Black shark 4
Device type (phone/TV): Phone
Android version: 13

I want to suggest this feature:
I would like to see an option to force a landscape screen when the game is launched, like the official Xbox Gamepass app.

Added in v0.5


Updates for minecraft 1.12 released June 7, 2017

Since your release Minecraft has changed clay to terracotta, added glazed terracotta, added concrete and concrete powder, and changed the beds texture and colors. Bed now use block entities to determine color. I don't think your scripts/blocks.json has a way to handle that.
Also, how is static/data/blocks.png used? I'd like to help and it would help if the readme gave an overview of how this all works. I've modified scripts/blocks.json to add the latest blocks (not sure about beds though as I've said) and I can upload a new static/data/textures.png but I was hesitant because I didn't know what compression settings you saved it with. The default in photoshop lowered the file size so I didn't want to lose any quality. Any suggestions?

Someone decoded the 1.13 chunk format. Hope you can use this to help update this app to 1.13 compatible.
https://www.minecraftforum.net/forums/minecraft-java-edition/recent-updates-and-snapshots/2894808-minecraft-1-13-new-chunk-format-fully-decoded-read


⚠ Bofrost.be has degraded performance

In 6e5ae97, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1414 ms

Resolved: Bofrost.be performance has improved in 370c032.


okhttp 4.* min sdk 21 (android 5)

you can bump library_min_version to 21 or set okhttp to version 3.12.12 which is the last version that support sdk older than 21

@tonyofrancis your last commit almost 2 month ago, since a lot of developers relay on this in their apps and it's a lot of work to write our own or move to another library.
if you can not maintain this project anymore you can transform the repo to any of the contributors or any developer willing to take the work.

@badrazizi Thanks for reporting this issue. I will be setting the okhttp version to 3.12.12 to support older version.

@badrazizi  A new version of Fetch 3.0.12 & AndroidX version 3.1.6 has been released


Install extensions from terminal with a file

Hello.
I think it would be interesting to have some terminal command that could also install multiple extensions from a file like "extensions.json".
My idea would be to have something like
code --install-extension extensions.json

/duplicate

@sandy081 can you link the original please?

I am sorry that I cannot find quickly but I am sure I came across this feature request.


[BEAM-7397] Avoid String.format in state namespace key construction

This is a pretty trivial change, but has a nice little perf impact for jobs that use lots of state.  The existing impl generated state keys using String.format, this changes it to simply use a string concat, which is significantly faster.
@lukecwik
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

Build test failures look like unrelated spark test issues creating directories in jenkins? 🤷♂

Run Java PreCommit

Will merge on green

Thanks!


Mwpw 142003: Mini Compare Chart card fixes

Third one is the charm...
Fixes multiple Mini Compare Chart card alignment and spacing issues.
Resolves: MWPW-142003
Note: Alignment across cards will show broken in test URL because prices are not resolved due to CORS policy. To properly test alignment, please use this URL: https://main--cc--adobecom.hlx.page/drafts/axel/mini-compare-chart-edgecase?milolibs=MWPW-142003--milo--axelcureno
Test URLs:

Before: https://main--milo--axelcureno.hlx.page/drafts/axel/mini-compare-chart?martech=off
After: https://mwpw-142003--milo--axelcureno.hlx.page/drafts/axel/mini-compare-chart?martech=off

I am requesting changes since I have identified a couple of issues:

all browsers - you cannot tab anymore through clickable elements in the card
safari and firefox - in the 3rd card, there's a small overlapping between "Save 19% ($US7.99/mo) for the first year if you buy now" and "Secure transaction"
all browsers - mobile view is broken, the cards don't adjust width; the same issue is visible in the before link though.

@narcis-radu thanks for the review, I have addressed the mentioned issues. The Test URL from my fork will not reflect the actual changes (alignment inside cards) because prices fail in milo-forked URLs. To see the changes with prices displaying properly please see https://main--cc--adobecom.hlx.page/drafts/axel/mini-compare-chart-edgecase?milolibs=MWPW-142003--milo--axelcureno
For testing mobile, this card has mobile-specific styles and height calculations triggered upon page load. Please refresh the page when testing in mobile. We decided not to trigger mobile view when the page is resized to avoid performance impact (since most users will not go from desktop to mobile in the same window).

@Axelcureno - I still cannot tab through items when I use your feature branch. The #1 item from my initial feedback is still problematic.

@Axelcureno - I still cannot tab through items when I use your feature branch. The #1 item from my initial feedback is still problematic.

@narcis-radu I forgot to mention, this is fixed in a separate ticket: https://github.com/adobecom/milo/pull/2078

@Axelcureno 1)On mobile version the pricing text is "per" is broken in next line https://main--cc--adobecom.hlx.page/creativecloud?milolibs=MWPW-142003--milo--axelcureno

Regarding CTA alignment its different in different page, which on'es correct?
https://main--cc--adobecom.hlx.page/creativecloud?milolibs=MWPW-142003--milo--axelcureno

 https://main--cc--adobecom.hlx.page/products/premiere?milolibs=MWPW-142003--milo--axelcureno  3)'Best value' either should be in center align with product title or at top - plz review https://main--cc--adobecom.hlx.page/creativecloud?milolibs=MWPW-142003--milo--axelcureno  4)Strikethrough pricing font size is bit larger https://main--cc--adobecom.hlx.page/creativecloud?milolibs=MWPW-142003--milo--axelcureno

I have addressed the mentioned comments @Roycethan, please check:

https://main--cc--adobecom.hlx.page/products/premiere?milolibs=MWPW-142003--milo--axelcureno

Thanks @Axelcureno 1,2,3 are fixed now

Also observed one weird issue with quantity selector here: https://main--dc--adobecom.hlx.page/acrobat/features?milolibs=MWPW-142003--milo--axelcureno
4) On inner edit box theres up/down for increment/decrement when i use that the quanity is not getting applied to the Buy now CTA Href. although it works when using dropdown.
will loop @VKniaz  as well since he has worked on it before
fyi @afmicka

Also observed one weird issue with quantity selector here: https://main--dc--adobecom.hlx.page/acrobat/features?milolibs=MWPW-142003--milo--axelcureno 4) On inner edit box theres up/down for increment/decrement when i use that the quanity is not getting applied to the Buy now CTA Href. although it works when using dropdown. will loop @VKniaz as well since he has worked on it before
fyi @afmicka

@Axelcureno Plz check 4 above

Thanks @Roycethan . Quantity selector issue is not related to this card but to the component itself. Will follow up offline to get browser details from your testing.
Regarding issue #4, strikethrough should be in normal font weight and smaller font size as per Consonant XD:
XD:

With PR changes:

@Axelcureno  Tracking Quantity selector issue 4 in different ticket https://jira.corp.adobe.com/browse/MWPW-146254 since its also repro on hlx.live.
@afmicka when you get chance plz also do regressions around and then we can sign off cc @Axelcureno

@Axelcureno your changes are underlining links in the cards by default while on prod they underline on hover. Should that be like that?
https://main--cc--adobecom.hlx.live/creativecloud/file-types/image?milolibs=MWPW-142003--milo--axelcureno&martech=off

cc: @Roycethan

@Axelcureno not sure if i missed something in the requirements but shouldn't these be aligned and equal height as on prod?
https://mwpw-142003--milo--axelcureno.hlx.page/drafts/axel/mini-compare-chart?martech=off

@Axelcureno not sure if i missed something in the requirements but shouldn't these be aligned and equal height as on prod? https://mwpw-142003--milo--axelcureno.hlx.page/drafts/axel/mini-compare-chart?martech=off

Hi @afmicka, misalignment happens in my forked branch due to prices not loading. You can verify the alignment here: https://main--cc--adobecom.hlx.page/drafts/axel/mini-compare-chart-edgecase?milolibs=MWPW-142003--milo--axelcureno

@Axelcureno not sure if i missed something in the requirements but shouldn't these be aligned and equal height as on prod? https://mwpw-142003--milo--axelcureno.hlx.page/drafts/axel/mini-compare-chart?martech=off

About links underlined by default, Consonant requested this change for all cards. I just confirmed with them this morning:

@Axelcureno prices are loaded with modheader on. I get that experience with prices on the page

NLE fix, error is gone , but still see the price-stotage overlapping :

NLE fix, error is gone , but still see the price-stotage overlapping on 3rd card :

The issue has been fixed, please check again 🙏🏼 @Roycethan @afmicka

@Axelcureno The overlapping of pricing is fixed now on both desktop/mobile . Badge padding around is adjusted.
Storage options are stcked one below other as per XD which fixed the padding issue.
One outstanding issue which i still see on mobile is mis-alignment of CTA's and other components:

@yesil @Axelcureno Links that have this issue are:
https://main--cc--adobecom.hlx.page/drafts/axel/mini-compare-chart-edgecase?milolibs=MWPW-142003--milo--adobecom
https://main--cc--adobecom.hlx.page/drafts/axel/mini-compare-chart?milolibs=mwpw-142003--milo--axelcureno

Hi @Axelcureno! I see that some unit tests are failing 🤔

@Axelcureno The mis-alignment on Mobile issue is fixed on latest commit.Thanks. On a side note can you plz check why this pricing font is having font H4 ans font size is bit bigger 24 px https://mwpw-142003--milo--axelcureno.hlx.page/docs/library/kitchen-sink/merch-card?martech=off cc @afmicka plz do some more regressions

@Roycethan 24px font size is correct for this card:
We moved away from using H2s inside merch cards because it hurts accessibility and SEO. There was a ticket for this change. Now, merch cards use headings in a descending hierarchal order starting fro H3.

This PR has changes requested from @narcis-radu - this cant be merged with changes requested and thus isn't ready for stage.

@narcis-radu as per our requirements, tab click is to navigate within inner clickable elements of a merch card and arrow keys to navigate between cards. You may check this functionality in the test URL.
This PR is getting escalated because it's blocking important CC and DC migrations.
cc @yesil @mokimo @npeltier

As confirmed by PdMs, the requested changes by @narcis-radu are not in accord with our requirements, hence adding Ready for Stage label.
For any questions, please reach out to April Burness, Lucy Sogard or Nick Lam.
cc @mokimo @npeltier @yesil @3ch023

Hi @Axelcureno
The aem-psi-check is failing for this PR. Could you please take a look at it?

Hi @Axelcureno The aem-psi-check is failing for this PR. Could you please take a look at it?

Thank you, I have retriggered and it is now passing all checks :)


ciao-controller: add fatal logging for controller HTTP services

Fatally log if controller HTTP services return an error. These are run
inside a goroutine as ListenAndServeTLS blocks so it is necessary to
report these errors inside an anonymous function run via a goroutine.
Signed-off-by: Rob Bradford<EMAIL_ADDRESS>

Coverage decreased (-0.02%) to 65.77% when pulling 1bfa33be102a060875fc39eb7cd073081d4453e9 on rbradford:controller-service-error-logging into cf0419a9a9c72f0f9867dd31b56d605e85c8d82a on 01org:master.

Coverage decreased (-0.01%) to 65.781% when pulling 6171eb6d7633e95f38037b9c6d59d2867baedb17 on rbradford:controller-service-error-logging into cf0419a9a9c72f0f9867dd31b56d605e85c8d82a on 01org:master.

@markdryan: as we discussed yesterday.
CC: @kaccardi
The use of the sync.WaitGroup / shutdown problems are discussed in #1326

@markdryan Acknowledged. I hope we can resolve it by the approach outlined in #1326


Documentation: add initial benchmarks

@kelseyhightower How are we going to keep history of the benchmark result? i suggest to put it under /doc/bench/2.1.0

@xiang90 Yeah. Let me move this.

LGTM


Support for state/alarms sensors.

I would be great  to know the actual state of the inverter, pv links and battery and in case of any error on the system receive a notification with a HA automation. Model model_64259.json has good sensors that can be useful in this case.

Fixed in Commit 12962dc


Add trans pride flag

To-Do

[x] Trans pride flag
[ ] Window size detection on Unix-type systems

@NamedKitten has elected to deal with window size detection, so I'm merging this now.


Is it possible to try locally without paying Neo4j license?

The Bloom service is accessible without paying for a Bloom Server license?
I would like to try and show to my company the results.

I don't know about Bloom, but you can use the community edition of neo4j locally.

Hi @telekoteko
We are not using bloom for the vizualisation, we use a library called NVL (which is the same one used in bloom, hence probably the confusion)
The license of the library permits its use for valid Neo4j EE license which include Enterprise but also Neo4j Desktop and Aura (Free, Pro or EE)

@prakriti-solankey @karanchellani can we close this issue? thanks!

Sure @msenechal


Deleting a user is not supported on Windows VM

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command
az vm user delete --ids
Describe the bug
Hi, I am trying to delete a user across multiple VMs in different subscriptions, the az cli, 'az vm user delete' allows you to provide ids for all the VMs irrespective of subscription.  when i use the az vm user delete cmd, i get the error "Deleting a user is not supported on Windows VM".
However i noticed a workaround  on this issue, which doesn't quite cover my use-case. Any ideas on how to resolve?
Thanks
Expected behavior
The local account user  should be deleted from the VMs

@zhoxing-ms for awareness

Same as #23453

@yonzhan can i ask why this was closed. I havent gotten a response on my issue.

@zhoxing-ms this issue isnt quite the same as https://github.com/Azure/azure-cli/issues/23453#issuecomment-1207589188 , seeing as the workaround there resolved his issue because it only addresses deleting a user on VMs in the same resourcegroup,the Pwshell cmdlet does not do the same for my usecase where i am trying to delete a user across multiple vms in multiple subscriptions.

This problem was by designed on CLI side code link. We need the Compute team to confirm whether the current Windows VM supports deleting the user
@grizzlytheodore Could you please help confirm whether the REST virtual-machine-extensions/update can support deleting users for Windows VM or does the service team have relevant plan?

@zhoxing-ms thanks for the feedback.looking forward to the response.

this is a question for CRP team specifically someone who works in extensions. I can refer manuj bhardwaj, but I cannot find his github. I will share him this thread via email

turns out Manuj does not work with VM Ex. I forwarded email to correct team.

@nkuchta As you mentioned in the email thread: The Windows version of VM Access doesn't support deleting a user account. May I ask if service team has any plans to support it in the future?

@JereO-Nc I hope you had a chance to look at the above comment from our Compute Service team. The Windows version of VM Access doesn't support deleting a user account.
I have developed a sample PS script  as a workaround to achieve your requirement. You can modify it to serve your requirement. Hope this helps.

Not working no option to define which user u need yo delete or RG>

Parameter cannot be processed because the parameter name 'subs

Update the script delete local user what we request is domain user

@aamad777 As mentioned in the above comment, you need to tweak the PS script as per your requirement.
To answer your question related to define the username which needs to be deleted, This is mentioned under the Pre-requisites section in this Powershell script.
Within the script.ps1 file you need to add this line Remove-ADUser -Identity GlenJohn . This line will help you to remove the AD user.

@navba-MSFT  it only work when i delete   Get-AzSubscription -SubscriptionName $sub.Name | Set-AzContext however
I need to delete Azure account not AD account i tried the script nothing has been removed

@JereO-Nc Could you please confirm if you have tried the above script and if it helped you ?

@navba-MSFT  Tried the same nothing delete even it give succuss put when i checked using local account the  user still exists with file and folder

@aamad777 Thanks for getting back. May I know the feasibility of opening a support ticket ? Our support professionals can have closer look at the issue and troubleshoot this further.

Not sure what you mean @navba-MSFT  ?? this is already a ticket no?

@aamad777 This is a GitHub issue. My suggestion is to open a support ticket. So that our Microsoft Support Professionals can contact you over email / phone and work with you over a screen sharing session and troubleshoot this issue.
The process to open a support ticket is explained here.

@JereO-Nc As mentioned in the above comment. The Windows version of VM Access doesn't support deleting a user account. However, I have developed a sample PS script as a workaround to achieve your requirement of deleting a user across multiple VMs in different subscriptions. You can modify it to serve your requirement. Please test this and get back to us with an update.

@JereO-Nc The action is now pending on you to test the above workaround and get back to us. If you need any further assistance on this issue in future, please feel free to reopen this thread. We would be happy to help.

@navba-MSFT  pardon the late response. i didnt get notified with these updates, will test the script and get back to you.


HBASE-24363 Fix failed ut TestAssignmentManagerMetrics for branch-2.2

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 47s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ branch-2.2 Compile Tests _

+1 :green_heart:
mvninstall
6m 54s
branch-2.2 passed

+1 :green_heart:
compile
1m 18s
branch-2.2 passed

+1 :green_heart:
checkstyle
1m 36s
branch-2.2 passed

+1 :green_heart:
shadedjars
4m 11s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
branch-2.2 passed

+0 :ok:
spotbugs
3m 22s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 21s
branch-2.2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 42s
the patch passed

+1 :green_heart:
compile
0m 55s
the patch passed

+1 :green_heart:
javac
0m 55s
the patch passed

+1 :green_heart:
checkstyle
1m 17s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m  1s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
27m 26s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
javadoc
0m 34s
the patch passed

+1 :green_heart:
findbugs
3m 24s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
164m  2s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 35s
The patch does not generate ASF License warnings.

232m 46s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1705/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1705

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux e2d7029dbeba 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1705/out/precommit/personality/provided.sh

git revision
branch-2.2 / 0b4f272f2d

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1705/1/testReport/

Max. process+thread count
4598 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1705/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

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

This message was automatically generated.


Add exclude parameters to be passed in. Also make it work with precommit.

[x] Can pass libraries you want to exclude. python caniusepython3/__main__.py --exclude django-health-check drf-ujson poster foursquare -r requirements.txt Fixes https://github.com/brettcannon/caniusepython3/issues/198
[x] Also works with precommit now, so it can be run in the commit stage or overriden to run in manual stage, which helps in CI checks in builds. Fixes https://github.com/brettcannon/caniusepython3/issues/197

Please keep PRs to a single task, so either remove the pre-commit stuff  or the --exclude proposal and make a new PR for what you broke out.

@brettcannon done

closed in favor of https://github.com/brettcannon/caniusepython3/pull/200


So delete it and commit again, and let's add the .idea to the .gitignore file - git will ignore all of the files in this directory and don't commit them no matter what.


Recognize X-Box 360 controller as Xbox controller

On Linux I get Microsoft X-Box 360 pad as controller.getName(), so currently isXboxController() returns false for me.
cc @WilliamHayward

@Tom-Ski Can you have a look at this please? :smiley:

@winniehell Please take a look at an old PR I made some time ago, I encountered the same problem as you: https://github.com/libgdx/libgdx/pull/3969/files#diff-c282a5cbb1ce4dcd0eee21184296851eL146

@tdgroot Thank you! Where did you get the other values for the regex from?

I tested it using several Xbox controllers, the controller names weren't really consistent.

The proposed patch (or the regex) would also correct and add support for XBox one controllers which would be great. Under Linux I see "Microsoft X-Box One S pad" which does map out to the identical commands as the 360.
As a side note it would probably be worth listing the example strings that forced the regex/contains to look the way it does so that when looking at it in the future their is a clear list of the minimum things that need to pass.

I've had to do this exact fix inside of my own projects(under linux, at least), so +1 from me!

I agree that we should collect the existing names before we craft the regular expression. I only have access to one controller. Maybe can collect the different names here and I open a new pull request with the corresponding regular expression?
Also I am wondering if there isn't a more reliable criteria to identify a controller such as vendor ID?


Replace len() with list.lenght as py2py pass

len is currently (in #282) implemented as an external code in .eo, so no need for this issue now


High latency ingame. What am I doing wrong??

I play Half-Life Alyx on my gaming laptop and Quest 2.
The laptop has a Ryzen 9 5900HZ and a Radeon 6800M.
I'm on Fedora Workstation, with everything constantly updated.
Half-Life Alyx is run with the following launch options
DRI_PRIME=1 gamemoderun %command% -novid -console -vconsole +vr_fidelity_level_auto 0 +vr_fidelity_level 3 +vr_render_scale 1.0
ALVR streamer on the laptop is the latest binary, and the client on the Quest 2 is from the Quest store.
I connect my laptop to my Quest 2 through a Tenda V12. The router, laptop and the headset are all in the same room. The WiFi is 5GHz 867Mbps. Unfortunately connecting the laptop to the router via LAN is just too annoying to do, and doesn't help the situation much.
Here are my settings

And here are some statistics

I constantly get bad compression-like pixellation, or slowdowns that make the game lag behind my movements. When this happens, I close the client and reopen it, and it usually gets better, but not for long.
Can anyone please help me get a smoother experience?
General Troubleshooting

[x] I carefully followed the instructions in the README and successfully completed the setup wizard
[x] I read the ALVR GitHub Wiki

Environment
Hardware
Note: https://linux-hardware.org/?probe=fd9d1ea7b8
CPU: AMD Ryzen 9 5900HX
GPU: AMD Radeon 6800M
GPU Driver Version: mesa-vulkan-drivers 24.1.5-2.fc40, mesa-va-drivers-freeworld 24.1.5-1.fc40
Audio: Pipewire
Installation
ALVR Version: 20.10.0
ALVR Settings File: All the settings are at default.
SteamVR Version: 2.7
Install Type:

[ ] Packaged (exe, deb, rpm, etc)
[x] Portable (zip)
[ ] Source

OS Name and Version: Fedora Linux 40 (Workstation Edition)

You say you are using the latest but in the issue you also specify v20.10.0 which is not the latest.  There was an important change made in v20.11.0 which will benefit you greatly if you are running on a laptop (assuming you have audio enabled).
Also...
I would start by setting your resolution to "Very Low", turn off "Adapt to Framerate" and set it to 30 to begin with.  Also use h264 to begin with.  Once you have it working, try switching to HEVC and messing about with the bitrate.
Afaik 120hz is not supported, switch that to 90.
Good luck!

@dyargici 20.11is really new and I didn't notice it.
Thanks for all the info, I'll try it out tomorrow and report back :)

Don't use 120 - you cannot have high quality at this setting in my experience.

@vibudh007 - how did you get on?

@dyargici Sorry for the late reply.
Not sure if it was the 90Hz setting, the low resolution setting or the 30-50Mbps bitrate, but Half-Life Alyx is orders of magnitude better to play. Thank you for your advice, it should be visible to all new users.
Anyone who comes across this issue should do what dyargici says. It works.


*: support TLS and authentication for Thanos Ruler queries

Closes #1778

[X] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
Similar to #1838, this PR adds TLS and authentication support to Thanos Ruler for the query API endpoints. The YAML configuration format is very similar to the one used for configuring Alertmanager.
Verification
I have adapted the end-to-end tests to use the new parameters. The tests already exercised static addresses and file SD.

Rebase needed ):


Update rollup.conf.js

Updated config so that this script will also run on the forum (and any other "unity.com" page). Also, apparently @updatedURL and @downloadURL are deprecated invalid (ignore my wrong commit message!) headers according to TamperMonkey's linter but for all I know they could be valid headers on other extensions of that nature so I doubt it needs changing :)

Hi! The userscript is designed to fix a specific problem on the Unity Answers site. It serves no purpose on other subdomains. Do you have an example of this problem appearing on the forum or other sites?
In any case, there is a way for a user to modify match rules in TamperMonkey. See the official FAQ.
https://www.tampermonkey.net/faq.php#Q103

Hi! Annoyingly I can't seem to find the forum page I thought I saw the issue on 😅I thought I'd seen it somewhere on the forum, hence the update. Also, yep I totally agree, people can update the TamperMonkey script themselves. I'll close this pull request as it seems unnecessary now :)


No URLs will be polled as dynamic configuration sources.

hi,
I am using :
<dependency>
    <groupId>com.netflix.hystrix</groupId>
    <artifactId>hystrix-core</artifactId>
    <version>1.3.16</version>
</dependency>

From time to time I see the following log message:
WARN No URLs will be polled as dynamic configuration sources. ("com.netflix.config.sources.URLConfigurationSource)

URLConfigurationSource is located in archaius-core which is a dependency of hystrix-core.
Any ideas why hystrix wants to use a URLConfigurationSource ?

I also encountered this problem, why create a config.properties, do not want to create config.properties do not want to see this warning, because useless

If Hystrix works without config.properties, it's merely an info. Why'd the user create a useless file to suppress this warning?

Imho to fix this issue (simple lumber way), just removal of one line is needed: URLConfigurationSource.java#L126
Information will not be lost, as next line will print similar info.
Proper fix probably should be done in archaius in DynamicPropertyFactory class. It should not use some URLConfigurationSource if not requested. According to nice documentation it will automatically try to get config from url saved in system property URLConfigurationSource.CONFIG_URL = "archaius.configurationSource.additionalUrls". What about to check, if given property is set and use URLConfigurationSource only when it has some value?

为什么非要创建一个没有用的文件呢，就没有别的方法吗？

Another approach for your Spring Boot properties file: logging.level.com.netflix.config.sources.URLConfigurationSource=ERROR

Had to delete application and git clone + fresh install... tried all the suggestions here

@benjchristensen I suggest re-opening this issue, as it's still an issue.

We use Zuul, which requires Hystrix. System is working as expected without the need of any additional configuration. So this warning is misleading. Nothing is wrong with configuration.
Get rid of the warning please. Reduce log level or delete it completely.

Had similar problem. WARN -- No URLs will be polled as dynamic configuration sources.
Next line is INFO - To enable URLs as dynamic configuration sources, define System property archaius.configurationSource.additionalUrls or make config.properties available on classpath.
Just added a blank config.properties file to the classpath (in the  resources folder) and warning is gone.
Can also define system property: https://github.com/Netflix/archaius/wiki/Getting-Started  and https://www.baeldung.com/netflix-archaius-spring-cloud-integration

the warning is still in place. to silence it adding an empty config.properties still helps. so i would guess it will stay in because after 6 years i don't think a fix will happen.

Add Spring web starter then you will be good


Create SBoM consumer plugins

Describe the Feature
Tern can generate SBoMs in different formats. Since it understands the metadata and has a data model for container images, it could possibly consume SBoMs of different formats.
Use Cases

Users can provide their own SBoMs of the software components they know of and Tern can produce a report incorporating that information.
Users can provide an SBoM of the starting container image and Tern can generate a contextual SBoM (i.e. not containing the software components listed in the previous SBoMs.

Implementation Changes

Under formats create a consumer abstract base class which has an abstract method consume
For JSON, YAML and SPDXJSON, create a consumer.py which will implement consume and return an ImageLayer object containing the list of packages and any layer information.

Closing this. We have consumer plugins for JSON and SPDX JSON formats.


Edit and Post Recording from 5/13/2019 Community Call

Edit zoom recording from 5/13 meeting and post to YouTube channel for PresQT.

The file has been edited and added to OSF. Please post on the You Tube Channel.


Fix additional tests

This fixes two additional tests which results in the first set of tests now completely passing.
This fixes the duplicate "No manual entry" issue - there was an extra call to gripe_no_man https://github.com/ZOSOpenTools/man-dbport/compare/fix_more_tests?expand=1#diff-8d71d7400911dfb47d851d83a0f5cc613c47ad9d039fed71a66940c4c76f4b50L29
However, this now means that the next set of tests are run...and unfortunately all fail.
It also fixes how test results are calculated
I've also added -i check in the event that one of the tests starts regressing

I will open an issue for the new test failures

Created issue https://github.com/ZOSOpenTools/man-dbport/issues/51


Add option to hide Ranking Table in round_robin stages

Thanks for your PR!

Released in https://github.com/Drarig29/brackets-viewer.js/pull/90


Roll engine 288a855bfd8e..9d7cd70ce534 (3 commits)

https://github.com/flutter/engine/compare/288a855bfd8e...9d7cd70ce534
git log 288a855bfd8e0f32024b1457319def24b47670e7..9d7cd70ce5346d1c8bf1d1a3595cdc9f314c0825 --no-merges --oneline
9d7cd70ce add observatoryUrl property to FlutterEngine (flutter/engine#8987)
34a524837 Add matrix4 param to Linear gradients (flutter/engine#8985)
aa63f090f libtxt: add a BoxHeightStyle option based on the height of the strut (flutter/engine#8927)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Remove a duplicate lookup in apply_state_transitions

Objective
Remove a duplicate resource lookup and an unnecessary panic.

bors r+


Links to Java source files in analysis reports are broken

See the report in this PR for an example. Expand a performance problem and click on the Java link in the stack trace.
The link in our report is missing folders ahead of the "package" folders:
https://github.com/land-of-apps/sample_spring_app/blob/29ab2aa99862b7979e28450b0dcbb31ca1d1cd07/org/springframework/samples/petclinic/MySqlIntegrationTests.java#L60
The link should however be:
https://github.com/land-of-apps/sample_spring_app/blob/29ab2aa99862b7979e28450b0dcbb31ca1d1cd07/src/test/java/org/springframework/samples/petclinic/MySqlIntegrationTests.java#L60
Note the additional src/test/java/ fragment in the second (correct) URL.

Searching for the file using this pattern results in a reasonable result - https://github.com/search?q=repo%3Aland-of-apps%2Fsample_spring_app+org%2Fspringframework%2Fsamples%2Fpetclinic%2FMySqlIntegrationTests.java&type=code
The user is at least a click or two away from seeing the right file, instead of getting no link at all.


Pyenv's latest version is 2.2.3 now

Appreciate your great work! 🙏
Though I can specify the pythonv_version to the one I wanted, could we bump the default version of "pyenv" to the latest 2.2.3?
https://github.com/pyenv/pyenv/releases

And due to the breaking change which was introduced in 2.0.0:
Split startup logic into PATH and everything else (#1649 (comment))

So, the markosamuli.pyenv is required to insert eval $(pyenv init --path) into the profile file for a shell.

I am also grateful for all the work so far on this great project.
My team would also very much appreciate an update to get access to the latest pyenv.

I've updated the pyenv version, but the shell init might be broken now as I've been using my dotfiles instead of modifying those with Ansible:

https://github.com/markosamuli/dotfiles/blob/master/.zprofile
https://github.com/markosamuli/dotfiles/blob/master/pyenv/pyenv.zsh
https://github.com/markosamuli/dotfiles/blob/master/pyenv/completion.zsh

I've moved the PATH configuration to .zprofile, .bash_profile and .profile files.

The v6.1.0 release should fix this.


chore: bump support matrix to include GK 3.16.0

Description
What this PR does / why we need it:
Bumps Ratify GK support to include Gatekeeper 3.16.0
Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #
Type of change
Please delete options that are not relevant.

[ ] 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 not work as expected)
[ ] Helm Chart Change (any edit/addition/update that is necessary for changes merged to the main branch)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] Does the affected code have corresponding tests?
[ ] Are the changes documented, not just with inline documentation, but also with conceptual documentation such as an overview of a new feature, or task-based documentation like a tutorial? Consider if this change should be announced on your project blog.
[ ] Does this introduce breaking changes that would require an announcement or bumping the major version?
[ ] Do all new files have appropriate license header?

Post Merge Requirements

[ ] MAINTAINERS: manually trigger the "Publish Package" workflow after merging any PR that indicates Helm Chart Change

while we are on this topic, @akashsinghal , it looks like GK supported version are n and n-1. Why do we validate 3 version ? https://github.com/open-policy-agent/gatekeeper/blob/master/docs/Release_Management.md#supported-releases


Organizations on stillmaintained.com?

I used to have my organization's projects on still, but now I'm getting a broken image.  If I go to the image directly (http://stillmaintained.com/phpcollab/phpcollab.png) I get an internal server error.  If I click the "manage my packages" I don't see my organization listed.
I thought Organizations was added in issue #25.
Any insight you could provide on this?
Thanks!

+1 how doe I add orgs?

+1 I'd like to use this too.

Sadly, I'm guessing that Stillmaintained isn't being maintained.

+1 for last comment


Update README.md

added Decentralized AI project ELNA on IC

Dear @alexjolly28,
In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.
If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.
— The DFINITY Foundation

I've signed  DFINITY's CLA


function sendMessage error

hi，
I use function sendMessage after accept friend request,
when the new friend request and the bot not in the same network, the program stopped and has error.
while if the new friend request and the bot are in the same network, it can work well
error log as follows:

WARN PuppetWebBrowser execute() exception: unknown error: r.isBrandContact is not a function
(Session info: chrome=53.0.2785.143)
(Driver
WARN PuppetWebBridge execute() exception: unknown error: r.isBrandContact is not a function
(Session info: chrome=53.0.2785.143)
(Driver info: chromedriver=2.24.417412 (ac882d3ce7c0d99292439bf3405780058fcca0a6),platform=Mac OS X 10.11.6 x86_64)
WARN PuppetWebBridge proxyWechaty() exception: unknown error: r.isBrandContact is not a function
(Session info: chrome=53.0.2785.143)
(Driver info: chromedriver=2.24.417412 (ac882d3ce7c0d99292439bf3405780058fcca0a6),platform=Mac OS X 10.11.6 x86_64)
ERR PuppetWebBridge send() exception: unknown error: r.isBrandContact is not a function
(Session info: chrome=53.0.2785.143)
(Driver info: chromedriver=2.24.417412 (ac882d3ce7c0d99292439bf3405780058fcca0a6),platform=Mac OS X 10.11.6 x86_64)
ERR PuppetWeb send() exception: unknown error: r.isBrandContact is not a function
(Session info: chrome=53.0.2785.143)
(Driver info: chromedriver=2.24.417412 (ac882d3ce7c0d99292439bf3405780058fcca0a6),platform=Mac OS X 10.11.6 x86_64)
ERR Wechaty send() exception: unknown error: r.isBrandContact is not a function
(Session info: chrome=53.0.2785.143)
(Driver info: chromedriver=2.24.417412 (ac882d3ce7c0d99292439bf3405780058fcca0a6),platform=Mac OS X 10.11.6 x86_64)
/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/promise.js:2291
throw error;
^
WebDriverError: unknown error: r.isBrandContact is not a function
(Session info: chrome=53.0.2785.143)
(Driver info: chromedriver=2.24.417412 (ac882d3ce7c0d99292439bf3405780058fcca0a6),platform=Mac OS X 10.11.6 x86_64)
at WebDriverError (/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/error.js:27:5)
at Object.checkLegacyResponse (/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/error.js:505:15)
at parseHttpResponse (/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/http.js:398:13)
at doSend.then.response (/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/http.js:330:11)
at process.tickCallback (internal/process/next_tick.js:103:7)
From: Task: WebDriver.executeScript()
at Driver.schedule (/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/webdriver.js:414:17)
at Driver.executeScript (/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/webdriver.js:563:16)
at Browser.execute (/Users/lijiarui/git/wechaty/src/puppet-web/browser.js:437:38)
at Bridge.execute (/Users/lijiarui/git/wechaty/src/puppet-web/bridge.js:346:32)
at execute.then.then (/Users/lijiarui/git/wechaty/src/puppet-web/bridge.js:332:19)
at ManagedPromise.invokeCallback (/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/promise.js:1303:14)
at TaskQueue.execute_ (/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/promise.js:2731:14)
at TaskQueue.executeNext_ (/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/promise.js:2714:21)
at asyncRun (/Users/lijiarui/git/wechaty/node_modules/selenium-webdriver/lib/promise.js:2584:27)

before use sendMessage,  the log shows, the parameter isn't null

INFO Bot sendMessage(Class Wechaty(web), {content: hi，我是桔子蜜，最好用的微信群助手，回复1，加入测试1群，回复2，加入测试2群, to: @323bce54d6559e43a712796d14340d68, room: null})

sendMessage is not a api. it just a demo in example, and it had already deprecated.
please use Room.say() / Message.say() / this.say() instead.

another tip:
after request.accept(), the request.contact might not ready(yet).
you can try to check if contact data is full loaded. if not, wait 3s and call contact.ready(), then check & try again.

thanks!
After I use bot.send and use message.ready(), it works well
because I use sendMessage you gave in the example, which  doesn't ready（）, so the bug occurred.
but when I test the bot.send, i found the doc you gave may revise:
Message.get(prop): String|Contact|Room|Date

Get prop from a message.

Supported prop list:

id :String
from :Contact
to :Contact
content :String
room :Room
date :Date

from :Contact
to :Contact
should change to:
from :String
to :String
because， when I use message.set('to', contact), bug occurred but I use   message.set('to', contact.id) all works well.

yes, the doc is out dated.
I suggest you to use the new say() methods:

Message.say(content: string)
Room.say(content: string, replyTo: Contact)

instead of Wechaty.send()
and when you have to create a Message manualy, better to use

Message.from(contact: Contact)
Message.to(contact: Contact)
Message.content(content: string)

instead of Message.set(...), because they work better.


Add support for DynamoDb ImportTable and related actions

Hello Team!
AWS have recently introduced an API to load data from S3 into a DynamoDb table. I thought it could be useful to support it.
This PR adds actions to:

ImportTable
ListImports
DescribeImport

I tested it against the real AWS, but importing one CSV row takes ~2 minutes, which is not great.
DynamoDb local does not support those actions. I could add support to the FakeDynamoDb, but first I'd like to see if you think it's worth adding the actions in the first place :relaxed: .
Thanks!

Hey. Thanks for the contribution! We'll look at it soon - it's definitely worth adding stuff that people might find useful - even if Local dynamo lags a bit! 🙃

This all looks good - would be great if we can get this in the Fake so we can at least cover it with one test case if it turns out that real DynamoDb is too slow - actually we're a bit pondering why it's so slow in real life. Have you got any insight into why that is?

Thanks :relaxed:
Not sure why it is so slow, I reckon it is because it needs to setup the infrastructure to run the import?
I've added a test implementation using FakeDynamoDb as suggested.

Thanks again! Next time feel free to add a HatTip to the CHANGELOG as well... you can be immorrtalised forever in the release notes! 😉


i have some  issues about running evaluate

IN def get_eval_report(),
prediction = str(int(not bool(int(prediction))))
ValueError: invalid literal for int() with base 10: 'does anyone have any idea where on the new cipro website one can check if a company name exists/ does not exist on their database?'
What is the reason for this error in this code, I can't understand.
Thank you,I hope you will help if you have time

Do you still have this issue? If I don't hear from you, I will close it.


Pr tensor node augmentations

@swetha097 In many places I have intentionally left a blank line before a comment to denote a block of code and is permissible wrt Google coding standards

A blank line before a comment line usually helps readability — the introduction of a new comment suggests the start of a new thought, and the blank line makes it clear that the comment goes with the following thing instead of the preceding.

Merged


No warning for unreachable break

while(true) {
...
}
gives no warning if the break statement is missing or unreachable.
////////////////////////////////////////////////////////////////////////////////////
Editor Version: 8124

"or unreachable" seems strange to pair with this request. Aren't any unreachable statements just dead code (a warning that already exists).
Or, for the following code, would you like two warnings (DEAD_CODE, INFINITE_LOOP):
while (true) {
  // no break statements.
  functionThatCouldThrowOrExit();
  // no break statements.
}

I think he meant "or if the break is unreachable" as in:
while (true) {
  if (3 < 2) break;
}

Basically, don't be fooled just because there is a break statement if it can never get you out of the loop.

This is a duplicate of a newer bug which has more information, https://github.com/dart-lang/sdk/issues/29196.


Forced encryption even if turned off on both sides

Shouldn't that:
https://github.com/microsoft/mssql-jdbc/blob/145c10e90148087e48c6c18f0f3449bcbb3d1708/src/main/java/com/microsoft/sqlserver/jdbc/SQLServerConnection.java#L3551-L3555
be
if (!isTDS8 && TDS.ENCRYPT_OFF != negotiatedEncryptionLevel && TDS.ENCRYPT_NOT_SUP != negotiatedEncryptionLevel) {
    tdsChannel.enableSSL(
        serverInfo.getParsedServerName(),
        serverInfo.getPortNumber(),
        clientCertificate,
        clientKey,
        clientKeyPassword,
        false
    );
    clientKeyPassword = "";
}

if I use the driver with my mssql 14 server and encryption is turned off on both sides, it still tryes to connect via ssl.

Even when encryption is "OFF" on both sides, encryption is still used to encrypt the login information. After login, the connection returns to unencrypted.

Ok, I do understand, but this should be configurable as well. IMO the credentials are not necessarily more valuable than the transferred Information. 🤔
Do the MSSQL Server demand such a behaviour or would i.e. MSSQL 2014 also allow to receive the login data unencrypted as well?
The Problem I have is that java 17 does not want to talk to my MSSQL 14 due to a RSA 1024bit encryption issue. Java 17 wants at least 2048bit and I'm only able to change thigs on the client side. Turning encryption off would be a good solution because I'm in a closed network with no internet access.

SQL Server would only accept an unencrypted login if the client specifies that it doesn't support encryption. This is used for backwards compatibility with very old clients. It's not something we would put it any modern client.
I don't know which setting it would be off hand, but you should be able to modify the Java 17 config to allow RSA 1024 bit encryption: https://www.java.com/en/configure_crypto.html

@David-Engel Thanks for your reply. So there is no way of getting this as a configurable option? I would write a PR if possible? If not I consider selfhosting a modified version of your code. Would it be enough to just change the lines I marked or would it be more of an effort to make it possible?

So there is no way of getting this as a configurable option? I would write a PR if possible? If not I consider selfhosting a modified version of your code. Would it be enough to just change the lines I marked or would it be more of an effort to make it possible?

There's a little more to it in the negotiation prior. But I would not support adding it as an option.

I don't know which setting it would be off hand, but you should be able to modify the Java 17 config to allow RSA 1024 bit encryption: https://www.java.com/en/configure_crypto.html

I know that, but I can't do this due to ensurance reasons :(

If you have those kinds of requirements, I would think downgrading it at any point, whether in the Java config, or via driver options, would go against the same reasons. It sounds like your only option, within your requirements, is to get the server config fixed.

Thank you for the clarification 👌😇

Btw, jTDS does connect without (just saying :)

Btw, jTDS does connect without (just saying :)

And jTDS sends your credentials in plain text. Most people care about that kind of thing. :)

Btw, jTDS does connect without (just saying :)

And jTDS sends your credentials in plain text. Most people care about that kind of thing. :)

Not in a unittest environment or in a dmz.


Fine-tune MobileNet-SSD

Hello,
I would like to fine-tune MobileNet-SSD with my own dataset and own classes. There is an example in Caffe for fine tuning an image classification pb (http://caffe.berkeleyvision.org/gathered/examples/finetune_flickr_style.html) but it is not suitable for a multiobject detection. Is there any code/example/tutorial available?

@didulle
If you found any resource for custom classes training share here
Thanks,

I also fine-tune  my own model with my own datasets, my iterations is 240000, fixed other parameters, but the final model also can detect some objects that classes in pre-trained model, such as person, It can detect very well, but it's not in my classes. Does anyone know the reason?


Reactified

This reactifies things a bit more properly:

We don't create CompletableFuture instances to be completed later, but chain the ones from the DB all the way down to the user
Added RxUtil for some common RX utils, I should add more stuff there
Had to add RxActionQueue and copy default actions instead of extend them
Reactified flush/delete/load

Some global RX recipes:

Don't create CF instances to be completed later, just chain the one you get which has what you're waiting for
It's fine creating CF instances already completed (null or value)
Chain CF instances that you create in a for loop with thenCombine
Turn finally{...} into .whenComplete((v,x) -> { if(x != null) { ... }}) (whenComplete, like try does not expect a return)
Turn catch(X x){...} into .handle((v,x) -> { if(x instanceof X) { ... }}) (handle, like catch needs you to care about return values)
Turn rxA(); B() into rxA().thenAccept(v -> B()) (sequence with then and drop result with thenAccept)
Turn rxA(); return B() into rxA().thenApply(v -> B())(sequence with then and return result with thenApply)
Turn rxA(); rxB() into rxA().thenCompose(v -> rxB()) (sequence with then and chain with thenCompose)

Rebased.

Merged https://github.com/DavideD/hibernate-rx/pull/2 thanks


How to query the event by timestamp

Issue
Currently I am trying to query the audit.app.create events by calling CC V2 API. when I only query the events by type by following command, it works well.
curl --header 'Authorization:  bearer XXXXX' -k https://<ENTRY_OF_CF>/v2/events?q=type:audit.app.create
But when I tried to add a query condition for the timestamp, it seems not works. I have tried several different ways to do the query, but none of them works. Could you please tell me how to do the query for the timestamp?
curl --header 'Authorization:  bearer XXXXXX' -k https://<IP_OF_CF_ENTRY>/v2/events?q=type:audit.app.create&q=timestamp%3E2017-11-08+00%3A00%3A00-00%3A00
curl --header 'Authorization:  bearer XXXXXX' -k https://<IP_OF_CF_ENTRY>/v2/events?q=type:audit.app.create&q=timestamp>2017-11-08T00:00:00.00000Z
Many thanks!

@maoyi8212 Looking at your question, the docs for that endpoint might be helpful. In particular, the format for timestamp that we use is like "2016-06-08T16:41:23Z".
If this doesn't work, we'd be happy to help you in the #capi Slack channel.

@elenasharma I have tried your format, but it seems still not works. Just like the filter condition is not exisit. The same results outputs just like the only query condition is "type=audit.app.create "
And it seems that I can not join the capi slack channel, because   @get-an-invite-at.slack.cloudfoundry.org or @vmware.com email address is required.

@maoyi8212 Hmm, that is odd behavior from the endpoint. You can get an invite by going to http://slack.cloudfoundry.org/.

@elenasharma I have put a request in Slack, but got no response yet.
Actually now I can query the events by timestamp, for example: curl --header 'Authorization: bearer XXXXX' -k https://api.run.pivotal.io/v2/events?q=timestamp>2017-11-14T08%3A55%3A53Z
I can also query the events by the type, for example: curl --header 'Authorization: bearer XXXXX' -k https://api.run.pivotal.io/v2/events?q=type:audit.app.create
But when I combined the two query condition together, the second query condition seems never take effect. So my question is that whether the combination query is supported by CC API event entry? My CC API version is 2.98. Many thanks!

@maoyi8212 did you post in the #capi Slack channel in the Cloud Foundry organization? Here is the link to that channel. I might have missed your question, but I don't see one.
As to your question, we do support using multiple query parameters. An example request for querying for events by type and timestamp would be: cf curl '/v2/events?q=type:audit.organization.create&timestamp%3e2017-11-17:22:06:00'.

@elenasharma I have found the issue. When I use curl command directly to call the API, the issue happens, but when I switch to "cf curl", the results seems no problem. But I do not know why the curl command has the issue.
Thanks a lot for your great support!


When using the new text-embedding-3 models the scores are a lot lower

Hey,
when i try to use the new text-embedding-3 models for creating the embeddings and for querying, i get a lot lower scores for the same query.
with ada-2, a query could get a result score of 0.8, but with text-embedding-3 it goes below 0.5, but returns the same content.
Is there a reason for this?

That's a function of the embeddings model and nothing I have control over. It implies that they're generating a more diverse range of embeddings... Can you share some examples (query + text being compared to)

Here is an example using the example from the readme of vectra:
with "text-embedding-ada-002":
Querying green...
[0.9027890493383421] blue
[0.8750171543194056] red
[0.8316836924030466] apple
Querying banana...
[0.9025824326098169] apple
[0.8489727589250824] oranges
[0.840552337334082] blue
with "text-embedding-3-small":
Querying green...
[0.5587630540517711] blue
[0.4586459570036867] red
[0.3330212746409029] oranges
Querying banana...
[0.463723740085403] apple
[0.36792568686955635] oranges
[0.3011467689281706] blue
with "text-embedding-3-large":
Querying green...
[0.5854194924173858] red
[0.5425629350657741] blue
[0.3589804053636035] oranges
Querying banana...
[0.4618476040380141] apple
[0.39727599664880175] oranges
[0.37006686089236474] blue


Lazy Formatter

Traceback (most recent call last):
  File "/home/alisamar/Desktop/lldb_folly_formatter/src/lazy/lazy_formatter.py", line 45, in LazySummary
    computed = bool(lldb.value(valobj).value_.storage_.hasValue)
  File "/usr/lib64/python3.9/site-packages/lldb/__init__.py", line 14445, in __getattr__
    raise AttributeError("Attribute '%s' is not defined" % name)
AttributeError: Attribute 'value_' is not defined

Tests pass!


Signup doesn't execute when using email and password authentication for 2 time.

When I am signing up using email and password authentication then it will execute for the first time perfectly but if I log out and then so the same process of signup it doesn't execute. Is there any cache error or something else I want to know.

Didn't get reply


Fixed todo in User.py line 159/160

So the todo was "TODO Figure out why the logic likes having +1 of the actual count
before it decides its time to actually punish the user properly".
The problem is that we set  self.duplicateCounter  to 0 and we can't see if a message is a duplicate until we have another message/duplicate. If we have 2 duplicates we can set the counter to 1 but after the 5th duplicate its at 4 and not at 5.
So my solution is to either set the counter to one or subtract 1 in every check against "warnThreshold", or add 1 to counter at every check.
For performance reasons I would prefer to initialize the  duplicateCounter with 1 and if its ever needed to look the user object up via a command then just subtract 1 from the duplicateCounter.
I don't want to make a decision here so its up to you what you do :)

Sorry for only just noticing this, I will look into it further tonight during further punishment implementation and decide on a course of action.

Note, given its preset to 1 now rather then 0 and that needs to be kept in mind. Im looking into possibly turning it into a get/set or something with a public/private setup to remove the need to worry about the actual value outside of the class

Yeah as I'm working with your project in my bot maybe I come  up with a nice solution like that or something better who knows and no problem regarding the time offset of you noticing that. Hope it was helpful anyway

Should be resolved now, and I appreciate your feedback in issues and pull requests :)
Its always helpful, keep it up! 👍


Can you release a binary package?

I can help write a manifest via scoop so that we can install lpless easily.

Never mind, I saw you have released lpless.exe in nuget package LinqpadLess, I will try to use that instead.


[FEATURE]  no decoration

My problem with this is it's not very interesting/useful. There are already some questionable snippets like "reset all styles"...
We should keep snippets to things like this: https://9elements.com/io/css-border-radius/ (make a PR for this!)

I agree with @atomiks on this one, it's not super interesting. I vote for closing this, honestly.