Example does not work

What am I missing? What do I do wrong? What is it that I do not understand?
First session:
docker@boot2docker:/mnt/sdd1/tmp$ git clone https://github.com/buildkite/docker-puppeteer.git
Cloning into 'docker-puppeteer'...
remote: Enumerating objects: 408, done.
remote: Total 408 (delta 0), reused 0 (delta 0), pack-reused 408
Receiving objects: 100% (408/408), 131.98 KiB | 707.00 KiB/s, done.
Resolving deltas: 100% (193/193), done.
docker@boot2docker:/mnt/sdd1/tmp$ cd docker-puppeteer/example
docker@boot2docker:/mnt/sdd1/tmp/docker-puppeteer/example$
docker@boot2docker:/mnt/sdd1/tmp/docker-puppeteer/example$ # Build and run the example tests
docker@boot2docker:/mnt/sdd1/tmp/docker-puppeteer/example$ docker-compose -f docker-compose.integration-tests.yml run tests
WARNING: The Docker Engine you're using is running in swarm mode.

Compose does not use swarm mode to deploy services to multiple nodes in a swarm. All containers will be scheduled on the current node.

To deploy your application across the swarm, use `docker stack deploy`.

Creating example_web_1 ... done
wait-for-it.sh: waiting 15 seconds for web:80
wait-for-it.sh: web:80 is available after 0 seconds

  1) "before all" hook in "{root}"
  2) "after all" hook in "{root}"

  0 passing (2s)
  2 failing

  1) "before all" hook in "{root}":
     Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

  2) "after all" hook in "{root}":
     TypeError: Cannot read property 'close' of undefined
      at Context.after (/integration-tests/index.test.js:32:17)

Started HeadlessChrome/75.0.3765.0

Second session;
docker@boot2docker:/mnt/sdd1/tmp$ ls -la /mnt/sdd1/tmp/docker-puppeteer/example/screenshots
total 8
drwxr-xr-x    2 root     root          4096 Nov 15 09:38 .
drwxr-xr-x    4 docker   staff         4096 Nov 15 09:38 ..

I see:
docker@boot2docker:/mnt/sdd1/tmp$ cat /mnt/sdd1/tmp/docker-puppeteer/example/integration-tests/index.test.js
const assert = require('assert')
const puppeteer = require('puppeteer')

let browser
let page

before(async() => {
  browser = await puppeteer.launch({
    args: [
      // Required for Docker version of Puppeteer
      '--no-sandbox',
      '--disable-setuid-sandbox',
      // This will write shared memory files into /tmp instead of /dev/shm,
      // because Docker’s default for /dev/shm is 64MB
      '--disable-dev-shm-usage'
    ]
  })

  const browserVersion = await browser.version()
  console.log(`Started ${browserVersion}`)
})

beforeEach(async() => {
  page = await browser.newPage()
})

afterEach(async() => {
  await page.close()
})

after(async() => {
  await browser.close()
})

describe('App', () => {
  it('renders', async() => {
    const response = await page.goto('http://web/')
    assert(response.ok())
    await page.screenshot({ path: `/screenshots/app.png` })
  })
})

http://web/ is not a valid URL. Replace with something sensible.


Create dispatch task request/response messages

These new message types will be used to indicate that a task request should be assigned to the best available fleet by the task dispatcher.

Do you want the task state to be embedded in the response? Wouldn't it be simpler if the dispatch node simply publishes a task state over the usual websocket connection?

Having it in the response is easier because we don't want to return anything to the client until we write the task state into the db. Waiting on the ws will require us to keep state across different servers.

Having it in the response is easier because we don't want to return anything to the client until we write the task state into the db. Waiting on the ws will require us to keep state across different servers.

I'm not sure I follow the logic here.
The way I think about it is this:

The client makes a REST API request asking for a task to be dispatched.
The api-server forwards that request to the ROS2 topic.
The dispatcher node responds to the request saying "I acknowledge your request and am handling it. Now keep an eye out for a task with an ID of #xyz, that will be the task that you requested."
At almost the exact same time, the dispatcher node will publish an initial task state over the websocket connection. The initial state will effectively say "This task is queued and waiting assignment".

I don't see a reason to have multiple channels of truth for the task state. I would think that just burdens the dashboard implementation with needing to handle multiple different sources of task state information. Especially when the task state will probably arrive over the websocket connection within milliseconds of the REST response (maybe even before the REST response, depending on race conditions).

we don't want to return anything to the client until we write the task state into the db

I think right now our use of databases is not very well defined, and that should be the next thing we focus on after we have this task pipeline functional. If we could have the task dispatch node ensure that the new task state is written into the database before it sends the response to the api-server, would that address your concerns? I know none of the ROS2 nodes currently write anything directly to the database, but I think we should start having them do that.

@koonpeng Per our discussion, I've updated the response message to include a task state field when the response is a successful one. I've also added a dispatch field to the task state where the dispatcher can put dispatch-related information.

I'm going to go ahead and merge this so we're not blocked on this PR. @koonpeng let me know if you have any remaining concerns and we can follow up in another PR.


Script to add scheme adds deployment target 7.0

Which is too low.

Better option is to just warn the user if their deployment target is too low. I think requiring 8.0 deployment target and above is very reasonable.

I will add a fix in deploy_detox.rb.

No longer relevant. Detox is injected in runtime, and should work as long as officially iOS >= 9.0 and unofficially >= 8.0.


Problem with loading test #915 (Biccoca tests)

I deleted the file 00915.h5 of ~/.strip/unittests (as recommended). However, It is not automatically downloaded when I read it:

Please, any other suggestion/recommendation?
Regards

You cannot delete a file without updating the internal database in ~/.strip/unittests/unittests.db. Now that you have removed it, you should better delete the whole directory ~/.strip/unittests/ and re-run the command again.


#178 Kafka <IP_ADDRESS>

fixes #178

Upgraded Kafka to <IP_ADDRESS>
Bump version
Gitignore build files

waiting for CI

@suyograo output PR here https://github.com/logstash-plugins/logstash-output-kafka/pull/133

@original-brownbear change the version and merge.


The Rename Symbol Dialog Ignores Exclamation Mark

Issue Type: Bug
The rename symbol dialog ignores exclamation marks in symbol names, though the actual rename parameters are correct.

Write a symbol in any language supporting exclamation marks in identifiers names, using Julia for the snippet below, but it's reproducible with other languages.
function test!()
end

test!()

Run the Rename Symbol command.

VS Code version: Code 1.57.1 (507ce72a4466fbb27b715c3722558bb15afa9f48, 2021-06-17T13:26:56.255Z)
OS version: Linux x64 5.12.9-1-MANJARO snap
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-2760QM CPU @ 2.40GHz (8 x 2800)

GPU Status
2d_canvas: unavailable_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: disabled_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
3, 3, 3

Memory (System)
7.65GB (0.22GB free)

Process Argv
julia-vscode --no-sandbox --crash-reporter-id e9b4c761-c989-4fe7-8f42-5278286bd6a4

Screen Reader
no

VM
0%

DESKTOP_SESSION
xfce

XDG_CURRENT_DESKTOP
XFCE

XDG_SESSION_DESKTOP
xfce

XDG_SESSION_TYPE
x11

Extensions: none
A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
vspre833:30321513
vshan820:30294714
vscus158:30321503
pythonvsuse255cf:30323309
vscorehov:30309549
vscod805cf:30301675
vscextlangct:30333562
binariesv615:30325510
aatestflightcf:30331712
bridge0702cf:30333156

This is up to each individual extension, the UI passes on whatever has been typed in the input box

All extensions are disabled as indicated in the system report. I used Julia
just to demonstrate it.
On Tue, Jul 6, 2021, 1:43 PM VS Code Issue Triage Bot <
@.***> wrote:

This issue is caused by an extension, please file it with the repository
(or contact) the extension has linked in its overview in VS Code or the
marketplace https://aka.ms/vscodemarketplace for VS Code. See also our issue
reporting https://aka.ms/vscodeissuereporting guidelines.
Happy Coding!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/128042#issuecomment-874689325,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI3OMSIOBGYN4R7OSS73W5DTWLT7NANCNFSM474H6PEA
.


Is it possible to configure it so that the user can still interact with the background?

Hi,
Nice one ! Pretty awesome features and simple to use. Just one question, is it possible to configure it so that it is still possible to interact with the background while the bottom sheet is on?
I need to make a media player bottom sheet that stays visible while navigating and it is not very clear for me either or not it is possible using react native raw bottom sheet?
Thanks!

Since the RBSheet is implemented on top of the React Native Modal, we are unable to show or click any elements that are located behind it.


Materialize error pages and announcement modal

In this PR:

Replaced bootstrap modal with angular material dialog for announcements
Replaced bootstrap classes with angular material components in error pages
Renamed dialog template to reflect the service and directive that govern it
Slightly adjusted some of the phrasing on error pages

Notes:

Previously, the announcement description was limited to screen readers only and the image was hidden on small screens. It looks goofy to have a dialog with no content, so I set up the description to show when the image is hidden. Functionality on large screens is the same as it was before.

Screenshots

Coverage remained the same at 33.517% when pulling fb64bc220bd5a720d4a578321e050dcd0765513e on thevoiceofzeke:buckyMakeover into ac731fb57e362221bad8cc6f856338aa14c4ea80 on UW-Madison-DoIT:master.

Coverage remained the same at 33.517% when pulling fb64bc220bd5a720d4a578321e050dcd0765513e on thevoiceofzeke:buckyMakeover into ac731fb57e362221bad8cc6f856338aa14c4ea80 on UW-Madison-DoIT:master.

Coverage remained the same at 33.517% when pulling fb64bc220bd5a720d4a578321e050dcd0765513e on thevoiceofzeke:buckyMakeover into ac731fb57e362221bad8cc6f856338aa14c4ea80 on UW-Madison-DoIT:master.

@timlevett Hmm...maybe I'll mark this WIP and take that on next week.
...unless of course someone else feels like taking care of it.

Coverage remained the same at 33.517% when pulling 7fdd5725faa678e43f67b86e42f0d8cc36a38f3f on thevoiceofzeke:buckyMakeover into ac731fb57e362221bad8cc6f856338aa14c4ea80 on UW-Madison-DoIT:master.

Coverage remained the same at 33.517% when pulling 3db3bd3baf604ae5a69b9669bf43a6c3baceab97 on thevoiceofzeke:buckyMakeover into ac731fb57e362221bad8cc6f856338aa14c4ea80 on UW-Madison-DoIT:master.


can't immediately kill queued runs

