How to do continue training when a job failed

Hi, for example I am training a job using this yaml, how to do continue training if this job failed? Thanks.

You can add a load path as a trainer argument in that yaml to resume a job from an earlier checkpoint.
Something like this:
trainer:
  _target_: composer.Trainer
  device: gpu
  max_duration: 850000ba
  eval_interval: 10000ba
  device_train_microbatch_size: 16
  run_name: ${name}
  seed: ${seed}
  load_path: # Path to checkpoint to resume training from
  save_folder:  # Insert path to save folder or bucket
  save_interval: 10000ba
  save_overwrite: true
  autoresume: false
  fsdp_config:
    sharding_strategy: "SHARD_GRAD_OP"


TypeError: value.toLowerCase is not a function

When running 'av init' I receive this error  'TypeError: value.toLowerCase is not a function'
TypeError: value.toLowerCase is not a function
at Object.validate (/Users/foo/workspaces/barProject/node_modules/availity-workflow/cli/commands/init.js:118:33)
init.js:118
Looks like around that line its calling toLowerCase on the 'keywords' array of strings instead of the individual strings.

I ran into the same problem today, I'll take a look and see if I can correct the issue.

Fixed in v1.2.12


incorrect perl statement from LDAS update

Two line bug fix - original change from Land folks were not correct perl statements.

@rtodling: My mistake, sorry about the syntax error.  I'm confused, though, about including "$ldashome" in the first query, and "$ldasenshome" in the second query.


 Thx

Can you be more specific? Are you asking how to run it, or about how it does what it does?


[Release] sdk/resourcemanager/mobilenetwork/armmobilenetwork/4.0.0

https://github.com/Azure/sdk-release-request/issues/4967

The breaking changes are introduced and approved in this PR.

/azp run go - armmobilenetwork - ci

The breaking changes are introduced and approved in this PR.

@JeffreyRichter , please approve this major version release by adding label of Approved Major Version, thanks.


SP5: fix hidden potables rendering

Fix plotables rendering when one plotable is hidden.
This should fix issue #3089.

Great catch @KroMignon, thanks!


Forcing anti-aliasing

Some drivers like AMD Radeon allow us to "force anti-aliasing" in the driver settings across DirectX 9, 10 and 11 games (maybe in Vulcan or Direct 12 but I have not tested those as I do not own any.)

Would it be possible for similar to be done when in using DXVK maybe as a config option?
Because some games don't support it in the options or don't do it for cameras in the game, so it would be nice to have.
The driver for AMD and Nvidia can do it, so maybe DXVK can...! Or does it need to be lower level...?
You tell me! 🐻
Thank you for your time! This feature would be greatly appreciated. 😺

The option in the AMD control panel doesn't let you enable AA, it lets you switch between regular multisampling and supersampling. This could be done, but a) most D3D10/11 games don't support MSAA to begin with (which is a requirement for this to work) and b) the only thing this would really do in the end is murder performance for almost no visual gain.
So, no, not going to implement it.


Create Glue database and tables for CloudTrail to be used with Athena

Allows us to query the CloudTrail logs with Athena
CloudTrail logs by default are stored in an S3 bucket, in a structure that makes it difficult to read or query. Creating the Glue table (with a structure that works with the CloudTrail logs) allows Athena to read and query the logs.

I added a bit more of a why 👍

I added a bit more of a why 👍

ship it :shipit:


Error while using /auth/local

I am getting strange error...
I registered two users using my "/users" service in same session.
I was able to login with first user using his username and password.
However when I tried to login with second user, it threw following error.
NotAuthenticated: Invalid login.    at Object.construct (D:\POCs\feathers_tutorials\my_feather-app\node_modules\core-js\modules\es6.reflect.construct.js:23:24)    at NotAuthenticated.ExtendableBuiltin (D:\POCs\feathers_tutorials\my_feather-app\node_modules\feathers-errors\lib\index.js:21:28)    at NotAuthenticated.FeathersError (D:\POCs\feathers_tutorials\my_feather-app\node_modules\feathers-errors\lib\index.js:99:87)    at new NotAuthenticated (D:\POCs\feathers_tutorials\my_feather-app\node_modules\feathers-errors\lib\index.js:152:85)    at D:\POCs\feathers_tutorials\my_feather-app\node_modules\feathers-authentication\lib\services\local.js:144:27    at allFailed (D:\POCs\feathers_tutorials\my_feather-app\node_modules\passport\lib\middleware\authenticate.js:94:18)    at attempt (D:\POCs\feathers_tutorials\my_feather-app\node_modules\passport\lib\middleware\authenticate.js:167:28)    at Strategy.strategy.fail (D:\POCs\feathers_tutorials\my_feather-app\node_modules\passport\lib\middleware\authenticate.js:284:9)    at verified (D:\POCs\feathers_tutorials\my_feather-app\node_modules\passport-local\lib\strategy.js:82:30)    at D:\POCs\feathers_tutorials\my_feather-app\node_modules\feathers-authentication\lib\services\local.js:120:18    at D:\POCs\feathers_tutorials\my_feather-app\node_modules\bcryptjs\dist\bcrypt.js:261:17    at D:\POCs\feathers_tutorials\my_feather-app\node_modules\bcryptjs\dist\bcrypt.js:1198:21    at Immediate.next [as _onImmediate] (D:\POCs\feathers_tutorials\my_feather-app\node_modules\bcryptjs\dist\bcrypt.js:1078:21)    at processImmediate [as _immediateCallback] (timers.js:383:17)
Here is my code:
https://github.com/rohitketkar87/useradministration-feathers.git

However i noted that if I register users in two different sessions (stopping server and starting again), I am able to login from both using /auth/local.
I am suspecting that when we use same sessions, "token" is same in both the cases which is causing trouble I guess.

In general Feathers services are stateless so there isn't really any sessions. I did notice some strangeness in https://github.com/rohitketkar87/useradministration-feathers/blob/master/services/userservice.js however. For example userService.after in https://github.com/rohitketkar87/useradministration-feathers/blob/master/services/userservice.js#L173 will register a new hook every time findUserById is getting called so I'm thinking that other things in that file are getting changed up while the application is running.
I'm not sure if services/userservice.js is necessary the way it is right now since your Controller routes could just call the user service directly (via req.app.service('users')):
app.post('/api/registeruser', function(req, res, next) {
  console.log('mainCtrl : registerUser starts');
  var userInput = req.body;

  req.app.service('users').create(userInput).then(function(user) {
    res.json(user);
  }).catch(next);
});

@daffl true, right now its not necessary but actually when code grows, it helps to maintain modular structure. Moreover I was just trying out how to fit feathers in my template.
Anyway, to add further to issue, I traced log. I found out that when I try to register second user, userService.before({ 	 		create: [authentication.hooks.hashPassword(), function(hook) { 	 			console.log('****hook**** '+JSON.stringify(hook, null, 4)); 	 			hook.data.createdAt = new Date(); 	 		}] 	 	}); is getting called twice. Please find attached log.
login error.txt

@daffl Resolved the issue. Thanks. Registered all the hooks at one place. Latest code uploaded to repo.


Revert "KiCad: add footprint cache"

Reverts github/gitignore#3043
Recently filed pull request #3013 adds the fp-info-cache line in duplicate but in a more sensible location.
Propose we revert #3043 and remove the duplicate line as it's just clutter.

@bartlettD thanks!


Improve expression type parsing

Currently implicit expressions are recognized only when there is a trailing $. This lacks flexibility in the expressions. There should be ways to improve this identification of expression types vs. regular strings.

Current proposed solution (credits @soamvasani) is to use "{...}" to mark a string as an expression. It 'abuses' the fact that in javascript {<expr>} == <expr> to avoid the need to parse the expression before feeding it to Otto.

Fixed. Current solution is a { ... } syntax.


Minor update to testing readme

Updates typo and adds more instructions for brew to the testing section
of the readme.
Additions

Adds instructions for running brew info to find chromedriver installation path.

Changes

Updates typo in somewhere text.

Review

@sebworks
@jimmynotjim

:+1:


RegEXP in tags on problem panel.

Hello!
Need support for regexp in tags in the problem panel to avoid unnecessary problems.

Hi,
I had a lot of headaches with the hostgroups regex.
Can you try with?
/((?!NOALERT).)*/
Mi regex to exclude "build" and show 24x7 is:
/^((?!BUILD).)*24x7/
Hope this can help you.

Hi! Do you use problems view in Zabbix? Ho do you use tags there? It's tricky to implement regex filter for tags since it's not supported by Zabbix API. It supports like operator which can make filtering more flexible, so maybe I can add it at least.

I am facing the same problem.
@alexanderzobnin
In zabbix problems view to not show the tag "component:updates" we filter the tags like this:

Zabbix v6.0.6
Grafana v9.0.1
Zabbix plugin v4.2.9
I hope this helps

Another solution could be the use of the dashboard settings variables with the regexfilter.
But it doesn't look like i can get the tags through that.

Me too

@alexanderzobnin It might be very usefull for my team too.

@alexanderzobnin very necessary functionality!

Any news about this feature? T clould be good to exclude problems by filtering tags by regex


add number of processor to make command

following https://www.pyimagesearch.com/2020/03/25/how-to-configure-your-nvidia-jetson-nano-for-computer-vision-and-deep-learning/ , compiling protobuf for nvidia jetson NX which has 6 processor, adding -j$(nproc) to the make command to make it quicker

Thanks for this pull request.


some error occurred

Thankyou for the great mod! But some error occurred.
How do I fix it?
[Error  : Unity Log] NullReferenceException: Object reference not set to an instance of an object
Stack trace:
ValheimVRM.EyeSync.Update () (at :0)
thankyou

Thank you for your report. Fixed this issue in ver1.0.9.


Revert "Update 03_p01_ch01_03.md"

Reverts yysfz/from-en#1

1066 words italian change translation


Danger stops updating PR's comment after branch was merged

I thought that danger should continue to updating comment until PR will be closed but it doesn't. After I merge branch into the master, danger stops updating a comment. Probably, it's a feature? How to alter this behavior? I want to keep PR open to test danger against.

After removing a comment in PR, danger doesn't put it back :-(


Use en-dash instead of hyphen in license between years

English grammar states that an en-dash should be used, not a hyphen between two years.
Examples of each
em-dash: —
en-dash: –
hyphen: -
Em-dash
An em-dash is typically used as a stand-in for a comma or parenthesis to separate out phrases—or even just a word—in a sentence for various reasons (i.e. an appositive). Examples where an em-dash should be used:

School is based on the three R’s—reading, writing, and ’rithmetic.
Against all odds, Pete—the unluckiest man alive—won the lottery.
I sense something; a presence I've not felt since—

