Running when Python 3 is the default

It doesn't take much to make grumpy work under a non-default installation of 2.7, so I thought it could be somehow supported or at least described in the README.
I have Python 3 as my default (as in, python or /usr/bin/env python will invoke Python 3), but I do have python2.7 installed as well. I presume this is a setup common for a lot of people, so I thought grumpy could be supported on system configurations like this.
There are only two changes needed:

in the Makefile, all calls to @python need to be @python2.7 - but to make this universal, the invocation of python here could be parametrised by using a variable, so that one could set the python version of choice in the Makefile header.
tools/* use #!/usr/bin/env python, which will point to Python 3, so I had to change this to /usr/bin/env python2.7 and I'm not quite sure what's the best way to make this parametrised. Perhaps using the aforementioned variable and using find/replace when copying the tool scripts to the build folder

I hardcoded these two changes and tested it out, worked like a charm.

Does anybody have a sense whether the name python2.7 is always available when Python 2.7 is installed? Or are there some installs where it's only installed under python. On both my MacBook and my Ubuntu workstation python2.7 is available. Does it seem reasonable just to hardcode the name python2.7 in the Makefile and shebang lines?

Not specific to python2.7 but python2
Related to PEP 394 - https://www.python.org/dev/peps/pep-0394/
and v2.7.12 implements that - https://hg.python.org/cpython/raw-file/v2.7.12/Misc/NEWS

You may interest this - http://stackoverflow.com/a/26056481

I created https://github.com/google/grumpy/pull/88 to address this. Comments welcome.

@kokes Thanks for the suggestions! I implemented it basically as you described. Let me know if you encounter any issues with it.


Adding a git repo adds it to the source control view even when disabled

VSCode Version: 1.44.2
OS Version: Win 10.0.18363

Steps to Reproduce:

Create a new empty directory and open in VSCode
Set the following workspace settings: "git.autoRepositoryDetection": false and "git.detectSubmodules": false.
Open terminal and create a new subrepo: git init sub. Observe that despite the workspace settings, the repo is discovered and opened.

If VSCode is restarted, the repo is no longer found, which is in accordance with the git.autoRepositoryDetection setting.

Does this issue occur when all extensions are disabled?: Yes

This is #37947.
The git.scanRepositories setting might be a workaround for now.

@joaomoreno This is not #37947, this issue is not about nesting since the top work directory is not a repo. It is about VSCode behaving inconsistently when a repo inside the work directory is created. It will add the new repo to the source control view even when it shouldn't according to settings. VSCode must be restarted to fix it. Please consider to reopen the issue.

Sorry, I misread the instructions indeed! 👍

The problem is that we're still detecting git repos based on file change events, despite this setting value: https://github.com/microsoft/vscode/blob/27a97b9534e6ddf6c7cf8a4270827157e1eb9288/extensions/git/src/model.ts#L93-L93

Suggestion from @jrieken:

make it a tri-state setting: on open, on edit, never


Fixed bug where accept flag wasn't propagated correctly when using auto-forwarding

In reference to issue #131. Compiles and passes all unit tests.
Thanks,
James.

Hmmm, just found another possible problem so don't accept the PR yet. Refer to issue #131 for more information on it.


[BUG] Build Failure --> error: dangling pointer to an unnamed temporary may be used [-Werror=dangling-pointer=]

Reproduction steps
Steps to Reproduce:

Follow the Building steps as mentioned in the link,
https://github.com/project-chip/connectedhomeip/tree/master/examples/lighting-app/nrfconnect#building

When the Final build command is fired the below error is generated and the build is not completed.
Dangling_Pointer_error.log

Building files for the nrf kit and came across this issue.
Bug prevalence
Frequent
GitHub hash of the SDK that was being used
v1.0.0
Platform
nrf
Platform Version(s)
MatterStack v1.0.0
Anything else?
No response

This got fixed in #23827 which unfortunately fixed it as a drive-by and hence this did not get backported to 1.0...

Thankyou for the Clarity, Boris! Great Help!

@SiddMishra20 Thank you for finding this!


Character exported from Reallusion's 3DXchange does not animate properly

https://forum.unity.com/threads/dual-quaternion-skinning-for-unity.501245/page-3#post-5906549

Apparently the issue was not in the script
https://forum.unity.com/threads/dual-quaternion-skinning-for-unity.501245/page-3#post-6026843


New version: Devito v0.13.3

Registering package: Devito
Repository: https://github.com/ChevronETC/Devito.jl
Created by: @jkwashbourne-oss
Version: v0.13.3
Commit: b81e4eb9da3f3404e4bb3c4bf03351b592c20bf0
Reviewed by: @jkwashbourne-oss
Reference: https://github.com/ChevronETC/Devito.jl/commit/b81e4eb9da3f3404e4bb3c4bf03351b592c20bf0#commitcomment-145055332
Description: Julia wrapper for Devito functionality. Part of the COFII framework.

[merge approved]


Fixed Documentation Issue

What issue do you see here? The actual numbers in Markdown lists (other than the first) are not relevant. This API documentation already renders correctly:
https://hyperledger.github.io/fabric-gateway/main/api/node/interfaces/Contract.html

But still, this should be fixed.
If one goes through the code, I guess.

But still, this should be fixed. If one goes through the code, I guess.

Sorry I do not agree. TSDoc/TypeDoc aware IDEs (for example VS Code) render the numbering correctly in doc comments when mousing over the type. I prefer not to use explicit numbering (that will be ignored anyway) to avoid people having to remember to update the number sequence if adding or removing list entries, and the confusion it causes if they forget.


Update postBuild to the latest lab extension

This should fix the lab issue on Binder.

Haha I was just making a PR for that, thanks Jeremy!

I was about to :-D

We'll need to reset stable onto master so Binder can pick up this change.

Why doesn't binder get changes from master?

That's because the link in the README uses the stable branch: https://mybinder.org/v2/gh/QuantStack/ipycytoscape/stable?filepath=examples

But we can manually try on master with the following Binder link:
https://mybinder.org/v2/gh/QuantStack/ipycytoscape/master?filepath=examples

Oh! right. Thanks Jeremy, I'm gonna update it. Unless, you're already doing it. hehe

Done :)

<3


Bug: Query: stringification not supported for node type collection

ArangoDB 3.1.6
LET wuigerhojfoeru = FULLTEXT(authors, "name", "prefix:li")[*]._key

FOR doc IN authors

FILTER doc._key IN wuigerhojfoeru
SORT doc.name
RETURN doc.name

leads to Query: stringification not supported for node type collection (exception location: arangod/Aql/AstNode.cpp:2186). Please report this error to arangodb.com (while optimizing plan) (exception location: arangod/RestHandler/RestCursorHandler.cpp:131). Please report this error to arangodb.com

Fixed in 3.1.8 and devel.

3.1.8 is available for download now.


Update class DataLoaderIterWrapper to accommodate extra returned values

At https://github.com/davidtvs/pytorch-lr-finder/blob/master/torch_lr_finder/lr_finder.py#L453
 inputs, labels = next(self._iterator)
Could it be changed to
 inputs, labels, *rest = next(self._iterator)
to accommodate cases where more values are returned?
For example, if a weighted loss is used, then also the weights need to be returned to calculate the loss. This weighted loss can be found in U-Net original paper. Another example is to return the training data file name being used; this is useful for debugging.

hmm, I think handling loss weights could be something that we could handle in the future is there's a good idea on how to do it.
What I can do short term is implement your suggestion from the first post so that no error is thrown; however, the content of *rest will not be used

This has been implemented for python 2 and 3 in v0.1.5

Thanks!


Doc is pointing to a wrong BE Service - Customizing Task Scheduling - MI 4.2.0

Description
Description :
I tried out this document[1] to build a custom Task and found out the pointed BE service is incorrect[2][3]. Above BE should point to here[4].
Can you please check this and update the document?
[1]. https://apim.docs.wso2.com/en/4.2.0/integrate/develop/customizations/creating-custom-task-scheduling/#customizing-task-scheduling
[2]. https://apim.docs.wso2.com/en/4.2.0/integrate/develop/customizations/creating-custom-task-scheduling/#starting-the-back-end-service
[3]. https://github.com/wso2-docs/WSO2_EI/blob/master/Back-End-Service/stockquote-deployable-jar-2.2.2.jar
[4]. https://github.com/wso2-docs/WSO2_EI/blob/master/Back-End-Service/axis2Server.zip
Steps to Reproduce
$subject
Version
4.2.0
Environment Details (with versions)
No response

PRs:
https://github.com/wso2/docs-mi/pull/1087
https://github.com/wso2/docs-mi/pull/1086
https://github.com/wso2/docs-apim/pull/8731


Change hebrew date at sunset

Description:
The hebrew date should  change at sunset
Related issue (if applicable): N/A
Pull request in home-assistant.io with documentation (if applicable): N/A
Example entry for configuration.yaml (if applicable):
N/A
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:
N/A
If the code communicates with devices, web services, or third-party tools:
N/A
If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

@balloob What I'm trying to do is as follows:

get today's sunset (since we might be after sunset, I take the one matching start of day)
If we're after the sunset I increment today by one, since the Hebrew calendar changes at sunset.

Thinking of it again maybe get_astral_event_date makes more sense in this case?

Ah, I think you're right. I thought you just wanted to get the next sunset. Changing back is fine.


[CI] Bump Flax and Jaxlib versions to fix Jaxlib install error

The Flax dependency Orbax v0.1.8 has enforced the deprecation of installing Orbax as a standalone package. Flax v0.6.8 (which is currently pinned in the script docker/install/ubuntu_install_jax.sh) attempts to install Orbax as a standalone package and raises an error about doing so.
Going forward, the package orbax-checkpoint should be installed instead. Flax v0.6.8 does not recognise this and attempts to install Orbax instead of orbax-checkpoint and the installation fails.
In order to resolve Jax installation issues, bumping the version of Flax to be at least 0.6.9, which resolves the problem. Flax >= 0.6.9 does not pin the version of orbax-checkpoint that it installs and the latest version requires Jax >= 0.4.9 to be installed so the two must be updated together.
Issue: 15420 raised that details the error which occurs and steps to reproduce.
cc @leandron @Mousius @lhutton1 @ekalda @areusch @konturn @tqchen

Thanks @Liam-Sturge @yongwww


Improve CHANGELOG order

A good practice in changelog, place last version in first place
Expected order:
## [0.5.1] - Add nullable Layout
You can access layout safely with
Layout.maybeOf(context)

## [0.5.0] - First Version

Current order:
## [0.5.0] - First Version

## [0.5.1] - Add nullable Layout
You can access layout safely with
Layout.maybeOf(context)

True! Thank you for taking your time to submit the issue.
I have fixed it. It will be available in pub.dev in the next release


test: use T.TempDir to create temporary test directory

Description
A testing cleanup.
This pull request replaces os.MkdirTemp with t.TempDir. We can use the T.TempDir function from the testing package to create temporary directory. The directory created by T.TempDir is automatically removed when the test and all its subtests complete.  This saves us at least 2 lines (error check, and cleanup) on every instance.
Reference: https://pkg.go.dev/testing#T.TempDir
func TestFoo(t *testing.T) {
	// before
	tmpDir, err := os.MkdirTemp("", "")
	assert.NoError(t, err)
	defer os.RemoveAll(tmpDir)

	// now
	tmpDir := t.TempDir()
}

Type of change

[x] Refactoring

@Juneezee good catch, thx :)


docs: Undoc deprecated functionality

Partially addresses #3454

Should deprecated features be documented to the same level as their replacements?

Deprecated: https://altair-viz.github.io/user_guide/generated/api/altair.selection_single.html#altair-selection-single
Replacement: https://altair-viz.github.io/user_guide/generated/api/altair.selection_point.html#altair.selection_point

Mainly impacts API Functions where it is much clearer which selection_ functions should be used, since 3/5 are no longer there.

Thanks!


Repository is missing Rvalues_200815.csv

I am trying to run the measureclass.py script, and it fails with the following errory:
FileNotFoundError: [Errno 2] No such file or directory: 'Rvalues_200815.csv'

I searched the repo and no mention of this file.

file now added.


ci(postman): Add merchant account deletion test case

Type of Change

[x] CI/CD

Description

This PR adds a check to address missing Merchant Account - Delete validation
Motivation and Context

Existing collection does not have that test.
How did you test it?

curl --location --request DELETE 'https://sandbox.hyperswitch.io/accounts/postman_merchant_GHAction_962727c4-7e78-418d-8656-f4bec6f0ba30' \
--header 'Accept: application/json' \
--header 'api-key: <api-key>'

{
    "merchant_id": "postman_merchant_GHAction_962727c4-7e78-418d-8656-f4bec6f0ba30",
    "deleted": true
}

merchant_key_store table:

Checklist

[x] I reviewed the submitted code
[x] I added unit tests for my changes where possible

@pixincreate This test case is not required as we already have in the payment connector folder


Where to place third-party files ?

Hey guys,
I'am new to ngx and want to know where I have to place and bind third party CSS & JS Files
like this one here http://sdk.zurich.com/frontend-atomic/demo/zurich/01_styleguide/2_setup.html
thx

Without npm package... Create a lib folder or something and put it there. It is not a common thing at all, but I guess no all libraries cares about npm.

If I create a lib folder, where I have to register the parts and will the lib folder be compiled to my dist output ?

put your lib folder or third party library in src/public .. everything in src/public/ will get compiled with your dev and dist versions. src/public/lib/ becomes lib/ in your script src.
starting on line 247 in /webpack.config.js ..
CopyWebpackPlugin can copy over any other outside directories you need.

thanks guys for this blazing fast answers, i will try this today.


Render to arbitary size surface.

lottie is a vector image format and it can be render to arbitary size surface losslessly. But Animation::render() requires a Size struct which can't be constructed. Only Animation::size() can return a Size struct.
What about add a constructor function to Size struct?

Thoughts: Can we model a Surface struct(like the C++ class defined in rlottie.h) and make Animation::render render on it?
struct Surface {
  bytes: Vec<BGRA8>,
  width: u64,
  height: u64,
}

The Size struct was made private because it used size_t which was a different type depending on the target architecture. However, I found an option to just make bindgen use usize so it can be made public again.
Also, a Surface struct sounds like a good idea.


Sse fixes

Realized was adding more newlines that what was in the input. Added regression testing on the server SSE interface as well.

I'm not sure I follow this change. The idea was to allow incomplete lines; the mandatory line break shouldn't be considered content.

We chatted about this and it seems the fix is incorrect.
The format requires a string without a line break to be represented as data: foo\n\n. The two line break characters are part of the format. A string with a line break would be data: foo\ndata:\n\n.
@appleton, are you seeing something else?

The only time I see additional newlines is right at the start (for the app I'm testing anyway). Here's an example of the complete output:
id: 1
data:
data:

id: 29
data: -----> Node.js app detected
data:

id: 86
data:
data: -----> Reading application state
data:        package.json...
data:

id: 112
data:        build directory...
data:

id: 138
data:        cache directory...
data:

id: 170
data:        environment variables...
data:

id: 359
data:
data:        Node engine:         0.10.x
data:        Npm engine:          unspecified
data:        Start mechanism:     npm start
data:        node_modules source: package.json
data:        node_modules cached: true
data:
data:

id: 506
data:        NPM_CONFIG_PRODUCTION=true
data:        NODE_MODULES_CACHE=true
data:
data: -----> Installing binaries
data:        Resolving node version 0.10.x via semver.io...
data:

id: 556
data:        Downloading and installing node 0.10.36...
data:

id: 627
data:        Using default npm version: 1.4.28
data:
data: -----> Building dependencies
data:

id: 668
data:        Restoring node modules from cache
data:

id: 703
data:        Pruning unused dependencies
data:

id: 737
data:        Installing any new modules
data:

id: 769
data:
data: -----> Checking startup method
data:

id: 950
data:        No Procfile; Adding 'web: npm start' to new Procfile
data:
data: -----> Finalizing build
data:        Creating runtime environment
data:        Exporting binary paths
data:        Cleaning npm artifacts
data:

id: 981
data:        Cleaning previous cache
data:

id: 1022
data:        Caching results for future builds
data:

id: 1048
data:
data: -----> Build succeeded!
data:
data:

id: 1220
data<EMAIL_ADDRESS>/tmp/build_bb6ae88bd648876c0c9e7df4d0308523/appleton-sample-node-app-38b87b6
data:        â”œâ”€â”€<EMAIL_ADDRESS>data:        â””â”€â”€<EMAIL_ADDRESS>data:
data:

id: 1253
data: -----> Discovering process types
data:

id: 1314
data:        Procfile declares types -> web
data:
data: -----> Compressing...

id: 1326
data: done, 5.7MB
data:

id: 1346
data: -----> Launching...

id: 1427
data: done, v192
data:        https://pure-anchorage-9855.herokuapp.com/ deployed to Heroku
data:
data:

id: 1427
data:

As @tt says the double newlines in the middle of the stream are correctly signifying a string with a linebreak.

Got it yeah right now I omit the newlines at the start as well so all
should be good.
On that note, the SSE 204 termination is done in the io branch.

This was fixed in the io branch


Update dependencies

Fixes #32
At least bundle exec rspec seems to pass fine here.

Coverage remained the same at 97.525% when pulling 3f44525c929b9d7d3bfea05fbbd66b3398c8cf38 on Keruspe:master into c8d1a3aaad9e8ccc6609c129f7e8980992d21749 on localmed:master.

Coverage remained the same at 97.525% when pulling 0cfe817f130b6eb666d0e9c339ab85d9254d585e on Keruspe:master into c8d1a3aaad9e8ccc6609c129f7e8980992d21749 on localmed:master.

Coverage remained the same at 97.525% when pulling 02ed6a1dd776da88630912780766220087e1f775 on Keruspe:master into c8d1a3aaad9e8ccc6609c129f7e8980992d21749 on localmed:master.

Thanks!


Maximum call stack size exceeded

(node) warning: Recursive process.nextTick detected. This will break in the next version of node.Please use setImmediate for recursive deferral.
The message kept pumping up while doing the extracting then finally throw 'Maximum call stack size exceeded' and program died.
My unzip is version 0.1.9 and with node v0.10.23.And the size of my zip file is only 3.53MB

This repo has switched to setImmediate instead of next tick for some time. But the call stack exceeding happens because of a bug in the code for dealing with zip files that have compressed files stored in the streaming mode of the zip format.
This bug has been fixed by https://www.npmjs.com/package/unzip2, as this repo seems abandoned.
It comes down to the lines in lib/parse.js:161
setImmediate(function() {
  self._pullStream.unpipe();
  self._pullStream.prepend(extra);
  self._processDataDescriptor(entry);
});

should be
self._pullStream.unpipe();
self._pullStream.prepend(extra);
setImmediate(function() {
  self._processDataDescriptor(entry);
});


[Bug]: RHODS/ODH Dashboard is not accessible if we add application namespace to istio smmr

ODH Component
Model Serving
Current Behavior
When we incorporate the application namespace(redhat-ods-applications(downstream) a/opendatahub(upstream) into the smmr for monitoring, the RHODs/ODH dashboard becomes inaccessible.
Expected Behavior
All the service should be enabled and dashboard should be accessible after adding the namespace
Steps To Reproduce

Create a cluster
Install RHODS v2 or upstream ODH
Follow the kserve installation guide
RE-Enable(if it is already enable) RHODS dashboard component in DSC
Launch the dashboard. Observed application not available error

Workaround (if any)
Remove the application namespace from SMMR config
What browsers are you seeing the problem on? (If applicable)
Chrome
Open Data Hub Version
ODH: 2.2.0

RHODS: 2.3.0

Anything else
No response

The smmr config: https://github.com/opendatahub-io/caikit-tgis-serving/blob/main/demo/kserve/custom-manifests/service-mesh/smmr-rhods.yaml#L9C30-L9C30
https://github.com/opendatahub-io/caikit-tgis-serving/blob/main/demo/kserve/custom-manifests/service-mesh/smmr-odh.yaml#L9

ok it is not related to what @vaibhavjainwiz is working on. I will take care of this issue.

AFAIU, this issue is out-dated and there is no issue now. So I am closing this ticket. If it needs more help, please reopen it.


Fix Token program ESM exports

This is the second of two PRs to address https://github.com/solana-labs/solana-program-library/issues/3502, this one for the Token program JS (https://github.com/solana-labs/solana-program-library/pull/3507 for Memo, which it builds on top of). This fixes the issues that build systems have with .mjs files by exporting .js files. It also restructures the project to be similar to Wallet Adapter / the TypeScript monorepo template I've been using.
Fixes #3502

Also, the node runner for the GitHub action needs to be updated to version 16

Done, thanks! Running now.

Published in<EMAIL_ADDRESS>

This change is breaking TypeScript node.js import of @solana/spl-token:
✖ ERROR: Error [ERR_REQUIRE_ESM]: require() of ES Module<EMAIL_ADDRESS>from solana.client.js not supported.
index.js is treated as an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which declares all .js files in that package scope as ES modules.
Instead rename index.js to end in .cjs, change the requiring code to use dynamic import() which is available in all CommonJS modules, or change "type": "module" to "type": "commonjs" in<EMAIL_ADDRESS>to treat all .js files as CommonJS (using .mjs for all ES modules instead).

@raymondfeng I'm not sure what solana.client.js or where this error is coming from. Can you open a new issue with details on how to reproduce the issue?


Don't show h1 on table of contents

Andrei made a change related to this but it's still in review. Moving to backlog for now.


Lightline integration?

This is somewhat of a newbie question but I'm unsure about how to integrate this with vimscript?
I'm new to these Lua plugins and well not sure what to do.
Does this plugin or Lua need to be configured to integrate with other plugins like lightline?
Thanks!

I don't use lightline or vimscript, but if you can use the output of a vim command as the component in lightline, you can probably use lua print(require("wpm").wpm()) which outputs the wpm count (you can replace wpm() by sorted_graph(), historic_graph() or any other wpm function listed on the README.
Try to have a look at adding a component with a function maybe (check this example in the README)
let g:lightline = {
      \ 'colorscheme': 'wombat',
      \ 'active': {
      \   'left': [ [ 'mode', 'paste' ],
      \             [ 'gitbranch', 'readonly', 'filename', 'modified' ] ]
      \ },
      \ 'component_function': {
      \   'gitbranch': 'FugitiveHead'
      \ },
      \ }

maybe you can create a function that calls the required command and use that (maybe with execute inside the function)
See if you can do something with that. Let me know if you have any issues or it doesn't work, and I'll try to have a look at it.
Again, I don't use lightline or vimscript, so someone else might know better

I tried defining a command in lua like so:
vim.api.nvim_create_user_command('DisplayWpm',
  function()
    return "abc"

    -- TODO: Use wpm instead.
    -- return require("wpm").wpm()
  end,
  {})

And the lightline configuration looks like this:
let g:lightline.component_function = {
  \ 'gitinfo':         'lightline#hunks#composer',
  \ 'filetype':        'lightline#devicon#DevIconFiletype',
  \ 'fileformat':      'lightline#devicon#DevIconFileformat',
  \ 'cocstatus':       'coc#status',
  \ 'currentfunction': 'lightline#coc#CocCurrentFunction',
  \ 'wpm':             'DisplayWpm'
  \ }

But I can't get it to recognize the command because I think it's looking for a vimscript function. Doing this does work like I expect:
function! DisplayWpm()
  return "wpm:"
endfun

But I naively don't know how to call lua functions in here :sweat_smile:

Try using execute inside the vim function to execute lua require("wpm").wpm(), I think that might work


Use vcpkg in a Visual Studio project without user-wide integration

          We don't have enough information to solve this issue, unfortunately; if you have any more information to help us solve this, please reopen!

Originally posted by @JackBoosY in https://github.com/microsoft/vcpkg/issues/18868#issuecomment-930830469
I think I have the same problem. Our software uses libcurl by just copying its headers and library files into our source tree and referencing said copies. For a side project I installed cpr trough vcpkg which also brought along libcurl. If I integrate vcpkg user-wide the main software fails to link because now it's able to find 2 copies of libcurl and can't decide which one to link. Modifying it is not an option. I'd like to integrate vcpkg for a single solution/project.

See if vcpkg integrate project can do what you need.

->vcpkg help integrate
Synopsis: Integrates vcpkg with machines, projects, or shells
Examples:
vcpkg integrate install
Makes installed packages available user-wide. Requires admin privileges on
first use
vcpkg integrate remove Removes user-wide integration
vcpkg integrate project
Generates a referencing NuGet package for individual Visual Studio project
use. Windows-only

According to the documentation NuGet isn't recommended. Admittedly, it is going to be Windows-only anyway but I think there should be an easy cross-platform way to integrate into a project just like vcpkg integrate install does user-wide.


Output test files don't delete/rename themselves after doing the same operation in src files

TypeScript Version:  1.8.5
VSCode Version: 1.2.1
OS Version: Windows 10
I am building a VS Code extension, and on writing test cases for it, I noticed that files in out/test get created automatically upon creation in /test but don't get renamed or deleted.
Steps to Reproduce:

Start Yo Code and scaffold a TS extension (Yo code)
open the extension in VS Code and build once (by pressing F5 ) to get typescript compiler running on watch mode (need to select "Launch Extension" from debug pane)
go to /test and create main.test.ts. See that the file is generated in out/test folder.
Delete (or rename) main.test.ts .
In /out/test , you will see that the old files are still there.

Expected behavior:
main.test.ts should get deleted
Actual behavior:
Files don't get updated
I raised this issue to VS Code team, they directed me here.
8495

The compiler does not delete files. if you have renamed a source file, you need to delete the generated files as well. I do not think we have any intentions to change this behavior.


add cos integration

This change's primary purpose is to add the COS dependency to access it through the measureCLI code path.
Because we are still working through implementing the persistence cache layer, the code "doesn't do anything" yet.
I added a CosExampleCLI for testing and example purposes,
which I can revert if people don't think that should be merged back.
additional notes
I spent a bit of extra time (08a91bd) trying to support the concept of "dynamic COS client configuration"
but ultimately decided to revert the commit because I thought it wasn't worth the added complexity.
Let me know if you feel differently, and I will revert the revert.
# example usage
java -classpath code.jar com.ibm.cohort.cli.MeasureCLI \
-Ccos.request.timeout=5000 -Ccos.tcp.keep.alive=true ...

references

WFFHCOHORT-307

@CoreySandersOH @dkwasny-ibm
re:

Where the code lives

After thinking about this a bit more,
I do agree that it should be it's in a separate module for the same reasons you two are talking about.
I guess my question would be should I also move parts of the caching interface code as well at that point?

@CoreySandersOH @dkwasny-ibm
re:

Where the code lives

After thinking about this a bit more, I do agree that it should be in a separate module
for the same reasons you two are talking about.
I guess my question would be:
should I also move parts of the caching interface code as well at that point?

I think in a "purist" world, the caching interface would be in its own project that gets imported into both the engine and the COS caching layer.  I don't see a need to go that far with this release though.  Having the caching layer show up in the engine project along with the simple JCache implementation doesn't feel all that bad.
The main reason for separating the code is to prevent dependency bloat.

08cb335 moves the primary chunk of cos access code to its own module and
a6031bb changes the configuration pattern to be more in line with how we pass fhir server information via the cli
(via a json file reference)

Nitpicking a bit here, but if I were to pick a name for the new COS module, I would have picked cohort-cache-cos with the expectation that there might be a cohort-cache-rdms module later (or anything really) and I would want them to sort together. I would also expect possibly a cohort-cache module to exist at some point that defines the required interfaces for each of those sub modules. Again, they would all group together.
Looking at the code you have written so far, is it right that these are all infrastructure classes for providing an interface to COS and the actual work to implement the cache is coming later in this same project? With just what is there right now, it feels more like cohort-dao-cos than cohort-cache-cos. Not sure we would want to break the dao and the cache implementations up into separate projects, but it feels like we could if there was an expectation that other parts of the system might want the same DAO layer.
I got onto the previous point about dao vs. cache because the package names didn't seem to match the module name. I was expecting something like com.ibm.cohort.engine.cache.cos vs. com.ibm.cohort.engine.cos, Based on the functionality that is there, I don't mind the exist package name, but it took me some time to grok what was going on and if it were me I probably would put service or dao in the package name before cos.

I'm personally OK with the proposed name change, but will leave it up to Dan.
My current thought is that the new module is a starting point for all the cache work.  Breaking out COS specific code for other parts of the project to use can come later when the need arises.
I'm also OK with the proposed package name change, and will leave the decision to Dan.
My view is that we can use the more generic com.ibm.cohort.engine.cos package in a new module if/when we extract usecase agnostic features of our COS integration.

I will leave the branch around to see if we want to revisit this next sprint but closing this PR for the time being.


Speed up /atom and /atom/movable init times

Speed up /atom/proc/<init> and /atom/movable/proc/<init> by setting the built-in list vars inside their meta objects.

Might want to drop a comment mentioning that the lists are initialized internally.

Might want to drop a comment mentioning that the lists are initialized internally.

Will do once I'm at my computer again tomorrow.


add a contributor

@all-contributors please add @l1m0n3 for code and maintenance

@all-contributors please add @l1m0n3 for code and maintenance


Example code missing :page prop

If you add the the following line of code as shown in the example:
<pagination :records="500" @paginate="myCallback">
you get the following error:
[Vue warn]: Missing required prop: "page"
The example line needs to read:
<pagination :page=1 :records="500" @paginate="myCallback">
Once the :page prop is added the code works as expected - this change to the README.md code would make it work straight away for new users.

Thanks. I've updated the README and the package (the prop is now a v-model)


Allowing disabling of cache on ResultsGenerator.

Coverage remained the same at 100.0% when pulling 1f628c46f340139349c926af29256244b7a1677f on nocache into bbd4ba8679cb74a11fbd965f2f610cb19c3dfcc4 on master.


Build system integration

Problem
Combine CMake and GN to generate and build an Open IoT SDK application.
Change overview

Add basic Open IoT SDK platform adaptation

add platform adaptation files - with empty implementation now
add GN build integration to the Open IoT SDK platform
add crypto layer adaptation to the Open IoT SDK platform
add system layer adaptation to Open IoT SDK platform - CmsisRTOS locking and system mutex

Create platform GN project in config/openiotsdk/chip-gn directory

add base default arguments - .gn  and .args.gni
add toolchain configuration in toolchain/BUILD.gn  - create GGC toolchain definition (use GN template) and pass ar, cc and cxx arguments, set as no clang.
add mbedtls configuration in mbedtls/BUILD.gn - empty target, SDK has its own mbedtls
add BUILD.gn file - create openiotsdk target and add CHIP library as dependency

Implement Cmake bridge file with GN configuration file generation and building CHIP library as an external project - pass compiler settings from Cmake to GN

create Python script to build GN arguments file

Add a simple SDK application in examples/shell/openiotsdk  directory

main - use CMSIS-RTOS to start the demo thread and print messages from it. Include CHIP  config header ( lib/core/CHIPConfig.h  ) and print the CHIP port ( CHIP_PORT  ) value.
cmake - configure and build Open IoT SDK, configure and build CHIP library, build an application

Add Open IoT SDK example build script - build and run SDK examples

Testing
Build and run a simple example

your c++ fix was merged, I guess time to update the git submodule?

@paul-szczepanek-arm Next MR with CI changes


Show up app window, when new message comes up

Is it possible to show up app window, when you getting new message, like option, while app don't have notifications?

Hi @riwz , could you specify that better?

if i getting new message, a whole skype window shows up on the screen, instead notification window


feat: add dynamic len back references

Improves compression ratio.
Rough idea; instead of having a fix number of bits to store the address of the backref, make that a function of our position in the stream. For example, if we are processing byte 278, addressable space is 277bits so we just need 9 bits to store the backref address. We have an arbitrary max bound size at 1 << 20 (20 bits).
This PR also simplifies a bit the compression loop; using a circular buffer to avoid re-computing back references that we may have computed already as part of the previous "retry window i+1 i+2..." .

This works, but I suggest holding off on merging until after the release for two reasons:

Involves nontrivial circuit modifications that don't save on constraints
I think its "non-uniform" nature (varying backref length) can make things difficult should we choose to wizardify.

So counter arguments here;

I thought modifications for that were trivial? (i.e. known at compile time / easy to implement?). If not indeed, no rush to merge, BUT
If we don't wizardify, which is also a likely scenario, we want to avoid breaking changes in the circuit, and... since we have a bunch of these breaking changes in the upcoming release, I'ld rather pack what we know we will do there rather than force a new circuit setup / blob parsing etc breaking change in the next one.

This works, but I suggest holding off on merging until after the release for two reasons:

Involves nontrivial circuit modifications that don't save on constraints
I think its "non-uniform" nature (varying backref length) can make things difficult should we choose to wizardify.

So counter arguments here;

I thought modifications for that were trivial? (i.e. known at compile time / easy to implement?). If not indeed, no rush to merge, BUT
If we don't wizardify, which is also a likely scenario, we want to avoid breaking changes in the circuit, and... since we have a bunch of these breaking changes in the upcoming release, I'ld rather pack what we know we will do there rather than force a new circuit setup / blob parsing etc breaking change in the next one.

Okay I'm convinced once staticcheck is happy I'll merge.


[BEAM-5253] - Go SDK PubSub example currently broken

When running the streamin pubsub example in golang which (needs to run on dataflow in order to use the pubsub beam feature), I discovered that the functions where not exposed / registered correctly and hence causing a panic when running with the dataflow runner

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

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

Java

Python

---

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

@herohde

Made a comment in the JIRA.

Will close this as its not needed, only a small optimization according to @herohde.
It failed for me when using go run wordcap.go --runner=dataflow .....
But when making a build with go build wordcap.go and then ./wordcap --runner=dataflow ..  it works as expected.


Inline image previews are not sorted in the same order as the links

Apparently, our markdown inline image previews show the images in a random order, not the same order that the links to the images appeared in the message.  This can be quite confusing!
See, for example, https://chat.zulip.org/#narrow/near/186339/stream/design/topic/Starred.20messages.20in.20app

@zulipbot claim

@sampritipanda I spend like 5 minutes investigating this a few days ago, and mostly found that I'm not sure why this isn't sorted as it is.
InlineInterestingLinkProcessor in bugdown/__init__.py calls walk_tree to walk through the HTML structure of the message, so it's possible that the bug is something to do with walk_tree not using the human-expected order.


Version 0.1.7 release

I'd like a new release to be made with the fix from #16, so that I can open a PR on trust-dns to update. :)
This is where I encountered the panic described in #15.

I published the new version :)


Allocate temp data on the fly for some casting operations

@eric-haibin-lin
@reminisce

@cjolivier01 trust me harouwu is not working on sparse...

for clarification, this is a temporary fix. after alpha release we should move all resource request to exec/attach_op_resource_pass.cc


It is possible to select past dates in the event date filter

Description of issue
There is a comment from Jun on https://trello.com/c/PQIdwgZb/125-ui-events-listing-filter-events-list-by-date that states the one of the ACs should be that "User won't be able to select a date before today", but the behaviour currently allows this.
Note: This is also an A/C of https://trello.com/c/JVwbl10D/154-whole-app-not-showing-past-events. It has been added there as a development task to complete that ticket so who ever picks this up please advance that ticket when complete 👍
Expected behaviour
Should not be able to select dates before today.
Steps to reproduce

Go to the Events page and open the date filter
Select a date before today's date

It is selectable

OS / Browser / Device model
Galaxy S8
iPhone 7+
Screenshots
As indicated by the dot, today's date is 18th April

Covered in this story: https://trello.com/c/JVwbl10D/154-whole-app-not-showing-past-events

MVP could be to not be able to select any dates before the first event of Pride

Should we design something for the empty events listing instead? Will need UXD

On the other filter pages we just disable the submit button if the applied filter would result in 0 events. We could do that quite easily for the dates dialog. So if a user selects any date range where no events would appear the button underneath would be grey, disabled, and say "No events" like it does on the filter modal.

I think handling the no events case is still important. It will be applied in a number of places for example, event listing, curated events (e.g. favourite events) etc. Also would be used as a placeholder if the user fails to load the data from contentful on their first load.

Let's knock back past dates 50% opacity

For now I've created the PR above. I've just faded the days, but they are still selectable. Let me know if I should do more:

Hey @robbiemccorkell
Past dates should be not selectable if poss.
Noticed that the current date is also knocked back, but this should be full opacity

Ok I'll try making them non-selectable.
🤦♂  no idea how I didn't notice the current date was faded!

haha thanks :)

@JunBadger Previous days are now both faded and not selectable in the calendar in the above PR


Tracing

Added Jaeger tracing to the Offers API.

Coverage increased (+0.05%) to 89.081% when pulling ebd6c1147895f2186953e80c09e5823edd9c1bb4 on tracing into c6d701cdc2f3f99d636a2cd099f58fa3763959db on master.


Moved android hackathon demo to ./hackathon/android.

I believe it's failing the test because of a symlink path that went bad when we changed the folder, to wit:
{ Error: ENOENT: no such file or directory, stat '/home/travis/build/PolymerLabs/arcs/hackathon/android/src/main/assets/pipes_shell_dist'}

Ah yeah. That won't resolve because the dist folder won't have been created on travis. On my local machine I do have that dist folder, and the linter is trying to lint the webpacked sources. It appears to be very unhappy with them... :stuck_out_tongue: My guess is it's the /src/ folder in the middle of that path that is causing the issue. I'll fix sigh.

Nice. Sorry for the bad diagnosis, thank you for ignoring it, lol.


Allow changing gamespeed with non-numpad keys

Allow changing gamespeed with non-numpad +/- keys

Please provide more in deph description which keys will be used instead, also needs a description update of the feature in hte patcher itself

Agree, I can update the UCP description text.
As for which keys: it is the + and - keys that are not on the numpad (without shift modifier fyi).
I feel it is best to describe it as the non-numpad +/- keys, it is the most accurate way to describe it and is the equivalent of how the current behaviour is best described as using the numpad +/- keys.
I am open to suggestion, so let’s confirm this before we update the commit.

The mapping of non numpad + and - keys might be different for different keyboard languages.
Are you using key combinations? For example on a us keyboard the + is achieved by shift =, or is it the = key without shift?

Maybe the event keycode can clarify any confusions. From my understanding:  + (=171)   - (=173)
https://keycode.info/

my own plus would be 187, minus would be 189. hm interesting

Would be good to have this tested by as many different keyboard languages
as possible. German, english us, english uk, russian, chinese, etc.
On Thu, 25 Feb. 2021, 11:41 pm Krarilotus<EMAIL_ADDRESS>wrote:

my own plus would be 187, minus would be 189. hm interesting
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Sh0wdown/UnofficialCrusaderPatch/pull/746#issuecomment-785867130,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALGGWTREVPWCZQBN67J667LTAZAQXANCNFSM4WFRTRFQ
.

I can't figure out what the code does, but to your question, it compares esi with 187 and 189.


replace 3600 seconds with 1 hour

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2524   +/-   ##
=======================================
  Coverage   45.07%   45.07%
=======================================
  Files          93       93
  Lines        9780     9780
=======================================
  Hits         4408     4408
  Misses       4654     4654
  Partials      718      718

Flag
Coverage Δ

#linux
49.1% <100%> (ø)
:arrow_up:

#windows
41.58% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

metadata/content.go
40.5% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM


Remove stale dateutils package

It seems like python-dateutil seems to have replaced dateutils, the latter of which installs strange dependencies like argparse (stdlib since 2.6). The tests also only install python-dateutil, so this just makes the install dependencies equal to the test dependencies

Also added some code to make/upload wheels cuz wheels are the new hype. 🍺

Sorry, this is now definitely two things in one...removing dateutils and using wheels. The test script builds the wheel, installs it, then deletes the source directory (deleting the source directory can be skipped in Python 3.4+ with the -I flag that causes Python to ignore the local environment)
I can split it up if desired...if I can figure out how to use git such magically. ✨  Wow. 😟


Include NAT fixes in build until they are committed to the stable VPP…

… branch

Coverage decreased (-0.2%) to 76.63% when pulling b82da658163a60d2e8d29e9a5718771482a5c959 on rastislavszabo:master into 08b951cbba2223d6df4caa0a485dbe4316957065 on contiv:master.


add a new retention rule to the nuget server.

Make sure to check the following when opening a pull request:

If it's a large contribution, please open a regular issue first and discuss with the team
Make sure the PR targets the dev branch

Having that said: thank you for your contribution already!

Hi @Ortems, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

I have do the requested changed.
Thank you all for the review

If you refer to this PR, then it looks like it doesn't contain all the changes requested.
I do see the updated comment in the config file, and the call to FindPackagesById, but it doesn't look like the other feedback was addressed (renaming the setting to PackageVersionRetention, renaming the property to PackageVersionRetention)
Please push your updates to this PR branch here instead to facilitate review and discussion (you can reopen this PR).
Thanks!

I have add new test for this feature and rename the property to PackageVersionRetention.
PS: I am a beginner with github
Thanks! :)


Styling headers

Is there any way to style row and column header in bold?
Because now we have opportunity to change font size and font family, but not font weight:
columnHeaderCellFont: '12px sans-serif'
rowHeaderCellFont: '12px sans-serif'

Sure, but it's unconventional compared to HTML/CSS.  With there is a ticket to improve this behavior, but for now, you need to use a event handler to hook into the drawing of the data, then alter the canvas context.  Usually you just need to change a few parameters, then the grid will draw the specified data using the style you set in the event handler.
These two links will help you with what you're trying to do.
https://tonygermaneri.github.io/canvas-datagrid/docs/#tutorial--Conditionally-set-colors
https://developer.mozilla.org/en-US/docs/Web/API/Canvas_API/Tutorial/Drawing_text
Cheers


Constant error returned by RPC does not equal to original

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go1.10.2 darwin/amd64
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
What did you do?
I have a couple of error type variables defined globally. For example,
var ErrFileNotExist = errors.New("File does not exist")

On server side, this is used as a return value in an RPC, for example,
func (t *Type) F(args *T1, reply *T2) error {
   ...
   // some code here.
   return ErrFileNotExist
}

But on client side invoking this RPC and getting the return value, compare it with the global ErrFileNotExist, it does not match. That is,
...
// client set up
e := c.Call(...)
log.Prinln(e) // File does not exist
log.Println(e == ErrFileNotExist)  // false

If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
Playground link (RPC cannot run):
https://play.golang.org/p/RFOF_c1DT0F
What did you expect to see?
Shouldn't the value equality be preserved? It works on local call.
What did you see instead?
The returned value does not equal to the global ErrFileNotExist by value.

The error from the rpc call is of type ServerError (which wraps the underlying error). The global error is of type errorString. So I don't think they can be equal.

@meirf Yeah, I believe so. Is there any workaround?

Specific to this exact error ""File does not exist" that doesn't seem too exceptional - maybe just include a bool in the reply struct.
More generally, I've seen pattern of a very thin client library for each rpc server. The global error could live in that library. The library could look at the reply and/or error and decide if that library call should return library error.
But I'm not an expert on error best practices. Best to close this issue, read https://godoc.org/github.com/pkg/errors, dave cheney's blogs and ask around in go nuts for more advice.

Thanks @meirf for the analysis. Closing as this is not a bug. @vonzcy - please have a look at Questions wiki page for places to ask for advice.


shorten meta description of landing page to < 160

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/index.yml
89
34
0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 1b11e60:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/index.yml
:white_check_mark:Succeeded
View (qsharp-preview)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/index.yml
87
53
0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 7c36ebe:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/index.yml
:white_check_mark:Succeeded
View (qsharp-preview)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Likes for comments and lists

Is there a way where I can sync likes for comments and lists?
I see that the trakt website have likes_comments and likes_lists 🚀

It's available and documented here: https://trakt.docs.apiary.io/#reference/users/likes/get-likes
You can filter to get only comments or only lists otherwise you can use the type in the response.

Is there a way to get all items?
Like the watchlist or something like that?
Otherwise I have to iterate trough it? 😬

From this endpoint, you can get all the likes a user made. The information can't be cached efficiently otherwise. You need to keep that data your side and match it with the list of comments you get from other endpoints.

I read the docs wrong, pagination is optional.
Thanks!


local API's forgotPassword throwing errors.

Bug Report
local API's forgotPassword is throwing errors when used:
TypeError: Cannot read properties of undefined (reading 'config')
    at forgotPassword (/Users/user/directory/node_modules/payload/src/auth/operations/forgotPassword.ts:51:9)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Steps to Reproduce
I am trying this
await payload.forgotPassword({
  collection: "users",
  data: {
    email,
  },
});

reseting the password via the /admin/forgot works without a problem though. So it cannot be my email setup I guess.
Other Details
"payload": "1.6.15",

Hey @MoSattler — we'll check into this to see what we can discover.
Thanks for the report - will have more info for you shortly!

I've been experiencing a similar problem in the same project with payload.create() and payload.update()`.
I have a collection of countries with an alpha2 country code and a country name. I couldn't get the local API to set any other fields of the collection than country.name, even if I configured them similarly and fed the same data.
Again, with the REST API it works seamlessly.


Split VirtualService destinations analyzer

Split virtualservice.DestinationAnalyzer into virtualservice.DestinationHostAnalyzer and virtualservice.DestinationRuleAnalyzer. This makes them more narrowly scoped, and prevents the DestinationRule check from being unnecessarily turned off if service discovery is disabled (see https://github.com/istio/istio/pull/17365).
[X] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@sushicw: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
87a7a1d1ce7df93d53af51ad415db2c19f2085d8
link
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sushicw: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
87a7a1d1ce7df93d53af51ad415db2c19f2085d8
link
/test lint

racetest_istio
87a7a1d1ce7df93d53af51ad415db2c19f2085d8
link
/test racetest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@sushicw: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

racetest_istio
87a7a1d1ce7df93d53af51ad415db2c19f2085d8
link
/test racetest

lint_istio
87a7a1d1ce7df93d53af51ad415db2c19f2085d8
link
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test lint

@sushicw: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
87a7a1d1ce7df93d53af51ad415db2c19f2085d8
link
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sushicw: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-tests_istio
8fd0b4ff170c85a307125d37f20a88b803bd22e0
link
/test integ-mixer-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


InputEventMouseMotion (and other input events) should have "speed" renamed to "velocity" as it is represented as a vector.

Issue description:
Speed is a scalar quantity, and is the magnitude of velocity. In godot, Since speed is represented as a vector, it is actually a velocity since it represents both magnitude and direction.
Physics Classroom Link

Speed
Vector2

Default
Vector2( 0, 0 )

Setter
set_speed(value)

Getter
get_speed()

becomes

Velocity
Vector2

Default
Vector2( 0, 0 )

Setter
set_velocity(value)

Getter
get_velocity()

See https://github.com/godotengine/godot/issues/16863.


KAFKA-3865: Fix transient failure in WorkerSourceTaskTest.testSlowTaskStart

Ping @ewencp

Making the changes to synchronous is less sensitive to timing. I think It should be fine as unit tests are checking the correctness for basic cases.

As discussed offline with @hachikuji, the addition of future.get() is to ensure that the exception thrown by the executor is propagated.  Also it is not safe to throw the InterruptedException as the flag set by the exception is only cleared with next awaitLatch.

LGTM. Will merge to both trunk and 0.10.0 since it just cleans up the unit test and should hopefully help w/ stability of 0.10.0 builds too.


Google cookies are not deleted

Describe the Bug/Feature
I'd expect this addon to delete all cookies if I've closed all non-built-in pages, for example if about:preferences is the only open tab.
The following gif shows an example where this is not happening. Note that firefox sends my search to google.com, which redirects to google.de. Maybe that has something to do with the issue.
Screenshot in which the bug can be seen

Steps for anyone to reproduce the bug
See gif.
Your settings
See gif (I've only activated cookie deletion and set the delay to 0 minutes).

OS/version: Arch linux, 16.07.2017
Browser/version: firefox 54.0.1, build id<PHONE_NUMBER>2252, clean profile
Cookie AutoDelete version: 1.4.1

This is on the addon page

If you get notifications for a google cookie (with the name: NID) that can't be deleted, then it's probably this Firefox bug. The workaround is to just delete that cookie from Firefox's cookie manager.

Another option, if you get annoyed with the notifications, put google.de in the GreyList.
Related Firefox bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=1362834

Also you need to refresh the about: preferences page after deleting.

Ah I see, sorry for bothering then. I didn't know that firefox sends stuff to google non-stop... I assume they have to give them every URL I'm visiting to check whether it's "safe"? Kind of disturbing... Do you know whether this can be turned off?

You can try this:
https://support.mozilla.org/en-US/kb/how-does-phishing-and-malware-protection-work#w_how-do-i-use-the-phishing-and-malware-protection-features

I assume they have to give them every URL I'm visiting to check whether it's "safe"? Kind of disturbing... Do you know whether this can be turned off?

That's not true. See this excellent article by @fmarier as to how Safe Browsing (SB) works. Francois is a Mozilla engineer who works in this area (tracking protection, safe browsing, private browsing mode etc). SB works by using local lists (yes these are curated and maintained by a third party: google), and full URLs are never sent, only part URLs, hashed, and hidden with extra "noise"/traffic, etc (and I think it now uses Mozilla APIs?, or at least a new version with even more safeguards) .. but wait, there's more ... Mozilla strip out any extra parameters from google that could be used in any way for ID'ing, as well as using a separate cookie jar and so on. A LOT of work has gone into this, especially over the last couple of years.
This separate cookie jar for SB should be protected I believe - maybe Francois can elaborate on that.
Using google as the source means a couple of things which should be no-brainers. Google is in a prime position to amass this information and get it right with their resources - so the quality should be great. And it saves duplication of effort, time and money for Mozilla.
You can also have a look here at our repo ( https://github.com/ghacksuserjs/ghacks-user.js/blob/master/user.js ) and note section 0410's on Safe Browsing as to how to turn on/off various parts - such as blocking real time checks for items not covered - eg binaries not in local lists.

This separate cookie jar for SB should be protected I believe - maybe Francois can elaborate on that.

I don't mind that it's in a separate cookie jar. The issue is the WebExtension API call for all cookies browser.cookies.remove() should be able to remove it. I can grab it just fine with browser.cookies.all(), but it doesn't get removed.

@Thorin-Oakenpants thanks for the explanation and the link!
I guess google could still include the 32-bit prefix of the hash of urls of major sites in the list cached by firefox, and then not return the URL of the major site when firefox queries for the precise list of malicious URLs with that prefix upon visiting that site.  Thanks again!


Updatable Edges with Floating Edges corrupt the position of Anchor

Describe the Bug
I ran into this problem while creating FloatableEdges whith UpdatableEdge. The element (EdgeAnchor) that is used to move with the edge is drawn in a place that is internally defined, and does not take into account the curve that is created using CustomEdge. So the user is unable to find the element to drag.
Your Example Website or App
https://codesandbox.io/s/eloquent-ardinghelli-qgkv3h
Steps to Reproduce the Bug or Issue

Try to drag the EdgeAnchor
You find that EdgeAnchor isn´t on the line path.

EdgeAnchor is rendering next to the Node´s Handle, depending on data-handlepos. (class="react-flow__edgeupdater")
But the real path of Edge can be different.
Expected behavior
As proggrammer creating Custom Edge. I would like to have some possibilities to adjust the position of these EdgeAnchors from my Custom Edge, because there I know the real source & target positions.
Screenshots or Videos

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 11.7.3]

Additional context
No response

It is possible to adjust their position with your own not REACT code. But it's not a completely clean solution.
`  if(lineElement.current){ //Some element with react reference in custom node
const source = lineElement.current.parentElement?.querySelector(".react-flow__edgeupdater-source");
const target = lineElement.current.parentElement?.querySelector(".react-flow__edgeupdater-target");
// sy, sy, tx, ty - Positions of start and end of path.
if(source){
source.setAttribute("cx",String(sx));
source.setAttribute("cy",String(sy));
}
if(target){
target.setAttribute("cx",String(tx));
target.setAttribute("cy",String(ty));
}
}`

good catch. We are drawing the anchors before you are drawing your custom edge so currently there is no clean way to do this right. Not sure if we will change it. Thanks for sharing your workaround!


No docker-compose for production?

You provided docker-compose.yml in the root that builds the Dockerfile.dev images.  Do you not have a compose file that builds the Dockerfile images for production or am I missing something here?
Also, the link to the blog post in the README is missing, it only has https://.

@socketwiz Great question!
No, we do not use docker-compose for production, just for development. Production workflow looks different:
Github commit to production branch -> CI test/build/push containers -> CI run Ansible script to run changed containers on production.
I am also curious if someone uses docker-compose to run containers in a production environment. Could be handy for the small application deployment.
Let's see if some of our readers going to share this with us.


Create a tutorial for creating and using one (or more) of the example program sets

To help others create their own programs and use the existing programs, I want to create a document that walks through the code of e.g., Lunar Lander from World to several displays/controls to illustrate (in the MVC framework from implementation-notes.md) everything in action.

This was replaced by video demos and is obsolete with Creator.


Update site-recovery-deployment-planner.md

I updated the VMware CLI link that is broken. This is the correct one: https://code.vmware.com/web/dp/tool/vsphere-powercli/6.0

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@jporrata : Thanks for your contribution to the Azure documentation! The author, @nsoneji, has been notified to review your proposed change.

This file is no longer used and has been superseded by a new set of tutorials at https://docs.microsoft.com/en-us/azure/site-recovery/tutorial-vmware-to-azure
Please close this PR. Looks as though the latest tutorial now points to the correct link

#sign-off

In the existing document link we have aka.ms and pointing to the correct download link.  We don't need this PR. discard this changes.


Run test suite with PHP 8.3

Changes

Run test suite with 8.3, which is released in 2 weeks
Run psalm action with 8.2 instead of 8.1 (8.1 is skipped in tests.yml)
Run test when config file has changes

@Nielsvanpach
BTW, there is one failing test, it's not connected to changes you made, so you can ignore it. I'm struggling to fix it for a long time, I will appreciate any help with this. This test (and incorrect types returned by the plugic) blocks the whole release (which should be great and work in some cases better than LaraStan 🚀 )

it's finally releases https://github.com/psalm/psalm-plugin-laravel/releases/tag/v2.9.0 🎉


making kvm device for the virt-v2v pod optional

we want to be able to run virt-v2v on nodes without KVM capabilities (ci)

with this PR in-place managed to run migration against fake vmwhere and converter pod https://github.com/eslutsky/forkliftci/actions/runs/3532779911/jobs/5927561806

as discussed  offline,  we need to add new  parameter which control this setting, and turn it on by default

We merged #38 few days ago, now we delete it?

no we are going to create a cascading parameter which will disable it for CI , so it will be ON by default.

Codecov Report
Base: 33.94% // Head: 33.94% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (0645c68) compared to base (03f0b4d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   33.94%   33.94%   -0.01%
==========================================
  Files          42       42
  Lines        8077     8078       +1
==========================================
  Hits         2742     2742
- Misses       5118     5119       +1
  Partials      217      217

Impacted Files
Coverage Δ

...m/kubev2v/forklift/pkg/controller/plan/kubevirt.go
1.81% <0.00%> (-0.01%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

SonarCloud seems to be confused again.

SonarCloud seems to be confused again.

yeah, I've started looking at that and then got distracted by codecov..


Update to basic-ssg script for local dev, and CONTRIBUTING instructions

I was able to reproduce this but it does not happen on every run.
I suspect this may be caused by https://github.com/roc-lang/roc/pull/6792.


Heavy Lag while manipulating lots of vertexes  with Graph Editor

Expected behavior and actual behavior
Expect:Smooth operation on Graph Editor.
Actual:Heavy Lag while manipulating vertex  with Graph Editor(Studio 2.2).
Steps to reproduce the problem
Add vertex 500 more times.
Important Questions
Runninng Mode

[X] Embedded, using PLOCAL access mode
[ ] Embedded, using MEMORY access mode
[ ] Remote

Misc

[ ] I have a distributed setup with multiple servers. How many?
[ ] I'm using the Enterprise Edition

OrientDB Version

[ ] v2.0.x - Please specify last number:
[ ] v2.1.x - Please specify last number:
[X] v2.2.0 - Please specify last number:

Operating System

[ ] Linux
[ ] MacOSX
[X] Windows
[ ] Other Unix
[ ] Other, name?

Java Version

[ ] 6
[ ] 7
[X] 8

@jiezg
The Graph editor is not supposed to work with a lot of nodes.  The limit should be around 1k nodes

@jiezg
There is some improvement in Graph Editor starting from 2.2.8.
But again the Graph Editor is not supposed to work with too many vertexes/edges.

Thank U ！
在 2016-09-23 18:30:48，"Enrico Risa"<EMAIL_ADDRESS>写道：
@jiezg
There is some improvement in Graph Editor starting from 2.2.8.
But again the Graph Editor is not supposed to work with too many vertexes/edges.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


fix annoying horizontal scroll bar in repos in dabshboard

For issue  https://github.com/codepod-io/codepod/issues/118

Can you also give me permission to push to your branch, so that I can make these trivial tweaks as well?

Can you also give me permission to push to your branch, so that I can make these trivial tweaks as well?

Of course, invitation sent


Setup web application

Is your feature request related to a problem? Please describe.
Need a web app to interact with the services.
Describe the solution you'd like
A Vue JS app using Ant Design components.
Describe alternatives you've considered
React JS
Additional context
Add any other context or screenshots about the feature request here.

Too broad of an issue.


Update BCR review github actions

Getting the fix from https://github.com/bazelbuild/continuous-integration/pull/1920
Also removed paths filter for dismiss_approvals workflow since it'll be a presubmit check for all PRs.

This fails on fork repos. I opened https://github.com/bazelbuild/bazel-central-registry/issues/1630. Thanks!


Make 'make package' unconditionally copy the files

The ; \ at the end is not needed if there is only a single command.
Are we sure build_package.py works well if the output folders already exist? I usually do rm -rf build/*; find . -name "*.pyc" -delete; make (aliased to r or reb) to deploy CodeChecker locally.

This was super confusing for me. I spend quite some time to understand why bin/codechecker wasn't the same than from the git repo and why make package didn't fix my problem.

I agree that it is confusing and I am in favor of this change.

I think this was done because of our internal packaging, I need to recheck if it is still needed.

I think the reason for that check in the main Makefile was that running the tests should not rebuild the package if it is already there, because we loose some configuration changes done after packaging.
We should remove the package dependencies from the test targets in tests/Makefile so the package will not be overwritten.
This will change the current behavior a bit, because right now you could run the tests even if there was no package available but with this change you will need to run make package manually before the tests can run.

@gyorb if I remove the package dependencies, will you accept this PR? I have been bitten with this issue one more time...

@sylvestre I've created another pull request #1414 with some additional changes.
I still need to verify that it will work with our internal test infrastructure, but it will do a full package clean for the package target as in this change.

Will be superseded by #1414.


Pass the check if any one of the library (of the arch) satisfies the requirement.

Pass the check if any one of the library (of the arch) satisfies the requirement.
Fix:
https://github.com/microsoft/vscode/issues/204124
https://github.com/microsoft/vscode/issues/203967

@microsoft-github-policy-service agree

Pinging @deepak1556

The loop over libstdc++.so.6 works on my system. Looks good for me. Thanks.
I've no system with more than one libc.so.6 (which is probably very problematic).

Thanks for confirming!

@deepak1556 The suggestions all look good to me.  Do I need to commit the suggestions or resolve the comments?

Feel free to do either, commit suggestions from github or make the changes locally and commit them (which will also resolve the comments)

I like your exit checks.  Easier to read!  Thanks for all the efforts on VSCode!


v14.1.9 simplify Game Model prototype

[Current coverage][cc-pull] is 98.66%

Merging [#289][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #289   diff @@
==========================================
  Files            24         24
  Lines           374        374
  Methods          77         77
  Messages         77         77
  Branches          0          0
==========================================
  Hits            369        369
  Misses            5          5
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by 8d2e064
[cc-base-branch]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/branch/master?src=pr
[cc-pull]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/pull/289?src=pr


Upgrade to Spring Boot 3.3.0

Upgrade to Spring Boot 3.3.0, current Maven Wrapper, Dependencies incl. Webjars.

Hi @dsyer - As I have done a couple of times, I updated the Spring Petclinic dependencies this time just in time for SpringIO. I also tested the container images and native image builds.

Thanks @patbaumgartner


How to force hide a CalloutView using MapViewPlus?

Can we have a new method which will hide any visible CalloutView from the Map?
mapViewPlusObj.hideActiveCalloutView()

Does mapViewPlusObj.deselectAnnotation(annotation:animated:)work?


Update xfails info in README.md

Merge: 22a00e8 bb89502
Author: Jussi Kukkonen<EMAIL_ADDRESS>Date:   Fri Nov 22 13:21:05 2024 +0200

    Merge pull request #233 from loosebazooka/patch-1

    undefined

No idea what "undefined" is here but ok


Set up Byebug debugger

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

Byebug is an easy to use debugger for Ruby. If used with any spec, it's usage is similar to the :focus inclusion filter in rspec.

Sorry this is proving so difficult!

Thanks again @vidusheeamoli!


Change email identification to IDs (flarum/core#897)

Forgive me because the issue is not fresh in my mind (and there's a lot to read in #897), but what's the rationale behind this change?
One flaw I see is that since the email is no longer used as "identification", if you already have an existing Flarum account with your fb email address, it's now impossible to log into it via Facebook. If you tried to do so, you'd be presented with the registration form and then you'd have to change the suggested email address to something that's not taken.
Storing the facebook ID is probably a good idea in general, but not necessarily as the sole identifier...

This is the way it works for the GitHub Auth if I'm not mistaken. Emails can't be relied on for an identifier. They change and are seen as such. If someone changes their email on Facebook, what happens then? The id is a static attribute, and should be used as the sole identifier (it's literally called ID as in identification if that's not evidence enough).

This is the way it works for the GitHub Auth if I'm not mistaken. Emails can't be relied on for an identifier.

I think you mean Twitter Auth - that's correct, but it's only the case because the Twitter API doesn't actually allow us to get the user's email, so we have to make them enter it manually.

If someone changes their email on Facebook, what happens then?

Good point, and I'm not saying we shouldn't make use of the Facebook ID at all - I'm just saying that it's crap UX when I'm unable to link my existing Flarum account with my Facebook account to make logins easier for myself in the future. Matching emails is one way to do this. I'm sure there are other ways we can think of too. Food for thought.

link my existing Flarum account with my Facebook account

This is almost always done by signing into Facebook while signed into a service's account.
Your idea poses a security risk as well. Say a user creates a Flarum account, they don't have a Facebook account. Someone signs up for a Facebook account with that user's email address and uses that to sign into the first user's Flarum account. This is only possible is Facebook fails to confirm the email. This might not be a huge issue as Facebook is relatively trustworthy, but an issue that could be huge for a different SSO provider. It's best for unity in the way that SSO is handled in Flarum so that the issue you mentioned can be fixed across the board.

Done with b8 release


Adding support for neo4j

This PR should add support for neo4j to test containers, as discussed in discussion #579
Some notes:

neo4j is the default db name, I don't believe there are env variables to auto create a different one
using a different username throws an error by the db, hence sticking it with neo4j
docker image also expects NEO4J_AUTH as a combo of username and password, so I've tried to set that up in a way that makes sense

Thanks for your pull request. I need a few more days until I can do the review (I am on the road).

@HofmeisterAn I've pushed up what I believe to be the changes you intended. Let me know if everything looks all right, or if you'd like any changes. Sorry it took me a bit to get back to you on this one, I should have a quicker turn around moving forward.

9ab251cd2ace82a64d9a9cec046f6363400b2043, closes #579.


Flesh out some ConMon delivery steps

Changes proposed in this pull request:

Updates template to be more useful.

security considerations
Updates some manual steps for preparing conmon. No impact on system security.

@pburkholder is this ready for review?


Support for multiple languages

Successor to https://github.com/joachimschmidt557/nochfragen/pull/3 (accidentally closed)

Thanks @Mineorbit!


Add support for impala

By passing in authmech=3 and user/pass in the jdbc url you can interact with Impala even when Kerberos is enabled

This can be done by changing the jdbc url ?
On 28 January 2016 at 20:44, Antonios Chalkiopoulos <
<EMAIL_ADDRESS>wrote:

By passing in authmech=3 and user/pass in the jdbc url you can interact
with Impala even when Kerberos is enabled
—
Reply to this email directly or view it on GitHub
https://github.com/eel-lib/eel/issues/23.

Build two aliases ImpalaSource / ImpalaSink
JDBCDriver = "com.cloudera.impala.jdbc3.Driver"
jdbc:impala://host:21050;AuthMech=3;UID=UserName;PWD=Password

Why can't you do JdbcSource("jdbc:impala://host:21050;
AuthMech=3;UID=UserName;PWD=Password") ?
On 29 January 2016 at 12:47, Antonios Chalkiopoulos <
<EMAIL_ADDRESS>wrote:

Build two aliases ImpalaSource / ImpalaSink
JDBCDriver = "com.cloudera.impala.jdbc3.Driver"
jdbc:impala://host:21050;AuthMech=3;UID=UserName;PWD=Password
—
Reply to this email directly or view it on GitHub
https://github.com/eel-lib/eel/issues/23#issuecomment-176734554.

This is a duplicate of functionality already present in jdbc source


M3 4683 update cypress

The main changes in the test files deal with how the stubbing/modifying of requests/responses works in this cypress update. I also cleaned up and refactored some other things. The yarn lock is what I am the least sure about as far as if I did it correctly, but cypress 6.2.1 is working on this branch.
How to verify that that the tests pass:

yarn up
yarn cy:e2e (runs headless)
Result should be "✔  All specs passed!"

Make sure your .env contains:
REACT_APP_LOGIN_ROOT='https://login.linode.com'
REACT_APP_API_ROOT='https://api.linode.com/v4'
REACT_APP_APP_ROOT='http://localhost:3000'
REACT_APP_LAUNCH_DARKLY_ID=(CMR flag)
REACT_APP_CLIENT_ID=(https://cloud.linode.com/profile/clients, copy ID)
MANAGER_OAUTH=(create in https://cloud.linode.com/profile/tokens > "Add a Person Access Token", Save the token that appears in the modal, you will only see it once)

Looks like the tests are failing because of the yarn lock file being changed... not sure if that means I did it incorrectly or not.

Looks like the tests are failing because of the yarn lock file being changed... not sure if that means I did it incorrectly or not.

Seems like it may be similar to this: https://github.com/cypress-io/cypress/issues/6789#issuecomment-605629069 Try bumping to the most recent version of testing-library/cypress

Seems like it may be similar to this: https://github.com/cypress-io/cypress/issues/6789#issuecomment-605629069 Try bumping to the most recent version of testing-library/cypress

@Jskobos that may have done the trick?

@Jskobos that may have done the trick?


it8xxx2_evb: kernel.tickless.concept test fail

Describe the bug
tickless_concept.test_tickless_slice test fail in this testsuite: kernel.tickless.concept
Logs and console output
DEBUG   - DEVICE: *** Booting Zephyr OS build v3.6.0-472-gc975910f84ff (delayed boot 1ms) ***
DEBUG   - DEVICE: Running TESTSUITE tickless_concept
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: START - test_tickless_slice
DEBUG   - DEVICE: elapsed slice 110, expected: <100, 110>
DEBUG   - DEVICE: elapsed slice 70, expected: <100, 110>
DEBUG   - DEVICE:
DEBUG   - DEVICE: Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/tickless/tickless_concept/src/main.c:48: thread_tslice: t >= SLICE_SIZE is false
DEBUG   - DEVICE:
DEBUG   - DEVICE: elapsed slice 20, expected: <100, 110>
DEBUG   - DEVICE:
DEBUG   - DEVICE: Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/tickless/tickless_concept/src/main.c:48: thread_tslice: t >= SLICE_SIZE is false
DEBUG   - DEVICE:
DEBUG   - DEVICE: elapsed slice 20, expected: <100, 110>
DEBUG   - DEVICE:
DEBUG   - DEVICE: Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/tickless/tickless_concept/src/main.c:48: thread_tslice: t >= SLICE_SIZE is false
DEBUG   - DEVICE:
DEBUG   - DEVICE: FAIL - test_tickless_slice in 0.232 seconds
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: START - test_tickless_sysclock
DEBUG   - DEVICE: time 270, 480
DEBUG   - DEVICE: time 490, 690
DEBUG   - DEVICE: PASS - test_tickless_sysclock in 0.423 seconds
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: TESTSUITE tickless_concept failed.
DEBUG   - DEVICE:
DEBUG   - DEVICE: ------ TESTSUITE SUMMARY START ------
DEBUG   - DEVICE:
DEBUG   - DEVICE: SUITE FAIL -  50.00% [tickless_concept]: pass = 1, fail = 1, skip = 0, total = 2 duration = 0.655 seconds
DEBUG   - DEVICE: - FAIL - [tickless_concept.test_tickless_slice] duration = 0.232 seconds
DEBUG   - DEVICE: - PASS - [tickless_concept.test_tickless_sysclock] duration = 0.423 seconds
DEBUG   - DEVICE:
DEBUG   - DEVICE: ------ TESTSUITE SUMMARY END ------
DEBUG   - DEVICE:
DEBUG   - DEVICE: ===================================================================
DEBUG   - DEVICE: RunID: 78ccabe142cf8aa9280d00b5b3e5ab4e
DEBUG   - DEVICE: PROJECT EXECUTION FAILED

To Reproduce
Steps to reproduce the behavior:
twister -p it8xxx2_evb --device-testing --west-flash="../../itetool/loader.sh" --device-serial=/dev/ttyUSB0 -s tests/kernel/tickless/tickless_concept/kernel.tickless.concept

OS: Linux
Toolchain: zephyr-sdk-0.16.5
v3.6.0-472-gc975910f84ff

no repro


Data Science with Python : Drug Classification

Welcome to 'DSWP' Team, good to see you here
This issue will helps readers in gaining all the guidance that one needs to know about Drug Classification. Tutorial to Drug Classification using various algorithms and how it's applied using sample code.
To get assigned to this issue, add your serial numbers mentioned in the spreadsheet of "Data Science with Python", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
If you had referred any resources, add them up in "DS Resources". Similarly if you had used datasets, include them in "DS Datasets".
Domain : Machine Learning
Mentors Assigned : Arya / Yash Goel
Points to Note :
The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Changes should be made inside the Datascience_With_Python/ directory & Datascience_With_Python branch.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'GWOC' contributors of 'DSWP' domain.
All the best. Enjoy your open source journey ahead. 😎

@prathimacode-hub  Ma'am I would like to work on this issue.I will explain the detail information regarding it.
Type: Documentation
Sr.no: 428

@prathimacode-hub  Ma'am I would like to work on this issue.I will explain the detail information regarding it.
Type: Documentation
Sr.no: 428

@prathimacode-hub Ma'am I would like to work on this issue.I will explain detailed information regarding it.
Type: Documentation
Sr No: 146

@prathimacode-hub Ma'am I would like to work on this issue.I will explain the detail information  and also add some code, algorithms regarding it.
Type: Documentation
Sr.No: 146

Sr no: 161
Batch: 5
Name: Deepika Pavundoss
Type : Documentation
I would like to explain about the libraries used here, process and the code in detail
@prathimacode-hub

/assign

Don't self assign yourself. Issue is been assigned to @Deepika-Pavundoss for documentation

Hello, I am Deepthi M with serial number:172, Batch-5. I would like to do video on this issue.
My approach: a detailed explanation

Issue assigned to @deepthi1107 for video contribution


Add confetti to og:image and web page on CWV pass & 100/100 scores

Closes #14

Summary by CodeRabbit

New Features

Introduced a celebratory confetti effect triggered by specific conditions related to Lighthouse performance metrics to enhance user experience.

The og:image is proving hard to test. I am making changes directly to the OpenGraph folder yet don't see a change propagating?

I think we might need to do something like overlay a confetti SVG on the open graph image as we won't really be able to animate anything on it.

Makes sense. I'll have a muck around with the OpenGraph and find an SVG to layover the image. I'll also move the rendering logic to setup function with computed property as you did in the app.

@danielroe - I've used the static class from Stack Overflow's design library Stacks. It is under MIT 👍

I don't seem to be getting any visual feedback from the preview deployment. Is there some form of cache that might be taking time to refresh?
defaults: {
    cacheMaxAgeSeconds: 60 * 60 * 24
},

The thing to do is open the Nuxt Devtools (little Nuxt logo on the page) and open up the OG image devtool... it should look like this. (You'll even get HMR when editing.)

Bear in mind there are some limitations on how OG images work; they are rendered by Satori so support a subset of Tailwind and style attributes.

Lovely, that helps big time! 💚

@danielroe - good to merge? 🔥

Just want to see if  the extra css in the lighthouse page can be server only

Thanks a lot for picking neoconfetti 🙏😁

@PuruVJ - can open a new issue on the repository, but it would be nice to have static confetti as well. Seems very little support for static confetti apart from Stacks by StackOverflow, and not a massive fan of copying and pasting code.

Sure, go ahead please! And Pull requests welcome too, won't be able to implement this for a while myself 🙏


[EDA] - CA Add Application related fields to Contact

Critical Changes
Changes
We have added several new fields to the Contact object to track Former First, Middle, and Last Names; Citizenship Status; and Date Deceased.  New customers will see the new Citizenship Status field on the HEDA Contact Layout; existing customers can add manually any desired fields to Page Layouts.
Issues Closed
Closes #1229
New Metadata
Contact.Former_First_Name__c
Contact.Former_Middle_Name__c
Contact.Former_Last_Name__c
Contact.Citizenship_Status__c
Contact.Date_Deceased__c
Deleted Metadata
Testing Notes
https://salesforce.quip.com/bRXRAu2xIDxt#aSGACAqHLWk

Tested / Passes QE.


ACM-13184: update release image list for mce 2.7

https://issues.redhat.com/browse/ACM-13184

/cc @philipwu08 @fxiang1

/lgtm
Probably also need @philipwu08 to lgtm as I'm not on the owners list for this repo.

Looks like Phil is on PTO. I will manually merge then when the tests are done.

/approve


Comunication to PC

I noticed a clone of these being sold on aliexpress and ebay with a comunication module. Any idea how this comunication module could be added to your design? It would be really cool to be able to hook up this swr module to node red.

Thanks for your email. I'm aware of the clones, Owen Duffy is reviewing one
in his blog.
Sorry I can't help with your request. I'm not doing any more development on
this project and comms interfaces are not my strong point.
Regards
Mike
On 28 October 2022 19:54:39 randal007 @.***> wrote:

I noticed a clone of these being sold on aliexpress and ebay with a
comunication module. Any idea how this comunication module could be added
to your design? It would be really cool to be able to hook up this swr
module to node red.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.