While testing #619 I noticed (what I think is) an unrelated issue: if you kill the run immediately after it has been killed you get an error:
% lviv run reverse_hash/input2 | tee >(grep -o '[0-9]\+' | while read id; do lviv kill "$id"; done)
1023373367
https://localhost:4000/run/#1023373367/uq
Request to http://localhost:4001/killRun failed with 500. Cannot read properties of undefined (reading '0').

Full response: {'error': {'message': "Cannot read properties of undefined (reading '0')", 'code': -32603, 'data': {'code': 'INTERNAL_SERVER_ERROR', 'httpStatus': 500, 'stack': "TypeError: Cannot read properties of undefined (reading '0')\n    at Hosts.getHostForRun (/Users/benwest/Documents/GitHub/vivaria/server/src/services/Hosts.ts:34:26)\n    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n    at <anonymous> (/Users/benwest/Documents/GitHub/vivaria/server/src/routes/general_routes.ts:673:18)\n    at resolveMiddleware (/Users/benwest/Documents/GitHub/vivaria/node_modules/.pnpm/@trpc+server@10.45.2/node_modules/@trpc/server/dist/index.mjs:421:30)\n    at callRecursive (/Users/benwest/Documents/GitHub/vivaria/node_modules/.pnpm/@trpc+server@10.45.2/node_modules/@trpc/server/dist/index.mjs:451:32)\n    at callRecursive (/Users/benwest/Documents/GitHub/vivaria/node_modules/.pnpm/@trpc+server@10.45.2/node_modules/@trpc/server/dist/index.mjs:451:32)\n    at callRecursive (/Users/benwest/Documents/GitHub/vivaria/node_modules/.pnpm/@trpc+server@10.45.2/node_modules/@trpc/server/dist/index.mjs:451:32)\n    at <anonymous> (/Users/benwest/Documents/GitHub/vivaria/server/src/routes/trpc_setup.ts:40:20)\n    at <anonymous> (/Users/benwest/Documents/GitHub/vivaria/server/src/routes/trpc_setup.ts:12:10)\n    at callRecursive (/Users/benwest/Documents/GitHub/vivaria/node_modules/.pnpm/@trpc+server@10.45.2/node_modules/@trpc/server/dist/index.mjs:451:32)", 'path': 'killRun'}}}
Request to http://localhost:4001/killRun failed with 500. db return error: expected 1 row; got 0. query: "SELECT \"setupState\" FROM runs_t WHERE id = $1".

I'm guessing there is some sort of race condition where we were previously testing this by humans having to type in the run id, but if you script it it goes more quickly