En-dash
An en-dash is used to connect values in a range or that are related. A good rule is to use it when you're expressing a "to" relationship. Examples where an en-dash should be used:

in years 1939–1945
pages 31–32 may be relevant
New York beat Los Angeles 98–95
When American English would use an em-dash – following British and Canadian conventions.

Hyphen
A hyphen is used to join words in a compound construction, or separate syllables of a word, like during a line break, or (self-evidently) a hyphenated name.

pro-American
cruelty-free eggs
em-dash
it's pronounced hos-pi-tal-it-tee
Olivia Newton-John

Build fails due to NPM:
Loading "cssmin.js" tasks...ERROR
>> Error: Cannot find module 'pako/lib/zlib/messages'
Loading "uglify.js" tasks...ERROR
>> Error: Cannot find module 'pako/lib/zlib/messages'
Warning: Task "cssmin:minifyCore" not found. Use --force to continue.

Build fixed

Thanks @juthilo

Personally, I'd prefer to keep the file as plain ASCII text.

To my knowledge the en-dash is valid ASCII though I could be wrong. From ISO 8859-1:
DEC	OCT	HEX	BIN	Symbol	HTML Number	HTML Name	Description
150	226	96	10010110	–	�	–	En dash

Take a gander at http://en.wikipedia.org/wiki/ASCII#ASCII_printable_code_chart
ASCII only has 0x2D, the hyphen-minus, as far as dashes go.

Ah ok. I was looking at http://www.ascii-code.com/ which shows it under extended ASCII code / ISO 8859-1.
Thanks for the additional info.

What are your thoughts of using the hyphen shortcuts (- for hyphen, -- for en-dash, and --- for em-dash) common in applications such as Microsoft Office? I can understand and respect the desire to keep it within the standard printable ASCII

Doesn't seem particularly worth it to me. But I don't know how the rest of the team feels obviously.

How important is the ASCII aspect of license files? Could we not rename it to LICENSE.md and do whatever we want?

It's tradition basically; ASCII plaintext is readable everywhere. Although I have seen exceptions when the author's name contains non-ASCII characters and they don't wish to transliterate their name.

Could we not rename it to LICENSE.md and do whatever we want?

If we really want the nice dash, might as well just keep it as LICENSE and use UTF-8.
Is using the pedantically-correct dash worth the inevitable mojibake that will result in a few folks seeing e.g. "Copyright (c) 2011â€“2015 Twitter, Inc" though?

I'm fine leaving it as-is. Not a huge issue :).

Just want to jump in and mention that the printable ASCII set doesn't even cover all of English, much less other languages (latin-based or otherwise). For example, "naïve" or "née" or "vis-à-vis."
Since neglecting accents does not render words unreadable—just semantically incorrect—saying "ASCII plaintext is readable everywhere" is accurate (almost) only for English text.
I'm not saying you change policy here... just want to inform in case something similar comes up another time.

That's a writability problem rather than a readability problem though.

@listrophy: Actually,  every English spell-checker I've ever used insists that the accented versions are invalid and suggests the accents be stripped.
(For French loanwords like "fiancé/fiancée" where both forms are still valid English, "English pronunciation intuition" seems to be in the process of phasing out the masculine form so the "ee" suffix can take the place of the "é" as a pronunciation guide.)

@ssokolow just checking your hypothesis here on Mac OSX Yosemite.
Every English dictionary I have, and every spell checker I've tried in a number of text-friendly apps seem to refute the claim that "Every...accented versions are invalid and suggests the accents be stripped."
Also my personal experience of decades reading both domestic and international publications, as well as the style guides I have at-hand, indicate that the phasing out of accented words in English is not, in fact, happening at all.

Thank you for the contrasting data points. I'll keep them in mind in the future.
However, to clarify, I was very careful to say that it was every spell-checker that I personally have used. The last Macintosh I ever used was a Mac SE running System 7 when I was a very young child and, at that age, I had no interest in writing.
The underlying point I'd intended to convey was that the un-accented forms are perfectly valid English (though, as you've now pointed out, not the preferred forms according to various style guides).
Also, please take note that I'm speaking from a viewpoint that, given their rarity in day-to-day common usage (likely, due to a lack of easy ways to type combining diacritics when using Windows with a US ASCII layout), the accented forms will probably go away eventually because, based on the following evidence, common usage tends to drag formal usage along, kicking and screaming:

Not even formal English uses the words ye, thou, thee, thy, or thine anymore.
...nor does it use "mine" in place of "my" before a word that begins with a vowel, despite still using "an" in place of "a" under the same conditions.
The misuse of tragedy to mean something other than "a literary work in which the hero can't escape their downfall" is so widely accepted that it's not uncommon for an individual to be unaware of the original meaning.

(I don't like that "to whom" seems to be dying out in common speech, given that "to who" feels like "a apple", but I can see the rationale. It's extra complexity that appears to impart no additional expressiveness to the language relative to what is inferred from context... just like the distinction between "ye" and "you" or the "-th" conjugation for verbs used with a 3rd-person singular pronoun.)

@ssokolow These reasons might be good enough for a Windows-using English speaker, but what about the billions of people in the world who are not? And to bring the conversation back to Bootstrap, what about the (tens of? hundreds of?) thousands of non-English speakers who use this project? Are they to be rendered second-class for the sake of convenience?

Irrelevant to my point.

You responded to a comment about keeping the license-related text, which is in English, in ASCII plaintext for maximum compatibility when feasible.
I referenced Windows because, last I checked, Windows had something like 95% of the market share in the English-speaking world... which makes its lack of support for combining diacritics a significant shaping force in how English words are spelled in day-to-day writing.

Ok folks, at the risk of sounding like a party pooper...let's get back on track here. This issue is about a minus/hyphen vs a dash. We're not here to discuss general character encoding, grammar, non-english languages being second class, or anything grander than: should that one hyphen be replaced.


(fleet) add username/password login for the OCI client

This PR adds direct username + password login for the installer OCI client to simplify setup with artifactory/nexus & the likes.

/merge


feat(frontend): optional index canister

Motivation
Make the index canister ID optional in the core type definition without changing the way we load information - i.e. in a backwards compatible manner.
Notes
This PR is relatively large, so Antonio and I reviewed the code together offline.
Changes

custom-token.services.ts: extend toCustomToken to support optional Index canister ID
IcTransactions.svelte: silently ignore listing transactions if no index
ic-token.schema.ts: set Index Canister as optional in the base type IcCanisters
ic-add-custom-tokens.service.ts: extend support to, in the future, to allow user to register custom token without index canister id
icrc-wallet.worker.ts: assert index is set. in the future this should not be called without index
add few TODOs
adapt tests

Relates to https://github.com/dfinity/oisy-wallet/issues/2883


Combine _id and _in

What I use
{
  _or: [
    {
      id: {
        _id: [
          "@deep-foundation/core",
          "dockerSupportsJs"
        ]
      }
    },
    {
      id: {
        _id: [
          "@deep-foundation/core",
          "Any"
        ]
      }
    },
  ]
}

What I want
{
    id: {
      _id: {
        _in: [
          [
            "@deep-foundation/core",
            "dockerSupportsJs"
          ],
          [
            "@deep-foundation/core",
            "Any"
          ]
        ]
      }
    }
  }

@ivansglazunov what do you think about that feature?

Or even like this:
{
  "id": [
    [
      "@deep-foundation/core",
      "dockerSupportsJs"
    ],
    [
      "@deep-foundation/core",
      "Any"
    ]
  ]
}


Write code to publish algorithm for data processing to Ocean marketplace

It should be easier to get a quick and simple algorithm for processing data working compared to a training algorithm. In any case, algorithms that processs raw data into useful forms are valuable.
Write code (e.g. a notebook) to publish a data processing algorithm on the marketplace. This algorithm can be published before the data processing code has been finalised. We only need to give it a path to the location where the algorithm will be on GitHub and a suitable Docker image.
See an example of a notebook for a different algorithm here https://github.com/AlgoveraAI/onshore/blob/main/notebooks/4-publish-model.ipynb

Going through the C2D quickstart flow. When I  try this
#run barge: start ganache, Provider, Aquarius; deploy contracts; update ~/.ocean
./start_ocean.sh  --with-provider2
(Sorry for the whiteouts, just my shell prompt). I tried the seed phrase on OP,s barge repo and also 'oceanprotocol' as passwords. Appreciate if you have any insights or help me diagnose, Cheers mnkyNtigr


Really good job but possibility for notifications?

Thank you for this web scraper! But is it possible to get an email notification if the price goes down or if the price gets lower than an requested price?

This sounds great! I have never made a pull request here on GitHub tbh so not familiar with that haha.. :)


Why i can not see my new objects created with the cli?

This is  a bug request.

[* ] Are you running the latest version?
[*] Are you reporting to the correct repository?
[*** ] Did you check the documentation?

Description:
There are many errors in the documentation. That should be known. It starts with the fact that, for example, it only becomes clear with the command "composer install" that PHP 7.4 is necessary.
The section for the cli has the first error at point 0. The documention says that backslashes have to be set, but you have to take forward slashes.
But all of that is not so bad.
The menu is a bit less clear then. That's why I asked the issue #279.
It only works if you write “index” on the route without anything else before or after index. Thought that was the solution. But ...
After that it says in the documentary you should see the new menus after the next "yarn build". Unfortunately, this is not the case. Is the reason now that I wrote something wrong in the menu / route or is the cli broken?
Too bad it takes a lot of time to search ...
Steps to Reproduce

config the cli with a new object, for example Dog or Cat
yarn && yarn build
start the app in localhost:8080

Expected behavior
new menus or tables or forms for dogs or cats
Actual behavior
standard app without the new object
git status:
Untracked files:
(use "git add ..." to include in what will be committed)
app/Dog.php
app/Forms/
app/Http/Controllers/Dogs/
app/Http/Requests/
app/Tables/
client/src/js/pages/dogs/
client/src/js/routes/dogs.js
client/src/js/routes/dogs/
database/migrations/2020_03_05_192933_create_dogs_table.php
database/migrations/2020_03_05_192934_create_structure_for_dogs.php
which of the files do you need? i can upload

Okay, it seems that the menu-link itself works, if i write that by hand. Also i did not respect the rest with the validation and the filling of templates for the tables and the forms. Here i was to fast.
But It rests also the permissions, they are not there ... and the menu topics in the surface ...
I will see this week.
Maybe you could tell something in short to the two points.
thy a lot in advance
Dirk

Hi @whizzly ,
I take it that you are missing the permissions? Perhaps you can clarify what is the current issue?
If that is indeed the case i.e. the permissions missing, please make sure that you reload the application (refresh the page) after running the migrations.
The reason for needing the reload is that the app permissions are loaded on startup.
Basically, after using the cli to create a structure what I usually do is:

run the migrations (php artisan migrate)
add the new routes
check that the routes are fine by running, php artisan route:list, verifying that my new routes are in the list and that they point to the new controllers
do a front-end build with yarn build or, if I was using serve, stop and re-run yarn run serve
go into the browser, reload the page and check that the new menu is there
open the menu, check that the table is loading and displays no results.

From this point on, I usually revert the migrations, and start modifying as required:

add columns to the table by editing the create table migration
add the new attributes to the model's $fillable
add relationships if needed
edit the table builder, update the select & include the new columns
edit the table template & include the new attributes
edit the form template & include the new attributes
edit the request validation & add the new attributes
re-run the migrations
reload the page, go into the new menu, create a new model instance using the form
go back, check that the new model is shown in the index table
rinse & repeat

Yes, the documentation is slightly out of date, we're aware of this and it's on the to-do list. If you're so inclined, you can even make PRs for punctual updates :)
In the meantime, I hope I've helped and do let us know if you bump into other things

hi gandesc,
thank you very much for the answer. I work on this in my free time and accordingly i can only answer when i am not at work. I will try to process the steps you recommended and when i have any other problem i open a new issue especially for that problem. At this place, i close this one.
nice weekend


Pull CCZ in from openfermion-cirq

Combine the benefits of both (openfermion-cirq's allows raising to a power; cirq's has a decomposition that avoids adjacency issues).

The current CCZ doesn't have powering.  Do we really want that?  It just does a default decompse into CNOT and T.


Implement option groups' and subcommand sections' support as mixins

Currently:

option groups' support is hard-coded into cloup.Command
subcommand sections' support  is hard-coded into cloup.Group.

It may be useful to implement it as a mixin so that:

they can be more cleanly mixed with other click extensions
they can be mixed with multiple click command classes, e.g. cloup.Group could extend OptionGroupMixin.

Done, but I made the deliberate choice of not mixing OptionGroupMixin in cloup.Group. Groups should not have many options.

Done, but I made the deliberate choice of not mixing OptionGroupMixin in cloup.Group. Groups should not have many options.


Bump version to <IP_ADDRESS> and add release notes

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

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

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

Codecov Report

Merging #61 (73e995f) into main (7a7e88a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #61   +/-   ##
=======================================
  Coverage   63.38%   63.38%
=======================================
  Files          30       30
  Lines        1065     1065
  Branches      354      360    +6
=======================================
  Hits          675      675
  Misses        372      372
  Partials       18       18

Continue to review full report at Codecov.

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


CUDA out of memory on A6000 [HiFA]

I have been getting CUDA out of memory errors when running the HiFA configs. With hifa.yaml it happens almost immediately (iteration 5 or so) with prolificdreamer-hifa.yaml it happened after 12903 iterations.

half num_samples_per_ray in the config/hifa.yaml
thought it was an error as well when running on an A100, but with the number set to 256 it needs about 35GB of VRAM


.fullText()

Coverage remained the same at 100.0% when pulling eada9da8c3d11fb7caef818e870f2f1fafef50ee on kg-fulltext into 2f4c718f343578daa54a90ae5c188b2683ba86b3 on master.


When pressing enter inside a list, indentation is unintuitive

In the following example: (with | representing the cursor)
+ Some list item with text in it.|
+ Some other list item from the same list

pressing enter would put the cursor on the same indentation level as the list markers, and not inside the list content:
+ Some list item with text in it.
|
+ Some other list item from the same list

rather than:
+ Some list item with text in it.
  |
+ Some other list item from the same list

I feel this is unexpected mostly because when doing the same when the list item already has more than one line in it, the latter happens. Though I do believe most code-oriented text editors put you on the deepest indentation level that hasn't yet closed.
I do understand the editor is trying to be helpful by letting me add a list item, so this is a trade off we have to make, but I claim generally, if you're in the middle of a list, you likely want the latter behaviour.

The more I wrote this out the more I saw the use case of lists with single-line items. Maybe this is a bit far, but could we dynamically adapt depending on whether the other list items are single-line or not?

Maybe this is a bit far, but could we dynamically adapt depending on whether the other list items are single-line or not?

I wonder whether that would be confusing and counter any intuition one starts to build for how the editor behaves.


Fields' required flag as callable

I'm introducing the support to set Fields' required flag as callables, so we will be able to validate the existence of fields depending the values of other ones.

Coverage increased (+0.2%) to 78.348% when pulling a42ba9266d81327047be1bd40ddb7a914eb0612a on puentesarrin:required_as_callable into 76746893df27b8fd963dc562a5100124a9969c2a on bodbdigr:master.


Differences between XETLA and oneMKL/oneDNN

Since either oneMKL or oneDNN provides optimized GEMM and CONV implementations on Xe GPU, why users need to approach XETLA?
is xetla an open source version of oneMKL?
Good to explain the position among different intel products when intel gives more options to users.
thanks

XeTLA is a collection of SYCL/eSIMD template, target for the building block to implement high-performance GEMM, CONV and adjacent computations with reasonable efforts.
XeTLA provides reusable, flexible C++ templates for subgroup level, workgroup level, kernel level primitives. The developers can implement & optimize kernels by specializing and tuning with different data types, tiling policies, algorithms, fusion policies, etc.
XeTLA primitives try best to hide/abstract different Xe HW implementation details from SYCL/DPC++ developers.

Hello,
I understand XeTLA provides building blocks that can be convenient (for non-experts) to write efficient kernels.
But suppose I have to perform a simple matrix-matrix multiplication (maybe many times for two matrices of size 4096 x 4096). In this case is XeTLA is going to be more performant than oneMKL gemm? I am finding that the latter is faster.

For simple GEMM + specific shape which oneMKL or oneDNN can support, XeTLA could get similar performance. That's not the key user scenario for XeTLA. XeTLA could get some performance advantages when fusing some GEMM based kernels to one bit kernel.


Implement proper balance transfer in task pallet and minor refactoring.

Add unit tests!
The rest seems fine.

I will update tests to check balances after creation/updation and deletion of tasks.

ping for review @JosephKnecht-lab

ping for review!


add Cryptobuyer to Segregated Witness Support list

Please add our project to the SegWit support list! Thx

Thanks a lot!


Request to delete package from pub

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?
1.
2.
3.
What is the expected output? What do you see instead?
What version of the product are you using? On what operating system?
Please provide any additional information below.
Personnal informations are visible in package description

This issue has been moved to dart-lang/pub#450.


Appu

This Pull Request resolves Issue #
Description:
In this Pull Request we have done the following additions/edits for the code:

I claim

Are there any updates here or should we close the PR and label it as spam?

We can close it


Allow disabling of splunk-kubernetes-logging monitoring agent

What would you like to be added:
A configuration parameter to allow disabling of the the  splunk-kubernetes-logging monitoring agent, as it produces a lot of log messages which aren't required from our point of view.
All generated logs relate to plugins that have no really meaning for us, but take up a large proportion of the log data that is ingested (~8k/24h).
{ [-]
   output_plugin: false
   plugin_category: input
   plugin_id: object:2b2a09afc6d0
   retry_count: null
   type: prometheus_monitor
}

Why is this needed:
We do not want to include useless information in our Splunk indexes, as this comes at a cost in many forms.
Imho the following ticket should not have been closed, as it addresses the same issue:
https://github.com/splunk/splunk-connect-for-kubernetes/issues/458
The proposed configuration parameter did not work for us:
splunk-kubernetes-objects:
  enabled: false

As the monitor agent is running inside the splunk-kubernetes-logging system components it's also really clear for me that this cannot work.
There is already a PR for it, but unfortunately there seems to be no activity from Splunk side:
https://github.com/splunk/splunk-connect-for-kubernetes/pull/340

Thank you for reporting this. I had misinterpreted #458. I agree with you.
I will review that PR soon.
Thanks!

PR is merged. Thank you!

Thanks a lot! Is there an ETA or a Milestone Planning for Chart Releases? When will the latest features be published?

i am planning to release one in near future.

@rockb1017 ,
Has this change adopted and release? We have the same requirement to disable unnecessary log.

explicitly setting monitoring_agent_enabled: false should disable these logs


Update docker images

Change Description
Fixes #7629
Steps to Test
Steps for reviewers to follow to test the change.
Pull Request Checklist
Testing

[ ] Your PR passes all CI checks.
[ ] Tests covering the positive and negative (error paths) are included.
[ ] Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

[ ] The change obeys the Code Documentation and Commenting guidelines, and lines wrap at 80.
[ ] Commits follow the Ideal Git Commit Structure.
[ ] Any new logging statements use an appropriate subsystem and logging level.
[ ]  There is a change description in the release notes, or [skip ci] in the commit message for small changes.

📝 Please see our  Contribution Guidelines for further guidance.

Thanks for the PR. Unfortunately it's never as easy as just bumping the numbers, things often break. Left a few comments.

Thank you for the comments, I'm just starting out to get hands-on with LND and have a lot to learn.

Ah, sorry, I just checked the full diff, but now saw that there are 6 commits instead of 3 or 4. Can you please squash the commits?
And signing them is recommended too, if you want to go down that rabbit hole.

Ah, sorry, I just checked the full diff, but now saw that there are 6 commits instead of 3 or 4. Can you please squash the commits? Commits should always tell the story of what you want to change, not the full story of the code review process. And signing them is recommended too, if you want to go down that rabbit hole.

Oh, I thought they are all gonna be squashed and merged. Fixed them now.


PageCache storing original header names

Q
A

Is bugfix?
✔

New feature?
~

Breaks BC?
✔ a bit

Fixed issues
#19187

I've added additional data to HeaderCollection storing the original name of the header so it can be provided with toArray(true). This change makes a small BC with the filtering option for PageCache since now the names are compared to the case-sensitive versions of the headers but I can add some code to keep it as before.

👍


[FEATURE REQUEST] add host graphing and listing with graphite backend

hi guys,
i've seen that hosts and graphs are only displayed in bosun if stored in openTSDB.
any ways to implement this with graphite backend ?
we use graphite to store our hosting platform metrics and we'd like to use bosun as an alerting system.
Not being able to see hosts and graph in Bosun is needed (at least until grafana fully implements alerting through bosun).
hoping this feature request is easy to implement,

We don't have any plans to implement this right now, but would be happy to accept a pull request for this feature. It may be a bit difficult since for openTSDB it only works if you send a copy of the data to Bosun as well so it can be indexed. For Graphite we'd probably want to use their API to enumerate the possible metrics instead.

with graphite this is pretty hard because it's so un-standardized:

there's a bunch of different agents that report the data with different metrics (both how the metrics are called as well as how the data is reported)
you can typically configure your agent with different metric prefix or naming schemes
collectd in particular has an option i hate: storeRates, which changes the reported data (it basically derives it), but under the same given key

all these things mean it becomes very difficult to "just provide graphite dashboards that query the right data and visualize it the right way".
with Grafana we "solve" 1 by just providing different dashboards for different agents and we will solve 2 by letting people finetune their prefix and naming scheme. and 3 for now will be "solved" by just stating which setting we're compatible with.
example : https://grafana.net/dashboards/24
if anything i think we should take this stuff out of bosun and move it into grafana (also for opentsdb)

@Dieterbe @gbrayut thanks for your replies
how we can go on , on this ?
should i put should work on this ? i'd be happy to try :)


Add a QuantumCircuit.transform_registers method

What is the expected behavior?
Add a method to QuantumCircuit that allows you to transform the registers (and bits/qubits) of a circuit. One use case of this popped up in interoperability with other libraries as described in https://github.com/Qiskit/qiskit-terra/issues/6860#issuecomment-891950249.
class QuantumCircuit:
  ...
  def transform_registers(
    self,
    new_qregs: Optional[List[QuantumRegister]] = None,
    new_cregs: Optional[List[ClassicalRegister]] = None,
) -> None:

Example use case:
qreg = qiskit.QuantumRegister(1, name="old")
circuit = qiskit.QuantumCircuit(qreg)
circuit.h(qreg)

print(circuit)
#        ┌───┐
# old_0: ┤ H ├
#        └───┘

circuit.transform_registers(new_qregs=[qiskit.QuantumRegister(1, name="new")])

print(circuit)
#        ┌───┐
# new_0: ┤ H ├
#        └───┘

print(circuit.qubits)
# [Qubit(QuantumRegister(1, 'new'), 0)]

print(circuit.qregs)
# [QuantumRegister(1, 'new')]

This should allow different register sizes (e.g., a circuit with one 2-qubit register and one 3-qubit register to get transformed to one 6-qubit register, and vice versa), error on wrong number of bits in new registers, etc.
#6860 described an issue I ran into when trying to transform registers "manually." A special case implementation of this can be found here.

This seems like an odd candidate for doing in-place, like in your example - is there a reason not to construct a new circuit object? It logically seems like a "new circuit" operation to me, and QuantumCircuit really isn't designed to have its backing data removed from underneath it.
At any rate, this may have some overlap with #6621 - if we merge that in some form, then this would be easy to do: you'd simply take the indices of all the qubits/clbits in your instructions, then make a new circuit with the new register layout, and replay the instructions onto it using indices.  This would make it easy to handle registerless qubits and clbits as well, and likely wouldn't even need to access any internals of QuantumCircuit.
I'm not really in a position to give the feature request the full go-ahead or say no to it, but I do think if we do it, it should wait til after #6621 to avoid duplicating logic.


Where is the straight forward, direct link to all the keyboard shortcuts of this extension?

From @fmorriso on December 17, 2017 14:23
"You can see all the keyboard shortcuts in the extension's contribution list."
What does this mean?  I cannot find it.
Please provide a straight-forward, easy-to-find, direct link on the main page of this extension to a list (such as markdown or pdf)  of the replacement keyboard shortcuts this extension supplies.
Copied from original issue: rebornix/vscode-vs-keybindings#26