There is a race condition that I know can happen:
https://github.com/METR/vivaria/pull/602#discussion_r1823047512
I'll start by adding a more sensible error message in this case. There is still an open question of whether it's ok to kill a run that started setup or would that leave dangling resources (I'm currently assuming the latter)

@Xodarap
https://github.com/METR/vivaria/pull/624


Bug ? in get roleAssignment RESTful API

Describe the bug
I am looking at this page: https://docs.microsoft.com/en-us/rest/api/authorization/roleassignments/listforresource.
I notice that if i specifies the resource be a blob storage account, I get all the roleAssignments that inherited from upper level, roleAssignments at storage account level, and roleAssignments of lower level(roles that i assign to lower level blob containers).
But if I specifies the resource to be a container, I only get the roleAssignments at storage account and roles that storage accounts inherited. I could't get roles that I assign to the container.
I also tried to specify the resource to be a blob file under container, and I got same roles as I specified resource as container.
To Reproduce
Steps to reproduce the behavior(I used Python requests to trigger the APIs):

Get your management bearer token
session = requests.Session(),
s.headers.update({'Authorization': 'Bearer ' + bearer,
'tenant_id': 'common',
'x-ms-date': datetime.datetime.utcnow().strftime('%a, %d %b %Y %H:%M:%S GMT'),
'x-ms-version': '2018-03-28'
})
Recourse is a blob file:
resp = s.get("https://management.azure.com/subscriptions/<subscriptionID>/resourceGroups/<group-name>/providers/Microsoft.Storage/storageAccounts/<blobstorage>/blobServices/default/containers/<container>/blobs/<fileName>/providers/Microsoft.Authorization/roleAssignments?api-version=2019-04-01-preview")

Resource is a container:
resp = s.get("https://management.azure.com/subscriptions/<subscriptionID>/resourceGroups/<group-name>/providers/Microsoft.Storage/storageAccounts/<blobstorage>/blobServices/default/containers/<container>/providers/Microsoft.Authorization/roleAssignments?api-version=2019-04-01-preview")
Resource is a storage account:
resp = s.get("https://management.azure.com/subscriptions/<subscriptionID>/resourceGroups/<group-name>/providers/Microsoft.Storage/storageAccounts/<blobstorage>/providers/Microsoft.Authorization/roleAssignments?api-version=2019-04-01-preview")
Expected behavior
I am not sure what I should expect from giving container or blob file as resource because the documentation didn't specifies, but i think definitely not the current behavior

@rloutlaw can you please take a look at this issue?


Loader cannot find MoltenVK_icd.json in app bundle.

Executable in app bundle, that load ../Frameworks/vulkan.1.dylib using dlopen+dlsym, should set current working directory of itself to its base directory (i.e. appname.app/Contents/MacOS/ for appname.app/Contents/MacOS/appname) in order to relative paths in VK_DRIVER_FILES to work. Otherwise loader cannot find ../Resources/MoltenVK_icd.json and further ../Resources/../Frameworks/libMoltenVK.dylib. This is because by default current directory is / for apps in app bundles.
This is not true:

The macOS loader also looks in the application's bundle for ICD and layer JSON files. It looks in the bundle's Resources/vulkan/icd.d directory for ICD JSON files

quoted from here.

Okay... what do you want us to do? We do not control the current working directory of the app--according to your description, this is happening before MoltenVK even gets a chance to load.

@cdavis5e there is two ways:

to find a way to get absolute path of some fixed subpath in app budle directory and use it as base path to search ICD descriptions relative to it
to correct the documentation in accordance with reality

Ah, ok. This should be addressed in loader repo. Sorry.

Eh, it's all right.


Add readme section on using fetchgit

I like to use fetchgit to get the repository, rather than a channel, since it makes my configuration.nix more self-contained.

Also see #54.

Well, I think having the alternatives in the readme is fine so long as any drawbacks are documented.
I also think that while not having channel rollback is a shame, it's wrong to say that you don't have rollbacks - nixos-rebuild rollbacks are 90% of what I want!

fetchTarball is a bit faster on small repositories and pinning is also possible: https://github.com/nix-community/NUR#pinning


Feature/rabbitmq connector

RabbitMQ Source/Sink Connector.

This change is

Hi. The JMS connector would have been fine I guess. I was checking the code and wanted to contribute something. Maybe a specific connector for RabbitMQ could make the configuration more convenient in some cases but for sure they overlap. Cheers.


Asset's selection lost

Wallet tab
Select asset
Switch to other tab
Come back to wallet

Nothing is selected. Asset selection should stay as before.

Send view should not save asset id. Asset in send should be the same as selected here or BEAM if nothing selected

This task was about saving selection in assets section in the wallet, not in send view. Here for example Shekel is selected but this selection is not saved if you switch from wallet view to another tab and then switch back

Checked


[Snyk] Upgrade workbox-webpack-plugin from 6.5.4 to 6.6.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade workbox-webpack-plugin from 6.5.4 to 6.6.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 2 months ago, on 2023-05-26.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Patch and project coverage have no change.

Comparison is base (1e821c3) 81.53% compared to head (fdddf86) 81.53%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #728   +/-   ##
=======================================
  Coverage   81.53%   81.53%
=======================================
  Files          31       31
  Lines         352      352
  Branches       45       45
=======================================
  Hits          287      287
  Misses         60       60
  Partials        5        5

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


'Python has stopped working' on Spyder launch, 'No symbol file loaded for Qt5Core.dll'

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
'Python has stopped working' message appears when attempting to launch Spyder
What steps reproduce the problem?

Launch Spyder

What is the expected output? What do you see instead?
A normal working Spyder. Spyder does open as normal but immediately the 'Spyder is not working' message appears.
Paste Traceback/Error Below (if applicable)

After selecting the debug option when the 'Python is not working' window appears. This is the outcome:

Versions

Spyder version: 4.0.0
Python version: 3.7.1
Qt version: 5.6.2
PyQt version: 5.6.0
Operating System name/version: Windows 10

Qt version: 5.6.2
PyQt version: 5.6.0

You're using very old versions of Qt/PyQt. Please update and try again.

Updating qt and pyqt leads to the same 'Python is not working' message and this error when debugging:

I think you have another Qt5 version installed in your system (due to another program doing that). Try to find it by searching for Qt5Core.dll and removing those not installed by Anaconda.
That's the only advice I jave for you. If the problem persists, please report it here:
https://github.com/ContinuumIO/anaconda-issues/issues/


Handle closing the zoom on scroll

When I click on an image and zoom it
Then I scroll the page
Then I should see the zoomed image smoothly return from whence it came

When I tap on an image on a mobile device and zoom it
Then I use my finger to scroll the page in any direction (say a 10px drag)
Then I should see the zoomed image smoothly return from whence it came

It would be sweet if we could make this with a single implementation and not have to specify between mobile and desktop.

Tangentially related issue that has to do with being able to specify what the actual scroll container is: https://github.com/rpearce/react-medium-image-zoom/issues/136

@spencerfdavis how's this bit going? I did some necessary leg-work and have that merged into master. Happy to pick this up if you need

@rpearce
Hello, is this bug fixed? Because my iPhone is still failing on this one. When zoomed in and scroll, it does not zoom out. Is it my problem. My Zoom component is clean, which means I didn't set any attributes.
Thank you.

@sunmoon-idegu What version of iPhone and iOS are you using? Have you updated to the latest version of this project?


Appropriate error type for unable to saveAddressBook after Fees update

Currently it is throwing a warning in the constructor of LogicManager which I think violates some code quality stuff. It should also be made known to users that if the addressbook is unable to save the data upon fees update they should have a choice to exit the application and retain the previous lastUpdated.

@angkoonhwee any updates on this?

I will change this tomorrow sir


Fix the formatting

@sivolko Can you review the table and fix the issue with the layout?

@sivolko Can you review the table and fix the issue with the layout?

Yes, I can see Storage , Logging section is broken and security part is getting carousel . I'm fixing it

resolved

I still issue with dockerextensions.collabnix.com page esp for

Resolved Now


what exactly means "outstanding operation"

Version of Beast
300
hi, i see randomly that assert in basic_stream.hpp goes off - there is a comment next to the assert:
// If assert goes off, it means that there are
// already read or write (or connect) operations
// outstanding, so there is nothing to apply
// the expiration time to!
//
BOOST_ASSERT(
    ! impl_->read.pending ||
    ! impl_->write.pending);

so my question is when the operation is outstaning? i know this is somehow connected with calling stream.expires_after() but i can't figure out when this happens. what is the call sequence of async_write()s and async_read()s and expires_after()s to cause that?
EDIT:
hm...from time to time the program also crashes few lines after above on
BOOST_VERIFY(impl_->write.timer.expires_at(expiry_time) == 0);
what are the reasons why the expires_at fails?

This error is the code politely telling you that you have broken the fundamental rule of Asio. You must never have more that one asynchronous write in progress at a time on a given stream.
You must wait for the completion handler of the previous write to be invoked before you initiate the next asynchronous write.

ok, ta. yea i know that rule - however i was pretty sure that that i did not break the rule....well...obviously not...

You need to call expires_after before the read or write it applies to, not after.


Import Error

I got an error on step 4.
ImportError: No module named 'azureml.datacollector'
I run pip install azureml.datacollector and got the message saying:
Requirement already satisfied: azureml.datacollector in c:\users\my username\appdata\local\amlworkbench\python\lib\site-packages (0.1.0a13)
Any ideas on how to fix it?

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

ID: 7e24f06c-a1e1-8897-9a92-52ef8f44d439
Version Independent ID: 7f7bed83-0028-1f60-9cf8-dd50603dbf24
Content: Use the model data collection feature in Azure Machine Learning Workbench
Content Source: articles/machine-learning/desktop-workbench/how-to-use-model-data-collection.md
Service: machine-learning
Product: unspecified
GitHub Login: @aashishb
Microsoft Alias: aashishb

@CindyBai Hi, thank you for your feedback. For issues not relative to document, please use MSDN forum and Stack Overflow so that out team can help well.

@CindyBai We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Machinepoollet is restarting periodically

Describe the bug
Upon investigation of the host, dmesg, it seems machinepoollet either falls short of memory, or has a memory leak within.
[5264803.260313] Memory cgroup out of memory: Killed process 2788175 (machinepoollet) total-vm:753308kB, anon-rss:25672kB, file-rss:26896kB, shmem-rss:0kB, UID:65532 pgtables:212kB oom_score_adj:999
[5264805.306357] oom_reaper: reaped process 2788175 (machinepoollet), now anon-rss:0kB, file-rss:0kB, shmem-rss:0kB
[5269777.612991] machinepoollet invoked oom-killer: gfp_mask=0x400cc0(GFP_KERNEL_ACCOUNT), order=0, oom_score_adj=999

To Reproduce
Check the dmesg on the host, or may be kubelet logs.
Expected behavior
Not to restart.

This issue can be addressed by patching the memory resources for the machinepoollet pod as the defaults are pretty low (30Mi).


[App]: Abnormal Behavior when writing a report using Korean input methods

Current Behavior
I think this is an extension of #3525, the abnormal behavior happening to Korean characters along with Chinese and Japanese.
When trying to type in Korean (using MacOS Monterey) on the report page, the following abnormal behaviors happen:

Location of the cursor will suddenly change to different places on the line
The characters will be duplicated erratically

Expected Behavior
I had no problem writing Korean in the description section and expected it to behave the same for the main part of the report.
Steps To Reproduce

Go to create a new report
Type in the main part of report in Korean

Screenshots
https://user-images.githubusercontent.com/63901494/172969636-d97aa4bb-013c-4b08-ba14-4356b98fb212.mov
Environment
OS: MacOS Monterey
Browsers: Google Chrome
Version: Chrome version 102.0.5005.61 (Official Build) (arm64)
Additional Context
No response

Hey @Taehee-K, thanks for flagging this. I'll file a ticket for this bug.

Yes I can create a report in Korean now
thank you!


Barclays epdq gateway updates

Should no longer deal with Solo/Switch or collect start date / issue number for UK maestro cards.

Missed something, will send an updated pull request shortly.


WIP: Pin github actions' dependencies

What type of PR is this?
/kind cleanup
What this PR does
Bump all github actions to their latest released version, whilst also pinned them against the specific git commit id.
Updated all job's images to latest (ubuntu-20.04 or macos-11) also pinning them.
Why we need it:
Pinned dependencies reduce several security risks:

They ensure that checking and deployment are all done with the same software, reducing deployment risks, simplifying debugging, and enabling reproducibility.
They can help mitigate compromised dependencies from undermining the security of the project (in the case where you've evaluated the pinned dependency, you are confident it's not compromised, and a later version is released that is compromised).
They are one way to counter dependency confusion (aka substitution) attacks, in which an application uses multiple feeds to acquire software packages (a "hybrid configuration"), and attackers fool the user into using a malicious package via a feed that was not expected for that package.

More information refer to ossf
Which issue(s) this PR fixes:
Partially fixes https://github.com/kubernetes-sigs/security-profiles-operator/issues/653
Does this PR have test?
N/A
Special notes for your reviewer:
Running the latest version of buildah on ubuntu-18.04 led to the error: "fuse-overlayfs" is not found. Hence why the motivation of doing both changes together.
Does this PR introduce a user-facing change?

NONE

/test pull-security-profiles-operator-test-e2e

Codecov Report

Merging #738 (97c813d) into main (c2c7fcf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #738   +/-   ##
=======================================
  Coverage   53.94%   53.94%
=======================================
  Files          41       41
  Lines        4121     4121
=======================================
  Hits         2223     2223
  Misses       1831     1831
  Partials       67       67

/test pull-security-profiles-operator-test-e2e

Looking into the test failure as it is not the usual flake test - potentially this PR will require further changes.
/hold

/test pull-security-profiles-operator-test-e2e

/test pull-security-profiles-operator-test-e2e

/test pull-security-profiles-operator-test-e2e

/test pull-security-profiles-operator-test-e2e

/unhold


Unable to detect jobs.

Hello Guys, I have configured merge-gatekeeper as a workflow by following the doc, but when I do the PR to my repository merge-gatekeeper is getting triggered but it says 0 out of 0 jobs found. I need like if any of the other workflows in the repo fails that should stop the merge(as the merge-gatekeeper acts as a check). Please help.

Hi @Marellasai ! The above description indicates that you are having the Merge Gatkeeper run against a commit ref that has no GitHub Actions / CI jobs associated. Merge Gatekeeper's main idea is to tie itself with a commit ref, and check its status via calling GitHub API. If GitHub API returns no associated job, it would show up with 0 for everything.
It would be difficult to debug further without the actual configurations for your CI suite (can be a mock) - would you be able to provide more?


x/build: add longtest builders for all first class ports

We currently have no longtest builders for arm64 or arm. Although I know getting compute on non-amd64 architectures is harder, and longtests on arm might take forever, there's a lot of stuff that is skipped on -short. For example, ECDSA is only tested with one curve on -short. This does catch most mistakes, but it also fails to test a lot of architecture-specific code in other curve implementations. Ideally, a longtest builder would be a requirement for first class ports.
(Apologies if there's already an issue open for this, searching long tests didn't bring up anything.)
/cc @golang/release

We currently have no longtest builders for arm64 or arm.

That's #49649.
(See also #35678 and #49055 for darwin/amd64 and darwin/arm64.)

Aha! Figured it'd be a duplicate. It's the original's birthday, too!

Feel free to close or keep it as an umbrella issue if it feels useful!


Exporting to PNG and embedding the scene crashes Chromium

Multiple people on my team (using various Chromium browsers: Chrome, Arc, etc.) have found that exporting a PNG and embedding the scene completely crashes the browser.

Same here...
It only crashes when "Embed scene" is selected.
It crashes for PNG and SVG exports with scened embedding for me on Chrome + Arc.

I'm seeing this too, on regular, stable Chrome.

Hello. If anyone can still reproduce, what is your OS and Chrome version?
Does the crash happen on all drawings, or just bigger ones? If on larger ones, what are the dimensions & size? You can check by right-clicking on canvas and opening the "stats for nerds":

This is still occurring for me on macOS Chrome v111.0.5563.64 (Official Build) (arm64) on all drawings, i.e. this small one:

Still crashing... even with very tiny drawing:

Browser:
Arc Version 0.94.0 (37738)
Chromium Engine Version 111.0.5563.64

Strange.
@namoscato @bhb603 both of you have 111.0.5563.64, could this be it? I have 111.0.5563.65, and I tried to find 111.0.5563.64 build, but couldn't find it in https://commondatastorage.googleapis.com/chromium-browser-snapshots/index.html (best I could do is 111.0.5563.0, which still works).
Could you try update if it allows you, and if not, try one of the newer channels (Canary, Beta, Dev) if it's fixed for you?

Yeah, as noted in https://github.com/excalidraw/excalidraw/issues/6349#issuecomment-1473580678, it works for me in v113.0.5656.0 (canary).

Oh, I missed that was you. Alright, so the working theory is this is a browser issue fixed in newer versions. We'll keep this open for a while and close unless new info is discovered.

I'm having this issue too, also on 111.0.5563.64
Google Chrome | 111.0.5563.64 (Official Build) (arm64)
Revision | c710e93d5b63b7095afe8c2c17df34408078439d-refs/branch-heads/5563@{#995}
OS | macOS Version 12.6.3 (Build 21G419)
JavaScript | V8 11.1.277.13
User agent | Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.3

Seems 111.0.5563.65 isn't available for me yet through the usual google chrome updates, hopefully i'll be to able to report back too that it's working, once I get that build.

I'm also on Version 111.0.5563.64 (Official Build) (arm64), experiencing the issue since updating to this version and don't have a different version available to me on the stable channel.
@dwelle notice that this is an arm64 version (I'm on an m1 mac). If you're on x64 (Intel) it could explain why you don't see this version and others ONLY see this version.
Anyway, frustrating as it is to just wait, the issue seems specific to this version of Chrome, and based on people reporting - canary fixes it, so it may be worth just waiting it out.
If anyone does want to research this and needs data from someone who can reproduce (not sure if there's a way to write logs that survive a browser crash)-- let me know, I'd be happy to help.
As already reported, this happens consistently even with a fresh drawing of a single line, when exporting to an image with scene data embedded.


This means that the problems as the were before has disappeared.


Functionality suggestion

Hi,
First off my compliments on the app, very nice and easy to use!
I’ve read through some of the history and I believe that it has been possible to enable throttle assist on different modes these bikes have? At least from what I understood it was accidentally enabled/disabled on some previous versions of this app?
Would be a very nice feature if it would be possible to toggle throttle assist manually for each of the modes the bike has!
Kind regards,
Dennis

Thanks! I'm glad you've found the app useful. My guess is that previous versions assumed all bikes were in the US. US and EU have different meanings between the modes, especially as it relates to throttle assist. This wasn't removed, rather EU support was added. However, if the app incorrectly detects the bike region, you can always switch regions in the Edit menu (where you can also change the bike name).
Hopefully that helps?

I found the region settings, just to clarify, would setting the region to US then give the EU bikes throttle assist at lower modes?

From the official app, here's what it says about the differences in the two regions (PAS is Pedal Assist):
US:
- 1: Class 1 - PAS Only, 20 mph
- 2: Class 2 - PAS & Throttle, 20 mph
- 3: Class 3 - PAS Only, 28 mph
- 4: OFF-ROAD - PAS & Throttle, no limit

EU:
- 1: EPAC - PAS, 25 km/h
- 2: 250W - PAS, 35 km/h
- 3: 850W - PAS, 45 km/h
- 4: OFF-ROAD - PAS/Throttle, no limit

On my bike, which is set to US, the throttle works on all modes. I haven't tried really riding the bike in EU mode though. As for what will happen with your particular bike? I can't be sure. If you try it, I'd love to know what happens.

K6
On Fri, May 26, 2023, 10:40 AM sinneD1988 @.***> wrote:

Reopened #17 https://github.com/blopker/superduper/issues/17.
—
Reply to this email directly, view it on GitHub
https://github.com/blopker/superduper/issues/17#event-9352732520, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A7MFN5WVPSBPUIBIDRN4EWTXIDMF7ANCNFSM6AAAAAAYMFJXLI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Sorry for my late reply. Got to test the bike with both US and EU region settings but they don’t make any difference.
So for an EU bike you don’t get the throttle assist as it seems.

Huh, thanks for the info. I do know the region doesn't actually switch until you change the mode as well. So, flip the region, then tap the mode button to change the mode number. That triggers the app to actually update the mode on the bike. If you get a chance to try that again, let me know!


Execute the benchmarks with stack

Benchmarks should be run with stack:
stack bench

Pierre:
As a developer, I would like to know the performance of TorXakis in isolation.
Running ToXakis while generating an SMT log file enables separately measuring the time needed by the SMT solver and calculate the time needed by TorXakis.


Fix: Removed comments that were breaking the code

The comments were causing a "no matching files found" error for the embed pattern. After removing the comments, the server builds successfully.

Thanks for spotting this! I've fixed the issue while supporting client-embedded builds by moving out those lines into release.go, so that they only get run when you do go build -tags release. Plain go build should work now.


Obfuscate Secret for encrypting state file in terragrunt output

When running terragrunt with an Encrypted GCS backend for state files the encryption password is printed on the Output.
This is bad especially when running terragrunt in a pipeline since the encryption password will be left over in the historical logs of every run
I currently just sed it out but this is prone to break if the output string was to change, ideally terragrunt could treat those as sensitive and just print XXXX instead ?
for example
terraform init -backend-config=encryption_key=XXXXXXX -backend-config=project=my-project -backend-config=bucket=my-bucket -backend-config=prefix=terragrunt/zone/terraform.tfstate -get=false -get-plugins=false -backend=false -from-module=git::ssh://git@gitlab.com/xxxx/gke-terraform-overlay?ref=master -no-color /XXXXXXX/.terragrunt-cache/YWtrf-rqN2s9tYuJa3oRVZjMG0w/1t5BPiEk_I5qZT1KJxm6W-P2lvA

Answering myself,
I moved the encryption key to
terraform {

    extra_arguments "project" {
      commands = ["${get_terraform_commands_that_need_vars()}"]

      env_vars = {
          GOOGLE_ENCRYPTION_KEY = "<MYKEY>"
      }
  }
}

and is not showing anymore in the cli.


WIP: run as non-root user inside container

Description
Make sure the operator does not run as root when running in a container.
Type of change
Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

Environment

Runtime version(Java, Go, Python, etc): Go Lang 1.12.x

I did not test this yet, but I am using this setup for a different project which is working. Please do not merge yet.

What's the latest in this issue? Can we get this wrapped up?

I still have not had a chance to test this. If someone else has sometime to test this that would be super helpful.

@seanmalloy  this PR is good. Unit tests, e2e, and full cluster-seed example all passed.


ivi-dance-with-a-twist mishandles empty sizes

If the first call to ivi-dance-with-a-twist sets the bufferSize to zero, we should just return without calling again. Currently we call with a zero size, which reports an error (at least for some drivers/APIs).

We have to be a little careful here - multiple parameters can be ivi-dance-with-a-twist; maybe some could be zero size and some could be non-zero size? In that case I would think we'd want to do the call anyway. (and hopefully the underlying API would handle that...)


Fix Vercel backend issue

👨🏻💻 What does this PR do?
This PR will fix the vercel deployment for our backend app.
🧪 How has/should this change been tested?
Manual API Test.
📝 Any notes for the reviewer?
📜 Documentation

Cheers bro, looks good!


Case sensitivite (Delete, Edit, Star, Unstar, View)

Description
delete, edit, star, unstar and view are case sensitive for the NAME parameter while find is not case sensitive. This is not reflected in the user guide and might prove a little inconsistent given that find is not case sensitive.

Suggested improvement

Make these commands case insensitive (might be preferable for sake of consistency)
Or update the user guide to specify that they aren't

I think it is already mentioned that the 'find' command would be case insensitive in the user guide
Screenshots


different FlutterActivity use same flutter engine not work

I add a Flutter Activity with my native android app.
in order to use less memory ，i make different FlutterActivity use same flutter engine .
i open four Activity one by one (ActivityA -> FlutterActivityA -> ActivityB -> FlutterActivityB).
(ActivityA and ActivityB is android native Activity，FlutterActivityA And FlutterActivityB is FlutterActivity.)
now the top Activity is FlutterActivityB，then i finish FlutterActivityB and ActivityB quickly.
after finish FlutterActivityB and ActivityB, the top Activity is FlutterActivityA.
if the FlutterActivityB onDestroy after the FlutterActivityA onResume，the FlutterActivityA can not renderer
and can not interact with user. FlutterActivityA can not update and  renderer flutter ui any more.
i think is a CacheEngine bug(different FlutterActivity use same flutter engine), FlutterActivityB onDestroy after the FlutterActivityA onResume effect FlutterRenderer stopRenderingToSurface, thus, FlutterActivityA cannot Renderer again.

Hi @HXiaoMing
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample?
did you use this method :
startActivity(
      FlutterActivity
        .withNewEngine()
        .build(currentActivity)
      );

Hi @HXiaoMing
Can you please provide your flutter doctor -v, your flutter run --verbose and a minimal complete reproducible code sample? You can create a sample project that reproduces the issue and upload it on GitHub and then share the link here.
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


File uploading with aiohttp (status 500)

Description
Connexion return 500 error.
My swagger scheme:
requestBody:
        content:
          multipart/form-data:
            schema:
              type: object
              properties:
                fileName:
                  type: string
                  format: binary

Handler:
async def create_task(self, fileName, **request):
        print(fileName)

Html form:
<form method="post" enctype="multipart/form-data" multipart action="">
    <label for="fileName">Choose file to upload</label>
    <input type="file" id="fileName" name="fileName">
    <button>Submit</button>
</form>

Expected behaviour
FileField(name='fileName', filename='script.sql', file=<_io.BufferedRandom name=12>, content_type='application/octet-stream', headers=<CIMultiDictProxy('Content-Disposition': 'form-data; name="fileName"; filename="script.sql"', 'Content-Type': 'application/octet-stream')>)
Actual behaviour
  File "/lib64/python3.6/site-packages/aiohttp/web_protocol.py", line 418, in start
    resp = await task
  File "/lib64/python3.6/site-packages/aiohttp/web_app.py", line 458, in _handle
    resp = await handler(request)
  File "lib64/python3.6/site-packages/aiohttp/web_middlewares.py", line 119, in impl
    return await handler(request)
  File "/lib64/python3.6/site-packages/aiohttp/web_middlewares.py", line 119, in impl
    return await handler(request)
  File "/lib64/python3.6/site-packages/connexion/apis/aiohttp_api.py", line 32, in oauth_problem_middleware
    response = yield from handler(request)
  File "/lib64/python3.6/site-packages/connexion/decorators/coroutine_wrappers.py", line 21, in wrapper
    connexion_response = function(connexion_request)
  File "/lib64/python3.6/site-packages/connexion/decorators/uri_parsing.py", line 143, in wrapper
    response = function(request)
  File "/lib64/python3.6/site-packages/connexion/decorators/validation.py", line 147, in wrapper
    data.update(dict.fromkeys(request.files, ''))  # validator expects string..
AttributeError: 'bytes' object has no attribute 'update'```

### Additional info:

This problem was solved with patching aiohttp_api.AioHttpApi.get_request method.

-Python 3.7.2
-Version: 2.2.0

Ping. I need solved for the problem. What is status the issue?

Ping. I need solved for the problem. What is status the issue?

@kornicameister create new PR.
But it doesn't merged yet.
I use monkeypatch from this PR :)

Is there already a solution on that? I have the problem that I definitely need to upload files to the server and this bug seems to still persist.

Should be fixed by #1222


[BUG] SkiaSharp 2.88.0 PackageReference break csproj pack logic

Description
When referencing the SkiaSharp package (2.88.0) in a csproj where NuGet pack is enabled using
<GeneratePackageOnBuild>true</GeneratePackageOnBuild>

The following errors are generated
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\skiasharp.nativeassets.win32\2.88.0\runtimes\win-x86\native\libSkiaSharp.dll' is not added because the package already contains file 'content\libSkiaSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\skiasharp.nativeassets.win32\2.88.0\runtimes\win-x86\native\libSkiaSharp.dll' is not added because the package already contains file 'contentFiles\any\net472\libSkiaSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\skiasharp.nativeassets.win32\2.88.0\runtimes\win-x64\native\libSkiaSharp.dll' is not added because the package already contains file 'content\libSkiaSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\skiasharp.nativeassets.win32\2.88.0\runtimes\win-x64\native\libSkiaSharp.dll' is not added because the package already contains file 'contentFiles\any\net472\libSkiaSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\skiasharp.nativeassets.win32\2.88.0\runtimes\win-x64\native\libSkiaSharp.dll' is not added because the package already contains file 'content\libSkiaSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\skiasharp.nativeassets.win32\2.88.0\runtimes\win-x64\native\libSkiaSharp.dll' is not added because the package already contains file 'contentFiles\any\net472\libSkiaSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\skiasharp.nativeassets.win32\2.88.0\runtimes\win-arm64\native\libSkiaSharp.dll' is not added because the package already contains file 'content\libSkiaSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\skiasharp.nativeassets.win32\2.88.0\runtimes\win-arm64\native\libSkiaSharp.dll' is not added because the package already contains file 'contentFiles\any\net472\libSkiaSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\skiasharp.nativeassets.win32\2.88.0\runtimes\win-arm64\native\libSkiaSharp.dll' is not added because the package already contains file 'content\libSkiaSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\skiasharp.nativeassets.win32\2.88.0\runtimes\win-arm64\native\libSkiaSharp.dll' is not added because the package already contains file 'contentFiles\any\net472\libSkiaSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\harfbuzzsharp.nativeassets.win32\<IP_ADDRESS>\runtimes\win-x86\native\libHarfBuzzSharp.dll' is not added because the package already contains file 'content\libHarfBuzzSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\harfbuzzsharp.nativeassets.win32\<IP_ADDRESS>\runtimes\win-x86\native\libHarfBuzzSharp.dll' is not added because the package already contains file 'contentFiles\any\net472\libHarfBuzzSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\harfbuzzsharp.nativeassets.win32\<IP_ADDRESS>\runtimes\win-x64\native\libHarfBuzzSharp.dll' is not added because the package already contains file 'content\libHarfBuzzSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\harfbuzzsharp.nativeassets.win32\<IP_ADDRESS>\runtimes\win-x64\native\libHarfBuzzSharp.dll' is not added because the package already contains file 'contentFiles\any\net472\libHarfBuzzSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\harfbuzzsharp.nativeassets.win32\<IP_ADDRESS>\runtimes\win-x64\native\libHarfBuzzSharp.dll' is not added because the package already contains file 'content\libHarfBuzzSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\harfbuzzsharp.nativeassets.win32\<IP_ADDRESS>\runtimes\win-x64\native\libHarfBuzzSharp.dll' is not added because the package already contains file 'contentFiles\any\net472\libHarfBuzzSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\harfbuzzsharp.nativeassets.win32\<IP_ADDRESS>\runtimes\win-arm64\native\libHarfBuzzSharp.dll' is not added because the package already contains file 'content\libHarfBuzzSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\harfbuzzsharp.nativeassets.win32\<IP_ADDRESS>\runtimes\win-arm64\native\libHarfBuzzSharp.dll' is not added because the package already contains file 'contentFiles\any\net472\libHarfBuzzSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\harfbuzzsharp.nativeassets.win32\<IP_ADDRESS>\runtimes\win-arm64\native\libHarfBuzzSharp.dll' is not added because the package already contains file 'content\libHarfBuzzSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5118: File 'C:\Users\username\.nuget\packages\harfbuzzsharp.nativeassets.win32\<IP_ADDRESS>\runtimes\win-arm64\native\libHarfBuzzSharp.dll' is not added because the package already contains file 'contentFiles\any\net472\libHarfBuzzSharp.dll'
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5100: The assembly 'content\libSkiaSharp.dll' is not inside the 'lib' folder and hence it won't be added as a reference when the package is installed into a project. Move it into the 'lib' folder if it needs to be referenced.
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5100: The assembly 'contentFiles\any\net472\libSkiaSharp.dll' is not inside the 'lib' folder and hence it won't be added as a reference when the package is installed into a project. Move it into the 'lib' folder if it needs to be referenced.
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5100: The assembly 'content\libHarfBuzzSharp.dll' is not inside the 'lib' folder and hence it won't be added as a reference when the package is installed into a project. Move it into the 'lib' folder if it needs to be referenced.
C:\Program Files\dotnet\sdk\6.0.407\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tasks.Pack.targets(221,5): error NU5100: The assembly 'contentFiles\any\net472\libHarfBuzzSharp.dll' is not inside the 'lib' folder and hence it won't be added as a reference when the package is installed into a project. Move it into the 'lib' folder if it needs to be referenced.
    0 Warning(s)
    24 Error(s)

Code
  <ItemGroup>
    <PackageReference Include="HarfBuzzSharp" />
    <PackageReference Include="ReactiveUI" />
    <PackageReference Include="SkiaSharp" />
    <PackageReference Include="Topten.RichTextKit" />
  </ItemGroup>

  <PropertyGroup>
    <GeneratePackageOnBuild>true</GeneratePackageOnBuild>
  </PropertyGroup>

Workarround
Adding these properties to my csproj ensure the native files are not added as content
    <!-- Workarround to ensure the Native files of SkiaSharp and Harfbuzz are not included in the package -->
    <ShouldIncludeNativeSkiaSharp>False</ShouldIncludeNativeSkiaSharp>
    <ShouldIncludeNativeHarfBuzzSharp>False</ShouldIncludeNativeHarfBuzzSharp>

Expected Behavior
I expect a NuGet package of my assembly is generated containing my assembly and a dependency on the SkiaSharp package.
The assemblies part of the SkiaSharp and Harfbuzz packages should not be considered as input files to pack.
Actual Behavior
MSBuild fails with the below error. The SkiaSharp (and HarfBuzz) win32 package is injecting the native assemblies as content. Which is picked up by MSBuild.
Possible solution
The build targets included in the package contain logic to have it included as content.
One solution could be to use a None include instead of a Content included
Another solution could be to also set the Pack attribute, to ensure the content files is not considered as a file to pack.
        <!-- include everything -->
        <Content Include="@(_NativeSkiaSharpFile)">
            <Link>%(Dir)%(Filename)%(Extension)</Link>
            <Visible>False</Visible>
            <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
            <Pack>False</Pack>
        </Content>

Basic information
Using Visual Studio 2022 17.4 and .NET 6 SDK version 6.0.407

I have a similar behavior with the native SkiaSharp libraries being included throughout all transitive dependencies
Ex:
AppA -> LibA -> LibB -> SkiaSharp
AppA, LibA and LibB will all embed the native dependencies in the nupkg, even when AppA end LibA are pulling it transitively

I also just stumbled upon this issue.
I have added the following code to Directory.Build.targets, but would prefer to have the Pack property set in the nuget's .props file, as is outlined in the bug report as a possible solution.
  <ItemGroup>
    <Content Update="@(_NativeSkiaSharpFile)">
      <Pack>False</Pack>
    </Content>
  </ItemGroup>

Setting ShouldIncludeNativeSkiaSharp to False is not an option in my case because I need to run my application from the bin folder.

This problem still occurs when building NuGet-Packages. I'm using version 2.88.6.
Sometimes, but not always, an error occurs during compilation when the compiler tries to copy the DLLs into the bin directory.
We also have to start our application from the bin folder, but we also pack NuGet packages for some assemblies.
Is there a satisfying solution for this?


Time limit for git input mode

Users can set max_time_for_a_git_repo to set a time limit for when the input is a git repo. When the time spend on analysis goes over the limit, next commits will not be analyzed anymore.

in the current implementation, it is not guaranteed that if a time_limit is set, the most recent commits will be analyzed before reaching the time limit. Check here. So do not merge it yet please.

Hi @khaes-kth
Thanks for this nice feature. We had a similar one on Astor, but not here, so it's nice.

it is not guaranteed that if a time_limit is set, the most recent commits will be analyzed before reaching the time limit. Check here.

I dont see the problem. Why the current implementation is not working fine?

So do not merge it for now please.
Okey, tell me if you need something about.

Regards
Matias

Hi @martinezmatias ,

I dont see the problem. Why the current implementation is not working fine?

Let say the repo contains n commits. If max_nb_commit_analyze is set to m and max_time_for_a_git_repo is set to t seconds, Coming starts from the (n-m+1)th commit and continues until all the (n-m+1)th to (n)th commits are analyzed or the time spend on the analysis is more than t seconds. So there is a possibility that only the commits (n-m+1)th to (n-m+i)th (i < m) are analyzed and time is over which means some of the most recent commits will not be analyzed.
This can be seen as a feature as it is or we can try to start the analysis from the (n)th commit and go backward through the development history until reaching the (n-m+1)th commit to make sure that the analyzed commits are the most recent ones.
However, I think if we have the list of all analyzed commits, that would be enough for the Repairability project we are working on right now. So I added this data to the output here:
JsonArray analyzedRevisions = new JsonArray();
root.add("analyzed_revisions", analyzedRevisions);

	for (Object revision_commit : finalResult.getAllResults().keySet()) {
	analyzedRevisions.add(revision_commit.toString());

I am OK with the current version and if you think no more change is required, it is time for a merge. But if you think we should always analyze the most recent commits, I should see how we can do that.

Hi @khaes-kth

So there is a possibility that only the commits (n-m+1)th to (n-m+i)th (i < m) are analyzed and time is over which means some of the most recent commits will not be analyzed.

That could happen, it's fine, which it should be very clear is to show a message that says something like "We reach the TIME_MAX of 1 hour and we analyzed X out of Y commits"

we can try to start the analysis from the (n)th commit and go backward through the development history until reaching the (n-m+1)th commit to make sure that the analyzed commits are the most recent ones

Yes, that's another feature (BTW, really simple to implement).

But if you think we should always analyze the most recent commits, I should see how we can do that.

I would say no: if we arrive to the TIMEOUT and there was not enough time to process the latest n commits, we should simply report the message. (The reverse order navigation would be another feature, not related to this one).

I am OK with the current version and if you think no more change is required, it is time for a merge.

Perfect. I merge! Thanks a lot.


KAFKA-5247: Materialize committed offsets in offset order

With this patch, offset commits are always materialized according to the order of the commit records in the offsets topic.
Before this patch, transactional offset commits were materialized in transaction order. However, the log cleaner will always preserve the record with the greatest offset. This meant that if there was a mix of offset commits from a consumer and a transactional producer, then it we would switch from transactional order to offset order after cleaning, resulting in an inconsistent state.

cc @hachikuji @guozhangwang

@apurvam The failing integration test looks like it could be related.

@hachikuji I don't think so. The error from PlaintextConsumerTest.testAsyncCommit is: 'org.apache.kafka.common.errors.TopicExistsException: Topic 'topic' already exists.'
That seems to be a symptom of a leaked producer which hasn't been cleaned up in another test.

retest this please

Hmm.. Yeah, just seems too coincidental to have one of the offset commit tests failing. Given overall flakiness though, it definitely could be unrelated.


Readable local logs (! after previous PR is merged !)

Coverage decreased (-0.2%) to 67.078% when pulling cea1888133235d7a96bb95668710c26d74ab1580 on readable-local-logs into d0c7b53a689d01f8c21cd219008f530f7345928a on master.


Edit not working?

Edit button is only available when I choose an id field of documents,
and clicking it doesn't do anything.

Expected behavior:

Edit button should be available on the document itself
and work

Maybe Edit button is supposed to be available for any field of the document? I'm not sure what was intended.
Clarification: Double clicking does nothing as well.
Effectively this means I can only view documents, which makes this amazing plugin useless.
Thx!

As of now, when you will Right Click on the _id field, you will get the option to Edit the document. This has been the designed fliw from the start. Please try to validate if this is working for you.

Hi @dorbareket, is the above approach working for you? If not, please let me know. Thanks

Right clicking on _id and clicking on Edit opens a window titled Edition which allows to edit each key-value.
If it is not working for you, it would be great if you can share a short video for the same.
Also, please share if any error is coming on clicking Edit. Errors come at bottom right corner and are also available via a Alert icon in bottom bar on right side.

I just discovered I can create new documents and edit them, but I cannot edit existing documents using your tool (it does work from 3t studio or DataGrip).
There's no issue logged under notifications, it's just that absolutely nothing happens.
Here's a video:

Thanks!

Hi @dorbareket , sorry for delayed response. Are you still facing the issue? Did u check if u have edit access to the database your are trying to edit?

@geetesh-gupta yes I'm an admin, I use Studio3T and I can edit whatever i want

Hi, @geetesh-gupta. I experience the same edit issue as dorbareket. Tried with no user (local default) and with different admin user setups. Edit does not work as expected. Thanks!

I just now created a new collection with a document using Studio3T, then went to the plugin, loaded that collection, right clicked on the document _id, I was able to see the edit menu and edit the document.
One thing I observe in @dorbareket video, that the document you are trying to edit, is having the _id as 1, which in an ideal scenario should be something like 6697fc91313970259c0b06fc. Maybe that's the issue.
@berndlueneburg Can you share the screenshot of your collection, maybe if their is any similarity?

I just now created a new collection with a document using Studio3T, then went to the plugin, loaded that collection, right clicked on the document _id, I was able to see the edit menu and edit the document.
One thing I observe in @dorbareket video, that the document you are trying to edit, is having the _id as 1, which in an ideal scenario should be something like 6697fc91313970259c0b06fc. Maybe that's the issue.
@berndlueneburg Can you share the screenshot of your collection, maybe if their is any similarity?

I have the same issue and I agree that having an _id that is not an uidd like "my_settings" is the problem. But why is this a problem? In other clients like studio 3T that doesn't happen, it seems like a bug on your side for me, we can not change our _id values just for updating the data.
Thanks for your help in advance.

I just now created a new collection with a document using Studio3T, then went to the plugin, loaded that collection, right clicked on the document _id, I was able to see the edit menu and edit the document.
One thing I observe in @dorbareket video, that the document you are trying to edit, is having the _id as 1, which in an ideal scenario should be something like 6697fc91313970259c0b06fc. Maybe that's the issue.
@berndlueneburg Can you share the screenshot of your collection, maybe if their is any similarity?
No, i use standard MongoDb Ids:

I also use standard ids. I just created that temporary DB for the video and to try with a clean db

So, we are able to know the issue that _id is not in standard format. Will check for the support for other IDs. As of now, it only supports the ObjectIds.

currently i can't check because i can't add my local db due to #15

So, we are able to know the issue that _id is not in standard format. Will check for the support for other IDs. As of now, it only supports the ObjectIds.

as I understand @dorbareket and i use standard MongoDb ObjectIds and the Edit issue appears anyhow. So it cannot be the problem of a different id format. I think the hottest hint is that newly added documents within the MongoDb Browser are editable but existing ones not. Does this eventually help you?

I tried creating documents from Studio3T. I was able to edit them in the plug-in.

So umm

With the plugin, I can totaly create a document with string/numberInt _id. The new document is NOT EDITABLE, albeit edit button is in color. On creation I can add fields, but not afterwards.

I tried creating a document from Studio3T with objectId and indeed I am able to edit it from the plugin.

So @geetesh-gupta I think you are right, edit only works when _id is an ObjectId. It doesn't work if it's a string/NumberInt

So the plugin allows creating documents with _id that is not ObjectId, but wont' allow editing it

Correct @dorbareket. Will try to see if we can fix it.


How to install this app?

So how to install this app? I ask, because there's no .IPA file.

It's a tweak, not a standalone app, you can install it using a package manager such as Cydia, Sileo, Zebra, etc


No proper redirection to original URL after login

Setup

Rancher version: v2.8.3
Rancher UI Extensions: v2.8.3
Browser type & version: Vivaldi (current version)

Describe the bug

To Reproduce

Open https://op-prg2.openplatform.suse.com/dashboard/c/c-m-t7lmqchx/explorer/apps.deployment/qem-bot/qem-bot
Log in

Result

A different URL than https://op-prg2.openplatform.suse.com/dashboard/c/c-m-t7lmqchx/explorer/apps.deployment/qem-bot/qem-bot is shown

Expected Result

https://op-prg2.openplatform.suse.com/dashboard/c/c-m-t7lmqchx/explorer/apps.deployment/qem-bot/qem-bot is shown

Candidate to backport, unless it's somehow quick to sort out for 2.9.0

The scenario points to a bookmark to a specific page that requires login to access. Sounds like a lost preference.


patch operand image and job image with enviroment variable

add operand image and job image into container env variable
patch resource yaml file with those image

/lgtm

Hi @qpdpQ
Could you please help to add two more images for UI?
https://github.com/IBM/ibm-user-management-operator/blob/192494b6d193941e59ad99dad9f5ecb25e98db06/internal/resources/yamls/ui.go#L333
https://github.com/IBM/ibm-user-management-operator/blob/192494b6d193941e59ad99dad9f5ecb25e98db06/internal/resources/yamls/ui.go#L503


"Permission denied !" on a binary that has permission !

while doing this PR to add podman as a driver to minikube (which will run a systemd inside a container using podman)
https://github.com/kubernetes/minikube/pull/6515
I noticed the binary files that minikube copies into the podman container, are not executable.
root@minikube:/var/lib/minikube/binaries/v1.17.2# ./kubectl
bash: ./kubectl: Permission denied

# ls -lah kubectl
-rwxrwxrwx 1 root root 42M Jan 21 18:07 kubectl

because I had other issues with podman cp command
I decided to go into the container myself and use wget to download them and then chmod +x manually but I still get permission denied !!!
this seems to be the only blocker to add run minikube in podman and I would appreciate any help on this
sudo podman exec -it minikube /bin/bash

root@minikube:/var/lib/minikube/binaries/v1.17.2# wget https://storage.googleapis.com/kubernetes-release/release/v1.17.2/bin/linux/amd64/kubectl
 41.48M   223MB/s    in 0.2s
2020-02-06 07:53:03 (223 MB/s) - 'kubectl' saved [43491328/43491328]

root@minikube:/var/lib/minikube/binaries/v1.17.2# chmod +x kubectl
root@minikube:/var/lib/minikube/binaries/v1.17.2# ./kubectl
bash: ./kubectl: Permission denied
root@minikube:/var/lib/minikube/binaries/v1.17.2# chmod a+x kubectl
root@minikube:/var/lib/minikube/binaries/v1.17.2# ./kubectl
bash: ./kubectl: Permission denied

it worth noting that minikube creates the container with --cgroup-manager cgroupfs
and --privileged and with sudo
(unlike docker it doesn't let me create my container without sudo)
versions :
$ podman version
Version:            1.7.0
RemoteAPI Version:  1
Go Version:         go1.12.10
OS/Arch:            linux/amd64
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 19.10
Release:	19.10

Could you check if this is an SELinux issue? setenforce 0?
Could this be seccomp blocking it?

Does minikube has file capabilities? That could conflict with no new privileges

interestingly it only has this issue in /var/lib folder but if I move it to /usr/local/bin I can execute it !

Is /var/lib directory mounted with the noexec option?

Can you run mount in the container and provide the output?

I think this is probably a volume mount of /var or /var/lib - you may need to add the exec option onto the volume mount

@mheon you are right ! for anyone having same issue
this fixed my problem
 mount -o remount,exec /var

I am just curious why same image and same container doesnt act same way on docker and podman

Our volume mounts are, by default, mounted noexec,nodev,nosuid for security; Docker does not do this. You can append the exec option to mounts if you need it; podman run -v /var:/var:exec for example.

@mheon thank you very much for clarifying it ! it makes sense now ! I consider this issue closed for me, but I also recommend adding this to your website documentation for others


frontend: Add prettier code formatting tool

Using config from create react app prettier docs.
https://create-react-app.dev/docs/setting-up-your-editor/#formatting-code-automatically
The .prettierrc provides an eslint rules compatibility option.
How to use
Files that are git staged get prettier run on them.
See Editor integration section of prettier docs.
Testing done
Tried prettier on a couple of our files.
fixes #185

There's some prettier documentation here on Integrating with Linters, in which they recommend eslint-config-prettier.
It also comes with a script for checking conflicting rules. Which finds this conflict.
npx eslint-config-prettier `find src -type f -name \* | grep ".ts\|.js\|.tsx"`
The following rules are unnecessary or might conflict with Prettier:

- @typescript-eslint/member-delimiter-style

The docs for this rule: @typescript-eslint/member-delimiter-style

There's some prettier documentation here on Integrating with Linters, in which they recommend eslint-config-prettier.
It also comes with a script for checking conflicting rules. Which finds this conflict.
npx eslint-config-prettier `find src -type f -name \* | grep ".ts\|.js\|.tsx"`
The following rules are unnecessary or might conflict with Prettier:

- @typescript-eslint/member-delimiter-style

The docs for this rule: @typescript-eslint/member-delimiter-style

AFAICS running prettier didn't change the members' delimiter, so we can delete that rule if that's correct and it's the only thing impeding prettier to use our eslint rules.

AFAICS running prettier didn't change the members' delimiter, so we can delete that rule if that's correct and it's the only thing impeding prettier to use our eslint rules.

To make sure eslint would not fix prettier changes, I tried this:
$ ./node_modules/.bin/prettier --write src
# 2800 or so changed lines
$ git commit . -m "prettier changes"
$ yarn lint --fix src
$ git status
On branch prettier
nothing to commit, working tree clean

We can either

make one big code formatting commit
do it as we go, and only format files files as they change

Probably 2 is the safest option, with less noise too. If we went with 1 big commit, there's some trick you can do to get git blame working again.

To make sure eslint would not fix prettier changes, I tried this:
$ ./node_modules/.bin/prettier --write src
# 2800 or so changed lines
$ git commit . -m "prettier changes"
$ yarn lint --fix src
$ git status
On branch prettier
nothing to commit, working tree clean

We can either

make one big code formatting commit
do it as we go, and only format files files as they change

Probably 2 is the safest option, with less noise too. If we went with 1 big commit, there's some trick you can do to get git blame working again.

AFAICS running prettier didn't change the members' delimiter, so we can delete that rule if that's correct and it's the only thing impeding prettier to use our eslint rules.

Yeah, true. It does seem prettier leaves those delimiters alone. So if we stick with using ;, then prettier should not rewrite them.

AFAICS running prettier didn't change the members' delimiter, so we can delete that rule if that's correct and it's the only thing impeding prettier to use our eslint rules.

Yeah, true. It does seem prettier leaves those delimiters alone. So if we stick with using ;, then prettier should not rewrite them.

Why not add the eslint-plugin-prettier now?

"Runs Prettier as an ESLint rule and reports differences as individual ESLint issues."

This would mean we'd have to apply those 2800 lines of changes so make lint would be happy.

Why not add the eslint-plugin-prettier now?

"Runs Prettier as an ESLint rule and reports differences as individual ESLint issues."

This would mean we'd have to apply those 2800 lines of changes so make lint would be happy.

I made estlint --fix run in the lint-staged, and added a npm run format command.
I was going to add the app/ config, which is a duplicate. Then I considered, that we'd need to use this config in each app going forward. We can do this with a shared configuration in a package like a @kinvolk/prettier-config.
However, what if we had no prettier configuration and left all style decisions up to prettier?
The benefits of leaving all style decisions up to prettier are:

no prettier config needed for each app
code is closer to 'standard' JS ecosystem style

The downsides are:

there are more changes, because of the single quote.

$ git diff | wc -l
   14772

a) go with prettier for all style decisions, and no config.
b) keep single quotes, and add a shared @kinvolk/prettier-config package.

I would probably go for a), because it's a one time cost.

I made estlint --fix run in the lint-staged, and added a npm run format command.
I was going to add the app/ config, which is a duplicate. Then I considered, that we'd need to use this config in each app going forward. We can do this with a shared configuration in a package like a @kinvolk/prettier-config.
However, what if we had no prettier configuration and left all style decisions up to prettier?
The benefits of leaving all style decisions up to prettier are:

no prettier config needed for each app
code is closer to 'standard' JS ecosystem style

The downsides are:

there are more changes, because of the single quote.

$ git diff | wc -l
   14772

a) go with prettier for all style decisions, and no config.
b) keep single quotes, and add a shared @kinvolk/prettier-config package.

I would probably go for a), because it's a one time cost.

From chat about prettier config...

Material UI: https://github.com/mui-org/material-ui/blob/next/prettier.config.js
CRA: https://github.com/facebook/create-react-app/blob/master/.prettierrc
react too: https://github.com/facebook/react/blob/master/.prettierrc.js
redux: https://github.com/reduxjs/redux/blob/master/.prettierrc.json

so I think we should really check again how to minimize the changes between prettier + eslint
and if we can have a @kinvolk/prettier or whatever to share the config as we do for eslint, let's do that (edited)

From chat about prettier config...

Material UI: https://github.com/mui-org/material-ui/blob/next/prettier.config.js
CRA: https://github.com/facebook/create-react-app/blob/master/.prettierrc
react too: https://github.com/facebook/react/blob/master/.prettierrc.js
redux: https://github.com/reduxjs/redux/blob/master/.prettierrc.json

so I think we should really check again how to minimize the changes between prettier + eslint
and if we can have a @kinvolk/prettier or whatever to share the config as we do for eslint, let's do that (edited)

There's a shared config PR here: kinvolk/eslint-config#3 which needs to be merged/published first (testing instructions in there).

I'm working on dependency upgrades on headlamp and kinvolk/eslint-config first, so prettier/@typescript-eslint can work (which requires a newer dependency).

The kinvolk/eslint-config 0.4.0 PR https://github.com/kinvolk/eslint-config/pull/4, then the dependencies update PR https://github.com/kinvolk/headlamp/pull/190 should be merged before this one is done again.


mGear Core: Added lineWidth of curves with collect_curve_data on curve.py #148

https://github.com/mgear-dev/mgear4/issues/148
The function of collect_curve_data on curve.py now supports a lineWidth of curve shapes.
And it allows getting them while importing.

Thanks for the PR @maglev2468ng   I will check it and merge ASAP. 😄

ah didn't see this pull request:) made the same one.

@maglev2468ng  I have finally merged your PR. I am sorry it took so long to do it.
Thanks again for the PR!


A function to pull all features in a dataframe

Hi,
It will be great to have the function avaialble to pull all the fatures once for all.
Use Scenario：
This can help do some quick ML project prototype.
Thanks.

Hello @Zakk-Yang,
I recommend checking the README. But if it's not there or not clear, please let me know.
Kind Regards,
KJ


Incorrect package json exports in @equinor/fusion-framework-react-app

Describe the bug
The package json exports are incorrect in @equinor/fusion-framework-react-app.

My tsconfig
{
...
    "target": "ESNext",
    "module": "ESNext",
...
}

Stacktrace
@cc-reports/powerbi:build: src/hooks/usePbiHelpers.ts(2,29): error TS7016: Could not find a declaration file for module '@equinor/fusion-framework-react-app/http'. 'C:/Users/gustav.eikaas/repo/cc-reports/node_modules/.pnpm/@equinor+fusion-framework-react-app@5.0.0_@equinor+fusion-framework-module-http@5.2.0_<EMAIL_ADDRESS>implicitly has an 'any' type.
@cc-reports/powerbi:build:   There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The '@equinor/fusion-framework-react-app' library may need to update its package.json or typings.
@cc-reports/powerbi:build: src/hooks/useWorkspaceBookmark.ts(6,8): error TS7016: Could not find a declaration file for module '@equinor/fusion-framework-react-app/bookmark'. 'C:/Users/gustav.eikaas/repo/cc-reports/node_modules/.pnpm/@equinor+fusion-framework-react-app@5.0.0_@equinor+fusion-framework-module-http@5.2.0_<EMAIL_ADDRESS>implicitly has an 'any' type.
@cc-reports/powerbi:build:   There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The '@equinor/fusion-framework-react-app' library may need to update its package.json or typings.
@cc-reports/powerbi:build: src/fusion-framework/frameworkConfig.ts(1,34): error TS7016: Could not find a declaration file for module '@equinor/fusion-framework-react-app'. 'C:/Users/gustav.eikaas/repo/cc-reports/node_modules/.pnpm/@equinor+fusion-framework-react-app@5.0.0_@equinor+fusion-framework-module-http@5.2.0_<EMAIL_ADDRESS>implicitly has an 'any' type.
@cc-reports/powerbi:build:   There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The '@equinor/fusion-framework-react-app' library may need to update its package.json or typings.
@cc-reports/powerbi:build: src/fusion-framework/createRender.tsx(5,8): error TS7016: Could not find a declaration file for module '@equinor/fusion-framework-react-app'. 'C:/Users/gustav.eikaas/repo/cc-reports/node_modules/.pnpm/@equinor+fusion-framework-react-app@5.0.0_@equinor+fusion-framework-module-http@5.2.0_<EMAIL_ADDRESS>implicitly has an 'any' type.
@cc-reports/powerbi:build:   There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The '@equinor/fusion-framework-react-app' library may need to update its package.json or typings.

Here is an example of how it should be done:
https://github.com/equinor/fusion-framework/blob/main/packages/framework/package.json#L6
Tool for checking if everything is configured correctly
https://arethetypeswrong.github.io/?p=%40equinor%2Ffusion-framework-react-app%405.0.0

We are getting the following error currently
[plugin:vite-plugin-eslint] Package subpath './dist/types/types' is not defined by "exports" in
<EMAIL_ADDRESS>Occurred while linting C:\Projects\equinor\plant-query\src\frontend2\src\index.ts:1
Rule: "import/namespace"

looks like it is related to this one

<EMAIL_ADDRESS>released now


pylint: pylint message ids are not shown in any output

From<EMAIL_ADDRESS>on 2011-05-26T13:33:14Z
What steps will reproduce the problem?

run pylint What is the expected output? What do you see instead? I want to suppress some pylint errors and warnings, but currently the output does not show me the IDs to do this.  Why is there no mapping between the reported item and the message ID?  This actually looks like a pylint issue because I ran it from the command line and the output is the same. What version of the product are you using? On what operating system? Spyder 2.0.11, python 2.6.1, Windows XP SP3

Original issue: http://code.google.com/p/spyderlib/issues/detail?id=665

From<EMAIL_ADDRESS>on 2011-05-31T12:28:30Z
I have discovered the flag to turn this on in pylint and it appears the output is properly parsed by Spyder.  We just need an option to display this information in the pylint widget.

From pierre.raybaut on 2011-05-31T13:56:42Z
This issue was closed by revision 85ac63f7a91c .
Status: Fixed

From<EMAIL_ADDRESS>on 2011-06-01T09:35:57Z
Much appreciated, however I tried to pull default from the repository and got an exception so I had to revert back to 2.0.11, which I couldn't pull from repo because there didn't seem to be a tag for it.  Maybe I was pulling from the wrong repo?

From pierre.raybaut on 2011-06-01T09:51:30Z
Take a look at this wiki page for the repositories location: https://code.google.com/p/spyderlib/wiki/Roadmap

From pierre.raybaut on 2011-05-31T13:56:44Z
This issue was closed by revision d62af58429a2 .


Autocomplete: suggest function parameters

Could you add function parameters autocompletion like you did with procedures in #1552?
This is currently not working (on Oracle at least).

duplicate of #3061


Chore: Add value placeholders & preview for badge generator

⚠⚠⚠ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma
Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

Description
Fixes comment in #2915
New package "vite-plugin-commonjs" is added to allow for import of the server config.js file.
Also fixes bugs:

cert-exp badge does not work because the color is set incorrectly.
the "Badge Prefix/Suffix" does not actually describe what it does.

Type of change

Bug fix (non-breaking change which fixes an issue)
User interface (UI)

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[x] I have commented my code, particularly in hard-to-understand areas
(including JSDoc for methods)
[x] My changes generate no new warnings
[ ] My code needed automated testing. I have added them (this is optional task)

Screenshots (if any)

@louislam
I think this could be a valuable addition to the 1.22.1 milestone.
What do you think?

@louislam I think this could be a valuable addition to the 1.22.1 milestone. What do you think?

No problem, it is indeed hard to use without knowing the default value.

@louislam I think this could be a valuable addition to the 1.22.1 milestone. What do you think?

This branch is not based on the 1.22.X branch, so it has to be in 1.23.0.


Add RTLD_NEXT and RTLD_DEFAULT

This change adds RTLD_NEXT and RTLD_DEFAULT to libc. These two constants are useful when calling dlsym.

Thanks!


On a Macbook scrolling with the parameters can create problems

@freedumbs00 do you have a screenshot for it?


proper file change watching

the tmon.py is a primitive script put together for demo purposes. I would be very glad to put the re-running out of scope ot testmon.
to consider:

watchdog has https://github.com/gorakhargosh/watchdog#shell-utilities
xdist has http://pytest.org/latest/xdist.html#running-tests-in-looponfailing-mode (which maybe should be out of py.test or it's plugins?
there is https://github.com/guard/guard but it's ruby

In short term, testmon will probably be failing with it's bugs so I will not want to run it automatically that much. Also in long term I quite struggle with console test outputs so I would probably run py.test manually or use PyCharm to run a test suite.

Yes, any external tool like watchmedo from watchdog would do.
A few examples might be provided in the documentation, and/or as example files - keeping the tmon.py script, but e.g. not installed automatically - or with a different name.
I think that testmon might also provide a default "tricks" file for watchdog maybe.

There is also https://github.com/seb-m/pyinotify and https://github.com/copton/react/ using it (the latter (currently) leaks however, see PRs).

It might be nice if testmon would not need to restart py.test at all, but could loop over it internally, using e.g. watchdog.
That would certainly improve performance / the feedback loop, since py.test takes some time during the collection phase alone.
Just some food for thought.

To looping over internally: The biggest cost of restart at the moment seems to be test collection. And we do want to have freshly created tests collected so the real task here is optimizing pytest's collection (e.g. using cache)
And another contender for the functionality
pytest --looponchange
https://bitbucket.org/pytest-dev/pytest/pull-request/253/take-in-pytest-cache/
As far as I could understand it uses polling :(

Polytester seems to be a very nice tool for this: https://github.com/skoczen/polytester

https://github.com/joeyespo/pytest-watch Is the "official" way to reexecute pytest --testmon on file changes.


The npm version supports "versionPrefix"

I want to select the latest version of "@ wangeditor/editor for vue", which should be 5. x.x.
However, "1.0.2" is recently released, and only "1.0.2" can be selected at present
https://img.shields.io/npm/v/@wangeditor/editor-for-vue：

使用 https://img.shields.io/npm/v/@wangeditor/editor-for-vue?versionPrefix=5 无效：

Hmm. It is hard to know exactly what has happened here, but this behaviour can be explained by looking at the dist-tags. See
https://registry.npmjs.org/-/package/@wangeditor%2Feditor-for-vue/dist-tags
{
  "latest": "1.0.2",
  "last": "5.1.0",
  "next": "5.1.12"
}

I think my recommendation here would be that you want to follow the 'last' or 'next' tag with either
https://img.shields.io/npm/v/@wangeditor/editor-for-vue/last
https://img.shields.io/npm/v/@wangeditor/editor-for-vue/next
https://img.shields.io/npm/v/@wangeditor/editor-for-vue
but it is hard to know the exact intent of the package authors here.
We shouldn't implement your suggested solution though, well meaning as it is.

Hmm. It is hard to know exactly what has happened here, but this behaviour can be explained by looking at the dist-tags. See
https://registry.npmjs.org/-/package/@wangeditor%2Feditor-for-vue/dist-tags
{
  "latest": "1.0.2",
  "last": "5.1.0",
  "next": "5.1.12"
}

I think my recommendation here would be that you want to follow the 'last' or 'next' tag with either
https://img.shields.io/npm/v/@wangeditor/editor-for-vue/last https://img.shields.io/npm/v/@wangeditor/editor-for-vue/next
but it is hard to know the exact intent of the package authors here.
https://img.shields.io/npm/v/@wangeditor/editor-for-vue is basically the same as https://img.shields.io/npm/v/@wangeditor/editor-for-vue/latest
We shouldn't implement your suggested solution though, well meaning as it is.

thank you


Bugfix: Show Details on Interaction Toggle Retention

This fixes the issue with the toggle state retention for issue #88

Thanks! just a heads up manually added translations will be overwritten, I'll see if I can find any translation for this
https://github.com/matt8707/ha-fusion/tree/main/scripts/translations

Thanks! just a heads up manually added translations will be overwritten, I'll see if I can find any translation for this https://github.com/matt8707/ha-fusion/tree/main/scripts/translations

No problem if you think there is something better to describe this function that already has a translation feel free to swap it up.


DB: Table Land Integration

[ ]   Create users table

id

name

lastLoggedIn

custodialWalletAddress

custodialWalletPrivateKey

createdAt

updatedAt

[ ]  Create table for user_models

id

userId

name

cid

createdAt

updatedAt

Caveat: work on william's repo

This is wrong,  when creating a user, you do not have access to the user's web3auth id. Else you want users to connect wallet when creating account

@techyNonso so web3auth id can be anything, it can be event user's public key:
https://github.com/Web3Auth/web3auth-core-kit-examples/blob/main/node-sdk/node-backend-example/index.js

Here: https://web3auth.io/docs/integration-builder/builder/nextjs?lang=REACT&chain=ETH&evmFramework=WEB3&customAuth=NONE&mfa=DEFAULT&whitelabel=NO&useModal=YES&web3AuthNetwork=TESTNET&rnMode=EXPO&stepIndex=0
I think web3auth id would be in here:
  const getUserInfo = async () => {
    if (!web3auth) {
      uiConsole("web3auth not initialized yet");
      return;
    }
    const user = await web3auth.getUserInfo();
    uiConsole(user);
  };

Wow there is cooler nextjs snippet: https://web3auth.io/docs/integration-builder/builder/nextjs?lang=NEXT&chain=ETH&evmFramework=WEB3&customAuth=NONE&mfa=DEFAULT&whitelabel=NO&useModal=YES&web3AuthNetwork=TESTNET&rnMode=EXPO&stepIndex=0

You need to stop comparing your app to fully web3 app. On sign up form , there is no wallet connection so you cannot get any ID yet

Here: https://web3auth.io/docs/integration-builder/builder/nextjs?lang=REACT&chain=ETH&evmFramework=WEB3&customAuth=NONE&mfa=DEFAULT&whitelabel=NO&useModal=YES&web3AuthNetwork=TESTNET&rnMode=EXPO&stepIndex=0
I think web3auth id would be in here:
  const getUserInfo = async () => {
    if (!web3auth) {
      uiConsole("web3auth not initialized yet");
      return;
    }
    const user = await web3auth.getUserInfo();
    uiConsole(user);
  };

This function here will prompt for a wallet connection.

Mind you, a user can have more than one wallet

@Abhay-2811 its ready now

@Nasfame updating db on tableland costs gas are you sure you want to  update it every time user logs in ( talking about lastLoggedIn column )

Yes let's worth it.

making user_models's id auto-incrementing and integers ( 0,1,2.. such that )

Lets close this.. @Nasfame

Okay


cpu_dummy_reads.nes is failing

https://github.com/christopherpow/nes-test-roms/raw/master/cpu_dummy_reads/cpu_dummy_reads.nes
I found more info here: https://github.com/christopherpow/nes-test-roms/tree/master/cpu_dummy_reads/source

This might help
https://www.reddit.com/r/EmuDev/comments/9zes0i/nes_what_are_dummy_reads_and_how_to_emulate_them/

That's going to be complicated to fix, not going to bother for now


Updated to prep for html tag removal.

Removed the  tags and changed text so that there is no mention of highlighted text. This topic is now ready to have the html tags removed from the doc. I believe from the notes, there are only  tags that need to be converted (removed from the markdown). Please do not modify html that is actually part of the tutorial.

OK to merge and thanks for merging. Thanks


Wrong orientation of patch faces in guide curve case

The patch faces resulting from MakePatches are not oriented well. The normal vectors should point "outwards". At the moment, they are pointing "inwards":

This is probably the same bug as mentioned earlier. It is actually not possible to detect, what should be correct if we don't have a solid. In case of a solid it is easier, as we simply could take a point at infinity and check if it is outside (it should be), or not. In the latter case, we just have to invert the shell.
I think, the direction of the patches depends only on the directions of the guides and profiles. So we just have to provide guides and profiles with the "correct" direction.

I agree. To provide guides and profiles that yield the correct orientation is the cleanest way. However, in the end all surfaces will be closed, right? Then one could use the solid for detecting the orientation.

I am not entirely shure, if we'll get a solid in all cases. E.g. if we provide a entry and exit plane of the engine, the nacelle might not be a solid.

The normal of a surface patch is determined by a right handed coordinate
system attached to the (u,v) = (0,0) point of the surface patch. Here
the tangent of the profile defines the x-direction and the tangent of
the guide defines the y-direction. The normal is the z direction.
As a result, the resulting shell is correctly orientated if

The direction of rotation of the wing profiles is the y-direction, and
the wing guides run from the fuselage to the wing tip (in y-direction)

The direction of rotation of the first (upper) nacelle profiles is in
y-direction and the direction of rotation of the nacelle guides is in negative
x-direction.

The direction of rotation of the fuselage profiles is negative x-direction, and
the fuselage guides run in x-direction (or vice versa)


authentication-flows-app-scenarios#scenarios-and-supported-authentication-flows incorrectly says that Device code flow works with B2C

[Enter feedback here]
https://docs.microsoft.com/en-us/azure/active-directory/develop/authentication-flows-app-scenarios#scenarios-and-supported-authentication-flows mentions that you can sign-in to B2C with Device code flow. We just learnt that this wasn't the case. The corresponding line needs to be fixed.

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

ID: f7f1af72-01ef-2a13-8498-e2c21261c1e0
Version Independent ID: 33f65db3-8bd6-33ad-675e-b0337c56596f
Content: Microsoft identity platform authentication flows & app scenarios
Content Source: articles/active-directory/develop/authentication-flows-app-scenarios.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

#assign jmprieur

#sign-off


Add license

Most of our dependencies use a BSD-3-Clause license.
Some use MIT
one used some Apache license.
I'm not sure if this affects the license we get to choose.
If nothing else, BSD-3-Clause seems to be a popular choice with flutter, so right now I am between BSD-3-Clause and GPLv3

Also, I'm not sure what name(s) I should put in the license.
Maybe it should just be a list of all of our names, until it gets handed off to DiscoTray?