From @cpmcgrath on February 5, 2018 0:15
If you go to the Extension list in Visual Studio Code, find Visual Studio Keymap (Doesn't need to be installed), it brings up details of the Extension. Up the top, it shows 4 sections: Details, Contributions, Changelog, Dependencies. If you choose Contributions it shows all the commands and what they map to.

From @fmorriso on February 5, 2018 11:39
Looking in Contributions instead of something like Documentation or Keyboard Shortcuts is not straight forward.     You cannot play this type of peek-a-boo with the documentation of how to use an extension and expect wide-spread adoption.

From @cpmcgrath on February 5, 2018 20:12
I agree that it's more hidden than ideal, however, that's the standard documentation location for this in Visual Studio Code.
With that in mind, I think if you believe it's too hidden I'd suggest raising it with the Visual Studio Code team.
By using it as the shortcut documentation, it means the extension only has to maintain a single list of changes instead of two. I agree with the Author's decision to use it.


Question: Have you tried to couple this with Threee.js

Hi,
I am the maintainer of the Asset-Importer-Lib Repo. Thanks for all your great work here. I am impressed that you have addressed the wasm issue already.
I am currently thinking about integrating a js-based renderer to our model repo. Rendering the stuff would be easy by using trree.js. But I need to intregrate omehow the aiScene intermedfiate format to three.js.

Thanks in advance!
Kim Kulling

Hi @kimkulling, it's nice to see you here, and thank you for the wonderful assimp library.
Earlier I've made an experiment to build a three.js based viewer. It exists on a branch, but never made it to main because of the limited time I had for this experiment. You can find the code here: https://github.com/kovacsv/assimpjs/tree/viewer/docs
My approach was to convert to glb with assimp and visualize it with three.js. For that a trick is needed to load a binary buffer as glb: https://github.com/kovacsv/assimpjs/blob/viewer/docs/source/viewer.js#L56.
Probably it would be a better approach to use the assimp json format directly because some data is lost during GLB conversion, too.


[FEA] Create a post block to be able to feed continuous list features to an MLP model

Description
Currently, we are not able to feed continuous list features to an MLP model since the aggregation of this list features are not working as expected.  We tested the following in the InputBlockV2 as continuous arg, but it does not work:
        continuous=mm.ContinuousFeatures.from_schema(schema_model.select_by_name(['length_of_stay_list']),
                                                                           post=SequenceAggregator(SequenceAggregation.MEAN),
                                                                           aggregation='concat')

if we can make SequenceAggregator work for this case, that'd be solving the issue. It does not work now, since right now it has hard requirement that the input should be 3D. it should be able to work with 2D tensor as well.
This is an important feature for GTC'23 Merlin tutorial.

fixed by https://github.com/NVIDIA-Merlin/models/pull/969


The demo is not available

The demo is unavailable at the moment.

It seems like the demo is down again.

@drasive try again, thanks for reporting.
I also just turned up the number of instances and gave them a lot more memory, so hopefully it will stop going down.


Canonicalization of column header

Hi, thanks for releasing the PubTables1M dataset. It took me a lot of time to clean the PubTabNet dataset, and the oversegmentation problem is probably the most tricky part. The release of PubTables1M not only increases the amount of data but also provides a good solution for the oversegmentation problem.
However, in Algorithm 1, step 10

for each cell in the column header do recursively merge the cell with any adjacent cells above and below in the column header that span the exact same columns

might lead to problems like:

Mistakenly merging nonblank cells in column header. For example, in PMC1064102_table_2:

nonblank cell 1-1 (text: None), blank cell 2-1 and nonblank cell 3-1 (text: 3 (4)b) are merged. However, None and 3 (4)b are not semantically coherent and they correspond to different row headers (Addition and Gene), so we should only merge nonblank cell 1-1 (text: None) and blank cell 2-1.
Similarly, blank cell 0-0, nonblank cell 1-0 (text: Addition...a), blank cell 2-0 and nonblank cell 3-0 (text: Gene) are merged, but we should only merge nonblank cell 1-0 (text: Addition...a) and blank cell 2-0.

The vanilla row just below the column header might be mistakenly recognized as a part of the column header, then it will be merged into the last row of the real column header under the rule of step 10. This might cause a significant mismatch between correct and wrong samples, since the visable border between the column header and the adjacent vanilla row is a strict rule for splitting cells. For example, in PMC1064102_table_0:

nonblank cell 0-0 (text: RNA no.) and nonblank cell 1-0 (text: 1) are mistakenly merged.

* Sorry I can not upload the images since I am using the company's network.
Column oversegmentation usually occurs in top-aligned spanning cells with one or zero text line. Hence, it is helpful to merge (nonblank or blank) cell and blank cells below it, but I doubt that it is not worthwhile to merge nonblank cells.
Besides, errors caused by step 10 can not be easily corrected, maybe it should be removed from the algorithm?

BTW, there is another confusing problem. For example, in PMC1079800_table_0, every column in the initial column header have exactly one complete cell that only spans that column, which arecolumn header cell 0-0 (text: Median age (range, yrs)) and column header cell 0-0 (text: 64 (60-73)). According to step 5, the while loop should be terminated immediately. However, the following three vanilla rows without any spanning cell are mistakenly recognized as a part of the column header, which is not consistent with the algorithm.

Hi, thanks for the kind words, great questions, and great points raised.
I'll try to get to all of them eventually but for now I wanted to give you an explanation for the last example you brought up.
We derived PubTables-1M from a set of source HTML tables in the PMC dataset. Unlike most source table data, such as FinTabNet, this data did include a header annotation. In the case of PMC1079800_table_0 the original header annotation looks like this:

Median age (range), yrs	64 (60–73)

Age category, n (%)

  >60 and < 65 years	100 (56)

  ≥ 65 years	80 (44)

As you can see, in this case the original annotation is wrong with regards to the column header, which does happen quite a lot.
You're right that our algorithm for annotating headers would have only annotated the first row as being part of the header.
However, in creating PubTables-1M we tried to strike a balance between correcting and trusting the original annotations. So we only tried to extend the header to more rows, never reduce it to fewer rows. We do say this in the paper, but maybe we can make this clearer:

These assumptions enable us to determine if a row or column header is only partially annotated and if so, to extend it to additional columns or rows, respectively.

Why we only extend the header and never shrink it is we thought there was value in letting the original annotations override our simple algorithm in cases like these, in case there were edge cases we had not considered. Given that there will always be edge cases, it's not obvious if there would be more errors or fewer errors if we let our algorithm always determine the header and override the original annotators.
I hope this explains that example for you. Thanks again for the question!


Trivy Security scan error - Need proof

This is my dockerfile which I am using for installing multiple versions of node .
FROM ubuntu
ENV NVM_DIR /usr/local/nvm
RUN mkdir -p $NVM_DIR
RUN curl https://raw.githubusercontent.com/creationix/nvm/v0.39.7/install.sh | bash
&& . $NVM_DIR/nvm.sh
&& nvm install 14.21.3
&& nvm install 16.15
&& nvm install 18.16.1
&& nvm install 20.11.0
RUN chmod +x $NVM_DIR/nvm.sh
ENV NODE_14_PATH $NVM_DIR/versions/node/v14.21.3/bin
ENV PATH $NODE_14_PATH:$PATH
ENV NODE_16_PATH $NVM_DIR/versions/node/v16.15.1/bin
ENV PATH $NODE_16_PATH:$PATH
ENV NODE_18_PATH $NVM_DIR/versions/node/v18.16.1/bin
ENV PATH $NODE_18_PATH:$PATH
ENV NODE_20_PATH $NVM_DIR/versions/node/v20.11.0/bin
ENV PATH $NODE_20_PATH:$PATH
RUN export NVM_DIR="/usr/local/nvm"
&& [ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"
&& [ -s "$NVM_DIR/bash_completion" ] && . "$NVM_DIR/bash_completion"
When I execute this command below:-
trivy image --severity HIGH,CRITICAL --ignore-unfixed image_name
I am getting these vulnerabilities:-
image
My issue was closed giving reason , nvm doesn't has runtime dependencies. And it is from code side . I don't have any code and stuff .
My need is to install multiple versions of nvm inside my image and instantiate it when required.
Can you try from your end this plain Dockerfile , build it and check with trivy. Just abruptly closing the issue by giving a reason without proof doesn't help .

@ljharb closed the issue. Can you substantiate with a proof that there is no issues ?

Please don't open a new issue; comment on #3343 instead. This is just spam.
I am the sole maintainer. We have had a full security audit. I am the only person who is most qualified, on the entire planet Earth, to answer your question, and I already answered it. If you create another spam issue, I'll block you from the repo.


linux build error:  error: calling a constexpr host function("ceil_div") from a global function("nms_kernel") is not allowed.

 The experimental flag '--expt-relaxed-constexpr' can be used to allow this.

1 error detected in the compilation of "/tmp/tmpxft_00008508_00000000-6_nms_cuda.cpp1.ii".
CMakeFiles/torchvision.dir/build.make:338: recipe for target 'CMakeFiles/torchvision.dir/torchvision/csrc/cuda/nms_cuda.cu.o' failed
make[2]: *** [CMakeFiles/torchvision.dir/torchvision/csrc/cuda/nms_cuda.cu.o] Error 1
CMakeFiles/Makefile2:78: recipe for target 'CMakeFiles/torchvision.dir/all' failed
make[1]: *** [CMakeFiles/torchvision.dir/all] Error 2
Makefile:132: recipe for target 'all' failed
make: *** [all] Error 2

I checked cuda.cmake in libtorch and there is list(APPEND CUDA_NVCC_FLAGS "--expt-relaxed-constexpr"), apparently it didn't work. Any clue how to fix this?
CMakeLists.txt

This is related to https://github.com/pytorch/vision/pull/2127
@zasdfgbnm could you send a PR fixing this?

It is strange that why it is not compiling with --expt-relaxed-constexpr, but we can just make this function __host__ __device__ so that we no longer rely on --expt-relaxed-constexpr. See https://github.com/pytorch/vision/pull/2217


Move all datetime handling into a shared core module

See https://github.com/GoogleCloudPlatform/gcloud-python/pull/1021#discussion_r35710351
We have (essentially) a novel approach to datetime in every sub-package, so we should get unified.

From https://github.com/GoogleCloudPlatform/gcloud-python/pull/1044/files#r36545736: Hmm, maybe _EPOCH should be moved up to gcloud._helpers.

@tseaver I'm going to take a crack at this.

@dhermes OK, cool.


bugix/ protect internal data in multi thread env.

Map is not thread safe but some mojo are marked thread safe.
This PR use a safe structure.
PS: null check added because ConcurrentHashMap does not accept null value (diff with HashMap)

This PR is expected to fix https://github.com/mojohaus/versions/issues/1018#issuecomment-2248374042


create cache layer for non-string input hashing functions

When trying to run rule sets that contain thousands of hashes, processing time can significantly increase.  Allowing the hash module to cache the last computed value, offset, and length it can quickly return the already computed hash value avoiding the cost of recomputing for each rule.
This doesn't add any value to folks with various offsets/lengths.  For users with the same offset + size for rules, this can decrease processing time.

This solution only stores a single hash of each type (sha256/sha1/md5). A solution using a hash table is implemented in 22ce5e09dd49ccfb175524e8c6066984d1ecfb63

We debated trying to do something like this but decided against it on the assumption rules likely wouldn't have that many offset/length variations.  There was also the part of figuring out how to implement the hash table.  Thanks for putting something together and pulling it into the main branch!


Add frontend testing, require node 12

Add basic frontend unit testing infrastructure using jest in ESM mode
Rename 'make test' to 'make test-backend'
Introduce 'make test-frontend' and 'make test' that runs both
Bump Node.js requirement to v12. v10 will be EOL in less than a month.
Convert all build-related JS files to ESM.

I opted to run frontend tests run as part of the compliance pipeline because they complete fast and are not platform-specific like the golang tests.
make test seems to never be triggered in drone, so this rename only affects users.

Labeling as breaking because of the bump to the minimum Node.js version.

do we actually need to up the required version of node for this change?

bump of node version in docs is needed too

do we actually need to up the required version of node for this change?

Yes, bump is kind of required unless I want to opt to configure jest in CJS (legacy) mode as the ESM mode requires a feature only present in Node 12.16.0 or above.

bump of node version in docs is needed too

I think I handled this with the change in docs/config.yaml, or did I miss anything?

node eof ref: https://nodejs.org/en/about/releases/

Drone is already done: https://drone.gitea.io/go-gitea/gitea/38171/1/9
It's executed as part of the compliance pipeline because it's not platform dependant like the go tests are. Maybe we should move it to the actual test pipelines later for consistency.


Remove extra variable

Don’t create rand, just set the seed of the random module itself.

Hey Benjamin,
I will take care of this issue.
Nelaven


Unable to generate accessibility reports for dynamic web pages

Hi,
This tools is really awesome and works perfectly fine for static web pages. But i am facing difficulty in generating reports for dynamic web pages.
What would you recommend?
Is there any update coming sometime soon which is related to the above problem?
Thanks,
Sameer

@supasani what are you looking for when you say dynamic pages?


Common ancestor for Sun and Moon

It would be practical if Sun and Moon had a common ancestor that had properties that both classes have the same (date, altitude, rise, set ....).
Right now I have two almost identical methods, although one would be enough.
Thanks, Michal Kůs

Hi Michal, we are aware that could be better to have a common ancestor for both Sun and Moon class, they also share the same codebase, but keep them separated is an our choice.  For now they will kept separated, in the future we will consider to join them.
Thanks :)


chore: Test 3.3.4-RC1

Motivation:
Test 3.3.4-RC1

This is not for merge, right? Might be good to mark it 'draft'

Yes , I want to expose some compiler issue

see #612


Crashed: NSOperationQueue 0x283a84820 (QOS: UNSPECIFIED)

Crashlytics has reported on the following crash:
Crashed: NSOperationQueue 0x283a84820 (QOS: UNSPECIFIED)
EXC_BAD_ACCESS KERN_INVALID_ADDRESS 0x0000000aa4826e30

0  libobjc.A.dylib                0x1a3840d70 objc_msgSend + 16
1  Starscream                     0x1055ac1a8 $S10Starscream16FoundationStreamCAA8WSStreamA2aDP5write4dataSi0B04DataV_tFTW (WebSocket.swift:233)
2  Starscream                     0x1055b26f0 $S10Starscream9WebSocketC12dequeueWrite33_E91840E7069A0D2D302D042308E9D928LL_4code15writeCompletiony10Foundation4DataV_AC6OpCodeOyycSgtFyycfU_ (WebSocket.swift:1254)
3  Starscream                     0x1055a9e64 $SIeg_IeyB_TR (<compiler-generated>)
4  Foundation                     0x1a50e682c __NSBLOCKOPERATION_IS_CALLING_OUT_TO_A_BLOCK__ + 16
5  Foundation                     0x1a4feea28 -[NSBlockOperation main] + 72
6  Foundation                     0x1a4fedefc -[__NSOperationInternal _start:] + 740
7  Foundation                     0x1a50e8700 __NSOQSchedule_f + 272
8  libdispatch.dylib              0x1a40916c8 _dispatch_call_block_and_release + 24
9  libdispatch.dylib              0x1a4092484 _dispatch_client_callout + 16
10 libdispatch.dylib              0x1a4068e14 _dispatch_continuation_pop$VARIANT$armv81 + 404
11 libdispatch.dylib              0x1a40684f8 _dispatch_async_redirect_invoke + 592
12 libdispatch.dylib              0x1a4074afc _dispatch_root_queue_drain + 344
13 libdispatch.dylib              0x1a407535c _dispatch_worker_thread2 + 116
14 libsystem_pthread.dylib        0x1a427417c _pthread_wqthread + 472
15 libsystem_pthread.dylib        0x1a4276cec start_wqthread + 4

This has happened on an iPhone X running iOS 12.1.0. The version of Starscream is 3.0.5.

same here

I have similar issues. My assumption is that need to move to Network.framework as soon as possible.

Same issue here:

Same issue here also, any help will be appreciated, thanks

It seems  should need a read-write lock for outputStrream and inputStream, as the function cleanup() may hang up, and at the same time  other function execute some code to access the variables.

Check out #653. It completely changes the way this is handled and hopefully should resolve this issue.

Hey guys!
How did you done https://github.com/daltoniam/Starscream/issues/671 task??

Version 4.0.0 has been released which removed the use of the operation queues which should resolve this specific crash. Thanks (and also for the patience of a new release as it has been a while!)

Same issue here also, any help, advance thanks


Select: Select component trembles on zoom (#17856)

PR without descriptions and refer relative issues  will be closed directly.


Dockerfile: Ensure that the .ort data directory exists

It is common practice to mount a host-side ORT config directory into the Docker container at /home/ort/.ort/config. While Docker infers the permissions for the config directory from the corresponding directory on the host, for the intermediate .ort directory (which did not exist in the container prior to this change) there is no corresponding directory on the host, so Docker uses root:root ownership. Fix the ownership to be ort:ort by ensuring that the .ort directory exists in the container.
Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

/cc @borisbaldassari

Not sure this is a good change

Why not? What's your concern? It has been tested to fix a real permission issue.

at minimum it should be a mkdir -p which fails silently

Done.

Hmm, this seem to indeed break the CI tests:
java.io.FileNotFoundException: /home/ort/.ort/cache/analyzer/maven/remote-artifacts/journal.tmp (No such file or directory)

Very weird, Docker drives my crazy :zany_face:

Hmm, this seem to indeed break the CI tests:

Fixed.

Not sure this is a good change

Why not? What's your concern? It has been tested to fix a real permission issue.

at minimum it should be a mkdir -p which fails silently

Done.

Any update @tsteenbe? Tests pass now.


docs: (IAC-507) Change empty tags {} map in all example tfvars

Changes
Add doc that the value for the tags variable cannot be an empty list and to leave the tags variable commented out in each .tfvars file if no tags will be specified.
Add a valid "key" = "value" argument to the tags variable and comment it out so that by default, the value for tags from variables.tf will be used.
Testing
Will mix and match sample input file testing for coverage across all three cloud providers.
Verified that using a tags value defined as an empty list generates the expected error for sample-input-minimal.tfvars
Error: Invalid value for variable

  on variables.tf line 109:
 109: variable "tags" {
    ├────────────────
    │ var.tags is empty map of dynamic

ERROR: You must provide at last one tag.

This was checked by the validation rule at variables.tf:114,3-13.}

Development tests with no tags value defined in sample-*.tfvars file. See internal ticket for additional detail.

scenario
sample file
.tfvars tags value
AWS tags used
Result

1
sample-input-minimal.tfvars
none, commented out
default tag from variables.tf
cluster creation was successful with 3 expected node groups - cas,default,and generic

2
sample-input-ha.tfvars
none, commented out
default tag from variables.tf
cluster creation failed after 35 minutes with the Error: message below. Issue is related to files/custom-data/additional_userdata.sh content. IAC-974 was opened to track the issue. Re-ran without custom-data file and cluster creation was successful with all expected node groups

3
sample-input.tfvars
none, commented out
default tag from variables.tf
cluster creation was successful with 5 expected node groups - compute,cas,stateless,stateful,default

4
sample-input-singlestore.tfvars
none, commented out
default tag from variables.tf
cluster creation was successful with 6 expected node groups - compute,cas,stateless,stateful,default,singlestore

5
sample-input-gpu.tfvars
none, commented out

Not tested

6
sample-input-connect.tfvars
none, commented out

Not tested

7
sample-input-custom-data.tfvars
none, commented out

Not tested

8
sample-input-byo.tfvars
none, commented out

Not tested

@thpang, please let me know if CONFIG-VARS.md update reflects what we discussed, ready for re-review.


Add shrinkwrap to the makefile

Steps to implement:

Fork the repo
Change the deps command in the makefile to the following:
rm -rf node_modules/ npm-shrinkwrap.json && npm install && npm shrinkwrap
Make sure it runs, and open a PR

That's it and welcome to open source :grinning: :tada:

Great first timers only! You've probably already considered this but I do have a small concern.
Do we definitely want this in the deps task? It means whenever someone clones and runs deps they're potentially making a change to the repo, potentially a chance to the code and therefore potentially a change to functionality. This would defeat essentially the object of shrinkwrap - keeping the code exactly the same for a particular commit.
This is definitely useful, but we need to ensure people don't clone then immediately do this when all they wanna do is reproduce and track down a bug which may well be caused by a specific version of a dependency.
See what I mean? Or am I missing something?

:100:
I've updated the issue accordingly.
We need to update the documentation to include a mention of the makefile btw

Yeah you're right we do. I can do that quickly no prob!
Good update - I can get fully behind this! :)

Hello! Gonna twoot bout this from @youfirstpr today! Thanks so much for making things.

Thank you. What time are you tweeting as I may have some more for you :wink:

I only do one a day, but as my next one would be on Christmas day I could do a bonus round. Tweets usually go out at 1PM :)

I will try and have it ready by then :)

Ahhhh so much activity. I will find a coffee shop and try to help :P
On 23 Dec 2015 10:31, "Mordechai Zuber"<EMAIL_ADDRESS>wrote:

I will try and have it ready by then :)
—
Reply to this email directly or view it on GitHub
https://github.com/basicallydan/forkability/issues/76#issuecomment-166857725
.

Thank you @Charlotteis for the ol' tweetage :)

Think this can be closed now :)

Yes! Thanks Charlotte!
See you some time at some event in London I hope!
On 2 Jan 2016 11:54, "Charlotte Spencer"<EMAIL_ADDRESS>wrote:

Think this can be closed now :)
—
Reply to this email directly or view it on GitHub
https://github.com/basicallydan/forkability/issues/76#issuecomment-168384027
.

So it should be closed then? :stuck_out_tongue_closed_eyes:

Haha sorry I thought Charlotte had closed it too!
--Dan
danhough.com http://www.danhough.com - @basicallydan
http://twitter.com/basicallydan
On 3 January 2016 at 16:21, Mordechai Zuber<EMAIL_ADDRESS>wrote:

Closed #76 https://github.com/basicallydan/forkability/issues/76.
—
Reply to this email directly or view it on GitHub
https://github.com/basicallydan/forkability/issues/76#event-504230023.

I don't have those permissions!


FIX: oneapi_backend building WIN

Description
icx was ignoring -fsycl for WIN build of oneapi_backend. -fsycl is required for sycl compilation.
Since icx MVCS compatible driver, so it is needed add extra link args before /link command.

https://ecmd.jf.intel.com/commander/link/jobDetails/jobs/ed7cb862-f456-f17e-b838-a4bf010d0e2e?objectId=job-ed7cb862-f456-f17e-b838-a4bf010d0e2e&filterName0=jobsPageSearch&filterDepth=1

http://intel-ci.intel.com/ed7cb862-f456-f17e-b838-a4bf010d0e2e

http://intel-ci.intel.com/ed7d387a-6d09-f1a7-93b7-a4bf010d0e2e


Leverage "Optional" functionality while generating multiple resource type names

Tool version
v4.0.0
Is your feature request related to a problem? Please describe.
Optional or excluded components are required during the filling of fields (Generate section), when the Generate Multiple Resource Type Names option is selected. The names of excluded components do not appear in the generated names, ALWAYS.
It may happen instead that the name of a component sometimes needs to be included, while at other times it should be omitted in generated names.
So, it would be useful to extend the usage of 'optional' component when the Generate Multiple Resource Type Names option is selected.
Regards,
Luca

Hi Luca,
This is already part of our backlog, the current tool design makes implementation difficult. We will continue to evaluate this capability for future releases.

Bryan


Specify text for multiple siblings with list

I'm want to create this simple html with Emmet.
    a
    b
    c

I would imagine it to be possible with a syntax like this.
 td{a,b,c}
But this only produces the output of a,b,c	, which is not what I desire. Is this at possible with Emmet?

You should write a, b and c each on new line, select them and wrap with abbreviation.
Or write td*3 them tab & write content
Sent from my iPhone

On 15 Aug 2018, at 01:09, samurdhilbk<EMAIL_ADDRESS>wrote:
I'm want to do a simple thing with Emmet.
I want to create this html.
a
b
c

I would imagine it to be possible with a syntax like this.
td{a,b,c}
But this only produces the output of a,b,c, which is not what I desire. Is this at possible with Emmet?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Welcome screen links don't work

When ever I try to open one of them, a tab opens then immediately closes.

close your ad blocking software and try again.


Table.head() failure when number of rows < default

from cogent3 import make_table

table = make_table(data=dict(a=['a'], b=['b']))
table.head()

raises an index error. I fully expect that table.tail() will fail also.

Hi Gavin! this works well for me, no index error raised, however, I think this could potentially be a bug:

table = make_table(data=dict(a=[i for i in range(10)], b=[i for i in range(10)]))
table.head(nrows=5)

it turns out the output has got ten rows.

oops, sorry Gavin, I spotted the bug, only raised when using a notebook.
I am on the way of fixing, cheers.


Input not working when out of focus

EDIT : I tried with a totally new project, I only have one script that is the exact script from your example project, I ticked the "run in background" option in player settings and it still doesn't register input if I click another window
Maybe the project is outdated or I didn't understand something.
I everything exactly like in your example project, everything works fine but if I lose focus, it stops working.
Here is the code where I use it
using System;
using System.Collections;
using System.Collections.Generic;
using System.Runtime.InteropServices;
using UnityEngine;
using UnityRawInput;

[System.Serializable]

public struct WindowSavedStruct
{
    public string title;
    public IntPtr hWnd;
    public WindowSavedStruct(string title, IntPtr hWnd)
    {
        this.title = title;
        this.hWnd = hWnd;
    }
}
public class WindowSwitcher : MonoBehaviour
{
    public bool WorkInBackground = true;
    public bool InterceptMessages = false;

    [DllImport("user32.dll")]
    internal static extern IntPtr FindWindow(string lpClassName, string lpWindowName);
    [DllImport("user32.dll")]
    internal static extern bool ShowWindow(IntPtr hWnd, int nCmdShow);
    [DllImport("user32.dll")]
    internal static extern IntPtr SetForegroundWindow(IntPtr hWnd);

    public EasySwitchMenu easySwitchMenu;
    public int selectedWindowIndex = 0;
    public GameObject DofusProcessUIPositionPrefab;
    public GameObject gridGameobject;

    public List<WindowSavedStruct> dofusWindowsList = new List<WindowSavedStruct>();

    public void OnEnable()
    {
        RawInput.WorkInBackground = WorkInBackground;
        RawInput.InterceptMessages = InterceptMessages;

        RawInput.OnKeyUp += LogKeyUp;
        RawInput.OnKeyDown += LogKeyDown;
        RawInput.OnKeyDown += DisableIntercept;

        RawInput.Start();
    }

    private void OnDisable()
    {
        RawInput.Stop();

        RawInput.OnKeyUp -= LogKeyUp;
        RawInput.OnKeyDown -= LogKeyDown;
        RawInput.OnKeyUp -= DisableIntercept;
    }

    private void OnValidate()
    {
        // Apply options when toggles are clicked in editor.
        // OnValidate is invoked only in the editor (won't affect build).
        RawInput.InterceptMessages = InterceptMessages;
        RawInput.WorkInBackground = WorkInBackground;
    }

    private void LogKeyUp(RawKey key)
    {
        Debug.Log("Key Up: " + key);
    }

    private void LogKeyDown(RawKey key)
    {
        Debug.Log("Key Down: " + key);
    }

    private void DisableIntercept(RawKey key)
    {
        if (RawInput.InterceptMessages && key == RawKey.Escape)
            RawInput.InterceptMessages = InterceptMessages = false;
    }
    public void CapturedKeyPressed(RawKey key)
    {
        Debug.Log(key);
        if (key == RawKey.W)
        {
            SwitchToNextWindow();
        }

    }

    private void SwitchToNextWindow()
    {
        if (selectedWindowIndex < dofusWindowsList.Count - 1)
        {
            selectedWindowIndex++;
        }
        else
        {
            selectedWindowIndex = 0;
        }

        Debug.Log("Switching to window: " + dofusWindowsList[selectedWindowIndex].title);
        SwitchToWindow(dofusWindowsList[selectedWindowIndex].hWnd);
    }

    private void SwitchToWindow(IntPtr hWnd)
    {
        if (hWnd != IntPtr.Zero)
        {
            SetForegroundWindow(hWnd);
            ShowWindow(hWnd, 5);
            SetForegroundWindow(easySwitchMenu.AppWindow);

            Debug.Log(hWnd);

        }
        else
        {
            Debug.LogWarning("GetFocusWin() failed, hWnd is 0!");
        }
    }

    public void RemoveClosedWindows()
    {
        List<WindowSavedStruct> tempWindowSaved = new List<WindowSavedStruct>();

        foreach (WindowSavedStruct window in dofusWindowsList)
        {
            if (FindWindow(null, window.title) == IntPtr.Zero)
            {
                tempWindowSaved.Add(window);
            }
        }

        foreach (WindowSavedStruct window in tempWindowSaved)
        {
            if (FindWindow(null, window.title) == IntPtr.Zero)
            {
                dofusWindowsList.Remove(window);
            }
        }

        RefreshUI();
    }

    public void RefreshUI()
    {
        // Delete all children gameobjects from gridGameobject
        foreach (Transform child in gridGameobject.transform)
        {
            Destroy(child.gameObject);
        }

        int index = 0;

        foreach (WindowSavedStruct window in dofusWindowsList)
        {
            GameObject newDofusProcessUIPosition = Instantiate(DofusProcessUIPositionPrefab, gridGameobject.transform);
            newDofusProcessUIPosition.GetComponent<DofusProcessObject>().Initialize(window.title, index, this);
            index++;
        }
    }
}

I made a huge rookie mistake. I simply didn't tick the workInBackground option in the inspector and it was public


Issue deadlock (#6173)

Pull request opened by the merge tool on behalf of #6173

Merged into branches iso6 in 9ffb1cf468c35dc3a4868c901d5793ab44a0697e


Update README for Syntax highlighting

Closes #367
I had to dig a bit to find out that markdown-to-jsx does this.  It also passes lang- (ie lang-json) as an attribute to the DOM node for <code> but I left that out for the sake of brevity.

Some element mappings are different frommdx-js , being that markdown-to-jsx doesn't have its own table, it's natural to assume that people would assume the table from mdx-js works.  It's useful to point out the differences.
It's probably preferable for mdx-js to have its own table mapping of components.  If you agree I can add one.


fix(changelog): correctly set the tag message for the latest release

Description
Now we support the following use case:
$ git cliff --unreleased --tag 1.2.3 --tag-message "Release 1.2.3"

Motivation and Context
closes #836
How Has This Been Tested?
Locally.
Types of Changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (no code change)
[ ] Refactor (refactoring production code)
[ ] Other

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[x] I have formatted the code with rustfmt.
[x] I checked the lints with clippy.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 39.92%. Comparing base (fe37d14) to head (c0a02a3).

Files with missing lines
Patch %
Lines

git-cliff/src/lib.rs
0.00%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #854      +/-   ##
==========================================
- Coverage   39.94%   39.92%   -0.02%
==========================================
  Files          20       20
  Lines        1660     1661       +1
==========================================
  Hits          663      663
- Misses        997      998       +1

Flag
Coverage Δ

unit-tests
39.92% <0.00%> (-0.02%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update zap delete paths of WebStorm

Since WebStorm 2016.1, the plist file created under ~/Library/Preferences/ is renamed as jetbrains.webstorm.cdacf2ec.plist instead of the old com.jetbrains.webstorm.plist, and it looks like that the random string appears in the middle of the name is version specific.
I also add the command line launcher executable, it's usually created when you confirm the config dialog appeared after the registration.

I know it's not a preference file, but shouldn't zap delete perform a clean uninstall that remove every file created by the cask?

No, zap is to delete user-created files. Since that isn’t one, it should be in uninstall, not zap.

Thanks for your explanation, so would it be valid if I move it to the uninstall field instead?

Thanks for your explanation, so would it be valid if I move it to the uninstall field instead?

Yes. We should likely also do it for all other JetBrains casks.

If I want to change other JetBrains cask files, should I open a single pull request or open standalone ones for each cask instead?

We prefer one PR per cask. However, if you really want to make a single PR, at least make each cask change a different commit.

Rebased, and I will open another pull request to fix related JetBrains casks later on today.

Thank you for the contribution. It was merged directly as 2a728edfbca937f71602e4b889d36d6a03f28f96 to keep commit history cleaner. Your contribution is still credited to you.


feat: Add mul_inv and inv_mul versions of div_le_one_of_le

These are exactly div_le_one_of_le with a / b replaced by a * b⁻¹ and b⁻¹ * a.
They come up frequently because some tactics produce inverses as normal forms over division.  The iff versions with 0 < b already exist as mul_inv_le_one_iff and inv_mul_le_one_iff; the point of these is to have a weaker nonnegativity assumption on b.

Possibly I should add inv_mul_le_one_of_le too?

Yes, and iff versions assuming 0 < b. Also please edit the PR title, I cannot comprehend it 😄

Is that title better?
I'll add the inv_mul version now.  The iff versions already exist: mul inv, inv mul.

Now that's better!

@YaelDillies: Would you be up for a review and approval if it seems reasonable?

What’s the next step after “maintainer merge”?

Nothing on your part. A maintainer will merge the PR or come up with a comment I hadn't thought of.

Thanks! :tada:
bors merge


Add toast component to show notifications

New Behavior
Description
Displays a toast notification at the bottom of the page. This was designed to solve #527 where the requirement is to show a toast message in the bottom of the page. So, this component is currently limited to solve that purpose. Later if required for other purposes, we can generalize it further by adding a container to show multiple toast messages at the same time and also enable customization to choose location of the toasts. Eg: top-left, bottom-right, etc.,
Screenshots

Hi @rohithbalaji123 , thank you so much for build this cool component! I left few comments on the PR.


fix segfault on missing CSP

These routines:
  NCryptOpenStorageProvider()
  winpr_NCryptOpenStorageProviderEx()

would segfault with PKCS11 support enabled, if the user did not supply a service provider name ("CSP"); they assumed the parameter pszProviderName would always be non-NULL, but it's NULL in that case.

@freerdp-bot test please

Oh good catch, I'm wondering how I've not seen it before

@freerdp-bot test again please

@freerdp-bot ok, please test this nice pull request again now that it is properly formatted.


[FLINK-35077][cdc] Add license checking CI scripts

Currently, Flink project has CI scripts checking if dependencies with incompatible licenses are introduced.
Flink CDC module heavily relies on external libraries (especially connectors), so running similar checking scripts during every CI would be helpful preventing developers introducing questionable dependencies by accident.

Hi @PatrickRen, I just pushed a hotfix commit to lock dependency version and suppress potential  breaking changes as follows:
The public API of some Rubyzip classes has been modernized to use named
parameters for optional arguments. Please check your usage of the
following classes:
  * `Zip::File`
  * `Zip::Entry`
  * `Zip::InputStream`
  * `Zip::OutputStream`

Please ensure that your Gemfiles and .gemspecs are suitably restrictive
to avoid an unexpected breakage when 3.0 is released (e.g. ~> 2.3.0).
See https://github.com/rubyzip/rubyzip for details. The Changelog also
lists other enhancements and bugfixes that have been implemented since
version 2.3.0.

Could you please help re-run the CI? Thanks!


Changing appearance doesn't do anything (Lion)

See screenshot: http://cl.ly/1B3i2r2V0S060L2S3F1c

Not all these options work on every interface. I'm pretty sure the two you changed don't affect the Bezel interface. Have you tried any of the other options?

This worked before on SL and briefly it worked in Lion, but Lion messed it all up ;)

Ha fail! Thanks!


Experimental feature for unboxing.

This is what I had in mind for #103.
@greghaskins - does it majorly suck, or does it add value?

Nice.
Does it work with concrete types or only interfaces? I'm thinking people would really like to use this for mocks, which are frequently (with Mockito at least) concrete types.
Still mulling over the unbox() entry point. Not the worst thing, but another import/function for people to learn. My first thought was overloading let like let(Something.class, () -> new Something()).
There may also be some sneaky tricks to even avoid passing in that Class object; I feel like maybe Hamcrest or something has some code that just works with generics somehow – unbox(let(() -> new Something()); would be pretty slick.

The inbox function as provided can be used to surround any supplier, of which both let and junitMixin are such... but we could provide overloads to wrap it further.
It can ONLY work for interfaces. This is a technical constraint to do with dynamic proxies, but also makes sense if you think about how concrete types can have accessible instance variables and multiple constructors, which even Mockito has limitations over.
The type must be specified in the call because the stuff you mentioned about inferring type only works for generic types themselves, not for actual Class objects. In other words, to know at run time the interface to supply, you need a Class object. Even Mockito.mock requires a class object as input.

My first thought was overloading let like let(Something.class, () -> new Something()).

I've extended my POC to have the equivalent of this - I reverse the order of parameters from the above, because that's what felt natural, but it's easily refactored. I did it for junitMixin and let, but left the unboxer documented and public as this tool could conceivably be used by end-users for their own combinations of things.
How does it look?

Understood ✅ . A couple thoughts:

If it only works with interfaces, we should fail fast (throw an IllegalArgumentException) to help people realize that quickly, rather than getting a stacktrace from deep in the standard library.
Since this is experimental, and limited to just interfaces (for now – back of my mind is already wondering how to use ByteBuddy down the road...), let's just use a single entry point like unbox. If this feature takes off and people really like it, we can add the convenience overloads of let and junitMixin.
I don't know what the best name for unbox() would be, but I can't think of anything significantly better. unwrap(), proxy(), get(), and from() come to mind; I'm not in love with any of them.

So, I should remove the overloads of let and junitmixin? It is easy to check if the passed in class is an interface.
Naming the method was hard. It's an auto get or an unwrapper or a de supplier or... nothing sounded good. Accessor?


Migration step 22

#104
migration step 22 to 24

Please make all the unit tests in steps/unit-tests fail by adding a expect(true).toBe(false) and update the slides accordingly, making the goal of the exercise simpler to understand: "just fix the tests" (with a step by step of course).

Please make all the unit tests in steps/unit-tests fail by adding a expect(true).toBe(false) and update the slides accordingly, making the goal of the exercise simpler to understand: "just fix the tests" (with a step by step of course).

done too ;)


⚠ Processors has degraded performance

In f642022, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 50 ms

Resolved: Processors performance has improved in 8acc89d.


Update to work with latest Phoenix.js

This fixes a few issues and warnings to work with Phoenix 1.2

Can you squash all these commits please and I'll merge it in! I tried doing it myself but it no longer count's as your merge 😞

Sure, np!

I'll create a new PR, I did it in another branch


Better error handling and error messages when network connection is lost

Since IFM Maps is used heavily on mobile devices users from time to time lose their network connection when using the app (or the index.html is simply cached) and the XHR/AJAX requests fail with error messages that give no indication of what the problem is.

One solution could be to use Offline.js: http://github.hubspot.com/offline/docs/welcome/
We should however also improve the error handling in the AJAX requests:
http://stackoverflow.com/questions/1730692/jquery-ajax-how-to-detect-network-connection-error-when-making-ajax-call


Update bson to 4.10.0

About this PR
📦 Updates org.mongodb:bson from 4.2.3 to 4.10.0
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.mongodb", artifactId = "bson" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.mongodb", artifactId = "bson" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #171.

Superseded by #176.


Add a regression test to ensure compression is enabled

We added a regression test to the Kopia repo but should add one to Corso as well to ensure that if Kopia behavior changes - we catch it.

I was mucking with this yesterday a little bit and couldn't find a good signal on it with what we already have available (byte counter for uploaded data). We may need to do some more digging on how to accomplish this


Dashboard guild.farewell.embed.description not save

In the dashboard, in the embed farewell, in the descriptions field it does not save. But everything else yes

Fixed


Save Button in the pdf viewer does not work

Does someone has an idea, why the save button in the pdf viewer is not working.
After creating the Data and putting it into an iframe, I want to open the pdf in a new window.
So far it works well, but in the new window the pdf save button in the pdf viewer is not working at all.
Also not the save as.. functionality.
This is the code I have used for
Chrome is my standard browser
var sDataURI = oPDF.output('datauristring'); var iframe = "<iframe width='100%' height='100%' src='" + sDataURI + "'></iframe>"; var oNewwindow = window.open(); oNewwindow.document.open(); oNewwindow.document.write(iframe); oNewwindow.document.title = sTitle; oNewwindow.document.close();
Many thanks in advance for any kind of hint.

I wouldnt know where to "fix" this. Seems to be more like pdf viewer problem than a pdf generation problem.


Does not profile iOS app in release configuration ("profilerMode": "trace")

Does not profile iOS app in release configuration ("profilerMode": "trace"); but works for iOS (Debug). Also works for Android Release / Debug both configurations.
Shows below output in debug console on vs code -
/usr/local/share/dotnet/packs/Microsoft.iOS.Sdk.net8.0_18.0/18.0.8303/tools/bin/mlaunch
WARNING: dotnet-dsrouter is a development tool not intended for production environments.
info: dotnet-dsrouter-14198[0]
Starting dotnet-dsrouter using pid=14198
info: dotnet-dsrouter-14198[0]
Starting IPC server (/Users/username/ios-port.lock) <--> TCP client (<IP_ADDRESS>:9000) router.
info: dotnet-dsrouter-14198[0]
Successfully connected new device, id=2.
But does not create speedscope file at end of session
using below launch configuration -
{
"name": ".NET Meteor Profiler",
"type": "dotnet-meteor.debugger",
"request": "launch",
"profilerMode": "trace",
"preLaunchTask": "dotnet-meteor: Build"
}
Softwares -
VS Code - Version: 1.94.2 (Universal)
.NET Meteor - v5.7.0
Xcode - Version 16.0 (16A242d)
dotnet workload list
maui                       8.0.82/8.0.100        SDK 8.0.400

Hi, @nikamvd ! It is iOS device or simulator?

Hi, @nikamvd ! It is iOS device or simulator?

Its physical device: iPhone 12; iOS 16.3

https://github.com/xamarin/xamarin-macios/issues/21530

Fixed in 5.8.2


added known facet for bitwarden

bitwarden is a popular open source password manager that supports FIDO U2F security keys. This PR adds the facet endpoint for bitwarden as described in the README.

Thanks!


Add link to community post on building a framework with vinxi

Instead of only showing coming soon, also have a link to this great post for now:
Community post on the topic: Bullding a React Metaframework with Vinxi

how does changeset work for these doc changes where I don't want to bump any of the packages?

how does changeset work for these doc changes where I don't want to bump any of the packages?

We don't add any changesets for docs stuff

Thanks for this! Been meaning to add that


Skip empty data ({}) in text logging

We should skip empty data in text logs. E.g. have
INFO  Index check found zero issues, nice job!

Instead of the current:
INFO  Index check found zero issues, nice job! {}

It reads better

This is done


Google Analytics

Feature request
Feature description

Remove Google Analytics
Hello! I regularly use CDNJS, so I felt like this is an issue I really should speak on.
I'm quite confused as to why this brilliant resource uses Google Analytics, which tracks people across the internet, becoming the subject of million-dollar lawsuits. I'm quite worried that we are setting a bad example here.
Personally, I think we, as a community, should research alternatives to Google Analytics.
Thanks! :D

The plan is to continue using GA, it's industry standard and what we've used for a long time. If you don't like it, you are welcome to use a browser extension to block it -- we do nothing funky to force it.


Manage MFA user settings - ambiquous location of step 2

There isn't a clear indication of where one can find the MFA settings in this section. There needs to be an assumption that users are coming here after already setting it up months ago, and telling them to go to a page which isn't clearly marked isn't very useful.

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

ID: 3ed95865-eddb-4ad1-d2e2-5bd618151c1b
Version Independent ID: ef32bb26-e78a-caad-5b41-d7e49f6ef9b4
Content: Set up multi-factor authentication for Office 365 users
Content Source: Office365-Admin/security-and-compliance/set-up-multi-factor-authentication.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

Please fix this. I have no way of getting to the manage MFA portal in this new UI.

Under Manage MFA Features, step 2 has "On the multi-factor authentication page, select the check box next to the people you want to manage." What is the multi-factor authentication page and how do you get to it?

@StuBeck32 can you explain exactly what isn't clear? The page has a link to your Active users in the O365 Admin Center and a pictorial about how to set a user up with MFA.

Can you please explain what part of my comment you’re misunderstanding? I copied the step and the part that I’m not understanding, so I’m not sure how to be more clear.

@StuBeck32 - in the preview mode it is difficult to find - the topic has been updated. Does it seem more clear now?

Yes, this is much more clear @Sirkkuw . I'm still confused as to what @tseward needed more details on.


dict: 64bit atomic operations must be 8-byte aligned.

Atomic operations on non-8 byte aligned values results in segfaults on some
32bit architectures. This aligns all 64 atomic operations as well as places
compile time checks to prevent regression.

How does that work? Is the const expected to be unsigned and if it's negative cause a compilation error? Is there precedent for this in other code that we could reference?

Because unsafe.Offset returns a uintptr, the - cause a compiler error (negative can't be assigned to unsigned or some such) when the offset is off. I'm not sure if there is a precedent other than fixing it and hopefully documenting it somewhere. I tried to make the failure a little obvious by adding a comment to the line. Much better IMHO than the random panics I was seeing which didn't say anything about alignment.
I could also see this being in a unit test/test code, but it would have the chance of being separated from the variables and this seemed like an easy way to place the "requirement" near the code which is placing the requirement.

SGTM. Can you just add a more detailed comment describing the intent and mechanism?


Update ScrollTo.elm

@simplystuart  Issue#1
Create initFor (alternative to init) which takes an id of a scrollable element as an argument.

@simplystuart
There would be a problem with this if it were used for more than one element, or for the window and another element.  I have developed and tested a further version which deals with this problem.  It requires that a msg is defined for each element/window and that Status becomes Status msg, and init and initFor take a msg constructor as an argument.  Obviously, ScrollTo.subscriptions needs calling for each window/element.
I wanted you to be able to review the versions separately, but being a bit of a Github newbie, I'm not sure of the best way to effect this.


Layout Issing on iPad portrait mode

Register:

Edit Region Details:

Club Dashboard:

Edit Team :

League dashboard:

Edit league:

League Briefing. (Club, Region):


Split testing-base into separate infrastructure project

Previously :testing-base included both code that ran in the gradle daemon and in the test worker. This meant that test worker code could accidentally depend on code that it should not -- like things from :core. Previously, developers had to be sure to not accidentally depend on these restricted dependencies, and the build did not enforce that boundry.
This commit splits :testing-base into a separate :testing-base-infrastructure, where the infrastructure project contains all code that runs on workers and does not depend on projects that are not intended to run in workers.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build


Allow omitting count query in PaginatedCriteriaBuilder

The total count might not always be necessary. Introduce a method e.g. withCount(boolean) that can be used to specify the desired behavior. PagedList#getTotalSize() should return -1 if it is disabled.

That's especially important now because the spring data integration has an option for omitting the count: https://www.jhipster.tech/tips/019_tip_infinite_scroll_with_slice.html


Add NoEmitOnErrorsPlugin documentation

@simon04 on the assumption that at some point we may implement something like #1017 (and also the convention helps with consistency/readability), I think @bebraw and I are both slowly shifting our commit messages over to the standard defined here.
I'm starting to use the squash and merge option to rewrite prs that don't follow the convention on merge, so if you could try that as well it would be helpful. You don't necessarily have to follow the convention on each branch but we should at least attempt to stick to it on master.


Improve fleet integration management

Situation 1: Mandatory Definition of All Integration Inputs
Customer is encountering an "issue" where they must explicitly define all integration inputs within their Fleet Terraform configurations, even for those that are not enabled. This is necessary to prevent Terraform from showing diffs between plans. For example, in the AWS integration, which offers 50 different inputs, they only enable 4. Despite this, they need to define every input, importing their default values, to avoid plan discrepancies.
Here's an example of the issue with a disabled input still needing to be defined:
resource "elasticstack_fleet_integration_policy" "server-system" {
  name              = "system"
  namespace         = "default"
  description       = "example"
  agent_policy_id   = elasticstack_fleet_agent_policy.server_policy.id
  integration_name  = elasticstack_fleet_integration.this["system"].name
  integration_version = elasticstack_fleet_integration.this["system"].version

  input {
    enabled       = true
    input_id      = "system-logfile"
    streams_json  = jsonencode(jsondecode(file("integration_configs/server/system/system-logfile.json")))
  }
  ...
  input {
    enabled       = false
    input_id      = "system-httpjson"
    streams_json  = jsonencode(jsondecode(file("integration_configs/server/system/system-httpjson.json")))
  }
}

If customer remove the last input and its associated streams, the plan continuously shows a diff suggesting the removal of the undefined input. This requires customer to "backport" the default streams for all inputs, even disabled ones, and to update them with each integration version change.

This seems to also affect multiple streams within a single input (e.g. the Prometheus integration).
Would love to omit the disabled-by-default streams from the streams_json argument. Not sure how easy would that be given this is a json string and probably not part of the schema. Maybe worth having streams as blocks just for the enabled argument and the rest of their config could be kept as json.